vm.expectEmit doesn't work with anonymous events

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (1a4960d 2024-03-20T00:17:21.177444508Z)
What command(s) is the bug in?
forge test`
Operating System
Linux
Describe the bug
anonymous events (i.e. LOG0) cannot be matched with vm.expectEmit(...)
// SPDX-License-Identifier: MIT
pragma solidity ^0.8.25;

import {Test} from "forge-std/Test.sol";

event Foo(address emitter) anonymous;

contract Dummy {
    function doEmit() external {
        emit Foo(address(this));
    }
}

contract AnonEventTest is Test {
    Dummy dummy;

    function setUp() external {
        dummy = new Dummy();
    }

    function testEmit0() external {
        vm.expectEmit(address(dummy));
        emit Foo(address(dummy));
        dummy.doEmit();
    }

    function testEmit1() external {
        vm.expectEmit(true, true, true, true, address(dummy));
        emit Foo(address(dummy));
        dummy.doEmit();
    }

    function testEmit2() external {
        vm.expectEmit(true, true, true, true);
        emit Foo(address(dummy));
        dummy.doEmit();
    }

    function testEmit3() external {
        vm.expectEmit(false, false, false, true);
        emit Foo(address(dummy));
        dummy.doEmit();
    }
}

Hi @duncancmt thanks for flagging this
Anonymous events are handled, except for the edge case where no indexed fields exist (i.e. contains no topics) in the event.
Created a draft here to implement a fix: https://github.com/foundry-rs/foundry/pull/7461

Due to specific limitations of vm.expectEmit and not wanting to introduce any footguns we'll have to implement a new cheatcode to deal with anonymous events: https://github.com/foundry-rs/foundry/issues/7545


Migrate tooltip (or imlement tooltip in react-components)

The tooltip window is not available in the VsCode plug-in. Either migrate the tooltip window or implement tooltips in the react components for each graph.

This is fixed by the tooltip implementation of the thei-trace-extension for timeline-charts:
https://github.com/theia-ide/theia-trace-extension/pull/283.


Support for C operators on RVV types

Now clang support some C operators (such as: arithmetic, bitwise and subscript) for SVE sizeless types. I think we could support these operator for RVV sizeless too. Actually in some real cases where vl=vlmax and without mask, using operators instead of builtin is more clear and would enable more ir/backend optimizations.
I create a llvm patch here:  https://reviews.llvm.org/D158259

I am not sure this issue should discuss here or in https://github.com/riscv-non-isa/rvv-intrinsic-doc since it isn't an instrinsic problem.

Thanks for bringing this topic up! We briefly touched this today in the SIG Toolchain call. @eopXD (chair of the vector intrinsics TG) will discuss this within the Vector Intrinsics TG.

Hi @jacquesguan,
We will be discussing this in the RVV C intrinsics meeting today [0]. If possible, may you attend the meeting and propose what you have in hand?
Thank you.
[0] Meeting reminder 10/03

Hi @jacquesguan,
We will be discussing this in the RVV C intrinsics meeting today [0]. If possible, may you attend the meeting and propose what you have in hand?
Thank you.
[0] Meeting reminder 10/03

Sorry, I was on vocation in that day and just read this comment right now.

Hi JianJian,
No problem. It was discussed on the meeting that there is not much push back for this syntax sugar that can help porting for libraries that targets other architectures. So I guess we can proceed by proposing your targeting behavior in the intrinsics document.
On the other hand, I am curious about the optimizations you mentioned that would be enabled. Wouldn't it be the same if vlmax is specified in the intrinsic functions?

would

If we use intrinsics, I believe that IR combiner and DAG combine wouldn't work.
by example,
vec_c = vec_a + vec_b; vec_d = vec_c - vec_b;
If we write it in intrinsic, vec_d will still be calculated by vadd and vsub not directly the value of vec_a.


baseURL

I know it's no longer strictly necessary, but consider running the browser loader in NodeJS for example - it can be useful to be able to set what the base path should be for normalizing unnormalized URLs.
It can also be nice to be able to do the following in the browser:
System.baseURL = '/js/';
System.import('blah');

And to be able to control that resolution. I think it's an important feature to keep here.

Also perhaps it's about time to call it baseURI rather?

I think this feature should be provided by custom loaders, not the built-in ones. Also, it's URL; URI is an obsolete term that doesn't correspond to anything in real software (i.e. all software just uses a single parser, known as the URL parser).

So we make it stick to document.baseURI / process.cwd(), and then if users want more than that they can add a custom resolve step?

Yeah. I don't think there should be a loader-specific way of customizing the base URL of a document. It would be weird to have that apply only to resources loaded through the loader (JS files, maybe HTML imports, what else?) but not others (CSS files, images, ...?)

Ok sure I can go with that - I guess it is specifically the NodeJS case that makes it necessary, and a custom resolve can make it work out there fine, or even chdir.

but not others (CSS files, images, ...?)

There is a way https://developer.mozilla.org/en-US/docs/Web/HTML/Element/base

Yes, that was precisely my point, base element for some resources and loader baseURL for others is bananas.

Thinking about it some more ... is a design goal of this spec to work independent of environment?  Outside of the browser, I could see how some "base" would be extremely important.
The Node case does make it necessary.  How do you balance influencing factors of node vs browser vs Rhino vs Nashhorn etc?

My point of view is that base is an environment-specific thing, just like the actual mechanics of translating a URL or filename into a stream of bytes is an environment-specific thing. In node it is process.cwd() modifiable with process.chdir. In the browser it's document.baseURI modifiable with the <base> tag. In Rhino it's something else.

@justinbmeyer if you need some sort of custom base, you can rely on sites() to define the mapping for some of your files.
If we introduce baseURL, you have this big problem:
<script>
System.baseURL = '/static/whatever/';
System.import('path/to/foo.js');
</script>
<script src="path/to/foo.js" type="module"></script>

That's very confusing, because you break the reflection between the script type module and the Imperative api to load the exact same module since you have the ability to modify that value. Even more important, this breaks the mental model that developers have today when they think about fetching things over the network.
Essentially, we want to stick to the principle described by @domenic.


how to use spyder-reports in spyder

hello  i have installed spyder-reports plugin,but when i open spyder ,i don't know how to use this, is there some course or documention

This plugin is unmaintained at the moment and it's also incompatible with Spyder 4.


Error in test_operator.py::test_deformable_convolution

Description
AttributeError: module 'mxnet.symbol.numpy_extension' has no attribute 'deformable_convolution'.
Looks like src/operator/contrib/deformable_convolution.cc has been removed or renamed from codebase, closest attribute I found is deconvolution but I'm not sure if it's about the same thing
Occurrences
To replicate, python -m pytest tests/python/unittest/test_operator.py -k "test_deformable_convolution"

@r3stl355 this has been moved to the npx namespace #19046 and you need to re-build mxnet or update your source code.

Issue disappeared after full rebuild, learning from my own mistakes here


Add validation for rollingCountBadRequests

This fix adds validation for rollingCountBadRequests (added to the dashboard circuit template in v1.4.7), and also sets it to a default of 0 to provide backwards compatibility with older streams.

NetflixOSS » Hystrix » Hystrix-pull-requests #142 SUCCESS
This pull request looks good

Thanks for the PR.  Sorry for the delay merging - I was out of the country on vacation.  In addition, I opened #836 to see if there's a more general approach to allowing dashboards to be more resilient against older metric streams


Change empty template body from error severity to warning severity

Change empty template body from error severity to warning severity.
Original:
# templateName // error

Currently:
# templateName // warning

Pull Request Test Coverage Report for Build 82862

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 4 files lost coverage.
Overall coverage remained the same at 78.597%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder.Dialogs/ObjectPath.cs
1
76.24%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/OAuthPrompt.cs
1
57.23%

/libraries/Microsoft.Bot.Builder.Azure/CosmosDbStorageOptions.cs
2
50.0%

/libraries/Microsoft.Bot.Builder/MemoryTranscriptStore.cs
2
86.09%

Totals

Change from base Build 82853:
0.0%

Covered Lines:
5402

Relevant Lines:
6873

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.5.3
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.5.3


Unity 2018.4.9 changes

Changes applied to scene, prefab, and meta files by Unity 2018.4.9 just by opening and saving all the Mirror examples and touching all the prefabs without actually changing them.

Is this a good idea? Any newer version of Unity can upgrade any prefab from an older version (within reason) on-the-fly. The same cannot be said for downgrading. Re-serializing the prefab data in a future version of Unity will destroy it for use in past versions, regenerating the defaults when it is re-imported.

This annoys me to. But I believe unity was crashing on linux in that version

I opened this for discussion as much as anything.  I'd like you all to take a careful review of what Unity is changing.  Some of it is due to Mirror changes over time, e.g. m_whatever is renamed.  There are ArraySegment bits in there too.
At this point, I can't hardly open a Mirror project linked to master without some of these jumping right into the Changes wanting to be committed.
Please...don't skim / headline surf...read through the files and discuss.

This annoys me to. But I believe unity was crashing on linux in that version

Wrong, Unity was having Linux problems on 2018.2.20
2018.4.9 is the most recent LTS version (September 13th)
I'm on 2018.4.8 right now.

No, since 2018.3.8f1 they have broken physics resulting in crash on start on linux for some people, it's only fixed in 2019.2.

No, since 2018.3.8f1 they have broken physics resulting in crash on start on linux for some people, it's only fixed in 2019.2.

My bad. I think 2018.2.20 was the memory leak issue.
Maybe that's why I get a crash every 1 out of 300~1000 starts? I still have a rare bug where my linux game instance crashes a few secs right after start (always post transport initialization, which means, it doesn't happen instantly, but after a seconds).

Most of the changes are from just mirror changes and are fine.
Have you tried opening the project in older unity after this change?  does it still work?   especially the room example that changed a lot?

All examples play fine in 2018.4.9 and 2018.3.14 with these changes applied.

No, since 2018.3.8f1 they have broken physics resulting in crash on start on linux for some people, it's only fixed in 2019.2.

My bad. I think 2018.2.20 was the memory leak issue.

2018.2.20 had a memory leak. That's what prompted me to get the team to move to 2018.3 at least as I could reproduce the leak leaving it run overnight on a server.

:tada: This PR is included in version 3.20.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.19.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add support for translations

Hi - I needed to make this change for my project and thought others might need to do the same.

Thanks a lot!
I'm assuming based on the fact that the tests are passing that this works fine when the i18n addon is not present and translate is set to false. Have you perhaps also tested what happens when the addon is not present and translate is set to true?
I'd also like there to be an automated test that verifies that it indeed properly translates labels when translate is true, although I'd be happy to look into that myself.

Updated.  Thanks for taking the time to look and suggest improvements.  I'm new to open source and happy to take suggestions.

Excellent, thanks again!
Merging... Will see if I can release as 0.8.0 later today.

Thanks for taking the time to look and suggest improvements. I'm new to open source and happy to take suggestions.

Absolutely no need to thank me! I always feel honored when someone contributes to a project of mine, I'm really not that much of an open source hero myself yet.


[VT-70] vit;  = info

David Patrick on 2013-01-01T21:19:15Z says:
I'm sure that the "info" report is a challenge for vit, but once it's working, it should be invoked with < enter > or :info

Migrated metadata:
Created: 2013-01-01T21:19:15Z
Modified: 2014-02-09T02:23:41Z

steve rader on 2013-03-24T14:32:16Z says:
Added in vit-1.0


champions.rb bug

Please change line 50 in champions.rb from:
e['worst'].to_i +
to:
e['worst-case'].to_i +

@yegor256 seems like there is some weird missing in the script

@yegor256 please do something about this issue

@rultor release, tag is 0.3.2

@rultor release, tag is 0.3.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.3.2

@yegor256 Done! FYI, the full log is here (took me 9min)

@kennyx46 the bug is fixed, new version released. thanks for reporting!

@kennyx46 please, close the ticket if everything is fine now


automatic paste your Url

great thanks your work.
if  it's possible after copy the youtube Url, then the Downloader can automatically paste the Url, or automatically pop-up a tips " whether you want to download this video or audio ? " click "OK" directly start the download,

I don't see the problem with copying and pasting yourself. I wouldn't want an application to intercept my clipbaord buffer.


Use index.json from tldr.sh

Solves same problem as #16, i.e. no tldr page can ever be found. The difference is that this PR changes the url instead of adding the -L option to curl. I think this approach is a bit better because

tldr.sh seems to be the new canonical domain -- tldr-pages.github.io redirects to tldr.sh
https is used

Commit msg below
The index.json file has been 301 redirected to a new url, which caused
curl to fetch nginx's default 301 template to be saved instead of the
actual index. With the index file malformed, no tldr page could ever be
found.
The old url (at github.io) redirects to the new location (at tldr.sh),
but curl doesn't follow redirections without -L.
New url also uses https.

Fixed, thanks for the PR!


Fix "too many branches" lint errors

Description
Prospector 1.2 currently brings up several "too many branches" errors in the code base.

tern/analyze/docker/analyze.py
  Line: 122
    pylint: too-many-branches / Too many branches (18/15)

tern/analyze/docker/run.py
  Line: 102
    pylint: too-many-branches / Too many branches (24/15)

tern/formats/default/generator.py
  Line: 22
    pylint: too-many-branches / Too many branches (16/15)

tern/formats/spdx/spdxtagvalue/generator.py
  Line: 142
    pylint: too-many-branches / Too many branches (16/15) (col 4)

To Do
We can tackle each "too many branches" instance and submit an issue and a corresponding PR. There may be too many changes to submit a PR for all of them.
Background
We will ignore the errors in the meantime. So this is a technical debt task.

Hi @nishakm , can you please assign me this issue?

@sjha2048 Do you want to work on this whole thing or just part of it? I can only add an assigned tag to the issue. So just let me know.

I would like to work on the first file then move towards other ones gradually.

I would like to work on the first file then move towards other ones gradually.

Cool! Please file an issue indicating the prospector error with the first file. Reference that issue in your commit message. Thanks so much for working on this!

I came up with this issue too, i split the deeper branches into a separate function to fix it. Is this the proper way to deal with this issue?

This is what i got by running prospector . in tern/tern:
formats/spdx/spdxtagvalue/layer_helpers.py
  Line: 91
    pylint: too-many-branches / Too many branches (22/15)

tools/container_debug.py
  Line: 121
    pylint: no-else-continue / Unnecessary "else" after "continue" (col 16)

tools/verify_invoke.py
  Line: 88
    pylint: too-many-function-args / Too many positional arguments for function call (col 4)

I came up with this issue too, i split the deeper branches into a separate function to fix it. Is this the proper way to deal with this issue?

Yup! The function for which the "too-many-branches" error comes up needs to be decomposed into smaller functions and then called.

@ForgetMe17 this is what I get:
tern/analyze/docker/helpers.py
  Line: 45
    pylint: too-many-branches / Too many branches (18/15)

tern/extensions/scancode/executor.py
  Line: 48
    pylint: too-many-branches / Too many branches (18/15)

tern/formats/default/generator.py
  Line: 22
    pylint: too-many-branches / Too many branches (24/15)

tern/formats/spdx/spdxtagvalue/layer_helpers.py
  Line: 91
    pylint: too-many-branches / Too many branches (22/15)

Some of these I can tackle quickly as it is code I had recently been working on:
tern/extensions/scancode/executor.py
tern/formats/spdx/spdxtagvalue/layer_helpers.py

I will make an issue for these.
How about you tackle these?
tern/analyze/docker/helpers.py
tern/formats/default/generator.py

I can make an issue for each one of these and assign them to you.

Yes! i will work on them.

Fixed.


documentation, GLUE and bart

Removed credits.
Added tested code to run all GLUE tasks with one command. Encoders can be changed on the fly based on config.
bart summarization documentation updated with latest run. (previous documentation was obsolete)

Codecov Report

Merging #26 (6eda0bb) into main (80590b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   73.10%   73.10%
=======================================
  Files          20       20
  Lines        1372     1372
=======================================
  Hits         1003     1003
  Misses        369      369

Flag
Coverage Δ

unittests
73.10% <ø> (ø)

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

Continue to review full report at Codecov.

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


Spending limit: no logo nor description for LQTY token

Description
No logo nor description appears for an spending limit denominated in LQTY token
Environment

Browser: Firefox
Wallet: Ledger
Chain: Ethereum mainnet

It happens with desktop app for linux too (3.17.2 and 3.22.1).
Steps to reproduce

Create an spending limit in LQTY
Go to the main spending limits section

Expected result
The description and the logo for the limit, like for Ethereum
Obtained result
A blank space
Screenshots

It works fine with other tokens.
Does LQTY logo show up on the Assets page? If not, please follow this article: https://help.gnosis-safe.io/en/articles/4261681-add-an-erc-20-token-to-the-safe-token-list

Closing due to inactivity. Please feel free to reopen if you can provide more info.


Integrate CMake formatter similar to clang-format for C++ code

Maybe we should use some format tools for CMake files as well (we already have it for .hpp and .cpp).
cmake-format is a good candidate, I always use it for my own projects.
Originally posted by @Jinming-Hu in https://github.com/Azure/azure-sdk-for-cpp/pull/5065#discussion_r1372498324

I think this would be a pretty good item for MQ.


Launcher Demo App (OSX): Cannot select files

On OSX 10.11.13, the Launcher Demo App does not allow one to select files for the demos: Manage Websites and Files, and View and Manage Private Data.
With either of those demos, if one uses the option to upload individual files, a file selection dialogue is displayed, but all files listed are greyed-out and non-selectable.  Only entire folders may be selected. i.e. The option to select files behaves in exactly the same way as the option to select folders.
I did try to drag & drop a file onto the View and Manage Private Data demo, but that resulted in the image file I used being displayed within the Launcher Demo window, leaving me with no way to return to the demo interface.  I had to quit and restart the Launcher Demo.

same problem

Should be resolved with the release version 0.2.1

Yes, this is resolved for me in the latest version of the Launcher and Demo Apps.


ToDo app: Page name already taken error.

I've gotten to the Python tutorial step in which we create the reusable TodoApp class and when I run the code I get the following error:
level=fatal msg="Connect app error: Page name is already taken"
I tried writing my own code following the example and then copied the example code, and the result was the same in both cases.
Do I need to create a config file to make this work, or should it work on its own?
Cheers,
todo.txt

I figured the issue: the last line of tutorial code:
pglet.app("todo-app", target=main)
works if you remove "-app"
pglet.app("todo", target=main)

Thanks for the update and trying Pglet!
Honestly, it's been kind of surprising (and pleasant) to see a sudden spike in Pglet usage, so may I ask where did you know about Pglet and what kind of app are you working on and what language? We are still trying to better understand Pglet use cases and need to plan an upcoming release for any new/breaking changes.

Hello, I’m really enjoying working with Pglet. I learned about it from the TLDR daily email, which highlights at least two cool new libraries everyday. I’m working in Python and modifying the todo list demo a bit so I can use it for my boss. He’s obsessed with task lists, so Pglet is the perfect solution.
On Dec 3, 2021, 4:22 PM -0800, Feodor Fitsner @.***>, wrote:

Thanks for the update and trying Pglet!
Honestly, it's been kind of surprising (and pleasant) to see a sudden spike in Pglet usage, so may I ask where did you know about Pglet and what kind of app are you working on and what language? We are still trying to better understand Pglet use cases and need to plan an upcoming release for any new/breaking changes.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Cool, thanks for letting know!


response.json(string) return invalid json

Brief Description
response.json(string) returns invalid json
Sample Code
in a controller:
* index(){
yield response.json("asdf")
}

got asdf, should be "asdf"

I didn't get what u mean by asdf and "asdf", since both are strings

"xxx"

is a string and is a valid json string,
but without quote, it's not a valid json string.
response.json expect returns a valid json, that can be parsed directly, but now, if I use response.json("asdf"), it returns a invalid json

Maybe because returning only "asdf" is not valid either?

response.json() expects an object and convert it to JSON string. Since adsf is a string already you can return it as it is using response.send()

@RomainLanz
``
JSON.parse('"asdf"');  // valid
JSON.parse('asdf'); // invalid

@thetutlage maybe response.json(string) should throw an exception instead of send invalid json string?

Getting back here.
So "asdf" may be a valid JSON syntax representing a JSON value but per the RFC-4627 (which defines application/json format) you cannot use it as the complete body of an HTTP response.
You must use response.send() if you only want to send this string.


build prefix command is naive

This function:
https://github.com/ament/ament_tools/blob/master/ament_tools/build_types/cmake.py#L346-L364
Currently sources dependencies of the package being built in an arbitrary order if they have a local_setup.* file.
Instead it should source the local_setup.* files for the recursive build dependencies and they should be sourced in topological order.

build_dependencies in this context are not the direct build dependencies of the package specified in the manifest. This list already contains all recursive dependencies in topological order.
Therefore I don't think anything needs to be changed here?

Based on the output of #62 it looks like that the order is correct. The problem is that the environment files include not only the recursive dependencies but also upstream packages - basically all packages of the workspace.

@dirk-thomas But they should be skipped because they don't have environment hooks inside the workspace? or is it a problem when you install two workspaces to the same install space?

Sorry, I meant downstream packages. E.g. ament_cppcheck doesn't have any dependencies but currently uses the environment of all packages in the workspace (which is only the case when rebuilding a workspace again).

Oh, it's not just the dependencies, but also anything that comes before in the topological order.

It includes even packages which come afterwards in the topological order.

For packages with dependencies on other ament packages the list and order is correct. But for packages without dependencies on other ament packages the list contains all packages from the workspace. The fix is simple: #63.


added fixes for the unusable strict parameters

Only changed a few lines of code in discovery.py to fix the issues with the script failing on later versions of urllib and the unusable 'strict' parameter for the parse function. Tested and working.

Hi,
Thanks for the pull request and identifying the issue! Unfortunately this will change wig's behavior when run under Python 3.2 as this version defaults to 'strict'=True (https://docs.python.org/3.2/library/html.parser.html). I think a check is needed for the current python version, so that the results aren't different on 3.2.
I've just committed a variation of your request - is it possible for you to test if this solves the issue you're seeing?

Sure I will check to make sure. Version checking is definitely optimal in this case, unless you want to force everyone to upgrade. ;)

On Nov 2, 2015, at 5:22 PM, jekyc<EMAIL_ADDRESS>wrote:
Hi,
Thanks for the pull request and identifying the issue! Unfortunately this will change wig's behavior when run under Python 3.2 as this version defaults to 'strict'=True (https://docs.python.org/3.2/library/html.parser.html). I think a check is needed for the current python version, so that the results aren't different on 3.2.
I've just committed a variation of your request - is it possible for you to test if this solves the issue you're seeing?
—
Reply to this email directly or view it on GitHub.

Works great! I will try to get rid of this pull request on my end. Thanks


fix(qr-code): inline transparent style to show preview in cordova

Fixes lack of camera preview in Cordova

Tested in Chrome, FF, Brave, Safari, and FF browser for Android - LGTM.

looks a bit weird but I assume there is no better way than to modify the body background :see_no_evil:

looks a bit weird but I assume there is no better way than to modify the body background 🙈

This is how this plugin works


Sourcemaps support

This adds a --source-map / -s flag for generating source maps through the CLI.
In addition, when used with ncc run, source maps support is automatically injected into Node with a build of source-map-support/register.
In the API I changed the input option from sourceMap: true to sourcemap: true as it seemed more consistent. Happy to change this back if anyone disagrees. Also added a filename option to the API as the source map needs a filename to reference to.

Codecov Report

Merging #61 into master will decrease coverage by 0.51%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   67.69%   67.17%   -0.52%
==========================================
  Files           3        3
  Lines         260      262       +2
==========================================
  Hits          176      176
- Misses         84       86       +2

Impacted Files
Coverage Δ

src/cli.js
0% <0%> (ø)
:arrow_up:

src/index.js
91.37% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Question: how expensive are source maps? In other words, is there a case for not enabling them for the prod output?

Hmm, I just tested the average time it takes the tests to run with vs without sourcemaps and it was 67s with and 61s without.
I guess I'm just used to builds where source maps times dominate build times... but that certainly isn't the case here.
I've added a commit to set the default to use sourcemaps.
We could still stick with having them off for build by default, although that would need a separate --source-map flag.


Update CEJST dataset to 1.0

CEJST 1.0 adds a bunch of new columns and updates some underlying calculations. You can learn more about the changes from these resources:

Biden-Harris Administration Launches Version 1.0 of Climate and Economic Justice Screening Tool, Key Step in Implementing President Biden's Justice40 Initiative | CEQ | The White House
White House Releases version 1.0 of the Climate and Economic Justice Screening Tool (CEJST)
1.0-cejst-technical-support-document

This PR updates the extract and transform code to handle the 1.0 data.  30% of the counties_wide_format.justice40_dbcp_index values changed between the beta and 1.0 update. Of the 30% of counties with new values, 55% of the values increased and 45% decreased.

Maps!

Not super legible without interacting with it in Jupyter. Here is some more legible information:

There are 22 counties whose index was non-zero in the beta version and zero in 1.0.
There are 200 counties whose index was 0 in the beta version and non zero in 1.0.

It would probably be more legible as raw differences than as percent. It's only a 6 point scale, IIRC


Raise Ecto.QueryError

Resolves #766

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


source-book-ids metadata affected on reimport

just upgraded the plugin to 1.16.6 and decided to re-import all my kindle clippings to confirm the fix for #26
after the re-import, running a git diff shows no change in authors pages, but it does show a diff like the below, one for every singe book. it looks like the the source-book-ids field somehow got messed up.
diff --git a/pages/highlights%2Fbook%2FAbundance%3A The Future Is Better Than You Think.md b/pages/highlights%2Fbook%2FAbundance%3A The Future Is Better Than You Think.md
index a283b65..0197e5e 100644
--- a/pages/highlights%2Fbook%2FAbundance%3A The Future Is Better Than You Think.md
+++ b/pages/highlights%2Fbook%2FAbundance%3A The Future Is Better Than You Think.md
@@ -1,9 +1,9 @@
 type:: Book
-last_sync:: 2022-08-19T06:00:10.749Z
+last_sync:: 2022-08-20T17:59:47.115Z
 author:: Diamandis, Peter H.;Kotler, Steven
 alias:: Abundance - Highlights
 title:: highlights/book/Abundance: The Future Is Better Than You Think
-source-book-ids:: c09ed53e1147551a70d5e4de53802532100ac485f3914c5be09f5f85
+source-book-ids:: c,0,9,e,d,5,3,e,1,1,4,7,5,5,1,a,7,0,d,5,e,4,d,e,5,3,8,0,2,5,3,2,1,0,0,a,c,4,8,5,f,3,9,1,4,c,5,b,e,0,9,f,5,f,8,5,28c432845df7230962345a2f69705cac1f29621c17a2fff060e6b6b3

@Dieterbe, I used the values that you supplied to write a unit test for book ID generation. Should be working now!


Accessing files directly is possible?

Hi,
I'm working on my first project with PicoCMS, so we have a folder in root (/signature) with some images there. I want access those files directly but I think Pico is blocking it.

How are you trying to access them within Pico? I cannot replicate this issue.

Placed a test folder in the root Pico directory,
Placed several test .jpg images in the directory,
Created a test.md markdown file in the contents-sample folder,
Placed an image tag pointing to the above test folder/file ![AltName](http://localhost/test/test.jpg)
Viewed the page at http://localhost/test
Works as expected.

The .htaccess file (https://github.com/picocms/Pico/blob/master/.htaccess/) has exceptions for files and directories on Lines 5&6, so Pico shouldn't be blocking it @marciotoledo

I think I had problem with file permission. Sorry, it's working fine now.


Modal demo is not working correctly

Modal demo is not working correctly
https://react.lightningdesignsystem.com/components/modals/
Testing in Chrome 64

Yes, a few other teams noticed this morning. The markup is present I'm guessing it's something to do with CSS classes.

Should have a fix for you by tomorrow.

@george-norris-salesforce  https://github.com/salesforce/design-system-react/releases/tag/v0.8.7

Ok cool! Just wanted to make sure you all knew about it.


First simple vendor code splitting test

Coverage decreased (-1.2%) to 84.25% when pulling 328bfc7155c017d417a3091e833270c54cf4308d on dev into eb5bac1bf164310385357a5b9e24a30695b84cab on master.


Address 404 error due to change in Navigation URL structure

Change-Id: I2201e3dc299c9d90b614880a8b111b4e8a1eeb71

Recently the ORS changed how they process URLs and it will no longer accept a URL ending with a slash. This fixes the URL in both the Android and iOS code to address a 404 error that was occurring in the Navigation component.

Not sure what is going on with the tests here. I just ran them manually and all passed.


FR: support page preview

It would be very nice if it were possible to hover over the links in the fileclass view and get page previews of the note,
such as the one I get when I hover over links in my  markdown notes and press cmd

This is something I was trying to achieve. Unfortunately that's not currently possible:
The fileclass view is a custom view where the "link" is just a regular html link and doesn't inherit all the cool features of internal links in markdown views.
I've barely managed to reproduce the click-to-open behavior, but couldn't do the same for the hover part.

Dataview supports this out of the box including the use of html in Dataviewjs. For example, you can do this: dv.el('span', "[[Test Page]]")
Maybe worth looking into @mdelobelle?


Image scanning should be based on actual running container image

I suspect that the operator is scanning images based on the workload specification. This might not always be accurate, and I think it would be better if the scanning was based on the actual image the Pod container is running on.
We have developed an in-house operator similar to trivy-operator, but would like to abandon this work and put our effort into the trivy-operator instead. There might be several ways to fix this issue, but in our operator we watch pods (filtering events) instead of the workload. That allows us to pull out the actual running container image digest from the Pod containerStatuses, and that will always give accurate results.
Possibly related issue: https://github.com/aquasecurity/trivy-operator/issues/101

+1 for scanning images by digest.


Added missing api-key message to translate

We need more feedback when things don't work.

this should use @hook.api_key. @hook.api_key should give this error whenever running an explicit command (not a regex/other hook) that's missing the appropriate key.


Tab count limit

The new wrap tabs feature is awesome, but I've noticed that I accumulate a large number of tabs, so they limit my editor space. I usually end up just closing all of them and starting over.
It would be great if there was an auto close tab setting that closed your least recently focused/edited one when you create one beyond the tabCountLimit.
For example, if tabCountLimit = 5, on the 6th file that I open, the least recently interacted w tab would be closed.

I think what you want are these settings:
"workbench.editor.limit.enabled" , "workbench.editor.limit.perEditorGroup" and "workbench.editor.limit.value"

Thanks! @jeanp413


⚠ HDAtoms has degraded performance

In dab181d, HDAtoms ($HDATOMS) experienced degraded performance:

HTTP code: 200
Response time: 2210 ms

Resolved: HDAtoms performance has improved in 730b254 after 1 hour, 21 minutes.


Adjust Sound devices

A feature to mute audio in or adjust audio sources would be verry nice!

This is the only feature that's missing for me as well. A list of my mixer devices with the ability to mute/unmute each one.


Geo widget unit tests (WIP)

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes [link to issue]
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

This PR will be closed because new PR https://github.com/opensrp/fhircore/pull/3273/files has been created


update: refactor blas/ext/base/dnannsumors to follow current project conventions

Resolves #1481 .
Description

What is the purpose of this pull request?

refactor blas/ext/base/dnannsumors to follow current project conventions
Related Issues

#1481

Does this pull request have any related issues?

This pull request:

resolves #
fixes #

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@Planeshifter @Pranavchiku @kgryte can you please review my pr.


Travis not failing on failed test

If a failed test happens it keeps on trucking to a success state.  plumber maybe?
Pay little attention to the library.  WIP.
https://travis-ci.org/RoundingWellOS/marionette.toolkit/builds/52438697

oh, interesting. I'll look into this this weekend.

I'm seeing the same issue. My gulp coverage task is throwing errors that I can't seem to track down. I'd love to help solve the problem if you could point me in the right direction.
https://travis-ci.org/nicksrandall/kotojs

Resolved in master. For me, upgrading is usually as simple as copy+pasting the new Gulpfile.


Add product data

@lauraluiz
For our new shop, how do we fill the shop with product data? With the client or the CLI? The example product data should be under version control.

We already have data, but we still don't have the data in the repo.

Why don't we have the data in the repo?

Data already provided by @CT-Szilvia


Check path concatenation for pre-calculated partials

@brendanjmeade Can you check https://github.com/brendanjmeade/celeri/commit/75a2ec7f0bf976f2f1757e733065eff1ca3fac4c to make sure what I've done is okay?
In celeri.read_data, I defined command.file_name so that I could reference it later:
https://github.com/brendanjmeade/celeri/blob/75a2ec7f0bf976f2f1757e733065eff1ca3fac4c/celeri/celeri.py#L57
In celeri.get_elastic_operators, I find the path and then concatenate it to the .hdf file to read in saved partials. Otherwise I was getting a "file not found" error.
https://github.com/brendanjmeade/celeri/blob/75a2ec7f0bf976f2f1757e733065eff1ca3fac4c/celeri/celeri.py#L1243

@jploveless What are your current thoughts on this?
We could go with absolute paths, which would be super simple.  However, it would also mean that each user has to edit input files to run any model to make them
We could go with relative paths and it could work well if we document a clear standard for this.  That would make sharing easier at the cost of having to adhere to a standard.  Now that I write that, it sounds like a better idea than I anticipated because it would force us all to store files in the same way.
We could also support both via a flag in in the command.json file.
I was leaning toward absolute but now I'm thinking relative, like you suggested, is more appealing.

I think for me the big issue is that the path to the pre-calculated operators is specified in command.json and it would therefore be clearest to specify a path relative to the location of command.json. I think what confused me before is that both the geometry files, which are in the same directory as command.json, and the elastic operator file were both specified with just ./ notation. My own thinking about a typical workflow is to keep all input files in a single directory (or a command/, segment/, block/, station/, mesh/, result type of structure, and it makes sense to me to keep the pre-calculated elastic operators tied to a particular block/station geometry. This is why I always stored it in a specific result directory in Blocks.
One approach could be to write a celeri_directories helper function that just sets up empty directories for each type of input file (like blocksdirs.m as well as a result/ directory into which the actual results as well as copies of that run's input files could be stored. And maybe we could set up celeri_ui to write into a structure like this by default?

I think for me the big issue is that the path to the pre-calculated operators is specified in command.json and it would therefore be clearest to specify a path relative to the location of command.json. I think what confused me before is that both the geometry files, which are in the same directory as command.json, and the elastic operator file were both specified with just ./ notation. My own thinking about a typical workflow is to keep all input files in a single directory (or a command/, segment/, block/, station/, mesh/, result type of structure, and it makes sense to me to keep the pre-calculated elastic operators tied to a particular block/station geometry. This is why I always stored it in a specific result directory in Blocks.

So something like this:
project_name
├── notebooks
│   ├── block_model.ipynb
│   ├── visualize_results.ipynb
│   └── resolution_tests.ipynb
├── command
│   ├── command_001.json
│   └── command_NNN.json
├── segment
│   ├── segment_001.csv
│   └── segment_NNN.csv
├── block
│   ├── block_001.csv
│   └── block_NNN.csv
├── station
│   ├── station_001.csv
│   └── station_NNN.csv
├── mesh
│   ├── mesh_001.msh
│   └── mesh_NNN.msh
├── precomputed_operators
│   ├── elastic_001.hdf5
│   └── elastic_NNN.hdf5
├── output
│   ├── 2022-02-20-17-01-39
│      ├── 2022-02-20-17-01-39.log
│      ├── model_segment.csv
│      ├── model_block.csv
│      ├── model_velocity.csv
│      ├── rotation_velocity.csv
│      ├── strain_rate_velocity.csv
│      ├── okada_velocity.csv
│      ├── tri_velocity.csv
│      └── elastic_velocity.csv
│   └── NNNN-NN-NN-NN-NN-NN
│      ├── NNNN-NN-NN-NN-NN-NN.log
│      ├── model_segment.csv
│      ├── model_block.csv
│      ├── model_velocity.csv
│      ├── rotation_velocity.csv
│      ├── strain_rate_velocity.csv
│      ├── okada_velocity.csv
│      ├── tri_velocity.csv
│      └── elastic_velocity.csv
└── README.md

This would be very organized for projects but a little different from what we have in the repository now.  Maybe we could put command/, segment/, block/, station/, mesh/ in a data folder under project_name?

One approach could be to write a celeri_directories helper function that just sets up empty directories for each type of input file (like blocksdirs.m as well as a result/ directory into which the actual results as well as copies of that run's input files could be stored. And maybe we could set up celeri_ui to write into a structure like this by default?

This is an excellent idea!  Let's agree on the folder structure first!

I think this sounds great! I might suggest we store the precomputed operators within a specific output folder, or else have some way of attaching the corresponding segment, station, and mesh files to each .hdf5 file so that we could make sure we're working with the same model geometry (and make minor changes if need be, i.e. only re-calculate operators for the segments whose geometry have changed, like in Blocks).

Putting the precomputed operators on the output folders is a good idea!  Here's the latest structure.   We'll need to consider the exact structure of where the mesh parameter file goes (in the mesh folder, or in the command folder? I think the latter) and therefore how it should reference the path of the actual mesh files. I'm sorry, I think I'm making this more complicated than it needs to be!

I think either approach would be fine. I'm not sure why I lean toward keeping it as a separate parameter file rather than embedded in the command file; just old habits, I think!

Maybe separate is better so that it would make it easier to swap out a bunch of meshes at once?  Let's keep it separate.  So the proposed folder structure is now:
project_name
├── README.md
├── notebooks
│   ├── block_model.ipynb
│   ├── visualize_results.ipynb
│   └── resolution_tests.ipynb
├── command
│   ├── command_001.json
│   └── command_NNN.json
├── segment
│   ├── segment_001.csv
│   └── segment_NNN.csv
├── block
│   ├── block_001.csv
│   └── block_NNN.csv
├── station
│   ├── station_001.csv
│   └── station_NNN.csv
├── mesh
│   ├── mesh_params_001.json
│   ├── mesh_params_NNN.json
│   ├── mesh_001.msh
│   └── mesh_NNN.msh
└── output
    ├── 2022-02-20-17-01-39
    │  ├── 2022-02-20-17-01-39.log
    │  ├── elastic_operators.hdf5
    │  ├── model_segment.csv
    │  ├── model_block.csv
    │  ├── model_velocity.csv
    │  ├── rotation_velocity.csv
    │  ├── strain_rate_velocity.csv
    │  ├── okada_velocity.csv
    │  ├── tri_velocity.csv
    │  └── elastic_velocity.csv
    └── NNNN-NN-NN-NN-NN-NN
       ├── NNNN-NN-NN-NN-NN-NN.log
       ├── elastic_operators.hdf5
       ├── model_segment.csv
       ├── model_block.csv
       ├── model_velocity.csv
       ├── rotation_velocity.csv
       ├── strain_rate_velocity.csv
       ├── okada_velocity.csv
       ├── tri_velocity.csv
       └── elastic_velocity.csv

Looks great! Seems like this issue will have some overlap with #87 as well as UI development. I can try to write a directory setup function sometime soon.

I've completed this with : https://github.com/brendanjmeade/celeri/commit/36343b5f9a5e847e02aa11fbf9ff6b2c3917232a
It does now require you to create your own "runs" folder in the root celeri folder to store model runs in.  The .gitignore doesn't track anything in this folder.


Adding this plugin in serverless.yml breaks API Gateway Handler

Steps to reproduce:
In a working serverless project:

Install: npm install -D serverless-offline-sns
Add serverless-offline-sns to plugins in serverless.yml
run npm offline start
curl the API Gateway route, fails with the below error message
comment the plugin in serverless.yml, functionality starts to work again

The error displayed:
{"message":"The security token included in the request is invalid.","code":"UnrecognizedClientException","time":"2019-04-23T09:33:15.640Z","requestId":"fc5da850-64db-8786-a965-761720a08619","statusCode":400,"retryable":false,"retryDelay":63.06871720931615}

Weird, looks like an AWS credentials issue. Does configuring AWS creds help?


Can you put bots or at least triggers under source control?

I think it is reasonable to share triggers (or even bots?) under source control and / or share them with team members. Specially if you have a people in different locations with their own network ...

Yes, great idea, this is actually something I've been considering for a while. I built a Ruby tool xcskarel which can actually dump Bot config into a file (designed for this purpose).
Let me know what exact workflow would be appropriate for you, there are many ways we could go about this.

Thanks @czechboy0 , greatly appreciate your swift :) response.
I guess this overlaps with https://github.com/czechboy0/Buildasaur/issues/1
This is how I imagine it may work:
I create a bot and I want to share it with the rest of the team so that they can run their local bots, experiment with it, and when we change it we want to be able to keep track of changes and review the changes ....
I have experience with TravisCI and drone.io and similar CI systems, but very new to XCServer and Buildasuar (awesome name by the way, I pronounce it buildozer btw).
let me look into your xcskarel to see how it works first.

Thanks @alimoeeny, that sounds great. Buildasaur saves its preferences to ~/Library/Application Support/Buildasaur - would you want to add those into source control?

Dude, you save my server password and my ssh passpharse in plain text in ServerConfig.json and Projects.json . This is totally unacceptable.
Can we do keychain?
Do you want me to send a PR for this?
And yes, I think Having Build templates under source control would be enough. Thanks.

There is a ticket for this already: #137. Feel free to contribute if you really need it ASAP :) (just mark it in the ticket if you do, @Brett-Best was planning on implementing that, so let's make sure you guys don't duplicate your work).

Thanks, I'll wait for @Brett-Best

Cool :+1:

@alimoeeny I am hoping to have this done in the next week or so, feel free to submit a PR before that time though.


Stm32 pwm use common pins (Nucleo G0B1, G474, L476, WB55)

In order to ease testing this PR moves timer pwm pins of several nucleo boards to common arduino PINs.

I have updated the prescaler values to match commonly used values (1 and 10.001).
Note:
As discussed with @ABOSTM, we should rethink the st,prescaler binding outside of this PR.
Currently this defines the prescaler register value which is (real-prescaler - 1).
As this is not really what one might expect, I think this should either be documented in the binding or changed, such that the dts prescaler value matches the real-prescaler value.


Migration lock

It would be wise to ensure migrations are not run by multiple processes at once. This could happen if an application would try to run migrations on startup and is then launched multiple times at once (which could be typical for a horizontally scalling application).
Therefore adding a simple lock to the migration process would ensure that at most one process could run migrations at any given time.
Notes:

This could also be implemented outside of this library. Still, it is nice if the RedisMigrator supports this.
This could be optional if users want to opt-in into this. Maybe it should be opt-out to make the library more secure for new users?

we can use something on the following lines, what do you think?
	locker := redislock.New(client)

	// Try to obtain lock.
	lock, err := locker.Obtain(ctx, , migrator.migrations.ID,100*time.Millisecond, nil)
	if err == redislock.ErrNotObtained {
		fmt.Println("Could not obtain lock!")
	} else if err != nil {
		fmt.Errorf("failed due to following error: "err)
	}

	// Don't forget to defer Release.
	defer lock.Release(ctx)
	fmt.Println("I have a lock!")

In you draft you wrote locker.Obtain(ctx, , migrator.migrations.ID,100*time.Millisecond, nil) which seems a bit weired.
I guess a proper key should be added here, something like `prefix + "migration-lock"
https://github.com/fabiante/kvmigrator/blob/4c0dea2a7dfb55460d24e655c281645186e00fb6/migrate.go#L42-L45
That is assuming that the lock is implemented as a function receiver on RedisMigrator which seems to make sense.

Besides that, I have not yet used redislock  that much but it seems to make sense to reuse this instead of writing a manual lock implementation :+1:


Create scubed.rbind.io.json

Add Sophie Lee's blog

Hi @sophie-a-lee,
Thank you so much for contributing and providing resources ☺ I updated the JSON a bit so that it reflects the requirements.
For the bot to pick up only R-related content (and to keep the feed clean), can you add a tag that identifies this content? (Similar to your other categories, the RSS feed would then look like this: https://scubed.netlify.app/categories/r/index.xml (or whatever you want to name the post ☺). Then all posts within the R category are picked up.
If you're also on Bluesky, there's no the possibility to add this to your JSON as well 🙌
Let me know if you have any questions!

I reopened it because it makes it easier to find it again - I hope that's OK! ☺ I also added your bluesky handle - if this looks good to you, give me a quick 👍 ☺


Output markdown skeletons from a legislative code section

We should have a feature that, given the identifier of a legislative code section, outputs a catala skeleton that contains the text of the law articles in the appropriate hierarchical ordering, plus placeholders for the implementation.
An identifier instance is for example LEGISCTA000006147017 which identifies the section "Livre premier : Assiette et liquidation de l'impôt (Articles 1 A à 1656 quater)" from the tax code.

Notes for self : notes on how to experiment with the Legifrance API using httpie
Retrieve token:
export LF_TOKEN=$(http -f POST https://oauth.aife.economie.gouv.fr/api/oauth/token grant_type=client_credentials scope=openid client_id="my-client-id" client_secret="my-client-secret" | jq ".access_token")


[DDW-699] Add Cardano wallet build deps to nix

This PR is for discussion purposes regarding the integration of the new wallet backend API into Daedalus.
It's for open discussion and will likely never be merged in this form. For now we are just trying to determine the best approach to adding the build dependencies.
cc @cleverca22

@Sam-Jeston we should have a DDW ticket for this. Can you please add it to our current sprint? https://iohk.myjetbrains.com/youtrack/agiles/100-15/101-1392
Thanks!

As we now have a good idea of what this will look like, it is on hold until we get 2 things from cardano-wallet, such that we can actually do an initial integration.
IOHK nix cache for cross-platfrom builds: https://github.com/input-output-hk/cardano-wallet/pull/103
Initial IPC interface: https://github.com/input-output-hk/cardano-wallet/pull/388

@Sam-Jeston please let me know what we need to do with this PR since the DDW-699 task has been closed?

Closing this PR unmerged as per discussion with @Sam-Jeston.


Prophet facebook different result

Hi,
Why i have different value for same input when execute prophet facebook?
What can i do?
This are the logs for two execution.
Prophet Log 8.txt
Prophet Log 7.txt
Thank you

Hi @MarcoBIConsultant , the Prophet algorithm can produce some variance in the results as the model has random components. This is discussed in some posts on the prophet repository e.g. #253.
These differences are usually minor, but I see your data has very large numbers which may exaggerate the effect. I would try passing the take_log=true parameter or scaling the data in Qlik to a useful precision.

Is there any way to get reproducible results in the algorithm (by setting a random seed)?

The results are reproducible apart from a small degree of randomness. These variances should be minor and won't matter in most cases as the predictions come with a level of uncertainty.
However, take a look at this discussion in issue 849 on the Prophet GitHub repo. It points out some ways to reduce stochasticity of the model. I'll add a parameter for the random seed that will be applied as mentioned in that thread.
I'm working on some other changes so this will be available with the next release.

Closing this issue as the questions have been addressed and changes committed.


MySQL and DateTimeOffset? LockoutEnd

I'm running the identity system against a MySQL DB and get SQL syntax errors for DateTimeOffset. MySQL does not support the datatype.
Found some related notes on the interwebs

@Quentinb, please report bugs related to core library in main repository. With samples )
Thanks

@sdanyliv thanks for the feedback. I have created one here


The task_dir successfully cleans when the file is absent.

fix #11071

@Burning1020 @mxpv Thank you for viewing

The same issue exists on the rust-extensions side. Please have a look. @mxpv @dcantah
https://github.com/containerd/rust-extensions/pull/342


Enable support for armv6l

Used in Raspberry Pi Zero

 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.

I'm sorry for taking so long to get back to you. We have difficulties to properly test all of our executables in every relevant scenarios for this architecture. Therefore, we are not able to accept it as supported. We would like to focus our resources more on recent architectures.
Thank you for your contribution and I'm sorry that I have to reject it.


Move TransformerDemo into Examples

Move the remaining code in the Transformer folder into Examples. This could go either in Examples/GPT2-WikiText2 or a new folder along the lines of Examples/GPT2-Generation.

I started speculating on some renaming in the Readme within PR #377, to bring it closer in line with the GPT2 training model. Agreed that "Transformer" sounds a little too generic now and that it would be nice to continue consolidating examples.


openmv won't save > 1024 images with certain micro-SD cards

Observed behavior
@efarq writes in email "TinyCamMLs Maiden voyage" on 23-Oct-24: "So TinyCamML2,3, and 5 stopped saving images once they hit about 1000 images saved (which was like yesterday) but did keep recording in their datalog. I reset the tinycammls so they are still out there running currently and I’ll probably check on them Friday. TinyCamML1 seems to be totally fine, which is interesting because it is the only one whose sd card is different (picture below) and it has the voltage divider. All of the sd card formats are the same, so I’m thinking this sounds like some sort of specific sd card bug? I’m going to mess around with the extra TinyCamML and see if I can recreate it but let me know if y’all have any ideas."
Expected behavior
TinyCamMLs keep saving images as long as there is available space on micro-SD card

This thread and in particular this comment suggests using ImageWriter rather than save. save evidently spends time finding open space on the SD card that increases as more space is filled up, maybe getting to the point that the save operation isn't completing when the device goes to sleep.
Another quick band-aid solution could be to add a delay or query the result of
https://github.com/TinyCamML/Boron-and-OpenMV/blob/bcfeacb64a2c065acb4354f63079edc074f35b6f/Firmware/OpenMV/PublishOpenMV.py#L74
and have a blocking loop below it that doesn't let you proceed (and go to sleep) until it responds. I don't know if the save method returns anything, but that's worth looking into also.
Lots of forum posts about seemingly related issues! https://forums.openmv.io/t/saving-stream-of-jpeg-images-to-sd-card/1648

imagewriter is deprecated, so it would be image.ImageIO . this would essentially save a video file, that we would decode later.. this is not that fun, so i suggest a possible hack: new directories every day
so before the while loop:

def get_today_folder():
"""Create and return the folder path for today's date."""
date_str = utime.strftime("%Y-%m-%d")  # Format: YYYY-MM-DD
folder_path = f"./images/{date_str}"
if not os.path.exists(folder_path):
os.mkdir(folder_path)
return folder_path

and in teh loop:

Read current time from UART
poll = uselect.poll()
poll.register(uart, uselect.POLLIN)
poll.poll()
curr_time = uart.read().decode('utf-8')

# Create a file name using the current time and flood state
file_name = f"{curr_time}_{floodstate}.jpg"

# Get the path for today's folder and save the image
today_folder = get_today_folder()
img.save(f"{today_folder}/{file_name}")

# Log the data in DataLog.txt
with open("DataLog.txt", 'a') as file:
    log_entry = f"{curr_time},{floodstate},{gc.mem_alloc()}\n"
    file.write(log_entry)

Just wanted to let y'all know I am currently testing this out right now and I think its working. I'm going to let it run for a few days at home before I commit the code, but basically I added a one second delay before turning off the OpenMV in the Boron's code, and then in the OpenMV code, I added this function definition, using the Boron's timestamp as the basis:
def get_today_folder():     ts = int(curr_time) #time from boron     tsdt = list(time.localtime(ts)) #convert from tuple to list     tsdt[0] = (int(tsdt[0])-30) #subtract 30 years bc micropython starts timestamps at 2000     date_str = f"{tsdt[0]}-{tsdt[1]}-{tsdt[2]}" #extract date     folder_path = f"{date_str}"     if not folder_path in os.listdir():         os.mkdir(folder_path)     return folder_path
and at the bottom:
`today_folder = get_today_folder()
with open("./DataLog.txt", 'a') as file:
    file.write(curr_time + "," + floodstate + "," + str(gc.mem_alloc()) + "\n")

img.save("./" + today_folder + "/" + file_name + ".jpg")`


Enable ability to request all albums from the server

Just like songs, we should be able to get all the albums from the server and display them

Fixed


Removed Python 3.3 support

Now I see, my PR#238 duplicates this. I will remove it.
Dropping python 3.3 makes CI build for other issues (#324 and related PR #325) passing CI.

👍
Python 3.3 is EOL and no longer receiving security updates.
It's also little used.
Here's the pip installs for tablib from PyPI for last month:

python_version
percent
download_count

2.7
56.97%
27,044

3.6
26.61%
12,632

3.5
11.36%
5,393

3.4
4.98%
2,362

3.7
0.06%
30

2.6
0.01%
7

3.3
0.01%
5

Source: pypinfo --start-date -45 --end-date -15 --percent --pip --markdown tablib pyversion

You could also add python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*', to help pip know what to install.

Unfortunately, there's a maintainership issue with tablib. A second maintainer is badly needed.

py33 should be removed from tox.ini as well


Test with all Julia versions

Codecov Report

Merging #10 into master will increase coverage by 1.8%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #10     +/-   ##
=========================================
+ Coverage   88.88%   90.69%   +1.8%
=========================================
  Files           4        4
  Lines          90       86      -4
=========================================
- Hits           80       78      -2
+ Misses         10        8      -2

Impacted Files
Coverage Δ

src/exports.jl
100% <0%> (ø)
:arrow_up:

src/unbound_args.jl
100% <0%> (ø)
:arrow_up:

src/Aqua.jl
100% <0%> (+28.57%)
:arrow_up:

Continue to review full report at Codecov.

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


IntelliSense-like Autocomplete

In a Python file, if I start typing "from math import s", I want it to suggest "sqrt". Is this possible with ncm2?
This would be similar to VS Code functionality.

Found the needed information here
https://github.com/prabirshrestha/vim-lsp/wiki/Servers-Python


Make GOV.UK Frontend assets paths work with http prefix

What’s changed
You no longer need to change the Sass variable govuk-assets-path when using a http prefix; the tech docs gem will add the prefix for you.
Identifying a user need

Users who want to deploy their tech docs to GitHub pages need to handle an additional path prefix to all URLs; they may want to use the http_prefix setting for this. PR #197 added this http prefix to all assets from the tech docs gem, but required the user to also override govuk-assets-path themselves so that assets from GOV.UK Frontend had the correct path. This PR should mean that this additional step is no longer required, making configuration easier for users.

If I understand what this is doing correctly, then I think there's a neater way to do this that doesn't need us to monkey patch the asset_path function.
GOV.UK Frontend allows us to define which function to use when generating paths for images ($govuk-image-url-function) and the font ($govuk-font-url-function).
If no custom function is set, the default behaviour is just to prefix the $filename with $govuk-images-path / $govuk-fonts-path (which default to "#{$govuk-assets-path}images/" and "#{$govuk-assets-path}fonts/" respectively).
I think we can instead pass our own function instead which does the same thing but then calls asset-path with the resolved path including the filename. So something like this:
$govuk-assets-path: "/assets/govuk/assets/";

@function tech-docs-image-handler($filename) {
  @return asset-path($govuk-image-url-function + $filename);
}

$govuk-image-url-function: 'tech-docs-image-handler';

As ever, happy to talk through it if it's helpful!

Thanks for this @36degrees, this is much nicer!
I've pushed a different version of the code now based on your suggestion.

I'm guessing there's no easy way to test this, given the functionality we're interested in is really part of Sprockets?
Otherwise I'm happy with this approach, bar a couple of very minor comments.

Ah tests is a good idea, I'll add some.


ssh call parser issue with not numberic  in methods_util.py

/usr/bin/python3.8 /root/git/spectrum-protect-sppmon-0.3beta/python/sppmon.py --cfg=/root/git/spectrum-protect-sppmon-0.3beta/config_files/sppconnections_default.conf --hourly
...

executing ssh commands, which are labled to be executed for OTHER ssh clients
executing process_stats ssh commands
Error when parsing result, skipping parsing of this result
Exception in FILE: methods_utils.py, Line: 90, Exception: <class 'ValueError'>
Exception Message: value is not numeric
Exception args: ('13123705+total,',)
Error when parsing result, skipping parsing of this result
Exception in FILE: methods_utils.py, Line: 90, Exception: <class 'ValueError'>
Exception Message: value is not numeric
Exception args: ('13123705+total,',)
Error when parsing result, skipping parsing of this result
Exception in FILE: methods_utils.py, Line: 90, Exception: <class 'ValueError'>
Exception Message: value is not numeric
Exception args: ('13123705+total,',)
getting vadps
getting storages
inserting storage info into database
Storing script metrics
total of 9 exception/s occured
Stored script metrics sucessfull

Hello @DominicMW,
A few questions about this error:

What kind of other client were you requesting? What is the OS of that system?
Did this issue occur another time? Can you reproduce it?
If neither of the points above are true: Do you still have the logfile of any of those errors?

I'd like to check and debug this error with live data. It seems to me like the format of any ssh-command did change. I have to manually adjust the ssh-command-parsing.

Hi @NielsKorschinsky , the problem was reported from a customer. I will ask him for more detailed information of examples of the ssh calls/responses from the machines he added.

Thanks Dominic!
The error occured most likley on the SPP-Server (or the ssh-client labeled as SERVER).
I need two steps of verification, where the first one is the most important. Only execute the second one if the same error did occur again.

Step: Execute SPPMon with the --verbose, --processStats and --debug simultanously. If an error is thrown, I need the log file (Result of following command, printed at end of SPPMon):
check log for details: grep "PID XXXX" C:\Path\to\log\sppserver.log > sppmon.log.XXXX

Please attach it as a file since it will be bigger.

If a error is thrown in step 1, you will see just above it on which client it did occur. This should be the host-address of your SPP-Server. If not, make sure only your spp-server is labled as SERVER.

Please do manually log in into this system and execute the following commands. I need a full copy of the terminal output.
mpstat
free
top -b -n1 -p $(pgrep -d',' -f mongod)
top -b -n1 -p $(pgrep -d',' -f beam.smp)
top -b -n1 -p $(pgrep -d',' -f java)

I asked the customer for this information.


Crash with large wallet

From Sunil https://github.com/AlphaWallet/alpha-wallet-ios/issues/2293#issue-746194844:

Know it is a test build, but was slow especially for wallets with a large number of tokens and transactions. Crashed often, you can use to test this 0x26aaD4D82f6c9FA6E34D8c1067429C986A055872

@hboon i found that this crash caused in web3swift library ( not sure if that is right crash, but i checked this one several times). when we trying to call GetBlockTimestampCoordinator.

Hmm.. do you have a fix for it or can you paste a stacktrace and how you reproduced it?


fixed typos, change key word proxy to dimension, add more explanations in Issue Resolution Duration for Filters

fixed typos, change key word proxy to dimension, add more explanations in Issue Resolution Duration for Filters

I am not sure about changing proxy to dimension. They are not synonyms.
Changing this term changes the narrative and meaning of the text (possibly in a good way).
Was there a discussion about this somewhere?

Was there a discussion about this somewhere?

No, this is the place we start the discussion first. :)

I am not sure about changing proxy to dimension. They are not synonyms.
Changing this term changes the narrative and meaning of the text (possibly in a good way).

"proxy" from my understanding is more like represented as “act on behalf of”, like Java Dynamic Proxy Classes or Proxy server.
But here Although code changes can be a **proxy** to the coding activity of a project, makes me confused somewhat.
If I changed it to dimension, it looks more clear that code change is a dimension of measuring the the coding activity of a project
I am not a native English speaker. I hope that I understand correctly on the these metrics, which is quite important and nice! 👍

No worries @eyehwan. It is a good discussion and edits are most certainly welcome.
Both terms work here but they mean different things.
Dimension may be the better word but I would like to get some other opinions on it because it does change the meaning of the text. I am not sure if it changes it in a substantial way, but if it does, we would need to send this metric back to review.
@sgoggins @foundjem -  what do you think?

The Evolution working group thinks this line change to code-changes-lines, code-changes, change-requests-accepted, change-requests, and change-requests-declined should be reverted:
  Although code changes can be a proxy to the coding activity of a project,
  Although code changes can be a dimension to the coding activity of a project,

During the July 7, 2021 Evolution working group meeting, we discussed these points at length. "Proxy" has a very specific meaning that clearly represents how people use many Evolution metrics as a "proxy for activity". This word will remain, and we welcome your other changes!! THANK YOU!!!

@sgoggins  Thanks for the declarations! Then I will push another patch for the rest of fixes.

I have updated another patch, to revert dimension back to proxy.


Samir marin patch 1 node16 and update actions/core to latest

Currently running workflows with this action complains that node 12 will be depreciated and the set-output command is deprecated and will be disabled soon:

Updating actions.yaml to ref Node 16 as suggested by warning.

ran npm update @actions/core to grab the latest (1.10.0) actions/core version, this update adds logic to actions/core to use the new process for setting outputs described here

@jgiannuzzi are you still maintaining this repo? i'm not super versed with JS actions let me know if this is not the right way to go about updating these.


feat: status conditions

This includes/requires #24 & #23

adds a basic controller test
sets ES condition=Ready and refreshTime
adds a noop provider (for e2e testing)

I'll rebase once the above mentioned PRs are merged.

Is this good to merge @moolen?

Yes it is! I just opened up https://github.com/kubernetes-sigs/kubebuilder/issues/2018 to document that issue in kubebuilder


[reflection] impl array reflection

Objective

Currently reflection is not supported for arrays.
Fixes: #1213

Solution

Implement reflection for arrays via the Array trait.
Note, Array is different from List in the way that you cannot push elements onto an array as they are statically sized.
Now List is defined as a sub-trait of Array.

Can Array be made a super trait of List? List is just a growable Array, right?

Can Array be made a super trait of List? List is just a growable Array, right?

Ideally, this was my first thought, however, both traits need to have a unique clone_dynamic function.
However, I guess I could just change these to be clone_dynamic_array and clone_dynamic_list perhaps?

Can Array be made a super trait of List? List is just a growable Array, right?

Yep :)
Just did some refactoring to make List: Array

@NathanSWard are you able to rebase this? I'm tackling reflection and would like to merge this in.

I just tried rebasing this and it's real gnarly. I think we should probably copy-paste and remake this PR fresh, crediting Nathan appropriately.

I just tried rebasing this and it's real gnarly. I think we should probably copy-paste and remake this PR fresh, crediting Nathan appropriately.

@alice-i-cecile I rebased this PR onto main as #4701. If everything there looks okay (and no objections from @NathanSWard), I think we can go ahead and close this one.


17 merge tesia and victors changes into concerto

This adds vtcode and victor_files_n_codes. The most recent version is vtcode. We will clean this up before merging into main.
vtcodes needs to be merged with victor_files_n_codes first, but because these are notebooks, merging will be challenging.

[ ] biolog_files
[ ] check-if to files...

[ ] create_community_model... ; we never went over this together because gapfill model was still being worked on for this
[ ] generate_carveme ; ^ same as above
[ ] generate_rich_media ; relative filepaths to model (../victor_files_n_codes/Generation_of_rich_media_csv_file/updated_modelcsc031FixedCompartments.xml) generates FileNotFound error, otherwise runs locally


Is it possible to set an icon for 'inactive' tab bar item and a different icon for 'active' ?

Our app uses 1 icon for the currently active tab (eg. icon2_selected.png) and a different icon if that same tab does not have focus (eg. icon2_not_selected.png).
Is it possible to set this on the navigationOptions.tabBar object ? (or perhaps some other way?)

sorry i just saw the property in docs. perfecto!
icon: ({ focused }) =>  (
            focused
            ? <Image source={require('../images/home-active.png')} size={25}  />
            : <Image source={require('../images/home.png')} size={25}  />
         )

React Navigation and gesture handle version mentioned here.
 "react-navigation": "^3.11.0",
 "react-native-gesture-handler": "^1.3.0",

This is the way, I achieved.
navigationOptions: ({navigation}) => {
            return {
                tabBarIcon: ({focused}) =>(
                    <TabBarIcon navigation={navigation}
                                image={ focused ? require( './src/assets/icons/bottom_tab_navigator/feed_tab_active.png')
                                : require( './src/assets/icons/bottom_tab_navigator/feed_tab_inactive.png')} />
                )
            }
        }

just ignore the TabBarIcon, its just my custom component. But, you can review this mentioned below to just got an idea.
const TabBarIcon = (props) => {
    const { image, navigation } = props;
    return(
        <TouchableOpacity
            onPress={() => navigation.navigate(navigation.state.routeName)}
        >
            <Image style={{padding: 0, width: 20, height: 20}}
                   source={image}/>
        </TouchableOpacity>
    );
};


remove the fontsize check in xarray.tests.test_plot.test_no_args, bec…

…ause it can be greater than 12 for high-resolution screens

[x] Closes #1584
[ ] Tests passed
Not sure why continuous integration checks are failing. Do not hink this is because of the change, since the master branch is also failing the continuous integration checks.
[x] Passes git diff upstream/master | flake8 --diff
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API
Change is too small for whats-new.rst or api.rst.

Thanks!


Add tests for Sentinel 2 products, limited WKT support

This is not complete WKT support, but most CRSes defined as WKTs we've seen end with an AUTHORITY[] section containing their EPSG code. This adds a simple fallback that extracts this EPSG code using a regexp if it exists.
It's enough for Sentinel product support, and tests are included for the current Sentinel Level1 and ARD products.
See #30 for full WKT support.

Pull Request Test Coverage Report for Build 283

12 of 23 (52.17%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 88.38%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cubedash/summary/_extents.py
12
23
52.17%

Totals

Change from base Build 280:
-0.6%

Covered Lines:
1293

Relevant Lines:
1463

💛  - Coveralls


It doesn't close when pressing on confirmation

Please provide more description with some code samples if possible

This issue would be closed since no further information was provided. Feel free to re-open this issue later if you wan to.


Support for functions with default arguments

Now, console can call functions with default arguments.
Given function definiton
def my_command(required, optional_int : int =5, optional_string : String ="abcdef")

one can register it and call
> my_command abcde
>my_command abcde 7
> my_command abcdef 7 efg

Is there some place where I should write tests for this new functionality?

Is there some place where I should write tests for this new functionality?

Nope...

I've faced a problem. This code causes an error if not all arguments are filled in.
func test(arg0: bool, arg1: int = 42, arg2: String = "default") -> void:
	prints(arg0, arg1, arg2)

Спасибо! Прости, что форсил коммиты...


[Hotfix][CheckStyle][Min] Fix DummyInputFormat class lacks Javadoc

What is the purpose of the change
Fix DummyInputFormat class lacks Javadoc
Brief change log
Add Javadoc to DummyInputFormat class
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no / don't know)
The runtime per-record code paths (performance sensitive): (no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: ( no / don't know)
The S3 file system connector: (no / don't know)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Hi @dianfu, Could you help to have a review. This occured when I built javaDocs in my local, and it is failed for DummyInputFormat  lacks javaDocs. After added, succeed to build javaDocs.


Add possibility to customize contents of pre_randomize

The constraints_utils macro has to implement pre_randomize(), in order to insert constraints. There are cases, though, where the user would like to also add code to pre_randomize(). Something like the following might be ok:
`constraints_utils_begin(some_type)
  `constraints_utils_pre_randomize_begin
    // statement 0
    // statement 1
  `constraints_utils_pre_randomize_end
`constraints_utils_end

The statements between the pre_randomize_begin/end macros should be added to the implementation of pre_randomize.

One question that remains is how to handled super.pre_randomize(). Shall the default contents of pre_randomize_begin/end contain a call to super.pre_randomize()?

Actually, it might be a better idea to stop using pre_randomize() all-together. We might be able to do it by creating a dummy rand object inside the class and using that object's pre_randomize() as a callback to handle global/instance constraints.

Fixed in #11


DOCS Explicitly noting lack of PHP 7 support

Coz you know... apparently 5.3.2 is ambiguous about not supporting PHP 7

I think it would be more helpful to include a section such as this, to answer the inevitable question.
## Why doesn't SilverStripe 3 work with PHP 7?
Unfortunately, SilverStripe has classes named the same as PHP reserved words, such as "Int" and "Float". This means that we are unable to make SilverStripe 3 support PHP 7 without breaking backward compatibility. SilverStripe 4 will work with PHP 7 and will be released in 2016. Until then, we recommend that you use PHP 5.6.

Sure. I'll add it

@dhensby ping

@chillu done.


Re-exclude renaissance-* tests in vendor ibm

Was removed by "Updated disable format" (#2772).
Signed-off-by: Jason Feng<EMAIL_ADDRESS>

@llxia @renfeiw could you please review?