Safari Date Checking for not working.

Dates that are valid are being recognized as invalid on safari browser.

Date Validation was unsound, fixed by parsing each piece of birth date in format: yyyy-mm-dd


Added Oculus Manifest generation tool for Quest

Overview
Added Oculus Manifest generation tool for Quest
For the Quest to detect the controllers properly, a custom AndroidManifest.xml file is required.  Ported the Oculus tools to enable this for XRTK projects.
In the future, this will be handled by the Build Window
Changes:

Fixes: #13

Notes
A manual task, users will need to use the "Mixed Reality Toolkit -> Tools" options to generate the manifest. Has safety features to check if there is an existing file and won't overwrite until it's removed.

Comments addressed where possible without affecting the license too much


Getting all correct solutions, but challenge isn't accepted

I've tried a number of strategies, but am not able to get this challenge accepted. Thanks for looking at this issue.  Val
Challenge Return Largest Numbers in Arrays has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
var arrNew = [];
var arr = [];

function largestOfFour(arr) {
  // You can do this!

  for (var i=0; i < arr.length; i++) {
    var largest = 0;
    for (var j=0; j < arr.length; j++) {
      if (arr[i][j] >= largest) {
        largest = arr[i][j];
      }
    }
  arrNew.push(largest);

    }
   arr = arrNew;

 return arr;
  }

largestOfFour([[4, 9, 1, 3], [13, 35, 18, 26], [32, 35, 97, 39], [1000000, 1001, 857, 1]]);

Global variables are bad news. Think about what your code does if that function is run twice (or just give it a try and see what it does). Thanks and happy coding!

Thanks so much, Logan! I'm still trying to figure out what determines if
the solution is accepted or not. So this was definitely helpful.
Val
On Wed, Mar 2, 2016 at 11:12 AM, Logan Tegman<EMAIL_ADDRESS>wrote:

Closed #7359 https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7359.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7359#event-575142983
.


VSTS find tagged builds

Hi all,
Is it possible to find builds that was tagged? In build dashboard > Recently completed, is there any way to have a column that shows the tag assigned in each build version?
Thank you

Not an issue with a task in this repo.  Please use developer community for service issues.


chore(deps): update apt/apk dependencies in Dockerfile orchestrator-chaincode-init

Updated dependencies.json

/e2e

End to end tests: :hourglass_flowing_sand: STARTING


Customisation accordingly to manual

Hello, @joeldenning,
Please take a look.
Thanks in advance

Published at https://www.npmjs.com/package/@esm-bundle/markdown-it


Pinned items open wrong folder

If I click on a pin item in the side bar it opens the recycle bin. I have to click off and click again for the correct folder to be displayed.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 1903 18362.175]
App version: [v0.11.0]

Additional context
Add any other context about the problem here. Does this problem occur again after restarting the app?
Log file
Please upload the log file here so that we can understand your issue better. You can access it from Settings->About->Open log location. You can also find this file by going to %localappdata%\Packages\49306atecsolution.FilesUWP_et10x9a9vyk8t\LocalState, you should see a file in this directory called debug.txt or/and debug_fulltrust.txt.

@Ricky55 Can you fill out the issue template so we can better investigate this issue?

@Ricky55 Can you fill out the issue template so we can better investigate this issue?

@Ricky55 We need a little more information from you in order to investigate this issue, please reopen this issue once you have filled out the issue template.

@Ricky55 We need a little more information from you in order to investigate this issue, please reopen this issue once you have filled out the issue template.


Is there a way to implement client side pagination?

I have an API endpoint which returns all data for specific resource, I want to implement client side pagination using vuetable-2
Consider API response has 100 records, I want to display each page with 10 records in client side.
Do vuetable-2 supports this feature?

@kaarthikeyansp It's already there. See the tutorial for the usage.
You can even extend it to suit your need.


breakpoints are not correct when saving the source file after changed the file

VSCode Version: 1.33.1
OS Version: win 10

Steps to Reproduce:

Add one break in source file (for example at line 5)
start the debugger, it will start the debugger adapter
edit the source file (for example to add 10 line codes before the line 5),
save the source (ctrl + s)
Vscode send the "setbreakpoints" request to the debugger adapter, but the location for the breakpoint is still 5 instead 15

Does this issue occur when all extensions are disabled?: Yes

It is up to the debugger to update the breakpoint location.
@whlkw  What debugger are you using?

@isidorn I am writing the vscode extension for a common lisp dialect language. We think VSCODE is a powerful IDE.

It is up to the debugger to update the breakpoint location.
Do you mean that the debugger should track the file changes, and track the breakpoints location for the file changes? I was thinking that the breakpoints information are from vscode side.

I want to clarify one point:
In the above "step 5", after saved the source file, in the VSCODE UI the breakpoint is in line 15, but the request received by debugger adapter is line 5.
If I "restart" the debug session, the breakpoint line information is correct in the "setbreakpoints" request.

@isidorn Any update on this?

No. Can you update the initial comment the have all the details clearly and consily please. Thanks

I have updated the initial comment for the issue.

Yes, I can reproduce and I acknowledge the issue.
VSCode should always send its notion of the line number, not what the debug adapter has reported.

Thanks! Hope it can be fixed in future release!


Implemented a localization pattern

There is already a localization api specific to the module, we need to look at how it work in the previous iteration and adopt or improve that pattern.

Done in 3c72a3b55f8f417cd6e22e473e8d9f87d3a368a3


invite: Fix rendering invite modal for realms without default channels.

The default streams checkbox only exists in DOM if there are default channels in the realm, so we should make sure that we check whether there are default streams in the realm before accessing the checkbox element in set_streams_to_join_visibility.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Nice, merged, thanks @sahil839!


[sk] Added runtime variable docs

Summary
Added docs for using runtime variables. The Creating Runtime Variables In-Editor section is left empty since in-editor support for adding runtime variables hasn't been added yet.
cc:
@wangxiaoyou1993

For runtime variable, the common use case is to pass a date string so that we can transform the data for date partitions when running pipelines in production. Can we give that example here?

Added this new example + rearranged some of the sections of the docs


cmdkey targetname argument not fully documented

The windows cmdkey command can be used to add credentials for accessing remote resources.
cmdkey
For the most part, it is clearly documented. One part that is not is the targetname parameter. For example, I use it to connect to SQL Server databases like this:
cmdkey /add:my.remote.sqlserver:1234 /user:me /pass:secret
I discovered that I can add the port number on the end quite by accident. I need it since my company uses non-standard ports for SQL Server. Recently, I discovered that you can also specify:
cmdkey /add:*.my.domain
to add a default credential for all remotes in a certain domain. Apparently you can't add a port number to the end (or I can't figure out how).
What I'm now wondering is, can the full specification for targetname be documented?

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Dark Theme

I am curious about the thought process behind requiring certain amount of staking for a color scheme.
I would like to suggest to allow the Dark theme for free.
Bringing new people into the eco system and the official wallet not allowing a dark theme unless they stake I think is very off putting. Some people simply cannot do that and a dark theme is pretty much a must for certain people.

Staking 1 Luna sounds not that hard.


Table UI Requirements

The UI is designed to populate the clipboard with data that will be used by the AHK script to input commands to the terminal in game.

There will be a table for managing player names. Each row should have a checkbox, text field for the player name, and a display for the associated hotkey (for now, hotkeys are hard-coded).
Automatically check and enable the associated checkbox once the name field is not empty.
If a name field is empty, the associated checkbox should be unchecked and disabled.
Any time a name is edited or a box is checked, the clipboard should be updated. The clipboard will need to contain each name from the table, trimmed and separated by new line characters. If a box is unchecked, the name should be saved to the clipboard with a prefix of 'IGNOREDPLAYER:'. This will allow the AHK script to skip these when cycling through players, but allow assigned hotkeys to work regardless of checkbox status.
There should be a checkbox to toggle all checkboxes in the checkbox column's header. If any box is both unchecked and enabled, this master checkbox should also be unchecked. Upon checking, all checkboxes where the input name is not empty should be set to checked. Upon unchecking, all checkboxes should be unchecked.
There should be a button to clear all name fields.
There should be a button to allow the user to update their clipboard without having to edit any box or name.

https://github.com/Ryan-Florida/Lethal-Company-Guy-in-Chair/pull/1 Hits all of these.

If any box is both unchecked and enabled, this master checkbox should be unchecked.

To be clear on this, does "unchecked and enabled" mean there's a name in a given box but the checkbox has been unchecked? If that's the case, which I'm assuming it is, then need to toggle off the master checkbox in this case in the above PR.

With the assumption mentioned above, this PR has all the goods.

With PR #1 and commit 456e15c, the basic functionality desired has been added.


doc: update crate documentation

Codecov Report

Merging #74 (2952e95) into master (5b9b597) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   94.41%   94.41%
=======================================
  Files           5        5
  Lines         680      680
=======================================
  Hits          642      642
  Misses         38       38

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 5b9b597...2952e95. Read the comment docs.


getWebpackVersion does not work properly

Do you want to request a feature, report a bug or ask a question?
Bug
What is the current behavior?
svg-sprite-loader have webpack 4 in dependencies.
getWebpackVersion return version for webpack in {project}/node_modules/svg-sprite-loader/node_modules/webpack/package.json
What is the expected behavior?
getWebpackVersion should return version for webpack in {project}/node_modules/webpack/package.json
If the current behavior is a bug, please provide the steps to reproduce, at least part of webpack config with loader configuration and piece of your code.
I just run my nuxt.js project and get this error:
basic chunk optimization/{project}/node_modules/svg-sprite-loader/lib/utils/get-module-chunk.js:14
    chunks = Array.from(module.chunksIterable);
                   ^

TypeError: Cannot convert undefined or null to object`

Please tell us about your environment:

Node.js version: 8
webpack version: 3
svg-sprite-loader version: last
OS type & version: macOs

Проще по русски написать :)
Ошибка возникает в get-module-chunk.js, условие webpackVersion.IS_4 верно, хотя у меня стоит 3 вебпак.

Could you please create repo with minimal setup to demonstrate a problem (package.json, webpack config, SVG image and piece of your code). If you don't want to create a repository - you can create a gist with multiple files.

fixed..

Fixed in<EMAIL_ADDRESS>please update

@kisenka I had to go back to the version<EMAIL_ADDRESS>because my Cypress was breaking after 4.1.3.
It couldn't find the require.main.require('webpack...').
Although, the last version worked well with my App and storybook.

@williansabiao which NodeJS version is used?

The Cypress uses his own Node build. It's v8.2.1 😞 .
I just saw they are working on to update it. So, probably it is the reason I'm getting the error in the new versions of svg-sprite-loader, right?
I guess I can update to a newer version when Cypress update as well. That's ok. Thanks.

Just wanted to highlight that this change (PR #314 ) breaks builds where webpack is started from a global Node.js application such as a global Gulp installation.
In that case the main entry point for Node.js will be the global Gulp and its paths (global paths) will be present in require.main.paths. If you don't happen to have webpack globally installed the webpack lookup will fail.
I have found the following workarounds:

Use local Gulp through npx or similar
Patch require.main.paths with the local "node_modules" folder in my gulpfile.js so that the webpack lookup works (however this makes "global" "local", your milage may vary). This is the solution I have chosen since I need to run global Gulp (using the Task Runner in Visual Studio)

Not sure if this is a problem that this of concern for this module, but since others might encounter the same issue I post my findings here.

I'm also running into this problem when running ESLint (with eslint-import-resolver-webpack).
This line:
const webpackVersion = require.main.require('webpack/package.json').version;
Causes this error:

Error: Cannot find module 'webpack/package.json'

I use lerna and there is two directories: with builder and with sites. When I bootstrap builder to each site, causes this error:

Error: Cannot find module 'webpack/package.json'

Is there any way to make it work with eslint-import-resolver-webpack?

@williansabiao @linqcan @Nettsentrisk @ivan-podgornov @ViieeS
Using autodetecting webpack version was very bad idea. I've published alpha version without detecting webpack version, could some of you guys help me with trying it out?
npm install svg-sprite-loader@drop-webpack-version-detector

@kisenka works fine! Thank you!

Published in<EMAIL_ADDRESS>


Build p1

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

Please ignore this, closing


expose with_public_ip var of metadata service

@Erin-Boehmer how did this work for you, since you added the option not to assign the IP in the first place in #37 ?
I believe @andreinechaev ran into this issue with ECS not being able to pull images when it doesn't have IP address but runs in a public subnet. If it was a private subnet, I think it would be fine granted NAT gateway and Internet gateway are set up as usual.
We should maybe add a note that you should either

use a public subnet and remember to set with_public_ip to true
use a private subnet, then with_public_ip doesn't matter (?)
use a public subnet, leave with_public_ip to false, but configure the module to pull container images from a private repo accessible from within the VPC

I'm now trying to think what's the best way to communicate this to module users 🤔 Maybe add something in the description for the subnet setting too.

@oavdeev  Agree, i ran into the same issue today.
I was running this:
https://github.com/outerbounds/terraform-aws-metaflow/blob/v0.7.1/examples/minimal/minimal_example.tf
And because ECS is set to automatically use a private ip, the ECS cannot pull the image for metadata service.

Therefore now the minimal example would not work, since the minimal example uses public subnets for the ECS.
According to AWS, private subnet should have with_public_ip  set to false.
Link: https://docs.aws.amazon.com/AmazonECS/latest/developerguide/task_cannot_pull_image.html


[Feature]: alt_title needs to defer to title  over mapping_name when using both in the same metadata update

Is your feature request related to a problem? Please elaborate.
When using alt_title and title in the same metadata update PMM chooses to use the mapping_name for the item instead, thereby ignoring the functionality of title.
Describe the solution you'd like
When using both for a metadata update title should win over the mapping name.
Additional Information
Discord discussion of this with mattyteds: https://discord.com/channels/822460010649878528/992508369048973343/992540360893005834

fixed in 1.17.2


Make the migrated code run successfully

(base) ➜  example git:(master) ninja -C /Volumes/MyExternal/ExternalRefCode/engine/src/out/android_debug_unopt_x64; /Volumes/MyExternal/ExternalRefCode/flutter/bin/flutter --local-engine-src-path=/Volumes/MyExternal/ExternalRefCode/engine/src --local-engine=android_debug_unopt_x64 run --device-id emulator-5554


createNonInterpolatedUIImageFromCIImage flips the image

I just started using this guide on how to generate QR codes for my app and it mostly works, but I notice that the -createNonInterpolatedUIImageFromCIImage method posted here actually flips the given image vertically. I verified this by comparing it to UIImage +imageWithCIImage, which gave me an interpolated image that was the right orientation. Adding a line
scaledImage = [UIImage imageWithCGImage:scaledImage.CGImage scale:1.0 orientation:UIImageOrientationDownMirrored];
into the end of the method flips it back to vertical.

A better way of doing this is probably within the existing CGContext. Should add these 2 lines before CGContextDrawImage(context, CGContextGetClipBoundingBox(context), cgImage);
CGContextTranslateCTM(context, 0, size);
CGContextScaleCTM(context, 1.0, -1.0);


Don't overwrite user-provided account ID (fixes #64)

When originally adding that change I definitely wanted to avoid overriding it. When I used non-master key values, the other API calls didn't work because the wrong (mismatched) account ID was being sent

Interesting. Will test with different keys in a bit


Nginx version 1.18.0 is being used by few components in Automate

Describe the bug
Although, Automate Release notes says, Nginx 1.21.0 is used.
But it is noticed few components still use Nginx 1.18.0.
To Reproduce
Steps to reproduce the behavior:

in list form if possible

Expected behavior
A description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

OS: [e.g. rhel6]
Browser [e.g. chrome, safari]
Automate Build Number [e.g.<PHONE_NUMBER>2009]

Additional context
Add any other context about the problem here.

More just adding for some relevance. This is the latest release of a2.
There are newer versions of chef/openresty-noroot available based on a new version of the chef/chef-server-nginx hab pkg.
Also provided a command to identify the hab pkgs that currently have a Transitive dependency on core/nginx/1.18.0
[root@ip-10-254-1-152 ~]# chef-automate version
Version: 2
  CLI Build:<PHONE_NUMBER>0314
  Server Build:<PHONE_NUMBER>4140
[root@ip-10-254-1-152 ~]# while read -r nginx; do echo $nginx; hab pkg exec $nginx nginx -v; done < <(hab pkg provides nginx)
chef/automate-es-gateway
nginx version: nginx/1.21.3
core/nginx
nginx version: nginx/1.18.0
chef/automate-load-balancer
nginx version: nginx/1.21.3
chef/openresty-noroot
nginx version: openresty/<IP_ADDRESS>
[root@ip-10-254-1-152 ~]# find /hab/pkgs -name "TDEPS" -type f -exec grep -H 'nginx' {} \;
/hab/pkgs/chef/automate-ui/2.0.0/20211216090314/TDEPS:core/nginx/1.18.0/20200506101012
/hab/pkgs/chef/automate-builder-api-proxy/0.1.0/20211118152647/TDEPS:core/nginx/1.18.0/20200506101012
/hab/pkgs/chef/automate-cs-nginx/14.4.4/20211118152358/TDEPS:chef/chef-server-nginx/14.4.4/20210520121142
/hab/pkgs/habitat/builder-api-proxy/8997/20200812161534/TDEPS:core/nginx/1.18.0/20200506101012

For the purpose of building in particular nginx modules, components/automate-es-gateway and components/automate-load-balancer both build nginx from source rather than using the upstream core/nginx package, and those versions were bumped in #5405. Which is different than the chef/openresty-noroot package built from the chef/chef-server/src/openresty-noroot git repo.
Another way to find the transitive dependencies in the product without installing it is the chef/automate/tools/hab-dep-graph/main.go tool:
btm@btm-thinkpad:~/src/automate (main)$ go run ./tools/hab-dep-graph --help
Generate dot file for Habtiat package dependencies

The dot file can be passed into the dot command from GraphViz

    go run ./tools/hab-dep-graph | dot -Tsvg -odeps.svg

You can generate the graph for a particular package by passing it as an argument:

    go run ./tools/hab-dep-graph chef/compliance-service

Or combine this with other tools to create the dep graph for a particular release:

    pkg_list="$(curl https://packages.chef.io/manifests/dev/automate/latest.json 2>/dev/null | jq -r '.packages | .[]')"
    go run ./tools/hab-dep-graph $pkg_list

Usage:
  hab-dep-graph [PKG_IDENT...] [flags]

Flags:
  -c, --channel string    Channel to pull non-fully-qualified package dependency data from (default "unstable")
  -d, --debug             Enabled debug output
  -h, --help              help for hab-dep-graph
      --ignore-versions   Ignore dependency versions when building the graph

Use the current release:
btm@btm-thinkpad:~/src/automate (main)$  pkg_list="$(curl https://packages.chef.io/manifests/dev/automate/latest.json 2>/dev/null | jq -r '.packages | .[]')"
btm@btm-thinkpad:~/src/automate (main)$ go run ./tools/hab-dep-graph $pkg_list | grep core/nginx
# [snip the stderr output]
"habitat/builder-api-proxy/8997/20200812161534" -> "core/nginx/1.18.0/20200506101012"
"chef/automate-ui/2.0.0/20211216090314" -> "core/nginx/1.18.0/20200506101012"
"core/nginx/1.18.0/20200506101012" -> "core/bzip2/1.0.8/20200305225842"
"core/nginx/1.18.0/20200506101012" -> "core/glibc/2.29/20200305172459"
"core/nginx/1.18.0/20200506101012" -> "core/libedit/3.1.20150325/20200319193649"
"core/nginx/1.18.0/20200506101012" -> "core/ncurses/6.1/20200305230210"
"core/nginx/1.18.0/20200506101012" -> "core/openssl/1.0.2t/20200306005450"
"core/nginx/1.18.0/20200506101012" -> "core/pcre/8.42/20200305232429"
"core/nginx/1.18.0/20200506101012" -> "core/zlib/1.2.11/20200305174519"

Note that when running this tool fully qualified package names you need to be really conscious of what channel (-c option) is being used to find a fully qualified package name for.


Crash when connecting a client

msgflo:fbp components-changed [ 'DetectPartyMods' ] +23ms
/opt/c-flo/node_modules/msgflo/src/protocol.coffee:24
if (component.definition) {
^
TypeError: Cannot read property 'definition' of undefined
at fbpComponentFromMsgflo (/opt/c-flo/node_modules/msgflo/src/protocol.coffee:19:15)
at Library.<anonymous> (/opt/c-flo/node_modules/msgflo/src/protocol.coffee:216:16)
at emitTwo (events.js:106:13)
    at Library.emit (events.js:192:7)
at Library._updateComponents (/opt/c-flo/node_modules/msgflo/src/library.coffee:133:6)
at /opt/c-flo/node_modules/msgflo/src/library.coffee:184:8
at FSReqWrap.oncomplete (fs.js:112:15)

@bergie This bugreport has insufficient information.
What program crashed? How did you start this process (options/arguments)? What do you mean by "connecting a client" (is DetectPartyMods the client?).

Well, DetectPartyMods is what is (very likely) being sent as the component in the discovery messages, and those files have c-flo/DetectPartyMods - which is not the same.


QualysVM Solution Migration.

https://microsoftapc.sharepoint.com/:f:/t/SentinelCATTeam/EjNJJ0EbWUNCqz7U9_ETWi8B8R-yY7Yeb_dggqKcVb9lbg?e=qixj3Y   Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

@NikTripathi , please add “status”: “Available” property in analytic rules

@NikTripathi,  in data connector “isPreview” property need to set to “false”

@v-dvedak , Changes done as per suggestions.
Thanks


Tracking wlroots releases

How do you manage compatibility with wlroots releases? Currently pywlroots only supports wlroots 0.12.0, whereas wlroots is currently on 0.13.0 (only recently). Between releases they track breaking changes in an issue such as this for 0.14.0: https://github.com/swaywm/wlroots/issues/2844. Do you aim to exclusively support the current release or maintain some kind of backwards compatibility?

Just saw that you did update to work with 0.13.0 so ignore that part :)

This is a great question. With wayland, it looks like everything is generally added in a nice backwards compatible way, and then through the protocols, that we can generally keep track of our own versions. But I was thinking it might make sense to change the versioning here to keep inline with the wlroots version that is supported. So something like 0.13.x, where our bugfixes can be new patch versions, but signifying that it supports wlroots 0.13. While we are still in the early phase, supporting the latest version is probably good enough, we can try to maintain backwards compatible support as much as makes sense and as much as it makes sense to keep up the support for the older versions. In which case, having the versions following the wlroots version would make keeping separate branches alive for separate versions easier. That is probably more maintenance than we need at this point, but a good thing to think about as we move this into qtile and need to support it more.

Happy to have thoughts or feedback on the best way to version this and maintain support for upstream wlroots functionality!

Oops forgot to reply. I noticed you changed the version to 0.13.0.

In which case, having the versions following the wlroots version would make keeping separate branches alive for separate versions easier. That is probably more maintenance than we need at this point, but a good thing to think about as we move this into qtile and need to support it more.

Agreed completely. If people really wanted to they could package the latest 0.13.x version as pywlroots-0.13 after we're done with it, if they wanted to keep using the older version.

Sounds good. Yeah, that agrees with how I was seeing that.


Student id card

Summary

Added the Student ID Card feature which had not yet been implemented in flutter.
The Student ID Card now displays the user's picture, barcode, classification, name, college and major.
The user can expand their barcode by tapping the barcode.
Changelog

[General] [Add] - Added UI for student_id_card.dart as well as logic to complete student_id_data_provider.dart and all associated student_id_[...].dart models.
Test Plan

Tested with different iOS Simulators: From largest (iPhone 11 Pro Max) to smallest (iPhone SE).
Tested with Android Emulators too such as: Pixel 3a, Pixel 2a
Tested with edge cases such as long name, college and/or major, now scales dynamically.
Tested the readability by scanning the screens with multiple cameras and scanning applications (no real world testing).

Tablet patch is included now. Support for iPadOS and Android tablets.


automationtstpro:main

Hi,
Your homework has been reviewed.
Great! Keep up the good work!
Best wishes from Automation Test Pro,
Khanh


SEAB-6423: Consistently impose SourceFile limits

Update
Alrighteee... the original PR has been overhauled.  Important changes:

The SourceFile size/binary limiting code is now in LimitedSourceFileBuilder, which demonstrates the "step builder" pattern:
http://rdafbn.blogspot.com/2012/10/step-builder-pattern-definition-and.html
Other helpers have moved to SourceFile.
We now apply a 10MB per-file download limit in the low-level GitHub download code: the code checks the file size of a Blob before downloading, and if it's >10MB, it is not downloaded at all.   This prevents a trivial DOS, but allows us to download large-but-not-entirely-unreasonably-sized files (<10MB) and analyze their content, so we can apply content-specific limits and talk about them in the related error messages.
Will create followup tickets to limit the overall number of bytes downloaded/stored per version, and to add status information to SourceFile that will signal that the file is complete, the content is an error message, etc.
Suggest we don't further increase the notebook file size limit, or spend the time to make it configurable, until we see how the above tickets pan out.
Our code mangles binary file content - input bytes are mapped as UTF-8 to Unicode, and non-UTF-8 binary sequences get converted to code point 65533:
https://www.fileformat.info/info/unicode/char/0fffd/index.htm
If we want to accurately download and save binary content in the Dockstore database, there is some significant refactoring to be done.
My initial goal was to limit the scope of this PR so we could hotfix if desired, but in retrospect, hotfixing it feels too risky.

Description
Whilst diagnosing https://ucsc-cgl.atlassian.net/browse/SEAB-6423, we discovered that when registering a 2.5MB notebook file, the corresponding SourceFile's content is null.  This causes the UI "Preview" notebook tab to report that the notebook does not have any content.
Investigation revealed that the webservice only enforces the maximum size and non-binary SourceFile requirements on the old entry registration path.  On the .dockstore.yml registration path, SourceFiles are still effectively limited, but only because:

the GitHub API query used to retrieve a file's contents returns null when the file's size is >= 1MB.  This null is stored as the corresponding SourceFile's content.
Postgres won't store strings containing the nul (0) character.  So, when the webservice attempts to create an entry containing a binary SourceFile, a database exception is thrown.  (I haven't confirmed the latter but am pretty sure it's true.)

To address the above, this PR:

adds code to successfully retrieve the content of files larger than 1MB, via the "blob" GitHub API.
increases the maximum size limit of a Jupyter notebook file to 3MB.
adds a set of helper methods to create, copy, and duplicate SourceFile instances.
moves the requirements enforcement code to the create helper method.
changes existing webservice SourceFile creation/manipulation code to use the helper methods.

If we consistently use these SourceFile helper functions, we make it easy to tweak the SourceFile creation process (to add a new requirement, for example) and ensure that, should we modify the SourceFile object itself, all fields get copied correctly.
As before, when a file doesn't meet the size or non-binary requirements, we set the corresponding SourceFile's content to an error message.  For example, "Dockstore does not store binary files".  However, code that subsequently uses the SourceFile has no way to differentiate the error message from valid file content.  So, after this PR, when a "too large" notebook is registered, the notebook's "Preview" tab will try to parse the error message as a Jupyter notebook, and fail in a different way.
Thus, in a followup PR, I propose to modify SourceFile so that we can differentiate a normal SourceFile from one that didn't meet our requirements.  Perhaps we store the error message in a different field, or add a flag or enum that  differentiates "COMPLETE" and "ERROR" content.   An enum would allow us to support other states, such as a "SUMMARY" state, which would describe an abbreviated form of the full content (perhaps useful if we diversify into indexing AI models, which we might not want [or be able] to store in complete form).
It can be argued that [some of?] the helpers should be methods of SourceFile, etc.  If anyone feels super-strongly, we can tweak.  There's a rationale behind the current design, but I've already typed too much in this description, so I won't dump it here.
This PR is a little bit scary from a reliability/correctness point of view.  The new code passes our existing tests, and appears to be covered.  However, given the variety of states and contexts involved, it's very difficult to confirm that we haven't introduced a bug, without spending tons of time auditing the existing tests and filling any holes.  Given the way surrounding code is currently architected, it's hard to test, which makes verifying correctness even more challenging.  It'll help to visually inspect very carefully and be on the lookout for sourcefile-related problems after we deploy to qa, but it's still a little risky.
Review Instructions
Register the notebook which is linked in the ticket, and confirm that Dockstore successfully read it, by visiting the notebook page and confirming that the Preview is correct.
Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6423
Security and Privacy
No concerns.

[x] Security and Privacy assessed

e.g. Does this change...

Any user data we collect, or data location?
Access control, authentication or authorization?
Encryption features?

Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

increases the maximum size limit of a Jupyter notebook file to 3MB.

From a different conversation, the limit per file is possibly less important than the limit per version (or even per workflow?).
Link follow-up ticket here if spun off

Thus, in a followup PR, I propose to modify SourceFile so that we can differentiate a normal SourceFile from one that didn't meet our requirements.

:+1:

Ok, I overhauled the PR to address most of the feedback.  Please refer to the "Update" section at the top of the PR description for more information.


Loom metamodel should perform single pass of streaming inference in INITIALIZE

The current implementation of INITIALIZE does not actually initialize any Loom models, it only prepares database tables to store outcomes of loom analyses.
https://github.com/probcomp/bayeslite/blob/39e96af568d8f11528583b263042b4711e0cd5ff/src/metamodels/loom_metamodel.py#L309-L324
Instead, INITIALIZE should perform a single subsample annealing pass through the data, which will ensure that models are actually initialized and queries can be run against them. Model improvement can then be achieved using additional SMC passes through the dataset with ANALYZE.

Should document this behavior.


Issues with the collision for rotated portals

I made a video about them. And here's an explanation of what happened in it: I made two examples of the rotated portals. A - a portal rotated along the Z axis by 90 degrees. B - a portal rotated along the Z axis by 180 degrees. (Note: It doesn't matter if the portal rotated along the Z or X  axis. The result are the same.) As you can see at the example A, the player almost instantly falling through the portal after starting walking. The example B is a bit better. The player can walk in the portal for about a second but then he teleports.
Also the players can't crouch in the portals that has non upside-down rotation. (<180 degrees)

Probably fixed in 0.39


Missing spinner when IPFS multimedia takes a while to show up

Example of heavy-weight IPFS blob:
https://tzcomet.io/#/token/KT1RJ6PbjHpwc3M5rw5s2Nbmefwbuwbdxton/75206
( https://hicetnunc.art/objkt/75206 also takesx a while to display it for me )

Spinner seen !


Update Firefly III image

What this PR does / why we need it:
Firefly III image has moved
Latest version of Firefly III at fireflyiii/core:latest
Which issue (if any) this PR fixes:
#512

Thanks! :+1:


Toggle ligature failing in develop

Toggling ligatures (both on and off) always fails in develop mode.

How ligatures are set/unset changed in Verovio. This is fixed in Verovio by this commit.


Pin proj4

See https://github.com/conda-forge/cartopy-feedstock/pull/16

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


New tag

Please add [center][/center] and [left][/left] and [right][/right] tags.
Also add support for [color] tag but additionally using text alias instead a hex code.
For example [color="#FF0000"][/color] => [color=red][/color]
Thank you in advance!

PR is ready to be merged and to update pub.dev. Will probably do so tomorrow.


error C2373: '__pfnDliNotifyHook2': redefinition;

ERR

 win_delay_load_hook.c
C:\Users\zetsin\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\src\win_delay_load_hook.c(34): error C2373:
'__pfnDliNotifyHook2': redefinition; different type modifiers [D:\GitHub\node-winode\node_modules\ref\build\binding.vcx
proj]
  C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\delayimp.h(134): note: see declaration of '__pfnDliNot
  ifyHook2'

WHILE

npm install ref

ENV

Microsoft Windows [Version 10.0.10240]
(c) 2015 Microsoft Corporation. All rights reserved.

>node -v
v4.4.7
>npm -v
3.10.2
>node-gyp -v
v3.4.0
>python -V
Python 2.7.12
>msbuild /ver
14.0.25420.1

npm -g install npm@next?

Thanks a lot, @addaleax :+1:
Something wrong happened:
"...\npm\node_modules\npm\node_modules\node-gyp\src\win_delay_load_hook.cc':** No such file or directory

after  npm -g install npm@next
So I did that:
cd "...\npm\node_modules\npm\
npm install

It works, finally

Something wrong happened:

That does not look like something that should happen. Do you still have the npm-debug.log from that installation attempt?

Let me do it again.
rd/s/q "...\npm\node_modules\npm"
npm -g install npm@next
cd "...\npm\node_modules\npm\node_modules\node_gyp"
dir

I found that there is no "src" folder in "node_gyp", so the wrong happened when i use node-gyp build(e.g. npm install ref)

"...\npm\node_modules\npm\node_modules\node-gyp\src\win_delay_load_hook.cc':** No such file or directory

yeah a typo, im sorry @bnoordhuis

No problem.  Just to be sure, can you delete your node_modules directory and install npm@next again?
If you still can't build ref, can you post the full build log of npm install ref?

I found a workaround, go to AppData\Roaming\npm\node_modules\npm and type npm remove node-gyp then npm install node-gyp.
I think this is due to using npm-windows-upgrade. It guides to use --production flag so I guess it installs npm with --production too. So node-gyp was missing src/.
First I had a problem that the upgrade was installing<EMAIL_ADDRESS>and it had the previous node-gyp. I posted a comment about it. So I had<EMAIL_ADDRESS>in my global node_modules but 3.3.1 inside npm.
But now I'm finally able to build. :)

fwiw npm@next is probably not going to fix this, yet, there was a problem in the upgrade that went out with 3.10.4, see https://github.com/npm/npm/issues/13256 (my fault somehow I believe). It's been fixed in tree and I imagine it'll be out soon in a 3.10.5. npm@2 works fine though if you're able to use v2.

@rvagg I think I may have the same issue with npm 2.14.12. I have pretty much the same environment as @zetsingithub, except for the npm version. I was trying to install mariasql and got:
win_delay_load_hook.c
C:\Users\Andre\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\src\w
in_delay_load_hook.c(31): error C2373: '__pfnDliNotifyHook2': redefinition; dif
ferent type modifiers [D:\Projects\betarena_chat\node_modules\mariasql\build\de
ps\libmariadbclient\mysys_ssl\mysys_ssl.vcxproj]
  C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\delayimp.h(134
  ): note: see declaration of '__pfnDliNotifyHook2'

Do you know if this issue might affect npm@2 as well?

yep, you need 2.15.9: https://github.com/npm/npm/releases/tag/v2.15.9
I think npm i npm@2 -g should be enough to upgrade you, we don't carry it in our Node LTS releases yet but it'll get there.

@rvagg That did the trick, Thank you!

Had the same issue, npm install npm@next fixed it for me, if anyone else is in the same boat.

npm -g install npm@next appears to be the magic sauce.  Thanks for posting that!

Same here. I tried to install the oracledb module and got the same redefinition; different type modifiers for __pfnDliNotifyHook2 error.
Doing a npm -g install npm@next did fix this issue.

npm -g install npm@latest should do the trick now that it's been pushed out in a stable release, no need to jump forward to next which can sometimes contain regressions.

Hade the exact problem when trying to install node module "bcrypt".  Installed it without any errors now, cheers!

I had same problem with "os-service", but I have a solution also.
npm install os-service --force
cd node_modules/os-service
node-gyp rebuild

I had a lot of trouble getting this working for Oracle drive...same error with node-gyp.  These are the prerequisites.  I think this will also work for other versions of Visual Studio but I used the lastest Community edition 2015.
You must have at least 3.10.8 version of NPM.  This is critical.
npm -g install npm@next
Make sure Node globally uses the same version 2015
npm config set msvs_version 2015 --global
1) Delete node_modules directory in your project
2) npm install

if you are working on an old version like I was, a simple npm install -g npm (without the @next worked.

npm -g install npm@next worked. Thanks.

For me npm i -g npm@next didn't work, but the trick from @RanzQ to reinstall node-gyp worked ! Thanks a lot this was driving me crazy

What shall I do if I don't want to use next?

@XadillaX It’s been a while since you had to use npm@next to resolve this; just upgrade your npm or node to the latest version and you should be fine.

just to note that npm install -g npm@next broke my npm x.x
so i uninstalled node and installed again, tried it all again and the npm got broken again. Any command i try to execute, even only "npm" just causes a lot of errors like "cannot find npmlog module".
Need to say that i'm using nvm to install and use node 5.3.0 as my old project needs. And the first issue i had was installing node-oracledb. This project worked months ago, and it still works in my macosx but in windows i'm with these errors.
Will try to install node 5.3.0 and use just npm install -g npm as @jasonswearingen suggested

note: most of the workarounds here recommend upgrading node-gyp by upgrading npm, which is fine. But in some cases users may instead wish to upgrade just node-gyp, which also fixes this issue (since the underlying bug was fixed in https://github.com/nodejs/node-gyp/pull/952).

@springmeyer I am using node-gyp v3.6.2 and still having this problem (node v6.11.0 and npm v3.10.10). Does that release contain the fix you mentioned?

Yes @jacobq - as far as I can tell the fix landed in v3.3.3: https://github.com/nodejs/node-gyp/issues/956. Perhaps you have an older version being invoked without realizing it?

@springmeyer any ideas how that might happen? For example, is it possible for another module to list it in package.json or something?
I'm currently trying the suggestion from https://github.com/nwjs/nw.js/issues/5088
//PfnDliHook __pfnDliNotifyHook2 = load_exe_hook;
decltype(__pfnDliNotifyHook2) __pfnDliNotifyHook2 = load_exe_hook;

ERR

 win_delay_load_hook.c
C:\Users\zetsin\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\src\win_delay_load_hook.c(34): error C2373:
'__pfnDliNotifyHook2': redefinition; different type modifiers [D:\GitHub\node-winode\node_modules\ref\build\binding.vcx
proj]
  C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\delayimp.h(134): note: see declaration of '__pfnDliNot
  ifyHook2'

WHILE

npm install ref

ENV

Microsoft Windows [Version 10.0.10240]
(c) 2015 Microsoft Corporation. All rights reserved.

>node -v
v4.4.7
>npm -v
3.10.2
>node-gyp -v
v3.4.0
>python -V
Python 2.7.12
>msbuild /ver
14.0.25420.1


IS vcpkg compatible with vs2019?

when I use vcpkg installed ffmpeg, vs2019 can not find the ffmpeg header file  ?

Have you run vcpkg integrate install and added -DCMAKE_TOOLCHAIN_FILE=[vcpkg root]\scripts\buildsystems\vcpkg.cmake to VS?
Was your setup working with vs2017?
To answer your question vcpkg is supposed to be vs2019 compatible.

thanks,It's ok now .first I install ffmpeg with x86-windows-static, not ok。when install with x86-windows,it's ok now.


Run edge cases with static_layer only

Follow these steps to toggle static layer generation
https://wiki.jpl.nasa.gov/display/operasds/Making+changes+to+settings.yaml+on+Mozart

Performed static_layer toggle

Some errors so will have to revisit


moved the set -o lines under the ACCESS_TOKEN block

Attempting to run immuno on compute1 results in varscan failing. @tmooney suggested moving the set -o lines after the wget block. As of now, since errext and pipefail are set before wget, when the command is executed wget does not find the server and it causes a non-zero exit. I have tested this change and it appears that this fix allows varscan to execute successfully.

Performed a test on a full dataset and confirmed that the varscan results are identical (except for expected differences including file paths and time stamps).


Snowflake Tag Association - Object Identifier Mapping Incorrect

Provider Version
0.56.3
Terraform Version
1.3.9
Describe the bug
When using the snowflake_tag_association resource to associated a tag to column(s).
The object_identifier block is not properly capturing the object.
Example:
resource "snowflake_tag" "pii-tag" {
  provider       = snowflake.sys
  name           = "PII_TAG"
  database       = module.security.db.name
  schema         = snowflake_schema.pii-schema.name
  allowed_values = toset(["SSN", "DOB", "NAME", "EMAIL", "ADDRESS", "PHONE", "PRN", "TIN", "EIN", "ID", "CARD NUMBER", "OTHER"])
}

resource "snowflake_tag_association" "table_association" {
  tag_id      = snowflake_tag.pii-tag.id
  object_type = "COLUMN"
  tag_value   = "VALUE"
  object_identifier {
    database = "DB"
    schema   = "SCHEMA"
    name     = "TABLE.COL"
  }
}

snowflake_tag_association.table_association: Creating...
╷
│ Error: error associating tag to object: ["DB"."SCHEMA"."TABLE.COL"] with command: [ALTER TABLE "SEC"."TAGS"."TABLE" MODIFY COLUMN COL SET TAG "SEC"."TAGS"."PII_TAG" = 'PRN'], tag_id [SEC|TAGS|PII_TAG]
│
│   with snowflake_tag_association.table_association,
│   on security.tf line 75, in resource "snowflake_tag_association" "table_association":
│   75: resource "snowflake_tag_association" "table_association" {
│
╵
make: *** [apply] Error 1

The object looks like ["DB"."SCHEMA"."TABLE.COL"] but the rendered Sql defaults to the location of of the tag (in this case "SEC"."TAGS").
ALTER TABLE "SEC"."TAGS"."TABLE" MODIFY COLUMN COL SET TAG "SEC"."TAGS"."PII_TAG" = 'PRN';

This issues makes it impossible to apply tags to objects that aren't colocated in the same schema.
Expected behavior
The above resource would render a SQL statement like
ALTER TABLE "DB"."SCHEMA"."TABLE" MODIFY COLUMN COL SET TAG "SEC"."TAGS"."PII_TAG" = 'PRN'

Code samples and commands
Please add code examples and commands that were run to cause the problem.
Additional context
Add any other context about the problem here.

Can I get some love over here? I think this would help some folks out. Column level governance in terraform is one PR away.

@sfc-gh-kumaurya @joshtree41 is this fixed?

@sfc-gh-ngaberel Yes, this will be available in the next release v0.64.0.

Oh yeahhhhh, thank you @sfc-gh-kumaurya! I'll test it out after I see the new version pop up


Windows 11

Is Windows 11 (Pro, latest build) supported? I only see mentions through Windows 10.

Yes, supported.
Official INI is not support any Windows 11 build, but you can replace rdpwrap.ini and get support for your build.
For more information:

Download RDP Wrapper 1.6.2
Execute RDPConf
Open issue section in this repository
Type in search your termsrv.dll version (from RDPConf). Example 10.0.XXX.YYY


deprecation warnings on Ember 2.4.2

Seeing these when using an imgix-image component on an Ember 2.4.2 app:
DEPRECATION: Using the injected `container` is deprecated. Please use the `getOwner` helper instead to access the owner of this object. [deprecation id: ember-application.injected-container] See http://emberjs.com/deprecations/v2.x#toc_injected-container-access for more details.
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:16376:7)
        at raiseOnDeprecation (http://localhost:4200/assets/vendor.js:16286:12)
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:16376:7)
        at invoke (http://localhost:4200/assets/vendor.js:16392:7)
        at deprecate (http://localhost:4200/assets/vendor.js:16345:32)
        at Object.deprecate (http://localhost:4200/assets/vendor.js:26461:37)
        at Object.defineProperty.get (http://localhost:4200/assets/vendor.js:11582:26)
        at .<anonymous> (http://localhost:4200/assets/vendor.js:131841:18)
        at Descriptor.ComputedPropertyPrototype.get (http://localhost:4200/assets/vendor.js:25263:28)
ember.debug.js:6208 DEPRECATION: A property of <myapp-ember@view:-outlet::ember611> was modified inside the didInsertElement hook. You should never change properties on components, services or models during didInsertElement because it causes significant performance degradation. [deprecation id: ember-views.dispatching-modify-property]
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:16376:7)
        at raiseOnDeprecation (http://localhost:4200/assets/vendor.js:16286:12)
        at HANDLERS.(anonymous function) (http://localhost:4200/assets/vendor.js:16376:7)
        at invoke (http://localhost:4200/assets/vendor.js:16392:7)
        at deprecate (http://localhost:4200/assets/vendor.js:16345:32)
        at Object.deprecate (http://localhost:4200/assets/vendor.js:26461:37)
        at exports.default._emberMetalMixin.Mixin.create._Mixin$create.scheduleRevalidate (http://localhost:4200/assets/vendor.js:53421:22)
        at http://localhost:4200/assets/vendor.js:23840:32
        at Object.BasicStream.notifySubscribers (http://localhost:4200/assets/vendor.js:32430:11)

Thanks for bringing this up. Looks like the problem code is here: https://github.com/imgix/ember-cli-imgix/blob/master/addon/mixins/imgix-path-behavior.js#L208-L210
We'll review and see what the path forward here is. Likely just need to update the addon to take advantage of more public APIs.

The second deprecation warning about modifying properties within didInsertElement points to a legitimate performance concern. We've been experiencing browser crashes when moving between pages of collections containing many imgix-image components. Deferring the resize count update with Ember.run.once seems to solve the issue. PR incoming…


dependency management and changes for main.yml file... #issue 81 and #issue71

here in this we are changing from setup.py system to requirements.in file ,we have updated required libraries in requirements.in file by seeing import files and when we run
pip-compile requirements.in
requirements.txt file is generated,according to files mentioned in requirements.in and it is used for better dependency management.so further we don't need the file setup.py so we removed it
this following pr resolves the following issue 71 and  issue81
the following issues are run in my forked repo even workflow was runned in my local environment ,here are screenshots of of the tests passed for 3.9 and 3.10.9 versions of python

so ,once pls review my pr and help me knew if anything is needed

Codecov Report
Base: 86.22% // Head: 86.32% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (42eea2d) compared to base (8b90b20).
Patch has no changes to coverable lines.

: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             @@
##           master      #82      +/-   ##
==========================================
+ Coverage   86.22%   86.32%   +0.09%
==========================================
  Files          12       12
  Lines         421      424       +3
==========================================
+ Hits          363      366       +3
  Misses         58       58

Impacted Files
Coverage Δ

...cell_membrane_segmentor/cell_membrane_segmentor.py
87.28% <0.00%> (+0.10%)
:arrow_up:

...neage_population_model/lineage_population_model.py
81.73% <0.00%> (+0.17%)
:arrow_up:

...n/embryo_generator_model/embryo_generator_model.py
84.61% <0.00%> (+0.24%)
:arrow_up:

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

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

pls don't merge this pr there are some code changes should be added


Rename Azure AD in Microsoft Entra ID

Would it be possible to remove mentions of Azure AD in the document and manage it by Microsoft Entra ID?

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

ID: 5c6b2cf2-eb38-1bd8-23b1-13b18ca1bdc0
Version Independent ID: 5c6b2cf2-eb38-1bd8-23b1-13b18ca1bdc0
Content: Azure security baseline for Azure Virtual Desktop
Content Source: benchmark/azure/baselines/azure-virtual-desktop-security-baseline.md
Service: virtual-desktop
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@marion-msft
Thanks for your feedback! We will investigate and update as appropriate.

Hi @marion-msft
Thank you for bringing this to our attention. Currently, Azure AD documents are being updated to Microsoft Entrа Id. Once all these documents have been updated, related documents will also be modified accordingly. It will take some time to go live for the changes soon.
if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

hi Ajay, you're welome. I have no furter comments at the moment.


Add the ability to add a "message" field when inserting alerts into InfluxDB

Similar to levelField, idField and durationField, having a "messageField" would be pretty handy, allowing the message for the alert to be persisted. This allows for Grafana annotations for alerts to have a nice, user-readable explanation of the conditions that triggered the alert.
We're currently hacking this functionality by having an HTTP server receive the alert's JSON (which contains the message) and uploading to InfluxDB so not super high priority for us, but would be nice to streamline this data flow.

This was implemented a while back.


Project 1 Feedback

Discussed in https://github.com/airavata-courses/neo/discussions/48

Originally posted by vikrantdeshpande09876 February  9, 2022
Tested out the application on my local machine. Runs fine! 💯
Here's a few issues I observed:

The "edit" feature doesn't seem to work. It's essentially just doing the same thing as the trashcan icon (clearing the workspace).
All microservices are allocated the maximum memory, but it might be prudent to set CPU resources LIMITs based on actual functionality of each.
The login feature pops up a Google login each time, but shouldn't this be required only while registering? Signing in could be made available based on a static username and password too (if someone doesn't want to link their Google account to this system).

Best,
Team Scapsulators.

Thanks Team Scapsulators!
That's a good point about allocating resources based on microservice functionality. We'll look into our runtime logs to identify our CPU-intensive services. Once identified, we can update our docker-compose YAML file.
We are also aware of these minor UI issues and have identified how to fix them - should be a quick update. Thanks for reviewing!


base64

Can you please make a base64 encode/decode library~~~(most base64 decoder is not working when string contains new emoji unicode6)

I hava a question when I use the  function base64decode.  The string 'jdncksnckndc'  is a valide base64 ,
but when I try to base64decode('jdncksnckndc'), it cause a URI malformed error


Issues with the project

This project doesn't seem to be building properly, has tagged releases with no artifacts so we can't install manually, and since it isn't building the Macro Deck plugin installer is still installing an old version of this.
Further, OBS ships websocket support natively. The documentation for this project suggests that it doesn't work with anything but a not-supported-in-latest-OBS version of the plugin that is now natively built-in on OBS.
I don't understand what I'm missing as all of this seems to indicate the project doesn't work, but there are no issues complaining about this.

These issues should be resolved after the latest release (v2.0.0). The build artifacts are back in the releases, this was deployed to the Macro Deck extension store, and the documentation has been updated to indicate it works with the built in websocket plugin. Thanks for pointing these out.


Issue: physical deletion of recordings

Currently, the stopping/deletion endpoint just only delete the files in memory (in order to avoid show in the list) but the file is still remaining in the disk.
IMHO, it could be better if we delete from disk too.

Fixed in PR #26

Fixed in PR #26

This issue is solved

This issue is solved


fix: staked amount issue

I settled all balance option as true.

stake amount is not shown on accounts / multi-sig page yet


Plugin doesn't support this project's cordova-android version. cordova-android: 6.4.0, failed version requirement: >=7.0.0

Hi,
I install<EMAIL_ADDRESS>in the<EMAIL_ADDRESS>When I run build, it shown below.
%% Plugin doesn't support this project's cordova-android version. cordova-android: 6.4.0, failed version requirement: >=7.0.0
%% Skipping 'cordova-plugin-firebase-messaging' for android

My cordova version is 8.0.0.
I can't find the cordova-android plugin.
Anyone have idea ?
Thank you.

You have to upgrade Cordova-android version (not cli)
Cordova platform rm Android
Cordova platform add android@latest

You have to upgrade Cordova-android version (not cli)
Cordova platform rm Android
Cordova platform add android@latest

Hi @chemerisuk ,
I cannot use cordova platform add android@latest.
When I use meteor build code, it will be reset to default(6.4.0).
May I know, what's version can be used in the 6.4.0 ?
Thank you.

Try versions before v1

Hi @chemerisuk,
When I use the 0.14.0, it can not be build by<EMAIL_ADDRESS>So I change another fcm plugin.
Thank you.


Prettier 1.15, jsxSingleQuote and inline snapshots

I'm trying Jest with the newly released Prettier 1.15 that supports jsxSingleQuote. I have set it to true in my .prettierrc and after running Jest, tests look like this:
it('...', () => {
    const tree = renderer.create(<Link to='/'>Example</Link>).toJSON();
    expect(tree).toMatchInlineSnapshot(`
<Link
  to="/"
>
  Example
</Link>
`);
});

I've got two questions here:

Is it expected that Jest applies Prettier to the entire file? We don't use Prettier in this project, I was sort of assuming Jest only runs the snapshots through Prettier but before I updated to 1.15, I was getting <Link to="/"> on line 2.
Is it expected that the snapshots themselves get double quotes? All tests are passing but it seems a bit strange.

Thanks.

Yes, formatting just the snapshot part proved difficult,/cc @azz
Yes, the serialization has nothing to do with prettier, that's done by pretty-format

Thanks. I have to admit though that at this point I don't quite understand why Jest needs to format my source code – what would happen if I, for example, used different JSX quotes inside my renderer.create snippet?

We don't format jsx, we format the result from React.createElement passed through some render function.
For prettier discussion, please see #6380

Oh, I probably understand now, it took me a while :) So when Jest transforms this:
expect(tree).toMatchInlineSnapshot();

to this:
expect(tree).toMatchInlineSnapshot(`
<Example>hello</Example>
`);

it does so by reading my source file into AST via Prettier, appending the snapshot in it and calling prettier.format, right? That's pretty drastic but I can understand why it was easier than doing string manipulation (though it doesn't seem like a very complex one..).
Thanks!


[stftpitchshift] New port

Describe the pull request

What does your PR fix?
add new stftpitchshift port for jurihock/stftPitchShift

Which triplets are supported/not supported? Have you updated the CI baseline?
all, no

Does your PR follow the maintainer guide?
yes

If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?
yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thanks for the new port!


Nokogiri/hpricot Needed

I also needed to run gem install nokogiri in order for Gulp to work.
Perhaps something to add to readme?
Error:
No suitable adapter for Premailer was found, please install hpricot or nokogiri (RuntimeError)

+1


Better .plt section support for PPC thunk functions

Binary ninja does not handle references to the .plt section for the PPC architecture if thunk functions are involved.
For instance in the library libosipparser2.so.1.1.0 in the function msg_osip_body_parse() though binja is smart enough to know the register value of r30 when I mouse over it (0x57acc) it does use that value for the memcpy() thunk function sub_33880 call at 0x1e59c and instead leaves the thunk function unlabeled without an xref inside.
Thunk function sub_33880 code (taken from IDA):
lwz       r11, (memcpy_plt - 0x57ACC)(r30) # __imp_memcpy
mtctr     r11
bctr

There are many more examples like this inside libosipparser2.so.1.1.0  - I have attached a binary for reference. I have been using libosip2-4.1.0 if you prefer to compile your own.
libosipparser2.so.11.0.0.zip

I believe I have encountered this issue again with another PPC binary, libtasn1.so.6.2.4 (attached). This time the lack of proper detection of the underlying thunk function leads to a control flow issue where asn1_der_decoding falls through to the next function asn1_der_decoding_element
This illustrates the proper detection on x64:

And here's the same call for the ppc binary:

And diving into the thunk functions on PPC:

The final thunk function on PPC. If this xref was properly detected I assume binja would label it as "no return"

I have also attached the PPC binary for reference.
libtasn1.so.6.2.4.zip


PM page to select documents for an FP

Resolves #PMPage
Creates a PM Page where they can select documents for their FP
<Brief description of the added features or other changes/updates (either bulletpointed list, small paragraph, etc.)>

@rgychiu @arpanlaha
feature completed.


Implement new flyout menu for audio in Windows 10

New flyout menu for audio interface on Windows 10.
Replace audio interface that appears when hitting play/pause and using the volume slider on keyboard with a new, modern-looking flyout that fits Windows 10 and follows fluent design -- ModernFlyouts, available for download on the Microsoft Store, is a great example of how this could be implemented.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

dup against #6273


Combine http exception handlers

! @markhibberd @novemberkilo
As discussed earlier combine the two handlers. Only checks that weren't overlapped were
InternalIOException and ResponseTimeout

👍


How to favorite Tweet using REST client

Is this possible or not?
tyvm

already found it
ty

how?

@greyguy123

@acegilz https://github.com/sferik/twitter/blob/master/lib/twitter/rest/favorites.rb


SWITCHYARD-2906 Context properties are not carried back on fault route

Can one of the admins verify this patch?

ok to test


fix: prevent infinite loop causing stream freeze in DASH multi-period streams

Added a condition to ensure requiredIntervalUs > 0 in DashMediaSource.
Resolves an issue where negative requiredIntervalUs triggered an infinite loop in onPlaylistUpdateRequested(), leading to application/stream freezes.

This fix restores smooth playback for multi-period DASH streams.
This fixes #1698.

Hi @totallyunknown,
Would it be possible to open a new PR from an individual-owned fork? We can't push changes to organization-owned forks like this one unless we have collaborator access. If that's not possible then we can still merge this PR but it will result in an 'evil' merge. See more info here: https://github.com/androidx/media/blob/release/CONTRIBUTING.md#push-access-to-pr-branches


Added nim_tinyfiledialogs package

:]

I would suggest renaming this to tinyfiledialogs, the fact it's a Nim package is already clear by the fact that it's in Nimble and import nim_tinyfiledialogs is too verbose :)

E: tinyfiledialogs: a package by that name already exists.

Yea, I just found a package that wraps the same library :[
Yet, my bindings do include a color picker so I think I'll just rename them to tinyfiledialogs.nim (?) (verbose)

@dom96 Should I open another PR to fix the name conflict or?

this is what I get for not waiting for CI lol, sorry


Beta

These changes should be good to go. Thanks!

Did you clear the outputFileWidget error that we were seeing in the console?


unknown command "node-pools" for "kube-aws" with v0.9.5-rc.6

Hello
I'm running into an issue wherein I'm not able to delete the node pool according to the docs tagged with this release version as noted below.
Please let me know if I'm missing something.
Thanks!
kube-aws version
kube-aws version v0.9.5-rc.6

According to the docs tagged for this release Destroy the cluster

If you created any node pool, you must delete these first by running kube-aws node-pools destroy --node-pool-name  or kube-aws destroy will end up failing because node pools still references AWS resources managed by the main cluster.

so going from the docs, the command to delete the node pool should be:

kube-aws node-pools destroy --node-pool-name  foo

However, when I try to run that command, it seems like it's not a valid sub-command/option?
kube-aws node-pools destroy --node-pool-name nodepool1
Error: unknown command "node-pools" for "kube-aws"
Run 'kube-aws --help' for usage.

kube-aws --help
Manage Kubernetes clusters on AWS

Usage:
  kube-aws [command]

Available Commands:
  calculator  Discovery the monthly cost of your cluster
  destroy     Destroy an existing Kubernetes cluster
  init        Initialize default node pool configuration
  render      Render deployment artifacts
  status      Describe an existing Kubernetes cluster
  up          Create a new Kubernetes cluster
  update      Update an existing Kubernetes cluster
  validate    Validate cluster assets
  version     Print version information and exit

Use "kube-aws [command] --help" for more information about a command.

Was hoping I might be able to use the older version to delete my node-pools created with a newer version of kube-aws (that seems to be missing the node-pool sub-command), but seems like perhaps the node-pools configuration has changed and now it is in the main cluster.yaml file versus in sub directories
/usr/local/bin/kube-aws-v0.9.3-rc.5 node-pools destroy --node-pool-name nodepool1
Error: Error parsing node pool config: open node-pools/nodepool1/cluster.yaml: no such file or directory

To move forward created the nested sub-dirs and copied the cluster.yaml file into the tree and then deleted the node-pools again with the older kube-aws version
mkdir -p node-pools/nodepool1/
cp cluster.yaml !$
/usr/local/bin/kube-aws-v0.9.3-rc.5 node-pools destroy --node-pool-name nodepool1
CloudFormation stack is being destroyed. This will take several minutes

Hi @cmcconnell1, thanks as always for trying kube-aws!
Sorry about the documentation - it seems that we've missed updating the doc before tagging the release. I guess you can consult the latest documentation in the master branch without problems as of today.
Anyway, since v0.9.5, kube-aws node-pools sub-command is removed and integrated into the top-level up, update and destroy commands via #315.
Basically, you can now define your node pool(s) under the worker.nodePools[] array in cluster.yaml and then running kube-aws up creates all the cfn stacks including a control-plane and node pool(s).

Hello @mumoshu
Thanks for the quick response.
AIR, I initially did try to just do a kube-aws destroy but wound up with errors within the cloudformation API, that's when I checked the doc's and went down that route.
Looking back from earlier today AM, I think these are the initial errors from the
kube-aws destroy  before trying the node-pool option sub-command:
13:38:22 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	ops-dev	The following resource(s) failed to delete: [Nodepool1, Controlplane].
13:38:21 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	Controlplane	Embedded stack arn:aws:cloudformation:us-west-1:076613928512:stack/ops-dev-Controlplane-1LCJW7NEY3OY8/7e0128e0-1641-11e7-b531-500c2171aec6 was not successfully deleted: Export ops-dev-Controlplane-1LCJW7NEY3OY8-Etcd0EIP cannot be deleted as it is in use by ops-dev-Nodepool1-GNSGPW1EUPS
13:38:10 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	Controlplane
13:38:08 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	Nodepool1	Embedded stack arn:aws:cloudformation:us-west-1:076613928512:stack/ops-dev-Nodepool1-GNSGPW1EUPS/2b362c80-1643-11e7-a1c3-500cf8ee6035 was not successfully deleted: The following resource(s) failed to delete: [IAMRoleWorker].
13:37:57 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	Nodepool1
13:37:55 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	ops-dev	User Initiated
12:07:44 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	ops-dev	The following resource(s) failed to delete: [Nodepool1, Controlplane].
12:07:43 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	Controlplane	Embedded stack arn:aws:cloudformation:us-west-1:076613928512:stack/ops-dev-Controlplane-1LCJW7NEY3OY8/7e0128e0-1641-11e7-b531-500c2171aec6 was not successfully deleted: Export ops-dev-Controlplane-1LCJW7NEY3OY8-OpsDevIgwPublic1b cannot be deleted as it is in use by ops-dev-Nodepool1-GNSGPW1EUPS
12:07:31 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	Controlplane
12:07:28 UTC-0700	DELETE_FAILED	AWS::CloudFormation::Stack	Nodepool1	Embedded stack arn:aws:cloudformation:us-west-1:076613928512:stack/ops-dev-Nodepool1-GNSGPW1EUPS/2b362c80-1643-11e7-a1c3-500cf8ee6035 was not successfully deleted: The following resource(s) failed to delete: [IAMRoleWorker].
12:05:52 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	Nodepool1
12:05:49 UTC-0700	DELETE_IN_PROGRESS	AWS::CloudFormation::Stack	ops-dev	User Initiated

I got in a bit of a mess by manually trying to purge, detach, disassociate resources that I saw in the CF event logs.
Will pick back up on this tomorrow AM.
Thanks

Another note for anyone getting stuck with orphaned kube cluster infrastructure components
(especially those using deis workflow) is that you will want to delete and un-munge anything done outside of kube-aws--including IAM roles you might have applied to cluster resources; your deis ELB, etc., before destroying your kube cluster.
Else the Deis ELB persists and has a network interface associated and it's contained within a kube cluster subnet.  So these will fail to delete and cause your cnf stack to fail upon delete, and ultimately prevents any clean kube-aws destroy from completing.
A tool that I seem to not be able to live without now is awless, which is so much more useful to show relationships and help track down dependencies, cruft, etc.
awless show subnet-123ae456
| PROPERTY ▲ |           VALUE           |
|------------|---------------------------|
| CIDR       | <IP_ADDRESS>/24              |
| Default    | false                     |
| ID         | subnet-123ae456           |
| Name       | ops-dev-OpsDevIgwPublic1b |
| Public     | true                      |
| State      | available                 |
| Vpc        | vpc-abc12ca1              |
| Zone       | us-west-1b                |

# Relations:
us-west-1[region]
	↳ @terradatum-main[vpc]
		↳ @ops-dev-OpsDevIgwPublic1b[subnet]

Siblings: @EXTERNAL-1A[subnet], @EXTERNAL-1B[subnet], @INTERNAL-1A[subnet], ... (display all with flag --siblings)

Note: Confirmed to work with v0.9.6-rc.2 according to https://github.com/kubernetes-incubator/kube-aws/issues/189#issuecomment-294894954
Thanks for your confirmation @cmcconnell1 👍

Also - thanks for the tips! I believe it should be noted in our documentation. I'll open an another issue to track the documentation.


Add link to latest release to long version.

Hello I added a simple link to the latest GitHub release to the long version string. Please let me know if you would prefer a different message.
This is for issue #1023.

Hey @hjarrell - I think I should have been more specific in the issue that I filed.
What we're looking for here is version-specific output, so if the version they're on is not the latest (say, version 1.6.0), it should print out a link to that release instead of the latest one.
As a side note, issue https://github.com/cloudflare/wrangler/issues/397 seems semi-related to this as well.

Thanks for the feedback! I'll look at that issue and update the PR with a version-specific output.

solved by #1190


You can take Items out of the menu with Hotkeys

Using Hotkeys to swap content of your hotbar with things inside the menu allows to put and take items to and from the Menu.
This means that InventoryClickEvent is not cancelled.

Thanks for reporting, it will be solved for next update!


Add jasmine 2.4.1 to teaspoon-jasmine

Adds jasmine 2.4.1 to teaspoon-jasmine.

Thanks! Yeah, this is why we broke them off into separate "sub" gems.. at some point later we'll probably extract them and have different repos, but this makes life easier for now. I'll merge, leave this on master for a bit, and then release it after some testing has happened.


docker: fix typo in README.md

https://github.com/tungbq/devops-basic/tree/main/topics/docker#official-website-documentation-of-your_topic
Just a quick task to change from Official website documentation of YOUR_TOPIC to Official website documentation of docker

Hi, I created a PR for this issue 😄


Add url note support for pushbullet

Description:
This will allow the user to push a note with an URL inside (as a data) which will open the recipient's browser (Pushbullet behaviour for url note)
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#1182
Example entry for an action:
action:
  service: notify.NOTIFIER_NAME
  data:
    title: Send URL
    message: This is an url
    data:
      url: google.com

Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

🐬


README

README
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN  the prework project repository
WHEN I view the README for the repository
THEN I can see a professional README that provides information on the project and how to use it

Added a simple README to the project, closing ticket.


feat: Add compareSnapshot option to allow test failure

This is useful when you want to just generate full report with failed snapshots.
Our project is changing fast and forcing all visual tests to fail makes it hard to maintain visual tests. We would like to use cypress-image-diff to review all visual changes periodically, but we do not want to fail full e2e test suite each time.
Sorry for not contacting to about this feature earlier, but it's a simple change so I just created and we can discuss it in this MR.
This changes api a bit (adds options object instead of threshold) but it is fully backward compatible.

Perhaps in addition to this we could have a global flag that ensures all tests are allowed to fail... slightly bigger change though, but we can add it to the backlog. I am approving and merging this PR, thanks for your help!

@katilius I had to revert your changed as it broke the pipeline.. if you want to give it a go again, this time your PR will trigger the CI build and it will warn you about the failures.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactoring proposal for networking implementation

Issue/Task Number:  none
Overview
This Pull Request is a refactoring proposal for HTTPClient and its related classes such as Request, RequestBuilder and Endpoint
Changes
In this project, there are 6 classes to construct and send HTTPRequests. ClientConfiguration, HTTPClient, APIEndpoint,  Request, RequestBuilder, and RequestParameters. But some of the properties and tasks, which are necessary for http requests, are covered and implemented over multiple places.
For example there is a property for http header in Endpoint (which is responsible for specifying where to send requests), and session task for http request is created in Request class using a session defined in HTTPClient class.
Implementation Details
Tasks that each class is responsible for
※ I use protocol for interface definitions but it is just for examples in the PR.
ClientConfiguration
ClientConfiguration  stays the same.
Endpoint
Endpoint is only responsible only for specifying which endpoint to request and what type of task it is.
The interface looks something like this
enum Task {

    case requestPlain

    case requestParameters(Parametrable)
}

protocol Endpoint {

    var path: String { get }

    var task: Task { get }
}

This property and method in Endpoint will be moved to RequestBuilder.

var additionalHeaders: [String: String]?

func makeURL(withBaseURL baseURL: String) -> URL?

RequestBuilder & Request
ReqestBuilder is responsible only for building http requests and adding necessary information such as http headers and body parameters.  It takes ClientConfiguration as a parameter. All the logic in building HTTPRequest should be implemented here in this class to minimize the dependencies.
Request is responsible for building a request with RequestBuilder and handling response object mapping it to a model. (RequestBuilder and Request may be mergeable since Request class is just wrapping RequestBuilder)
The interface looks something like this
protocol RequestBuilder {

    func contentTypeHeader() -> String

    func acceptHeader() -> String

    func encodedAuthorizationHeader() throws -> String

    func addRequiredHeaders(toRequest request: inout URLRequest) throws

    func buildHTTPURLRequest(withEndpoint endpoint: APIEndpoint) throws -> URLRequest
}

By making this change and moving methods originally implemented in RequestParameters, RequestParameters class will not be necessary anymore.
HTTPClient
HTTPClient is responsible only for sending HTTPRequests. session.dataTask is called in Request class for current implementation, but I think it should be called in HTTPClient.
Interface looks something like this
protocol SessionTask: class {

    func resume()

    func cancel()
}

protocol HTTPClient {

    func request<ResultType>(toEndpoint endpoint: APIEndpoint, callback: Request<ResultType>.Callback?) -> SessionTask?
}

Overall
By applying those changes, each class will have their own tasks in order to execute requests, and also multi-covered tasks and properties which exist right now will be removed , which will make the codebase simpler and easier to read.
Impact
It should be adapted step by step because the changes will be a bit large and it may be hard to review.
Whether this proposal is accepted or not, It is still possible and better to remove RequestParameters and move its tasks to RequestBuilder.

Hey, thanks a lot for the proposal.
I thought about refactoring the networking logic too. As you said some class are responsible of different task and this should be refactored.
I'd love to see what you can end up with the changes that you proposed. I don't have the time right now to work on it but if you'd like to submit a PR implementing the proposed changes it'd be awesome.
Just a quick note: I think I split out RequestBuilder when I added the new SocketClient so you may need to make sure that we don't have conflict with this.

@mederic-p Thank you for the replay and the quick note. I will implement the changes and submit pull requests.

@yuzushioh Thanks for the proposal! In the future, please create an issue to discuss proposals :)

#46


Air Sinai A220 (Livery request)

no comment -

But it is real

ik, also someone added a srilankan a319 in multiliveries which is also blank, what's next? A blank A350?

ik, also someone added a srilankan a319 in multiliveries which is also blank, what's next? A blank A350?

in game, it is the basic livery

For A35K

I completely intended for that to be in the joke (totally did not make that sentence up after you said that 😉)

ik, also someone added a srilankan a319 in multiliveries which is also blank, what's next? A blank A350?

I think it was luke

Yeah


Enhancement: Export report data of teams report useractivityuserdetail command

@VelinGeorgiev
As we discussed here, I am planning to add an additional parameter to output the result in json file for the following command:
teams report useractivityuserdetail [options]
I will implement the same for the following commands once we successfully merge this change

teams report useractivityuserdetail
Get details about Microsoft Teams user activity by user.
Usage
teams report useractivityuserdetail [options]

Options

Option
Description

--help
output usage information

-p, --period <period>
The length of time over which the report is aggregated. Supported values D7, D30, D90, D180

-d, --date [date]
The date for which you would like to view the users who performed any activity. Supported date format is YYYY-MM-DD

-p, --path [path]
The path to save the report data to

--verbose
Runs command with verbose logging

--debug
Runs command with debug logging

Examples
Gets details about Microsoft Teams user activity by user for the last week
teams report useractivityuserdetail --period D7

Gets details about Microsoft Teams user activity by user for July 13, 2019
teams report useractivityuserdetail --date 2019-07-13

Gets details about Microsoft Teams user activity by user for July 13, 2019 and exports the report data in the specified path
teams report useractivityuserdetail --period D7 --path '/Users/joe/Desktop/report.json'

@sprider , I would suggest we make it the same format as the spfx upgrade commands. They have
-f, --outputFile [outputFile]`|Path to the file where the upgrade report should be stored in
`-o, --output [output]`|Output type. `json|text|csv`. Default `text`

You can have a look how Waldek did it in that pull request https://github.com/pnp/office365-cli/pull/1037
If we'd go with the json format first, we would have to convert the response body from CSV to json. I would propose the following json object structure to match what is in the csv coming back with the response where the keys are column name and the value is the column row value. The keys of the json objects can be created dynamically based on the column names in the csv response from the graph.
[
  {
    "Report Refresh Date":"<data>",
    "Web": "<data>",
    "Windows Phone": "<data>",
    "Android Phone": "<data>",
    "iOS": "<data>",
    "Mac": "<data>",
    "Windows": "<data>",
    "Report Date": "<data>",
    "Report Period": "<data>"
  },
  {
   ...
  }
];

@VelinGeorgiev Sure, I will review the same and start this weekend.

Thank you Joseph! I will assign it to you.

@VelinGeorgiev
Could you please put this on hold? I will pickup later or someone can work on this. Got up with few activities. Thank you

@sprider , Thanks for the heads up!

@VelinGeorgiev, Please assign this to me. I will work on this.


Java 8 Support

Do you guys think it's good if we create a Java 8 build?
@bodia , @tzclucian

Can't really come up with an example from the code right now, but keeping up with the latest Java versions sounds good to me.
It is something we should aim to achieve.
We could even push for a Java 9 build 😃

I'm building with JDK 1.8_144 without issue.

I'm also using JDK 1.8. Do we want to push for JDK 8 for now?

Yeah, creating java 8 build is a good idea.

https://github.com/authenteq/java-bigchaindb-driver/pull/34

This can now be closed.


Update Eudic extension to use applescript

Eudic supports AppleScript since Version 3.6.3 and it's way more effective to use AppleScript than Service.
With EuDic service, its window always freezes for a few seconds, which is no longer a problem using AppleScript.

@invariant Is it possible to make a alias for Option Values in Config.plist?
Users may be confused with bundle id, like com.eusoft.eudic:

No, but that is something I would like to add. "Display name" for options which map to underlying values. Thanks for suggestion.


Fix after-deploy hook not uploading

Add a missing return so deployS3() is run after sls deploy.

Thanks!


CNTRY codelist now has DENMARK in capital as the only country

The CNTRY codelist was modified in January to have the code for Denmark changed to capital (DENMARK). This now get included in output products and it looks strange that Denmark is the only country where this is the case.
I suggest the code to be changed back to Denmark again.

@Kadji, could you please let us know what was the reason for editing this code on the 2021-01-29? Reference Management Group did not receive any information on this edit.


Introduce Docker for handling cross-compilation & pre-build steps

We can introduce standardized way to interact with Docker for handling cross-compilation for compiled languages as well as performing any required pre-build task for a function (ex: Run webpack for a Node.js function written in ES2015 to prepare babelified and optimized code for final .zip)
Then it should be much easier to add ever growing list of runtimes:

Java (#4)
Rust (#19)
C (#72)
Swift (#86)
and more yet to come...

What would Docker be used for? -1 for any Docker stuff in core for me, you can't deploy containers to Lambda, but if you just mean for computing the builds I think that's unnecessary, in most cases it's pretty simple, Docker wouldn't help there I don't think.

@tj I came up with this Docker proposal while thinking about how to bring Swift support to Apex. Docker is completely out of the picture for runtimes we support today. However, once we get to the point where we support multiple compiled languages via shim, each of them having their own requirements, things can get complicated quickly. It can be far easier just telling users that setting up a Docker host somewhere is the only requirement and users being able to instantly deploy functions written in any programming language (while cross-compilations happen behind the scene via pre-built Docker containers) can be a truly magical experience. I know I'm overthinking this at this early stage though. Also, adding lots of languages may not be the goal of Apex.

Rather than adding support for docker(which might result in a somewhat large download, container + cross compilers + runtime), Maybe we can add support to spawn a new ec2 instance with Amazon Linux or point to an existing Amazon Linux instance for pre-deploy stuff. After all the code is only going to run on Amazon Lambda, which is running on Amazon Linux.

Different projects have a different requirements in terms of building and packaging, and supporting it in apex can add unnecessary complexity.

@pilwon maybe packer in combination with Atlas is better suited for this step?

Hmm all sounds like too much work to me haha. Most languages let you cross-compile with a few options , I just think we shouldn't support the ones that can't, it's not worth it. If people really want that they could devise their own technique and just add the output to the zip and use "shim": true when we have that

perhaps another way is a more generic runtime as plugin, ie. BYOB (binary ;-).

{
  "hooks": {
    "build": "make"
  }
}

should be enough to do whatever people want with ad-hock runtimes or builds now


Extra account "deleted" from the subscription

When about to delete an account from the subscription, two disappeared instead of one. Correct number after page refreshed.

Needs to be rechecked after this current release came (is it still relevant)

@PaTara43  re-check this please

Okay, so now it indeed does not remove another account.
But when adding one, it disappears after transaction complete

all good after page refresh

Do you do this on the domain robonomics.app or dev.robonomics.app?
This fix is currently only available on dev.robonomics.app

This is now work in dev branch. Will be added in the next release.

This is now work in dev branch. Will be added in the next release.

Tested again on commit 0e4836b and have same situation as in the beggining.
When delete account from "USERS IN SUBSCRIPTION", dissapeare one more account:
Begin: 4 accounts

Deleting account:

After delete: 2 accounts

And again reload page helps

Update problem on comment https://github.com/airalab/robonomics.app/commit/24120c0c9a5d51c67d620cfe2bf4def7a6ea0248
tested on https://dev.robonomics.app/#/rws-setup.
When delete NOT the last account from subscription list, disappear two account. Example:
I have this accounts list:

when I want to delete account in the middle of the list, we have got the next( i have 6 accounts):
deleting account:

after finish see next. One more account disappear(only 4 accounts):

Still, after refreshing the page, again i see 5 accounts:

This problem appears only when user want to delete account in the middle of the list!
@vol4tim please check it


Route ingest request

We should preconfigure a nginx ingress (keeping it disable by default is ok if we judge it makes sense) routing ingest requests
to indexers and routing search request to searchers.
Right now the extra hop is unnecessary

A default config for an AWS ALB would be nice as well.


The API of demo, is 404

For example , https://demo.getqor.com/api/products/1/color_variations/1.json is 404 page
How to solve it?

the document should be wrong, color variations will be included in /api/products/1.json, so there will be no nested color variations API like above URL....
Let me think how to write an example for your case.


Torrent 'ready' event not fired

Anyone can tell me why the 'ready' event isn't firing sometimes?

Are you attaching an event handler to the event right after the engine is created? If it has already fired, it won't fire again when you attach an event handler later. If you just need some code to run after 'ready', use engine.ready()

Yeah @bsuh, this was the problem. Thanks!


command-line interface

This pull request adds a command-line interface including usage information and instructions to install the command-line program from npm. With this patch the interface is now:
$ google-closure-compiler-js file1.js file2.js > minified.js

Or the program can also read from stdin, similar to other minifiers like uglifyjs:
$ browserify main.js | google-closure-compiler-js > minified.js

I have signed the contributor license agreement.

Google does not accept an ed25519 public key as a valid email address, so I can't add it.

@samthor Help please

LGTM with nits which you can fix later (or I will)-

code style isn't quite consistent (please use {} everywhere, arrow functions preferred, etc)
mixed feelings on flag camelCasing vs long_form - I'd like to support both, to match other versions of the compiler - but it's not finished quite correctly everywhere right now.

Thanks for the PR substack. Thanks for allowing these things to be worked on afterwards samthor. :clap:

@samthor Care to do a npm version bump?

I'll publish later on today.
On 15 January 2017 at 11:22, Cory Simmons<EMAIL_ADDRESS>wrote:

@samthor https://github.com/samthor Care to do a npm version bump?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/closure-compiler-js/pull/45#issuecomment-272663948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHRkG2FnrT2DfriwZpioEdx8BPJcoo6ks5rSWbbgaJpZM4LLUH2
.


Update for Godot 4.0.stable, improve demo scene

Tweak level for easier previewing of motion blur.
Tweak graphics settings in the demo for better quality.
Use physical key inputs so that WASD movement can be used on any keyboard layout.
Capture mouse input by default in the demo.

This closes https://github.com/davisbrandon02/motion-blur-godot-4.0/issues/2.

Preview
Epilepsy warning: Video contains flashing due to fast camera rotation.
https://user-images.githubusercontent.com/180032/226503817-38490e73-b87d-40e0-896d-cd5b365ac41a.mp4

Thanks for this, I'm using your version of the files.

Do you happen to know if it's possible to exclude something from the shader processing? I'm asking this cuz my game is third person and the shader makes the player mesh blur too much, I was wondering if I could prevent the player mesh from blurring.

Other than making a specific material use Depth Draw Never, no.

Does not work in stable version 4.3. What can you suggest?
Ran the demo scene. Quick mouse movement = nothing.

Does not work in stable version 4.3. What can you suggest? Ran the demo scene. Quick mouse movement = nothing.

This shader was likely broken by reverse Z in 4.3, since it still works in 4.2.2.
That said, in 4.3. I recommend using https://github.com/sphynx-owner/JFA_driven_motion_blur_addon, as it's much more capable.