Dopex - ARBITRUM QA

Review Info

Description
Info

Subgraph Reviewed
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/id/QmaiMb6pLnPnpJooz9yoi2q2t4tHmpRWMWSpgsQpU6jPLe&tab=protocol

Schema Version
1.2.0

Subgraph Version
1.0.0

Methodology Version
1.0.0

Evidence Spreadsheet
https://docs.google.com/spreadsheets/d/1JRe3Nilw6npQAwHx5ut3Zl8OM6MegBlq/edit?usp=sharing&ouid=104027705798814439445&rtpof=true&sd=true

Metrics to Review
Revenue

We have calculated the daily total revenue/protocol-side revenue using data from the official website, and the results show that it should be around 10,000 USD instead of only 679.43 USD.
Refer to the sheet 'Support-Implied- Revenue-&-TVL'.

March 16, 2023:

Subgraph - Daily Total/Protocol Side Revenue
Official Website - Daily Total/Protocol Side Revenue
Absolute difference
Difference %

679.43
10,279.45
-9,600.02
-93.39%

Refer to the sheet 'Protocol'

Total Premium

We have also calculated the daily total premium based on the official website data, and our results indicate that it should be approximately 34,000 USD, which is significantly higher than the reported amount of 7,763.815 USD.
Refer to the sheet 'Support-Implied- Revenue-&-TVL'

March 16, 2023:

Subgraph - Daily Total Premium
Official Website - Daily Total Premium
Absolute difference
Difference %

7,763.82
34,264.84
-26,501.02
-77.34%

Refer to the sheet 'Protocol'

Pool Overview - TVL

TVL data for two out of the 15 pools have shown discrepancies of more than 100% when compared with the official website data. These pools are the 'ETH MONTHLY CALLS SSOV V3 3' and 'DPX WEEKLY CALLS SSOV V3' pools, which had discrepancies of 359.64% and 632.68%, respectively.

March 16, 2023:

Subgraph - ETH MONTHLY CALLS SSOV V3 3 Pool
Official Website - ETH MONTHLY CALLS SSOV V3 3 Pool
Difference %

368,629.81
80,200
359.64%

Subgraph - DPX WEEKLY CALLS SSOV V3 Pool
Official Website - DPX WEEKLY CALLS SSOV V3 Pool
Difference %

67,406.35
9,200
632.68%

While there are six pools with double-digit discrepancies, we believe that these discrepancies may be attributed to differences in time and token prices.

Refer to the sheet "Pool_Overview" for details

Pool Snapshot
rDPX MONTHLY CALLS SSOV V3 3 Pool
1.) rewardTokenEmissionsAmount and rewardTokenEmissionsUSD

We calculated the implied market price of $rDPX using the rewardTokenEmissionsAmount and rewardTokenEmissionsUSD data from Subgraph. However, our result does not match CoinGecko's market price data.

Subgraph - Implied $rDPX Market Price Pool
CoinGecko - $rDPX Market Price
Difference %

0.00000000000000028
40.36
-99%

2.) inputTokenBalances

The inputTokenBalances data should only be around 107,000, as shown in the Arbiscan explorer.

Subgraph - inputTokenBalances
Aribiscan - inputTokenBalances
Difference %

107663491032186000000000.00
107,000
99%

3.) dailyVolumeByTokenAmount

The volume data appears to be largely inflated and is not in line with the daily volume by token in USD.
Refer to the sheet "Pool_snapshot" for details

Please refer to the attached worksheet for more details.

@corerouter QA for Dopex

@corerouter QA for Dopex

Gotcha. I will do some analysis and then reply to the findings. Thanks!

@corerouter to be clear, the revenue on their website is ~7 day new revenue. And that is why it is much higher than our daily new revenue on the subgraph side?
@SerenityFund does what core router said satisfy the QA or are there any pieces that are still missing?
From my perspective the issues seem to be made whole from core router's explanation. Before we finalize anything (@bye43 / @ishraq8) do you want to review/ do a final sign off?

The subgraph looks good. there may be an additional field added as collateralVolume for completeness, i can take a look at that when we add it.

@ishraq8 Is this completed?

One more thing, took a look at the polygon deployment. network is listed as "arbitrum_one" @corerouter .

One more thing, took a look at the polygon deployment. network is listed as "arbitrum_one" @corerouter .

Problem fixed as shown below. Thanks @ishraq8


Support MPI distributed training

I have this in mind for the Mojo target issue - which is really about having the Makefile support composability like the one for llama.cpp. Probably copy-pasta most of what llama.cpp has so the build is using mpicc. Would still need to write the MPI code.

definitely! but this is pretty far down the line, i think we first need to get the 1-GPU version to be super solid.

I regularly write MPI code, so this shouldn't be too complicated to implement. I've started to look though the CPU version to get started. However, I do have questions regarding the ML side.
There a few options I can see:

Data Parallelism using MPI_Allreduce to average gradients
-  I think we would do this around here:
-  https://github.com/karpathy/llm.c/blob/master/train_gpt2.c#L906C1-L906C5
Tensor parallelism (similar to lamma.cpp)
Model Parallelism

Is there preference to how this could be scaled with MPI? If option 2 or 3, seem like the best option, do you have a suggestion as to where in the code I should dig into?

Sounds great! I expect to get started with the backward pass somewhere over the weekend most likely.
(I spent today optimizing the forward pass still)
Once we have the backward pass getting data parallel training in will be super awesome

I would do MPI-2 as MPI IO is all you need and it is most widely supported.

The MPI version of this is mostly working at this point. I've tested it up to 8 nodes. It reduces training by many hours.
@karpathy Do you still have interest in a NCCL version? If so, are resources for multi-GPU resource that you could share?

This is interesting. @Yiltan, are you still working on an MPI version?

@ibrumar Im non longer working on the MPI version. The GPU version of the project is more performant and I think somebody else has already contributed a NCCL version for distributed training


hwmv2: Out of tree hwmv1 boards/socs do not work

Describe the bug
Seemingly getting a second zephyr checkout on 3.6 and commenting the deprecation of a board then prevents the original board from being found. Indeed it seems that scripts/list_boards.py just returns nothing at all when using the SOC and BOARD roots of the zephyr 3.6 directory.
To Reproduce
Comment out a board deprecating and try building for it with BOARD_ROOT and SOC_ROOT set to the zephyr 3.6 directory. As a bonus just run the list_boards script, e.g. /tmp/aa/zephyr/scripts/list_boards.py --board-root=/tmp/bb/zephyr/ --arch-root=/tmp/aa/zephyr --soc-root=/tmp/bb/zephyr/ gives no output at all. Running the 3.6 script with the same parameters minus SOC root gives a full output /tmp/bb/zephyr/scripts/list_boards.py --board-root=/tmp/bb/zephyr/ --arch-root=/tmp/aa/zephyr:
...
nios2:
  altera_max10
  qemu_nios2
posix:
  native_posix
  native_posix_64
  native_sim
  native_sim_64
  nrf52_bsim
  nrf5340bsim_nrf5340_cpuapp
  nrf5340bsim_nrf5340_cpunet
riscv:
  adp_xc7k_ae350
  beaglev_fire
  esp32c3_devkitm
  esp32c3_luatos_core
  esp32c3_luatos_core_usb
...

Expected behavior
Boards to be detected and work
Impact
Showstopper, release blocker
Environment (please complete the following information):

Commit SHA or Version used: 72a17b6d887ef20b3cc08a25d8ea1d622792d57a

same is seen for Zephyr modules.


LAB_02b-Manage_Governance_via_Azure_Policy.md

Current Translation:
Tugas 3: Terapkan kunci sumber daya
Suggested Translation:
Tugas 3: Menerapkan kunci sumber daya
Description:
Should be in active sentence, not imperative sentence

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

I can't find Current Translation.


[MM-40739] Make-width-fit-content

Summary

Ticket Link

Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

SAFARI

CHROME

FIREFOX

Release Note

Styling change

Hello @bhimeshchauhan,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.
This is a standard procedure for many open source projects.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

/check-cla

@hmhealey It was a different issue but I have added the fix. Thanks for noticing that! 💪

/e2e-test

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/108434

/update-branch


aws_iam_policy_attachment used for additional policies

aws_iam_policy_attachment creates exclusive attachments, I think it makes more sense to use aws_iam_role_policy_attachment in those cases:
https://github.com/terraform-aws-modules/terraform-aws-lambda/blob/5ab6ccbd5566cb2e3cdeb0d2cae5ce4950be711c/iam.tf#L226
https://github.com/terraform-aws-modules/terraform-aws-lambda/blob/5ab6ccbd5566cb2e3cdeb0d2cae5ce4950be711c/iam.tf#L238

This should be the case for all attachments as it can override or removing existing policies from other roles.

As far as I can see, these two are the only ones that allow using existing policies. Other attachments either create new policies or use inline policies.

I just ran into this issue as well... does anyone know if there are any plans to correct this?

I agree, it makes sense, fixed in #45, v1.20.0 has been just released.


Need for offcanvas menu and vertical accordion menu

I think it would be very helpful if there is offcanvas menu and vertical accordion menu (whithout ID )

First part is a duplicate of #24718. No plans for the second part other than our existing accordion options in our collapse plugin.


bug: after disconnecting from metamask burner wallet is connected

Is there an existing issue for this?

[X] I have looked through the existing issues

Which method was used to setup Scaffold-ETH 2 ?
git clone, npx create-eth@latest
Current Behavior
In some cases (see steps to reproduce) after disconnecting from metamask you are connected to burner wallet
https://github.com/scaffold-eth/scaffold-eth-2/assets/25638585/b6847d71-de79-4ef0-8337-4c8663c614e0
Expected Behavior
I'm not connected to any wallet after disconnect
Steps To Reproduce

Connect Metamask
Refresh the page
Disconnect from Metamask

Anything else?
No response

closing this since it's default behavior :(
Fixing this requires to change default connection of burner-connector to false and changing logic on the client, so I think it's not worth it
cc @carletex @technophile-04


refactor(runtime-core): simplify traverse

Here it seems that the Array can be merged into the Set Map and used forEach to traverse.

@baiwusanyu-c Of course, if I consider the performance issue to the extreme, I need to close this PR, but I found that forEach is also used in other places.

Traverse is used in deep watch and can be dealing with unknown object size, and therefore can be very perf sensitive. This is intentional for perf.

Thank you, got it.


Fix memory leaks in busywork/bin/fabricLogger

Description

Although Tcl uses reference-counting garbage collection for temporary objects, some objects like the HTTP "tokens" returned by http::geturl are defined in a global symbol table and have infinite lifetimes unless explicitly freed. There were a few places I forgot to delete these objects in the fabricLogger utility. Those places have been identified and fixed.
Motivation and Context

Fixes #1746
How Has This Been Tested?

I've run busywork test cases and followed the memory utilization of the fabricLogger using top. The obvious memory leak is now gone.
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

The continuous integration build process will run make checks to confirm that tests pass and that code quality meets minimum standards. You may optionally run this locally as PRs will not be accepted until they pass.
Signed-off-by: Bishop Brock<EMAIL_ADDRESS>

@bcbrock Anyone in particular who you'd like to review this PR? I'm not qualified to review Tcl, but based on what you've written LGTM.

@srderson As of today, although other people have used busywork, I have not had any conversations with anyone about any of the Tcl implementations, so I can't recommend any other source-level reviewers. This one is so straightforward that I would humbly suggest that it be integrated. On the next busywork PR (which is more complex) I will suggest a review process.


ag-grid license

Hi,
Can you clarify something please? The ag-grid is not MIT for all uses. Your library and license agreement seems to effectively hide this. Can you add clarity to your page or correct me if I'm wrong?
Thanks,
John

ag-grid is free
ag-grid-enterprise is not free
ag-grid-aurelia is free and can be used with either, however if you use with ag-grid-enterprise, then you need an enterprise license.


dialogSize property does not effect width on flutter for web

@imtoori  dialogSize property only changes the height, it does not change the width on flutter web.
It's a Material Modal Bottom Sheet's issue,
If you wrap SelectionDialog in Center Widget that fixes the issue
Originally posted by @ali-raza-flutter in https://github.com/imtoori/CountryCodePicker/issues/59#issuecomment-717760133

Thanks for the heads up
For the next time consider opening a pull request :)
it would be nice both for you and the maintainer of the project

Thanks for the quick response.
I will keep that in mind 👍


Move CloudWatch metrics and alarms to CDK

For SOC 2 compliance, Medplum has a set of CloudWatch metrics and alarms.  At present, those are managed via AWS Console, but that is unmaintainable.  We should move them to CDK for proper versioning and change tracking.  Furthermore, it would make it easier for self-hosting customers to get the benefit of the work as well.
Unauthorized API Calls
{ ($.errorCode = *UnauthorizedOperation) || ($.errorCode = AccessDenied*) }

Sign In Without MFA
{ ($.eventName = ConsoleLogin) && ($.additionalEventData.MFAUsed != Yes) }

Root Account Usage
{ $.userIdentity.type = Root && $.userIdentity.invokedBy NOT EXISTS && $.eventType != AwsServiceEvent }

IAM Policy Changes
{($.eventName=DeleteGroupPolicy)||($.eventName=DeleteRolePolicy)||($.eventName=DeleteUserPolicy)||($.eventName=PutGroupPolicy)||($.eventName=PutRolePolicy)||($.eventName=PutUserPolicy)||($.eventName=CreatePolicy)||($.eventName=DeletePolicy)||($.eventName=CreatePolicyVersion)||($.eventName=DeletePolicyVersion)||($.eventName=AttachRolePolicy)||($.eventName=DetachRolePolicy)||($.eventName=AttachUserPolicy)||($.eventName=DetachUserPolicy)||($.eventName=AttachGroupPolicy)||($.eventName=DetachGroupPolicy)}

CloudTrail Configuration Changes
{ ($.eventName = CreateTrail) || ($.eventName = UpdateTrail) || ($.eventName = DeleteTrail) || ($.eventName = StartLogging) || ($.eventName = StopLogging) }

Sign In Failures
{ ($.eventName = ConsoleLogin) && ($.errorMessage = "Failed authentication") }

Disabled CMKs
{($.eventSource = kms.amazonaws.com) && (($.eventName=DisableKey)||($.eventName=ScheduleKeyDeletion)) }

S3 Policy Changes
{ ($.eventSource = s3.amazonaws.com) && (($.eventName = PutBucketAcl) || ($.eventName = PutBucketPolicy) || ($.eventName = PutBucketCors) || ($.eventName = PutBucketLifecycle) || ($.eventName = PutBucketReplication) || ($.eventName = DeleteBucketPolicy) || ($.eventName = DeleteBucketCors) || ($.eventName = DeleteBucketLifecycle) || ($.eventName = DeleteBucketReplication)) }

ConfigService Changes
{($.eventSource = config.amazonaws.com) && (($.eventName=StopConfigurationRecorder)||($.eventName=DeleteDeliveryChannel)||($.eventName=PutDeliveryChannel)||($.eventName=PutConfigurationRecorder))}

Security Group Changes
{ ($.eventName = AuthorizeSecurityGroupIngress) || ($.eventName = AuthorizeSecurityGroupEgress) || ($.eventName = RevokeSecurityGroupIngress) || ($.eventName = RevokeSecurityGroupEgress) || ($.eventName = CreateSecurityGroup) || ($.eventName = DeleteSecurityGroup)}

Network ACL Changes
{ ($.eventName = CreateNetworkAcl) || ($.eventName = CreateNetworkAclEntry) || ($.eventName = DeleteNetworkAcl) || ($.eventName = DeleteNetworkAclEntry) || ($.eventName = ReplaceNetworkAclEntry) || ($.eventName = ReplaceNetworkAclAssociation) }

Network Gateway Changes
{ ($.eventName = CreateCustomerGateway) || ($.eventName = DeleteCustomerGateway) || ($.eventName = AttachInternetGateway) || ($.eventName = CreateInternetGateway) || ($.eventName = DeleteInternetGateway) || ($.eventName = DetachInternetGateway) }

Route Table Changes
{ ($.eventName = CreateRoute) || ($.eventName = CreateRouteTable) || ($.eventName = ReplaceRoute) || ($.eventName = ReplaceRouteTableAssociation) || ($.eventName = DeleteRouteTable) || ($.eventName = DeleteRoute) || ($.eventName = DisassociateRouteTable) }

VPC Changes
{ ($.eventName = CreateVpc) || ($.eventName = DeleteVpc) || ($.eventName = ModifyVpcAttribute) || ($.eventName = AcceptVpcPeeringConnection) || ($.eventName = CreateVpcPeeringConnection) || ($.eventName = DeleteVpcPeeringConnection) || ($.eventName = RejectVpcPeeringConnection) || ($.eventName = AttachClassicLinkVpc) || ($.eventName = DetachClassicLinkVpc) || ($.eventName = DisableVpcClassicLink) || ($.eventName = EnableVpcClassicLink) }

Organizations Changes
{ ($.eventSource = organizations.amazonaws.com) && (($.eventName = AcceptHandshake) || ($.eventName = AttachPolicy) || ($.eventName = CreateAccount) || ($.eventName = CreateOrganizationalUnit) || ($.eventName = CreatePolicy) || ($.eventName = DeclineHandshake) || ($.eventName = DeleteOrganization) || ($.eventName = DeleteOrganizationalUnit) || ($.eventName = DeletePolicy) || ($.eventName = DetachPolicy) || ($.eventName = DisablePolicyType) || ($.eventName = EnablePolicyType) || ($.eventName = InviteAccountToOrganization) || ($.eventName = LeaveOrganization) || ($.eventName = MoveAccount) || ($.eventName = RemoveAccountFromOrganization) || ($.eventName = UpdatePolicy) || ($.eventName = UpdateOrganizationalUnit)) }

Fixed in https://github.com/medplum/medplum/pull/2825


Updating tracy version.

Yes, time to update the capture tool. Let me do that.

thanks!

(and no rush!)

Okay uploaded a new tracy capture tool. Let's see how it goes.

Worked on the moto, at least.
Now have GPU stats though we still have the issue with the zones not deduplicating ;(

Really wish we knew why that happened.

Sampling and all worked on the moto, so at least android works - just something about the pixel phones

Hmm, mysterious "segmentation fault" coming from both Pixel phones. On both phones, captures succeeded before adb shell cd /data/local/tmp/iree-benchmarks/TFLite/MobileNetV2-fp32,imagenet/iree-dylib-sync__CPU-ARM64-v8A__little-core,full-inference,experimental-flags && TRACY_NO_EXIT=1 taskset 0f /data/local/tmp/iree-benchmarks/traced-tools/iree-benchmark-module --flagfile=flagfile. There might be something weird specific to this case?

Actually searching found other "segmentation fault" but doesn't affect things??

The segfault might be happening during Tracy's complex (involving thread exit hooks), somewhat suspect (e.g. TSan failures) shutdown sequence?  It would be interesting for sure to see that captured in a debugger.
As far as I'm concerned, this update is a necessary step anyway to playing with Tracy on recent Android devices, so 👍 to merging this as soon as CI is green.


check child is json or not in zookeeper.it will be continue if not.

What is the purpose of the change
catch the exception when the url can't be deserialization.related issue
Brief changelog
add the json check when ANY_VALUE.equals(url.getServiceInterface()) at ZookeeperRegistry#doSubscribe.
Verifying this change
the child will work normal if the child is json.It can still work when dubbo-admin run.

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] GitHub Actions works fine on your own branch.

@CrazyHZM hello,In the last commit, I only make the logger for warnings and did not change any logic. However, it still did not pass the checks.That is strange.I checked the logs, and it seems to be a timeout issue with Zookeeper. Could the success of the checks be related to network factors?

@AlbumenJ I catch the exception,but i can't pass the CI/CD.I am trying to fix it.

@AlbumenJ Take a look,please.

Also, pls add some test cases

@AlbumenJ I had finish it,could you take a look please?


hotfix/restrooms -> dev

This change is

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Make OSS product long name actually long

'Code - OSS' didn't look great in the launcher.
Fixes #2982

@joaomoreno is this is fine for Windows and Mac? Matches format of official build.

I believe this was a purposeful choice. We don't want to apply the Visual Studio brand to the OSS flavour of Code. @chrisdias has the word on this.

It might actually be useful in communication to refer to the distinction as "vscode" and "osscode" (or "oscode"?).  This would have probably helped out in issue #60 (see comment 6).  Compare to Chromium and Google Chrome.
Maybe @chrisdias has an opinion on this?

The OSS version is just that - the OSS version, it is not Visual Studio. It is only "Visual Studio" when we build the distro which applies our branding, licensing, telemetry, etc. We should not merge this in.

@chrisdias thanks for the clarification :)

@Tyriar Close #2982?


Rename signingkey and SignatureKey

Signed-off-by: feihujiang<EMAIL_ADDRESS>I think signingkey and SignatureKey here isn't really a signing key, it is just the name of the key related to the signature. I renamed them to avoid confusing and fixed the wrong case in test_signed_object.

@ojalatodd I updated this PR, please have a look.

@vaporos I updated this PR according to your advice, please take a look.


Implement KYC, interest calculation, digital identity, atomic transactions, anonymous transactions and max limit in CBDC

This commit adds the following features to the Norwegian CBDC based on the ERC20 standard:

KYC compliance: Banks perform KYC checks on users with Verifiable Credentials before they can use the CBDC
Digital identity: Integrated with BankID and EU's upcoming digital identity wallet
Interest calculation: Both positive and negative interest rates are supported
Max limit: Implements a cap on the amount of CBDC a user can hold in their wallet
Anonymous transactions: Implemented stealth addresses based on Vitalik's concept to enable private and secure transactions without revealing user identities

Please refer to the blog post (WIP) for more details on each feature implementation.

@FredrikBakken


Improve Schema Scan by column prune

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：
For https://github.com/StarRocks/starrocks/issues/2976
Problem Summary(Required) ：
Core improve point: Only fill necessary columns to result chunk.
For SQL:  select count(column_name from information_schema.columns.
Before this PR:
       - FillChunk: 4s966ms
       - NumDiskAccess: 0
       - PeakMemoryUsage: 0
       - RowsRead: 0
       - RowsReturned: 6.08M

Afore this PR:
       - FillChunk: 394.469ms
       - NumDiskAccess: 0
       - PeakMemoryUsage: 0
       - RowsRead: 0
       - RowsReturned: 6.08M

run starrocks_be_unittest

run starrocks_be_unittest


World generation mappings

I did some more world generation mappings.

This needs an update to fix the merge conflict.

I mean, the "netherrack" doesn't matter at all in the name. It can be configured to use any block.

Hmm, this is like naming a dimension type. You can create new instances of this feature with different registry name (like create mymod:replace_blobs feature with this class), yet meanwhile you can create configured versions of the netherrack_replace_blobs that replace blobs with different targets than netherrack. So yeah it's an either way.
Also rerequesting as to invalidate that previous change request.


component_expand issue

I'm having trouble getting this to work:
let g:lightline = {
	  \ 'active': {
	  \   'right': [ [ 'my_flag' ] ]
	  \ },
	  \ 'component_expand': {
	  \   'my_flag': 'MyFlag', " doesn't work
	  " \   'my_flag': 'MyFlagPre', " works
	  \ },
	  \ 'component_type': {
	  \   'my_flag': 'error',
	  \ }
	  \ }

function! MyFlagPre() abort
  return '%{MyFlag()}'
endfunction

function! MyFlag() abort
  return &modified ? '+' : ''
endfunction

The idea is to evaluate the component in the context of current window.
Basically I would like to understand why I have to create wrapper functions that do %{...} and if there is a cleaner way, something like
let g:lightline.component_expand_raw': {
	  \   'my_flag': '%{MyFlag()}',
	  \ }

You call lightline#update() correctly, right? Calling this function is required to update the contents of component_expand.

Actually I don't call it anywhere. What's the best way to do it in my case? Set a timer, or use autocommand?

You can call lightline#update on the TextChanged event, but since it is triggered too frequent so it sacrifice the performance. I don't think the modified flag is an error so I propose to start with component_function. Users are recommended to use component_expand for components indicating some errors or warnings. Most of syntax-checking plugins have hooks triggered after their checks where users can update the lightline statusline.

Thank you, that made things a lot clearer to me.


CAMEL-9587 - camel-restlet - Make it easy to turn on gson or jackson

With this PR one can explicit set the converters to use with:
from("restlet:....?enabledConverters=JacksonConverter,my.company.MyConverter")

Notes:

the converted need to be installed
by default all the converter found by the Engine are enabled
converters not used are removed from the engine which is a singleton s

To make it easy to add Jackson/Gson extension there are now two more features:

camel-restlet-jackson
camel-restlet-gson

Hope it cope with the requirements.

It would be good to have an option on the component level also so you can set this "globally" and not must use the endpoint uri config.

Agree but wouldn't be better to have it only on component level ?
Configuring it on different levels could lead to errors as it tweak a singleton

Yeah component level only makes more sense when its global on restlet itself also

Done


WT command option parsing error

Windows Terminal version (or Windows build number)
1.11.2921.0
Other Software
Windows  10.0.22000.0
Steps to reproduce
run wt -version command, single hyphen
Expected Behavior
No response
Actual Behavior

So, that's definitely weird, but I suspect that's just an artifact of the way CLI11 parses flags. It's treating -version similar to -v -e -r -s -i -o -n. The -v it recognizes, but the rest are not.


Unable to fetch other user outlook calendar events using MSGraphClient in SPFX webpart

Category
Question
Hi All,
I am facing issue while fetching other O365 tenant users calendar events using MSGraphClient in SPFX Webpart. I am able to get records for api "/me/events", "/users/events" but whenever I am specifying user like "/users/{id/upn}/events" its giving 403 access denied. I provided permission scope for Microsoft Graph to read user calendar in package-soution.json as webAPIPermissionRequests. Tenant Admin also approved the permission scope.
Below is my scope as in package-solution.json file
"webApiPermissionRequests": [
  {
    "resource": "Microsoft Graph",
    "scope": "User.Read.All"
  }
  ,

  {
    "resource": "Microsoft Graph",
    "scope": "Calendars.Read"

  },
  {
    "resource": "Microsoft Graph",
    "scope": "Calendars.ReadWrite"

  }

]
Also below is my code snippet which I used.
**this.props.msGraphClientFactory.getClient()
.then((client: MSGraphClient):void=>{
  client.api('/users/{id/upn}/events').get((err, response:any,rawResponse?:any)=>{
    console.log(response)
    console.log(rawResponse)
  })

})**
The access token which is being generated also includes my permission scope which I have defined.
Looking forward to any suggestions.. where am I going wrong? Thanks in advance

Hi Team,
Looking forward to your response

The user that is initiating this request from SPFx must also have access to that user's calendar that they are hitting. The permissions you are getting are delegated permissions, not application permissions, so the access token is in the context of the current user. The scopes you've granted the app say the app CAN use those permissions, but it doesn't mean the actual user has access to the target user's mailbox.
This permission is granted within Outlook.

Hi Andrew,
Thanks for your valuable input, indeed if i generate token and pass it along with the Api call as auth header I am getting the response. Is there anything I am missing on permission part in my config json file? or i need to pass explicitly the token from spfx to the api ?  Looking forward to your inputs.

Please re-read my comment.
Specifically This permission is granted within Outlook.
Permission grants to the Microsoft Graph only say "This application (SPFx) can DO [these permissions] on that application (Graph)" It doesn't say "Jane as given John rights to read her calendar".
That permission is defined within the mailbox. Not in SharePoint. Not in Microsoft Graph.

Hi Andrew,
If I understood correctly , user needs to specifically share their
calendars with the respective person from their outlook, like John will
share his calendar from his outlook with Jane and Jane needs to accept that
to get this working.
Regards,
Soumyadeep
On Tue, Sep 10, 2019, 10:04 PM Andrew Connell<EMAIL_ADDRESS>wrote:

Please re-read my comment
https://github.com/SharePoint/sp-dev-docs/issues/4599#issuecomment-529924979
.
Specifically This permission is granted within Outlook.
Permission grants to the Microsoft Graph only say "This application (SPFx)
can DO [these permissions] on that application (Graph)" It doesn't say
"Jane as given John rights to read her calendar".
That permission is defined within the mailbox. Not in SharePoint. Not in
Microsoft Graph.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SharePoint/sp-dev-docs/issues/4599?email_source=notifications&email_token=ANDZGII3JPTOLYNALVB23BLQI7EB3A5CNFSM4IUP6SB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LXEJY#issuecomment-530018855,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANDZGIMC3B2RY3TST5NXXMLQI7EB3ANCNFSM4IUP6SBQ
.

Sort of... Jane doesn't need to accept anything. Once John shares his calendar with Jane, then Jane has permissions to access it. She can open it up in Outlook on desktop, Outlook Web, Outlook iOS, etc.
However, that wouldn't be enough to access it using SPFx via the Microsoft Graph. The reason why is because SharePoint would need the Calendar.Read permission to use that scope with the Graph. You've already done that part though so you should be good. Mentioned it just for the sake of completeness.

Thanks Andrew for your inputs.
On Wed, Sep 11, 2019, 12:49 AM Andrew Connell<EMAIL_ADDRESS>wrote:

https://camo.githubusercontent.com/f8fa196bb641bf905e67fb6bd8a8ba3e165c4744/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f6c314a3977586f433857344a466d5245592f67697068792e676966
Sort of... Jane doesn't need to accept anything. Once John shares his
calendar with Jane, then Jane has permissions to access it. She can open it
up in Outlook on desktop, Outlook Web, Outlook iOS, etc.
However, that wouldn't be enough to access it using SPFx via the Microsoft
Graph. The reason why is because SharePoint would need the Calendar.Read
permission to use that scope with the Graph. You've already done that part
though so you should be good. Mentioned it just for the sake of
completeness.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SharePoint/sp-dev-docs/issues/4599?email_source=notifications&email_token=ANDZGIJNTMMF3EEXQCBRJMDQI7XL3A5CNFSM4IUP6SB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6MGPJY#issuecomment-530081703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANDZGIN5TBIYEJQUOKKOI3TQI7XL3ANCNFSM4IUP6SBQ
.


More enhancements needed for TaxBrain results tables

The tax analysis described in Tax-Calculator issue 1335 reveals the need for several more enhancements to the TaxBrain results page.

For some reason the terminology for the diagnostic tables and the difference tables are different.
The difference tables sensibly use "by" (as in "by income decile") while the diagnostic tables use the strange terminology "per" (as in "per income decile").  Can we standardize on "by"?

The Print|Copy|CSV|Excel boxes just below the upper Total Liabilities section disappear when users change from Change to either Current Law or Reform.  And the boxes don't come back when clicking on Change.  This means there is no way to download the Current Law or Reform Total Liabilities.  Surely this is not by design.  Can we make it so the four boxes don't disappear?

When clicking on either Current Law or Reform the row labels in the Total Liabilities table don't switch to a more appropriate label.  In all three versions of the table the row label ends in Change, which is appropriate in only one of the three cases.  Can we drop Change from the end of each of the table row labels?

The title of the Total Liabilities table ends with (2017) --- when that is the Start Year --- in the Current Law and Change version of the Total Liabilities table even though all ten years are shown in the table.  Can we remove the (2017) at the end of those two table titles?

A key statistic is missing from the Total Liabilities tables: the ten-year totals.  Most users will want the ten-year total for each row in the Total Liabilities table, but it is not there at the end of the row where most users would expect to see it.  Can we add a ten-year total column on the far right?

The downloaded CSV file has all the numerical values enclosed in quotation marks (like "1,234.5").  This is a problem for users who have uploaded reform/assumption files to TaxBrain and want to download TaxBrain results for analysis on their local computer using command-line tools.  Can the commas and enclosing quotation marks be removed from the downloaded CSV file?

@MattHJensen @feenberg @PeterDSteinberg @brittainhard @enelson1995

@martinholmer, I added Point 1 and Point 4 to #534. Point 2 was already listed in #534.
Would you mind closing this issue and opening a new one for the 10-year totals? I would be happy to do so if you'd prefer.


Window Controls interferes with dragging compendium effects on to tokens

Per the title, when Window Controls is active (and all modules other than libwrapper disabled), I am 90% of the time unable to drag status effects on to tokens.

The browser dev console, reveals nothing - it doesn't show some code failing to fire, but rather it doesn't even register that anything was attempted during each drag and drop.
Occasionally, for a short spurt, I will be able to drag effects for a 5 second window before it stops working again.
Foundry Version 10.291
Pathfinder 2e System Version 4.7.4

@uninvitedguest , maybe the problem comes from the issue I just submitted #37 . Do you use top or bottom taskbar?

Sounds like a likely explanation!


Can't run my Main2.java file

Hi everyone! I am trying to run my Main2.java file but this error always pop up. Not sure why though....

Hi all, I have resolved the problem. The solution is to use Ctrl Z instead of Ctrl D. :) What I found online is, Ctrl-Z is the DOS command code for Windows (NOTE THIS) for end of input (the UNIX equivalent is Ctrl-D).


Add proxy option

I miss a proxy option for this tool so that we can pass a proxy URL like we do with the youtube-dl tool.
Is it possible to use youtube-search-python through a proxy even without such option? I was trying something like that:
    {...}
        http_proxy = f"http://{set_proxy()}"
        os.environ['HTTP_PROXY'] = http_proxy
        os.environ['http_proxy'] = http_proxy
        orig_get = requests.get
        proxies = {
            'http': http_proxy,
        }
        requests.get = partial(orig_get, proxies=proxies)

    try:
        videosSearch = CustomSearch(query=query,
                                    region=region,
                                    language=language,
                                    searchPreferences=set_search_pref(),
                                    limit=limit*2)
    {...}

#63
This feature has already been proposed.
What is also fascinating about your example is that it doesnt proxy anything.
I can see you are using requests package for proxying, and this would probably work if we'd be using it. We aren't dependend on requests package nor we are using it.

Oh cool, sorry for this post then and thanks for the clarification about my "solution".
Best! ;)

@mytja sounds amazing! Could you provide a code example of how we have to set up the proxy to use it with youtube-search?

how about socks5://<IP_ADDRESS>:1080 @mytja


Survey panel scale and position

Great tool, thanks for making this available.
I tried to customise a survey in regards to default scale and position in the world.
However, I'm unable to set a scale of position in the editor, at play it always reverts to default values. Tried to scale a parent gameobject or the survey, the survey gamebject and also the page & questionnaire prefabs. Also tried commenting out some of the localscale commands in the script, but no luck. Can you help pls.
Another thing, I tried to add a close button to the final page, using the lastPage prefab, but not sure how to go about this. If I change the script, so the footer is displayed for the last page, it is overwritten by the default next/previous buttons.
BTW, it was fairly simple to add the functionality to save the survey result on a server with webrequest.post and a php script.

For resizing, please see the Manual (https://github.com/MartinFk/VRQuestionnaireToolkit/wiki/📖-Manual). Section 5 explains how to do this. It is supported since version 1.1 in case you are still using 1.0. The world-anchoring described in section 4 only works in ConfigurationMode (when running the scene). It is misleading in the documentation -> I will update this. The size/position settings are stored in a separate file and loaded when starting the scene.

I am afraid, I do not really understand what you want to do. On the final page there is a submit button, right? Do you want to change the text to "Close"?

Awesome! If you don't mind it would be great if you could share this with us (you would get the credit for it). I think many people would benefit from this given the current situation.

Thanks.
I'll try the resizing this way.
At present, the last page of a survey ( after submit ) is just acknowleding and thanking the participant. This page does not, by default, have a button.  However, I mananged to add a footer with close button to the last page prefab. So this works now.
For the server side data upload, here is what i did:
c# code, added to existing Export.cs
IEnumerator UploadPost(string data)
{
WWWForm form = new WWWForm();
form.AddField("unitypost", data);
using (UnityWebRequest www = UnityWebRequest.Post("https://domain.com/surveyData.php", form))
{
www.downloadHandler = new DownloadHandlerBuffer();
yield return www.SendWebRequest();
        if (www.isNetworkError)
        {
            Debug.Log(www.error);
        }
        else
        {
            string responseText = www.downloadHandler.text;
            Debug.Log("Response Text from the server = " + responseText);
        }
    }
}

call: StartCoroutine(UploadPost(string));
I re-formatted the survey result in export.cs, so each participant is a row in the server csv file. Easier to analyze the data in this way:
dateTime;Q1;Q2;Q3;Q4;Q5
22-02-2021-08-28-11;1;2;3;4;5
22-02-2021-08-28-41;5;4;3;2;1
22-02-2021-11-53-05;1;1;1;1;1
22-02-2021-12-05-55;1;2;3;4;5
server side:
empty text file: surveyData.txt
Php script: surveyData.php :

Thanks you very much for your input!
Let me know if you need anything else.


AbstractDataSourceExtension now uses Config.getPropertyNames()

Signed-off-by: Laird Nelson<EMAIL_ADDRESS>

Hi, @tjquinno and @barchetta; I am always shaky on my attempts to apply PRs against the 1.x branch.  This PR is basically equivalent to #2322 but for the 1.x branch.  At your convenience.


Java client pagination API

Coverage increased (+0.02%) to 93.181% when pulling 43958fee7e0d03cb7e8a76737d813c7c29950039 on mprimi:GENIE-673-java-client-pagination into 155913f0ba1977c391238ab07cee089fa967ea6f on Netflix:master.


definition of ZIsDistance missing

Receive this error just opening in unity (2017.2.0p2) the toolkit (tiltbrushtoolkit-UnitySDK-v14.0.0.unitypackage)
Assets/TiltBrush/Scripts/Editor/ModelImportSettings.cs(226,46): error CS0117: TiltBrushToolkit.BrushDescriptor.Semantic' does not contain a definition for ZIsDistance'

That's defined on line 28 of Assets/TiltBrush/Scripts/BrushDescriptor.cs. You probably have a duplicate definition somewhere, perhaps from importing Poly Toolkit. Poly Toolkit and Tilt Brush Toolkit cannot currently coexist in the same Unity project, at least not easily.
It may work if you do something like this:

import Poly Toolkit first, which has its own copy of TBT in Assets/PolyToolkit/Internal/tbt
import Tilt Brush Toolkit (but only those files which haven't already been imported by Poly Toolkit)
the "ZIsDistance" compile error. Either search-and-replace "Semantic.ZIsDistance" with "Semantic.XyIsUvZIsDistance", or add a definition for that enum value here:

Assets/PolyToolkit/Internal/tbt/Scripts/BrushDescriptor.cs line 28
XyIsUvZIsDistance,
ZIsDistance = XyIsUvZIsDistance,   // <-- add this line

If that works out for you, I can patch TBT so this works a bit better out of the box.

Correct. I was playing with both sdk, separately but in the same project. I have no need for both in the same project and trying them in separate projects worked fine. Thanks.


Dart analysis server Error

Analyzer Feedback from IntelliJ
Version information

IDEA AI-181.55<IP_ADDRESS>56338
2.1.0-edge.a2eb050044eec93f0844667b8b6132e858467e4e
AI-181.55<IP_ADDRESS>56338, JRE 1.8.0_152-release-1136-b06x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.14 unknown, screens 1440x900  Retina

Exception
Dart analysis server, SDK version 2.1.0-edge.a2eb050044eec93f0844667b8b6132e858467e4e, server version 1.20.5, error: Analysis failed: /Users/am/dev/projects/flutter/learn/smart_qfnu/lib/bloc/notification_bloc.dart context: exception_20181030_150930_322
Exception: Element not found in summary: package:rxdart/src/subjects/publish_subject.dart;package:rxdart/src/subjects/publish_subject.dart;Observable
#0      SummaryResynthesizer.getElement (package:analyzer/src/summary/resynthesize.dart:308:9)
#1      ElementHandle.actualElement (package:analyzer/src/dart/element/handle.dart:345:42)
#2      ClassElementHandle.actualElement (package:analyzer/src/dart/element/handle.dart:35:43)
#3      ClassElementHandle.type (package:analyzer/src/dart/element/handle.dart:111:29)
#4      _ReferenceInfo._buildType.<anonymous closure> (package:analyzer/src/summary/resynthesize.dart:996:48)
#5      InterfaceTypeImpl.typeArguments (package:analyzer/src/dart/element/type.dart:1389:26)
#6      DeclarationResolver._setGenericFunctionType (package:analyzer/src/generated/declaration_resolver.dart:523:37)
#7      DeclarationResolver.visitVariableDeclarationList (package:analyzer/src/generated/declaration_resolver.dart:476:7)
#8      VariableDeclarationListImpl.accept (package:analyzer/src/dart/ast/ast.dart:11481:15)
#9      FieldDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:4387:17)
#10     RecursiveAstVisitor.visitFieldDeclaration (package:analyzer/dart/ast/visitor.dart:834:10)
#11     DeclarationResolver.visitFieldDeclaration (package:analyzer/src/generated/declaration_resolver.dart:197:11)
#12     FieldDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:4382:49)
#13     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:8304:20)
#14     ClassDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:1795:13)
#15     RecursiveAstVisitor.visitClassDeclaration (package:analyzer/dart/ast/visitor.dart:690:10)
#16     DeclarationResolver.visitClassDeclaration.<anonymous closure> (package:analyzer/src/generated/declaration_resolver.dart:89:13)
#17     DeclarationResolver._walk (package:analyzer/src/generated/declaration_resolver.dart:543:13)
#18     DeclarationResolver.visitClassDeclaration (package:analyzer/src/generated/declaration_resolver.dart:88:5)
#19     ClassDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:1765:49)
#20     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:8304:20)
#21     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2437:21)
#22     RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:714:10)
#23     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2430:49)
#24     DeclarationResolver.resolve (package:analyzer/src/generated/declaration_resolver.dart:51:12)
#25     LibraryAnalyzer._resolveFile (package:analyzer/src/dart/analysis/library_analyzer.dart:566:31)
#26     LibraryAnalyzer._analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:102:9)
#27     __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection/runtime/libcompact_hash.dart:370:8)
#28     LibraryAnalyzer._analyze (package:analyzer/src/dart/analysis/library_analyzer.dart:101:13)
#29     LibraryAnalyzer.analyze.<anonymous closure> (package:analyzer/src/dart/analysis/library_analyzer.dart:78:14)
<asynchronous suspension>
#30     _PerformanceTagImpl.makeCurrentWhileAsync (package:analyzer/src/generated/utilities_general.dart:222:21)
<asynchronous suspension>
#31     LibraryAnalyzer.analyze (package:analyzer/src/dart/analysis/library_analyzer.dart:77:43)
<asynchronous suspension>
#32     AnalysisDriver._computeAnalysisResult.<anonymous closure> (package:analyzer/src/dart/analysis/driver.dart:1244:71)
<asynchronous suspension>
#33     PerformanceLog.runAsync (package:front_end/src/base/performance_logger.dart:52:21)
<asynchronous suspension>
#34     AnalysisDriver._computeAnalysisResult (package:analyzer/src/dart/analysis/driver.dart:1220:20)
<asynchronous suspension>
#35     AnalysisDriver.performWork (package:analyzer/src/dart/analysis/driver.dart:1025:23)
<asynchronous suspension>
#36     AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:1916:24)
<asynchronous suspension>
#37     AnalysisDriverScheduler.start (package:analyzer/src/dart/analysis/driver.dart:1857:5)
#38     new AnalysisServer (package:analysis_server/src/analysis_server.dart:397:29)
#39     SocketServer.createAnalysisServer (package:analysis_server/src/socket_server.dart:81:26)
#40     StdioAnalysisServer.serveStdio (package:analysis_server/src/server/stdio_server.dart:37:18)
#41     Driver.start.<anonymous closure> (package:analysis_server/src/server/driver.dart:504:21)
#42     _rootRun (dart:async/zone.dart:1124:13)
#43     _CustomZone.run (dart:async/zone.dart:1021:19)
#44     _runZoned (dart:async/zone.dart:1516:10)
#45     runZoned (dart:async/zone.dart:1463:12)
#46     Driver._captureExceptions (package:analysis_server/src/server/driver.dart:550:12)
#47     Driver.start (package:analysis_server/src/server/driver.dart:502:7)
#48     main (file:///b/build/slave/Mac_Engine/build/src/third_party/dart/pkg/analysis_server/bin/server.dart:12:11)
#49     _AsyncAwaitCompleter.start (dart:async/runtime/libasync_patch.dart:49:6)
#50     main (file:///b/build/slave/Mac_Engine/build/src/third_party/dart/pkg/analysis_server/bin/server.dart:10:10)
#51     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:287:32)
#52     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

1540883369963 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/models/upload_photo_model.dart","errors":[]}}
1540883369964 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/models/login_model.dart","errors":[]}}
1540883369964 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/models/response_model.dart","errors":[]}}
1540883370088 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/main.dart","errors":[]}}
1540883370094 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/service/news_service.dart","errors":[]}}
1540883370095 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/service/notification_service.dart","errors":[]}}
1540883370097 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/demo/clip_test.dart","errors":[]}}
1540883370100 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/photo_view_page.dart","errors":[]}}
1540883370101 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/every_day/address_book_page.dart","errors":[]}}
1540883370102 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/home/notification_page.dart","errors":[]}}
1540883370103 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/home/news_page.dart","errors":[]}}
1540883370107 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/home/every_day_page.dart","errors":[]}}
1540883370109 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/home/my_info_page.dart","errors":[]}}
1540883370110 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/webview_page.dart","errors":[]}}
1540883370113 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/splash_page.dart","errors":[]}}
1540883370114 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/login_page.dart","errors":[]}}
1540883370115 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/home_page.dart","errors":[]}}
1540883370116 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/skill_details_page.dart","errors":[]}}
1540883370120 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/award_details_page.dart","errors":[]}}
1540883370121 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/part_time_page.dart","errors":[]}}
1540883370122 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/award_edit.dart","errors":[]}}
1540883370128 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/skill_page.dart","errors":[]}}
1540883370129 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/my_details_page.dart","errors":[]}}
1540883370130 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/award_page.dart","errors":[]}}
1540883370131 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/change_password_page.dart","errors":[]}}
1540883370136 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/pages/my/skill_edit.dart","errors":[]}}
1540883370137 => {"id":"6","method":"analysis.updateContent","params":{"files":{"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/bloc/notification_bloc.dart":{"type":"add","content":"import 'base_
1540883370139 <= {"event":"analysis.errors","params":{"file":"/Users/am/dev/projects/flutter/learn/smart_qfnu/lib/widgets/image_swiper.dart","errors":[]}}
1540883370148 <= {"id":"6","result":{}}
1540883370382 <= {"event":"server.error","params":{"isFatal":false,"message":"Analysis failed: /Users/am/dev/projects/flutter/learn/smart_qfnu/lib/bloc/notification_bloc.dart context: exception_201810

Dart analysis server, SDK version 2.1.0-edge.a2eb050044eec93f0844667b8b6132e858467e4e, server version 1.20.5, error: Analysis failed: /Users/am/dev/projects/flutter/learn/smart_qfnu/lib/bloc/notification_bloc.dart context: exception_20181030_150930_322
Exception: Element not found in summary: package:rxdart/src/subjects/publish_subject.dart

Duplicate of #34555


[ansible/artifactory] Bugfix/305

PR Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] Title of the PR starts with installer/product name (e.g. [ansible/artifactory])
[X] CHANGELOG.md updated
[ ] Variables and other changes are documented in the README.md

What this PR does / why we need it:
Fixes excessive logging on single server Artifactory.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #305
Special notes for your reviewer:
edits binarystore.xml based on server count.

looks good , thanks for the changes , will take it in next patch release

fixed via #308 FYI - Added an additional change of removal of binary_store variable , closing this


More info from HEKA file

Hi,
Cool project & preprint!
Just in case you're interested: I've recently spent some time getting lots more data out of HEKA files (including protocols, compensation settings etc). Code is on github if you want to have a look https://github.com/myokit/myokit/blob/main/myokit/formats/heka/_patchmaster.py
Cheers,
Michael
(P.S. Not a real issue, feel free to close!)

Hi, Thanks a lot. This fits to our plans to extend the information extracted from the HEKA files ..  Would be definitely a nice enhancement. One would have to make sure to keep BPE working if  file specific information are not available in other file formats in the downstream analysis since multiple formats can be pooled into the same analysis.
Cheers,
David

Thats good, I will see if I can add this


'NoneType' object has no attribute 'attrs' when upgrading to 2.0.0

I just tried upgrading my project to django-bootstrap 2.0.0, and encountered the following error when loading websites.
Traceback (most recent call last):
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/core/handlers/base.py", line 145, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/core/handlers/base.py", line 143, in _get_response
    response = response.render()
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/response.py", line 105, in render
    self.content = self.rendered_content
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/response.py", line 83, in rendered_content
    return template.render(context, self._request)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/backends/django.py", line 61, in render
    return self.template.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 171, in render
    return self._render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/test/utils.py", line 95, in instrumented_test_render
    return self.nodelist.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 936, in render
    bit = node.render_annotated(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 903, in render_annotated
    return self.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/test/utils.py", line 95, in instrumented_test_render
    return self.nodelist.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 936, in render
    bit = node.render_annotated(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 903, in render_annotated
    return self.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/loader_tags.py", line 150, in render
    return compiled_parent._render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/test/utils.py", line 95, in instrumented_test_render
    return self.nodelist.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 936, in render
    bit = node.render_annotated(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 903, in render_annotated
    return self.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
    result = block.nodelist.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 936, in render
    bit = node.render_annotated(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 903, in render_annotated
    return self.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/loader_tags.py", line 62, in render
    result = block.nodelist.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 936, in render
    bit = node.render_annotated(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/base.py", line 903, in render_annotated
    return self.render(context)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/django/template/library.py", line 192, in render
    output = self.func(*resolved_args, **resolved_kwargs)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/templatetags/bootstrap4.py", line 627, in bootstrap_field
    return render_field(*args, **kwargs)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/forms.py", line 39, in render_field
    return renderer_cls(field, **kwargs).render()
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/renderers.py", line 84, in render
    return mark_safe(self._render())
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/renderers.py", line 515, in _render
    html = self.post_widget_render(html)
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/renderers.py", line 363, in post_widget_render
    html = self.list_to_class(html, "checkbox")
  File "/home/rixx/.local/share/virtualenvs/pretalx/lib/python3.8/site-packages/bootstrap4/renderers.py", line 341, in list_to_class
    label.input.attrs["class"] = label.input.attrs.get("class", []) + ["form-check-input"]
AttributeError: 'NoneType' object has no attribute 'attrs'

This is a direct regression to #181, where django-bootstrap4 assumes that all inputs have to be inside labels (which is not idiomatic or at least not required in HTML).

Thanks. Apparently there is no test on this regression. Will look into that.
The major version dump in 2 is because of the drop n support fro Python and Django versions. You should be fine with django-bootstrap4<2.

Is there any chance you can provide a breaking test? In v1.1.1 we fixed it based on symptoms, but a breaking test would be much better?
Copy/paste here is fine if you do not want to do a PR.


why "common.cc : static const int32 kDefaultTransferNumObjecs = 512;"?

common.cc : static const int32 kDefaultTransferNumObjecs = 512;
why "kDefaultTransferNumObjecs = 512"???

Elevating it helps on some benchmarks. But I recently got evidence that it
hurts some other benchmarks badly enough. So we'll likely revert back to
old value of 32 very soon.
But why you're asking? Do you have some other workloads where it hurts?
On Jan 11, 2017 18:49, "bedragon"<EMAIL_ADDRESS>wrote:

common.cc : static const int32 kDefaultTransferNumObjecs = 512;
why "kDefaultTransferNumObjecs = 512"???
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gperftools/gperftools/issues/862, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA6AkjjUnXIBNQoZV59xdrKwitzV1U_ks5rRZS4gaJpZM4LhUHd
.

Thanks answer my question，I learn tcmalloc source code. I found it to be different, in different versions。so，Its value depends on your experience？

Some benchmarking and some wild guessing.


hlint-3.0 regression: Unrecognized language extension 'Trustworthy'

Consider the following program:
{-# LANGUAGE Trustworthy #-}
module Bug where

On hlint-2.2.11, this produces no hints, as expected:
$ hlint --version
HLint v2.2.11, (C) Neil Mitchell 2006-2020
$ hlint Bug.hs
No hints

With hlint-3.0, however, it crashes:
$ hlint --version
HLint v3.0, (C) Neil Mitchell 2006-2020
$ hlint Bug.hs
hlint: IMPOSSIBLE: Unrecognized language extension 'Trustworthy'
CallStack (from HasCallStack):
  error, called at src/Hint/Extensions.hs:266:11 in hlint-3.0-b26bf2d7d97a0573cc460418b91c96800f45ab73c13e16368e3465671e6a15bf:Hint.Extensions

hlint-3.0 crashes similarly if you enable Safe or Unsafe.

Eek. Thanks for the report. Looking into it now. Seems to be caused because Trustworthy doesn't have an entry in the GHC extensions data type: https://hackage.haskell.org/package/ghc-lib-parser-<IP_ADDRESS>00412/docs/GHC-LanguageExtensions-Type.html#t:Extension. CC @shayne-fletcher

The Haskell98 and Haskell2010 extensions also trigger the same crash.

Thanks - I should have a fix within about 15 mins.

Fix pushed. I'll wait for CI to kick in (15 mins or so) then make a release of HLint 3.0.1. Problem is that these are language pragmas that don't have an associated extension, which we assumed could never happen. I now handle that case in the extension hint, and have added test cases for it.

Yes, they aren't extensions and thus don't appear in xFlags. It turns out there is a dedicated field in DynFlags of type SafeHaskellMode.
-- | The various Safe Haskell modes
data SafeHaskellMode
   = Sf_None          -- ^ inferred unsafe
   | Sf_Unsafe        -- ^ declared and checked
   | Sf_Trustworthy   -- ^ declared and checked
   | Sf_Safe          -- ^ declared and checked
   | Sf_SafeInferred  -- ^ inferred as safe
   | Sf_Ignore        -- ^ @-fno-safe-haskell@ state
   deriving (Eq)

Released as 3.0.1. Thanks for the bug report!


updated headers

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots

[x] Testing

[x] Changes have been manually tested

What are the relevant tickets?
Closes #792
What's this PR do?

Updates heading tag of title to H1
Updates heading tag of course-title to h2

How should this be manually tested?

Verify heading for course-banner and course-title is h1 and h2

Any background context you want to provide?
I have updates the headings for the course banner and title however course content headings such as these

are set by users and the markdown for it also is for a h1
Screenshots (if appropriate)

Oh, I thought Abdurrehman was going to work on this. In any case, this change should be part of his course-v2 theme, so we can look at it with everything else. Can you move it over there, maybe after he merges #784 ?

I have updated the headings for the course banner and title however course content headings such as these
are set by users and the markdown for it also is for a h1

Oh, that's going to be challenging. Is there any way to transform that in the theme, with hugo?

I have updated the headings for the course banner and title however course content headings such as these
are set by users and the markdown for it also is for a h1

Oh, that's going to be challenging. Is there any way to transform that in the theme, with hugo?

Yes, we can hugo render hooks to update the level for headings in a specific theme. Render hooks for heading were introduced in v0.71

Why is this blocked?

Why is this blocked?

This was blocked by compact design changes

Also @Wassaf-Shahzad this could use a rebase now that https://github.com/mitodl/ocw-hugo-themes/pull/807 is merged

@gumaerc I think we should remove this styling
  h2 {
    margin-bottom: 1.25rem !important;
  }

  h3 {
    font-size: $font-normal !important;
  }

from course-v2/assets/css/instructor-insights.scss and we'll be good to go! Right?
ps:  I think this PR will also fix another issue https://github.com/mitodl/ocw-hugo-themes/issues/632 so we can also close #632 once this PR is merged.

@MAbdurrehman12 @gumaerc I updated instructors-insights.css However I had to remove update it for both course v1 and v2 due to how we have configured webpack.

Can you give me some before and after URLs I can look at, to see this change? I'd like to have an example of a regular interior page, and an Instructor Insight page.

Also, in courses-v2 we should be following the color styles specified in https://projects.invisionapp.com/d/main?origin=v7#/console/21537330/468374061/preview?scrollOffset=60 -- no more red

https://ocw-hugo-themes-course-pr-794--ocw-next.netlify.app

@pdpinch Updated heading colors

https://ocw-hugo-themes-course-pr-794--ocw-next.netlify.app/

@pdpinch Updated heading colors

Sorry, I'm still seeing the red headings in the build at https://ocw-hugo-themes-course-pr-794--ocw-next.netlify.app/
Where can I see this in course-v2?

@pdpinch We'll need to merge https://github.com/mitodl/ocw-hugo-themes/pull/824 before we'll start seeing course-v2 builds with Github actions / Netlify.
@Wassaf-Shahzad I'm not sure why we need to also remove the changes from the course version of instructor-insights.scss? I added these lines back in and took a look at the site rendering with course:

Running again with course-v2 I get:

Can you elaborate on why you believe we need to remove the CSS from both?

Where can I see this in course-v2?

https://ocw-hugo-themes-course-v2-pr-794--ocw-next.netlify.app/

@pdpinch We'll need to merge #824 before we'll start seeing course-v2 builds with Github actions / Netlify.
@Wassaf-Shahzad I'm not sure why we need to also remove the CSS from the course version of instructor-insights.scss? I added these lines back in and took a look at the site rendering with course:
Running again with course-v2 I get:
Can you elaborate on why you believe we need to remove the CSS from both?

@gumaerc i have not idea why it didn’t  work for me before but its working know so i added back the code.

If I review this PR based solely on #792, then I think this meets the acceptance criteria. However, there seems to be a lot of other considerations. Could we get @gumaerc to give a final approval in the absence of @pdpinch?

Thanks @gumaerc. Let's talk with @pdpinch next week about the H1 issue. @Wassaf-Shahzad you should be good to go now.


Kernel to quickly compute comparisons of arrays

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
DataFusion (and likely other libraries) often wants to implement its own fast comparison functions
For example, comparing the contents of an array element by element to form an output BooleanArray
https://github.com/apache/arrow-datafusion/blob/7617d78809d4ff5bde31142e0744c70024e40635/datafusion/physical-expr/src/expressions/in_list.rs#L50
the arity::unary is close to what I want: https://docs.rs/arrow/17.0.0/arrow/compute/kernels/arity/index.html but it can not produce a BooleanArray (because BooleanArray is not a PrimitiveArray)
fn in_list_primitive<T: ArrowPrimitiveType>(
    array: &PrimitiveArray<T>,
    values: &[<T as ArrowPrimitiveType>::Native],
) -> Result<BooleanArray> {
    Ok(arity::unary(array, |x| values.contains(&x)))
}

Describe the solution you'd like
I would like a function such as unary_cmp that I could call like:

fn in_list_primitive<T: ArrowPrimitiveType>(
    array: &PrimitiveArray<T>,
    values: &[<T as ArrowPrimitiveType>::Native],
) -> Result<BooleanArray> {
    Ok(arity::unary_cmp(array, |x| values.contains(&x)))
}

There is a proposed implementation: https://github.com/apache/arrow-datafusion/pull/2826
Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.

Instead of copying unary, seems we can reuse exist compare code like #1991


Items in list, position when adding new (bootstrap tab-pane incompatible)

Hi, With this http://textextjs.com/manual/examples/tags-with-items.html example,
When I try to add a new tag the cursor goes to the beggining of the textarea, under the already wrote tags.
As i see, in the example there is a "dinamyc" padding-left. It does not work on mine for any reason.

maybe is just me? This is a screenshot for the issue.
http://i43.tinypic.com/nm18a8.png
When I start the textext with items, if I try to add a new Tag the prompt does not begins after the already inserted tags.

I found it to be a problem with Tab content. http://twitter.github.com/bootstrap/javascript.html#tabs
if i use the textext inside one of the tab-pane

any news on this? :( i'm totally stuck


Possible memory leak?

Version
1.0.0-beta.26
Steps to reproduce

Run a large test suite
I cannot provide a minimal reproduction because any minimal reproduction would not be helpful here. I am looking for a discussion. to see if others see this as well as to possible solutions.

What is expected?
Vue components are garbage collected
What is actually happening?
80% of VueComponent instances are retained in memory; even after calling wrapper.destroy after each instance. Heap size continues to grow and grow over time. Garbage collection occurs (either automatically or by force), but only a minimal amount is removed from memory.

Could this be related to #892? I'm experiencing both symptoms: as I re-run my tests with a mocha watcher they get slower (some event timeout) and the heap size keeps growing. Eventually, I get a JavaScript heap out of memory error and the process crashes.
I've also observed that manual GC don't help and a lot of VueComponent objects never get cleaned.

Thanks for the report.
The main issue I found is cached constructors added to component options objects, which results in lots of VueComponent instances that can't be garbage collected. This is likely why VueComponent objects weren't removed even after calling $destroy. I've opened a PR to remove the component that's mounted cached constructor.
I'll also add a cleanUp function that can run in a beforeEach to call $destroy on all mounted instances.
Another issue I found is that some plugins keep references to instances. I'll investigate this further.

Thanks a lot @eddyerburgh.
I've had the chance to quickly test the new version (1.0.0-beta.27) this afternoon. My numbers show an improvement of roughly 50% of the heap growth compared to the last version. This means that, on subsequent runs of the test suite, the heap only increases ~50KB instead of the ~100KB growth I was observing before.
At this point, it's not clear for me which module is responsible for the remaining leaks, but your change certainly helped!

Thanks for this progress! I have done some further testing and found some leaks within vue-router. There are event listeners that hold onto components forever. Trying to determine if there is a quick solution to this particular problem. Not sure if anyone else is also using vue-router.

Related: https://github.com/vuejs/vue-router/issues/2341

I have seemed to find this vm.$emit closure as a culprit for holding onto vue components forever:
function logEvents (vm, emitted, emittedByOrder) {
  var emit = vm.$emit;
  vm.$emit = function (name) {
    var args = [], len = arguments.length - 1;
    while ( len-- > 0 ) args[ len ] = arguments[ len + 1 ];

    (emitted[name] || (emitted[name] = [])).push(args);
    emittedByOrder.push({ name: name, args: args });
    var x = emit.call.apply(emit, [ vm, name ].concat( args ));
    console.count(x);
    return x;
  };
}


[FALSE POSITIVES] Malware Detection Function

Please use this issue to submit false positive reports from the malware scanner

Hi, the FS25_Courseplay beta, available here: https://github.com/Courseplay/Courseplay_FS25/releases/tag/<IP_ADDRESS> is currently detected as possible malware.


add missing pkg:glob dependency

Coverage remained the same at 83.846% when pulling bbf3d4b20038f1697c10d15b7e74d398f292be86 on oops into dde34a9f5207fa826c5364bc2171cc9d4f572674 on master.


Feature/816 migration scripts

Colton is out for a couple of weeks so I'm going to work on addressing some of the issues I found during my review.


Parsing AT+CMGR response, falling on empty ,, to option and trailing string

I have already started poking inside serde_at to fix some issue.
Here is the AT command response with escaped characters:
+CMGR: \"REC UNREAD\",\"+48123456789\",,\"23/11/21,13:31:39+04\"\r\nINFO

They are two issues when parsing it, first is the ,, with is a placeholder to put size of the read message when in PDU mode with I don't use, in text mode it is empty. So I put in the response struct: pub size: Option<usize>, but that was still falling, so I changed in serde_at deserialize_option to this:
        match self.parse_whitespace() {
            Some(b'+' | b',') | None => visitor.visit_none(),
            Some(c) => visitor.visit_some(self),
        }

Not sure if this is the best idea, but it seams to work and not break anything?
But the second issue is bigger with I am stuck on with no idea how to best fix. After I fixed the first issue when I run my code I got this error: CustomErrorWithMessage("invalid length 4, expected struct Message with 5 elements") with as far I know does come from next_element_seed() not realizing the \r\nINFO is the last part of the response and should be parsed to INFO.
The problem here is how to adjust that part of the code, because after parse_whitespace() it is already at I character because \r\n are omitted. I feel like serde_at  should have "put everything left into the last struct field" but it needs to be aware of that INFO is just an example and it can contain ,\r\n basically any character because it is just the content of the SMS message received. Will be gratefully for any help dealing with that, I hope I only need help with the right direction to fix it and come up with a PR (and hope the first fix is fine too).
Almost forgot, here is the Message struct:
#[derive(Clone, Debug, Deserialize)]
pub struct Message {
    state: String<256>,
    pub sender: String<256>,
    pub size: Option<usize>,
    pub date: String<256>,
    pub message: String<256>,
}

Chaning next_element_seed() to this:
 fn next_element_seed<T>(&mut self, seed: T) -> Result<Option<T::Value>>
    where
        T: de::DeserializeSeed<'de>,
    {
        if let Some(b'\r' | b'\n') = self.de.peek() {
            self.de.eat_char();
            if let Some(b'\r' | b'\n') = self.de.peek() {
                self.de.eat_char();
            }
            self.de
                .parse_whitespace()
                .ok_or(Error::EofWhileParsingValue)?;
        } else {
...

Does work for that case but that is super ugly I am pretty sure wrong, and changing INFO to INFO,ttt does break it right away, and that should work fine.

Huh, interesting!
Nice find on the empty parameters, don't think I have ever used that, so I probably haven't accounted for them when writing the parser. Your solution seems reasonable.
Regarding the other issue, the format looks super strange. I have never come across a command that mixes like that without it being some kind of newline separated list.

Ahh .. The contents of text messages..

No matter what, your message should probably be a Vec<u8> instead of String, as it is not quoted.

I am fine with using Vec or Bytes for it, that doesn't unfortunately fix the issue with parsing it, both still give the same error of invalid length

I am not sure what the correct way to handle this in serde deserializers would be.
What we want here is to just take the buffer remainder as the last parameter, but I am not sure if that is safe to do generally..

Yes, that would be best way to handle it.
Hmmm I am wondering if #[serde(skip)] on the last field would be a good idea and then allowing to be the last argument handled by AtatResp directly? Could be done before or after serde parsing, but I am not sure if serde has an option to output what was left after parsing so it might be better to do before? But that would require more parsing to happen inside AtatResp and only parse a slice of the read response to serde_at


config_flow error setting up/login eero account v1.2.1 - HAOS 9.5 - HA 2023.4.2

Hi,
I'm encountering an error trying to setup home-assistant-eero  integration.
If I enter my phone number... I don't get a text and nothing happens (included country code).
If I enter my email, I get an error.

home-assistant-eero=v1.2.1 (installed via HACS)
HAOS=9.5
HA=2023.4.2
eeroOS=v6.14.0-4275
Running on Proxmox on a NUC.
Here is what I see in the logs:
2023-05-10 14:30:26.763 ERROR (MainThread) [custom_components.eero.config_flow] Status: 401, Error Message: error.login.unknown
2023-05-10 14:30:26.763 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 235, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 146, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 180, in post
    return await super().post(request, flow_id)
  File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 72, in wrapper
    result = await method(view, request, data, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
    result = await self._flow_mgr.async_configure(flow_id, data)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 271, in async_configure
    result = await self._async_handle_step(
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 367, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/eero/config_flow.py", line 77, in async_step_user
    self.user_input[CONF_USER_TOKEN] = self.response["user_token"]
TypeError: 'NoneType' object is not subscriptable

I took a peek at the code and added a logger line to confirm it's receiving my input...

I personally use my US-based phone number to login so that's the only method I can directly confirm.
From your logs, it seems like it doesn't recognize your email address:
ERROR (MainThread) [custom_components.eero.config_flow] Status: 401, Error Message: error.login.unknown
Maybe someone else on here can confirm the email method works? That would help narrow it down a bit.

Thanks for quick reply. I can confirm my email is correct... And Amazon account since 2006 on same email. (and same phone number)
I did try phone number in format: "CAAANNNNNNN" (where C=country code, AAA=area code, NNNNNNN=number)
I see the prompt message Enter the verification code that was sent to however, I don't receive anything. (checked erro app & amazon app, too) No messages in the logs.
I can edit the code and add a logger message anywhere... is there a file/function/line you think will help debug the issue?

Hmm, what do you use to sign into the Eero app? Your email I assume? And to confirm, you're not using the sign-in via Amazon method, correct?

I had no clue what you meant by sign-in via Amazon method... so I logged out and tried to log in again...
I guess I did choose the Sign in with Amazon method when I setup eero :(
Now, the NOTES in the README makes sense.
I did get it working. Thanks. I'll add some steps below for the next person searching.
Create a new Eero account (not linked to Amazon) then, give it admin access to current account.

Sign out on eero app.
Click New to eero? then click Continue with email or phone. Then, sign out.
Sign in original account (Amazon sign in method)
Add admin to account.

Go to settings -> network users -> Invite an admin -> email yourself the share link

Sign out of original account and sign in new account. Open admin invite link and accept.
Sign in your original account.
Use the dummy account you just created to setup this eero integration.


[Docs] the example for the LQIP technique with a blur effect is not complete - alts and src attributes are missing

The code sample that's included on the blur-up plugin page is incomplete. src and alt attributes are missing. These things not only affect SEO but also make HTML code invalid. On the other hand If we add alt attribute then the loading effect is broken, because instead of blank placeholders we end up with blank placeholders with alt text :/
So we need both: alt and src attributes, but in turn, adding these breaks the smooth loading effect because the initial image is not blurred out.
So, IMO the soultion would be to include alt and src atteibutes and improve the CSS code - add blur effect for the initial image as well.  Thanks to this the low-quality image would be blurred-out before the JS code does its magic.

You can simply add an alt attribute. About the src: This plugin was created because people wanted  not to use the lqip markup and instead also wanted to lazyload the low quality source. As I understand your issue you don't want to use data-lowsrc but you want to use the lqip src for the fadeover effect, right?

You can simply add an alt attribute.

Not really, because If we add the src attribute then the initial image (the image that's loaded before JS code is processed) is not blurred out. Some CSS changes would be required in order to blur-out the initial image too. It would be great if the code sample contained such a CSS code.

As I understand your issue you don't want to use data-lowsrc but you want to use the lqip src for the fadeover effect, right?

Yes, and I also want to use alt attributes - because images without alts are not valid and also not SEO friendly. If I add alt attributes but omit the src attribute then alt text is shown for a fraction of a second before JS code is processed. That looks bad.
So my point is that the example is not complete and produces code that's not w3c compliant.

@wujekbogdan How did you end up implementing this?


Add Neon Devnet Deployment

Website
https://neon-labs.org/
ChainId
245022926

Please provide an RPC


xilem_web: Add event_target_value helper function

I took the opportunity to create a new util module in which all such helpers can end up.


Handle SAML response with status other than Success

The home IdP of a user can respond with a SAML Status element with a StatusCode other than "urn:oasis:names:tc:SAML:2.0:status:Success".
We need to handle this better.

Show status to user in readable form. See saml-core-2.0-os section <IP_ADDRESS> for a more readable version.
Some status code require action form engine:

AuthnFailed : Typically sent when a user chooses cancel on the IdP. We should forward status to SP.
NoPassive : For passive request: Forward status to SP.

Optional StatusMessage: Show in message to user

RequestDenied
AuthnFailed

This is handled in #48 (AuthnFailed) and #61 (RequestDenied).


Does uv build respect the lock file?

Is it possible to build packages with pinned dependencies from the lock file.
In poetry it's not supported
https://github.com/python-poetry/poetry/issues/2778

Are you referring to pinning the versions in the built wheel's metadata? Or using pinned dependencies while building (like, pinned build dependencies)?

I believe this is a duplicate of https://github.com/astral-sh/uv/issues/8729.


Mhang/use media comp static resource

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

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

\azp run net - Azure.Communication.MediaComposition - tests

\azp run net - Azure.Communication.MediaComposition - tests

/azp run net - Azure.Communication.MediaComposition - tests

/azp run net - Azure.Communication.MediaComposition - tests

close and will raise another PR.


Android lint stats

Android lint stats
https://github.com/marketplace/actions/android-lint-statistics

CodeFactor found an issue: Trailing spaces
It's currently on:
.github\workflows\android-lint-stats.yml:48
Commit 55b93738b6c66a88e4a779b308af3cc2d3f0dd68

CodeFactor found an issue: Trailing spaces
It's currently on:
.github\workflows\android-lint-stats.yml:27
Commit 55b93738b6c66a88e4a779b308af3cc2d3f0dd68


draft: transfer event checks

Add helper to assert that all of the expected transfers occur and only those transfers

Codecov Report
Patch and project coverage have no change.

Comparison is base (d807d28) 100.00% compared to head (77d8776) 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##               1.5     #1061    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files           46        46
  Lines         2652      2414   -238
  Branches       396       354    -42
==========================================
- Hits          2652      2414   -238

Flag
Coverage Δ

foundry
?

production
100.00% <ø> (ø)

reference
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 29 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


More accurately reflect necessary setup for D9.

The Problem/Issue/Bug:
When the user clones a lean repository but has not yet run the
composer install command, then using ddev config --auto will not
work because the files that the detection needs are not yet present.
How this PR Solves The Problem:
Updates the documentation that users were following and not having luck with.
Manual Testing Instructions:
Clone a lean repository for a Drupal 9 site (say, from a Pantheon site using Integrated Composer). Run ddev config --auto before doing a composer install. Note that the generated config.yaml will not be Drupal 9.
Automated Testing Overview:
Documentation update only.
Related Issue Link(s):
Issue not filed. Discussed in Drupal Slack: https://drupal.slack.com/archives/C5TQRQZRR/p1646971651968869
Release/Deployment notes:
n/a

Thanks so much for taking the time to improve this! I learn something every day.


Create Functional Builder API

A functional API to put together containers. Will also be used for image/container macro.

macros are gonna use current builders


Update resource file project settings after renaming/moving the files

Fixes #19132.

As @0ttoman mentioned in #19132, there are some other resources with the same problem, so I'll extend this PR to cover all of them before it's merged.

Can you fix the conflicts? :)

Looks like GitHub's online editor inserted spaces instead of tabs, but I think I managed to fix the conflict properly locally and push it now :)


Specifying Lerna Version in lerna.json

Issue Description
We need to look into how we specify our Lerna version in the lerna.json file.

Issue Type

[ ] New Feature
[ ] Enhancement
[ ] Bug
[x] Other

At this moment, lerna is on version 3.4.0. List of 3.0 breaking changes: https://github.com/lerna/lerna/releases/tag/v3.0.0

Duplicate of https://github.com/cerner/terra-core/issues/1792


"OP SAFT"

As far as I can tell, the term "OP SAFT" is used for the instrument. but is not a defined term.  The term "SAFT" is defined.  Is this intended?

Whoops, the term "OP SAFT" is quoted (and therefore defined?) in the title of the instrument.

That's right, I think. There aren't any disclaimers for headings not being operative / binding, and the pseudo-preamble provides for all "undefined capitalized terms" to be defined with reference to the annex, so theoretically including the defined term in the annex should be enough for a court to find that the parties intended for the term "OP SAFT" to refer to https://github.com/BranDAOn/OP-SAFT/blob/main-(non-U.S.-investor)/FORM-OF-OP-SAFT.md (or the document) generally.


How to make hover and click popup work both?

Hi,
I am pretty new to all of this... I try to have a hover on mouse over AND a click event showing different content.
for now the code looks like this, but I cannot figure out how to include the hover without messing up with the click popup.
Thanks in advance!
// CALL THE CARTODB SQL API HERE IN URL FORMAT
$.getJSON('http://soundgoods.cartodb.com/api/v2/sql?format=GeoJSON&q=SELECT * FROM soundgoods_mixtape_map_1', function(data) {
    var geojsonMarkerOptions = {
        radius: 8,
        fillColor: "#FFCC00",
        color: "#323232",
        weight: 2,
        opacity: 0.5,
        fillOpacity: 0.4
        };

    var geojson = L.geoJson(data, {

        pointToLayer: function (feature, latlng) {
            var popupOptions = {maxWidth: 500};
            var popupContent = '' +
                    feature.properties.soundcloud +
                    '
' + "Post & tracklist" + '
' + '';
            return L.circleMarker(latlng, geojsonMarkerOptions).bindPopup(popupContent,popupOptions,{

            });

        }

    });
    markers.addLayer(geojson);

    // CONSTRUCT THE MAP
    var map = L.map('map').setView([0,0],3);
    baseLayer.addTo(map);
    markers.addTo(map);
});

Hi @wolframlange this is an issue tracker for Leaflet development, not for asking for help on how to use it. For this, you should better use Stack Overflow or GIS Stack Exchange for example: it's more likely that your question will be answered there.
See also https://github.com/Leaflet/Leaflet/blob/master/CONTRIBUTING.md#reporting-bugs for contributing guide lines on this issue tracker.
Thank you and enjoy Leaflet :)


Draft: HeroIcon (TO DO: Dynamic importing)

Description:
-- Added HeroIcon component with a few icons in place, can either add more to the component or make imports dynamic if needed
-- NOTE: Can change the color of the icon using color prop, but the color must be defined in the Tailwind config file.

Closing this PR, going to refactor this component.


Worked on modernizing asynchronous python calls and resolving frontend errors

Summary

Switched over to async await instead of using
asynchronous generators.
Cached User Articles on Addition to allow frontend to reflect new
changes
Made nSubjects an optional parameter in the database
General Style updates (using static methods where appropriate) and removing unnecessary imports

Test Plan
Replicated errors received on frontend including
https://brainspell.herokuapp.com/json/v2/edit-global-article?github_token=b38a4968302c2948a9e5275c88de298c6107bb01&key=8a54a6d2574394f0a6d50b37d2a67f955196ae98&pmid=18367612&edit_contents={%22experiments%22:[{%22id%22:90000,%22caption%22:%22Areas%20showing%20greater%20activation%20for%20high-conflict%20trials%20than%20low-conflict%20trials,%20on%20trials%20where%20no%20response%20was%20required.%20The%20top%20half%20of%20the%20table%20describes%20activity%20as%20a%20result%20of%20the%20direct%20
and
https://brainspell.herokuapp.com/json/v2/edit-local-article?collection_name=naturalistic&github_token=b38a4968302c2948a9e5275c88de298c6107bb01&key=8a54a6d2574394f0a6d50b37d2a67f955196ae98&pmid=18367612&key_value_pairs={"90000":[]}&exclusion_reasons={"90000":"excluded"}

Verified these errors are now resolved.
Further tested the add-to-collections page to verify frontend clearly reflected newly added articles.
Also verified changes persist to DB and persist across refreshes of the page.

General Comments
Tornado recommends the use of native async over coroutines:
https://www.tornadoweb.org/en/stable/guide/coroutines.html

This looks good to me - good call on removing Python 3.4


How to pick darker color?

I understand that the color selection is based on the wheel.png file.
There are a lot of color variations for blue and red for example. It's almost impossible however to pick a dark green. Is there a reason for that?
Thanks,
Michael

Hi msevestre, thanks for opening the issue.
The goal of this component is to change colors in a HSL / HSV model. The color selection is not based on the image, but on the angle and position of the thumb. Changing the background image would not suffice, you would need to change how the color is retrieved.
This is easy, and I would welcome a PR that is backgrounds compatible, but it's implementation it's not planned.
Related 👉 https://github.com/netbeast/react-native-color-wheel/issues/13

I don't think this is related to #13 whatsoever. I am not asking for black color. Just asking for darker color for green.

The goal of this component is to change colors in a HSL / HSV model. [...] Changing the background image would not suffice, you would need to change how the color is retrieved.

Yes this is clear that some code will need to be implemented to make it work. But don't you think that this would make sense to have a green color range that is bigger (for a lack of a better word) that what is available now? The green is almost the same throughout the wheel area
 whereas blue and red go from light to dark.

This is easy, and I would welcome a PR that is backgrounds compatible,

I am not sure what you mean with background compatible. If you give me some pointers however on how this could be implemented, I could give it a try

Sorry I meant backwards compatible (editing soon).
I am afraid you already have available 360º of hue. The only way to get more green options, would be to have a higher definition, but I wonder how the user experience would be.
You can turn a color darker, reducing it's luminosity or brightness, properties of the HSV format. Normally other color pickers offer a second spinner or UI element to increase the range of the colors, but this wheel is supposed to be the wheel only.
I can't come up with an elegant and easy solution, knowing those restrictions. If you can come up with one, I'm open to it.
More info about color and encodings:
https://en.wikipedia.org/wiki/HSL_and_HSV


OwO

This fucky wucky puww wequest wiww send a secwet message when you use the nyew command that isn't made yet - pwease check it uwu

Stop opening shitpost PR's or i will make it so you cant submit anymore


Strict function / block prototypes

Use strict prototypes in function / block declarations.
(-Wstrict-prototypes)

I signed it!

Fixed indents.