Event Handler for MessageReceived(int id, string msg)

Hola, first and foremost thank you very much for contributing this codebase to the open-source community. It saved me a lot of frustration right off the bat. However, I'm now running into a issue that I believe is a bug unless I'm misunderstanding the id parameter of the MessageReceived event handler.
To Reproduce

Open a Server Listener
Open multiple client connections to the given Listener
Set a break point on the internal void InvokeMessageReceived(int id, string text) function (called when (State.Header == "MESSAGE") evaluates as true)
Have client 1 send multiple messages using SendMessage() to the server

The first message has an id of 1 as expected, but the second message has an id of 2. My understanding is that the id value passed should indicate which client id sent the message. If I've misunderstood the id parameter, please explain the proper use of it, and could you also explain how I might determine which client id sent the message to the Server Listener? Again I was thinking it would be the Id value inside the passed State.
Likewise, the MessageReceived event does not seem to be firing for additional clients. Only the first client seems to raise this event, despite it being attached to the Listener. I believe this is a bug but it's possible that I've modified something and that it may not be reproducible on your end. Please let me know what you are able to find.
Thanks for your time!

I'm having a bit of trouble replicating this issue. You need to set the events for each client you're declaring so that might be the issue? Could you send me a snippet of your code so I could determine what is going wrong?

I have set the events using a BindEvents() function similar to in the examples provided in the repository (by the way perhaps we should update the readme to include a proper example BindEvents() function? Took me a bit of digging to get to). I am passing the Listener by reference which the examples did not, but this should not cause any issue, should it?
public void BindEvents(ref ServerListener listener) {

    listener.ProgressFileTransfer += Progress;
    listener.MessageReceived += MessageReceived;
    listener.MessageSubmitted += MessageSubmitted;
    listener.CustomHeaderReceived += CustomHeaderReceived;
    listener.ClientDisconnected += ClientDisconnected;
    listener.ClientConnected += ClientConnected;
    listener.FileReceived += FileReceived;
    listener.ServerHasStarted += ServerHasStarted;
    listener.MessageFailed += MessageFailed;
    listener.ServerErrorThrown += ErrorThrown;

    Application.ApplicationExit += new EventHandler(this.OnApplicationExit);

}

And my MessageReceived() function hooked up to the Listener's MessageReceived event works great for the first client, but does not seem to get triggered by the second client.
private void MessageReceived(int id, string msg) { // Client #2's messages are not raising this event / firing this function.

   Console.WriteLine("[MsgReceived(" + id + ")] Contents: " + msg); // The ID printing here alternates between 1 & 2 even when both messages were transmitted by Client Connection w/ ID=1.

    /// ...

}

Your listener events are correct. The ID Parameter corresponds to the int of the Dictionary that stores all the connected clients. So maybe your client disconnects and reconnects ? I've yet to implement a unique identifier per socket, for now the first connected client = 1, second = 2 and so on.
// Returns all currently connected clients public IDictionary<int, ISocketInfo> GetConnectedClients() {      return ConnectedClients.ToDictionary(x => x.Key, x => (ISocketInfo) x.Value); }

My clients are not disconnecting but the second client will not raise the event for MessageReceived. Or, if they are disconnecting, ClientDisconnectedInvoke never fires. I was able to also determine that the connections are unstable when a second client connects, and the original MessageReceived event will often stop firing entirely. Could the the event bindings be getting set when the first client connects, but destroyed at some point when a new client connects? Any ideas for continuing this troubleshooting would be much appreciated, 'cause I'm pretty close to out of ideas.

There might be an issue with your events, it could be a bug too. I’ve actually rewritten this library check https://github.com/Cloet/AsyncClientServer/tree/SimpleSockets?files=1 it doesn’t implement SSL yet but it should work I just tested it with 500 clients yesterday, also it gives a guid to each client. That way you can check if it’s the same client or not.

I appreciate the link to the other project but the reason I went with your library was for the SSL implementation. Nonetheless, I will refer to it regarding the guid logic and see what I can come up with. Thanks!

I’ve actually just added SSL to that project!

Oh, perfect! I'll see what I can come up with when I can dig into the other project. Thanks again, and great work :)

No not yet, tomorrow I'll probably rewrite the readme, list all the changes and add a lot of new info. I'm also doing a bit of testing with ssl at the moment but it should already work fine.

Well I've discovered that I needed to add a byref parameter to my BindEvents() method based on how I'm creating my SocketClients. I'm able to get data from both clients now with the correct clientId as discussed previously (for testing I've made each client send distinctly identifiable messages to the server), but I've discovered some strange behavior. I can send maybe 10 to 15 messages from client 1, but then no more messages will trigger the MessageReceived event of the server - until and unless I start sending messages from client 2 to the server. Then I'll get the server's MessageReceived event to trigger another 10 or so times, and the messages that are printed to my log file are almost perfectly alternating between client 1's message and client 2's messages. Any idea why this might happen?
Also, I was previously using SendMessage() instead of SendMessageAsync(), though I've now switched to using the async method. Still encountering the alternating received messages.

It seems like the Ssl listener has some issues receiving messages, I’ve fixed the majority, I’m still working on Ssl receiving.

I know where my mistake is but I just haven’t had time yet. The issue is when i’m sending a lot of messages at once (test is 10k messages from 500 clients) i’ll receive 4.8m messages instead of 5m. Smaller test works fine though.
When i’m done fixing this i’ll update the Readme (it needs a lot of work) i’ll add a parameter for the timer.


Hierarchy not updated on change

Not all levels in a hierachy are updated.
It can show one item, then when expanded numbers show more, even though there are 15 results


v2.6.0 release

Fixes the breaks in 2.5.0 and includes an additional couple of fixes and features.
A full length screenshot of things to make sure I've not messed things up this time:

LGTM, unless we want to fold in a fix for #1237 ?

Yeah, included. Did not look to have any breaking changes or anything in it.


Frontend: Introduce -experimental-serialize-external-decls-only option

This option is designed to be used in conjunction with -experimental-lazy-typecheck and -experimental-skip-all-function-bodies when emitting a resilient module. The emitted binary module should contain only the decls needed by clients and should contain roughly the same contents as it would if the corresponding swiftinterface were emitted instead and then built.
This functionality is a work in progress. Some parts of the AST may still get typechecked unnecessarily. Additionally, serialization does not trigger the appropriate typechecking requests for some ASTs and then fails due to missing types.
Resolves rdar://114230586

@swift-ci please test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


[ACR] az acr login: Change warning to info

Related command
az acr login
Description
Default message level is WARNING, however this is not really a warning. It should be info level.
WARNING: You can perform manual login using the provided access token below, for example: 'docker login loginServer -u 00000000-0000-0000-0000-000000000000 -p accessToken'
Testing Guide
az acr login --name $myACRName  --expose-token
History Notes
[az acr login] az acr login: Change warning level to info

This checklist is used to make sure that common guidelines for a pull request are followed.

[X] The PR title and description has followed the guideline in Submitting Pull Requests.

[X] I adhere to the Command Guidelines.

[X] I adhere to the Error Handling Guidelines.

ACR

/azp run

@luisgepeto Any update?Please note that we are launching the release for this sprint this week. Please resolve all comments by tomorrow, otherwise the release of this PR will have to be postponed to the next sprint (on 07-02)

@luisgepeto Since we haven't received a response to this PR for a long time, I will put it in the backlog first. After you have resolved its comments, we will consider adding it to the release milestone


Add Dark Mode

This is the PR you didn't know you wanted! It adds a new (highly opinionated) dark mode functionality using the prefers-color-scheme CSS media feature. The preference is automagically picked up via the settings in the operating system.

I love it! Let's run it on Nightly for a little while to experiment, and then I'll merge it.

The lights have now been turned off on nightly.justplanningpoker.com (how appropriate!).

Awesome! I may have removed a bg-light class on the nav header, that I noticed had a ever so slight gray background. On my screen at home I couldn't see the difference from white. I might need to add it back and override that specific class in the dark mode. Other than that the light mode should be unaffected :)

I didn't notice it either. In fact, I think it looks better without it. Let's keep it.


In Matmul, is FP16 x FP16 accumulated to FP16 or FP32?

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 16.04
TensorFlow installed from (source or binary):
binary
TensorFlow version (use command below):
1.7
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
9.0
GPU model and memory:
V100 16GB
Exact command to reproduce:

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
It is not clear in documents that, in Matmul, FP16xFP16 is accumulated to FP16 or FP32.
This choice affects not only training accuracy, but also TensorCore computing performance on Volta GPU.

Nagging Assignee @jart: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jart: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Thank you for the feedback on doc improvement. We'd welcome a contribution that makes our docs more clear on this matter.


repo description typo

two things:

the Github repo description currently reads:  A ollection(sic) of Ember computed macros
this library is awesome

this library is awesome

Thank you! I guess I named the project correctly then lol
Mind submitting a PR?

@kellyselden I would but I can't because it's part of github, not your source:

Oh my bad, I thought you were talking about package.json description. Fixed.


Dual policies for $LATEST qualifier

Description
When the publish variable is set to false and you try to add permissions for function invocations from EventBridge, the module attempts to create two separate permissions, one with the $LATEST qualifier defined and one without. The permission without the qualifier succeeds, but the one with the $LATEST qualifier fails.
Versions

Terraform: 1.0.11

Provider(s):

+ provider registry.terraform.io/hashicorp/aws v3.67.0
+ provider registry.terraform.io/hashicorp/external v2.1.0
+ provider registry.terraform.io/hashicorp/local v2.1.0
+ provider registry.terraform.io/hashicorp/null v3.1.0

Module:
2.27.1

Reproduction

Steps to reproduce the behavior:

terraform apply
Code Snippet to Reproduce

module "generate_s3_url_function" {
  source  = "terraform-aws-modules/lambda/aws"
  version = "~> 2.0"

  function_name = "nebula-notify"
  description   = "Notification of new data dumps"
  handler       = "index.lambda_handler"
  runtime       = "python3.9"

  publish = false

  source_path = "${path.module}/../lambdas/notify"

  allowed_triggers = {
    GenerateUrl = {
      principal  = "events.amazonaws.com"
      source_arn = aws_cloudwatch_event_rule.generate_url.arn
    }
  }
}

resource "aws_cloudwatch_event_rule" "generate_url" {
  name          = "S3GenerateURL"
  description   = "Generate S3 URLs"
  event_pattern = <<EOF
{
  "source": ["aws.s3"],
  "detail-type": ["Object Created"],
  "detail": {
    "bucket": {
      "name": ["${aws_s3_bucket.nebula.id}"]
    }
  }
}
EOF
}

resource "aws_cloudwatch_event_target" "generate_url_lambda_function" {
  rule = aws_cloudwatch_event_rule.generate_url.name
  arn  = module.generate_s3_url_function.lambda_function_arn
}

Expected behavior

Only one permission is created.
Actual behavior

Two permissions are attempted to be created. Here is the plan:
  # module.generate_s3_url_function.aws_lambda_permission.current_version_triggers["GenerateUrl"] will be created
  + resource "aws_lambda_permission" "current_version_triggers" {
      + action        = "lambda:InvokeFunction"
      + function_name = "nebula-notify"
      + id            = (known after apply)
      + principal     = "events.amazonaws.com"
      + source_arn    = "arn:aws:events:us-east-2:623699964423:rule/S3GenerateURL"
      + statement_id  = "GenerateUrl"
    }

  # module.generate_s3_url_function.aws_lambda_permission.current_version_triggers["GenerateUrl"] will be created
  + resource "aws_lambda_permission" "current_version_triggers" {
      + action        = "lambda:InvokeFunction"
      + function_name = "nebula-notify"
      + id            = (known after apply)
      + principal     = "events.amazonaws.com"
      + qualifier     = "$LATEST"
      + source_arn    = "arn:aws:events:us-east-2:623699964423:rule/S3GenerateURL"
      + statement_id  = "GenerateUrl"
    }

Terminal Output Screenshot(s)

│ Error: Error adding new Lambda Permission for nebula-notify: InvalidParameterValueException: We currently do not support adding policies for $LATEST.
│ {
│   RespMetadata: {
│     StatusCode: 400,
│     RequestID: "6c89684c-8a0d-4e1e-8f67-3742ca724cf0"
│   },
│   Message_: "We currently do not support adding policies for $LATEST.",
│   Type: "User"
│ }
│
│   with module.generate_s3_url_function.aws_lambda_permission.current_version_triggers["GenerateUrl"],
│   on .terraform/modules/generate_s3_url_function/main.tf line 193, in resource "aws_lambda_permission" "current_version_triggers":
│  193: resource "aws_lambda_permission" "current_version_triggers" {

Additional context

Currently, this is a behavior of AWS and you need to specify create_current_version_allowed_triggers = false.
Please see Q4 in FAQ.


[QUESTION]Is there a way SKBitmap/ImageSource => SkPicture

Is there any way to convert SKBitmap/ImageSource to SkPicture?
or draw Bitmap with SKMatrix?

If I think what you are asking is there a way to take the SKBitmap pixels and turn them into the drawing commands of SKPicture? then no.
But, if you are just wanting to draw with a matrix, then you should be able to just save the canvas, do the transformation, draw the bitmap and then restore:
var canvas = ...;
using (new SKAutoCanvasRestore(canvas)) {
    canvas.Translate(10, 10);
    canvas.DrawBitmap(...);
}


Fix hard-coded path in recipe_runoff_et.yml

Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[x] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[x] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[x] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[x] Please use yamllint to check that your YAML files do not contain mistakes

Modified recipe/diagnostic

[x] Assign the author(s) of the affected recipe(s) as reviewer(s)

Closes #1486

Too fast too furious, Mattia :grin: - the tests were failing, did you fix that? Didnt have the chance to see why they were failing

The installation test failed, but not because of this PR.
It looks something related to Julia.
@jhardenberg has the same problem in another PR #1528.

Aha, good to know, will have a look, it's about time our environment broke again, we been good for too long now :grin:


LLDP discover

I did not found MIB of LLDP. This protocol is userful to NOC

But you can import it in, using the import.rb script, can't you?


janitorial: eclipse warnings

I don't know if you are interested in this class of warnings.
Hope you will accept the pull request.
Thanks anyway.

once travis is green I will merge it. yes any PR is welcome even if it is only removing warnings on eclipse :)

thanks


Release notes are unnecessarily long

Please limit the information included in the release notes to what is actually included in the release.
By adding the same 4-5 paragraphs for every release, it makes it incredibly tedious to work out what was actually changed.
For example, I'm 90% sure that nothing in the "New Features" section has actually changed in over two years.

Thanks for the feedback. :-)


Configure the description from the official project description

🌱 Some helpful links to get started:
Configuration Guide: Get familiar with the project’s configuration by visiting our configuration page.
Further Customization: For more advanced customization options, check out the official GitHub documentation.

Hello. This is my first time contributing to open source
Can I get a guide on what is required here?

I recommend to look for a project with a much larger community. That will make your journey a much easier one. This is just a issue holder repo, that will never contain any code.

ok. Thank you so much


confusing error message

Guestbook's console output:
Error talking to the datastore: no matching index found. recommended index is:
- kind: greeting
  ancestor: yes
  properties:
  - name: date
    direction: desc

I can't work out what I'm supposed to do with that. Sure, I can access the data store online, and start 'create entity' but there's a little correlation with fields in that message above, and controls in the form for 'create entity'.  Why doesn't Guestbook initialize its own schema?

Hey there!

Datastore is a schemaless database, so there's nothing to initialize.
What Datastore does require you to do is set up indexes based on the queries you'd like to perform. I'm sorry that wasn't clearer for this demo; take a look here for an example of what you'll need to do.

I didn't write Guestbook. I'm commenting on its default output which is hugely confusing to noobs. I was suggesting some more advice for noobs. Not sure why you're explaining about schema-less and closing the issue, versus writing in the README or associated page online - something explaining a more step by step running of the Guestbook demo.


Update scala-library, scala-reflect to 2.12.15

Updates

org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.12.14 to 2.12.15.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-patch

Codecov Report

Merging #144 (73c536b) into master (8e5099b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #144   +/-   ##
=======================================
  Coverage   80.35%   80.35%
=======================================
  Files           6        6
  Lines         112      112
=======================================
  Hits           90       90
  Misses         22       22

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 8e5099b...73c536b. Read the comment docs.


UN-411: Insights to Stories rename app

Related ticket(s):
https://national-archives.atlassian.net/browse/UN-411
About these changes
Calls rename_app to update insights to stories in deployment and local environments
This update only works on first call, subsequent calls have not effect since the change has taken place.
However another PR post deployment will remove these changes.
How to check these changes
This PR is required to be merged into feature/UN-411-insight-to-story.
Observe the logs for echoed output.
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly myself before handing over to reviewer.
[x] Included the ticket number in the PR title to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

For Reviewer
Once PR is merged, check and arrange to deploy on platform-sh.

Closed - follow on PR-601


Bug with wp.pow

Seems to be a bug with wp.pow for negative values:
@wp.kernel
def test(tt: wp.array(dtype=wp.float32)):
  tt[0] = wp.pow(-1., 2.)

tt = wp.zeros((1,), dtype=wp.float32, device='cuda')
wp.launch(test, dim=1, inputs=[tt], device='cuda')
print(tt)

The above prints [nan] while it should print [-1]. Could be an issue with my graphics card who knows but if not then it's probably a bug somewhere. (I'm getting the correct value with device='cpu'.)

I was seeing the same issue on a different example. For me it was random and about 50% of computations produced the nans. My work around was to just not use pow and do the multiplications out.

This is the default behavior for powf() on CUDA when using --fast-math (which is on by default) (https://forums.developer.nvidia.com/t/unsuspected-work-of-pow-function-pow-device-function-works-incorrectly-witn-negative-numbers/8089/2).
I agree though that this can be quite unexpected. I will revisit the use of --fast-math, we may actually add an option to turn this on/off per-module.

Just following up, we disabled --fast-math by default. You can now enable it on a per-module basis using wp.set_module_options({"fast_math": True}).
Best,
Miles


fix: run-sidecar.sh for vm.

Description:
fix run-sidecar.sh for vm.

Testing done:
Done

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)? no

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (release/v1.4@6c7ade8). Learn more about missing BASE report.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff               @@
##             release/v1.4     #577   +/-   ##
===============================================
  Coverage                ?   14.18%
===============================================
  Files                   ?      820
  Lines                   ?    65272
  Branches                ?        0
===============================================
  Hits                    ?     9257
  Misses                  ?    55840
  Partials                ?      175

Flag
Coverage Δ

unittests
14.18% <100.00%> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Refactor GraphConnector status aggregation

GraphConnector currently assumes that all collections it returns need to report a status to it. This means that all collections created in it need to take and manage a StatusUpdater parameter and call it at the right time (there may be race conditions between calling the status updater and when GraphConnector initializes the WaitGroup it uses to track collection completion)
Going forward, it may be less error-prone to design and implement another way to gather statistics from the collections

Scrubbing backlog. Closing some of the older items that haven't been prioritized - will let them resurface if there is pain.


Json output - weird numbers

Getting different numbers (1907.9999999999999999) on output than in my input when setting output mode to json. csv output mode works fine.
% cat /tmp/x.csv
name,value
a,1908.0
b,1908.0
c,1908.0

% duckdb -json /tmp/x.db "select * from read_csv_auto('/tmp/x.csv')"
[{"name":"a","value":1907.9999999999999999},
{"name":"b","value":1907.9999999999999999},
{"name":"c","value":1907.9999999999999999}]

% duckdb -csv /tmp/x.db "select * from read_csv_auto('/tmp/x.csv')"
name,value
a,1908.0
b,1908.0
c,1908.0

% duckdb --version
DuckDB 3222accba

This looks like a rendering choice by the SQLite shell output to me and not like something the DuckDB engine has any influence on.

This looks like a rendering choice by the SQLite shell output to me and not like something the DuckDB engine has any influence on.


lsp/refs: Just in time init for refs util

This was being slow to start and adding around 100ms to all regal command runs, even though this is only needed for the lsp.
This change makes the initialization of the pq var happen at the first call instead.

Since we both thought the same, let's use that :)

Thanks both, have updated.


No error message for invalid sql statements

Error description
When executing an invalid query there is no feedback to the user that something went wrong. The result tab just shows the result of previous query. Also in the log tab, there is no indication that something went wrong.
An invalid query can be

a wrong SQL statement
valid SQL but with wrong table name

Expected behavior
Some kind of error message/warning. Maybe for beginning, it is possible to show the error message from DataFusion itself?

Can you provide an example query?  I had been working that and got it to work correctly, for example on parser errors(this is on latest main):

Your latest PR seems to fix this issue - cool!
Now, I have another issue. After executing an incorrect query (wrong path to the file with data), next query is not executed anymore. At least, it does not appear in the history and the error message that the path is wrong does not disappear as well (cf. screenshot)

.

Got it - will look into.

So i think its actually because of the semicolon.  If you delete the semicolon after the bad query and add another one again i think it will work.  I suppose the expected behavior would be keeping the semicolon status (i.e. execute after hitting enter).

I will need to think about this a little more.  Off the top of my head im not sure how straight forward this is to handle in a simple way as i could always need to be looking through all the editor text for a semicolon to determine if the flag should be set.

the status is tracked in app.editor.sql_terminated

I think I have an idea. Rather than using enter in edit mode and tracking semicolon to execute the query I use enter mode in normal mode to trigger the query. If no semicolon it simply won't execute and I don't have to worry about tracking status. Maybe I could provide a message to inform user if we wanted to be extra friendly but don't think that's required.  User would just have to remember to go from edit to normal mode to execute but I think that's okay. What do you think?

I wonder if one needs a semicolon at all. Right now, the editor always contains a single query and unless this changes in the future, having a semicolon is only required to recognize that the user wants to execute a query. Thinking about other tools, the same problem is often solved by using another key combination e.g. "shift+enter". Could that maybe be a more user-friendly solution?
On the pro-side, if chosen carefully, this "execute query" key combination could be used in the normal and edit modes such that the users will do less mistakes.
On the contra-side, the complexity of key handling increases.
How do you see it?

sorry, pressed wrong button and closed the issue accidently

I had in mind that people could do DDL plus a query in the same editor which would require the semicolon in the editor.  i thought there was value in this for interactive data analysis and it provides a good interface before doing DDL through ~/.datafusion.
I definitely also had an interest in something like shift-enter, as i use that in datagrip all the time.  However, i think that would be non-trivial to implement (although i do have an idea for this already).  But i will create a separate issue for this point.
Switching to only executing in normal mode i think provides a good tactical solution to this issue for 0.1 release (thats my focus right now) while a longer term solution can be investigated further.  hope thats ok!

Sure - good idea! And you are absolutely right about caring more about 0.1 release. If one does not stop thinking about improvements, one will never come to a release... this happens all the time to me. :)


⚠ Contemporanea Group has degraded performance

In 320e8be, Contemporanea Group ($SITE_CNTMG) experienced degraded performance:

HTTP code: 200
Response time: 11257 ms

Resolved: Contemporanea Group performance has improved in 6012639.


call to get ui5 control is slow

Describe the bug
We started using wdi5 service in wdio tests and we noticed that it takes 1.2 or more second for any interaction with ui5 control to happen. I.e. to retrieve a proxy object of a Title or a Tile takes time. And after adding additional time logging I saw that most of the time is used waiting for sap.ui.test.RecordReplay.waitForUI5() function execution.
This ui5 library function is checking if all the waiters are not waiting. That includes promise waiter which gets two primises just before calling waitForUI5(). One promise comes from puppeteer library and another from the waitForUI5 method itself.
Few questions come to mind while investigating this slow performance:

Does anyone gets this slow performance (~1200ms) from the wdi5 browser.asControl function? I have checked that wdio DOM element access has very small 30-60ms response.
Am I doing something wrong with my selectors or configuration?
Could it be that that is another issue with wdi5 or there is a slow inherited waitForUI5 function logic in ui5 library.

Also I have switched from puppeteer to selenium driver and then only one promise was is _promiseWaiter object internal watch list. But that did not reduce delay of waitForUI5() execution.
To Reproduce
Call browser.asControl() to get a proxy object of any ui5 control. It should take around 1.2 seconds.
Expected behavior
I expect the control search to take 100ms or less.

wdi5/wdio-ui5-service-version: 0.8.2
UI5 version: 1.98.0
wdio-version (output of wdio --version): @wdio/cli (version: 7.16.16)
node-version (output of node --version): v16.14.0
OS: MacOS Monterey (12.2.1)
Browser + Version [Chrome: Version 99.0.4844.51 (Official Build) (arm64)]

Hi,
thank you for fast response. I have tested this issue with 0.9.0-rc1 version.
The issue persists.
Just to illustrate the performance issue I am adding changes which helped me to track the slow but. As you can see waitForUI5 takes around 1.2 seconds.
This is my changes to track time in getControl():
async function clientSide_getControl(controlSelector) {
    controlSelector = await Promise.resolve(controlSelector) // to plug into fluent async api
    return await browser.executeAsync((controlSelector, done) => {
        const printTime = (message) => {
            const d = new Date()
            console.log(`${d.getHours()}:${d.getMinutes()}:${d.getSeconds()}.${d.getMilliseconds()} -::- ${message}`)
        }
        const waitForUI5Options = Object.assign({}, window.wdi5.waitForUI5Options)
        if (controlSelector.timeout) {
            waitForUI5Options.timeout = controlSelector.timeout
        }
        printTime('Starting waitForUI5')
        window.bridge
            .waitForUI5(waitForUI5Options)
            .then(() => {
                printTime('Starting retrieving control')
                window.wdi5.Log.info("[browser wdi5] locating " + JSON.stringify(controlSelector))
                controlSelector.selector = window.wdi5.createMatcher(controlSelector.selector)
                const element = window.bridge.findDOMElementByControlSelector(controlSelector)
                printTime('Completed retrieving control')
                return element
            })
            .then((domElement) => {
                // window.wdi5.Log.info('[browser wdi5] control located! - Message: ' + JSON.stringify(domElement));
                // ui5 control
                printTime('Retrieving a control again')
                const ui5Control = window.wdi5.getUI5CtlForWebObj(domElement)
                printTime('Getting control id')
                const id = ui5Control.getId()
                printTime('Control id retrieved')
                window.wdi5.Log.info(`[browser wdi5] control with id: ${id} located!`)
                const aProtoFunctions = window.wdi5.retrieveControlMethods(ui5Control)
                // @type [String, String?, String, "Array of Strings"]
                done(["success", domElement, id, aProtoFunctions])
            })
            .catch((error) => {
                window.wdi5.Log.error("[browser wdi5] ERR: ", error)
                done(["error", error.toString()])
            })
    }, controlSelector)
}

module.exports = {
    clientSide_getControl
}

this is the console output:
2022-03-22 19:10:49.064699 FLP Bootstrap Scheduling Agent :: MODULE 'schedulingAgent' /undefined/ : Loading finished - FLPScheduler
VM800:6 19:10:59.287 -::- Starting waitForUI5
VM800:6 19:11:0.499 -::- Starting retrieving control
Log.js?eval:456 2022-03-22 19:11:00.498100 [browser wdi5] locating {"forceSelect":true,"selector":{"controlType":"sap.m.GenericTile","properties":[{"header":"Product Explorer"}]},"wdio_ui5_key":"sap.m.GenericTile_-_-_-_[{\"header\":\"Product Explorer\"}]"} -
Log.js?eval:456 2022-03-22 19:11:00.502000 Found a DOM reference for the control 'Element sap.m.GenericTile#__tile0'. Executing 'sap.ui.test.actions.Press' on the DOM element with ID '__tile0' -  sap.ui.test.actions.Press
VM800:6 19:11:0.503 -::- Completed retrieving control
VM800:6 19:11:0.503 -::- Retrieving a control again
VM800:6 19:11:0.503 -::- Getting control id
VM800:6 19:11:0.503 -::- Control id retrieved
Log.js?eval:456 2022-03-22 19:11:00.502699 [browser wdi5] control with id: __tile0 located! -
VM820:6 19:11:0.664 -::- Starting waitForUI5
VM820:6 19:11:1.873 -::- Starting retrieving control
Log.js?eval:456 2022-03-22 19:11:01.872699 [browser wdi5] locating {"forceSelect":true,"selector":{"controlType":"sap.m.GenericTile","properties":[{"header":"Product Explorer"}]},"wdio_ui5_key":"sap.m.GenericTile_-_-_-_[{\"header\":\"Product Explorer\"}]"} -
Log.js?eval:456 2022-03-22 19:11:01.874500 Found a DOM reference for the control 'Element sap.m.GenericTile#__tile0'. Executing 'sap.ui.test.actions.Press' on the DOM element with ID '__tile0' -  sap.ui.test.actions.Press
VM820:6 19:11:1.876 -::- Completed retrieving control
VM820:6 19:11:1.876 -::- Retrieving a control again
VM820:6 19:11:1.876 -::- Getting control id
VM820:6 19:11:1.876 -::- Control id retrieved
Log.js?eval:456 2022-03-22 19:11:01.875100 [browser wdi5] control with id: __tile0 located! -
Log.js?eval:456 2022-03-22 19:11:03.299699 Outer shell hash changed from 'Shell-home' to 'Product-display' -  sap.ushell.services.ShellNavigation
Log.js?eval:456 2022-03-22 19:11:03.313899 Tile clicked: -  sap.ushell.ui.launchpad.Tile
VM867:6 19:11:3.378 -::- Starting waitForUI5
Log.js?eval:456 2022-03-22 19:11:03.420600 NavTargetResolution: custom resolver DefaultAdapter resolves #Product-display -
Log.js?eval:456 2022-03-22 19:11:03.606899 LoaderExtensions.registerResourcePath ('sap/epd/ps/app/explorer', 'resources/sap/epd/ps/app/explorer') -
Log.js?eval:456 2022-03-22 19:11:03.609300 Loading manifest via URL: resources/sap/epd/ps/app/explorer/manifest.json?sap-language=EN -
Log.js?eval:456 2022-03-22 19:11:03.665899 Including /resources/sap/f/themes/sap_fiori_3/library.css -  sap.ui.core.Core.includeLibraryTheme() -
Log.js?eval:456 2022-03-22 19:11:03.699399 Including /resources/sap/uxap/themes/sap_fiori_3/library.css -  sap.ui.core.Core.includeLibraryTheme() -

Hi @edvardas-kireilis, first of all thanks for providing detailed info + context and for already digging quite deep 👍
we brainstormed over your issue and will give a couple of ideas a try. that being said, i can state that no one has reported these long waiting times before. that's why we have the hunch that the FLP might be involved into things - so double checking here: judging by your logs above, you're testing a UI5 app, starting off from a tile in the FLP, correct?

Hi Volker. You are correct. We are starting using FLP and app tile. I would like to add that there seems to be a consistency with every browser.asControl() call.
I have tried to use custom waitForUI5 function logic replacing promise into callbacks. I managed to reduce overall asControl() execution time from 1.2sec to 0.3sec. But the time varies now.
Maybe wdi5 library could try this approach as well. But for that to work puppeteer has to be replaced with selenium driver because puppeteer is adding a promise of it's own.
Thank you for you quick response. Looking forward to a resolution.

Hi. Is there any update on the ideas to solve this issue?

are you using puppeteer with the WebDriver protocol as a capability?
anyway, could you please share your wdio.conf.(j|t)s?
and the test code in question?

As a side note:
it would be nice to be able to configure window.wdi5.waitForUI5Options.interval value which currently is 400 ms. Maybe reduction of it could speed up the execution in some cases.

As a side note:
it would be nice to be able to configure window.wdi5.waitForUI5Options.interval value which currently is 400 ms. Maybe reduction of it could speed up the execution in some cases.

+1 - we've also seen that already and tracked it in #143

It seems that puppeteer is use by default, unless selenium driver is configured.
This is my relevant configuration

exports.config = {
  specs: [
    './test/wdi5/specs/**/*.spec.js'
  ],
  exclude: [
  ],

  maxInstances: 1,

  capabilities: [{
    maxInstances: 1,
    browserName: 'chrome',
    acceptInsecureCerts: false
  }],
  logLevel: 'warn',

  bail: 0,

  baseUrl: 'http://localhost:50001',

  connectionRetryTimeout: 1000 * 60 * 30,

  connectionRetryCount: 3,

  services: [
    'ui5',
    ['selenium-standalone', { drivers: { firefox: '0.29.1', chrome: true, chromiumedge: 'latest' } }]
   // puppeteer is used if above is not added
  ],

  framework: 'mocha',

  reporters: [
    'spec',
    ['junit', {
      outputDir: './'
    }]
  ],

  //
  // Options to be passed to Mocha.
  // See the full list at http://mochajs.org/
  mochaOpts: {
    timeout: 1000 * 60 * 30
  },

  wdi5: {
    screenshotPath: path.join(exports.getResultLocation(), 'screenshots'), // [optional] using the project root
    screenshotsDisabled: false, // [optional] {Boolean}; if set to true screenshots won't be taken and not written to file system
    logLevel: 'error', // [optional] error | verbose | silent
    platform: 'browser',
    url: '/flpSandbox.html#Shell-home',
    deviceType: 'web', // [mandatory] native | web
    skipInjectUI5OnStart: true,
    waitForUI5Timeout: 1000 * 60 * 40 // [optional] maximum waiting time while checking for UI5 availability
  }
}

thanks for providing the config. We have up to date

not included selenium-standalone
no test-setup for puppeteer with WebDriver

in any of our test suites.
I'll take this as the opportunity to do so, including adding performance-relevant metrics → #202. Then we'll be able to investigate any potential performance bottlenecks.
That being said, this setup will take us a while - so asking for your patience until ~ end of next week.
Or, since you obviously have a very good grasp on WebdriverIO + wdi5 in general and already put in metrics, we'd very much appreciate you PR-ing some wdio-setup with puppeteer and WebDriver, e.g. in /examples/ui5-js-app. That would give us all a head-start 🤗

PR to improve to fix this issue:
https://github.com/js-soft/wdi5/pull/209


Add py35 tox test config

When you get a chance, can you review and maybe merge this one ?
Adding py35 support will be very useful.

First, I'm a little surprised that this small of a change adds some additional Python3.X support. Thanks for contributing that!
Unfortunately, it looks like setuptools/pip have entirely dropped support for Python3.2. As a result, the tests are failing with your patch. The good news is that the new Python versions you've added to the test suite are passing. Do you mind dropping the 3.2 test runs from the Travis config?
I recognise that somewhere along the lines we hit a breaking change because of setuptools, but it would have affected users long ago if it was going to so I'll release your change under a minor rev.
As an aside, what use cases do you have for a library like this? Nowadays, everything is either using Docker or a process manager like upstart or systemd. The usefulness of unix daemons has dropped to an extremely low level because almost all process managers ask for non-forking daemons. Are you working in an environment where process managers/Docker are not available?

I have updated PR to remove py32. Thanks for looking into it!
I really just want to learn and educate myself how a system process spawner was made, and I happen to be more of a developer than system admin so getting to know how some of the basic system blocks are made is what I want to do.

@tonytan4ever Hey, I wan't to apologize for letting this PR sit around for so long. I usually try to be responsive to PRs and issues for my project here and I really dropped the ball on this one. Somehow, I got under the impression that I had already merged this so I didn't think about it.
Sorry for the extremely long delay.


fix: Correct capacity reservation target

Description

Correct capacity reservation target to work without needing to set an open capacity reservation

There are two capacity_reservation_specification definitions - one in the instance resource and one in the spot request resource. In the original PR only one of the definition was updated correctly to match the default value type of {}. This is now synchronized across the two resources

Motivation and Context

Ref #285
Closes #286
Closes #287

Breaking Changes

No

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

This PR is included in version 4.1.4 :tada:


feat: add new kubestronaut

Hello Fabio, thanks for doing PR but I will have to close it. I am gathering all the info from all new Kubestronauts and add them at once at the end of each week. All the best


#_on_core_channel_exit upon exiting neovim

Hey there @roxma, loving this plug, super fast and works well so far. Just have one minor issue, with this plugin installed and enabled via vim-plug, i get the following output when exiting neovim:
E118: Too many arguments for function: 5
E118: Too many arguments for function: <SNR>76_on_core_channel_exit
E118: Too many arguments for function: 6

Please let me know if there are any steps you'd like me to do for debugging or any additional info you'd need.
I'm on OS X 10.11.6, iterm2 nightly, neovim-HEAD.
Thanks!

@roxma as a quick update, upon entering a js file, i get the following message in neovim's command input: E118: Too many arguments for function: 24.
Also, I'm getting regular segfaults with not touching anything with a file/buffer opened with the following:
nvim(93754,0x700000801000) malloc: *** error for object 0x7fe8dbc25a18: incorrect checksum for freed object - object was probably modified after being freed. *** set a breakpoint in malloc_error_break to debug

I've got jedi, and tern_for_vim installed as well.

E118: Too many arguments for function: <SNR>76_on_core_channel_exit Should be fixed now.
I'm using neovim 1.7, It seems you're using an unstable version, neovim-HEAD, this is possibly related to the malloc issue.

@roxma i'll flip back to stable on neovim and see if the malloc issues resolve. thanks much!!

@roxma last little item from that fix for this issue.. upon exit, now seeing: W19: Deleting augroup that is still in use. i know this is a minor item, just want to help you get this thing super stable!

will also post back about moving to neovim stable.

@roxma no crash yet on 1.7; will stick to that for a while.
also, still, i'm wondering how you're getting javascript completion; i have tern_for_vim installed with the proper npm package installed. e.g., what i had working for deoplete/deoplete-ternjs.
i'd love to get that awesome JS completion like you have in your asciicinema!
thanks again :D


Swift setup cannot create directory error

When running the latest installer build, I ran into the issue:
Cannot create the directory 'C:\Library\Developer\Platforms\Windows.platform\Developer\SDKs\Windows.sdk\usr\lib\swift\windows\x86_6MSVCRT.swiftmodule'. A file with this name already exists. Please rename or remove the file and click retry, or click Cancel to exit

This should most likely have an overwrite option (or overwrite by default), as it was created by a previous installer.

This also occurred in that directory with:
SwiftOne.swiftmodule
Swift.swiftmodule
WinSDK.swiftmodule

Manually deleting the files solved the issue and allowed the installation to succeed

Ah, the install over the existing installation process is currently not working :-(.  I want to figure out how to do that (#298 was my initial attempt to get that to work).  The problem is that for the clang resource headers, I use a tool to enumerate the headers and install them rather than explicitly list them out.  If we can get that fixed up, we should be able to support in-place upgrades.  The current workaround is to run the uninstaller first :-( (you should be able to go to "Add/Remove Progarms" and select Modify and then uninstall.

This should be fixed at this point with the installer overhaul in the 5.10 release.


PHP Fatal error:  Allowed memory size of 268435456 bytes exhausted

Trying this HTML minifier on a live production. Sometimes it throws:
PHP Fatal error:  Allowed memory size of 268435456 bytes exhausted (tried to allocate 81920 bytes) in tiny-html-minifier.php on line 33

How can we fix this? And is this happening because of false HTML markup (like unclosed tags)?

So far this script does not care at all about open or closed tags. It just looks at < and > so that's not the problem.
I'm not sure what it is and because I can't reproduce the problem, it's hard to fix it.

I have the same issue. If you want a FTP access to reproduce it, lemme know.

This should be fixed by the latest commit.


in paddlespeech/cls/exps/panns/deploy/predict.py   from paddleaudio.features import melspectrogram error

For support and discussions, please use our Discourse forums.
In paddlespeech/cls/exps/panns/deploy/predict.py  file
import numpy as np from paddle import inference from paddleaudio.backends import load as load_audio from paddleaudio.datasets import ESC50 **from paddleaudio.features import melspectrogram** from scipy.special import softmax
the melspectrogram is error import,
when in 'PaddleSpeech/examples/esc50/cls0' dir
run  './run.sh 4 cpu ./export /audio/dog.wav'
got
--- Running analysis [adjust_cudnn_workspace_size_pass] --- Running analysis [inference_op_replace_pass] --- Running analysis [ir_graph_to_program_pass] I1101 10:49:58.253744 1347346 analysis_predictor.cc:714] ======= optimize end ======= I1101 10:49:58.253801 1347346 naive_executor.cc:98] ---  skip [feed], feed -> x I1101 10:49:58.254617 1347346 naive_executor.cc:98] ---  skip [linear_4.tmp_1], fetch -> fetch ['/home/wayne/audio/dog.wav'] W1101 10:49:58.256059 1347346 device_context.cc:447] Please NOTE: device: 0, GPU Compute Capability: 7.5, Driver API Version: 11.4, Runtime API Version: 11.2 W1101 10:49:58.257632 1347346 device_context.cc:465] device: 0, cuDNN Version: 8.1. Traceback (most recent call last):   File "/home/wayne/project/baidu/PaddleSpeech/paddlespeech/cls/exps/panns/deploy/predict.py", line 143, in <module>     results = predictor.predict(wavs)   File "/home/wayne/project/baidu/PaddleSpeech/paddlespeech/cls/exps/panns/deploy/predict.py", line 119, in predict     feats = extract_features(wavs)   File "/home/wayne/project/baidu/PaddleSpeech/paddlespeech/cls/exps/panns/deploy/predict.py", line 57, in extract_features     feat = MelSpectrogram(waveforms[i], sr, **kwargs).transpose()   File "/home/wayne/.local/lib/python3.8/site-packages/paddleaudio/features/layers.py", line 145, in __init__     self.fbank_matrix = compute_fbank_matrix(   File "/home/wayne/.local/lib/python3.8/site-packages/paddleaudio/functional/functional.py", line 179, in compute_fbank_matrix     fftfreqs = fft_frequencies(sr=sr, n_fft=n_fft, dtype=dtype)   File "/home/wayne/.local/lib/python3.8/site-packages/paddleaudio/functional/functional.py", line 145, in fft_frequencies     return paddle.linspace(0, float(sr) / 2, int(1 + n_fft // 2), dtype=dtype)
import change to
‘from paddleaudio.compliance.librosa import melspectrogram’
then can work.

please use paddleaudio==1.0.1 then the script before change can work
by the way, https://github.com/PaddlePaddle/PaddleSpeech/blob/develop/paddlespeech/cls/exps/panns/deploy/predict.py we use paddlespeech.audio now, which version of PaddleSpeech script now?

commit 4b3f6c615ebb77a3577e552d34339a85e3de73b5 (HEAD -> develop, origin/develop, origin/HEAD)
Merge: 38eb3361 b1ef4349
Author: Hui Zhang<EMAIL_ADDRESS>Date:   Wed May 18 15:49:09 2022 +0800
Merge pull request #1913 from Honei/timelimit

[asr][server]add asr time limt configuration

commit 38eb3361f41c0b6323644dd72734e1723aa85f4b
Merge: fb4d255f 61e4b44b
Author: TianYuan<EMAIL_ADDRESS>Date:   Wed May 18 13:05:41 2022 +0800
Merge pull request #1918 from Jackwaterveg/develop

I also use paddlespeech.audio

oh, the commit is too old for now, so paddleaudio==1.0.1 is well


Choose a time tracking tool.

Determine how we will do time tracking.
Choose a time tracking tool.
Do some little research on time tracking.
Documentation of what we found in the wiki

I have chosen to use Microsoft Project as a project management, resource management and time-tracking tool.


GDNative method argument name receive the name of the class

Godot version: Godot 3.3 RC 8 (but it's on the master as well)

OS/device including version: N/A

Issue description:

godot_nativescript_set_method_argument_information incorrectly sets the name of the class as the name of the argument.
the variable defined here should be used here instead of p_name.
Steps to reproduce: N/A
Minimal reproduction project: N/A

I'll complete with Steps to reproduce and Minimal reproduction project soon.
It involves creating a GDNative library, so it's not straightforward to make it.

GDNative was substituted by the GDExtension, so this issue is probably irrelevant, I'll close for now, reopen if needed.


Respond to greetings in polite script

This patch adds an additional response to greetings, as well as short responses for short thank-yous, for extra politesse.

Can you merge master in and reconcile the conflicts?

@atmos sure thing, I did it.


[warning] failed package validation and/or copy for commit 5d233e2de534b3f96a2e810f2d972510bb3132f7

Hi @conda-forge/opentelemetry-exporter-jaeger! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

Hi @conda-forge/opentelemetry-exporter-jaeger! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

Hi @conda-forge/opentelemetry-exporter-jaeger! This is the friendly automated conda-forge-webservice!
It appears that one or more of your feedstock's outputs did not copy from the
staging channel (cf-staging) to the production channel (conda-forge). :(
This failure can happen for a lot of reasons, including an outdated feedstock
token. Below we have put some information about the failure to help you debug it.
Rerendering the feedstock will usually fix these problems.
If you have any issues or questions, you can find us on gitter in the
community chat room or you can bump us right here.
error messages:

invalid feedstock token

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/opentelemetry-exporter-jaeger-feedstock#6.

Think this was fixed a while ago. Closing...


ReactRedux configuration doesn't play well with TypeScript tools

Current configuration of ReactRedux project doesn't work well with TypeScript code actions. In my case I have two files:

/ClientApp/logs/components/LogsList.tsx
/ClientApp/logs/components/OperationLogs.tsx

When I try to add import to LogsList.tsx from OperationLogs.tsx the following line is added:
import { LogsList } from 'ClientApp/logs/components/LogsList';

Visual Studio doesn't show any errors but then when I run application I get such error:
NodeInvocationException: Prerendering failed because of error: Error: Cannot find module "ClientApp/logs/components/LogsList"
I have already reported to TypeScript repository that in such case they could use relative path https://github.com/Microsoft/TypeScript/issues/15888#issuecomment-323339210, but it would be nice to have that correctly configured in template.

I think the syntax you need to use is:
import { LogsList } from './ClientApp/logs/components/LogsList';

i.e., with a leading ./.
However, I'm guessing that you already know this, and the problem you're reporting is that the TypeScript tooling code action is generating the incorrect import. If so, that would be an issue for the TypeScript tooling folks to address. I'll close this since it's not something we can fix in this repo, but please let me know if you think we are doing something wrong in this repo!

I don't think that adding ./ will help - then it will be incorrect path and I believe that TypeScript tools works according to tsconfig.json so probably there is the issue.

The actual correct relative path will depend on where the file is and where it's being referenced from, of course.

I believe that TypeScript tools works according to tsconfig.json so probably there is the issue.

I'm not clear on whether you're saying there's a fault in this repo or not. If you think something is wrong here, can you give an example of what change to the tsconfig.json you think we should make, and a scenario where it makes a difference?


Profile and Discord Image Verification

Feature to build profile and Discord image verification
This feature will verify if the photo of an individual is the same as the discord's photo.
Points:

User must be verified by RDS bot!!
User photo and their discord photo can almost have a difference of 20%(Allowed due to pixel-2-pixel checking)
If a user's photo verification fails, moderators will be notified about this discrepancy.(Open to suggestions for other ways to handle this). This is to done by adding new documents of users who's verification failed.

Data model

{
    userId: string,
    discordId: string,
    discordImage: string,
    date: timestamp
}

This will require a new API endpoint from RDS bot that will provide the user details.

https://github.com/Real-Dev-Squad/discord-slash-commands/issues/66

Will require addition of a new package for image processing jimp
this feature will change the yarn.lock file.

{
    userId: string,
    discordId: string,
    discord: {
        url: string,
        approved: boolean,
        date: Timestamp
    }
    profile: {
        url: string,
        approved: boolean,
        date: Timestamp,
        publicId: string
    },
    status: `ACCEPTED` | `PENDING` | `REJECTED`
}

updated model for the #1924


Requirements for SMTP check

Hey, what are the server requirements to get smtp: { valid: true }?
I know on aws the outbound port 25 is not allowed.
Locally in Java I get a successful smtp check but with your tool, I always get smtp: { valid: false }

@sfarkas1988 Hi , could you please specify the issue more  , at which server its giving SMTP false every time

I do have a list of private email lists I can't share.
But the point is, I never get a successful smtp = true response.
Which might be the reason that the smtp check itself does not always work on all environments.
There should be some requirements for the server where I run the command like proper DNS setup so the responding SMTP server would accept the request. This is how I understand the whole SMTP thing.


Consistent storage account naming between dev and prod

Which issue this PR addresses:
I've had to re-open this PR a few times due to e2e failures resulting from invalid tenant. I've moved over to the main trunk repo and hopefully e2e will succeed.

What this PR does / why we need it:
https://msazure.visualstudio.com/AzureRedHatOpenShift/_git/ARO-Pipelines/pullrequest/9703384?_a=files changed the way that the prod storage accounts are named (using storageAccountDomain + "oic" rather than prefix + "oidc" + region). This PR is meant to align our dev storage accounts to that as much as possible so that they follow the same naming convention.
In the case of the shared dev environments (v4-eastus, v4-westeurope, etc) those will likely have to stay as-is using the resource group name since in a local or shared dev environment there is no storageAccountDomain or RP-config defined for those environments.

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

How do you know this will function as expected in production?

/azp run ci,e2e

/azp run e2e


Documentation or implementation bug. Custom ViewHolder cannot be cast to eu.davidea.viewholders.FlexibleViewHolder

Issue is correct for the current snapshot.
I'm dealing with sticky headers. Nowhere in docs mentioned, that instead of custom ViewHolder inheritance there should be FlexibleViewHolder used. And generic constraints of interfaces of provided abstract classes, like AbstractHeaderItem and AbstractSectionableItem expect to get classes inherited from RecyclerView.ViewHolder
However StickyHeaderHelper class's method getHeaderViewHolder(int position) have direct casting to FlexibleViewHeader and of course will failed if there is simple ViewHolders used with error similar to this one:
FATAL EXCEPTION: main
Process: test, PID: 29504
java.lang.ClassCastException: Fragments.HeaderViewHolder cannot be cast to eu.davidea.viewholders.FlexibleViewHolder
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.getHeaderViewHolder(StickyHeaderHelper.java:266)
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.updateHeader(StickyHeaderHelper.java:153)
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.updateOrClearHeader(StickyHeaderHelper.java:143)
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.initStickyHeadersHolder(StickyHeaderHelper.java:122)
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper.access$000(StickyHeaderHelper.java:40)
at eu.davidea.flexibleadapter.helpers.StickyHeaderHelper$1.run(StickyHeaderHelper.java:80)
at android.os.Handler.handleCallback(Handler.java:739)
 at android.os.Handler.dispatchMessage(Handler.java:95)
at android.os.Looper.loop(Looper.java:148)
 at android.app.ActivityThread.main(ActivityThread.java:5417)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)

So i think either it's need to fix docs and probably introduce new abstractions which will be statically checking correct typing, or fix the code to not assume it's use FlexibleViewHolder items.

Hello @ivankarpey, thank you for your message.
Indeed that is still not ok, I introduced this constraint as temporary fix for crashing bugs since 2-3 snapshots ago, to allow people continue to develop. Before next beta7, I have to find a final solution, that is good for everybody. I cannot put this in documentation because it is not closed yet.
However, thank you that you have spotted this, I'm going to fix this.
For documentation, the wiki page Headers and Sections clearly explain that sections are identified by IHeader and ISectionable interfaces.
New and netter docs will follow.

@ivankarpey, I refactored the item interfaces signatures to be compliant with StickyHeaderHelper.

Thanks. Looks like docs much clearer now. I'm plan to use flexible adapter pretty aggressively in my app and will try to provide some more improvements if will find any.

Got same exception when my header class extends AbstractHeaderItem and normal item extends AbstractSectionableItem. HELP IN NEED.
I do as the doc said, but still got a ClassCastException, with 5.0.0-b7

@jinsen47, can you check if getLayout() of both classes returns a different layout?
If not resolved, can you paste here the 2 items and the stacktrace you get?


Publish terrautils to pypi

We want the terrautils package to be published.
Completion criteria:

[x] Terrautils published to pypi
[x] Documentation describing how to do this in the future
[x] Review documentation with team members

@jterstriep hoping to finish this week

Terrautils has been uploaded to PyPI in both source and wheel distributions. Two version are currently available. Version 1.1.0 supports the terraref namespace so code would 'import terraref.terrautils', version 1.0.0 supports the original 'import terrautils'.
Installation automatically installs all dependencies listed in the setup.py. This is slightly different from the group's practice of placing all dependencies in requirements.txt. Setup.py was updated to include all
dependencies from requirements.txt. One additional dependency was pyclowder but this has been published to PyPI and terrautils now pulls it from PyPI.
One remaining dependency is GDAL which is left as an exercise to the user. We might need to add better guidance.
Publishing to PyPI is straightforward and well documented in python's packaging guide. Beware that
PyPI is currently transitioning from pypi.python.org to pypi.org. The two systems are linked but there are some changes which makes older documentation incorrect. The TL;DR versions is as follows:
python setup.py build sdist
python setup.py build bdist_wheel
twine upload -r pypi dist/*

A couple additional considerations. Publishing implies versions which are currently specified directly in setup.py. These should be matched with tags on GitHub. Lastly, I am the owner of the terrautils package. Other people should create PyPI accounts and I can add them as owners.

This introduces the terraref package space (1.1) so using pip install will need to use the new import.
pip install terrautils==1.0.0 will be the same as before.

tag & push current docker image as 1.0.0.
new version of docker becomes 1.1.0 and extractors building from it will require the new import syntax.

https://github.com/terraref/terrautils/pull/46

@robkooper Please add max-zilla as owner

follow on with: https://github.com/terraref/computing-pipeline/issues/429


Minor issue when filling out qso in lower case

When uploaded it doesn't change and display in upper case. same with callsign and sota ref

Yep - understood. Solution is to force UPPER() for callsigns and SOTA / POTA references. Will need something a little more slick for Maidenhead grids.

Should be fixed by that last commit. Pushed to live

All fixed doesn't matter now if summit qso in lower case


Added Cloudmersive PDF connector

This checklist is used to make sure that common guidelines for a pull request are followed.

[ X ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Added requested updates @dougelliott @sriyen-msft

Made requested updates @sriyen-msft


Request

Hi
Is there any possibility that we can use hijri calendar instead of Georgian Calendar
Regards
Adeel Arshad

you can adjust the localization used, but i think that probably stops working sort of using a different calendar, given how many assumptions i've probably made in the code about Georgian calendar things (like week length). I'm not really familiar with other cal types so it'd be hard for me to support them but i'd be open to a PR


Create the topics from the python code instead of the docker image

What does this PR do?

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

Current dependencies on/for this PR:

master

PR #13918

PR #14135

PR #14136   👈

PR #14137

PR #14103
PR #14095
PR #14094

PR #14105

PR #14080
PR #14078
PR #14077
PR #14076
PR #14079

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

master

PR #13918

PR #14135

PR #14136   👈

PR #14137

PR #14103
PR #14095
PR #14094

PR #14105

PR #14080
PR #14078
PR #14077
PR #14076
PR #14079

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

master

PR #13918

PR #14135

PR #14136   👈

PR #14137

PR #14103
PR #14095
PR #14094

PR #14105

PR #14080
PR #14078
PR #14077
PR #14076
PR #14079

This comment was auto-generated by Graphite.


BulkReadAsync:  SQLite Error 1: 'unknown database [dbo]'

Hello, I tried to run with this code a unit test code, with entity framework in which the table named "bottler" has a default schema (dbo). Try with BulkInsertAsync and other methods and it worked correctly with the default scheme, but with BulkReadAsync there are conflicts when performing the test
`
        var connection = new Microsoft.Data.Sqlite.SqliteConnection("DataSource=:memory:");
        connection.Open();

        var option = new DbContextOptionsBuilder<ABIDBwscContext>()
            .UseSqlite(connection)
            .EnableSensitiveDataLogging()

            .Options;
        var context = new ABIDBwscContext(option);
        if (context != null)
        {
            context.Database.EnsureDeleted();
            context.Database.EnsureCreated();
        }

        await context.AddAsync(new Bottler() { BottlerId = 1, BottlerNombre = "ANDINA" });
        await context.SaveChangesAsync();
        var entities = new List<Bottler>() { new Bottler() { BottlerId = 1 } };
        var ListConverted = entities.Select(x => new Bottler() { BottlerId = x.BottlerId, BottlerNombre = x.BottlerNombre }).ToList();
        await context.BulkReadAsync(ListConverted, new BulkConfig() { UpdateByProperties = new List<string> { nameof(Bottler.BottlerId) } });
        Assert.AreEqual("ANDINA", ListConverted.First().BottlerNombre);

`

The issue might be due to DataSource=:memory:
There is a note in ReadMe:
Bulk Tests can not have UseInMemoryDb because InMemoryProvider does not support Relational-specific methods.

Hello again,
I tried again assigning the default schema "main" to the efcore modelbuilding and everything worked without problems, it seems that assigning the value "dbo" in the EFcore configuration generates some kind of problem with sqlite(memory), but luckily changing to "main" the library works correctly with sqlite in memory
In other words:
 default_scheme = scheme == string.Empty? "dbo" : "main";
then
entity.ToTable("bottler", default_scheme);
in the context
and add the parameter in new context as optional:
var context = new ABIDBwscContext(option, "test");

Glad you solved it, will close the issue.
Just to Note that Sqlite does not support schemas like SqlServer, at least not multiple different ones.
There schema name is the name of an attached database.


Dodanie Paid

https://docs.spreecommerce.org/developer/customization/deface_overrides_tutorial#what-is-deface :)
We could always just copy-paste the erbs from the spree gem and customize but this is not following DRY


Fix Jupyter notebook header sizes

This PR fixes #141310

Closing since #141310 was closed by the author while we were waiting for feedback on the request. If the request gets enough interest we can reconsider this


Nick: Watch pdf417mrz.xyz

Nick requests the watch of the watch_keyword pdf417mrz\.xyz. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

pdf417mrz\.xyz has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Any way to achieve two level multi-tenancy?

Hello,
Thanks for implementing and supporting such a good gem we use in many of our projects!
We use the gem in our project to separate events such as fairs and conferences as tenants. They share users and have out-of-tenant configurations such as domains and routes, etc:

What we want is to add a completely new level of tenancy: vendors. Vendors should have their own separate scope of events, users and configurations:

As my experience with the activerecord-multi-tenant and common sense I see "Variant 2" as a viable option, where I would have two multi_tenant tables: events and vendors. On the contrary, events still should be dependent on vendors and it can mix up some stuff with database queries and the MultiTenant.current_tenant state.
"Variant 1" seems to me more logical and readable but I have no idea where to start with modifying the gem to make it work. I suppose it is doable to make it possible to pass the tenant as an array of a vendor and an event model as so: MultiTenant.with([current_vendor, current_event]) but I wanted to ask you, the authors, what you think about it.

Hey @vit-panchuk
I can not see any difference between Variant 1 and Variant2. Could you explain the difference? And could you give me some examples in code blocks?
Thanks

@gurkanindibay, thx for the quick response!
Variant 1 would be considered something like that:
# vendors: id, name, etc
class Vendor  < ApplicationRecord
end

# users: id, name, vendor_id, etc
class User < ApplicationRecord
  multi_tenant :vendor
end

# urls: id, domain, route, vendor_id, etc
Class Url < ApplicationRecord
  multi_tenant :vendor
  has_one :event
end

# events: id, name, vendor_id, etc
class Event < ApplicationRecord
   multi_tenant :vendor
   belongs_to :url
end

# exhibitors: id, name, event_id, vendor_id, etc
class Exhibitor < ApplicationRecord
  multi_tenant :vendor, :event # obviously, pseudocode
end

The Event model itself would have multi_tenant :vendor.
The User model has only multi_tenant :vendor because users are shared among events of the same vendor but are not shared among vendors.
All models inside the event tenant (e.g. Exhibitor, SideEvent, Product) would need to have both event_id (already implemented) and vendor_id (needs to be added) references and they would know they are inside an event and inside a vendor.
Obviously, if I decide to go this way, MultiTenant.current_tenant and other MultiTenant API should be modified to recognize the hierarchy of tenants (vendors > events).
Variant 2:
# vendors: id, name, etc
class Vendor  < ApplicationRecord
end

# users: id, name, vendor_id, etc
class User < ApplicationRecord
  multi_tenant :vendor
end

# urls: id, domain, route, vendor_id, etc
Class Url < ApplicationRecord
  multi_tenant :vendor
  has_one :event
end

# events: id, name, vendor_id, etc
class Event < ApplicationRecord
   belongs_to :vendor # or maybe I can even use `multi_tenant :vendor` here?
   belongs_to :url
end

# exhibitors: id, name, event_id, etc
class Exhibitor < ApplicationRecord
  multi_tenant :event
end

This way I am getting two federated "islands" of tenants that know nothing about each other existence (e.g. Exhibitor model does not have a direct reference to a vendor but it is still inside an event). No MultiTenant modifications, etc, but I am not sure it actually possible to track two MultiTenant.current_tenant states simultaneously (one for current_vendor and another for current_event).

So you meant it actually possible to have something like this?:
# vendors: id, name, etc
class Vendor < ApplicationRecord
end

# events: id, name, vendor_id, etc
class Event < ApplicationRecord
   multi_tenant :vendor
end

# exhibitors: id, name, event_id, vendor_id, etc
class Exhibitor < ApplicationRecord
  multi_tenant :vendor
  multi_tenant :event
end


Fastnlight: Blacklist<PHONE_NUMBER>6

Fastnlight requests the blacklist of the number<PHONE_NUMBER>6. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Spevacus in Charcoal HQ


[Feature]: extensionAlias support

🚀 Feature Proposal
Similar to webpack/rpsack extensionAlias
https://webpack.js.org/configuration/resolve/#resolveextensionalias
https://github.com/webpack/webpack/issues/13252#issuecomment-1824282100
Be able to resolve import "myFile.js" to myFile.ts, to enable ESM
Motivation
ESM requires file extensions, so we need to import using .js, but this breaks jest that it is looking for .ts file instead
Example
import "myFile.js" to myFile.ts, to enable ESM
Pitch
solve another ESM problem and make module resolution faster
/** @type {import("next").NextConfig} */
const nextConfig = {
  experimental: {
    // Remove .js from import specifiers, because
    // Next.js and webpack do not yet support
    // TypeScript-style module resolution out of the box
    // https://github.com/webpack/webpack/issues/13252#issuecomment-1171080020
    // https://github.com/vercel/next.js/pull/45423
    // https://github.com/vercel/next.js/issues/58805
    // https://github.com/vercel/next.js/issues/54550
    extensionAlias: {
      '.js': ['.ts', '.tsx', '.js', '.jsx'],
      '.jsx': ['.tsx', '.jsx'],
      '.mjs': ['.mts', '.mjs'],
      '.cjs': ['.cts', '.cjs'],
    },
  },
}

Hey @sibelius ! 👋 Long time no see, hope everything's good with you.
Considering that part of the TypeScript 5.7+ ecosystem is moving to fully-specified import specifiers with *.ts* extensions (motivated by Node.js type stripping), would allowImportingTsExtensions from TypeScript 5.0 with .ts file extensions work for you?
I wrote a bit in a similar issue over here:

https://github.com/vercel/next.js/issues/69426#issuecomment-2407789177


Enable markdown in project descriptions

[x] render markdown

[x] style headings appropriately, or demote headings

[x] update documentation

:tada: This issue has been resolved in version 8.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


pcluster createami should point out that Berks is missing

When the local ChefDK installation is not properly setup (i.e when a user installs chefdk via Homebrew rbenv-chefdk formula), the following happens:
$ pcluster createami --ami-id ami-09226b689a5d43824 --os alinux2 --config conf/config --region ap-southeast-2 --cluster-template tothill
Building AWS ParallelCluster AMI. This could take a while...
Base AMI ID: ami-09226b689a5d43824
Base AMI OS: alinux2
Instance Type: t2.xlarge
Region: ap-southeast-2
VPC ID: vpc-7d2b2e1a
Subnet ID: subnet-3ad03d5c
Template: https://s3.ap-southeast-2.amazonaws.com/ap-southeast-2-aws-parallelcluster/templates/aws-parallelcluster-2.6.1.cfn.json
Cookbook: https://s3.ap-southeast-2.amazonaws.com/ap-southeast-2-aws-parallelcluster/cookbooks/aws-parallelcluster-cookbook-2.6.1.tgz
Packer log: /var/folders/wz/__dd9trs0kl4jb3rs83704y80000gn/T/packer.log.20200427-144543.zn8dxbyo
Packer status: 	exit code 1

No custom AMI created

It makes it seem as if there was some sort of packer issue, when in reality it's berks/chefdk at fault... the error message should point (earlier?) that Berks is missing, not Packer.
Environment:

OSX Catalina
aws-parallelcluster-2.6.1
parallelcluster config: https://github.com/umccr/infrastructure/blob/master/parallel_cluster/conf/config

/cc @reisingerf

Thank you for reporting this. I marked it as enhancement in our backlog.

Closing this because Berkshelf has been removed from the client requirements since version 2.8.0.
Reference https://github.com/aws/aws-parallelcluster/blob/develop/CHANGELOG.rst#280

Great!


fixing SNOWMED to SNOMED in docs

Coverage remained the same at 94.397% when pulling 9ec5cbab98cc32583e2f63af1e8bdeef67d8e5dd on goelalex:patch-1 into 11b47989a2240d13ac4cc380b378a4b4118bc73e on medplum:main.


It does nothing after Enter

I've run base on your guide at README, but after 1 hour, nothing happened except the line could not detect version_main.therefore, we are assuming it is chrome 108 or higher.
This is my command: poetry run python bookphucker https://bookwalker.jp/de99571adf-73d3-4b0a-9384-08c8ecbcc189/
Then, it occurs and nothing else comes after 1 hour:

I've pressed Ctrl + C to break and this is:

I've try to download chromedriver 114 win 32 from this link and put the chromedriver.exe at the same folder with config.json, but the error is not solved.
The line could not detect version_main.therefore, we are assuming it is chrome 108 or higher is exactly like your example image. So must i wait for a longer time than 1 hour ?

That's definitely abnormal, I've just tested this script and it's still working for me, so I cannot reproduce your issue.
Please set headless in your config file to false to examine where it gets stuck.
That may also be an upstream issue, perhaps you can try ubuntu subsystems in Microsoft app store.
Normally the script should've at least thrown a timeout error instead of wasting an hour of yours, i'm sorry for your loss of time🫠.

Set headless = false has solved my issue. But it can't download. Must i buy that light novel before downloading ? Cuz there is a image error.png appear in my folder.


Getting started instructions

Hello, I'm following the thread at https://github.com/dotnet/fsharp/issues/14527.
Very cool to see that somebody is working on this.
I think it would be beneficial to have a "getting started" file in this repository.
How can people play around with this? How can they contribute? How do I run the tests?
More contributors could be drawn in if this information was easily accessible.

Hi @nojaf,
Thank you for commenting, I think what your suggesting sounds like a good idea!
I will try and get something along the lines of a getting started section once I get the time.


Android targeting API Level 30 needs permission

Hi!
My App with targetSdkVersion=30 crashed on Android 11 calling shareViaEmail with java.lang.ArrayIndexOutOfBoundsException
I think the reason is that you now need permission to have full access to the list of all installed apps:
https://medium.com/androiddevelopers/package-visibility-in-android-11-cc857f221cd9
Solution:
Add the following to AndroidManifest.xml
<uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

or -

Adding following permission to config.xml to generate AndroidManifest.xml with this permission.
<config-file parent="/manifest" target="AndroidManifest.xml" xmlns:android="http://schemas.android.com/apk/res/android">
   <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />
</config-file>

I hope this helps someone,
GTM

thanks. Exception for google Search..
java.lang.ArrayIndexOutOfBoundsException:
at java.util.ArrayList.remove (ArrayList.java:506)
at nl.xservices.plugins.SocialSharing$1.run (SocialSharing.java:199)
at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1167)
at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:641)
at java.lang.Thread.run (Thread.java:923)

Yes, this solve the issue on Android 11
AndroidManifest.xml
<uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

I'm facing the same issue.
Thanks for posting the solution here!
The bug is basically that the list is empty and trying to remove an item from an empty list causes the app to crash.
I don't mind sending a PR to either check for an empty list and raise a toast or add the relevant permission to the manifest.
This plugin serves me really well and I'd like to contribute back a bit :-)

Hi, when I added the <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" /> the app now just crashes on startup

@chrisjdev when this fix will be released for cordova? https://www.npmjs.com/package/cordova-plugin-x-socialsharing

@chrisjdev when this fix will be released for cordova? https://www.npmjs.com/package/cordova-plugin-x-socialsharing

Sorry, in my comment referencing this issue, I was saying my change didn't completely solve the issue, so I wasn't planning to do a pull request. Also, for other reasons, I'm planning to switch to using share() instead of shareViaEmail(), so I likely won't spend any more time on this.

Also worth mentioning that the solution to add a permission requires adding an explanation in the google play store as can be read here:
https://support.google.com/googleplay/android-developer/answer/9888170
This is the solution I went with as it didn't need any change in the plugin code to fix this.
I'm no longer sure, due to the above google restriction, that this is the right solution.
Having said that, I'm not sure what is the right solution in this case...

@HarelM Yes, setting global permission is therefore no longer a good option.
I have struggled through this (and ended up with  @bhandaribhuminpfizer solution):

Remove <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

add to config.xml:

 <config-file parent="/manifest" target="AndroidManifest.xml">
            <queries>
                <intent>
                    <action android:name="android.intent.action.SENDTO" />
                    <data android:scheme="mailto" />
                </intent>
            </queries>
        </config-file>

Remove android platform and add it again to generate AndroidManifest.xml correctly:

cordova platform remove android
cordova platform add android

Tested with corova-android 9.1.0 on Android 11 and Android 12
Feedback is welcome

@GloriaTM thanks for the info!!
I've tested this and it seems to work as expected in android 12 but doesn't open the mail client or the ability to select in Android 11 (one plus device). It doesn't crash as before though too.
Not sure what to do here though as the Android 11 device is not mine to test with...
I'll see if I can reproduce this in the simulator...

My mistake, everything works as expected, I've updated my previous comment.
Thanks for providing the solution!

In my case this works:
<platform name="android">
  ...
    <config-file parent="/manifest" target="AndroidManifest.xml" xmlns:android="http://schemas.android.com/apk/res/android">
        <queries>
            <intent>
                <action android:name="android.intent.action.SENDTO" />
                <data android:scheme="mailto" />
            </intent>
        </queries>
    </config-file>
</platform>

For me, I just removed the namespace tags, and it worked:
           <config-file parent="/manifest" target="AndroidManifest.xml">
                <queries>
                    <intent>
                        <action name="android.intent.action.SENDTO" />
                       <data scheme="mailto" />
                   </intent>
               </queries>
        </config-file>

To sharing on Fb and email with Android 12, i need to add fb package
<config-file parent="/manifest" target="AndroidManifest.xml">                 <queries>                     <intent>                         <action name="android.intent.action.SENDTO" />                        <data scheme="mailto" />                    </intent>                    <package android:name="com.facebook.katana" />                </queries>         </config-file>

Ionic/Angular/Capacitor app:
\android\app\src\main\AndroidManifest.xml
I have removed <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />
and added this:
<queries>
           <intent>
            <action android:name="android.intent.action.SENDTO" />
            <data android:scheme="mailto" />
        </intent>
    </queries>

Still, it is not working in android 12 and above devices. app is getting crashed

I have a working app targeted to Android 13.
I don't know if I'm doing something different, but it's working for me.
If it helps, the following is my repo:
https://github.com/IsraelHikingMap/Site/tree/3997912dc56df39cd84a6100c06241364eab7d05/IsraelHiking.Web

Thanks for the quick response. Currently, I am using angular 12.1.4 and cordova -11. It was working fine in ios. In android, greater than 11 android versions getting crashed.
this.socialSharing.shareViaEmail(body, subject, []).then(()=>{
});
app was closing automatically after calling then()

Did you follow the above suggestion?
https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/issues/1150#issuecomment-1114292916
Here's my commit that solved this:
https://github.com/IsraelHikingMap/Site/commit/bb9502865d6dc73b63f86eb9da244fa44f1c24fe
I believed this crashed for me too before this fix.

I tried to update social-sharing plugin -6.3.0 and also followed the above procedure which you suggested.Still, no luck in result.

Hi!
My App with targetSdkVersion=30 crashed on Android 11 calling shareViaEmail with java.lang.ArrayIndexOutOfBoundsException
I think the reason is that you now need permission to have full access to the list of all installed apps: https://medium.com/androiddevelopers/package-visibility-in-android-11-cc857f221cd9
Solution: Add the following to AndroidManifest.xml <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

or -

Adding following permission to config.xml to generate AndroidManifest.xml with this permission.
<config-file parent="/manifest" target="AndroidManifest.xml" xmlns:android="http://schemas.android.com/apk/res/android">
   <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />
</config-file>

I hope this helps someone, GTM

If you add this persmission in Manifest file, app may got rejected due to the play console policy.
Please refer below link for complete fixes...
https://stackoverflow.com/a/73151052

Hi!
My App with targetSdkVersion=30 crashed on Android 11 calling shareViaEmail with java.lang.ArrayIndexOutOfBoundsException
I think the reason is that you now need permission to have full access to the list of all installed apps: https://medium.com/androiddevelopers/package-visibility-in-android-11-cc857f221cd9
Solution: Add the following to AndroidManifest.xml <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />

or -

Adding following permission to config.xml to generate AndroidManifest.xml with this permission.
<config-file parent="/manifest" target="AndroidManifest.xml" xmlns:android="http://schemas.android.com/apk/res/android">
   <uses-permission android:name="android.permission.QUERY_ALL_PACKAGES" />
</config-file>

I hope this helps someone, GTM

Thanks it helped me


Add Microbiology and Bioinformatics feed

Couldn't properly test it yet due to issues with the EuropePMC API.


feat: defi earn grouping blended APY

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3988   👈

This comment was auto-generated by Graphite.


Adding similar functionalities as in UltraEdit

added winattr for when copying files the copies should keep the attributes from original

when creating copy of a file take the time stamp from original

added 'maxVersionCount' which adds the ability to set the maximum amount of copies

added 'saveOneStepAhead' for setting if the current stand of the file should be versioned or only the previous versions

'filename' adds a new way to version the files (similar to UltraEdit) and 'filenamePattern' limits the files which should be versioned by this pattern

Sorry, there were some issues what i noticed at the last moment.
https://github.com/zabel-xyz/local-history/pull/45 is the new pull request.


Drawer causes screen shift when content overflows vertically

link:
https://codesandbox.io/p/sandbox/drawer-with-scale-forked-6zc9pz
the screen shifts when drawer is opened and when drawer is closed (the opening animation hides it because the background scales backwards, much more visible on drawer close). The issue is there whether shouldScaleBackground is enabled or disabled

i get the same issue

This is fixed in #406


Using Bellman Ford Algorithm

Is your feature request related to a problem? Please describe.
In a realistic scenario it is seen that there can be delay in traffic or a sudden emptiness of roads(smaller lanes to the main road) which can make the speed of the car vary greatly. Hence it is necessary to be able to deal with negative weights and the detection of negative cycles.
Describe the solution you'd like
Instead of Dijkstra's algorithm can Bellman Ford algorithm be applied.
Additional changes
Make necessary changes in the Propogation of Updates also and comment on the efficiency by adding to the documentation.

can i do this?


add 64 bit arm support

Starting August 1, 2019:
All new apps and app updates that include native code are required to provide 64-bit versions in addition to 32-bit versions when publishing to Google Play.

added arm64-v8a support in native-reflow branch

merged into master


Restore integration tests

This is a version of the integration tests that runs and passes on TravisCI. I achieved this by using travis to encrypt the credentials.yml file with the username and password for a service account in my Okta dev instance. It has a few issues and I'm interested in feedback on how to address them.

It has an okta metadata.xml file directly included in the repository, which will only work with the credentials in the encrypted credentials.yml file
It also has a URL to the same metadata file for the HttpMetadata test

Having these direct ties to a single account right in the repo seems like it could be confusing to people. I could encrypt these as well, but that seems like overkill. Any thoughts on other ways to make this more clear?

Another issue I see with this is that it will not build on Bamboo which is used to deploy to the Spring Snapshot repository.
Rather than a yml file couldn't we use system properties or environment variables to specify the credentials?

Yes, we can move those into environment variables. Mark K and I will start on that now. It looks like there is also some flakiness in at least one of the tests which should be addressed.

We extracted those into environment variables, but they're still using Travis' encryption utilities. There may still be work to achieve Bamboo integration.
I like the idea of somehow using a public account for this, and not even needing to worry about encryption. However I'm not sure how possible it is with Okta. There may be other IDPs that support some sort of test account.

Closed via: https://github.com/spring-projects/spring-security-saml-dsl/commit/8ce2845119c06b56013cdbc1129d3ab183bee710


help me out,i see this while running my projects

Gradle sync failed: Unexpected lock protocol found in lock file. Expected 3, found 0.
			Consult IDE log for more details (Help | Show Log)

cc @mit-mit @devoncarew

@wivvv can you clarify when you see this? I.e., some repro steps would be nice. Also please include the output of flutter doctor

Without specific information on how or when this happens, we're not able to act on it, so I'm going to close this issue. Please don't hesitate to file a new bug if you still see this.


[TDX-2195] Dev portal app reg with multi-runtime group support

Summary

The feature this documentation is being updated for is still behind a feature flag and has not been rolled out to customers yet, so this PR is not ready to merge.
Reason

https://konghq.atlassian.net/browse/TDX-2195
Application registration is opening support for any runtime group. Only compatible with Kong Gateway versions >= 3.0
Testing

@Mierenga Will you be adding additional changes or is this ready for a copy edit? Thanks!

@cloudjumpercat There will be additional changes in the next couple weeks before it is ready for docs review, thanks for checking in!

@Mierenga Hi! Just checking in to see if you have any more changes for this PR. Thanks!

@cloudjumpercat This info is ready for review from the docs team, thanks!

I'm pushing this to a feature branch with this PR
https://github.com/Kong/docs.konghq.com/pull/4716
So they can both go out together on Monday


[GAL-180] Mint poster

Added a mint poster page.
Basically, reuse the previous poster page and most of the mint function from the membership card page.
The page should inherit all the transaction states, errors, and transaction hash from the mint card page.
Notes

Since most of the mint method is redundant, I moved some of the mint function into their own hooks. Just in case, if we want to refactor all the mint methods from one hook (It can be another PR)
I didn't touch anything in mint membership; there should be any membership regression issue on this PR.
The token ID will be set for 2 right now for testing purpose while waiting for the smart contract updates

Screenshot

Banner

Navigation

Mint page

Desktop

Mobile

Error State

Desktop

Mobile

Success State

Transaction hash

Transaction hash + Success state

Video
https://share.cleanshot.com/1UXl52

thanks so much for the PR, jakz!! in the future do you mind adding some screenshots for the feature? example:

pic of new banner
pic of new mint page
mobile vs. desktop view

Thanks for addressing @jakzaizzat ! looks great.
I pushed a couple of small changes, just a heads up:

updated some copy
disabled mint if feature flag is off (since the poster page url is already public and accessible, if the flag is off users will see a "mint opening soon" message instead.
immediately hide notification dot when I click the mint page link in the dropdown
don't show notification dot next to username if feature flag is off

I think it's good to go, and we can finalize + deploy the contract and prepare the token in the meantime.

Thanks @kaitoo1 for fixing it! I totally missed it 🤦

all good! just some small polish items I noticed

@kaitoo1 Just small update, i removed the limit 1 poster message since we already filter it on the initial load page.
Plus, the error code UNPREDICTABLE_GAS_LIMIT not 100% refer to amount of mint

@kaitoo1 Just small update, i removed the limit 1 poster message since we already filter it on the initial load page.
Plus, the error code UNPREDICTABLE_GAS_LIMIT not 100% refer to amount of mint

ok, sounds good that makes sense!  thanks for the update.


Less than 1 epoch training duration

Could you rebase and then I think we can merge this?

@limiteinductive rebase done