DELETE doesn't delete fresh data

System info: I tried 1.4.0, 1.4.1 and 1.4.2 on debian linux. I'm not able to reproduce it on 1.3.7
Steps to reproduce:
We have a unit test that does the following steps:

Clean our test-database (drop database and create database)
Insert some data
Delete some of the data
Query for the deleted data

Expected behavior: The deleted data should not be in the result of step 4.
Actual behavior: Until 1.3.7 it worked as expected. Since 1.4.0 some of the deleted data is retuned by the result of step 4, which makes our test red.
Additional info:
We thought it might be a timing problem, but even if we add a sleep of 10 seconds between step 2 and 3 AND between step 3 and 4 we can easily reproduce the problem.
Below is the log-output of the influxdb service during our test:
Nov 27 17:20:51.310521 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z DROP DATABASE TSM service=query
Nov 27 17:20:51.312877 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+DATABASE+TSM&u=+ HTTP/1.1" 200 171 "-" "InfluxData.Net.InfluxDb" ef6246ed-d38e-11e7-80d6-000000000000 2516
Nov 27 17:20:51.315628 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z CREATE DATABASE TSM service=query
Nov 27 17:20:51.317332 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=CREATE+DATABASE+TSM&u=+ HTTP/1.1" 200 173 "-" "InfluxData.Net.InfluxDb" ef630643-d38e-11e7-80d7-000000000000 2100
Nov 27 17:20:51.321191 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+MEASUREMENT+tsm_meta_rp.tsm_data&u=+ HTTP/1.1" 400 72 "-" "InfluxData.Net.InfluxDb" ef63ddd7-d38e-11e7-80d8-000000000000 89
Nov 27 17:20:51.332277 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z DROP MEASUREMENT tsm_typical_data_age service=query
Nov 27 17:20:51.332390 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+MEASUREMENT+%22tsm_typical_data_age%22&u=+ HTTP/1.1" 200 191 "-" "InfluxData.Net.InfluxDb" ef659388-d38e-11e7-80d9-000000000000 339
Nov 27 17:20:51.354930 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z CREATE RETENTION POLICY tsm_meta_rp ON TSM DURATION 0s REPLICATION 1 service=query
Nov 27 17:20:51.357041 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?p=%5BREDACTED%5D&q=CREATE+RETENTION+POLICY+tsm_meta_rp+ON+TSM+DURATION+inf+REPLICATION+1&u=+ HTTP/1.1" 200 222 "-" "InfluxData.Net.InfluxDb" ef690abd-d38e-11e7-80da-000000000000 2320
Nov 27 17:20:51.359788 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z SHOW RETENTION POLICIES ON TSM service=query
Nov 27 17:20:51.359918 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=SHOW+RETENTION+POLICIES+ON+TSM&u=+ HTTP/1.1" 200 204 "-" "InfluxData.Net.InfluxDb" ef69c719-d38e-11e7-80db-000000000000 260
Nov 27 17:20:51.362819 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z SHOW RETENTION POLICIES ON TSM service=query
Nov 27 17:20:51.362953 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=SHOW+RETENTION+POLICIES+ON+TSM&u=+ HTTP/1.1" 200 204 "-" "InfluxData.Net.InfluxDb" ef6a3c05-d38e-11e7-80dc-000000000000 268
Nov 27 17:20:51.373083 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z SHOW RETENTION POLICIES ON TSM service=query
Nov 27 17:20:51.373283 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=SHOW+RETENTION+POLICIES+ON+TSM&u=+ HTTP/1.1" 200 204 "-" "InfluxData.Net.InfluxDb" ef6bc835-d38e-11e7-80dd-000000000000 334
Nov 27 17:20:51.375442 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z DROP DATABASE TSM service=query
Nov 27 17:20:51.377294 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+DATABASE+TSM&u=+ HTTP/1.1" 200 171 "-" "InfluxData.Net.InfluxDb" ef6c2c68-d38e-11e7-80de-000000000000 1949
Nov 27 17:20:51.379727 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z CREATE DATABASE TSM service=query
Nov 27 17:20:51.381461 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=CREATE+DATABASE+TSM&u=+ HTTP/1.1" 200 173 "-" "InfluxData.Net.InfluxDb" ef6cd1cb-d38e-11e7-80df-000000000000 1985
Nov 27 17:20:51.383868 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+MEASUREMENT+tsm_meta_rp.tsm_data&u=+ HTTP/1.1" 400 72 "-" "InfluxData.Net.InfluxDb" ef6d79d0-d38e-11e7-80e0-000000000000 73
Nov 27 17:20:51.386617 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z DROP MEASUREMENT tsm_typical_data_age service=query
Nov 27 17:20:51.386759 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=DROP+MEASUREMENT+%22tsm_typical_data_age%22&u=+ HTTP/1.1" 200 191 "-" "InfluxData.Net.InfluxDb" ef6dd893-d38e-11e7-80e1-000000000000 336
Nov 27 17:20:51.409606 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z CREATE RETENTION POLICY tsm_meta_rp ON TSM DURATION 0s REPLICATION 1 service=query
Nov 27 17:20:51.411274 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?p=%5BREDACTED%5D&q=CREATE+RETENTION+POLICY+tsm_meta_rp+ON+TSM+DURATION+inf+REPLICATION+1&u=+ HTTP/1.1" 200 222 "-" "InfluxData.Net.InfluxDb" ef715e4c-d38e-11e7-80e2-000000000000 2056
Nov 27 17:20:51.414152 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z SHOW RETENTION POLICIES ON TSM service=query
Nov 27 17:20:51.414303 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=SHOW+RETENTION+POLICIES+ON+TSM&u=+ HTTP/1.1" 200 204 "-" "InfluxData.Net.InfluxDb" ef721033-d38e-11e7-80e3-000000000000 286
Nov 27 17:20:51.417470 ubuntu influxd[4156]: [I] 2017-11-27T16:20:51Z SHOW RETENTION POLICIES ON TSM service=query
Nov 27 17:20:51.417590 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "GET /query?db=TSM&p=%5BREDACTED%5D&q=SHOW+RETENTION+POLICIES+ON+TSM&u=+ HTTP/1.1" 200 204 "-" "InfluxData.Net.InfluxDb" ef7276fe-d38e-11e7-80e4-000000000000 312
Nov 27 17:20:51.466050 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "POST /write?db=TSM&p=%5BREDACTED%5D&precision=ns&rp=tsm_meta_rp&u=+ HTTP/1.1" 204 0 "-" "-" ef7a014a-d38e-11e7-80e5-000000000000 49
Nov 27 17:20:51.533784 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "POST /write?db=TSM&p=%5BREDACTED%5D&precision=ns&rp=autogen&u=+ HTTP/1.1" 204 0 "-" "-" ef8385de-d38e-11e7-80e6-000000000000 5441
Nov 27 17:20:51.609196 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:20:51 +0100] "POST /write?db=TSM&p=%5BREDACTED%5D&precision=ns&rp=tsm_meta_rp&u=+ HTTP/1.1" 204 0 "-" "-" ef8f0803-d38e-11e7-80e7-000000000000 5435
Nov 27 17:21:01.543393 ubuntu influxd[4156]: [I] 2017-11-27T16:21:01Z DELETE FROM tsm_data WHERE f_l0 = 'aggnode1' AND f_l1 = 'aggnode2' AND f_l2 = 'aggnode51' AND f_l3 = 'datanode62' AND f_l4 = '' AND time >= -9223372036854772800 AND time <= -9223372036854771800 service=query
Nov 27 17:21:01.546139 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:21:01 +0100] "GET /query?db=TSM&epoch=ns&p=%5BREDACTED%5D&q=DELETE+FROM+tsm_data+WHERE+f_l0+%3D+%27aggnode1%27+AND+f_l1+%3D+%27aggnode2%27+AND+f_l2+%3D+%27aggnode51%27+AND+f_l3+%3D+%27datanode62%27+AND+f_l4+%3D+%27%27+AND+time+%3E%3D+-9223372036854772800+AND+time+%3C%3D+-9223372036854771800&u=+ HTTP/1.1" 200 33 "-" "InfluxData.Net.InfluxDb" f57ba274-d38e-11e7-80e8-000000000000 3093
Nov 27 17:21:11.551815 ubuntu influxd[4156]: [I] 2017-11-27T16:21:11Z SELECT * FROM TSM.autogen.tsm_data WHERE f_l0 = 'aggnode1' AND f_l1 = 'aggnode2' AND f_l2 = 'aggnode51' AND f_l3 = 'datanode62' AND f_l4 = '' AND time >= -9223372036854772800 AND time <= -9223372036854771800 ORDER BY time ASC LIMIT 100 service=query
Nov 27 17:21:11.552201 ubuntu influxd[4156]: [httpd] <IP_ADDRESS> -   [27/Nov/2017:17:21:11 +0100] "GET /query?db=TSM&epoch=ns&p=%5BREDACTED%5D&q=SELECT++%2A+FROM+autogen.tsm_data+WHERE+f_l0+%3D+%27aggnode1%27+AND+f_l1+%3D+%27aggnode2%27+AND+f_l2+%3D+%27aggnode51%27+AND+f_l3+%3D+%27datanode62%27+AND+f_l4+%3D+%27%27+AND+time+%3E%3D+-9223372036854772800+AND+time+%3C%3D+-9223372036854771800+ORDER+BY+time+ASC+LIMIT+100&u=+ HTTP/1.1" 200 2224 "-" "InfluxData.Net.InfluxDb" fb72c96b-d38e-11e7-80e9-000000000000 1002

Do you have a script that we can run to reproduce this?

Thanks Jason for the quick response.
Yes I stripped my test down to the attached script.
This is a transcript of my usage of the script:
lars@ubuntu:~$ sudo dpkg -i influxdb_1.3.7_amd64.deb dpkg: warning: downgrading influxdb from 1.4.2-1 to 1.3.7-1 (Reading database ... 94349 files and directories currently installed.) Preparing to unpack influxdb_1.3.7_amd64.deb ... Unpacking influxdb (1.3.7-1) over (1.4.2-1) ... Setting up influxdb (1.3.7-1) ... Installing new version of config file /etc/influxdb/influxdb.conf ... Processing triggers for man-db (2.7.5-1) ... lars@ubuntu:~$ sudo systemctl restart influxdb lars@ubuntu:~$ /mnt/hgfs/D/data/influx-delete2.sh {"results":[{"statement_id":0}]} {"results":[{"statement_id":0}]} {"results":[{"statement_id":0}]} {"results":[{"statement_id":0}]} lars@ubuntu:~$ sudo dpkg -i influxdb_1.4.2_amd64.deb (Reading database ... 94349 files and directories currently installed.) Preparing to unpack influxdb_1.4.2_amd64.deb ... Unpacking influxdb (1.4.2-1) over (1.3.7-1) ... Setting up influxdb (1.4.2-1) ... Installing new version of config file /etc/influxdb/influxdb.conf ... Processing triggers for man-db (2.7.5-1) ... lars@ubuntu:~$ sudo systemctl restart influxdb lars@ubuntu:~$ /mnt/hgfs/D/data/influx-delete2.sh {"results":[{"statement_id":0}]} {"results":[{"statement_id":0}]} {"results":[{"statement_id":0}]} {"results":[{"statement_id":0,"series":[{"name":"tsm_data","columns":["time","f_changed","f_l0","f_l1","f_l2","f_l3","t_old_data","vl_0_label52","vl_1_label54","vl_2_label56","vl_3_label58","vl_4_label60"],"values":[["1677-09-21T00:12:43.1452272Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452272,-0.08063101069679558,-0.0806310102,529122247631452272,-1100342216],["1677-09-21T00:12:43.1452273Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452273,-0.08063101069679558,-0.0806310102,529122247631452273,-1100342216],["1677-09-21T00:12:43.1452274Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452274,-0.08063101069679558,-0.0806310102,529122247631452274,-1100342215],["1677-09-21T00:12:43.1452275Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452275,-0.08063101069679558,-0.0806310102,529122247631452275,-1100342215],["1677-09-21T00:12:43.1452276Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452276,-0.08063101069679558,-0.0806310102,529122247631452276,-1100342214],["1677-09-21T00:12:43.1452277Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452277,-0.08063101069679558,-0.0806310102,529122247631452277,-1100342214],["1677-09-21T00:12:43.1452278Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452278,-0.08063101069679558,-0.0806310102,529122247631452278,-1100342213],["1677-09-21T00:12:43.1452279Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452279,-0.08063101069679558,-0.0806310102,529122247631452279,-1100342213],["1677-09-21T00:12:43.145228Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452280,-0.08063101069679558,-0.0806310102,529122247631452280,-1100342212],["1677-09-21T00:12:43.1452281Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452281,-0.08063101069679558,-0.0806310102,529122247631452281,-1100342212],["1677-09-21T00:12:43.1452282Z",1511864140346845600,"aggnode1","aggnode2","aggnode51","datanode62","True",529122247631452282,-0.08063101069679558,-0.0806310102,529122247631452282,-1100342211]]}]}]}

@lsitzmann Thanks for the repro script.  #9173 should fix this on master once merged.  I'll backport to 1.4 as well.


IndexError: list index out of range (similar to #208)

t = yf.Ticker("XEF.TO").info
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ubuntu/yfinance/yfinance/ticker.py", line 138, in info
    return self.get_info()
  File "/home/ubuntu/yfinance/yfinance/base.py", line 416, in get_info
    self._get_fundamentals(proxy)
  File "/home/ubuntu/yfinance/yfinance/base.py", line 305, in _get_fundamentals
    s[s.index == 'ratingYear']['Value'].values[0],
IndexError: index 0 is out of bounds for axis 0 with size 0

I got this issue by cloning the master branch with the latest commit being 9cdda84. The issue is similar to #208. Simply checking if the entry exists before accessing it would fix this bug.

PR #354 should fix this. However the code is failing the codefactor check. I didn't have a chance to check if it's because of my code changes or something else.


Bump Service Downloader

Thanks for taking care of bumping this!

Just an update on this, @Charles-Gagnon discovered the changes he made to the service-downloader don't support tars for linux and mac so this is on hold till we solve that problem and update the package again.

@Charles-Gagnon is this PR good to go?

Looks good to me - if you're happy go ahead and merge it in


Added jax support for standardize_dtype

This PR adds jax support for standardize_dtype.
One thing that came up during this PR that I think is worth asking is: should ALLOWED_DTYPES be backend specific? See test for where things get a bit ugly with torch / jax config.

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

Comparison is base (1a480f4) 78.97% compared to head (784b2d0) 56.23%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #18764       +/-   ##
===========================================
- Coverage   78.97%   56.23%   -22.75%
===========================================
  Files         336      336
  Lines       34040    34040
  Branches     6668     6668
===========================================
- Hits        26883    19141     -7742
- Misses       5580    13447     +7867
+ Partials     1577     1452      -125

Flag
Coverage Δ

keras
56.23% <100.00%> (-22.64%)
:arrow_down:

keras-jax
?

keras-numpy
56.23% <100.00%> (ø)

keras-tensorflow
?

keras-torch
?

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

Files
Coverage Δ

keras/backend/common/variables.py
97.45% <100.00%> (-2.55%)
:arrow_down:

... and 183 files with indirect coverage changes

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


Typeguard-friendly Array#every

~Fixes #~ I'm not aware of any issue
This PR tries to address the following. However, it may not play well with mutations.
function is_number(v: unknown): v is number {
  return typeof v === "number";
}

function weird_sum(collection: Array<string | number>) {
  if (collection.every(is_number)) {
    // currently, typescript raises an error here
    const sum: number = collection.reduce((acc, v) => acc + v);
    return sum;
  }
  return 0;
}

weird_sum([5, "abc", 42]);

Please verify that:

[ ] There is an associated issue in the Backlog milestone (required)
[x] Code is up-to-date with the master branch
[x] You've successfully run gulp runtests locally
[ ] There are new or updated unit tests validating the change

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: fazouane-marouane sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Duplicate of https://github.com/microsoft/TypeScript/pull/38200


Examples do not work with 0.92 release

Version 0.9.3 needs to be issued as a release here and on pypi.
If one tries to run the current examples after installing from 0.92 one gets syntax errors. See the below error form running compr-rotor.py:
error.txt

I've uploaded the new release on github and pypi now:
https://github.com/spacether/pycalculix/releases/tag/v0.9.3
https://pypi.python.org/pypi?:action=display&name=pycalculix&version=0.9.3
Examples now run.


How do I rate-limit some but not all endpoints?

Hi - Thank you for taking the time to create this!
How do I protect multiple endpoints in express?
const rateLimiter = RateLimiter(app, app.locals.redisClient0);
rateLimiter({
    path: '/api/fetch',
    method: 'post',
    lookup: 'headers.x-forwarded-for',
    total: 120,
    expire: 1000 * 60 * 60
});

It seems like the path is a string, not an array. Please let me know - thanks!

https://github.com/ded/express-limiter#as-direct-middleware

Is doing this in the whitelist option with the request object an alternative? Asking because I absolutely want to avoid hitting the storage backend on maintenance and high frequency routes. Those are very small in number all others endpoints are not, which makes direct middleware unfeasible.

Just check the code: it is.


Barebones grpc aio attempt

put together a bare diff with ideas from #217  - might be a good starting point for @MHDante 's idea

closing based on conversation in #217


Crystal Scribing Tool Bug

Under 0.572 of Forbidden Magic for 1.7.10, I noticed when I crafted a second set of Crystal Scribing Tools that I received about twice as many research points when both were in my inventory.  Decided to launch a creative world and filled my inventory with them, and sure enough, I had 400 research points in about 3-4 minutes.
I'm guessing this is not intended functionality since it's ripe for abuse so I decided to hunt down the github and post.
I should probably mention that this functionality of the scribing tools is only available when Thaumcraft research is set to easy.  Forgot to post that initially.

That's the intended functionality when research is set to easy.  Guess I can look into putting some sort of limit on filling your inventory with them.

I figured that the tools weren't intended to stack the return based on the number of them in your inventory.  If that is intended, than this is more of a balance concern than a bug and I realize how subjective balance can be.
If this is balance, maybe having a config option for the amount of research points returned or the number of ticks before the event handler fires would allow users to adjust to their preference.


Unable to get current page route，Expect a function  import { getCurrentRoute } from 'vue-router'

Reproduction
Unable to get current page route
Steps to reproduce the bug
import { getCurrentRoute } from 'vue-router'
Expect a function from inside the vue-router  getCurrentRoute()
Whether vue-router provides such a function
Expected behavior
import { getCurrentRoute } from 'vue-router'
Expect a function from inside the vue-router  getCurrentRoute()
Whether vue-router provides such a function
Actual behavior
vue-router does not provide this function
Additional information
No response

Remember to use discussions or the Discord server instead for questions next time 😉


Trimmed russian advancement

Some translated achievements are not displayed correctly, I have noticed often only the last word in a single line display is often not displayed.
MC: 1.19.2.
PaginatedAdvanncements version: 2.2.1-1.19
Some example:

Screenshots

And another mod for example:

Screenshots

Interesting. The vanilla advancements screen does not have that issue?

It's not all that clear-cut :DDD
On vanilla, description displays as it should, but the name of the advancement is trimmed...

Screenshots

As with PaginatedAdvancements text larger than 1 line is displayed correctly

Screenshots

Wack. Alright, I will see what I can do, ty! 😄


[fix]Use "more" as default pager (fix windows problem)

Use "more" instead of "less" as default pager
Fixes: https://github.com/edgexfoundry-holding/edgex-cli/issues/127
Signed-off-by: Diana Atanasova<EMAIL_ADDRESS>

I have tested the fix on both linux and windows
@mkbhanda, please test the fix works on Mac


Paella player preview image versus the new previewPortrait: Opencast only creates landscape preview image

Describe the bug
The Opencast preview image only displays in Paella when the player window is in landscape mode (wider than it is tall).
When the player window is in portrait mode (taller than it is thin) the preview image dissapears and a default portrait image is shown.
Issue:
Opencast default paella-player only extracts one preview image from the mediapackage.
https://github.com/opencast/opencast/blob/develop/modules/engage-paella-player-7/src/js/EpisodeConversor.js#L384-L391
Paella sets the extracted preview image as the landscape image and retrieves the preview portrait image from the player config or uses a backup Paella player portrait image:
https://github.com/polimediaupv/paella-core/blob/main/src/js/Paella.js#L94-L99
To Reproduce
Steps to reproduce the behavior:

Go to https://stable.opencast.org/paella7/ui/watch.html?id=ID-3d-print (or publish a new mp)
Make the player window thinner than it is tall, by dragging in the sides of the window.

Expected behavior
The preview image continues to display.
Screenshots
Wide & short:

Tall & thin:

User environment:
All browsers, All devices

Our site resolved by adding this line to the Opencast EpisodeConversor.js
    ...
    manifest.metadata.preview = playerPreviewImage;
+   // TODO: For now, using same image for both landscape and portrait mode
+   manifest.metadata.previewPortrait = playerPreviewImage;

Issue is also mentioned in ticket https://github.com/opencast/opencast/issues/4368

Also mentioned in https://github.com/polimediaupv/paella-core/issues/19 which was the patch to create the Paella image fallback for portraitPreview so it the missing config for the portrait preview doesn't through an error.
For our site, it's unexpected to not have the same preview image when changing aspect ration. This might not be the case for other sites. Perhaps it can be a configuration option for EpisodeConversor to use the same playerPreviewImage for both preview and previewPortrait when there is preview portrait flavor in the mediapckage.


Support configurable Density in js and darwin targets

On uikit:
We set the density initially and when screen size changes (e.g. orientation change).
On mac and js:
We set the density only initally for now.
P.S. I didn't find how to change the screen density on iphone simulator.

Tested with widgets sample (images are layed out as expected):

Also, added a support for orientation change:
The red box has fillMaxWidth() modifier.  On the first screenshot, it takes the same width as when in portrait orientation.
On a second screenshot, the red box width is correct.

I didn't find how to change the screen density on iphone simulator

@dima-avdeev-jb, could you check, if it possible to change density on a real iOS device? I suppose, viewWillTransitionToSize should be called when we do that (on whole application is recreated, I am not sure), because it provides size in points.

I didn't find how to change the screen density on iphone simulator

@dima-avdeev-jb, could you check, if it possible to change density on a real iOS device? I suppose, viewWillTransitionToSize should be called when we do that (or the whole application is recreated, I am not sure), because it provides size in points.

There are no settings to adjust the scale on iOS devices. Only font size (But it's another task).

It seems, besides Text size, there are another features that can be configured. Created an issue for the future

Found bug with keyboard:
You can try to focus TextField and open Keyboard with Cmd+K shortcut.
https://user-images.githubusercontent.com/99798741/210410155-ce326795-8018-43f0-92e6-7aabb3b5efce.mp4


MNT: set MINIFORGE_HOME: C:\Miniforge

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #20.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Fixes #20

From @jaimergp:

Hey, we are optimizing the provisioning overheads on Windows (download Miniforge, install dependencies, etc), and we will change the default installation path from C:\Miniforge to D:\Miniforge. This will reduce the installation times significantly. We are submitting this PR here because this feedstock was redefining CONDA_BLD_PATH to the C: drive. In that case, we do not want to move Miniforge to C:; we want both paths to be in the same drive! This should not create new builds, just protect against the upcoming change in conda-smithy, once rerendered.
More information at https://github.com/conda-forge/conda-smithy/pull/2076

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

we still have to bump the build no right?

Not really, the packages won't be uploaded if they already exist. This is not changing how the packages are built yet, only defending against the upcoming conda-smithy release. You can merge with a [ci skip] commit too and that would be enough.

10 days have passed to I'm merging this with [ci skip] to unblock https://github.com/conda-forge/conda-smithy/pull/2076. Thanks!


Bugfix/tool 7 tag in bitbucket incorrectly creates

Added code to extract GitPushEvent's GitReference type (tag or branch) from reference string, formatted as:
refs/{type}/{branchName}
This is used by BitBucketResource to filter incoming push events, dropping tag pushes (with logging) and forwarding others to any listening services.
I also changed the logging level from info to debug in GitHubResource to be consistent with BitBucketResource.

All looks good

All looks good, feel free to merge in


Offer content is not transferred into created openBIS project

Describe the bug
When you create a project based on an existing offer in the offer manager, the resulting project does not seem to have the information copied properly from the offer.
To Reproduce
Steps to reproduce the behavior:

Create a project from an offer with the OM
Select the project in the projectbrowser
Look at the project description and project manager field

Expected behavior
All information are copied from the offer (objective, PM, customer info etc.)

concerns https://github.com/qbicsoftware/offer-manager-2-portlet/issues/277

Solved in #430


The node is not working on Windows

The node appear to not work on Windows (see attached flow).
SOX is working: I'm able to record on the command line using the following command
rec test.wav
I'm also able to successfully record using the "working" flow provided. When I start it, it will produce a "test.wav" file that skip the initial silence and stop recording after 5 second of silence.
`rec test.wav silence 1 00:01 0.1% 1 00:05 0.1%"
However, when the same command (at least I think I set all the parameters in the correct way) is issued using your node, it will issue a "msg.payload: isSilence" and stops recording immediately.
Testing Flow
[{"id":"1b84bae8.7c9985","type":"tab","label":"Flow 1","disabled":false,"info":""},{"id":"cfb56183.362e38","type":"Mic","z":"1b84bae8.7c9985","name":"Record","outputPayloadType":"stream","endian":"little","bitwidth":"16","encoding":"signed-integer","channels":"1","device":"default","rate":"22050","silenceTimeout":"1 00:01 0.1% 1 00:05 0.1%","exitOnSilence":"true","debug":"false","active":true,"x":429,"y":131,"wires":[["8973ef57.0b587","113d012d.3d0ed7"]]},{"id":"113d012d.3d0ed7","type":"file","z":"1b84bae8.7c9985","name":"Output file","filename":"z:\libri\byosh\experiments\test.raw","appendNewline":false,"createDir":true,"overwriteFile":"true","x":650,"y":226,"wires":[]},{"id":"8973ef57.0b587","type":"debug","z":"1b84bae8.7c9985","name":"","active":true,"console":"false","complete":"payload","x":1077,"y":236,"wires":[]},{"id":"4028bd6e.c367cc","type":"trigger","z":"1b84bae8.7c9985","op1":"start","op2":"stop","op1type":"str","op2type":"str","duration":"10","extend":false,"units":"s","reset":"","name":"start-stop","x":195,"y":190,"wires":[["cfb56183.362e38"]]},{"id":"d69a816f.15cbc8","type":"inject","z":"1b84bae8.7c9985","name":"Start test","topic":"","payload":"true","payloadType":"bool","repeat":"","crontab":"","once":false,"x":104,"y":61,"wires":[["4028bd6e.c367cc"]]},{"id":"99979c46.ae80f","type":"inject","z":"1b84bae8.7c9985","name":"Start","topic":"","payload":"true","payloadType":"bool","repeat":"","crontab":"","once":false,"x":174,"y":407,"wires":[["1af7e417.7179d4"]]},{"id":"1af7e417.7179d4","type":"exec","z":"1b84bae8.7c9985","command":"rec z:\libri\byosh\experiments\test.wav silence 1 00:01 0.1% 1 00:05 0.1%","addpay":false,"append":"","useSpawn":"true","timer":"60","oldrc":false,"name":"Record","x":616,"y":410.5,"wires":[["8973ef57.0b587"],[],["8973ef57.0b587"]]},{"id":"1e850018.5360b8","type":"inject","z":"1b84bae8.7c9985","name":"Stop","topic":"","payload":"true","payloadType":"bool","repeat":"","crontab":"","once":false,"x":172,"y":524,"wires":[["8ed69269.77669"]]},{"id":"8ed69269.77669","type":"change","z":"1b84bae8.7c9985","name":"","rules":[{"t":"set","p":"kill","pt":"msg","to":"SIGTERM","tot":"str"}],"action":"","property":"","from":"","to":"","reg":false,"x":397,"y":530,"wires":[["1af7e417.7179d4"]]},{"id":"bfee75bf.9f0688","type":"comment","z":"1b84bae8.7c9985","name":"Not Working","info":"","x":89,"y":31,"wires":[]},{"id":"e3648c86.d8189","type":"comment","z":"1b84bae8.7c9985","name":"Working","info":"","x":151,"y":375,"wires":[]}]

Hi aizzi, you set the Stop record on silence property to true.
Is it the same behavior when is set to false ?

Isn't it because Mic node's outputPayloadType is stream?
Why don't you set it buffer?
Otherwise, to work on stream output, I think there are some points that must be fixed:

File node does not support stream input

If input stream object, just stringify-ed stream object itself will be written.
Not the data the stream treats.

Do not connect Mic node to two or more child nodes

In my experiments, if stream output is divided to multiple child nodes,
only one child node can receive the stream's events such as data or end.
This means, the other child nodes can receive no data.

Not only streamObj.on(event), but also streamObj.read() or streamObj.pipe() does not work.

It looks file node and debug node are connected to Mic as Mic's children.
Disconnect the debug node.

I have also worried about the the latter stream behavior.
Perhaps, not only Mic node, stream-output nodes may need some special care, generally.


Fix some regressions following Infinispan 7.1.1.Final upgrade.

In particular, org.jboss.as.test.clustering.cluster.web.passivation.FineSessionPassivationTestCase takes way longer than it should.

Linux with security manager Build 916 is now running using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823

Linux Build 5937 is now running using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823

Windows Build 1047 is now running using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823

Linux with security manager Build 916 outcome was FAILURE using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823
Summary: Tests failed: 1 (1 new), passed: 839, ignored: 319 Build time: 0:05:10
Build problems:
Failed tests detected
Failed tests
org.wildfly.clustering.web.infinispan.session.SessionEvictionSchedulerTestCase.test: org.mockito.exceptions.verification.WantedButNotInvoked:
Wanted but not invoked:
transactionBatch.close();

Failure looks relevant.

Linux Build 5937 outcome was FAILURE using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823
Summary: Tests failed: 1 (1 new), passed: 2926, ignored: 313 Build time: 0:47:32
Build problems:
Failed tests detected
Failed tests
org.wildfly.clustering.web.infinispan.session.SessionEvictionSchedulerTestCase.test: org.mockito.exceptions.verification.WantedButNotInvoked:
Wanted but not invoked:
transactionBatch.close();

Windows Build 1047 outcome was FAILURE using a merge of 90a19ddaa5d57488cb5b30255ebe73ebf78d5823
Summary: Tests failed: 1 (1 new), passed: 2925, ignored: 313, muted: 1 Build time: 0:47:35
Build problems:
Failed tests detected
Failed tests
org.wildfly.clustering.web.infinispan.session.SessionEvictionSchedulerTestCase.test: org.mockito.exceptions.verification.WantedButNotInvoked:
Wanted but not invoked:
transactionBatch.close();

Windows Build 1055 is now running using a merge of 2dc3ed0d1f87396fe814cb35b1aefe07b268ca46

Linux Build 5945 is now running using a merge of 2dc3ed0d1f87396fe814cb35b1aefe07b268ca46

Linux with security manager Build 924 is now running using a merge of 2dc3ed0d1f87396fe814cb35b1aefe07b268ca46

Linux with security manager Build 924 outcome was SUCCESS using a merge of 2dc3ed0d1f87396fe814cb35b1aefe07b268ca46
Summary: Tests passed: 840, ignored: 319 Build time: 0:06:35

Windows Build 1055 outcome was SUCCESS using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935
Summary: Tests passed: 2927, ignored: 313 Build time: 0:46:00

Linux Build 5945 outcome was FAILURE using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935
Summary: Tests failed: 1 (1 new), passed: 2925, ignored: 313, muted: 1 Build time: 0:47:17
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.smoke.webservices.WSTestCase.testManagementDescriptionMetrics: java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)

Windows Build 1057 is now running using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935

Linux Build 5947 is now running using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935

Linux with security manager Build 926 is now running using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935

Linux with security manager Build 926 outcome was FAILURE using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935
Summary: Tests failed: 1 (1 new), passed: 839, ignored: 319 Build time: 0:06:23
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.smoke.webservices.WSTestCase.testManagementDescriptionMetrics: java.lang.AssertionError: null
	at org.junit.Assert.fail(Assert.java:86)
	at org.junit.Assert.assertTrue(Assert.java:41)

Linux Build 5947 outcome was SUCCESS using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935
Summary: Tests passed: 2926, ignored: 313, muted: 1 Build time: 0:37:35

Windows Build 1057 outcome was SUCCESS using a merge of 15049eb0542db5ad335853a4d9a42604b7dd0935
Summary: Tests passed: 2927, ignored: 313 Build time: 0:46:19


submit an edit record doesn't direct user to the record page

https://dev.gpcrconsortium.org/chaise/recordedit/#1/assets:cpm/uri=%2Fhatrac%2Fassets%2FUSC%2Fcpm%2F2017%2F01%2F78f585c2ff5e597f80c67e72e40275ea2644a0861d334b1087fec9261b6fbeef.csv@sort(last_modified_timestamp::desc::,id)

Make modification to the record
click submit, nothing happens but the record has already been modified
click submit again, an error is thrown at the user
If you check the record, it is clear that it has already been modified.

When I try to reproduce this bug I see the following error:
ERROR: insert or update on table "cpm" violates foreign key constraint "cpm_last_update_user_guid_fkey" DETAIL: Key (last_update_user_guid)=(https://auth.globus.org/6ccebd18-ef90-11e5-8e41-cb8595f578a7) is not present in table "person".
@hongsudt, is GPCR using some form of row level security that checks if a user is in the person's table to be able to edit or create entities?

This issue can no longer be reproduced. There are some other issues that are still present, but those are addressed in other github issues. Closing this issue.


Created dogs.md

Add ASCII art for dogs for issue #1

Thanks for PR
Please add dogs.md file to the index (add link in README to the file)

Added dogs.md to Readme.md in #7

Hey,
there's something wrong with the PR's. First, you shouldn't have made another PR with fixes. You could simply commit and push on patch-1 branch (here). Second, if you take a look at your patch-2 branch, there are no memes.md and dogs.md files. Probably you haven't added them to commit. I will close another PR as it is duplicate and you try commiting the fixes again.
Thanks


Fix offline not starting after rehydrate

Order matters, swapping two words could fix a critical bug 🎉
The offline middleware wasn't aware of the outbox actions after rehydrate until another action was dispatched, so when the app starts and it doesn't dispatch any other action the offline wouldn't start and the actions would remain there forever.
Now the middleware is applied first and all of the actions go through it, including the rehydrate one.
Reference: https://redux.js.org/api-reference/compose
Key: Composes functions from right to left.

I can confirm this works. Adding a quick example that can help peeps dropping by :)
Change is the order of the arguments in composeEnhancers()
My store creation before (actions not firing before other action is fired):
const store: Store<IAppState> = createStore(
        offlineEnhancer.enhanceReducer(rootReducer),
        initialState,
        composeEnhancers(
            applyMiddleware(offlineEnhancer.middleware as any, epicMiddleware),
            offlineEnhancer.enhanceStore,
        )
    );

My store create after (offline queue now immediatly dispatches)
   const store: Store<IAppState> = createStore(
        offlineEnhancer.enhanceReducer(rootReducer),
        initialState,
        composeEnhancers(
            offlineEnhancer.enhanceStore,
            applyMiddleware(offlineEnhancer.middleware as any, epicMiddleware),
        )
    );

Thanks for a great solution 👍

Glad you got it working! Thanks for the example!! This repo is not really related to @redux-offline, it might be worth it to PR the docs in Redux-Offline to add an Epics example such as the one we have for sagas there! Or add your solution to a related issue, such as the ones linked in the https://github.com/redux-offline/redux-offline/pull/229 PR.

@sorodrigo That's a great idea. I have never made a PR to open source projects before, so didn't really dare to do it. But your suggestion makes 100% sense to it. I'll make a PR tomorrow :) (and add my solution to a couple of issues).


sesman does not log anything if authentication fails

This is what i get when entering a wrong username:
==> /var/log/xrdp-sesman.log <==
[20190605-20:01:26] [INFO ] A connection received from <IP_ADDRESS> port 33274

==> /var/log/xrdp.log <==
[20190605-20:01:26] [INFO ] xrdp_wm_log_msg: sesman connect ok
[20190605-20:01:26] [DEBUG] xrdp_wm_log_msg: sending login info to session manager, please wait...
[20190605-20:01:26] [DEBUG] return value from xrdp_mm_connect 0
[20190605-20:01:28] [INFO ] xrdp_wm_log_msg: login failed for display 0

==> /var/log/xrdp-sesman.log <==
[20190605-20:01:28] [DEBUG] Closed socket 8 (AF_INET <IP_ADDRESS>:3350)

# grep -A4 Logging /etc/xrdp/sesman.ini
[Logging]
LogFile=xrdp-sesman.log
LogLevel=DEBUG
EnableSyslog=1
SyslogLevel=DEBUG

According to the code sesman should log a lot more, but it doesn't and i don't know why, help please?

[20190822-08:26:09] [INFO ] A connection received from <IP_ADDRESS> port 41390
pam_authenticate failed: Authentication failure
[20190822-08:26:11] [DEBUG] Closed socket 8 (AF_INET <IP_ADDRESS>:3350)

It is logged when running sesman in foreground. It should be logged to logfile, too.


Merge the DotNet_###Test "platforms" back into the right place

I originally added these platforms to prevent the .NET components from
building when you built the entire solution, and to prevent them from
building in CI.
It turns out that managing an extra thousand project-platform-config
triples is an absolute pain, and that we should have been building
these things in CI the entire time. So.
This should make life a lot easier.
As a bonus, this PR enables the WPF test harness to build for ARM64.

@msftbot merge this in .01 days

@msftbot merge this in .01 minutes

Interesting, that x86 failure. I'll dig.


Win32 exception in Vlc.DotNet.Core.Interops.dll

I have an issue about Vlc.DotNet.
Summary:
What:
http://prntscr.com/ke9xdf
Error in text:
Exception thrown: 'System.ComponentModel.Win32Exception' in Vlc.DotNet.Core.Interops.dll
An unhandled exception of type 'System.ComponentModel.Win32Exception' occurred in Vlc.DotNet.Core.Interops.dll
%1 is not a valid Win32 application
When:
The moment this line is called the application breaks.
vlcControl.MediaPlayer.EndInit();
How:
To reproduce the error, i will upload the full code to my github.
https://github.com/metalglove/AnimeSync
using the example code from this:
https://github.com/ZeBobo5/Vlc.DotNet/blob/master/src/Samples/Vlc.DotNet.Wpf.Samples/MainWindow.xaml.cs
Generic information

Vlc.DotNet version : <IP_ADDRESS>
Vlc.DotNet project used : WPF
libvlc version : http://download.videolan.org/pub/videolan/vlc/last/win64/
.net version : v4.6.1
Project language : C#
Project build architecture : AnyCPU
Operating system : Windows 10

In the Project properties, in the Build tab, is the "prefer 32 bits" box checked?
Does it work with a x86 libvlc?
By the way, don't commit libvlc in a folder, it's really huge. I'd recommend that you get it as described here : https://github.com/ZeBobo5/Vlc.DotNet/wiki/Getting-started#with-nuget-

The prefer 32 bits box is checked.
Upon trying 32 i get a other error.
System.AccessViolationException: 'Attempted to read or write protected memory. This is often an indication that other memory is corrupt.'
And yeah i should of made a ignore file for that.

Alright, i have got it working.
But how do i know what the required plugins are for this to work?

It depends of what you are trying to do with your application. For example, if you are just playing files, you don't have to redistribute the plugins that reads media from http, rtsp and other protocols.

What would i need if i were to only do it from http?

I wouldn't touch the codec folder too much, but you can probably remove most of the access plugins except http, also keep directx for hardware acceleration.
For other folders, try to delete plugins and see if it's broken. it's the easiest way to know for sure

Okay, thanks for your help!
i hereby close this issue as it is resolved.

"Alright, i have got it working." Wonderful. How?

Probably by following the wiki /samples and not trying to load a x86/x64 version in the other type of process.


kernal died restarting in spyder

Description
What steps will reproduce the problem?

Every time I start Spyder (I can't run any code)

What is the expected output? What do you see instead?
I would expect IPython to function normally.
Please provide any additional information below
I have tried to resetting Spyder to default and reinstalling Anaconda, but I get the same error code.
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.23.1 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.4 (OK)

Also, your version of Spyder is over half a year out of date. You should update with conda update spyder from Anaconda prompt. Furthermore, as @dalthviz mentioned, please try the steps listed in the Spyder Troubleshooting Guide, specifically the Basic First Aid and Emergency CPR sections, which resolve the great majority of Spyder installation issues.

Closing because of lack of response.


add method adjustAjaxConfig to allow customization of ajax request

Customization the ajax request is necessary if one wants to use contentType: 'application/json' for example.

I think it would be simpler to introduce a second argument options to the makeRequest method that would get merged with the default options. That way you can simply override makeRequest in a custom authenticator:
makeRequest(data) {
  return this._super(data, {
    contentType: 'application/json'
  });
}

@kbullaughey: any updates on this (see my above comment)?

That occurred to me, but makeRequest is called automatically by authenticate, and it would be strange to pass an extra parameter there, which would in turn get passed to makeRequest.  I think it depends whether we see this as a configuration option or a behavior. If you'd like it to be a configuration option, where would be the best place to put it? Should it be devise-specific or available to the other authenticators? If it's conceptually more like a behavior, I think achieving it by overriding a method is a natural way to accomplish it, and this seems to be practiced quite broadly in ember (e.g., model hooks, lifecycle hooks, event handlers and such). I'm happy to implement whatever solution you think would fit well.

Yes, the solution I point out above is based on your specific devise authenticator overriding the default one. In your custom authenticator you'd override the makeRequest method, passing on any XHR options you want to set:
makeRequest(data) {
  return this._super(data, {
    contentType: 'application/json'
  });
}

That way the content type would be set for all requests that the devise authenticator makes (if the makeRequest method accepts a second argument with request options as I said above).

Ah. That works. I guess we can count on anyone with this particular problem doing the same, and don't need to introduce any additional public API for it.

I'll make a separate PR

PR is issue #886.


Refactor remove sleep on db update failure

Pull Request Test Coverage Report for Build 551121366

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

Totals

Change from base Build 551098579:
0.0%

Covered Lines:
1430

Relevant Lines:
5330

💛  - Coveralls


experiment_id: Correction to 3 geoMIP experiment parents and start years

Ben Kravitz has approved the following changes to experiment_id info. associated with 3 experiments: G6solar, G6sulfur, and G7cirrus.
For each of these replace parent_experiment_id="historical" with parent_experiment_id="ssp585" and replace start_year of 2015 with 2020.

These changes are now in place, please take a peek at CMIP6_experiment_id.html and let us know of any further tweaks required
@taylor13 @glevava @bkravitz

I think it looks good.  Thanks for your help!


python 3 support

Add python 3 support.

Addressed in v1.0 branch.

@alimanfoo, just a note of thanks for addressing this!

@erictheise, no problem!


Enterprise Architect

The Aruba models listed above are all older models. Newer models (eg. 310 series) need to be certified and added to this list.

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

ID: ba1c5bd5-da94-25ac-4d42-d2573235fa84
Version Independent ID: cda59715-81fe-8e04-13b7-03fa269d12e1
Content: Skype for Business - Networking Wi-Fi
Content Source: Skype/SfbPartnerCertification/certification/networking-wifi.md
Service: skype-for-business
GitHub Login: @jambirk
Microsoft Alias: jambirk

@paul-jensen this page reflects certifications that have been completed. Currently the Wi-Fi networking certification is no longer active, therefore newer models will not be added to the page.

@dougand  Thank you very much for the contribution and sharing this explanation. @paul-jensen
Hope this comment is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


RxMobius: Prefer package visibility over public for internal classes

This is the same as #102, it can't be done until we make a new major version. At which point, we're planning on moving everything to a new namespace so that we don't in fact break existing usages.

Thanks for explaining will close the PR

@pettermahlen from the talk, should the classes then be called MergedTransformer2, and so on?
and use the 2 implementation which is package protected, 🤔 ?

I would think the point is that we don't want MergedTransformer in the publicly exposed namespace, so in the new namespace (com.spotify.mobius2, maybe), we won't include it. So in terms of the talk, I'm expecting us to publish a new artifact and new namespaces, and leave the current Mobius 1 artefact and namespaces around 'forever'. That means that Mobius 1 and 2 will be able to coexist in the same binary, and we won't be taking away functionality from users, while still publishing a new thing that we think is better.


Implement missing Contributor Hub text, and modify existing content

Implement/modify text in the following categories within the Contributor Hub:

[x] Open Positions & Bounties:

Implement preceding document

[x] Employment:

Rename to "Contributor Onboarding".

Translate "Contribution Types & Time Commitments" to its own category

Translate "Trail Period" to its own category

[x] Worker Compensation:

Alter font to default in "Experience Guidelines"

Fix compensation typo for level 5 on chart

[x] Performance and Career Development:

Remove until agreeance is made by Contributor Experience Squad

[ ] How we Work:

Capitalize "we"

[ ] Equal opportunity:

Work with Contributor Experience Squad to develop content

[ ] Decentralized Communication Technologies:

Update Gov 2.0 to passed xDai proposal (include ann collateral?)

[ ] Standards of Conduct:

Keenan will review the contributor covenant. If it is a desired addition, will need to pick what we like and run past the community - and then prepare content in its place. Otherwise, will be removed.

[ ] xDai worker proposal update

Employment:
Translate "Contribution Types & Time Commitments" to its own category
Translate "Trial Period" to its own category

Upon review, it makes sense to keep these together in the new "contributor onboarding" category. Removes additional clutter, and can be hyperlinked directly. (+ they both fit the onboarding topic)


Question: Did you guys ever experience a crash when extending ItemAxe?

Hmmm... You need this for your mod, right? ^^
Do you have any crash-reports? If yes, they could help.
We didn't experience such crashes

sorry for replying so late, it's cause I figured out my problem

Ok, good to hear :)


TOMEE-2883 Translate to Portuguese:examples/projectstage-demo

Linkt to Jira: https://issues.apache.org/jira/browse/TOMEE-2883

thank @hbelmiro for review.


Cannot find module 'meteor/accounts-base'

meteor create testapp --bare
cd testapp
meteor npm i -S graphql graphql-load subscriptions-transport-ws apollo-live-server apollo-live-client apollo-client apollo-cache-inmemory apollo-link apollo-link-http apollo-link-ws express apollo-server-express uuid graphql-subscriptions body-parser graphql-tools graphql-type-json
meteor add cultofcoders:apollo
meteor

it looks like it use accounts package defaultly

@crapthings are you sure you're on the latest version ?

Yes, you are correct. I'll work on removing this dependency. For now just add it to play with it.


Tidy spacing in generated migrations.

# -*- coding: utf-8 -*-
from __future__ import unicode_literals

import datetime
from django.conf import settings
from django.db import migrations, models

class Migration(migrations.Migration):

    dependencies = [
        ('clients', '__first__'),
        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
    ]

    operations = [

vs
# -*- coding: utf-8 -*-

from __future__ import unicode_literals

import datetime
from django.conf import settings
from django.db import migrations, models

class Migration(migrations.Migration):
    dependencies = [
        ('clients', '__first__'),
        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
    ]

    operations = [

(MIght be tempting to further separate imports that start with "import " for ultimate niceness.)

Two newlines after imports is PEP8, so this change is wrong.
Also, newline before dependencies seems more clearer IMO.
So, I think these changes are unnecessary.

I agree with @coldmind . Thank you for the patch @lamby , though.

Sadly generated migrations cannot be customized to everyone's favorite coding style. That's the tradeoff of generated code. No matter what we do, some would prefer slightly different formatting.


TextField - iOS - Unable to click where the Title is shown

Hi there!
I'm currently using UraniumUI.Material nugget package with version 2.5.6 for a .NET 7 MAUI project.
The TextField works fine on Android, but I'm unable to click on some parts of the TextField on iOS. In the screenshot, clicking on the area in the red box will not focus the field.

Just wanted to check if there was any plan to have this fixed soon. It's the only issue preventing me from releasing an app!

I couldn't reproduce on iOS with default TextField parameters, Can you share your TextField usage code clock to help me to reproduce?
Normally, Title has InputTransparent="True" but in some cases it may not work

Thank you for your reply!
Setting InputTransparent to True  makes the field completely uneditable. And setting it to False has the same issue mentioned in this ticket.
This is the whole file
<?xml version="1.0" encoding="utf-8" ?>
<ContentPage
    xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
    xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
    xmlns:d="http://schemas.microsoft.com/dotnet/2021/maui/design"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    xmlns:material="http://schemas.enisn-projects.io/dotnet/maui/uraniumui/material"
    mc:Ignorable="d"
    x:Class="MySolution.Views.LoginPage"
    Shell.TabBarIsVisible="False"
    BackgroundColor="Black"
>
    <ContentPage.Content>
        <StackLayout Orientation="Vertical">
            <StackLayout Padding="15,0,15,0" VerticalOptions="Center" BackgroundColor="Black">
                <material:TextField
                    Title="Your email"
                    InputTransparent="True"
                    HorizontalOptions="Fill"
                    AccentColor="LightGray"
                    TextColor="White"
                    TitleColor="LightGray"
                    BorderColor="LightGray"
                    InputBackgroundColor="DarkGray"
                    Text="{Binding LoginRequest.Email, Mode=TwoWay}"
                />

                <material:TextField
                    Margin="0,10,0,0"
                    Title="Password"
                    HorizontalOptions="Fill"
                    IsPassword="True"
                    AccentColor="LightGray"
                    TextColor="White"
                    TitleColor="LightGray"
                    BorderColor="LightGray"
                    InputBackgroundColor="DarkGray"
                    Text="{Binding LoginRequest.Password, Mode=TwoWay}"
                >
                    <material:TextField.Attachments>
                        <material:TextFieldPasswordShowHideAttachment />
                    </material:TextField.Attachments>
                </material:TextField>

                <Button
                    Margin="0,25,0,15"
                    VerticalOptions="Center"
                    Text="LOG IN"
                    BackgroundColor="Red"
                    HeightRequest="40"
                    Command="{Binding LoginCommand}"
                />
            </StackLayout>
        </StackLayout>
    </ContentPage.Content>
</ContentPage>

Hi @enisn ! Any update on this issue?

I also see this issue on iOS/iPadOS, and it happens on WinUI as well. If you click the Placeholder text, the control does not get focus.


Request to improve: recommend using locate to determine path to libclang.so on Linux

Is there any particular reason for not using "locate libclang.so" instead of "find ..."? I believe that there are some cases in which this fails, specifically libclang is not in $PATH, but then the "find ..." can be used as a fallback. This is due to the long time that find takes to process all the entries.

I cannot find my libclang.so by locate command.
I don't think it is not useful.
% locate libclang.so
[1]    6698 exit 1     locate libclang.so
% find /usr/lib -name libclang.so
/usr/lib/llvm-3.8/lib/libclang.so


#167398979 built user sign in endpoint

What does this PR do?
It creates a user sign-in endpoint.
Description of Task to be done?

It adds a user sign-in controller in the auth controller file.
It validates the information submitted during sign-in such as email and password.
It adds a route for user sign-in.

How should this be manually tested?

Start the Server and POSTMAN.
Browse the following endpoint route: localhost:3000/api/auth/signin.
Make sure that you have an account or create one in advance.
Submit the required information - email and password - with POST method.
Check if the validations work once you don't submit any of the required information.
After passing the validations, you should receive a 200 HTTP Status Code along with the information related to your account such as firstname, lastname, and email.

Looks good, and after testing this endpoint, It is working well.

@Placide, when the user ties to create an already existing user, the status code is 409 for conflict, not 400, that's a bad request.
The password validation is not strong enough. I think that forcing the user to use is strong password is better by including upper and lower cases and symbols in the password.
When the user uses invalid credentials, the status code is 401, not 400.

@Placide, when the user ties to create an already existing user, the status code is 409 for conflict, not 400, that's a bad request.
The password validation is not strong enough. I think that forcing the user to use is strong password is better by including upper and lower cases and symbols in the password.
When the user uses invalid credentials, the status code is 401, not 400.

@Placide, when the user ties to create an already existing user, the status code is 409 for conflict, not 400, that's a bad request.
The password validation is not strong enough. I think that forcing the user to use is strong password is better by including upper and lower cases and symbols in the password.
When the user uses invalid credentials, the status code is 401, not 400.

Thanks! I'm gonna implement that.

Everything works fine, you can only implement Eric's feedback

I have implemented all of the feedback!


Feature/56 copy scenes to build settings/dev  (Do After for PR#24)

DEPENDENCIES

https://github.com/MoralisWeb3/web3-unity-sdk/pull/21 must be merged first before proceeding below...

CONTEXT
The Moralis "API Examples" has 15 scenes. There will be more in the future. Before the user can use the full scope of the demo, all scenes must be added to the build settings. This ticket adds a menu option to help.
TICKET

https://gabronickwontdiefromcovid.com/unity-squad/ethereum-unity-boilerplate/-/issues/56

CHANGES

SDK, Add a scene related utility
Samples~/Moralis API Examples/, Add the menu item. Rename a competing class.

TESTING

Target: Standalone. Tested with success in Unity Editor.

RESULT
See pic...

This PR is now redundant to https://github.com/MoralisWeb3/web3-unity-sdk/pull/36 and will be closed, unmerged.


Any chance of having .NETStandard support with this?

I love the changes you've made but it's not .net standard compliant.  Any chances getting that feature in here?
Thanks!

Closed


Final PR process

On the PR for the release, ensure that the following is done:

check for TODOs
proofread all text

update changelog


npm package required. we do not use bower, may you please publish over npm ?

Available at https://www.npmjs.com/package/realtime-messaging


lowercase 'datatype: dateTIME' for certain dimension types in base looker output

This line should be lowercase:

https://github.com/LewisCharlesBaker/droughty/pull/73 resolves this issue


Allow to specify client certificate path explicitly

As per discussion in Slack: https://prestosql.slack.com/archives/CFPVDCDHV/p1589924393017200
Currently presto.dbapi.connect api does not have a parameter to explicitly specify client certificate path to connect with SSL/TLS. Client cert should be explicitly configurable.

A current workaround, from referenced slack thread:

After creating the connection object I do
conn._http_session.verify = '/path/to/cert'

before creating a cursor.


Extend validation

Hi Jonathan,
Right now I'm trying to add dynamic validation of a cropped image. How can I extend the CropValidator or the AbstractImageValidator to achieve this?
The image sizes are managed in a separate yaml file and I would like to only declare them once, not twice, for a liip filter and the crop validation. I wrote a provider to retrieve the image sizes from the yaml file. I then should pass it to the configuration of the validator in some way.
Hope you can help!
Tim

Can you post an example of your custom yaml configuration file for image sizes ?
Is it really an issue to have this information duplicated ? From my experience, these kind of thing do not change a lot. You will lose a lot of time for this optimization with little benefit in my opinion.

If you already have all information mandatory for the crop validator in a service, you can inject it directly in the service by overriding the default one :
jb_fileupload.crop.validator:
        class: "%jb_fileupload.image.crop.class%"
        tags:
            - { name: jb_fileuploader.validator, alias: Crop }

You can either completly override this validator service to inject your provider as an argument or create a custom compiler pass to add your provider as an argument in the default jb_fileupload.crop.validator and override the class to change the implementation. I had a blog post on the last solution (http://blog.bouzekri.net/2015/02/02/surcharger-n-importe-quel-service-symfony/) but it is in french ...

Thank you for your fast reply! I think for now I will just duplicate the information, it's the easiest way. It was more an optimisation. Right now I'm diving into Symfony, I will return and try your suggestions when I'm more advanced ;). Thank you for your help!


Remove timer from extended non-conformance tests

Now run by prow: https://github.com/openshift/release/pull/665.
@stevekuznetsov

@bbguimaraes you need to run sjb/generate.sh


How can we customize export of documents to include non-column related data?

Recently, we've been trying to add the search performed (text search + filters) to the export file obtained when downloading the documents' metadata.
Unfortunately, we've come to realize that this endeavor was not possible due to the fact that the download file is prepared in the backend, and only the final file ready for download is sent back to the frontend.
We thought that we could intercept the file before it being downloaded and change the filename through the header 'content-disposition' but it was unsuccessful.
Do you have any idea how this could be implemented with the existing export query component or is this something that we would need to create a new component to implement?

Hi Guillaume,
The front-end is probably the easy part. You could modify the existing component to your needs.
The difficulty is that there is currently no way to customize the backend for the export query... Perhaps you could implement your own export endpoint via plugin?

Hi Eric,
I understand, I'll talk to my team and see what we decide to do. I'll close this issue for now since your message is answering my question.
Thanks!


remove include/nana/config.hpp from git?

Is include/nana/config.hpp computer generated? If so it should be removed from git repo and added to the gitignore. My machine generates a different one after the build and shows up in git status as modified.
On a related note, is there a reason why fixes are not merged into master right away? I understand new features may go into a development branch first but I don't understand why fixes for the master are not on the master right away. If a new user downloads the repo and tries to build he could run into the same problem again unless he knows to check out the hotfix branch instead, which is unlikely. If you want to have versioned releases I suppose you can always create named tags for the corresponding commits.

This file defines default configurations, the library can be built by using project solution coming with the repo. If you use CMake to generate a new solution/makefile, it will be overwritten by the CMake generated one.
Master branch always keeps the status of the latest release. Some fixes may be deprecated in next release due to big improvement of feature in other branch. This method can make thing easier to maintain the repo.

Is there a way to make config.hpp`` not show up in my git status? It would be nice to be able to do git add .without addingconfig.hpp```.
I still think it is easier if fixes to master are merged into master right away. Reading the mail list I can already see a few cases where people are reporting issues that were fixed in the hotfix branch already, including issue #77 reported by me. Are you concerned about the merge when you merge with a development branch off the release commit? Would git rebase help in that case? You can use rebase to always keep a linearized history in master. See the workflow here which I know worked well.


If getValue is declared as an extension method ktlint thinks its unused

Example:
import com.natpryce.konfig.PropertyGroup
import com.natpryce.konfig.getValue
import com.natpryce.konfig.stringType

interface EventConfiguration {
    val url: String

    companion object {
        private object rabbitmq : PropertyGroup() {
            val url by stringType // This is where getValue is used.
        }
}

This is how getValue is defined:
operator fun <G : PropertyGroup, T> ((PropertyLocation, String) -> T).getValue(group: G, property: KProperty<*>) =
        group.key(property.name, this)

operator fun <SCOPE, T> ((PropertyLocation, String) -> T).getValue(scope: SCOPE?, property: KProperty<*>) =
        Key(property.name, this)

Kt-lint thinks that import com.natpryce.konfig.getValue is an unused import so you have to disable the check using // ktlint-disable.

@JLLeitschuh Thank you. Fixed in 0.8.2 (Maven Central sync might take up to 25min).

While you are at it you should fix it for setValue


Support multiple load_balancer entries

Hi all, here's a scenario , let me know what you think of it

It is good to have split horizon DNS for a CF instance, i.e. on the devbox you have a named/bind9 server that resolves *.cf.azurelovecf.com to <IP_ADDRESS> inside the VNet, and the Public IP outside the VNet.   This avoids hairpin NAT routing to the load balancer public IP address which sometimes can be slower and flaky than remaining within the VNet.
In a high availability enterprise scenario, we should use an Azure Load Balancer created ahead of time and have it load balance to two HAproxy instances annotated with the load_balancer cloud property.
However an azure load balancer can only have a Public OR a Private IP address, it cannot have both.
Therefore I think it makes sense to be able to have two (or more?)  load_balancer entries, one for a Private load balancer and one for a Public load balancer.

Let me know if this is a reasonable enhancement.  Right now as workaround, I'd need to create two resource pools and jobs for haproxy.

@svrc-pivotal It is a good idea. Let us do some investigation.

Related to https://github.com/Azure/azure-powershell/issues/2042

@svrc-pivotal Now Azure has supported to create one external LB and one internal LB in the same NIC. Could you have a try?

@svrc-pivotal Need to correct. Azure can support to create create one external LB and one internal LB in the same NIC but the port rules cannot be same. We are still investigating.

There is a question and answer at this link about multiple load balancers, saying that:

Azure NICs can participate in multiple load-balancer: one external and one internal.
LB rules on the external and internal load-balancers cannot use the same backend port.
Only the primary NIC can be used in load-balancer backend pools.

I had tried to create rule tcp 22:22 for both external and internal load balancers and assigned those two lbs to the same VM, I got error code "RulesUseSameBackendPortProtocolAndIPConfig" when creating the VM.
Hi @svrc-pivotal, are you expecting to use same backend port in load balancers in CF? Azure does not support such feature now.

Hi @svrc-pivotal ,
Now in Azure the secondary NICs can also be used in LB backend pools, so this feature is doable. However, the secondary NICs can't be probed by LB because by default the outbound traffic will go to gateway of primary NIC. In order to make LB work with the secondary NICs we need to configure a policy route in the VM, which means we need a bosh release to configure the network additionally. I had a PR in networking-release, trying to implement the policy routing for this scenario, but it is not reviewed / merged yet.
It looks like this is not a common scenario, and the issue has last for a long time. Not sure if you have already solved the issue by any other methods, so my question is - do you still need this feature?

@gossion At the moment we deploy separate VMs if we need separate load balancers.  I have at least two large Azure customers deploying separate VMs for internal vs. external facing HAproxies for example.
I guess my point was that Azure is the only cloud I know (out of AWS, OpenStack, GCP, vSphere) that requires multiple NICs for multiple load balancers, so this should be a common case.   Multi-NIC could be a workaround but seems too early.
You can close this issue if it doesn't look like Azure LBs will ever allow a single NIC to expose to multiple load balancers, as it's a larger concern than just this CPI :)

This should work with standard LB skus and maybe should be revisited, especially considering SNAT with standard LBs requires a public LB with unused inbound rules to denote SNAT outbound IPs/ports (scenario 2 here - https://docs.microsoft.com/en-us/azure/load-balancer/load-balancer-outbound-connections )

Not stale - waiting on updates from the BOSH team.

this issue should be fixed with PR #638 which was released with v37.6.0.


operator [N] lms-moodle-operator (0.4.0)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/retest

/merge possible

/merge possible

/hold cancel

/merge possible

/merge possible

@mporrato, @tomasbakk, this operator has as dependencies the other four operators which are being added in previous PR: Moodle Operator, Postgres Operator, Keydb Operator and NFS Operator. I suppose pipelines in this PR will fail until those are merged.

/retest

/merge possible

/merge possible

/merge possible

/merge possible

@mporrato, @tomasbakk, this operator has as dependencies the other four operators which are being added in previous PRs: Moodle Operator, Postgres Operator, Keydb Operator and NFS Operator. I suppose pipelines in this PR will fail until those are merged.

Thanks. Now all pipelines have passed

/retest

/merge possible

/merge possible

/merge possible


Docker development setup

Disclaimer: I would implement this myself (as I did everything else in the project thus far), but I've spent a few days of googling/stackoverflowing with subsequent failed attempts and barely any progress. I'm sure it's not as complicated/obscure as this somehow became to me, and I presume my proposed solutions are incomplete and/or misguided.
Problem
The project's Docker setup is not fit for development: There's no live-reloads/compiler-watches, and therefore the only way to view any edit is to subsequently rebuild the affected images & relaunch containers.
Solution Criteria

Automatic updates, from host files to container files, for all relevant files

Relevant files include:

That end's (i.e frontend/backend directories) entire src subdirectory
The entire shared directory

Relevant files exclude:

node_modules subdirectories
archive subdirectories (exclusion preferred, not critical)

Automatic recompilation following changes in/of source files, updating respective output (dist) contents

Both frontend & backend compilers have a --watch flag which fulfills this criterion, and both have their full command saved as npm run buildDev in their respective package.json

Automatic server restart upon changes in compiled server files

Available with the nodemon npm packagePossibly helpful command I ended up withnodemon -e ts --exec \"tsc && node ./dist/backend/src/server.js\"

No need for server restarts to view frontend-only changes

Adhere to Docker best-practices (e.g single dockerfile per folder, multi-stage builds for minimal images, etc)

Implementation Suggestions

Create two independent purpose-written docker-compose files

Seems necessary (excluding the "independent" part, that's just my preference) considering the setups' differing requirements: The former setup uses bind-mounts, the following step requires different property values

Utilizing multi-stage builds, split each dockerfile into dev/prod setups and target them using the target build property in each setup's (or at least the former's*) respective docker-compose file

* I've seen conflicting opinions about which setup, dev or prod, should come first. As of writing this I'm leaning towards dev, but open to being convinced otherwise.

progress made at https://github.com/EvAvKein/Distillogue/commit/fae791b82d413af44845ddc6d0ce37dfe36e6c83

implemented ^^
'works on my machine' :P
resolved the WSL bug, can't imagine why it wouldn't work on linux (but will test anyway when switching my laptop's OS soon), don't have a mac to test it with.
...and any further bugs would probably merit a distinct issue anyway


regression-testadd nested data type to create table

Proposed changes
add nested data type case to product table statement
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Curious behaviour of quat, given three arguments

The signature of quat, for three arguments, is [^Float x ^Float y ^Float z]. This would lead the innocent programmer to believe that numbers are expected, but in fact they're not.
jme-clj.core=> (quat 1.0 2.0 3.0)
Execution error (ClassCastException) at jme-clj.core/quat (core.clj:225).
java.lang.Double cannot be cast to com.jme3.math.Quaternion

jme-clj.core=> (quat (quat 1 2 3 4) (quat 4 5 6 7) 2)
#object[com.jme3.math.Quaternion 0x43700dc8 "(7.0, 8.0, 9.0, 10.0)"]

This is because the arity three constructor for Quaternion expects two quaternions and a float.
I suspect what you intended could be achieved by:
(defn quat
  "Returns a [Quaternion](https://javadoc.jmonkeyengine.org/v3.4.0-stable/com/jme3/math/Quaternion.html)
   object. If no args are passed, the object will
   have the values <0, 0, 0, 1>; if three arguments `x`, `y`, `z` are passed, such
   that all are numbers which can be cast to floats, the object will have the values
   <`x`, `y`, `z`, 1.0>;  if four arguments `x`, `y`, `z`, `w` are
   passed, such that all are numbers which can be cast to floats, the object
   will have those values.

   An exception will be thrown if any argument is passed which is not a number, or
   is too large to be cast to a float.
   "
  ([]
   (Quaternion.))
  ([^Float x ^Float y ^Float z]
   (Quaternion. x y z 1.0))
  ([^Float x ^Float y ^Float z ^Float w]
   (Quaternion. x y z w)))

@simon-brooke thanks for reaching out! It seems that I did not test it well enough, I'm happy to accept PR for that one 🙌🏻

OK, I'm working on a lot of documentation. I'll drop you at least one pull request later this weekend. Would you prefer

one pull request with just code fixes and a separate one with documentation, or
just one pull request?

One pull request should be fine I think 👍🏻

OK, I'm running slower than I'd like, the weekend's over. But I'm working on it. You can get a view of progress on my fork, which is here.

Wow, looks so cool. It seems that you put so much effort!

@simon-brooke it seems that linting is broken on master after the PR, could you check and fix it, please?

What are you seeing/which linter are you using? If the problem is
At src/jme_clj/core.clj:null:
Consider using:
  and
instead of:
  #(and %1 %2)

(reported by Kibit), the following patch fixes it. It's also better, more idiomatic code. Shall I submit a new pull request?
illuminator:jme-clj simon$ git diff
diff --git a/src/jme_clj/core.clj b/src/jme_clj/core.clj
index 9307b96..156eec5 100644
--- a/src/jme_clj/core.clj
+++ b/src/jme_clj/core.clj
@@ -307,7 +307,7 @@
    (Quaternion.))
   ([x y z]
    (cond (and (instance? Quaternion x) (instance? Quaternion y) (number? z)) (Quaternion. x y z)
-         (reduce #(and %1 %2) (map number? [x y z])) (Quaternion. x y z 1.0)))
+         (every? number? [x y z]) (Quaternion. x y z 1.0)))
   ([^Float x ^Float y ^Float z ^Float w]
    (Quaternion. x y z w)))

Yes please

did you run lein lint?

My apologies, I appear to have missed this. I have now run lint, and added a new commit to the pull request; there are eleven remaining instances of 'long lines' failures, but each of these is due to a link which cannot be broken or shortened.


DVC trees do not update properly after adding a file

In this repo https://github.com/iterative/iterative.ai
I'm trying to add file this way:
https://user-images.githubusercontent.com/3659196/218895696-4487e09d-982b-499d-978c-27cd32f4d34a.mov
and it updates both trees properly.
But if I do it this way (just a different location):
https://user-images.githubusercontent.com/3659196/218895986-b79ebf2a-cde7-4f64-8725-c16055d59e4c.mov
It updates files around to ignored which is nor correct as far as I understand?
I hit an issue before but can't reproduce when it was not updating the tree at all (only one new greyed out file added), no modifications visible.

Managed to reproduce the second scenario:
https://user-images.githubusercontent.com/3659196/218898539-fa32860a-a604-4b7e-acc4-ca7c79370c70.mov

Might be fixed, we need to try it again.

I have confirmed that this is fixed.


Update to .NET 5 and EF Core 5

Hello just wondering if the project will have a release for .NET 5 eventually?
As I understand it's a .NET Standard 2.0 project so it should work with the new version of dotnet and EF Core?
I'm still kind of confused about the required changes, so this is mostly to clarify the future of the project.

It should work with .NET 5, but I think EFCore 5 now only supports .NET Standard 2.1, so this lib would need to be changed to target that in order to use EFCore 5 instead of 3.

Yeah I gave it ago, it seemed to work but most of the bulk operations were running significantly slower compared to EF Core 3.1. So I just ended up reverting back to 3.1 for now.

From reading some compatibility info on .Net 5 understanding was that it should work without changes.
And @Dylan-Erskine confirmed it with a test.
As for the speed will check that in the near future.

Alright, thanks for the info @borisdj and everyone else. We'll probably stay on 3.1 for a while then and I'll keep checking for any changes meanwhile.

I have created a library to implement this on EF Core 5.
await ctx.DeleteRangeAsync(b => b.Price > n || b.AuthorName == "zack yang");
await ctx.BatchUpdate()
.Set(b => b.Price, b => b.Price + 3)
.Set(b => b.Title, b => s)
.Set(b=>b.AuthorName,b=>b.Title.Substring(3,2)+b.AuthorName.ToUpper())
.Set(b => b.PubTime, b => DateTime.Now)
.Where(b => b.Id > n || b.AuthorName.StartsWith("Zack"))
.ExecuteAsync();
Github repository: https://github.com/yangzhongke/Zack.EFCore.Batch
Report: https://www.reddit.com/r/dotnetcore/comments/k1esra/how_to_batch_delete_or_update_in_entity_framework/

Lib is upgraded  to v5.0.0 with support for .Net 5 for EF Core 5.
Target in v5 is .NetStandard 2.1

@ErikEJ I've noticed that with .Net 5 simple BulkInsert is around 2 times slower then with .Net 3.1
Am I correct to assume that there were some significant changes that had effect on efficiency.
Secondly is there anything that can now be configured to speed it up more.
If not, is this how it will remain, or are some improvements planed in new minor versions, or at least in next mayor release .Net 6

just using SqlBulkCopy you see this?
Is the slowness in EF Core part, or in SqlBulkCopy?

Or in other words: which components does .NET 3.1 imply, and what components does .NET 5.0 imply?
EF Core version, Microsoft.Data.SqlClient version?

Haven't done detailed inspection yet.
Just compared same Test before and after switching to .Net 5.
Previously lib. was targeting .NetStandard 2.0 and had nuget dependencies:
Microsoft.EntityFrameworkCore 3.1.13
Now it is targeting .NetStandard 2.1 and using Microsoft.EntityFrameworkCore 5.0.4
And also some sitax had to be changed for new version, example:
GetTargetType() -> TargetEntityType
IsDependentToPrincipal() -> ParentNavigation.IsOnDependent
GetColumnName() -> GetColumnName(ObjectIdentifier)
Compared Test was following (on 3.1 just under 2s, and with 5.0 took little over 4s):
var entities = new List<Item>();
for (int i = 1; i < 100000; i++)
{
	var entity = new Item
	{
		Name = "name " + i,
		Description = "info " + Guid.NewGuid().ToString().Substring(0, 3),
		Quantity = i % 10,
		Price = i / (i % 5 + 1),
		TimeUpdated = DateTime.Now
	};
	entities.Add(entity);
}
context.BulkInsert(entities);

Also to note that Test project itself was changed,
Target:  netcoreapp3.1 -> net5.0
Nugets:
Microsoft.EntityFrameworkCore.SqlServer 3.1.13 -> 5.0.4
Microsoft.Extensions.Configuration.Json 3.1.13 -> 5.0.0
NET.Test.Sdk 16.9.1 -> 16.9.4

Did some more digging.
Operation sqlBulkCopy.WriteToServer(dataTable) runs the same aroud 1s on both version.
So it seems slow neck is in some of prepare method of the library, either extension code itself or one of EFCore methods that are used.

@andrysvyrid @ajcvickers

Found the problem, one inner method of lib had inside a loop the call to .GetColumnName() instead of using Dictionary.


Add ability to pass resource/config files

I am using a python library which is initialized using a config file and uses several other resources to initialize itself. I do not see anyway to pass those config and resources directly using "sparse submit" by making them part of jar resources. That way all dependencies are at one place, instead of user pushing such resources separately to all workers.

Any updates on this? Am I missing something trivial?

The python library is doing something like an /etc/foo-library.conf lookup? In most scenarios I'd say that belongs in the realm of system configuration -- a package with a system conf file should be installed by the package/configuration manager because it's shared between multiple runs of this library.
I don't believe streamparse has any pre-submit hooks to run arbitrary fabric commands.
One option would be to symlink that conf to where the jar is unpacked, but I think that filename may be variable.
Is it possible to override the python library to specify an alternative lookup path to that file? Possibly a global or a constructor argument. You could submit that conf file in your topology and find it relative to the file being executed.

yes config file is configurable via constructor, and currently I am going to previous folder from spout location and getting that config. e.g. spout is in resources/spouts and config is in resources, I hope that will always be supported.
The main problem is that usually I won't keep config in src (security: as it contain auth details), but it will come from somewhere else during submit. We need a way to hook into jar creation so that I can point to my config/resource files
Ideally streamparse config.json should take path to such extra resources and such path should also be configurable from sparse submit

quickest workaround could be to have SPARSE_RESOURCES env variable which can point to file or folder and then can be copied to _resources in prepare_topology method

Another workaround without changing code:
Added external_resources dir , added it to .gitignore
symlink required resources to external_resources and add external_resources to project.clj
there could be different symlinks for dev, stg, prd configs, and python code can load each based on an environment.


Add __init__.py in cli/commands

Just noticed we were missing a __init__.py file :)
It fixes this error from mypy:
 error: Cannot find implementation or library stub for module named 'strawberry.cli.commands.codegen'

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)


Saving results to file after CVAT annotations

How do you download only the updated and newly added tags from CVAT annotation in FiftyOne?
dataset.export does not add the new annotations.

If you are using the FiftyOne integration with CVAT, then you need to make sure to load the annotation back into FiftyOne first before exporting them to disk. Also note that tags are only available for image datasets in the CVAT format, not videos, this is a limitation of CVAT.
Please share a reproducible example if you believe there is a bug.


How to use with emacs?

Hi first of all thanks for creating this tool/lib. Can I use this with emacs-tree-sitter some how? Any suggestion on how to configure it? tree-sitter for emacs seems to want to load the language specification by using dynamic lib.

I have to dig into emacs-tree-sitter a bit and figure this out. Will get back to you...

It looks like you were able to get what you needed out of https://gitlab.com/woolsweater/tree-sitter-swifter, so I’ll close this for now. If anyone else comes across this and wants to try it out, it sounds like the instructions on https://github.com/emacs-tree-sitter/tree-sitter-langs#building-grammars-from-source were what worked for you — feel free to reply if you have anything to add on top of that that might apply to this repository as well.

I think I want to reopen this again. Your parser seems more complete.
I downloaded it and runned npm install from root. It created a tree-sitter-swift.node file. Emacs would like to have a swift.dylib file instead so I tried renaming it.
But it fails to start because 'error in process filter: Language load failed: "dlopen(.emacs.d/elpa/tree-sitter-langs-20220821.1313/bin/swift.dylib, 0x0005): tried: '.emacs.d/elpa/tree-sitter-langs-20220821.1313/bin/swift.dylib' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64e'))"
I am doing it correctly, can I build a .so or .dylib directly. And how do I change from x86_64 to silicon?


obsolete iausofa library

The directory
https://github.com/sdss/coordio/tree/master/cextern/sofa
is an obsolete version of
https://github.com/liberfa/erfa/tree/master/src
I propse to use the "git subtree" command to replace the obsolete local version and to ensure that the other version (which is known to be maintained actively) is always in use.
Besides: one could get the RPM or Debian packages from
https://build.opensuse.org/package/show/home:rjmathar/iausofa-c
with the zypper or apt command to avoid compiling anything within this module.

Hi @rmathar,
Do you know what are the improvements of the newer versions of SOFA? In principle I don't mind updating it, but I since we only use a few features I feel that doing that manually may be better. I've found that adding subtree or modules ends ad making managing the repository more complicated, and people that clone from source not always follow the instructions to initialise the submodule.
Similarly, I would prefer not to depend on an RPM or external installation because that would make coordIO not easily work on macOS, unless you know of an easy way to install SOFA on Mac. Since SOFA is a very simple pure C library compiling it doesn't seem to be likely to cause errors (assuming one has a basic compiler).

i) The changes within the IAU SOFA releases are documented in
https://www.iausofa.org/archive.html
ii) To keep the software up-to-date with the upstream release has nothing to do with "RPM" or "external installation" or Macs. The RPM's mean that people can get up-to-date versions of the software by bypassing the obsolete variant maintained here.


Update TimeZoneInfo XML to move PS1 example to correct place

Title
The XML file had a PowerShell example, but in the wrong place thus while the example displayed, it displayed on the wrong page. This PR attempts to correct that!
Summary
The example was rendered under the SupportsDaylightSavingsTime property. It was moved to be on the correct page, demonstrating the GetSystemTimeZones method.
Details
I goofed when entering the sample in the first place. This fixes that mistake
Suggested Reviewers
@mairaw

This looks good, @doctordns. @mairaw, the PowerShell example always displays despite the language filter, and there is no PowerShell filter. Is that the expected behavior at this point?

@rpetrusha - thanks. As for the lack of language filter for PowerShell etc is a 'feature' of the platform at the moment. It's expected behaviour but hopefully not for the long term.

I was hoping that it had already changed, @doctordns. In any case, this looks good, and I'll merge your PR now.

Sorry, @doctordns. Instead of commenting, I closed your PR, so reopening begins a new build. I'll merge when the build completes.

That's still an issue, unfortunately. 😢


Optimize: Pull with ~70 devices takes ~7 minutes

Pull with a large amount of devices should be optimized.

Optimized by splitting datastore, and replaced per-device commit with a single per-transaction commit

Down to 1m:30s with ~80 devices. 🎉


InsecurePlatformWarning: A true SSLContext object is not available

When I try to post a message I get this warning
In [1]: from slacker import Slacker
In [2]: slack = Slacker('abc-123')
In [3]: slack.chat.post_message('#random', 'hello from slacker')
/Users/pablo/envs/slacker-cli/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:79:
InsecurePlatformWarning: A true SSLContext object is not available.
This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail.
For more information, see
https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
Out[3]: <slacker.Response object at 0x107726350>

Messages are still going through. See the original issue. I'm not sure if we should suppress these warnings in slacker.

Messages are still going through. See the original issue. I'm not sure if we should suppress these warnings in slacker.

I never see this message before
I thought this message was related with the recent security incident in slack
http://slackhq.com/post/114696167740/march-2015-security-incident-and-launch-of-2fa

It's not related. You wouldn't see any of these warnings if you follow the same steps with requests==2.2.1.

The question is, should we suppress these warnings?

In a library like slacker, I think it is not necessary suppress the warnings, because the developer who use the library can ignore these.
what is your opinion ?
in the case of a command line application like https://github.com/juanpabloaj/slacker-cli , I think it is necessary suppress the warning messages.

Yeah, that's what I was thinking too. Thanks for the ticket @juanpabloaj.


Height change extras

Contains
Attempt to fix #2535 .
The character's jump speed,movement speed,interaction range and step height are a function of the player's height.I acquired the functions through play testing and graph plotters.
Earlier, a change in the player's height was made through a change in the his position through teleportation.This is probably what caused the player to sink through blocks at dwarf heights.This pull request, on the other hand just changes the eye height of the player.

[x] Increase/Decrease jump speed according to size. Giants should be able to jump higher and dwarfs lower.
[x] Giants should take longer strides, increase runFactor?
[x] Unable to pick up blocks off ground when height > 10. Bug!
[x] Physics bug: Eye height translates gazeMountPoint to a value more than expected.
[x] Trickier: Enable giants to be able to automatically climb up (no jump) blocks of smaller elevation.
[x] Height too small, makes you fall through blocks.
"Increase/Decrease damage from fall. (30m giant should not die of a 15m fall). Define function/proportionality constant same as above."->this sub issue is dealt by a pull request in the health module

How to test
Just change the player's height and his capabilities should increase or decrease accordingly.
Outstanding before merging

[ ] Potions should use a AffectHeightComponent instead of affecting the PlayerConfig settings. These should override the Default (config) values and should be serialized to the world save.

Can one of the admins please verify this patch?

@skaldarnar "Potions should use a AffectHeightComponent instead of affecting the PlayerConfig settings. These should override the Default (config) values and should be serialized to the world save."
This is actually a follow up.I only mentioned it because it was mentioned as one of the extras in the issue

I think that this command should, under the hood, use a ScaleCharacterEvent such that other systems can also react on it (e.g., the Health system also needs to adjust values on height change).
Merging this now as clear improvement and good progress to get the logic right.


Improve Minecraft issue template

Link to the Mojang bug tracker /summary instead of /issues page which contains links to guidelines and help resources
Remove Markdown formatting from _Java_ because GitHub does not appear to support Markdown for the description

@Marcono1234 Thanks for the contribution!

Here is how it now looks with the GitHub 1.20.0 app on iOS 14.6. The description of Minecraft-related bugs is unfortunately truncated, but I presume players of the Java edition are not using this app for filing issues.

@KalleOlaviNiemitalo - Thanks for this feedback!  I'll look to truncate that description.

but I presume players of the Java edition are not using this app for filing issues.

Probably rather unlikely indeed, isn't it? The user came here from the header of the JVM crash report which they viewed on their PC.

but I presume players of the Java edition are not using this app for filing issues.

Probably rather unlikely indeed, isn't it? The user came here from the header of the JVM crash report which they viewed on their PC.

Those choices should pop up when users follow that URL in their browser.  I've made a change (landed 15 mins ago) which I think should force those options to pop up.  Let's see if the volume slows down :-)
@Marcono1234 - thanks for all of your triage BTW - much appreciated!  Are you available to chat on the joint Slack we have?

Are you available to chat on the joint Slack we have?

We Mojang bug tracker volunteers are on the Mojang Slack, not sure if there is any connection between it and your Slack.

Are you available to chat on the joint Slack we have?

We Mojang bug tracker volunteers are on the Mojang Slack, not sure if there is any connection between it and your Slack.

Yes I'm on there, Martijn Verburg - please say hello :-)

We are multi channel guests so it looks like I cannot find you without already sharing a channel with you. Could we do it the other way around, could you message me instead please? My user name on Slack is marcono1234 (like here).


Docs should be included with release

New docs folder wont currently be added to a release build - should be included in the future while docs are there.
.travis.yml example area
e.g.
elif [[ $TRAVIS_OS_NAME == "windows" ]]; then
7z a -tzip $FILE_NAME README.md install.bat LICENSE q examples;
elif [[ $TRAVIS_OS_NAME == "linux" || $TRAVIS_OS_NAME == "osx" ]]; then
tar -zcvf $FILE_NAME README.md install.sh LICENSE q examples;

Hi Simon - I'm including your Travis CI patch into the next docs update as a separate commit

Closing the ticket because the patch is merged.
Simon, thanks for pointing out.