Bujo: Record Time not behaving as expected

Im using Bujo in conjunction with Dendron. I have a markdown file open that contains a time track table. I can insert a task using Bujo: Schedule Entry command. When I place the cursor on the row of the task I am starting and run the command Bujo: Record Time the time is inserted into the "Tracker" column of the table. i.e., current_time- e.g., "14:30-"
However when I run Bujo: Record Time it does not behave as demonstrated here and here
Instead it results in current_time-previous_time- e.g., "15:00-14:30-"
If I repeat it will be "15:01-15:00-14:30-" etc.
I am running the latest VScode update on Windows 10.

I found that if there was no space between current_time- or 11:30- and the pipe | then the above behaviour occurs. I had to manually place a space between the dash - and the pipe and the command worked as expected.
Also thanks for making this extension. I think it will be very useful for me.

Thanks for looking into this, @Aodhagan! The issue seems to be due to the regex I use to parse the time record. I am glad to hear that adding the space between the dash and the pipe fixed it. However, I will check to see if I can make the regex a bit more robust.

Just an FYI. As it stands Bujo: Record Time will behave correctly or as expected if the time track table is fully expanded - as in the editor window is wide enough so that each row of the time track table is on a single line of the editor window.
If you narrow the width of the editor window such that the rows of the table wrap over multiple lines of the editor then the above issue occurs.


dependent parameters

Suppose I want to define an application that takes parameters like this:
app --dir /data --in ./src/input --out ./out/result

where the meaning of -in and -out are file paths that are relative to --dir.
I would like to define the parameters --in and --out dependent on the parameter --dir so that I could have access to the value of the --dir parameter in the custom parsers for --in and --out.
In this example, the custom parser for --in would verify that the relative path, ./src/input, exists in the directory specified by --dir, ie., /data/src/input.

What this would really mean I think is the ability to lift an arbitrary effect to the parser monad, so you can write the existence check and use the fail I've just added to express this. The interesting question here is what would happen to the usage info generator.
Note that you can produce the combined directories right now, just without checking the file's existence, with something like
for {
  data <- namedParam()
  relIn <- namedParam()
  relOut <- namedParam()
} yield (data / relIn, data / relOut)

One solution of course would be to not solve this and defer the extra validation to outside of the parameter parsing, but I understand that would create more boilerplate.
Another possibility would be to provide two functions when lifting the effect, the real one and a pure simulation that can be called in the usage graph builder.
It would look something like:
case class Path(s)
case class ValidatedPath(path: Path)

def validate(path: Path): Task[ValidatedPath]

for {
  root <- namedParam[Path]()
  child <- namedParam[Path]()
  full = root / child
  validatedFull <- lift(validate, ValidatedPath(_))
} yield validatedFull

which means that during parsing we run the effect validate but for usage graph building we always assume the paths are validated.
What do you think?

Thanks for the feedback, I really like the validate approach, it seems to be more general than the use case in this issue. For example, one could use this mechanism to ensure that a combination of options is acceptable.
For example, imagine that an application could be invoked with any 2 of 3 options: -a, -b, -c.
So, -a -b, -a -c, -b -c would be OK but -a -b -c would not.
I believe that this use case could also be handled with your validate idea.


travis: test with stable and beta, but not nightly

It's probably a good thing if the spaceapi builds with the latest stable
toolchain, not only with beta / nightly.
Also I removed nightly because recently we hit a compiler bug: https://travis-ci.org/coredump-ch/spaceapi/builds/69862138, see https://github.com/rust-lang/rust/pull/26807 which fixed the error.
But maybe that's a good thing, so we uncover bugs in rustc.

Do you know, why the build is failing?

Have you looked at it? It fails to download the toolchain and timeouts after 10 minutes.

I totally agree about building on stable and beta :) Back when we introduced testing, stable was not yet available.


AO Soft Limit bug fix

When merged this pull request will:

Fix an issue where the module would effectively check AOs for every player whether or not it applied to their team.

Reported by @goosko https://github.com/Global-Conflicts-ArmA/Olsen-Framework-Arma-3/issues/110


Update to React 0.13.0

React is already at version 0.13.3, and because the peer dependency was set to 0.12.0 npm errors, because The package react does not satisfy its siblings' peerDependencies requirements!.
With this PR the React version is upped to 0.13.0.

thx @mitchellvanw. bumped and pushed to npm


settings: disable unlinking of last external account

this only applies if the user can't login to their local account
(either ACCOUNTS_LOCAL_LOGIN_ENABLED=False, or they don't have a
password set)

closes #251

of course, we could still make this behaviour optional by adding yet another configuration item, and checking for it in the relevant places


Test/add e2e pipeline

Add e2e task with mocha reporting. Outputs created as artifacts and uploaded to S3 artifacts bucket. Slack notification sent to notifications-e2e-test-results channel.
Ticket: An addendum to https://hee-tis.atlassian.net/browse/TIS21-1020

Looks really good, but we should change deploy-prod to depend on cypress-e2e

Cheers Andy. I just wondered if we wanted to prevent E2E from being a blocker to deploying to prod to avoid similar issues encountered with admins-ui.


Can't save anymore

I just came across a problem when I can't save new entries anymore, its seems that super table has some limit problem. Does someone came across this problem already ? Or know a solution ?

This can be a common problem - have you had a look at the Troubleshooting section?

Ah sorry I totally skipped the Troubleshooting section, it worked with the php.ini.


Improve your score on google pagespeed insights

Webp Image Generator
https://webpack.js.org/plugins/image-minimizer-webpack-plugin/#generator
SVG Optimizer
https://dev.to/smelukov/webpack-5-asset-modules-2o3h
https://github.com/tigt/mini-svg-data-uri


[Request] Add a Hotkey for the "Run" function

Note: for support questions, please use one of the following channels.
Reddit: /r/gbstudio
Discord: Join Chat
This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[ ] bug report
[X] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?

Feature.

What is the current behavior?

Must click the "Run" button to compile/run the emulator.

What is the expected behavior?

Use Ctrl+P (Win)/Cmd+P (Mac) to trigger Run function.

What is the motivation / use case for changing the behavior?

Editor efficiency.

Please tell us about your environment:

Version: 1.0.0
Platform: all

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

This has been present since the first public release as Ctrl/Cmd + B and is listed in the documentation  https://www.gbstudio.dev/docs/keyboard-shortcuts/ also the shortcut is listed against the menu item under the Game menu. You can also export the ROM using Ctrl/Cmd + Shift + B

Thank you. Sorry -- maybe the bug report should be for a hover element on the Run button then (the tools have them).


[Feature Request] transpose feature in matmul compute api

Is your feature request related to a problem? Please describe.
There is an argument of transpose in matmul_tiles kernel API but it seems to be unsupported properly
Describe the solution you'd like

support transpose feature both in0 and in1 in WH B0

Additional context

I wonder if there is a support plan.

It seems that the transpose feature for in1 in the matmul APIs is working properly in WH_B0.
Is there any support for this feature for in0 as well?


Response preview: right-click a value to create/save a variable

Insomnia Version: 7.0.5
Operating System: 10.15.1 Catalina

Details
I would like to be able to right-click a value in Preview pane

and save it as a variable - either new, or update the existing one.
In the above example I would update my accessToken variable after the token expires, and thus it would take me just one click to have all my acess tokens throughout all calls, updated.

Thanks for the suggestion! Going to close this as a Duplicate of #1796 so we can continue the discussion in one place.

Duplicate of #1796


Access to Handlebars.partials inside global helpers file

We want to be able to render partials from inside helpers. For example, in our global helpers file, when we try something like:
var Handlebars = require('handlebars');

module.exports = {
    something: function(options) {
        var partial = Handlebars.partials['partial-name'];
              html = partial(options.hash);
        //.......
    }
}

That partial partial-name is not available because Handlebars.partials is an empty object {}.
What is the best way to create/import/inject the Handlebars.partials object (as defined in the partialsDir option) so our helpers can instantiate them?
Thanks in advance!

This seems very much related to #101, and in PR #105 I'm now passing along the partials to the data channel during rendering making them available inside of helpers via options.data.ExpressHandlebars.partials.
Let me know if you think that will solve what you're after here, and if so, I'll close this out once #105 is merged.

Not particularly, but that's ok. The goal is to allow development of helpers and partials that work both server and clientside. Clientside, handlebars puts partials at Handlebars.partials. So, I'd still have to register them globally anyway with the proposed solution. But I will close this and potentially submit a pull request later.
Thanks for your work on this. :+1:

@tuxsudo thankyou for the hint. I was wondering how can I access Handlebars instance in my helpers file when using express-handlebars package.
this is the hack I used
// app.js

var hbs = exhbs.create({
    defaultLayout: 'main',
    helpers: require('./lib/hbs-helpers'),
    // other options
});

and in lib/hbs-helpers.js
Handlebars = require('handlebars'); // the official handlebars package
module.exports = (function(Handlebars) {
   fallback: function(actual, defaultValue) {
       var out = actual || defaultVal;
       return new Handlebars.SafeString(out); // now I can access Handlebars instance
   }
})(Handlebars);

This is a hack as I am not good with documentations so is there an integration with express-handlebars.


Publish this on brew

I created this fork because I had this issue and this feature request, but got no reaction. So i implemented both in my own fork.
@chmac came up with the idea.
The original repo seems to be unmaintained (one commit in the last five years).
Btw: I think it's already possible to install a repo (like DerZyklop/prettyping) via brew directly. But for sure it's less convenient.
I am open to publishing my fork on brew, but I am not very familiar with that topic. So if anyone is familiar, feel free to leave some hints on what to do, or pull requests if that’s necessary.

Btw: I think it's already possible to install a repo (like DerZyklop/prettyping) via brew directly. But for sure it's less convenient.

I believe you need to create your own "tap", which is brew speak for a repository. Then to install one runs the brew tap command to add your tap, and then the packages in that tap become available. I think the tap needs to be a separate repository from the main one.
Maybe it's possible to publish on brew with something like prettyping-improved as the name or so?


Identity request (ID1, ID2, ID3, ID4)

ID1:
First Address: 3nGYWT67gvbPcfcMmee7DnQWygViib21PSnESnQaT5oNcaKcDw
Second Address: 3HhNo8Ahub5Rrvp67QCs8av6uCxEddDiU4MC4CMro4LJYWGQ7J
Device info: Oneplus 6T A6013 with Android 10
File: PubID2.jpg contain public information
ID3:
Document type: Passport without real data
Nationality: North Ireland
device: Oneplus 6T A6013 with Android 10
process: I use fake passport picture followed by a fake selfie. I've tried verification but after "Verification in Progress...." (about 11mins), system give me response of  "Verification failed" - I think that this system work great

I resubmit separately


:hammer:[refractor][made changes for tests]

fix #85
@blackgirlbytes please check this pr!

looking!

changed to pnpm , hope this works !

installed pnpm locally

hey @blackgirlbytes , can you check this time?

Awesome!🎉 Thanks for all the help @blackgirlbytes🙌 !


New wine venues

Next time, please verify that the JSON file is valid (e.g. with http://jsonlint.com/) or run the application to test it before submitting PR.


Data provided by a keras.utils.sequence DataGenerator has None dimensions when used to fit Model subclass

Code is written using Google Colab on Tensorflow Version 2.4.1
A DataGenerator subclassing keras.utils.sequence is providing data of correct shape when used to fit a sequential model. However, when used to fit a custom model subclassing keras.Model the input dimensions become (None, None, None, None) in the call method during the first training epoch.
When the custom model is fit using the DataGenerator the input dimensions to the call method should be (16, 256, 128, 3).
A simplified recreation of the issue is provided on Google Colab in the following notebook:
Colab Notebook

Was able to reproduce the issue with TF v2.3, TF v2.4 and TF-nightly. Please find the gist of it here. Thanks!

cc @amahendrakar @rmothukuru

Can you try switching your input pipeline to be tf.data based instead? That should make it generally more performant anyways.

@05hazza06,
Can you please respond to the above comment? Thanks!

The tf.data dataset implementation worked as expected. Thanks.
Still unsure about why Keras sequence didn't work but the issue is sorted.


feat: throw on non-zero exit

Introduces a new throwOnError option which will cause tinyexec to
throw any time a non-zero exit code is encountered.
If the exit code is null, we will not throw since it means something
went very wrong anyway (the process is still running and shouldn't be,
since we saw the close event by then).
If the exit code is greater than 0, we will throw a
NonZeroExitError which has an exitCode property.
For example:
try {
  await x('foo', [], {throwOnError: true});
} catch (err) {
  if (err instanceof NonZeroExitCode) {
    err.exitCode; // the exit code
    err.result; // the tinyexec process
    err.result.killed; // getters on tinyexec process
  }
}

cc @benmccann

have updated to be a throwOnError option rather than a breaking change


Critical issues (potential blockers)

[ ] https://github.com/OpenVoiceOS/ovos-messagebus/issues/4
[ ] https://github.com/OpenVoiceOS/ovos-listener/issues/15
[ ] https://github.com/OpenVoiceOS/OVOS-workshop/issues/67

last one should not be a blocker, but solved by https://github.com/OpenVoiceOS/OVOS-workshop/pull/72

https://github.com/OpenVoiceOS/ovos-ocp-audio-plugin/issues/82  is padatious matching the wrong intent, this is not an issue with docker but a general problem with mycroft
padatious intents are not deterministic and will depend on the number of skills installed also using padatious intents, once you have all skills installed hopefully disambiguation will become better and lower the confidence of that match


delete unit_env

Change Name to M5Unit-ENV

Hello @Tinyu-Zhao. I am confirming your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests. They will merge this pull request when it is done. You will get a notification from GitHub at that time.

Solved in https://github.com/arduino/library-registry/pull/1576#issuecomment-1178737746


Add support for a MAX_RUNNER env var

The idea here is that we add the ability for a MAX_RUNNER environment variable to be set. Ideally this would look like setting the environment variable in the repo settings and using it in the action. This would allow for owners to set an absolute max to be used to for any workflows. This would enable us to set waits to ignore hitting AWS/cloud provider limits. This would check against the cloud provider for the number of instances spawned using the gha-runner platform.
See here for more info on repo wide configuration.

This could also just look like having runner creation wait until it is able to provision with an included timeout. This should not be default, but it should be an option.


Update typing-extensions

Description
For compatibility with iFixit/piwik-pro-piping
qa_req 0 (@addison-grant testing)

Testing this

I ran the tests on the repo which passed. I also used the library in Piwik PRO Piping and the tool implemented worked.
You can see the table created in the BigQuery table piwik-pro-warehousing.piwik_pro_warehousing.events
I think that counts as QA :sunny:


-[LCCanOpenURL canOpenURL:] deadlocks if called from SpringBoard

If you define overloads for canOpenURL: and canOpenURLString: that call directly into [UIApp canOpenURL:], you avoid the deadlock when people use your API from inside SpringBoard.

Oops. I have forgotten that case.
Thanks for your report.
BTW what kind of license AppList is?

Looks good :+1:
Thanks for the heads up on licensing. I've added LGPL licensing text to the AppList repo.

Thanks for review.
Okey, I'll inherit that license.

I consider libcanopenurl sufficiently differentiated from AppList that it need not have the same license if that's what you'd prefer. It's a pretty standard LightMessaging project that happens to use some of the same patterns I use in many of my projects

Sure. I'll keep current MIT license.


fix recompose version range for flowtype

#563

there are also internal references into renamed folders must be changed also

This is not relevant


Loan - Edit - add tagged items

  when I edit a loan
     then I see a fourth option to select loan items
         any loan items attached to a tag/keyword

@tmcelrath this is done, right?

AS in, I can add all tagged items to a loan? Yes.


Modules and plugins

This issue is a placeholder to describe the order of the steps we need to take to make working with Scribe a joy:

DEV env for Scribe and plugins
Move sources into plugin repositories: https://github.com/guardian/scribe/issues/146
Split integration tests by plugin repository
Revise module syntax to satisfy consumers with non-AMD needs
Provide tag releases for sources (enabling consumers (us) to build from source themselves)
Look into ES6 modules

@theefer hello, you say that it's usable via CommonJS, but I'm not really sure I've understood how this is achievable. I can see in the examples there is a way of building a sort of standalone package, but there doesn't seem to be a way to make Scribe play nicely in an actual CommonJS environment.
If it's CommonJS compatible, I would expect to be able to require('scribe-editor') and it to "just work"®, but I get a 'define is undefined' error. Is there anything I'm missing?


Add typescript definitions

I took a first pass at running everything here through tsc and this was the result:
export function generateParticles({ count, height, margin, seed, width, }: {
    count: any;
    height: any;
    margin?: number;
    seed: any;
    width: any;
}): {
    x: number;
    y: number;
    vx: number;
    vy: number;
    line: any[];
}[];

export function moveParticle({ amplitude, damping, frequency, lengthOfStep, particle, }: {
    amplitude: any;
    damping: any;
    frequency: any;
    lengthOfStep: any;
    particle: any;
}): void;

export function generateField({ amplitude, count, damping, height, margin, particles: suppliedParticles, scale, seed, width, }?: {
    amplitude?: number;
    count?: number;
    damping?: number;
    height: any;
    margin?: number;
    particles: any;
    scale?: number;
    seed: any;
    width: any;
}): any;

Looks like the return type for generateField is wrong–are you open to adding type defs to the distributed library?

@romellogoodman if you are open to having them added, I'd be happy to open a PR with config, types, and a naive pass at supporting scripts for your publishing process.


Updated jwt creation code

Couple of changes in this PR:

Changes jwt creation by checking onetrust cookie and removing any reference to the user's id in their token when they login
adds sentry error reporting to all three functions (auth-start,auth-callback,get-workshop)
add more config variable building for the sentry DSNs for the functions
add client-side sentry error and performance reporting

Not sure if it has to do with being a preview deploy, but after attempting to auth with my GitHub, it redirects me back to the Netlify auth-callback function with the following error:
{"error":"Cannot read property 'search' of undefined"}

Attempting to go back to any workshop after, it sends me straight to that error message without any redirect in between


Spyder editor don't open file with CTRL+F11 in remote kernel mode

I have similar problem ([https://github.com/spyder-ide/spyder/issues/13868]).
I make a remote connection with nanopi using remote spyder kerner.
I can run and debug files,
BUT, when i want to step into (CTRL+F11) function which locate in another file, Spyder Editor DON'T open this file. But debug proccess still continued (I can see in in Variable Explorer).
I have this problem only with remote kernels.
Versions on nanopi:
spyder-kernels 2.5.2
Python 3.10.12
Versions on Linux OS:
spyder-kernels 2.5.2.
Python 3.10.12

Hey @DaryMile, thanks for reporting. We'll take a look at this problem after we release Spyder 6.0 in a few weeks.


Add rewriter to simplify loop statements generated by the decompiler

Contributing a Pull Request
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Contributing a feature

[x] I have opened a new issue for the proposal, or commented on an existing one, and ensured that the Bicep maintainers are good with the design of the feature being implemented
[x] I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
[x] I have appropriate test coverage of my new feature

Fixes #1788

Codecov Report

Merging #1873 (a5ade33) into main (cf4e7ec) will increase coverage by 0.48%.
The diff coverage is 94.92%.

@@            Coverage Diff             @@
##             main    #1873      +/-   ##
==========================================
+ Coverage   95.11%   95.59%   +0.48%
==========================================
  Files         371      370       -1
  Lines       21929    22036     +107
  Branches       15        0      -15
==========================================
+ Hits        20857    21066     +209
+ Misses       1072      970     -102

Flag
Coverage Δ

dotnet
95.59% <94.92%> (+0.04%)
:arrow_up:

typescript
?

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

Impacted Files
Coverage Δ

...p.Decompiler/Rewriters/DependsOnRemovalRewriter.cs
97.33% <ø> (ø)

...piler/Rewriters/ParentChildResourceNameRewriter.cs
92.39% <ø> (ø)

...ep.Decompiler/Rewriters/TypeCasingFixerRewriter.cs
100.00% <ø> (ø)

src/Bicep.Core/Syntax/SyntaxRewriteVisitor.cs
80.00% <87.96%> (+4.05%)
:arrow_up:

...piler/Rewriters/ForExpressionSimplifierRewriter.cs
94.94% <94.94%> (ø)

src/Bicep.Core.UnitTests/Utils/PrintHelper.cs
100.00% <100.00%> (ø)

src/Bicep.Core/Syntax/ForSyntax.cs
96.07% <100.00%> (ø)

...rc/Bicep.Core/Syntax/Rewriters/CallbackRewriter.cs
100.00% <100.00%> (ø)

src/Bicep.Core/Syntax/Visitors/CallbackVisitor.cs
100.00% <100.00%> (+100.00%)
:arrow_up:

...itTests/Rewriters/DependsOnRemovalRewriterTests.cs
100.00% <100.00%> (ø)

... and 20 more


Redo lift

Removed RefTerm::lift. I'm not yet sure if my changes are complete, cus in concrete there's gonna be invalid 'lift local var' and we want to report errors (now lift is restricted to defcalls).

I seems that the changes are complete.
bors r+


[WM-1696] Convert Womtool Inputs/Outputs to CBAS

@kpierre13 it seems the conversion for type Array[Map[X,X]] might not be working as expected. Using your branch to create new method and see the new method being populated in Submission Config page in CBAS UI, when I tried to create a new method for assemble_refbased workflow, in it's outputs it displays Array[Unsupported.. for output assemble_refbased.ivar_trim_stats (as shown below).

Comparing it to the output definition for our pre-staged assemble_refbased it seems that the Map type might not be converted from WomTool to CBAS as expected.
Output definition for assemble_refbased.ivar_trim_stats based on your branch:
{
    "output_name": "assemble_refbased.ivar_trim_stats",
    "output_type": {
      "type": "array",
      "non_empty": false,
      "array_type": {
        "type": "array",
        "key_type": "String",
        "value_type": {
          "type": "primitive",
          "primitive_type": "String"
        }
      }
    },
    "destination": {
      "type": ""
    }
  }

Output definition for assemble_refbased.ivar_trim_stats from pre-staged workflow:
{
    "output_name": "assemble_refbased.ivar_trim_stats",
    "output_type": {
      "type": "array",
      "non_empty": false,
      "array_type": {
        "type": "map",
        "key_type": "String",
        "value_type": {
          "type": "primitive",
          "primitive_type": "String"
        }
      }
    },
    "destination": {
      "type": "none"
    }
  }


bundle-stats.html not generated if compare is true

I have a react app that uses bundle-stats. Currently updating webpack 5, I'm also updating bundle-stats. I removed<EMAIL_ADDRESS>(latest version working) and installed<EMAIL_ADDRESS>I noticed bundle-stats.html not generating with when compare is true. However, it works as expected when compare is false.
Node.js 10.14.2
webpack: 5.9.0
bundle-stats-webpack-plugin: 2.3.0
Webpack config that might be relevant:
{
  context: '$home/app/client'
}

bundle stats plugin:
BundleStatsWebpackPlugin({
        compare: true,
        json: true,
        outDir: "..",
      })

Thanks for raising the issue @ktw1016, will be helpful if possible to share more details about your issue:

are you getting any errors / warnings?
are the reports generated?

Found out that with config baseline=true, reports are not generated at all if there is already node_modules/.cache/bundle-stats/baseline.json. Here is the output:
**Read baseline from ../../../node_modules/.cache/bundle-stats/baseline.json
Write baseline data to ../../../node_modules/.cache/bundle-stats/baseline.json
Bundle Size — 32.6MB (0%).**
en:
  assets by chunk 22.5 MiB (id hint: vendors) 19 assets
  56 assets
  orphan modules 347 KiB [orphan] 165 modules
  runtime modules 6.5 KiB 12 modules
  modules by path ./node_modules/ 7.43 MiB 1788 modules
  modules by path ./src/ 2.37 MiB 569 modules
  ../data/faq.csv 12.5 KiB [built] [code generated]
  assert (ignored) 15 bytes [built] [code generated]
  en (webpack 5.9.0) compiled successfully in 43217 ms

However, if baseline.json is missing, report is generated. Output:
**Missing baseline stats, see "baseline" option.
Write baseline data to ../../../node_modules/.cache/bundle-stats/baseline.json
Bundle Size — 32.6MB (+100%).**
en:
  assets by status 32.6 MiB [compared for emit]
    assets by chunk 22.5 MiB (id hint: vendors) 19 assets
  assets by status 3.25 MiB [emitted]
    **asset ../bundle-stats.json 1.65 MiB [emitted]
    asset ../bundle-stats.html 1.17 MiB [emitted]**
    asset ../../../node_modules/.cache/bundle-stats/baseline.json 439 KiB [emitted]
  orphan modules 347 KiB [orphan] 165 modules
  runtime modules 6.5 KiB 12 modules
  modules by path ./node_modules/ 7.43 MiB 1788 modules
  modules by path ./src/ 2.37 MiB 569 modules
  ../data/faq.csv 12.5 KiB [built] [code generated]
  assert (ignored) 15 bytes [built] [code generated]
  en (webpack 5.9.0) compiled successfully in 42267 ms

hi @ktw1016! I was able to replicate the issue on one of the example repos - https://github.com/relative-ci/example-bundle-stats-webpack-config/tree/e00452845d1cce478a2f183298623b057fd3792e. After I upgraded html-webpack plugin to fix a deprecation warning, the problem disappeared.
I'm not 100%, but I suspect a deeper problem with the new way Webpack 5 is processing assets. In the meantime I updated the way bundle-stats is processing the reports and released a beta version (2.4.0-beta.6).
If you manage to give it a try, let me know how is going.

Hi @vio , thanks for getting back to me. What's the html-webpack plugin you upgraded?
I tried with<EMAIL_ADDRESS>And I'm still getting the exact same error as described above. Slightly different output however.
Missing node_modules/.cache/bundle-stats/baseline.json output (report is generated):
Missing baseline stats, see "baseline" option.
Bundle Size — 4.7MB (+100%).
(node:126) [DEP_WEBPACK_COMPILATION_ASSETS] DeprecationWarning: Compilation.assets will be frozen in future, all modifications are deprecated.
BREAKING CHANGE: No more changes should happen to Compilation.assets after sealing the Compilation.
Do changes to assets earlier, e. g. in Compilation.hooks.processAssets.
Make sure to select an appropriate stage from Compilation.PROCESS_ASSETS_STAGE_.
en:
assets by path .js 4.7 MiB
assets by chunk 2.76 MiB (id hint: vendors)
assets by status 1.46 MiB [big]
asset vendors-node_modules_graphiql_esm_index_js-node_modules_isomorphic-fetch_fetch-npm-browserify-359772.app-en.min.js 816 KiB [emitted] [minimized] [big] (id hint: vendors)
asset vendors-node_modules_html2canvas_dist_html2canvas_js-node_modules_jspdf-autotable_dist_jspdf_-247707.app-en.min.js 682 KiB [emitted] [minimized] [big] (id hint: vendors) 1 related asset
17 assets
assets by path ../ 2.21 MiB
asset ../.json 1.19 MiB [emitted] [big]
asset ../**.html 1.02 MiB [emitted] [big]
orphan modules 2.46 MiB [orphan] 976 modules
runtime modules 6.63 KiB 12 modules
modules by path ./ 9.23 MiB
modules by path ./node_modules/ 6.58 MiB 1155 modules
modules by path ./src/ 2.65 MiB 352 modules
../data/faq.csv 12.5 KiB [built] [code generated]
assert (ignored) 15 bytes [built] [code generated]
If there is already node_modules/.cache/bundle-stats/baseline.json (report not generated):
en:
assets by chunk 21.3 MiB (id hint: vendors) 19 assets
56 assets
orphan modules 347 KiB [orphan] 165 modules
runtime modules 6.62 KiB 12 modules
modules by path ./node_modules/ 7.43 MiB 1788 modules
modules by path ./src/ 2.37 MiB 569 modules
../data/faq.csv 12.5 KiB [built] [code generated]
assert (ignored) 15 bytes [built] [code generated]
en (webpack 5.10.2) compiled successfully in 48450 ms
Read baseline from ../../../node_modules/.cache/bundle-stats/baseline.json
Write baseline data to ../../../node_modules/.cache/bundle-stats/baseline.json
Bundle Size — 31.09MB (0%).

@ktw1016

What's the html-webpack plugin you upgraded?
html-webpack-plugin

Thanks for trying<EMAIL_ADDRESS>but i was still missing couple bits.
There is a new beta version available<EMAIL_ADDRESS>that is solving the deprecation warning and is working for your project as well.
~/w/r/_/i/client on master ⨯ npm run IB_prod_stats                                                                                00:20:03

> node build_code/webpack_starter.js PROD NO-WATCH STATS EN main_client

  app: main_client,
  prod: true,
  babel: true,
  languages: en,

Read baseline from ../../../node_modules/.cache/bundle-stats/baseline.json
Bundle Size — 4.68MB (0%).
en:
  assets by path *.js 4.68 MiB
    assets by chunk 2.76 MiB (id hint: vendors)
      assets by status 1.46 MiB [big]
        asset vendors-node_modules_graphiql_esm_index_js-node_modules_isomorphic-fetch_fetch-npm-browserify-359772.app-en.min.js 816 KiB [compared for emit] [minimized] [big] (id hint: vendors)
        asset vendors-node_modules_html2canvas_dist_html2canvas_js-node_modules_jspdf-autotable_dist_jspdf_-247707.app-en.min.js 682 KiB [compared for emit] [minimized] [big] (id hint: vendors) 1 related asset
      17 assets
  assets by status 3.08 MiB [emitted]
    asset ../bundle-stats.html 1.63 MiB [emitted] [compared for emit] [dev]
    asset ../bundle-stats.json 1.45 MiB [emitted] [compared for emit] [dev]
  orphan modules 2.45 MiB [orphan] 975 modules
  runtime modules 6.5 KiB 12 modules
  modules by path ./ 9.21 MiB
    modules by path ./node_modules/ 6.58 MiB 1155 modules
    modules by path ./src/ 2.64 MiB 352 modules
  ../data/faq.csv 12.5 KiB [built] [code generated]
  assert (ignored) 15 bytes [built] [code generated]

  WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
  This can impact web performance.
  Assets:
    vendors-node_modules_graphiql_esm_index_js-node_modules_isomorphic-fetch_fetch-npm-browserify-359772.app-en.min.js (816 KiB)
    vendors-node_modules_html2canvas_dist_html2canvas_js-node_modules_jspdf-autotable_dist_jspdf_-247707.app-en.min.js (682 KiB)

  en (webpack 5.9.0) compiled with 1 warning in 77746 ms

Hi @vio , indeed it's fixed.
I'll let you know if I find anything else. Thanks for the fix!


[BUG] - Create workflow with shared storage

When trying to create a new workflow that uses S3 shared storage but some of the required secrets are not filled (access_key or secret_key) the validation is show "Resource not found", we must show a custom message.

@vinicvaz is this fixed already in dev?

@luiztauffer I don't think so.

Also, I think there are other error notifications that are showing with dictionary and details

Not sure about the current reported error, but definitely we are not handling income api errors properly, working on it rn


Support riscv64gc-unknown-freebsd

Also see https://github.com/rust-lang/rust/pull/91284

Seems reasonable to me! If you wouldn't mind pinging this when the upstream rust-lang/rust PR has merged I'll merge this here as well.

It has been merged. Thanks!


Connection to BMS lost, aggregate-driver crashes

Hello,
i have a problem with crashing with the driver. It runs 3-4 weeks without any problems.
But then it could happen that for a short moment the connection to one of my 4 BMS (JBD)
is lost for a short moment. And then the driver crashes and also a reboot does not solve the problem.
I then have to restart the driver with the restart script when the Cerbo is up again.
It shows me the following error in the log:
ERROR:root:Occured during step Read battery state, Battery SerialBattery(LLT/JBD). ERROR:root:Read trial nr. 1 Traceback (most recent call last):   File "/data/dbus-aggregate-batteries/aggregatebatteries.py", line 842, in _update     sum(VoltagesSum_dict.values()) / settings.NR_OF_BATTERIES TypeError: unsupported operand type(s) for +: 'int' and 'NoneType' Terminated
My actual version of the Venus OS is 3.41
best regards
Steffen

Same here, just as i check VRM Portal for new updates things start to go wrong - what you see at the end 15:32 is when i run
/data/dbus-aggregate-batteries/restart.sh manually:
INFO:root:  CVL: 54.0V, CCL: 280A, DCL: 280A
INFO:root:  Bat. voltage: 54.0V, Bat. current: 0A, SoC: 98.2%, Balancing state: 0
INFO:root:  Min. cell voltage: Pack2_C5: 3.372V, Max. cell voltage: Pack2_C3: 3.379V, difference: 0.007V
INFO:dbusmonitor:com.victronenergy.battery.ttyUSB1 disappeared from the dbus. Removing it from our lists
INFO:dbusmonitor:com.victronenergy.battery.ttyUSB0 disappeared from the dbus. Removing it from our lists
ERROR:root:Fri Sep  6 15:21:11 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 1
ERROR:root:Fri Sep  6 15:21:12 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 2
ERROR:root:Fri Sep  6 15:21:13 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 3
ERROR:root:Fri Sep  6 15:21:14 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 4
ERROR:root:Fri Sep  6 15:21:15 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 5
ERROR:root:Fri Sep  6 15:21:16 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 6
ERROR:root:Fri Sep  6 15:21:17 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 7
ERROR:root:Fri Sep  6 15:21:18 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 8
ERROR:root:Fri Sep  6 15:21:19 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 9
ERROR:root:Fri Sep  6 15:21:20 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 10
ERROR:root:Fri Sep  6 15:21:21 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 11
ERROR:root:Fri Sep  6 15:21:21 2024: DBus read failed. Exiting.
INFO:root:Fri Sep  6 15:21:23 2024: Starting AggregateBatteries.
INFO:root:registered ourselves on D-Bus as com.victronenergy.battery.aggregate
INFO:root:Fri Sep  6 15:21:23 2024: Initial Ah read from file: 536Ah
INFO:root:Fri Sep  6 15:21:24 2024: Starting battery monitor.
INFO:root:Fri Sep  6 15:21:24 2024: Connected to DBus, and switching over to GLib.MainLoop()
INFO:dbusmonitor:===== Search on dbus for services that we will monitor starting... =====
INFO:dbusmonitor:Found: com.victronenergy.settings, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.settings has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.system, scanning and storing items
INFO:root:Fri Sep  6 15:21:25 2024: Searching Settings: Trial Nr. 1
INFO:dbusmonitor:       com.victronenergy.system has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.vebus.ttyS4, scanning and storing items
INFO:root:Fri Sep  6 15:21:25 2024: com.victronenergy.settings found.
INFO:dbusmonitor:       com.victronenergy.vebus.ttyS4 has device instance 276
INFO:dbusmonitor:Found: com.victronenergy.battery.aggregate, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.aggregate has device instance 0
INFO:dbusmonitor:===== Search on dbus for services that we will monitor finished =====
INFO:root:Fri Sep  6 15:21:30 2024: Searching batteries: Trial Nr. 1
INFO:root:Fri Sep  6 15:21:30 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:21:35 2024: Searching batteries: Trial Nr. 2
INFO:root:Fri Sep  6 15:21:35 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:21:40 2024: Searching batteries: Trial Nr. 3
INFO:root:Fri Sep  6 15:21:40 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:21:45 2024: Searching batteries: Trial Nr. 4
INFO:root:Fri Sep  6 15:21:45 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:21:50 2024: Searching batteries: Trial Nr. 5
INFO:root:Fri Sep  6 15:21:50 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:21:55 2024: Searching batteries: Trial Nr. 6
INFO:root:Fri Sep  6 15:21:55 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:22:00 2024: Searching batteries: Trial Nr. 7
INFO:root:Fri Sep  6 15:22:00 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:22:05 2024: Searching batteries: Trial Nr. 8
INFO:root:Fri Sep  6 15:22:05 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:22:10 2024: Searching batteries: Trial Nr. 9
INFO:root:Fri Sep  6 15:22:10 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:22:15 2024: Searching batteries: Trial Nr. 10
INFO:root:Fri Sep  6 15:22:15 2024: 0 batteries found.
INFO:root:Fri Sep  6 15:22:20 2024: Searching batteries: Trial Nr. 11
INFO:root:Fri Sep  6 15:22:20 2024: 0 batteries found.
ERROR:root:Fri Sep  6 15:22:20 2024: Required number of batteries not found. Exiting.
INFO:root:Fri Sep  6 15:22:22 2024: Starting AggregateBatteries.
INFO:root:registered ourselves on D-Bus as com.victronenergy.battery.aggregate
INFO:root:Fri Sep  6 15:22:22 2024: Initial Ah read from file: 536Ah
INFO:root:Fri Sep  6 15:22:22 2024: Starting battery monitor.
INFO:root:Fri Sep  6 15:22:22 2024: Connected to DBus, and switching over to GLib.MainLoop()
INFO:dbusmonitor:===== Search on dbus for services that we will monitor starting... =====
INFO:dbusmonitor:Found: com.victronenergy.settings, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.settings has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.system, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.system has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.vebus.ttyS4, scanning and storing items
INFO:root:Fri Sep  6 15:22:23 2024: Searching Settings: Trial Nr. 1
INFO:dbusmonitor:       com.victronenergy.vebus.ttyS4 has device instance 276
INFO:root:Fri Sep  6 15:22:23 2024: com.victronenergy.settings found.
INFO:dbusmonitor:Found: com.victronenergy.battery.aggregate, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.aggregate has device instance 0
INFO:dbusmonitor:===== Search on dbus for services that we will monitor finished =====
INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB0, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB0 has device instance 5
INFO:root:Fri Sep  6 15:22:34 2024: Searching batteries: Trial Nr. 1
INFO:root:Fri Sep  6 15:22:34 2024: SerialBattery(Seplos) found, named as: Pack2.
INFO:root:Fri Sep  6 15:22:34 2024: 1 batteries found.
INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB1, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB1 has device instance 1
INFO:root:Fri Sep  6 15:22:39 2024: Searching batteries: Trial Nr. 2
INFO:root:Fri Sep  6 15:22:39 2024: SerialBattery(Seplos) found, named as: Pack2.
INFO:root:Fri Sep  6 15:22:39 2024: SerialBattery(Seplos) found, named as: Pack1.
INFO:root:Fri Sep  6 15:22:39 2024: 2 batteries found.
ERROR:root:Fri Sep  6 15:22:40 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 1
ERROR:root:Fri Sep  6 15:22:41 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read battery state, Battery Pack2.
ERROR:root:Read trial nr. 2
ERROR:root:Fri Sep  6 15:22:42 2024: Error: unsupported operand type(s) for +=: 'float' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack1.
ERROR:root:Read trial nr. 3
ERROR:root:Fri Sep  6 15:22:43 2024: Error: unsupported operand type(s) for +=: 'float' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack1.
ERROR:root:Read trial nr. 4
Traceback (most recent call last):
File "/data/dbus-aggregate-batteries/aggregatebatteries.py", line 842, in _update
sum(VoltagesSum_dict.values()) / settings.NR_OF_BATTERIES
TypeError: unsupported operand type(s) for +: 'float' and 'NoneType'
Terminated
INFO:root:Fri Sep  6 15:32:17 2024: Starting AggregateBatteries.
INFO:root:registered ourselves on D-Bus as com.victronenergy.battery.aggregate
INFO:root:Fri Sep  6 15:32:17 2024: Initial Ah read from file: 536Ah
INFO:root:Fri Sep  6 15:32:17 2024: Starting battery monitor.
INFO:root:Fri Sep  6 15:32:17 2024: Connected to DBus, and switching over to GLib.MainLoop()
INFO:dbusmonitor:===== Search on dbus for services that we will monitor starting... =====
INFO:dbusmonitor:Found: com.victronenergy.settings, scanning and storing items
INFO:root:Fri Sep  6 15:32:19 2024: Searching Settings: Trial Nr. 1
INFO:dbusmonitor:       com.victronenergy.settings has device instance 0
INFO:root:Fri Sep  6 15:32:19 2024: com.victronenergy.settings found.
INFO:dbusmonitor:Found: com.victronenergy.system, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.system has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.vebus.ttyS4, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.vebus.ttyS4 has device instance 276
INFO:dbusmonitor:Found: com.victronenergy.battery.aggregate, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.aggregate has device instance 0
INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB0, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB0 has device instance 5
INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB1, scanning and storing items
INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB1 has device instance 1
INFO:dbusmonitor:===== Search on dbus for services that we will monitor finished =====
INFO:root:Fri Sep  6 15:32:24 2024: Searching batteries: Trial Nr. 1
INFO:root:Fri Sep  6 15:32:24 2024: SerialBattery(Seplos) found, named as: Pack2.
INFO:root:Fri Sep  6 15:32:24 2024: SerialBattery(Seplos) found, named as: Pack1.
INFO:root:Fri Sep  6 15:32:24 2024: 2 batteries found.

Hello guys, it is correct to crash and restart if a battery is disconnected or send an invalid value. Otherwise would aggregate batteries provide invalid charge parameters which is dangerous to the battery. Try to monitor the failing instance of the dubs serial battery in order to understand which value goes to none.

Hello, last night i had again a connection loss to one of my four battery packages but there the restart has worked properly.

I do agree that it should crash, but why isn‘t it recovering by itself but after a restart only?
I mean the data from serialbattery is the same before and after the crash

If my battery crashes and restarts, the aggregate restarts as well. If there is a error message about incompatible operands 'int' and 'none', some battery instance, even found, doesn't send the given quantity.

Hi,
I do have exactly the same issue. Just install both dbus-serialbattery and dbus-aggregate-batteries.
I do have 2 LFP pack (16cells). Both are visible on the Cerbo, with all details.
So what kind of setup we should apply in serialbattery driver to ensure the aggregate-batteries driver works fine ? (ie in config.default.ini of serialbattery)
Thank you

here is log of serialbattery.ttyUSB0 (other batterie is same in USB1):

The JPG seems to be broken, please try again. And check with dbus-spy command if all batteries are visible on the dbus and which values do they send.

The right guy who can help you with the serial-battery setup is @mr-manuel .

If you see errors in the dbus-serialbattery open a issue in the repository: https://github.com/mr-manuel/venus-os_dbus-serialbattery/issues/new/choose
Please read carefully the instructions and don't skip anything.

Hi
With the corrected link to the missing above jpg:

Your cerbo screenshots contain no relevant info. Please check the aggregate batteries log file and observe dbus by dbus-spy command. Find which path contains 'None' value. If this is the case, my program must crash and restart.

Hi,
dbus-spy does not show any error message or line with 'None' value. See attached screenshots. I ony share for 1 battery, the 2nd is fully aligned with similar details (well, its name change, ... ;) )

Hi, and the Cerbo Gx screenshot

Hi, now I see the problem. Your paths are not as usual. Your voltages:

My voltages:

The exception is raised in the 842, where the sum of "Voltages/Sum"  from the lines 661 and 662 are calculated. But you have the path "Cell/Sum" instead. The value of not existing path "Voltages/Sum" is "None".
@mr-manuel : Have you renamed the paths in some update?

No this is selectable since many years.
https://github.com/Louisvdw/dbus-serialbattery/blob/fd0fc1a193427df6ca0fadd141aa57826f761819/etc/dbus-serialbattery/config.default.ini#L342-L347

Hi,
That is the issue and the solution !
As soon I change the BATTERY_CELL_DATA_FORMAT to 1, I get the dbus-aggregate_battery working on Cerbo !
Many thanks for your kind support and time ! Definitely really appreciate :)
Best regards

But isn't that the default, so if it's not changed manually it should be 1 already?
Cells have the correct format on my end, without any customization - and aggregate_battery works perfectly for me, unless i trigger firmware search from the VRM Portal.
Serial Battery disconnects then, aggregate batteries has no values anymore and crashes as well. What happens then is, as serial battery reconnects, it has no values for 20-30 seconds until the connection is re-established fully again - and at this time aggregate batteries has given up already. I think key would be if serial battery would not post anything to dbus until it's fully up, or aggregatebatteries handles this situation better.

Just did look dbus-spy while things happen. The Cell voltages have empty values for around 20-30 seconds. Aggregate batteries is in a dead end afterwards

Please read readme. You can increase number of trials before the program gives up. But even given up, it should be restarted automatically by Linux.

Please see my post with the log above:
it's exiting after trial 11 to search for batteries and then restarts. Here it finds the batteries then, however with still empty values. At the read thread it just goes down to retry 4 until it fails and doesn't recover from there anymore (even while the settings.py is still set to 10 read retries per default, it doesn't seem to make it)
INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB1, scanning and storing items
INFO:dbusmonitor: com.victronenergy.battery.ttyUSB1 has device instance 1
INFO:root:Fri Sep 6 15:22:39 2024: Searching batteries: Trial Nr. 2
INFO:root:Fri Sep 6 15:22:39 2024: SerialBattery(Seplos) found, named as: Pack2.
INFO:root:Fri Sep 6 15:22:39 2024: SerialBattery(Seplos) found, named as: Pack1.
INFO:root:Fri Sep 6 15:22:39 2024: 2 batteries found.
ERROR:root:Fri Sep 6 15:22:40 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack2.
ERROR:root:Read trial nr. 1
ERROR:root:Fri Sep 6 15:22:41 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read battery state, Battery Pack2.
ERROR:root:Read trial nr. 2
ERROR:root:Fri Sep 6 15:22:42 2024: Error: unsupported operand type(s) for +=: 'float' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack1.
ERROR:root:Read trial nr. 3
ERROR:root:Fri Sep 6 15:22:43 2024: Error: unsupported operand type(s) for +=: 'float' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack1.
ERROR:root:Read trial nr. 4
Traceback (most recent call last):
File "/data/dbus-aggregate-batteries/aggregatebatteries.py", line 842, in _update
sum(VoltagesSum_dict.values()) / settings.NR_OF_BATTERIES
TypeError: unsupported operand type(s) for +: 'float' and 'NoneType'
Terminated

HW with a robust and galvanic isolated interface between batteries and Cerbo.

As I wrote in the dbus-serial-battery discussion about the hardware some time ago, it is essential to use galvanic insulation. Otherwise a part of (and in case of BMS disconnection the whole) battery current through ground of the serial line with all consequences you can imagine.

it's exiting after trial 11 to search for batteries and then restarts. Here it finds the batteries then, however with still empty values. > At the read thread it just goes down to retry 4 until it fails and doesn't recover from there anymore (even while the settings.py is > still set to 10 read retries per default, it doesn't seem to make it)

I think I've found it out. In aggregatebatteries.py, in try block beginning in line 578 in the variable step is written by text describing what is just being done. In the exception bloc beginning in the line 819 this text is printed in order to identify the issue. This is the case here,

ERROR:root:Fri Sep 6 15:22:43 2024: Error: unsupported operand type(s) for +=: 'float' and 'NoneType'.
ERROR:root:Occured during step Read V, I, P, Battery Pack1.
some path is 'None'.

After 4 trials the missing value is present. But then an panhandled exception in the line 842 is raised. The voltages of all batteries are in the dictionary, some of them is 'None'. The assignment of 'None' into dictionary seems to be legal, therefore no exception starting new trial. But the line 842 is outside of the trial block, the exception causes immediate crash. I'll correct it in the program at the next opportunity.
But even in this case the program should be restarted again and again, there could be something wrong with ./service/run or the symbolic link, see the lines 12-15 in reinstall-local.sh.
The easiest way to check if it is working is to execute sh ./ restart.sh from the work directory.

An additional feedback from my side, it has also crashed with the new driver but with the help from mr-manuel (Thx) the problem could be localized to a cabeling problem. It seems that the USB cables are laid to close to the batterie cables.
I will reorganize the cables and hope that the problems are solved then.

It seems that the USB cables are laid to close to the batterie cables.
The battery stabilizes the voltage, not the current. The AC component from charger/inverter flows through, there is no way to filter it in case of 50Hz. That is the reason why the current measurement of JK BMS is inaccurate. I am not even sure in case of Smart Solar, running at high switching frequency, whether the current ripples are filtered sufficiently. So you have time-variable magnetic field around your battery which coupled into your USB cable. But I wonder that the shielding of the cable didn't do its job. Was the shield on both sides connected to the reference potential (digital ground)? And the galvanic insulation between the USB to serial converter and the BMS is essential.

Hello again,
no i had the same problem again (Connection lost to one BMS, there i have to reorganize the cables) but after it comes back
the driver does not come up again. There where 4 BMS recognized (correct count of BMS, 4 installed) but the driver crashes therefore.
INFO:root:Thu Oct 10 12:15:01 2024: Connected to DBus, and switching over to GLib.MainLoop() INFO:dbusmonitor:===== Search on dbus for services that we will monitor starting... ===== INFO:dbusmonitor:Found: com.victronenergy.settings, scanning and storing items INFO:dbusmonitor:       com.victronenergy.settings has device instance 0 INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB0__0, scanning and storing items INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB0__0 has device instance 6 INFO:dbusmonitor:Found: com.victronenergy.system, scanning and storing items INFO:root:Thu Oct 10 12:15:02 2024: Searching Settings: Trial Nr. 1 INFO:root:Thu Oct 10 12:15:02 2024: com.victronenergy.settings found. INFO:dbusmonitor:       com.victronenergy.system has device instance 0 INFO:dbusmonitor:Found: com.victronenergy.solarcharger.socketcan_can0_vi2_uc432290, scanning and storing items INFO:dbusmonitor:       com.victronenergy.solarcharger.socketcan_can0_vi2_uc432290 has device instance 2 INFO:dbusmonitor:Found: com.victronenergy.vebus.ttyS4, scanning and storing items INFO:dbusmonitor:       com.victronenergy.vebus.ttyS4 has device instance 276 INFO:dbusmonitor:Found: com.victronenergy.solarcharger.socketcan_can0_vi1_uc429774, scanning and storing items INFO:dbusmonitor:       com.victronenergy.solarcharger.socketcan_can0_vi1_uc429774 has device instance 1 INFO:dbusmonitor:Found: com.victronenergy.solarcharger.socketcan_can0_vi0_uc430176, scanning and storing items INFO:dbusmonitor:       com.victronenergy.solarcharger.socketcan_can0_vi0_uc430176 has device instance 0 INFO:dbusmonitor:Found: com.victronenergy.battery.aggregate, scanning and storing items INFO:dbusmonitor:       com.victronenergy.battery.aggregate has device instance 0 INFO:dbusmonitor:===== Search on dbus for services that we will monitor finished ===== INFO:root:Thu Oct 10 12:15:07 2024: Searching batteries: Trial Nr. 1 INFO:root:Thu Oct 10 12:15:07 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD). INFO:root:Thu Oct 10 12:15:07 2024: 1 batteries found. INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB1__0, scanning and storing items INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB1__0 has device instance 3 INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB2__0, scanning and storing items INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB2__0 has device instance 5 INFO:root:Thu Oct 10 12:15:12 2024: Searching batteries: Trial Nr. 2 INFO:root:Thu Oct 10 12:15:12 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD). INFO:root:Thu Oct 10 12:15:12 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD)2. INFO:root:Thu Oct 10 12:15:12 2024: JBD-AP20S003S-L20S-300A-300A-B- found, named as: SerialBattery(LLT/JBD)3. INFO:root:Thu Oct 10 12:15:12 2024: 3 batteries found. INFO:dbusmonitor:Found: com.victronenergy.battery.ttyUSB3__0, scanning and storing items INFO:dbusmonitor:       com.victronenergy.battery.ttyUSB3__0 has device instance 4 INFO:root:Thu Oct 10 12:15:17 2024: Searching batteries: Trial Nr. 3 INFO:root:Thu Oct 10 12:15:17 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD). INFO:root:Thu Oct 10 12:15:17 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD)2. INFO:root:Thu Oct 10 12:15:17 2024: JBD-AP20S003S-L20S-300A-300A-B found, named as: SerialBattery(LLT/JBD)3. INFO:root:Thu Oct 10 12:15:17 2024: JBD-AP20S003S-L20S-300A-300A-B- found, named as: SerialBattery(LLT/JBD)4. INFO:root:Thu Oct 10 12:15:17 2024: 4 batteries found. Traceback (most recent call last):   File "/data/dbus-aggregate-batteries/aggregatebatteries.py", line 842, in _update     sum(VoltagesSum_dict.values()) / settings.NR_OF_BATTERIES TypeError: unsupported operand type(s) for +: 'float' and 'NoneType'

After a restart.sh the driver comes up again and works as expected, but it dosn't come up automatically

Hi, please replace the aggregatebaterries.py by the one I've just pushed and try. Now if '/Voltages/Sum' is None, the adjusted number of trials (and not only one as before) will be done before crash. But if the program after crash does not restart, I cannot help you. It is your local Linux issue and I'm not a Linux specialist.
Here is the code change (from line 662):
here an exception is raised and new read trial initiated if None is on Dbus
volt_sum_get = self._dbusMon.dbusmon.get_value(self._batteries_dict[i], "/Voltages/Sum")
if volt_sum_get != None:
VoltagesSum_dict[i] = volt_sum_get
else:
raise TypeError(f"Battery {i} returns None value of /Voltages/Sum.")

Thanks for the changes, i have replaced the file and rebooted the Cerbo. I will monitor this for the next weeks what happens then when a connection lost appears again.

I can confirm that the driver now successfully recovers, when serial battery looses the connection to both Packs for some time. Thanks for this fix!


Allow shortName to contain a slash

Right now, short names of resources can't contain a slash and this PR fixes that issue.

What is the use case?

Let's imagine a resource class Foo, the default endpoint will be /foo/{id} but I want it to be /bar/foo/{id}.


Multiple errors trying to run on basic

Trying to run v2.03 on Transcend WiFi SD Linux fail with the following errors :
bash -x /etc/init.d/ftp-sync .

: not found/ftp-sync: line 10:
+
: not found/ftp-sync: line 44:

CONFIG_FILE=/etc/ftp-sync/ftp-sync.conf

: not found/ftp-sync: line 46:
+
: not found/ftp-sync: line 48:
+
: not found/ftp-sync: line 50:
/etc/init.d/ftp-sync: line 51: syntax error: unexpected "("

Will be hard to reproduce your problem on this kind of linux based distribution :/
Can you provide more information about this distrib ?


The texts in the current background are not easily visible.  Figure out how this can be achieved without compromising the looks.
(Try reducing bg opacity or overlay)

Sir I have opened a pull request for this issue can you look into it.


Play DRM-protected content securityLevel L1

I'm unable to play content with top securityLevel L1:
you can try this media content from google exoplayer repo for L1:
        "name": "HW secure all (L1)",
        "uri": "https://storage.googleapis.com/wvmedia/cenc/h264/tears/tears.mpd",
        "drm_scheme": "widevine",
        "drm_license_uri": "https://proxy.uat.widevine.com/proxy?video_id=GTS_HW_SECURE_ALL&provider=widevine_test"

I tried it on Emulator as well as real devices and they both fail to play the content.

You can use videoFormat
FORMAT_SS -> C.CONTENT_TYPE_SS
                FORMAT_DASH -> C.CONTENT_TYPE_DASH
                FORMAT_HLS -> C.CONTENT_TYPE_HLS
                FORMAT_OTHER -> C.CONTENT_TYPE_OTHER
                FORMAT_RTSP -> C.CONTENT_TYPE_RTSP

@TDUser91  In china, I can't to do this!  would you please feedback to (media)[https://github.com/androidx/media] ?

try {
                                val mediaDrm = FrameworkMediaDrm.newInstance(uuid!!)
                                // Force L3.
                                mediaDrm.setPropertyString("securityLevel", "L3")
                                return@setUuidAndExoMediaDrmProvider mediaDrm
                            } catch (e: UnsupportedDrmException) {
                                return@setUuidAndExoMediaDrmProvider DummyExoMediaDrm()
                            }

Maybe is DummyExoMediaDrm error

@liuchuancong I've found the root of the issue. BetterPlayer doesn't work with Widevine L1/L2 because of textures. It uses VideoPlayer and the way VideoPlayer processes video is using textures in software level and in order to play L1 protected-content it has to be Hardware level. What I've found is that I have to use SurfaceView (You can read more here: (https://source.android.com/docs/core/graphics/architecture and https://medium.com/androiddevelopers/android-hdr-migrating-from-textureview-to-surfaceview-part-1-how-to-migrate-6bfd7f4b970e)

after change i used  SurfaceView ,but has lots of bug,like black screen @TDUser91

@TDUser91  https://github.com/liuchuancong/better_player/commit/1bfa2bdad01587ab9a860d3b6f1f8bc8a35f9d61

@TDUser91 I don't konw how to fix this, then i must delete this change


Design reusable consistent react components

[Story Points: 4]
[Priority: High]
[Risk: Medium]
Components to design:

Button
Link
Toast
Loading
Inputs
Branding Colors
Fonts

demo_issue_71.webm


[FLINK-26276] Introduce ITCase for table store source and sink

Fix some bugs: see commits
Introduce ITCases

@LadyForest @tsreaper I think I have resolved your comments, please take a look again.


ResAuc Project Blog

Added ResAuc Project Blog Post:

Added a new Markdown (.md) file for my ResAuc project blog post.

Included several images and a background banner in the assets folder to enhance the post.

Updated the author.yaml file with my name for attribution.

Added Project Blog Post:

Added a new Markdown (.md) file for my project blog post.

Included several images and a background banner in the assets folder to enhance the post.

Updated the author.yaml file with my name for attribution.


Flag --auto-open-devtools-for-tabs does not work

A bunch of searchable info on the web indicates that it is possible to open DevTools automatically when you launch msedge programmatically, e.g. via launch.json launch command. But is seems not to be working anymore.
Also some sources indicate that for it to work, it should be the fist instance of msedge (or chrome for that matter) to work.
In my test I found that --auto-open-devtools-for-tabs switch does not work with or without existing msedge instance, while the same flag works with chrome.
Tested via vs code launch.json and with cmd start msedge --auto-open-devtools-for-tabs
MS Edge version:

Microsoft Edge
119.0.2151.44 (Official build) (64-bit)

Revision
44557a29097af522913330e5b41f74810bdfb1a2

Chromium version
119.0.6045.105

Operating system
Windows 10 Version 22H2 (Build 19045.3570)

JavaScript
V8 <IP_ADDRESS>

Just tested this with Edge 120 from the command line, and it worked:
"C:\Users\____\AppData\Local\Microsoft\Edge SxS\Application\msedge.exe" --auto-open-devtools-for-tabs.
Trying with Edge stable in a minute ...

I dont know if is related but I had a trouble with DevTools not oppening when hit F12 or Inspect element button, I have opened an issue telling about it. Then one dude come and comment, he was having the same issue, I saw his name and recognize as Brazilian name (like me). Then my first idea was to change the main language of my browser from pt-BR to en-US then boom...the DevTools window opened.
So, in my case it looks like there is mistake with the language of the browser.

I think these are two different issues. The pt-BR problem is impact all ways to open DevTools. But here the problem only concerns the --auto-open-devtools-for-tabs option.

@captainbrosset I can confirm that the flag works on the later versions, e.g. current Dev build, but only if no instance is already running. Otherwise the flag is ignored.
Tested on 121.0.2220.3 (Official build) dev (64-bit)

Thanks for confirming. 120 releases in 2 weeks from now, so this bug will go away on its own.

but only if no instance is already running. Otherwise the flag is ignored.

I believe this is expected, and was already the case before the bug.

Only takes effect when the msedge.exe instance is not running. Otherwise, the flag will be ignored. You can use tasklist | findstr "msedge.exe" to check.
Tested on 126.0.2592.113 (official version) (64 bit)


vite:import-analysis Failed to resolve entry for package jsforce

When trying to import jsforce in a vite project i am receiving the error:
vite:import-analysis Failed to resolve entry for package jsforce
This appears to be the same as this issue https://github.com/vitejs/vite/issues/7725, which is that the path in the module field of the package.json file must resolve.
In jsforce:
https://github.com/jsforce/jsforce/blob/c83cbd4b0aa4c1706cbe1edaf1eb73ecf64fdf4d/package.json#L26
This path appears to be invalid? I am not familiar with either the project or the intended use of the module path, so I'm not really sure what the right resolution is here?
I can tell you that updating the module path to (what appears to me to be) the correct entry point "module": "./index" seems to correct the issue.

can confirm the error and the solution by @matindow


✨(backend) Contract endpoint: download

Feature Request
We need a endpoint that return the download url of a contract.

I think that this download url could be part of the Contract get/list serializers.
All user that request contract or contract list will want to be able to download it, and this is not restraint by any ability.

Indeed, but I don't think we want to store this link within Joanie database so we will have to retrieve the link from signature provider. That's why I imagined a dedicated endpoint to retrieve this download link instead to retrieve this information on each get/list requests.


LND fails after unlocking wallet (panic: runtime error: invalid memory address or nil pointer dereference)

After I unlock my wallet, I get this error anytime I try to run a lncli command:

[lncli] rpc error: code = Unavailable desc = grpc: the connection is unavailable

This is what I'm seeing in my logs:

Mar 22 15:24:34 thundroid lnd[667]: 2018-03-22 15:24:34.423 [INF] CHDB: Checking for schema update: latest_version=0, db_version=0
Mar 22 15:24:34 thundroid lnd[667]: 2018-03-22 15:24:34.549 [INF] RPCS: password RPC server listening on <IP_ADDRESS>:10009
Mar 22 15:24:34 thundroid lnd[667]: 2018-03-22 15:24:34.549 [INF] RPCS: password gRPC proxy started at <IP_ADDRESS>:8080
Mar 22 15:24:34 thundroid lnd[667]: 2018-03-22 15:24:34.550 [INF] LTND: Waiting for wallet encryption password. Use lncli create to create wallet, or lncli unlock to unlock already created wallet.
Mar 22 15:32:59 thundroid lnd[667]: 2018-03-22 15:32:59.367 [INF] LNWL: Opened wallet
Mar 22 15:32:59 thundroid lnd[667]: 2018-03-22 15:32:59.536 [ERR] SRVR: invalid password
Mar 22 15:32:59 thundroid lnd[667]: panic: runtime error: invalid memory address or nil pointer dereference
Mar 22 15:32:59 thundroid systemd[1]: lnd.service: Main process exited, code=exited, status=2/INVALIDARGUMENT
Mar 22 15:32:59 thundroid systemd[1]: lnd.service: Unit entered failed state.
Mar 22 15:32:59 thundroid systemd[1]: lnd.service: Failed with result 'exit-code'.

I notice that the log says "invalid password", but in terminal it's showing:

Input wallet password:
lnd successfully unlocked!

Any idea what could have gone wrong?
I tried creating a new wallet password (but using the same seed that lnd gave me the first time), and this issue still persists.

How much memory does what machine you're running on have? Does it have any swap?

What's the full panic trace?

Memory on my machine is 1994 MB, and only about 25% of it is being used.
I don't know what a full panic trace is, nor how I can retrieve it 😅

Was this resolved?

Yes :)
For anyone else who experiences this same issue: looks like this was caused by switching from testnet to mainnet. When I deleted macaroons.db, admin.macaroons, and readonly.macaroons, then no more errors after that.


Bounding Box problem with tract_querier...

Dear Demian,
I'm currently use tract_querier with no problem up to now when I ran the following command line :
tract_querier -t whole_brain.trk -a JHU_hau_diff.nii.gz -q JHU_queries.qry -o test --interactive
and I get the following message:
Loading files
Calculating labels and crossings
Traceback (most recent call last):
File "/homes_unix/haemi/Python-2.7.8/bin/tract_querier", line 284, in
main()
File "/homes_unix/haemi/Python-2.7.8/bin/tract_querier", line 183, in main
tracts, img, affine_ijk_2_ras, options.length_threshold, options.threshold
File "/homes_unix/haemi/Python-2.7.8/lib/python2.7/site-packages/tract_querier/tract_label_indices.py", line 71, in init
self.tract_bounding_boxes = compute_tract_bounding_boxes(self.tractography)
File "/homes_unix/haemi/Python-2.7.8/lib/python2.7/site-packages/tract_querier/tract_label_indices.py", line 134, in compute_tract_bounding_boxes
bounding_boxes[i] = BoundingBox(ras_coords)
File "/homes_unix/haemi/Python-2.7.8/lib/python2.7/site-packages/tract_querier/aabb.py", line 23, in new
"Bounding box must have 6 components or be a list of points")
ValueError: Bounding box must have 6 components or be a list of points
In fact, it works when the trk file has been created with one seed per voxel during the tractography processing (MRtrix, see Girard et al. (2014), NeuroImage 98:266-278)) and it crashes when the trk file has been created with 20 seeds per voxels. From my point of view, the difference at the level of the input trk file for tract-querier is about the number of streamlines and therefore the size of the file (up to 7 Go for the file coming from 20 seeds per voxels). This is the only difference I'm thinking about between a trk file that works and the one that did not work.
I'll greatly appreciate your help for that.
Best regards,
Laurent Petit
GIN-UMR5296
CNRS-CEA-Univ. Bordeaux
France

Dear Laurent,
This might be due to a problem in the header of te trk file you are using.
If you find it possible to send me your files somehow I will be happy to assist
All the best
Demian
Demian Wassermann, PhD
<EMAIL_ADDRESS>Research Scientist
Athena Project Team
INRIA Sophia Antipolis - Méditerranée
2004 route des lucioles - FR-06902

Sorry I missed the point that the file is 7gb. Does this work with 2 seeds per voxel? The size of the file should not be an issue.
Best regards
Demian

Thanks Demian for your quick answer!
I don't think that the original number of seeds used to create the trk file
is the problem. But I'm not sure also... I checked with another file where
I think there were 10 seeds per voxels and it works... but I can't reach
the person who created it for now to definitively be sure of that. I can
run a whole tractography with 2 seeds and get the results in 24h... this is
a long process!
Here is a location with the files I used that gave me the error message.
https://www.dropbox.com/sh/t0z47mivddbqcfa/AABllofqYomOOoyXBYVcaLHIa?dl=0
https://www.dropbox.com/sh/t0z47mivddbqcfa/AABllofqYomOOoyXBYVcaLHIa?dl=0
I forgot to tell you that I have no problem to open the trk file in
TrackVis for example.
Thanks again for your help,
Laurent
Laurent PETIT
GROUPE D'IMAGERIE http://www.gin.cnrs.fr
NEUROFONCTIONNELLE http://www.gin.cnrs.fr
UMR5296 - CNRS - CEA - UNIV. BORDEAUX
see my publications here
http://www.researchgate.net/profile/Laurent_Petit3/publications/?ev=prf_pubs_p1
Université de Bordeaux, PAC Carreire
146, rue Léo Saignat, CS 61292
33076 Bordeaux Cedex
Mobile : +33 (0)671281912
E-mail<EMAIL_ADDRESS>2015-01-23 15:44 GMT+01:00 Demian Wassermann<EMAIL_ADDRESS>
Sorry I missed the point that the file is 7gb. Does this work with 2 seeds
per voxel? The size of the file should not be an issue.
Best regards
Demian
—
Reply to this email directly or view it on GitHub
https://github.com/demianw/tract_querier/issues/14#issuecomment-71202662
.

OK, digging a little bit more digging. I think I found the problem.
I believe is the fact that the tractography algorithm is producing tracts that are just one point long while my tool requires a tract to be a segment of at least 2 points.
You can remove those by using the command tract_math included in the tool:
tract_math  <my_tractography> tract_remove_short_tracts .01 <my_tractography_without_short_tracts>
Hope it works!
Best
Demian
--
Demian Wassermann, PhD
<EMAIL_ADDRESS>Research Scientist
Athena Project Team
INRIA Sophia Antipolis - Méditerranée
2004 route des lucioles - FR-06902

This is it! It works after removing that single points. Thank you very much!
Have a nice weekend!
Laurent
2015-01-23 18:02 GMT+01:00 Demian Wassermann<EMAIL_ADDRESS>
OK, digging a little bit more digging. I think I found the problem.
I believe is the fact that the tractography algorithm is producing tracts
that are just one point long while my tool requires a tract to be a segment
of at least 2 points.
You can remove those by using the command tract_math included in the tool:
tract_math tract_remove_short_tracts .01
Hope it works!
Best
Demian
Demian Wassermann, PhD
<EMAIL_ADDRESS>Research Scientist
Athena Project Team
INRIA Sophia Antipolis - Méditerranée
2004 route des lucioles - FR-06902
—
Reply to this email directly or view it on GitHub
https://github.com/demianw/tract_querier/issues/14#issuecomment-71225915
.

Hi Laurent,
Glad it worked, I also changed the new version such that it provides a more descriptive message in case you run into the same problem again.
All the best
Demian Wassermann, PhD
<EMAIL_ADDRESS>Research Scientist
Athena Project Team
INRIA Sophia Antipolis - Méditerranée
2004 route des lucioles - FR-06902


Fix subscription button position on Gist

Introduced in #5623
RGH's selector is less specific than GitHub's .gisthead .pagehead-actions .thread-subscription-status.
Test URLs
https://gist.github.com/kidonng/1f70b2e3de50b519483f6c79dd3d82e2
Screenshot

Before
	After

P.S. apologize for suddenly disappearing for the last two months, I'm going to be busy for real life for another month. Will try to pick up the work I left when I (really) come back.

https://github.com/refined-github/refined-github/pull/5665 ??


Add toolchain option to cross-build

Current cross-build uses the toolchain installed in the host so, we need to add user custom toolchain option to cross-build.

Now, I added target-compiler-triple option for cross-building.

hi, if  target-compiler-triple=aarch64-none-linux-gnu will broken compiler that  output error
    /usr/bin/ld: unrecognised emulation mode: aarch64linux
    Supported emulations: elf_x86_64 elf32_x86_64 elf_i386 elf_iamcu elf_l1om elf_k1om i386pep i386pe
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    gmake[1]: *** [CMakeFiles/cmTC_0ca6b.dir/build.make:100: cmTC_0ca6b] Error 1
    gmake[1]: Leaving directory '/home/seanchann/source/opensrc/flutter-extension/decodetest/build/elinux/arm64/release/CMakeFiles/CMakeTmp'
    gmake: *** [Makefile:127: cmTC_0ca6b/fast] Error 2


Drawer Navigator now works properly

Added some routeName in the conditional in order for it to ignore some of the Drawer Navigator default routes specified in React Navigation library.
Also removed useless "else" statement because a "return" will get excuted first.

@parakhod btw I only tested it with Stack Navigator and Drawer Navigator


Update group results etc for >2 exp groups

need group results & animal avgs  (stats) to calculate for multiple groups correctly

Still needs more help with the stats which I'm not sure I'm doing correctly
Also, should update to automatically handle new groups, or define them somehow?


Initial terminal control

Incorporate ideas / work from:

https://github.com/vkottler/coral/issues/7
https://github.com/vkottler/workspace/issues/21
https://github.com/project-81/firmware/issues/26

Need to start with some kind of event loop / signal handler / epoll thing?

I think next we may need to implement https://github.com/vkottler/coral/issues/8 (FIFO interface that standard input goes to, or other input sources like test input).

Got the basic application processing stdin correctly, next up:

how should we write output? need a terminal writer thing that takes an integer file descriptor

Next steps: use signalfd to handle signals, using epoll.


Update chaos-bugbounty-list.json

add cafebazaar bugbounty program

Thank you @akhilarb

Thank you @akhilarb


Ruin destroyed construction w/ Messenger attached, cannot re-attach Messenger

Describe the bug
The game is no longer playable. After ruining the construction, you should be able to attach the Messenger to a new construction, instead, an error message pops up.
To Reproduce
Steps to reproduce the behavior:

Attach Messenger to construction
Ruin construction from step 1
Attempt to select 1 new construction to attach the Messenger to -> error message saying Messenger already shares a space with a construction
OR Attempt to select 0 constructions to attach the Messenger to -> error message saying you must select a construction

duplicate of https://github.com/ymichael/everdell/issues/67


Proxy should indicate Mixer not available in RESPONSE_FLAGs

At present If Mixer (istio-policy) is not available, then we get
`[2018-08-14T09:26:52.187Z] "GET /apis HTTP/1.1" 503 - 0 69 4 - "<IP_ADDRESS>" "FxVersion" "bae0a141-3e55-9240-9074-8429433ffc32" "mysvc" "-"```
Both UPSTREAM_HOST and RESPONSE_FLAGS are empty.
Mixer client should record the reason in RESPONSE_FLAGS

[ ] At present use UNAUTHORIZED_EXTERNAL_SERVICE response flag
[ ] Add UNAVAILABLE_EXTERNAL_SERVICE or equivalent flag to envoy and use it so that access logs distinguish between the two.

/cc @qiwzhang

Implemented in 1.1:
mixer/test/client/failed_request/failed_request_test.go:  "context.proxy_error_code": "UAEX",


--config should be used for the local now.json, not the global .now.json

In my opinion, we don't need to introduce another flag for the global .now.json file because we already have -t for specifying the token...

Nice :) Looking forward to this as I build two different versions of my project and want to deploy them to different endpoints.

Who not to introduce a new flag for the project configuration containing now.json
Anyways, we should support flags for defining both now.json files. The one with auth and the one with project preferences.

The question is: Why for the global one? Having two flags for setting config files will be really confusion, no matter how we'll call them.


Removing an Instance from InstanceSetParams element.

When changing the selected instance from an instance to "none" crashes.

Application has crashed: C++ exception
mods/BeardLib-Editor/Classes/Map/Elements/coreinstance.lua:144: attempt to index a boolean value
SCRIPT STACK
clbk() @mods/BeardLib-Editor/Classes/Map/Elements/coreinstance.lua:221
RunCallback() 485
SetValue() 475
SetValue() 77
SetSelectedItem() 105
ContextMenuCallback() 59
MousePressed() 220
mouse_press() 401
lib/managers/mousepointermanager.lua:339

Cannot replicate. Need more info.

Unable to replicate anymore, closing. If the issue happens again Ill open this up again.


Qt samples - Fix category in metadata json

Quick fix to category (should be Display information, not DisplayInformation or Display Informtion)
for these 4 Qt samples.
Thanks!

@DonMoJa can you please make sure these updates make their way into the v.next branch?

Thanks for doing this @markjdugger!

I'll do it right now. @markjdugger Thank you for doing this!


--registry-config flag is not honored

Currently, the helm registry commands look in a hardcoded place for the docker config file:
https://github.com/helm/helm/blob/2eab781b35019ce452738012092f9df4ce9fdf00/cmd/helm/root.go#L416
There is a flag which is supposed to do this, --registry-config but as shown in the above code, it is not honored.
This brings up 2 issues

The default spot it checks for the config file is slightly unintuitive.
There is currently no way to instruct helm to use a different location

Happy to do a PR

Thanks for identifying this. Feel free to work on a fix.

@bacongobbler PR as mentioned above.

fixed in #7356.


support async send msg can get sequenceId when throw exception

Motivation
When sending messages asynchronously fails, an exception will be thrown, but it is not known which message is abnormal, and the user does not know which messages need to be retried。
Proposed changes
This change can be supported on the client side,   when throwing an exception need to set sequenceId
org.apache.pulsar.client.api.PulsarClientException
public class PulsarClientException extends IOException {
    private long sequenceId = -1;

    public PulsarClientException(String msg, long sequenceId) {
        super(msg);
        this.sequenceId = sequenceId;
    }

close via #6825


Solve linter issue

On our local machines we aren't checking the linter cheks so we are able to create pull requests that failed due to this issue.
We need to find a way to make lint checks locally.

We did! In my case, I had both Prettier and ESLint extensions installed in VSCode. Disabling the Prettier extension did the trick for me.
Also, here's my settings.json file for ESLint in case you find it useful:
{
  "javascript.updateImportsOnFileMove.enabled": "always",
  "workbench.editorAssociations": {
    "*.pdf": "default"
  },
  "grunt.autoDetect": "on",
  "editor.codeActionsOnSave": {},
  "eslint.codeActionsOnSave.rules": null,
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "prettier.singleQuote": true,
  "editor.formatOnSave": true,
  "window.zoomLevel": 2,
  "eslint.execArgv": null
}

Hey guys, we found a solution for this, basically the problem was that when you clone a repo git automatically uses for the entire project the linebreaks of the operating system you are using and since linter was checking if we had unix linebreaks there was a problem there.
What we decided is to remove the linebreak check on eslint, since it doesn't matter the linebreak you have once you pull the changes git automatically will assign you the linebreaks of your OS.

Ok. There's a command or a setting in VSCode for the line brake too, I can't remember what it is but I had the same problem initially, until I ran that command :)


Bugfix/api docs

some small api docs changes

Verified using:
npm install apidoc
npx apidoc --config apidoc.json --input isso/ --output _apidoc/
firefox _apidoc/index.html

(we probably ought to document this procedure somewhere and also upload those generated apiDocs somewhere)
Thanks for the contribution!