PIDCache causes forever increasing memory usage

Expected Behavior
When many objects that clean themselves are in use with Cluster, memory shouldn't forever grow.
PIDCache should self-maintain and clean itself.
Actual Behavior
Memory usage forever grows. PIDCache grows forever. Each PID holds a Process reference, which causes ActorProcess and RemoteProcess instance count to grow.
Steps to Reproduce the Problem

Run the code in the following repo: https://github.com/DenizPiri/ProtoActorClusterPIDCacheMemoryLeak/blob/main/ProtoActorClusterMemoryLeak/Program.cs

Specifications

Version: 0.25.0
Platform: Windows

We see this issue in our production instances. Using PidCacheInvalidation doesn't really help, as it is designed only to solve the problem of remote PIDs, local PIDs still stay in PIDCache forever, together with their ActorProcess instances.

Currently, this pull request partially addresses the issue: https://github.com/asynkron/protoactor-dotnet/pull/1300
I believe, if the thoughts mentioned here would be implemented, it would be fully solved: https://github.com/asynkron/protoactor-dotnet/pull/1300#issuecomment-979163900

https://github.com/asynkron/protoactor-dotnet/pull/1300#issuecomment-979901080 should fully resolve this issue right now. Once that is merged, this can be closed.

#1300 is merged. Closing this one.
Once the new release is out, I will try it on production to verify that the RemoteProcess leak fix also works, if the issue persists, I will create a new issue or revive this one.


Defined new revisions for golang dep: github.com/ClickHouse/clickhouse-go

@AndrienkoAleksandr This says the license is MIT: https://github.com/ClickHouse/clickhouse-go/blob/6b1ba590788873fd6993f50f823bb90274a23ba5/LICENSE
If you agree, I will update the PR.

If you agree, I will update the PR.

Thanks, fixed.

I also fixed license info for all my another pr's...

Ran component through the scanners since I noticed it had not been harvested yet and it correctly declared the license. Closing PR as it is no longer needed.


Plugin secretstore incompatible with NetBox 3.5

django.core.exceptions.ImproperlyConfigured: Plugin netbox_secretstore requires NetBox maximum version 3.4.99.

Looks like it got deprecated in favor of https://github.com/Onemind-Services-LLC/netbox-secrets

i'm going to attempt the migration

Looks like we need to wait for https://github.com/Onemind-Services-LLC/netbox-secrets/pull/54 to land


Dev

Codecov Report

Merging #74 (936ab76) into master (52bce48) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #74   +/-   ##
=========================================
  Coverage     65.47%   65.47%
  Complexity       85       85
=========================================
  Files            11       11
  Lines           223      223
  Branches          8        8
=========================================
  Hits            146      146
  Misses           66       66
  Partials         11       11

Impacted Files
Coverage Δ

...in/java/io/github/portlek/reflection/RefField.java
0.00% <ø> (ø)

...n/java/io/github/portlek/reflection/RefMethod.java
0.00% <ø> (ø)

.../portlek/reflection/constructor/ConstructorOf.java
64.28% <ø> (ø)

...va/io/github/portlek/reflection/field/FieldOf.java
76.92% <ø> (ø)

.../io/github/portlek/reflection/method/MethodOf.java
73.68% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f17faa3...936ab76. Read the comment docs.


foxy bug

I tested the foxy-devel branch and found that rqt_publisher doesn't responding to a change in a message. I think the problem is in commit where _change_publisher_expression was deleted but https://github.com/ros-visualization/rqt_publisher/blob/foxy-devel/src/rqt_publisher/publisher.py#L158 doesn't change

Specifically it looks like this commit was the one that removed the change handler 367049ecc4ce3cabed9d5ca452fb478a98a3ba21
But this is definitely a bit of an issue, as it makes this plugin near useless.

I have now confirmed that reverting that specific commit fixes the issue. So the removed code was not in fact dead code... Ill open up a PR in a minute or so that fixes this against foxy-devel.

#28 fixed the issue, thanks @coalman321 !!
I've released the fixed version for foxy/galactic/rolling:

https://github.com/ros/rosdistro/pull/30066
https://github.com/ros/rosdistro/pull/30067
https://github.com/ros/rosdistro/pull/30068

so the issue will be fixed in next patch release.


Use prop-types instead of React.PropTypes

React.PropTypes is deprecated as of React 15.5.

Will it work with older React versions or this must be a breaking change?

See https://www.npmjs.com/package/prop-types#compatibility

I just pulled from npm version 3.12.3 and on the src folder there's still React.PropTypes instead of PropTypes...

Probably because of https://github.com/okonet/react-dropzone/blob/master/src/index.js#L378?
@jochenberger mind creating a new PR that removes it as well?

Not sure if that's the reason. shouldn't I have the exact same code I see here on the master branch on git when I pull that tgz file from npm? I've pulled it, and the src/index.js still has React.PropTypes

The relevant change is not in 3.12.3 AFAICT.

I think chore commit type doesn't trigger a release.


[8pt] Implement FR stream segment break to align with the MS boundary points

Enforce FR breaks at AHPS locations to ensure consistency with MS.

This is a follow-on (modification) to #97.

This is a follow-on (modification) to #97.


[RFC] [DOC] Add feedback mechanism

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Currently, the only way for users to leave feedback on documentation is to enter a Github issue. While this works fairly well, we have no quantitative way to measure if a user is happy or unhappy with the content.
Therefore, we should add the a feedback widget to each doc page, whether its a star rating (https://www.twilio.com/docs/autopilot/guides/how-to-build-a-chatbot), or a thumbs up, thumbs down mechanism. When a user selects the feedback mechanism, a modal would pop up allowing them to explain their rating.
To add this mechanism, it would require:

Changing the default template in Jekyll.
Creating a liquid include or some other form to add interaction to the rating.
Work with design to determine how the mechanism should look [Nice-to-have]

We can use Google Analytics to capture the data.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Here is a potential solution from a fellow colleague: https://github.com/facebook/react-native-website/pull/1449 and https://github.com/facebook/react-native-website/pull/1458/files

This is interesting. The first thing that comes to mind is something that AWS does on each of their documentation pages (example). They offer a simple 'yes/no' on 'was this helpful' - then a link to provide specific feedback - then a link to make edits via GitHub.

@krisfreedain This is very similar to what I'm implementing.

I'd like to know what they use? I looked into qualtrics recently and was told we cannot use their feedback mechanism. We can only use surveys.

@hdhalter - sent an intro email to you and an AWS docs person I know. Hopeful they can provide technical details, or know someone to assist in getting that information.

Implemented and backported to 2.0 - https://github.com/opensearch-project/documentation-website/pull/2264

Closing this issue because it has been implemented in #2264


DataFilterTest.NormalSpaceDataPointsFilter fails assert in debug mode

NormalSpaceDataPointsFilter<T>::inPlaceFilter has an assert to ensure that normals are normalized:
assert(normals.col(i).head(3).norm() == 1);

This one triggers an exception for me when running the DataFilterTest.NormalSpaceDataPointsFilter test:

utest: /home/cris/SW/SOUP/libpointmatcher/pointmatcher/DataPointsFilters/NormalSpace.cpp:91: void NormalSpaceDataPointsFilter::inPlaceFilter(NormalSpaceDataPointsFilter::DataPoints&) [with T = float; NormalSpaceDataPointsFilter::DataPoints = PointMatcher::DataPoints]: Assertion `normals.col(i).head(3).norm() == 1' failed.

I replaced the assert with an if() std::cout << ... statement, which results in a long list of normals being displayed. They are all in the range [1,sqrt(2)], which leads me to believe that these are 2D vectors that have been normalized such that the max element is 1 (L_infinity norm).
These all happen when nbSample is 240 (the 2*nbPts2D/3 case in the test), not for other values of nbSample.
All tests (including this one with the assert disabled) pass successfully.

@MathLabu could you have a look?

Hi!
In fact the NormalSpaceDataPointsFilter was not designed to handle 2D point-cloud. I mechanically added the 2D test case (this is where the assertion failed) in the utest, but that was a mistake.
About the assert, we need normalized-normals to compute the normal-space as a 2D space in polar coordinates Phi-Theta (Rho=1, so no need to take it into account).
I propose 2 solutions to fix this issue:

add support for 2D case in the filter (just need to build normal-space in 2D, i.e. a polar angle histogram)
remove the 2D test case

Do you need the 2D support? If so I can implement it quite quickly, otherwise just removing the 2D test case should be enough.

Thanks for your insight. Makes me feel better about having removed that test. :)
When nbSample=nbPts2D, is it not also a 2D test case? That one doesn't trigger the assertion.

You're welcome ! I'm going to do a PR to fix the issue adding a check of point cloud dimension in the filter.
nbSample=nbPts2D is a 2D test, but this one does not trigger the assertion because there is a check at the beginning of the filter to verify if the number of sample requested is superior or equal to the number of points: if so, nothing is done, otherwise we perform the algorithm.


Release OpenBLAS 0.2.20

@martin-frbg
We are going to release a new version. Any other works to be done?

[x] Test arm soft abi #1221
[x] Merge #1225 to fix #1117 (if we are OK with picking from wemsaar's tree while he is unavailable)

@xianyi Do you have an ETA for the 0.2.20 release?

@boegel  about Jul 15

I will correct PR1199

@xianyi as already noted in #1118 I would like to grab the fix for #1117 from wernsaar's fork as Werner appears to be unavailable. I also have a tentative fix for #1155 and I would appreciate your comment on #1137 (handling of unaligned accesses) but these can wait. Seeing that the 0.2.20 release date has been slipping since early march I'd really like to see it released rather sooner than later now, especially with the upcoming summer holidays in the northern hemisphere. (I know there is work underway on improving BLAS support for IBM z13 but it seems @quickwritereader is only about three quarters through at the moment, so this too should probably end up in 0.2.21)

Arm soft abi support is the most awaited one due to new ndk version.  I am glad to hear that it was solved.
As for IBM z13 we now have competitive  BLAS 3 for real and complex numbers. and I think it can be released as first version or as experimental.

It looks like we can release a new version this week.

Can you consider hard-to-debug crasher fix #1234 ?

@brada4 I just merged #1234

Thanks.

I have committed a fix to disable the new ReLAPACK option (did not even expect that big PR to get merged before 0.2.20) so I think everything should be ready now. The other PRs I am currently creating can go in right after the release I think (the L1 misdetection fix should improve performance on the latest
two generations of Intel chips so I hope 0.2.21 will not take as long to appear, but I have not done any benchmarks).

Is there an updated ETA? What is missing apart from the Changelog? Anything one can do to help?

@pbregener I am working on changelog and release a new version.

Great, thanks! 😄


Tenant detach error: tenant directory doesnt exist

Steps to reproduce
Not entirely sure about that. But in logs It can be seen that in both cases there is this sequence:

create timeline call that times out and continued in the background while control plane is disconnected
Tenant detach call arrives when timeline is still being created
Tenant detach fails because tenant directory doesnt exist (what is confusing is that I think at this point tenant directory should exist so the source of the error needs investigation)
Tenant detach succeds despite the fact that on the previous step the directory was missing
Timeline creation handler finishes

Failures in timeline create with a message that uininit mark already exist are related to this issue. If timeline creation is retried when previous call timed out but execution is still continued in the background then this error is the expected outcome.
Note that this appeared around restart. Not sure if this is important, but may be related. At least it may be the cause of timeline create request timing out
Expected result
No errors.
I think there are two problems. First there is a race between timeline creation and tenant detach. This is wrong.
Second, detach shouldnt fail when tenant directory is missing. This I expect to be fixed in https://github.com/neondatabase/neon/issues/4250. I have a WIP version of that, will link a PR once I open it.
Actual result
ERROR request{ path=/v1/tenant/X/detach request_id=X}: Error processing HTTP request: InternalServerError(Failed to run cleanup for tenant X
Caused by:
    0: local tenant directory "/storage/pageserver/data/tenants/X" removal
    1: No such file or directory (os error 2)

Environment
prod
Logs, links

Explore-logs-1.txt
Explore-logs-2.txt
Release thread where the error was first seen in logs https://neondb.slack.com/archives/C03H1K0PGKH/p1689691603398319?thread_ts=1689678688.598589&cid=C03H1K0PGKH
https://neonprod.grafana.net/goto/d62m0_jVg?orgId=1

Sounds like a problem we can solve to some level with more utils::completion :) (I think this pattern is repeating.)
Basically we should only be able to create new timelines if let Some(guard) = self.timeline_creation.lock().unwrap_or_else(|e| e.into_inner()).clone() {  given a Tenant::timeline_creation: Mutex<Option<completion::Completion>>.
On detach we should tenant.timeline_creation.lock().unwrap_or_else(|e| e.into_inner()).take(); and then await on the associated Tenant::timeline_creations_done: completion::Barrier to make sure everything is done.


Implement 3D and transposed 3D convolutions

Since the Conv3d PR broke the CI for an unknown reason (yet) we reverted it to allow other work to continue.
This PR tries to add the commit back (#1945)  to main without breaking the CI.

The push history has disappeared from the PR thread but here are the results when tweaking the tasks_max field of RuntimeOptions for wgpu with 7 test runs:

Max Tasks
✅
❌

1
7
0

32
3
4

I'll push a rebase with main and the modification of the default max tasks to 1, if the run passes I'll merge to main.


NEXTAUTH_SECRET doesn't work in production

I tried getting the stack working all the way through but kept getting an error where NextAuth would say I didn't have a secret in production even though I had NEXTAUTH_SECRET set in prod for my app.
It seems like I have it working by adding secret: process.env.NEXTAUTH_SECRET, in auth-options.ts.
It's a small change but I can't seem to create a branch or PR.

The deployed version of this project has working sign in so not sure what you're doing wrong:
https://create-t3-turbo.vercel.app
Did you follow the setup guide in the readme?

Yes I did follow all of the instructions besides the expo stuff so I'm not sure why it seems to need this.


Too low order of integration for VectorDomainLFIntegrator

Hello,
We've noticed that the order of integration for VectorDomainLFIntegrator::AssembleRHSElementVect in line 174 in liniteg.cpp is set as
int intorder = el.GetOrder() + 1;
while for DomainLFIntegrator::AssembleRHSElementVect in line 41 it is
ir = &IntRules.Get(el.GetGeomType(), oa * el.GetOrder() + ob);
with oa = 2, ob = 0 for default. Similiarly for VectorFEDomainLFIntegrator::AssembleRHSElementVect   in line 291
int intorder = 2*el.GetOrder();
We had to increase the order in VectorDomainLFIntegrator::AssembleRHSElementVect to obtain the right solution to our problem.
Is there any reason why the order is set lower for the vector valued case? Or why don't we have the chance to set something like oa and ob in the scalar care?
The same applies for BoundaryLFIntegrator::AssembleRHSElementVect and VectorBoundaryLFIntegrator::AssembleRHSElementVect.
Thanks
Tamas

Hi Tamas,
Admittedly these default values are somewhat ad hoc, and for historical reasons we may have ended up with different choices in closely related integrators. I am open to unifying these defaults, @v-dobrev may provide more details why these particular values were chosen.
That being said, you can always overwrite the quadrature rule with the LinearFormIntegrator:: SetIntRule method. This gives you full control of the order and type of integration used.
Tzanio

I'm closing this issue now. Feel free to re-open if you'd like to discuss it further (or better yet propose a PR :)).


🧐[问题] Please, how can I change the language used in this component?

https://protable.ant.design/intl


Phase 3: Managers Commitment Information

Description
Managers provide information and sign commitment forms that outline the expectations for their roles. (Duration: 1 Week)

Minimum of 2 hours per week commitment.
Minimum of 6 months commitment.
Agree to the Code of Conduct.
Agree to governing document.
Agree to make their name public.
Provide bio, management experience, and the reasons why they chose their theme.

Output

Signed documents from all managers
Bios, management experience, and reasoning from all managers

Supporting Documents

Phased Plan

Closing issues associated with deprecated project plan. Please see https://github.com/orgs/DevRel-Foundation/projects/7 for new project plan


Organization & owner

If I create an organization with my Work Account (O365 & Azure tennant) is this organization tied to my user?
By default I am owner of the organization, but what if the account of the owner get suspendend?
What is the impact of this on the existing organization created by that user?

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

ID: b1b9dd76-4131-bf69-f9bc-c7f7ef562609
Version Independent ID: 8d3bfbae-3398-c0a2-ca4d-c4782a3b4370
Content: Create, configure a new organization - Azure DevOps Services
Content Source: docs/organizations/accounts/create-organization.md
Product: devops
GitHub Login: @chcomley
Microsoft Alias: chcomley

@StijnBt, just following up with you on this - is there anything I can help you further with? Please let me know. Thanks!

As for the second question, "By default I am owner of the organization, but what if the account of the owner gets suspended? What is the impact of this on the existing organization created by that user?" I'm not sure I understand - are you referring to the impact on the members or the organization, or on resources like projects and builds in the organization? Thanks for clarifying...

Can the organization remain to exist if the original creator is no longer part of the O365/Azure tennant?
Sorry for the delay...

Hi @StijnBt. The answer is yes, an organization can remain if the original creator is removed from the Azure AD tennant. Make sure you don't remove EVERY member of the organization from the Azure AD tenant. I'll be closing out this issue, however please let me know if you'd like any further clarification. Thank you!


Make it clear that debloat scripts are not fully open source (OO Shutup)

Introduction
In Tweaks section there are Recommended Selections. All of them (Desktop, Laptop, minimal) are checking the box next to OO Shutup. Maybe I'm getting the wrong impression but I would expect from open source enthusiast to make debloating script open source. Don't get me wrong, I understand that some non-open-source scripts may be better than what you are able to create (but tbh. I am still searching for fully open source debloat script) but I wouldn't expect that you would recommend to me script to debloat that is not open source without warning.
Describe the solution you'd like
I would like, at least, when you hover over "Run OO Shutup" that it would display that OO Shutup is not open source.
Describe alternatives you've considered
There might be additional indication, for example little asterisk next to "Run OO Shutup" and explained below that this script is not open source, what this script does (If I am not wrong OO Shutup have many options?), maybe why you trust it. This indication can also be done by changing color of text/background etc.
Additional context
Section where you install software probably don't need that because people probably already know what they want and if it is open source or not (although little asterisk or other indication next to every non-open-source app might work) but debloating script is something that most people probably don't know how works and are resorting to recommended settings.

I don't know if you (@ChrisTitusTech) just didn't saw it yet and it was closed automatically or you saw it and decided to leave it to be closed automatically. Isn't 2 weeks too short (if there even should be time limit because I guess some issues might take very long to resolve)?


[freetype] Fix cmake usage

What does your PR fix?
Fixes #28167.
Promotes native CMake usage.

FTR multiconfig usage with CMake 3.24.2 was tested in https://github.com/microsoft/vcpkg/pull/27408#issuecomment-1288097793.


Add support for configurable time units

This change adds support for configuring time units to be one of: ns, us, ms, s.
Fixes #26

This PR depends on #39.


Update scalafmt-core to 3.2.0

Updates org.scalameta:scalafmt-core from 3.1.2 to 3.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, early-semver-minor, semver-spec-minor

Superseded by #78.


[Bug]: 0 amount LN invoices doens't always work on Android

Describe the bug
bitkit ios -> bitkit android, 0amount - works
bitkit android -> bitkit ios, 0mount - doesn't work
bitkit android -> bitkit ios, 1 satoshi - works
bitkit android -> WOS, 0 amount - doesn't work
bitkit android -> WOS, 5 sats - works
bitkit android -> phoenix, 0 amount - works
bitkit android -> clightning, 0 amount - doesn't work
Reproduce
see above
Screenshots / Recording
No response
Operating system
Android
Bitkit version
v106
Log output
bitkit_ldk_logs_1708524918614.zip

@Jasonvdb did you had time to have a look?

I think this might be resolved in LDK 121. Almost done with the android update and then will confirm.


Added scenarios and guidance for card design

Docs Build status updates of commit 01d42d7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0f9dfcb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 09fdfcd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5912630:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit cb161ec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 26ae02a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Amount doesn't parse u64::MAX

Test case:
assert_eq!(
    ua_str(&ua_sat(u64::MAX).to_string_in(D::Satoshi), D::Satoshi),
    Ok(Amount::MAX)
);

Fails with too big. This if looks wrong but if I comment it out other tests fail and it's hard to understand why, so if anyone can help, that'd be great.
https://github.com/rust-bitcoin/rust-bitcoin/blob/eadc6a6c9eb8c397f01f41db99b5c78bb22a437d/units/src/amount.rs#L612-L614

Amount is i64 in Bitcoin core.
The current code is correct.
I'm surprised the other methods don't check the u64 bounds.

That's absolutely not relevant here since this is Rust, not C++ and thus this kind of thing is very surprising for Rust developers. (We've already agreed to write Rust-idiomatic code over Core-like.) We've been discussing whether to restrict the range to 21M BTC and the consensus is no. (IIRC I was the only one who would like it but I'm not that convinced lately.)
Also given we have MAX equal to u64::MAX the code as a whole is absolutely not correct. If we wanted to restrict it artificially we have to enforce the invariant everywhere and also have the correct constants. Which at this point I don't think we want to do. And if there's a limit, I absolutely prefer 21M BTC over some arbitrary number.


Dart DevTools exited with code 255 Flutter VS Code

When I try to open Dart DevTools in VS CODE. Shows error that "Dart DevTools exited with code 255".
Here is my flutter doctor -v output
PS D:\FlutterProjects\expenses_app> flutter doctor -v
[√] Flutter (Channel stable, v1.12.13+hotfix.9, on Microsoft Windows [Version 10.0.18362.720], locale en-US)
    • Flutter version 1.12.13+hotfix.9 at D:\Flutter\flutter
    • Framework revision f139b11009 (5 days ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at D:\AndroidSDKComponentSetup
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = D:\AndroidSDKComponentSetup
    • Java binary at: D:\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[!] Android Studio (version 3.6)
    • Android Studio at D:\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code (version 1.43.2)
    • VS Code at C:\Users\SagarKhurana\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.1

[√] Connected device (3 available)
    • SM J200G                  • 320136c19c48b45b • android-arm   • Android 5.1.1 (API 22)
    • Redmi 8                   • ad4810d10506     • android-arm64 • Android 9 (API 28)
    • Android SDK built for x86 • emulator-5554    • android-x86   • Android 10 (API 29) (emulator)

! Doctor found issues in 1 category.

Thanks

Hi @sagarkhurana00786
Can you see the same error when running DevTools from command line?
flutter pub global activate devtools
flutter pub global run devtools

@VladyslavBondarenko thanks for showing your concern
the output of flutter pub global activate devtools
PS D:\FlutterProjects\expenses_app> flutter pub global activate devtools
Package devtools is currently active at version 0.2.3.

Precompiled devtools:bin\devtools.
Installed executable devtools.
Warning: Pub installs executables into D:\Flutter\flutter.pub-cache\bin, which is not on your path.
You can fix that by adding that directory to your system's "Path" environment variable.
A web search for "configure windows path" will show you how.
Activated devtools 0.2.3.
PS D:\FlutterProjects\expenses_app>

and output of flutter pub global run devtools
PS D:\FlutterProjects\expenses_app> flutter pub global run devtools
Serving DevTools at http://<IP_ADDRESS>:9100
In flutter pub global run devtools it wont show anything beyond Serving DevTools at http://<IP_ADDRESS>:9100
Please help
Thanks

Hi @sagarkhurana00786
now that you successfully activated devtools
did you try launching it from vscode as described in the docs?
Thank you

@iapicca on following docs still getting same error code 225
I still don't know where are the problem and whats the problem
Thanks

@sagarkhurana00786
if you run your app from terminal, you can see such log
Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h Repeat this help message.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).
An Observatory debugger and profiler on Android SDK built for x86 is available
at: http://<IP_ADDRESS>:60230/J7_wS_YhTuo=/

Run in another terminal flutter pub global run devtools and then open http://<IP_ADDRESS>:9100
enter in the Connect to a running app field address of your running app http://<IP_ADDRESS>:60230/J7_wS_YhTuo=/
can you see exited with code 255 error now?
or only when opening DevTools from VS Code?

@VladyslavBondarenko
Now in this process, I'm not facing any issue after putting my flutter app URL then immediately I'm connected to Dart Dev tools.
But Why it is showing error When I try to open it from VS Code
Thanks, Everyone

@sagarkhurana00786
It may be not the issue with Flutter,
please create an issue in the Dart extension repository and link there current one.
Closing this for now, comment if it turned out that it's not only problem with VS Code extension and I'll reopen

@VladyslavBondarenko Okay Then I create an issue on that
Thanks 👍


API structure

jaltol
|__ django files (views, admin, apps, urls)
|__ all the API endpoint codes
gee_configs
|__ any configuration related to GEE
geoprocessing
|__ ee_core
|__ lulc
|__ imd
don't include any sensitive information or credentials in the repo.

Archiving this


Benchmark update

I have updated the branch with the fixes you requested. I cannot test them since I do not use Visual Studio. Hope it works!


Istio 1.5.6, istiod not creating istio-ca-root-cert configmap for namespaces where istio injection is enabled

HI Guys,
with reference to the below comment, the config map creation issue where istio injection is enabled on the namespace should have fixed in istio 1.5.6 , but that's not the case as this has hit us badly again in production today.
Based on the confirmation. we have migrated all our environments from Istio 1.5.4 to Istio 1.5.6 just to fix this burning issue, but the issue still exists
Whenever a tenant onboard we create a namespace for the tenant and all the deployments of that tenant happens in his own namespace and whenever istiod stops creating istio-ca-certs for new namespaces, the deployments for all these new tenants will fail
the stop gap solution is to restart the istiod pod and it will resolve the issue, but it's a bad hack and it's not a good customer experience
Cloud Provider: GCP
Managed Kubernetes: Yes, we are using GKE
Kubernetes Cluster Version:  1.16.10-gke.8
No of Nodes: 120
Istio Version: 1.5.6
No of Namespaces: 350 currently & growing
Can you guys please suggest, how can this problem be fixed asap as it's happening on our production environments

"@rajivml @Mdirfan6 Actually this is fixed in 1.5.6, please see the release announcement
https://istio.io/latest/news/releases/1.5.x/announcing-1.5.6/.
Originally posted by @knight42 in https://github.com/istio/istio/issues/22463#issuecomment-648777698
"

I have verified the istio-leader configmap, it's pointing to the right istiod pod which is of istio 1.5.6 version itself

Please let us know if any additional logs are needed, after we restarted istiod problem went away and in couple of hours again the problem started, will changing the istiod pod replicas to 1 will temporarily solve the issue if it's an leader election issue among istiod pods ?

any update on this ? this issue is happening consistently on our prod environment at-least once a day , any suggestion on how to fix this for good would be helpful

cc @howardjohn @tariq1890 @knight42

Can you post the istiod logs? does this issue happen always/permanently or only after some period of time? I mean if you restart istiod, then create a new namespace, will it update the configmap?

it's happening many times a day and when we are restarting istiod, it's creating missing configmaps in the namespaces where istio injection is enabled and we have temporarily scheduled a cron like this to restart istiod when ever configmap creation is not happening .
I will share the logs shortly , I have just now asked our IT admin to create an sink (GCP bucket) to upload the istiod logs as gcp console is giving an option to download only 300 lines of logs
#!/bin/bash

echo "Validating istio setup"
temp_namespace="validate-istio"
#create temp namespace
kubectl create namespace $temp_namespace
echo "$temp_namespace namespace created. Labelling namespace ... "
sleep 10
kubectl label namespace $temp_namespace istio-injection=enabled
sleep 30
#validating istiod
counter=1
while [ $counter -le 3 ];
do
  # check configmap
  response=$(kubectl -n ${temp_namespace} get configmap | grep istio-ca-root-cert | awk '{print $1}')
  echo "$response"
  if [ \( ! -z "${response}" \) -a \( "${response}" = "istio-ca-root-cert" \) ]; then
    echo "Istio Setup successfully completed and working fine."
    kubectl delete namespace $temp_namespace
    echo "$temp_namespace namespace deleted"
    exit 0
  else
    kubectl -n istio-system rollout restart deployment istiod
    echo "${counter} iteration restarting istio ..."
    sleep 30
  fi
  counter=$((counter+1))
done

echo "Istio setup validation failed"
kubectl delete namespace $temp_namespace
echo "$temp_namespace namespace deleted"

@howardjohn plz find the logs downloadable at below link, attached logs worth of couple of hours
https://uipath-my.sharepoint.com/:u:/p/rajiv_chodisetti/EQUC5shCdeZPvg2Kt9PpKw8BPJd5N9p1zv49VkeZaFt0sA?e=3smp8J
https://uipath-my.sharepoint.com/:u:/p/rajiv_chodisetti/ERa8gKX6yFBDtdCw6bc1XuIB_IS8tlYg893h0zdqO6sYpQ?e=nwoD9l

Hi, is anyone looking into it

@myidpt @howardjohn since you worked on https://github.com/istio/istio/issues/22463, could you help to take a look at this issue? It is not clear to me what is the root cause, and whether the root cause to this issue is the same as the one fixed at #22463.

This issue is still consistently happening on all our environments i.e alpha, staging and prod where istio 1.5.6 is being used and we put a cronjob to restart istiod pod whenever we run into such issue which is not an elegant solution. So, appreciate any support here, if more logs are required please let me know

@howardjohn Hi John, do we observe similar issues on Istio 1.6 or later versions?

Not that I know of, probably fixed by #22598. We can re-open if we see it in 1.7+

@howardjohn Nope, this fix was live in 1.5.6 and use 1.5.6 in production and this issue happens on our production on a day to day basis and we have a cron setup to restart istiod when this happens https://github.com/istio/istio/pull/22598

@howardjohn Nope, this fix was live in 1.5.6 and use 1.5.6 in production and this issue happens on our production on a day to day basis and we have a cron setup to restart istiod when this happens https://github.com/istio/istio/pull/22598


setup.env URL limit

the URL written by WIRETRUSTEE_AUTH0_AUDIENCE did not add / at the end.
The management side will not start.
I suggest you strengthen the documentation or improve the automatic judgment of the program.

Unfortunately, we can't enforce that, as the Auth0 audience value can be any string.

updated doc on #291


⚠ Bofrost.lu has degraded performance

In 21dd60b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1926 ms

Resolved: Bofrost.lu performance has improved in 2ff44c4 after 8 minutes.


Support vectorizer multi2vec with auto schema

As of now (with #1756 merged) the new multi2vec can't be used with implicitly created classes, even when the default vectorizer is multi2vec. This is because it would leave both imageFields and textFields empty and it would have nothing to vectorize.
Instead, we could let the module parse the schema and decide that any text/string field will be added to textFields and any blob field will be added to imageFields. For more detailed configurations, the user has to explicitly create a class.

@antas-marcin did we already add this or do we still need to?

Keep.


configure dockerhub registry for windows toolbox boot2docker?

I want to connect vscode to manage an arbitrary docker registry with exposed API. I'm running vscode on a windows 10 host with docker windows toolbox (i.e. through docker-machine and boot2docker).  the docker registry is running virtual machine.
c:\host>docker-machine env default
SET DOCKER_TLS_VERIFY=1
SET DOCKER_HOST=tcp://<IP_ADDRESS>:2376
SET DOCKER_CERT_PATH=C:\Users\laurence\.docker\machine\machines\default
SET DOCKER_MACHINE_NAME=default
SET COMPOSE_CONVERT_WINDOWS_PATHS=true
REM Run this command to configure your shell:
REM     @FOR /f "tokens=*" %i IN ('docker-machine env default') DO @%i

I've exposed the docker API on the vm using this method and I have updated the Windows DOCKER_HOST env var accordingly. https://stackoverflow.com/questions/27949254/docker-remote-api-not-accessible/49939177#49939177.

edit /var/lib/boot2docker/profile.

DOCKER_TLS=no
DOCKER_HOST=<IP_ADDRESS>:2375

then these return valid data...

curl http://localhost:2375/containers/json
curl http://localhost:2375/images/json

vscode docker explorer sidebar observations:

main docker tab is empty.
docker Images tab is empty.
only docker Containers contains valid data.
running Compose Down/Compose up on a compose file is not working (no errors, no actions)
running Build Image on a docker file is not working (no errors, no actions).

thanks

i have partial functionality available by setting DOCKER_HOST = tcp:/url:port on the windows host.


Impossible character escapes in strings

I have a simple macro:
dstr: MACRO
    REPT _NARG
        db \1
        shift
    ENDR
    db 0
ENDM

And I pass it this string:
    dstr "Line 1\nLine 2"

And I get this error:
ERROR: engine.asm(7) -> engine/title.asm(1357) -> dstr(3) -> dstr(1):
    Unterminated string

I then try to pass this string instead:
    dstr "Line1\\\nLine2"

And I get this error:
ERROR: engine.asm(7) -> engine/title.asm(1357) -> dstr(3) -> dstr(1):
    Illegal character escape '
'

I would expect either of these variants to work, since they're right there in the manual!

Well, I figured out that the correct answer was \\n. Which both isn't intuitive and not really pointed to in the manual. Plus the Unterminated string error doesn't give much insight as to what to do - I only knew what to do because I had encountered it using commas.

Well, it makes sense. The \\ is converted to \ after the first pass. If you do the same to the n, \n will be converted to a newline.
Where is the problem in the manual?
I agree that the error messages aren't always helpful, but it's quite hard to get the parser to say useful things in some cases...

Well, the manual states that \, is an escape sequence, so when I got an Unterminated string error after using a comma, I thought of escaping it. But I expected \n to work as well, which it didn't.
In retrospect, \\n does make sense, but it's not obvious to have the idea -- it took me a day to have that idea.

That's also my guess, although the parser actually attempts to escape the line feed, instead of calling it an unterminated string. But that's just nitpicking.

However, even though this is not a bug, maybe the documentation can be updated. The documentation has an example, actually:
Line continuations work as usual inside macros or lists of arguments of macros.
Strings, however, are a bit trickier. The following example shows how to use
strings as arguments for a macro:

    PrintMacro : MACRO
        PRINTT \1
    ENDM

        PrintMacro STRCAT(\"Hello\"\,  \
                          \" world\\n\")

Maybe it's just in the wrong place.

I think it is. It also feels weird to have to escape everything, because there are no quotes wrapping the argument. Intuitively, it feels like you shouldn't have to escape those...

Well, that's a macro argument, and macro arguments are a tricky thing...
The problem is that the parser isn't great and there are many hacks everywhere. This is a problem that isn't going to go away.
The only thing I can do is to change the docs, but I don't know where to put that, so I'm open for suggestions.

RE: They are tricky things: https://github.com/rednex/rgbds/issues/63

I'm going to close this. If you have a suggestion on how to change the docs feel free to propose it.


Incompatible with latest version of chardet

Since commit a98992c5818129949b11912cbf11b5b4972430c0 polyfile uses a narrower version specifier for the version of chardet expected. In the meantime chardet 5.1.0 has been released (and packaged on Arch Linux), which leads to a DistributionNotFound error since pkg_resources is explicitly imported at runtime:
Traceback (most recent call last):
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 581, in _build_master
    ws.require(__requires__)
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 909, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 800, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (chardet 5.1.0 (/usr/lib/python3.10/site-packages), Requirement.parse('chardet~=5.0.0'), {'polyfile'})

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/polyfile", line 33, in <module>
    sys.exit(load_entry_point('polyfile==0.5.0', 'console_scripts', 'polyfile')())
  File "/usr/bin/polyfile", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 992, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/python3.10/site-packages/polyfile/__init__.py", line 1, in <module>
    from . import nes, pdf, jpeg, zipmatcher, nitf, kaitaimatcher, languagematcher, polyfile
  File "/usr/lib/python3.10/site-packages/polyfile/nes.py", line 6, in <module>
    from .polyfile import register_parser, InvalidMatch, Submatch
  File "/usr/lib/python3.10/site-packages/polyfile/polyfile.py", line 8, in <module>
    import pkg_resources
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3260, in <module>
    def _initialize_master_working_set():
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3234, in _call_aside
    f(*args, **kwargs)
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 3272, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 583, in _build_master
    return cls._build_from_requirements(__requires__)
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 596, in _build_from_requirements
    dists = ws.resolve(reqs, Environment())
  File "/home/prettybits/.local/lib/python3.10/site-packages/pkg_resources/__init__.py", line 795, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'chardet~=5.0.0' distribution was not found and is required by polyfile

This was observed on Arch Linux with the AUR package for polyfile at version 0.5.
Can the version specifier be safely loosened to chardet>=5.0.0 again?

On a related note, the current install_requires dependencies are (as of the time of posting this comment):
https://github.com/trailofbits/polyfile/blob/438628fea2d32ee97b9f23a7aef7ffa3fdc80a0a/setup.py#L26-L39
With the disclaimer that I fully understand (1) why versioning dependencies can be important (2) "virutalenvs" exist for a reason (3) I might be wishing for something that isn't possible and (4) there are a multitude of different viewpoints on what is "right" in terms of dependency versioning, etc.
If at all possible, it would be nice if there was a way to remove restrictions on specific versions for dependencies. I am fully aware that it's easier said than done. Certainly a > or >= is much easier to integrate than a == or ~=.
This particular issue is probably more impactful on rolling distributions like Kali or Arch as compared to Ubuntu or Debian (at least, Debian other than sid)...
That said, if there's any way to remove "hard" dependencies - that'd be great. Alternatively, using a submodule and/or integrating specific functionality from a specific version of a different module/package might be an option too. Though I recognize and detest the idea of not maximizing code-reuse (i.e. having, essentially, multiple versions of a package installed because the code from one package is incorporated as a sub-part of another.)


Crystal::Macros::Call Missing location info

https://play.crystal-lang.org/#/r/7ki6
struct Foo
  macro method_missing(call)
    {{raise "Unknown method: '#{call.name}' at #{call.filename}:#{call.line_number}:#{call.column_number}"}}
  end
end

Foo.new.some_method # => Error: Unknown method: 'some_method' at nil:nil:nil

Pulled from https://forum.crystal-lang.org/t/missing-location-info/1125

I don't think there's a way to implement this. The method_missing macro is executed once per different call (name, argument number and size) and cached for later reuse. So the location would be the one for the first call.

@asterite Meaning like if you had
# a_file.cr
Foo.new.some_method

# some_other_file.cr
Foo.new.some_method

That the location info would only be correct for the one in a_file?  Wouldn't that not be a big issue since it would fail on the first time it sees an undefined method, which the location data would be related to that?  Or does the location data get calculated on a diff pass or something?

It's not per file. It's global. Even in the same file if you do:
Foo.new.some_method
Foo.new.some_method

then the macro will be executed just once.

Wouldn't that not be a big issue since it would fail on the first time it sees an undefined method, which the location data would be related to that?

It might makes sense for reporting an error message. It doesn't make sense in general if people want to use the location for something else. Like, I think providing a filename when it's not actually the correct filename is not good. So for now it's nil meaning "we can't provide that".

Also, please try to avoid method_missing. I'd really like to remove it from the language.

It might makes sense for reporting an error message. It doesn't make sense in general if people want to use the location for something else

True true, prob better to keep things consistent then.

Also, please try to avoid method_missing. I'd really like to remove it from the language.

Hmm okay, mainly was just using it to customize the error message, just a slightly better UX than the default for my use case.
But, can probably go ahead and close this if there isn't anything to be done?


Add Russian recourses

Added resource files for Russian language:

etc.

You just gave me a great idea! Time to translate.

Hey, what does NavigateLink means? It's confusing me.

@charlesmilette

I knew that it means that, but where does the string appear?
If it appears in a tooltip, I'd go for "Naviguer vers le lien"
Somewhere else, I'll use "Lien de navigation"
Etc...
That's why I'd like to know where does it appears

@charlesmilette
Is used only here:
namespace FirstFloor.ModernUI.Windows.Navigation
{
    /// <summary>
    /// The routed link commands.
    /// </summary>
    public static class LinkCommands
    {
        private static RoutedUICommand navigateLink = new RoutedUICommand(Resources.NavigateLink, "NavigateLink", typeof(LinkCommands));

        /// <summary>
        /// Gets the navigate link routed command.
        /// </summary>
        public static RoutedUICommand NavigateLink
        {
            get { return navigateLink; }
        }
    }
}

No example for tooltip etc in sample app. I translated it verbatim.

Thanks!

@Dvornik Why every Russian translation begins with lower-case letter? And why the "ok" is written in English but not in Russian "ок"? This seems pretty strange. Thanks in advance.

@kungfux Good question.
I left as it was in the English version, for consistency. MUI automatically changed capitalization for tabs'  and sub tabs' headings, so  lower-case looks appropriate.
But I agree, that is look unusually for Cyrillic. I re-read guidline https://msdn.microsoft.com/windows/uwp/style/typography - ALL CAP recommended only for certain pages, no information about lower-case, so I'll fix it.
I'm thinking about adding property for turn on/ turn off automatic capitalization changing.

@Dvornik Thanks for reply.
As I can see in this repository and your fork as well, the 1st letter is upper-case for English translation. But it's okay if you are going to change the Russian translation in correct way.
By the way, do you know, does appropriate translation is chosen automatically at runtime or startup based on system codepage? As I understand the answer is no. Is it so?

As I can see in this repository and your fork as well, the 1st letter is upper-case for English translation.

@kungfux It's not:

MUI convert case at runtime for styling purpose. I can't fix it with just in resource file. In my res file, the first key is upper-case too.
@charlesmilette can't change it too only with res file.
I think the best solution is adding property for turn on/ turn off automatic capitalization changing.

translation is chosen automatically at runtime or startup based on system codepage

Exactly: it' standard WPF behavior, not just MUI. Very useful.

@Dvornik I meant English translations in the code

Exactly: it' standard WPF behavior, not just MUI. Very useful. No additional code needed.

Great, thanks!

I meant English translations in the code

@kungfux Is just resource designer's auto-generated file. The right file with translation: https://github.com/firstfloorsoftware/mui/blob/master/1.0/FirstFloor.ModernUI/Shared/Resources.resx

@Dvornik Got it. You're right. Thanks for your time.


Got TypeError when pre-trainning model

Hi,
when pre-trainning model, I got TypeError as the following:

Traceback (most recent call last):
File "exp.py", line 615, in
pretrain(args)
File "exp.py", line 530, in pretrain
verbose=True, eval_top_pred_only=args.eval_top_pred_only)
File "/home/vladimir/cg-rl/CG-RL-main/evaluation.py", line 65, in evaluate
decode_results = decode(examples, parser, args, verbose=False)
File "/home/vladimir/cg-rl/CG-RL-main/evaluation.py", line 26, in decode
hyps = model.parse(example.src_sent, context=None, beam_size=args.beam_size)
File "/home/vladimir/cg-rl/CG-RL-main/model/parser_pre.py", line 777, in parse
token = primitive_vocab.id2word[token_id.item()]
TypeError: 'method' object is not subscriptable

After checking the class VocabEntry I found the member variable id2word defined in line 18 has the same name with the member method defined in line 35. I think the member method is redundant and should be removed. Is it right? @lighter47

No, this class is from Tranx and no problem. I can't tell the reason, maybe you can send me more information via email.

Uhh, I don't know the reason, too. I met this problem when using the dataset built by myself. The environment is the same as your Python3 environment. However, I think the member method id2word in line 35 of the file components/vocab.py is redundant, and removing it could solve this problem.
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年4月12日(星期二) 晚上7:58
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [DeepLearnXMU/CG-RL] Got TypeError when pre-trainning model (Issue #4)
No, this class is from Tranx and no problem. I can't tell the reason, maybe you can send me more information via email.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Ok I found the reason. I process my dataset using the latest  version of tranx, but your code is the previous version of tranx. There are some differences between the two versions in the class VocabEntry.

And I found the package dataset should be in your code. Could your please add the package to your code?

have you solved this problem? @Vladimir-Chan

have you solved this problem? @Vladimir-Chan

If I remember correctly, using Pytorch 1.3.0 may be helpful.


⚠ Quirk Club Argentina (Payment Gateway) has degraded performance

In 0e78f63, Quirk Club Argentina (Payment Gateway) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/payment-gateway-ml) experienced degraded performance:

HTTP code: 200
Response time: 4663 ms

Resolved: Quirk Club Argentina (Payment Gateway) performance has improved in 18325ea after 11 minutes.


There is no user setup

"During Beaker's setup, you filled out a name, picture, and bio for your profile. You can visit your profile drive by clicking your picture at the top right of the browser."
I installed the browser and there was no setup. How can I create a profile like the docs show? There is no explanation for this. It's not mentioned in the getting started, the FAQs, etc. It's like everything assumes a profile is created upon installation.

Those docs were for a feature in the 1.0 beta that we ultimately removed. I'll be removing that from the docs. Sorry for the confusion!

if the user profile is no longer a part of beaker, how can i find the directory page where on a previous install i was able to find other people's sites & or how can i find existing hyper:// pages to visit and learn from?


Create a generic number ↔ name and let EVP use it

This creates a generic, per library context number ↔ name map, and has the EVP fetcher use it.
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Arguably, this namemap could be used by the property code...  I don't quite understand why crypto/property/property_string.c currently holds two databases, when they are really just string ↔ identity maps and those strings don't have any other inherent per se.

@levitte the reason that crypto/property/property_string.c holds two (name, value) databases is to distinguish between the names and the values.   There are strict rules about what forms names can take and these are enforced.  There are far laxer restrictions on values.  Keeping them separate seemed expedient from the point of view of validity checking.
I'm not going to object to a merging of the two, so long as the semantics don't also change.

There is a slight difference between the properties name ↔ value maps and this.  The property maps are known to be indexed sequentially (from zero) which makes a stack of names efficient, here the index doesn't have that property which makes a sparse array the better choice.
By switching the property implementation over, there will be a (small?) space cost but I don't think there will be a noticeable time cost.  I believe the simplification to a single name ↔ value map across the library is a beneficial one that outweighs associated the costs.

Merging the two name ↔ value maps in the property code into a single map will allow properties like: "fips=foo, foo=bar" (where foo is not a property name) to be specified and parsed without error.  I.e. the reservation of names without a period can be circumvented.

There is a slight difference between the properties name ↔ value maps and this. The property maps are known to be indexed sequentially (from zero) which makes a stack of names efficient, here the index doesn't have that property which makes a sparse array the better choice.

Uhmmmm, this namemap is indexed monotonically as well, but from one (zero indicates an "undefined" string).  A string gets added by pushing it in the stack...

There's one bit that I find highly disturbing, and it's that this namemap is per library context...  That would mean duplicating it unnecessarily if we're handling several library contexts within a unit.  It's just a string to number cache, essentially, so there's no reason to make the numbers different per context.
I could force it to end up in the default library context, but that would mean that there must be one, and the FIPS module currently forbids that...  I might have an idea or two on that, btw, but that's the matter of another PR.

I saw the use of sk_NAMEMAP_value and thought that it searched for some unknown reason.
Best to stick with the stack and it can directly replace the two property mappings.

Could we have this per context and default to the "default" context if none is directly set?

Properties index from 1 as well.

Could we have this per context and default to the "default" context if none is directly set?

We can.  Also, this implementation only allows one namemap (per library context), and what you said about keeping property names and values separate makes me think that we need a mechanism to have more than one.  This is perfectly viable, but needs an extension of the API.

I re-implemented it to make it possible to create free floating name maps.

Ooooh, this found me an interesting bug...  we weren't doing things quite right when there is no pre-existing nid for a name...

This isn't in draft any more

Ping?
(btw, in a conversation yesterday, it was suggested that the strings could be bitstrings, i.e. binary data rather than just text...  I don't mind the idea, but I also think that could be future development of this map functionality, it doesn't have to happen in this PR)

Doing the close+open thing to kick Appveyor

Merged.
0211740fcc47a954be19cceb65fb57a6f7deb797 EVP_FETCH: remove the need to transport the legacy NID through construction
1f79ddf5049ff53ad8a7cbab76e62d02d9ac099f util/find-doc-nits: Fine tune detection of POD markup in NAME section
2e49c05472ab76cee4e30c2eaa4fa576b9ae92c6 EVP_FETCH: deal with names without pre-defined NIDs
baff732dc654f8aab37953f6036399989c765604 Make the generic EVP fetching mechanism use the namenum map
f2182a4e6fef098014117eb4a4f35ca84cd2cf8e Create internal number<->name mapping API


[DS-4257] dspace.url and dspace.baseUrl config cleanup

This PR is the final stage of https://jira.lyrasis.org/browse/DS-4257 (Spring Boot webapp refactor).  I finally got back to this.
It refactors our configuration slightly to better differentiate between our URL-based configurations.
Two new configurations are added:

dspace.server.url = The URL of the server webapp (i.e. DSpace backend)
dspace.ui.url = The URL of the Angular UI (i.e. DSpace frontend)
dspace.ui.hostname = The hostname of the DSpace UI (This config is only really used as an "identifier" for DSpace in modules like OAI-PMH)

UPDATE: I've updated this PR to refactor out the need for a "hostname" configuration. See notes below under the dspace.hostname bullet.

These old URL configurations have been removed (replaced by the above):

dspace.baseUrl is removed and (usually) replaced by dspace.server.url.  However, in a few rare cases it was actually referring to the UI and was replaced by dspace.ui.url.
dspace.url is removed and (usually) replaced by dspace.ui.url. However, in a few rare cases it was actually referring to the backend and was replaced by dspace.server.url.
dspace.hostname is completely removed (with no replacement).  Instead, I've refactored the code that used dspace.hostname to instead parse the hostname from the appropriate dspace.*.url configuration.

Added a new Utils.getHostName() utility method to parse the hostname from any URL. Added tests to prove it works.  Refactored Java code that previously used dspace.hostname to now use this utility method.
Changed OAI-PMH's description.xml to use the existing oai.identifier.prefix (which previously defaulted to dspace.hostname).  Changed oai.identifier.prefix to now be set dynamically to the hostname of the dspace.ui.url (if not specified in the oai.cfg or similar)
Refactored OAI-PMH build process to no longer populate the description.xml using Ant filters. Instead, any configuration variables in the description.xml are now populated dynamically.  This ensures it can use the oai.identifier.prefix (now populated dynamically).

dspace.mobileUrl is completely removed (no longer used anywhere in our codebase and not applicable for Angular UI).

NOTE: Once this PR is merged, we need to ensure all DSpace 7 developers are aware of the new configurations.  Additionally, the REST API demo site must update its local.cfg to use the new configurations (or else errors may occur).

@tdonohue
Could you add a method similar to getHostName that return the dspace.server.url without /server?
I mean https://dspace7.4science.cloud/server => https://dspace7.4science.cloud
I need it in the #2651 to fix this configuration
Thanks

I find no guarantee that authentication-shibboleth.lazysession.loginurl and dspace.server.url share a longest prefix.  I would argue that the login URL must be configurable, not calculated.
Maybe there is a reason for a property dspace.host after all.

This is at +2 and was approved to merge in today's DSpace 7 meeting.  As discussed today, @atarix83 will create the "base" URL method he requires in his separate Shibboleth PR (If you need any help, Giuseppe, ping me on Slack as needed!)


chore: Extra codes removed

Coverage remained the same at 96.97% when pulling e82f5fabea1a44231044294877909fb9115a78ce on pnaik1:cleanup into 54287c732ac5de9c5a1c4ccf8084ce185617c87d on nodeshift-starters:main.


Getting Started Guide is too long.

Feature request
Create quick and short Getting Started Guide.
Expected behavior
I can play with vscode-webview-ui-toolkit as quickly as possible.
Current behavior
I have to run multiple commands and edit multiple files and I am forced to use TypeScript.
Use case
Simple preferably one file extension without any generators like npm install -g yo generator-code.

Hi,
Especially Getting Started Guide can no longer be implemented... various errors appear with a recent installation of generator-code to initialize the development context.
Retrieving the template extension from Github works.


ES Modules: Simplified "Polyfill Loader"

Jira
N/A
Summary
Incorporates the majority of the polyfill consolidation / simplification that's part of the broader ES Module work without any component or build tool related changes required, or inadvertently introducing any major breaking changes.
Details
When the full set of updates from https://github.com/boltdesignsystem/bolt/pull/1579 are in place, the Core polyfill loader essentially becomes unnecessary since we only include the necessary polyfills once in the main bolt-global Webpack bundle.
That said (and something I especially use @danielamorse and @remydenton's feedback on) — the upstream ES Module work does NOT auto-include polyfills in any of the independent bundles from Webpack since these may or may not require the polyfill loader in order to work.
Because of that, I'm a bit unsure if / where the polyfill loader is getting used downstream in Drupal which we'd need to account for here, either by manually adjusting the code downstream OR by keeping the original polyfill loader exactly the same and instead renaming this slimmed down version to something else...
Thoughts?
How to test

Review changes
Confirm tests passing
Confirm overall browsers supported continue to work as expected (no errors, etc)

PR was released with v2.14.0


error in state update of LPDDR5

https://github.com/CMU-SAFARI/ramulator2/blob/b7c70275f04126c647edb989270cc429776955d1/src/dram/impl/LPDDR5.cpp#L397-L423
In command such as CASRD, CASWR, RD16, and WR16, the m_final_synced_cycle should be updated with m_timing_vals, not m_timing. Current codes result in issuing WCK2CK command per every RD or WR request.
The code should be
node->m_final_synced_cycle = clk + node->m_spec->m_timing_vals("nCL") + node->m_spec->m_timing_vals("nBL16") + 1;

Thank you! This has been fixed.


[Card] top padding issue

Description
The top padding of the card is different in code and Sketch. Originally reported by @stellambaka
Reproduction Steps

Observe the <Card /> component in code and in Sketch. Notice the top padding is different

Meta

[x] TDS version: v1.0.1
[ ] Willing to develop solution
[ ] Has workaround
[ ] High impact

Screenshots

fixed. will push out in the next release. created a jira ticket for other bugs stella has found.
https://telusdigital.atlassian.net/browse/TDS-514

These bugs have been fixed and released in DSM.


Migrate editor from files to Postgres.

Rewrite serve.py for a Postgres backend. Python and JS tests pass unchanged.

Add scripts to initialize the schema, define default users, and migrate legacy data.

Integrate the editor processes with Docker Compose, now required by tests.

Move access tokens from URLs to cookies in preparation for authentication. Add a /login page to switch among users.

Simplify review mode, replacing git subprocesses with a "sync" operation.

Fix a bug in dataset upload where JS made incorrect assumptions about pbtxt file encodings.

Agreed that this should get more manual testing. Anyone can hit it now at http://<IP_ADDRESS>/.


Rename "_unit" to "_normalized", or similar

The flag indicating whether a datavalue has been changed to normalize the unit should be called something to do with normalization.
Changing this key and using the new versions should retroactively change the source dicts for the existing origins, so no blatting should be required.
Ensure this change is documented on the appropriate locations.

Close old issue.


get access_token in componentDidMount is undefined

I use @axa-fr/react-oidc-context for oidc handling of my project.
In the following code I want to pass access_token to Child component, and in componentDidMount  method of Child  component I want to get access_token and attach that to the request header in order to read some data from server. But in this method value of access_token is undefined.
AuthenticationProvider
<AuthenticationProvider
        configuration={oidcConfiguration}
        loggerLevel={oidcLog.DEBUG}
        isEnabled={true}
        callbackComponentOverride={CustomCallback}>

        <AuthenticationContext.Consumer>
          {props => {
            return (
              <Child access_token={props.oidcUser?.access_token}></Child>
              );
            }}
        </AuthenticationContext.Consumer>
</AuthenticationProvider>

Child component code:
export class Child extends React.Component {
  constructor(props) {
    super(props);
  }
  componentDidMount() {
     console.log(this.props.access_token); // is undefined
  }
  render() {
   //after componentDidMount run, this.props.access_token has value
    return

FROM PROPS: {this.props?.arg1}

;
  }
}

How can I get access_token in componentDidMount method.
Notice that I don't want to use @axa-fr/react-oidc-context-fetch.

Hi @kasranajafi1991,
Thank you for your issue,
Which version are you using ?

my react and @axa-fr/react-oidc-context version :
"@axa-fr/react-oidc-context": "^3.0.4-alpha.0",
"react": "^16.9.0",

Do you have openid in your requested scope?
const configuration = {
client_id: 'spa',
redirect_uri: 'http://localhost:3000/authentication/callback',
response_type: 'code',
post_logout_redirect_uri: 'http://localhost:3000/',
scope: 'openid profile email api offline_access',
authority: 'https://demo.identityserver.io',
silent_redirect_uri: 'http://localhost:3000/authentication/silent_callback',
automaticSilentRenew: true,
loadUserInfo: true,
triggerAuthFlow: true,
};

yes here is my configuration :
const configuration = {
    client_id: 'xxxxxx',
    redirect_uri: `${siteUrl}authentication/callback`,
    response_type: 'code',
    post_logout_redirect_uri: `${siteUrl}`,
    scope: 'openid profile phone api',
    authority: `${sosUrl}`,
    silent_redirect_uri: `${siteUrl}authentication/silent_callback`,
    automaticSilentRenew: true,
    loadUserInfo: true,
    triggerAuthFlow: true,
};

Have you tried to log all props.oidcUser ?
it is working on my project.

I have an idea. I'll look at this next tomorrow or next week.

May i close this issue?
Isbit still relevent in v4? https://www.npmjs.com/package/@axa-fr/react-oidc-context


refactor(AlgebraicGeometry): stalk maps

Introduces Scheme.Hom.stalkMap and LocallyRingedSpace.Hom.stalkMap with the API copied from presheafed spaces. Removes LocallyRingedSpace.stalk and LocallyRingedSpace.ΓToStalk. Instead uses Presheaf.stalk and Presheaf.germ directly.

The intent is to get rid of erws when working with stalks on schemes. Especially composition of stalk maps caused issues before.

!bench

(btw: I am not sure why the bot does not remove the merge-conflict label)

I think the increases are mostly due to the set of simp theorems about equivalences. Where are these needed?

I think the increases are mostly due to the set of simp theorems about equivalences. Where are these needed?

The general ones are used in stalkMap_hom_inv and co, which are not used in this PR, but which come up from time to time and are certainly annoying to prove on the fly.

!bench

(For the record, Christian is on holiday and I'm taking over)

bors merge

@mattrobball I'd like to try writing the elementwise lemmas by hand. They use ConcreteCategory.instFunLike instead of the right funlike instance so should probably be rewritten anyways.

bors r-

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

I have given up. I tried various hacks that seems to improve the runtime on my local machine but the slow down is still there. @mattrobball if you are interested please give it a look. If not I think let's just merge it.

Can you summarize any observations? It is hard to track from the git log. Also, you can use time -v lake env lean --profile XX/YY/ZZ.lean on your machine to get statistics for your file. This is helpful so you don't have to wait for !bench

So by the bench before and after the commit 859763, one can see that stalkMap_germ'_apply is single-handedly responsible for the +25% slowdown. I've tried various methods (providing types to arguments, filling in implicit variables by hand) that speeds up the elaboration time of that lemma from 2s to 1s on my local machine, but it doesn't seem to affect anything.
And the command is useful. Thanks! I didn't know about the --profile flag. I guess I can experiment some more now.

It doesn't give me instruction count though. Can I somehow get that locally?

Maybe. I don't know how and use CPU time as a proxy there. It is fairly stable on your own machine.

Yeah but I don't seem to see the +25% instructions (or +25% cpu time) that I'm supposed to see.

Odd. I can see the different between this branch and master on my machine. I will poke slightly and then merge

It seems nothing in this PR is the culprit for the increases. During unification, Lean unfolds everything to check that x =?= ↑⟨x, hx⟩ for (x : X) (hx : α.base x ∈ U) multiple times.
I am not sure whether we can optimize this outside of core.
@erdOne do you still prefer the custom apply theorems over the elementwise ones?

Here is a gist of one example of the problem.

Yes they are needed regardless.
I suspect unbundling TopCat.Presheaf.germ so that there is no coercion in the signature might help. I might try that next.

Any other changes anyone wants before it is merged?

I think it's good to go. Christian is on vacation.

bors merge


fix(typeahead): Use mousedown event instead of click event. Fixes #363

mousedown fires before blur, see http://stackoverflow.com/questions/10652852/jquery-fire-click-before-blur-event

Fix using a different approach https://github.com/valor-software/ng2-bootstrap/pull/398

Looks even better! (Y)

closed in favor of https://github.com/valor-software/ng2-bootstrap/pull/398


When I reject an order I get this error

NameError in Orders#update
Showing /app/app/views/order_mailer/order_denied.html.erb where line #2 raised:
undefined local variable or method `order' for #<#Class:0x007ff20614e100:0x007ff2060e2158>
Did you mean?  @order
Extracted source (around line #2):
1
2
3
4
5
6

  <%= order.user.name %>,

  You order, #<%= @order.id %>, has been declined for the following reason:
Rails.root: /app
Application Trace | Framework Trace | Full Trace
app/views/order_mailer/order_denied.html.erb:2:in _app_views_order_mailer_order_denied_html_erb__3217532425436443871_70338730451920' app/mailers/order_mailer.rb:5:in order_denied'
app/models/concerns/order_status.rb:60:in block (4 levels) in <module:OrderStatus>' app/models/concerns/order_status.rb:103:in update_status'
app/models/order_updater.rb:34:in update_status' app/models/order_updater.rb:13:in update'
app/models/concerns/users/order_manipulator.rb:35:in block in update_order' app/models/concerns/users/order_manipulator.rb:33:in update_order'
app/controllers/orders_controller.rb:39:in `update'
Request
Parameters:
{"utf8"=>"✓",
"_method"=>"patch",
"authenticity_token"=>"7nC1BO92B/J2/8GpqT1Nv5GgmN1/73vn6zKYN/1mGmXdInQfzTqPJL9ppzakIAr3m/a31TisauOMtQ1qPZYYlg==",
"order"=>{"order_details"=>{"item_id"=>["20"],
"quantity"=>["1"]},
"status"=>"reject"},
"email"=>{"reason"=>"We are unable to process your order at this time."},
"id"=>"155"}
Toggle session dump
Toggle env dump
Response
Headers:
None

fixed by #471


Create login

Create authorization page to block access from unauthorized users. The credentials should be passed as ENV vars during and checked in the login page. After login, the user token should be stored in the session and used to make authenticated requests

Can you assign me and describe what do you want to do for the login form.

Hey @QuangHuy1902, added some description to the issue. Feel free to work on this issue and open a pull request.
Some more context: this is meant to be a self-hosted app, so we just need to support 1 user, and it's not supposed to be linked to feeds and items (yet)


fix for checkbox on firefox

Issue #, if available:
Description of changes:
Changed the behavior of checkbox to rely on html and edited action section of css and scss to make the frontend universally behave the same on all browsers
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

changes approved


Possibility destroy video on leaving page

Hi,
I am trying to destroy video everytime when I leave page where it's playing but it doesn't work.
Is any recommended way how to do it with this pack?
Cound someone post me a piece of code how to destroy video?
Thank you so much

Can you paste your code that you have trying to do this? The destroy() method should be sufficient. So let's figure out what's happening 😄

`import {Component} from "@angular/core";
import {Location} from "@angular/common";
import {Page} from "../page";
import {ActivatedRoute} from '@angular/router';
import{ArticlesService} from '../../services/articles.service';
import * as SocialShare from "nativescript-social-share";
import {RouterExtensions} from "nativescript-angular/router";
import {Video} from "nativescript-videoplayer";
@Component({
selector: 'article-page',
templateUrl: 'pages/article/article.page.html',
styleUrls:  ['pages/article/article.page.css'],
providers: [ArticlesService]
})
export class ArticlePage extends Page {
constructor(private location: Location, private route:ActivatedRoute, private articlesService: ArticlesService, private routerExtensions: RouterExtensions) {
    super(location);

}

videoTap(vPlayer: Video) {
    vPlayer.destroy();
 }

}`
   <VideoPlayer #vPlayer                 src="{{articleVideo}}"                 autoplay="false"                  controls="true"                 width="100%"                 height="210"                ></VideoPlayer>
This is way how to I trying this.
But maybe it will be better do it some way how to turn off video everytime when I leave video page.
I have app where I can open article. Some articles has video. I play video. Go back and sound is still playing even when I play video from another article.
Thank u for help

I dont know the NS+NG too well. So in your videoTap function where is that defined and are you sure the argument vPlayer is an instance of the VideoPlayer? In plain NS a tap event passes an object of type EventData which you can then get an instance of the control via the .object property.
If you are receiving no method on undefined or something similar then you likely don't have an instance of the VideoPlayer so you'd need to make sure you get an instance and then call the destroy method on the player.
@bnussey - don't you use Angular? Can you see what's wrong with the snippet above? My assumption is ^^^ and that the tap event isn't setup correctly to be executed. I would also guess you can get the VideoPlayer instance with a global method of getViewById() or something similar with NG. Thanks

@bradmartin You are correct about the tap event in NNG.
The method I use to get a ui component like this is to use Angular's ViewChild.
In this case you would need to import ViewChild from @angular/core and then add the ViewChild to the class
export class ArticlePage extends Page {
    @ViewChild(Video) public vPlayer: Video;

    constructor(private location: Location, private route:ActivatedRoute, private articlesService:
    ArticlesService, private routerExtensions: RouterExtensions) {
        super(location);
    }

    videoTap(vPlayer: Video) {
        this.vPlayer.destroy();
    }
}

I use Angular and do it like so:
@ViewChild("videoPlayer") _videoPlayer: ElementRef;

Then you can access the destroy method by doing this:
this._videoPlayer.nativeElement.destroy();

Didn't see @ls-spavel he has summed it up perfectly, thank you for that. Closing this issue for now.

@ls-spavel @bnussey thank you so much for your help. I dunno why but my app still crashes.
[aqme] 254: AQDefaultDevice (1): skipping input stream 0 0 0x0 [aqme] 254: AQDefaultDevice (173): skipping input stream 0 0 0x0 [aqme] 254: AQDefaultDevice (173): skipping input stream 0 0 0x0 [aqme] 254: AQDefaultDevice (173): skipping input stream 0 0 0x0 Apr 11 09:29:45 --- last message repeated 4 times --- 1   0x10f8927cb NativeScript::FFICallback<NativeScript::ObjCMethodCallback>::ffiClosureCallback(ffi_cif*, void*, void**, void*) 2   0x10ff671ee ffi_closure_unix64_inner 3   0x10ff67b22 ffi_closure_unix64 4   0x110c682f4 -[UIControl sendAction:to:forEvent:] 5   0x110c6860f -[UIControl _sendActionsForEvents:withEvent:] 6   0x110c6754a -[UIControl touchesEnded:withEvent:] 7   0x11100c35d _UIGestureEnvironmentSortAndSendDelayedTouches 8   0x111007374 _UIGestureEnvironmentUpdate 9   0x111006da7 -[UIGestureEnvironment _deliverEvent:toGestureRecognizers:usingBlock:] 10  0x111005f6e -[UIGestureEnvironment _updateGesturesForEvent:window:] 11  0x110b5307a -[UIWindow sendEvent:] 12  0x110affc90 -[UIApplication sendEvent:] 13  0x1112e1f5c __dispatchPreprocessedEventFromEventQueue 14  0x1112d9eba __handleEventQueue 15  0x1135dec01 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ 16  0x1135c40cf __CFRunLoopDoSources0 17  0x1135c35ff __CFRunLoopRun 18  0x1135c3016 CFRunLoopRunSpecific 19  0x115568a24 GSEventRunModal 20  0x110ae25b4 UIApplicationMain 21  0x10ff6797d ffi_call_unix64 22  0x11f3036b0 file:///app/pages/article/article.page.js:64:21: JS ERROR TypeError: undefined is not an object (evaluating 'this.vPlayer.destroy') Apr 11 09:29:45 Roman-MacBook-Air com.apple.CoreSimulator.SimDevice.AB4B81E5-D9EF-40EB-B3D9-CDACC149964A.launchd_sim[39526] (UIKitApplication:org.romansimr.HelloWorld[0xebd3][39551][40046]): Service exited due to Segmentation fault: 11
my code looks like this
  <VideoPlayer #vPlayer                 src="{{articleVideo}}"                 autoplay="false"                  controls="true"                 width="100%"                 height="210"></VideoPlayer>

`import {Component, ViewChild, ElementRef} from "@angular/core";
import {Location} from "@angular/common";
import {Page} from "../page";
import {ActivatedRoute} from '@angular/router';
import{ArticlesService} from '../../services/articles.service';
import * as SocialShare from "nativescript-social-share";
import {RouterExtensions} from "nativescript-angular/router";
import {Video} from "nativescript-videoplayer";
@Component({
selector: 'article-page',
templateUrl: 'pages/article/article.page.html',
styleUrls:  ['pages/article/article.page.css'],
providers: [ArticlesService]
})
export class ArticlePage extends Page {
@ViewChild(Video) public vPlayer: Video;

constructor(private location: Location, private route:ActivatedRoute, private articlesService: ArticlesService, private routerExtensions: RouterExtensions) {
    super(location);
}

videoTap(vPlayer: Video) {
    this.vPlayer.destroy();
}

}
interface Article{
id:string;
article_heading:string;
article_text:string;
article_photo:string;
article_perex:string;
article_author:string;
strip_id:string;
}`

You wanna try and do it like mine? Also for multiple lines of code you can do ```, makes it a lot easier to read.

Your way give me this error. I am kinda frustrated :( I tried almost everything.
[aqme] 254: AQDefaultDevice (1): skipping input stream 0 0 0x0 [aqme] 254: AQDefaultDevice (173): skipping input stream 0 0 0x0 1   0x10263b7cb NativeScript::FFICallback<NativeScript::ObjCMethodCallback>::ffiClosureCallback(ffi_cif*, void*, void**, void*) 2   0x102d101ee ffi_closure_unix64_inner 3   0x102d10b22 ffi_closure_unix64 4   0x103a112f4 -[UIControl sendAction:to:forEvent:] 5   0x103a1160f -[UIControl _sendActionsForEvents:withEvent:] 6   0x103a1054a -[UIControl touchesEnded:withEvent:] 7   0x103db535d _UIGestureEnvironmentSortAndSendDelayedTouches 8   0x103db0374 _UIGestureEnvironmentUpdate 9   0x103dafda7 -[UIGestureEnvironment _deliverEvent:toGestureRecognizers:usingBlock:] 10  0x103daef6e -[UIGestureEnvironment _updateGesturesForEvent:window:] 11  0x1038fc07a -[UIWindow sendEvent:] 12  0x1038a8c90 -[UIApplication sendEvent:] 13  0x10408af5c __dispatchPreprocessedEventFromEventQueue 14  0x104082eba __handleEventQueue 15  0x106387c01 __CFRUNLOOP_IS_CALLING_OUT_TO_A_SOURCE0_PERFORM_FUNCTION__ 16  0x10636d0cf __CFRunLoopDoSources0 17  0x10636c5ff __CFRunLoopRun 18  0x10636c016 CFRunLoopRunSpecific 19  0x108281a24 GSEventRunModal 20  0x10388b5b4 UIApplicationMain 21  0x102d1097d ffi_call_unix64 22  0x11211e6b0 file:///app/pages/article/article.page.js:63:26: JS ERROR TypeError: undefined is not an object (evaluating 'this._videoPlayer.nativeElement') Apr 11 20:24:16 Roman-MacBook-Air com.apple.CoreSimulator.SimDevice.AB4B81E5-D9EF-40EB-B3D9-CDACC149964A.launchd_sim[42346] (UIKitApplication:org.romansimr.HelloWorld[0x4d44][42371][42589]): Service exited due to Segmentation fault: 11

This is the issue:
file:///app/pages/article/article.page.js:63:26: JS ERROR TypeError: undefined is not an object (evaluating 'this._videoPlayer.nativeElement')

We need to figure out what your instance of this._videoPlayer is undefined when you execute that and you should be good to go.

Oh my mistake! It's working! Thank you so much. I really appreciate it. :) 👍

In case others come across this, here is the relevant code that I found was required for the destroy method to work in Angular (and iOS):
html:
 <VideoPlayer #videoPlayer src="{{videoSource}}" autoplay="false" height="300"></VideoPlayer>
ts:
import { Component, OnInit, ViewChild, ElementRef } from "@angular/core";
import { Page } from "tns-core-modules/ui/page";

import { Video } from 'nativescript-videoplayer';
registerElement("videoPlayer", () => Video);
...
export class CoolComponent implements OnInit {
   @ViewChild("videoPlayer") _videoPlayer: ElementRef;
  ...

    ngOnInit(): void {
        this.page.on('navigatingFrom', (data) => {  //could also have this code in a separate function called on tap
            this._videoPlayer.nativeElement.destroy();
        })
    }

Note that destroying the element may cause problems if you navigate to the page via the back button (it won't re-register the element, I believe, but the element will have been destroyed).
For this reason, on the pages where I call destroy I remove the back button from any subsequent pages that are navigated to from that page (remove with:  <NavigationButton visibility="collapsed"></NavigationButton>)
I am still struggling with the fact that on slower devices (like older ipads, for example), the video player fails to load entirely. But that is another issue.


Error when installing via Homebrew: luv-static

$ rm -rf /Library/Caches/Homebrew/neovim--git
$ brew reinstall --HEAD neovim
==> Reinstalling neovim/neovim/neovim
==> Cloning https://github.com/neovim/neovim.git
Updating /Users/julieengel/Library/Caches/Homebrew/neovim--git
==> Checking out branch master
==> Downloading https://github.com/libuv/libuv/archive/v1.8.0.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--libuv-1.8.0.tar.gz
==> Downloading https://github.com/msgpack/msgpack-c/archive/cpp-1.0.0.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--msgpack-1.0.0.tar.gz
==> Downloading https://raw.githubusercontent.com/neovim/deps/master/opt/LuaJIT-2.0.4.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--luajit-2.0.4.tar.gz
==> Downloading https://github.com/keplerproject/luarocks/archive/5d8a16526573b36d5b22aa74866120c998466697.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--luarocks-998466697.tar.gz
==> Downloading https://github.com/mauke/unibilium/archive/v1.2.0.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--unibilium-1.2.0.tar.gz
==> Downloading http://www.leonerd.org.uk/code/libtermkey/libtermkey-0.18.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--libtermkey-0.18.tar.gz
==> Downloading https://github.com/neovim/libvterm/archive/1b745d29d45623aa8d22a7b9288c7b0e331c7088.tar.gz
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--libvterm-7088.tar.gz
==> Downloading https://github.com/jemalloc/jemalloc/releases/download/4.0.2/jemalloc-4.0.2.tar.bz2
Already downloaded: /Users/julieengel/Library/Caches/Homebrew/neovim--jemalloc-4.0.2.tar.bz2
==> Building third-party dependencies.
==> cmake ../third-party -DUSE_BUNDLED_BUSTED=OFF -DUSE_EXISTING_SRC_DIR=ON -DCMAKE_C_FLAGS_RELEASE=-DNDEBUG -DCMAKE_CXX_FLAGS_RELEASE=-DNDEBUG -DCMAKE_INSTALL_PREFIX=/usr/local/Cellar/neovim/HEAD-2daf54e -DCMAKE_BUILD_TYPE=Release -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_VERBOSE_MAKEFILE=ON -Wno-dev
==> make VERBOSE=1
Last 15 lines from /Users/julieengel/Library/Logs/Homebrew/neovim/02.make:
/usr/local/Cellar/cmake/3.6.2/bin/cmake -E make_directory /tmp/neovim-20160913-68203-gfr9kf/deps-build/build/tmp
/usr/local/Cellar/cmake/3.6.2/bin/cmake -E make_directory /tmp/neovim-20160913-68203-gfr9kf/deps-build/build/src/luv-static-stamp
/usr/local/Cellar/cmake/3.6.2/bin/cmake -E make_directory /tmp/neovim-20160913-68203-gfr9kf/deps-build/build/downloads/luv
/usr/local/Cellar/cmake/3.6.2/bin/cmake -E touch /tmp/neovim-20160913-68203-gfr9kf/deps-build/build/src/luv-static-stamp/luv-static-mkdir
[ 21%] Performing download step for 'luv-static'
cd /tmp/neovim-20160913-68203-gfr9kf/deps-build/build/downloads/luv && /usr/local/Cellar/cmake/3.6.2/bin/cmake -DPREFIX=/tmp/neovim-20160913-68203-gfr9kf/deps-build/build -DDOWNLOAD_DIR=/tmp/neovim-20160913-68203-gfr9kf/deps-build/build/downloads/luv -DURL=https://github.com/luvit/luv/archive/1.9.1-0.tar.gz -DEXPECTED_SHA256=86a199403856018cd8e5529c8527450c83664a3d36f52d5253cbe909ea6c5a06 -DTARGET=luv -DUSE_EXISTING_SRC_DIR=ON -P /tmp/neovim-20160913-68203-gfr9kf/third-party/cmake/DownloadAndExtractFile.cmake
CMake Error at /tmp/neovim-20160913-68203-gfr9kf/third-party/cmake/DownloadAndExtractFile.cmake:33 (message):
  USE_EXISTING_SRC_DIR set to ON, but
  '/tmp/neovim-20160913-68203-gfr9kf/deps-build/build/src/luv' does not exist
  or is empty.

make[2]: *** [build/src/luv-static-stamp/luv-static-download] Error 1
make[1]: *** [CMakeFiles/luv-static.dir/all] Error 2
make: *** [all] Error 2

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/neovim/homebrew-neovim/issues

System:
OSX  10.10.3
$ brew --version
Homebrew 0.9.9 (git revision 5a264; last commit 2016-09-12)
Homebrew/homebrew-core (git revision 6f89; last commit 2016-09-12)

$ nvim --version
NVIM 0.0.0-alpha+201503292107 (compiled Mar 30 2015 16:10:38)
Commit: 960b9108c2928b6cf0adcabdb829d06996635211
Build type: RelWithDebInfo
Compilation: /usr/local/Library/ENV/4.3/clang -Wconversion -O2 -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/tmp/neovim20150330-55338-i4l3xx/build/config -I/tmp/neovim20150330-55338-i4l3xx/src -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include/luajit-2.0 -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include -I/tmp/neovim20150330-55338-i4l3xx/.deps/usr/include -I/usr/local/opt/gettext/include -I/usr/include -I/tmp/neovim20150330-55338-i4l3xx/build/src/nvim/auto -I/tmp/neovim20150330-55338-i4l3xx/build/include

I think you need to update the formula, the flags have changed. Also see https://github.com/neovim/homebrew-neovim/issues/177#issuecomment-244624710 in case that doesn't work.

Ah, I was thinking I'd be something along these lines. I was just reading through #143, #147 and #148 which also have useful info.
Will try and report back! Thanks!

So, at first $brew update wouldn't want to work, because of Homebrew permission issues.
$ brew update
error: insufficient permission for adding an object to repository database .git/objects
fatal: failed to write object
fatal: unpack-objects failed
error: insufficient permission for adding an object to repository database .git/objects
fatal: failed to write object
fatal: unpack-objects failed
Error: Fetching /usr/local/Library/Taps/neovim/homebrew-neovim failed!
Fetching /usr/local/Library/Taps/homebrew/homebrew-dupes failed!
warning: failed to remove Library/Perl/Updates/5.18.2/darwin-thread-multi-2level/perllocal.pod
To restore the stashed changes to /usr/local run: 'cd /usr/local && git stash pop'
Updated Homebrew from e10234f to 3d3a03f.
Updated 2 taps (caskroom/cask, caskroom/versions).
No changes to formulae.

Then I found this link and tried $ sudo chown -R $USER /usr/local and then $ brew update again. That updated the formulaes, including the neovim one.
Then I tried to install neovim again by first $ brew uninstall neovim and then re-installing it via $ brew install --HEAD neovim. This time everything went smooth and successful! 🎉  Thanks for the help @fwalch!


resized heading image

Related Issue
Closes: #1725

Description

Resized heading images and renamed image extension.
Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

Add screenshots

added screenshot