Messages in threads appear in reverse order (Element)

Steps to reproduce

Create a thread, send some messages
Restart client to sync again
Open thread
See messages in reverse order

https://github.com/element-hq/element-web/assets/29222413/d26931a3-3640-4c62-b107-e652191b64bc
(see element-hq/element-web#27664)

I'm flailing around with this problem @ https://github.com/isosphere/conduwuit/tree/504_sort_threads
my operating understanding:

PduEvent.origin_server_ts should hypothetically be useful for sorting chronologically
that field is set by the server on receipt

I have an untested worry that there is a discrepancy between client sent time + order and server recieve time + order. As far as I've been able to understand, the server won't be able to tell if that happened.

Here's a capture between element and synapse.
 "timeline": {
    "events": [
        {
            "type": "m.room.message",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "1",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>468,
            "unsigned": {
                "age": 23055
            },
            "event_id": "$2OA5pqKmifrtyEmjbc_6tbJDm6rmsCCTXJ12egorYDA"
        },
        {
            "type": "m.room.message",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "2",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$2OA5pqKmifrtyEmjbc_6tbJDm6rmsCCTXJ12egorYDA"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>444,
            "unsigned": {
                "age": 22079
            },
            "event_id": "$T1DcOiCSWiLZbhNynC42gIFnANNg3Rz4hgNuuLXcU7M"
        },
        {
            "type": "m.room.message",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "3",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$T1DcOiCSWiLZbhNynC42gIFnANNg3Rz4hgNuuLXcU7M"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>659,
            "unsigned": {
                "age": 20864
            },
            "event_id": "$U59vv0DEWtbSi8aTejoZ8JJwb6CFY-1Xi2IjBdSIQ0Q"
        },
        {
            "type": "m.room.message",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "4",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$U59vv0DEWtbSi8aTejoZ8JJwb6CFY-1Xi2IjBdSIQ0Q"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>453,
            "unsigned": {
                "age": 20070
            },
            "event_id": "$R4AYXbSH9VnH5b0j8uBziJ8zO3R2riPMbPKBvdmWIlE"
        }
    ],
    "prev_batch": "s28_934_2_15_14_1_1_10_0_1",
    "limited": false
}

This is a synposis including details talked about in the conduwuit matrix group.
The only difference that looks significant to me between Synapse and Conduwuit is that Synapse's unsigned age parameter varies in descending order from oldest to newest messsage, and Conduwuit's value is always 1.
Here's where age was hardcoded to 1: https://github.com/timokoesters/conduit/commit/acfe381dd3064512272f8f47ea4dd388c04f1c39
Here's what the spec says about it: https://spec.matrix.org/v1.11/client-server-api/#room-event-format

The time in milliseconds that has elapsed since the event was sent. This field is generated by the local homeserver, and may be incorrect if the local time on at least one of the two servers is out of sync, which can cause the age to either be negative or greater than it actually is.

The issue most likely lies in underlying event relations code, specifically this line that seems to ignore the direction: https://github.com/girlbossceo/conduwuit/blob/main/src%2Fservice%2Frooms%2Fpdu_metadata%2Fmod.rs#L100

I set a breakpoint at that line and it does not get tripped by a new user joining a test room and clicking an existing thread, nor when starting a new thread and adding to it. I checked my environment by setting a breakpoint on the "typing" service and that tripped.

Can you compare the direction behaviour request responses between Synapse and conduwuit?

Can you compare the direction behaviour request responses between Synapse and conduwuit?

Clients: 2x element-desktop executed via flatpak with different profiles. Both clients connected to the same server.
Synapse
GET https://s.dataheck.com/_matrix/client/v1/rooms/!fpdVzFVTBbeJtYJCUS%3As.dataheck.com/relations/%24MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY?dir=b
{
    "chunk": [
        {
            "type": "m.room.message",
            "room_id": "!fpdVzFVTBbeJtYJCUS:s.dataheck.com",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "4",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$U59vv0DEWtbSi8aTejoZ8JJwb6CFY-1Xi2IjBdSIQ0Q"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>453,
            "unsigned": {
                "age":<PHONE_NUMBER>
            },
            "event_id": "$R4AYXbSH9VnH5b0j8uBziJ8zO3R2riPMbPKBvdmWIlE",
            "user_id": "@matt:s.dataheck.com",
            "age":<PHONE_NUMBER>
        },
        {
            "type": "m.room.message",
            "room_id": "!fpdVzFVTBbeJtYJCUS:s.dataheck.com",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "3",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$T1DcOiCSWiLZbhNynC42gIFnANNg3Rz4hgNuuLXcU7M"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>659,
            "unsigned": {
                "age":<PHONE_NUMBER>
            },
            "event_id": "$U59vv0DEWtbSi8aTejoZ8JJwb6CFY-1Xi2IjBdSIQ0Q",
            "user_id": "@matt:s.dataheck.com",
            "age":<PHONE_NUMBER>
        },
        {
            "type": "m.room.message",
            "room_id": "!fpdVzFVTBbeJtYJCUS:s.dataheck.com",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "2",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$2OA5pqKmifrtyEmjbc_6tbJDm6rmsCCTXJ12egorYDA"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>444,
            "unsigned": {
                "age":<PHONE_NUMBER>
            },
            "event_id": "$T1DcOiCSWiLZbhNynC42gIFnANNg3Rz4hgNuuLXcU7M",
            "user_id": "@matt:s.dataheck.com",
            "age":<PHONE_NUMBER>
        },
        {
            "type": "m.room.message",
            "room_id": "!fpdVzFVTBbeJtYJCUS:s.dataheck.com",
            "sender": "@matt:s.dataheck.com",
            "content": {
                "msgtype": "m.text",
                "body": "1",
                "m.relates_to": {
                    "rel_type": "m.thread",
                    "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY",
                    "m.in_reply_to": {
                        "event_id": "$MmkcrQpY6rGhBZY4JSNk-Oej0t9itZd01VBYmHSZlJY"
                    },
                    "is_falling_back": true
                }
            },
            "origin_server_ts":<PHONE_NUMBER>468,
            "unsigned": {
                "age":<PHONE_NUMBER>
            },
            "event_id": "$2OA5pqKmifrtyEmjbc_6tbJDm6rmsCCTXJ12egorYDA",
            "user_id": "@matt:s.dataheck.com",
            "age":<PHONE_NUMBER>
        }
    ]
}

It requested reverse chronological order, and that's what it got. Shows fine in element-desktop: chronological. I visually inspected 3 other similar requests for different threads, they match this behaviour.
Conduwuit
(I removed my dir overriding test for this)
GET http://sharedbudgets.local/_matrix/client/v1/rooms/!fhpvWsfeOxpE5JKZCB%3Asharedbudgets.local/relations/%24xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU?dir=b [^1]
[^1]: sharedbudgets.local = <IP_ADDRESS>
{
    "chunk": [
        {
            "content": {
                "body": "1",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$jA2D_6tXtlhFQi7EYBUZctS-I6CdJBInBtvZ4EuMZNE",
            "origin_server_ts":<PHONE_NUMBER>838,
            "room_id": "!fhpvWsfeOxpE5JKZCB:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {}
        },
        {
            "content": {
                "body": "2",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$jA2D_6tXtlhFQi7EYBUZctS-I6CdJBInBtvZ4EuMZNE"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$60IAks924vMegDh8RCsBL4sJc2o0F6U_n3XBBWlNlYA",
            "origin_server_ts":<PHONE_NUMBER>171,
            "room_id": "!fhpvWsfeOxpE5JKZCB:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {}
        },
        {
            "content": {
                "body": "3",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$60IAks924vMegDh8RCsBL4sJc2o0F6U_n3XBBWlNlYA"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$CS1UCmIZ_Lvo-sZIQRdLngS0c7zC94I3ZiknAU2WafM",
            "origin_server_ts":<PHONE_NUMBER>496,
            "room_id": "!fhpvWsfeOxpE5JKZCB:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {}
        },
        {
            "content": {
                "body": "4",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$CS1UCmIZ_Lvo-sZIQRdLngS0c7zC94I3ZiknAU2WafM"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$LP83ESqrzGuXVNExrt3fpVDJgqrBHVXXoogKt75GGlQ",
            "origin_server_ts":<PHONE_NUMBER>820,
            "room_id": "!fhpvWsfeOxpE5JKZCB:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {}
        },
        {
            "content": {
                "body": "5",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$xdd718Rv2y456IfBRnGN_hjlqgkrcI0QpWJ0dL7zLdU",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$LP83ESqrzGuXVNExrt3fpVDJgqrBHVXXoogKt75GGlQ"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$DPWJ4TDT5XTrGvfKSHfpVZB9Er281O5bvGsKfIB06x8",
            "origin_server_ts":<PHONE_NUMBER>258,
            "room_id": "!fhpvWsfeOxpE5JKZCB:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {}
        }
    ],
    "next_batch": "1337",
    "prev_batch": "18446744073709551615"
}

Okay, maybe conduwuit is wrong. paginate_relations_with_filter sees that the client requested dir=Direction::Backward but we're actually returning data chronologically.

Here's a response snippet from the commit referenced above; it's now in reverse order as requested by element:
{
    "chunk": [
        {
            "content": {
                "body": "3",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$7jry-CavwdYk3DEWIgzPEJ8ml9c9HvA7hhqkuV2oxKg",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$42gs7TDdW5voOQFpNgRfiJWzB6gYK83dk-mMRruzXSM"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$TIX0EEGSi-Fsfyrwfx1F8JipvwAzZ8jOjHZ83zggXq8",
            "origin_server_ts":<PHONE_NUMBER>141,
            "room_id": "!AX4HOUVizHqi0FSSTZ:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {
                "transaction_id": "m1723814631088.3"
            }
        },
        {
            "content": {
                "body": "2",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$7jry-CavwdYk3DEWIgzPEJ8ml9c9HvA7hhqkuV2oxKg",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$_M8VQjBJm3ivFBQnHwfmXWOtE1JA7fXnlIl0CC5VZdE"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$42gs7TDdW5voOQFpNgRfiJWzB6gYK83dk-mMRruzXSM",
            "origin_server_ts":<PHONE_NUMBER>847,
            "room_id": "!AX4HOUVizHqi0FSSTZ:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {
                "transaction_id": "m1723814630790.2"
            }
        },
        {
            "content": {
                "body": "1",
                "m.mentions": {},
                "m.relates_to": {
                    "event_id": "$7jry-CavwdYk3DEWIgzPEJ8ml9c9HvA7hhqkuV2oxKg",
                    "is_falling_back": true,
                    "m.in_reply_to": {
                        "event_id": "$7jry-CavwdYk3DEWIgzPEJ8ml9c9HvA7hhqkuV2oxKg"
                    },
                    "rel_type": "m.thread"
                },
                "msgtype": "m.text"
            },
            "event_id": "$_M8VQjBJm3ivFBQnHwfmXWOtE1JA7fXnlIl0CC5VZdE",
            "origin_server_ts":<PHONE_NUMBER>419,
            "room_id": "!AX4HOUVizHqi0FSSTZ:sharedbudgets.local",
            "sender": "@matt:sharedbudgets.local",
            "type": "m.room.message",
            "unsigned": {
                "transaction_id": "m1723814630381.1"
            }
        }
    ],
    "next_batch": "1675",
    "prev_batch": "1682"
}

This should be fixed now with https://github.com/girlbossceo/conduwuit/pull/555

Confirmed the latest main branch works for me! Thank you both! ❤


Adding support for Caffe2 and PyTorch

@Tomcli based on your investigation what`s needed on this side?

For CPU, simply add the new framework options and pull the images in the lcm/container_helper_extensions.go. Then we need to add some unit test and example jobs and should be good to go.

For GPU, they are on the same boat with Caffe and TensorFlow where we need to implement a trigger in LCM to switch the job between CPU and GPU seamlessly.

Fixed in #24


Clojurecript repl on remote repl

is the package capable of running cljs in remote repl?

It has some limited support for it. There were problems in the past connecting to a ClojureScript REPL but most of them should be fixed. I'd classify it under experimental right now.

On Jul 24, 2016, at 3:56 PM, karneaud<EMAIL_ADDRESS>wrote:
is the package capable of running cljs in remote repl?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

i was able to connect remotely to repl but when trying to run clojurescript via cljs.repl with rhino I'm able to get the prompt under user but not able to execute any code
to reproduce

connect to remote repl
required clj ```` (require
'[cljs.repl :as repl]
'[cljs.repl.rhino :as rhino])

(repl/repl* (rhino/repl-env)
{:output-dir "out"
:optimizations :none
:cache-analysis true
:source-map true})```

when connected ran (+ 1 1)

not sure what the problem is as I was able to do this locally via terminal on the host

I have a test repo that almost works with figwheel at https://github.com/gmp26/figwheel-atom-editor-starter. This follows the same figwheel setup as for intellij, but with an extra project.clj dependency on proto-repl. You have to run the script/repl.clj commands by hand once nrepl starts to get you into the cljs browser repl.
It starts up and works once, but shut down is problematic as :cljs/quit does nothing, and somehow the figwheel port 3449 (or 3450 in this case) never closes. Even closing down atom does not close the port.
In fact, none of the figwheel commands (stop-autobuild) .. (fig-status) work as they are all intercepted somewhere and barf.

@gmp26 With the most recent version of Proto REPL, you should now be able to use :cljs/quit, (fig-status), and the like and they should work correctly now.

@rgdelato Very nice. Not in a position to try this immediately, but good to know it's there.


[Backend] Add GET api for fetching the questions of Q&A

Deliverabels-

[ ] We need 2 GET api for fetching the question

Get all the questions
Get a particular question using id where id will be provided in the json body

Ref- https://github.com/HITK-TECH-Community/Community-Website/tree/main/backend/app/routes/Q%26A/question

Please assign this task to me


Button (tap) event runs twice

So this is a strange one, and was hard to reproduce.
Environment
Whatever version play.nativescript.org is running.
However, I'm seeing the same in my own setup with nativescript-angular 7.1.2
ONLY TESTED ON IOS!
Describe the bug
The tap event runs twice
To Reproduce
Check https://play.nativescript.org/?template=play-ng&id=oSZ0t7&v=7
The proof of concept is in the home.component.
Click the button once, and check the logs.
....
If you remove the *ngIf from the button it will run twice on the first click, and then only once the following clicks. I added a few setTimeouts to cycle the component, which will cause the problem to be apparent on every click.
It also seems related to the HostListener on the btnClick directive I'm using, which listens for the touch event. I'll leave you to figure out whats what.
Expected behavior
I expect the tap event to behave, and only trigger once.
Sample project
https://play.nativescript.org/?template=play-ng&id=oSZ0t7&v=7

The solution is just to do this:
import { Component, OnInit } from "@angular/core";
@Component({
selector: "Home",
moduleId: module.id,
templateUrl: "./home.component.html",
styleUrls: ['./home.component.css']
})
export class HomeComponent {
isClicked = false;
clickMe() {
    if (this.isClicked)
        return;
    this.isClicked = true;

    console.log('Clicked!');

    // THANKS: https://stackoverflow.com/questions/41659822/when-and-why-to-use-a-tick-in-angular-2
    // So the 0 is to make the change detection happen immediately and the empty function is because we don't want to do anything.
    setTimeout(() => {
        this.isClicked = false;
    }, 0);
}

}
Enjoy.

any update on it?

It still have bugs when you click fast. :'(

any update on it in 2020?

@tahir-jamil you can use this solution

@tahir-jamil you can use this solution

You call that a solution ?
I still have a lot of unwunted double tap, on ios and android
Please reopen

any solution in March 2023 ?

@gyan-nsf please open a issue on https://github.com/nativescript/angular with a reproduction. I can't reproduce a double tap issue on my projects


pr2_kinematics: 1.0.7-1 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository pr2_kinematics to 1.0.7-1:

upstream repository: https://github.com/PR2/pr2_kinematics.git
release repository: https://github.com/pr2-gbp/pr2_kinematics-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.18
previous version for package: null

pr2_arm_kinematics
* Fix pr2_arm_kinematics for indigo by adding cmake_modules
* Add dependency to cmake_modules to solve Eigen depdency on indigo
* Contributors: Ryohei Ueda

pr2_kinematics

No changes

@TheDash you can also rerun bloom with --pull-request-only. It will be much quicker since it won't actually do the rerelease, just reopen the pull request.

I did run it with --pull-request-only. It hasn't been released into the distribution.yaml before, and the release repo has changed from hydro to indigo, so It asks me some things.

Huh, maybe it doesn't work for that case, I would have expected it not to
produce a debian increment in the version number.

According to the README in the release repository you ran it twice without --pull-request-only, but I don't know if this pull request was the result of that last one or not:
https://github.com/pr2-gbp/pr2_kinematics-release/blob/master/README.md

Right. The data gleaned which lead me to the conclusion in my prior comment
was from making another bloom-release of a similar package (the
pr2_make_a_map one). I assumed that info would extrapolate here.
On Tue, Feb 10, 2015 at 3:22 PM, William Woodall<EMAIL_ADDRESS>wrote:

According to the README in the release repository you ran it twice without
--pull-request-only, but I don't know if this pull request was the result
of that last one or not:
https://github.com/pr2-gbp/pr2_kinematics-release/blob/master/README.md
—
Reply to this email directly or view it on GitHub
https://github.com/ros/rosdistro/pull/7192#issuecomment-73776163.

--
Devon Ash
View Devon Ash's profile


Less verbose output to prevent baseline regeneration

Hi @TomasVotruba,
thank you for the extension!
After adding to older code base and adding errors to the baseline (to check only new code at this moment - more fixups planned) we are forced to re-generate the baseline with quite a lot of changes.
Is it possible to force less verbose mode? Now we have over 4000 warnings on code that was not changed :)
Thank you for your time,
P.S: we could make the change, let me know where to change it and we can do it.

Hi, this is better used in a different way :)
Let's say we have a 70 % type coverage. Instead of ignoring errors, we always bump it to higher level then I have, e.g. 90 %.
 # phpstan.neon
 parameters:
		type_coverage:
-				return_type: 70
+				return_type: 90

Then we run PHPStan:
vendor/bin/phpstan

Fix as many erros as we can. Let's say we reach the level of 80.
Then instead of ignoring everything, we change it back to 80.
 # phpstan.neon
 parameters:
		type_coverage:
-				return_type: 90
+				return_type: 80

another option here would be to not print the actual percentages in the "error" but just let it state "missing param type" for example. the percentages can be printed as a hint. that would keep the changes on the baseline low while also providing useful information.


Fix init banks from blockstore not working properly error

Fix https://github.com/soonlabs/igloo/issues/16


[devops:bot] update deps in nix/sources.json (from: rhino-core@4cfecdc

update nix refernces for rhino-core and|or nixpkgs to match https://github.com/rhinofi/rhino-core/blob/4cfecdc1fa9761f002d244bc659bda3204777e9f/nix/sources.json (branch: dev)

superseded by #206


Stopping or restarting node.js debugger crashes vscode

VSCode Version: 1.6.1
OS Version: OSX El Capitan 10.11.6
Node version: 6.7.0

Steps to Reproduce:

Start vscode through the finder
Open a new folder
Create a new file app.js in the root of the folder with contents

var msg = 'hello world'
console.log(msg)

[optional] Confirm that "hello world" is printed when running the file with
node app.js
Click the gear in the DEBUG pane to create a Node.js launch.json config.
Hit F5 or click the run button to run app.js with the debugger attached
Expected behavior: "hello world" is printed to the console. No breakpoints are set, so the program runs and terminates, and the debugger detaches automatically.
Actual behavior: "hello world" is printed to the console. The debugger stays attached.
Hit Shift-F5 or the Stop Debugging button to detach the debugger and terminate the node program.
Expected behavior: The debugger detaches and vscode exits debug mode.
Actual behavior: Nothing happens for about 10 seconds, then vscode crashes.

Possible clue
If vscode was launched from the command line with
code
(see the shell command setup here https://code.visualstudio.com/docs/setup/mac)
Then step 7 produces the following output in the debug console, and vscode does not crash.
/usr/local/Cellar/node/6.7.0/bin/node[57339]: ../src/debug-agent.cc:149:void node::debugger::Agent::Stop(): Assertion `(err) == (0)' failed.
 1: node::Abort() [/usr/local/bin/node]
 2: node::RunMicrotasks(v8::FunctionCallbackInfo<v8::Value> const&) [/usr/local/bin/node]
 3: node::debugger::Agent::~Agent() [/usr/local/bin/node]
 4: node::debugger::Agent::~Agent() [/usr/local/bin/node]
 5: node::Environment::~Environment() [/usr/local/bin/node]
 6: node::Start(int, char**) [/usr/local/bin/node]
 7: start [/usr/local/bin/node]

Could you try with "type": "node2", our experimental new debugger, to see if it works any better? I'm also curious whether you see this with different versions of Node, or in vscode insiders build - code.visualstudio.com/insiders.

app.js not terminating properly because of this node bug: https://github.com/nodejs/node/issues/1788
I cannot reproduce the crashing with VS Code 1.6.1, macOS 10.11.6, and node 6.7.0.

@aheitzmann It is possible that this issue is caused by an installed extension. Can you try to run VS Code without extensions? From the command line, execute: code --disable-extensions and try your steps again to see if it reproduces.
@roblourens node-debug2 shows the same behaviour: app.js does not terminate properly but crashing does not occur.

That part is expected because of this bug unfortunately - https://github.com/Microsoft/vscode-node-debug2/issues/11

@weinand  Unfortunately the crash does not reproduce when vscode is launched from the command line. I get the "Assertion `(err) == (0)' failed" error in that case whether or not --disable-extensions is specified.
I tried simply uninstalling all of my extensions, and the crash still reproduces when launched from the finder.
My user settings are
// Place your settings in this file to overwrite the default settings
{
    "editor.rulers": [100],
    "editor.wrappingIndent": "indent",
    "diffEditor.ignoreTrimWhitespace": false,
    "files.trimTrailingWhitespace": true,
    "files.autoSave": "afterDelay",
    "typescript.useCodeSnippetsOnMethodSuggest": true,

    "vim.useCtrlKeys": true
}

and I'm attaching a zip of the project in case that helps with the repro.
nodecrashrepro.zip

@roblourens The crash also reproduces with the "type": "node2" debugger.

@roblourens The crash also reproduces with the latest vscode insiders build.

I've started seeing this issue as well with vscode 1.8.1, El Capitan, node version 6.9.1
Things were working just fine until yesterday, although I'm not sure what I have changed since then.
There was an update to vscode that I did install, but trying to revert to an earlier version still results in a  crash on debugger detach.
Has anyone else experiencing this? Since the last update to this issue was in October.

Please try the latest VS Code and a recent version of node.js. If the problem still exists, we can reopen this issue.


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #1

It's a good idea to open an issue first for discussion.

[/] Tests pass
[ ] Appropriate changes to README are included in PR

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
View this failed invocation of the CLA check for more information.
For the most up to date status, view the checks section at the bottom of the pull request.

I fixed My CLA


what is the machine Requirements , My machine is 8C 16G But is memory is become bigger and bigger, what is the problem , Memory leak

Environment
The CLI can be used to gather generic information about your system to assist with support.
No installation is required thanks to npx
npx @cardano-graphql/cli system-info

[PASTE_REPORT_HERE]
Alternatively, what release or Git revision are you using:
Platform

[ ] Linux / Ubuntu
[ ] Linux / Other
[ ] macOS
[ ] Windows

Platform version:
Runtime

[ ] Node.js
[ ] Docker

Runtime version:
node -v
docker version

Steps to reproduce the bug
What is the expected behavior?

Typically the RAM usage will grow as the cache in RAM increases, which is standard for a lot of databases. I've got servers with 64GB of RAM and typically it won't go above 29GB of RAM usage. So the minimum requirements are a lot lower, but if there's RAM it can use then it will.

It is 32 G ok

The official requirements according to db-sync (which is what cardano-graphql is based on) is 16GB, so 32GB should easily be enough unless you're doing hundreds + of concurrent queries. https://github.com/input-output-hk/cardano-db-sync


Dired 'move' shouldn't default to first filename found in directory

When I select files to move into a directory, selectrum will default to a file inside the directory. My intention is not to move/rename my file(s) to that filename, but rather move the files into that directory.

If the directory is empty or I don't (sometimes) complete the path with a /, it will work properly I think. Otherwise, i'm not sure how to move files into a directory that already has files in it.
I'm using version 20200504.1619 from melpa.

Why do you type .. instead of DEL?

I see. I guess a matter of preference - I can't remember why I had started using ... But behavior should be the same, no?

I  tried it and it works to select the directory you want the files to move in and press RET. Note that you can also select the prompt. What do you expect or where do go things wrong?

When you type ../ you are supposedly 1 directory up, so you should be able to copy files directly there. You should not have to go up 1 more directory to select the directory that is one up.
ie. if you have files in
/home/name/hi/
and you go to /home/name/hi/../ you are given options within name, but you can't actually copy files into name. You have to go up again to /home/name/hi/../../ to be given name as an option.
I find that unintuitive, as ../ should be equivalent to deleting hi/ in the minibuffer.

but you can't actually copy files into name. You have to go up again to /home/name/hi/../../ to be given name as an option.

You can copy file into name by selecting the prompt (using C-p). That is how selectrums UI work otherwise you would have to include ./ as a candidate. But for reading directory names it might make sense to include ./ as default (although somehow dired-move/copy don't use read-directory-name for some reason so this wouldn't work in this case).

Gotcha, I see.
Yeah, I would love for it to be default. C-p to me seems like the Counsel-M-x workaround I was having to use (in order to do things like edits with multiple cursors), which is actually what prompted me to leave the counsel, ivy, swiper suite, funnily enough.

It's an extra binding in this case, in other cases you would have an extra binding to skip the ./. The dired file movement commands don't seem to use read-directory-name. You probably don't want ./ included in regular find-file and friends?

I think ido's default for find-file is ./ and would drop you there with dired. So I would want to include it there too. Counsel-M-x also behaved this way I think.

I just had an idea for an alternative way to support your use case: There could be an option to select the prompt by default for file navigation (when there is no input) that would give you the same behaviour without needing the ./ as a candidate.

Okay, so the way I'd been using normal find-file a lot would be to hit C-x C-f C-f to get ./. But yeah, it looks like ./ would normally be excluded.

I just had an idea for an alternative way to support your use case: There could be an option to select the prompt by default for file navigation (when there is no input) that would give you the same behaviour without needing the ./ as a candidate.

That sounds awesome

I just had an idea for an alternative way to support your use case: There could be an option to select the prompt by default for file navigation (when there is no input) that would give you the same behaviour without needing the ./ as a candidate.

I have to think how that would have to be integrated with #70.

The way that Selectrum encourages you to select a directory is by removing the trailing slash. Is that acceptable for you, @tinglycraniumplacidly? I understand that it may be less intuitive, but I am hesitant to complicate the interface when simplicity is a design goal and there is a straightforward way to achieve the desired behavior already.

Cool, I think that's the best overall solution for what I'm looking for.
I will say I think it's not that intuitive to know to delete the trailing slash. (Maybe just me, but I think I've been burned in the past with other methods when deleting the trailing slash, where the file would overwrite a symlinked dir instead of placing the file in the dir? But that doesn't seem to be the case with selectrum).
Thanks!


Updated Dockerfile to include Vim and Nano editors inside Sparkler Container

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Is this related to an already existing issue on sparkler?
If so, mention that issue by referencing its number here.
Will it close an existing issue?
Say 'Closes #IssueNum' here.
How was this patch tested?
We are particularly interested in unit tests, integration tests, manual tests you did to ensure that the patch works as expected, so briefly describe them.
Please review
https://github.com/USCDataScience/sparkler/blob/master/.github/CONTRIBUTING.md before opening a pull request.

Add emacs too

Hi Professor, Emacs was already added.

Got it thanks

Thank you. :)

Hi, I have installed sparkler today and the issue persists:
$ nano
sh: 3: nano: not found
$ emacs sparkler/bin/seed-urls.txt
sh: 4: emacs: not found
$


Update increment value if ID is overridden on new document

Replaced checking of document field (i.e. _id) type as a number with checking if document is new (using isNew property) - http://mongoosejs.com/docs/api.html#document_Document-isNew.
Added check for provided ID override, where it checks the new ID is greater than the current count, else it increments the current count and sets the document ID with that value.

Well done, thank you.

No problem at all - I had to have this functionality for my current set up anyway, and I wanted to make this behave more like *SQL does for those importing tables over to mongo


Improve CI

I improved CI.

Ah, unfortunately, I can't accept the changes to setup-ruby. I like to ensure all my projects use the same actions I created so I don't have to think about it. Besides which, setup-languages already performs these operations: https://github.com/yettoapp/actions/blob/abbd75422c1118794948f6cc682efe5a7cf988b0/setup-languages/action.yml#L33-L40
Also, why delete the .ruby-version?

I like to ensure all my projects use the same actions I created so I don't have to think about it.

I changed to use yettoapp/actions/setup-languages.

Also, why delete the .ruby-version?

Repositories of commonly used Gems such as Puma, Faraday, and RuboCop do not contain .ruby-version file.
I think Ruby version should not be specified in the Gem repository, so that is why I added matrix to CI.
However, it's not a big problem, so it's okay to restore the .ruby-version file👌

For reference, below is the .gitignore for Ruby from github/gitignore. https://github.com/github/gitignore/blob/6eeebe6f49678aacd8311ce079842c971b3ebe96/Ruby.gitignore#L46-L50

Oh, that's really interesting. In all my years of Ruby I never knew that was suggested.
But, what if the code in the project is written using some new Ruby syntax, that the contributor doesn't have installed? In other words, how would I communicate to someone else that "Ruby 3.x.y is recommended to work with this project"?

How about using TargetRubyVersion in .rubocop.yml?
I just commited now.
Ruby version of this repository is 3.1, so I specified 3.1 in .rubocop.yml.
For reference, below is the .rubocop.yml of Faraday.
https://github.com/lostisland/faraday/blob/a9cf00425e3abc99b78952af44deb2912a65a882/.rubocop.yml#L10


Require description when creating a new project

Description
Disables the create button if a project description is not entered (minimum 8 characters). Adds a popup explaining the requirements.
Tests
No server side changes.
For frontend: go to sample upload page, click create project. Test that the "create project" buttons is disabled, and on hover has a popup explaining the requirements. Test that the popup changes when each requirement is fulfilled. The popup should disappear completely and the "create project" button enabled when all requirements are fulfilled.

Hey @lvreynoso! Thanks!
Few questions/comments:

I was wondering: why is there an 8 character limit on the project description?
I think a shorter description would be sufficient like "Complete all fields to create a project."
Can the tooltip be centered on the "CREATE PROJECT" button?

Thanks for taking a look Jenn! The 8 characters isn't a limit, it's a minimum; but it's also an arbitrary number, I can just check for at least 1 character in the field. I can also make the tooltip have a shorter description and centered on the Create Project button.

Updated popup:

Thank you @lvreynoso! Can you make the tooltip width smaller? So it's more balanced with more words on the bottom =]=] Thank you!!

More aesthetic popup display:


Connecting module to car battery

I've had this module for a couple of months now and it works like a charm!
I have it making  POST requests to my website using the gps_tracker demo with a few alterations that I made to the code but the problem I have is that whenever I start the car it takes several minutes before it can actually get a gps fix.
The GSM part works right away but the GPS takes a while so I'm thinking maybe I should connect the module to the battery and  put it to sleep when the car gets turned off, so that way maybe it takes less time to get a fix in the morning.
I don't have a lot of experience working with microcontrollers. I'm basically a self-taught web developer so Javascript, PHP, Node JS, etc, I'm good at but I don't exactly know how to make the right connections in order for this to work and not burn the module.
I bought a 12v to 5v-3amp converter and  based on this #129  I should connect the +wire from the car battery to both the VBAT (Pin 17) and to the VUSB/5V Pin ?
Any help would be greatly appreciated!

The A9G module supports only 3v5 to 4v2(idle value is 4v0) on VBAT pin, so if your module has little pot on board you can change it to 4.0v and then you're good to go.

OK, thanks.
So I've read a bit on how to step down from 5v to 4v and I think that what I would have to do is connect the 12v to 5v converter to the car battery and then, with either silicon diodes or resistors, step down the 5v to 4v and plug it in the VBAT pin, and I could read the voltage to trigger an event to make the module go to sleep if it's 4volts and wake up if it's 5volts.
The module would be connected to the car radio I think, that way whenever the car starts the module should get the 5volts.
But what happens when the car starts ? The module would burn by receiving 4 volts from the VBAT pin and 5 volts from VUSB/5V at the same time ??

I'm guessing you're using the pudding dev board, right? VBAT is already powered by 4.2V from an onboard 5V to 4.2V regulator. VUSB is not a power supply pin, it just monitors for USB power and turns on the module on.
What you need to do is find a power connection in your car that is always powered, even when the car is off. I think the OBD-II port is always powered with 12V. Now that your module is always powered you then need to add some code to monitor a 12V supply that only turns on when the car is on. The other 12V supply can't be directly connected, you need a [voltage divider](voltage divider). Vin to 12V, Vout to an input pin on the module. 10k and 3.9k will be suitable values for the resistors. When the pin reads HIGH, the car is on. When LOW, the car is off.
Now, the actual power saving. The GPS receiver must always be enabled, otherwise the position will be lost. There's not much documentation for the GPS receiver in the A9G, but there does seem to be GPS_SetStandbyMode() and GPS_SetLpMode() functions in https://github.com/Ai-Thinker-Open/GPRS_C_SDK/blob/master/libs/gps/src/gps.c and https://github.com/Ai-Thinker-Open/GPRS_C_SDK/blob/master/libs/gps/include/gps.h
You should put the GPS receiver into low power modes and disconnect from the GSM network when the 12V monitoring pin reads LOW. When the pin is HIGH you then reconnect to GSM and put GPS into full running mode again.
But be warned, the module is very unstable, it might only last a few days before it crashes.

Cool! I'll have to read up on your response because like I said I'm a noob in microcontrollers but now I know which way to go. It will take me a couple of weeks to get this going.
I haven't had any problems with the GSM network. The module connects very fast and it starts transimitting the data without problems. It's the GPS getting a fix that's my current problem. It didn't had that many problems a month ago or when I stopped using it that much to start working with the Google Maps API, but now after not using it for that period it struggles to get a GPS signal. Maybe it's an antenna problem so I'll try to buy a bigger one and see if that solves it.
Anyway thanks for response, I'll post the solution once I implement it based on the info you gave me.

hi @marcosasilvalepe ,
why do you want to connect it to car battery?Theres always a risk of power loss may be during cranking,
Instead try to hook up a ~1000mAh po bLiPo battery(make sure they stay cool in your car as they might burst!).
I recently bought this Module and am also trying to buid a car GPS tracker with MQTT.
Interested in your progress.

Hi @vkhungund I just don't want to lose the GPS signal because it takes too long for it to get a fix again once the module restarts.
Like I said I'm using the gps_tracker demo to make post requests with latitude, longitude, speed, timestamp and other variables to a php script in a website that captures them and inserts all the values in a MYSQL database, and on the client side I have a Javascript function that runs every 2 seconds and fetches the last row of the table with the latest values to slide the marker to the new position on the map. The module handles itself very well and it was pretty stable about a month ago but like I said I stopped using it to work on the Google Maps API and now for some reason it struggles to get a GPS fix.
I had actually bought a couple of 3.7v 500mah Li-ion batteries but because of covid-19 it has taken longer than usual for them to get here so I have to wait to keep playing with the module but I'll post when I make some progress. Cheers.

So to do a follow up, based on the answer by @zkemble (thanks for that) I ended up with this

It looks pretty amateur but like I said I'm a noob in microcontrollers and it works.
I checked the cables from the car radio and it turns out that there is a cable that is always powered with 12V so that the radio doesn't get disconfigured everytime you turn the car off, and another 12V cable that turns the radio on when the car is on.
So in the image the orange cable only gets power once the car is on, and the brown cable is connected to a 12V to 5V converter, that way the module always has power.
I switched to pulkin micropython version to program the module and by simply doing pin.value() you get either 0 when the car is off and that way the module needs to go to sleep, or 1 when it's on and it should transmit coordinates through the internet.
I'd like to install a small battery to the module now just to provide power in case it can't get it from the car battery, like if for instance someone stole the car and cuts the module's power cable, that way the module can transmit coordinates anyway at least for a few minutes or hours, but I don't know how to do that with my current configuration.
There are 2 VBAT pins in the module and if I connect a battery while getting power from the 5V cable at the same time, wouldn't it burn the module because it get's power from 2 different sources ??


License

Hi,
I really like to thank you to share such a great theme. It looks like the best fit for our next documentation project and we are taking into account to use it. One thing I have not found so far is the License you are sharing this project. Could you please add a license file?
Cheers
Johannes

The license file is MIT. It's inside the licenses folder: https://github.com/tomjohnson1492/documentation-theme-jekyll/tree/gh-pages/licenses

Ok, I was not searching for an folder. Thanks a lot


noindex searches that use keywords

We may want to noindex all search pages, but this will require some further thought.
This stops spurious search terms from being indexed by external search engines and limits search indexer access to search results pages which only use the facets we've defined (or just paginating through unfaceted results.)
I've tested the primary path, but haven't worried about being too comprehensive as it's a pretty trivial change.  I'll be pushing for noindexing these altogether anyway so this will likely change soon.
Relevant card is https://trello.com/c/4EAytHVV/165-plan-noindexing-of-finder-pages

Search page examples to sanity check:

http://finder-frontend-pr-1126.herokuapp.com/search/all
http://finder-frontend-pr-1126.herokuapp.com/search/research-and-statistics
http://finder-frontend-pr-1126.herokuapp.com/search/advanced?group=guidance_and_regulation&topic=%2Feducation
http://finder-frontend-pr-1126.herokuapp.com/drug-device-alerts
http://finder-frontend-pr-1126.herokuapp.com/find-eu-exit-guidance-business

Other finders

That'd be cool, I'll have a look thanks @bilbof .  I think I saw some contradictory advice somewhere too, so I'll merge this for now and iterate. 👍


Update MD file on new branch 2017-33-02_21-12-32

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

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.


Add social media icons

This PR adds twitter, facebook and youtube links to the page footer, as requested in issue #44. By the way, could someone double-check the links actually point to the right accounts? I'm not 100% sure.
Picture: https://i.imgur.com/zvUCgkK.png

Hmm. I'm not sure about that placement. There is no other navigation there in the site's design, it's very out of place. The design is that they should be in the top bar, but there isn't much room up there... maybe a dropdown?
Could an alternative be sidebar widgets? I know twitter and facebook (assuming it doesn't do tracking evilness) have widgets that would be nice. I don't know about youtube however...
All of the accounts are right btw!

I can't think of sites that don't have the general social media links in the footer off the top of my head. Of course, share links and such are usually placed in a very prominent manner, but that's a wholly different use-case. Could you link any example sites you have in mind?

But those sites follow a top nav + footer navigation style - dolphin-emu.org does not. dolphin-emu is straight up bootstrap, which was designed by twitter for... twitter, which is infinite scrolling (footers are impossible).
Honestly dolphin-emu.org would probably benefit from a footer navigation system. It would allow the top nav to be user facing things (forums, screenshots, guides, wiki, etc etc) while the footer can be more specific with things we are ok with not every user seeing (issue tracker, github pages, fifoci, dolp.in, all kinds of things that people have to memorize right now!) It would be a lot better than what we have now! But that would require an entirely new footer to accomplish, though...
Aaaanyway, that is kind of beyond the scope of this PR... I don't really think placing the icons in the footer underneath the copyright notice is good. It is against the design intentions of bootstrap, and it shows - discoverability is terrible! So, I think things should be done right with a proper footer, or stick to the boostrap design and put them in the top bar somehow.

By sidebars, you mean the wells in the front-page? (compability/ blog tags) Would they only exist on the front-page in that case? Where's the google plus icon located, I can't seem to find it.

The google plus widget was removed. :P I did say "was handled"! If I remember correctly, it was placed beside the RSS icon. That's not the best place for it, but, it worked ok.
As for the sidebar, yes the compatibility and blog tags are in it. No one actually uses a blog tag cloud, and it would be a loooot better to have a twitter sidebar there linked to various developers' twitter accounts and the official account, but meh. And a facebook like widget too. ...and whatever youtube has. But anyway, that's an alternative.

I'm a bit confused, I must say. When you're talking about the google +1 and widgets, do you mean "Share" buttons for twitter and facebook and such? The way I understood the issue, it concerned a need for actual links to the various social media pages.

With google+ it was kind of overlapping? Sorry, it's been a while...

I think it makes sense putting the links in the footer. That's where I had expected they would go and seems to be pretty common on other sites to do it like that. I think there could be less space between the icons but otherwise it looks good to me.
Also the Google+ page link is missing if it was intended to link to that too: https://plus.google.com/102301409253582998506

The Google+ page is no longer maintained, so it's probably best not to link to it...
And I object to the "everyone else does it" line of logic. Because well, it's incorrect! Here are some examples of sites that use secondary footer navigation:
reddit
the verge
arstechnica
polygon
microsoft.com (that one is really nice)
Notice that none of them actually put their social media links down there! Discovery is too poor for that. The Microsoft one is a good example of how I'd think Dolphin could use it - lots of misc links that don't need to have great discovery but are really nice to have, while the top navigation has primary navigation that has to be discovered and used by everyone.
Aaaanyway, if you look at those sites, you'll notice that they don't have social media links in either top or bottom navigation! They are always presented in the page somewhere, via "Social media widgets" (satan!) and things. Still, links in the top bar, a proper footer, or something in the sidebar would be better. Especially the sidebar, since we could put more dynamic content there.
@melonmanchan sorry to spill a little designer over your PR... But I do think what is implemented here has very poor discovery. :( If you still want to put the links in the footer like you have now without making a full on footer sub-navigation, there are some ways to increase discovery. For a good example... look down! github's footer is nicely composed with nice margins (albeit with minimal discovery still...). To increase discovery from that, add to the margins even more, and make it a light grey color ala microsoft.com, and it just might work!

I don't know if it's too late now but it would be nice to have these social media links ready for the 5.0 release, even if the implementation isn't 100% optimal. Currently it's a mess for users to tell which accounts are official and which ones are not.


Mujoco eq_active attribute not found

I was having an issue with the Kitchen environment training. I was receiving this error and the training was never starting
 co/index.py", line 628, in struct_indexer
    attr = getattr(struct, field_name)
AttributeError: 'MjModel' object has no attribute 'eq_active'

After some debugging I found the issue is with the dm_control package version. I solved it by updating to the version v1.0.16.
I could update the conda environment yaml, but not sure if everyone is receiving this error.

Solved by pip install mujoco==2.3.5


"Next page link" broken

When I click the right arrow to advance the table to the next page, it does not work.

Expected Behavior
Advances to the next page of the table.
Current Behavior
Fails with a JavaScript error:
Uncaught TypeError: _this.props.onChangePage is not a function
    at eval (webpack-internal:///../../node_modules/material-table/dist/components/m-table-pagination.js:67)
    at HTMLUnknownElement.callCallback (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:189)
    at Object.invokeGuardedCallbackDev (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:238)
    at invokeGuardedCallback (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:293)
    at invokeGuardedCallbackAndCatchFirstError (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:307)
    at executeDispatch (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:390)
    at executeDispatchesInOrder (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:415)
    at executeDispatchesAndRelease (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:3279)
    at executeDispatchesAndReleaseTopLevel (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:3288)
    at forEachAccumulated (webpack-internal:///../../node_modules/@hot-loader/react-dom/cjs/react-dom.development.js:3260)

Steps to Reproduce
We created the instance a few weeks ago, and updated to latest yesterday (7/22). Once we ingested enough components to trigger pagination this bug became apparent.
Your Environment

NodeJS Version (v12): 14.17.2
Operating System and Version (e.g. Ubuntu 14.04): Ubuntu 18.0.4
Browser Information: Chrome latest

Thanks for the report! This is a duplicate of #6452 and #6553, which are linked to the pull request #6581 that should fix this once merged. You can try this potential workaround in the meantime, and please report on that ticket if it worked for you.


🐛 [BUG] - Fix the alignment of the share button

🐛 Bug Report Form

I would like to fix this alignment.
I would like to work on it under gssoc23
🌐 Browsers
Which browsers are you experiencing the problem on? (Select all that apply)

[x] Chrome
[ ] Firefox
[ ] Safari
[x] Microsoft Edge
[ ] Opera
[ ] Brave

💻 OS
What is the impacted environment (operating system)? (Select all that apply)

[x] Windows
[ ] Linux
[ ] Mac
[ ] iOS
[ ] Android

📝 Sections Completed
Check the box of the sections you have completed:

[ ] Description
[ ] Reproduction URL
[ ] Reproduction Steps
[ ] Screenshots
[ ] Logs
[ ] Browsers
[ ] OS
[ ] Additional Context

🔍 Priority
Please select the priority of the bug:

[ ] High
[ ] Medium
[ ] Low

📜 Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct:

[x] I follow Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme.

Hi,
I would like to work on this issue under gssoc'23,
Can you assign it to me?

I'm a Gssoc'23 contributor
can you assign this issue to me
i was able to align the share button.


conda build not producing all python versions

This github action
https://github.com/fusion-energy/brep_part_finder/blob/main/.github/workflows/anaconda-publish.yml
runs a conda build command
conda build conda -c cadquery -c conda-forge --config-file conda/conda_build_config.yaml

Three versions of python are specified in the conda_build_config.yaml
https://github.com/fusion-energy/brep_part_finder/blob/main/conda/conda_build_config.yaml
However only one file gets uploaded to anaconda and this does not have a python version attached to the name
https://anaconda.org/fusion-energy/brep_part_finder/files
I am not quite sure why the conda build is not picking up all the python versions available for cadquery master
Any ideas @adam-urbanczyk

this command is much quicker to build the package but requires that mamba is installed
conda mambabuild conda -c cadquery -c conda-forge --config-file conda/conda_build_config.yaml

Changing the github action to use the super fast mamba boa builder fixed the issue :tada:


Update finitude.py

Updated line 319 from CarrierError to FinitudeError.

Daniel,  do I need to do anything to merge this into the code?  I ask because I have a few other additions and I'm not sure if this process is somehow waiting on me.

Sorry, no, the problem has been on my end. I will look at this today.
Thanks for sending.
On Sat, Jun 15, 2024 at 4:17 PM JJJ77TX @.***> wrote:

Daniel, do I need to do anything to merge this into the code? I ask
because I have a few other additions and I'm not sure if this process is
somehow waiting on me.
—
Reply to this email directly, view it on GitHub
https://github.com/dulitz/finitude/pull/8#issuecomment-2170967380, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVWYJJAWWNRHH35MCCFGWLZHTDRNAVCNFSM6AAAAABJMBZPXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZQHE3DOMZYGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Added new Action to submit a new search on the toolbar.

This is related to the issue #22 . Also solved issue with the position of the search related to the issue #26

👍 👍

:+1:


add function to clean accumulated pandas storage data

Addresses #169

I think that we can work on this and add others features
Thank you for your code and suggestions

Tip: next time push to dev so that we can test new features and stuff without breaking the code (as usually happen). This is not a complex upgrade so no worries, just saying for future occasions


Fixes for the parser and handling of .dk domains

Ensure .dk domains get passed to the WhoisDk parser and host. Add the registrar regex, so that registrars are also picked up for .dk domains.

Ping @richardpenman have you had time to look at this, please?


Match to bool

Is your feature request related to a problem? Please describe.
It would be nice if there was a way store whether there was a match into a boolean.
Describe the solution you'd like
I was thinking there could be an additional struct tag to configure this but I am not sure what it should look like. Booleans are currently supported but only to match true or false.
type B struct {
	HasSuffixFoo bool `regroup:"foo,flag"`
}

var re = regroup.MustCompile(`.*(?P<foo>foo)?`)

Describe alternatives you've considered
Currently I just check the value of the struct after matching.

Hi @eberkund!
Thank you for the suggestion :)
In case no match was found, an error will be returned. The type of the error will be NoMatchFoundError
So, you can verify a match that way.
Is that satisfy your requirement?

Let me try to do a better job at explaining. The use case here is for when you have an optional part of the pattern.
Imagine some inputs like this:
<EMAIL_ADDRESS>username=alice,email=alice@example.com,admin

and a corresponding Go struct like this:
type User struct {
    Username string
    Email string
    IsAdmin bool
}

My understanding is that there isn't really a nice way to handle this with regroup so I suggested having a "flag" option which would be true if that optional match group is present and false if it is not.
This is different from an input string that looks some like this (which is supported by the current features):
username=bob,email=bob@example.com,admin=false
username=alice,email=alice@example.com,admin=true

Ah I understand. You want to mark a field as a flag, so that the existence of the group alone will mark it as true.
That sound like a great addition. Would you like the contribute this feature?

Ah I understand. You want to mark a field as a flag, so that the existence of the group alone will mark it as true. That sound like a great addition. Would you like the contribute this feature?

Exactly, yeah I would not mind contributing. I will take a look when I have some time.


chore(deps): update common-dev-assets digest to 2433141

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
0ee568c -> 2433141

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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


Fatal Error LNK1120

Hi everyone,
I was just going through the class and was pretty happy to be able to follow it all because I'm super new to programming and I was actually understanding. Then came the class 32 "Constructors  for Initialization" and now I'm not able to build the game anymore. It ways gets this error:
1>------ Build started: Project: Section_03, Configuration: Debug x64 ------
1>Section03.cpp
1>Section03.obj : error LNK2019: unresolved external symbol "public: __cdecl FBullCowGame::FBullCowGame(void)" (??0FBullCowGame@@QEAA@XZ) referenced in function "void __cdecl `dynamic initializer for 'BCGame''(void)" (??__EBCGame@@YAXXZ)
1>Section03.obj : error LNK2019: unresolved external symbol "public: int __cdecl FBullCowGame::GetMaxTries(void)const " (?GetMaxTries@FBullCowGame@@QEBAHXZ) referenced in function "void __cdecl PlayGame(void)" (?PlayGame@@YAXXZ)
1>Section03.obj : error LNK2019: unresolved external symbol "public: int __cdecl FBullCowGame::GetCurrentTry(void)const " (?GetCurrentTry@FBullCowGame@@QEBAHXZ) referenced in function "class std::basic_string<char,struct std::char_traits,class std::allocator > __cdecl GetGuess(void)" (?GetGuess@@YA?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ)
1>Section03.obj : error LNK2019: unresolved external symbol "public: void __cdecl FBullCowGame::Reset(void)" (?Reset@FBullCowGame@@QEAAXXZ) referenced in function "void __cdecl PlayGame(void)" (?PlayGame@@YAXXZ)
1>C:\Users\Gustavo\source\repos\BCGame\x64\Debug\Section_03.exe : fatal error LNK1120: 4 unresolved externals
1>Done building project "Section_03.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
I've tried with both x86 and x64 and both brings this error. Not sure what to do :(
Thank you in advance!
Gustavo

Hi I'm having the same problem I believe it has to do with the new visual studio. I've gone over the work numerous times and even copy and pasted the code from the site and still get the same error. Please update so students can follow or at least add comments that can help us get on track.

Just to let you all now I have seen this and am working on a fix.

@abrasounds, could you share your code with us?

@sampattuzzi at this point I just copy/pasted the code from here to see if that would work and it didn't, still got the error. So I guess now I'll just wait for @BenTristem to fix it as apparently what causing this is the latest version of Visual Studio.

@abrasounds I'm not sure which stage of the project you are at. It doesn't look like a VS version issue. Perhaps you can give us more details.

I was able to (partially) reproduce the same error you bring us by removing FBullCowGame.cpp from the solution explorer. You may have not deleted the file from your computer but make sure that is visible from your explorer solution. Try right click + add existing file and lookup for the above mentioned file. Make sure you have everything in the right place and then try to build again the entire solution. I strongly suggest to download the entire section from here rather than copy pasting it. I have also noticed this:

1>Done building project "Section_03.vcxproj" -- FAILED.

Bear in mind this is Section_02. Just in case its helpful.
By the way, this works like a charm in VS2017 x64bit.

Is this a problem you feel other student will be held back by?
On 19 October 2017 at 16:24, Arregi<EMAIL_ADDRESS>wrote:

I was able to (partially) reproduce the same error you bring us by
removing FBullCowGame.cpp from the solution explorer. You may have not
deleted the file from your computer but make sure that is visible from your
explorer solution. Try right click + add existing file and lookup for the
above mentioned file. Make sure you have everything in the right place and
then try to build again the entire solution. I strongly suggest to download
the entire section from here rather than copy pasting it. I have also
noticed this:
1>Done building project "Section_03.vcxproj" -- FAILED.
Bear in mind this is Section_02. Just in case its helpful.
By the way, this works like a charm in VS2017 x64bit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/UnrealCourse/02_BullCowGame/issues/9#issuecomment-337943439,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB4LQHR69JLiszrRJrCGgEPmAaAtjzb7ks5st2nDgaJpZM4O4Wxo
.

I don't think so. I guess that depends on each student.


[MOD-3251] Stop heartbeats before sending the container snapshot RPC

Heartbeats make a connection to the gRPC proxy (modal.sock) which causes the container to open a FD. This open FD causes gVisor to crash during runsc checkpoint. I ran the repro over 300 times with 0 flakes.

Update: I have landed on a solution I am happy with, with almost no overhead (@thundergolfer and @luiscape I removed the bottleneck I spoke of). Here is a summary:

I introduce a asyncio.Condition variable that guards a boolean called waiting_for_memory_snapshot (same as the old name).
Before a ContainerCheckpoint or ContainerHeartbeat RPC, the Condition must be acquired, and the heartbeat will only run if the container is NOT waiting_for_memory_snapshot. This prevents heartbeat requests from executing while the checkpoint-restore process in in progress.
If the function is a snapshotting function (function_def.is_checkpointing_fn), then heartbeats will only turn on after the entire checkpoint-restore process
If it's not a snapshotting function, then heartbeats are enabled immediately, and will run during the container init phase.
Remove the old ContainerIOManager::_waiting_for_memory_snapshot logic, as it was broken and did not prevent heartbeats from happening when they shouldn't

The following invariants hold:

If snapshotting fn, then heartbeats run after the snap-restore phase
If snapshotting fn, then heartbeats do not run before the snap-restore phase
If not snapshotting fn, then heartbeats are always running


nested CTEs vs. multipe CTEs defined within a nonrecursive CTE

In Azure Synapse, how does one define a CTE that references another CTE without using WITH twice?
Related question: what exactly constitutes a CTE query definition? Is it different than a single CTE?
cc: @alittlesliceoftom @nandanhegde15 @julieMSFT
In the Features and Limitations of Common Table Expressions in Azure Synapse Analytics section of  docs/t-sql/queries/with-common-table-expression-transact-sql.md there are two bullet points that confuse me:

Multiple CTE query definitions can be defined in a CTE.
Specifying more than one WITH clause in a CTE is not allowed. For example, if a CTE query definition contains a subquery, that subquery cannot contain a nested WITH clause that defines another CTE.

The Guidelines for Creating and Using Common Table Expressions section has the same information:

Multiple CTE query definitions can be defined in a nonrecursive CTE ...
Specifying more than one WITH clause in a CTE is not allowed ...

did some playing around, and I think have an example of something worth being added to a the page linked above. Happy to make the PR if you agree. Basically it wasn't immediately clear from reading the docs whether it was possible to have CTEs reference one another inside a single WITH statement. Now I've figured it out...
this works
WITH att_a AS (
    SELECT name, id
    FROM [ajs_stg].[attributes]
    WHERE name = 'Campaign'
),
att_b AS (
    SELECT name, id
    FROM att_a
    WHERE id = 122529495
)
SELECT * FROM att_b;

but this doesn't work
WITH att_b AS (
    WITH att_a AS (
        SELECT name, id
        FROM [ajs_stg].[attributes]
        WHERE name = 'Campaign'
    )
    SELECT name, id
    FROM att_a
    WHERE id = 122529495
)
SELECT * FROM att_b;

@swanderz -- Anders, thank you for your feedback and research.
@VanMSFT -- Van, please review the above suggestion and if possible give Anders the go-ahead for a new PR.

Hi @swanderz! Thanks for the feedback and testing. Please go ahead and submit the PR, and we will validate it. We appreciate your help with our documentation! I'll close this GitHub Issue for now, since you found the answer. We look forward to the PR! Thanks again!


Add documentation for properties/methods inherited from the base shape classes

Searching for the term "isoperimetric quotient" does not show up with results for any of the classes where the property is inherited from the base class Shape3D. Only shapes with custom overrides show up (e.g. spheres always return exactly 1 and have a different docstring to indicate that). We need to add Sphinx documentation for properties/methods inherited from the base shape classes.
https://coxeter.readthedocs.io/en/latest/search.html?q=isoperimetric+quotient
https://github.com/glotzerlab/coxeter/blob/2414555de3f34b9fc46e81864852fef3fc21add5/coxeter/shapes/base_classes.py#L394

This seems to be a problem with RTD search, not with the documentation itself. If I actually search for the text using my browser on the API documentation page, I find the property documented under all of the shapes that support it, even the ones where it is inherited.

@vyasr You’re right. I missed this earlier, I must have made a mistake when I did the same in-page search. I suppose this can be closed, unless we want to figure out what is wrong with the search…

I don't know how much control we have over that, there's a decent chance we'll end up having to correspond with RTD folks to find the problem and that seems like more work than it's worth at this stage. I'm going to close this, we can reopen if someone runs into this again.


waypoint_replanner configuration parameter cleanup

Removed the following configuration parameters for waypoint_replanner:

float32 radius_thresh
int32 velocity_offset
int32 end_point_offset
int32 braking_distance
bool overwrite_vmax_mode

And one new configuration parameter
float32 lateral_accel_limit

If replan_curve_mode is true, limit the speed at each waypoint by the lateral_accel_limit and its curvature.
Remove some unused or undefined functions.

Verified in the vehicle at different max velocities (20 Kmh, 30 Kmh, and 40 Kmh).
Will be followed up by a PR in messages and utilities repo.
See original PR for more details: https://gitlab.com/astuff/autoware.ai/core_planning/-/merge_requests/51

Didn't meen to close this.

@JWhitleyWork, this PR needs to be merged next, since it deals with the new changes to the config message.
Apologies for not linking the PRs originally! I've updated it now.


Update error message and add the ERR error code

We now have some error messages that do not return
an ERR error code:
<IP_ADDRESS>:6666> cluster nodes
Cluster mode is not enabled

As far as i know this doesn't affect the client libs,
and doesn't affect the users. But I think it's right
to add the error code.
Note that Redis will actually count error messages and
standardize the error codes, like:
<IP_ADDRESS>:6379> info Errorstats
 # Errorstats
errorstat_ERR:count=2
errorstat_EXECABORT:count=1
errorstat_WRONGTYPE:count=1

We mixed using the status message and error output in
some places, this PR checks all redis::Error calls and
use error status to replace it. (Only the ERR case is
handled, others are not handled yet.)

It's not a direct objective of this PR, but I'd like to have the "ERR" + some_error_message construction only in one place as an implementation detail of the Redis protocol. Maybe, inside redis::Error or so.

It's not a direct objective of this PR, but I'd like to have the "ERR" + some_error_message construction only in one place as an implementation detail of the Redis protocol. Maybe, inside redis::Error or so.

@torwig thanks for the review. agree, that is a good suggestion.
we still have some places that mixed using the status message and error output. (e.g. NOSCRIPT)
i think we can leave it for a future PR as a cleanup. (pull out the error code and error message)


actions block multiple elements values as one state

Description
I have an actions block in a message with multiple elements, say a datepicker and a select dropdown. When an action is triggered by user changing any of the values, I need to get values for the whole group of elements instead of just the element that was actioned.
I cannot believe this isn't in the documentations. I've looked everywhere! And if it isn't possible then that is even a bigger issue. A block should have values of all elements when triggered. Otherwise how am I supposed to link the elements together when they're submitted?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[x] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Steps to reproduce:

Send an interactive message with an actions block with multiple elements
Interact with any of the elements in the message
Listen for action in Slack app

Expected result:
As this is a block with multiple elements, I should get values for all elements in the block
Actual result:
Receiving only the value for the element that triggered the interaction

Never mind actually, I found that values are grouped together stored in body.state of the payload coming back in app.action listener. How this isn't clearly outlined in the docs I don't understand 🤷

Hi @jzaynal - I'm glad you were able to find the solution to your issue! For any who might be wondering where in the documentation to find more information about the shape of interactive payloads like actions, this page is a good resource.
I've also added some additional links to the bolt js docs section on actions based on your feedback.


Converting the complete environment from Bash to Python because of portability

I wanted to ask if there are any plans to remove the bash completely from the project? In the presentation for the release of the project as open source, there was still talk of the need to switch to Python to make it more portable than before.

Unfortunately, I don't see any plans on the roadmap for this anymore. In my opinion, however, it would be quite desirable if Python could be used to depict the whole thing. This way you wouldn't have to do such a headstand under Windows to get the system up and running. The best way to do this is to simply run the program without external dependencies. That makes it entirely portable.

I added it to the long term plan. We would like to do it, but there are quite a few more important goals.

Thank you very much! Of course there are more important topics than this one, but I was just wondering why it was missing on the list. I'm closing the issue now! Thanks for the quick response!

I am re-opening this as we surely want to do it, although not right now.

I did some work towards converting all scripts to Python, which can be seen in my Branch.
It's still early work in progress. Apart from  retdec-decompiler and retdec-signature-from-library-creator every script is converted. Converted meaning they compile and look reasonable like a python script. However some scripts still emplace some non portable bash commands, which should be replaced by their python equivalent.
This could be used as a starting point, so any feedback or help would be greatly appreciated.

Done - everything is in master. We will do a new release with this in the next few days.

That's great news! Thank you for your efforts.


Running natural_earth.sh results in error "tolerance condition error"

When I run natural_earth.sh on master on my Arm M1 macbook pro I get the error described in the title.
For instance, if I comment out all of the ne source urls in the script except for the last one and run the script I get this:
./natural_earth.sh
NOTICE:  extension "postgis" already exists, skipping
CREATE EXTENSION
fetching https://naciscdn.org/naturalearth/10m/physical/ne_10m_ocean.zip
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 3114k  100 3114k    0     0  1936k      0  0:00:01  0:00:01 --:--:-- 1941k
Archive:  0.zip
  inflating: 0/ne_10m_ocean.README.html
 extracting: 0/ne_10m_ocean.VERSION.txt
 extracting: 0/ne_10m_ocean.cpg
  inflating: 0/ne_10m_ocean.dbf
  inflating: 0/ne_10m_ocean.prj
  inflating: 0/ne_10m_ocean.shp
  inflating: 0/ne_10m_ocean.shx
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: tolerance condition error
ERROR 1: Reprojection failed, err = -20, further errors will be suppressed on the transform object.

This is the GDALversion I am using:
ogr2ogr --version
GDAL 3.4.1, released 2021/12/27

I have also been able to reproduce this on an intel s3 instance so I do not think this is ARM related.
I posted a question on the GDAL mailing list re: the above and I received this answer:

This dataset includes latitude up to 90 degree north, which projects to infinity in EPSG:3857. You should use the -clipsrc option of ogr2ogr to for example cut at 85 degree

I'm not exactly sure how to use this option correctly. Looking at some examples online I found this
-clipsrc -180.1 -85.0511 180.1 85.0511 and tried it like so:
OGR_ENABLE_PARTIAL_REPROJECTION=true ogr2ogr -unsetFieldWidth -t_srs EPSG:3857 -clipsrc -180.1 -85.0511 180.1 85.0511  -nlt PROMOTE_TO_MULTI -f PostgreSQL PG:"dbname='ne' host='localhost' port='5432' user='myuser' password=''redacted" ne_10m_ocean.shp
The command no longer fails and I find 3 rows in the created table but just because there were no errors doesn't mean its the correct fix for this problem.
Could you confirm this is indeed a bug and the above is the correct fix?

I tried updating natural_earth.sh to
OGR_ENABLE_PARTIAL_REPROJECTION=true ogr2ogr -unsetFieldWidth -t_srs EPSG:3857 -clipsrc -180.1 -85.0511 180.1 85.0511 -nlt PROMOTE_TO_MULTI -f PostgreSQL PG:"dbname='$DB_NAME' host='$DB_HOST' port='$DB_PORT' user='$DB_USER' password='$DB_PW'"

i.e. adding -clipsrc -180.1 -85.0511 180.1 85.0511.
Most of the above errors disappear but I still get one or two.
Is no one else getting the above errors? I'm trying to work out if its an environment issue or I wonder if its anything to do with the datas' source urls being updated recently. Perhaps the raw natural earth data has changed?
(cc @ARolek )

Sorry I forgot to add in my last comment the error I am still seeing. Here it is
Archive:  61.zip
  inflating: 61/ne_10m_lakes_north_america.README.html
 extracting: 61/ne_10m_lakes_north_america.VERSION.txt
 extracting: 61/ne_10m_lakes_north_america.cpg
  inflating: 61/ne_10m_lakes_north_america.dbf
  inflating: 61/ne_10m_lakes_north_america.prj
  inflating: 61/ne_10m_lakes_north_america.shp
  inflating: 61/ne_10m_lakes_north_america.shx
Input geom 0 is INVALID: Self-intersection at or near point -93.810065734214476 50.475055438868417 (-93.810065734214475697 50.475055438868416502)
<A>
POLYGON ((-93.8179440637089925 50.4308355812790978, -93.8499163576565536 50.4208094165111191, -93.8967956691880374 50.4166883870498594, -93.9280410214826560 50.4276889160359048, -93.8921053046715883 50.4453518722649932, -93.8087717138336359 50.4519046170063419, -93.8132948526465356 50.4662333068101603, -93.8047481692926794 50.4895833234770492, -93.7875066521550735 50.5108989376523567, -93.7705259140755487 50.5210039017706691, -93.7308350412120319 50.5192439160760927, -93.7095650821384538 50.5116950072548505, -93.7322536045641499 50.5097336130517576, -93.7615301006236734 50.5113228920604271, -93.7726259035747489 50.4974251259773865, -93.7905495188592937 50.4839842391629503, -93.8102983765965064 50.4761144003827482, -93.8072212463625306 50.4621076555067987, -93.7644947641007604 50.4549225835861961, -93.7094546981023768 50.4529433803538438, -93.6758725440474223 50.4453499740687974, -93.6125031029548325 50.4601394845921334, -93.5948419728205891 50.4471434799242147, -93.6092826873922945 50.4406438041435905, -93.6440307038893565 50.4364653674272674, -93.6894898636331135 50.4364525552598053, -93.7311121568103971 50.4436828817032392, -93.7754336800916519 50.4418257547950617, -93.8179440637089925 50.4308355812790978))
</A>
ERROR 1: TopologyException: Input geom 0 is invalid: Self-intersection at -93.810065734214476 50.475055438868417

@adamakhtar hey there! apologies for the slow response to these comments.

Could you confirm this is indeed a bug and the above is the correct fix?

I need to give this a test myself. I'm not sure if this is an appropriate fix.

Is no one else getting the above errors? I'm trying to work out if its an environment issue or I wonder if its anything to do with the datas' source urls being updated recently. Perhaps the raw natural earth data has changed?

There was a PR not too long ago submitted which changed the data. I ran the import scripts after the change to confirm things were working prior to accepting the PR. The data might have changed.

ERROR 1: TopologyException: Input geom 0 is invalid: Self-intersection at -93.810065734214476 50.475055438868417

This error is not ideal, but does happen sometimes. I believe the data is still imported though and can potentially be fixed by running ST_MakeValid(geom) on the geometry. This error is different than the tolerance errors you're receiving.

Thanks @ARolek for the help. I'll look into ST_MakeValid(geom).


Update monitor-workspace.md

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

Hi @shemers -- This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.


stub function instead of object

possible to stub a function directly instead of passing in an instance of the function where it belongs to?
reason being is because i'm moving towards functional programming and would like to do away with classes for implementation

The way the stub function works is that it takes the instance and replaces the property with a spy on the function that was previously there on the instance. For regular functions, the only way to replace a function would be to assign a new variable in its place. You can already do that with the spy function. The README.md file has an example that can be found here. In the second example, isEven is being spied on. If you wanted to fake isEven, you could call spy with the fake function you want to serve as the isEven function for the test.


AEG blocked token URL

When I directly go to the URL I get the error:
curl "https://electrolux-wellbeing-client.vercel.app/api/mu52m5PR9X"
Payment required
DEPLOYMENT_DISABLED
I see the IOS app now uses two different URL's
https://ws.eu.ocp.electrolux.one
https://api.eu.ocp.electrolux.one

Are you able to make a patch for this new url?

Hi, I've tried to use this plugin, however I faced the same issue as @AlanSmithe . I don't want to advertise in any way, but I've  created a new plugin: https://github.com/tomekkleszcz/homebridge-electrolux-devices to connect my Comfort 600 AC, and Well A7 air purifier to Homekit. It uses these 2 URL's you've mentioned, so if you would like to try out my plugin I'll be more than happy.
@davidjbradshaw If you are still interested in maintaining this library, I think we could work together to create a patch and add support other devices based on the code I wrote.

wow nice, that was quick. Did Electrolux release api documentation or did you reverse engineer the IOS/Android app?

wow nice, that was quick. Did Electrolux release api documentation or did you reverse engineer the IOS/Android app?

I've reverse engineered it.

@tomekkleszcz my health isn’t great, so I can do much right now. I’m happy to either give you access to everything or hand it over to you.

Alternatively if you have a whole new plugin for the new API, maybe the best solution is just to update this to point to your project? Add something to the HomePage and and the log output for this project pointing users to your stuff.
If you want to do that, I’m happy to take a PR and publish it.


Using route key other than 'id' not working

Description
I have an existing model that uses the Spatie/Sluggable package, and uses the getRouteKeyName() method to set a custom route key: 'slug'. The listing page of Runway uses the 'id' field to generate the edit link (eg. http://localhost/cp/runway/product/1), but the Resource Controller edit method uses the routeKey() method in retrieving the model record. So it is trying to find a record whose slug is '1'. This results in an "Attempt to read property "name" on null" error, since the record cannot be retrieved.
Steps to reproduce

Add getRouteKeyName() method to your model returning a key name other than 'id' (eg. 'slug')
Navigate to a Runway index page and try to edit a record

Environment
Statamic 3.1.27 Solo
Laravel 8.42.1
PHP 8.0.7
doublethreedigital/runway 2.1.1

Ah whoops, I'll fix this today. Sorry!

Thanks very much. Didn't know where to put this given the bug report template structure, but I only just discovered this addon yesterday, and I'm really liking it. Thanks for the work!

No worries! 🙂

Closed by f7c52cb & will be fixed in the next release.


Optimize Hand & Optimize Joker correlation with hand size?

Description of the bug
At a certain hand size -- I think it's around 10 or 11 cards -- both of these checkboxes cannot both be on. Is this intentional?

Yes. Because at that point, the calculation would take an unreasonably long period of time.


rpc.buttonIncactiveLabel for private repos

Make it so the discord button in activity detects if a GitHub repo is private and then make it display different text / the buttonInactiveLabel & buttonInactiveUrl

Some reference (for my future self):
https://docs.github.com/en/rest/repos/repos#get-a-repository

we have a problem, and that is we have multiple git provider, gitlab, github and some scary third party git provider, if we were to add support for this, it gonna take a lot of effort and code


Bug in FetchDataTask [1.7.6]

I am using BaseDownloadTask and manage queue on my own. Every one of these items is correctly configured NOT TO use multi-connection (because of encryption with FileBufferedOutputStream), even though i can't download them because of below condition:
            if (hostRunnable != null && !isSupportSeek) {
                throw new IllegalAccessException(
                        "can't using multi-download when the output stream can't support seek");
            }

Why using hostRunnable crashes out entire downloading (no multi-connection)? Can this be fixed soon?

So you custom OutputStreamCreator and you don't support seek? if so, you need custom a ConnectionCountAdapter and force connection count to 1.

Shouldn't it be logic implemented in library itself?

Oh, sorry with my misoperation. I'm asking you the way you use FileDownloader.

So, can you show me how you custom the OutputStreamCreator ?

You said:

Every one of these items is correctly configured NOT TO use multi-connection
How do you configure items don't use multi-connection?

I'm using my OutputStreamCreator to encrypt on the fly but aren't you supposed to disable multi connection for all downloads with isSeekable = false?

Actually, Filedownloader will use single connection if the output stream doesn't support seek:
    private boolean isMultiConnectionAvailable() {
        //noinspection SimplifiableIfStatement
        if (isResumeAvailableOnDB && model.getConnectionCount() <= 1) {
            return false;
        }

        return acceptPartial && supportSeek && !isChunked;
    }

So, I need the whole FileDownloader log to check what's the problem.

//            if (hostRunnable != null && !isSupportSeek) {
//                throw new IllegalAccessException(
//                        "can't using multi-download when the output stream can't support seek");
//            }

This code breaks it.


example-cloud pom.xml

The project org.camunda.bpm.extension.springboot.example:camunda-bpm-spring-boot-starter-example-cloud:2.2.0-SNAPSHOT (C:\Users...\Desktop\example-cloud\pom.xml) has 1 error
Non-resolvable parent POM: Could not find artifact org.camunda.bpm.extension.springboot.project:camunda-bpm-spring-boot-starter-example-root:pom:2.2.0-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 4, column 11 -> [Help 2]
i can't build with dependency.

just tried with a fresh git clone ... works. And since also Jenkins and travis are green, this must be a local problem of yours. Try to delete everything you have under .m2/repo/org/camunda/bpm/extension/springboot and run man clean install again.


Implement pulldown and pullup on input

It looked like this was prepared already, I needed to enable a pull up on my gpio so I implemented it.
Pull up is verified on nrf52dk button1 - 4
The pull-down is still untested it is however symmetrical.
Should the function names be different?
Please enjoy.

Hey @simonsso, thanks for the contribution!
We are about to wrap up the v0.6.0 release, once that is wrapped I'll try to get this merged for you!

Thanks, Is there a better base line I could base this and futher patches on to minimise extra merge conflicts on?

Rebased


DP-460 Upgrade to PHP 8.1 / Laravel 9

@daniilly let's rid of merge commit. It's also a bit strange, that we see this set of 6 old commits there.