Add parser support for ES6 module feature

Initial support for handling import and export statements in the Chakra
parser. It is gated behind a config flag (-ES6Module).
We are treating the import and export keywords like a tag for declarations
or identifier references which should be physically stored in a special
slot array which we need to build up at module link time (or, at least
before bytecode generation time). The array is not built up as part of
this change but we are collecting all the exports, imports, and referenced
modules such that building that array should be possible. Once we build
this array, accesses to identifiers which are created via an import
keyword will load from that slot array and accesses to identifiers
declared via an export keyword will load and store from the same slot
array.
Currently, we are not hooking up export declarations to their imported
usages. If you import a binding from another module and access it, we will
only guarantee that the binding exists. The value of the binding will
always be undefined.
We are keeping track of exports, imports, and referenced modules in the
top-level ParseNode of the parser (which is subsequently handed-off to the
ByteCodeGenerator). When parsing a module we set a flag (fscrIsModuleCode)
which also causes us to create the root ParseNode as a knopModule type
node. That is in turn used to allocate the ParseNode itself as a PnModule
node, which derives from PnProg but adds the lists for imports, exports,
and referenced modules. We are still treating the top-level ParseNode as a
knopProg (which is to say, we reset the nop to knopProg after allocating
it) because we handle the case of knopProg everywhere and we don't need to
complicate things by also needing to handle knopModule. Instead we just
use the PnModule extensions when we are in a module-specific parsing path
and otherwise treat the node as if it was a PnProg ParseNode anyway. Note
that import and export statements do not themselves have special ParseNode
types.
This change adds an experimental method to Jsrt (JsExperimentalRunModule)
and a ch.exe hook (WScript.LoadModuleFile) for loading a module from a
file. Loading a module via this hook acts similarly to including a module
in a script tag like

Hi @boingoing, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Taylor Woll). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

So great to see modules moving forward!! Can't wait :-D

Is this the beginning of a modules feature branch or do you want to merge this into master and continue that way?

We are planning to work on the modules feature in the master branch. Since we have to coordinate with browser folks, it will be better if we can take advantage of existing machinery to get our code from master to them as quickly as possible.

:clap:  So cool to see this coming!

Looks great!

Big change! Looks good.

Agreed! Looking real good!


adding extra container to tasks

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

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

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

Codecov Report

Merging #2847 (eb2baea) into main (f3524c3) will decrease coverage by 1.62%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2847      +/-   ##
==========================================
- Coverage   28.12%   26.51%   -1.62%
==========================================
  Files         302      125     -177
  Lines       35574    13436   -22138
==========================================
- Hits        10006     3562    -6444
+ Misses      25568     9874   -15694

Impacted Files
Coverage Δ

src/ApiService/ApiService/OneFuzzTypes/Enums.cs
32.39% <ø> (ø)

src/ApiService/ApiService/OneFuzzTypes/Model.cs
70.00% <0.00%> (-0.18%)
:arrow_down:

src/ApiService/ApiService/onefuzzlib/Config.cs
0.00% <0.00%> (ø)

src/ApiService/ApiService/onefuzzlib/Defs.cs
0.00% <0.00%> (ø)

src/ApiService/ApiService/onefuzzlib/Scheduler.cs
15.16% <0.00%> (-0.27%)
:arrow_down:

src/agent/onefuzz-agent/src/debug.rs

src/agent/onefuzz-agent/src/scheduler.rs

src/agent/onefuzz-agent/src/setup.rs

src/agent/onefuzz-agent/src/work.rs

src/agent/onefuzz-agent/src/worker.rs

... and 172 more

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


CalDev sync?

Is your feature request related to a problem? Please describe.
I'm currently self-hosting NextCloud and I'm using NextCloud tasks within my instance. I would love be able to have a desktop app to work together with this. Therefore I would love if the app in the future could support CalDev sync (that is what NextCloud tasks support), so I can sync my tasks between computers.

@danielsteiner I am very interested in this, do you have any project where we can follow your progress?

@danielsteiner Are there any updates on this other project? :) Because I'm in love with Nextcloud tasks, and I have found really good desktop clients for Ubuntu and my android phone, but I still have nothing on Windows. :(

That is out of scope. Ao is just a wrapper that wraps to-do.microsoft.com. On the other hand, I am working on a cross platform solution for Card-/CalDAV display.

@danielsteiner Any news on this project?
Have the same problem like @TobiasDev.

That is out of scope. Ao is just a wrapper that wraps to-do.microsoft.com. On the other hand, I am working on a cross platform solution for Card-/CalDAV display.

@danielsteiner Any news on this project?
Have the same problem like @TobiasDev.


chore: enable cgo and use google buildbase

cgo needs to be enabled to link against boringcrypto, so we add that here.
In addition, we use the google-go.pkg.dev/golang image as the Go buildbase to ensure build-time requirements, like boringcrypto, are enabled.
We also use gke.gcr.io/gke-distroless/libc as our runtime image.
We add the "cryp/tls/fipsonly" import to ensure boringcrypto is linking properly at build time. We guard this with a build tag "boring" in a dedicated file.

LGTM, but shouldn't we do the same for rest of our cmds?

Maybe? Though I don't think we need to rush as those are electively deployed by users and not part of the managed suite of binaries. IMO they can be done later.

There might be a bit different perf characteristics with CGO, but recent resources says it should be minimal. Let's watch that with a bit more care (this is the most relevant to collector perhaps).

Thanks for the link - and good to know! I agree the collector is the most risky. I did some /prombench-marking for the PRs there and they looked good for the most part. Agreed we should proceed cautiously to start.


Dismiss function

The dismiss function didnt; t work

@KoenvGinkel You solved?

I did. had to use .bind(this) in the function

onPress={this.submitURI.bind(this)}
together with:
submitURI(){     this.fadeAnimationDialog.dismiss();   }


Nested allignments

I love HVN's align feature. It certainly makes my life a little easier! That being said, I frequently align lines internally, too. E.G. the = below:
fib n | n ≡ 0        = 0
      | n ≡ 1        = 1
      | otherwise    = fib (n - 1) + fib (n - 2)

It doesn't look like there is a way to do that with HVN as of yet. Is this something that's doable?
Thanks!
P.S. GitHub is aligning things weird, so the space numbers above don't actually match up. The above does capture what I'm going for visually, though.

You could try visually selecting a region (with capital V) then run ,a= to align equal signs. Does this do what you'd like?

Did you figure this out? Should I close the issue?


Review the bash scripts

the bash syntax seems incorrect, variable definitions should not be prefixed with $, the line carriage return should use \ instead of `
The naming convention between scripts should be reviewed : e.g rgname vs rgName

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

ID: 076b7e0e-8ca2-53fa-51f1-f1027f1d2da8
Version Independent ID: 4e99d739-6a5d-7b61-54c5-e5f1395e36d2
Content: Ultra disks for Windows VMs - Azure Managed Disks
Content Source: articles/virtual-machines/windows/disks-enable-ultra-ssd.md
Service: virtual-machines-windows
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

These changes apparently didn't get merged to master yet.
Also, bash shell scripts should not have spaces between the variable, the equals sign, and the expression.  For example:
#spaces between subscription and equals sign and ", gets an error
me@Azure:~$ subscription = "eef21314-653e-4eec-893f-81681da67b86"
bash: subscription: command not found
#remove the spaces is successful
me@Azure:~$ subscription="eef21314-653e-4eec-893f-81681da67b86"
the last command is correct.


[Bug Report] after upgrade to 2.0.0 build is broken

In our project we had a PR created today for a version bump from 1.2.6 to 2.0.0. The build for the PR is broken and I am a out of ideas what could cause it.
See https://github.com/redhat-developer/vscode-didact/pull/420 for the PR created by dependabot.
The build fails with the following error:
><EMAIL_ADDRESS>compile /home/circleci/vscode-didact
> tsc -p ./

src/test/suite/didact.test.ts:30:28 - error TS7016: Could not find a declaration file for module 'async-wait-until'. '/home/circleci/vscode-didact/node_modules/async-wait-until/dist/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/async-wait-until` if it exists or add a new declaration (.d.ts) file containing `declare module 'async-wait-until';`

30 import waitUntil = require('async-wait-until');
                              ~~~~~~~~~~~~~~~~~~

Found 1 error.

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>compile: `tsc -p ./`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>compile script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>vscode:prepublish script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/circleci/.npm/_logs/2021-02-17T08_06_25_008Z-debug.log

Is that a problem with the 2.0.0 version?

@lhein Thanks for reporting!
Could you try this, please?
import { waitUntil } from ('async-wait-until');

@lhein the issue's fixed, please upgrade to 2.0.1 🙂
(turns out I didn't update the files in package.json before publishing 2.0.0)

Thanks for fixing the issue, @devlato.

@lhein No worries!


highlighting is incorrect for bash script mixing in python code using the << EOF syntax

Information

Language: [Git Bash, Python]
Plugins: ['none']

Description
This piece of bash script will not highlight correctly when mixing in python code using the << EOF syntax

The code being highlighted incorrectly.
cat > hello.py << EOF
print("hello world")
EOF

Looks fine to me. What is the problem?


The gnss_d300 driver based on dora

Hardware Name or Vendor
GNSS_D300
Description
Position positioning: By receiving satellite signals, GNSS_D300 can locate its own position (longitude, latitude and altitude).This driver is for the USB interface of GNSS hardware.
Completed the release of the NavSatFix, QuaternionStamped, and TwistStamped speed parts corresponding to ros2.
Installation
Serial port connected to computer USB port.
The default value in nmea_getSentence_driver_dora.py is changed according to the situation after connecting the serial port:
serial_port = '/dev/ttyUSB1'
serial_baud = 115200
Development Guidelines
sudo chmod 777 /dev/ttyUSB*
RUST_LOG=debug dora start gnssD300_nmea_driver_dora.yaml  --attach --hot-reload
Output
Completed the release of the NavSatFix, QuaternionStamped, and TwistStamped speed parts corresponding to ros2.

[x] Create a new folder for each new hardware under the right categorisation.
[x] Add this README template into the folder as well as the source code.

@haixuanTao Excuse me sir, can you please review this submission and give me some advice，thank you!


Remove registry redirect

... it was causing an infinite redirect.
Signed-off-by: Ben Firshman<EMAIL_ADDRESS>(cherry picked from commit 4c6d5e3a25557b2a053e8f8e5aafc84f5f6aada8)

LGTM (if builds go green)

@jfrazelle I have no ability to rekick builds --- even so there seems to be an across the board issue with that one hello-world test.  (I'm more concerned about seeing the consistent test failure on multiple prs than this particular pr)  What say you?

Seems fine
On Friday, April 17, 2015, moxiegirl<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle I have no ability to rekick
builds --- even so there seems to be an across the board issue with that
one hello-world test. (I'm more concerned about seeing the consistent test
failure on multiple prs than this particular pr) What say you?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/12463#issuecomment-93978876.

Wah but I want my builds to be green before I merge!

I think there's a pr to fix that test
On Friday, April 17, 2015, moxiegirl<EMAIL_ADDRESS>wrote:

Wah but I want my builds to be green before I merge!
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/12463#issuecomment-93979800.

The PR was merged, but you'll have to rebase to get it. Forgot about that when I triggered a rebuild this morning :smile:

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fix-redirect-bfirsh" git@github.com:moxiegirl/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~868
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

nooooooooooo!

:'(


Revert "Reland engine display features"

Reverts flutter/flutter#89511
A few tests red on CI
https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_android new_gallery__transition_perf/2996/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android flutter_engine_group_performance/896/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__crane_perf/885/overview
/cc @andreidiaconu

Looks like the gallery repo also needs to update lockfiles, as those tests use the repo for testing. From what I can tell, these tests are not part of the checks that show up on GH https://github.com/flutter/flutter/pull/89511/checks
Am I missing something? I will open a separate PR on the gallery repo to update lockfiles and trey to reland everything again

Opened https://github.com/flutter/gallery/pull/557 which should make the gallery compatible.


"Investors" and "Report a bug" banner disappears after going to a room and returning to the home screen

Browser: Brave 1.20.103 (Chromium: 88.0.4324.152)
The title, when you go to a room and return the bottom banner ceases to exist. Might be a bug or a feature, not sure.

expected behavior, didn't want the bottom to get too stacked


Instance populates bootstrap_expect with 404

i've tried building and rebuilding this module several times, but keep hitting this wall - the consul agent is trying to come up as a server and it is also populating its config file with the html of a 404 error page..
`sudo cat /opt/consul/config/default.json
{
"advertise_addr": "<IP_ADDRESS>",
"bind_addr": "<IP_ADDRESS>",
"bootstrap_expect":

Error 404 (Not Found)!!1

404. That’s an error.
The requested URL /computeMetadata/v1/instance/attributes/cluster-size was not found on this server.  That’s all we know.,
  "client_addr": "<IP_ADDRESS>",
  "datacenter": "europe-west1",
  "node_name": "vault-cluster-99h3",
  "retry_join": ["provider=gce project_name=XXXX tag_value=vault"],
  "server": true,
  "ui": true,
"raft_protocol": 3
}`

fixed due to mismatch between meta data and config


Output for actions that update libman.json need to specify the file that was updated.

Print the full file path of the libman.json file that was updated.

I'm not sure what the context was here originally (IDE vs. CLI? What actions?  Which file(s) are being updated?)


Travis xcode 9

Subtle, and very annoying, bug in allSubclassesOf.
If the initial call to objc_getClassList returned fewer classes than were returned from the second call then the buffer was filled to the correct limit (bug 1 - some classes might get randomly missed out) but the loop went from 0 to the larger amount (bug 2 - buffer overrun).
Fixed by allocating the buffer to twice the expected amount. Not that fussed by the inefficiency here, it's deallocated almost immediately afterwards, and isn't going to be a vast allocation anyway.
I've also updated the travis file to work with the Xcode 9 image, with a single job still running xcode 8 until we decide to drop support for it.

Perhaps a comment in the source code linking to this PR would be useful for the future

@kerrmarin Hey - checkout commit f890268 :)

@kerrmarin I also removed the failed experiment which was buddybuild. Just plain didn't build it. Every time I hear '5 minutes to set up, will automatically detect your project settings' I shudder.


.Net Core 2.1 P1 - Can't website from fresh Angular web template: '@angular-devkit/core'

Issue Title
Can't website from fresh Angular web template: Error with @angular-devkit/core
General
Trying to start the website after creating a fresh project using the Angular web template nets me Error: Cannot find module '@angular-devkit/core'.
Adding @angular-devkit/core as a devDependency in package.json seems to fix the issue.

@Eilon @muratg @DamianEdwards

This issue was moved to aspnet/JavaScriptServices#1578


Added support for the labmode comms

This small change allows HoloViews comms to support labmode. To be clear about the responsibilities of labmode:

Labmode will not be defining Jupyter on the root so this is a pure fallback (and if Jupyter ever does get stubbed, it will have to support enough of the same API that this PR can be reverted).
Labmode is responsible for mirroring the Jupyter comms API.

I just want to make it clear that I don't want to add any more labmode specific code to HoloViews. This PR is simple enough that the only point I see for discussion is whether the check should be for window.comm_manager or maybe something else labmode can set...

Seems  reasonable to me.

I think this is now ready to review/merge.

Looks good.


Add Non-.gov URL API

Add http://search.digitalgov.gov/developer/govt-urls.html to http://open.gsa.gov/developer.
A potential slug is, "This API allows you to find government URLs that don't end in .gov or .mil. URLs are searchable by keyword, state, or scope."

Thanks, we are working on a site redesign so I'm closing the issue here and moving to that repo.
https://github.com/GSA/open-gsa-redesign/issues/35

Thanks for tracking these as issues, @rdgsa.  It's a big help!
A question - what's the current ETA on this redesign.  Given how easily we can edit the legacy site (I'm happy to file a PR), if it's going to be more than a month or so, could I recommend we still add this here?
This API is relevant to some recent policy that rolled out and is also just good stuff and worth getting out there anyway.


Update gungames.xml

[Saturn] removing Corpse Killer. It has no light gun support.

I will add / delete games as I correct them. There was some mistakes. Once ready to merge, I'll change the status.


[PE-D][Tester D] Addf command does not allow me to submit a recipe

I tried the following input fields for the addf command. After I am done, I press Save.

However, I got the following error:

I am unsure of why the recipe cannot be added since I entered most of the values as suggested in the placeholder text of the input boxes.

Labels: severity.High type.FunctionalityBug
original: eugenetangkj/ped#6

error messages fixed in #245


Stop node loop from using sstream << int on Windows

This works around an issue with node 7.7.4 on Windows.

LGTM


[GH-580] Fake Item and Traps

Motivation
Add the new fake item to the game and also add the new traps feature.
Closes #580
After merging this, lets merge https://github.com/lambdaclass/curse_of_mirra/pull/1772 into curse of mirra's front
Summary of changes

Implemented items mechanics
Added new item Fake Item
Added Arena.Game.Item and Arena.Game.Trap modules
Added Trap and TrapStatus message to messages.proto

Demo
https://github.com/lambdaclass/mirra_backend/assets/67444519/7dbfeded-5122-44c1-b423-a3c1b89348ae
How to test it?
You'll see that now an item could've this fields. Traps are entities too, so they pretty much will have the same parameters as any other entity. In the code you'll which fields are part of the additional info
  {
    "name": "fake_item",
    "radius": 200.0,
    "effects": [],
    "mechanics": {
      "spawn_bomb": {
        "name": "bomb",
        "radius": 200.0,
        "activation_delay_ms": 3000,
        "vertices": [],
        "mechanics": {
          "circle_hit": {
            "damage": 64,
            "range": 380.0,
            "offset": 400
          }
        }
      }
    }

They're pretty much self explanatory, but maybe we should add a field that addresses animation time.

Get a Fake Item
Use it nearby an enemy and it will leave a bomb
Wait until explodes
Boom!

Checklist

[ ] Tested the changes locally.
[ ] Reviewed the changes on GitHub, line by line.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

The feature is working as expected! 🪨

After talking with @manucamejo we should change the behavior from a bomb to a landmine

Addressed !

Maybe we should add a time of preparation or something like that because i think when you use the fake item the explosion is being triggered instantaneously

Done

Can we add some PR to test this? Either work on the client app or make a PR in unity repo. It's really hard to test its behavior when I cannot see the dropped item.

They should be visible now!


fix rmm managed memory resource initialization to resolve some intermittent …

…memory issues
Looks like rmm.reinitialize destroys memory resources assigned to all other devices besides the current device.  This way of initializing as in this PR and doing only once per process avoids this.  I think the intermittent cuda errors were due to hit or miss uses of destroyed resources on the c++ api rmm operations.   So this should address the issue.
Also, in umap, current device was set before the memory resource was initialized.  Needs to be the other way around.
also pin numpy < 1 in readme

build

build


Support higher resolution images

Right now, the generated pictures do have low resolution, even with high resolution input. Would be nice, if the output would be idk, 800x800px or something like that :)
And btw, thanks, it's useful for trolling crypto folks. :3

I'll work on this later today! Thanks for the feedback :3

Any update?


Added color to labels

Default Label's TextColor on Windows is almost invisible on the white background, unlike on Android, where the default TextColor for Label is Gray, so I added a TextColor property to the labels, with the value of the android label's default color, which is Gray.

 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: yairp03 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Also, fixed a bug in part 4 solution where there was a wrong closing tag on the MainPage.xaml view.

Make sense to me :) i use styles for all the other sections so that is probably why.


Is it possible to use $ref with x-go-type

We have two swaggers, one for data model and second for api implementation, now we are pointing objects from model in the following way:
Some:
    type: object
    properties:
      fake: # <- this tells the codegen that this is a real object. The property mentioned here is not used.
        type: integer
    x-go-type:
      import:
        package: some/some/swagger/gen/some
      type: Some
    x-nullable: true

Problem is when we are trying to generate JS client based on this swagger, js swagger not familiar with x-go-type tags, but when we are switching to $ref tags, server/client implementation will generate all data model object under new package
Example of change:
Some:
   $ref: `../swagger/swagger.yaml#/Some`

Is there way to use $ref with x-go-type:, or mark generated models as belongs to correct golang package
Environment
swagger version: 0.25
go version: 1.16
OS:   Mac

You can generate models and server or client separately.
When generating the server, you can skip models generation using the flag  "--existing-models=github.com/foobar/model", indicating models should be found in that package.
So you can stick with the $ref


Properties in product detail page are sorted randomly

PHP Version: 8.2.9   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
Properties are sorted correctly
Actual behaviour:
Properties are sorted randomly.
How to reproduce:
The customer has a problem here that the properties (Products > “Product XY” > Specifications > Properties) are sorted randomly.
For example, if you type “blue” in the search bar, “color / blue” appears on the last page.
In addition, every time you click in and out of the search, the search results change.

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-38535
[created from NEXT-38535, comment 499094]


Use Toolbar for service account from app-services-ui-components

@jgiardino told to keep this on hold until the issue related to Pagination for list service account api end-point is resolved

Rebase needed? or can be closed


Fix missing question option translations

These changes should've been made in #2005. They should fix the broken regression test build.

Looks good to me.


Fix random_string resource numeric input deprecation

Signed-off-by: Arthur Diniz<EMAIL_ADDRESS>Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
This module uses a resource called random_string. This resource has an input field that is deprecated and it's a good idea to update to the new format.
number -> numeric
No breaking changes.
Types of changes
What types of changes does your code introduce to <repo_name>?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING.md doc.
[x] I have added the necessary documentation (if appropriate).
[x] Any dependent changes have been merged and published in downstream modules.

Further comments
Before
╷
│ Warning: Attribute Deprecated
│
│   with module.securityhub.random_string.to_slack_cf_suffix,
│   on .terraform/modules/securityhub/event-to-slack-cf.tf line 7, in resource "random_string" "to_slack_cf_suffix":
│    7:   number  = false
│
│ **NOTE**: This is deprecated, use `numeric` instead.

After
Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # module.securityhub.aws_securityhub_account.default will be created
  + resource "aws_securityhub_account" "default" {
      + id = (known after apply)
    }

  # module.securityhub.aws_securityhub_standards_subscription.cis[0] will be created
  + resource "aws_securityhub_standards_subscription" "cis" {
      + id            = (known after apply)
      + standards_arn = "arn:aws:securityhub:::ruleset/cis-aws-foundations-benchmark/v/1.2.0"
    }

  # module.securityhub.aws_securityhub_standards_subscription.foundational[0] will be created
  + resource "aws_securityhub_standards_subscription" "foundational" {
      + id            = (known after apply)
      + standards_arn = "arn:aws:securityhub:sa-east-1::standards/aws-foundational-security-best-practices/v/1.0.0"
    }

Plan: 3 to add, 0 to change, 0 to destroy.

Hey @arthurbdiniz could you consider including the minimum AWS provider version that supports the new feature?

Hey @brunodasilvalenga, this was a change on the random provider and not AWS terraform provider.  As a warning deprecation, the old attribute will only be removed on the next MAJOR release of the provider following Provider Attribute Rename Docs.

Attribute Change Pull Request
terraform-provider-random release v3.3.0

So in my opinion the best approach is to define the version to >= v3.3.0.
required_providers {
    random = ">= 3.3.0"
}

If you have other approaches for this requirement I will be around to discuss and make the necessary changes.


add configuration option for flags

As I have two accounts in my 1Password vault, I couldn't receive any secrets.
So I added a “flags” config to configure 1Password global flags.

Thanks Kasi07! I was just going to PR the change myself and I'm happy to have checked the pull-requests first :D
Thanks @benvp for the quick review, would really love to have this change released!

Sorry for the delay. Didn't have any time yet. Just released 0.2.0. And thanks for your contribs!


Microsoft

[Enter feedback here]

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

ID: 7dc7b3e7-67d0-5dc2-3141-da171f825dda
Version Independent ID: a768939c-9297-1394-ec2f-ec90b478c1a6
Content: Azure permissions for General - Azure RBAC
Content Source: articles/role-based-access-control/permissions/general.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@Top-Chief
It would be great if you could add more about query to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!


updated the puppet scripts to remove the puppet deprecation warning

On Ubuntu 14.04 LTS I was getting these warnings:
"Variable access via 'lsbdistcodename' is deprecated. Use '@lsbdistcodename' instead."
"Variable access via 'domain' is deprecated. Use '@domain' instead."
"Variable access via 'fqdn' is deprecated. Use '@fqdn' instead."

This has been merged into master...thanks for the contribution!


exemplifying how to use Popper with data.world

This pull request solves https://github.com/systemslab/popper/issues/290

Hello @Franko1307! Thanks for updating the PR.

In the file pipelines/data-world/scripts/generate-data.py, following are the PEP8 issues :

Line 6:32: W291 trailing whitespace
Line 11:67: E231 missing whitespace after ','
Line 11:80: E501 line too long (112 > 79 characters)
Line 13:57: E231 missing whitespace after ','
Line 14:51: E231 missing whitespace after ','
Line 14:80: E501 line too long (88 > 79 characters)

In the file pipelines/data-world/scripts/update-dataworld-repo.py, following are the PEP8 issues :

Line 9:80: E501 line too long (119 > 79 characters)


Grad accum

Added Gradient Accumulation

Currently the PR conflicts with dev branch. Please update the branch and commit again :+1:


feat: Add groups and federated_id claims to the scope

federated_id is a dex specific that passes down the username and
connector used which is much easier to work with than the sub that dex
uses.
Whilst I was here add Groups because it may be useful to someone

/assign @ccojocar

/lgtm
/approve
/test this


Adding pom dependencies

Adding dependencies related to security api, jib and sonar.
JIRA Ticket Number
Type of change

[ ] Bug fix
[ x] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!


Bug: Components that use modelValue props throw TS overload errors

Describe the bug

When testing a component in TS that uses a v-model as defined with a modelValue prop and a update:modelValue emit, TS will throw an error when mounting the component with a ref for the modelValue.
To Reproduce

Component:
<template>
  <input
    type="text"
    data-test-id="input"
    :value="modelValue"
    @input="update"
  >
  {{ modelValue }}
</template>

Test:
import { describe, test, expect } from 'vitest'
import TST from '../tst.vue'
import { nextTick, ref } from 'vue'
import { mount } from '@vue/test-utils'

describe('Input', () => {
  test.only('thing', async () => {
    const text = ref('')
    const wrapper = mount(TST, { //TS will throw a "No overload matches this call."
      props: {
        modelValue: text, // Using text.value makes TS happy, but obviously you lose reactivity so you can't test.
      },
    })

    expect(wrapper.find('[data-test-id="tst"]').text()).toBe('')

    text.value = 'foo'
    await nextTick()

    expect(wrapper.find('[data-test-id="tst"]').text()).toBe('foo')
  })
})

Expected behavior

TS to accept a ref to be passed in as the modelValue.
Related information:

@vue/test-utils version: 2.0.0-rc.20
Vue version: 3.2.20
node version: 16.14.0
npm (or yarn) version: 8.3.1

Additional context

Tests run fine and builds work too, so it's really just an issue inside the IDE.

Hi @TheDutchCoder
Can you try to update VTU to v2.0.0 and Vue to v3.2.34 to make sure that this is not fixed in more recent releases?

You should use setProps instead of a ref as prop. Because the modelValue should be of type string and not of Ref<string>. Try this solution:
import { describe, test, expect } from 'vitest'
import TST from '../tst.vue'
import { mount } from '@vue/test-utils'

describe('Input', () => {
  test.only('thing', async () => {
    const wrapper = mount(TST, {
      props: {
        modelValue: ''
      },
    })

    expect(wrapper.find('[data-test-id="tst"]').text()).toBe('')

    await wrapper.setProps({
      modelValue: 'foo'
    })

    expect(wrapper.find('[data-test-id="tst"]').text()).toBe('foo')
  })
})

https://test-utils.vuejs.org/guide/essentials/passing-data.html#using-setprops

Hi @TheDutchCoder
Can you try to update VTU to v2.0.0 and Vue to v3.2.34 to make sure that this is not fixed in more recent releases?

Updating both seemed to have worked, thanks!
2.0.0 just crashes in VSCode when running tests (opened a bug for that), but it works in iTerm, so thanks!


fix: remove reactor formulas beta disclaimer

Description
Removes Reactor Formulas beta disclaimers

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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


Rec page and RL linking issues

Connected to https://github.com/cortezaproject/corteza/issues/1758
2 issues

if a1 module and a2 are linked via field => a1 rec page is created and a RL pointing to a2 is added => a2 rec page is created => rec page of a1 becomes the parent of rec page 2

if RL page is created for a1, then the hierrarcy is: RL a1 is the grandparent of a1 rec page, a1 rec page is the parent of a2 rec page. This makes sense. If the first issue is fixed, then this shouldn't be a problem

update: you don't have to select a reference field. You just:

create module a1 with string field
create module a2 with string field and rec field (don't add any configs)
add rec page for a1 with RL pointing to a2
add rec page for a2 and a1 rec page is the parent of a2 rec page

another update: this is present on 2022.3.5

Notes added to the original ticket https://github.com/cortezaproject/corteza/issues/1758


[SPARK-26560][SQL][2.4] Spark should be able to run Hive UDF using jar regardless of current thread context classloader

What changes were proposed in this pull request?
This patch is based on #23921 but revised to be simpler, as well as adds UT to test the behavior.
(This patch contains the commit from #23921 to retain credit.)
Spark loads new JARs for ADD JAR and CREATE FUNCTION ... USING JAR into jar classloader in shared state, and changes current thread's context classloader to jar classloader as many parts of remaining codes rely on current thread's context classloader.
This would work if the further queries will run in same thread and there's no change on context classloader for the thread, but once the context classloader of current thread is switched back by various reason, Spark fails to create instance of class for the function.
This bug mostly affects spark-shell, as spark-shell will roll back current thread's context classloader at every prompt. But it may also affects the case of job-server, where the queries may be running in multiple threads.
This patch fixes the issue via switching the context classloader to the classloader which loads the class. Hopefully FunctionBuilder created by makeFunctionBuilder has the information of Class as a part of closure, hence the Class itself can be provided regardless of current thread's context classloader.
Why are the changes needed?
Without this patch, end users cannot execute Hive UDF using JAR twice in spark-shell.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
New UT.

cc. @cloud-fan This is the backport PR for #27025

Test build #116022 has started for PR 27075 at commit 6470cdc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20814/
Test PASSed.

Test build #116022 has finished for PR 27075 at commit 6470cdc.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116022/
Test FAILed.

retest this, please

Test build #116028 has started for PR 27075 at commit 6470cdc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20820/
Test PASSed.

Test build #116028 has finished for PR 27075 at commit 6470cdc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116028/
Test PASSed.

thanks, merging to master!

Thank you, @HeartSaVioR and @cloud-fan !


Alignment of filters

Background:
https://trello.com/c/QedVprNS/463-minor-alignment-of-filters
Solution:

Adjusted media query values

 minor (alignment of filters)


Add leader address environment variable injection

This adds LWS_LEADER_ADDRESS environment variable injection constructed via related LWS pod labels. This avoids making the user specify in yaml via downward api.

What type of PR is this?
/kind feature

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #143
Special notes for your reviewer
Does this PR introduce a user-facing change?

Add LWS_LEADER_ADDRESS environment variable to all containers in a leaderworkerset.

/ok-to-test

/lgtm
/approve
/label tide/merge-method-squash


feat: add "autoCloseDocCommentDoSugesst" feature

Description
"intelephense" will suggest the completion item /** [Intelephense PHPDoc] */ in the case of /**| */.
In vscode-intelephense, */ is automatically inserted in the "Language Configuration" function of the VSCode itself. https://code.visualstudio.com/api/language-extensions/language-configuration-guide
This is not a problem with vscode-intelephense, but it is a problem when used from Vim. Completion items are also suggested in the following cases, but there is a glitch.
Before:

| is the cursor.

    /**|
    public function dummy1()
    {
      // ...snip
    }

    /**
     *
     */
    public function dummy2()
    {
      // ...snip
    }

After:
This is a bug that causes the dummy2 method to disappear... :(
    /**
     *
     * @return void
     */
    public function dummy1()
    {
        //
    }

Add feature
Added the ability to behave as follows.

Auto insert */ by typing /**.
Text will be /**| */. (| is the cursor)
Automatically triggers completion at the current cursor position.
/** [Intelephense PHPDoc] */ complemention items are suggested.

If this feature is not needed, it can be disabled. Set intelephense.client.autoCloseDocCommentDoSugesst to false.

DEMO (mp4)
https://user-images.githubusercontent.com/188642/161670727-b7a3f6d7-4833-4279-a21d-611992a40760.mp4


Signing a tx

Hi there,
I am trying to write an example for signing transactions.
Is it possible to sign a tx with multiple TransactionInputs using this lib?
It seems like the SignatureContext object only accepts one input.
Cheers.

well, sorry for interrupting, i figured it out.
Cheers.


Use private AWS NTP service instead of outbound global

Is your request related to a problem? Please describe.
Our network security folks only allow outbound <IP_ADDRESS>/0 access on 80 and 443.  NTP to <IP_ADDRESS>/0 violates that rule.
Describe the solution you'd like.
Looks like AWS has a private NTP server.
https://aws.amazon.com/blogs/aws/keeping-time-with-amazon-time-sync-service/
Can we use that, please?

thanks for pointing this out - we can't change the default at this time since that would be breaking, I have made a note for the next breaking release to make it the default. For now, I can add a variable to make this configurable

This issue has been resolved in version 18.26.0 :tada:


Kythe doesn't build with development build of Bazel

Hello dear Kythe,
I see Kythe implements a Bazel version check, but this doesn't work with the development build of Bazel which we use on our downstream pipeline (https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/989#a722f124-2b90-4559-a939-7c6dd6d921dd). The build is green with the released version of Bazel, it's the only empty version string of development Bazel build that causes trouble.
Thanks!

Thanks! This is a known issue (I commented on it in some of the earlier bugs about adding us to the Bazel CI).  I'll try and tackle it today.

Oh that would be great, thanks! I'll then keep Kythe enabled on downstream. Pls let me know if you couldn't get to it, and I'll temporarily disable Kythe then (we need to make sure downstream is green or red only because of our regressions because otherwise devs quickly learn to ignore the pipeline :).
Thanks!

@hlopko Let me know if #3760 is not a viable approach.  The other options seem to require making check_version either much more complicated (as a repository_rule) or omitting it altogether (which doesn't lead to particularly user-friendly errors on incompatible changes).

Thanks Shahms! This will do just fine.


Unable to get provider com.google.android.gms.ads.MobileAdsInitProvider

For some reason I get this error, the only thing i recall doing is test the ios version of the app for the first time so i installed the cocoa pods and run it in ios and it worked, then i try to run it in android again and the app does not even start, this is the complete error log.
tns platform remove/add does not seem to work either.
`FATAL EXCEPTION: main
java.lang.RuntimeException: Unable to get provider com.google.android.gms.ads.MobileAdsInitProvider: java.lang.IllegalStateException
                                                                          * The Google Mobile Ads SDK was initialized incorrectly. AdMob publishers    *
                                                                          * should follow the instructions here: https://goo.gl/fQ2neu to add a valid  *
                                                                          * App ID inside the AndroidManifest. Google Ad Manager publishers should     *
                                                                          * follow instructions here: https://goo.gl/h17b6x.                           *

                                                                              at android.app.ActivityThread.installProvider(ActivityThread.java:5177)
                                                                              at android.app.ActivityThread.installContentProviders(ActivityThread.java:4769)
                                                                              at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4709)
                                                                              at android.app.ActivityThread.-wrap1(ActivityThread.java)
                                                                              at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1415)
                                                                              at android.os.Handler.dispatchMessage(Handler.java:102)
                                                                              at android.os.Looper.loop(Looper.java:148)
                                                                              at android.app.ActivityThread.main(ActivityThread.java:5443)
                                                                              at java.lang.reflect.Method.invoke(Native Method)
                                                                              at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:728)
                                                                              at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:618)
                                                                           Caused by: java.lang.IllegalStateException: `

@javierpal I've got the same errors too. I have followed the instruction stated in https://goo.gl/fQ2neu
by adding

But unfortunately, nothing has changed!
here you can get more info about the errors, https://ads-developers.googleblog.com/2018/10/announcing-v1700-of-android-google.html
has mentioned about the change of admob sdk.

@shilik it's weird because i did it too and it worked fine later, one more thing the only line you need to add is:
<meta-data             android:name="com.google.android.gms.ads.APPLICATION_ID"             android:value="[ADMOB_APP_ID]"/>
inside manifest/application check if you have it right.

@javierpal
Yes, it is quite wierd.
I have tried  either
<meta-data android:name="com.google.android.gms.ads.APPLICATION_ID" android:value="[ADMOB_APP_ID]"/>
or
<meta-data android:name="com.google.android.gms.ads.APPLICATION_ID" android:value="ADMOB_APP_ID"/>
then did the following
npm run clean
adb logcat -d >log.txt
npm run watch:android
but
my app kept stopped and crached. :(

@javierpal
I have tried various combinations.
when the code snippets are added,  The "The Google Mobile Ads SDK was initialized incorrectly. "
error disappeared. but my app crashed.
finding:
As long as nativescript-admob is installed (whatever version),
my app crashed and stopped after re-compilation.

I am getting following error while running on android device

11-22 14:20:45.942  5293  5293 E AndroidRuntime: * The Google Mobile Ads SDK was initialized incorrectly. AdMob publishers    *
11-22 14:20:45.942  5293  5293 E AndroidRuntime: * should follow the instructions here: https://goo.gl/fQ2neu to add a valid  *
11-22 14:20:45.942  5293  5293 E AndroidRuntime: * App ID inside the AndroidManifest. Google Ad Manager publishers should     *
11-22 14:20:45.942  5293  5293 E AndroidRuntime: * follow instructions here: https://goo.gl/h17b6x.                           *
11-22 14:20:45.942  5293  5293 E AndroidRuntime: ******************************************************************************
I am not using any google AdMob plugin.
I am using following plugins
cordova-facebook-audnet-sdk 4.26.2 "Facebook AudienceNetwork SDK for Cordova"
cordova-plugin-device 2.0.2 "Device"
cordova-plugin-extension 1.5.4 "Cordova Plugin Extension"
cordova-plugin-facebookads 4.23.2 "FacebookAds"
cordova-plugin-ionic-keyboard 2.1.3 "cordova-plugin-ionic-keyboard"
cordova-plugin-ionic-webview 2.2.3 "cordova-plugin-ionic-webview"
cordova-plugin-splashscreen 5.0.2 "Splashscreen"
cordova-plugin-statusbar 2.4.2 "StatusBar"
cordova-plugin-whitelist 1.3.3 "Whitelist"

"... AT THE END OF THE MANIFEST/APPLICATION ", after two days of trial i finally get rid of the error.
grate answer thank you!!

@bereket-kassahun Just so I can make sure the instructions in this repo are still correcht, is your solution any different than adding this and the instruction here? If so, how can we make that more clear?

though i put this code on my manifest file but still my app is crushed

though i put this code on my manifest file but still my app is crushed
manifest:

"... AT THE END OF THE MANIFEST/APPLICATION ", after two days of trial i finally get rid of the error.
grate answer thank you!!

i had this problem to...
"... AT THE END OF THE MANIFEST/APPLICATION "


 It's important that new or existing end-to-end tests are run in CI as a required check for PRs to be merged to ensure that bugs aren't introduced.
Changes:

Run all jobs under integration.yaml on ubuntu-latest
Create a new e2e-test job to replace the existing integration-test job that spins up a Kubernetes cluster using Kind, builds and loads the image to the cluster, then runs the existing four e2e tests
Remove local Kind commands from the Makefile. The existing kind-create-cluster target was broken anyway and developers may wish to use other distributions e.g. Minikube or k3d

Closes https://github.com/kubeflow/spark-operator/issues/1416
Prerequisite work before adding an end to end test for the Yunikorn batch scheduler integration for https://github.com/kubeflow/spark-operator/issues/2098
Change Category
Indicate the type of change by marking the applicable boxes:

[X] Bugfix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that could affect existing functionality)
[ ] Documentation update

Checklist
Before submitting your PR, please review the following:

[X] I have conducted a self-review of my own code.
[ ] I have updated documentation accordingly.
[ ] I have added tests that prove my changes are effective or that my feature works.
[X] Existing unit tests pass locally with my changes.

/assign @ChenYi015 @vara-bonthu

Remove local Kind commands from the Makefile. The existing kind-create-cluster target was broken anyway and developers may wish to use other distributions e.g. Minikube or k3d

@jacobsalway  I forgot to push the charts/spark-operator-chart/kind-config.yaml file, which breaks the kind-create-cluster target in makefile. You can make a kind config file as follows and it can be used in e2e CI and local tests:
kind: Cluster
apiVersion: kind.x-k8s.io/v1alpha4
nodes:
- role: control-plane
  image: kindest/node:v1.29.2
- role: worker
  image: kindest/node:v1.29.2

/lgtm


Harden the HTTPS authority host check to explicitly check for https and disallow everything else

The Java check needs to be hardened. Unlike the other language SDKs, which check that the scheme string is exactly HTTPS and fails otherwise, the Java check only confirms that it is "not HTTP" (and assumes anything else is "HTTPS"). That, along with the fact that the setter doesn't do any checking, there could be a way for a malformed authority host url to get through.
Comparing to other languages, this check: https://github.com/Azure/azure-sdk-for-java/blob/0fbfd34590988f9bb78f9898e875366d17b5641d/sdk/core/azure-core/src/main/java/com/azure/core/http/policy/BearerTokenAuthenticationPolicy.java#L93-L95
Should be something like the following (unless we have other non-HTTP supported schemes beyond HTTPS in Java): if (!context.getHttpRequest().getUrl().getProtocol().equals("https")
Introduced in this PR (the intent of which I gather is to only allow HTTPS): https://github.com/Azure/azure-sdk-for-java/pull/6543
cc @joshfree, @billwert

Issue #37236 - Hardening the HTTPS Authority Host Check
Hello,
I wanted to express my support for the proposed change to harden the HTTPS authority host check in the Java SDK for Azure. It's crucial to ensure secure communication, and explicitly checking for "https" in the URL protocol.

Thanks for filing this @ahsonkhan!
@Vijay-948, if you want to create a PR with that change, we'd be more than happy to review it.

Sure I make a PR

I'm sorry to hear that you're facing difficulties with creating a pull request (PR) due to file path length issues. It seems like this problem is occurring at the file system level, and it's not something that can be directly resolved through a PR.

I'm sorry to hear that I'm facing difficulties with creating a pull request (PR) due to file path length issues. It seems like this problem is occurring at the file system level(file name is too long).


In server view, search tables

Issue Type: Feature Request
"Search Server Names" isn't crazy, but for people like me, much less helpful than "search tables". I only have 10 servers, but each server has a dozen databases and each database has a dozens of tables.
Searching the expanded tables (or even just a view filter, like VSCode's Explorer filter) would be a huge improvement.
Azure Data Studio version: azuredatastudio 1.5.2 (f74080c96310354bc92211f1826c0fdce78ca2f6, 2019-03-22T06:25:26.627Z)
OS version: Windows_NT x64 10.0.16299

@tigerhawkvok have you tried the Redgate Search extension?  It provides this functionality, but it is not integrated with the Servers pane.  In any case, I'll close as a duplicate of #860 since that is tracking the same request.


[Question] Unable to print with star-prnt on TSP100III

i have to support multiple devices and i was able to get this working on an epson printer using the esc mode however when trying to print on a star printer using star-prnt language, it does not work. i am printing from a mac using chrome. how can i debug why it is not working?

    let encoder = new ThermalPrinterEncoder({
      language: "star-prnt",
    });

    let result = encoder
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .text("The quick brown fox jumps over the lazy dog")
      .cut()
      .encode();

    console.log(result);

    const receiptPrinter = new WebUSBReceiptPrinter();

    await receiptPrinter.connect();

    await receiptPrinter.print(result);

    await receiptPrinter.disconnect();

i had added initialize then  removed it to test. i also had codepage set to auto

The library has been tested on and should work on a TSP100III.
What is happening exactly? Does it output anything?
Also, if it does print, but it looks weird, one thing you could try is to see how the dip-switches on the bottom of the printer are configured. It is possible the printer is set up to use ESC/POS instead of StarPrnt.

Okay, I have more information about this.
The TSP100 series itself does not support either Star Line Mode or StarPRNT, used by this library. It uses Star Graphics Mode internally and is not compatible with StarPrntEncoder.
The Windows printer driver does include an automatic software based simulator for Star Line. So if you print on Windows using the printer driver it will work. You can do this with the build in serial port emulator and WebSerialReceiptPrinter.
So, this printer is not compatible with WebUSBReceiptPrinter on Mac.


hi, how to train with multi gpus?

you can train 8 models at the same time on 8 gpus. :P the current code base is suitable for people with less gpu-resources like me at school. it would be good to write a multi-gpu version (with slim it shouldn't be that hard, you can just dump it into multiple tfrecord file and train it), but later. contributions are welcome.


Determine kind ( normal/build/dev) for all dependencies (not just edges)

Changes made

Get kind ( normal/build/dev) for all dependencies (not just edges)
All normal transitives of normal deps are normal
Any build deps to any dep and all normal deps to build deps are build deps, if they're not already in normal list
All deps to dev deps are dev, if they're not already in normal or build list

Note:
both guppy and cargo-tree filters only by edges, therefore, bot determining a final kind for each dependency by themselves.
guppy::graph::PackageLink::dev_only returns false for an edge between a normal dependency of a dev dependency to the root application. However, Isn't that edge is only counted in a dev build? Also, in that way how is the function different than guppy::graph::PackageLink::dev::is_present ?
closes #114

/land

:boom: Tests timed-out

/land

:boom: Tests timed-out

/land

:boom: Tests timed-out

/land

:boom: Tests timed-out

/land

:boom: Tests timed-out

/land

:boom: Tests timed-out

/land

:broken_heart: Test Failed - CI

@bmwill moved octocrab to build-deps in test crate in the prior PR which cargo-geiger doesn't scan. Therefore, needed to adjust some testing. There are three new commits inclduing fixups pushed after what was approved.

/land


Aux window: review usages of DOM.isActiveElement and DOM.isAncestorOfActiveElement

One thing I learned is that document.activeElement will return the documents active element, even if the window does not have focus. In presence of aux windows, this can mean that we have active elements per window, even though only one window or none at all have keyboard focus.
I am introducing new methods DOM.isFocusedElement and DOM.isAncestorOfFocusedElement as drop-in replacements that do the same as before but also check on ownerDocument.hasFocus():

If we have code that makes assumptions based on the active element, that code has to be reviewed and checked if it should adopt the new methods rather when caring about focus as well. In my findings through the code, in most cases I changed to the new methods.
See https://github.com/microsoft/vscode/issues/199867 for a related issue.

Actually, now that I think more about it, we should make isActiveElement and isAncestorOfActiveElement work as getActiveElement does: prefer the focused document and fallback to the main window.


modules/dreamview/fronted: redundancy file(package-lock.json,yarn.lock)

yarn.lock: https://yarnpkg.com/blog/2016/11/24/lockfiles-for-all/
package-lock.json: https://stackoverflow.com/questions/44297803/package-lock-json-role


[Video] set audio category to ambient every time a new player is made

Audio category could get out of sync with the player if you switch video

Nice. I made a small tweak on the native side so we avoid updating the session when unnecessary (it's a little slow). Looks good to me.

Ah perfect, nice one


Function: support dockerImage option

Ryan mentioned:

For DockerImage functions I just want them to deploy as DockerImages, when running locally it would be great if they just ran like normal Functions with the live debug.

Requests

Ryan https://discordapp.com/channels/@me/997591665315430430/1017162943571439688

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


No Bower release since 0.3.2

The package descriptor is currently at 0.4.1, the latest release states 0.3.2, which differs by a major. Is it by accident maybe or are there interdependent significant developments you didn't want to release just yet?

This worked for me
bower install --save git@github.com:openexchangerates/accounting.js.git#master

:+1:

Just installed 0.4.1 via bower. I'm pretty sure this issue can be closed.
bower i accounting.js --save
bower accounting.js#*        cached git://github.com/openexchangerates/accounting.js.git#0.4.1
bower accounting.js#*        validate 0.4.1 against git://github.com/openexchangerates/accounting.js.git#*
bower accounting.js#~0.4.1   install accounting.js#0.4.1

Correct. Tested as well, 0.4.1 installed as appropriate.


[ENH] suggested classifier refactor: _get_train_probs, delegated pipelines

A number of observations from looking at classifiers, suggested refactors.

a large number of current classifiers could be refactored as [transformer][classifier] pipelines, similar to https://github.com/alan-turing-institute/sktime/pull/3102 or https://github.com/alan-turing-institute/sktime/pull/3112, see there for recipe pattern
_get_train_probs should perhaps become a proper interface point.
storing transformed data should not be the responsibility of all classifiers, but only those that compute these, e.g., rotation forest

Idea for _get_train_probs:
this could simply become an option in fit_predict_proba, which is now the main interface point to produce in-sample or out-of-sample predictions on the training set.
I would suggest:

a capability tag, capability:oob-predict
if the tag is True, then fit_predict_proba can be given the arg cv="oob", and it produces the current output of _get_train_probs
similar for fit_predict (the current default would inherit the predictions from fit_predict_proba by mode)

question: any scenario where you do not, at the same time, want to also fit the estimator whenever one obtains oob predictions for the train set? (asking about the implication "if obtaining oob predictions, also want to fit to the entire training set)


Rules for 1.1.9

Just wanted to put this on your radar if you did not already know that 1.1.9 is out!

Hey hey, it's on my high priority list, I'm in Amsterdam at the moment,
will try do a release for 1.1.9 in couple days!
On Mon, 15 Aug 2016, 22:03 John<EMAIL_ADDRESS>wrote:

Just wanted to put this on your radar if you did not already know that
1.1.9 is out!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/artem-zinnatullin/RxJavaProGuardRules/issues/37, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7B3BcQ1DiTBcDRyIe0IB8IaEz8XVQzks5qgMX9gaJpZM4Jkt9I
.

Awesome! thanks for the great work!

@Jawnnypoo done! Should be available in 10-30 minutes

Awesome! Appreciate it!


Backend - Implement Reporting and Moderation Core Function - Moderation Dashboard

Requirements:

1.2.4

<IP_ADDRESS>

<IP_ADDRESS>.1
<IP_ADDRESS>.2
<IP_ADDRESS>.3
<IP_ADDRESS>.4

<IP_ADDRESS>

<IP_ADDRESS>.1
<IP_ADDRESS>.2
<IP_ADDRESS>.3

Description:
Make the reporting and moderation system available to the front teams in its entirety as described in the requirements.

1.2.4 Reporting and Moderation
The system shall have two distinct dashboards to review reported content and users.

<IP_ADDRESS> Moderation Dashboard available to IA moderators shall grant abilities for:

<IP_ADDRESS>.1 Removing inappropriate posts from the IA.
<IP_ADDRESS>.2 To warn users that create such inappropriate posts or misbehave under comment sections.
<IP_ADDRESS>.3 Banning users from the IA.
<IP_ADDRESS>.4 Reporting issues to System Administrators in case these issues require actions outside the scope of the Moderation Dashboard.

<IP_ADDRESS> Administration Dashboard available to System Administrators shall grant abilities for:

<IP_ADDRESS>.1 Removing inappropriate posts or IAs from the platform.
<IP_ADDRESS>.2 Warning users that create such inappropriate posts or misbehave under comment sections.
<IP_ADDRESS>.3 Banning users from the platform.

branch: backend/WA-526

spent: 3.5h


Hive-runner doesn't work on OSX under rbenv, only rvm

This is because the shim inserted for hived start breaks the bundling of ruby dependencies

It doesn't work with uru either, which is a problem for the Windows Hive. I think it may be to do with the use of the daemons gem. Specifically, hived executes the start_hive script but this picks up the wrong version of ruby.

Hi I've faced following problems under this configuration;
OS: MacOS High Sierra
Ruby version: 2.3.7
rbenv version 1.1.0
When I try to run
$ hived

/Users/iso/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/hive-runner-2.1.27/Gemfile not found

And I've changed   '../../Gemfile', part with following content in /Users/iso/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/hive-runner-2.1.27/bin/hived
#!/usr/bin/env ruby

require 'pathname'
ENV['BUNDLE_GEMFILE'] ||= File.expand_path(
 # '../../Gemfile',
  Pathname.new(__FILE__).realpath
)

After this change I tried again with command
$ hived --help

Usage: start_hive <command> <options> -- <application options>

* where <command> is one of:
  start         start an instance of the application
  stop          stop all instances of the application
  restart       stop all instances and restart them afterwards
  reload        send a SIGHUP to all instances of the application
  run           start the application and stay on top
  zap           set the application to a stopped state
  status        show status (PID) of application instances

* and where <options> may contain several of the following:

    -t, --ontop                      Stay on top (does not daemonize)
    -s, --shush                      Silent mode (no output to the terminal)
    -f, --force                      Force operation
    -n, --no_wait                    Do not wait for processes to stop

Common options:
    -h, --help                       Show this message
        --version                    Show version

[!] There was an error parsing `hived`: exit. Bundler cannot continue.

 #  from /Users/iso/.rbenv/versions/2.3.7/lib/ruby/gems/2.3.0/gems/hive-runner-2.1.27/bin/hived:31
 #  -------------------------------------------
 #
 >  Daemons.run(File.expand_path('..', __FILE__) + '/start_hive',
 #              show_status_callback: :test,
 #  -------------------------------------------

Do you have any idea?

@ismailakbudak Can you try and follow http://bbc.github.io/hive-ci/documentation/quick-start.html and use rvm instead of rbenv, probably it is easy to use.


Better handling Fuse Open flags

Alluxio Version:
Master
Describe the bug
In the beginning, Alluxio thinks open() is only used for reading.
After we encounter overwrite case, we realize open() can be used for overwriting, so we detection of
OpenFlags.O_WRONLY
https://github.com/Alluxio/alluxio/blob/8640ff37ead66cd1cb022666299878893904c8ab/integration/fuse/src/main/java/alluxio/fuse/AlluxioJniFuseFileSystem.java#L339
But there are multiple other open flags
https://github.com/mafintosh/fuse-bindings/issues/25
@maobaolong and I encountered the following cases to use open() for write() before
w'   # Open file for writing. The file is created (if it does not exist) or truncated (if it exists).
        #32769
8001 Hexadecimal = 32769 Decimal
'r+'  # Open file for reading and writing. An exception occurs if the file does not exist.
        #32770
8002 hexadecimal

Those two are not included in the Openflags
To Reproduce
Steps to reproduce the behavior (as minimally and precisely as possible)
Expected behavior
Correctly deal with different open() cases.
Don't support opening file for mixed of read and write.
But support other read only or write only case. For write only case, only support delete and write to a new file
Urgency
Describe the impact and urgency of the bug.
Are you planning to fix it
Please indicate if you are already working on a PR.
Additional context
Add any other context about the problem here.

2021-12-23 19:26:13,168 DEBUG AlluxioJniFuseFileSystem - Enter: Fuse.OpenOverwrite(path=/filebench/read/testF/00000007/00000001,flags=0xc002)
because of https://github.com/Alluxio/alluxio/commit/e205ed32bcccf21da148505ef5dbb45dc50a3b90
FYI @maobaolong

open flags: 0x8002 /filebench/composite/testF2/00000001/00000001/00000007
unique: 18712, opcode: OPEN (14), nodeid: 1015, insize: 48, pid: 16953
open flags: 0x8002 /filebench/composite/testF2/00000001/00000001/00000005
open[1097] flags: 0x8002 /filebench/composite/testF2/00000001/00000001/00000003
unique: 18710, success, outsize: 32
open[1098] flags: 0x8002 /filebench/composite/testF2/00000001/00000001/00000004
unique: 18708, success, outsize: 32
open[1099] flags: 0x8002 /filebench/composite/testF2/00000001/00000001/00000005
unique: 18712, success, outsize: 32
unique: 18713, opcode: READ (15), nodeid: 982, insize: 80, pid: 16951
read[1097] 98304 bytes from 0 flags: 0x8002

https://github.com/warrenfalk/fuselaj/blob/master/src/warrenfalk/fuselaj/FileInfo.java
some good references


Version 0.12.1

I also added package-lock.json as recommended by npm 5.

Codecov Report

Merging #701 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #701      +/-   ##
==========================================
+ Coverage   95.26%   95.28%   +0.02%
==========================================
  Files          83       83
  Lines        8990     8990
==========================================
+ Hits         8564     8566       +2
+ Misses        426      424       -2

Impacted Files
Coverage Δ

src/mapInteractor.js
96.12% <0%> (+0.28%)
:arrow_up:

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 31bf6e5...038c936. Read the comment docs.


Middleware ignored when deploying to prod

I've been struggling with this for a while now
I use latest nextjs 14 using the app router and put my middleware.ts into the root of my project as the documentation suggests.
When running npm run dev locally everything works fine and the middleware triggers.
But when running a deployment and I hit the cloudfront routes I just get routed through. No middleware is being triggered.
This is a serious issue as it makes secured routes public available.
This is my middleware setup:
/**
 * BIG WARNING
 * THIS IS NOT WORKING IN PROD middleware seems to be ignored
 */
'use client'
import { fetchAuthSession } from 'aws-amplify/auth/server';
import { NextRequest, NextResponse } from 'next/server';
import { runWithAmplifyServerContext } from '@/utils/amplifyServerUtils';
import { LINKS } from '@/utils/links/links';

export async function middleware(request: NextRequest) {
  const response = NextResponse.next();
  const authenticated = await runWithAmplifyServerContext({
    nextServerContext: { request, response },
    operation: async (contextSpec) => {
      try {
        const session = await fetchAuthSession(contextSpec);
        return session.tokens !== undefined;
      } catch (error) {
        console.log(error);
        return false;
      }
    },
  });

  if (authenticated) {
    return response;
  }

  return NextResponse.redirect(new URL(LINKS.ROOT_AUTH_LOGIN, request.url));
}

export const config = {
  matcher: [
    '/api/login/'
  ]
};

Are you running on windows? Open-next don't work well there, you need to use WSL

@conico974 Thanks! Just deployed on wsl and it worked. Would have never guessed that this was the problem. You saved me tons of time. 👍 I'll close the issue


Add remaining tests

https://github.com/metacall/deploy/blob/605258cf7c063a3b4d02bbb29aad8d951c2c701f/src/test/cli.integration.spec.ts#L159

@viferga i would like to work upon this issue

No tests pending.

@Creatoon can you give me some improvements or issue to be done on this repo, I would like to contribute in GSOC for this project , by contributing I can understand it better

@parteekcoder Use it and if you find any improvement or bugs, raise a pr or open an issue.


(hmac-auth) http_version in request-line is wrong

when i use http1.0 to send request to kong, the request-line string before hmac is
GET / HTTP/1 instead of GET / HTTP/1.0, this bug make my client always been rejected
the problem is this line, kong.request.get_http_version() return ngx.req.http_version()
https://github.com/Kong/kong/blob/88a530c68e09e0bde74a1904ee908054718f6195/kong/plugins/hmac-auth/access.lua#L133-L137
this is my test with resty
# /usr/local/openresty/bin/resty -e 'print(string.format("HTTP/%s", 1.0))'
HTTP/1
# /usr/local/openresty/bin/resty -e 'print(string.format("HTTP/%.01f", 1.0))'
HTTP/1.0

Hi, thanks for reporting this.
It has made me create two different requests:

#4838 because the docs for kong.request.get_http_version() were not correct
#4839 which is, in essence, your proposed fix.

@kikito there is another problem,  no request-line in HTTP2,
HTTP/2 or HTTP/2.0?

@JoveYu that is expected: none of the authentication plugins is compatible with http2.

OK thanks


Change google ad client

It  needs to be pasted in between the  and  tags
The code is:

Find out we can't just place on the subdomain, so I will need to ask Lemon to show you how to place on www.freecen.org.uk as well as FreeCen2.  Code is:

Working fine now on FreeCEN1 and FreeCEN2, but not test2

Hi @PatReynolds ,
Updated test2. Please have a look

Perfect!


integrations: Allow null pull request review body in GitHub Integration.

This PR allows the body of the pull request review to be null. The documentation says that it can null or string. Also added a fixture to test this behavior. Thanks to @kou for the fixture. :heart_decoration:
Fixes #26055.

Self-review checklist

[x] 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).

[x] Each commit is a coherent idea.
[x] 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.

Squashed and merged. @sbansal1999 in general there's no reason to add fixtures in a separate commit after a change that fixes the bug the fixtures test.
(It is useful to add them with tests in a commit before one doing a functional change, because then the diff in the test demonstrates really nicely the corrected behavior, but that doesn't work in the other order)

That makes perfect sense. Thanks for pointing it out. :heart_decoration:


⚠ Bofrost.at has degraded performance

In d66a396, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1869 ms

Resolved: Bofrost.at performance has improved in a44bbd2 after 17 minutes.


Update owin-startup-class-detection.md

Fixed broken MSDN links
When creating a new PR, please do the following and delete this template text:

Reference the issue number if there is one:
Fixes #Issue_Number

The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.

Suggest reviewers if you know who should review the PR, using the GitHub Reviewers UI.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@jboeshart  thanks.


Readme content of Search is unclear and inconsistencey

1
Section Link

Suggestion:
Links are not availble. Add following links:
Java Development Kit (JDK)
https://docs.microsoft.com/en-us/java/azure/jdk/?view=azure-java-stable
2
Section Link

Suggestion:
SearchIndexClient --> SearchServiceClient
The link is unavaliable. Change link to https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/search/azure-search-documents#create-a-searchserviceclient
3
Section Link

Suggestion:
client--> searchServiceClient
client--> searchServiceAsyncClient
client--> searchIndexClient
client--> searchIndexAsyncClient
Also change it in code snippet

https://github.com/Azure/azure-sdk-for-java/pull/10672
This is fixed by above PR.
Thanks @Luyunmt for reporting this!
Close the issue. Feel free to reopen if you have any questions.


Optional values cause toJSON to fail

Invoking toJSON() on objects that have optional fields or values that are set to null or (in some cases) are undefined causes null pointer/undefined pointer errors. Or in some cases a null or undefined value will be entered into the JSON which violates the schema.
This also includes some code-deduplication in the test harness.

Greg, you're tagged for review because this will need to land on flux_a for the curation study.

Hi all, I nuked the redundant assertions and added a minor change that inserts whether or not a field/value is required in the JSDoc.

Hi @daniellee1025, for some reason the GitHub interface won't let me add you as a reviewer (maybe you're not a member of the repository or something?). Anyways, this is the pull request that I mentioned during the scrum.

@jgibson It might be because my GitHub username is danlee1025(not sure why there are 2 different ones displaying...).
Anyways, these changes look fine for Flux.  We currently aren't using the toJSON functions so it should still work with our current code.


[205] Increasing socket count

The default TcpListener in Rust has a very conservative max socket count (lower than Go). Above that threshold, Rust will start dropping socket connections (even if the hardware has the capability to process more).
Depending on the number of concurrent connections configured in the stress testing tool, given you still had CPU and RAM left, it's possible that you simply hit the limit for socket connections and the Rust server started dropping new connections.

This PR increases the max socket count to the max amount possible which should let the Rust server dig a little deeper.

Nope, it didn't help. I keep hitting that magic number of 27k RPS, no matter how much memory or CPU I allocate. I'll test on VMs to avoid an additional layer of abstraction. But it hasn't been related to socket count so far.


Who has rights to edit a video?

The same video can be used in multiple courses, and hence has no clear owner.

I think it should be the creator of the video entry on tekvideo.
A followup question. Can the same video be included in multiple different video entries on tekvideo?

The domain supports it, but not currently implemented on the front-end.


jquery events not firing under neutronium application

hi @David-Desmaisons
trying to have my application detecting some change in the view, after the view model has changed its state. tried a simple  without any bindings to the view model. it gets triggered outside the neutronium application however it does not when running inside it.
Is this expected?
What i want basically is to fire some javascript code from the c# side, so that i don't need to have so many bindings between the view and the model. So be good to have some logic in the client side.
Can that be done? thanks
regards
Jorge Costa

Hello @jmecosta ,
in Neutronium all comunication between C# and javascript is done via View Model binding.
If you need to execute a javascript function trigerred in C#, the best way to do this is to use a property in a viewmodel and react when its value changed:
C# ViewModel:
public Data Data
{
    get {return _data;}
   set { _data=value; FirePropertyChanged(nameof(Data));}
}

Vue Vm:
   watch:{
       Data(newValue){
            //run the javascript function here
            executeCb(newValue);
       }
   }

To run the javascript function just do in your C# code:
vm.Data = data;

wow, how cool is that? Simple and clean. Great. thanks for the help. works great.


Preparing 2.7.0 release

Description of changes:
Minor changes in preparation for 2.7.0 release
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-sDRE8Pq0duHT
Commit ID: a0a14036395f3f0dfaf36f6a98fa79a4bb81d600
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Remove at/ test for reaping, move at/ tests to xt

          I'll remove the test at the end of the at/ test in favor of this one.

I probably should have made everything XT in the first place.
  I'll turn all that into xt tests


Fix Cannot find type 'UUID' in scope

I just saw that you import SwiftUI later on the same file for the View so this might be implicit, feel free to close if you want.


how to upgrade to node 4.2?

what do I need to change to make it work for node 4.2?
Cheers
Qiong

Change the two lines in https://github.com/felixrieseberg/azure-node4/blob/master/deploy_node4.sh#L99-L100 - but I can also update this script ;-)

thanks, that works :)


iOS keyboard reopens after close

Hi,
I've been trying to build a few buttons to sit above the keyboard using keyboard-attach. One of the buttons sole purpose is to close the keyboard.
I've tried every combination of the following, but always on iOS - when the buttons appear over a scrollable view (i.e. the view is bigger than the window) - OR maybe coincidence and the real issue is that the buttons appear over other input boxes, the keyboard - and the keyboard-open body class - are triggerred about half a second after the keyboard closes...
$scope.close = function() {
//do nothing, the blur when clicking the button should suffice
}

$scope.close = function($event) {
                cordova.plugins.Keyboard.close();
 }

$scope.close = function($event) {

                $event.preventDefault();
                $event.stopPropagation();
                var currElement = document.activeElement;
                currElement.blur();
                cordova.plugins.Keyboard.close();
                //$scope.bodyEl.removeClass('keyboard-open');
}

We did some digging by listening to the
window.addEventListener('native.keyboardshow', function(e) {
}

And it looks like the trigger for reshowing the keyboard is coming directly from the iOS code.
The only way I've managed to make this work is to add this code to all input elements of the page. Which temporarily disables the eventlistener inside ionic-keyboard so that the native.keyboardshow event is ignored.
angular.element(element).bind('blur', function (e) {
                            ionic.keyboard.disable();
                            $timeout(function() {
                                console.log('re-enabling keyboard');
                                ionic.keyboard.enable();
                            },500);
 })

Having looked at various bugs around this subject for a few days, It feels like its simillar https://forum.ionicframework.com/t/keyboard-reopens-when-closing-popup/18857
I also tried adding eventListeners like this:-
window.addEventListener('native.keyboardshow', function(e) {
                    console.log('native.keyboardshow fired', $scope.stayClosed, e.keyboardHeight);
                    if($scope.stayClosed){
                        console.log('Keyboard should remain closed', document.body.classList);
                        window.cordova.plugins.Keyboard.close();

                        var currElement = angular.element(document.activeElement);
                        console.log('activeElement', currElement);
                        currElement.blur();
                        $scope.bodyEl.removeClass('keyboard-open');
                        console.log('After Keyboard should remain closed', document.body.classList);
                    } else {
                        console.log('Allowing Keyboard Open', document.body.classList);
                        $scope.bodyEl.addClass('keyboard-open');
                        console.log('After Allowing Keyboard Open', document.body.classList);
                    }
                });

But when trying this, the keyboard itself would not reshow - the native.keyboardshow event occurs, my event forces the keyboard closed again, however the keyboard-attach buttons Remain on the page, as the 'keyboard-open' body class doesn't get removed - despite me explicitely removing it.
I was wondering if anybody else has found this issue?

It also happened to me. Has anyone found a solution yet?

+1

I have the same issue. :( With ion autocomplete in ion-header-bar. At the moment who I hide the ion autocomplete.
Any idea?

I figure out what it causes , I think it's an ios bug , when  the webview is up ,but the ios layout is there , the old input position is still there when the input blur it may trigger other input focus and therefore make the issue happen.
My sulution is blow:
html:
<input type="password" placeholder="password" maxlength="60" ng-model="password"
       ng-disabled="iosDebug"/>

js:
/* when blur or push login btn call the function below */
function iosKbDebug(){
    if(!ionic.Platform.isIOS()) return;
    $scope.iosDebug = true;
    $timeout(function(){
        $scope.iosDebug = false;
     },500);
}

This works for me
setTimeout(() => Keyboard.close(), 500);


Design a logo

idea: arrow up, resembles the I of isamuni and the up direction which relates to the meaning of the word isamuni

I think that we should create a contest on PAC group to give members the possibility to choose the Isamuni's logo.

@giupardeb what is the price for the contest?
I think that as of now users are very lazy and none will participate :(

@sic2 you are right..but I'll try

Latest proposal


"Invalid password or key" when trying to delegate via steemconnect

I have seen other similar issues such as https://github.com/steemscript/steemconnect/issues/170 but could not figure out what's happened. Tested in Chrome Stable (Windows).
I am using my active key as indicated

Well, I ended it up using my password and then change it immediately. that did work even though I think it should never be necessary


gulp compatibility

Make css url relative so that npm css rewrite packages can work (eg: gulp-rewrite-css or laravel-elixir-bower )
Update bower.json main section to handle all main files.

Where is Iran(fa-فارسی)

Merged, done!


depth_processor attribute

Cannot make "depth_processor" other than default. This way I can not limit the attribute values, for example, like the "mode" attribute on jit.alphablend.
This is possibly due to the C Macros used to define a non-default attribute, which throw errors "No matching function for call to 'class_attr_addattr_parse'" (etc.) since ta_jit_kinect2 is in C++
For now this is not really an issue, so I'll leave it as is.

See https://cycling74.com/sdk/MaxSDK-7.0.3/html/chapter_jit_mopdetails.html
to learn how to override default methods and attributes


Provide context data to getAnnounceOpts

I use getAnnounceOpts to extends announce with extra statistics but it is hard to manage multiple torrents without having any information about the torrent or the tracker involved.
This PR binds the tracker to this in getAnnounceOpts and adds the current opts in case we need it in the extensions.

What do you think @DiegoRBaquero and @feross ?


target renders null after unmount/mount

💬 Questions and Help
Hi! I'm looking for some help with some odd behavior I've encountered.
Here's the behavior I'm seeing. The setup, without drowning anyone in code, is:
There is a Root component. There is a Parent component (which is rendered by Root) that contains a Teleporter.Target and is in charge of rendering A and B (using the simple router mentioned below). The OldParent is like Parent in that it renders a .Target

unmount OldParent and mount Parent which renders a Teleporter.Target and mounts A, which renders a Teleporter.Source (ok)
within Parent, unmount A and mount B which contains no Teleporter.Source (ok)
unmount B and (re) mount A 👉 results in null teleportation
...
if I unmount the Parent component going back to OldParent and then re-mount it (which re-mounts/renders A) the teleportation works as expected.

I wrote my own baby router component which is what's rendering these A and B components based on some in-memory state. I don't think the router is relevant to the bug, but I mention it because it certainly might be. The rendering boils down to
const Route = useMemo(() => views[last(history).key], [views, history]);

...
// <Parent /> looks like
return (
  <View>
    <Route />
  </View>
)

// where <View /> basically looks like
<>
  {children}
  <Teleporter.Target as="footer" />
</>

The furthest I got while debugging is that in subsequent renders of Teleporter.Source, context.element is null so the Source renders null due to https://github.com/smooth-code/react-teleporter/blob/master/src/index.js#L36
throwing a debugger into the useCallback() block, though, shows that context.element is set to null during the unmount of a PreviousComponent (expected) but then immediately set to the correct element when Parent is mounted, never being called again. When the source in A attempts to render the second time, though, the element state is null because context.element is null within the useLayoutEffect hook.

Hello @shrugs, your problem looks complex. The code of react-teleporter is very simple. I suggest you to add it directly in your project and to try to debug it. If the bug is confirmed, feel free to submit a PR.

Fixed in 9a5611e9ba0cf66a4d646360413dad5b4d90fdab


Proposal: Allow pre-defined abbreviations in Checkstyle

The Google checkstyle configuration uses allowedAbbreviationLength = 1 for the AbbreviationAsWordInName rule. I have a class CORSResponseFilter that violates this rule, since the abbreviation CORS is too long.
The rule contains a allowedAbbreviations property that can handle exceptions to this rule.
Unfortunately, Checkstyle does not does not support overriding of previously defined rules at the moment (see Checkstyle issue 2873). How should be proceed with this?
I think we should modify the Google checkstyle rules, if all contributors (at the moment only @SoerenHenning) accept the modification.
Therefore I propose to add the allowedAbbreviations property with the default values XML,URL to the Google checkstyle section. Custom values can then be added, e.g., CORS.

On a second thought, this will prevent an easy "copy&paste" update process of your project's checktstyle.xml. However, I do not see an alternative but to copy&paste your custom abbreviations.

I would recommend against this and to keep the rule. For your CORSResponseFilter class, I think CorsResponseFilter would be a better name. The following StackOverflow thread describes it quite well:
https://stackoverflow.com/questions/2236807/java-naming-convention-with-acronyms
Please consider also the second answer regarding the Eclipse shortcut. Moreover, having not everything uppercase allows you to go through the single words of your type name by Strg + → / ←.
However, there seems to be no general rule and even the JDK seems to be inconsistent in this. But for me it seems like that they moved to not using uppercase abbreviations in later APIs. For example: HttpClient instead of HTTPClient (see JavaDoc).


Collect Device Manufacturer

Collect the manufacturer of the device as a string under the "device_manufacturer" field.
Example:
"device_manufacturer": "samsung",

Not sure if you'll find that many iOS device manufacturers really?

@jaakla Agreed.