[url_launcher] for web should support the tel and sms scheme in PWA

Use case
The current plugin supports http,https,mailto
const _mailtoScheme = 'mailto';

static final _supportedSchemes = {'http', 'https', _mailtoScheme};

The safari browser in the iPhone can be executed normally using the following code
  _launchURL() async {
    var url = 'tel:+1-408-555-5555';
    //if (await canLaunch(url)) {
      await launch(url);
    //} else {
      //throw 'Could not launch $url';
    //}
  }

Proposal
Add support for the tel and sms scheme links in url_launcher_web, For use in pwa.

/cc @ditman @mdebbar

I think this is a very good first contribution. @AngusPei, would you be interested in contributing a fix for the plugin?
There's some special handling needed for non http/https URLs (like mailto) in PWA, so all you'd need to do is refactor _mailtoScheme to a Set of _specialSchemes that need that handling.

platform_detect doesn't seem to have an implementation to detect iOS and Android for web and it doesn't seem like it's going to be added anytime soon, there might be a need to fork and add it, https://github.com/Workiva/platform_detect/issues/41 @ditman @Vanethos

@twisstosin why did you need to detect iOS vs Android? Just to enable the support when on mobile?

yeah @ditman, is there a possibility of tel and sms working for desktop?

@twisstosin, I'm not sure if there's a built-in tel/sms handler for desktop, but for example Skype used to handle tel links:
https://mycyberuniverse.com/create-html-link-that-starts-skype-call.html
(Not sure if this is true anymore). It is impossible to detect from the browser if there's any protocol handler registered, so you need to let the user click on it at least.

This feature is now published, on url_launcher_web 0.1.2


Fixed likes retriving bug

Instagram is not showing likes if one is not logged in. Thus we cannot just try to get number of likes under the photo and convert it to int, we need to test if we can do it by using try, except error handling. Otherwise whole program throws error.
We will need to discuss if we want to allow user log in option for purpose of retrieving likes. If someone needs to get likes, I wrote how to do it by using cookie here: https://github.com/timgrossmann/instagram-profilecrawl/issues/108

@justdvl Awesome, thank you very much.
I think login, at some point, might be needed.
However, crawling a profile is not an invisible task so I don't know how suitable that is.
It definitely should be an option.


[BEAM-2060] Add charset support in XmlSink

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[X] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[X] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[X] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[X] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @lukecwik

LGTM
Made some minor whitespace changes in a test and comment fixes and merged.


Add txredisapi to matrix-synapse dependencies

Motivation for this change
When using workers with synapse txredisapi is required.
Things done
I've been using basically this for half a year+ with a module with worker support.
I'm not sure if it makes sense at all to do the unit testing like this for the txredisapi package. If someone has an opinion on keeping/improving/or removing I'm all ears.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 128048 at 44a982a0 run on aarch64-linux 1

2 packages built successfully:

python38Packages.txredisapi
python39Packages.txredisapi


Use or delete unused nvm counters

From what I can tell, c_vbus_rst, c_deploy, c_downlink, c_logfail are all unused. We should use them or delete them.

delete c_vbus_rst and make c_state_err 8 bits

rename c_state_err to be c_software_error

Use c_downlink and create a c_uplink to count the number of packets sent/received.


Value 0 for argument GroupBy is not supported for POST to "Test History" ?

When I post this JSON: {"automatedTestName": "MyAwesomeTest1"} to URL: https://{accountName}.visualstudio.com/{project}/_apis/test/Results/testhistory?api-version=5.0-preview.1
I get the following response:
{
"$id": "1",
"innerException": null,
"message": "Value 0 for argument GroupBy is not supported.",
"typeName": "Microsoft.TeamFoundation.TestManagement.WebApi.InvalidPropertyException, Microsoft.TeamFoundation.TestManagement.WebApi",
"typeKey": "InvalidPropertyException",
"errorCode": 0,
"eventId": 3000
}
I do add media type request header: "application/json".
What am I missing? GroupBy is mandatory? According documentation none of the fields in the request body are mandatory.

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

ID: 82433bb4-2923-67b8-26d7-221a23b0afd6
Version Independent ID: 306a39a0-9d1e-6610-8aea-e7e54f73e77a
Content: Test History (VSTS Test)
Content Source: docs-ref-autogen/5.0/test/Test%20History.yml
Product: devops
GitHub Login: @wnjenkin
Microsoft Alias: whjenkin

@psoszynski  It seems like an issue with documentation. I am working with the team who handle document generation part to fix this.
meanwhile automatedTestName and GroupBy are required field.
Your json should be
{
"automatedTestName": "MyAwesomeTest1",
"GroupBy": "1"  // If you want result group by branches and "GroupBy": "2" to get results group by environment.
}
Please do let me know if still face the issue or have any concern or query.

Hei,
thx, that was a little better now. But still the response is missing data:
{
"automatedTestName": "MyAwesomeTest",
"maxCompleteDate": "2018-08-19T20:12:22.397554Z",
"trendDays": 7,
"groupBy": "branch",
"continuationToken": "",
"resultsForGroup": []
}
I guess that ResultsForGroup should not be an empty array, right?
regards, Przemek
pt., 10 sie 2018 o 08:49 Faizan2304<EMAIL_ADDRESS>napisał(a):

@psoszynski https://github.com/psoszynski It seems like an issue with
documentation. I am working with the team who handle document generation
part to fix this.
meanwhile automatedTestName and GroupBy are required field.
Your json should be
{
"automatedTestName": "MyAwesomeTest1",
"GroupBy": "1" // If you want result group by branches and "GroupBy": "2"
to get results group by environment.
}
Please do let me know if still face the issue or have any concern or query.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/vsts-rest-api-specs/issues/71#issuecomment-411992049,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6hs4EnpEX1hbjYTGLoNYFqJYkEj1E0ks5uPSzvgaJpZM4Vk8jX
.

Hi,
I think that ***/test/Runs should also return fully qualified test names
(in stead of simply test names in "testcase/name"). That would help in
automating the process for getting test histories. Right now, I need to
parse the source files to retrieve the fully qualified names. If test runs
returned qualified names of tests the parsing og source files would not be
necessary.
regards,
Przemek
wt., 9 paź 2018 o 09:26 Przemysław Soszyński<EMAIL_ADDRESS>napisał(a):

Hi,
now it works. Thanks.
Przemek
wt., 9 paź 2018 o 07:19 Faizan Ahmad<EMAIL_ADDRESS>napisał(a):

Hi @psoszynski https://github.com/psoszynski, First of all, apologies
for the late reply. The post slipped through the cracks.
Did you try with FullyQualifiedName in automatedTestName. I mean
NameSpace.ClassName.MyAwesomeTest.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/vsts-rest-api-specs/issues/71#issuecomment-428063158,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6hs2ldEzpCIZeafHUQEGSElR-l6jAaks5ujDHhgaJpZM4Vk8jX
.

Hi,
You can use automatedTestName. It is the fully qualified name of the test case.
Thanks,
Faizan


Where are the PL/SQL communities and enthusiasts?

As we move towards the launch of the new version of Exercism we are going to be ramping up on actively recruiting people to help provide feedback.
Our goal is to get to 100%: everyone who submits a solution and wants feedback should get feedback. Good feedback. You can read more about this aspect of the new site here: http://mentoring.exercism.io/
To do this, we're going to need a lot more information about where we can find language enthusiasts.

Is PL/SQL supported by one or more large organizations?
Does PL/SQL have an official community manager?
Do you know of specific communities (online or offline) that are enthusiastic about PL/SQL? (Chat communities, forums, meetups, student clubs, etc)
Are there popular conferences for PL/SQL? (If so, what are some examples?)
Are there any organizations who are targeted specifically at getting certain subgroups or demographics interested in PL/SQL? (e.g. kids, teenagers, career changers, people belonging to various groups that are typically underrepresented in tech?)
Are there specific groups or programs dedicated to mentoring people in PL/SQL?
Are there popular newsletters for PL/SQL?
Is PL/SQL taught at programming bootcamps? (If so, what are some examples?)
Is PL/SQL taught at universities? (If so, what are some examples?)

In other words: where do people care a lot and/or know a lot about PL/SQL?
This is part of the project being tracked in https://github.com/exercism/meta/issues/103

Oracle, obviously.
However, there is also a mildly divergent but highly effective implementation supported by PostgreSQL (called PL/pgSQL) and MariaDB (called Stored Procedures, but they also offer an Oracle compatibility mode), which are open source. Given that association with Oracle seems to be an active turnoff for engagement, according to this remark, it might be wise to inquire with one of those organizations about structuring an SQL track around their offerings.
PL/SQL -> PL/pgSQL seems like it would offer the least friction, and especially be fruitful, given that it is a highly popular open source database. However, there are still other implementations of what might better be called PL/SQM, which is a standard based on Oracle's PL/SQL. Obviously, studying PL/SQL seems like it would give one a leg up with any SQL/PSM implementation, but new students may not understand that at all.


Fixed #57: Incorrect EOF test in getMessages

It's possible to send a Logoff action. When this happens, Asterisk sends a response message and then close the connection. With this PR is possible to read the Logoff response message without throwing an exception if it's the last message and the connection is closed after that.
Also related to #121, because it's a good practice to send a logoff to disconnect, but sending it could throws an exception at this moment. This PR fix this.
Some little fixes on the code found debugging it.

Coverage remained the same at 91.932% when pulling 6cbc707e0d6b3dc7dbb999c4204ed4a31f86b4e1 on thomasvargiu:hotfix/57-read-and-eof into 61e58601c08731f754d21f5077000e034492f587 on marcelog:master.


examples: add simple work_item example module

This is a simple mostly empty example of a module using the PX4 work queue and running at a fixed rate. I was using something similar for probing system performance in a few areas and thought it might be potentially useful for others.

@bkueng Did you guys plan on creating any additional documentation? Minimally something linked from or a peer of http://dev.px4.io/master/en/apps/module_template.html

@hamishwillee thanks for raising this, and yes we should. Essentially it's the same thing, just running on a work queue. Do you want to add it quickly or should I write something?

@bkueng Probably best if you do it, since I still don't know anything about Work Queues - ie I don't know what is important and what is just an accident of the example. Hopefully I'll learn enough from the review to do something useful in future.


Multi-Account Support - find areas where OrgID must be added to filter criteria

IDPE code was written with an assumption that every user belonged to one and only one organization. For users that belong to more than one organizaion there places where the responses need to be filtered by organization id, else the caller will get objects that belong to more organization. Relying on the UI to always have a "org scoped" request does not control for non-UI users getting data that is not scoped to the org represented by their token

Created a bunch of grace tests to test the API for a user that belongs to multiple orgs.

Creation of dbrps looks like it creates the first dbprs across orgs, and but creates the second one only in the second org (org1 and org3 are the 2 orgs being used here)

await helpers.addDbrps(helperUrl, userToken1Org1, orgID1, org1BucketID, dbName1, rpName, true)
 await helpers.addDbrps(helperUrl, userToken1Org3, orgID3, org3BucketID, dbName3, rpName, true)

org 1 id 63527b3bb3359bd7
{
  id: '087e26f2e1776000',
  database: 'db1',
  retention_policy: 'rp',
  default: true,
  orgID: '63527b3bb3359bd7',
  bucketID: 'c9a8345244b8402f'
}

org 3 id 63527b3bb3359bd9
{
  id: '087e26f305b76000',
  database: 'db3',
  retention_policy: 'rp',
  default: true,
  orgID: '63527b3bb3359bd9',
  bucketID: '5f7e7ca89e40e34a'
}
{
  id: '087e26f41bb4d000',
  database: 'db1',
  retention_policy: 'autogen',
  default: true,
  orgID: '63527b3bb3359bd9',
  bucketID: 'e42f14d373dba4c0'
}

"e42f14d373dba4c0","orgID":"63527b3bb3359bd9","type":0,"name":"db1/autogen","description":"Auto-created from v1

is being auto generated in a v1 write test

The process for autogenerating a bucket and DB/RP is documented in this original issue under the section Autogenerating Buckets:
https://github.com/influxdata/idpe/issues/6948
Note specifically that the feature does not account for a user that is a member of multiple organisations. Can a user specify their default organisation? If not, how with this workflow choose a default org that a bucket and DB/RP is created for? Does the auth token contain authorisation to access multiple organisations or just a single?

@stuartcarnie @StacieClark  this isn't an OSS issue, right? Can we close this out here and move to idpe?


UseSharedCompilation likely causes a slower build

I saw the message

Using shared compilation with compiler from directory: /home/sam/.cache/bazel/_bazel_sam/c0ab60a4963be8b657cd35acdb19a608/external/dotnet_sdk/sdk/3.1.100/Roslyn/bincore

From the binlog on az ubuntu when a build failed.
This led to this SO post:
https://stackoverflow.com/questions/47590068/what-means-the-using-shared-compilation-with-compiler-from-directory-messag
Setting UseSharedCompilation to false got the build to succeed, but apparently slows down the build. As such, this issue can be closed, and a new optimization issue can be opened.

What's happening is the MSBuild property UseSharedCompilation is being set to false. As such we are not using the compiler server and you're paying the JIT cost for CSC on every build. That is the reason for the slow down.

Originally posted by @samhowes in https://github.com/samhowes/my_rules_dotnet/issues/33#issuecomment-809529789

The csc build process that gets started appears to not have permissions to write the the desired directories in the sandboxes on linux:
CSC : error CS2012: Cannot open '/home/vsts/.cache/bazel/_bazel_vsts/5c27df7f718ab35828b0e0cfe0993663/sandbox/linux-sandbox/62/execroot/rules_msbuild/bazel-out/k8-fastbuild/bin/tests/examples/Grpc/Protos/obj/netcoreapp3.1/Protos.Net.dll' for writing -- 'Read-only file system'

I think [persistent workers](https://docs.bazel.build/versions/main/persistent-workers.html_ is actually the intended solution for situations like this


Jakarta Support for SpringBootStarter

Hello, I would love to use JTE in my new project with Spring Boot 3, but JTE (jte-spring-boot-starter) 2.2.4 only supports javax as I can see.
When will jakarta namespace be supported?

Welcome @schowave! The javax.servlet-api dependency is an optional dependency and as far as I looked not used by any of the jte spring boot starter classes. I'm not that involved in Spring, but all new projects would probably rather use Spring Boot 3 over Spring Boot 2.
@atomfrede should we bump the spring boot version of jte starter to 3?

Yes I didn't had time to do it yet. Should we have one for sb 2 and one for sb 3 for some time?

That's a good idea!

I'll try to start with updating the current starter to latest 2.7.x release first and afterwards doing another artifact for boot 3.

Aww, thank you so much @atomfrede! That's a good plan!

Regarding the naming of the starter, what would you say is the "best" way?

sb3 gets a new name (e.g. jte-spring-boot-starter-3)
current starter is renamed to jte-spring-boot-starter-2
we use a different version schema (e.g. 3.XXXX) to indicate the compatible sb version

I tend to both option 1 and 2 and make it explicit, like this:
jte-spring-boot-starter-2
jte-spring-boot-starter-3
Which is probably the least confusing for new and existing users. When existing users upgrade, they will notice the missing spring-boot-starter and can then decide if they want to stick to 2, or upgrade to 3.
What do you think?

Sounds good 👍

Two different spring-boot-starter versions sound good!
In the future, the spring-boot-starter for version 3 will be the default, because of the jakarta.* namespace.
And with newer spring boot versions (e.g. 4), you are maybe not aligned to the spring boot versioning.
What do you think of (1)
jte-spring-boot-starter-javax
and
jte-spring-boot-starter
As alternative you could also do: (2)
jte-spring-boot-starter-javax
and
jte-spring-boot-starter-jakarta
In the Alternative 1 the current spring boot starter users will get an compilation error and move to the jte-spring-boot-starter-javax version and you can continue to update jte-spring-boot-starter without any further versioning problems.
In Alternative 2 you have the argument of @casid, that the artifact is gone.
Actually, I would prefer Alternative 1, because then you could go on without the SpringBoot 3 in the name of your lib.
Thank you for your support, I appreciate that! Both ways are fine for me!

I also thought about having a naming without any "compatible" version in the name so it will be the default when boot 2 goes out of support. But it will be a little strange for current users as their artifact becomes incompatible with their boot 2 application for no obvious reason. So I would say let's go with -2 and -3. When we remove the boot 2 starter eventually we can rename boot 3 starter.

That's fine with me! Thank you!

I also thought about having a naming without any "compatible" version in the name so it will be the default when boot 2 goes out of support. But it will be a little strange for current users as their artifact becomes incompatible with their boot 2 application for no obvious reason. So I would say let's go with -2 and -3. When we remove the boot 2 starter eventually we can rename boot 3 starter.

Or even keep -3 to be prepared for -4, that's probably gonna happen someday.

Thymeleaf is doing the the same. There is thymeleaf-spring3, thymeleaf-spring4 etc. Will try to push the rename today

@schowave, @atomfrede I just published version 2.2.6, which contains jte-spring-boot-starter-3. Thank you @atomfrede for making this!

@atomfrede and @casid: Thank you very much for your fast support. Tried it today and it is working perfectly!

Happy to help. Great to hear your feedback!


fixed the debugger settings doc

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Modified doc PR.
Related Issues/PRs
https://github.com/tracel-ai/burn/pull/1719
https://github.com/tiruka/base_rust/tree/reproduction-vscode-debugger (reproduction purpose)
Changes
Modified doc for debugger settings.
The below setting is not valid when using vscode debugger with launch.json. I modified the doc and make suggestions that developers should modify the Cargo.toml straight.
{
  "rust-analyzer.runnables.extraEnv": {
    "CARGO_PROFILE_DEV_DEBUG": true
  }
}

Testing
I made repo for reproduction. If you need it, please see it.
https://github.com/tiruka/base_rust/tree/reproduction-vscode-debugger

Thanks for sharing it! I see the cause of the issue is the same. So, Modifying the debug to true is straight when using launch.json.
BTW, you approved my pr, and should I wait for merging? I am not sure the github actions status to be reported.
If I need something to do, please kindly let me know.

BTW, you approved my pr, and should I wait for merging? I am not sure the github actions status to be reported. If I need something to do, please kindly let me know.

Nope, just forgot to merge it immediately 😅 sorry about that lol


Improvement support multiple agg function have same base table slot

Proposed changes
support multiple agg function have same base table slot
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


fix flow errors in crossdock

Coverage remained the same at 93.895% when pulling 66c0fa5164eec9a4b95d9fc6f20b7dc87a248d19 on fix_flow_errors into 29f3a3cb14790094a98d637611585a74cb4d426a on master.

Coverage remained the same at 93.895% when pulling 9dc03fc84507899cf457bc66f62c6b0f4565a0f0 on fix_flow_errors into 29f3a3cb14790094a98d637611585a74cb4d426a on master.

Coverage remained the same at 93.895% when pulling 594a5283ccfd342708a9141ad721e2ff075f37a7 on fix_flow_errors into f5f2f7e89772a20538613b2805bb7aba3cebfc79 on master.

Coverage remained the same at 93.552% when pulling 6d93c38f96f944d87e228b7a3d89d91b11574196 on fix_flow_errors into d09c598ab43f0aa9f52e5ad5f56ef34170300879 on master.


Autoscaling group, update existing, instead of creating new

In the Autoscaling.py example, the example is adding a load balancers to a new auto scaling group. Is there such an example that adds ELB's to an existing AutoscalingGroup? Updates an existing AutoScaling group?

There isn't an example for this - it might be possible, but this is more of a Cloudformation question than a troposphere question.


TASK: Fix typos in readme and comments, streamline language, add i18n…

… files for BackgroundImage mixin
Some of the German language like "erblassen" for fade or "rutschen" for scroll might be technically correct, but is very German and IMO not common.
Fixes: #39

Will merge soonish... but would like to have all other issues together in a 3.0 release.

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


"Read" no longer work after node-red restart

Hi,
While testing the flow following the youtube tutorial, the read function no longer works properly after restart of the node-red.
The gateway is finding my devices

but the read function is empty, even though it says 2 devices found

Any ideas on how to troubleshoot?
I'm currently testing it on Win10 virtual machine before implementing it on an edge device. After the issue occur I tried it on freshly installed node-red on my local machine, but there it cannot even find the devices.

Conversation moved to discord for diagnosis will update in comments when resolved

Issue surrounding hyper v installation. A solution was found via discord chat. Closing this case.


Added ZSE school from Olsztyn, Poland

Actual domain is .eu for school's website (zse.olsztyn.eu) but it uses @zsek.olsztyn.pl for students' emails so I'm not exactly sure where to put the file. I used the path JetBrains webpage autosuggested.

@Me-Wosh First stage of verification passed, official domain is zse.olsztyn.eu as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://zse.olsztyn.eu/bursa.html

@Me-Wosh Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Navigation Documentation Updates

[ ] Identify examples to create interactive code samples for
[ ] Create interactive code samples for identified scenarios
[ ] Differences between MUI and SDS
[ ] Link to MUI documentation
[ ] Update prop tables
[ ] Add code to interactive blocks
[ ] Add any additional code examples outside of Default
[ ] Incorporate nuances from the section below into the appropriate place

Now documentation updates are covered by an overarching single GH issue #539


option to 'connect the dots' in a scatter plot

cqid: VisIt00008958cqsubmitter: Mark Millercqsubmitdate: 08/11/09 This is related to 8959. Our scatter plot does not generate actual paths in 'phase space' even if the input data is 'phase related'.  I think when the numbers of points are low as in curve data (see 8959), we ought to have the option of 'drawing lines between points known to be neighbors in the input mesh'. I think this option would be useful even for 2 and 3D rect and unstructured grids of moderate size. Yes, you would get what appears to be inter-tangled lines, but the trend the lines would suggest might be more useful than just the dots themselves.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 143
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: option to 'connect the dots' in a scatter plot
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start:
Due date:
% Done: 0%
Estimated time:
Created: 06/21/2010 07:17 pm
Updated:
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 3 - Medium
Expected Use: 2 - Rare
OS: All
Support Group: DOE/ASC
Description:
cqid: VisIt00008958cqsubmitter: Mark Millercqsubmitdate: 08/11/09 This is related to 8959. Our scatter plot does not generate actual paths in 'phase space' even if the input data is 'phase related'.  I think when the numbers of points are low as in curve data (see 8959), we ought to have the option of 'drawing lines between points known to be neighbors in the input mesh'. I think this option would be useful even for 2 and 3D rect and unstructured grids of moderate size. Yes, you would get what appears to be inter-tangled lines, but the trend the lines would suggest might be more useful than just the dots themselves.
Comments:

Removing from reviewed state due to request from Teams.

Keith MacKay indicated a need to scatter plot two curves for a trajectory of the two curves, one plotted against the other.
In the case of a scatter plot where the two variables being plotted are curve variables, we might want to always connect the dots and maybe have an option to NOT connect the dots.


New SSAO method

As described here, this change will give both an improvement in the visual results of SSAO as well as a significant FPS improvement.
However, it should be noted that this PR doesn't remove the existing SSAO method, and the only way to change between them is to rename the original "SSAO.cs" to something Torque doesn't find like "SSAO.ncs".
This PR is mostly here for discussion and I will change it to a PR that replaces the current SSAO implementation if there is an interest. Personally I've had a lot more success with the new SSAO method.

It should now completely replace the old one.

The CI errors are all:
../../../../Engine/source/sfx/openal/LoadOAL.h:34:12: fatal error: 'al/al.h' file not found
#  include <al/al.h>
           ^

Which is perplexing.

Made a small change to Templates/Full/game/core/scripts/client/postFx/default.postfxpreset.cs but I doubt that triggered the bug.. I've no clue...

Has this been abandoned? @lukaspj, @eightyeight


Azure monitor query stream style migration

Migrating azure-monitor-query from Jackson Databind to azure-json.

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure:azure-monitor-query


Clone a repository / connect advances to page before auth completes

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce
Dev Setup tool -> Account -> Select GitHub -> within 5 seconds, the clone a repository dialog updates, before you've completed the github OAuth flow

Expected behavior
No response
Actual behavior
No response
Included System Information
No response

completed in #1185


Updated scopes

Scopes should be comma separated, I have seen people facing issue because of scopes were space separated.
Description
In the last Time to First Hello World, one of the developer could not generate an access token using space separated scopes. When he changed the scopes to be comma separated it worked fine. I would suggest that we change our documentation to comma separated scopes which works in all scenario.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

In postman the scopes are space-delimited so it might cause confusion to add the commas

Hi Mike,
I'm afraid, I have used the comma separated scopes in Postman and it works fine!

Let me give you some background here, We conducted Adobe Developer Challenge at the University of Utah on November 13th, we noticed that one student was unable to generate an access token using the space separated scopes. I verified everything and concluded that issue was because space separated scopes. When I replaced the scopes to be comma separated the postman OAuth 2.0 worked fine and generated the access token.


Feature Request: Pet Unlocked

Category
A new trigger
What are you missing?
I was wondering if it would be possible to add a pet unlocked trigger? That's something the old Action Queue had that was very useful.
Thank for your hard work!
Code of Conduct

[X] I agree to follow the code of conduct
[X] My request fits the contribution guidelines
[X] I have searched for similar issues and nothing similar exists

Done, it'll get published sometime this year :slightly_smiling_face:


chore: fix docstring and update cargo lint

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.26%. Comparing base (4f513f9) to head (2b6cfaf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage   95.26%   95.26%
=======================================
  Files          48       48
  Lines        3083     3083
=======================================
  Hits         2937     2937
  Misses        146      146

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


Align towncrier with defaults

As jaraco/skeleton is adopting towncrier, that upstream project is attempting to honor the default settings wherever possible.
These changes attempt to align with those defaults and re-use the functionality added upstream for managing towncrier.
Summary of changes

Move towncrier config to its own file.
Use the default 'newsfragments' for news fragments.
Use jaraco.develop.towncrier to check changes and determine the release kind.
Revert to towncrier default types.
Normalized headings in changelog.

Tests are failing on cygwin because the new jaraco.develop dependency has some logic for uploading secrets to Github, which requires PyNaCl, which is built on CFFI, which cygwin doesn't support. We'll have to find a way to exclude that dependency.

I'm going to push forward with merging this to unblock refreshing the skeleton. I welcome feedback, but for the most part, I expect this change to have minimal impact with one exception - existing PR changelogs will need to be updated:

change -> feature
breaking -> removal
misc -> bugfix (maybe)
The location may need to be updated (newsfragments/).


Use text/plain for logs instead of eventstream

Use text/plain format instead of eventstream

How will this impact the browser scenario?

Maybe this should be JSON.

Remove all text/eventstream results, replace with text/plain.


It only works with a single USB cable.

Hi guys, I'm having a problem mirroring the screen. It only works with a single USB cable, like it works 100%, but if I try to connect using another USB cable it appears:
scrcpy 2.3.1 https://github.com/Genymobile/scrcpy
ERROR: Could not find any ADB device
ERROR: Server connection failed
Press Enter to continue...
(NOTE: I know absolutely nothing about codes etc. and I'm Brazilian, so I'm sorry for my bad English)

You need USB cables which are not charge-only.


respect DO_NOT_TRACK env var per consoledonottrack.com

https://consoledonottrack.com
This adds a check for the environment variable DO_NOT_TRACK, a new proposed standard for user opt out of tracking and telemetry in command-line tools.

We would rather use our own variable for now (at least until this is much more widely adopted).

Nothing about supporting this new standard prevents you from continuing to use your own.

If projects don't adopt it unless it becomes widely adopted, how is it supposed to gain wide adoption?
That argument is a self-fulfilling prophecy..

I support this feature, especially as I'm convinced there's already a GDPR violation in the way Homebrew's  analytics are opt-out by default...

@grxnola, did you mean opt-in by default?

I would rather see the different packager managers sit together and work on that standard.
And especially big players should get involved, like the debian/ubuntu apt people, the fedora people, the nixpkg people, pip, node and all the others.
I have also the same opinion as https://github.com/syncthing/syncthing/pull/6158#issuecomment-554260354,
this looks more like a marketing move from yourself than something the community really needs/wants.
I also take the opportunity to remember to all our users that we have been really transparent about our analytics strategy: https://docs.brew.sh/Analytics.
@grxnola if you think there is any GPDR violation here, I would like you to give us the exact articles from the regulation which you think we violate. Please open a new issue for this (to not clutter this discussion). We are open to learn and improve what needs to be improved. But I do not think we are doing anything wrong right now, and already have reviewed this topic internally.
Also, if we really need a more formal decision on this, we can involve the Homebrew Technical Steering committee (https://docs.brew.sh/Homebrew-Governance).
I am personally not in favor of adding this env variable.

I would rather see the different packager managers sit together and work on that standard.
And especially big players should get involved, like the debian/ubuntu apt people, the fedora people, the nixpkg people <...>

Most other package managers do not send analytics by default (or even have the capability of doing so at all) - especially distribution package managers. I'm from the Arch Linux team and we e.g. do not even have rough info on the amount of installed systems, because there is no official data collection at all. Brew and other community-effort or domain-specific package managers are the outliers in that methodology i think.
I for one would really enjoy seeing analytics collectors like brew make this change, to allow for a more standardized and unified way to opt out of data collection. It is a change that will require adoption, and i think getting it introduced into brew & co is a great opportunity to get wider adoption in software like this.
The old methods of opting out are still respected, after all, so there is no compatibility to worry about.

Homebrew should respect a DO_TRACK environment variable.  Right now it is bundled with spyware and cannot be recommended.

Nothing about supporting this new standard prevents you from continuing to use your own.

@sneak None of the PRs you have opened have been merged yet. As-is: this is not a "standard", it is a webpage you have made with some links to PRs.

If projects don't adopt it unless it becomes widely adopted, how is it supposed to gain wide adoption?

Currently it has no adoption. It's not on Homebrew to change that.

especially as I'm convinced there's already a GDPR violation in the way Homebrew's analytics are opt-out by default...

Thanks for implying legal threats against volunteers who spend their free time making software you use for free!

I'm from the Arch Linux team and we e.g. do not even have rough info on the amount of installed systems, because there is no official data collection at all.

Run your package manager how you want/need and we'll do the same.

As long as spyware is bundled with Homebrew, Homebrew shouldn't be used or recommended or normalized. The solution is for Homebrew to remove its spyware by having explicitly opt-in telemetry only.

No, you've said the solution: you can stop using or recommending Homebrew.

Everyone who comes to this thread should take time to (re)read https://docs.brew.sh/Analytics and note that we tell people about our analytics and direct them to documentation which describes opt-out before we ever send anything (on initial installation or on brew update if you update from a version that doesn't have analytics).

We would rather use our own variable for now (at least until this is much more widely adopted).

Adopting this standard is zero-cost, I'm confused why this was closed.

You're denigrating this as a marketing effort and something in need of standardization, but if you were going to pick a generic env var to control tracking behavior, what would be a better choice than DO_NOT_TRACK?

Adopting this standard is zero-cost, I'm confused why this was closed.

@4n3w a few things:

it requires code changes which need to be maintained
as-is this PR won't work at all (evidently it was not tested)

and something in need of standardization

No-one from Homebrew has made that claim.
This conversation is not a good use of anyone's time at this point. When more projects have adopted this: we will reconsider it.


Review and Add the Guidelines for Writing Ballerina By Examples (BBEs)

Purpose
This is to review and add the "Guidelines for Writing Ballerina By Examples (BBEs)" to the /docs directory.
Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@             Coverage Diff             @@
##             master   #16456     +/-   ##
===========================================
+ Coverage     13.67%   14.17%   +0.5%
- Complexity     3202     3246     +44
===========================================
  Files          1515     1513      -2
  Lines         74455    73128   -1327
  Branches       9082     8981    -101
===========================================
+ Hits          10183    10368    +185
+ Misses        63214    61686   -1528
- Partials       1058     1074     +16

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/ballerinax/jdbc/SQLDatasource.java
59.3% <0%> (-0.59%)
16% <0%> (-1%)

...llerinalang/stdlib/task/actions/DetachService.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...erinalang/compiler/CompiledPackageSymbolEnter.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...lang/compiler/bir/writer/BIRInstructionWriter.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...erinalang/compiler/parser/BLangParserListener.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../compiler/semantics/analyzer/SemanticAnalyzer.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...etions/builder/BFunctionCompletionItemBuilder.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ballerinalang/stdlib/io/nativeimpl/WriteInt32.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...erinalang/compiler/bir/model/BIRNonTerminator.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...iler/tree/expressions/BLangTypeConversionExpr.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 122 more

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 f9dda8b...807e514. Read the comment docs.


DatePicker data-format Observe

How can i change attribute format at runtime and picker will be updated?
I have a multi language page and i need the selected value will be updated when user change language.
Thanks

Did you mean the locale attribute?

In 4.2.46 I added observation for attribute data-locale

In 4.2.46 I added observation for attribute data-format


Could not parse rpc data from daemon instance  0

m@r:~/z-nomp$ npm start

<EMAIL_ADDRESS>start /home/m/z-nomp
LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$PWD/node_modules/stratum-pool/node_modules/equihashverify/build/Release/:$PWD/node_modules/equihashverify/build/Release/ node init.js

2018-03-22 10:50:42 [POSIX]	[Connection Limit] (Safe to ignore) POSIX module not installed and resource (connection) limit was not raised
2018-03-22 10:50:42 [Master]	[CLI] CLI listening on port 17117
2018-03-22 10:50:42 [PPLNT]	[zcash] TimeShare processing setup with redis (<IP_ADDRESS>:6379)
2018-03-22 10:50:43 [Website]	[Server] Website started on localhost:8080
2018-03-22 10:50:43 [Master]	[PoolSpawner] Spawned 1 pool(s) on 4 thread(s)
2018-03-22 10:50:43 [Switching]	[Setup] (Thread 1) Loading last proxy state from redis
2018-03-22 10:50:43 [Pool]	[zcash] (Thread 1) Share processing setup with redis (<IP_ADDRESS>:6379)
2018-03-22 10:50:43 [Pool]	[zcash] (Thread 1) Could not parse rpc data from daemon instance  0
Request Data: {"method":"getinfo","params":[],"id":1521696043391}
Reponse Data:
!DOCTYPE html>
html lang="en">
head>
meta charset="utf-8">
title>Error
/head>
body>
pre>Cannot POST /
/body>
/html>

Check your daemon port (in pool_config/zcash.json, section "daemons"). Looks like you put http port and getting responses from http-server instead of daemon.

@crutched I have come across the same error. Could you explain what you  meant by replacing the daemon port. What should i keep in the config file?

@mrsaicharan1 you need to put port for rpc of the daemon in the "daemons" section of config. This port is custom for every coin.

Do you have any idea for zencoin? the rpc port of daemon?

Also, what IP do I put in? The VPS IP or what ?

@mrsaicharan1 if your daemon on the same machine - put <IP_ADDRESS> loop as ip. Usually rpc works with localhost only, you need to config it if you want to have your daemon on another machine.

I tried keeping that . It says the pool could not start :(. <IP_ADDRESS> IP
Port 80.
Do I need to change it elsewhere too?

@mrsaicharan1 I'm 100% sure that 80 port are never used as rpc port of the daemon.

Zencash or zencoin? check out zencash pool config example. you can change your rpc port in your node's config file. Line will look like this: rpcport=12345 where 12345 is your preferred rpc port number.
Did you start your node before starting z-nomp? Make sure it gives you reply when you make getinfo query.

Yeah,I set up the zencash node before. It gives the live status of number of blocks too. The node works fine. The only thing which is not running is the pool due to config issue.

{
"enabled": true,
"coin": "zen.json",
"address": "znYnbSYnMFSZqyk1VMzHE6QNiP7pp6RH7FE",
"_comment_address": "a transparent address to send coinbase rewards to and to transfer to zAddress.",

"zAddress": "zcLh1SzCYt64T1m17TTNXVSYz1txEFbyCL7qSU9oSEbjjf8wD3rWst1wMeeGqN1nmwENy7LAuQ3U1wywyzucwc8sfXCcLrj",
"_comment_zAddress": "a private address used to send coins to tAddress.",

"tAddress": "znhg6n4jpujWfvwxEkjV5aS3CrJbyPdb6Yc",
"_comment_tAddress": "transparent address used to send payments, make this a different address, otherwise payments will not send",

"invalidAddress":"znhGeka9zXmixvw6ufzGpcaSXcSACrjx5WZ",
"_comment_invalidAddress": "Invalid addresses will be converted to the above",

"walletInterval": 2.5,

"rewardRecipients": {
"znYnbSYnMFSZqyk1VMzHE6QNiP7pp6RH7FE":0
},

"tlsOptions": {
    "enabled": false,
    "serverKey":"",
    "serverCert":"",
    "ca":""
},

"paymentProcessing": {
    "enabled": true,
    "paymentMode": "prop",
    "_comment_paymentMode":"prop, pplnt",
    "paymentInterval": 20,
    "minimumPayment": 0.1,
    "maxBlocksPerPayment": 3,
    "daemon": {
        "host": "<IP_ADDRESS>",
        "port": 18231,
        "user": "zenuserorsomebettername",
        "password": "replacethiswithagoodpassword"
    }
},

"ports": {
    "3032": {
        "tls":false,
        "diff": 2.5,
        "varDiff": {
            "minDiff": 0.04,
            "maxDiff": 16,
            "targetTime": 15,
            "retargetTime": 60,
            "variancePercent": 30
        }
    }
},

"poolId": "main",
"_comment_poolId": "use it for region identification: eu, us, asia or keep default if you have one stratum instance for one coin",

"daemons": [
    {
        "host": "<IP_ADDRESS>",
        "port": ,
    "user": "zenuserorsomebettername",
        "password": "replacethiswithagoodpassword"
    }
],

"p2p": {
    "enabled": false,
    "host": "<IP_ADDRESS>",
    "port": 19333,
    "disableTransactions": true
},

"mposMode": {
    "enabled": false,
    "host": "<IP_ADDRESS>",
    "port": 3306,
    "user": "me",
    "password": "mypass",
    "database": "zcl",
    "checkPassword": true,
    "autoCreateWorker": false
}

}

Here's my zen.json. @igorvoltaic . Do I need to make any changes?

Have to input port and everything too.

OK, I've gone past this error and put in the correct rpc port. Now how do I tell the miners which IP to connect to?

I'm also getting an error right now saying daemon does not own pool address? Where do I fix this?

Hi,
Yes, your setup doesn’t have port settings in daemon section.
And yes, you need use taddress and zaddress of your daemon

On 5 Jul 2018, at 17:15, Saicharan Reddy<EMAIL_ADDRESS>wrote:
I'm also getting an error right now saying daemon does not own pool address? Where do I fix this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Of course not. Just use your usual zen address. Address of the pool will accumulate pool’s fee and you address will receive payments after 100 confirmations. (Minus fee)
If you want SSL support check out my pool request, it has nginx example

On 5 Jul 2018, at 17:27, Saicharan Reddy<EMAIL_ADDRESS>wrote:
username and password can be random?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

You mean the wallet address right?

Yes

Just like with any other pool.
Miner’s —server option should contain your pool’s IP address

Thanks @igorvoltaic !Honestly speaking, the setup was a pain in the butt but the community made it easier! Now my pool's ip and running!!!!

hey @igorvoltaic , the miner has connected to the pool, but the stats don't show up live on z-nomp website. http://<IP_ADDRESS>:8080

Usually takes several minutes. Does miner show you mining process? Can’t open pool’s web page

On 5 Jul 2018, at 17:47, Saicharan Reddy<EMAIL_ADDRESS>wrote:
It says that I have one authorized user on the cmd
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey @igorvoltaic . Done , mining has also started! Thanks anyways!!!

One last thing, I wasn't able to change pool  fee %. where do I do that?

In rewardRecipients section you set pool’s address and percentage. 1.0 stands for 1%

On 5 Jul 2018, at 17:51, Saicharan Reddy<EMAIL_ADDRESS>wrote:
One last thing, I wasn't able to change pool fee %. where do I do that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Update search.jet.html

Coverage remained the same at 32.957% when pulling a0dd09dbd873d79e0b0cda6d173241fa9d7a0071 on Kiloutre:patch-61 into e7031e461a8f8e217afce8fdf1e155ec3ce877b2 on NyaaPantsu:username-search.


Fix profile email field submission [MERGE #185 FIRST]

In #184, the profile email field was set to disabled to prevent user changing it.
Unfortunately this had a side effect of removing the value from the submitted payload, causing the server to respond with "incomplete params" error. This fix manually adds it back.
APPROVE AND MERGE #185 FIRST
This branch clones the branch in #185 . So please review that first.

@SheldonCheng I can see the same changes as #185 on this PR as well. Can you rebase again?

Recreated with #187


Corrected the types in documentation.

Summary updates -

new GraphQL(schema) accepts GraphQLSchema and NOT GraphQLSchemaGenerator
Updated GraphQL creation to use builder. The constructor that was in use is deprecated.

Thanks!


Wso2 iot Server - How can we make a change in the project?

Description:
I want to put an authentication mechanism to project. But I do not know what to do. How can I build the project? I can not make changes to this code. Because it's made into a bundle.
Can you help me ?

@omerstack you can find information about working with the source code in the following documentation.
https://docs.wso2.com/display/IOTS320/Working+with+the+Source+Code
Please note that the main component repositories that the IoT server is dependent on are,
https://github.com/wso2/carbon-device-mgt
https://github.com/wso2/carbon-device-mgt-plugins


Update to running directory in the generic ExecutionOptions class.

Updating the default run directory within the generic ExecutionOptions to be based on the first eight chateracters of the hash of combining the case title and the label. This helps to create a unique directory name for running the case but also helps to avoid OS related file path length issues (particularly in Windows).

The change seems straight-forward, and I'm all for it.
I wish that making this change broke a unit test though. What this tells me is there are not tests that cover that code.

Working on some tests now. Thanks for the feedback @theJollySin


fix state.merge error

This would be solving the same issue with #425. You may work on that PR instead if it solves the same issue.


Added documentation that explained how the module works

Hi,
Thank you for your contribution.
Could you check the following errors:
https://travis-ci.org/lizenn/erlang-dbus/builds/273584440?utm_source=github_status&utm_medium=notification

Errors fixed.  No functional changes to code.   Error was a bad -spec declaration.

Thank you very much for your contrib !


Licence

What is the licence agreement for this element? And can you please add it, thanks.

The license agreement is MIT.


Doesn't work with NextJS

Describe the bug
This library does not work with nextJS, because there is already some interaction with window during import time.
To Reproduce
Set up a NextJS + React App and use this library
When compiling the App, you will see an error like this:
- error Error [ReferenceError]: window is not defined
    at file:///home/veith/projects/ts/node_modules/.pnpm/reactjs-social-login@2.6.3_react-dom@18.2.0_react@18.2.0/node_modules/reactjs-social-login/dist/reactjs-social-login.modern.js:1:410
    at ModuleJob.run (node:internal/modules/esm/module_job:217:25)
    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:431:15) {
  digest: undefined
}

NextJS prerenders on the server, where the window object is not defined.
Various work-arounds listed here don't work, because this already happens at import time in this library.

duplicated #7 #109 #156 #148 #144

please use {srr: false}


Support for img srcset

Any plans or interest to make the Lightbox more responsive by supporting 'srcset'?

Last time I looked into srcset the browser coverage was really poor, though it seems to have improved http://caniuse.com/#feat=srcset
Would you be interested in submitting a PR?

@jossmac Great!  I'll work on a PR.  Since browsers that don't support it will just use src, it should be harmless.

@neptunian sounds good, I look forward to it

@jossmac  Can I introduce some other kind of image placeholder tool rather than the fillmurray.com one?   With srcset I'll need to grab various sizes of the same image which fillmurray.com doesn't seem to support.  This website does however: http://lorempixel.com/  Or if you have any suggestions...
Also, I notice the thumbnails are simply larger photos being squashed down, so could update that, if desired, as well.

@neptunian LoremPixel is a terrific service, can't wait to see the images you choose!
Good call on the thumbnails, i was pretty rushed when releasing the demo site, thanks mate :)

Resolved by #6

@jossmac  I hadn't updated the documentation yet in my branch that was merged.  Shall I do that?

@neptunian sorry! Would you mind submitting a separate PR with docs?


Add MathML parsing support

It would be great if somebody could make a PR and add MathML parsing support to the editor. TinyMCE 7 has it, but we have to reimplement it.

If all you want is MathML or LaTeX input, you basically want to just use MathJax to render the fragment and author the content as LaTeX. For details, see https://www.mathjax.org/ and the demo at https://www.mathjax.org/#demo
I'd recommend using longer start and end markers for the math input because a lone dollar is too easy to trigger accidentally. We're using [[$ and $]] for inline math limiters and [[$$ and $$]] for display mode limiters.
For even better implementation, you should probably also support http://mathquill.com/ to avoid enter LaTeX manually because most non-math-teacher users are not familiar with it.

I used https://github.com/umar221b/tinymce6-formula to create a plugin for TinyMCE6 that uses MathML as far as possible (modern browsers can deal with it), instead of rendered images as the original project did it. Please find my version zipped
tinymce6-formula.zip
To use it, move the dist directory into the TinyMCE plugins directory and call it "formula". Then enable the plugin "formula", add a button "formula" to the toolbar and set option
formula: {mlang: "mml"}
as we want to use MathML instead of LateX.
That should be it - in case of any bugs please just fix them and contribute to the original project (you may include my changes, I am just not familiar with the processes at GitHub)

Official editor seems to use markup such as <tiny-math-inline formula="m_{1,2} = \frac{y_2 - y_1}{x_2 - x_1}"></tiny-math-inline> instead of plain text $ m_{1,2} = \frac{y_2 - y_1}{x_2 - x_1} $ which would be natively supported by MathJax for the final rendering, too. Of course, you need to represent the markup with some elements in the editor area because TinyMCE doesn't easily support non-elements being rendered differently in the editor.
The important part is the serialization format for the output of the editor. One pretty nice format could be slightly mis-using official markup such as
 <math display="block" alttext="x = \frac{-b \pm \sqrt{b^2 - 4 a c}}{2 a}"></math>

(See the official documentation https://w3c.github.io/mathml-core/#the-top-level-math-element – I think the above empty element would even pass validation if needed but obviously the actual content is not human readable before using MathJax to render it for real.)
Then you could add a small piece of JavaScript to run before executing MathJax that takes all the <math> elements with empty contents in the DOM and replace those with text nodes with suitable syntax to work with MathJax. That piece of JavaScript could even look at the current MathJax config to figure out suitable start and end markers.
Also note that encoding of element attributes and plain text nodes is slightly different. Make sure to get encoding of <>"&' correct or you might introduce XSS vulnerability.
If you really wanted, you could also use MathJax client side to actually generate the full MathML output rendered by the MathJax library but I think it would be better to keep the serialized format as minimal as possible. The above markup already contains all the data the user inputted.
And if the user truly prefers to input raw MathJax instead of LaTeX, the UI support for that is basically a textbox that takes MathML as input and directly adds the markup into the editor.

@mikkorantalainen mentioned an important point, you should add to option extended_valid_elements the following:
annotation[],annotation-xml[],maction[],math[],menclose[],merror[],mfenced[],mfrac[],mglyph[],mi[],mlabeledtr[],mmultiscripts[],mn[],mo[],mover[],mpadded[],mphantom[],mroot[],mrow[],ms[],mspace[],msqrt[],mstyle[],msub[],msubsup[],msup[],mtable[],mtd[],mtext[],mtr[],munder[],munderover[],semantics[*]
Otherwise, TinyMCE will eliminate the MathML. IMPORTANT: You must filter the HTML thoroughly to avoid XSS or similar problems.

Thank you both for your input in this! The problem is, if I remember correctly, that when I allow math elements, then use the code plugin for inserting a formula and then click into the formula the editor turns read-only as long as I'm in the formula: It doesn't want to edit it. There's https://github.com/tinymce/tinymce/commit/628b0dac18d9ac23cd754f76908108b981a69b98 doing some extra stuff but that's GPL already. I'll take a look at the plugin you mentioned, thank you!

Personally I use LibreOffice Math formula objects within LibreOffice Writer (just think of LibreOffice Math). Their syntax is quite easy to use, and probably better than Latex. I wonder if it's used outside the LibreOffice world...

Official editor seems to use markup such as <tiny-math-inline formula="m_{1,2} = \frac{y_2 - y_1}{x_2 - x_1}"></tiny-math-inline> instead of plain text $ m_{1,2} = \frac{y_2 - y_1}{x_2 - x_1} $ which would be natively supported by MathJax for the final rendering, too. Of course, you need to represent the markup with some elements in the editor area because TinyMCE doesn't easily support non-elements being rendered differently in the editor.
The important part is the serialization format for the output of the editor. One pretty nice format could be slightly mis-using official markup such as
 <math display="block" alttext="x = \frac{-b \pm \sqrt{b^2 - 4 a c}}{2 a}"></math>

(See the official documentation https://w3c.github.io/mathml-core/#the-top-level-math-element – I think the above empty element would even pass validation if needed but obviously the actual content is not human readable before using MathJax to render it for real.)
Then you could add a small piece of JavaScript to run before executing MathJax that takes all the <math> elements with empty contents in the DOM and replace those with text nodes with suitable syntax to work with MathJax. That piece of JavaScript could even look at the current MathJax config to figure out suitable start and end markers.
Also note that encoding of element attributes and plain text nodes is slightly different. Make sure to get encoding of <>"&' correct or you might introduce XSS vulnerability.
If you really wanted, you could also use MathJax client side to actually generate the full MathML output rendered by the MathJax library but I think it would be better to keep the serialized format as minimal as possible. The above markup already contains all the data the user inputted.
And if the user truly prefers to input raw MathJax instead of LaTeX, the UI support for that is basically a textbox that takes MathML as input and directly adds the markup into the editor.

I think because there is already a math plugin
https://www.tiny.cloud/docs/tinymce/latest/math/
but it depends on 7.1 and later, i do not know if this would work here or not, i do not think..

HugeRTE is based on a commit later than 6.4, but earlier than 7.0 because they changed the license then and we want to keep the old one.


Fixes #87. Allow "inline" mode to be default if data is specified

This works as I would expect, LGTM.


[chip, testplan] Update SYSRST_CTL and ADC

This commit addresses the comments and updates from the testplan
review meeting held on 6/07/2021.
The meeting notes are below:
https://docs.google.com/document/d/1OhPP-HjciwKpIh0wWt1xqPqPf0Y0powmmww6xekwMeE/
Updated SYSRST ctrl and ADC ctrl sections of the testplan

Signed-off-by: Srikrishna Iyer<EMAIL_ADDRESS>

Look good to me.
-Eric


Use enum #repr field

Enum fields take 4 bytes, using the Fixedint encoding, even though most enums need at most one byte.
It is possible to use Varint encoding to save those bytes, as discussed in #157, however varint increases the size of integers that are already well matched to their types.  In a representative example in my codebase the fixedint encoding of one structure takes 100 bytes but the varint encoding takes 129 bytes.  Thus the gain from varint saving enum bytes can easily be outweighed by losses elsewhere.
Rust already supports #[repr(u8)] for enums.  Can not the fixint encoding use these or similar annotations?
Note: Another option would be to have a fixed enum size, but to have that enum type set in the config or provided as a type.  The same could be done with the length field.

This is discussed at length in #565 and #566, and there is a comment about this in the readme

Thank you for the references.  As mentioned, varint as suggested in the FAQ doesn't actually reduce the message size if the sizes are already accurate.  However I appreciate that simplicity is a priority for you and so I have made a fork and simply changed the hard-wired u32 to u8.  This suits my use case.


Expired URL

Issue description

the URL given as webshop in the document is expired.
the webpage is displaying this error and the configuration while assigning odata route path is showing error
the URL is this:-https://refapp-espm-ui-cf.cfapps.eu10.hana.ondemand.com/webshop/index.html.
the error shown in web page is this
:-404 Not Found: Requested route ('refapp-espm-ui-cf.cfapps.eu10.hana.ondemand.com') does not exist.
Feedback Type (Optional)
None
Page Title on SAP Help Portal (prefilled)
WebShop Example Application
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/cloud-integration/sap-cloud-integration/webshop-example-application

Hi @saikoushik-Pvs , thank you for your feedback! We’ll look into it and come back to you if we have any questions.

Hello @saikoushik-Pvs,
many thanks for the feedback! The system was temporarily unavailable, but this is fixed now. The URL is correct and the system should be accessible again.
Best regards
Peter

Thank you!


Drop rustc-serialize dependency

It has been deprecated: announcement.
Is there any functionality you would require from another library before this would be possible?

No obstacles. I'll do it when I have times to spend


Feature/add error handling for bqio

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Run Java PostCommit

"beam_PostCommit_Java" runs non-IO integration tests on direct runner. Would you mind trigger Dataflow PostCommit (which runs GCP IO integraiton tests on Dataflow) by adding file namedbeam_PostCommit_Java_DataflowV1.json ?

I went through the changes and mostly looks straightforward and looks good to me. Due to the line of changes I may missed something. Please run postcommit tests and see we do not break things (commented above). Also consider run Python xlang GCPIO Dataflow (trigger path beam_PostCommit_Python_Xlang_Gcp_Dataflow.json)

There are integration test failed:
org.apache.beam.sdk.io.gcp.bigquery.BigQueryIOStorageReadIT > testBigQueryStorageRead1MErrorHandlingArrow FAILED
    java.lang.RuntimeException at BigQueryIOStorageReadIT.java:162

org.apache.beam.sdk.io.gcp.bigquery.BigQueryIOStorageReadIT > testBigQueryStorageRead1MErrorHandlingAvro FAILED
    java.lang.RuntimeException at BigQueryIOStorageReadIT.java:162

See https://github.com/apache/beam/runs/21587463679 for details

"beam_PostCommit_Java" runs non-IO integration tests on direct runner. Would you mind trigger Dataflow PostCommit (which runs GCP IO integraiton tests on Dataflow) by adding file namedbeam_PostCommit_Java_DataflowV1.json ?

Turns out the tests hadn't gotten run, and I missed something. Thanks for the catch!


Architect Website

Issue Type:

[ ] Bug Report
[ yes] Feature Request
[yes ] Documentaion

Describe the bug

It is an Amazing Architect Website with some amazing features....

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

@rithik-sharma Kindly fill the description properly.


ROX-14920: Add Alertmanager statefulset to teardown

If deployed with MONITORING_ENABLED locally (e.g. to colima), teardown misses the statefulset, which means the stackrox-monitoring-alertmanager will keep running, which leaves the cluster in an unclean state.
Added the statefulset to the delete call, which leaves the cluster clean.
Tested on MacOS x86 / Colima.

Seeing that the automated checks are based on CircleCI: Are they expected to still work? Didn't we deprecate CCI?


Incorrect gvb00405u gvbmr88 message overlap

I had to move the reset of error data again, since the loop can be exited by PARMEOF label also.

I have run the build and the regression process seems to have completed successfully. Results are in:
/u/nbeesle/git/public/Performance-Engine/RTC23305


Unable to scroll layer list with scroll wheel

Using the scroll wheel to scroll a long layer list results in the map zooming instead.
jsfiddle showing this: https://jsfiddle.net/LnzN2/1454/
It works as expected without using grouped layer control: https://jsfiddle.net/LnzN2/1455/

I am seeing the same thing.

See #62

@NHellFire Sorry, didn't see that PR.  I don't think it updates the height of the layer control based on the window height to prevent the overflow.

PR #64 also updates the height the same way leaflet does. So that the layer control does not overflow the window.


[feat] UI for our AI Chat – no functionality required

Implement the following design and its components in the "Ai Styles" Placeholder section of the right properties panel.

Here's a more detailed screenshot:

This is the design for the "chat history" button. The "chat history" button should have a hover state label that says "Chat History" and be in an "active" state when the side panel is open. It would be ideal if there was a search bar in the top panel, but that can be added in a later PR. It is OK if the width of the chat history is a bit larger that the properties panel. The "Plus" button is a "New Chat" button and should have a hover state label on it too.

Please use the proper tokens for foreground / background, as our app is in light / dark mode now.
Also use proper text-sizing tokens from our app tokens indicated by this chart below. Use your best judgement – we can clean up the styles easily if there are issues.

i would like to work on it can you assign it to me

@Kitenite @drfarrell could you assign this to me?

👍  @abhiroopc84


gh-401 Updated schema diagrams to have unique reference ids

gh-401 Data Model Schema diagrams only showed one reference link between data classes when there were mutlitple links between classses.
Solution: Updated the diagram to use unique reference ids.
Original ids were created from the link source id and and link label, but label was always null.
New ids are craeted from the link soruce id and the link name.

@GBishop-PHSA I'd be willing to review/test this PR. Would you be able to rebase your branch from develop first though please - to bring it up to date?

@pjmonks  I've rebased against develop. Ready for review.

Tested this branch and the fix appears to correctly show the links between tables as described in issue #401.


Modify rb-page-title class structure to support other components

TP: https://rockabox.tpondemand.com/entity/9782
Just a simple naming convention fix. Requires someone to merge into story branch and pick up the corresponding Angular part.
Here's some sample mark-up (basically a static version of the directive's template) showing rb-page-title with a search-type rb-text-control.

            ← {{ backText }}

                    {{ heading }}

                <rb-badge state="{{ badgeStatus }}" collapsed="false"></rb-badge>

            {{ subheading }}

Wrong base. Same with right base at #214.


NERDTree opens on startup

I'm new to vim and spf-13 so please bother with me if it is trivial.
NERDTree opens open every time on startup. I find this a bit annoying. How can i prevent this?

wolfgangpfnuer  you rock thanks for your great tip
let g:nerdtree_tabs_open_on_gui_startup=0


Star textures not moving smooth with HMD (Oculus)

Stars seem to no update with every frame, same as ground textures when in orbit. Seems to be objects that are rendered at a distance do not update at the same frame rate as local objects. Running on GTX1060 3gb i7 7700 16gb ram

Sounds like a duplicate of #46


[JobRouter] Update readme

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


AutoResize not working when inserted in container (not directly in body)

When creating a new layout, it resizes correctly if appended directly to body (default).
If you create it in a div or other container, it doesn't resize.
example:
https://jsfiddle.net/rgmswer7/48/
Run this example and resize browser window. It won't resize.
Just remove '#coso' in the  new GoldenLayout() constructor and it will resize

Is possible to make it resizable also when is the container that is resized (not dependent by browser window)?

I think you would need to have an event listener on the container and then call the update function on the layout

Yes, I reverse-engineered from the source code the @ButchMonkey  solution; however I was wondering why that control has been inserted: the _isFullPage is set to false when I am using a container, disabling the resize.

I know about lack of a native resize event... :-(
I hope that in the future GL could use % instead of px to size its items to avoid a custom listener to container resize...

@ButchMonkey  I'm using the NPM package and inserting GL in my angular app, but it's giving an error that "property _isFullPage does not exist on type 'GoldenLayout'". I think typescript definitions need to be updated. As of now I wrote my code like this to make it work:
myLayout["_isFullPage"] = true;


Doesn't work with Babel (JavaScript)

When using Babel syntax highlighting you have to disable the default JavaScript package in order for it to be used in HTML script tags, and other such places.
After disabling the default JavaScript package, instead of switching to using Babel after restart like other syntax highlighters, markdown highlighting breaks in the first js/javascript block with a comment (both line or block comments).

It seems to be an issue with Babel
https://github.com/SublimeText-Markdown/MarkdownEditing/issues/438
The first comment is parsed as division plus a regex.


doomseeker: fix build by using qt wrapper hook

Current master immediately crashes; this fixes the build and the program runs
successfully. See #65399 for details.
Closed #79128

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build doomseeker

This should be backported to both 19.09 and 20.03 please.

@GrahamcOfBorg build doomseeker

@GrahamcOfBorg build doomseeker

Thanks for backports @veprbl :sparkles:


config file error

According to the documentation, the config.yml file should be as below.
hosts:
host_alias:
access_token: ''
host: foo.bar.companyname.com
id: AbCdEfGhIjKlMnOp
secret: QrStUvWxYz1234567890
However, auth.py, references the config file using the name 'kwargs', where the value of host is assigned the value 'looker' in cli.py, as such, throwing an error.
host = params['hosts'][kwargs['host']]['host']
I'm surprised no one else has encountered this issue so thought I'd bring it up.
I fixed the issue by finding and replacing all instances of '[kwargs['host']]' with '[host_alias]'.
Second issue I found was maybe because I'm running on a windows machine. In cli.py, line 29 was giving me issues because of file path backslashes, so had to use encode, decode for logging.config to be able to read this correctly.
LOGGING_LOG_PATH = os.path.join(LOGGING_LOG_PATH,'henry.log').encode('unicode_escape').decode()
Thanks,
Longfei

TLDR; replace host_alias with the string of your host alias and pass it as parameter as such:
Hi @josephaxisa Thanks for your comprehensive answers!
In that case I'll change the code back and just replace 'host_alias' with the actual host alias I decide on.
henry pulse --host=prod_looker
Regards, Longfei


Question: total memory usage (--mem) exceeded the RealMemory in pcluster with SLURM

Hello,
I have set RealMemory=58000 (though they are 64G instances) in the file slurm_parallelcluster_queue_partition.conf
When I run the following commands:
sbatch -N 1 -n 1 --mem=40000 --wrap="srun script.sh first_task"
sbatch -N 1 -n 1 --mem=40000 --wrap="srun script.sh second_task" and more
They will be assigned to single instance which lead to 'Cannot allocate memory' in Java. I assume they should be assigned to two instances since the total required memory already exceed the RealMemory.
Do I misunderstand something? Or do you have any suggestions on this unexpected behavior?
Also, these tasks won't be assigned to a 32G instance, so the RealMemory and --mem should already work.
Thanks very much.

Hi @wentingbian,
Could you check your slurm.conf, the SelectTypeParameters is CR_CPU or CR_CPU_Memory. If it is CR_CPU, it won’t schedule instance by memory constraints.

It is CR_CPU. Thanks very much!


Test only against node LTS

cash is the client-side library, I don't think that testing in multiple versions of Node.js is improving something for client-side libraries.
We are testing cash against multiple operating systems and browsers via SauceLabs (#208), but testing against multiple Node.js versions is excessive.

@fabiospampinato thanks for the quick merge! Would you mind to restart https://travis-ci.org/kenwheeler/cash/builds/440690608 ? I wonder if that was a momentary instability or some mistake in the configuration.

Restarted 👍

Restarted 👍

Thanks, that was a mistake in the configuration, here's the fix: https://github.com/kenwheeler/cash/pull/210/files


Added task to disable default installed apps

Added a task to disable apps that are enabled by default and you maybe not want. The variable for the list with apps you want to disable is "nextcloud_disable_apps"

@aalaesar sorry for bothering you, but can you check if we want this functionality be added to this role or not?
If not, I'll add it to my own Nexcloud customization role.
Cc @staticdev

@aalaesar sorry for bothering you, but can you check if we want this functionality be added to this role or not?
If not, I'll add it to my own Nexcloud customization role.
Cc @staticdev

hello @jspaans91  I was reviewing the code at the moment

@aalaesar sorry for bothering you, but can you check if we want this functionality be added to this role or not?
If not, I'll add it to my own Nexcloud customization role.
Cc @staticdev

hello @jspaans91  and thank you for the idea and the contribution
I was reviewing the PR ealier and I think your idea is great, as nextcloud is coming with more and more shipped application. One may not want/need them all.
As the role is a little big, I tend to look for minimising the number of tasks to do things. 🤔
The two main tasks to do it are here, there is also some refinement required to simplify how to use the list of apps returned by the occ cli without using tasks.
I can make suggestions if you're ok 😃

Thanks a lot for the contribution again @jspaans91! I also like the feature. I made some suggestions already, not sure if would be in line with @aalaesar was also thinking. Feel free to comment or propose alternatives.

Hoi @jspaans91 just a vrindelijke ping. Did you had opportunity to take a look at our code review? What do you think about it?

@jspaans91 I will closed this PR for inactivity, if you are able to reply us I will be happy to reopen.


Hiding isn't working

I just noticed that hiding isn't working in v1.2.0.  I believe I've traced the issue to
hide = () => {
	if (!this.immune) {
		this.setState({ isPopoverShown: false });
	}

When it comes to hiding, the this.immune value is true, so that setState never fires.  I'm not sure if this.immune is needed any more after the last code change.  It seems that just having a single callback in the popover store effectively takes care of the same need, but I could be wrong.  If you take out this.immune it seems that the code gets more straightforward.

Oh, right, immune isn't needed anymore. Got to write those tests! :)

They're a pain, but they help. :)
On Tue, May 10, 2016 at 1:56 AM Dan Caragea<EMAIL_ADDRESS>wrote:

Oh, right, immune isn't needed anymore. Got to write those tests! :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/terebentina/react-popover/issues/10#issuecomment-218074092


Removed Duplicates Solves issue #20

Duplicate files list:

stack.txt(1)
stack.txt(1)
circular.txt(1)
tree1(1)
tree1.txt

PR for issue #20

:wave: Hey @chanpreet1999!
Thank you for discovering issue #20 and solving it! I'll go ahead and merge this. :)


refactorization of server to javascript

urgency level: 5/5
todo: by end of the day
assigned: TSherpa10
The backend is currently in Typescript, which is hard to implement with Express and Node.js. We need to revamp this to make it easier for type declaration.
Pros: Easier to write code, easier to reference legacy (Sone) code, easier to interact with code (familiarity)
Cons: error handling, library compatibility
todo: link to mongodb again.
Vote: 4/4 in favor.

Done, did this during RCOS 10/17/23.


Installation of keen-query and keen-tracker separately using JSPM

I ended up using keen-js like this for now. If you consider it useful, please merge

Handling this in #388 – sorry for the delay, and thanks for your help with this @OrKoN!


How to problematically obtain Level0 statefile information

Hello everyone. This is amazing work you have been doing. I am building a set of blueprint leveraging your great work and I am stump with something I have not been able to figure out yet. I will be referring to the landingzone repo found here:
https://github.com/bernardmaltais/eslz-template
In my L1_blueprint_base I would like to automate the addition of users to the AAD Security Group created by the launcher as part of L0_blueprint_launchpad. For this I need to get the proper resource group id created.
How can I do this? I have started to implement some test code as part of the L1_subscription_base/code/5-L1_blueprint_base_members.tf file. Maybe you can point me to some documentation on how to do this? There are more info in the comment section at the top of the code.
Regards,
Bernard

I have figured a way to do what I was looking for by using the L0 prefix to create the name of the security group.


in flight support

c.SingleInflight is deprecated: This is a no-op. Callers should implement their own in flight query caching if needed. See github.com/miekg/dns/issues/1449.

I believe this should be addressed using darvaza.org/cache

Basic in-flight support introduced on #49


Use Marshal.SecureStringToGlobalAllocUnicode

Relates: https://github.com/elastic/elasticsearch-net/pull/3806#issuecomment-502700127
This commit updates the SecureStrings implementation to use the cross platform
Marshal methods that deal with unmanaged unicode strings.
Implement IDisposable on BasicAuthenticationCredentials and dispose of SecureString
instances when ConnectionConfiguration is disposed.

Added a test for SecureStrings

Thank you very much, the nightly build works.


Fix LineGraphStyle, remove padding

Hi,
Like the start of your project, am using it in a health app.  I needed to fix a few things to make it usable for us, both in our iOS and watchOS targets.
Next, I'll be implementing a TimelineGraph View class, which takes data: [(Date, Double)]

Yeah, looks like everything is in order, thanks for the contribution. As far as the timeline goes, it might be easier to express it as a struct internally. The plan in the near future is to allow you either pass in either the raw data types like you can now or structs that will let you set extra properties like labels or what have you for specific points. Particularly with a line graph it might be useful to be able to individually provide labels for each of the points somehow


[BUG] Indices that include knn_vector field fail when search includes "fields" parameter

Description
Right now, for indices that contain a knn_vector, we are not able to support the fields option in queries. This is because in KNNVectorFieldType, we do not implement ValuesFetcher. We should implement it similar to the geo types that handle arrays directly.
curl -X GET "localhost:9200/target_index/_search?pretty" -H 'Content-Type: application/json' -d'
> {
>   "fields": ["*"],
>   "query": {
>     "match_all": {}
>   }
> }
> '

{
  "error" : {
    "root_cause" : [
      {
        "type" : "unsupported_operation_exception",
        "reason" : "KNN Vector do not support fields search"
      }
    ],
    "type" : "search_phase_execution_exception",
    "reason" : "all shards failed",
    "phase" : "query",
    "grouped" : true,
    "failed_shards" : [
      {
        "shard" : 0,
        "index" : "target_index",
        "node" : "grr7rDoPSwKXqQcpNrcbJQ",
        "reason" : {
          "type" : "unsupported_operation_exception",
          "reason" : "KNN Vector do not support fields search"
        }
      }
    ],
    "caused_by" : {
      "type" : "unsupported_operation_exception",
      "reason" : "KNN Vector do not support fields search",
      "caused_by" : {
        "type" : "unsupported_operation_exception",
        "reason" : "KNN Vector do not support fields search"
      }
    }
  },
  "status" : 500
}

Repro steps
curl -X PUT "localhost:9200/target_index" -H 'Content-Type: application/json' -d'
{
  "settings" : {
    "number_of_shards" : 1,
    "number_of_replicas" : 0,
    "index.knn": true
  },
  "mappings": {
       "properties": {
       "target_field": {
           "type": "knn_vector",
           "dimension": 2,
           "method": {
               "name":"hnsw",
               "engine":"nmslib",
               "space_type": "l2",
               "parameters":{
                 "m":20,
                 "ef_construction": 23
               }
           }
      }
   }
  }
}
'

curl -X PUT "localhost:9200/_bulk" -H 'Content-Type: application/json' -d'
{ "index": { "_index": "target_index" } }
{ "target_field": [1.5, 5.5]}
{ "index": { "_index": "target_index" } }
{ "target_field": [2.5, 3.5]}
{ "index": { "_index": "target_index" } }
{ "target_field": [4.5, 5.5]}
{ "index": { "_index": "target_index" } }
{ "target_field": [1.5, 5.5]}
{ "index": { "_index": "target_index" } }
{ "target_field": [1.5, 5.5]}
{ "index": { "_index": "target_index" } }
{ "target_field": [2.5, 3.5]}
'

curl localhost:9200/target_index/_refresh?pretty
curl -X GET "localhost:9200/target_index/_search?pretty" -H 'Content-Type: application/json' -d'
{
  "fields": ["*"],
  "query": {
    "match_all": {}
  }
}
'

This is not a bug, you are trying to use efficient knn filtering, but used the nmslib engine, which does not support it. If you change the engine to lucene or faiss it should work as expected.
Here is the documentation for it: https://opensearch.org/docs/latest/search-plugins/knn/filter-search-knn/

@OliverLiebmann this is a bug - it doesnt have anything to do with filtering - more so use of fields in the search payload. See:
curl -X GET "localhost:9200/target_index/_search?pretty" -H 'Content-Type: application/json' -d'
{
  "fields": ["*"],
  "query": {
    "match_all": {}
  }
}
'

@jmazanec15 feel free to assign to me I understand the required fix.

Thanks @samuel-oci ! Assigned

@samuel-oci are you taking a look at this one?


Fix up a few shellcheck warnings

I ran platform-upgrade through shellcheck and implemented all the recommended fixes.

Looks reasonable. Want to fix this up and perhaps merge with the other PR?

I already implemented everything from #24 in here, so if you're happy with the "big one" then I'll close this.


Sidebar path

Describe the bug
The sidebar does not support custom sorting
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
Hope you can customize the sorting
Screenshots
http://jinlan.synology.me:25227/image/JFD
Desktop (please complete the following information):

OS Version: [e.g. Windows 10 20H2 19042.804]
App version: [e.g. v<IP_ADDRESS>]

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

Thanks for this report. Please describe this issue with more details and post the log files.

@gave92 He is request a new feature: resorting the Pinned directory in sidebar.

Thanks for clarifying, I misread this.
Unfortunately the requested feature is already implemeted but disabled due to a bug in the WinUI library


Native-sql-engine of oap-1.1-spark-3.1.1 cannot pass stability test.

We using the latest code of oap-1.1-spark-3.1.1 to run TPC-DS all queries for 10 rounds. The data scale is 1.5TB and the cluster contains 1 master and 3 workers. Each worker contains  512GB DRAM. The configuration of spark is showed below:

Q23b.sql cannot pass on the first round and error message is like:

@zhouyuan @zhixingheyi-tian Please help to track the issue. Thanks!


Custom Input with Multiline throws error on Change

Hey,
While working with a custom Input with multiline prop, the Scroll View is throwing an error when I try to change the value:

The sourcemap pointed to these lines in index.js:
// onChange 在 onContentSizeChange 之前触发
    // onChange 在 onSelectionChange 之后触发
    _onChange = event => {
        const target = event.target || event.currentTarget;
        const inputInfo = this._getInputInfo(target);
        inputInfo.text = event.nativeEvent.text;
    }

This must be something while deepcloning, I'm not sure. Custom TextInput without multiline prop works properly.
I'm using version 1.7.0
Thanks!
UPDATE
It seems that it's not returning the text value on onChange

at 232 line:
    _getInputInfo(target) {
        return this._inputInfoMap[target] = this._inputInfoMap[target] || {};
    }

this._getInputInfo(target) It'll never be undefined, do you modify the source code?
Or try the latest version v1.7.3

No I didn't. Also, I'm still getting the error with the 1.7.3.

This error is currently circumvented

@baijunjie Should I try 1.7.4?

@luco Yes

@baijunjie Worked perfectly. Thanks


PHP variant has no session support.

Running this php file:
# index.php
<?php
die(session_start());

returns
PHP message: PHP Fatal error:  Uncaught Error: Call to undefined function session_start() in /srv/index.php:3

PHP docs indicate that PHP sessions are enabled by default.  Alpine ships PHP sessions as a separate package, php7-session.
I think that package should be added to the abiosoft/caddy:php image but if not, we should document how to add it.
It may be a good idea to compare php7 -i in abiosoft/caddy:php to php -i in php:fpm-alpine and add a few things that people expect to be there.
I solved this for my case with
FROM abiosoft/caddy:php
RUN apk --update add --no-cache \
    php7-session \
    && rm -rf /var/cache/apk/*

Fixed. Kindly pull the image again.
Thanks.


The operation couldn't be completed (NSURLErrorDomain error)

I have this error message on ClubHouse
with mitmProxy ...

You should read the section: https://github.com/Seia-Soto/clubhouse-api#bypassing-ssl-pinning

You should read the section: https://github.com/Seia-Soto/clubhouse-api#bypassing-ssl-pinning

already read. I've followed every step...

Can I get your tweak settings and iOS version stuff?

I am sorry to say that I won't help reverse engineering and setting up MITM proxy at all time except for only some situation, but for some cases such as Clubhouse updated and we need updates on our code to mitigrate user bans. Also, it seems your issue is temporal. Please, close if your issue isn't ciritical.

Checked application again on my side and nothing changed. I am closing this issue due to inactivity.

I have this error message on ClubHouse
with mitmProxy ...

Kindly read the Bypassing ssl in the readme again to fix your issue.