a:hover style does not seem to be applied to Link or NavLink

Version
"react-router-dom": "^4.2.2",
"react-router-redux": "^5.0.0-alpha.8",

Test Case
n/a
Steps to reproduce

Try this code:

            <Link to="/records/create" className="subnav_link">Add a New Record</Link>

          <Spacer />

             e.preventDefault()}}>
              Show Inactive

With this CSS:

#subnav {
  display: flex;
  flex-direction: row;
  justify-content: center;
  align-items: center;
  flex: 0;
  width: 100%;
  padding-top: 8.9rem;
  padding-right: 0.4rem;
  padding-left: 0.4rem;
  box-sizing: border-box;
  user-select: none;
  font-size: 2.3rem;
  line-height: 2.3rem;
  font-weight: 600;
  color: red; }

a.subnav_link:link {
  text-decoration: none; }

a.subnav_link:hover {
  cursor: pointer;
  color: gray; }

a.subnav_link:visited {
  color: red; }

a.subnav_link:active {
  color: red; }

.subnav_menu-spacer {
  margin-left: 0.4rem;
  margin-right: 0.4rem;
  color: gray; }

Expected Behavior
The two links should have the same CSS applied to them.
Actual Behavior
Link hover state doesn't seem to work when it has the exact same DOM appearance as a normal  tag.
Here is a GIF:

Here is the DOM. They look the same, so I don't know how this is possible:

I tried using Link and NavLink and also activeStyle but it is the same behaviour.

You have an extra squigly bracket on your onClick prop.
But this code works fine if you fix that: https://codesandbox.io/s/pqjwwz20q Something is going wrong on your end. It's not the library.

Hmm ok thanks. That squiggler was a rogue after I deleted the extra poop in there. SASS must be giving me some sass.

@timdorr I changed the style name to a unique name and it still doesn't work.
I think it has to do with this:
https://stackoverflow.com/questions/34381701/react-router-links-dont-get-visited-style
Radium also has all kinds of issues with it it looks like.
That sandbox probably works because it isn't really visiting the link in a real browser.


Update dashboard link

Related to: https://github.com/canonical/kubeflow-dashboard-operator/issues/190
Some of the links in the configmap changed upstream. This PR resolves the changes related to this charm.

Checked the rc.2 changes, no changes affect tensorboards.
This PR is ready for review.


Retry validateJobs for a while to avoid flaky errors

This PR tries to fix #8776.
What the test does is to modify the state in helix and then verify that the result is correct. IICU Helix, there is no read-after-write guarantee in Helix, so there is a race condition between the update and the read. In normal cases, Helix will change the state faster and therefore the test doesn't fail. But that is not guaranteed and in some executions it may happen that the change is not applied when the validate is done.
Sequence diagram when everything works as expected:

sequenceDiagram
    Test->>+Controller: update table config (0 */20 * ? * * *)
    Controller->>+Helix: update table config (0 */20 * ? * * *)
    Helix->>Helix: Change ideal state to (0 */20 * ? * * *)
    Helix->>-Controller: Ok
    Controller->>-Test: Ok

    Test->>+Controller: get job info
    Controller->>+Helix: get job info
    Helix->>-Controller: updated job info (0 */20 * ? * * *)
    Controller->>-Test: updated job info (0 */20 * ? * * *)

Sequence diagram when the change is not seen:

sequenceDiagram
    Test->>+Controller: update table config (0 */20 * ? * * *)
    Controller->>+Helix: update table config (0 */20 * ? * * *)
    Helix->>Controller: Ok
    Controller->>-Test: Ok

    Test->>+Controller: get job info
    Controller->>+Helix: get job info
    Helix->>-Controller: updated job info (0 */10 * ? * * *)

    Helix->>-Helix: Change ideal state (0 */20 * ? * * *)

    Controller->>-Test: updated job info (0 */10 * ? * * *)

The only solution know is to retry the validation with some timeout.

Codecov Report

Merging #8999 (187c6a5) into master (d2031e9) will decrease coverage by 53.50%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #8999       +/-   ##
=============================================
- Coverage     68.50%   15.00%   -53.51%
+ Complexity     4894      170     -4724
=============================================
  Files          1820     1773       -47
  Lines         95396    93334     -2062
  Branches      14276    14044      -232
=============================================
- Hits          65353    14002    -51351
- Misses        25458    78321    +52863
+ Partials       4585     1011     -3574

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
15.00% <ø> (+<0.01%)
:arrow_up:

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

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


ignored aborted_resync test after discussion about its relevance

testing locally

actually, no need to test locally because this strictly decreases the number of failing tests


--convert can import local module

I have an extensive xml file. It contains an xsd definition from which I generated an xml parser and class definitions via generateDS.
Now I want to use it in the git-history import.
I can also install the module manually via a setup.py, but I wanted to ask if you think adding a way to import local python modules is a good idea.
Nevertheless great tool! I have already some repos up and running :)

This came up in another of my projects just a few weeks ago:

https://github.com/simonw/sqlite-utils/issues/353

https://github.com/simonw/sqlite-utils/issues/353#issuecomment-991376639 has a workaround that should work with git-history too:
PYTHONPATH=. git-history ... --import transform

So if your module is called transform.py in the current directory the above should cause it to be imported.

Implementing the same solution as https://github.com/simonw/sqlite-utils/issues/353 feels like a good idea to me.


make tests for next review date more explicit

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.62%. Comparing base (24266bb) to head (1bda598).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #229   +/-   ##
=========================================
  Coverage     91.62%   91.62%
  Complexity      736      736
=========================================
  Files           132      132
  Lines          2196     2196
  Branches        163      163
=========================================
  Hits           2012     2012
  Misses          120      120
  Partials         64       64

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


set MQTT QoS in mosquitto

User story
As a wis2box operator, I would like to set the MQTT QoS capability so as to guarantee message delivery attempts (background info).
Acceptance criteria

MQTT endpoint is provisioned with QoS=1

Definition of done

[ ] unit/functional tests added/updated
[ ] documentation updated
[ ] live demonstration provided

It's always 1... selected as standard... you have no possibility to pick any other QoS.

not quite true... it can be overridden if desire... but one should not change that setting. It is always 1.


Sort custom-templated cell

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I have 2 props on a row, when prop1 is set to a specific value, I want to display prop2 instead of it.
I set up a custom <ng-template /> to display the appropriate value with success (by accessing let-row="row", I can access prop2).
When sorting said column, it automatically sorts by prop1, completely ignoring my custom markup.
Expected behavior
Some way of describing custom sort logic that can access the rest of the row's object (unlike comparator which can only access the column's default prop's value and nothing more).
(I wouldn't want ngx-datatable to try and automatically sort the rows automatically according to my new template's text, I want to specify this explicitly, but I can't seem to find a way)
Reproduction of the problem
<ng-template #cellTpl let-row="row">
    {{row.prop1 == 'specificValue'? row.prop2 : row.prop1 }}
</ng-template>

this.columns = [
    { prop: 'prop1' }
];

this.rows = [
    { prop1: 'A', prop2: 'B' },
    { prop1: 'specificValue', prop2: 'A' }
];

What is the motivation / use case for changing the behavior?
In almost every datatable I instantiate I have some sort of custom cell that I display differently, firstName + lastName for example, I'd like to be able to render them concatenated without changing my dataset.

Table version: 9.3.0

Angular version: 4.1.3

Browser: all

Language: TypeScript 2.3.4

This would be really useful for me too

For anyone who finds this issue, it's been resolved by #1074 . Should be in v10.4.0. See https://github.com/swimlane/ngx-datatable/blob/master/docs/api/column/inputs.md#comparator for info.


"Măgura", "Viroaga" are listed in the drop-down menu

Description:
When user writes "Măgura", "Viroaga" the location appears in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Măgura"

Expected results
The location appears in the drop-down menu as "Sat Măgura, CONSTANȚA, (Cerchezu)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Viroaga"

Expected results
The location appears in the drop-down menu as "Sat Viroaga, CONSTANȚA, (Cerchezu)"


Do not push artifact that have been downloaded from the cache

See original issue on GitLab
In GitLab by [Gitlab user @jjardon] on Feb 6, 2018, 01:17
From example, in this build: https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/51193892 :
[--:--:--][af6f6965][ pull:dependencies/base-sdk-image.bst] INFO    Downloaded artifact af6f6965
[00:01:05][af6f6965][ pull:dependencies/base-sdk-image.bst] SUCCESS base-sdk-bootstrap/dependencies-base-sdk-image/af6f6965-pull.147.log
[--:--:--][af6f6965][ push:dependencies/base-sdk-image.bst] START   base-sdk-bootstrap/dependencies-base-sdk-image/af6f6965-push.373.log
[--:--:--][af6f6965][ push:dependencies/base-sdk-image.bst] START   Pushing Artifact
[--:--:--][af6f6965][ push:dependencies/base-sdk-image.bst] START   Preparing compressed archive
[00:00:39][af6f6965][ push:dependencies/base-sdk-image.bst] SUCCESS Preparing compressed archive
[--:--:--][af6f6965][ push:dependencies/base-sdk-image.bst] START   Sending artifact
[00:00:54][af6f6965][ push:dependencies/base-sdk-image.bst] SUCCESS Sending artifact
[00:01:37][af6f6965][ push:dependencies/base-sdk-image.bst] SUCCESS Pushing Artifact

In GitLab by [Gitlab user @samthursfield] on Feb 6, 2018, 13:43
I think this is a regression. The CI log in question is using commit 315c4ef768d3c21ea7484a35052c17fb2e7ec1c4 which is very recent. We are obviously missing a unittest here.

In GitLab by [Gitlab user @juergbi] on Feb 7, 2018, 10:04
Yes, I assume this is happening since the corresponding check in Element._skip_push() was removed in 51c17e114. With multiple configured remotes, this is a bit more complicated. Although, we could simply skip pushing if at least one remote already has the artifact.
[Gitlab user @samthursfield] Do you see a reason why we can't just re-add that check?

In GitLab by [Gitlab user @samthursfield] on Feb 7, 2018, 10:50
The user now chooses which remotes to push to using the push flag. I think we should push to any remote that is marked push and doesn't already have the artifact. So we should check that in the _skip_push() check so that if there's no need to push anywhere we don't create a 'push' job at all, but we should also ensure that if we have 2 remotes marked 'push' and one lacks the artifact, we push to that 1 remote.

In GitLab by [Gitlab user @LaurenceUrhegyi] on Feb 7, 2018, 16:20
assigned to [Gitlab user @samthursfield]

In GitLab by [Gitlab user @samthursfield] on Feb 14, 2018, 18:23
I think there are two options for fixing this.

modify the OSTreeCache._remote_refs dict so that it stores every remote that has a ref, not just the highest priority one. This should allow us to determine if any of the configured push remotes do not contain the ref, and push it accordingly.

only allow the user to have one 'push' remote configured, so the old logic can be used again. This is simpler but may be considered a regression by the folk who wanted multiple cache support in the first place.

I don't like the idea of having multiple push remotes but not pushing to any of them if one already has the artifact because it would make the push behaviour a bit unpredictable; some elements would end up in all configured caches and others would only be in one, and you'd have to know the initial state of all the caches in order to work out how any given build was going to behave in terms of pushing.

In GitLab by [Gitlab user @samthursfield] on Feb 15, 2018, 18:19
mentioned in merge request !276

In GitLab by [Gitlab user @juergbi] on Feb 26, 2018, 15:27
closed via merge request !276


Added offset parameter to TextureAtlas::from_grid_with_padding

Objective
Increase compatibility with a fairly common format of padded spritesheets, in which half the padding value occurs before the first sprite box begins. The original behaviour falls out when Vec2::ZERO is used for offset.
See below unity screenshot for an example of a spritesheet with padding

Solution
Tiny change to crates/bevy_sprite/src/texture_atlas.rs
Migration Guide
Calls to TextureAtlas::from_grid_with_padding should be modified to include a new parameter, which can be set to Vec2::ZERO to retain old behaviour.
from_grid_with_padding(texture, tile_size, columns, rows, padding)
                                  |
                                  V
from_grid_with_padding(texture, tile_size, columns, rows, padding, Vec2::ZERO)

sure thing. Just give me a sec to find the migration-guide file :)

I've edited the PR description. Lmk if there's anywhere else you'd like me to write a note for it :)


Unable to save new watcher

Kibana version: 6.3.0
Elasticsearch version: 6.2.4
Server OS version:Red Hat Enterprise Linux Server release 6.10 (Santiago)
Browser version Version 64.0.3282.167 (Official Build) (64-bit)
**Browser OS version: CentOS release 6.9 **
Original install method (e.g. release pkg link, gulp, from source, etc.): from zip
**Describe the bug: When trying to save a new watcher from kibana dashboard, I get error like Error: Save watcher:Save 'watcher':Index 'watcher'
Error: Index 'watcher'
    at new SentinlError (http://hostname:6702/bundles/sentinl.bundle.js:3:71993)
    at WatcherService.<anonymous> (http://hostname:6702/bundles/sentinl.bundle.js:22:346324)
    at tryCatch (http://hostname:6702/bundles/vendors.bundle.js:133:364594)
    at Generator._invoke (http://hostname:6702/bundles/vendors.bundle.js:133:366669)
    at Generator.prototype.(anonymous function) [as throw] (http://hostname:6702/bundles/vendors.bundle.js:133:364878)
    at step (http://hostname:6702/bundles/sentinl.bundle.js:22:341023)
    at http://hostname:6702/bundles/sentinl.bundle.js:22:341184
    at <anonymous>

The watcher index is up and available in es.
{"watcher":{"aliases":{},"mappings":{"doc":{"properties":{"sentinl-user":{"properties":{"password":{"type":"text"},"sha":{"type":"text"},"username":{"type":"text"}}},"sentinl-watcher":{"properties":{"actions":{"type":"object","enabled":false},"condition":{"type":"object","enabled":false},"custom":{"type":"object","enabled":false},"dashboard_link":{"type":"text"},"disable":{"type":"boolean"},"impersonate":{"type":"boolean"},"input":{"type":"object","enabled":false},"report":{"type":"boolean"},"save_payload":{"type":"boolean"},"sentinl":{"type":"object","enabled":false},"spy":{"type":"boolean"},"title":{"type":"text"},"transform":{"type":"object","enabled":false},"trigger":{"type":"object","enabled":false},"username":{"type":"text"},"wizard":{"type":"object","enabled":false}}}}}},"settings":{"index":{"creation_date":"1539007031135","number_of_shards":"5","number_of_replicas":"1","uuid":"PjGWJhImTvqEAaZyJN68hA","version":{"created":"6030099"},"provided_name":"watcher"}}}}

All watchers fail to save. Sample:
{
  "_index": "watcher",
  "_type": "watch",
  "_id": "new",
  "_source": {
    "trigger": {
      "schedule": {
        "later": "every 5 minutes"
      }
    },
    "input": {
      "search": {
        "request": {
          "index": [
            "<mos-{now/d}>",
            "<mos-{now/d-1d}>"
          ],
          "body": {}
        }
      }
    },
    "condition": {
      "script": {
        "script": "payload.hits.total > 100"
      }
    },
    "transform": {},
    "actions": {
      "email_admin": {
        "throttle_period": "15m",
        "email": {
          "to"<EMAIL_ADDRESS>          "from"<EMAIL_ADDRESS>          "subject": "SENTINL Alarm",
          "priority": "high",
          "body": "Found {{payload.hits.total}} Events"
        }
      }
    }
  }
}

**
Steps to reproduce:

Add new sentinl watch using 'New' button on top right
Add conditions actions to it
Click on 'Save'.

Expected behavior: A new watcher should be added ideally
Sample of data you have in Elasticsearch
Full watcher document
**Screenshots (if relevant): **
**Errors in browser console (if relevant):
[object Object]: Save watcher:Save 'watcher':Index 'watcher'
Error: Save watcher:Save 'watcher':Index 'watcher'
Error: Index 'watcher'
    at new SentinlError (http://hostname:6702/bundles/sentinl.bundle.js:3:71993)
    at WatcherService.<anonymous> (http://hostname:6702/bundles/sentinl.bundle.js:22:346324)
    at tryCatch (http://hostname:6702/bundles/vendors.bundle.js:133:364594)
    at Generator._invoke (http://hostname:6702/bundles/vendors.bundle.js:133:366669)
    at Generator.prototype.(anonymous function) [as throw] (http://hostname:6702/bundles/vendors.bundle.js:133:364878)
    at step (http://hostname:6702/bundles/sentinl.bundle.js:22:341023)
    at http://hostname:6702/bundles/sentinl.bundle.js:22:341184
    at <anonymous>**

Provide Kibana logs and/or server output (if relevant):
**Any additional context: Sentinl Config from kibana.yml
sentinl:
      es:
        host: 'localhost'
        port: 9200
        default_type: 'doc'
        default_index: 'watcher'
        alarm_index: 'watcher_alarms'
        alarm_type: 'sentinl-alarm'
      settings:
          email:
                active: true
                user: manu
                host: smtp.something.com
          report:
                active: true

**
'

The problem is here
sentinl:
      es:
        default_type: 'doc'
        default_index: 'watcher'

Currently, Sentinl saves watchers only in index specified in kibana.yml var kibana.index, by default .kibana. Just delete default_index: 'watcher' and default_type: 'doc' and you will be able to save watchers.
Also, soon we add a possibilty to save watchers in any index.
Please reopen the issue if you still have this problem.

Removed the configuration for default index and default type. Still getting same error.
Here is complete kibana.yml
`server.port: 6702
server.host: <IP_ADDRESS>
elasticsearch.url: "https://hostname:9200"
elasticsearch.username: "user"
elasticsearch.password: "pass"
elasticsearch.ssl.verificationMode: none
elasticsearch.requestHeadersWhitelist: ["Authorization", "sgtenant"]
elasticsearch.requestTimeout: 30000
elasticsearch.shardTimeout: 30000
tilemap.url: "http://a.tile.openstreetmap.fr/hot/{z}/{x}/{y}.png"
xpack.reporting.csv.maxSizeBytes: 104857600
xpack.security.enabled: false
searchguard.session.ttl: 86400000
searchguard.cookie.ttl: 86400000
sentinl:
es:
host: 'localhost'
port: 9200
alarm_index: 'watcher_alarms'
alarm_type: 'sentinl-alarm'
settings:
email:
active: true
user: manu
host: smtp.something.com
report:
active: true
`

Are you sure the error you get does contain Index 'watcher' words? Please paste the error here.
Make sure Kibana is restarted after the config change.
I see you use X-Pack. Make sure the user you use for login has rights to save docs in the system index (.kibana by default).
Also, in order to make Sentinl execute watchers automatically by schedule, you should have a Sentinl user with system index read/monitor and watcher index write/read permissions in X-Pack. Then put this user in the kibana.yml
sentinl:
  settings:
    authentication:
      enabled: true
      username: sentinluser
      password: sentinlpassword

Are you sure the error you get does contain Index 'watcher' words? Please paste the error here.
Make sure Kibana is restarted after the config change.
I see you use X-Pack. Make sure the user you use for login has rights to save docs in the system index (.kibana by default).
Also, in order to make Sentinl execute watchers automatically by schedule, you should have a Sentinl user with system index read/monitor and watcher index write/read permissions in X-Pack. Then put this user in the kibana.yml
sentinl:
  settings:
    authentication:
      enabled: true
      username: sentinluser
      password: sentinlpassword

Hi, I have restarted the server. Uninstalled and reinstalled sentinl also.The complete error when saving the watcher is:
Error: Save wizard:Save 'watcher':Index 'watcher' Error: Index 'watcher'     at new SentinlError (http://ptpmongo1.newark:6702/bundles/sentinl.bundle.js:3:71993)     at WatcherService.<anonymous> (http://ptpmongo1.newark:6702/bundles/sentinl.bundle.js:22:346324)     at tryCatch (http://ptpmongo1.newark:6702/bundles/vendors.bundle.js:133:364594)     at Generator._invoke (http://ptpmongo1.newark:6702/bundles/vendors.bundle.js:133:366669)     at Generator.prototype.(anonymous function) [as throw] (http://ptpmongo1.newark:6702/bundles/vendors.bundle.js:133:364878)     at step (http://ptpmongo1.newark:6702/bundles/sentinl.bundle.js:22:341023)     at http://ptpmongo1.newark:6702/bundles/sentinl.bundle.js:22:341184     at <anonymous>
I can visualize the chart also from the query but unable to save it. My elastic search user has admin role in sg_roles.yml:
sg_all_access:   readonly: true   cluster:     - UNLIMITED   indices:     '*':       '*':         - UNLIMITED   tenants:     admin_tenant: RW
Do I need to create roles for sentinl alerting?(Example I have a role for x-pack alerting with following config in sg_roles.yml as mentioned here https://docs.search-guard.com/latest/search-guard-xpack-alerting ):
sg_alerting:   cluster:     - indices:data/read/scroll     - cluster:admin/xpack/watcher/watch/put     - cluster:admin/xpack/watcher*     - CLUSTER_MONITOR     - CLUSTER_COMPOSITE_OPS   indices:     '?kibana*':       '*':         - READ     '?watches*':       '*':         - INDICES_ALL     '?watcher-history-*':       '*':         - INDICES_ALL     '?triggered_watches':       '*':         - INDICES_ALL     '*':       '*':         - READ
Result of hostname/watcher:
{"watcher":{"aliases":{},"mappings":{"doc":{"properties":{"sentinl-user":{"properties":{"password":{"type":"text"},"sha":{"type":"text"},"username":{"type":"text"}}},"sentinl-watcher":{"properties":{"actions":{"type":"object","enabled":false},"condition":{"type":"object","enabled":false},"custom":{"type":"object","enabled":false},"dashboard_link":{"type":"text"},"disable":{"type":"boolean"},"impersonate":{"type":"boolean"},"input":{"type":"object","enabled":false},"report":{"type":"boolean"},"save_payload":{"type":"boolean"},"sentinl":{"type":"object","enabled":false},"spy":{"type":"boolean"},"title":{"type":"text"},"transform":{"type":"object","enabled":false},"trigger":{"type":"object","enabled":false},"username":{"type":"text"},"wizard":{"type":"object","enabled":false}}}}}},"settings":{"index":{"creation_date":"1539007031135","number_of_shards":"5","number_of_replicas":"1","uuid":"PjGWJhImTvqEAaZyJN68hA","version":{"created":"6030099"},"provided_name":"watcher"}}}}
Result of hostname/watcher_alarms:
{"watcher_alarms":{"aliases":{},"mappings":{"sentinl-alarm":{"properties":{"@timestamp":{"type":"date"},"action":{"type":"text","fields":{"keyword":{"type":"keyword","ignore_above":256}}},"attachment":{"type":"binary"},"error":{"type":"boolean"},"level":{"type":"text","fields":{"keyword":{"type":"keyword","ignore_above":256}}},"message":{"type":"text","fields":{"keyword":{"type":"keyword","ignore_above":256}}},"payload":{"type":"object","dynamic":"true"},"report":{"type":"boolean"},"type":{"type":"text","fields":{"keyword":{"type":"keyword","ignore_above":256}}},"watcher":{"type":"text","fields":{"keyword":{"type":"keyword","ignore_above":256}}}}}},"settings":{"index":{"creation_date":"1528969442464","number_of_shards":"5","number_of_replicas":"1","uuid":"tZQYa1h2TiKrs4V0yfofMw","version":{"created":"6020499","upgraded":"6030099"},"provided_name":"watcher_alarms"}}}}

Try to remove Kibana webpack bundles and restart it
rm -rf kibana/optimize/bundles/*


call: Support also SIP URI with missing display name.

In function call_connect SIP URIs like "sip:user@host" are not supported. The
log tells us:
call: sipsess_connect: Function not implemented
This is because the angle brackets are present and the display name is missing.
The patch achieves that the empty display name is ignored and the URI is
accepted.

Christian Spielberger writes:

In function call_connect SIP URIs like "sip:user@host" are not
supported.

Is the URI in call_connect a Request-URI?  If so, the above is not a
valid SIP-URI.
-- Juha

Hi Juha,
it might be, but not coercibly. The call_connect paddr may contain the display name or not. But if so, the call to sip_addr_decode removes the display name and angle brackets in order to get a correct call->peer_uri.
regards,
chris.

Christian Spielberger writes:

it might be, but not coercibly. The call_connect paddr may contain the
display name or not. But if so, the call to sip_addr_decode removes
the display name and angle brackets in order to get a correct
call->peer_uri.

OK, in name-addr, display-name may be empty:
name-addr =  [ display-name ] LAQUOT addr-spec RAQUOT
-- Juha

Christian Spielberger writes:

it might be, but not coercibly. The call_connect paddr may contain the
display name or not. But if so, the call to sip_addr_decode removes
the display name and angle brackets in order to get a correct
call->peer_uri.

call_connect() is not API function.  does this mean that also some API
function would not accept name-addr without display-name?

thanks, well spotted @cspiel1
I wanted to fix this for some time now. I will make some new testcases
and merge your patch.

Thanks @cspiel1
I merged your patch and added a testcase for SIP uri with angle brackets here:
https://github.com/alfredh/baresip/commit/ee2afbd4a0b9268893ed0ebc08b726850d81e14e


[FEATURE REQUEST] Separate file for each chapter option

I have novel Website some website have cloudflare etc so its hard to scrape from them to own website.
I have used your plugin was hoping you could add feature called - Seprate file for each chapter.
Basically, it will create. txt file for each chapter like chapter 1, chapter 2, etc all .txt.
Thank you.

That's already supported.

can u also add seprate folder followed by name like this in picture
https://prnt.sc/1qrfte8

I don't quite see a reason to add this; you can do archive this effect pretty easily with a script/re-namer. Furthermore I'm not working on this version of Novel-Grabber much.


Version 2.7.7

Rate Limiter - new settings to prevent DDOS
Monitoring - new metrics on notifications
Smart Charging - fix - targetStateOfCharge can be null

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: ClaudeROSSI:x: mahdibenromdhane:x: lmargaronmahdibenromdhane seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


JLCPCB 7-digit Manufacturer Part #'s Will Not Show in Plugin

Describe the bug
When I search for parts from LCSC with Part #'s consisting of 7 digits after the "C", the part will not show despite the part being In Stock, Extended, or Basic.
To Reproduce
Steps to reproduce the behavior:

Go to 'LCSC Plugin'
Click on 'Select Part' for any of the PCB components
Enter any LCSC component with any of the filters and make sure the components have 7 numbers following the 'C' for example:
C5122489
Observe the component not appearing despite being In-Stack, and Basic or Extended

Expected behavior
Component should appear in the listing.
KiCad Version

Application: KiCad PCB Editor (64-bit)
Version: (6.0.7), release build
Libraries:
    wxWidgets 3.1.7
    libcurl/7.83.1-DEV Schannel zlib/1.2.12
Platform: Windows 10 (build 19043), 64-bit edition, 64 bit, Little endian, wxMSW
Build Info:
    Date: Jul 26 2022 02:49:38
    wxWidgets: 3.1.7 (wchar_t,wx containers)
    Boost: 1.79.0
    OCC: 7.6.0
    Curl: 7.83.1-DEV
    ngspice: 37
    Compiler: Visual C++ 1929 without C++ ABI
Build settings:
    KICAD_USE_OCC=ON
    KICAD_SPICE=ON

Wasn't sure if this was a bug and should be fixed, or if it would technically be a new feature request if it is not implemented yet. My guess was that it would be a bug technically since it is a feature it should already have. Please let me know if you can fix this. There is no time crunch, and if there's anything I can do to help I will try to as much as I can to help keep this amazing plugin going for KiCAD users.

Hi, I think this is a common misconception about the LCSC numbers as LCSC numbers != JLCPCB numbers.
The JLCPCB parts database is just a subset of the entire LCSC portfolio of parts, so not all of them are officially available for assembly by JLC.

Another user already requested that all LCSC numbers should be available in this plugin, but that brings a problem with it.
At the moment we search the parts db supplied by JLC, thats why your numer is not found. If we want to go for all LCSC numbers we need a new data source which we don't have at the moment.
Furthermore, how can we make sure that JLC will assemble a part if its a "LCSC only" part!?
Let me know what you think ...

I checked with JLC via their chat function:
**JLCPCB**
What do you need help with? PCB Assembly
Hello. How may I help you?

**me**
Hi, can I include all LCSC parts in my design for assembly or just those listed by JLC in the parts section?
Specificly C5122489

**JLCPCB**
please note we support 1 side or 2 sided assembly now (if need 2 sides assembly, pls choose "standard assembly"). and that only parts in our part lib(jlcpcb.com/parts) and have stock will be selected, and only selected parts will be charged and assembled. And for the through hole parts, also need be in our part lib and have in stock, and make them in the Bom and CPl with same designator and similar coordinates, once it is selected, will be mounted as well.
so pls check if they are also in our part lib,. thanks

**me**
Ok, thank you for clarification!

**JLCPCB**
you are welcome friend
and if no stock , you may try these 2 ways
for some parts without stock or not in our part lib, you may try these 2 features (but should meet the MOQ and attrition for assembly, for small parts, pls add 15 pcs if possible):
a) try part pre order as this link https://support.jlcpcb.com/article/151-how-to-build-your-own-parts-library-in-jlcpcb
b) Try global sourcing part purchasing https://support.jlcpcb.com/article/192-how-to-use-jlcpcb-global-sourcing-parts-service (first , the platforms such as Mouser, Digikey should have in stock, then we can order. and need the order of the component finished( in the part order history, it shown "in warehouse"), then you can submit the SMT order))
And option b , you may take it as priority, since other platforms are with stock, so we can buy from them directly
Pls note part pre-order or global sourcing part purchasing usually will be done in 9-15 working days usually.
And only the part purchasing is finished (part order history will show "complete") .Then pls submit the SMT order(put the part number in the bom and cpl, and the part will be selected from "my part",need to meet the MOQ and attrition) later. (more details pls refer to https://support.jlcpcb.com/article/212-how-to-use-my-own-parts-for-pcb-assembly-order)
if any other questions, just be free to ask , have a good day. bye

If I get it right, only parts listed in their parts section are allowed. So I'll close this one.

Got it, thanks for your help Bouni! I am a student over in university and I was working with an industrial advisor for my Electrical Engineering Senior Design PCB project, and I was just about ready to rip my hair out trying to find these parts lol.  Anyway thank you again for all of your help and for making that plugin!
One thing I never did get down is that my imported gerber file into JLCPCB always had funky placements and wrong polarity for ICs, but it wasn’t too much of a serious issue since I just moved all the parts where they should be and double checked polarity w/ silkscreen I put on the board.
Sincerely,
Alexander Quilty

From: bouni @.>
Sent: Wednesday, January 4, 2023 6:13:36 AM
To: Bouni/kicad-jlcpcb-tools @.>
Cc: ACEQ @.>; Author @.>
Subject: Re: [Bouni/kicad-jlcpcb-tools] JLCPCB 7-digit Manufacturer Part #'s Will Not Show in Plugin (Issue #217)
I checked with JLC via their chat function:
JLCPCB
What do you need help with? PCB Assembly
Hello. How may I help you?
me
Hi, can I include all LCSC parts in my design for assembly or just those listed by JLC in the parts section?
Specificly C5122489
JLCPCB
please note we support 1 side or 2 sided assembly now (if need 2 sides assembly, pls choose "standard assembly"). and that only parts in our part lib(jlcpcb.com/parts) and have stock will be selected, and only selected parts will be charged and assembled. And for the through hole parts, also need be in our part lib and have in stock, and make them in the Bom and CPl with same designator and similar coordinates, once it is selected, will be mounted as well.
so pls check if they are also in our part lib,. thanks
me
Ok, thank you for clarification!
JLCPCB
you are welcome friend
and if no stock , you may try these 2 ways
for some parts without stock or not in our part lib, you may try these 2 features (but should meet the MOQ and attrition for assembly, for small parts, pls add 15 pcs if possible):
a) try part pre order as this link https://support.jlcpcb.com/article/151-how-to-build-your-own-parts-library-in-jlcpcb
b) Try global sourcing part purchasing https://support.jlcpcb.com/article/192-how-to-use-jlcpcb-global-sourcing-parts-service (first , the platforms such as Mouser, Digikey should have in stock, then we can order. and need the order of the component finished( in the part order history, it shown "in warehouse"), then you can submit the SMT order))
And option b , you may take it as priority, since other platforms are with stock, so we can buy from them directly
Pls note part pre-order or global sourcing part purchasing usually will be done in 9-15 working days usually.
And only the part purchasing is finished (part order history will show "complete") .Then pls submit the SMT order(put the part number in the bom and cpl, and the part will be selected from "my part",need to meet the MOQ and attrition) later. (more details pls refer to https://support.jlcpcb.com/article/212-how-to-use-my-own-parts-for-pcb-assembly-order)
if any other questions, just be free to ask , have a good day. bye
If I get it right, only parts listed in their parts section are allowed. So I'll close this one.
—
Reply to this email directly, view it on GitHubhttps://github.com/Bouni/kicad-jlcpcb-tools/issues/217#issuecomment-1370854313, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AX2JS2IVNXSLGS6LVPM6CNTWQVSPBANCNFSM6AAAAAAQFEBNSY.
You are receiving this because you authored the thread.Message ID: @.***>


Speed up iter_models (x2-x3 improvement)

It is actually quite slow to iterate over a polars dataframe with the index, it it better to use iter_rows.

import patito as pt

class MyModel(pt.Model):
    a: int
df = MyModel.DataFrame(pl.DataFrame({"a": range(1_000_000)}))

# main
%timeit -r1 -n1 list(df.iter_models(validate_model=False)) # 7.56 s
%timeit -r1 -n1 list(df.iter_models(validate_model=True)) # 6.62 s

# pr
%timeit -r1 -n1 list(df.iter_models(validate_model=False)) # 3.53 s
%timeit -r1 -n1 list(df.iter_models(validate_model=True)) # 2.39 s

Maybe we should expose buffer_size as well ?
NB: Weird that it is faster to validate the model, right ?

NB: Weird that it is faster to validate the model, right ?
Yes. I immediately checked the dataframe dtypes and height, but they are unchanged. I have no idea what is going on there 😅

Good work! Let's leave buffer_size out of the equation until someone needs it.


The basic info of an author's response must contain the affiliation info.

Endpoint:
http://apis.impactu.colav.co:8010/app/person/640f41b51c64b4bf1b57fe10
This information is already in the authors' search results so follow the same structure in this endpoint.
Example:

http://apis.impactu.colav.co:8010/app/search/person?limit=10&page=1&sort=citations&keywords=diego+alejandro+restrepo

affiliation info added to person schema:

bf9ae1e548a0a85ec625c2d6ea18bdd283da6270


added configurarations options

Added a configuration key called models_list, in case someone needs to draw a chart for a subset of the models in the rails application.
example configuration:
models_list:  User.descendants
default is still ::ActiveRecord::Base.descendants.sort_by(&:name)
the method could be anything that will return an array of active record models or even a string containing an array of models:
"[User, Role]"
Also added
hide_columns: true
to hide columns by default for readability purpose
not settings the option or setting it to false will show columns like it is in the  current setting

@francescob
Thank you for your pull request!
Could you help me better understand:

What problem this PR is trying to solve?
Who would benefit from this change and how?

This context would help me better evaluate the proposed changes.


FIX: Allow filtering of a DataList by subclass fields

FIX: Allow generating DataList aggregates of subclass fields
NEW: Add DataQuery::joinSubclassTableForField()
NEW: Add SearchFilter::getDbNameAndAddToQuery()
Fixes #1683

Paging @tractorcow and @dhensby since it's an ORM change

I have another way I would solve this problem. Comment on the other issue.

I want to flag with @unclecheese that this PR is all about providing more tools for query manipulation, and as such might overlap with eager loading work. It might be useful to ask "what additional APIs does DataQuery need for both use-cases?"

@sminnee Does this still make sense to pursue, or should we close it out? (Eager loading still hasn't shipped 😅)

I'm going to close this as rendered obsolete by #8426 - let's aim to get that merged instead!


sensu-client crashing (OOM)

Seeing this on multiple sensu clients with ~9 checks each.
The sensu-client process begins consuming more and more memory, until it OOMs and crashes.
Ubuntu systems.  Monitored with pidstats to confirm that it is the sensu process itself chewing up all the memory.  Client version 0.20.3.
Log output below:
{"timestamp":"2015-08-24T18:14:24.796698+0000","level":"warn","message":"reconnecting to transport"}
/opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/sensu-spawn-1.3.0/lib/sensu/spawn.rb:67:in `block in read_until_eof': failed to allocate memory (NoMemoryError)
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/sensu-spawn-1.3.0/lib/sensu/spawn.rb:67:in `loop'
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/sensu-spawn-1.3.0/lib/sensu/spawn.rb:67:in `read_until_eof'
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/sensu-spawn-1.3.0/lib/sensu/spawn.rb:100:in `child_process'
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/sensu-spawn-1.3.0/lib/sensu/spawn.rb:31:in `block in process'
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/eventmachine-1.0.3/lib/eventmachine.rb:1037:in `call'
        from /opt/sensu/embedded/lib/ruby/gems/2.0.0/gems/eventmachine-1.0.3/lib/eventmachine.rb:1037:in `block in spawn_threadpool'

Should note, client went from less than 1% memory usage to OOMing in less than 30 minutes.

you can cause the client to OOM if you pipe too much data to it :D


Upgrade to Hibernate ORM 5.4.21.Final

I haven't tested it locally: entrusting CI to test on all DBs.

Awesome. Thanks. On it.


Icon Request: Nachnahme

Hi,
I'd need "Nachname" for german market.

Hi, we're discussing the translation topic in #32


deep pyramid supervision

Hi!
I found that deep pyramid supervision is used in training PIPO( train_concave0.py)?
However, deep pyramid supervision is not  used in training PIPO-FAN( train_sf_partial.py).
Is that right?

Thanks for asking. I finetuned PIPO-FAN by using the pretrained PIPO. So deep pyramid supervision is included.

Thank you very much for your answer！


feat: migrate to Pathlib

Addresses https://github.com/ManimCommunity/manim/issues/485
Picked some of the files not addressed by previous or currently pending PRs

cc: @leotrs @huguesdevimeux

Thanks for your contribution!

Thanks. I will probably pick more files in subsequent PRs when I find time 👍

Sounds good!
@behackl I know the docs are building correctly, but would it be possible for you to eye-ball real quick the changes to the manim-directive before merging? I feel RTD can be a bit brittle...

@leotrs Corrected a minor typo I had left behind: https://github.com/ManimCommunity/manim/pull/630/commits/2205086c57b9107b636f220ab60e9f7e5caca886

Has anyone seen similar exception before?
(venv3) ➜  docs git:(path_lib) ✗ make html
Running Sphinx v3.1.2

Exception occurred:
  File "/Users/manvendra.s/work/learn/manim/venv3/lib/python3.8/site-packages/pangocairocffi/__init__.py", line 11, in <module>
    from ._generated.ffi import ffi
ModuleNotFoundError: No module named 'pangocairocffi._generated.ffi'
The full traceback has been saved in /var/folders/r2/drncpk212xxdw_zd5ns7nrrn09q2nk/T/sphinx-err-s2gtdcs3.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!

Has anyone seen similar exception before?

Yes: see https://docs.manim.community/en/latest/installation/troubleshooting.html

Oh my god I love having a functional troubleshooting page chef's kiss

@manu-chroma : Looks really great!
can you merge the current master into your branch?
There are currently two merge conflicts.

@kolibril13 I still haven't solved the issue highlighted by @behackl in the above comments.
If that's ok, I'd be happy to resolve the merge conflicts.

@manu-chroma : any updates on this?

I merged master into this, and as manim_derective had changed a lot, I discarded the changes to pathlib there.


gateway-api: ALPN support

This feature is hidden behind enable-gateway-api-alpn flag on the operator gwapi cell.
The implementation will change envoy listener configuration to expose ALPN suggesting both HTTP/2 and HTTP/1.1.
Fixes: #30794

/test

The code itself looks good, but I think we should spend some time thinking about what this means for users.
Could you walk through some scenarios for me?

Bascially, what I'm trying to think of here are situations where we should call out in the docs how this could break people before they enable it.

@youngnick enabling ALPN doesn't change anything for connections explicitly specifying their protocols, it only plays if client asks for available ALPN protocols (i.e initiates negotiation).
With ALPN exposed, you can exec curl commands with either of --http1.1, --http2 or --http2-prior-knowledge, commands will work fine.
It could have broken clients if I negotiated http2, but my client only serves http1.1.
But if you check the helm chart values, enabling ALPN will also enable appProtocol support which will opt to use HTTP1.1 by default for upstreams.
We can call out in the docs that if you enable ALPN, it will also enable appProtocol, so if one's upstream is serving HTTP2, they should specify appProtocol: kubernetes.io/h2c on their Service specs.

someone turns this on and is doing other TLS connections over HTTP2?

If their backend is only serving HTTP2 and they don't specify appProtocol: kubernetes.io/h2c and they enable ALPN (enabling appProtocol alongside), their requests will fail with 502 protocol error.
(unless they're using GRPCRoute which will force HTTP2 on the upstream whether you have appProtocol or not)

someone turns this on and is doing other TLS connections over HTTP1.1 (only)?

Nothing will break.

If their backend is only serving HTTP2 and they don't specify appProtocol: kubernetes.io/h2c and they enable ALPN (enabling appProtocol alongside), their requests will fail with 502 protocol error.

Here, I was talking about hypothetical protocol negotiating scenario. ALPN doesn't change much for this case, because what breaks the traffic flow is enabling appProtocol, not ALPN.

@youngnick Anything else we need for this to go in?

/test

/test

@youngnick those flaky checks again. 😩


PR review guidelines

for example, how much approvals should be for merging

PR approvals also require active contributors/reviewers with familiarity with the language in use... I love code review, but it's very hard if no one has the time or makes it a priority.
Obviously reviewing themes/templates would be easier than reviewing buidlers, etc... for submitting new themes I'd imagine an issue template with a checklist would be a good start...

let's then split this topic to 3 separate questions:

how many approvals needed for templates
..for color-schemes
for all other infra repos

I think it's related to #8.  It never gets anything done. Only project owners, and its delegations can approve, and only one approval is required.
With that being said, we can have as many reviews as we want, and the owners should request for more community feedback when fit. At the end of day, if we (as community members) can always fork if we are unhappy.

Why would we need more than the approval of a single core team member/maintainer (other than perhaps major changes to the spec itself)?  This quickly leads back into governance... but so far it seems like we'll likely have a small friendly "core team" for a while...
Another question (or perhaps part of what you're trying to ask?) is how much reign maintainers of individual projects hosted here should have (builders, templates).  I'm happy to have PRs reviewed for node-builder, but if that were to take forever or become a convoluted process I'd rather quickly change my mind (out of necessity to get work done).
I wouldn't be super inclined to start putting set in stone review requirements on individual project maintainers unless the review time was guaranteed to be same day/next day... Another data point: as the Base16 Highlight.js Template is very closely tied to Highlight.js project if it were ever to most to the base16-project I'd expect 100% autonomy to set my own PR review policies - as to fully maintain control of the larger Highlight.js release process.

I think setting explicit numbers of reviewers will end up getting tabled for now. We're not big enough to need it quite yet.
Until it becomes a problem, spec changes need to have a general consensus (I'm explicitly not setting a number of approvals here), scheme additions I don't have a strong preference on so I think anyone can merge them, but changes to existing schemes should be approved/merged by an owner (or someone on a scheme maintenance team if we add one), and template changes can be merged in by any one of the maintainers if they feel strongly about it (ping the template or project owners if there's any question).

Maybe be slightly more careful with anything that auto-publishes (Go, NPM, etc) but I'm not sure we have things that auto-publish yet?

I don't think we do yet, other than the schemes which get cloned directly from git (vim, emacs, etc).

other than the schemes which get cloned directly from git (vim, emacs, etc).

Wait, what automation are you talking about?  I'm not sure I follow.

@belak How do you feel about marking main as protected and requiring PRs, not direct commits?  I've almost committed directly on accident a few times now...

other than the schemes which get cloned directly from git (vim, emacs, etc).

Wait, what automation are you talking about? I'm not sure I follow.

It's not really automation, just that vim schemes are generally installed via plugin managers which clone the git repo... And there's a Rube Goldberg machine that builds the emacs package so it integrates with the package manager.

@belak How do you feel about marking main as protected and requiring PRs (as a dev workflow), not direct commits? I've almost committed directly on accident a few times now...

Good idea. I'll do this later today.

I'm closing this in favor of #84, where we're collecting all remaining project management related questions.


[WIP] deploy a new VM with network ids

Fix https://github.com/leap-solutions-asia/auto-scaling/issues/18

controller

add network uuid when deploy a new VM

dashboard

choose network uuid interface on Settings page

I'll merge it.


How to load meta information from msql to metacat

I am running docker version can you let me know how to load data from external mysql database to metacat

Could you clarify what data you are referring to? Are you looking to add a mysql database as a source? If so the RDS connector should allow you to do that.

I am trying to add MYSQL and MSSQL
Also

Is there an API for connectors

@satishjb hi, 这个问题您解决了吗？我遇到了同样的问题，我应该如何连接mysql数据源呢


Is it possible to remove reliance on Client ID and Client Secret?

Hi Ivan
Thanks for this new tool!
When opening the tool with one of my connections, I get an error "This tool is using server to server connection to get Power Automate flow runs. Please use connecitons that is authenticated with client ID and client secret."
Is this essential, or would it be possible to use the tool using other connection methods?
Many thanks, Alex

If we want to keep a seamless experience we need to use client/secret authentication, otherwise, you would get a popup for login every single time when you open the tool.
Client/secret authorization for integrations is essential on every big project so I don't feel it's a big overhead if you need to use this type of connection nowadays.
Will do a research and check how integrate other login methods, but without ruining the UX of the tool.

Thanks for the rapid reply.
This may just be my misunderstanding of how the Flow Execution History tool works differently from other tools in the XrmToolBox for it's connection to Dataverse / integration with Dataverse. The In-App Notification Builder for example doesn't require this.

Now I see what bothers you, but the data about flow runs is fetched by Power Automate API which is not fetching data from Datavserse so the normal connection is not enough.
Flow API access token needs to be fetched before executing API calls and the best way to do it is by client/secret which is already there in the connection that way user is not bothered by additional login popups.

Thanks for explaining that! Please feel free to close this issue.

I've published a new release where you can authenticate against Power Automate with username and password if you don't want to use client and secret.


Wrong Session path

In the file /opt/rh/httpd24/root/etc/httpd/conf.d/php55-php.conf the default session path will be set.
This directory is not writable when running a container on ose3 environment in non priviledged mode.
For a fix see pull request #49 #50

Both PR are merged in PR #53 now.

@derdanu thanks for fixing this!


Support Other Connection Types (specifically a pgx.Pool)

We use sqlc, which support pgx. The generated code is is more pgx specific with the interface methods as follows
type DBTX interface {
	Exec(context.Context, string, ...interface{}) (pgconn.CommandTag, error)
	Query(context.Context, string, ...interface{}) (pgx.Rows, error)
	QueryRow(context.Context, string, ...interface{}) pgx.Row
}

So creating a test db doesn't work since a database/sql DB does not satisfy the interface.
We worked around this by using https://github.com/jackc/pgx/blob/master/stdlib/sql.go which can create a database/sql DB from a pool for our production code and then use the pgtestdb DB for tests. But it would be nice if we could find a way to support the pgx.Conn and pgx.Pool types. pgxpool implements a nearly identical interface to pgx connections.

Thank you very much for the suggestion! This should be resolved as of 11a64cc by the pgtestdb.Custom method, now documented here:

https://github.com/peterldowns/pgtestdb#pgtestdbcustom
https://github.com/peterldowns/pgtestdb#how-do-i-connect-to-the-test-databases-through-pgx--pgxpool-instead-of-using-the-sqldb-interface

Please let me know how this works, or if this is insufficient for some reason.


Display a warning when multiple documents resolve to the same path

Problem Description
Google Drive allows multiple documents with the same name to exist in the same folder. Library currently only allows a single item to occupy a given path.
Feature
When this situation happens in a Google Drive or shared folder, Library should detect it and emit a warning that says only one document will be accessible through Library.

A subtask of this might be to ensure that, if there are multiple documents that match a url, the same document should get chosen every time.
Currently, library seems to vary which of the documents gets shown, which can be confusing.

Google Drive also allows multiple folders of the same name to exist in the same folder. It would be good to also cover that case.

Closed via #46 !


Laradock change permissions on host mysql DB

Hi there when I run
docker-compose up -d nginx mysql

and then
docker-compose down

The permissions on my host in folder
/var/lib/mysql

are change so all my local system have a problem.... This is really dangerous and I do not understand why the docker changes my permissions on local machine... :/

This seems weird. Can you try replacing the data container definitions in your docker-compose.yml file with the below:
### Databases Data Container ################################

    data:
        build: ./data
        volumes:
            - ./data/mysql:/var/lib/mysql
            - ./data/postgres:/var/lib/postgres
            - ./data/mariadb:/var/lib/mariadb
            - ./data/memcached:/var/lib/memcached
            - ./data/redis:/data
            - ./data/neo4j:/var/lib/neo4j/data
            - ./data/mongo:/data/db

...that should store the database files in a directory local to your Laradock installation.
I'm not entirely sure what the purpose of binding to /var/lib is - perhaps so that you can use existing databases on your system? It does sound a little scary!

Issue #96 May also be relevant here.

I can confirm that if I change docker-composer.yml to contain:
Databases Data Container
data:
    build: ./data
    volumes:
        - ./data/mysql:/var/lib/mysql
        - ./data/postgres:/var/lib/postgres
        - ./data/mariadb:/var/lib/mariadb
        - ./data/memcached:/var/lib/memcached
        - ./data/redis:/data
        - ./data/neo4j:/var/lib/neo4j/data
        - ./data/mongo:/data/db

Then docker will not change the permissions. But it is weird to me that docker is changing the permissions. Is it safe to use it then?
Probably I still do know only little about docker...

Upgrade your LaraDock git pull origin master.
(LaraDock v4 works only with Docker for Mac/Windows. If you want to keep using LaraDock toolbox (VM) use LaraDock v3).
Rebuild your containers docker-compose build --no-cache (This might wipe your Data container)
Try again.. best of luck :)


Spike: Continuous delivery

It'd be amazing to be able to publish new versions of the extension semi-automatically from successful CI builds on a nominated deployment branch. What would be involved? Is this a thing we can even do without exposing signing keys? How would we handle rollbacks?

This is not the simplest thing that might work.


Opening a new position sometimes takes unacceptably long with very weird behavior.

Look at the video, something is not right.
https://github.com/get10101/10101/assets/224613/bf0f9cb6-2ce1-4871-b1d7-0632a239c369
2024-04-09_122223.log

I'll add critical because I believe a normal user would have closed the app already.

Looks like you are losing the connection in between. Otherwise you wouldn't see the catch up dialog.

related to https://github.com/get10101/10101/issues/2089

Looks like you are losing the connection in between. Otherwise you wouldn't see the catch up dialog.

But there is no reason for me to lose connection.

Looks like you are losing the connection in between. Otherwise you wouldn't see the catch up dialog.

But there is no reason for me to lose connection.

Can you maybe add the relevant logs when you opened the position?

Related to https://github.com/get10101/10101/issues/2415

Looks like I'm in a broken state now :(:
on my side I see this:

[WARN] TIME: 2024-04-16T11:00:59.179567Z r: dlc_manager::manager: Dlc channel timed out in State SettledAccepted { counter_next_per_update_point: PublicKey(6ff23dbe7ab0aeb721b6d22062e325b66e4a69516c7abbd6fbd6c23f3b9df01077b7bfdf4d4f5cb0497019453149cdfe202373cc62826f241fab423ced31d96f), own_next_per_update_point: PublicKey(e716c65c283c1c0ba6c2ccb176e6e00cb278ae1dbe70df25a62432627d9fcdfdd3c54d4451e1407be0b18238bd6d56b97061a5281448790914b280b48498e8b2), own_settle_adaptor_signature: EcdsaAdaptorSignature(02d72a6ebcf0b2e6a311e0b7286595d5b1e93a4ad8b068a4e59bb19be5a6d15eb503a88bea00c3b70612981488021d69f47ee965c9d04fea7554556f2ab74834033ac5783da03473c03e18005aaad3e9996d9fa48cc26cb0e628f7ba0829156535ad5d1558a4f851b945128dcc7966b9aa99396f76528d3e7178c38f9ab8b1aa816308c859936af606a66110fdf659e6508e2c6e1ccc0e098d340f40038f73fc378e), settle_tx: Transaction { version: 2, lock_time: PackedLockTime(0), input: [TxIn { previous_output: OutPoint { txid: dc21b526971e07b77637ab38aa811a24be12aaab5bae6c1d11eae813aecbb01d, vout: 2 }, script_sig: Script(), sequence: Sequence(4294967295), witness: Witness { content: [], witness_elements: 0, last: 0, second_to_last: 0 } }], output: [TxOut { value: 1586053, script_pubkey: Script(OP_0 OP_PUSHBYTES_32 9a3a48d1d9d001cacf40e5e26f3abfd77b28259ed99f893376cc07433e854bc1) }, TxOut { value: 1336598, script_pubkey: Script(OP_0 OP_PUSHBYTES_32 86d426dd1c9ab0e1ef6379659fdb6f0962920c8fab8eda5b252d52992da1d8a0) }] }, timeout:<PHONE_NUMBER>, own_payout: 1334078, counter_payout: 1583533 }. Skipping force-closure of dlc channel

2024-04-16_210102.log

Looks like I'm in a broken state now :(:
on my side I see this:

[WARN] TIME: 2024-04-16T11:00:59.179567Z r: dlc_manager::manager: Dlc channel timed out in State SettledAccepted { counter_next_per_update_point: PublicKey(6ff23dbe7ab0aeb721b6d22062e325b66e4a69516c7abbd6fbd6c23f3b9df01077b7bfdf4d4f5cb0497019453149cdfe202373cc62826f241fab423ced31d96f), own_next_per_update_point: PublicKey(e716c65c283c1c0ba6c2ccb176e6e00cb278ae1dbe70df25a62432627d9fcdfdd3c54d4451e1407be0b18238bd6d56b97061a5281448790914b280b48498e8b2), own_settle_adaptor_signature: EcdsaAdaptorSignature(02d72a6ebcf0b2e6a311e0b7286595d5b1e93a4ad8b068a4e59bb19be5a6d15eb503a88bea00c3b70612981488021d69f47ee965c9d04fea7554556f2ab74834033ac5783da03473c03e18005aaad3e9996d9fa48cc26cb0e628f7ba0829156535ad5d1558a4f851b945128dcc7966b9aa99396f76528d3e7178c38f9ab8b1aa816308c859936af606a66110fdf659e6508e2c6e1ccc0e098d340f40038f73fc378e), settle_tx: Transaction { version: 2, lock_time: PackedLockTime(0), input: [TxIn { previous_output: OutPoint { txid: dc21b526971e07b77637ab38aa811a24be12aaab5bae6c1d11eae813aecbb01d, vout: 2 }, script_sig: Script(), sequence: Sequence(4294967295), witness: Witness { content: [], witness_elements: 0, last: 0, second_to_last: 0 } }], output: [TxOut { value: 1586053, script_pubkey: Script(OP_0 OP_PUSHBYTES_32 9a3a48d1d9d001cacf40e5e26f3abfd77b28259ed99f893376cc07433e854bc1) }, TxOut { value: 1336598, script_pubkey: Script(OP_0 OP_PUSHBYTES_32 86d426dd1c9ab0e1ef6379659fdb6f0962920c8fab8eda5b252d52992da1d8a0) }] }, timeout:<PHONE_NUMBER>, own_payout: 1334078, counter_payout: 1583533 }. Skipping force-closure of dlc channel

2024-04-16_210102.log

I've rolled back my channel state and was able to close the position. Thanks @luckysori for the hand holding.

For debugging:
check the DLC_Messages table for how long does it take from start to end.

Updated logs from closing the position just now
2024-04-24_112613.log


✨ Add an artifact loader for .yaml

Builds on and extends https://github.com/laminlabs/lamindb/pull/2172

test (storage) failure was fixed in https://github.com/laminlabs/lamindb/pull/2266, i don't want to introduce conflicts so leave it failing here.


[FEATURE] Add support for Seq

At the moment, Apalache does not translate the Seq operator. We should add support similar to TLC. Given an option or a predefined operator that encodes the bound on the length of sequences, we can produce the set of all sequences up to that length.

Perhaps relevant: https://github.com/tlaplus/CommunityModules/blob/2ebcbd910bcf5a2cc7f62cdb3a11c9e2f2d15641/modules/SequencesExt.tla#L47-L59

This is discussed in: https://apalache.informal.systems/docs/apalache/known-issues.html#using-seqs


Respect buffer's original dimension when updating reglBuffer data

Currently, buffer's dimension will be always reset to 1 when calling buffer(array) to update its data.
This default behavior will lead to some incorrect context, e.g. wrong size parameter when calling gl.vertexAttribPointer in REGLVAO.
Respect buffer's original dimension will solve it.

Sorry this has sat for a long time, but it's my opinion that perhaps the current behavior is correct. The docs say:

To reinitialize a buffer in place, we can call the buffer as a function:

In that sense, I believe it's a full reinitialization with interpretation of arguments identical to the initial creation of the buffer.
Assuming some existing buffer, you can achieve the correct dimension by either specifying the dimension explicitly, e.g.
existingBuffer({
  data: [1, 2, 3],
  dimension: 3
})

or you can nest the data as an array of arrays, e.g.
existingBuffer([[1, 2, 3]])

If you want to replace data in-place without reinitializing, buffer.subdata is  more performant.
If this is still of interest, let me know if you have opinions. Thanks!


Only download Kaldi when it is requested to be installed.

Build-from-source.sh downloads Kaldi and installs gfortran, also when Kaldi should not to be installed.
I suppose it's better to also check the no_kaldi flag when trying to download Kaldi.

Good catch, thanks 👍


Add ppa:chris-lea/zeromq to the whitelist

Resolves issue #149.

@meatballhat The repository seems to be moving a little slow. Is there any way for me to volunteer to get things up to speed?

+1
We would really benefit from this PR being merged. Currently we have to compile ZeroMQ, which is a quite long process , and also we're tied to the legacy infraestructure because of the need to use sudo to build ZeroMQ.

Is there anything holding this up?

I would also benefit from this. My build times on Juno are ~30min because I need sudo to install libzmq3-dev for ZMQv4.x which (from what I have gathered from the docs) means that I can't use caching. Instead, Juno needs to recompile the universe for every build and in haskell that takes a long time if you have a lot of deps.
The whitelisted Travis-CI version of libzmq3 is for ZMQv3.0. I'm happy to help if help is needed.

hello?

bump


Allow searching for TV shows

Currently TV shows are not searched. We should refactor the code so you can specify which media you want to search.

Search types: https://github.com/pkkid/python-plexapi/blob/0f9cb40d5832bcb63e61240564d46fe0b3f5871c/plexapi/utils.py#L17-L19


Rename to Door Counter

Title.

door counter assumes we're counting doors. and that's an easy task. one ... two. upl has two doors. you don't even need electronics for that one!

We are counting doors, more specifically the open ones

good point


Display UK timezone on audit log

Description of change
Time displayed was previously UTC. This update forces the app to use London, regardless of server timezone or user's timezone
Story Link
https://trello.com/c/q7YTMdcA/272-time-in-bops-is-all-gmt-never-got-updated-to-bst
Decisions
I decided to go down the route of simply displaying London timezone times in the audit log instead of UTC because this way it avoids messing with what is stored in the DB and also avoids a dependency on server time. Following the "principle of least surprise", I reckon that simply updating the display to UK current time in this way would be what a planner might expect, even if they were temporarily in another timezone.

 Time in BOPS is all GMT (never got updated to BST)

closing in favour of #308


CORE-18840 Ignore auto-reformat commit in git blame.

Already included in https://github.com/corda/corda-runtime-os/pull/5253, so closing this PR.