show waypoints on livemap only works if header is in range

I like the new feature "show additional waypoints on live map", but: if the header is coming out of range, the waypoint disappeares... respectively if header ist far away, the e.g. "Final" will never appear...
the 2nd thing about this new feature: I find a waypoint on the livemap, klick on it, lets say it is named  "final", an i dont know, which cache it belongs to....

the first part was a known limitation of the current implementation.
the second part could be solved by a button/label on the details screen.

What about a visual connection between the Waypoints and the corresponding Cache on the live map? A thin (1px) line from the Cache to all its Waypoints for example. This way its done in Locus and its working nicely, from what i have experienced.
Maybe make it an option. And wasn't there a discussion about displaying solved mystery finals somewhere? It would be nice if one would have the option to only show waypoints that are "somehow" marked as Finals. This way you could show all solved Finals on the live map, not missing them when you are passing by.

There are quite some ideas and options floating around right now and I think it is good to get all these ideas. I need still some time to digest them and form a concept how this could be designed in an efficient and understandable manner. One thing is, for example, that we should look into the possibilities for modifying the base cache markers by different modifiers (like found. own, final location, stored, ....) instead of producing and storing graphics that contain all possible combinations - that would enlarge c:geo considerably.
So keep these ideas coming and we should start a discussion how this could all fit together nicely without another set of options :)


[BUG] Error with number 77

Describe the bug
When the OCR reads "77", it writes "717", I've noticed that this problem only occurs in the "Rss Gathered" field
Edit: Sometimes same problem for "Helps" field, and for number finish per "1", script write "11"
To Reproduce
Just run a scan lol
Expected behavior
The script should write "77" and not "717"
Your Setup (please complete the following information):

OS: Android
Bluestacks Version: <IP_ADDRESS>2
Android of Bluestacks: Pie 64-bit
Tracker Version: v3.0.0

Hm good to know, but I don't know how much can be done for that. Maybe i can try some more teseract settings or different pre processing. The problem is you can't auto detect if those fileds messed up (i think this can also happen for the "dead" field).

Yes I know it's complicated to fix, I haven't noticed any error on the "dead" fields yet, I checked each field of my scan and the error doesn't happen all the time it's strange


Implement interface to insert and retrieve info from table 'storages' and 'storage_policy'

1.add insertStorageTables() and getStorageCapacity() to  insert and retrieve info from table 'storages' in DBAdapter
2.add testInsertStorageTables() to test  insert and retrieve info
3.add insertStoragePolicyTable() and getStoragePolicyTableItem() to  insert and retrieve info from table 'storage_policy' in DBAdapter
4.add test method testInsertStoragePolicyTable() to test  insert and retrieve info
5.delete variable mapStorageCapacity ,because it is useless

@yanpingyou upstream has updated, please resolve the conflict.


Uploading sourcemaps to rollbar via capistrano

This was implemented in a bit of a haste. Let me know if that's something you guys are interested in.

Looks interesting! We'll review :)

Looks great. Thanks for PR


IOU - Settled up elseware displayed in LHN when settled using paypal

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

As userB, add a Paypal.me handle i.e tomrhys
As userB, send a request to userA for $1.00 such that userB is owed
As userA, navigate to the iouReport settle the request by selection paypal option

Expected Result:
Report comment updated as settled $ using Pay pal me for both parent and child
Actual Result:
Parent report updated as settled elsewhere
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/7043d8c1-cf12-4fd3-9c1b-d470980ed4e8
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU settled up elsewhere displayed after settling with paypal
What is the root cause of that problem?
The backend will return ...using PayPal.me  but our regex explicitly matches only Paypal.me
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/libs/ReportUtils.js#L1103
What changes do you think we should make in order to solve the problem?

we can easily fix this issue by updating the regex to / Paypal.me$/i adding i which is case insensitive.
What alternative solutions did you explore? (Optional)

This may be a duplicate of https://github.com/Expensify/App/issues/23022. This one was created first, but that one has already had the external label added so I'm going to hold off until we confirm whether they are dupes.

Dupe off https://github.com/Expensify/App/issues/23022


feat: Add Batch Processing Utility

Issue number: #168
Summary
Changes
This PR adds the Batch Processing Utility to Powertools for AWS Lambda (.NET).
With this, we can bring support for simple utilization of the AWS Lambda function response type ReportBatchItemFailures where partial batch item failures can be reported, and thereby help reducing the number of items that are being re-processed. The utility will automatically monitor the processing of each item within a batch, and report which items failed to be processed. This enable developers to focus on writing business logic while benefiting from the Batch Processing Utility automatically doing the reporting of partial failures within a batch.
User experience
Add the nuget package:
dotnet add package AWS.Lambda.Powertools.BatchProcessing

Consuming batches from SQS
Function handler using attribute-based configuration:
[BatchProcesser(RecordHandler = typeof(CustomSqsRecordHandler))]
public BatchItemFailuresResponse SqsHandlerUsingAttribute(SQSEvent _)
{
    return SqsBatchProcessor.Instance.ProcessingResult.BatchItemFailuresResponse;
}

Implementation of CustomSqsRecordHandler:
public class CustomSqsRecordHandler : IRecordHandler<SQSEvent.SQSMessage>
{
    public async Task<RecordHandlerResult> HandleAsync(SQSEvent.SQSMessage record, CancellationToken cancellationToken)
    {
        /*
         * Your business logic.
         * If an exception is thrown, the record will be marked as a partial batch item failure.
         */
        return await Task.FromResult(RecordHandlerResult.None);
    }
}

With this, the framework automatically creates an instance of a batch processor and uses the provided record handler for the per-record processing logic. All logic around catching exceptions and keeping track of partial batch item failures is handled automatically by the framework. On top of that, there are FIFO specific logic around handling partial failures - from the docs:

If you're using this feature with a FIFO queue, your function should stop processing messages after the first failure and return all failed and unprocessed messages in batchItemFailures. This helps preserve the ordering of messages in your queue.

... and this is also handled automatically by the framework.
Consuming batches from DynamoDB Streams
[BatchProcesser(RecordHandler = typeof(CustomDynamoDbStreamRecordHandler))]
public BatchItemFailuresResponse DynamoDbStreamHandlerUsingAttribute(DynamoDBEvent _)
{
    return DynamoDbStreamBatchProcessor.Instance.ProcessingResult.BatchItemFailuresResponse;
}

Consuming batches from Kinesis Data Streams
[BatchProcesser(RecordHandler = typeof(CustomKinesisDataStreamRecordHandler))]
public BatchItemFailuresResponse KinesisDataStreamHandlerUsingAttribute(KinesisEvent _)
{
    return KinesisDataStreamBatchProcessor.Instance.ProcessingResult.BatchItemFailuresResponse;
}

Consuming batches using the utility
public async Task<BatchItemFailuresResponse> HandlerUsingUtility(SQSEvent sqsEvent)
{
    var result = await SqsBatchProcessor.Instance.ProcessAsync(sqsEvent, RecordHandler<SQSEvent.SQSMessage>.From(x =>
    {
        Logger.LogInformation($"Inline handling of SQS message with body: '{x.Body}'.");
    }));
    return result.BatchItemFailuresResponse;
}

Consuming batches using processor and handler from IoC
public async Task<BatchItemFailuresResponse> HandlerUsingUtilityFromIoc(SQSEvent sqsEvent)
{
    var batchProcessor = Services.Provider.GetRequiredService<CustomSqsBatchProcessor>();
    var recordHandler = Services.Provider.GetRequiredService<CustomSqsRecordHandler>();
    var result = await batchProcessor.ProcessAsync(sqsEvent, recordHandler);
    return result.BatchItemFailuresResponse;
}

Configuration
The Batch Processing Utility supports the following configuration:

MaxDegreeOfParallelism This is used to control the parallelism of the batch item processing. With a value of 1, the processing is done sequentially (default). Sequential processing is recommended when preserving order is important - i.e. with SQS FIFIO queues. With a value > 1, the processing is done in parallel. Doing parallel processing can enable processing to complete faster, i.e., when processing does downstream service calls. With a value of -1, the parallelism is automatically configured to be the vCPU count of the Lambda function. Internally, the Batch Processing Utility utilizes Parallel.ForEachAsync Method and the ParallelOptions.MaxDegreeOfParallelism Property to enable this functionality.

ErrorHandlingPolicy This is used to control the error handling policy of the batch item processing. With a value of DeriveFromEvent (default), the specific BatchProcessor, determines the policy based on the incoming event. For example, the SqsBatchProcessor looks at the EventSourceArn to determine if the ErrorHandlingPolicy should be StopOnFirstBatchItemFailure (for FIFO queues) or ContinueOnBatchItemFailure (for standard queues). For StopOnFirstBatchItemFailure the batch processor stops processing and marks any remaining records as batch item failures. For  ContinueOnBatchItemFailure the batch processor continues processing batch items regardless of item failures.

The configuration items can be set in different ways:

Using POWERTOOLS_ environment variables.
Using the BatchProcesser attribute.
Using the ProcessingOptions object passed to BatchProcessor.ProcessAsync().

Checklist
Please leave checklist items unchecked if they do not apply to your change.

[x] Meets tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
No
RFC issue number: #168
Checklist:

[x] Migration process documented
[x] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

The PR is now more or less ready to go :) Will sync with @hjgraca before submitting.

Codecov Report
Patch coverage: 47.78% and project coverage change: -2.06 :warning:

Comparison is base (38130a1) 69.31% compared to head (1ec18a0) 67.26%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #337      +/-   ##
===========================================
- Coverage    69.31%   67.26%   -2.06%
===========================================
  Files           79       96      +17
  Lines         3464     3803     +339
===========================================
+ Hits          2401     2558     +157
- Misses        1063     1245     +182

Flag
Coverage Δ

unittests
67.26% <47.78%> (-2.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ertools.BatchProcessing/BatchProcesserAttribute.cs
0.00% <0.00%> (ø)

...rocessing/DynamoDb/DynamoDbStreamBatchProcessor.cs
0.00% <0.00%> (ø)

...rocessing/Internal/BatchProcessingAspectHandler.cs
0.00% <0.00%> (ø)

...cessing/Kinesis/KinesisDataStreamBatchProcessor.cs
0.00% <0.00%> (ø)

...Lambda.Powertools.BatchProcessing/RecordHandler.cs
0.00% <0.00%> (ø)

....Powertools.BatchProcessing/RecordHandlerResult.cs
0.00% <0.00%> (ø)

...Powertools.Common/Core/PowertoolsConfigurations.cs
80.95% <0.00%> (-13.50%)
:arrow_down:

...owertools.BatchProcessing/Sqs/SqsBatchProcessor.cs
88.88% <88.88%> (ø)

...ambda.Powertools.BatchProcessing/BatchProcessor.cs
95.04% <95.04%> (ø)

...tools.BatchProcessing/BatchItemFailuresResponse.cs
100.00% <100.00%> (ø)

... and 8 more

... and 1 file with indirect coverage changes

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

Thanks for reviewing @amirkaws


Java.Lang.Character

Add functions and test cases to : Java.Lang.Character.cpp , Java.Lang.CharacterTest.cpp

Please only comment in header file and check FIXME again.

About FIXME in function digit,  it's not my code, but I will check it when I finish java.lang.Character !

i mean you should only comment in header file
you comment in both header and source file, they are duplicate.


[plan] Milestone 2

Milestone 2 : Production-level implementation of Milestone 1 on Cosmos-SDK stargate version

1) Backend

Rebase and Refactoring

Rebase codebase to Cosmos-SDK version equal or later than 0.40.0-RC4
Refactoring codebase for rule-based decimal management and better error handling

Functional Improvement

Half-half fee payment for consistent pool price during fee payment process
Implement spam prevention methodologies

flat ATOM fees for each Liquidity Module message
proportional fees for each liquidity pool withdrawal

Utilities

RPC/CLI implementation
Genesis export

Maintenance

Deal with issues on the repository from AiB or other community members
Global test coverage +80% with below range of files

all *.go files in x/liquidity/keeper and x/liquidity/types
excluding *_test.go, *.pb.go and *.pb.gw.go

2) Test and Simulation

Implement various invariant check functions for verifying wide range of error possibilities
Implement scenario-based random simulation test program for

extensive edge case finding
testing the validity of the Liquidity Module model with various economic assumptions
provide testing report with simulation results

3) Documentations

Update spec documentation for new functionalities
Write white paper for Liquidity Module

mathematical explanation documentation for Liquidity Module calculations
comparison with Uniswap, Reasoning for Liquidity Module mechanisms
comprehensive economic simulation result for validation of Liquidity Module model

4) Frontend

Finalize wire frame for web interface of Liquidity Module
Provide sample frontend source code for reference (React)
Provide sample API communication codebase for frontend UI (RPC or gRPC)
Provide sample Keplr integration (Javascript)
Provide simple web interface connected to testnet for entire user-flow testing of Liquidity Module

schedule

1~2 week

Codebase

Implement spam prevention methodologies : flat fees for each Liquidity Module message (in Atom)
Strategy for rule-base decimal management
Improve test coverage

Simulation

Simulation spec and range

3~4 week
5~6 week
7~8 week

Milestone2 SoW Items

Turn initial proof of concept into a production level module based on Stargate version 0.40.0.

Code coverage of minimum 80% with all tests passing with success;

continuous

Unit testing on CLI functions that pass with success;

continuous

Qualitative review and enhancement of current unit tests; (qualitative review?)

continuous

Use of appropriate production-level error trapping, handling and logging;

continuous

Add the following additional features:

Pool Identification;

spec update and implementation by 2021-01-29

Half/Half Fee Payments;

done

Fee structure for Spam;

spec update and implementation by 2021-01-29

Remaining features from milestone one in the specification e.g. Event emission;

spec document by 2021-02-05
codebase update by 2021-02-12

Strategy for rule-based decimal management.

spec document by 2021-02-05
codebase update by 2021-02-19

"Light" paper outlining

Rationale for module design;
Explanation of how the model works, through the use of order book and AMM technologies;
Explanation of the Swap matching algorithm;
Outline assumptions and constraints of the model;
Comparison analysis versus other liquidity engines.
Version 0.1 by 2021-01-29
Version 0.2 by 2021-02-26

Production-level developers reference documentation that

Covers code’s public API and deployment;
Is based on Golang’s Godoc convention: https://blog.golang.org/godoc;
by 2021-02-19

HTTP API specification implementation based on the specification of Milestone One. This will
include gRPC as well as HTTP API.

by 2021-02-19

Rosetta API integration

by 2021-02-26

Testing, that will include the following:

Simulation testing using Cosmos SDK's functionality provided by the x/simulation module;

simapp simulation implementation by 2021-02-12
economic simulation test result by 2021-02-19
economic simulation report by 2021-02-26

Invariant checks;

spec document by 2021-01-29
implementation by 2021-02-12

Performance testing, particularly around End-Block.

performance testing by 2021-01-29
full report by 2021-02-05

Milestone2 SoW Weekly Schedule

by 2021-01-29

Pool Identification : spec update and implementation
Fee structure for Spam : spec update and implementation
Light Paper : V0.1 Finalize
Invariant checks : spec document
Performance testing : testing execution

by 2021-02-05

Remaining features from milestone one in the specification e.g. Event emission : spec update
Strategy for rule-based decimal management : spec document
Performance testing : report

by 2021-02-12

Remaining features from milestone one in the specification e.g. Event emission : implementation
Simulation testing using Cosmos SDK's functionality : simapp simulation implementation
Invariant checks : implementation

by 2021-02-19

Strategy for rule-based decimal management : implementation
Production-level developers reference documentation : finalize
HTTP API specification implementation including gRPC as well as HTTP API : implementation
Economic simulation result

by 2021-02-26

Light Paper : V0.2 finalize
Rosetta API integration
Economic simulation report

Milestone2 SoW Items

Turn initial proof of concept into a production level module based on Stargate version 0.40.0.

Code coverage of minimum 80% with all tests passing with success;

continuous

Unit testing on CLI functions that pass with success;

continuous

Qualitative review and enhancement of current unit tests; (qualitative review?)

continuous

Use of appropriate production-level error trapping, handling and logging;

continuous

Add the following additional features:

Pool Identification;

spec update and implementation by 2021-01-29

Half/Half Fee Payments;

done

Fee structure for Spam;

spec update and implementation by 2021-01-29

Remaining features from milestone one in the specification e.g. Event emission;

spec document by 2021-02-05
codebase update by 2021-02-12

Strategy for rule-based decimal management.

spec document by 2021-02-05
codebase update by 2021-02-19

"Light" paper outlining

Rationale for module design;
Explanation of how the model works, through the use of order book and AMM technologies;
Explanation of the Swap matching algorithm;
Outline assumptions and constraints of the model;
Comparison analysis versus other liquidity engines.
Version 0.1 by 2021-01-29
Version 0.2 by 2021-02-26

Production-level developers reference documentation that

Covers code’s public API and deployment;
Is based on Golang’s Godoc convention: https://blog.golang.org/godoc;
by 2021-02-19

HTTP API specification implementation based on the specification of Milestone One. This will
include gRPC as well as HTTP API.

by 2021-02-19

Rosetta API integration

by 2021-02-26

Testing, that will include the following:

Simulation testing using Cosmos SDK's functionality provided by the x/simulation module;

simapp simulation implementation by 2021-02-12
economic simulation test result by 2021-02-19
economic simulation report by 2021-02-26

Invariant checks;

spec document by 2021-01-29
implementation by 2021-02-12

Performance testing, particularly around End-Block.

performance testing by 2021-01-29
full report by 2021-02-05

Milestone2 SoW Weekly Schedule

by 2021-01-29

Pool Identification : spec update and implementation
Fee structure for Spam : spec update and implementation
Light Paper : V0.1 Finalize
Invariant checks : spec document
Performance testing : testing execution

by 2021-02-05

Remaining features from milestone one in the specification e.g. Event emission : spec update
Strategy for rule-based decimal management : spec document
Performance testing : report

by 2021-02-12

Remaining features from milestone one in the specification e.g. Event emission : implementation
Simulation testing using Cosmos SDK's functionality : simapp simulation implementation
Invariant checks : implementation

by 2021-02-19

Strategy for rule-based decimal management : implementation
Production-level developers reference documentation : finalize
HTTP API specification implementation including gRPC as well as HTTP API : implementation
Economic simulation result

by 2021-02-26

Light Paper : V0.2 finalize
Rosetta API integration
Economic simulation report

outdated


Layer Legend

Describe the Task
The users need to see a legend for the layers displayed.
Legend for IAPP sites, legend for record types.
Consider a static page to show all the layer symbology
Need to create the styling first before the legend.

@CrystalChadburn I think the new map maker page covers this??

@micheal-w-wells  no this is the pdf of the two letter codes for the species.  I just have to make it.  then we can have a button to upload it on the app itself or just on our website for now.


I installed the addon and it wont show up on preferences

Please let me know the blender version.
Meanwhile you can try - unzip the zip file and install the py file directly.


Feature: Scorecard Dependency-diff API, CLI and Action

Is your feature request related to a problem? Please describe.
The scorecard Action only reports the Scorecard results for the repository under analysis. The project will surface scorecard results for the dependencies for new pull requests.
Describe the solution you'd like

Add an API design that can fetch direct dependency changes (diffs) from a pull request using the GitHub Dependency Review API, surface Scorecard checks and scores for those dependencies and return the results.

Add a new Scorecard sub-command CLI, scorecard dependency-diff, which uses the above API and outputs the result dependencies as JSON.

Add a new Scorecard Action running on a pull request, use the API to visualize the dependency-diff and their Scorecard checking results in the PR comments as markdown.

Additional context
Assignee: @aidenwang9867

Is your feature request related to a problem? Please describe. The scorecard Action only reports the Scorecard results for the repository under analysis. The project will surface scorecard results for the dependencies for new pull requests.
Describe the solution you'd like

Add a design that can fetch direct dependency changes (diffs) from a pull request using the GitHub Dependency Review API, get dependencies of these direct dependencies (indirect ones) and vulnerabilities of both direct and indirect dependencies.
Information regarding Dependencies and vulnerabilities can be retrieved from the BigQuery dataset Open Source Insights.
Visualize the result to the PR comments like what CodeCov does, with information showing if a dependency is removed or added. Also, give vulnerability alerts (VulnID, VulnSource, CVSS3Score, Severity, etc.) either together in the comments, or in the repo Security tag.
Blocks the PR if vulnerabilities found.

Describe alternatives you've considered
Additional context This design is Scorecard-focused and is expected to be used in a pull request as a part of the Scorecard Action pre-submit check automation.
With this one in place, the Scorecard Action will be further empowered to check changes of dependencies between a base commit and a head commit in GitHub pull requests, giving detailed alerts to repository maintainers if vulnerabilities (CVEs, GHSAs, OSVs, etc.) are found in the added/updated dependencies.
Assignee: @aidenwang9867

This is cool. BigQuery needs authentication to run these checks. Would we require the user to provide those credentials?

I also wonder whether it'd be useful to have a

command line option to get the results for a dependencies, rather than run on a project itself
a --from-bq option in Scorecard to fetch a result from BQ instead of running scorecard on a project itself

Note that the project is focused on exposing the functionality in the Action first. I'm asking the questions above out of curiosity :-)

Yeah I'm preparing for my first pull request and get quite confused now where I should put my code to. Currently I'm testing my code locally in a main function and visualize the results to the StdOut.

Is your feature request related to a problem? Please describe. The scorecard Action only reports the Scorecard results for the repository under analysis. The project will surface scorecard results for the dependencies for new pull requests.
Describe the solution you'd like

Add a design that can fetch direct dependency changes (diffs) from a pull request using the GitHub Dependency Review API, get dependencies of these direct dependencies (indirect ones) and vulnerabilities of both direct and indirect dependencies.
Information regarding Dependencies and vulnerabilities can be retrieved from the BigQuery dataset Open Source Insights.
Visualize the result to the PR comments like what CodeCov does, with information showing if a dependency is removed or added. Also, give vulnerability alerts (VulnID, VulnSource, CVSS3Score, Severity, etc.) either together in the comments, or in the repo Security tag.
Blocks the PR if vulnerabilities found.

Describe alternatives you've considered
Additional context This design is Scorecard-focused and is expected to be used in a pull request as a part of the Scorecard Action pre-submit check automation.
With this one in place, the Scorecard Action will be further empowered to check changes of dependencies between a base commit and a head commit in GitHub pull requests, giving detailed alerts to repository maintainers if vulnerabilities (CVEs, GHSAs, OSVs, etc.) are found in the added/updated dependencies.
Assignee: @aidenwang9867

This is cool. BigQuery needs authentication to run these checks. Would we require the user to provide those credentials?

Thanks Naveen and, great question :D
My answer is no - we won't ask the users to provide credentials.
Instead, I am actively building a REST API service on a GKE with an external IP, putting the authentication & authorization process on the service so that the Scorecard project can connect to my REST APIs to query the BigQuery dataset.

Instead, I am actively building a REST API service on a GKE with an external IP, putting the authentication & authorization process on the service so that the Scorecard project can connect to my REST APIs to query the BigQuery dataset (without additional credentials needed).

OK, Probably that should have been called out. Is there a design for this?  So that we can understand and have a consensus.

Instead, I am actively building a REST API service on a GKE with an external IP, putting the authentication & authorization process on the service so that the Scorecard project can connect to my REST APIs to query the BigQuery dataset (without additional credentials needed).

OK, Probably that should have been called out. Is there a design for this? So that we can understand and have a consensus.

I am doing some modifications on my design doc, and then i'll share it with you via email by the end of tomorrow, so that we can discuss it in Thursday's meeting.

Cool @aidenwang9867! Can you please post it in the google groups? Thanks

np, but seems I'm not in the group yet, I'll ask Laurent to drag me in.

Based on what was presented in the Bi-Weekly scorecard meeting and the design document.
Here are the things that we have discussed.

v0 was supposed to be only running scorecard against dependencies for the repository and not build the REST endpoint for scorecard.
It was discussed that GitHub API for figuring out the dependencies doesn't support transitive dependencies for ecosystems like python, npm, and a few others.
deps.dev provides transitive dependencies for a given repository
It was discussed as part of v0 utilizing bot the GitHub API and deps.dev to get all the dependencies. But if a repository adds a new dependency in a PR then deps.dev wouldn't be aware of that and so this new check would miss those transitive dependencies.

Based on what was presented in the Bi-Weekly scorecard meeting and the design document.
Here are the things that we have discussed.

v0 was supposed to be only running scorecard against dependencies for the repository and not build the REST endpoint for scorecard.
It was discussed that GitHub API for figuring out the dependencies doesn't support transitive dependencies for ecosystems like python, npm, and a few others.
deps.dev provides transitive dependencies for a given repository
It was discussed as part of v0 utilizing bot the GitHub API and deps.dev to get all the dependencies. But if a repository adds a new dependency in a PR then deps.dev wouldn't be aware of that and so this new check would miss those transitive dependencies.

Thank you so much Naveen, for summarizing the things we discussed!
cc @laurentsimon  @azeemshaikh38

It was discussed as part of v0 utilizing bot the GitHub API and deps.dev to get all the dependencies. But if a repository adds a new dependency in a PR then deps.dev wouldn't be aware of that and so this new check would miss those transitive dependencies.

by "repository" you mean the repository with scorecard Action installed? I suppose you mean a dependency adds a new dependency which is a transitive dependency for the repository; and it's likely that deps.dev will not know about it because they run their scan every week or so. Is this what you meant?

It was discussed as part of v0 utilizing bot the GitHub API and deps.dev to get all the dependencies. But if a repository adds a new dependency in a PR then deps.dev wouldn't be aware of that and so this new check would miss those transitive dependencies.

by "repository" you mean the repository with scorecard Action installed?
Or do you mean:

a dependency P1 of the repository adds a new dependency P2
repository adds P1 in a PR (P2 is therefore a transitive dependency)
deps.dev has stale data because they only refresh their dataset every X days.

Is this what you meant?

@naveensrinivasan in case u missed this message from Laurent :D

Recent release of dependency-review-action supports this, but there may be edge case for projects not in our API dataset.
@jeffmendoza (paraphrased):

It should be seen as a success when GitHub adds a feature that Scorecard/Allstar previously developed and added to the release notes (when we deprecate this code path).

@spencerschrock proposed the deprecation, so I'm assigning this to him.
Note that we also need to check on whether the https://github.com/ossf/scorecard-dependencyanalysis should be archived.


Line highlight fails inside merge conflict areas when background color set in theme

VSCode Version: Code 1.19.1 (0759f77bb8d86658bc935a10a64f6182c5a1eeba, 2017-12-19T09:46:23.884Z)
OS Version: Windows_NT x64 10.0.16299

Highlighting of current line works fine inside inline merge conflict blocks, but if the color theme changes the background color of these areas, the line highlighting disappears.
Steps to Reproduce:

Choose a theme with line highlighting, I used Solarized Dark
Add a 'merge conflict' to the editor, for instance

<<<<<<<
check if
line highlighting
works here
=======
>>>>>>>

Check that line highlighting works inside the merge conflict area

Set the background color for merge conflicts, for instance add to settings:

    "workbench.colorCustomizations": {
		"merge.currentHeaderBackground": "#207972",
		"merge.currentContentBackground": "#0D4A4E"
    },

Now current line is no longer highlighted inside the merge conflict area

Can you add documentation about "diffEditor.insertedTextBorder" as well?  I just hit this today.

@bobbrow "diffEditor.insertedTextBorder" is documented here: https://code.visualstudio.com/docs/getstarted/theme-color-reference#_diff-editor-colors
I don't think the border needs to be transparent, does it?

Bah, I copied the wrong value.  I meant diffEditor.insertedTextBackground

@jibsen had made a similar comment above, but it looks like the documentation was not updated for this property or diffEditor.removedTextBackground

Thanks @bobbrow, I updated the descriptions.

Thanks!


Fix/Quick start

Please briefly answer these questions:

what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)

Previously, Quick Start only supported running on Linux. For developers using macOS (especially those equipped with Apple silicon Macs), cross-compilation was required. This PR will support macOS users in performing Quick Start.

what changes does this pull request make?

Update QUICK_START.md.
Moved the process of compiling on the host machine to Docker in quick-start.

are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

Please modify the commit message in that our CHANGELOG.md is generated from the git commit logs. Some commit messages in this pr, like "fix: fix macOS quick start", "fix: ignore target folder to reduce size of context copied into docke…", will mislead git-cliff to generate incorrect CHANGELOG. How about "feat: add macOS quick start support" and "chore: ignore target folder to reduce size of context copied into docke". FYI: https://github.com/datenlord/Xline/blob/master/CONTRIBUTING.md#commit-message-guidelines

Please modify the commit message in that our CHANGELOG.md is generated from the git commit logs. Some commit messages in this pr, like "fix: fix macOS quick start", "fix: ignore target folder to reduce size of context copied into docke…", will mislead git-cliff to generate incorrect CHANGELOG. How about "feat: add macOS quick start support" and "chore: ignore target folder to reduce size of context copied into docke". FYI: https://github.com/datenlord/Xline/blob/master/CONTRIBUTING.md#commit-message-guidelines

Sorry, I didn't carefully consider the commit type and have already made the necessary changes.

We can merge chore: ignore unnecessary files & fix typos and chore: remove unnecessary builds into one commit


Third blog heading

This has conflicts. Will close now & try again next time.  Make your commits longer so you avoid this.
Combine 3-4 sentences all in 1 commit together.


Release 2022.12.24

This release adds the following PRs:

#28
#29

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


New release

Hey there,
Would it be possible to release a new version with the latest changes on main?
It would help me if I can use the changes of my PR and it might even help with this issue:
https://github.com/ImperiumMaximus/ts-lsp-client/issues/22

Hello, new version 1.0.3 with your change has been published.


navbar with social icons in tailwindCSS

Description
**Need to create code for navbar with social icons with Image as per below image. **

The final code should be placed under path tailwind_components/navbars with file name as navbar_social_icons.jsx
The file as to be imported to tailwind_components/navbars/collection.js file as per example and added to the navbarCollections list
Post these process, go to components tab (components/navbars) in website and check whether the codepen is updated

The navbar should be mobile responsive with hamburger menu (make it simple)

For PR reference - https://github.com/jsvigneshkanna/tailwind_ui_components/pull/76/files
For any issue while creating / importing the component code file, check below sample files
https://github.com/jsvigneshkanna/tailwind_ui_components/tree/master/tailwind_components/buttons
For icons please refer - https://heroicons.com/
Screenshot of the tailwind component

Are you taking this fetaure implementaion
No, This is up for grabs

Hi @jsvigneshkanna again, I can work on this, can you please assign me?

Hi @jsvigneshkanna , I can work on this, can you please assign me?

Hi @uttampandit ,
Yeah sure, also check for other navbars if you can build those

@udaykahar ,
Can you please take issue #157

@udaykahar , comment over there #157 , will asign that to you

sure

Hello sir, can you assign this issue to me?

Welcome @Viren0602
Please make sure the final result is same as screenshot and mobile responsive

No plagiarism to be followed

If this issue is still open I would love to take it on.

Sure @mdb1710 , please make sure this is not plagiarised from other sites
This should be mobile responsive

Sir I have raised a PR for #156 again. Now its not a plagiarised from other sites.


Group / Series / Property tags

When setting tags on these documents, need to create them as an array.
Currently, if you put "tag1 tag2 tag3" in the text field, you get attributes.tags = ["tag1 tag2 tag3"].  Should actually be ["tag1", "tag2", "tag3"].   So this code should probably split() the textfield string, and then filter out any blank strings.

I think this is what we're currently doing.
I did a little test with the Series creation workflow, logging the data we would send for a newly created Doc. This is what I see:

As another sanity check, I inspected the data used to present the form in the WordPress dashboard and it seems to be coming back properly.

So I believe we are sending and receiving the proper data structures.
Perhaps there are more details for triggering the bug or some circumstance I haven't accounted for. Let me know!

  I was assuming they were space-separated.  Maybe some hint text for that field would solve my ignorance, then.

Oh, I see! Yep, we should definitely add a hint.  Thanks!

Added a hint in the same way that WordPress does for post tags:

See: 7926d3a099ffbe4e2ae51fd55bd2f9dadb4b3043.

Confirmed!


can't compile ts file in project

node_modules/geolib/es/types.d.ts(4,96): error TS8020: JSDoc types can only be used inside documentation comments.
export declare type GeolibGeoJSONPoint = [GeolibLongitudeInputValue, GeolibLatitudeInputValue, GeolibAltitudeInputValue?];

Oh, weird. Which TypeScript version are you using? Can you describe a bit more what exactly you're doing? Which files you are importing, which functions you are using, etc.?

"typescript": "^2.7.2"

Im only using getDistance for it

after changing to the latest typescript the error was fixed.
Thanks!

Do u want me to close the issue?


Option to close the browser window when I close the http-server [suggestion]

I run the http server with -o option that open a browser window.
Can you please add an option that closes the browser window when I shut down the script?
It should be a simple kill on the pid but it's a bit problematic to implement on my own on windows.

This one seems a bit out of scope and directly tied to testing. Opening a browser window when the server runs is already a "nice to have." Closing it (especially via killing a process) seems a bit risky relative the the gains you'd get.
It's likely "easier" to find a script that kills a process relative the the appearance/disappearance of another script.
If you disagree, I'm happy to take a look at it more if you can find some cross-platform module that would help this happen.

I run bash on windows (MinGW) and it looks like it doesn't really work if I try to kill chrome with a script.
For example if I run:
./chrome.exe & echo $!
and then kill {the pid that was printed}
the chrome window doesn't close even though I'm getting the following afterward:
[1]+  Done                    ./chrome.exe
So yeah I'd be happy if you could help with that.

Thanks @SelaOAL. I'm running the same stack here, fwiw. However, this feature's not likely to be a priority and falls into the "nice to have" category. That said, if code magically appeared here, I'd be happy to take a look. 😸
Thanks!
🎩


Can we use two two-dimensional lasers as a three-dimensional  laser?

A two-dimensional laser placed horizontally and a two-dimensional laser placed vertically constitute a data acquisition device, which can obtain both horizontal and vertical data. If we convert the two data into the form of point clouds, can we use them as input data of slam 3d cartographer?

Yes, Cartographer supports the use case of using two 2D Lidars as input for the 3D SLAM pipeline. We use a similar setup for our mapping backpack. See the backpack_3d launch and configuration files.
Demo instructions are available in the demo documentation.

Can you specify which configuration file uses two two-dimensional lasers as input for 3Dslam input?

See backpack_3d.launch and backpack_3d.lua.
In our case we are using two multi-line LiDARs. Depending on your system setup using two 2D LiDARs can work as well.

I have done this already, but the result is not good. Can you help me look at it? the config file and the bag file is b0-2014-07-11-10-58-16.bag

Ok, with only two 2D LiDARs there is not enough overlap between the scans and the mapped area to accurately resolve the 6D pose. Therefore, the quality of the map is degraded.
Reducing the resolution might lead to somewhat better matches as there should be more overlap between the scans and the map.
To gain a high quality 3D map with full 6D pose estimation with Cartographer 3D LiDARs are needed.


docs(guides): fix typo in multitrack progress

@dcvz it worked!

The way things are configured I still need a review in order to merge

Yea, that's fine. I just need a review is all. I saw that you reacted to my comment and didn't leave a review so I thought you assumed I could just merge.


Clean up SCL explanation

This clean-up is for style and presentation, only.

@tnguyen-rh thanks! sorry for the delay :)

LGTM (@bparees, merging this as README fix)


Spring: Add Semgrep metavar for exact path/route variables

Attempt at fixing/improving https://github.com/mschwager/route-detect/issues/10 and https://github.com/mschwager/route-detect/issues/12
Adding a $PATH metavar in the rules allows you to really extract the exact values
Only on unauthorized paths for now, I tried also supporting that for the other rules but it would break weirdly
The final graph will show something more readable like:
ln 22: PostMapping "/api/book/{bookId}"

(Can't run the tests on my machine, weird python parallel processing issues)

The parallelism bug is gone, but now I'm having a weird issue.
poetry run semgrep --test --config routes/rules/ tests/test_rules/
....

Found rule id mismatch - file=/Users/gabrielmarquet/Desktop/route-detect/tests/test_rules/spring.java 'ruleid' annotation with no YAML rule={'spring-route-unauthenticated'}
Failing due to rule id mismatch. There is a test denoted with 'ruleid: <rule name>' where the rule name does not exist or is not expected in the test file.

The parallelism bug is gone, but now I'm having a weird issue.
poetry run semgrep --test --config routes/rules/ tests/test_rules/
....

Found rule id mismatch - file=/Users/gabrielmarquet/Desktop/route-detect/tests/test_rules/spring.java 'ruleid' annotation with no YAML rule={'spring-route-unauthenticated'}
Failing due to rule id mismatch. There is a test denoted with 'ruleid: <rule name>' where the rule name does not exist or is not expected in the test file.

I believe this means there is a test with a ruleid, and the rule isn't firing there.


[RFE] add 'unsupported browser warning' for IE11 on ovirt-web-ui 1.5+

add 'unsupported browser warning' for IE11 on ovirt-web-ui 1.5+
From ovirt-web-ui 1.5+, IE support is dropped. Add a warning advising user to use Edge, Chrome, Firefox, etc.
@lwrigh @lizsurette does PatternFly have anything? Google found me: https://github.com/cockpit-project/cockpit/issues/9438#issuecomment-407384005

@gregsheremeta PatternFly doesn't have anything too specific for situations like that as far as I know. There are inline notifications that could work in this scenario: https://www.patternfly.org/pattern-library/communication/inline-notifications/#overview

@bond95 @lwrigh they ended up with https://github.com/cockpit-project/cockpit/pull/9781 all text, no logos.
we do not support GNOME Web, but we do try to support Safari, so let's replace GNOME Web with Safari and otherwise try to make it look similar.

@gregsheremeta @bond95 That sounds like a good solution to me with going with all text. I can do a quick mockup that features Safari instead of GNOME Web.

@gregsheremeta @lwrigh yeah, with all text is better.

Here's the mock up featuring the unsupported browser message and the supported browser options:

Updated mock up:

@lwrigh +1 nice.

@bond95 the second one is nicer though;)


Dynamically link algorithms

Line 247 (and maybe others) statically link to Algorithm 1. This should be a dynamic link.

Doesn't look possible. See #70.


Running --interactive mode still modifies my libs.versions.toml file

Describe the issue
As per documentation, running in interactive mode should not modify the libs.versions.toml file - but it does.
It changes how my plugins are declared, from { id = "...", version = "..." } to plain string declaration, it removes blank lines, it removes seemingly unused versions and libraries, it changes how I arrange my bundles, etc.
Steps to reproduce
Just run interactive mode and see that your file is changed.
The expected interactive file is also created...

Yes, currently it will still format the toml file. I think I did this to process unused libraries. The formatting is kind of inherent to the plugin and it would also be applied when adding the updates (basically every time the file is rewritten).
I'll take a look to see if the format pass can be removed when running interactive though. It might need waiting until the plugin has switched to the internal resolver for that. After switching there's no detection of unused dependencies so it's possible that the current behaviour isn't needed anymore.


Feature - jekyll placeholders

[ ] Researching including content for each post as opposed to a link to a page for each post?

[ ] use post title, body, etc with placeholders on main page?
[ ] Style with Sass as appropriate

#37

sjoyal still here ?

I don't want to irritate or insult you by acknowledging I'm a work in progress, Need a Guide dog lol

#33


Art Space app: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-art-space?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-2-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-art-space#4
In which task and step of the codelab can this issue be found?
Last Step, comparing my solution to that of developers
Describe the problem
Can the code for the Artspace app get added to a repository so we can see the approach of the development team?
Steps to reproduce?
NA
Versions
NA
Additional information
NA

Closing, functioning as intended.


Break en/search endpoint into different routes for each topic

Currently the en/search endpoint takes the topic as part of the post body. Given that the logic and format of the output are different for each topic, it might make more sense to give each topic its own route, and therefore its own function. The common code currently at the end of the function https://github.com/ulsdevteam/ccvgd-backend/blob/6e824481f93ae251376ca0ac711d714d44b50a9d/app_func/en_search.py#L594-L644 could be moved into its own function to be called by each route.

The same would apply to the village/search endpoint in https://github.com/ulsdevteam/ccvgd-backend/blob/6e824481f93ae251376ca0ac711d714d44b50a9d/app_func/villageDetails.py

This is actually what the front end wanted to show the length of return value so it would be different every time, I will talk with the front end and then decide whether we should leave this part.
Thanks for point this out!


Add assertions for privateKey, txHash, emit event

Codecov Report

Merging #228 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #228      +/-   ##
==========================================
- Coverage    97.5%   97.48%   -0.02%
==========================================
  Files          79       78       -1
  Lines        3040     3020      -20
==========================================
- Hits         2964     2944      -20
  Misses         76       76

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 a7feeeb...b408326. Read the comment docs.


Update the QGB branch to use cosmos-sdk v0.46.x and latest celestia-app master

Update the QGB branch to use cosmos-sdk v0.46.x and latest celestia-app master.
Opened to separate the PRs as of: https://github.com/celestiaorg/celestia-app/pull/600 and https://github.com/celestiaorg/celestia-app/pull/600#pullrequestreview-1064491249
Closes: https://github.com/celestiaorg/celestia-app/issues/517

Codecov Report

Merging #601 (1922693) into qgb-integration (195da78) will increase coverage by 1.20%.
The diff coverage is 55.58%.

@@                 Coverage Diff                 @@
##           qgb-integration     #601      +/-   ##
===================================================
+ Coverage             9.76%   10.96%   +1.20%
===================================================
  Files                   55       57       +2
  Lines                 9036     9240     +204
===================================================
+ Hits                   882     1013     +131
- Misses                8075     8125      +50
- Partials                79      102      +23

Impacted Files
Coverage Δ

e2e/qgb/Service.go
0.00% <ø> (ø)

e2e/qgb/qgb_network.go
0.00% <0.00%> (ø)

x/payment/keeper/keeper.go
62.50% <0.00%> (+6.94%)
:arrow_up:

x/payment/types/events.go
0.00% <ø> (ø)

x/qgb/keeper/msg_server.go
0.00% <0.00%> (ø)

x/qgb/module.go
3.57% <ø> (+0.06%)
:arrow_up:

x/qgb/types/abi_consts.go
69.23% <ø> (ø)

x/qgb/types/codec.go
0.00% <ø> (ø)

x/qgb/types/validator.go
39.79% <29.62%> (ø)

x/payment/types/builder_options.go
41.26% <30.18%> (-58.74%)
:arrow_down:

... and 15 more

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


Add count

Do the checklist before submitting the PR:

[x] Have you read the guidelines mentioned in CONTRIBUTING.md?
[ ] Have you squashed your commits?

Q: What version of Node.js you've used in the project? (e.g. Node v10.11.0)
A:v11.6.0
Q: Mention the Issue Number! (e.g. Fixed #8)
A:#107

Give additional information regarding the PR below:

I tried to squash my commits, however I am not able to no matter what :-/. There is a extra commit because of that

Codecov Report

Merging #114 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #114   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          61     63    +2
  Lines         823    869   +46
=====================================
+ Hits          823    869   +46

Impacted Files
Coverage Δ

src/count/index.js
100% <100%> (ø)

src/index.js
100% <100%> (ø)
:arrow_up:

src/validation/count.js
100% <100%> (ø)

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 b17560f...a8e828a. Read the comment docs.


[RFC] Implementation of User Status Update Script

We need a script which will automatically mark user status based on the task assigned to them
Approach 1:
This approach directly queries the tasks collection for each user to determine if they have any assigned tasks. If no tasks are found, the user's status is updated to "idle" in the status collection.
The Steps are as follows :

Retrieve all users from the Firestore "users" collection.
For each user, query the "tasks" collection using their ID as the assignee field value to check if they have any assigned tasks.
If the query result is empty, it means the user doesn't have a task assigned. Update their status to "idle" in the "status" collection.
This approach directly checks each user's task assignments to identify idle users and update their status.

Approach 2:
In this approach, all tasks are retrieved from the tasks collection, and the assignees' user IDs are extracted. Then, for each user, their ID is checked against the assignees list. If the user's ID is not present, their status is updated to "idle" in the status collection.

Retrieve all tasks from the Firestore "tasks" collection.
Extract the assignees (user IDs) from the tasks.
Loop through all users and check if their ID is not in the list of assignees.
If a user's ID is not found in the assignees list, update their status to "idle" in the "status" collection.

IMO the first approach looks good as we only query the users, the second one requires to query tasks then loop over all the users to find the user
One question does this mark the Mavens idle?

IMO the first approach looks good as we only query the users, the second one requires to query tasks then loop over all the users to find the user One question does this mark the Mavens idle?

Mavens don't create a account on RDS.


bind() deprecated in jQuery, should be on()

autocomplete.init.js still uses $(document).bind('DOMNodeInserted'), but bind() has been deprecated in jQuery 3, and the slim version does not include it anymore. I think we can just replace the call with on().
(Well, the DOMNodeInserted event we catch is also deprecated, but that is covered in #851)

Agreed, looking forward to review a patch.


Add information about models: Docs

I have noticed that some methods return promises, it'd nice if the examples use await just to show that a promise is expected.
In addition I have created the following code that gets the model and injects models into the Vue prototype. Can be added as part of Vuex ORM somewhere.

/**
 * Map Vuex ORM models to computed properties to provide fluent Vuex ORM API out of the box
 * run it in a Vue component as `computed: {...mapVuexORMModels(['modelName','modelName',...])}`
 * @param models {Array} array of model names to map out
 * @return {Object} mapped models instances
 */
function mapVuexORMModels(models) {
  let mappedModels = {};

  // create the Vuex mapGetters obj
  for (let model of models) {
    mappedModels[model] = function () {
      // console.debug('context of this: ', this);
      return this.$store.getters[`entities/${model.toLowerCase()}`]().model;
    }
  }

  return mappedModels;
}

This should kind of information about Models should be added elsewhere like in the Models section.

Thank you so much for the improvement! 🎉  Awesome!


Minimum progress filter gives incorrect total row count

Filtering the Subject - Users page by the Minimum User Progress component will filter the table but give a Total Row count that does not match the number of results

:+1:


Using C++ Vector as container for display buffer

Test using Vectors instead of static buffers. The benefit is that Vectors can be redimensioned post instantiation. That way Bluethooth/WiFi provisioning can take more RAM at the start.
// Old way
//uint8_t _buffer[PLOGIC014_BUFFER_SIZE];
// Vector test:
    vector<uint8_t> _buffer;
    vector<uint8_t>::iterator buffer_it;

branch: test/vector  Related to #42
Read https://github.com/martinberlin/cale-idf/wiki/CalEPD-news

With the display buffer stored in a vector, when size is > 65536, it just restarts after throwing this exception:
0x400d6b9b: __cxa_end_catch at /home/martin/esp/esp-idf/components/cxx/cxx_exception_stubs.cpp:13

Backtrace:0x400d1f2f:0x3ffb24b00x40085cf5:0x3ffb24d0 0x4008b88e:0x3ffb24f0 0x400d6b9b:0x3ffb2560 0x400daebc:0x3ffb2580 0x400d7b3a:0x3ffb25a0 0x400d7bc8:0x3ffb25e0 0x400d7c1e:0x3ffb2600 0x400d6cbb:0x3ffb2630 0x400eb2d4:0x3ffb2690 0x40088cc5:0x3ffb26b0
0x400d1f2f: panic_abort at /home/martin/esp/esp-idf/components/esp_system/panic.c:354

My measurements before and after initializing the vector are:
Free RAM before buffer: 292468
Free RAM after buffer: 226932
Probably out of RAM, only doable initializing the buffer in SPI Ram, WROVER or similar ESP32.

This works but it was just a test. Don't think is the wisest idea to use Vectors for a display buffer.
Anyways, here is the branch:  https://github.com/martinberlin/cale-idf/tree/test/vector


ModelState.AddModelError in Blazor Form

Is your feature request related to a problem? Please describe.
I am trying to add custom validation error message AFTER the form has been submitted successfully for an existing field.
In Razor Pages I can do something like: ModelState.AddModelError("Form.Field", "Something happened!");  which allows the error message to be displayed when the page is returned. (Bonus Point doing it like this because it allows the field CSS to also be displayed as errored field)
I can't find a way to do this in Blazor.
Describe the solution you'd like
I need a way to do ModelState.AddModelError in Blazor. Is it possible to expose ModelState altogether in Blazor?
Additional context
We apparently cannot simply use custom validation annotation because the validation may be async. (https://github.com/aspnet/Mvc/issues/7791 required for this behavior)

Related to issue, might qualify as separate issue:
protected override ValidationResult IsValid(object value, ValidationContext validationContext)
{
    var service = (IExternalService) validationContext
                         .GetService(typeof(IExternalService));
    // use service
}

In Blazor, GetService returns NULL!

EDIT 2: Solved the issue, although with a different approach:
https://github.com/ryanelian/FluentValidation.Blazor
This library allows writing validators registered to the .NET Core DI, which easily allow custom logic (even async) to be developed easily!

I implemented custom error message in this blog post https://remibou.github.io/Using-the-Blazor-form-validation/ (search for "Display validation error from the server").

Thanks for contacting us, @ryanelian .
Glad that you've managed to find a solution for this.


test(client): fix flaky e2e test

TestClient:
https://github.com/gotd/td/runs/1606566289
client_e2e_test.go:81: start: create auth key: failed to read response: read: read abridged: EOF

Also https://github.com/gotd/td/runs/1608990230
client_e2e_test.go:82: start: create auth key: key exchange verification failed: hash mismatch

Looks like this is resolved.
I've also reduced external tests flakyness.

Looks like this is resolved.
I've also reduced external tests flakyness.


Possibly flaky test in VerticalListTests

URL of codelab
Dogglers
In which task and step of the codelab can this issue be found?
Running the tests at the end
Describe the problem
The test vertical_scrolling is failing for me. I don't think its a bug with my application code though because when I put a sleep after performing the swipe it is passing the test. So it looks like a race condition when the view is being checked if its displayed before the swipe animation is completed. Error:
androidx.test.espresso.base.DefaultFailureHandler$AssertionFailedWithCauseError: 'is displayed on the screen to the user' doesn't match the selected view. Expected: is displayed on the screen to the user      Got: "MaterialTextView{id=2131230864, res-name=dog_name, visibility=VISIBLE, width=115, height=71, has-focus=false, has-focusable=false, has-window-focus=true, is-clickable=false, is-enabled=true, is-focused=false, is-focusable=false, is-layout-requested=false, is-selected=false, layout-params=androidx.constraintlayout.widget.ConstraintLayout$LayoutParams@fb2e21e, tag=null, root-is-layout-requested=false, has-input-connection=false, x=21.0, y=530.0, text=Faye, input-type=0, ime-target=false, has-links=false}"
Steps to reproduce?
Run the vertical_scrolling test on an emulator. I am using Pixel 2 API 28 (Android 9). All animations have been disabled
Versions
Android Studio Arctic fox 2020.3.1 patch 4
Api level 28
Additional information
Include screenshots if they would be useful in clarifying the problem.

Same experience here with a physical Pixel 2 w/ Android 11.
A delay of 1000ms after swipeUp allowed the test to pass.
Android Studio Bumblebee | 2021.1.1 Patch 1
Build #AI-211.7628.21.2111.8139111, built on February 1, 2022
Runtime version: 11.0.11+0-b60-7590822 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 12.1

Same problem, delay of 1000 ms helps as well.


Resume Zone Schedule Option

Similar to hitting "per schedule" on your thermostat, create a switch per zone that resets any manual holds to the scheduled activities.
Describe the solution you'd like:
For each zone, create a switch that is "on" when the default schedule for that zone is being followed, and is "off' when anything deviates from it (timed hold, permanent hold, vacation mode(?)). This can be grouped with the thermostat and fan objects on a per zone basis.
Additional context:
In the "config" object in the api, this can be determined by looking at the "holdActivity", "hold", and "otmr" values. If I recall correctly, you can reset holdActivity to an empty string, hold to "off" and otmr to an empty string.

For reference, this is a screenshot of the function desired from the thermostat interface

.

Just tried it out, looks great! Thank you!

Awesome!
I'll release it as the stable version in the next week or so. Theres a few things I want to do first to make the syncing between the activity buttons work better.


[CT-1294] [Feature] Make Postgres' indexes config cumulative

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
dbt on Postgres supports creating indexes but indexes are not cumulative when specified at multiple levels.
My situation is that there is a specific column I want to index in ~70 models so my first thought was to put that config in dbt_project.yml. But it looks like {{ config( indexes = [ ... ] ) }} in model scripts override rather than extend project-level index config.
Hooks are cumulative. Tags are additive. Grants have inheritance rules. So why not make indexes cumulative/additive/inheritable as well?
Describe alternatives you've considered
For now I have added indexes configuration in every model script, which is tedious and easy to forget for new scripts.
Who will this benefit?
Folks using Postgres
Are you interested in contributing this feature?
Not at this time, sorry
Anything else?
https://getdbt.slack.com/archives/C0172G2E273/p1664821151316579

thanks for opening @owenprough-sift! this sounds reasonable. I'm going to edit the labels and we'll see if we or someone from the community can pick this up in the future

I just came across this issue as I ran into the same situation (I have a field that I want indexed across all of my models, but I also have separate indexes I want to add to most models). It seems more intuitive to me to have the dbt_project.yml +indexes value be additive with the config within the model


fixed table alignment when sidebar closes

🛠 Fixes issue #182
📇 Changes proposed: The alignment of the table is fixed when the sidebar collapses. The day-night icon doesn't collide with the table anymore
✅ Check List (Mark all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[ ] This PR contains changes in the documentation
[x] This PR contains changes in the Code of the project
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
-It's a UI fix, for the issue that I proposed.
📷 Screenshots

@hasnainmakada-99, please verify and merge the changes for the issue #182.

@hasnainmakada-99, please verify and merge the changes for the issue #182.

Sure @replyre

@all-contributors please add @replyre as a contributor of code


what is starting uri of application

if you run locally  -> http://localhost:3000/xx
but also "/" is redirected to the defaultLanguage.

the 3000 = portnumber is configurable.
the "xx" is one of the languages you support (should be in the language table)
standard "en" is the defaultlanguage (and as we are from Belgium and speak Flemish,
"nl" is also in the demo setup)

you can see this in:
siteServer.all("/" + app.languages[iL].id + "/*", function (req, res) {
app.servePage(req, res);
});


Adding package manager

Adding package manager

Already added the setup.py and setup.cfg used for https://pypi.org/project/python-amazon-ad-api/


Implements do_segment_bounds and DoToSegmentPositionBackend.

🎉 New feature
Related to https://github.com/maliput/maliput_sparse/issues/15
Summary
Implements:

Lane::do_segment_bounds()
Lane::DoToSegmentPositionBackend()

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

I still need to dive into testing however I'd like just a double check here @agalbachicar .


Check the repo file

A link check for the repo file would have avoided https://github.com/ocaml/opam-repository/pull/26065

Docs on the repo file: https://opam.ocaml.org/doc/Manual.html#repo


Rename _index.html to _index.md

According to Hugo documentation, The index page with a front matter section should be named _index.md rather than _index.html. I'm not sure if these weird namings are confusing the site builder. This PR also attempts to fix the problem and see if Hugo still hangs in an unpredictable way.

preview:

https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/
https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/case-studies/
https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/community/
https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/docs/tutorials/kubernetes-basics/
https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/partners/
https://deploy-preview-38834--kubernetes-io-main-staging.netlify.app/training/

Not sure if this is a draft or not? The PR description implies it's an experiment @tengqm

@sftim This PR is a correction of file names. Hugo only expects _index.md, not _index.html. The introduction of _index.html is by Docsy, IIUC. So this PR at least fixes this problem, which is not a big deal.
The other purpose is for diagnosing the netlify build hanging problem. To that end, I'm not sure if this one can really help. In another experiment, I found that the netlify build may hang when building an _index.html file for a localization. This is something we can try, but I don't know if it can really resolve it.
So, this PR is no longer a draft.

According to Hugo documentation, The index page with a front matter section should be named _index.md rather than _index.html.

My gut feeling is that the Hugo docs are misleading. Let's check in with the Hugo project to confirm that the specific name _index.md is essential, or actually if _index. with any suffix counts the same.

My gut feeling is that the Hugo docs are misleading. Let's check in with the Hugo project to confirm that the specific name _index.md is essential, or actually if _index. with any suffix counts the same.

There could be a possibility that _index.<suffix> works the same way. However, the so called _index.html file we have today are actually markdown files. Renaming these files (as documented by Hugo) doesn't hurt anyway.
Just now, another timeout was caught using the test PR #36741. This time, time out was triggered by a similar case -- content/pl/training/_index.html is actually a markdown file.

Ah, right. If files contain Markdown, let's not mark them as HTML.

content/en/docs/tutorials/kubernetes-basics/_index.md looks like HTML; I wouldn't rename it from content/en/docs/tutorials/kubernetes-basics/_index.html.

No. The file contains a frontmatter section at the very beginning.
---
title: Learn Kubernetes Basics
linkTitle: Learn Kubernetes Basics
no_list: true
weight: 10
card:
  name: tutorials
  weight: 20
  title: Walkthrough the basics
---

<!DOCTYPE html>
...

I don't recall the reason for the front matter in the HTML files, but the content has been present for some time, if not always.
I think these files are parsed as HTML with some templating.
Perhaps there is something else causing the problem?

I don't recall the reason for the front matter in the HTML files, but the content has been present for some time, if not always. I think these files are parsed as HTML with some templating.
Perhaps there is something else causing the problem?

@kbhawkey You are right. I'm not 100% this will help resolve the build stuck problem. However, I believe these files are not supposed to be named as '.html' files. The Docsy theme may allow '.html' files as the "default index", but I failed to find any evidence in GoHugo code. Having some of the "default index" files named as ".html" is at least inconsistent with the rest of all directories. Renaming them back to '.md' won't hurt anyway.

No. The file contains a frontmatter section at the very beginning.

That's normal in Hugo, whether what follows is HTML or Markdown.

I am very dubious about taking a file that's HTML, and telling Hugo to treat it as Markdown. Hugo uses the file extension to work out what static rendering is required.

I am very dubious about taking a file that's HTML, and telling Hugo to treat it as Markdown. Hugo uses the file extension to work out what static rendering is required.

If you spend sometime reading the source code hugofs/files/classifier.go, you will see how Hugo tells if a file is HTML. It checks if a file has its extension spelt as ".html" or ".htm". Hugo treats such files as HTML content.
Also, in the same source, Hugo checks if the first non-space character in a file is '<', and use that to determine if a file is HTML content.
These two logics are hardcoded.
Take one of the files I renamed in this PR, as an example:
---
title: Community
layout: basic
cid: community
community_styles_migrated: true
---

<img> ...

This file is NOT HTML. It is a YAML. It can be treated as a markdown. This frontmatter is used everywhere in a .md file, so I am not doubting if Hugo can take good care of it.
However, if you say this is a HTML. I don't agree and the code is not written that way.

I think we found a fix now, or at least a good-enough workaround.

Can we close this? The .html suffix on the filename seems more correct to me.

(We could ask a Hugo maintainer such as @bep to confirm that .html files are allowed to start with front matter)

This a long thread, but I'm pretty sure this isn't what you want to do. The naming in Hugo for these section/branch files are _index.[extension].  where extension can be a number of formats known to Hugo: html, md, asciidoc etc.
So, html files with front matter is fine -- it will not be passed into any Markdown processor, but you can use shortcodes etc. and it gets a layout (e.g. single.html) applied to it.

@bep I'm eager to know how .html files are treated differently from .md files. I did not spend much time digging into the code details but I'm sure there are subtle differences. The code checks the file format based on its file extension and the first few characters. After that check the code handles HTML file differently from Markdown files. I also checked the docsy module. It looked to me that _index.html thing is specific to Docs. In Hugo, it should be _index.md files.

If we grep '_index' across the Hugo code base, we only see '_index.md' files. This means that most of the time we test the logic using a "_index.md" file rather than a "_index.html" file.
In files like hugofs/files/classifier.go, we check if a file is a HTML file and then looks into the file content and check the first few characters to see if it really is a HTML.

The above logics are not so obvious. I was suspecting there could be some subtle errors that lead to the site build timeout.
Another reason behind this PR is that I think we should stick to one way or the other way when naming a _index (i.e. a branch). Most of the time, we call them a .md file, but sometimes we call it a .html although the file is actually a valid Markdown rather than a valid HTML.
This PR is not complete. If we agree that this is the right thing to do, we may want to rename all _index.html files to _index.md when the file has ---\n(frontmatter)\n---\n at its head.

Hey @tengqm , thank you for taking up this initiative! Could we move this discussion to an issue on the Hugo forum, seek clarity, and come back to the PR to make changes, if any? We can reference this PR & the discussion we've had here in that issue/support request. IMO, that would help us resolve any queries that you might have and possibly, get to the root of the issue, quicker.

Em... I believe this is the right move. Even though this is not fixing any existing issue,
as an engineer, I prefer the safest way to code.
I would not call a file with a ---\n(frontmatter)\n---\n(other stuff) a HTML. It can be treated as markdown without an issue.
I did spend some limited time reading the Hugo source code, the logic there has quite some
different scenarios interwined together. It is tricky to get things done right, while
getting things to subtle failures is not so straightforward to debug.
Anyway, this PR is doing no harm to the website. Fortunately, we may fix the dangling
timeout issue by lucky accident.

👋 @tengqm
It seems reasonable to continue to investigate the file extensions of _index files, though it is not clear that the extension or file type is affecting the build.
I suggest to close this pull request.

/close

The intention of this PR was pretty clear, though it was not a complete one.
We are not naming non-html contents as .html files.


Exporting freeze

I'm having a problem with Export to pages. It freezes once it shows "Exporting documents..."
Do you have any idea what is the reason?

Hi,
I am not sure if it is the template that is causing the freeze. Can you give me more details about the version of Pencil you are using (Pencil Version, OS Version etc...) ?
Best Regards,
Madhav

I've just figured out the problem. Some with firefox plugin, it's not the template. Sorry for the incovenience. Found the solution here: http://thegeekettespeaketh.com/2013/03/solved-cant-export-png-from-evolus-pencil/

Ok, glad to see the issue is resolved for you. Thanks for your contribution.


iOS 10/XCode 8 Property extending/overridding in NativeScript

In iOS 9/XCode 7 everything was the class functions so overriding it was just a simple
{ x: function(y) { /* blah */} } and x would be overridden on the new extended class.
However with XCode 8 / iOS 10 -- I can't seem to find a way to override any of the (readonly) properties.  For example a ton of UIViewController items are now readonly properties instead of functions -- what is the proper way to override these?
For example:
https://developer.apple.com/reference/uikit/uiviewcontroller/1621419-shouldautorotate?language=objc
https://developer.apple.com/reference/uikit/uiviewcontroller/1621435-supportedinterfaceorientations?language=objc
https://developer.apple.com/reference/uikit/uiviewcontroller/1621438-preferredinterfaceorientationfor?language=objc

Instance Properties
You can find an example of overriding instance properties here. The syntax is:
var MyViewController = UIViewController.extend({
    get modalInPopover() { return this.super.modalInPopover; },
    set modalInPopover(x) { this.super.modalInPopover = x; }
}, {
    name: "MyViewController"
});

Static Properties
As stated in the limitations section you can override only instance methods/properties... but you still can "hide" the native class property by redefining it like this:
var Student = Person.extend({});

Object.defineProperty(Student, 'staticPropertyName', {
    get: function() {
        return Person.staticPropertyName + " Modified";
    },
    set: function(value) {
        Person.staticPropertyName = value;
    }
});

Keep in mind that this is still not really overriding the property because the new implementation will not be triggered if the property is accessed from native code.

If I understand you correctly; this is a pretty significant limitation now.
A huge number of methods are now read only properties in xcode 8 and if we can't override them in NS code that is a significant issue.
So basically as it stands now we cannot override any of the UIViewController properties which allows us to manage rotation, sizing and several other properties that are the only way to do some things in iOS.
@jasssonpet any work arounds that you can think of for modifing the autorotate value in the primary UIViewController or Navcontroller?
Any idea for a time line to fix this issue since xcode 8 has now a huge number of read only props that you are supposed to override to change functionality?

@NathanaelA All the properties you linked in your first comment are instance properties. You can override instance properties the way I described in my previous comment. The fact that the properties are readonly makes no difference... you still can override their getter (they have no setter). Limitation exists only for class(aka static) properties . I believe the properties that handle rotation and sizing are also instance properties, so you can easily override them. In other words, the transformation of instance methods into instance properties is not an issue, because NativeScript supports overriding both of them.

Thanks, I have added a pull request to the main core (https://github.com/NativeScript/NativeScript/pull/2827) make this a lot simpler in the future, as the UINavContoller is created when the frame is created which is at the start of the app; once it is created; it is too late to add any new prototype functions to it to override any functionality.  I have managed to find a hack around that will work in the meantime; but actually updating the prototype before it is created would be the best solution.


Changed to 26.1 instead of 27.1 after 26.

Followed link later in documentation to confirm that was the desired numbering.

There are 2 more 27.1 Apps for websites in the tables on the same page, but only 1 of those is followed by 27 (the other one is followed by 28). You may want to add those to your PR content.

Thank you.  I fixed those two additional 26.1, vs 27.1.  There is a different format where some sections will skip something altogether, but the bookmarks match something specifically later in the documentation.

Very well, fair enough. Good point, I was uncertain about the last incorrect table link, where the actual ordinal number 27 was missing between 26 and 28. Happy to see that you have got it covered. 👍


Modify validations.py to Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #1

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

I already have signed CLA


ESP32-S3-N8R2 SDMMC in 4-bit does not run on 40 Mhz with IDF 4.4.2 (IDFGH-8778)

    > @ThanosSiopoudis @ntwallace Thank you for reporting the issue! Could you please try to apply the attached patch and see if it improves the situation?

0001-Draft-sdmmc-fix-unreasonable-i-o-clock-phase-setting.patch.txt

@igrr Can this patch be applied to Stable 4.4.2?  Facing the same issue with ESP32-S3-N8.   Compiling with latest master is ok, compiling with 4.4.2 can only mount if 20 MHz and always fails on 40 MHz.   Using SDMMC/4-bit.  We must use the 4.4.2 as the ESP-ADF is not yet fully supported with v5.1 latest (any time frame for v5 to support ADF? or patches?)
Thanks
Originally posted by @NARRob in https://github.com/espressif/esp-idf/issues/8521#issuecomment-1225803453

Is there a patch for ESP32-S3 for SDMMC to run at 40 Mhz.   The same exact code runs 40 MHz when compiled with v5.1, but fails to mount when compiled with 4.4.2 or 4.4.3.  It only can run on 20 MHz.
We need to use 4.4.2 or 4.4.3 for ESP-ADF code.

I am building and running with arduino (esp-idf v4.4.3) sd_mmc using native esp-idf API and it is working, both, SPI and mmc 4-bits. SPI reporting max 20MHz and mmc 40MHz with my sd card.
        esp_vfs_fat_sdmmc_mount_config_t mount_config = {
            .format_if_mount_failed = true,
            .max_files = 5,
            .allocation_unit_size = 16 * 1024
        };
        _is_mmc = mmc;

        if(mmc)
        {
            sdmmc_host_t host = SDMMC_HOST_DEFAULT();
            host.max_freq_khz = SDMMC_FREQ_52M;
            host.flags &= ~SDMMC_HOST_FLAG_DDR;
            ret = esp_vfs_fat_sdmmc_mount(path, &host, &slot_config_mmc, &mount_config, &_card);
        }
        else
        {
            sdmmc_host_t host = SDSPI_HOST_DEFAULT();
            host.max_freq_khz = SDMMC_FREQ_DEFAULT;
            spi_bus_initialize((spi_host_device_t)host.slot, &bus_cfg, SDSPI_DEFAULT_DMA);
            slot_config_spi.host_id = (spi_host_device_t)host.slot;
            ret = esp_vfs_fat_sdspi_mount(path, &host, &slot_config_spi, &mount_config, &_card);
        }

@chegewara Thanks.  You message did not indicate if it was the ESP32-S3.   Maybe adding the ESP-ADF got something to do with it.  I will try using 4.4.3 alone without ADF to see if that's the case.  Thanks

Yes, of course it is S3. Im just not sure right now if its with or without PSRAM, because i have few different devkits, including N32R8 OPI mode. Here you can see pins i am using, but S3 can use any free pins with matrix:
#define SD_CMD	1
#define SD_CLK  2
#define SD_DAT0 4
#define SD_DAT1 5
#define SD_DAT2 6
#define SD_DAT3 7

The fix has been merged internally and will be available on Github soon. For reference, commit ID will be https://gitlab.espressif.cn:6688/espressif/esp-idf/-/commit/ef64e4e5b3fb4878ed7df62a054f44ae5b131519.


export newTokenProvider function for library usage

Exports the token/newTokenProvider() function for library usage.
In Crossplane Helm Provider, we are interested in authenticating to AKS non-interactively when supplied with AKS kubeconfigs that has execProvider section relying on kubelogin and user interaction.
Using the CLI is currently not an option and we aim to handle this programmatically in go code. We checked the execCredentialPlugin.Do(), however it outputs to stdout with no chance to configure and ultimately we are interested in obtaining the token provider only, not the converted kubeconfig.
Feedback is appreciated, thanks!

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

Comparison is base (3e83c88) 64.77% compared to head (48fc079) 64.77%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #371   +/-   ##
=======================================
  Coverage   64.77%   64.77%
=======================================
  Files          23       23
  Lines        1766     1766
=======================================
  Hits         1144     1144
  Misses        549      549
  Partials       73       73

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

Hi, we have merged the library changes to the master branch. I have submitted a pr for updating the usage in crossplane side: https://github.com/crossplane-contrib/provider-helm/pull/207 Let's move the discussion there, hope we can finalize the required setup there. I will close this PR as it doesn't apply any more.


Crashing when trying to import a class from non existing module

Trying to compile the following code (which uses a class in a package that doesn't exist), in a file import_class.py:
from doesntexist import SomeClass

s = SomeClass()
s.start()

If I compile the previous code with VOC and try to run it not having the doesntexist module in the classpath, I get the following error:
Exception in thread "main" java.lang.ClassCastException: org.python.java.Module cannot be cast to org.python.Callable
	at python.import_class.__init__.module$import(import_class.py:4)
	at python.import_class.__init__.main(import_class.py)

I'd expect to get an ImportError, like:
$ python import_class.py
Traceback (most recent call last):
  File "import_class.py", line 1, in <module>
    from doesntexist import SomeClass
ImportError: No module named 'doesntexist'

I think another instance of the same issue (failing to detect an import error), if you change the code to:
import doesntexist

s = doesntexist.SomeClass()
s.start()

You get:
Exception in thread "main" NameError: name 'SomeClass' is not defined
	at org.python.java.Module.__getattribute__(Module.java:34)
	at python.import_class.__init__.module$import(import_class.py:4)
	at python.import_class.__init__.main(import_class.py)

The expected behavior would still be the aforementioned ImportError.

Oh sorry, just realized I created an issue in the wrong project.


https - howto?

I have "https = true" set in my nitter.conf, and "NITTER_HTTPS=true" set in my .env, but links are still generated as http://, instead of https:// -- is there something i've missed to get this to work?
thanks

Added an environment variable INSTANCE_HTTPS=1 that would add https = true in the nitter config https://github.com/sekai-soft/nitter/commit/962b6eb39acd8641625530414cb271c4bb4c589d


9049 DEV Update dashboardCtrl.js to map sshKey integration with sshKeys sysInt and enable the integration by default

https://github.com/Shippable/pm/issues/9049

This is done. verified on rc.


We need a hyperlink button

We need a button so that we can update our latest anouncements/informatio.
This function should support hyperlink so that when user clicks in it he/she will be taken to appripriate place.

Labeling as enhancement.

Update:
Now user can create post with notice category. These post will be shown as announcement in home screen.


Implement data type enum

We need an enum describing all available/supported data types. It could be based on https://docs.oracle.com/javase/8/docs/api/java/sql/JDBCType.html

Are you wanting an enum that just defines the sizes, or do you want an enum that contains a store of sorts for each result?

Should consider that JDBC lack of some features like supporting JSON like column types. And it is also not so good for working with array types.

Closing this for now since there is a basic enum in place.
/// RDBC Data Types
#[derive(Debug, Copy, Clone, PartialEq, Eq)]
pub enum DataType {
    Bool,
    Byte,
    Char,
    Short,
    Integer,
    Float,
    Double,
    Decimal,
    Date,
    Time,
    Datetime,
    Utf8,
    Binary,
}


x/tools/gopls: fix autocompletion of structs in slices

go version

go version go1.12.5 windows/amd64

gopls version

version v0.1.3, built in $GOPATH mode

On VSCode:
This seems OK (ctrl + space):

After deleted &cli.Command before Name:, press ctrl + space:

It will not show the completions of the struct. It should show completions like the first one.

Thank you for reporting this issue. We will investigate.

Hey! I'm looking for a good way to start contributing to the Go project. I spent a little time looking into this issue. It appears that the root of the issue is that it is a slice of pointers, which confuses the code in completion.go:enclosingCompositeLiteral(). The suggestions work correctly if it is a slice of structs (not pointers to structs).
I'll try to put together a fix for this unless someone does first.


Consistent writing

Super nitpick, but kinda bugged me :-) This makes things more consistent with
enter password for new key

enter password again

enter password for new backend

(all lowercase)
I noticed some other inconsistencies: Sometimes log messages begin with the first word being capitalized, sometimes they don't. And sometimes they end with periods and sometimes they don't.
Not a huge deal I guess, but looks kinda "unpolished" (which I guess it is :-))
Maybe open an issue to make all user-faced logging use a consistent style?

Heh, you beeing nitpick got me through the math exams. Talking about Fundamentalmaschen. Merged.


fix some problem

fixes #50 #53

fixed https://github.com/CaiJingLong/flutter_ijkplayer/issues/47#issuecomment-488310917

update version to 0.2.4


C++ debugging broken as of VSCode 1.58

Does this issue occur when all extensions are disabled?: No, since features of the extension itself are broken.

VS Code Version: 1.58 / 1.59
OS Version: macOS Big Sur

Steps to Reproduce:

Install the C/C++ Extension from Microsoft or CodeLLDB by Vadim Chugunov
Configure a launch configuration similar to this

{
	"name": "Name",
	"preLaunchTask": "Build App",
	"type": "cppdbg",
	"MIMode": "lldb",
	"request": "launch",
	"expressions": "native",
	"cwd": "${workspaceFolder}",
	"program": "path/to/executable"
}

Try to debug

Observed behavior

VSCode's status bar does not turn orange and breakpoints do not hit. The console works though.
Since both of the extensions mentioned stopped working in VSCode 1.58 I assume that the issue is in VSCode rather than in the extensions.

/extCpp

As I mentioned in the bug description, I assume that the issue is in VSCode rather than in the extensions mentioned since both of these extensions stopped working after the update to VSCode 1.58, but work fine in 1.57.

@connor4312 We believe this is a VS Code bug/regression, because it started happening after a VS Code update and affects multiple debugger extensions.

@sean-mcmanus can you reproduce with latest vscode insiders? If yes with what debugger extensions can you reproduce since you say that it affect multiple? Thanks

I just tested the latest insider build and with only the two mentioned extensions installed debugging worked as intended. Either it is fixed in the insider build or something else in my regular installation is causing the problems. I'll check my regular installation now for other possible causes and report back.

I tested again and now both extensions stopped working. I did not change any settings. I just ran the CodeLLDB debugger (which worked initially) then tried the C/C++ Extension debugger (which did not work) and then tried the CodeLLDB debugger again which then also stopped working.
I also tried to delete the .vscode-insiders folder in my personal folder and installed the extensions again, but it seems that it is broken now.

Let's wait for @sean-mcmanus to investigate on the C++ side first. Once he provides more details we will know more.

@isidorn I haven't reproed the issue (and we haven't gotten widespread reports as far as I know). It might be something particular with the user's setup. @WardenGnaw is our debugger expert. He might know how to get more debugger logging/diagnostics.

From the original issue in vscode-cpptools:

The issue is that VSCode doesn't turn orange to indicate it is in debugging mode and that breakpoints don't work anymore.

It seems like the debug adapter has not activated so it would not produce any debug logs if we enable them in the launch.json

I just ran the CodeLLDB debugger (which worked initially) then tried the C/C++ Extension debugger (which did not work) and then tried the CodeLLDB debugger again which then also stopped working.

This indicates that it seems to occur across extensions.

I just did a bit more testing and found out how to get it back to working for one time before failing again:

delete the file ~/Library/Application Support/Code/CachedExtensions/user
restart VS Code
debug again

The contents of the user file created when launching VS Code are identical to the contents of the deleted version. I assume the the lack of this file triggers some reinitialization of the user extensions within VS Code that makes them work again for one session.

Same problem on Intel MacBook Pro with Big Sur 11.5.1.
In 1.59, the external console starts, but the input doesn't work.
I've downgraded to 1.57, reinstalling C++ extension, and everything is fine.

I believe this issue is related to incorrect cwd. As others are reporting, my instance of lldb is not loading any symbols or other init files. A quick test -exec shell pwd shows that the current directory is the program directory, not the cwd directory as it should be. Further, copying the program files to the cwd directory and then running the program from there - as a test - makes everything work again.

@isidorn - Hopefully if what I think I've found (see immediately above) holds up, it should also be easily repeatable? Thanks!

@mikebeaton Can you open an issue in the C++ extension for your issue? Its a bit separate since in your scenario the debug adapter is launched and there should be logs by adding the following to your launch.json.
"logging": {
   "engineLogging": true,
   "traceResponse": true
}

They should output to the Debug Console and please include those in the new issue.
This issue is indicating the debug adapter never starts.

I had those enabled already, it's just mentioning lots of things that can't be found. Changing the directory as indicated, it can be found.
Also, I thought the issue was the same, because without the debug logs the only real difference is that, as the OP says, breakpoints are not hit.
Let me just double-check whether the issue for me is also related to v1.58 (it's definitely recent).

@WardenGnaw You are correct: I have a separate issue, which has already been reported a few hours ago (just now that I have pinned it down!): https://github.com/microsoft/vscode-cpptools/issues/8084

There are no action items on the vscode side. Thus closing this.

This is incorrect for me. My MacBook Pro does not have a Touchbar, and I still have the problem (I normally clicking the buttons in the debug toolbar using the mouse cursor).

@isidorn, could we reopen this issue as I found a way to reproduce it now (see my comment above)?

It turns out the error was on my side. I pressed the launch button instead of the launch and debug button. Thanks anyway.


Remove obsolete otherPages variable from the scripting integration tests

It looks like this has accidentally been copy/pasted from the Textfields and focus block, which is the only one in which a test actually uses it. We can therefore safely remove it from all other blocks where no test uses it.

/botio integrationtest


fix(scripts): wrong variable names on rpm install

Fixes #181
Some variable names in the install_rpm() function were wrong.
This causes download() to get called with its first argument empty, which in turn causes the curl -sfLo issue mentioned.

before

after


:sparkles: Add dependabot to notify when actions are updated

The actions used by this action are rather stable and so this won't often be required but a useful convenience in the long term to keep all the used actions up to date.

Great to have. Thanks!


virtfs file: update cursor position on fd_read

If a handle is backed by InMemoryFile, fd_read (turned into Handle::read_vectored) doesn't update the cursor position properly and thus prevents the caller from detecting EOF.
This also fixes a similar issue in fd_pread, which doesn't advance the read offset when reading data into multiple iovecs.

@ueno the failure on Windows looks...weird. I can help out here with debugging, but won't be able to look at this until evening today.

@ueno the failure on Windows looks...weird. I can help out here with debugging, but won't be able to look at this until evening today.

I suspect that this is because on Windows fd_pwrite returns after writing each iovec, as I see the similar behavior in fd_pread.
Let me try to call fd_pwrite in a loop to ensure that everything is written.

@ueno the failure on Windows looks...weird. I can help out here with debugging, but won't be able to look at this until evening today.

I suspect that this is because on Windows fd_pwrite returns after writing each iovec, as I see the similar behavior in fd_pread.
Let me try to call fd_pwrite in a loop to ensure that everything is written.

FWIW, if this is the case, we should probably unify the interface in WASI though. I'm happy for us to do this in a subsequent PR so that we don't block virtfs fixes and improvements in this one though.