Graph not generate

I'm using exactly the same example from the plume-getstart-basicusage, however, when I try to export the cpg use driver.exportGraph("./graph3.json"), it only include the node META-DATA.
How should I fix it?
Just run with 'mvn clean compile exec:java'

4.0.0
de.noshak.plume_ex
plume-example-maven
1.0-SNAPSHOT

11
11
de.noshak.plume_ex.Tinker

io.github.plume-oss
plume
0.0.3

io.github.plume-oss
cpgconv

io.github.plume-oss
plume

jcenter
https://jcenter.bintray.com/

The documentation needs an update since the release of Plume 0.3.0 :laughing: Also, Plume has moved away from Bintray to Jitpack since Bintray announced that hosting on jcenter will no longer be free after a certain point this year.
Here is an example repository that I have update to the latest version of Plume and have tested to make sure the JSON works: https://github.com/plume-oss/plume-examples. Let me know if you have any issues with this.

The documentation needs an update since the release of Plume 0.3.0 😆 Also, Plume has moved away from Bintray to Jitpack since Bintray announced that hosting on jcenter will no longer be free after a certain point this year.
Here is an example repository that I have update to the latest version of Plume and have tested to make sure the JSON works: https://github.com/plume-oss/plume-examples. Let me know if you have any issues with this.

I got it, thanks so much for your reply! I would try it soon.

The documentation needs an update since the release of Plume 0.3.0 😆 Also, Plume has moved away from Bintray to Jitpack since Bintray announced that hosting on jcenter will no longer be free after a certain point this year.
Here is an example repository that I have update to the latest version of Plume and have tested to make sure the JSON works: https://github.com/plume-oss/plume-examples. Let me know if you have any issues with this.

Thanks so much for your help and it works.
Could I ask another question about plume TinkerGraph?
When I generate the cpg, I find that only includes node typr, doesn't have the node content/value.
For instance, here is a statement 'int a = 5'
Parse it into cpg, 'a' node is a variable and have a type 'indentifier' and value 'a'. However, the result now only have the type 'identifier', toghther with some other info.
Is there any way to generate the node value together with the cpg? Or could I do anything to achieve this?
Thanks so much for your help!

So currently Plume is still in the process of migrating to the standard ShiftLeft compliant CPGs. In your case the statement int a =5 would produce something like:
IDENTIFIER(a)-REF->LOCAL(a)
CALL("<operator>.assignment")-AST->IDENTIFIER(a)
                                                   -AST->LITERAL(5)

Whether this is correct or not I am in the process of sorting out and making it similar to Ocular/Joern output

Hi David, I'm sorry to bother you again, however, I meet the same issue again when I try to check your comment before about the output structure.
For this time, I pull the repo plume-examples and run ./gradlew tinkerGraphApp in the root folder.
However, there is still only one line with label 'META_DATA' generated in xml and 10 lines in json for the Basic1.java, without the real cpg nodes and edges.
What's wrong with that and how should I fix that?

Okay so I've just upgraded Plume version and added the logging dependencies (so now you can set it to debug logging too). I have performed a clean pull and ran ./gradlew tinkerGraphApp then ran cat graph.xml and I see loads of lines etc.
If you still run into this issue then could you post your logs set to debug? The configuration can be found under src/main/resources/log4j2.xml. Set the following:
        <Logger name="io.github.plume.oss" level="info" additivity="false">
            <AppenderRef ref="console"/>
        </Logger>

to
        <Logger name="io.github.plume.oss" level="debug" additivity="false">
            <AppenderRef ref="console"/>
        </Logger>

I tried to do so as you suggested and the result is shown in following pictures:

Basically, it told me no class found in the example file.

That is a really weird file name - are you running on an M1 Mac? I know that if a source file is given then it is first compiled, moved to /tmp/plume/build, analyzed, and then the temp files are cleaned again. Perhaps the architecture might be responsible for complications. I should also add logging on which files are compiled and loaded.
Plume is ready for another small release so when that is up I'll add better logging to help debug the problem.

No, I'm running on MBP-2016 and MacOS Big Sur v11.2.3
Does the source file have to be able to be compiled?
I'm going to try it on windows also.

So Plume only analyzes the bytecode so for convenience, if it is given a source file it will attempt to get the system's JDK to compile it. All the unit tests run Linux

Note that this is sensitive to the file extension

@jz48 Just following up, you manage to figure out the issue?

Hi David, I try it on ubuntu and it works. The example does generate the cpg with nodes and edges! There is no doubt that it works.
On the other hand, I find that (maybe) it could only deal with the compilable source file. For others, it would generate an empty cpg case compile fail.
Thanks so much for you patient and kindly answer and reply! Wish you could make it batter and better!

Thank you! I'm happy to hear you got it sorted. I think if anything I should add Windows and OSX to the CI/CD!

@jz48 This PR: https://github.com/plume-oss/plume/pull/126 fixes the issue. Essentially the temporary directories given back by Java did not end in a file separator in Linux but did in Mac and Windows - super weird! But this did highlight a few more issues further down the line anyway but now Plume tests on these two OS's now too. Release v0.3.7 is on the pipeline now and should be released soon to address this!


Wrong project reference resolution on Release mode

Opened on CodePlex by dungpa
Visual F# Tools' project system resolves project references to Debug mode although the current project is on Release mode. Here is an excerpt of FSharpProjectOptions that shows the issue:
GetProjectCheckerOptions: ProjectFileName: C:\VFPT\tests\FSharpVSPowerTools.Tests\FSharpVSPowerTools.Tests.fsproj, ProjectFileNames: [|"C:\AppData\Local\Temp.NETFramework,Version=v4.5.AssemblyAttributes.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\ProjectParser.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\ExternalProjectProvider.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\TestHelpers.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\MockDTE.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\Mocks.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\VsTestBase.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\SyntaxConstructClassifierTests.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\ResolveUnopenedNamespaceSmartTaggerTests.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\HighlightUsageTaggerTests.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\FormattingTests.fs";
 "C:\VFPT\tests\FSharpVSPowerTools.Tests\DepthTaggerTests.fs"|], FSharpProjectOptions: [|"-o:obj\Release\FSharpVSPowerTools.Tests.dll"; "--debug:pdbonly";
 "--noframework"; "--define:TRACE";
 "--doc:bin\Release\FSharpVSPowerTools.Tests.XML"; "--optimize+";
 "--platform:anycpu";
 "-r:C:\Program Files (x86)\Microsoft Visual Studio 10.0\Common7\IDE\PublicAssemblies\EnvDTE.dll";
 "-r:C:\Program Files (x86)\Microsoft Visual Studio 10.0\Common7\IDE\PublicAssemblies\EnvDTE80.dll";
 "-r:C:\VFPT\packages\Foq\lib\net45\Foq.dll";
 "-r:C:\VFPT\packages\FSharp.Compiler.Service\lib\net45\FSharp.Compiler.Service.dll";
 "-r:C:\Program Files (x86)\Reference Assemblies\Microsoft\FSharp.NETFramework\v4.0\<IP_ADDRESS>\FSharp.Core.dll";
 "-r:C:\VFPT\packages\FSharp.ViewModule.Core\lib\net45\FSharp.ViewModule.Core.Wpf.dll";
 "-r:C:\VFPT\src\FSharpVSPowerTools.Core\bin\Debug\FSharpVSPowerTools.Core.dll";
 "-r:C:\VFPT\bin\FSharpVSPowerTools.dll";
 "-r:C:\VFPT\src\FSharpVSPowerTools.Logic\bin\Debug\FSharpVSPowerTools.Logic.dll";

Notice that *.Core.dll and *.Logic.dll refer to their  bin\Debug folders.
It sounds strange but this affects Visual F# Power Tools as a client because we rely on resolution information from Visual F# Tools' project system.
Downstream bug report  https://github.com/fsprojects/VisualFSharpPowerTools/issues/790

comments
latkin wrote Nov 25, 2014 at 5:37 PM [x]
Can you provide a repro solution?

This now definitely needs a repro solution, as per the updated CONTRIBUTING.md guidelines.  We should close for now I guess
@dungpa - can you provide a repro solution?

The issue appears to be intermittent. Feel free to close.
I'll request to reopen the issue if I find a definite repro.


[release-4.13] OCPBUGS-18556: operator catalogs from oc-mirror fail to deploy because of invalid caches

This PR is a manual cherrypick of:

#651- commit dba4a0cfd0a9fd29c1e4b5bc1da737e1153cc679
#676 - commit 00c48bb918216619941134925565af2f0f8bda34

It allows for the cache to be regenerated upon catalog rebuild, so that the cache is valid in regards to the newly filtered declarative config and passes cache-integrity check upon catalog startup on the cluster.

/jira refresh

/lgtm

/label backport-risk-assessed

/label qe-approved

/label cherry-pick-approved

/jira refresh


Accessing Nested Objects in JSON - Example should match challenge

Challenge Accessing Nested Objects in JSON -
The example code does match the actual challenge very closely, which causes questions.
The example should be updated to be Array->Object->Array.

I would like to work on this as my first contribution

@paycoguy Take a look at the contributing guide to get started. There's also the Help Contributors Gitter page if you need help. Good luck!

Is this issue supposed to reference Accessing Nested Arrays in JSON instead?  The Array->Object->Array"= reference makes more sense to me on the Accessing Nested Arrays in JSON challenge.

You are correct:
http://www.freecodecamp.com/challenges/accessing-nested-arrays-in-json
The one with "myPlants" - the example does not match the challenge.


Properly handle @example jsdoc

Treat it as preformatted similar to markdown fence?

added the logic to block overflow, did not fix block underflow

This is untested but the issue should be resolved at this point. The block overflow logic does the state transition detection and updating of state and both places check for the state.


Use formatted numbers for members count

For the list of members we should format the number and show "1,000" or "1k" instead of "1000". Here:

Ideally we should use vue-i18n to do the correct formating instead of numeral, if might give a way to keep a transalation message with plural and singular and give locale formating for example in French it would return "1 000"

hello want to ask, let's say the member is 1000000 and then we choose locale french, do you want it to show 1 000 000 or 1 m?

We should show like the voting power in the votes, 1k, 1m

i see thank you for the explanation, but using vue-i18n right instead of
numeraljs?
On Tue, Sep 21, 2021, 7:19 PM Sam @.***> wrote:

hello want to ask, let's say the member is 1000000 and then we choose
locale french, do you want it to show 1 000 000 or 1 m?
We should show like the voting power in the votes, 1k, 1m
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/snapshot-labs/snapshot/issues/667#issuecomment-923929559,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5B4AN3PFWQZTKCBZ7GQ23UDBZ5PANCNFSM5EMTYOMQ
.

i see thank you for the explanation, but using vue-i18n right instead of numeraljs?

I'm not sure how that would work, I would just us numeraljs

because in the description bonustrack said that we should use vue-i18n instead

Maybe you can confirm with Bonustrack, I'm not exactly sure what he intended to achieve.

i see, thanks anyway
On Tue, Sep 21, 2021, 8:38 PM Sam @.***> wrote:

because in the description bonustrack said that we should use vue-i18n
instead
Maybe you can confirm with Bonustrack, I'm not exactly sure what he
intended to achieve.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/snapshot-labs/snapshot/issues/667#issuecomment-923998734,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5B4AMDHFF65AY6TERH7YDUDCDFHANCNFSM5EMTYOMQ
.


code_transformers tests failing on VM as Dart SDK is not present

 Issue by blois
Originally opened as dart-lang/sdk#17030

The build does not have the Dart SDK present.
Repro steps:
 - Clear out the build folder
 - python tools/build.py --mode=release --arch=ia32,x64 runtime
 - python tools/test.py -mrelease --copy-coredumps --write-debug-log --write-test-outcome-log -t60 pkg/code_transformers/test/resolver_test

 Comment by sethladd

Hi, Is this issue still valid?

Added Pkg-CodeTransformers label.

 Comment by sethladd

Removed Priority-Unassigned label.
Added Priority-Low label.


PARQUET-3031: Support to transfer input stream when building ParquetFileReader

Rationale for this change
Support to transfer the parquet file inputstream when building the ParquetFileReader, so that we can re-use the existing inputstream and reduce the open file rpcs.
What changes are included in this PR?
As title.
Are these changes tested?
Existing UT. It only a new constructors.
Are there any user-facing changes?
No break change.
Closes #3031

On many file systems, a seek backwards to read the data after reading the footer results in slower reads because the fs switches from a sequential read to a random read (which typically turns off pre-fetching and other optimizations enabled in sequential reads).
It might be worth considering if reusing the stream is worth it.

Thanks parthchandra for the comments.
For our company internal managed spark, we reuse the inputstream for parquet file.
Before that:
A spark task will open the file multiple times to read footer and data.
When the HDFS nameNode is under high pressure, it will cost time.
After that, it only open the parquet file for one time.

This is the testing 3years ago on Spark-2.3.

It reduces 3/2  hdfs RPC requests to namenode.
And after this Spark patch in community [https://github.com/apache/spark/pull/39950]([SPARK-42388][SQL] Avoid parquet footer reads twice in vectorized reader),  the solution might reduce 1/2 hdfs RPC requests.

It looks reasonable to me and users can choose their best fit.
cc @gszadovszky @steveloughran

Thank you all.

For the record, I merged @Fokko 's Parquet 1.15.0 PR to Apache Spark repository.
To @turboFei and @wangyum , if you want, you can make a PR to use this new technique in Apache Spark as illustrated in the above comments.
Thank you!


Cleanup HTML

General cleanup
Make everything work with percentages instead of constant pixel sizes.
Load Time ? https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage
Redundant? https://github.com/goktug97/org-roam-server/blob/85523cfbad2a347a20039817e4c281451b841f81/index.html#L191

https://github.com/goktug97/org-roam-server/commit/3147e12a9e49810d9f33c8d714ab0e1db828683b


Fix Radio in RTL

Fix Radio in RTL direction

After doing some quick research on components, I found some issues about supporting RTL, which I will Add to this PR.

Can we close this in favor of #334?

Yes


Database entry modification tab not removing "annotations" and "href" in light editing mode

title

plain_text and text.link also


Update nginx.conf

Fixed correct number of spaces on line 48. Also added two lines after 48, with line 50 being the portion to allow the Nextcloud app to connect properly.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Use new router's @RouteConfig syntax

There is a new router for Angular 2.0 (and 1.4) - https://github.com/angular/router
It would be good if angular2-now used the same syntax instead of @State
@Component({})
@View({
  template:
    `

`,
  directives: [RouterOutlet, RouterLink]
})
@RouteConfig({
  path: '/user', components: {
    master: 'userList',
    detail: 'user'
  }
  // or:  path: '/user', component: UserComp
})
class MyComponent {}

Thanks for the suggestion. I'm currently using ui-router in my apps and
haven't had the chance yet to try the new Angular 1.4 router.
I'll investigate the syntax and see what I can do.
On 21 May 2015 at 13:42, Nicholas Albion<EMAIL_ADDRESS>wrote:

There is a new router for Angular 2.0 (and 1.4) -
https://github.com/angular/router
It would be good if angular2-now used the same syntax instead of @State
@Component({})
@View({
template:

,
directives: [RouterOutlet, RouterLink]
})
@RouteConfig({
path: '/user', components: {
master: 'userList',
detail: 'user'
}
// or:  path: '/user', component: UserComp
})class MyComponent {}
—
Reply to this email directly or view it on GitHub
https://github.com/pbastowski/angular2-now/issues/4.

There's an example here.  It took me a while to figure out that the UserComp is actually a wrapper of the decorated class.
For a page component:
@Component
@View({ template: 'Page 1'})
class Page1 {}

If you want to refer to it from an application Component, you need to wrap it:
import {Page1} from 'demo/Page1';
import {Page2} from 'demo/Page2';

@Component
@View({ template: '

'})
@RouteConfig([
   {path: '/page1', component: {default: Page1}}
   {path: '/page2', component: {default: Page2}}
])
class MyApp {}

default means that the Component applies to the default ng-viewport.  The example in my first post on this issue shows how you'd use multiple outlets at one URL (you might update the header separately to the main content area)
I also had to configure the $componentLoaderProvider to use the Component class, instead of a string:
jobapp.config(function ($componentLoaderProvider) {
  $componentLoaderProvider.setCtrlNameMapping(function (component) {
    return component;
  });

  $componentLoaderProvider.setComponentFromCtrlMapping(function (component) {
    return component;
  });

  $componentLoaderProvider.setTemplateMapping(function (component) {
    return component.template || component.templateUrl;
  });
});

This is working for me:
export function RouteConfig(instructions) {
  if (!instructions || !(instructions instanceof Array))
    throw new Error('@RouteConfig: must provide one or more route instructions');

  return function (target) {
    for (var i = 0; i < instructions.length; i++) {
      var instruction = instructions[i];
      var component = instruction.component;
      if (undefined !== component && typeof component !== 'string') {
        component = component.name.replace(/Controller$/, '');
        component = component[0].toLowerCase() + component.substr(1);
        instruction.component = component;
      }
    }

    angular.module(currentModule)
        .run(['$router',
          function ($router) {
            $router.config(instructions);
          }
    ]);
  }
}

...example usage:
import {Home} from './home/home-controller';
import {Page1} from './page1/page1-controller';

@Component({
  selector: 'my-app',
})
@View({
    templateUrl: 'components/app.html'
})
@RouteConfig([
  {path: '/', redirectTo: '/home'},
  {path: '/home', component: Home, as: 'home'},
  {path: '/page1', component: Page1, as: 'page1'}
])
class MyApp {
}

// also provide `home.html` in same directory
// In Angular 2, I suppose this would actually be a @Component, but the @View is implied by the name
@Controller({
    name: 'HomeController'
})
export class Home {
}

Nice work.
Can you chat on gmail/skype now?

I think I just skyped you.

Closing. I do not have enough time to integrate the new router the way I would like to at the moment.
If you need routing please consider using ui-router, which works with the @State annotation.


Pathway WP2909 cannot be read, due to BioPax issue

  pathway: WP2909	111761
[   609.244] Info:  Start reading the XML file: /tmp/wp-cache/Arabidopsis thaliana/WP2909.gpml
[   609.289] Info:  Recognized format [Namespace: prefix "" is mapped to URI "http://pathvisio.org/GPML/2013a"]
Warn:  Unknown property: IRIS_ID
Error: Biopax element [Element: <bp:PublicationXref [Namespace: http://www.biopax.org/release/biopax-level3.owl#]/>] ignored
	java.lang.IllegalArgumentException: Property IRIS_ID is not valid for The gene for fragrance in rice PubMed http://rdlibrary.app.pmi//doc_num.php?explnum_id=70021.: Property IRIS_ID is not valid for The gene for fragrance in rice PubMed http://rdlibrary.app.pmi//doc_num.php?explnum_id=70021.
java.lang.IllegalArgumentException: Property IRIS_ID is not valid for The gene for fragrance in rice PubMed http://rdlibrary.app.pmi//doc_num.php?explnum_id=70021.
	at org.pathvisio.core.biopax.BiopaxNode.addProperty(BiopaxNode.java:57)
	at org.pathvisio.core.biopax.BiopaxNode.loadXML(BiopaxNode.java:153)
	at org.pathvisio.core.biopax.BiopaxNode.fromXML(BiopaxNode.java:142)
	at org.pathvisio.core.biopax.BiopaxElement.refresh(BiopaxElement.java:80)
	at org.pathvisio.core.biopax.BiopaxElement.setBiopax(BiopaxElement.java:294)
	at org.pathvisio.core.model.GpmlFormat2013a.mapBiopax(GpmlFormat2013a.java:949)
	at org.pathvisio.core.model.GpmlFormat2013a.mapElement(GpmlFormat2013a.java:443)
	at org.pathvisio.core.model.GpmlFormatAbstract.readFromRoot(GpmlFormatAbstract.java:589)
	at org.pathvisio.core.model.GpmlFormat.readFromXmlImpl(GpmlFormat.java:182)
	at org.pathvisio.core.model.GpmlFormat.readFromXml(GpmlFormat.java:121)
	at org.pathvisio.core.model.Pathway.readFromXml(Pathway.java:734)
	at org.wikipathways.wp2rdf.WPREST2RDF.main(WPREST2RDF.java:235)

which pathvisio version are you using?

Not with PathVisio, but with GPMLRDF. However, because the problems seems to be in org.pathvisio.core, I filed it in this repo. It may also be automatically resolved with the upgrade. I just tried it with PathVisio 3.3, and that does not seem to have problems loading the pathway, but a similar exception does show up in the logs:
Error: Biopax element [Element: <bp:PublicationXref [Namespace: http://www.biopax.org/release/biopax-level3.owl#]/>] ignored
        java.lang.IllegalArgumentException: Property IRIS_ID is not valid for Removal of feedback inhibition of delta 1-pyrroline-5-carboxylate synthetase, a bifunctional enzyme catalyzing
the first two steps of proline biosynthesis in plants. PubMed http://rdlibrary.app.pmi/do
c_num.php?explnum_id=120751.: Property IRIS_ID is not valid for Removal of feedback inhibition of delta 1-pyrroline-5-carboxylate synthetase, a bifunctional enzyme catalyzing the first
two steps of proline biosynthesis in plants. PubMed http://rdlibrary.app.pmi/doc_num.php?
explnum_id=120751.
java.lang.IllegalArgumentException: Property IRIS_ID is not valid for Removal of feedback inhibition of delta 1-pyrroline-5-carboxylate synthetase, a bifunctional enzyme catalyzing the firs
t two steps of proline biosynthesis in plants. PubMed http://rdlibrary.app.pmi/doc_num.ph
p?explnum_id=120751.
        at org.pathvisio.core.biopax.BiopaxNode.addProperty(Unknown Source)
        at org.pathvisio.core.biopax.BiopaxNode.loadXML(Unknown Source)
        at org.pathvisio.core.biopax.BiopaxNode.fromXML(Unknown Source)
        at org.pathvisio.core.biopax.BiopaxElement.refresh(Unknown Source)
        at org.pathvisio.core.biopax.BiopaxElement.setBiopax(Unknown Source)
        at org.pathvisio.core.model.GpmlFormat2013a.mapBiopax(Unknown Source)
        at org.pathvisio.core.model.GpmlFormat2013a.mapElement(Unknown Source)
        at org.pathvisio.core.model.GpmlFormatAbstract.readFromRoot(Unknown Source)
        at org.pathvisio.core.model.GpmlFormat.readFromXmlImpl(Unknown Source)
        at org.pathvisio.core.model.GpmlFormat.readFromXml(Unknown Source)
        at org.pathvisio.core.model.Pathway.readFromXml(Unknown Source)
        at org.wikipathways.client.WikiPathwaysClient.toPathway(WikiPathwaysClient.java:214)
        at org.pathvisio.wpclient.WikiPathwaysClientPlugin.openPathway(Unknown Source)
        at org.pathvisio.wpclient.WikiPathwaysClientPlugin$2.doInBackground(Unknown Source)
        at org.pathvisio.wpclient.WikiPathwaysClientPlugin$2.doInBackground(Unknown Source)
        at javax.swing.SwingWorker$1.call(SwingWorker.java:295)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at javax.swing.SwingWorker.run(SwingWorker.java:334)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)
Warn:  Unknown property: IRIS_ID

The PublicationXref elements contained bp:IRIS_ID elements which do not match the schema. These elements were removed in 2023: https://classic.wikipathways.org//index.php?pwTitle=Pathway%3AWP2909&diff=125975&oldid=117228.


update dependencies: Scala 2.12.12, 2.13.3, guava 23.0, bcprov 1.66

To keep dependencies up to date, this repo could be added to https://github.com/scala-steward-org/repos/blob/master/repos-github.md and let scala-steward do his thing :)

@lemastero thanks for the PR! On steward, will check, thanks!


REST API  in Flask

Type
REST API using Flask
Description
Build a REST API to expose the trained model, we will POST an image to /predict endpoint and we need class and confidence as response.
Tools

Flask
Keras/Tensorflow/PyTorch
REST API
Modeling

Have you read the Contributing Guidelines on Pull Requests?
Yes
Note: One can use the test script which can be found in models/ dir, pretrained models can be found in the same dir.
Reference: Flask APP for OCR

I would like to work on this issue.
Please assign me this issue.

Sure @sameersrivastava13, I am assigning you to work on this. If possible, test your API endpoint from the React client app. You have 5 days to create a PR but earlier the merrier.

Hey @macabdul9 I want to try out this issue. Please assign me..

And also I want some more reference material if you can provide..😅

Hey @macabdul9 I want to try out this issue. Please assign me..

yeah you can ask and take this issue as I will be not active for few days. All the best.

@kritikaparmar-programmer you can work on this, I expect a PR within the stipulated time. You should check out this repo reddit-flair-detector for client server architecture.


Long running stream query connection being reallocated before it's done

I have a long-running stream query (swag 10 minutes) on the stack  in many threads (one query per account fwiw). Under the consumer of those queries, there are other standard queries. For some reason, Hikari seems to be allocating still in use connections from the streaming queries to the downstack queries thus triggering

java.sql.SQLException: Streaming result set com.mysql.cj.protocol.a.result.ResultsetRowsStreaming@6541fef9 is still active. No statements may be issued when any streaming result sets are open and in use on a given connection. Ensure that you have called .close() on any active streaming result sets before attempting more queries.

The error, of course, doesn't give enough information for me to know for certain that it isn't a query which my code thinks it finished; however, since the streaming query uses use I believe it will close the connection when it's done.
        database.executeExpression(expression) { statement ->
            statement.fetchSize = Integer.MIN_VALUE
            statement.executeQuery().use { rs ->
                while (rs.next()) {
                    action(rs)
                }
            }
        }

Any contrary hypotheses?
My gut is to create a one time connection outside the connection pool per streaming query as the overhead is dwarfed by the query run time (streaming on the order of a million rows). Any how-to for that or better ideas?

Ah, simple cause not so simple fix.
Default behavior of Hikari or ktorm is reuse any open connection in the call stack rather than create a new connection (or get one from the pool). So, I need to have my streaming call basically forbid the reuse. It's not obvious how.

@dmitchell can you wrap a query in a transaction? Semantic of transaction is to forbid connection re-use while it's active.

I wonder if this is somehow related to the fact that action is a suspend function, and ktorm+hikairi release current connection at suspension point

Thank you @IRus for that explanation. It clarifies the cause. I've gotten around it by not using Hikari for the streaming query and making a connection per query. the overhead is minuscule compared to everything else.


FaceRecognition.Create working on memory (taking byte[] as parameter)

Hello Takeuchi-san.
I have created a face recognition system using your wonderful work, thank you very much.
Only, my enterprise works with microservices, and in order to implement the f.r. system into one I would need to work independently of the file system. There, I stumble upon FaceRecognition.Create, which only takes a folder path as input.
I've been looking into it and basically what Create does is deserialize all 4 model files using DlibDotNet. Deserialize method in DlibDotNet.Dnn.LossMmod.cs and .Dnn.LossMetric.cs both take a file, check if it exists and turn it into a byte array, then they do the proper deserializing.
I was wondering, could it be possible to create an overload for these Deserialize methods that takes directly a byte array, and extend that up to the Create method, so that FaceRecognition.Create(byte[] resnetModel, byte[] faceDetector, byte[] 5faceLandmarks, byte[] 68faceLandmarks) could be used as an overload (along with the already existing method print)?
I really love your work and would like to keep using it as you publish it, without internal tinkering.
Thank you again.

@ADD-eNavarro
Can not agree with your suggestion.
But I can add the following interface
var model = new Model {  ResNet = File.ReadAllBytes("test.dat"), ... } ;
var fr = FaceRecognititionDotNet.Create(model);

Because

I do NOT want to add new argument if FRDN support new model file

However

DlibDotNet does not support deserialize from byte array

So we have to update DlibDotNet at first.
Thanks.

Hi again!
Your solution seems right to me.
Are you going to open PR in DlibDotNet to implement deserialization from byte array?

@ADD-eNavarro
Yes. Just start to implement deserialize.
https://github.com/takuya-takeuchi/DlibDotNet/tree/deserialize
But I should take a long time for long long build time.
Because DlibDotNet.Native and DlibDotNet.Native.Dnn uses numerous template class.
Please be patient.

Hello again.
I will be patient, I know good work takes time.
But my boss wants me to sketch a roadmap for the deployment, so I was wondering: would it be possible to have some idea of how long are we talking about? Weeks, months? Any information will be very useful.

@ADD-eNavarro
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/commit/9a3f39aa4df48576e5366ea105889caf5bb2552d
It could work on win and Linux.
osx can not pass test due to dependencies.
So I can't publidh nuget package now.

https://github.com/takuya-takeuchi/FaceRecognitionDotNet/wiki/Quickstart#1-initialize

Hello again.
With a colleague we're trying to create the NuGet package from the development branch, but we get an error, missing DlibDotNet native dlls. So we were wondering if you could help on this matter, maybe even create the NuGet package yourself from this branch (and yes, we are aware it wouldn't work on osx).

@ADD-eNavarro

but we get an error, missing DlibDotNet native dlls.

You can pick them from nuget file.
https://www.nuget.org/packages/DlibDotNet/

@ADD-eNavarro
Release <IP_ADDRESS>
https://github.com/takuya-takeuchi/FaceRecognitionDotNet/releases/tag/<IP_ADDRESS>


API Get Account return Credit Card Value is NULL

Credit Card can be null

I have vule in credit card, but return value is null

issue spotted


Bug: Searching doesn't work with "path" option

When using the path option, searching doesn't work anymore. No Error, no Results.
Example:
// schema
const AccountModel = new Schema("test", {
    address: { type: "string", path: "$.config.address" }
});

const repo = new Repository(AccountModel, client);
await repo.createIndex();

// add 2 examples
await repo.save({
    address<EMAIL_ADDRESS>});
await repo.save({
    address<EMAIL_ADDRESS>});

// search
const s = repo.search().where("address").eq("*test*");
const accounts = await s.return.all();

console.log("ACCOUNTS s", s);
console.log("ACCOUNTS", accounts.length);

prints:
[DEBUG] ACCOUNTS s [
  <ref *1> Search {
    schema: Schema {},
    client: Client {},
    sortOptions: undefined,
    rootWhere: WhereString {
      negated: false,
      search: [Circular *1],
      field: Field {},
      value: '*test*'
    }
  }
]
[DEBUG] ACCOUNTS [ 0 ]

-> 0 Results
But when I remove path: "$.config.address"
I get all 2 results:
[DEBUG] ACCOUNTS s [
  <ref *1> Search {
    schema: Schema {},
    client: Client {},
    sortOptions: undefined,
    rootWhere: WhereString {
      negated: false,
      search: [Circular *1],
      field: Field {},
      value: '*test*'
    }
  }
]
[DEBUG] ACCOUNTS [ 2 ]

When using path, the JSON document you save has to have that path in it. So, you need to call save like this:
await repo.save({
  config: {
    address<EMAIL_ADDRESS>  }
} );

Before you specified path, the path was defaulting to $.address, which is why is worked before.

you are right, of course!
It confuses me, that everything is saved anyway, no matter what schema.
Thank you @guyroyse for your help!


Solana resources

https://dev.to/edge-and-node/The-complete-guide-to-full-stack-solana-development-with-react-anchor-rust-and-phantom-3291

Hi @rishiyadav1923 , is this a suggestion to include this article?

https://dev.to/edge-and-node/The-complete-guide-to-full-stack-solana-development-with-react-anchor-rust-and-phantom-3291

This page doesn't exist anymore, @rishiyadav1923.

the link doesn't work.


After install multi format extensions, "editor.formatOnSave" base on which one?

VSCode Version: 1.25.1
OS Version: macOS 10.13.6

Steps to Reproduce:

install extensions: "Beautify" and "Prettier - Code formatter"
then change settings "[javascript]": { "editor.formatOnSave": true }

Does this issue occur when all extensions are disabled?: No
So when I save .js file, the auto format sometime is "Beautify" sometime is "Prettier"
I wanna assign one to format my file, but I cannot find the setting.
Thanks.

Yeah, that's a know problem and it depends on the order in which the extensions are activated. We have blogged about this https://code.visualstudio.com/blogs/2016/11/15/formatters-best-practices and the question is why do you need two formatters for one language? Allowing multiple formatters is tracked here: #41882


LIS 4.2.3-5 is current now

@steveesp LIS update...

@shirgall : Thanks for your contribution to the Azure documentation! The author, @steveesp, has been notified to review your proposed change.

#sign-off @shirgall @jimdial


more: use dev tty instead of mio to avoid panics

Fixes #2362

Can you test this out with the filedescriptor feature rather than the use-dev-tty? The filedescriptor is better documented in crossterm overall.

Can you test this out with the filedescriptor feature rather than the use-dev-tty and see if it works the same? The filedescriptor is better documented in crossterm overall.

I tried filedescriptor feature and it did not fix the problem. According to the crossterm code filedescriptor feature enabled via use-dev-tty and use-dev-tty itself used as feature flag to opt-out mio as event source on unix.

Alright! I will add docs and file a PR for the use-dev-tty since it isn't documented in the README or lib.rs.

Thanks for your PR :)


Extremely slow training when compared to Detectron2 model

Hi. I have been using detectron2 to train the instance segmentation model. But I want to try and use mmdetection to improve the training/inference speed.
However, on the contrary the training has been extremely slow. The model that took 1 day for training in detectron2 is taking 33 days on mmdetection. The number of iterations and batch size are set to the same value
I have shared the configuration that I have used both in the mmdetection and detectron2 below
detectron2 config
from detectron2.config import get_cfg
cfg = get_cfg()
cfg.merge_from_file(model_zoo.get_config_file("COCO-InstanceSegmentation/mask_rcnn_R_50_FPN_3x.yaml"))
cfg.DATASETS.TRAIN = (train_dataset_name,)
cfg.DATASETS.TEST = (val_dataset_name,)
cfg.DATALOADER.NUM_WORKERS = 2
cfg.MODEL.WEIGHTS = model_zoo.get_checkpoint_url("COCO-InstanceSegmentation/mask_rcnn_R_50_FPN_3x.yaml")
cfg.SOLVER.BASE_LR = learning_rate
cfg.SOLVER.WEIGHT_DECAY = 0.01  # Increase weight decay

cfg.SOLVER.IMS_PER_BATCH = batch_size

# Set your configuration settings here
cfg.SOLVER.LR_SCHEDULER_NAME = "WarmupCosineLRWithRestarts"
cfg.SOLVER.BASE_LR = learning_rate
cfg.SOLVER.MAX_ITER = iterations
cfg.SOLVER.WARMUP_ITERS = int(iterations * 0.1)
cfg.SOLVER.WARMUP_FACTOR = 0.001
cfg.SOLVER.COSINE_RESTART_INTERVAL = int(iterations / 5)  # Set the restart interval (in iterations) between the cycles
cfg.SOLVER.COSINE_END_LR = 0  # Set the minimum learning rate value after annealing

cfg.MODEL.ROI_HEADS.BATCH_SIZE_PER_IMAGE = 128
cfg.MODEL.ROI_HEADS.NUM_CLASSES = len(thing_classes)  # Set number of classes
cfg.MODEL.ROI_HEADS.SCORE_THRESH_TEST = min_object_detection_confidence_inference

# output folder
cfg.OUTPUT_DIR = str(output_dir)

# training code
trainer = CustomTrainer(cfg)
trainer.resume_or_load(resume=False)
trainer.train()

mmdetection config
# Imports
import mmcv
import mmengine
from mmdet.apis import init_detector, inference_detector
from mmdet.utils import register_all_modules
from mmengine import Config
from mmengine.runner import Runner

# config and checkpoint
config_name = 'mask_rcnn/mask-rcnn_r50_fpn_2x_coco.py'
checkpoint_name = 'mask_rcnn_r50_fpn_2x_coco_bbox_mAP-0.392__segm_mAP-0.354_20200505_003907-3e542a40.pth'

# load model and config
model = init_detector(config_path, checkpoint_path, device='cuda:0')
cfg = Config.fromfile(config_name)

# update the config
from mmengine.runner import set_random_seed

# Modify dataset classes and color
cfg.metainfo = {
    'classes': segmentation_labels,
    'palette': [(190, 77, 37), (37, 150, 190), (220, 20, 60), (190, 150, 60)]
}

# Modify dataset type and path
cfg.train_dataloader.dataset.ann_file = str(train_json)
cfg.train_dataloader.dataset.metainfo = cfg.metainfo

cfg.val_dataloader.dataset.ann_file = str(val_json)
cfg.val_dataloader.dataset.metainfo = cfg.metainfo

cfg.test_dataloader = cfg.val_dataloader

# Modify metric config
cfg.val_evaluator.ann_file = str(val_json)
cfg.test_evaluator = cfg.val_evaluator

# Modify num classes of the model in box head and mask head
cfg.model.roi_head.bbox_head.num_classes = len(segmentation_labels)
cfg.model.roi_head.mask_head.num_classes = len(segmentation_labels)

# We can still the pre-trained Mask RCNN model to obtain a higher performance
cfg.load_from = checkpoint_path

# Set up working dir to save files and logs.
cfg.work_dir = 'workdir'

# We can set the checkpoint saving interval to reduce the storage cost
cfg.default_hooks.checkpoint.interval = 3
cfg.default_hooks.logger.interval = 10

# Set seed thus the results are more reproducible
set_random_seed(0, deterministic=False)

# Training code
runner = Runner.from_cfg(cfg, )
runner.train()

Can anyone help in understanding what is slowing down the training in mmdetection
More details about the detectron2 code is available at https://pastecode.io/s/40if5x0w

Hi @hhaAndroid Can you please look into this issue whenever you have some time. Thanks

Experiencing a similar issue. I wanna train rtm-ins on RTX 3090 4GPUs. But, it takes 18 days.. :( What's wrong?

Hi @Wooho-Moon . What I realized is that number of iterations
parameters in different from epochs.

Could you provide more detail?? I wanna know more precisely!

Why I confused is that RTMDet-ins use EpochBasedTrainLoop class in mmengine according to debug. Max_iter parameter is used in IterBasedTrainLoop, but it cannot be used un EpochBasedTrainLoop. Am I wrong?

I think there is a promblem here.
This code is in mmengine.runner.loop.py
outputs = self.runner.model.train_step( data_batch, optim_wrapper=self.runner.optim_wrapper)
if I use single gup, the excution time is 0.6~1.0 s. But, when I use mult gpu, the excution time is around 2sec. It's really wired.


Bootstrap Dropdowns and Popovers call parent element's hide function

When a Bootstrap Dropdown item is selected or a Popover closed, the popped up content is closed. However, a call is also made to the hide() function of the parent:

      <button class="btn btn-secondary dropdown-toggle" type="button" id="dropdownMenuButton" data-toggle="dropdown">
        Dropdown button
      </button>

        Action
        Another action
        Something else here

    <button id="Button" type="button" class="btn btn-lg btn-danger" data-toggle="popover" data-placement=bottom title="Popover title" data-content="And here's some amazing content. It's very engaging. Right?">Click to toggle popover</button>

In the above code, messages will appear on the console when the dropdown or popover are dismissed. Looking at the stack trace, the actual call to the hide function is coming from jQuery.
This happens in both Bootstrap 3 and 4.
Is there anyway to prevent this from happening? If not, is there a way to detect if hide() is being called from jQuery?
Here's a jsfiddle: https://jsfiddle.net/ghenne/yeo3nh17/

I don't understand, what's your issue here ? What's the bad behavior you want to show us ? Because if you don't and unknown methods to your DOM elements you won't have any issue here 😟
Our plugins always throw an hide and hidden events so you'll be able to detect when hide is called.

In that case, is there any way to detect that the hide was called by Bootstrap?
I can then get Button.hide to execute its code only if it isn't called by Bootstrap.

you only have to listen to our hide or hidden events to execute your code when our internal hide method is called.
For example:
$('#myPopover').on('hide.bs.popover', function () {
 // your code
})

Thanks - that works.


[Event Request] Codeunit 32000000 "Ref. Payment Management" - SetLinesOnBeforeGenJnlLineInsert

A new Event Publisher is need in the Codeunit 32000000 "Ref. Payment Management".
Event:
local procedure SetLinesOnBeforeGenJnlLineInsert(CustLedgEntry: Record "Cust. Ledger Entry"; var GenJnlLine: Record "Gen. Journal Line")
Code
[Scope('OnPrem')]
procedure SetLines(RefPaymentImported: Record "Ref. Payment - Imported"; JnlBatchName: Code[20]; JnlTemplateName: Code[20])
begin
                    if AccCode <> '' then
                        GenJnlLine.Validate("Bal. Account No.", AccCode)
                    else
                        GenJnlLine.Validate("Bal. Account No.", CustLedgEntry."Bal. Account No.");
                    SetLinesOnBeforeGenJnlLineInsert(CustLedgEntry,GenJnlLine);
                    GenJnlLine.Insert(true);
                    LineNro := LineNro + 10000;
                    LastDocNro := IncStr(LastDocNro);
                end;
            until RefPaymentImported.Next = 0;
end;

Need to update few fields from CustLedgEntry to GenJnlLine in the SetLines Procedure related to Combine SEPA Payments. The new event should be invoked before GenJnlLine.Insert(true); code. Could you please add the requested Publisher?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Fix add payment card doesn't disappear after adding a card

Explanation of Change

Fixed Issues

$ https://github.com/Expensify/App/issues/52564
PROPOSAL: https://github.com/Expensify/App/issues/52564#issuecomment-2478148847
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Log into an account that has the free trial expired and doesn't have a payment card
Navigate to Concierge/Expensify chat where they're being asked to add a payment card
Press add payment card button
Add the card, go back to the same chat, and verify the button no longer shows up

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/user-attachments/assets/68b707f1-a736-45ad-ad81-ab08c801e0d4

iOS: Native

iOS: mWeb Safari

https://github.com/user-attachments/assets/25a3b1cb-e9cb-45da-a129-add28a7367df

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/a92627f5-4900-497f-9c6d-e5a19b4a729f

MacOS: Desktop

Yeah, I'm also running out of account to test this PR 😂

little bump @AndrewGable

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Error during npm init riot

`npm init riot
npm WARN exec The following package was not found and will be installed: create-riot
This utility will walk you through creating a package.json file.
It only covers the most common items, and tries to guess sensible defaults.
See npm help init for definitive documentation on these fields
and exactly what they do.
Use npm install <pkg> afterwards to install a package and
save it as a dependency in the package.json file.
Press ^C at any time to quit.
package name: (apps)
version: (1.0.0)
description:
git repository:
keywords:
author:
license: (ISC)
About to write to C:\projects\riot\apps\package.json:
{
"name": "apps",
"version": "1.0.0",
"main": "index.js",
"scripts": {
"test": "echo "Error: no test specified" && exit 1"
},
"author": "",
"license": "ISC",
"description": ""
}
√ Please select a template · webpack

Downloading the template files

It was not possible to download the template zip file
Error details:
C:\Users\rudin\AppData\Local\npm-cache_npx\939675aa98f97328\node_modules\create-riot\src\utils.js:38
throw new Error(error)
^
Error: StatusCodeError: 404 - {"type":"Buffer","data":[52,48,52,58,32,78,111,116,32,70,111,117,110,100]}
at panic (C:\Users\rudin\AppData\Local\npm-cache_npx\939675aa98f97328\node_modules\create-riot\src\utils.js:38:9)
at downloadFile (C:\Users\rudin\AppData\Local\npm-cache_npx\939675aa98f97328\node_modules\create-riot\src\utils.js:84:5)
at processTicksAndRejections (node:internal/process/task_queues:93:5)
at async download (C:\Users\rudin\AppData\Local\npm-cache_npx\939675aa98f97328\node_modules\create-riot\src\index.js:27:19)
at async Proxy.main (C:\Users\rudin\AppData\Local\npm-cache_npx\939675aa98f97328\node_modules\create-riot\src\index.js:109:19)
npm ERR! code 1
npm ERR! path C:\projects\riot\apps
npm ERR! command failed
npm ERR! command C:\Windows\system32\cmd.exe /d /s /c create-riot
npm ERR!      C:\Users\rudin\AppData\Local\npm-cache_logs\2021-06-14T09_53_08_551Z-debug.log
`

I was local problem. I run cmd under administrator, npm cache clean --force and npm install -g create-riot and all works. Thanks!


VRTK_InteractUse.ManageInteractTouchListener is never used

Environment

Source of VRTK: Github
Version of VRTK: fc51ae54c5b8b11bdbd9c6b7d551c4a58bdb47e6
Version of Unity3D: 5.4.4f1
Hardware used: Vive
SDK used: SteamVR

Steps to reproduce
This is a code problem, but also probably an issue when overriding the use button on Interactable Objects. Therefore one should test the example scenes and test whether overriding the use button on the IO works correctly (see below).
Expected behavior
VRTK_InteractUse.ManageInteractTouchListener is used or removed from the code base.
Current behavior
VRTK_InteractUse.ManageInteractTouchListener is never used. It looks like it should be, though. Since the listener method it adds to the VRTK_InteractTouch script's events do some stuff with regard to the used button for using.
I guess overriding the use buttons in VRTK_InteractableObject is currently broken because of this.

Note: I'm logging this here to remember @thestonefox. I've not checked whether the mentioned override bug is really there.

Fixed with https://github.com/thestonefox/VRTK/commit/80520da1c0821cdfc5239652686828e192cea73c


Allow open classes to be mocked

Just added a change so that open classes can be mocked for there corresponding open functions.

Can you give use cases for this?
So far, we've intentionally restricted the mocking feature to interfaces.
The reason is that you can only mock abstract / open methods, and therefore only track & verify mocked methods.
Let's consider this class:
open class Foo {
    fun bar() {}
    open fun baz() {}
}

This test will succeed:
@Test
fun testFoo() {
    val foo = MockFoo(mocker)

    mocker.verify {
        foo.bar() // Foo.bar is not open, so it cannot be intercepted by the mocker.
    }
}

In this example, instead of verifying that Foo.bar was actually called, the method is simply called.
The code is correct, but it is perceived wrongly. Even though Foo.bar was never called before the verify block, the test succeeds.
By allowing to mock open classes, we give to the developer the responsibility of tracking which methods are open (and therefore tracked by the mocker), and which are not. This really feels like a recipe for hard-to-debug errors and tests that succeed when they really shouldn't.
Finally, this opens the possibility for lazy developers (which we are all) to mark a method open just to be able to track it in tests, and I am not comfortable encouraging this bad practice.


Milestone 1: Create a data model

🎯 In this milestone, I:

created a new Rails project with a Postgres database
created migration files based on the ERD provided
generated project schema based from migration files 🚀

@AmaduKamara, thanks for the review 🙏🏽


changing now to how.

Changing now to how in a sentence which describes MTBF.

@pradeepgururani : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Good catch, thanks! 😸
#sign-off


Shortcut Guide: Extensions for programs

Description of the new feature / enhancement
The Shortcut Guide for Windows is useful, but having the ability to do the same in an application would be much cooler. The shortcuts could be stored in separate files on GitHub that the PowerToy can get when updated.
Scenario when this would be used?
Open Microsoft Word and be able to see the shortcut keys in that program to be able to bold text, insert table etc.
The same could go for Adobe products like PhotoShop.
Supporting information
No response

/dup #890


Create endpoint for 1-time address generation using xPub

Summary
Create an endpoint which lets a user pass in an xpub and hdpath and return the address in legacy and cash address format.
Details

GET /address/fromXPub/:xpub/:hdPath

Example: GET /address/fromXPub/xpub661MyMwAqRbcG4CnhNYoK1r1TKLwQQ1UdC3LHoWFK61rsnzh7Hx35qQ9Z53ucYcE5WvA7GEDXhqqKjSY2e6Y8n7WNVLYHpXCuuX945VPuYn?hdPath=m/0/0
Then in the REST codebase do the following to get an address from an xPub and hdPath:
let node = BITBOX.HDNode.fromXPub('xpub661MyMwAqRbcG4CnhNYoK1r1TKLwQQ1UdC3LHoWFK61rsnzh7Hx35qQ9Z53ucYcE5WvA7GEDXhqqKjSY2e6Y8n7WNVLYHpXCuuX945VPuYn')

let childNode = BITBOX.HDNode.derivePath(node, "m/0/0")
BITBOX.HDNode.toCashAddress(childNode)
// bitcoincash:qqz7p9mklzzz7fhqnfltduxr4xx6dnjffyymecxfgd
// also return in legacy format
BITBOX.HDNode.toLegacyAddress(childNode)
// 1Y5Rrx5CFAPM1LGancbtxerAfBxoV9VrR

Of course you'll want to get the xpub and the hdPath from the req and not hardcoded like it is there.
Request originally from Ekliptor

What’s the current status on 1-time address generation using xPub via http://bitcoin.com  rest api?

I realized we can't use BIP44 because it requires hardened children and XPub can't create hardened children. So we're defaulting to an hdpath of 0 w/ the option to pass in a different hdpath as a query string param.
This will be live in the next REST deploy


Clicking the Run app! button is running the old rather than the new version

Pop!OS pop-os_amd64_intel_45.iso
Old version stored at read-only location /isodevice/Applications/Subsurface-x86_64.AppImage
Running AppImageUpdate-x86_64.AppImage 6fbc9e9
Gui seems to do something and suggests application has been updated to the latest version
Where is the latest version stored? It does not show the filename nor location of the new file
Clicking the Run app! button shows that is not the latest version (screenshot) - it is running the old rather than the new version

me@host:~$ Qulo/AppImageUpdate-x86_64.AppImage /isodevice/Applications/Subsurface-x86_64.AppImage
AppImageUpdate version 1-alpha (commit 6fbc9e9), build 122 built on 2017-11-12 00:58:28 UTC
XOpenIM() failed
Checking for updates...
Fetching release information for tag "continuous" from GitHub API.
... done
Starting update...
Fetching release information for tag "continuous" from GitHub API.
Updating from GitHub Releases via ZSync
zsync2: Reading seed file: /isodevice/Applications/Subsurface-x86_64.AppImage
zsync2: Usable data from seed files: 32.641444%
zsync2: Renaming temp file
zsync2: Fetching remaining blocks
zsync2: Downloading from https://github-production-release-asset-2e65be.s3.amazonaws.com/2319498/25016776-c70a-11e7-82a9-201d837e0259?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20171112%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20171112T063052Z&X-Amz-Expires=300&X-Amz-Signature=be5edc9f6eeba986778e15c31dbb3ade0d1b75fb0f3cdeb5c220918b38550ee2&X-Amz-SignedHeaders=host&actor_id=0&response-content-disposition=attachment%3B%20filename%3DSubsurface-4d04f7431-x86_64.AppImage&response-content-type=application%2Foctet-stream
zsync2: Verifying downloaded file
zsync2: checksum matches OK
zsync2: used 23252992 local, fetched 47984640
Update successful

On the command line, the situation is somewhat better:
me@host:~$ Qulo/appimageupdatetool-x86_64.AppImage /isodevice/Applications/Subsurface-x86_64.AppImage
Checking for updates...
Fetching release information for tag "continuous" from GitHub API.
... done!
Starting update...

Fetching release information for tag "continuous" from GitHub API.
0% done...
Updating from GitHub Releases via ZSync
0% done...
zsync2: Subsurface-4d04f7431-x86_64.AppImage found, using as seed file
zsync2: Reading seed file: /isodevice/Applications/Subsurface-x86_64.AppImage
19.5233% done...
zsync2: Reading seed file: Subsurface-4d04f7431-x86_64.AppImage
96.7198% done...
zsync2: Usable data from seed files: 100.000000%
zsync2: Renaming temp file
zsync2: Fetching remaining blocks
zsync2: Verifying downloaded file
100% done...
zsync2: checksum matches OK
zsync2: used 71237632 local, fetched 0
100% done...
Update successful. New file created: Subsurface-4d04f7431-x86_64.AppImage
me@host:~$ ls
Subsurface-4d04f7431-x86_64.AppImage
Subsurface-4d04f7431-x86_64.AppImage.zs-old

Has nothing to do with the location of the original file being read-only. Can be easily reproduced like this:

Download https://github.com/Subsurface-divelog/subsurface/releases/download/v4.7.3/Subsurface-6f65f5191-x86_64.AppImage  (v4.7.3)
Use AppImageUpdate on it
Click "Run app!"
Check the about box - we are still running v4.7.3 despite that Subsurface-a157fa5c6-x86_64.AppImage (v4.7.4) has been downloaded

Should be fixed already. Please retry.


Cdb 267 disable controls

Issue related 267
zoomControl was no working. I fixed it, and write tests (I tried to they were unit tests)
Also, there was an error about naming in the documentation. It's search, and no searchControl (I guess it was an error because zoomControls was near).
Please, @javisantana , could you review it?

@javisantana I updated documentation here, because there's no doc about cartodb.js in CartoDB/docs repo.
What do you think about last commit? I'm not completely sure about this (specifically in this.mobile)

please @javisantana , what do you think about this pr?

:es:


Add CONSOLE_JSON for deploying to GCP K8S env

Add json console logging
Summary
Added CONSOLE_JSON to start dump the logs as UTC timestamp and as json.
Change Type
Please delete any irrelevant options.

New feature (non-breaking change which adds functionality)

Testing
Before:

After:

Checklist
Please delete any irrelevant options.

My code adheres to this project's style guidelines
I have performed a self-review of my own code
I have commented in any complex areas of my code
I have made pertinent documentation changes
My changes do not introduce new warnings

I noticed you added docker build scripts, can you explain them a bit? I don't have issues with including but noticed you didn't mention them.
Thanks for the PR!

I noticed you added docker build scripts, can you explain them a bit? I don't have issues with including but noticed you didn't mention them.
Thanks for the PR!

Yep, I use these to build the docker and push it to the custom Google GCP docker registry. You can pass arguments where to push. I know there are scripts to do this in GitHub Actions, but these are for the official repo and cannot be easily used to push to a custom registry.


PublicIP: extend farm data with available public pools

The farm object need to be extended to:

Add list of ip range
An  ip range can be defined as

A full subdomain
List of pubic IPs
Single IP

Each pool also has a list of reserved IPs which is defined as a map to reservation ID that holds ownership of this IP.

Is it better to query this information by aggregating all the reserved IPs from the database ? This is a heavy task to do to validate the free IPs each time.

API Endpoints
A farmer can manage his pools as follows

Base URL /api/v1/farms/<farmid>
POST <base-url>/ranges add a new range
GET <base-url>/ranges (not needed since this is part of the farm object after all)
DEL <base-url>/ranges/<id> (validate that are no ip used from that range)

Part Of: https://github.com/threefoldtech/home/issues/925

An ip range can be defined as

A full subdomain
List of pubic IPs
Single IP

I think as far as the explorer goes, we should just care about a list of public IP's in the data model and the api endpoints. This keeps operation simple. Client side tools can easily expand a subnet to a list op IP's (in case anyone ever adds a giant subnet). And a single IP is just a list of IP's with 1 element.
That way the data model can just be something extremely simple like:
[
  {
    "addr": "xxx.xxx.xxx.xxx",
    "used": false
  },
  {
    ...
  }
]

Can't we just use farm-update endpoint for all this since we just wanna keep a list of IP's on a farmer object?

You could I guess, but keep in mind that for deletion of IP's you must first validate that they are not currently in use. So every update will need to first check that the field of IP addrs is the same as it used to be, and if not, check that any removed ip is not used. Also, since the GET will return if an ip is used or not, the POST will also include this property, meaning you need to check that existing IP values are not flipped, and new IP's are in used == false state. Or you can make the POST take a list of IP's and modify this to the object on the backend, but that means that a client will need to reduce the object to a list of IP's under the hood as well

@LeeSmet I don't get what you suggest ? When I say the farmer needs to have a list of iprange defined, it can be a subnet or a subnet of single IP
<IP_ADDRESS>/24
<IP_ADDRESS>/32

then we get the 'used' ips part
which is just a list of all ips used from that farmer ?
So clients can expand the ranges, and find the free spots


Package names are too generic

All of the packages in these libraries are overly generic with respect to the Adobe domain space, and more qualifications are needed to ensure collisions don't occur when in a shared classloader with other artifacts.
For example:
Workspace is rooted at com.adobe.
AIOException is rooted at com.adobe.exception
At the least, additional package name of aio should be added to ensure separation and protection from conflicts/collisions. e.g.:
com.adobe.aio...
Is it slightly redundant? Yes, but given that large volume of packages within the AEM space exist in the com.adobe space, it would help prevent future issues.

agreed ! I'll resolve this asap

done


GitHub Actions has encountered an internal error when running your job.

Description
GitHub Actions has encountered an internal error when running your job.

https://github.com/pindoio/pindo-cli/actions/runs/842362071

Area for Triage:
python
Question, Bug, or Feature?:

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[+] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Image version

Image version where you are experiencing the issue.
Expected behavior
It should run build and test
Actual behavior
GitHub Actions has encountered an internal error when running your job.
Repro steps
A description with steps to reproduce the issue. If you have a public example or repo to share,
please provide the link.

https://github.com/pindoio/pindo-cli/actions/runs/842362071

https://www.githubstatus.com/incidents/plk7zdvpftby

Hello @kenessajr,
It seems there were some issues with GitHub environment, and now it should works as expected. Could you please confirm that everything is fine at the moment?
We are looking forward to your reply.

I'm going to close the issue until further updates.
In case of any questions, feel free to contact us.

@Darleev I'm still having the same issue. it was working a couple of hours ago. The error https://github.com/pindoio/pindo-cli/actions/runs/853659656

@kenessajr it's another github action incident: https://www.githubstatus.com you check check it here.

Just saw it!


Invoke-DbaWhoIsActive fails when used with Verbose switch

Bug Report

General Troubleshooting steps

[ Y] Verified running the latest release of dbatools?
[ Y] Verified errors are not related to permissions?
[ Y] Can duplicate in new/clean PowerShell session (clean = powershell -NoProfile)?

Steps to Reproduce

When the command Invoke-DbaWhoIsActive is called using the Verbose switch
eg Invoke-DbaWhoisActive -SqlInstance LensmanSB -ShowSleepingSpids 2 -Verbose
The command fails with error: Invalid query.
This is a result of the Verbose switch being evaluated as a Passed Parameter.
While there is a check that each parameter matches values in the $paramdictionary the null values are not filtered and a Null Parameter with Value of True is passed to the SQL Command

Looks like you already fixed this in PR #3203. Marking this closed.


Link activeClassName not working.

Hi, I'm trying to use ActiveClassName to style my links, but the property is not being processed, and is instead being passed down to the native DOM. Here's my code:

						<Link activeClassName="active" href="/admin/home">Home</Link>
						<Link activeClassName="active" href="/admin/pages">Pages</Link>
						<Link activeClassName="active" href="/admin/media">Media</Link>
						<Link activeClassName="active" href="/admin/themes">Themes</Link>
						<Link activeClassName="active" href="/admin/plugins">Plugins</Link>

And the resultant HTML:
HomePagesMediaThemesPlugins

I as on /admin/home when I copied the above HTML.
Please let me know if you need any more information.

Possibly related to #365 ?

Any updates on that? I have the very same issue.

Any updates?  Same problem here.

I see the same behaviour. Link from match does not have activeClassName applied when on href route.

As per this comment: https://github.com/preactjs/preact-router/issues/333#issuecomment-543839970, this is fixed for me if I use import { Link } from 'preact-router/match'; rather than import { Link } from 'preact-router'; (preact-router v3.2.1)

The active class with Link from match doesn't work for SSR pages.

<Link> from preact-router/src/index.js should just reexport from preact-router/match/src/index.js


RabbitMQ: use int64 for pageSize to avoid conversion problems

Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes https://github.com/kedacore/keda/issues/2318

/run-e2e rabbitmq*


Refactor agent setup guide

This PR aims to improve the agent's documentation.
Link to GH Pages site

The opening comment, I think could use a few tweaks.

Welcome to the JF Agent Documentation! The agent is a tool used to connect to your systems to give you back metrics and other data to help your organization. If you are looking to set up your agent for the first time, see here.

I'm not the greatest at copy.. so maybe even this could use some help.

Welcome to the JF Agent Documentation! The agent is an on-premise configuration tool you can use to send data to Jellyfish in order to power the metrics and insights that will help your organization. You can use this documentation to guide you in setting up, or troubleshooting.

As per https://jelly-ai.slack.com/archives/CPERQK8TV/p1625665714233400 decided to merge GH pages work


Guided assembly w/ GENCODE GTF results in large, single-exon transcripts spanning entire mRNA

When I use the -G option, the StringTie output GTF will contain many large single-exon transcripts that span an entire gene (example attached - w/ -G is top, w/o -G is bottom model). The reference GTF I am using comes from the GENCODE website. Have you ever seen this issue? These long exons are introduced in both assembly and merge mode with the -G option while the de novo assembly mode never has this issue. Any thoughts on why this might be the case and how I can remedy?
With best regards,
J.T.

We've had problems in the past with Gencode GTF having explicit "gene" features aberrantly declared with a "transcript_id" attribute, which made StringTie treat them as single-exon transcripts spanning the entire locus. I remember fixing this for recent versions of StringTie though -- what version of StringTie were you using when you got these results?

This was using the 1.3.0 Linux binary and the gencode v20 transcript model (gencode.v20.annotation.gtf).
Please let me know if it would be helpful to have portions of the output.
On Oct 31, 2016, at 10:28 AM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
We've had problems in the past with Gencode GTF having explicit "gene" features aberrantly declared with a "transcript_id" attribute, which made StringTie treat them as single-exon transcripts spanning the entire locus. I remember fixing this for recent versions of StringTie though -- what version of StringTie were you using when you got these results?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257308185, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXNwip8lweISdqp53xduqNJbGOjbnvks5q5fsLgaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.

Ouch, I was just able to reproduce this bug for Gencode v20. It turns out that the fix I thought I had there wasn't actually working for that Gencode GTF. Sorry about this, I just pushed a fix for that fix which should now allow proper parsing of these older Gencode GTF files. I prepared a patched StringTie version with this fix applied:
http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.Linux_x86_64.tar.gz
(source in the master branch or at http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.tar.gz)
If you could give it a try to confirm if it works OK now for you it would be great -- thank you!

Dear Geo,
Tested and working. You’re a wizard! Thanks for the quick fix.
Cheers,
J.T.
On Oct 31, 2016, at 10:27 PM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
Ouch, I was just able to reproduce this bug for Gencode v20. It turns out that the fix I thought I had there wasn't actually working for that Gencode GTF. Sorry about this, I just pushed a fix for that fix which should now allow proper parsing of these older Gencode GTF files. I prepared a patched StringTie version with this fix applied:
http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.Linux_x86_64.tar.gz
(source in the master branch or at http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.tar.gz)
If you could give it a try to confirm if it works OK now for you it would be great -- thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257475446, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXN9HDlXOo-KsrYa8kOVjATReKL9NAks5q5qOggaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.

Dear Geo,
Sorry to amend my previous response…
The assembly now works correctly, but long transcript length exons return once more after using the --merge step.
Which version of GENCODE and later works well?
Thanks again,
J.T.
On Oct 31, 2016, at 10:27 PM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
Ouch, I was just able to reproduce this bug for Gencode v20. It turns out that the fix I thought I had there wasn't actually working for that Gencode GTF. Sorry about this, I just pushed a fix for that fix which should now allow proper parsing of these older Gencode GTF files. I prepared a patched StringTie version with this fix applied:
http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.Linux_x86_64.tar.gz
(source in the master branch or at http://ccb.jhu.edu/software/stringtie/dl/stringtie-1.3.0b.tar.gz)
If you could give it a try to confirm if it works OK now for you it would be great -- thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257475446, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXN9HDlXOo-KsrYa8kOVjATReKL9NAks5q5qOggaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.

This is strange, the same code loads that reference annotation for the --merge option, so those gene-wide exons shouldn't be there anymore, even for the merge step. Are you sure that neither of these happen:

you had files in your merge list that were generated with the older (non-patched) stringtie ?
you ran the older (non-patched) stringtie program for the merge step?

As for your other question, I first downloaded the last Gencode GTF version (v25) to test this issue, and I noticed they no longer have those malformed "gene" feature lines with "transcript_id". I did not check any other releases between v20 and v25.

My code is shown below as a pair of Snakemake rules. They both reference the same stringtie binary and in this toy example only merge the one gtf file.
I can re-run with v25 no problem. If I can be of any more help tracking down potential issues, please let me know. Again, thanks for your help!
rule StringTie:
input: out/FCH9EFLADXX-HUMbghEAACRAAPEI-225.Aligned.sortedByCoord.out.bai, out/FCH9EFLADXX-HUMbghEAACRAAPEI-225.Aligned.sortedByCoord.out.bam
output: out/FCH9EFLADXX-HUMbghEAACRAAPEI-225-stringtie.gtf
benchmark: out/benchmarks/FCH9EFLADXX-HUMbghEAACRAAPEI-225.StringTie.txt
wildcards: sample=FCH9EFLADXX-HUMbghEAACRAAPEI-225
/ifs/e63data/poirierj/bin/stringtie-1.3.0b.Linux_x86_64/stringtie             -G /ifs/e63data/poirierj/indexes/GRCh38/gencode.v20.annotation.gtf             out/FCH9EFLADXX-HUMbghEAACRAAPEI-225.Aligned.sortedByCoord.out.bam             -p 6             -o out/FCH9EFLADXX-HUMbghEAACRAAPEI-225-stringtie.gtf
rule merge:
input: out/FCH9EFLADXX-HUMbghEAACRAAPEI-225-stringtie.gtf
output: out/all-merge/merged.gtf
benchmark: out/benchmarks/merge.txt
/ifs/e63data/poirierj/bin/stringtie-1.3.0b.Linux_x86_64/stringtie             --merge             -G /ifs/e63data/poirierj/indexes/GRCh38/gencode.v20.annotation.gtf             -o out/all-merge/merged.gtf             -p 12             -c 2.5             -T 1             -f .1             -i             out/FCH9EFLADXX-HUMbghEAACRAAPEI-225-stringtie.gtf
On Nov 1, 2016, at 12:20 PM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
This is strange, the same code loads that reference annotation for the --merge option, so those gene-wide exons shouldn't be there anymore, even for the merge step. Are you sure that neither of these happen:

you had files in your merge list that were generated with the older (non-patched) stringtie ?
you ran the older (non-patched) stringtie program for the merge step?

As for your other question, I first downloaded the last Gencode GTF version (v25) to test this issue, and I noticed they no longer have those malformed "gene" feature lines with "transcript_id". I did not check any other releases between v20 and v25.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257612386, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXN3getgPGWgtejCGgWwDFqKLAgdaHks5q52bigaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.

I noticed that you used the -i option for the merge, which may have something to do with this -- perhaps there is some (albeit faint) evidence of retained introns there.. Do you get the same transcript-wide exons when you don't use the -i option?
I would like to take a look at that single input GTF you are "merging" with the annotation there, could you please send that to me? (e-mail should be fine).

I just re-ran without the -i option and it did not help. I am attaching a screenshott of TP53 w/ the original StringTie GTF on top, the coverage data below, and then the result from the merge below that, which has an additional long exon that doesn’t seem to be read supported. I’ve also uploaded both GTFs to a google Drive for you to view here:
https://drive.google.com/open?id=0B3FkuPnNupZFbGJKTkw2MVJqd3c
Please let me know if all is OK and you can access.
Thanks again,
J.T.
<EMAIL_ADDRESS>On Nov 1, 2016, at 1:41 PM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
I noticed that you used the -i option for the merge, which may have something to do with this -- perhaps there is some (albeit faint) evidence of retained introns there.. Do you get the same transcript-wide exons when you don't use the -i option?
I would like to take a look at that single input GTF you are "merging" with the annotation there, could you please send that to me? (e-mail should be fine).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257636407, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXN2SxxvQo6nHGwSuQNYPeKLGsVL7Zks5q53nZgaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.

I don't see the same problem with the merge file -- I ran the merge myself but I also downloaded your merge output file and definitely the gene-wide single-exon transcript is not there for TP53.
So you're probably talking about a different issue now -- I guess you forgot to attach a screenshot so I don't know what "additional long exon" you are referring to (perhaps you can give me the transcript name and coordinates/number of this exon?) but it clearly seems to be a different problem from the gene-long single-exon transcript bug reported by the original post.

Actually, seeing exons not supported by coverage, but supported by reference transcripts (-G file) is normal for the merge operation.. The reference transcripts are actually merged in (assembled with compatible isoforms assembled in the samples) -- so if you were referring to such an exon, that's intended behavior for the merge operation.. If you do not want such an un-expressed exons in there, do not use the -G option with --merge. The point of the merge is to build a comprehensive and non-redudant set of putative isoforms for the purpose of their abundance estimation in each sample. This set might include isoforms that are not expressed in any of the samples (if -G was used); if an exon is not in a sample it will have zero coverage.. and the expression estimates for the related isoforms will be adjusted accordingly.

Thanks a lot for looking into this. I think that we can consider this issue closed!
J.T.
On Nov 1, 2016, at 5:03 PM, Geo Pertea <notifications<EMAIL_ADDRESS>wrote:
Actually, seeing exons not supported by coverage, but supported by reference transcripts (-G file) is normal for the merge operation.. The reference transcripts are actually merged in (assembled with compatible isoforms assembled in the samples) -- so if you were referring to such an exon, that's intended behavior for the merge operation.. If you do not want such an un-expressed exons in there, do not use the -G option with --merge. The point of the merge is to build a comprehensive and non-redudant set of putative isoforms for the purpose of their abundance estimation in each sample. This set might include isoforms that are not expressed in any of the samples (if -G was used); if an exon is not in a sample it will have zero coverage.. and the expression estimates for the related isoforms will be adjusted accordingly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/gpertea/stringtie/issues/75#issuecomment-257696474, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGTXN-TfAasZPh-SVsHii0gr3Nh8Mn3qks5q56k_gaJpZM4KlDpb.
=====================================================================
 Please note that this e-mail and any files transmitted from
 Memorial Sloan Kettering Cancer Center may be privileged, confidential,
 and protected from disclosure under applicable law. If the reader of
 this message is not the intended recipient, or an employee or agent
 responsible for delivering this message to the intended recipient,
 you are hereby notified that any reading, dissemination, distribution,
 copying, or other use of this communication or any of its attachments
 is strictly prohibited.  If you have received this communication in
 error, please notify the sender immediately by replying to this message
 and deleting this message, any attachments, and all copies and backups
 from your computer.


Cryptocoins don't reset to correct count

Sometimes cryptocoins reset to an amount smaller than what was actually spent.
Seems to be inconsistent but happens often.
Seems to occur more often with multiple upgrades selected.

Same problem here. @ochadenas

corrected a dubious calculation, but I'm not quite sure if that fixes the problem. Will have to take a closer look.

Should be fixed in 1.16.


Feature Request: specify character encodings on import and export

Original author<EMAIL_ADDRESS>(November 12, 2010 18:23:40)
Handling character encodings is a very common part of data cleansing for me.  At the basic level, this means giving the option of specifying a character encoding on both import and export.  Going further, if raw bytes are a supported type, you could offer character encoding/decoding as a column transformation.  To make it really rich you could suggest the mostly likely character encodings at import time.
Original issue: http://code.google.com/p/google-refine/issues/detail?id=198

@jackyq2015 @wetneb Don't we already handle this ?  Or only on import and we need to do more work to support giving options for changing encoding on export ?  This is an old issue and I feel we pretty much give most users what they most need regarding encoding support.  So I think we can close this, Yes ?

Agreed, closing.


Improve ClusterClaim Wait Experience

Previously claim.spec.namespace is set as soon as a cluster is assigned to the claim, but the user then needs to wait for the cluster to wake from hibernation and be fully running before it can be used.
This change adds a new ClusterClaim condition, ClusterRunning, which will be True when the cluster is ready to be used, and can be waited for with oc wait --for=condition=ClusterRunning --timeout=600s ClusterClaim myclaim.
/assign @abhinavdahiya
/cc @suhanime

Ready for review.

/lgtm


Local push isn't working anymore

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.17.0
Amplify CLI Version
10.4.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Amplify Categories
Not applicable
Amplify Commands
push
Describe the bug
This started happening out of the blue, I can't really understand or think of how to replicate it.
I tried to deploy my environment locally and this started to happen.
I used to deploy using the following command amplify push --forcePush --yes
I went to try today and I keep getting this message now:
Deployment failed.
Deploying api GraphQLAPI [ ==-------------------------------------- ] 1/23
	GraphQLAPITransformerSchema3C… AWS::AppSync::GraphQLSchema    UPDATE_COMPLETE
Deploying function CreateStripeCheckoutSession [ ----------------------------------
	LambdaFunction                 AWS::Lambda::Function          UPDATE_FAILED
Deploying function WebhookPayments [ ---------------------------------------- ] 0/5
	AmplifyResourcesPolicy         AWS::IAM::Policy               CREATE_FAILED
^[[A
Rolled back (4 of 3)
🛑 ["Index: 2 State: {\"deploy\":\"waitingForDeployment\"} Message: Resource is not in the state stackUpdateComplete"]

Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/

Is anyone able to help me? I can't get past this. Can't really get help on the troubleshooting page or can't find anything about it.
I would appreciate any help since I can't deploy through the console (https://github.com/aws-amplify/amplify-cli/issues/11292) and now I can't deploy manually. I'm 100% blocked now, thinking of alternatives from amplify because this is really slowing me down :/
Expected behavior
Make the proper deploy of my changes to the cloud
Reproduction steps
Run amplify push --forcePush --yes and this is happening.
GraphQL schema(s)

# Put schemas below this line

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response

Could you give a little more detail? Which categories are you deploying, what are the contents of your amplify/cli.json, and when you're looking at the error messages in cloudformation you should scroll down until the first error message that appeared, the screenshot above shows the recovery stages of the rollback but it doesn't show the error that occurred to cause it.
If you're willing, you may also send your amplify folder to<EMAIL_ADDRESS>

Hi @marcvberg
I end up destroying my env and recreating again. Luckily this was only on my dev, it could've been pretty bad if this was prod I can close this for now and I will re-open if happens again.
If this helps with anything in the future here is cli.json content.
{
  "features": {
    "graphqltransformer": {
      "addmissingownerfields": true,
      "improvepluralization": false,
      "validatetypenamereservedwords": true,
      "useexperimentalpipelinedtransformer": true,
      "enableiterativegsiupdates": true,
      "secondarykeyasgsi": true,
      "skipoverridemutationinputtypes": true,
      "transformerversion": 2,
      "suppressschemamigrationprompt": true,
      "securityenhancementnotification": false,
      "showfieldauthnotification": false,
      "usesubusernamefordefaultidentityclaim": true,
      "usefieldnameforprimarykeyconnectionfield": false,
      "enableautoindexquerynames": false,
      "respectprimarykeyattributesonconnectionfield": false,
      "shoulddeepmergedirectiveconfigdefaults": false,
      "populateownerfieldforstaticgroupauth": false
    },
    "frontend-ios": {
      "enablexcodeintegration": true
    },
    "auth": {
      "enablecaseinsensitivity": true,
      "useinclusiveterminology": true,
      "breakcirculardependency": true,
      "forcealiasattributes": false,
      "useenabledmfas": true
    },
    "codegen": {
      "useappsyncmodelgenplugin": true,
      "usedocsgeneratorplugin": true,
      "usetypesgeneratorplugin": true,
      "cleangeneratedmodelsdirectory": true,
      "retaincasestyle": true,
      "addtimestampfields": true,
      "handlelistnullabilitytransparently": true,
      "emitauthprovider": true,
      "generateindexrules": true,
      "enabledartnullsafety": true
    },
    "appsync": {
      "generategraphqlpermissions": true
    },
    "latestregionsupport": {
      "pinpoint": 1,
      "translate": 1,
      "transcribe": 1,
      "rekognition": 1,
      "textract": 1,
      "comprehend": 1
    },
    "project": {
      "overrides": true
    }
  },
  "debug": {
    "shareProjectConfig": false
  }
}```


fix: 🐛 Add position prop to the CornerDialogProps type

Overview
As per the documentation and the source code, the CornerDialog component should have position as an optional prop, however, that's not the case in index.d.ts.
✅  Closes: #1368
Screenshots
Current documentation page

Documentation

[x] Updated Typescript types and/or component PropTypes

Good catch! Thanks 😄


Fix scrape interval and duration tooltip not showing.

I have the same problem as #10470. I just copy this prop from another tooltip and it works. This may fix #10470.

close/reopen to trigger ci

I added a line of console.log(!scrapeTooltipOpen); to print log in the toggle function.
      <Tooltip
        isOpen={scrapeTooltipOpen}
        toggle={() => {
          setScrapeTooltipOpen(!scrapeTooltipOpen);
          console.log(!scrapeTooltipOpen);
        }}
        target={CSS.escape(id)}
        style={{ maxWidth: 'none', textAlign: 'left' }}
      >
        <Fragment>
          Interval: {interval}

        </Fragment>
        <Fragment>
          Timeout: {timeout}
        </Fragment>
      </Tooltip>

I found that when I focused on scrape duration, the console quickly printed true and false 2 lines.

May be related to this issue https://github.com/reactstrap/reactstrap/issues/1728.

@garenchan Ah thanks, that's annoying, definitely seems like some reactstrap bug. But your change makes it work better, so looks good for now :)


only trigger chart linting if charts change

What this PR does / why we need it:
only trigger chart linting if charts change
Special notes for your reviewer:
Checklist

[ ] DCO signed
[ ] Chart Version bumped
[ ] helm-docs are updated
[ ] Helm chart is tested

@dwerder ?

@pierluigilenoci ?

I was curious to understand what had happened this PR.

Now another question comes to me.
Why close one PR to open another with almost the same changes? 😆

Because this PRs were opened nearly at the same time. And the one by Stefan fixed more stuff.


[mt-source] Fix ce-source missing the namespace

Proposed Changes

Propagate source namespace to the st adapter.

Release Note

Docs

/lgtm

/test pull-knative-sandbox-eventing-kafka-integration-test-channel-consolidated


fix: bug 1896 - switching entity types needs to clear name field

Switching between entity types needs to clear the name field. Switching from pre-trained to custom for example left the pre-trained name linger in the name field. Now clear the name field, thereby requiring a new name be supplied.

Reworked logic. Name field cleared if previous type was prebuilt if type is changed. Further, restrict names prefixed with "builtin-" as users enter the name. Third, preserve the name when possible.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: