Confusing (or incorrect?) parse result with try_table

When I parse the following .wast:
(module
  (tag $e-i32-i32 (param i32 i32))

  (func $throw-1-2 (i32.const 1) (i32.const 2) (throw $e-i32-i32))

  (func (export "test-throw-1-2")
    (block $h (result i32 i32)
      (try_table (catch $e-i32-i32 $h) (call $throw-1-2))
      (return))
    (if (i32.ne (i32.const 2)) (then (unreachable)))
    (if (i32.ne (i32.const 1)) (then (unreachable)))))

With the main branch, I get this parse tree: (simplified by removing some unrelated parts)
Module {
    ...
    kind: Text(
        [
            ...,
            Func(
                Func {
                    ...
                    kind: Inline {
                        locals: [],
                        expression: Expression {
                            instrs: [
                                Block(...),
                                Call(Id("throw-1-2")),
                                TryTable(
                                    TryTable {
                                        block: BlockType { ... },
                                        catches: [TryTableCatch { kind: Catch(Id("e-i32-i32")), label: Id("h")  }],
                                    },
                                ),
                                Return,
                                End(None), // end TryTable
                                I32Const(2),
                                I32Ne,
                                If(...),
                                Unreachable,
                                End(None), // end If
                                I32Const(1),
                                I32Ne,
                                If(...),
                                Unreachable,
                                End(None), // end If
                            ],
                        },
                    },
                    ty: TypeUse {
                        index: None,
                        inline: None,
                    },
                },
            ),
        ],
    ),
}

Here we have a block, try_table, and two ifs. In total this should have 4 ends, but it only has 3.
Is this a bug, or is my reading is wrong?

Full debug output for the module:
[crates/spec-test/src/spec.rs:232] m = Module {
    span: Span {
        offset: 29,
    },
    id: None,
    name: None,
    kind: Text(
        [
            Tag(
                Tag {
                    span: Span {
                        offset: 39,
                    },
                    id: Some(
                        "e-i32-i32",
                    ),
                    name: None,
                    exports: InlineExport {
                        names: [],
                    },
                    ty: Exception(
                        TypeUse {
                            index: None,
                            inline: Some(
                                FunctionType {
                                    params: [
                                        (
                                            None,
                                            None,
                                            I32,
                                        ),
                                        (
                                            None,
                                            None,
                                            I32,
                                        ),
                                    ],
                                    results: [],
                                },
                            ),
                        },
                    ),
                    kind: Inline,
                },
            ),
            Func(
                Func {
                    span: Span {
                        offset: 75,
                    },
                    id: Some(
                        "throw-1-2",
                    ),
                    name: None,
                    exports: InlineExport {
                        names: [],
                    },
                    kind: Inline {
                        locals: [],
                        expression: Expression {
                            instrs: [
                                I32Const(
                                    1,
                                ),
                                I32Const(
                                    2,
                                ),
                                Throw(
                                    Id(
                                        "e-i32-i32",
                                    ),
                                ),
                            ],
                        },
                    },
                    ty: TypeUse {
                        index: None,
                        inline: None,
                    },
                },
            ),
            Func(
                Func {
                    span: Span {
                        offset: 143,
                    },
                    id: None,
                    name: None,
                    exports: InlineExport {
                        names: [
                            "test-throw-1-2",
                        ],
                    },
                    kind: Inline {
                        locals: [],
                        expression: Expression {
                            instrs: [
                                Block(
                                    BlockType {
                                        label: Some(
                                            "h",
                                        ),
                                        label_name: None,
                                        ty: TypeUse {
                                            index: None,
                                            inline: Some(
                                                FunctionType {
                                                    params: [],
                                                    results: [
                                                        I32,
                                                        I32,
                                                    ],
                                                },
                                            ),
                                        },
                                    },
                                ),
                                Call(
                                    Id(
                                        "throw-1-2",
                                    ),
                                ),
                                TryTable(
                                    TryTable {
                                        block: BlockType {
                                            label: None,
                                            label_name: None,
                                            ty: TypeUse {
                                                index: None,
                                                inline: None,
                                            },
                                        },
                                        catches: [
                                            TryTableCatch {
                                                kind: Catch(
                                                    Id(
                                                        "e-i32-i32",
                                                    ),
                                                ),
                                                label: Id(
                                                    "h",
                                                ),
                                            },
                                        ],
                                    },
                                ),
                                Return,
                                End(
                                    None,
                                ),
                                I32Const(
                                    2,
                                ),
                                I32Ne,
                                If(
                                    BlockType {
                                        label: None,
                                        label_name: None,
                                        ty: TypeUse {
                                            index: None,
                                            inline: None,
                                        },
                                    },
                                ),
                                Unreachable,
                                End(
                                    None,
                                ),
                                I32Const(
                                    1,
                                ),
                                I32Ne,
                                If(
                                    BlockType {
                                        label: None,
                                        label_name: None,
                                        ty: TypeUse {
                                            index: None,
                                            inline: None,
                                        },
                                    },
                                ),
                                Unreachable,
                                End(
                                    None,
                                ),
                            ],
                        },
                    },
                    ty: TypeUse {
                        index: None,
                        inline: None,
                    },
                },
            ),
        ],
    ),
}

Thanks for the report! I believe the parsing of the s-expression form had an accidental bug which will be fixed by https://github.com/bytecodealliance/wasm-tools/pull/1333

This is fixed in version 70.0.0.


Module to convert raw scraped data into a standardised format

Raw scraped data from Issue #4 would need to be processed before it can be used for training the models. We need a module that aggregates the raw data into a single dataset (.csv file) containing the training features and labels.
Each tweet tweeted by someone the user follows should be considered as a data point. All the tweets that were interacted with (liked, retweeted, or commented on) should be classified as a positive instance.
Features should include the tweet text, the user who tweeted the tweet, the global tweet interaction metrics (count of likes, retweets, comments), and the tweet time.
More complex features can also be thought of and included.

Dibs!

Can you please take up #9 first so that we have the scraping module ready?


[Feature for #651] Added Contents button in toolbar for Ebooks

I have added a 'Contents' button in the Ebook Reader toolbar. It uses the epub.js library to scan the XML for the contents section. If the section is found, it navigates to that section otherwise it navigates to the beginning of the book.

Known bugs
In some cases, navigating to the beginning even though contents section is present.
Cause: 'Contents' not listed in the XML of epub file.
Would love to hear your solutions.

@llaske Please review.

Fixed the console error and made the required changes.
Regarding the issue, there is no direct recommendation provided in the epub.js docs. I found this implementation in one of the examples in the repository and refined it to work with our Reader.

Nice. Thanks.


Initial conditions for concentrations

The user should be able to assign initial values for concentrations as parameters.
In case where nothing is set, the default value should be 0 for all the concentrations.

Done in #68


Add performance analyzer plugin to the manifest

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
Add performance analyzer plugin to the manifest
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/2650
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Currently PRs for PA and RCA have not been merged.
https://github.com/opensearch-project/performance-analyzer/pull/267
https://github.com/opensearch-project/performance-analyzer-rca/pull/223
Will merge this PR until clear there.

Codecov Report

Merging #2674 (b3e68e1) into main (9e80f02) will increase coverage by 5.75%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #2674      +/-   ##
===========================================
+ Coverage   94.24%   100.00%   +5.75%
===========================================
  Files         162         6     -156
  Lines        4313       105    -4208
  Branches       19        19
===========================================
- Hits         4065       105    -3960
+ Misses        248         0     -248

Impacted Files
Coverage Δ

src/test_workflow/test_recorder/log_recorder.py

...est_workflow/bwc_test/bwc_test_start_properties.py

src/test_workflow/test_args_path_validator.py

src/test_workflow/bwc_test/bwc_test_runner.py

...test_workflow/perf_test/perf_multi_node_cluster.py

src/manifests/distribution.py

src/test_workflow/test_kwargs.py

src/test_workflow/perf_test/perf_test_suite.py

src/ci_workflow/ci_target.py

src/ci_workflow/ci_test_manifest.py

... and 146 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


How to pass object to :style?

There is ability to pass object with multiple properties in v0.14 ?
What I need:
al-init="myStyle="{backgroundColor: 'red'}"
...
:style="myStyle"

but it does not work in 0.14 (before that I used al-style and everything worked http://jsfiddle.net/vxhdcj3f/1/)

Made the conversion of the object into a string manually, but this makes the code heavier
http://jsfiddle.net/vxhdcj3f/2/

Hello!
it was deprecated

usually you don't need it (I've never used it in real)
it requires deep-watch (it's slower than simple watch)
ugly api

instead you can use :style for different props, like:
<div
    :style.background-color="color"
    :style.border-color="color">

example: https://jsfiddle.net/uzjgowmg/
also in majority of cases, :class is used, or if you need something complex you can use (direct) directive.

@lega911 thanks for the quick response. I did it using directive and deep watching


Flake8 rule 'E501' reported an invalid point

Flake8 reported an invalid point for the rule E501, with the messge line too long (98 > 79 characters).

Debug information:
Atom version: 1.15.0
Flake8 version: 3.3.0 (flake8-docstrings: 1.0.3, pydocstyle: 1.1.1, mccabe: 0.6.1, pycodestyle: 2.3.1, pyflakes: 1.5.0) CPython 3.6.1rc1 on Windows

Are you able to share code to reproduce this?
There is currently a known bug with unicode characters that might be causing this, but without code it's impossible to say.

https://github.com/Tikani/BioStat/blob/master/lab1.py
Atom reported error right at the first line and it's strange because I didn't see such a long string there.

Looks like line 37 is the longest line, and it's full of Unicode characters so it's very likely that pycqa/flake8#306 is the problem here. It's been fixed in their master branch, so the next release of flake8 should fix this for you.
The message is only reported on the first line since that is the only line guaranteed to be in the file and we can't use the position from flake8 since we know that it isn't valid.


test(provider): RequestsProvider of BlockchainProvider2

Closes https://github.com/paradigmxyz/reth/issues/10343
The change turned out to be more invasive than expected because I added an argument for randomly generating requests into random_block and random_block_range.

how about otherwise making new function for when new param is used? seems like most calls take None for that param, so mainly not needed. this can be done easily by adding a third function, with the body of the current function, so that the function body of the original function, can just call my_func(arg1, arg2, .., None)

yeah makes sense, agreed. Maybe we can just pass a struct instead of fields one by one, and the struct will have a Default implementation.

hmm, actually we have just one optional field now which is requests_count. I think we should leave it as-is and refactor into a struct when needed.


feat(github plugin): resolve github commits and pull requests

enrich commits with pullRequestId and enrich pr with commitSha
Summary
enrich commits with pullRequestId
enrich pr with commitSha
Screenshots
prs table:

commits table:

Does this close any open issues?
Will close issue #950

pr/commits relationship is reprsented in comma separated format? it doesn't seem right.

pr/commits relationship is reprsented in comma separated format? it doesn't seem right.

Fixed in the new pr


Revert "audit: check the version does not decrease."

Reverts Homebrew/brew#1366

https://bot.brew.sh/job/Homebrew Core/10325/version=el_capitan/console
https://bot.brew.sh/job/Homebrew Core/10325/version=yosemite/console


[FEATURE]: Sync Discord Role to Admin Permissions

Scope
Web, In-Game Menu
Feature Description
1.) Create Permission Groups
2.) Allow Permission for Group Mapping to Discord Roles
3.) Now admins are automatically given permissions
Use Case
Way easier management of admins and permissions
Proposed Solution
Use existing whitelist abilities and apply that to this use case
Additional Info
No response

Thanks for the suggestion, but it's not gonna happen.
txAdmin itself must be the single source of truth of all permissions.


Merge prTesting_201908110939 branch to master branch

Docs Build status updates of commit 5d3b83d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[QUESTION]: error getting credentials - err: exit status 1, out: ``

Is your question answered elsewhere?

[X] searched the existing issues.
[X] read the documentation.

What is your question?
I'm following setp-by-step the doc:
https://milvus.io/docs/install_standalone-docker.md
But when I run the following command:
milvus % sudo docker compose up -d
I get the error:
2023/12/15 15:43:24 must use ASL logging (which requires CGO) if running as root
[+] Running 0/0
 ⠋ standalone Pulling                                                                                0.0s
error getting credentials - err: exit status 1, out: ``

Hardware Overview:
Model Name:	MacBook Pro
Model Identifier:	MacBookPro18,4
Model Number:	Z15J000EMPO/A
Chip:	Apple M1 Max
Total Number of Cores:	10 (8 performance and 2 efficiency)
Memory:	32 GB
Anything else?
No response

[Fixed]
The problem was that I was running as sudo
The fix is to run as:
$ docker compose up -d

[Fixed]
The problem was that I was running as sudo
The fix is to run as:
$ docker compose up -d


Several bug fixes added including category bug. Added segmap saving.

Thanks for the contribution, I appreciate figuring out the bugs. I won't accept this PR because:

It would be better to integrate the SegMask flow as part of the UI itself. Instead of a CMD argument.
The bug fixes should be a separate PR from this one.

As discussed over DM, it would have been great if you could make a separate PR that I could accept, but you asked me to copy the changes directly. Thus, instead I have fixed those bugs slightly differently. Closing this PR for now. Thanks for you help!


QUESTION: Using this with other starter templates

Hello there,
I am looking to build a simple side project demo and have been using this starter template for an astro app: https://github.com/markteekman/accessible-astro-dashboard/tree/main
Wondering if your component library able to implement with other templates that already have design systems? I see that as part of the installation it overrides the globals.css file (this starter app already has that defined) so will this basically lead to conflicts?

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Fix: Minor mistakes with docs

So sorry for having scuffed this up 15 times

 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.


1.8.34 - Create a global Day and Night plugin

Request to create a global day and night plugin for all players to enjoy...

This is not the place for feature requests. Please refer to https://github.com/runelite/runelite/issues/new/choose
Possibly already requested in form of #7451


Implement manifest selection

Closes #17 and #8
TODO:

[x] cleaner implementation and cleanup
[x] handle new modules addition changes on manifest
[x] watch manifest for on disk changes
[x] status bar item for current active manifest

Most of the things are now implemented and probably need a commit squash.
One thing I'm not sure is whether the status bar item should open the active manifest or activate select-manifest command.
It also needs more testing for edge cases

Btw, I sent you an invite for having write access to the repository just in case :)

thanks :D

Could use a rebase

I'll squash every commits for easier rebase

It shows this message when there is some error on the manifest


maba:webpack:compile --env=prod doesn´t work

Running the command
php bin/console maba:webpack:compile --env=prod

results in
[Symfony\Component\Console\Exception\CommandNotFoundException]  There are no commands defined in the "maba:webpack" namespace.

Seems like maba:webpack:compile doesn´t accept any params

Did you register MabaWebpackBundle inside AppKernel? Maybe it's in dev environment if block?

Yes, in fact the php bin/console maba:webpack:compile command, without the env param works fine.

If it's working in dev environment (which is default if you do not pass --env) and not in prod, I would suggest to:

check if bundle is configured to be used in production environment (see AppKernel). It must be outside of if (in_array($this->getEnvironment(), array('dev', 'test'), true)) {
clear var/cache/prod just to be sure that it's not cache-related somehow

In any way, I cannot see how this could be related to the bundle itself and not to the configuration of it in your current set-up.

Just had a similar problem after updating from a 0.5.x version. Even console cache:clear failed trying to access DynamicAssetProvider class, which presumably no longer exists. Manually removing var/cache/prod fixed it.


Why does this need browser history permission?

When installing via the chrome extension store:

.. why does this need browser history?

This is to run the script again when the search conditions change from the sidebar.
To detect URL changes, the chrome.webNavigation API is used instead of the popstate event, since the popstate event is not available in Edge or Chrome. To use this API, browser history permission is required.
There is another way to monitor DOM changes, but I'm not using this method at the moment because it causes a lot of unnecessary processing than the method of detecting URL changes.
However, after reading your comment, I feel that it would be better to stop using the chrome.webNavigation API and switch to the method of monitoring DOM changes so that users will not be worried.


Use generics

This is a first stab at fixing #121
It tries to leverage the new generics support from PHPStan to obtain a cleaner way to detect Prophecy's mocks, and allow the ObjectProphecy<OriginalClass> PHPDoc notation.
This is still WIP: I've tried to follow @ondrejmirtes suggestiong from https://github.com/Jan0707/phpstan-prophecy/issues/121#issuecomment-593096285, but I don't know how to proceed to make PHPStan recognize the calls of the original class on the ObjectPropecy.

@localheinz WDYT? You're welcome to take this and put it forward.

@Jean85 @ondrejmirtes
As you have seen in my attempt to make use of genetics in #155, I have no idea how to sort this out.
I’ll happily take a look, though, when you have.

@localheinz I've looked now at yours PR, and we have many parts in common. I didn't split the stub per-class, I'll try that because I have some autoload errors right now, it might help.
@ondrejmirtes I've (partially) followed your suggestions, and I've restored a big part of the previous codebase, except from ObjectProphecy (which is replaced with a generic) and RevealDynamicReturnTypeExtension (since its job of intersecting mocked types is now done upfront by the generic plus the WillExtendOrImplementDynamicReturnTypeExtension).
Almost all seems fixed, the only thing that I need to fix now is the return type inferred on ObjectProphecy calls; it sees the return type of the original method... MethodsClassReflectionExtension doesn't seem to work! Any tips?

Dump what's going into the hasMethod and getMethod, I guess that hasMethod returns false.

Thanks @ondrejmirtes, that led me in the right direction!
CI wa crapping out on 7.4 + --prefer-lowest. Now it's all green!! :tada:

@ondrejmirtes
Can you take a look and let me know if this makes sense, please?

Thank you, @Jean85 and @ondrejmirtes!


Clip Children has weird behaviour with background image

Tested versions

Reproducible in: 4.2.stable

System information
Godot v4.2.1.stable - macOS 14.2.1 - Vulkan (Mobile) - integrated Apple M2 Pro - Apple M2 Pro (12 Threads), also reproducible on Windows 10
Issue description
I'm converting my project from Godot 3 to 4, and I've tried implementing a 2D sprite mask. Godot 3 implements this using PointLight2D, but Godot 4 needs to use a Sprite2D with Clip Children. But I get very strange behaviour when I have a background image behind the sprite mask, the mask copies the background image and moves around when moving the sprite mask or it's children around as well, even though the background image isn't a child of the mask.
I've uploaded a preview video of what I mean as well as the sample project since it's hard to explain. This is only an issue when Clip Children is enabled.
https://github.com/godotengine/godot/assets/36524720/29f6898e-159a-4ebd-aafd-e38842709a8d
Steps to reproduce
I've uploaded the sample project, just move the mask, star and Label nodes around to see the issue. Also panning, and zooming in/out in the editor produces the same issue.
Minimal reproduction project (MRP)
Godot_text_mask_bug.zip

Thank you for the report. This is a known issue and has an open report already: https://github.com/godotengine/godot/issues/75956
So I'll close this issue in favour of https://github.com/godotengine/godot/issues/75956 so we can track the issue in one place


GraphiQL: meta tags

This is a fix for character encoding issue in some browser, e.g. FireFox reports on missing encoding meta tag:

The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol.

Plus we align with other meta tags currently set in express-graphql, see https://github.com/graphql/express-graphql/commit/775914b5689cfc781c99cf705292e1dcffd2239a

Thanks!


Merge gitlab GitHub

Fixed parsing of z_id, before it was None when in cases where it should not have been.

This is a problem you should solve with tags. The semantic versions here
are used in pypi.
On Thu, Feb 13, 2020, 09:50 martinapippi<EMAIL_ADDRESS>wrote:

@martinapippi commented on this pull request.
In pygef/_version.py
https://github.com/ritchie46/pygef/pull/49#discussion_r378721915:

@@ -1 +1 @@
-version = "0.4"
+version = "0.4.1b"

I would prefer not to, I did this merge request in order to have the same
repository in gitlab and gihub. We needed to bump the version in gitlab
because if you don't bump the version and make changes in the docstrings
they are not updated in the documentation (Sphinx). Furthermore, we just
added a patch (1b) that was not present before.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ritchie46/pygef/pull/49?email_source=notifications&email_token=AAXCBGARJ5AACEES7WSPAGTRCUCWTA5CNFSM4KT2JQOKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCVLUMEA#discussion_r378721915,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXCBGAYX37HUU3GQQUU3KDRCUCWTANCNFSM4KT2JQOA
.


Add Worksheet::copy_range for Copy/Paste cells?

I'd like to copy a row and insert a couple of rows with same values as below lines:
        let mut sheet1 = book.get_sheet_mut(&1).unwrap();
        sheet1.insert_new_row(&9, &3);
        copy_range(&mut sheet1, "A8:L8", &1, &0);

My copy_range is similar to Worksheet::move_range but for Copy/Paste workflow.Can we add a Worksheet::copy_range as below?
fn move_or_copy_range(..., remove: bool)
pub fn move_range();
pub fn copy_range();

@piaoger
Thank you for contacting us.
Yes, I would like to know if you have any questions. I think it is probably possible.
We will address this in the next update.


feat(kotlin support): add support for kotlin DSL

Gradle offers a new way to write build.gradle files using Kotlin (Gradle DSL) which requires files ending in: kts. See here  for more info on Gradle DSL. These currently do not work with this package. This PR aims to add support for these files. Apart from the file ending there are minor syntactic changes, like double quotes and equals.
I've tested this locally against a project that has a build.gradle.kts file using something like:
node ../capacitor-set-version/bin/dev ~/Developer/example -v 1.5.0 -b 150

And it worked as expected (with regular build.gradle and build.gradle.kts and an iOS project), however, I am unable to get the tests working. I haven't used this test suite before, but tried to replicate the other test cases. Unfortunately I could not get them to pass. Any advice on this would be appreciated.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


404 when using the package under a sub-directory

Hi there!
I'm having a problem when using this package under a sub-directory.
When your app is hosted under a sub-directory like example.com/admin, the request made to /meta.json returns 404 (Not Found) because is trying to request  example.com/meta.json instead of example.com/admin/meta.json.
One quick solution could be append window.location.pathname to the fetch's url ${window.location.pathname}/meta.json.

hi @fedebabrauskas , sure will update the path location

Hi,
In my project it's working on localhost but not after deploying to our server.
The problem is that startpage path of the web app is different from context-root and meta.json is stored under context-root path.
So we have something like this: http://host/context-root/startpage
But we need to fetch: http://host/context-root/meta.json
The proposed solution:
${window.location.pathname}/meta.json
is not working because it's pointing to http://host/context-root/startpage/meta.json
To make it work with the context root path we need to change it to:
window.location.pathname.substring(0, window.location.pathname.indexOf("/",2))/meta.json

As @kris-pl83 said, we can't use window.location.pathname for all scenarios.
We could improve it to match more cases: window.location.pathname.substring(0, window.location.pathname.lastIndexOf('/')), but it's not the best solution.
Let's say that your webapp is running under /context-root/startpage. How would you know if your root directory is /context-root/startpage or /context-root?
A better approach IMO could be implementing a basePath variable in front of /meta.json, so we can configure it when using the library: useCacheClear({ basePath: '/context-root' }).

Hi,
I agree with @fedebabrauskas.
Adding additional paremeter could be better solution.
Regards,
Kris

Hi,
First, thanks a lot for this package!
I just wanted to chime in to say that a "basePath" property would be awesome and solve my use case with Create React App.


Un bash-ify build script

Please write simpler and more readable scripts!

@cherry-zededa-com closing since the patched file has changed too much anyway


Add WithdrawalFeeEstimate query.

Current dependencies on/for this PR:

main

PR #40   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 21, 2:40 PM: @@zhenlu merged this pull request with Graphite.


Write-Access for developers

to speed up new releases it is important that some developers get write access.
otherwise different people are working on the same issues
release often + release early
code should find it's way to the project and not queue up in a mailbox

this repo is actually the code being served by github.io, not the code one could run on one's own computers.
To get a cleaner structure, I have created a repo https://github.com/refugee-phrasebook/backendscripts, where additional code can be shared.
I have given write access to the Backend team, and I have added you to that team
HTH
S

Please review the permissions:
git tells me:
You may not have permission to access backendscripts. Check Preferences to make sure you’re still logged in.
remote: Permission to refugee-phrasebook/backendscripts.git denied to michal-fre.
fatal: unable to access 'https://github.com/refugee-phrasebook/backendscripts.git/': The requested URL returned error: 403

Am 17.09.2015 um 14:36 schrieb SebastianNordhoff<EMAIL_ADDRESS>this repo is actually the code being served by github.io, not the code one could run on one's own computers.
To get a cleaner structure, I have created a repo https://github.com/refugee-phrasebook/backendscripts https://github.com/refugee-phrasebook/backendscripts, where additional code can be shared.
I have given write access to the Backend team, and I have added you to that team
HTH
S
—
Reply to this email directly or view it on GitHub https://github.com/refugee-phrasebook/refugee-phrasebook.github.io/issues/19#issuecomment-141067108.

sorry - my fault - forgot to press the link in the invitation :)


Copyedit of test page

What's changed
Copyedit of docs test page re: admonitions
Related issues
PR #3259

LGTM


Grid size in monthly view should not be hardcoded

Blank cells in the last week of November are OK-ish but a full week of them really isn't.

I think it would be great if CalendarBorderView takes the actual number of weeks into account.

Fixed


Getting properties

Hello. I'm doing initial tests for getting and setting properties on our assets. I have 'mySkelMesh' loaded in the editor, and yet most of the properties do not return correct/any values.
A couple of the propeties return values:
import unreal_engine
skelMesh = unreal_engine.find_object('mySkelMesh')
unreal_engine.log(skelMesh)
<unreal_engine.UObject 'mySkelMesh' (0x000001E1064BDF00) UClass 'SkeletalMesh'>
unreal_engine.log(skelMesh.Skeleton)
<unreal_engine.UObject 'Base_Skeleton' (0x000001E12DC18400) UClass 'Skeleton'>
unreal_engine.log(skelMesh.Materials)
[<unreal_engine.UObject object at 0x000001E122EC4DB0>, <unreal_engine.UObject object at 0x000001E122EC4DB0]
But the all-important SourceFilePath does not return anything:
unreal_engine.log(skelMesh.SourceFilePath)
nor does .LODInfo return accurate values set to 2.0 with the editor:
unreal_engine.log(skelMesh.LODInfo[0].ScreenSize)
returns: 1.1674562077543703e-26
If it helps, you can use this to list the values of all the properties on this SkelMesh too:
for property in skelMesh.properties():
....unreal_engine.log('{0} is: {1}'.format(property,skelMesh.get_property(property)))
Do you have any suggestions?
Thank you.

Hi, regarding SourceFilePath (if i remember correctly) it is a field of AssetData, it does not show as a valid property of SkeletalMesh (or its parents).
What is strange is the skelMesh.LODInfo[0].ScreenSize that does not return the value you set. I'll check it.
Thanks for reporting

This has been fixed thanks to proper structs support in latest releases


Caster tax is removed

With current default settings the 0.1s between hard casts for eg BLM seems to be removed. Is this on purpose?
see https://www.fflogs.com/reports/a:4pg7BQ1Tzxtkv6LM#fight=last&type=casts&view=events

Either grab the opcode file again from StaticData folder, or by downloading the newest release at here. If ActorCast opcode is incorrectly set, XivAlexander won't be able to tell whether it's a cast, resulting in effective removal of caster tax.

Thank you very much for the reply and fix!


Fix has on non-inherent fields

See the following example to reproduce:
#show figure: it => it.has("gap")
#figure[]

It should print false, but prints true on main.
This PR fixes that issue.

I think this PR is good but I am tempted to add a second has method generated by the elem macro to avoid needing to allocate a Value when checking whether a field exists 🤔

Thanks!


Grammar in ngraph Logistic Regression Part 2

ngraph/examples/walk_through/Logistic_Regression_Part_2.ipynb
says : "The axes creation is conecptually the same as before, except we now add a new axes H to represent the new feature space."
should said: "The axes creation is conceptually the same as before, except we now add a new axes H to represent the new feature space."

I have fixed this in the internal version.


How to use in Angular?

I want use it in Angular. And I can not find example. Can you provide examples?

Hi, I  implemented with angular 6.
in angular.json add
 "scripts": [ ...               "node_modules/jexcel/dist/jexcel.min.js",               "node_modules/jsuites/dist/jsuites.js"             ]    "styles": [             ...               "node_modules/jexcel/dist/jexcel.min.css",               "node_modules/jsuites/dist/jsuites.css"             ],
in any component.ts  add
declare var jexcel: any; //declare jexcel type
then you can use
   tubuExcel = jexcel(document.getElementById('my-spreadsheet'), {       onchange : this.save,       oninsertrow: this.insertrow,       ondeleterow: this.deleterow,     });
Also in component.html add

Hi, is it possible you to create an Angular example here? https://codesandbox.io/
It would be very much appreciated, thanks.

@Mekonom Sir, which jexcel version did you apply with angular 6? Could you please give a sandbox or code example? Thanks

https://codesandbox.io/s/jexcel-and-angular-cexs1 there are some third party @types that can be found and used.

@paulhodel , thank you Paul, I figured it out. https://codesandbox.io/s/jexcel-and-angular-forked-2boyp With Angular 6, there is fine without @type/jexcel, but you should use some previous version of jexcel, cause typescript version is required in a narrow range for Angular 6.

We are working on the official @types for both distributions Pro and CE. This should be included in the next releases

We are looking forward to it. Good job, Paul! You make a different!


Feat: Donations

Context
When donating to a grant (either from a user or from a matcher) we want to deposit the money into the grant account first before releasing it to the grant creator. Once an arbitrary constrain has been cleared (e.g. a deadline has passed), the money will be transferred to the grant creator, else, the grant can be cancelled and the money be refunded to each donor.
Implementation
A Donation will be a PDA derived from a grant and a user, so a user can only have one escrow account per grant to avoid multiple escrow accounts. Users should be able to donate again, which will increase the funds of the Grant account.
Another Link account will be created along each donation, which will be derived from the grant + donation_count of it to be able
to track all current donations related to the grant.
To cancel a donation requires that the grant it relates to has already been cancelled.
Action Items:

[x] Donation struct
[x] Donation creation
[x] Make sure a user can only have one Donation account per grant
[x] Grant release (give the money to the receiver)
[x] Donation cancellation (return the money to the donor)
[x] Increase the amount donated
[x] Make sure all Donations related to a grant can be queried without prior knowledge of the users who donated
[x] Make sure release and cancel actions can only be made when an admin or DAO authorizes it
[x] Cancelling a grant enables the cancellation of all donations related to it
[x] Add helper functions for create/increment a donation, and to cancel grant + refund grant's donations

Great suggestion, I'll make sure to do that when merging with your PR

Just merged the changes from #23 which is already on main, which required some refactoring work on the anchor side, and a lot of it on the tests side.

Did not review app yet btw

./src/transactions/index.ts:3:31
--
13:33:29.618 | Type error: Cannot find module '../../../target/types/grants_program' or its corresponding type declarations.
13:33:29.618 |
13:33:29.619 | 1 \| import * as anchor from "@project-serum/anchor";
13:33:29.619 | 2 \| import { Program } from "@project-serum/anchor";
13:33:29.619 | > 3 \| import { GrantsProgram } from "../../../target/types/grants_program";

I noticed your test suite doesn't seem to want to run on my local setup:
ts-mocha -p ./tsconfig.json -t 1000000 tests/suites/donations.test.ts tests/suites/grants.test.ts

  0 passing (0ms)

^ nvm got it. Assuming you just run anchor test and not anchor run test at all

Type error: Property 'metadata' does not exist on type '{ version: string; name: string; instructions: { name: string; accounts: { name: string; isMut: boolean; isSigner: boolean; }[]; args: { name: string; type: string; }[]; }[]; accounts: { name: string; type: { kind: string; fields: ({ ...; } | { ...; })[]; }; }[]; types: { ...; }[]; errors: { ...; }[]; }'.


Schematic Error at P1

The schematic shows an inconsistency at P1 USB Power:
P1-1 is labeled VBUS yet is connected to GND
P1-5 is labeled GND but is connected to VBUS
P1 labeling is correct, the connection just seem to be transposed.

Thanks for catching this! the SVG export seems to have caused some issues. The schematic is now in PDF format and should be correct.
Thanks!


Memory Leak in OverlayManager?

I am debugging my app for memory leaks, as I'm getting some OOM's after I stop/restart the application a bunch of times.
I am using LeakCanary to track the leaks https://medium.com/square-corner-blog/leakcanary-detect-all-memory-leaks-875ff8360745#.8fzmjudm6.
It complains about LocationFactory keeping a reference to the Context (main activity), causing a leak of the whole activity and its fragments.
I have traced the issue down to the OverlayManager creating a LostApiClient in line #155, looks like it's for the MyLocationOverlay, even though I don't use it.
OverlayManager(MapView mapView, MapController mapController, MapDataManager mapDataManager,
      MapStateManager mapStateManager, LostApiClient lostApiClient) {
    if (lostApiClient == null) {
      lostApiClient = LocationFactory.sharedClient(mapView.getContext(), connectionCallbacks);
    }

Am I doing anything wrong that is causing this leak? Here's the trace from LeakCanary
03-20 22:20:02.558  8421 19229 D LeakCanary: * tv.nebular.conectabike.MainActivity has leaked:
03-20 22:20:02.558  8421 19229 D LeakCanary: * GC ROOT static com.mapzen.android.location.LocationFactory.context
03-20 22:20:02.558  8421 19229 D LeakCanary: * leaks tv.nebular.conectabike.MainActivity instance
03-20 22:20:02.558  8421 19229 D LeakCanary: * Retaining: 42 MB.
03-20 22:20:02.558  8421 19229 D LeakCanary: * Reference Key: a8d9245d-589f-4d73-944c-c5434eaddb61
03-20 22:20:02.558  8421 19229 D LeakCanary: * Device: samsung samsung GT-I9505 jgedlteue
03-20 22:20:02.558  8421 19229 D LeakCanary: * Android Version: 6.0.1 API: 23 LeakCanary: 1.5 00f37f5
03-20 22:20:02.558  8421 19229 D LeakCanary: * Durations: watch=316611ms, gc=292ms, heap dump=9873ms, analysis=39346ms
03-20 22:20:02.558  8421 19229 D LeakCanary: * Details:
03-20 22:20:02.558  8421 19229 D LeakCanary: * Class com.mapzen.android.location.LocationFactory
03-20 22:20:02.558  8421 19229 D LeakCanary: |   static $staticOverhead = byte[16]@315642689 (0x12d05341)
03-20 22:20:02.558  8421 19229 D LeakCanary: |   static shared = com.mapzen.android.lost.internal.LostApiClientImpl@317419328 (0x12eb6f40)
03-20 22:20:02.558  8421 19229 D LeakCanary: |   static context = tv.nebular.conectabike.MainActivity@316192512 (0x12d8b700)
03-20 22:20:02.558  8421 19229 D LeakCanary: * Instance of tv.nebular.conectabike.MainActivity

Thanks in advance !

@nebular thanks for the details report. It sounds like the activity is getting destroyed and recreated but the application is not. Not holding onto a reference to an Activity from a static field is one of the easiest ways to get a memory leak and we are guilty of that here.
https://github.com/mapzen/android/blob/master/core/src/main/java/com/mapzen/android/location/LocationFactory.java#L14
Perhaps we could get around this by using the application context when the shared location client is created rather than the activity context.

Using the application context will definitely work, as well as using a normal instance of LostClient instead of a Shared one, then clearing it in mapView.onDestroy(). I'd also suggest to create the LostClient only if / when the "LocationOverlay" is used.


Soft affinity for executor side data cache

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

https://github.com/hit-lacus/spark/issues/1


KeyVaultCredential does not reflect changes in www-authenticate header

If the customer moves a key vault from one tenant to another, the www-authenticate header changes, but the KeyVaultCredential class does not detect the new value and use the previous forever. The process keeps requesting tokens for the wrong tenant and becomes unable to access that specific key vault again.  This caused an incident that required rebooting of servers in the cloud.
The issue is that http_bearer_challange_cache is global to the process and is never updated, or checked if the cached challenge is out of date.  The proposed fix is when the request fails with 401 we check to see if the returned challenge matches our cached version.  If not we need to update the cache and re-authenticate.

Wont fix this unless we have a request to fix it.


Update for private nuget sources

Private nuget sources are not working with Visual Studio for Mac without editing the nuget.config and specifying the private artifactory username and password. Could this page be updated with information on how to get it to work?
Thanks,
Jonathan

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

ID: 1e32dfe2-b389-c1ec-cd29-14c081d140a2
Version Independent ID: 3d0606b2-e98a-819e-6090-1ca8b7d1ed51
Content: Including a NuGet package in your project - Visual Studio for Mac
Content Source: mac/nuget-walkthrough.md
Product: visual-studio-mac
Technology: vs-ide-general
GitHub Login: @heiligerdankgesang
Microsoft Alias: dominicn

#reassign: @anandmeg

@JRosanowski, thanks for your feedback.
The scenario you're trying seems like it's a possible product issue. Please file a product issue with details of your specific scenario in the developer community forum.
Thanks again!

So no intention of documenting supported behaviour in the product? Great...I and anyone else attempting to do this will just waste time googling how to do it. Thanks.

Hi @JRosanowski - any version of Visual Studio for Mac 17.x will store username and passwords for private NuGet sources in the keychain. It looks like you've mentioned using JFrog's Artifactory to host your NuGet repository. While I'm not intimately familiar with this scenario, if this is what you're doing, I suspect a product or config issue.
After reading your initial Issue again in conjunction with your most recent comment, let me ask a quick clarifying question. Are you:

Saying that this scenario isn't working unless you add the info to the nuget.config file and you want a more acceptable solution OR
Saying that adding the information for the Artifactory credentials to the nuget.config file is a reasonable solution and that you just want it documented on the NuGet package documentation page

Am I correct in assuming the latter?
Thanks for your time!


000377.xml has labels "dog"

000377.xml has labels dog and should be hat instead.

wtf is that ,I saw it, too


Remove PS_Transcripts from Desktop

@mandiant/commando-vm why are you logging PS to the Desktop? https://github.com/mandiant/VM-Packages/blame/3e31720901e68b02683cfbe014421c28500224c7/packages/common.vm/tools/vm.common/vm.common.psm1#L1149
I find it annoying to get this folder every time I open PowerShell, trying to figure when/where to disable this or if there is a good reason to keep it. @mandiant/flare-vm opinions?

I guess just to make it easy to find. I'm happy to relocate it if we can find another convenient spot. I'm sure you guys dont use the logging as much as we do, but I use the logs almost every engagement I'm on for timestamps, screenshots, command syntax, etc.

My suggestion if we moved it would be the Start Menu

@day1player what about moving it and pin it to the taskbar (if it is possible to implement, see https://github.com/mandiant/VM-Packages/issues/708)?

@day1player what about removing VM-Configure-PS-Logging and moving it to the commando-vm config file?

@mandiant/flare-vm do you see a use case of having logging enabled for us?

I vote we keep it, but let's move it if it's an annoyance on the desktop

@mr-tz also mentioned he doesn't find it annoying. Ok, let's keep it for now and we can discuss removing at a later point if needed.


Improving generated code of amplify codegen (swift)

Is your feature request related to a problem? Please describe.
This feature request/improvement is strictly related to amplify codegen
I've noticed the incredible amount of redundancy (type-wise) when a simple schema is created as code generator will create a duplicate type for every operation (and level) a query has indicated, when this could easily be addressed with type aliases when getting a type.
This is frustrating as I need to write several times the same type conversion to map things to a type that isn't absurd all across my code.
Describe the solution you'd like
Imagine the following schema:
enum Gender {
	male
	female
	unspecified
}

type Person {
	id: ID!
	firstName: String!
	surname: String!
	address: String
	gender: Gender
}

input ModelPersonFilterInput {
	id: ModelIDFilterInput
	name: ModelStringFilterInput
	surname: ModelStringFilterInput
	address: ModelStringFilterInput
	gender: ModelGenderFilterInput
	and: [ModelPersonFilterInput]
	or: [ModelPersonFilterInput]
	not: ModelPersonFilterInput
}

type Query {
	getPersonBy(id: ID!): Person
	listPeople(filter: ModelPersonFilterInput, limit: Int, nextToken: String): Person
}

input CreatePersonInput {
	id: ID
	name: String!
	surname: String!
	address: String
	gender: Gender
}

input UpdatePersonInput {
	id: ID!
	name: String
	surname: String
	address: String
	gender: Gender
}

type Mutation {
	createPerson(input: CreatePersonInput!): Person
	updatePerson(input: UpdatePersonInput!): Person
}

When I download the schema.json and call amplify codegen the generated file will contain no less than 6 or 7 type definitions for Person which are absolutely identical. I completely understand those with the name Create and Update passed in as part of the mutation because well... inputs were defined like that. But my CreatePerson type will define internally a Person which gets redefined in UpdatePerson, ListPeople and GetPersonBy
And I am specifying here something without connections with other structures in "many to many" relationships.
Describe alternatives you've considered
Ideally I would like a single type definition for person and if it is hard to prevent conflicts with other types, create type aliases for each of the operations (mutations, queries or subscriptions) just pointing the one definition. It makes refactoring/data structure revisions so much manageable.
The schema.json file contains the type definition in a single place (as an object) so the block declaration should be able to create a direct connection to the already specified type instead of having to redeclare it (or just replace the reference with yet another type).
If that direct reference is problematic (or we don't want to introduce a massive backwards compatibility issue as it will break the code of many) then we could just use a type alias in place of the structure definition.

Thanks for your enhancement request. If you take a closer look, the optionality of fields in CreatePersonInput or UpdatePersonInput is different from Person. This is low priority and could potentially break many existing customers.

Thanks for your enhancement request. If you take a closer look, the optionality of fields in CreatePersonInput or UpdatePersonInput is different from Person. This is low priority and could potentially break many existing customers.

@phani-srikar duplicate code results in hundreds of thousands of lines of code, can this be addressed please?


Trouble building on Ubuntu 24.10.

Regression?
Yes
Hyprlock Info and Version
Trying to build hyprlock on my ubuntu using
cmake --no-warn-unused-cli -DCMAKE_BUILD_TYPE:STRING=Release -S . -B ./build
cmake --build ./build --config Release --target hyprlock -j`nproc 2>/dev/null || getconf _NPROCESSORS_CONF`

but got this error:
/home/nir/hyprlock/src/core/Fingerprint.cpp:12:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   12 | static const auto FPRINT        = sdbus::ServiceName{"net.reactivated.Fprint"};
      |                                          ^~~~~~~~~~~
/home/nir/hyprlock/src/core/Fingerprint.cpp:13:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   13 | static const auto DEVICE        = sdbus::ServiceName{"net.reactivated.Fprint.Device"};
      |                                          ^~~~~~~~~~~
/home/nir/hyprlock/src/core/Fingerprint.cpp:14:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   14 | static const auto MANAGER       = sdbus::ServiceName{"net.reactivated.Fprint.Manager"};
      |                                          ^~~~~~~~~~~
/home/nir/hyprlock/src/core/Fingerprint.cpp:15:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   15 | static const auto LOGIN_MANAGER = sdbus::ServiceName{"org.freedesktop.login1.Manager"};
      |                                          ^~~~~~~~~~~
/home/nir/hyprlock/src/core/Fingerprint.cpp: In member function ‘std::shared_ptr<sdbus::IConnection> CFingerprint::start()’:
/home/nir/hyprlock/src/core/Fingerprint.cpp:53:83: error: ‘ServiceName’ is not a member of ‘sdbus’
   53 | login      = sdbus::createProxy(*m_sDBUSState.connection, sdbus::ServiceName{"org.freedesktop.login1"}, sdbus::ObjectPath{"/org/freedesktop/login1"});
      |                                                                  ^~~~~~~~~~~

gmake[2]: *** [CMakeFiles/hyprlock.dir/build.make:132: CMakeFiles/hyprlock.dir/src/core/Fingerprint.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/hyprlock.dir/all] Error 2
gmake: *** [Makefile:136: all] Error 2

does anyone know how to go on from here?
Compositor Info and Version
 System/Version info
<Paste the output of the command here>

Description
Trying to build according to README
How to reproduce
cmake --no-warn-unused-cli -DCMAKE_BUILD_TYPE:STRING=Release -S . -B ./build
cmake --build ./build --config Release --target hyprlock -j`nproc 2>/dev/null || getconf _NPROCESSORS_CONF`

Crash reports, logs, images, videos
No response

sdbus-cpp >= 2.0.0

Issue persisting with sdbus-cpp = 2.1.0
-- The C compiler identification is GNU 14.2.1
-- The CXX compiler identification is GNU 14.2.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring hyprlock in Release with CMake
-- Checking deps...
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Found PkgConfig: /usr/bin/pkg-config (found version "2.3.0")
-- Found OpenGL: /usr/lib/libOpenGL.so
-- Checking for modules 'wayland-client;wayland-protocols;wayland-egl;hyprlang>=0.4.0;egl;opengl;xkbcommon;libjpeg;libwebp;libmagic;cairo;pangocairo;libdrm;gbm;hyprutils>=0.2.6;sdbus-c++>=2.0.0;hyprgraphics'
--   Found wayland-client, version 1.23.1
--   Found wayland-protocols, version 1.38
--   Found wayland-egl, version 18.1.0
--   Found hyprlang, version 0.5.3
--   Found egl, version 1.5
--   Found opengl, version 4.5
--   Found xkbcommon, version 1.7.0
--   Found libjpeg, version 3.0.4
--   Found libwebp, version 1.4.0
--   Found libmagic, version 5.46
--   Found cairo, version 1.18.2
--   Found pangocairo, version 1.54.0
--   Found libdrm, version 2.4.124
--   Found gbm, version 24.3.1-arch1.3
--   Found hyprutils, version 0.2.6
--   Found sdbus-c++, version 2.1.0
--   Found hyprgraphics, version 0.1.0
-- Found WaylandScanner at /usr/bin/wayland-scanner
-- Found wayland-protocols at //usr/share/wayland-protocols
-- Configuring done (0.6s)
-- Generating done (0.0s)
-- Build files have been written to: /home/adi/.cache/yay/hyprlock-git/src/hyprlock/build
[  5%] Building CXX object CMakeFiles/hyprlock.dir/src/core/Auth.cpp.o
[  5%] Building CXX object CMakeFiles/hyprlock.dir/src/config/ConfigManager.cpp.o
[  8%] Building CXX object CMakeFiles/hyprlock.dir/src/core/CursorShape.cpp.o
[ 11%] Building CXX object CMakeFiles/hyprlock.dir/src/core/Egl.cpp.o
[ 14%] Building CXX object CMakeFiles/hyprlock.dir/src/core/Fingerprint.cpp.o
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:12:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   12 | static const auto FPRINT        = sdbus::ServiceName{"net.reactivated.Fprint"};
      |                                          ^~~~~~~~~~~
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:13:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   13 | static const auto DEVICE        = sdbus::ServiceName{"net.reactivated.Fprint.Device"};
      |                                          ^~~~~~~~~~~
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:14:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   14 | static const auto MANAGER       = sdbus::ServiceName{"net.reactivated.Fprint.Manager"};
      |                                          ^~~~~~~~~~~
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:15:42: error: ‘ServiceName’ is not a member of ‘sdbus’
   15 | static const auto LOGIN_MANAGER = sdbus::ServiceName{"org.freedesktop.login1.Manager"};
      |                                          ^~~~~~~~~~~
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp: In member function ‘std::shared_ptr<sdbus::IConnection> CFingerprint::start()’:
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:53:83: error: ‘ServiceName’ is not a member of ‘sdbus’
   53 |     m_sDBUSState.login      = sdbus::createProxy(*m_sDBUSState.connection, sdbus::ServiceName{"org.freedesktop.login1"}, sdbus::ObjectPath{"/org/freedesktop/login1"});
      |                                                                                ^~~~~~~~~~~
/home/adi/.cache/yay/hyprlock-git/src/hyprlock/src/core/Fingerprint.cpp:54:25: error: ‘class sdbus::IProxy’ has no member named ‘getPropertyAsync’; did you mean ‘getProperty’?
   54 |     m_sDBUSState.login->getPropertyAsync("PreparingForSleep").onInterface(LOGIN_MANAGER).uponReplyInvoke([this](std::optional<sdbus::Error> e, sdbus::Variant preparingForSleep) {
      |                         ^~~~~~~~~~~~~~~~
      |                         getProperty
make[3]: *** [CMakeFiles/hyprlock.dir/build.make:135: CMakeFiles/hyprlock.dir/src/core/Fingerprint.cpp.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [CMakeFiles/Makefile2:87: CMakeFiles/hyprlock.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:94: CMakeFiles/hyprlock.dir/rule] Error 2
make: *** [Makefile:170: hyprlock] Error 2
==> ERROR: A failure occurred in build().
    Aborting...
 -> error making: hyprlock-git-exit status 4
 -> Failed to install the following packages. Manual intervention is required:
hyprlock-git - exit status 4

Am I doing something wrong elsewhere?

Hello,
I was facing the same issue while building Hyprlock v0.6.0 (tried both sdbus v2.0.0 and v2.1.0) on Ubuntu 24.04.
Following the solution to this issue fixes the problem: https://github.com/hyprwm/hypridle/issues/109
It seems to be a problem within the hypr ecosystem on Ubuntu.
I have no idea why the problem only shows up on Ubuntu and I have no other platforms to test things. I am running cmake version 3.28.3.


Error creating Db-connected services: tables with "_"

When I try to create a new db-connected rest service using a table with "_"(underscore), I get this error message:
"Error saving field".
I can see the files are generated, but Apigility UI cannot fetch the service.
I tried using:
"zfcampus/zf-apigility-admin-ui": "dev-develop",
If I use only apigility 1.3.1, there is still the JS bug during the service creation (#PR75), so I changed to dev-develop.
Any idea?

Been digging a bit in this, as I also ran into this.
It seems that when the new service is created, if the table name contains an underscore ( _ ), the API will convert this to what looks like CamelCase.
But when the UI then wants to create the fields for this newly created service, it (the UI) will still use the tablename with the underscore in it, when accessing the api (/apigility/api/module//rest/--Rest-<Table_name>-Controller/input-filter)
But the API will this time expect the controller-part being specified with the tablename camedcased:
(/apigility/api/module//rest/--Rest--Controller/input-filter)
But I do not know if this should be fixed in the UI-code or in the API-code ?

Hi Wesley, I got the same error.
http://stackoverflow.com/questions/32011684/apigility-error-saving-field-while-creating-db-connected-service?noredirect=1#comment51978970_32011684
I have no idea yet... but I'm looking for a solution.

Hello @gartner and @wesleywillians ... I found a solution.
In zfcampus-apigility-admin-ui change the function capitalizeFirstLetter in file src/apigility-ui/service/api.service.js adding the line "string = string.replace(/(\w)/g, function(,letter) { return letter.toUpperCase(); });" before the return.
I found a issue where were applied the solution but it seems not be in currently code... I don't know why.
https://github.com/zfcampus/zf-apigility-admin-ui/pull/59/files
I'm using "zfcampus/zf-apigility-admin-ui": "dev-develop".

It had already been fixed but it was undone in commit https://github.com/zfcampus/zf-apigility-admin-ui/commit/0cbaf331c6cebf9652b45619f0025f7ddbebeee2 .
@weierophinney can you tale a look because a bug was reopen in your commit?

I have no idea why that was removed, tbh; my guess is that I may have been working from an older commit when applying a patch. I'll get that updated shortly.

I fixed this issue with PR #83.

This issue is still open, and fix in capitalizeFirstLetter() function has been removed (can't guess why).
Can you change to status "open"?

Yes @leonardo-piseri, you are right. The fix #59  for this issue has been overwritten and this is why I have made a new pull request for that #100. We'll just have to wait for the next version to be updated.

Hi. Even after the fixes mentioned earlier, I am still encountering the errors.
The automatic discovery does not work for Oracle database (I am getting ORA-12504: TNS:listener was not given the SERVICE_NAME in CONNECT_DATA in /opt/apigw/apigility/vendor/zendframework/zend-db/src/Adapter/Driver/Oci8/Connection.php on line 122).
After hardcoding the credentials, I see the tables. However, when attempting to create db-connected services, I am getting "Error saving field". When I attempt to access them, I am getting "Unable to fetch service". I am currently stuck with this and I don't know what to do to solve the problem.
Any ideas?

This says closed but I am also having same issue.  Any updates?

Hello, i have the same error. I use Apigility 1.3.2


Some better things

[x] separate the tree library (tree.css, tree.js)
[x] replace ids in css with classes (because it is possible to add two trees at one page)
[x] add library docs
[x] replace an icon with beautiful one
[ ] add support for color themes
[x] refactor CSS-styles for tree. Class names should be more specifiсally, because now it is a library, and it may use in various pages, including pages with same class names. And now, markup is ugly(
[x] add beautiful font 'PT Mono' from google.fonts
[ ] add feature with creation of filter function from textarea by new Function
[x] add feature with selecting some nodes by filter function
[x] add sorting for object keys
[ ] add highlighting for expanded nodes
[ ] add search by js-path (for ex. root["phoneNumbers"]["0"]["type"])
[x] add displaing js-path for each node by click? And copy this path to search form (for editing path and finding similar or child nodes)
[ ] add expanding of root node by depth (count of descendants levels)
[ ] add expanding for all tree recursively by some function (for ex., expand all nodes with some count of children) Fix docs!
[ ] Add unit-tests for library
[ ] Add custom templates for values (by matcher + handler). For example:

strings which starts with "http://" displays as link
nodes with label "color" displays with circle of color in value at right side
numbers displays as "123 000 000" with spaces
nodes with label "image" by mouseover shows images preview
etc.

[ ] Add "find and get as array" feature. Not nodes marks, only result displays as array.
[ ] Change app.js' styles and class names. Now this is very ugly!

please work on this this would be useful

please work on this this would be useful


[Feature Request] Support VirtualBox Provider

As a developer, I am using virtualbox VM with k3s installed as my dev cluster. I would like to use auto k3s to provision new k3s cluster in virtual box automatically.

Don't need this feature anymore. Closing this issue.


Windows VS2022 installation

Hi, @glmcdona
Just find this repo interesting and would like to try, as alternative to strings.exe on Windows.
1. pip failed

System: Windows 10
Python: Python 3.10, installed via miniconda
C++ Compiler / IDE: I have Visual Studio 2019, 2022 both installed.

The output: it complains cl.exe failed.
2. C++ source modification suggestion
https://github.com/glmcdona/binary2strings/blob/main/src/extracted_string.cpp#L11
This line11 in extracted_string.cpp, convert NULL to std::string. However it will cause crash if you just try the minimal example on linux-x64 with clang:
#include <stdio.h>
#include <string.h>
int main()
{
    std::string m = (std::string)NULL;
    std::cout << "m is " << m << std::endl;
    return 0;
}

3. Build source and install on Windows
OK, based on the above two considerations:

I commented out line 11 in extracted_string.cpp
In an opened cmd.exe, python setup.py install still fail
Open VS2022 x64 native command prompt, then python setup.py install success.

I would suggest fix the pip install binary2strings experience on Windows. Thank you.

Thanks for your report, I'll look into it! I've had success with it on Windows 10 here, so like you suggested it may depend on build tools that are installed.

Fixed with #16. Apologies for the delay! Thanks for identifying the issue.

Fixed with release 0.1.10. You should be able to just do pip install binary2strings now.


Tests are broken on node 12, add to travis

Something to do with nock?

It looks like a lack of faked prebuilds for Node ABI v72 in https://github.com/ralphtheninja/a-native-module/releases is causing this.

PR to fix the tests at #120


[style]: Consolidate "View Profile" Link into a Single Line

What's the style idea?
Description:
The "View Profile" link in the profile box is currently displayed in two lines, with "View Profile" in the first line and an arrow "->" in the second line. This issue is to propose consolidating these into a single line for a cleaner and more compact presentation.
Details:
The current two-line format takes up unnecessary space and may disrupt the visual flow. Combining "View Profile" and the arrow into a single line will improve the overall aesthetics and user experience.
Benefits:
Improved visual presentation.
Consistent with design principles for compact and clean UI.
Please assign this issue or I'll directly submit a PR for this.
Add screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Assigned @piyushbarve98

Hii @shyamtawli this is the PR for this issue https://github.com/shyamtawli/devFind/pull/691

I will look into it.

The Project is migrated to tailwindcss if you want to work then please open a new PR.


Uncollected Pokemon and server restart

If you don't collect purchased Pokémon from "Claim Trades" and server restarts, it will put them back in "Trades" but you will be a seller.
If there is Pokémon in "Timed out Trades" when the server restarts, it disappears entirely.
v 1.2.0

Fixes in latest release


use url.URL, add chainguard actions

Adds github actions for deps and validation, unit test running.
Drops dep on knative.dev/pkg

Thanks for the review @mattmoor !


Thunderous Network Open Grant Application

Grant Application Checklist

[x] The application-template.md has been copied, renamed ( "project_name.md") and updated.
[x ] A BTC address for the payment of the milestones is provided inside the application.
[x] The software of the project will be released under the Apache license version 2.0 as specified in the terms and conditions.
[x] The total funding amount of the project is below CHF 30k at the time of submission.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: codemeit:x: kaicodexYou have signed the CLA already but the status is still pending? Let us recheck it.

We need more time to prepare the pallet specification. We will close the pull request for now until we're ready.
Thanks.


Simple chainlist report

Input:

Chain list API: https://content.subwallet.app/api/list/chain
Chain asset: https://content.subwallet.app/api/list/chain-asset

Output:

Chain Number
Number EVM
Number Substrate
XCM Support List: ChainAssetList.assetRefs of chain assets has value

Count number
Listing all XCM support

Staking List: substrateInfo.supportStaking = true

Count
Listing all network - symbol

link demo
https://codesandbox.io/p/sandbox/dazzling-kate-l37wkh?file=%2Fsrc%2Fstyles.css%3A40%2C14


better i/o timeout error on pull

see: https://forums.docker.com/t/pulling-docker-images-i-o-timeout/740 it is the 2nd most searched for topic on the forum... so I think we should make it less vague.

LGTM

LGTM

@jfrazelle Found the right incantation to get the Timeout info... it has a nice gift-wrapping:
		if urlErr, ok := err.(*url.Error); ok {
			if opErr, ok := urlErr.Err.(*net.OpError); ok {
				isTimeout = opErr.Timeout()
			}
		}

oooo pR it!
On Thu, Aug 6, 2015 at 11:05 AM, Brian Goff<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle Found the right incantation to
get the Timeout info... it has a nice gift-wrapping:
    if urlErr, ok := err.(*url.Error); ok {
        if opErr, ok := urlErr.Err.(*net.OpError); ok {
            isTimeout = opErr.Timeout()
        }
    }

—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/14839#issuecomment-128460441.


Fix example for CustomPathResolver

Adding constructor for CustomPathResolver
Fix to issue #991

thank you, @dobau!


feat: expand GitLab evalContext with the HEAD pipeline for the MR

Codecov Report
Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 2.34%. Comparing base (0b04c12) to head (e60d9f3).

Files with missing lines
Patch %
Lines

pkg/scm/gitlab/context_valuers.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##            main     #97      +/-   ##
========================================
- Coverage   2.34%   2.34%   -0.01%
========================================
  Files         51      51
  Lines       2431    2433       +2
========================================
  Hits          57      57
- Misses      2366    2368       +2
  Partials       8       8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Read only property

@santimirandarp
If you want to define it as a read-only property this can be done using Object.defineProperty. By default it is readonly.
Object.defineProperty(obj, "webkitRelativePath", {
    value: "test"
});

For me it is ok to do this and to have testcases that check if it is really read-only. Please feel free to open a PR.

You can also define it as a getter, like it is done in browsers. It would be like this:
fileList.push({
  name,
  ...
  get webkitRelativePath() {
    return 'test'
  }
})

I didn't know this. Thanks @targos !


Image Sharpness & Smoothness

I am trying to achieve common effects on images with JIMP. Need some help with applying sharpness and smoothness effects between 0-100 interval on image. Can someone help about these.

Need to revive the PR but this should be doable https://github.com/jimp-dev/jimp/pull/1067/files

@hipstersmoothie what is necessary to do at he pending PR?


Add Storage Key name to errors thrown by serializer

I have a bit of an edge case and wondered what your thoughts might be, @joshnuss.
This is where the serializer does its stuff when retrieving data from storage:
https://github.com/joshnuss/svelte-local-storage-store/blob/34b1994bd22180d704d7629938e9b27936aa375e/index.ts#L52
Any parsing errors that might occur when the storage event fires are unhandled. I wondered if it might be worth catching such errors and adding the key name to the error message before rethrowing?
As I say, this is quite an edge case; parsing errors should only happen if the value in storage is corrupted in some way, and given the same serializer will have been used to put the data there in the first place, this is unlikely.
But, I'm thinking of a situation where users tamper with storage (on their heads be it!) or if some other (unrelated) code manipulates storage in some way.
I am logging JavaScript errors centrally and, if corruption does occur, it would be useful to know which storage entry is causing the issue. Having the key name in the error message would help with this.

Hi @Holf,
Yes, that makes sense, I'd be open to a PR.


MMRCMS: Dynamic filter rework

Closes #1304
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

#1304
Why remove the option instead of fix it...?
I don't mind tho, thanks

#1304 Why remove the option instead of fix it...?
I don't mind tho, thanks

Because the website doesn't support it. You can check by going to https://mangas.in/manga-list and see that there is only sorting by title and by view count.

#1304 Why remove the option instead of fix it...?
I don't mind tho, thanks

Because the website doesn't support it. You can check by going to https://mangas.in/manga-list and see that there is only sorting by title and by view count.

Ohh, I see, since in "home" of the website it appeared I thought it would appear anywhere else lol


⚠ Turnos 24x7 has degraded performance

In 4d66637, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 883 ms

Resolved: Turnos 24x7 performance has improved in 01246af.


pm: device_runtime: option to enable from devicetree

Add the option to automatically enable PM device runtime on a device from devicetree. Devicetree is the only mechanism we have for per-instance configuration, which is why this route was chosen.
Absent this option, application code must be written to enable PM manually on devices, which cannot be done in a generic fashion.

I really liked it :) Just minor comment


Frontend Homework 14

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
2

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Refactor model, update pyproject.toml and testing

Chore(pyproject.toml): Update keywords and versions to reflect what is supported.
Testing/Refactor: Change names of models used for testing to avoid pytest confusion
Testing(warnings): Ignore from hypothesis
Feat(Justfile): Add mypy typecheck for test
Refactor(model): Merged into a single python file
Chore(version): MAJOR

Thank you!


[ONNX] Support for Split op

Style check fails:
2 of 878 source files failed the code-format check:
    src/ngraph/frontend/onnx_import/model.hpp
    test/onnx_import.cpp

In file included from ngraph/src/ngraph/frontend/onnx_import/ops_bridge.cpp:23:
ngraph/src/ngraph/frontend/onnx_import/op/split.hpp:90:75: warning: use of old-style cast [-Wold-style-cast]
                    return (left >= 0) ? std::min(left, right) : std::max((T)0, right + left);
                                                                          ^  ~

Error in CI running on clang 3.9:
error: unknown warning option '-Wno-zero-as-null-pointer-constant'; did you mean '-Wno-int-to-void-pointer-cast'? [-Werror,-Wunknown-warning-option]


[543] Added xdm:localTimezoneOffset to context/placecontext

https://github.com/adobe/xdm/issues/543
@kstreeter & @cdegroot-adobe

Hey @cluby-adobe this addition makes sense, but would it make sense to make the offset a string that has the same form of the offset defined in ISO8601?  This definition seems to be what they do in Javascript, but in Java they appear to support the string form.

@cluby-adobe, The issue is we don't know the actual timezone.  We only have the current offset.

Understood we would not have the actual timezone, just the offset.  ISO8601 has a notation for that (a "-" followed by the HH:MM of the offset).  That notation appears to be supported at least in the Java environment.

@kstreeter, Do you know of anything that tries to use this format for timezone offset on it's own outside of a date-time string?  I want to make sure there is value in using this format to justify the cost of dealing with in at various layers.  If every layer is just going to have to convert it into minutes to use it why don't we just represent it as minutes.


Move bitfield tests to proper place

and fix some unneed preparing / cleaning up

@marcosnils Please review. Thanks!

Merged to master, 2.10, 2.9 respectively.


pre/post-compile hooks w/o using custom buildpacks

would you be open to add support for something like:
$ heroku config:set BUILDPACK_PRE_COMPILE=custom_script.sh
which would enable custom actions on build time (like using "django-admin.py collectstatic" w/ use-env-compile labs feature) where $BUILDPACK_PRE_COMPILE script must exists in git repo.
this would remove requirement to create custom buildpack for simple stuff like mentioned collectstatic problem.

Just wanted to clarify, will this honor a python shebang like /usr/bin/env python or must it be /app/.heroku/python/bin/python (the result of running which python against my app dyno)? Just in case - is the location of which python consistent across dynos assuming they're on the same stack?

@alexkahn env python should work perfectly fine

Thanks!


add object type to required for register validation rule

related: https://github.com/react-hook-form/react-hook-form/pull/1061

thanks @kotarella1110


XML Not Updated on Create DI

Describe the Bug

When creating a DMN decision table the DMN contains no DI. Once you switch to the DRD view, the DI is added. This change is not reflected in the XML until you start modeling.

Steps to Reproduce

Create new DMN table
Switch to DRD view
Switch to XML view

Expected Behavior
XML should contain DI.
Possible solution: Remove the ability to create decision tables without DI. This option seems to only exist for historical reasons.
Environment

Camunda Modeler Version: v4.0.0

Can still be reproduced after #1811 using this diagram.


Add static files for dark theme

Kind of related to #864. there is no way (that I know of) to access the dark theme if you are just using static files.
On the Color/index.html in Docs, there is a section that indicates the dark theme can be accessed by using @import "calcite-web-dark";. I tried putting that in the console in Chrome - no dice. Also tried adding it to all.css in Chrome's sources window, but still no luck.
Probably mostly my failure to understand the vast majority of how CW works, but it would be nice if I could access the dark-theme by just loading all-dark.css file or whatever.

related to #864

Combining with https://github.com/Esri/calcite-web/issues/864 🌱


Moons can be larger than the planet that they orbit.

Moon sizes are generated using the solar system's moon datasets. It is generated randomly from the mean, stdev of existing moons. This makes it possible for a moon to be larger than it's planet.
This could be fixed by normalizing the mean and stdev relative to the planet that it orbits.

#73 should fix it.


dnsdumpster: fix csrf

Hmm, seems like I did not have the latest version..

Thanks for the pull request, it's fixed in the latest version.


Update docs for Calico v3.2.2

Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 0d9e9a44d153ee888c3c8f5339d93bfc3522f396
https://deploy-preview-2205--calico.netlify.com


Fix the file path of tar.c

Fixed the issue of moving filename, causing strncpy to copy not the full path, Also added cmake to make it easier to compile.

Can you please show an example of what the change fixes?

Please give both libraries proper names within CMakeLists.txt and rename the actual output files with set_target_properties.

您能否举例说明此更改修复了什么？

strncpy(entry -> name, filename + move, 100);
Here because the filename parameter is moved forward by 1, strncpy is not copying the full data of filename, for example, I want to copy the "/my_dir/my_file" file, because the filename is moved forward, causing strncpy to copy "my_dir/my_file", which is incomplete, if the name of the file does not contain the path, strncmp also only copies "y_file", instead of " my_file".

您能否举例说明此更改修复了什么？

Sorry, I made a mistake.


bugfix/feature: Allow users to manipulate images without needing to resize them

Description
Right now, Payload's image "transformation" feature is limited to images that requires resizing.
This is a simple fix/feature that allows us to "reuse" the original size of an uploaded media. This way, users can use Sharp's features like changing formatOptions and trimOptions without needing to give a new size to the images.
It also allows users to keep an original copy of the image for certain use cases.

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes

@DanRibbens friendly ping on this

Added a couple of tests + applied suggestion to not add a new property @denolfe

Looks great, @angelobartolome !


Switched the order of serializers to DillSerializer and PickleSerializer

Switched the order of serializers to DillSerializer and PickleSerializer
@benclifford We wanted to prioritize dill serializer, because dill is more reliable in some cases. For example, with the following snippet
    dfk = parsl.load(config)
    exe = dfk.executors['Midway_HTEX_multinode']
    f = exe.submit(square, None, 2)
    print(f.result())

The original order of serializers would give an error message of
AttributeError: Can't get attribute 'square' on <module '__main__' from '/home/yrao2/.conda/envs/parsl_probe/bin/process_worker_pool.py'>

What is going on here that is different from regular app submission, that makes this break but does not break app submission?

I was doing some config testing. With the following submission, the original order would also have the same issue.
            block_id = executor.start()
            fut = executor.submit(square, None, 3)
            print(fut.result())

but does parsl app execution work with the same config file?

Yes, I used the site_test configs on their corresponding sites.

@benclifford Let us look at the following two examples to compare dill and pickle.

Example 1: with one file main_only.py

# main_only.py
import dill
import pickle

def square(x):
    return x*x

if __name__ == "__main__":

    dill_serialized = dill.dumps(square)
    pickle_serialized = pickle.dumps(square)

    dill_deserialized = dill.loads(dill_serialized)
    pickle_deserialized = pickle.loads(pickle_serialized)

    print(dill_deserialized(5))
    print(pickle_deserialized(5))

python main_only.py would run without any issue.

Example 2: with two files in the same directory, main_lib.py and lib.py

# main_lib.py
import dill
import pickle
from lib import dill_serialize, pickle_serialize

if __name__ == "__main__":

    dill_deserialized = dill.loads(dill_serialize())
    pickle_deserialized = pickle.loads(pickle_serialize())

    print(dill_deserialized(5))
    print(pickle_deserialized(5))

# lib.py
import dill
import pickle

def square(x):
    return x*x

def dill_serialize():
    return dill.dumps(square)

def pickle_serialize():
    return pickle.dumps(sqaure)

python main_lib.py would show that dill part worked and pickle part did not work with error message of NameError: name 'sqaure' is not defined. This is because pickle.dumps() merely returns a reference to the original function, but dill.dumps() encodes the function body. The difference shows up when the function is defined in a different module. So we think dill is more reliable for serialization, and decide to prioritize it in our serializer.
The reason we did not see this kind of behavior before is that with the @python_app decorator, whether the function is defined in the same module or not will not matter. Both Example 3 and Example 4 as below work.

Example 3: with one file main_only_decorator.py

# main_only_decorator.py
import dill
import pickle
from parsl.app.app import python_app
from parsl.serialize import ParslSerializer

@python_app
def square(x):
    return x*x

if __name__ == "__main__":

    parsl_serializer = ParslSerializer()
    parsl_packed = parsl_serializer.pack_apply_message(square, 5, None)
    func, args, kwargs = parsl_serializer.unpack_apply_message(parsl_packed, locals())
    print(func)
    print(args)
    print(kwargs)

python main_only_decorator.py would work.

Example 4: with two files in the same directory main_lib_decorator.py and lib_decorator.py

# main_lib_decorator.py
from parsl.serialize import ParslSerializer
from lib_decorator import parsl_serialize

if __name__ == "__main__":

    parsl_serializer = ParslSerializer()
    func, args, kwargs = parsl_serializer.unpack_apply_message(parsl_serialize(), locals())
    print(func)
    print(args)
    print(kwargs)

# lib_decorator.py
from parsl import python_app
from parsl.serialize import ParslSerializer

@python_app
def square(x):
    return x*x

def parsl_serialize():
    parsl_serializer = ParslSerializer()
    return parsl_serializer.pack_apply_message(square, 5, None)

python main_lib_decorator.py would also work. The decorator makes this happen.

 Since this works hasn't moved in the past 6 months, I'm closing this.


fix docs

Currently CI docs fails during build for different reasons:

docstrings not present
refs between external packages (Observables and  Core) cannot be resolved.

To resolve 1), I have added some docstrings. The rest I have commented out from the @docs section
To resolve 2), I have commented out on and Core.NamedTuples
I am not sure whether there is some other way or whether this PR coolides with other docs activities of the GenieFramework.
@essenciary @PGimenez you should know better than me and could help perhaps.

this is good, thanks @hhaensel. We're currently focusing on fully documenting the StippleUI components, so nothing is being added to the Stipple.jl API docs atm.


[TF 2.0 API Docs] tf.AggregationMethod

Existing URLs containing the issue:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/AggregationMethod
Description of issue (what needs changing):

Correct Links
https://github.com/tensorflow/tensorflow/blob/master/python/ops/gradients_util.py is incorrect. The correct link should be https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/gradients_util.py.

Clear Description
The description is not opinionated about when to use this symbol, and unclear on what aggregation methods for combining gradients would be useful for.

Usage Example
No usage example is provided.

Parameters Defined
Parameters are poorly defined, and not formatted appropriately.

Returns Defined
Returns are not defined.

Raises Listed and Defined
Errors are not defined.

Visuals, if Applicable
No visuals are included.

@dynamicwebpaige I'm interested in working on this issue!

@dynamicwebpaige Can I please work on this?

@Rajiv2605 @Ayush517 - Absolutely! Any contributions that improve our API docs are extremely welcome. 😊

https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/AggregationMethod
While opening the link on this page, I am getting redirected to https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/ops/gradients_util.py which is the correct link mentioned in the issue. It's working fine I think.

Is this issue still open as it still has an open status?

How to use tf.AggregationMethod in tf2.0.
In tf1.0 it can be passthrough the
optimizer.minimize(loss, method_aggregation =  tf.AggregationMethod.EXPERIMANTAL_TREE)
And it can help if GPU is OOM.
But how to use that in TF 2.0?

Hi @dynamicwebpaige ,
Is this issue still Open? Asking so that I can work on it?
Thanks and Gratitude!

I am interested to work on it. Can I ? @dynamicwebpaige

Hi @chinmoysarangi,
Paige has a lot of these bugs, I doubt she working on this particular one right now.
Also if you're working on api pages be sure to have a look at this doc.

Is the issue still open? I'm interested to contribute

can I work on this issue?

The links on this page are fixed.


Active classname for all visible elements

Is your feature request related to a problem? Please describe.
I'm not 100% sure if it's a feature request, bug or is it already possible with current version.
Describe the solution you'd like
Right now if slidesPerPage > 1, only one item gets className BrainhubCarouselItem--active. Is there a way to add this className to every currently visible element? Or something else, like BrainhubCarouselItem--visible etc...
Describe alternatives you've considered
I tried working around it by altering classnames of children passed down to carousel and calculating (or assuming) when they are visible, but I think it's mixing responsibilities, so I dropped this idea.
Additional context
Add any other context or screenshots about the feature request here.

Thanks in advance for your help!

@PiotrSzlagura
It seems to be a bug.
Thanks for reporting.
Maybe we'll fix that but your PR will be more than welcome.

I'll try to approach this today :)

@PiotrSzlagura
We decided to improve your PR.
Or maybe you could do that but you need to consult that with @RobertHebel and @humbak .

I got a little bit lost with my assumptions and then forgot about this thread, sorry. Please tell me if my thinking is correct:

If slidesPerPage is integer, then there is no partially visible items
If slidesPerPage is float, then there are two partially visible items: one before first visible, and one after last visible item
There can never be more than 2 partially visible items.

Am I right?

@PiotrSzlagura
IMO slidesPerPage should be always an integer.

@piotr-s-brainhub I don't agree. Having possibility to set non-integer slidesPerPage is common in sliders libs (intentional or not) and I personally am using it a lot, so I think it's good that it's there.

@PiotrSzlagura
So I'm OK with that, it can be a non-integer.


Adding Compensated Summation to SpM-DV

This patch is an overarching solution to both Issue #109 as well as a feature enhancement that allows users to optionally compute SpM-DV using compensated summation.
Compensated summation is a technique for reducing the cumulative error that occurs during floating point summation. Such errors occur both because of differences in magnitude between the summands (where part or all of the smaller one may be rounded out) as well as due to catastrophic cancellation, where subtracting a large value from a sum will cause previously marginal values that were rounded out to become significant (and now wrong).
It does this by, in part, replacing simple floating point summation with an error-free transformation, such as 2Sum or Fast2Sum. These equations essentially perform the equation A + B = (Sum, Error).
The Sum2 algorithm uses this transformation to greatly reduce the final error in a summation. As you continue to add values into sum, you continue to separately add the errors together. At the end of the summation, you add the final sum value and the final error value together. The end result is that your final answer is as accurate (0 ULP) as performing the summation in 2x precision and then casting down to your original precision. In other words, if all of your values to sum are floats, your answer should be exactly the same as if you'd performed the summation using doubles and then cast the final answer as a float.
This code uses a parallel version of this algorithm, PSum2. This compensated summation algorithm was added to both csrmv_general and csrmv_adaptive. test-blas2 and clsparse-bench were both modified to allow the command line option to use compensated summation.
When running test-blas2 with compensated summation enabled, all double-precision tests pass with 0 ULP difference compared to a CPU-side algorithm performing the same algorithm. The single precision tests sometimes differ compared to the CPU calculations performed in double precision. This is because, on my test platforms, the AMD OpenCL runtime does not allow single precision denormals. Instead, it rounds all denorms to zero, causing a large error that we can't compensate.
While this replaces all of the additions in SpM-DV with many more FLOPs, the overhead of compensated summation is, in most cases, modest. On a FirePro W9100, the average single-precision slowdown of csrmv_adaptive is 5%, and the average double-precision slowdown is 2%. On a Radeon R9 Fury X (with a DPFP rate of 1/16 the SPFP rate, but with much higher bandwidth than the FirePro W9100), the slowdown is 10% for single-precision and 28% for double-precision.

(Note that the above graphic is not a comparison between the relative performance of the two cards. '1' in each bar is the performance in that category with traditional summation.)
Note that we do not perform a fully compensated dot product, as described in the Yamanaka, Ogita, Rump, and Oishi paper. Performing an error-free transformation on the products in our SpM-DV is not extremely computationally intensive, but carrying around the extra error values from the multiplications is burdensome in csrmv_adaptive. We've seen that the majority of error in our calculations comes from summation, so we deemed that good enough. Nonetheless, this means that we can't fully guarantee a 0 ULP accuracy compared to 2x precision calculations, because the error due to multiplies may compound.

@jlgreathouse
Give me an hour face2face tomorrow so that we get this PR merged in.  I would like to give you quick training/demonstration on interactive rebasing to squash the number of commits you have to a set of 'core' commits that makes sense.  I like to try to keep the repository commit history relatively simple, and rebasing is the tool to use.