ASP.NET Core 2.2 Conversion

Hello All,
I'm hoping someone can help me with the following error:
3>obj\Debug\netcoreapp3.0\NEP.Portal.RazorTargetAssemblyInfo.cs(16,12,16,92): error CS0579: Duplicate 'Microsoft.AspNetCore.Mvc.ApplicationParts.ProvideApplicationPartFactoryAttribute' attribute
I converted a 2.2 solution that consists of two .NET Core libraries and a single front-end ASP.NET Core project. I'm not sure what is causing this error and after research if I insert the following line in my csproj it will compile but fails to server any pages all resulting in a 404 error:
false
Any help greatly appreciated.
Regards,
Scott

@SSSikora can you share what your CSPROJ files look like?

Or, if possible, upload the whole app as a GitHub repo so we can take a look?

Thanks for the response Eilon. Below are the csproj files from the two libraries and the ASP.NET Core app. I've gone as far as to create all three projects in VS2019 using file new and then copying all of the assets into each and the problem remains. It's strange because it's a relatively small and simple application. Appreciate your help with this.
LIBRARY 1:

netcoreapp3.0

LIBRARY 2:

netcoreapp3.0

ASP.NET Core:

netcoreapp3.0
3.3
a74b6723-bd0e-4cb0-b79d-839a7aa59d1f

Again, thank you so much for any information you might have,
Scott

So you're trying to migrate from ASP.NET Core 2.2 to ASP.NET Core 3.0 Preview 3? It looks like some of the package references are still to 2.2, such as Microsoft.AspNetCore.Identity.

Thanks for another reply. I see what you mean and have removed references to that Identity package but the problem persists. I think it may have to do with the Telerik controls. When I expand them in the dependencies that show asp net core 2.1.6. Could that have something to do with it? Although I am using them with the 2.2. project so I don't see why that would be a problem.

@javiercn / @pranavkm - any ideas what might cause this?

Could you attach the obj\project.assets.json for the Web app? Perhaps something's pulling in a 2.x version of MVC \ Razor and not getting overriden by the targeting pack.

Thank you for the help...
project.assets.zip

Could you confirm this is from the project that targets the WebSDK (Microsoft.NET.Sdk.Web). I think it's called NEP.Portal? I don't see any of the Telerik specific package references in it, so just making sure I'm looking at the right file.
If it is the right one, could you include the preprocessed output for the file: dotnet msbuild /pp:out.txt and attach the out.txt that MSBuild spits out. The project.assets.json doesn't indicate any reason for you to see the build error.

A zero-byte file is generated and the error I'm getting in VS appears in the command prompt as the attached photo shows. Thank you, Pranav.

@SSSikora you must remove package reference to Microsoft.AspNetCore.* v2.0
My csproj corrected (inside an ItemGroup tag):
<PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.7.0-beta2" />
<PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="3.0.0-preview3-19153-02" />
<PackageReference Include="Microsoft.AspNetCore.Identity.UI" Version="3.0.0-preview3-19153-02" />
<PackageReference Include="Microsoft.AspNetCore.Mvc.NewtonsoftJson" Version="3.0.0-preview3-19153-02" />
<PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="3.0.0-preview3.19153.1">
  <PrivateAssets>all</PrivateAssets>
  <IncludeAssets>runtime; build; native; contentfiles; analyzers</IncludeAssets>
</PackageReference>
<PackageReference Include="Microsoft.VisualStudio.Web.BrowserLink" Version="2.2.0" />
<PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="2.2.3" />
<PackageReference Include="Microsoft.Web.LibraryManager.Build" Version="1.0.172" />
<PackageReference Include="Newtonsoft.Json" Version="12.0.1" />
<!--
<PackageReference Include="Microsoft.AspNetCore.CookiePolicy" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.HttpsPolicy" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Identity" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Localization" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Localization.Routing" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Mvc.Localization" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Mvc.Razor" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Mvc.TagHelpers" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Razor" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.ResponseCompression" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Rewrite" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.Routing" Version="2.2.2" />
<PackageReference Include="Microsoft.AspNetCore.Server.Kestrel" Version="2.2.0" />
<PackageReference Include="Microsoft.AspNetCore.StaticFiles" Version="2.2.0" />
-->

I had the same error, I corrected it as well

@Eilon that mean that the v3 included what meta package?
I do not want to reference 1000 packages that I do not need... (It's going to weigh my apps for nothing)

@andrestalavera - thank you very much for the information and taking the time to look at my issue.
-Scott

@SSSikora we have some migration information at the following links:

ASP.NET Core migration guide from 2.2 to 3.0: https://docs.microsoft.com/en-us/aspnet/core/migration/22-to-30?view=aspnetcore-2.2&tabs=visual-studio
https://github.com/aspnet/Announcements/issues/325
https://github.com/aspnet/AspNetCore/issues/3612
https://github.com/aspnet/AspNetCore/issues/3756

@SSSikora did the migration guides help?

I had previously found those guides and updated the app, but I believe my problem is that the Trlerik components are not yet working under 3.0 and so I have set this aside for now until they do.
Thank you to all who took time to comment. I am greatly looking forward to working with this next release.
-Scott

@SSSikora got it, thanks for the update. Should this bug be closed, or is there still any pending action for the ASP.NET Core team?

I don’t believe the problem lies with ASP.NET Core so the issue can be closed. Thanks again for the help.
From: Eilon Lipton<EMAIL_ADDRESS>Sent: Sunday, March 24, 2019 11:05 PM
To: aspnet/AspNetCore<EMAIL_ADDRESS>Cc: Scott S Sikora<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: EXTERNAL: Re: [aspnet/AspNetCore] ASP.NET Core 2.2 Conversion (#8383)
@SSSikorahttps://github.com/SSSikora got it, thanks for the update. Should this bug be closed, or is there still any pending action for the ASP.NET Core team?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/aspnet/AspNetCore/issues/8383#issuecomment-476039173, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHDoSAJKIJgy35zzd7rchLGXvhrbSnFNks5vaDzGgaJpZM4bnPZB.


iOS Deployment Target 11.0

Drop iOS 9.0 and 10.0
Build failed on Xcode13 Beta
Enum cases with associated values cannot be marked potentially unavailable with '@available'
Developer Forumns
https://developer.apple.com/forums/thread/682351
fixes #1

PR to forked https://github.com/el-hoshino/NotAutoLayout/pull/142

Renew branch https://github.com/ykws/NotAutoLayout/tree/feature/build-xcode13


ReferenceError: navigator is not defined; while using with next js.

import React from 'react';
import Chart from 'kaktana-react-lightweight-charts';

type LWChartsProps = {

}

const LWCharts: React.FC<LWChartsProps> = () => {
    const options = {
        color: '#f48fb1',
        lineStyle: 0,
        lineWidth: 1,
        crosshairMarkerVisible: true,
        crosshairMarkerRadius: 6,
        crosshairMarkerBorderColor: '#ffffff',
        crosshairMarkerBackgroundColor: '#2296f3',
        lineType: 1,
    }
    const lineSeries = [{
        data: [
            { x: 0, y: 8 },
            { x: 1, y: 5 },
            { x: 2, y: 4 },
            { x: 3, y: 9 },
            { x: 4, y: 1 },
            { x: 5, y: 7 },
            { x: 6, y: 6 },
            { x: 7, y: 3 },
            { x: 8, y: 2 },
            { x: 9, y: 0 }
        ]
    }]
    return (
        <div>
            <Chart options={options} lineSeries={lineSeries} autoWidth height={320} />
        </div>
    );
};

export default LWCharts;

import dynamic from 'next/dynamic';
const Chart = dynamic(() => import('kaktana-react-lightweight-charts'), {
ssr: false
});

import dynamic from 'next/dynamic'; const Chart = dynamic(() => import('kaktana-react-lightweight-charts'), { ssr: false });

I already solved it. But thanks anyways.

import dynamic from 'next/dynamic'; const Chart = dynamic(() => import('kaktana-react-lightweight-charts'), { ssr: false });

I already solved it. But thanks anyways.

how did you solve it?

import dynamic from 'next/dynamic'; const Chart = dynamic(() => import('kaktana-react-lightweight-charts'), { ssr: false });

I already solved it. But thanks anyways.

how did you solve it?

Check @ahmaddie1378's solution. That's is exactly how I solved it.

to avoid Promises' warnings add .then()
import dynamic from 'next/dynamic';
const Chart = dynamic(() => import('kaktana-react-lightweight-charts').then(module => module.default), {
  ssr: false
});


Revert "Feat/537: remove old contract for collateral bridge"

Reverts vegaprotocol/frontend-monorepo#998

This looked like it was required, but wasn't in the end. The deploy was locked to an old deploy and the nameless person trying to deploy a new version didn't noticed. Moving the deploy lock to the latest version made things work as expected.


[Bug] Incorrect Usage of temporal-sql-tool described in README

What are you really trying to do?
Update the README to match the correct usage of the tool.
Describe the bug
The README describes two usages of the temporal-sql-tool create-database command:
./temporal-sql-tool create-database -database temporal
./temporal-sql-tool create-database -database temporal_visbility
However, per the tool's help page global arguments must be supplied prior to any commands:
NAME:
   temporal-sql-tool - Command line tool for temporal sql operations

USAGE:
   temporal-sql-tool [global options] command [command options] [arguments...]

Supplying the -database argument after the create-database command results in an "Incorrect Usage" error:
Incorrect Usage: flag provided but not defined: -database
The two usages in the README should be reordered as follows:
./temporal-sql-tool -database temporal create-database
./temporal-sql-tool -database temporal_visbility create-database
Minimal Reproduction
N/A
Environment/Versions

OS and processor: Linux
Temporal Version: 1.18.4
Are you using Docker or Kubernetes or building Temporal from source? No

Additional context

./temporal-sql-tool -database temporal_visbility create-database

temporal_visibility


Zero-downtime restart

Hi :)
Zero-downtime restart has always been a PITA in both Unicorn and Puma.
What's Falcon's story?

That's a really good question and one I've thought about. Here are my thoughts.
async-container is responsible for starting and stopping child tasks.
I recently implemented using a shared file descriptor for multiple processes which works pretty well. That means it's possible to kill old processes and start new ones without rejecting incoming connections.
Right now I'm working on an opinionated virtual hosting solution for Falcon and part of this will be figuring out how to do zero-downtime restarts.
The idea right now is to have both a signal and perhaps an IPC mechanism for restarting specific hosts (could be as simple as touch tmp/restart.txt).
I think part of this can involve more reliability at the container level.
So, one limitation with async-container is that there is no logic for "keep 8 processes running at all times". If you start a container with 8 processes, they won't be restarted when the crash. If there was a policy for restarting them, it would be easy to have another process gracefully terminate them.
With HTTP/2 we need to be careful to use GOAWAY to ensure that we gracefully shut down existing connections.
guard-falcon already implements sort of zero-downtime restarts in a very limited context. It uses a shared socket in the parent process and restarts the application server processes as required. It's not perfect but it does work for its intended purpose. It doesn't do rolling restarts.
Maybe what you can do to help is clarify exactly how you think zero-downtime restarts should work and ideally what the process model should be so I can get a handle on exactly how this should work from your POV.

Guard::Falcon uses a shared endpoint and restarts the entire container every time file changes.
https://github.com/socketry/guard-falcon/blob/42fdf89439c7e077182f2affeca2bed4bc97dd21/lib/guard/falcon/controller.rb#L73-L123
There are potential options for optimising this - e.g. pre-fork which is only killed when files that affect the actual app server code change.

I think this is also a related issue: https://github.com/socketry/falcon/issues/17

I am going to brain dump some more things I've been thinking about.
Right now, the Falcon::Hosts is used to implement virtual hosting.
I feel like it's a very wide design. As in, the surface area of the API is proportional to the features required. As a test case, I'm thinking about how to add support for Let's Encrypt. This requires a user optionally specify per host whether they want to provision a certificate, or not, and the appropriate infrastructure to update the certs periodically, along with restarting the appropriate processes when the certs are updated in a graceful way.
I always liked the Rack way of layering middleware. I'm trying to figure out if rather than going "wide" we can go "deep" like Rack, for the specification of web hosts. To me, it seems like in order to go deep, some level of "configuration abstraction" is required. e.g. using a hash to define the configuration, then using a collection of these hashes (one per host?) to start the appropriate daemons. It would function like a database of configuration and then a stack of host middleware could act on it to do the right thing. To me, though, it seems a little bit too much indirection.
Configuration hashes are hard to document, hard to reason about, and it might just be simpler to go with some "wide" class like how Falcon::Host currently works, e.g. host.ssl_certificate_path = xyz.
As always, thoughts are welcome.

In theory this is now possible.  If the process has a connection in-flight it will be dropped, but if you use falcon virtual (experimental) the request would be retried up to 3 times (internally) if it is idempotent. You can access this functionality by falcon supervisor restart and the examples will show you how to do it, e..g falcon virtual examples/hello/falcon.rb. Feel free to try it out.


Enhancement request : Support for Typora

I it would be great to support Typora, so that HTML generated from Typora can use strapdown to generate nice dynamic TOCs. See
https://github.com/typora/typora-issues/issues/348
Perhaps you guys find some time to see what I am missing, as I cannot fix it.
Thanks a lot in advance
Frank

Strapdown works from an HTML file where you would write Markdown in the  tag, and transform that Markdown. In the HTML file you sent, Typora already transformed the Markdown inside of the  tag to HTML. At that point there is no Markdown left in the source file and Strapdown is not meant to work directly from HTML. I'm afraid that's not a supported use case.
I remember looking into tocify before doing my own thing for Strapdown. I think you are looking for something like that, that would work from the HTML itself, for you dynamic ToC. Or try to get Typora to support that from their converter!

Thanks a lot for your update.


HADOOP-16443 Improve help text for setfacl --set option

The help text associated with the command "setfacl --set" states:
--set     Fully replace the ACL, discarding all existing entries. The
             <acl_spec> must include entries for user, group, and others for
             compatibility with permission bits.

However the actual behaviour is a bit more subtle:

If the ACL spec contains only access entries, then the existing default entries are retained. If the ACL spec contains only default entries, then the existing access entries are retained. If the ACL spec contains both access and default entries, then both are replaced.

This PR will improve the help text to more align with the expected behaviour.

LGTM. The updated text comes from the javadoc of AclTransformation#replaceAclEntries()

+1, committed to branch 3.2+. Thanks!


Bug/dashboard

Bug description:

Dashboard rendering fails when service plan is updated as the corresponding dashboard URL is not updated in cloud controller.
In case of migration from one bosh to other, dashboard rendering code might query to older bosh to get tasks hence the dashboard rendering would fail.

To resolve the part 1 of the bug, a call to cloud controller to get the current plan of the instance was added in both DockerInstance and DirectorInstance.

To resolve part 2 of the bug, a new function getTasksForDashboard was added in BoshDirectorClient which queries only correct director instead of looping through all the primaryConfigs as original getTasks does. The 'correct' director is fetched from the cache using deployment name.

Minor modification in docker dashboard acceptance test to accommodate extra call to CC.

Added one unit test for getTasksForDashboard.

Pull Request Test Coverage Report for Build 1030

10 of 13 (76.92%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.006%) to 88.295%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

broker/lib/fabrik/DirectorInstance.js
1
4
25.0%

Totals

Change from base Build 1016:
-0.006%

Covered Lines:
6382

Relevant Lines:
6996

💛  - Coveralls

@abh1kg and @sauravmndl : please review this PR.

@sauravmndl has requested for changes. Further, there are some JSHint issues with this pull.

Yes, I will look into why the build is failing, I think it's the trufflehog issue. Will finalise it ASAP.


Avoid state vector normalization if it worsens the round offs

This fixes failure of
check/pytest -n0 cirq-core/cirq/circuits/circuit_test.py::test_final_state_vector
which happened because normalization of a state vector at np.complex64
precision can subtly increase the overall round-off error.
Follow up to #6522 and #6402

I have got consistent failures for the cirq-core/cirq/circuits/circuit_test.py::test_final_state_vector
test on my Debian-based desktop which were bisected to #6522.
For some reason these do not show in CI runs, but it is preferable to
have passing tests in local environments too.

maybe we should remove it normalization then?

The normalization does help for your original round-off example in #6402.
I think the case where it worsen the outcome is for state vectors with few
nonzero elements.  Here is how test_final_state_vector fails on my
desktop (the outcome may be different for other OS or CPU versions):
>>> x = np.exp(1j * np.pi/4, dtype=np.complex64)
>>> xn = x / np.linalg.norm(x)
>>> abs(x * x.conj() - 1)
5.960464477539063e-08
>>> abs(xn * xn.conj() - 1)
1.1920928955078125e-07

where the latter value for normalized xn is above the test tolerance.
With this PR the test passes and the round-off is better for #6402.


Optional media swipe gestures

Describe the feature you'd like to improve in Pock
Make swipe gestures optional.
For a long time I wondered why sometimes Spotify would skip a track when I tried to pause it.  Turns out there's a swipe feature I wasn't aware of, but after experimenting with it, I can't reliably skip tracks in the direction I want to (and often it skips when I want it to pause).
Unless it can be made more reliable, I'd suggest adding an option to disable it.

After playing with the gesture a bit I am pretty sure it's up and down, not left and right. It's hard to perform since there isn't much room to swipe up or down.

It seems like you have to swipe within the music widget to trigger the gesture according to PKView in PockKit, there must be a reason for that but it would be nice to add a boolean to let us swipe outside of the widget.
https://github.com/pigigaldi/PockKit/blob/13b84eaac269e69f78a985c9180d6bccb0fe4bf6/PockKit/Sources/PKView.swift#L76


Is there any update or any sort of maintenance for this project?

Yes, there will be some minor updates. Those including:

Enlist this mod to Sayonika (a mod-listing website)
Integrate to DDLC Mod Launcher (from Pi)(if he's still alive)(but I doubt not a lot of mods are going to integrate this mod launcher since the modding community is different nowadays)

However, there will be some updates that I will not be going to work on, which is:

No full support for fddme Mod Launcher (if there is something wrong with the compatibility of my mod in his launcher, that is not my responsibility)
Will not integrate into Doki-Doki Mod Launcher (DDML) by GanstaKingofSA (there are a few flaws that I do not want to solve, or the mod launcher is quite flimsy for "compatibleness" - I would prefer fddme)
No implementation of Travis CI for Sayonika - I only prefer list only, not auto mod-builder (I'm fine with the traditional way, not using rpatool btw)

If someone is going to fully resume this mod, these problems will likely to be solved. But so far no one has stepped forward to do so.

My schedule will likely to be packed with my classes in my college so I won't have time for those anyway, but if I have a chance, I will do it.


Add first and last variables

thanks for the pr, can you fix the formatting so that it's 2 spaces? then I can merge. thanks!

Hi, I've corrected the calculation of the last variable, it wasn't taking the start option into account, and re-indent to 2 spaces.


does not work when app killed by the user

I'm not sure if its the intended behavior or not but the callback does not fire if the user kills the app, the notification is always showing but it does nothing.
i tried with two tasks :

change a document in firebase
notify user using the flutter_local_notifications package

and in both cases, the plugin does not work when the app is killed (even if the foreground notification is still showing in the tray)
"extra: my use case is to run a callback periodically that check if the GPS is enabled, if yes update a document in firebase if not notify the user (this should work even if the user killed the app accidentally)"

I am facing the same issue when I switch off location permission from settings while the foreground service is still running
stacktrace-
java.lang.RuntimeException: Unable to start service changjoopark.com.flutter_foreground_plugin.FlutterForegroundService@2d593f7 with null: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String android.content.Intent.getAction()' on a null object reference         at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:3902)         at android.app.ActivityThread.access$1700(ActivityThread.java:212)         at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1846)         at android.os.Handler.dispatchMessage(Handler.java:106)         at android.os.Looper.loop(Looper.java:193)         at android.app.ActivityThread.main(ActivityThread.java:6912)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:860)      Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String android.content.Intent.getAction()' on a null object reference         at changjoopark.com.flutter_foreground_plugin.FlutterForegroundService.onStartCommand(FlutterForegroundService.java:23)         at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:3883)         at android.app.ActivityThread.access$1700(ActivityThread.java:212)          at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1846)          at android.os.Handler.dispatchMessage(Handler.java:106)          at android.os.Looper.loop(Looper.java:193)          at android.app.ActivityThread.main(ActivityThread.java:6912)          at java.lang.reflect.Method.invoke(Native Method)          at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)          at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:860)

I have a similar issue. For me, my OS is stopping the main app after a few hours and then the callback isn't called anymore.
From what I understand, the method channel gets destroyed in the process (and flutter gets detached from engine). I am not sure though since this is all pretty hard to debug for me.
Any ideas how to solve this so far? I was thinking of re-starting the activity via intent maybe. I want to avoid moving all the stuff into an own plugin (which should also keep running then just like the foreground service) since I want to keep it in Dart

For me the same. If the app has been closed by the user, the foreground service is  detached. App is killed and no more notifications.
Is there any fixes?
How to prevent the app being killed or how the delete the ongoing push notificaition too?
Thanks,
-Sue

did anyone find a solution for this?

Any idea?? still not working when app is closed, this means that the plugins is useless...

Is it working?if not how to resolve this?

Is it working?if not how to resolve this?


Bug: Page throws an error when no links are found in the bio

The image above shows the lines of code in question - if no links are present, matches becomes null and then when we try to map over it, an NPE is thrown.

@Satyam1203 I've opened a PR for this. please assign to me

@ShaneCreedon You already made a PR, right ? I'll merge it.. No point to assign it to you here and then go merge it.

@Satyam1203 Okay that's perfect! https://github.com/Satyam1203/resume-github/pull/6


Uncaught TypeError: Cannot read property 'nav' of undefined

When trying to access the Amplify Console from AWS website i get a white screen:
On the dev console i see a bunch of errors starting with:
Uncaught TypeError: Cannot read property 'nav' of undefined
And some more similar to this:
Refused to connect to 'https://console.aws.amazon.com/feedback/custsat/1/popquestion?locale=es&service=amplify&region=us-east-2' because it violates the following Content Security Policy directive: "connect-src https://us-east-2.console.aws.amazon.com/amplify/ https://bitbucket.org/ https://*.ccs.amazonaws.com https://aws-amplify-prod-us-east-2-access-logs.s3.us-east-2.amazonaws.com/ https://aws-amplify-prod-us-east-2-artifacts.s3.us-east-2.amazonaws.com/ https://e7auv6no3g.execute-api.us-east-1.amazonaws.com/ https://*.analytics.console.aws.a2z.com https://github.com/ https://phd.aws.amazon.com https://console.aws.amazon.com/amplify/ https://x1wkkmql32.execute-api.us-east-2.amazonaws.com/ https://gitlab.com/ https://console.aws.amazon.com/feedback/custsat https://ccs.amazonaws.com https://api.bitbucket.org/ https://api.github.com/ https://us-east-2.prod.signer.console-api.aws.amazon.com".

@agueric are you still seeing this? The Amplify console loads for me in us-east-2 and other regions as well. Is there a specific page this happens or some way to repro?

@agueric are you still seeing this? The Amplify console loads for me in us-east-2 and other regions as well. Is there a specific page this happens or some way to repro?

@kahdojay i'm still seeing the same problem. I've had it since at least 3 weeks. I thought it was some temporal problem but still seeing the same errors.

@agueric Thanks for reporting. Are you seeing the issue in any service consoles other than Amplify? Just trying to narrow down the issue.

No, actually if I visit another region I see the Start page. But on this one we do have a couple Apps running. I've tried another browser but still the same.
Should I try Incognito mode or some other test?

@agueric yes if you could confirm whether this happens in multiple browsers/incognito, and whether other service consoles are having the same issue in the same region (for example Rekognition, Batch, etc)

I've tested on an incognito browser and it did work. Using it for now. On other services there is not such problem.
I guess there's something wrong with cookies, cache, or something similar :/

Ok, we'll need to look into this further, I'm not able to reproduce the issue and haven't heard of other customers experiencing this yet but we will take a look. If you have a consistent repro we can follow let us know.

Hi @agueric are you still running into this issue or were you able to resolve it?

Hi.
I resolved it. It had to do with something in the browser and session. I
ended up clearing cookies and local storage on Chrome and it worked.
On Mon 22 Nov 2021 at 15:56 hloriana @.***> wrote:

Hi @agueric https://github.com/agueric are you still running into this
issue or were you able to resolve it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-console/issues/1920#issuecomment-975952977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3T32KB5B6J7EF5RYDKJA3UNK37TANCNFSM46BRHPOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@agueric Awesome. Thanks for reaching out and if you have any more questions or feedback, feel free to submit another issue. Thanks 🙂


Additional simplified interface

Is your feature request related to a problem? Please describe.
Calculating an indicator is currently possible in 2 ways (that i am aware of).
import pandas_ta as pta
df['rsi'] = pta.rsi(df['close'])
df['atr'] = pta.atr(df['high'], df['low'], df['close'], 20)
# or
df.ta.rsi(append=True)
df.ta.atr(20, append=True)

While usage of the above is "okish" for simple indicators like RSI (which only requires close) - it'll become painful with other indicators, which also require open, high, low, and maybe also volume).
The "dataframe interface" on the other hand would provide this simplification, but forces certain naming conventions - which can break further usage down the line (maybe a name(s) argument could be provided here, on top of the prefix / postfix, so the name(s) is 100% stable/predictable.)
(the above would result in "ATR_20" - which would change to ATR_21 if the number changes ...
While this can work for some cases, it can also cause issues if trying to find the right value for this.
Obviously this can become tricky for "multi-column" indicators ... where a list of names with the correct lenght might work.
As the "dataframe interface" already assumes certain facts about the underlying dataframe, an (additional) interface as follows would be great:
import pandas_ta.abstract as pta
df['rsi'] = pta.rsi(df)
df['atr'] = pta.atr(df, 20)

While for RSI, there's not much "saved letters" - for other indicators (like atr in the above sample) this can provide quite some simplification for the user.
It'd provide full control on the "resulting" dataframe column name, while simplifying the user's life.
Implementation-wise, i guess this can be mostly auto-generated - and might result in a slightly different import (similar to how the ta-lib python wrapper handles this).
note
Maybe i'm overlooking an alternative here which allows this already ... but i haven't quite found a way that provides both the simplicity (i'm lazy when typing), as well as freedom in column naming.

Hey @xmatthias,

Maybe i'm overlooking an alternative here which allows this already ... but i haven't quite found a way that provides both the simplicity (i'm lazy when typing), as well as freedom in column naming.

I am lazy when typing as well. So perhaps the third Programming Convention, Pandas TA Strategy, as mentioned on the README may be what you want.
It is a ta DataFrame Extension method, df.ta.strategy(), that one can automatically append indicators to the current df; less typing. This Jupyter Notebook shows examples on how to create Custom Strategies, including column naming like the last cell.
Here is a modified use case similar to the last cell in the Jupyter Notebook with Column Naming that you are interested in:

Note: When using 'col_names', there is [i] No mulitproccessing support for 'col_names' option. So this might be the only issue if there are a ton of indicators that need to be ran.

Potential New DataFrame Extension Property
I have also been thinking of adding another settable DataFrame Extension Property, like df.ta.study, which allows a user to specify a json file defining a Custom Strategy.
For example, consider sample.json:
{
    "name": "Sample JSON Strategy",
    "description": "Sample JSON Strategy Test",
    "ta": [
        {"kind": "ema", "length": 10, "sma": false},
        {"kind": "sma", "length": 50, "talib": false},
        {
            "kind": "bbands",
            "talib": true,
            "col_numbers": [0, 1, 2],
            "col_names": ["BBL", "BBM", "BBU"]
        },
        {
            "kind": "atr",
            "length": 50,
            "talib": true,
            "col_names": ["ATR"]
        }
    ]
}

from json import load as json_load
import pandas_ta as ta

tmpdf = ta.df.ta.ticker("PLTR", period="6mo", lc_cols=True)

def load_strategy(path:str):
    strat, json_strat = None, json_load(open(path))

    strat = json_strat[0] if isinstance(json_strat, list) and len(json_strat) == 1 else json_strat

    for indicator in strat["ta"]:
        if "col_numbers" in indicator and len(indicator["col_numbers"]):
            indicator["col_numbers"] = tuple(indicator["col_numbers"])
        if "col_names" in indicator and len(indicator["col_names"]):
            indicator["col_names"] = tuple(indicator["col_names"])

    if strat is not None and hasattr(strat, "description") and len(strat):
        return ta.Strategy(name=strat["name"], ta=strat["ta"], description=strat["description"])
    else:
        return ta.Strategy(name=strat["name"], ta=strat["ta"])

study = load_strategy("./sample.json")

df = tmpdf.copy() # Download
df = df.iloc[:,:-2]  # Remove 'dividends' and 'stock splits' columns

df.ta.strategy(study, timed=True, verbose=True)

df = df.iloc[:,3:] # Ignore 'open', 'high', 'low' and 'volume' for charting
df.drop(columns=["volume"], inplace=True)

# Plot everything but 'ATR' since it has a different scale
colors = ["black", "green", "red", "violet", "purple", "violet"]
df.iloc[:,:-1].plot(figsize=(16,10), color=colors, title=study.name, grid=True)

Result

Thoughts on loading a JSON strategy/study that I outlined with the help of a DataFrame Extension Property?
Hope this answers the question and helps!
KJ

@twopirllc

[i] No mulitproccessing support for 'col_names' option.

What would be the reason behind it? I was thinking we could do a first pass across all indicators to get the output column_names to check for any collisions and only start processing once sure that all column names are unique. Thoughts?

@xmatthias
What is a desired behavior when you want 2 or more of same indicator (say SMA5 and SMA50 or RSI14 and RSI50)?
df.ta.rsi(append=True)
df.ta.rsi(50, append=True)

When we print a Strategy, will printing the output column_names simplify user experience?
custom_a = ta.Strategy(name="A", ta=[{"kind": "sma"}, {"kind": "sma", "length": 50}, {"kind": "sma", "length": 200}])
print(custom_a)

Strategy(name='A', ta=[{'kind': 'sma', 'output_column_names': ['SMA_10']}, {'kind': 'sma', 'length': 50, 'output_column_names': ['SMA_50']}, {'kind': 'sma', 'length': 200, 'output_column_names': ['SMA_50']}], description='TA Description', created='Saturday June 19, 2021, NYSE: 7:59:38, Local: 11:59:38 PDT, Day 170/365 (47.00%)')

@twopirllc Any thoughts?


[Draft] fix building on Zephyr 3.7.0

Zephyr 3.7.0 is the latest LTS version, it should be supported for long term application development
Here is just a draft, some Kconfig need to be changed to new ones ( like CPLUSPLUS flag )

A very weird situation:
When I manage zephyr-rust via west:
manifest:
  version: "1.2"
  remotes:
    - name: zephyrproject-rtos
      url-base: https://github.com/zephyrproject-rtos
    - name: iotpi
      url-base<EMAIL_ADDRESS>
  defaults:
    remote: zephyrproject-rtos

  projects:
    - name: zephyr
      revision: v3.7.0
      import:
        name-allowlist:
          - cmsis
          - cmsis-dsp
          - mcuboot
          - picolibc
          - hal_stm32
    - name: zephyr-rust
      remote: iotpi
      revision: 3.7-support
      submodules: true

  self:
    path: rsmc

go inside zephyr-rust, execute:
west build -b qemu_cortex_m3 -p always samples/rust-app/

it failed as following:
  userspace: false
  /usr/lib/llvm-14/lib/clang/14.0.6/include/inttypes.h:21:15: fatal error: 'inttypes.h' file not found
  thread 'main' panicked at 'Unable to generate bindings: ClangDiagnostic("/usr/lib/llvm-14/lib/clang/14.0.6/include/inttypes.h:21:15: fatal error: 'inttypes.h' file not found\n")', src/main.rs:53:10
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

If I remove zephyr-rust from west.yml, and use git clone, then it pass build.

I was able to get 3.7 working in #54 but couldn't get it to work with prior 3.x versions, so I dropped them. I think it makes sense to support the 2.7 and 3.7 LTS versions now that 3.x has an LTS. Thanks for looking into it.
I'm not sure about the west module, but failing to find inttypes.h has come up in other contexts.


Download attachments for group mailbox posts

Attachment download needed an expand operation. Added that.
Added test coverage with gock. Tested with manual backup of posts which contain attachments(embedded/non-embedded). We can't add e2e tests with attachments, since API to create new conversations requires delegated access.
Note that https://github.com/alcionai/corso/issues/4991 is still an open item. We don't have a resolution for it right now, since attachment endpoint requires delegated token. Defaulting to let the backup fail for "too many attachments" error case. We don't know yet if we'd see that with group mailboxes, and whether it'd be the same error that we saw with exchange (recurring 503s).

Does this PR need a docs update or release note?

[x] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[ ] :no_entry: No

Type of change

[ ] :sunflower: Feature
[x] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

#

Test Plan

[x] :muscle: Manual
[x] :zap: Unit test
[ ] :green_heart: E2E

@pandeyabs - let's add a note clarifying the known issue here with large number of attachments,

Thanks @vkamra, good call! I'll add a known issue. Thanks Keepers for the review!


implement pythonic len()

len() in python is a simple builtin utility to count the number of items in a tuple,list,dictionary and then simply return the results. something like this:
func len(list):
var x = 0
for i in list:
x+=1
return x
Maybe I'm the only one, but I tend to implement something similar quite often in my scripts.

There is a size() func that does that. Not the same name, but I think is what you looking for.
see ex:
https://github.com/godotengine/godot/wiki/class_dictionary#size
https://github.com/godotengine/godot/wiki/class_array#size

Duplicate of #1960.

could have been implemented, but now adding that as a keyword would most
likely break existing games. Maybe for 3.0
On Wed, Dec 23, 2015 at 8:28 PM, Rémi Verschelde<EMAIL_ADDRESS>wrote:

Closed #3126 https://github.com/godotengine/godot/issues/3126.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3126#event-499837953.


where is WD_ROW_HEIGHT?

I am reading https://python-docx.readthedocs.io/en/latest/api/enum/WdRowHeightRule.html
But I can't find WD_ROW_HEIGHT. What happened?

WD_ROW_HEIGHT is a (shorter) "alias" for WD_ROW_HEIGHT_RULE. You include it from the same place as the full-length version and use it the same way.
Some of the enumeration names get long, so shorter versions are provided to keep the code more compact. They're noted as an "alias:" in the documentation page for that enumeration, as you can see on the page you linked.


More advanced joins

Hi there,
Currently I believe it's only possible to join two queries by an identical property (i.e. predicate to predicate)
It would be great to be able to join queries by joining on different fields. This kind of API would do I guess:
// QuadStore
const matchTermsA = {graph: 'g'};
const matchTermsB = {subject: 's'};
quadStore.query(matchTermsA)
    .join(quadStore.query(matchTermsB), [{ 'subject': 'graph' }]) // this would join (rather arbitrarily) a.graph == b.subject)
    .get((err, quads) => {});

Allowing to still pass strings would keep backward compatibility, and allowing objects would enable joining by different properties.
What do you think?
Jun

Hello @jmatsushita. This is a fantastic suggestion. I have a few hours of spare time coming up. I'll look into this and #40 .

Great ! :)

Some preliminary work about this here: https://github.com/jacoscaz/node-quadstore/blob/joins/test/quadstore.js#L586
I've had to slightly changed your proposal as I needed to maintain the ordering of term names:
quadStore.query(matchTermsA)
    .join(quadStore.query(matchTermsB), ['subject'], ['graph'])
    .get((err, quads) => {});

See pull request #43

Closing, available on NPM in<EMAIL_ADDRESS>.


Unable to compile Mica-WebView2

Unable to compile a clean import of Mica-WebView2 solution.
Error	MSB3231	Unable to remove directory "x64\Debug". The process cannot access the file 'MicaWebView2.log' because it is being used by another process.	Mica-WebView2	F:\microsoft WindowsAppSDK-Samples main Samples-Mica\cpp-WebView2\Mica-WebView2\MicaWebView2.vcxproj	209

That message can result from a background msbuild process.  If you still have trouble building that sample, please share a full binlog.  Thanks!


Fixed an issue with an error during pipeline execution

The error I am getting is the following -
ReferenceError: Response is not defined
 at getModelFile /node_modules/@xenova/transformers/src/utils/hub.js:462:42

The environment I am using is NodeJS v16.15.0 with TypeScript with the default CommonJS configuration.
This Commit fixes the issue.

This unfortunately breaks the caching system as FileResponse objects will now also be cached (leading to duplicated storage). According to the 16.15.0 (LTS) release notes, you should be able to resolve this by adding the --experimental-fetch flag. Alternatively, you can update to a more up-to-date version of NodeJS (18+) or use a polyfill

Thanks for the clarification 👍


Define dev container setup

This has also already been done, but maybe has some room for improvement.
I'm not sure if npm install happens automatically, or if I could spawn background tests for npm run dev and npm test on container creation, then attach new terminal windows on attachment


Dataclasses module missing from pip package

A clear and concise description of what the bug is.
I installed Superset using the pip package today following the "Install from Scratch" instructions.   Superset fails with a missing module error.
Expected results
superset commands should work
Actual results
All superset commands error with the following:
Traceback (most recent call last):
  File "/root/venv/bin/superset", line 11, in <module>
    load_entry_point('apache-superset==0.38.1', 'console_scripts', 'superset')()
  File "/root/venv/lib64/python3.6/site-packages/pkg_resources/__init__.py", line 476, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/root/venv/lib64/python3.6/site-packages/pkg_resources/__init__.py", line 2700, in load_entry_point
    return ep.load()
  File "/root/venv/lib64/python3.6/site-packages/pkg_resources/__init__.py", line 2318, in load
    return self.resolve()
  File "/root/venv/lib64/python3.6/site-packages/pkg_resources/__init__.py", line 2324, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/root/venv/lib64/python3.6/site-packages/superset/__init__.py", line 21, in <module>
    from superset.app import create_app
  File "/root/venv/lib64/python3.6/site-packages/superset/app.py", line 45, in <module>
    from superset.security import SupersetSecurityManager
  File "/root/venv/lib64/python3.6/site-packages/superset/security/__init__.py", line 17, in <module>
    from superset.security.manager import SupersetSecurityManager  # noqa: F401
  File "/root/venv/lib64/python3.6/site-packages/superset/security/manager.py", line 44, in <module>
    from superset import sql_parse
  File "/root/venv/lib64/python3.6/site-packages/superset/sql_parse.py", line 18, in <module>
    from dataclasses import dataclass
ModuleNotFoundError: No module named 'dataclasses'

This is because the 'dataclasses' module is apparently missing as a requirement from the python packaging.
How to reproduce the bug

Install Superset on a clean CentOS machine following the "Install from scratch" instructions
Try to run any superset command

Environment
(please complete the following information):

superset version: 0.38.1
python version: 3.6.8
node.js version:  not installed

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ x] I have reproduced the issue with at least the latest released version of superset.
[ x] I have checked the issue tracker for the same issue and I haven't found one similar.

dataclasses was added to the python core lib in 3.7. Looks like you're running 3.6.8. Newer versions of superset specify python version >= 3.7. https://github.com/apache/superset/blob/28c383af68d92bdbfe69fe8c90f8f6a769985b2b/setup.py#L156
I'd suggest your upgrade your python version as well as superset.


Can install pypi package pmtiles

Get the following error.  However, the pmtiles source distribution does have setup.py.  Any advice?
Error: https://github.com/opengeos/leafmap/actions/runs/11061972073/job/30735570607

Resolved 237 packages in 1.10s
error: Failed to prepare distributions
Caused by: Failed to fetch wheel: pmtiles==3.4.0
Caused by: /home/runner/work/_temp/setup-uv-cache/built-wheels-v3/pypi/pmtiles/3.4.0/KFhLlWVwzsoZmsd89qTP3/pmtiles-3.4.0.tar.gz does not appear to be a Python project, as neither pyproject.toml nor setup.py are present in the directory

Workflow: https://github.com/opengeos/leafmap/blob/master/.github/workflows/docs-build.yml#L40
pmtiles: https://pypi.org/project/pmtiles/3.4.0/#files
Install pmtiles on a local computer:

This looks like a cache bug (https://github.com/astral-sh/uv/issues/7485) can you upgrade to a newer version of uv?

@zanieb Many thanks for your prompt response! Update uv to v0.4.16 solves the issue.


use the plugin with users

Changed the source code so the application can work with users. Now the application stores the credentials in the database on the user model. This requires the model to have the following fields:

exact_accessToken
exact_refreshToken
exact_tokenExpires
exact_authorisationCode

Hi, I have created a new config exact_multi_user which is default to no. If you set the variable inside your .env file to true the plugin will store the credentials on the user object instead of the json file.

Since the package is still in development, I'm gonna merge this without testing (for now). Thanks for your work & input. I hope I can get back to you soon!

I think we at least need tot add the user table migration as well.


How can I use my own images for prediction?

Could you please tell me which file's code I need to use in order to predict the superquadric primitives of my own images?

Before making the prediction, what processing do I need to perform on my image?

Please tweak the content of tools/evaluate.py however you'd like to feed your own image instead of from the SPE3R dataset.
You can check core/dataset/build.py to see what kind of transformations are applied to image inputs during pre-processing.

I try a simply way without changing the code：
1、make a new folder in spe3r dataset
2、add this folder to splits.csv and set it to the first of test
3、SPLIT=test
4、python tools/evaluate.py --modelidx 0 --imageidx 0 --cfg experiments/config.yaml --split ${SPLIT} --save_dir ${SAVE_DIR} MODEL.NUM_MAX_PRIMITIVES ${NUM_PRIM} MODEL.PRETRAIN_FILE ${PRETRAIN}


Got java.lang.NoSuchMethodError: java.nio.ByteBuffer.flip() testing the version 1.2.11-rc-2

When testing the version 1.2.11-rc-2
We saw an exception triggered from https://github.com/jmxtrans/jmxtrans-agent/blob/master/src/main/java/org/jmxtrans/agent/StatsDOutputWriter.java#L231
The root cause is exactly https://stackoverflow.com/questions/61267495/exception-in-thread-main-java-lang-nosuchmethoderror-java-nio-bytebuffer-flip
which JDK version was used to compile and release into maven? if possible, can we add the --release 8 flag to avoid the issue

Thanks @endoplasmicR for the fix https://github.com/jmxtrans/jmxtrans-agent/commit/66c8e01991169bbb667ff5f51a766445cdbdb068
Can you please test v1.2.11-rc-3 ?

Thanks for merging!
I have just test it again and still got the same error, i have tried a different fix and looks like it got it resolved. sending a PR now

The fix is in PR:
https://github.com/jmxtrans/jmxtrans-agent/pull/157
this should ensure cross version compatibility

Fix in https://github.com/jmxtrans/jmxtrans-agent/releases/tag/jmxtrans-agent-1.2.1


This action was disabled due to block from instagra, action [Like] - IPV6

Has anyone found any solution for ipv6? every time I send any action I get this message "This action was disabled due to block from instagram!"
I've seen a staff saying that in ipv4 it's rolling normally, I'm thinking of using a rotating proxy
Does anyone else have another solution?
Thank you!

i think it's not up to ipv6.

It's not only the IPs, I've tried many IPV4 and IPV6 providers as well.
Please refear to this: https://github.com/huttarichard/instagram-private-api/issues/330


Highlights Flickering

Should on_invalidate_range/clear_highlights be called on every event?

@tiagovla it is only called when the language server tells me to refresh. Can you check whether the LSP actually calls this method https://github.com/theHamsta/neovim/blob/5099653ae1089e9180c560b7a4172674d0fa5398/runtime/lua/vim/lsp/handlers.lua#L25-L27 (e.g. by removing the handler). Maybe we should consider to not set this handler. It does not flicker for the language server I use (clang, sumneko_lua, rust-analyzer).

@jdrouhard did you ever experience something similar?

@tiagovla it is only called when the language server tells me to refresh. Can you check whether the LSP actually calls this method https://github.com/theHamsta/neovim/blob/5099653ae1089e9180c560b7a4172674d0fa5398/runtime/lua/vim/lsp/handlers.lua#L25-L27 (e.g. by removing the handler). Maybe we should consider to not set this handler. It does not flicker for the language server I use (clang, sumneko_lua, rust-analyzer).

I checked and it's not the on_refresh.

What is the language server you're using?

Pylance

EDIT: I see it calls on_invalidate_range/clear_highlights on cursor move. Will check the upstream PR later. You should maybe link this issue upstream that on_invalidate_range/clear_highlights is called to often. Actually it should only clear when it already has the highlight so the flickering is strange...

It is indeed the on_invalidate_range/clear_highlights. Clangd works fine for me (it doesn't call it on every event).

I was counting the number of times on_invalidate_range was called using clangd.

I encountered a similar thing with rust_analyzer. If I add or remove multiple lines in quick succession, there are multiple in-flight token requests. The tick check fails for the first response, which leaves the buffer un-highlighted momentarily, but then the 'correct' response arrives, and highlighting is correct again.
For now all I did was move the invalidate call below the error checking if, which seems to solve my problem. "Leftover" highlights are not really a problem IMO, but a stricter way to do this would be to only call invalidate if there's an actual error, since with the tick check it can be assumed that the response with the correct tick is on the way.

@tiagovla , you should probably check this recent patch and reopen the issue if you still see the flicker.

@tiagovla , you should probably check this recent patch and reopen the issue if you still see the flicker.

Funnily enough, I was checking it right now. It did fix it.


Hyper-V SR-IOV document

These instructions already exist in 'Hyper-V OOB Setup' files under Documentation.

Review from all stakeholder needs to be completed. Hence closing this PR.


different between demo website and python code

Hi,
I use your demo web site to extract a CNN news, but I found the results are different.
The url is http://edition.cnn.com/2014/03/27/sport/brooklyn-nets-owner-russia/index.html
By your demo website, it will show

Pro basketball is a staple of American athletics....

But I use python to get text by "Article.text", it will show

Story highlights Russian billionaire owner of Nets considers ownership move to Russian jurisdiction
The move is "not in any way tying it to the current political situation," the owner's rep said
Obama signed an order that can target Russian companies vital to its economy
Pro basketball is a staple of American athletics, taking its place with Major...

Did I use the wrong method?
My python is 3.6 and newspaper3k is 0.2.2.
from newspaper import Article

def news_extract(news_url):
    article = Article(news_url, fetch_images=False,)
    article.download()
    article.parse()
    text = article.text
    print(text)

source = 'http://edition.cnn.com/2014/03/27/sport/brooklyn-nets-owner-russia/index.html'
news_extract(source)

Thank you

Hey, I think you might want to call article.nlp() and then access the content via article.summary?

Hi,
I want to get the main news content only.
From the demo web site, it can show what I want.
But when I use Article.text, it not only shows main news content but also shows story highlights.
By Article.text

story highlights
Story highlights Russian billionaire owner of Nets considers ownership move to Russian jurisdiction
The move is "not in any way tying it to the current political situation," the owner's rep said
Obama signed an order that can target Russian companies vital to its economy
Pro basketball is a staple of American athletics, taking its place with Major League Baseball and the National Football League as one of the pre-eminent U.S. sports leagues. And it's big business, too, pumping tens of millions of dollars into the U.S. economy.
So the owner of the Brooklyn Nets raised eyebrows this week when he said he planned to transfer ownership of the team to a company in Russia -- at a time when tension between the U.S. and Russia is at its highest level since the end of the Cold War.
That raised two questions: Will it happen? Does it matter?
The billionaire owner, Mikhail Prokhorov, Monday reiterated his year-old intention to transfer the Brooklyn Nets ownership base to Russia, but has since toned down his rhetoric.

By demo web
'Pro basketball is a staple of American athletics, taking its place with Major League Baseball and the National Football League as one of the pre-eminent U.S. sports leagues. And it\'s big business, too, pumping tens of millions of dollars into the U.S. economy.\n\nSo the owner of the Brooklyn Nets raised eyebrows this week when he said he planned to transfer ownership of the team to a company in Russia -- at a time when tension between the U.S. and Russia is at its highest level since the end of the Cold War.\n\nThat raised two questions: Will it happen? Does it matter?\n\nThe billionaire owner, Mikhail Prokhorov, Monday reiterated his year-old intention to transfer the Brooklyn Nets ownership base to Russia, but has since toned down his rhetoric.\n\n"This is a long process which may or may not come to fruition and nothing is imminent," Prokhorov\'s company, Onexim Sports and Entertainment, which owns the NBA basketball team, said in a statement released via the Brooklyn Nets. "Of course, no steps in this direction could or would be taken without the full knowledge and approval of the NBA."\n\nOn Monday, the 48 year-old Russian national told reporters in Moscow, "I have already stated that I will transfer the basketball club (under the Russian jurisdiction)," according to Russian news agency ITAR-Tass.\n\nNot so fast, the NBA said, telling CNN the transfer has yet to begin.\n\n"The Nets are owned by Mikhail Prokhorov through a U.S.-based company," Mike Bass, executive vice president of NBA communications, told CNN. "We have received no official application, nor is there a process under way through our office to transfer the ownership of the Nets to another company."\n\nIf an application to change ownership was submitted by the Nets, 75% of NBA team owners -- or 23 of 30 -- would have to approve the move, Bass said. He said that the NBA\'s Board of Governors (the 30 team owners) would have to decide whether there are any issues with the transfer of ownership application.\n\nCurrently, the NBA has no rules prohibiting a foreign company from owning an NBA team.\n\nThe back-and-

@moveurbody I think that the demo website runs on an older version of newspaper (on the python-2 branch). This should explain the different results you're getting...

@yprez found the root cause for your inconsistency. It's sad that this is a regression and we should also work on improving extraction accuracy. I am closing this and separately investigating how to improve accuracy.

Thanks for filing @moveurbody