table extract error ?

The original pdf table:

The extract table:

This is the command:
python pdf2txt.py example.pdf -o example.html -t html
The example pdf: http://static.cninfo.com.cn/finalpage/2019-02-01/1205830955.PDF

Hi @bifeng, thanks for your question. You don't specify what is wrong with the output table. Obviously the layout of the numbers is wrong, but is that all? Or are there more issues that you want to address?
The wrong position of the text is because pdfminer groups the texts in the adjacent cells to aggressively. It does not take the cell dividers into account, only the vertical distance. You can change the default value (0.5) with LAParams.line_margin, for example by using the command line option --line-margin:
python pdf2txt.py example.pdf -o example.html -t html --line-margin 0.3

Example output:

You can close the issue if this fixes your problem.


Cu 86b085v5m arc cars

Task linked: CU-86b085v5m 023 'along the arc' vectors, cars able to make turns


Fixed #32559 -- Added step argument to FloatField.

ticket-32559

superseeds #14162
This pull request adds the missing documentation and release notes for the above.

buildbot, test on selenium.

buildbot, test on selenium.

buildbot, test on selenium.


[gui] Fix links in statistics tables

If we are for example in the checker statistics page and we set some
filter (run, checker name) and we click on a link we need to keep
the previous run/checker name filters to filter the report.
So all filters set in the statistics view should follow to the Reports view.

I can't see any change in the behavior. Could you please write a scenario that is different with this patch?
I did this:

Go to statistics page
Filter reports
Click on some link to jump to the reports.
Reports are now filtered and filter options are set correctly according to the clicked statistical number.

Go to the checker statistics page
Set a checker name filter
Click on a number.

The checker name filter will not be set at the report page.


Can anyone see the torrent speed?

Environment
Docker
Version
0.6.0
Describe the problem
First of all , I don't know if it is a bug. I am not a techy person. I just wanna check if anyone managed to use the torrent speed module. And how do you make it?
I filled in the username and the password and restarted both containers...but nothing happened...module shows nothing but 0 up & 0 down speed.
Any advice, thanks in advance!
Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

First thing to check is : Can the Homarr docker container access the service url you have set for your download client ?
If the link uses proxy's or external ips i will likely not work. You need to use something like <IP_ADDRESS>
There is a command to show your docker interface IP but I don't remember it by heart.
Also you can check the nature of the Error in the Dev console under the "network" tab see the result of the request.

First thing to check is : Can the Homarr docker container access the service url you have set for your download client ?
If the link uses proxy's or external ips i will likely not work. You need to use something like <IP_ADDRESS>
There is a command to show your docker interface IP but I don't remember it by heart.
Also you can check the nature of the Error in the Dev console under the "network" tab see the result of the request.

Thanks for the reply.
I guess it is something wrong with the ips.  After I have set the the qBittorrent port fowarding to the public ip and change it accordingly in Homarr, it worked! And the "Ping service dot" turned from yellow to green.
Do you know why it doesn't work on the local IP? Or it's just my ignorance that I can't make it work on Local IP 🤦🏻♂

First thing to check is : Can the Homarr docker container access the service url you have set for your download client ?
If the link uses proxy's or external ips i will likely not work. You need to use something like <IP_ADDRESS>
There is a command to show your docker interface IP but I don't remember it by heart.
Also you can check the nature of the Error in the Dev console under the "network" tab see the result of the request.

Thanks for the reply.
I guess it is something wrong with the ips.  After I have set the the qBittorrent port fowarding to the public ip and changed the address accordingly in Homarr, it worked! Also the "Ping service dot" turned from yellow to green.
Do you know why it doesn't work on the local IP? Or it's just my ignorance that caused the failure to  make it work on Local IP 🤦🏻♂

The resolution you achieved is the complete opposite of what I suggested aha.
It is supposed to work on local IPs but you need to make sure that Homarr can access the other containers on your network , using their name as a domain (ie sonarr if your container is named that way) or by using IP:port
It is not possible to change the behavior as I make it so that Homarr does the requests to your services and not your browser, so if you browser can access the used ip but not homarr it won't work.

First thing to check is : Can the Homarr docker container access the service url you have set for your download client ?
If the link uses proxy's or external ips i will likely not work. You need to use something like <IP_ADDRESS>
There is a command to show your docker interface IP but I don't remember it by heart.
Also you can check the nature of the Error in the Dev console under the "network" tab see the result of the request.

Thanks for the reply.
I guess it is something wrong with the ips.  After I have set the the qBittorrent port fowarding to the public ip and changed the address accordingly in Homarr, it worked! Also the "Ping service dot" turned from yellow to green.
Do you know why it doesn't work on the local IP? Or it's just my ignorance that caused the failure to  make it work on Local IP 🤦🏻♂

The resolution you achieved is the complete opposite of what I suggested aha. It is supposed to work on local IPs but you need to make sure that Homarr can access the other containers on your network , using their name as a domain (ie sonarr if your container is named that way) or by using IP:port
It is not possible to change the behavior as I make it so that Homarr does the requests to your services and not your browser, so if you browser can access the used ip but not homarr it won't work.

   "Ports": {
            "6881/tcp": [
                {
                    "HostIp": "<IP_ADDRESS>",
                    "HostPort": "6881"
                }
            ],
            "6881/udp": [
                {
                    "HostIp": "<IP_ADDRESS>",
                    "HostPort": "6881"
                }
            ],
            "8080/tcp": null,
            "8485/tcp": [
                {
                    "HostIp": "<IP_ADDRESS>",
                    "HostPort": "8485"
                }

Sadly, it still doesn't work...
I used "-p IP:port:8485" to deploy the QBittorrent...

Can't be replicated, needs more info. Maybe the instructions here can help you set up qBittorent correctly.

After I redeployed the Homarr in Host network, everything just works 🤦🏻♂ Why haven't I done it in first place


Change move constructor and assignment operator for Memory.

The move semantic expect behavior is to take ownership, this PR fixes this behavior for the Memory<T> class.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2355
@YohannDudouit
@tzanio
@camierjs + @v-dobrev + @pazner + @acfisher
06/26/21
⌛due 07/10/21
⌛due 07/17/21

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

As with the other PR, #2358, this PR goes against my original design. I prefer not to make this change, so I will not review this PR.

As @pazner was mentioning during the team meeting there is probably different valid choices for the move semantics behavior. This version leaves the moved objects in a "Reset" state, meaning that the origin is unusable after being moved (I think this is the expected standard behavior). An alternate behavior could just be to make a copy of the origin and to take ownership of the content of the origin object when possible (the origin is still usable but does not own the memory any more, and can therefore be deleted safely).
I don't think the move semantics is used anywhere so far, but in a way I agree with @tzanio that the behavior of the move semantics should be coherent with the behavior of the copy constructors. In this regard, I think that the only meaningful behavior for the copy constructor is to copy without taking ownership (otherwise dangling pointer bugs become very likely, or double free issues, since copying creates multiple owners). Ownership issues is one of the most frequent things that our users complain about, and for this reason I don't think we should take this very computer science question too lightly. I want to remind that the ownership aspect of the Memory<T> class goes beyond what pointers do, pointers always own their memory (they can always delete it, and create dangling pointers, double free, etc..). In many regards pointers are a bad abstraction of memory because they enable all sort of undesired features (bugs), modern C++ guidelines recommend to never use raw pointers (There is always a better solution provided by the standard library). There is absolutely no benefits in having a design that allows dangling pointers. A last remark, this design doesn't fully get rid of dangling pointers issues, since the "origin" is not aware of its aliases, a user can delete the "origin" and leave all aliases dangling. Making aliases safe would require a more extensive redesign.
To summarize, if we want the move semantics to mean "copy and take ownership", and the copy constructor to mean "copy and don't take ownership", it may make more sense to have the move semantics leave the "origin" in a valid state (contrary to what is done in this PR).

@YohannDudouit, am I correct that this is waiting on some documentation updates that we have discussed previously?

@v-dobrev can you confirm that your documentation concerns are addressed now?

Merged in next for testing...

Re-merged in next for testing...


No respect for "cache" option

There is no any code that handles "cache" option https://developer.mozilla.org/en-US/docs/Web/API/Request/cache
On IE11 this causes similar request always fetching from cache.

You're right. It's due technical limitations. See explanation here https://github.com/github/fetch/issues/438#issuecomment-261272466

This should be written in bold in README/Caveats section imho.

It's listed in Caveats of the web site.

It should be listed on top of page with big red font and blinking :)


lxc exec Error: Failed to fetch / 301 Moved Permanently

Required information

Distribution: Ubuntu
Distribution version: 17.04

Kernel version: 4.10.0-30
LXC version: 2.0.8
LXD version: 2.16 (zesty-backports)
Storage backend in use:

Issue description
After upgrading to lxd 2.16 i can't exec or start any container:
$ lxc exec local://xxx -- /bin/bash
error: Failed to fetch http://unix.socket/1.0/containers///xxx/exec: 301 Moved Permanently

I think you want "lxc exec local:xxx -- /bin/bash", the syntax isn't a URI/URL so there's never "//" in the LXD commands themselves.
The "local" remote is also usually your default remote, so "lxc exec xxx -- bash" would be equivalent.
I find it surprising that this ever worked, it sure wasn't meant to :)
(Syntax is "lxc  []:").

Thanks for the response.
I don't know why i got used to the syntax, i think i went into an error once and tried this instead.
The following worked for me:
$ lxc exec local:xxx -- bash
This did not work:
$ lxc exec xxx -- bash
error: The remote "" doesn't exist

I've just set default-remote: "local" in ~/.config/lxc/config.yml and $ lxc exec xxx -- bash is now working, great :)


Adlam text displayed in a block element with dir set to auto should be automatically right-aligned, unless the base direction is set to LTR, like this:

Adlam text displayed in a block element with dir set to auto should be automatically right-aligned, unless the base direction is set to LTR, like this:

The GAP
In WebKit the text appears as follows:

This is a serious issue, and indicates that first-strong heuristics are not being applied.  Note that this was not an issue for N'Ko, Arabic, Hebrew, Thaana, or Syriac text.
Tests & results
Interactive test, Text in languages that use RTL scripts will be automatically aligned with the right side of their container when dir is set to auto

Gecko: ✅     Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:96.0) Gecko/20100101 Firefox/96.0
Blink: ✅     Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.71 Safari/537.36
Webkit: ✅  Adlam used to be broken, but is now fixed (see the bug report).    Browser: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/15.2 Safari/605.1.15

Action taken
Webkit
Outcomes
This was fixed in WebKit, so it now works as expected for all 3 major browser engines.
Priority
Priority initially set to basic because this was a serious issue for Adlam users, but now set to OK.
Originally posted by @r12a in https://github.com/w3c/afrlreq/issues/13


Bring back typedef property expansion

In the newer version, I can only see the type of an argument which is an object, compared to how it was expanded in the previous versions.

It was actually very useful to see all properties on hover which served as a documentation.

I think if you hide the details of the type, you should also provide means to expand its properties via a UI, or provide a setting for it.

This issue was moved to Microsoft/TypeScript#25784


Syntax highlighting wrong halfway through file

 Environment data

Language Server version: 2022.3.0
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda):
python.analysis.indexing: null
python.analysis.typeCheckingMode: off

 Expected behaviour
Syntax highlighting consistent throughout the whole file
 Actual behaviour
Syntax highlighting changes in the middle of the file/ not working correctly
 Logs

Python Language Server Log
[Info  - 11:50:19 AM] (1861) Pylance language server 2022.3.0 (pyright 1ed262f6) starting
[Info  - 11:50:19 AM] (1861) Server root directory: /home/milo/.vscode-server/extensions/ms-python.vscode-pylance-2022.3.0/dist
[Info  - 11:50:19 AM] (1861) No configuration file found.
[Info  - 11:50:19 AM] (1861) No pyproject.toml file found.
[Info  - 11:50:19 AM] (1861) Setting pythonPath for service "ATP-pascal": "/usr/bin/python3.10"
[Warn  - 11:50:19 AM] (1861) stubPath /home/milo/ATP-pascal/typings is not a valid directory.
[Info  - 11:50:19 AM] (1861) Assuming Python version 3.10
[Info  - 11:50:19 AM] (1861) Assuming Python platform Linux
[Warn  - 11:50:20 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:50:20 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:50:20 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:50:20 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:50:20 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Info  - 11:50:20 AM] (1861) Searching for source files
[Info  - 11:50:20 AM] (1861) Found 3 source files
[Info  - 11:50:20 AM] (1861) Background analysis(1) root directory: /home/milo/.vscode-server/extensions/ms-python.vscode-pylance-2022.3.0/dist
[Info  - 11:50:20 AM] (1861) Background analysis(1) started
[Info  - 11:50:20 AM] (1861) Indexer background runner(2) root directory: /home/milo/.vscode-server/extensions/ms-python.vscode-pylance-2022.3.0/dist (index)
[Info  - 11:50:20 AM] (1861) Indexing(2) started
[Info  - 11:50:21 AM] (1861) Indexer done(2). indexed 431 files
[Info  - 11:53:24 AM] (1861) No configuration file found.
[Info  - 11:53:24 AM] (1861) No pyproject.toml file found.
[Info  - 11:53:24 AM] (1861) Setting pythonPath for service "ATP-pascal": "/usr/bin/python3.10"
[Warn  - 11:53:24 AM] (1861) stubPath /home/milo/ATP-pascal/typings is not a valid directory.
[Info  - 11:53:24 AM] (1861) Assuming Python version 3.10
[Info  - 11:53:24 AM] (1861) Assuming Python platform Linux
[Warn  - 11:53:24 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:24 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:24 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:24 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:24 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Info  - 11:53:24 AM] (1861) Searching for source files
[Info  - 11:53:24 AM] (1861) Found 3 source files
[Info  - 11:53:24 AM] (1861) No configuration file found.
[Info  - 11:53:24 AM] (1861) No pyproject.toml file found.
[Info  - 11:53:24 AM] (1861) Setting pythonPath for service "ATP-pascal": "/usr/bin/python3.10"
[Warn  - 11:53:24 AM] (1861) stubPath /home/milo/ATP-pascal/typings is not a valid directory.
[Info  - 11:53:24 AM] (1861) Assuming Python version 3.10
[Info  - 11:53:24 AM] (1861) Assuming Python platform Linux
[Warn  - 11:53:25 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:25 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:25 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:25 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:53:25 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Info  - 11:53:25 AM] (1861) Searching for source files
[Info  - 11:53:25 AM] (1861) Found 3 source files
[Info  - 11:53:25 AM] (1861) Indexer background runner(3) root directory: /home/milo/.vscode-server/extensions/ms-python.vscode-pylance-2022.3.0/dist (index)
[Info  - 11:53:25 AM] (1861) Indexing(3) started
[Info  - 11:53:26 AM] (1861) Indexer done(3). indexed 431 files
[Info  - 11:54:43 AM] (1861) No configuration file found.
[Info  - 11:54:43 AM] (1861) No pyproject.toml file found.
[Info  - 11:54:43 AM] (1861) Setting pythonPath for service "ATP-pascal": "/usr/bin/python3.10"
[Warn  - 11:54:43 AM] (1861) stubPath /home/milo/ATP-pascal/typings is not a valid directory.
[Info  - 11:54:43 AM] (1861) Assuming Python version 3.10
[Info  - 11:54:43 AM] (1861) Assuming Python platform Linux
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Info  - 11:54:43 AM] (1861) Searching for source files
[Info  - 11:54:43 AM] (1861) Found 3 source files
[Info  - 11:54:43 AM] (1861) No configuration file found.
[Info  - 11:54:43 AM] (1861) No pyproject.toml file found.
[Info  - 11:54:43 AM] (1861) Setting pythonPath for service "ATP-pascal": "/usr/bin/python3.10"
[Warn  - 11:54:43 AM] (1861) stubPath /home/milo/ATP-pascal/typings is not a valid directory.
[Info  - 11:54:43 AM] (1861) Assuming Python version 3.10
[Info  - 11:54:43 AM] (1861) Assuming Python platform Linux
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Warn  - 11:54:43 AM] (1861) Exception received when installing file system watcher: TypeError [ERR_FEATURE_UNAVAILABLE_ON_PLATFORM]: The feature watch recursively is unavailable on the current platform, which is being used to run Node.js
[Info  - 11:54:43 AM] (1861) Searching for source files
[Info  - 11:54:43 AM] (1861) Found 3 source files
[Info  - 11:54:44 AM] (1861) Indexer background runner(4) root directory: /home/milo/.vscode-server/extensions/ms-python.vscode-pylance-2022.3.0/dist (index)
[Info  - 11:54:44 AM] (1861) Indexing(4) started
[Info  - 11:54:44 AM] (1861) Indexer done(4). indexed 431 files

 Code Snippet / Additional information
Pylance used in combination of python3.10 in WSL on Windows 11

3.10

Fixed it by deleting the Python for VScode extention.
Credit to this comment: https://github.com/microsoft/pylance-release/issues/862#issuecomment-783313780
My apologies for opening an Issue, I've been at it for a few days but just found this solution. Maybe this will help someone in the future. I'll close the issue.


[feature request] GROUP BY tag_key with selector function should return the relevant timestamps

As expected (and liked), when I include a GROUP BY time() clause in my TOP() query, the returned timestamps are the start of the interval:
> SELECT TOP(value,1) FROM pies WHERE time >= '2015-11-05T18:48:13.361920531Z' AND time <= '2015-11-05T18:49:07.679323455Z' GROUP BY time(1m)
name: pies
----------
time			        top
2015-11-05T18:48:00Z	2000
2015-11-05T18:49:00Z	2

But, when I only GROUP BY  a tag key, it would be nice to get back the relevant timestamps:
> SELECT TOP(value,1) FROM pies GROUP BY kind
name: pies
tags: kind=apple
time			        top
----			        ---
1970-01-01T00:00:00Z	100

name: pies
tags: kind=pumpkin
time			        top
----			        ---
1970-01-01T00:00:00Z	2000

Same happens for LAST and likely for all selector functions. They should return the actual timestamp rather than epoch 0.

+1

Even when we not use GROUP BY it still returns the epoch 0 time as in SELECT last(field_key) from measurement_name in latest 0.10.1 version.
> SELECT last(price) from Revenue_daily
name: Revenue_daily
-------------------
time    last
0       7777

As a workaround, have you tried including time in your field selection list?
SELECT TOP(value,1), time
FROM pies
WHERE time >= '2015-11-05T18:48:13.361920531Z' AND time <= '2015-11-05T18:49:07.679323455Z'
GROUP BY time(1m)

SELECT time, last(price) from Revenue_daily

@PaulMest works for me

You could also use something like this:
SELECT value, time
FROM pies
GROUP BY kind
ORDER BY time DESC LIMIT 1

Not sure if it is faster or slower, though.

On 0.12.2
@jokogr  works for me
> select value,time from ambient_temperature order by time desc limit 1
name: ambient_temperature
-------------------------
time			value
2016-05-19T20:54:01.85Z	20.375

but @PaulMest  does not work
> select time,last(value) from ambient_temperature
name: ambient_temperature
-------------------------
time			last
1970-01-01T00:00:00Z	20.375

Adding a where clause gives a valid timestamp :
> select last(value) from ambient_temperature where time > now() - 1m
name: ambient_temperature
-------------------------
time				last
2016-05-19T20:57:08.146125643Z	20.40625

@rkuchan does the suggestion @PaulMest work?

@e-dard - it looks like the behavior has changed since I opened this issue - I'm on version 1.1. I'm now getting the expected timestamps when I GROUP BY  a tag key:
> SELECT top(water_level,2) from h2o_feet where time >= '2015-08-18T00:00:00Z' and time <= '2015-08-18T00:24:00Z' group by location
name: h2o_feet
tags: location=coyote_creek
time                                top
----                                 ---
2015-08-18T00:00:00Z  8.12
2015-08-18T00:06:00Z  8.005

name: h2o_feet
tags: location=santa_monica
time                                top
----                                  ---
2015-08-18T00:06:00Z  2.116
2015-08-18T00:18:00Z  2.126

I don't think including time in the SELECT clause does anything to affect the query. I wasn't aware that you could get any change in behavior by specifying time - wouldn't InfluxDB just assume that time is a  field key and ignore it (assuming you don't have a field key called time)?

Note: addressed in https://github.com/influxdata/platform/pull/360


Fix and close #99

Coverage decreased (-0.2%) to 77.165% when pulling a17852789b4a3566fdf1175a2cc9dcee938c593f on gmist:fix_99 into c07a906ed52f131bcdda67d84e493390c614b3ae on klen:develop.

@gmist Thank you for the PR.


WSL and Windows Ollama

I have installed Parllama on WSL2 and running Ollama on Windows. From within WSL, it is not able to connect with Ollama running on my PC IP, standard port. I am guessing because it is looking for Ollama on loopback within WSL. How can I point it to the regular IP of Ollama running on Windows? Cannot find any switch etc. to use for the same.

I will get this in tomorrow.


Add option to use local flex/bison

Based on #1132, but with option to use local flex/bison instead of making it the default.

Codecov Report

Merging #1137 (46c8427) into master (46a7d5d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1137   +/-   ##
=======================================
  Coverage   92.84%   92.84%
=======================================
  Files         337      337
  Lines       22500    22500
=======================================
  Hits        20891    20891
  Misses       1609     1609

Continue to review full report at Codecov.

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

Also:
Fixes #1122
Fixes #957


Load config from different files

Allow one of the configs to be loaded from a global config file
(~/.story_branch)
while others from the project root config file
(.story_branch)

@rpbaltazar looks ok.  Please add documentation coverage to README before merge.

I've updated the Readme to try to explicitly explain the situation.

Cheers Rui, looks ok.

Published to Rubygems


 It throws a lot of errors.

[x] Enable MyPY checks in CI
[ ] Pick an unannotated module, annotate it and whitelist it
[ ] continue

timeboxed for 1 day for now

When https://github.com/raiden-network/scenario-player/pull/509 is merged, all errors are fixed and untyped defs are checked. Further todos are:

Reconsider exclusion of service code (that code has been skipped until #502 is done)
Enforce types by setting disallow_untyped_defs = True

I think we already have most of the benefit without those todos and can consider this done (when #509 is merged). The above limitations are visible in the mypy.ini and can be worked as part of other commits.


linux-yocto-edison.bbappend: Add NFLX-2019-001 series of patches

Add patches for multiple TCP-based remote denial
of service vulnerabilities identified by Netflix.
Patch source:
https://github.com/Netflix/security-bulletins/blob/master/advisories/third-party/2019-001.md
Changelog-entry: Patches for TCP-based remote denial of service vulnerabilities
Signed-off-by: Vicentiu Galanopulo<EMAIL_ADDRESS>

@balena-ci retest


Improve WPF Live Data demo

This is the ScottPlot.Demo.WPF live demo

As you can see, the plot only updates when interacting with the window.
Is there a solution on my end or should I switch to WinForm for this kind of functionality?
ScottPlot.WPF 4.0.42
Microsoft Visual Studio Enterprise 2019
Version 16.7.5

Hey @MichelDequick thanks for posting this!
I'm on mobile and will take a closer look later today, but to force an update after data is modified you can call wpfPlot1.Render().
My guess is that the WPF demo isn't doing this important step when data updates. Resizing and interacting causes renders to be called for automatically, which is what probably leads to the behavior you observe.
Let me know if this helps!

Thanks for the fast response!
I tryed your suggestion and calling:
wpfPlot1.Render()

in
UpdateData()

Throws
System.InvalidOperationException: 'The calling thread cannot access this object because a different thread owns it.'

Hey @MichelDequick, so to hear this one is causing you so much trouble.
I'm about to merge a huge PR https://github.com/swharden/ScottPlot/pull/605 and roll a new major version in a few days, so my lean is to fix this problem after the merge and provide better documentation at that time. This may be similar to #609, and ultimately a problem where data triggers a render while a layout or mouse event has already started a render...
As far as an immediate fix, I suspect putting the render call inside a try/catch block will get you past the few instances where new data triggers an update at the same time a layout change or mouse interaction is causing an update. Let me know if this solves your issue, but either way I'll leave this issue open until I get time to come up with a better long-term solution.
Thanks again for your input!

After letting this issue behind for a bit I retried solving it without succes.
Reading up online suggested to pass the calls trough to the application invoker. Even thought this does not throw an error like previous methods, the plot still doesn't render. When forcing a render by dragging the window I notice that there is data lacking. This suggest that the UpdateData method is not called as frequently as it should, shifting the blame to the DispatchTimers.
Interested to hear your thoughts and insights

I have found a temporary temporary solution by using System.Timers.Timer instead of DispatcherTimer
and making this timer call Application.Current.Dispatcher.Invoke(new Action(() => { wpfPlot1.Render(); }));

I have found a temporary solution by using System.Timers.Timer instead of DispatcherTimer
and making this timer call Application.Current.Dispatcher.Invoke(new Action(() => { wpfPlot1.Render(); }));

Hey @MichelDequick,
I'm working to refactor the user controls to make this easier in the future, and I also plan to document this topic as part of the new FAQ #572 so I'll close this issue, but I really appreciate you for following-up to share your solution!
Scott


Expense - Pay elsewhere button and search icon are far apart

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.53-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat
Submit expense to the workspace chat
Go to expense report
Note the spacing between Pay elsewhere button and search icon
Go to Search
Note the spacing between Filters button and search icon

#Expected Result:
In the expense report (also workspaces page), the green button and search icon should not have big space
It should have the same spacing between Filters button and search icon in Step 7
Actual Result:
In the expense report (also workspaces page), the green button and search icon has big space
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/bc5263de-22b4-404c-ab6d-cb81716dda9c
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Pay elsewhere button and search icon are far apart
What is the root cause of that problem?
We are using styles.mr2 for the icon in HeaderWithBackButton making the header button seem further than it should.
https://github.com/Expensify/App/blob/5e975f5aedac0d150984282ada45eed1693bcdb6/src/components/HeaderWithBackButton/index.tsx#L181
What changes do you think we should make in order to solve the problem?

Remove the styles.mr2 from additionalStyles.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Expense - Pay elsewhere button and search icon are far apart
What is the root cause of that problem?
Search Button is outside this <View
https://github.com/Expensify/App/blob/5e975f5aedac0d150984282ada45eed1693bcdb6/src/components/HeaderWithBackButton/index.tsx#L194-L267
What changes do you think we should make in order to solve the problem?
Move SearchButton inside this View. and remove extra styling mr5 from searchButton
https://github.com/Expensify/App/blob/5e975f5aedac0d150984282ada45eed1693bcdb6/src/components/HeaderWithBackButton/index.tsx#L267
https://github.com/Expensify/App/blob/5e975f5aedac0d150984282ada45eed1693bcdb6/src/components/HeaderWithBackButton/index.tsx#L194-L266
What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

Don't need to fix this


Mac installation issue

I've completed the installation following the steps, but in the last one when the file "start.bat" has to be dragged on Terminal this alert appeared:
-bash: /Applications/NASA's Asteroid Data Hunter/start.bat: Permission denied
MacBookProMrMillares:~ mrmillares$
Any suggestion to solve it?

It should be "start.sh", there was a mistake in the user guide. I just fixed it. Thanks for finding this!

Almost succeeded, but not. Now this:
Automatic deployment tool - Start
TCSASSEMBLER
Unable to find any JVMs matching version "(null)".
No Java runtime present, try --request to install.
JAVA_HOME=
warurl=/Applications/NASA's Asteroid Data Hunter/asteroid/hunter.war
applicationpath=/Applications/NASA's Asteroid Data Hunter
/Applications/NASA's Asteroid Data Hunter/start.sh: line 32: /bin/java: No such file or directory

Please try this:

Delete the Asteroid Data Hunter folder from your Applications directory
Install the latest version of Java: java.com
Re-install the app.

Please let me know how it goes.

I did it all, but unfortunately the same alert message appears.

Please go to https://www.java.com/en/download/installed.jsp?detect=jre&try=1 - this page will then show you what version of Java you have installed. And if you don't have the latest version which is Version 8 Update 40, please upgrade, restart your browser and attempt to launch the app again. Please let me know how it goes.

Yes, the Mac (v10.10.2) is running the latest version of Java (Version 8 Update 40).

I am sorry about the issues you're experiencing - since this is now an open source project, we're working on the plan for ongoing support and maintenance. You'll probably need to wait until the next version is ready because I am unable to reproduce the error you're experiencing.

Thank you for the help anyway!

On first extraction, none of the executables in the jre/bin directory are marked executable by default for me, so this needs a <chmod +x *> before anything happens. Also, the included <asteroid-installer-mac.jar> is referenced as <asteroid-installer-linux.jar> in the bash script (typo?).
Now I need to figure out this error message:
<Error occurred during initialization of VM Unable to find JVM location at $JAVA_HOME>

If you actually have java installed, you can install using java -jar asteroid-installer-mac.jar and it will work.


Include QDK at Visual Studio install

It would be cool for people setting up new development environments to be able to select at install time to download and install the QDK. See the screen cap below for an example:

Thanks!

Great idea, thanks! We've marked it on our list of things we want to look into.

tebrikler

We're looking into this - tagging @ricardo-espinoza


Save All, overwrites the visible file

VSCode Version: Code 1.10.2 (8076a19fdcab7e1fc1707952d652f0bb6c6db331, 2017-03-08T14:02:52.799Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

html-snippets
abusaidm
0.1.0

vscode-javascript-snippet-pack
akamud
0.1.5

vscode-color
anseki
0.2.5

amvim
auiworks
1.21.1

autofoldyeah
bobmagicii
1.0.6

vscode-markdownlint
DavidAnson
0.7.1

vscode-simpler-icons
davidbabel
1.3.0

vscode-eslint
dbaeumer
1.2.8

vscode-html-css
ecmel
0.0.33

EditorConfig
EditorConfig
0.6.2

tslint
eg2
0.8.1

beautify
HookyQR
1.0.2

Angular2
johnpapa
2.0.3

vscode-simple-icons
LaurentTreguier
0.11.2

vscode-JS-CSS-HTML-formatter
lonefy
0.2.2

markdown-shortcuts
mdickin
0.7.1

cpptools
ms-vscode
0.10.4

debugger-for-chrome
msjsdiag
2.7.0

material-icon-theme
PKief
2.0.1

guides
spywhere
0.7.0

react-beautify
taichi
0.3.0

Angular2
UVBrain
0.4.1

vim
vscodevim
0.6.11

Steps to Reproduce:

Dirty at least one .js and a .html file
Activate Save All command

The visible file will be overwritten with a portion of the other file. There doesn't seem to be a pattern for the behaviour that I could isolate.

@gnmonsour this is caused by the vscode-JS-CSS-HTML-formatter extension and should be reported to them.


ndt-server.go: forward -datadir to ndt7

This change is

Pull Request Test Coverage Report for Build 501

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 62.4%

Totals

Change from base Build 497:
0.06%

Covered Lines:
780

Relevant Lines:
1250

💛  - Coveralls


ETFE 2355 - Add navigation banner to every page in the service

@LewisMitchell  It looks like the spacing of the Back button matches the prototype. Is it worth asking design?
I'll look in to the navbar section issue

Thank you for submitting a pull request on this HMRC repository. A member of the HMRC team will review your changes and action accordingly.
HMRC reviewer: please reference this runbook for guidance on how to proceed.

retest


RedBlackTree excessively copies structs by value

gassa (@GassaFM) reported this on 2013-02-14T15:06:12Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=9513
CC List

bearophile_hugs
schveiguy (@schveiguy)

Description
If we choose to store a struct in a RedBlackTree in the most intuitive way, it gets passed by value quite a few more times than one would expect.  This can severely reduce performance when the struct is large and/or has a non-trivial postblit constructor.

An example follows:
-----
import std.container;
import std.stdio;

immutable int LIMIT = 100000;

struct element
{
	static int postblit_counter;

	int x;

	int opCmp (ref element other)
	{
		return x - other.x;
	}

	this (this)
	{
		postblit_counter++;
	}
}

alias RedBlackTree !(element) container;

void main ()
{
	auto f = new container ();
	element.postblit_counter = 0;
	foreach (i; 0..LIMIT)
	{
		f.insert (element (i));
	}
	writefln ("%s", element.postblit_counter);
}
-----

Here, upon inserting 100,000 elements in the tree, we make 11,389,556 calls to this(this).

However, if we choose to override the default predicate for RedBlackTree (that is, "a < b") with one which accepts arguments by reference, the program makes just 300,000 calls to this(this).

The modification:
-----
import std.container;
import std.stdio;

immutable int LIMIT = 100000;

struct element
{
	static int postblit_counter;

	int x;

	int opCmp (ref element other)
	{
		return x - other.x;
	}

	this (this)
	{
		postblit_counter++;
	}
}

bool lessfun (ref element a, ref element b)
{
	return a < b;
}

alias RedBlackTree !(element, lessfun) container;

void main ()
{
	auto f = new container ();
	element.postblit_counter = 0;
	foreach (i; 0..LIMIT)
	{
		f.insert (element (i));
	}
	writefln ("%s", element.postblit_counter);
}
-----

These are both tested for D2 v2.059 for Win32.  Adding compiler options "-O -release -inline" does not change the result of the test.

!!!There are attachements in the bugzilla issue that have not been copied over!!!

bearophile_hugs commented on 2013-02-14T15:17:57Z
The second program gives me:

temp.d(28): Error: template instance RedBlackTree!(element, lessfun) RedBlackTree!(element, lessfun) does not match template declaration RedBlackTree(T, alias less = "a < b", bool allowDuplicates = false) if (is(typeof(binaryFun!(less)(T.init, T.init))))

schveiguy (@schveiguy) commented on 2013-02-15T06:58:45Z

(In reply to comment #3)
> (In reply to comment #1)
> > The second program gives me:
> >
> > temp.d(28): Error: template instance RedBlackTree!(element, lessfun)
> > RedBlackTree!(element, lessfun) does not match template declaration
> > RedBlackTree(T, alias less = "a < b", bool allowDuplicates = false) if
> > (is(typeof(binaryFun!(less)(T.init, T.init))))
>
> The function taking ref arguments can not work for T.init and T.init since ref
> implies lvalue.

Yes, this was recently more strictly enforced.  However, the function will only ever be called on lvalues inside the class

We should change the if statement to something like this:

if(is(typeof({T t; bool x = binaryFun!less(t, t);})))

That should fix the issue.

gassa commented on 2013-02-14T17:06:11Z
Created attachment 1188
Solution 2: template with auto ref

gassa commented on 2013-02-14T16:21:09Z
(In reply to comment #1)
> The second program gives me:
>
> temp.d(28): Error: template instance RedBlackTree!(element, lessfun)
> RedBlackTree!(element, lessfun) does not match template declaration
> RedBlackTree(T, alias less = "a < b", bool allowDuplicates = false) if
> (is(typeof(binaryFun!(less)(T.init, T.init))))

Hmm, I just tried with the current release, DMD 2.061, and it gives me the same error.  However, DMD 2.059 works with no complaint.

The problem arises from the constraint in RedBlackTree declaration:
-----
final class RedBlackTree(T, alias less = "a < b", bool allowDuplicates = false)
    if(is(typeof(binaryFun!less(T.init, T.init))))
-----

The function taking ref arguments can not work for T.init and T.init since ref implies lvalue.

Currently, I found two work-arounds, more or less clumsy.

The first one is to define the second function to avoid the check:
-----
// old one
bool lessfun (ref element a, ref element b)
{
	return a < b;
}

// new one
bool lessfun (element a, element b)
{
	return a < b;
}
-----

Obviously, this way, we need four identical functions to cover all possible cases.

The second one is to use template and auto ref:
-----
bool lessfun (T) (auto ref T a, auto ref T b)
{
	return a < b;
}
-----

This works fine.  And this could be a suitable solution for the original problem.  That is, implementing "a < b" in a similar way as the default predicate for the RedBlackTree!(AnyStructType) container may be not that bad.  But that makes me wonder: under the hood, does it still create the four identical functions?

gassa commented on 2013-02-14T17:05:27Z
Created attachment 1187
Solution 1: two redundant functions

gassa commented on 2013-02-14T17:04:46Z
Created attachment 1186
Example with excessive copying


Validation only when form is submitted

Is there a convenient way to display validation messages under each input only after the form is submitted?
I've read something here: https://github.com/ghiscoding/angular-validation/issues/76
, but I can't find the option and how to use it.

Use checkFormValidity from the ValidationService, this is a common functionality and you can find info in the Wiki - Submit and Validation

I'm using it for the submit button, but I don't know how to disable automatic validation on blur. I only want messages once the button is clicked

You can maybe use this Wiki - Global Options
// hide error messages showing under input elements (False by default)
// useful when we want to use Validation Summary only
hideErrorUnderInputs: false,

or define your own <span> with the logic you want and use the validation-error-to and associate it to your custom error span. Read more about this Wiki - display error to
Just go through the Wikis, there's a lot of info in there. I spent a lot of time building these Wikis so please go through them. The exact feature that you want is not built in, but there's multiple ways to do it with all the options and functionalities available already.


Use rows to avoid reinventing the wheel

rows already deals with a lot of common issues related to reading a CSV file and serializing data. With that in mind maybe we ca reduce the LOC to maintain in this project if we use rows to deal with:

[ ] importing and serializing reimbursement data (basically handling this transformation)
[ ] importing and serializing suspicions data (basically handling this transformation)
[ ] importing and serializing companies data (basically handling this transformation and paying attention to the creation of Activity data)

cc @turicas

Closing in favor of #211


GetPath test fails in Windows

\sui4j\src\test\resources\config\sui.keystore
    assertTrue(StringUtils.endsWith(fileBasedKeyStore.getPath(), "config/sui.keystore"));
This assertion is not working in Windows due to different path separator.
Will create a fix and push it in my forked repo. Don't have access to push here.

Can you please add me as collaborator so that i can push my branch with fix?

Can you please add me as collaborator so that i can push my branch with fix?

@teohaik invited, very glad that you are interested.

already fixed in #106


Bugfix/feature so term 97

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
if you change the schema you must patch the test databases as well, see Updating the schema
the PR must not fail unit testing

Description
Adds feature_so_term() method to Slice, as was done with Feature.
Use case
This is required by methods in Bio::EnsEMBL::Utils::IO::GFFSerializer, among others
Benefits
Possible Drawbacks
Allows fetching SO terms for slices, along with SO accession numbers, as can be done with features
Testing
Yes
If so, do the tests pass/fail?
Pass
Have you run the entire test suite and no regression was detected?
On master

Coverage increased (+0.002%) to 81.457% when pulling 7eb2992368ee2ea0fa791637752bde35fe1dc67c on bugfix/feature_so_term_97 into 9c7ee63989e60fcd556f169edfc49f5192587b2c on release/97.


Add possibility for flow level decorators to have options exposed at …

…the top level.
Also, minor other cleanups regarding PyLint

Superseded by #294


Gulp watch - recursive glob pattern not working

What were you expecting to happen?
I was expecting this glob pattern to work in gulp.watch. It should achieve to watch all files in react-app, and all subfolder recursively.
'./react-app/[!styles]{,**/}**'

It works with gulp.src!
What actually happened?
It did not work, It only watches files in react-app
const gulp = require('gulp');

// It works with gulp.src. In fact the method with {,**/}
// seems the only way to make it work.
const transpileJSX = () => gulp.src('./react-app/[!styles]{,**/}**')
    .pipe(require('gulp-babel')())
    .pipe(require('gulp-uglify')())
    .pipe(gulp.dest('./preload/react.dist/'));

gulp.task('jsx', transpileJSX);

// With gulp.watch this glob pattern does not watch the
// folder and all subfolders recursively
const watchJSX = () =>
    gulp.watch('./react-app/[!styles]{,**/}**', transpileJSX);

gulp.task('jsx:watch', watchJSX);

Meta

gulp - 4.0.2
node - 10.16.0
npm - 6.9.0
yarn - 1.16.0

+1, I have the same problem rightnow, I found it work by passing an ignore file
gulp.src([‘css/**/*.less’, ’!css/module/_variable.less’])

it is weird, whithout the ignore parts, glob does’t work fine

@bluebrown we rely on chokidar for our globbing on gulp.watch. If something doesn't match gulp.src, you should bring it up with them.


Simulate CMB for tube and load nested configurations properly

@msyriac can you check this is not breaking your notebooks?

Ok checking. Please use branches in this shared repo in the future, not forks, there are only two of us working on this repo.

No issues, works fine for me.


Passing SQL error in QuerySQLCheckerTool

Feature request
The SQL Database Agent utilizes the SQLDatabaseToolkit. This toolkit encompasses four distinct tools:

InfoSQLDatabaseTool
ListSQLDatabaseTool
QuerySQLCheckerTool
QuerySQLDataBaseTool

The QuerySQLCheckerTool is designed to identify and rectify errors within SQL code. It employs the QUERY_CHECKER prompt, which is structured as:
QUERY_CHECKER = """
{query}
Double check the {dialect} query above for common mistakes, including:
- Using NOT IN with NULL values
- Using UNION when UNION ALL should have been used
- Using BETWEEN for exclusive ranges
- Data type mismatch in predicates
- Properly quoting identifiers
- Using the correct number of arguments for functions
- Casting to the correct data type
- Using the proper columns for joins

If there are any of the above mistakes, rewrite the query. If there are no mistakes, just reproduce the original query.

Output the final SQL query only.

SQL Query: """

To enhance this, an augmented prompt named CUSTOM_QUERY_CHECKER has been crafted, which incorporates SQL errors returned by the SQLDatabase post-query execution.
The revised prompt reads:
CUSTOM_QUERY_CHECKER = """
Upon executing {query} on the {dialect} database, the following SQL error was encountered, delineated by triple backticks:

{sql_error}

Furthermore, please scrutinize the {dialect} query for typical mistakes, such as:
- Incorporating NOT IN with NULL values
- Preferring UNION over UNION ALL
- Using BETWEEN for non-overlapping ranges
- Witnessing data type inconsistencies in predicates
- Ensuring identifiers are aptly quoted
- Validating the precise number of function arguments
- Guaranteeing the appropriate data type casting
- Employing the correct columns during joins
- Abstaining from appending a semi-colon at the query's conclusion for {dialect}
- Imposing a restriction to yield a maximum of {top_k} results, employing LIMIT, ROWNUM, or TOP as mandated by the database variety. Note that the database type is {dialect}.

Kindly modify the SQL query suited for the {dialect} database. Return only the modified SQL query.
"""

---

With the implementation of this revised prompt, the SQLDatabase can now flawlessly execute the refined SQL without encountering errors.

### Motivation

With the implementation of this revised prompt, the SQLDatabase can now flawlessly execute the refined SQL without encountering errors.

### Your contribution

revised prompt

Thank you very much for your reply.
So, are you going to implement the above in your source and release in next version?

@mail2mhossain that's the bot replying. But I guess he gave you the correct answer. Let me see if the above can be integrated with codebase.

I don't think so we will be needing this in the main codebase. But you can use it as a custom tool.


Using source-directory w/o xattr fails

Output of restic version
restic 0.9.0 compiled with go1.10.2 on linux/amd64
How did you run restic exactly?

restic -r rclone:gdrive:restic --verbose --verbose init

enter password for new repository:
enter password again:
created restic repository 86c6846e1f at rclone:gdrive:restic

Please note that knowledge of your password is required to access
the repository. Losing your password means that your data is
irrecoverably lost.

restic -r rclone:gdrive:restic --verbose --verbose backup /mnt/photos

/mnt/photos is a mount:

///photo on /mnt/photos type cifs (rw,relatime,vers=default,cache=strict,username=,domain=,uid=1000,forceuid,gid=100,forcegid,addr=<IP_ADDRESS>,file_mode=0775,dir_mode=0775,soft,nounix,serverino,mapposix,noperm,rsize=1048576,wsize=1048576,echo_interval=60,actimeo=1,user=,x-systemd.automount)

This mount does NOT allow xattr.
restic output:
open repository
enter password for repository:
repository 86c6846e opened successfully, password is correct
lock repository
rclone: 2018/05/22 19:25:20 ERROR : locks: error listing: directory not found
load index files
rclone: 2018/05/22 19:25:23 ERROR : index: error listing: directory not found
rclone: 2018/05/22 19:25:23 ERROR : snapshots: error listing: directory not found
start scan on [/mnt/photos]
start backup on [/mnt/photos]
error: NodeFromFileInfo: Listxattr: xattr.List /mnt/photos : no data available
new       /mnt/, saved in 0.022s (0 B added, 13 B metadata)

Files:           0 new,     0 changed,     0 unmodified
Dirs:            1 new,     0 changed,     0 unmodified
Data Blobs:      0 new
Tree Blobs:      2 new
Added:      377 B

processed 0 files, 0 B in 0:16
snapshot 99ae013a saved
context canceled B, total 32856 files 13.350 GiB, 1 errors

Notice how restic does not process any files, even if it tells it has a total of 32856 files. After this restic quits and no data is in snapshot
What backend/server/service did you use to store the repository?
google drive via rclone
Expected behavior
restic creates a snapshot containing data
Actual behavior
restic creates an empty snapshot which can't be restored:

restic -r rclone:gdrive:restic --verbose --verbose restore 99ae013a --target /tmp/tmporestore

enter password for repository:
repository 86c6846e opened successfully, password is correct
restoring <Snapshot 99ae013a of [/mnt/photos] at 2018-05-22 19:25:23.762135498 +0200 CEST by <removed>> to /tmp/tmprestore
ignoring error for /mnt: Lchown: lchown /tmp/tmprestore/mnt: operation not permitted
There were 1 errors

restic -r rclone:gdrive:restic --verbose --verbose ls -l latest

enter password for repository:
repository 86c6846e opened successfully, password is correct
snapshot 99ae013a of [/mnt/photos] at 2018-05-22 19:25:23.762135498 +0200 CEST):
drwxr-xr-x     0     0      0 2018-05-10 23:22:39 /mnt

The source directory has an upper layer containing 158 files & directories:

ls -l /mnt/photos | wc -l
158

Steps to reproduce the behavior

create a source-directory which does not support extended attributes
try to create a backup

Do you have any idea what may have caused this?
Based on the errormessage I guess missing extended attributes on the source-directory
Do you have an idea how to solve the issue?
no
Did restic help you or made you happy in any way?
Very happy to find a backup - programm using rclone as backend, Works like a charm on local files.

I've produced the relevant section of a debug log for this. I would really appreciate any help towards solving this issue because nearly everything I have to backup is on a CIFS file system. It might be worth noting that recent samba updates have modified a few related things, one in particular is that stat --file-system --format=%T now returns smb2 instead of cifs.
2018/05/31 18:04:55 b2/b2.go:282	b2.(*b2Backend).List	1	returned 0 items
2018/05/31 18:04:55 archiver/archiver.go:663	archiver.resolveRelativeTargets	1	targets before resolving: [/mnt/resources/Images/ChildFolder]
2018/05/31 18:04:55 archiver/archiver.go:684	archiver.resolveRelativeTargets	1	targets after resolving: [/mnt/resources/Images/ChildFolder]
2018/05/31 18:04:55 archiver/tree.go:233	archiver.NewTree	1	targets: [/mnt/resources/Images/ChildFolder]
2018/05/31 18:04:55 archiver/tree.go:252	archiver.NewTree	1	result:
/mnt, root "/", meta "/mnt"
    /resources, root "", meta "/mnt/resources"
        /Images, root "", meta "/mnt/resources/Images"
            /ChildFolder, src "/mnt/resources/Images/ChildFolder"

2018/05/31 18:04:55 archiver/file_saver.go:75	archiver.NewFileSaver	1	new file saver with 2 file workers and 8 blob workers
2018/05/31 18:04:55 archiver/archiver.go:758	archiver.(*Archiver).Snapshot	1	starting snapshot
2018/05/31 18:04:55 archiver/archiver.go:479	archiver.(*Archiver).SaveTree	1	/ (1 nodes), parent <nil>
2018/05/31 18:04:55 archiver/archiver.go:479	archiver.(*Archiver).SaveTree	1	/mnt (1 nodes), parent <nil>
2018/05/31 18:04:55 archiver/archiver.go:479	archiver.(*Archiver).SaveTree	1	/mnt/resources (1 nodes), parent <nil>
2018/05/31 18:04:55 archiver/archiver.go:479	archiver.(*Archiver).SaveTree	1	/mnt/resources/Images (1 nodes), parent <nil>
2018/05/31 18:04:55 archiver/archiver.go:311	archiver.(*Archiver).Save	1	/mnt/resources/Images/ChildFolder target "/mnt/resources/Images/ChildFolder", previous <nil>
2018/05/31 18:04:55 archiver/archiver.go:391	archiver.(*Archiver).Save	1	  /mnt/resources/Images/ChildFolder dir
2018/05/31 18:04:55 archiver/archiver.go:204	archiver.(*Archiver).SaveDir	1	/mnt/resources/Images/ChildFolder /mnt/resources/Images/ChildFolder
2018/05/31 18:04:55 restic/node.go:653	restic.(*Node).fillExtendedAttributes	1	fillExtendedAttributes(/mnt/resources/Images/ChildFolder) [] Listxattr: xattr.List /mnt/resources/Images/ChildFolder : no data available
2018/05/31 18:04:55 archiver/archiver.go:402	archiver.(*Archiver).Save	1	SaveDir for /mnt/resources/Images/ChildFolder returned error: NodeFromFileInfo: Listxattr: xattr.List /mnt/resources/Images/ChildFolder : no data available
2018/05/31 18:04:55 archiver/archiver.go:147	archiver.(*Archiver).error	1	item /mnt/resources/Images/ChildFolder: error was filtered by handler, before: "NodeFromFileInfo: Listxattr: xattr.List /mnt/resources/Images/ChildFolder : no data available", after: <nil>
2018/05/31 18:04:55 archiver/archiver.go:569	archiver.(*Archiver).SaveTree	1	waiting on 0 nodes
2018/05/31 18:04:55 repository/repository.go:213	repository.(*Repository).SaveAndEncrypt	164	save id ac08ce34 (tree, 13 bytes)
2018/05/31 18:04:55 repository/packer_manager.go:64	repository.(*packerManager).findPacker	164	create new pack
2018/05/31 18:04:55 repository/repository.go:251	repository.(*Repository).SaveAndEncrypt	164	pack is not full enough (45 bytes)
2018/05/31 18:04:55 repository/packer_manager.go:87	repository.(*packerManager).insertPacker	164	1 packers
2018/05/31 18:04:55 archiver/archiver.go:544	archiver.(*Archiver).SaveTree	1	/mnt/resources, saved subtree Tree<0 nodes> as ac08ce34
2018/05/31 18:04:55 archiver/archiver.go:551	archiver.(*Archiver).SaveTree	1	/mnt/resources, dir node data loaded from /mnt/resources/Images
2018/05/31 18:04:55 restic/node.go:653	restic.(*Node).fillExtendedAttributes	1	fillExtendedAttributes(/mnt/resources/Images) [] Listxattr: xattr.List /mnt/resources/Images : no data available
2018/05/31 18:04:55 archiver/archiver.go:771	archiver.(*Archiver).Snapshot	1	saved tree, error: NodeFromFileInfo: Listxattr: xattr.List /mnt/resources/Images : no data available
2018/05/31 18:04:55 archiver/archiver.go:775	archiver.(*Archiver).Snapshot	1	err is NodeFromFileInfo: Listxattr: xattr.List /mnt/resources/Images : no data available, werr is <nil>
2018/05/31 18:04:55 archiver/archiver.go:781	archiver.(*Archiver).Snapshot	1	error while saving tree: NodeFromFileInfo: Listxattr: xattr.List /mnt/resources/Images : no data available
2018/05/31 18:04:55 restic/lock.go:97	main.unlockRepo	1	unlocking repository with lock PID 23119 on zeus by cameron (UID 1000, GID 100)
lock was created at 2018-05-31 18:04:53 (1.298249822s ago)
storage ID 323b0b7f
2018/05/31 18:04:55 cache/backend.go:37	cache.(*Backend).Remove	1	cache Remove(<lock/323b0b7fb6>)
2018/05/31 18:04:55 b2/b2.go:251	b2.(*b2Backend).Remove	1	Remove <lock/323b0b7fb6>

I do not believe this issue lies within Restic's control. It seems that recent samba and/or kernel changes have upset all extended attribute based tools. I've confirmed the same message no data available in even the basic attr binary, and I've seen numerous posts about this same issue across programs. I'm not too hip on the details, but technically it's no longer "unsupported" for samba yet it can cause issues with recent versions.
There is a surefire - perhaps only for the meantime - fix to be applied when mounting the network shares. Add the nouser_xattr option to the mount in fstab and remount the drives. This will essentially force the unsupported message, which Restic et al. will kindly ignore.

Can confirm: mounting with nouser_xattr allows restic to perform the backup

I had a recent similar error concerning xattr cifs errors where Duplicity reported errors. This was using Manjaro Linux .
Again mounting the file shares nouser_xattr as above did the trick. I have a feeling you're likely to see more of these unfortunately ..

I had a recent similar error concerning xattr cifs errors where Duplicity reported errors. This was using Manjaro Linux .
Again mounting the file shares nouser_xattr as above did the trick. I have a feeling you're likely to see more of these type of reported issues unfortunately ..

Is this still an issue? If yes then it should be sufficient to whitelist ENOTDATA when trying to read extended attributes in https://github.com/restic/restic/blob/5fd3dbccb73e6e439b63c3c6be6840a414d3e68f/internal/restic/node_xattr.go#L29 .

whitelist ENOTDATA

Yes, still an issue .
When we use samba share as the backup source it will encounter the same problem.
I fix it by ignoring the error ENOTDATA.  Every thing looks good so I cherry pick the commit and submit a PR.
Please take a look

using nouser_xattr and related GODEBUG from https://github.com/restic/restic/issues/2659 helped me fix the problems in my script (https://forum.restic.net/t/error-lstat-errno-524-during-backup/3272)

Closing this as it should be fixed by #3034.


Removing Python 3.8 from GitHub workflow and adding 3.12

Summary of changes (Bug fix, feature, docs update, ...)

Dropping support for Python 3.8. This has become necessary because the dependencies for openghg no longer support it (e.g. xarray). As a part of this, this also adds a check for Python 3.12 which was released on 02/10/2023.

Please check if the PR fulfills these requirements

[x] Closes #817  (Replace xxxx with the Github issue number)
[x] Added an entry in the latest CHANGELOG.md file if fixing a bug or adding a new feature

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

Comparison is base (e426ce0) 33.64% compared to head (1a85cba) 33.64%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##            devel     #818   +/-   ##
=======================================
  Coverage   33.64%   33.64%
=======================================
  Files         180      180
  Lines       14049    14049
=======================================
  Hits         4727     4727
  Misses       9322     9322

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


EIP-7594: Passive sampling

Since a node doesn't know when it should request the samples from its peers, it's better to passively receive them from the subnets, so that the node doesn't have to guess when to request the samples. This method is called "passive sampling"
The former method, now called "active sampling", will be used only if the node wants to do sampling in the past slots or the passive sampling fails.

I think we should just increase the custody requirement, though not all the way to SAMPLES_PER_SLOT. Custodying 6 to 8 subnets is already sufficient for the kind of security guarantees we need, and anything more than that is cheaper to do through peer sampling because there's no gossip overhead

I think we should just increase the custody requirement, though not all the way to SAMPLES_PER_SLOT. Custodying 6 to 8 subnets is already sufficient for the kind of security guarantees we need, and anything more than that is cheaper to do through peer sampling because there's no gossip overhead

I don't quite follow this. Is this off-topic? The purpose of this PR is that the nodes don't have to guess when the samples arrive at their peers.

I think this is on the research stage, so I will close this PR at the moment.


[Sema] Always look through optionals for unresolved member lookup

Previously, we only looked through optionals on UnresolvedMemberExprs when there were no results from Optional itself. However, in cases where Optional has a member with the same name, this would prevent programs from compiling even if the member on Optional was obviously invalid in the context of the UnresolvedMemberExpr. Consider this example:
import SwiftUI

extension UIColor {
  static let background: UIColor? = UIColor.black
}

let backgroundColor: UIColor? = .background

In SwiftUI, Optional conditionally conforms to View whenever Wrapped conforms to View. Additionally, View has an extension which provides an instance method named background(_:alignment:). This means that unresolved member lookup of background will find the method on Optional, and won't continue to look in UIColor for the user-defined background member.
This PR allows the above code to compile by unconditionally looking into Wrapped for unresolved member lookup, even if we already found results in Optional<Wrapped>. Because we already prefer overloads found on Optional to those found on Wrapped in CSRanking, situations where there are valid members on both should continue to choose the member on Optional (though this definitely needs a source-compat run).
Resolves SR-13815.

cc @xedin @hborla interested in both of your thoughts on this! It seems like this should be source compatible given our ranking logic but I'd like more eyes to make sure I'm not missing something...

@xedin I've taken your suggestion and lifted DeclViaUnwrappedOptional into ScoreKind for UnresolvedMemberExpr overload resolution. I've also added a couple tests to make sure that we resolve to overloads on Optional when there are matching overloads on Wrapped that would otherwise be preferred. Let me know what you think!

@xedin Yeah that makes sense! Is there a good example I could refer to about how to do that? I haven't written a test like that before.

I added a couple of unit tests which use ConstraintSystem and related types in unittests/Sema. You’d need to create a couple of fake declarations, generate constraints for a fake unresolved member reference and check that solution has an expected declaration picked and score doesn’t have member-via-unwrap increased.

Alright, I'll give those a look—thanks! Will ping you when this PR is ready for another look.

Thank you!

@xedin Finally got a chance to sit down with this and realized that relying on the solver output is overkill. All we really need is the type-checked AST, so I've switched to just checking the output of -dump-ast instead. Do you think that's sufficient, or do you still want a unit test here as well?

@Jumhyn I'd still prefer if we had that as a unit test but I think it's okay to use -dump-ast to verify that, at least it's more stable than -debug-constraints.

@Jumhyn @xedin After those changes member lookup here https://github.com/apple/swift/blob/a9f5107b44c71929521ffbf3a91c5081554f272d/lib/Sema/CSSimplify.cpp#L7183 will return both candidates for Optional and Wrapped types? Just checking as I'm trying to change the CSApply warning to a fix and right now we have to do another lookup ...

@LucianoPAlmeida Yes, the results from that line should include candidates from both Optional and Wrapped (but only when we’re doing lookup for an UnresolvedMemberExpr).

@LucianoPAlmeida Yes, the results from that line should include candidates from both Optional and Wrapped (but only when we’re doing lookup for an UnresolvedMemberExpr).

Great thanks, I think that could simplify things in that case :)

@LucianoPAlmeida Unfortunately, I probably won't have a chance for the next couple of days to write the unit test to get this mergeable, so you may have to wait for a bit for the changes here to be usable for your fix.

@LucianoPAlmeida Unfortunately, I probably won't have a chance for the next couple of days to write the unit test to get this mergeable, so you may have to wait for a bit for the changes here to be usable for your fix.

No worries @Jumhyn :)
Those are very simple changes so I've already "cherry-picked" to my branch to work with them. It did break the implementation that I had already so it will take some time for me to have something also.

@xedin @Jumhyn Quick question, in this case
enum Foo {
  case bar
  case none
}

let _: Foo?? = .none

Name lookup is actually returning only 2 viable candidates where the base is Foo?? and Foo?, so my question is if that is expected and intentional or it should have dig deeper and find also base Foo?

@LucianoPAlmeida It intentionally only unwrap a single layer of optionality at the moment.

@LucianoPAlmeida It intentionally only unwrap a single layer of optionality at the moment.

Ah, thank you @xedin :)
I think that will require another lookup then in order to record the fix correctly then

That would be okay since it's performed in diagnostic mode anyway.

That would be okay since it's performed in diagnostic mode anyway.

Humm, that may be a problem because since it is a warning we have to always record it ... but on the other hand the extra lookup will be done only in certain situations involving deep optionals. I'm finishing up the changes and pushing, so we can discuss it there :)

Thank you for your patience @xedin! Finally had a chance to sit down an write up the unit test(s). Let me know if this matches what you were picturing.
cc @LucianoPAlmeida, this will hopefully be merge-ready soon! :)

@xedin Updated per your comment! Would you mind kicking off the tests on this?

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - cd745f3c5d250b831b3ae680ca01afbd7c4fe852

Build failed
Swift Test OS X Platform
Git Sha - cd745f3c5d250b831b3ae680ca01afbd7c4fe852

@LucianoPAlmeida Latest push addresses your comments and fixes the test failure on Windows (in theory🤞). From what I could tell the Linux and macOS test failures were not related to the code touched here... mind kicking off another round of tests?

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test OS X Platform
Git Sha - 7b35e577c8d1103e05018a87badac6b392daa47e

@swift-ci please test MacOS Platform

@swift-ci Please test OS X platform

Windows failure looks like a flake. Rebased against main to see if that resolves the (seemingly unrelated) macOS failure... @xedin @LucianoPAlmeida one more round of tests, please?

@swift-ci please test

@swift-ci please test source compatibility

@xedin All checks passed!


GEOMESA-1747, GEOMESA-1748 Arrow integration

Arrow file-based data store implementation
Arrow accumulo iterators for push-down processing
Arrow WPS process and query response

single place for arrow work. Currently requires building https://github.com/apache/arrow/pull/452 and https://github.com/ccri/arrow-geometry

closing in favor of #1481 and #1482


Boolean input validation fails

Hi there,
I'm trying to use a boolean input from a terraform module, however terragrunt plan fails because the terraform variable validation block does not recognize the value that I'm passing.
This problem exists only when I use that terraform module with terragrunt.
If I don't pass that input, the variable is correctly initialized to its default value.
Here follows some code.
My input file:
inputs = {
  ...
  aft_vpc_endpoints           = false
}

The variable definition on the terraform module:
variable "aft_vpc_endpoints" {
  type        = bool
  description = "Flag turning VPC endpoints on/off for AFT VPC"
  default     = true
  validation {
    condition     = contains([true, false], var.aft_vpc_endpoints)
    error_message = "Valid values for var: aft_vpc_endpoints are (true, false)."
  }
}

The error I receive:
╷
│ Error: Invalid value for variable
│
│   on variables.tf line 94:
│   94: variable "aft_vpc_endpoints" {
│
│ Valid values for var: aft_vpc_endpoints are (true, false).
│
│ This was checked by the validation rule at variables.tf:98,3-13.
╵
ERRO[0192] 1 error occurred:
        * exit status 1

Terraform version: 1.1.7
Terragrunt version: 0.36.6

Hi,
since terragrunt passed inputs as environment variables, looks like it is an issue on how terragrunt parse variables, same error occurs when terraform is invoked with TF_VAR_xxx:
$ TF_VAR_aft_vpc_endpoints=true terraform plan
│ Error: Invalid value for variable
│
│   on main.tf line 1:
│    1: variable "aft_vpc_endpoints" {
│
│ Valid values for var: aft_vpc_endpoints are (true, false).
│
│ This was checked by the validation rule at main.tf:5,3-13.

References:
https://github.com/hashicorp/terraform/issues/30609

Hi @denis256, thank you for the reference!
I just tried updating to the latest alpha of Terraform 1.2.0 (v1.2.0-alpha20220413) and terragrunt plan seems to run smoothly.
I tried to find a page to see if version 1.2.0 is already scheduled for release, but I was unlucky: do you have any reference to something similar to a release calendar?

Same bug as here Waiting for the 1.2

Since we're now well past the Terraform 1.2.0, I'm closing this issue.


Write tests for the existing code

For testing e. g.  the waveform functions, consider creating a framework for asserting on a particular function shape.

Given that both the CLI and the Web version of the apps are going to be rewritten to some extent (#14, #18), there's little point in writing unit tests for their current code. I'm thus resolving this task. Yeah, finally!


"RGB" or "BRG"

Thanks for your work! I have a question, the input of the RGB stream is 'RGB' or 'BRG'?  Seems like in your codes,  using opencv to load images, whose format is 'bgr'..   THX!

It is RGB:
https://github.com/piergiaj/pytorch-i3d/blob/master/charades_dataset.py#L31
converts from BGR to RGB.

THX！


Optimising the scene menu creation process, remove CategoriesSet

No longer use registry->categories() to create parent items
Automatic sorting
removed CategoriesSet
fix: Any value entered does not match, because a non-existent UserData is used

I looked up where CategoriesSet is used in v2 and v3 and found that it is only used when creating menus. RegisteredModelsCategoryMap is perfectly capable of doing the job of CategoriesSet.
So I think the CategoriesSet can now be removed.
Please contact me if I have any incomplete considerations.
Thanks.

I use std::vector to store the RegisteredModelsCategory, and since the class seems to be used only to generate the Scene Menu, just traversing it without using lookups, using std::vector not only sorts the classes in the order they are registered, but also reduces the performance loss (though not by much)
I don't think it's a good idea to use ret->sortCategories() to sort at the end of the registration model. Because I won't know which model will be loaded last in my plugin_extensionbranch because of the plugin name change.
I have used a bool variable to flag whether or not the Menu needs to be sorted when it is generated, but I think the best way to do this would be to place a Button in the Menu to choose whether or not to sort it, and to be able to take effect the next time the Menu is generated.

Also I think it should just sort the subdirectories, for example Addition in the first.

Sort button function


Update RUN cmds for 3x TTMetal tests to include "ttmlir-translate --ttmetal-to-flatbuffer"

Needed after 5d60c17 today otherwise tests don't run on CI

Merging now to restore missing tests from CI- thanks Nick for cryptic mac build linking fix!


Upgrade Quarkus to 2.3.0.CR1

So we now have a situation where errors in docs on the main branch of Camel or its sub projects can break our build?
{"level":"error","time":1632310250575,"name":"asciidoctor","file":{"path":"docs/modules/ROOT/pages/list.adoc"},"source":{"url":"https://github.com/apache/camel-spring-boot.git","refname":"main","startPath":"docs"},"msg":"target of xref not found: latest@components::hwcloud-image-component.adoc"}

try again I fixed the SB xref


Stage 3

Testing strategy
The acceptance tests should be more strict with what we are checking:

For the incomplete test we should assert on the uploaded S3 parts and check their ETAGs. The ETAG returned for a part is related to the MD5.
For the completed upload, we should check the ETAG for the final upload.

The tests should be independent of the ordering ( no need for @FixMethodOrder(MethodSorters.NAME_ASCENDING) ).

It is ok to duplicate some code from the incomplete test.
The rule to delete all uploaded objects and multipart uploads should be run before every test.

Tech debt

fileUploader.setStrategy is a smell, fileUploader.upload(file, name) can receive the strategy as a parameter ( we should prefer immutability were possible )
rename UnfinishedUploadingFileUploadingStrategy(null) to MultiPartUploadFileUploadingStrategy(MultipartUpload upload) and perform the initialisation outside the class ( MultipartUpload should never be null )
reuse MultiPartUploadFileUploadingStrategy for large file upload

Resilience (now)
The last part of an incomplete upload can be smaller than 5MB and this could break the upload:

If the file is not complete, we could skip that part
If the file is complete, we could pad the last part to artificially increase its size, this seems to be what people are doing. Check out this file: https://github.com/alexmojaki/s3-stream-upload/blob/master/src/main/java/alex/mojaki/s3upload/MultiPartOutputStream.java#L94

We should be providing a part MD5 to the upload to prevent in-flight corruption. UploadPartRequest.withMD5Digest()
Resilience (next iteration)
The multipart can fail during upload (process could be killed, connection lost)

we need to be able to simulate these conditions in our acceptance test
getUploadedSize(alreadyUploadedParts) assumes that all the parts are uploaded and in sequence

could the parts be out of sequence?, one part failed to upload but not the others

Performance and resilience ( for future iterations, added here for reference )
Upload parts in parallel

using a ThreadPoolExecutor with fixed number of threads
the last part before completion is uploaded as one single part, should be broken down in smaller parts and uploaded in parallel

Right now, we retrieve all multipart uploads which is not scalable and not secure

we should upload files with a unique prefix per user (uniqueId)
we can retrieve uploads using the prefix, should only return the uploads for the user


Update installed software.

Most notably I updated ruby to 2.5.
As we anyway need to increase the major version for this I'm considering to also update the
baseimage to a more recent ubuntu version and thereby fixing some of the vulnerability alerts shown
in https://hub.docker.com/r/ad2games/docker-rails/tags/latest/

Hey, let's upgrade once more, merge and release this?


docs: Fix issues with hashes on ACT log line examples

Fixes #3405
Might be merge conflicts between this PR and #3423, I'm not sure. If there are, I'd rather this PR get rebased/merged second because it's the simpler of the two and will be easier to rebase.
Also includes the other adjustment mentioned in the linked issue to make things more uniform in LineEvent classes.
Also also fixes 0x00 and 0x0C lines having their last structure parameter dropped.

Rebased to fix merge conflicts

Thanks for fixing this too!


riot-tag attribute not working

I tried to use riot-tag attribute in a <div>but didn't work
http://plnkr.co/edit/GkpfWR?p=info

If you try it with the latest version (v2.1.0) of Riot it will work.

Thanks, it worked fine


java error while running juicer on cluster

Dear Juicer team,
I am running HiC data using CPU version on our server system at Ugent. Can you tell me am i doing anything wrong here or is it a memory issue.

[Experiment]$ less 118/log.txt
    at juicebox.tools.clt.old.AddNorm.run(AddNorm.java:84)
    at juicebox.tools.HiCTools.main(HiCTools.java:96)

real    0m1.973s
user    0m0.899s
sys     0m0.394s
Tue Jul  5 23:12:44 CEST 2022
Picked up _JAVA_OPTIONS: -Xmx150000m -Xms150000m
Not including fragment map
Error while reading graphs file: java.io.FileNotFoundException: /data/gent/Test_Bijal/Experiment/118/aligned/inter_30_hists.m (No such file or directory)
Start preprocess
Writing header
Writing body
/data/gent/Test_Bijal/scripts/common/juicer_tools: line 24: 3590737 Killed                  java -Djava.awt.headless=true -Djava.library.path=dirname $0/lib64 -Ddevelopment=false -jar dirname $0/juicer_tools.jar $*
real    0m30.377s
user    0m23.094s
sys     0m9.693s
Picked up _JAVA_OPTIONS: -Xmx150000m -Xms150000m
Error reading datasetnull
java.io.EOFException
at htsjdk.tribble.util.LittleEndianInputStream.readString(LittleEndianInputStream.java:119)
at juicebox.data.DatasetReaderV2.read(DatasetReaderV2.java:127)
at juicebox.tools.utils.original.NormalizationVectorUpdater.updateHicFile(NormalizationVectorUpdater.java:78)
at juicebox.tools.clt.old.AddNorm.run(AddNorm.java:84)
at juicebox.tools.HiCTools.main(HiCTools.java:96)
real    0m2.084s
user    0m0.909s
sys     0m0.367s
/data/Test_Bijal/scripts/common/juicer_tools is post-processing Hi-C for hg19
Data read from /data/Test_Bijal/Experiment/118/aligned/inter_30.hic.
Motifs read from /data/Test_Bijal/references/motif
ARROWHEAD:
Picked up _JAVA_OPTIONS: -Xmx150000m -Xms150000m
Exception in thread "main" jargs.gnu.CmdLineParser$IllegalOptionValueException: Illegal value '/data/Test_Bijal/Experiment/118/aligned/inter_30.hic' for option -z/--threads
at jargs.gnu.CmdLineParser$Option$IntegerOption.parseValue(CmdLineParser.java:202)
at jargs.gnu.CmdLineParser$Option.getValue(CmdLineParser.java:157)
at jargs.gnu.CmdLineParser.parse(CmdLineParser.java:485)
at jargs.gnu.CmdLineParser.parse(CmdLineParser.java:426)
at juicebox.tools.HiCTools.main(HiCTools.java:61)
***! Problem while running Arrowhead
***! Error! Either inter.hic or inter_30.hic were not created
Either inter.hic or inter_30.hic were not created.  Check  for results

i would appreciate if you look into this matter.
Regards
Bijal Thakkar

I also encountered this problem when running ARROWHEAD. There is no "merged_nodups.txt" in aligned.

Hi,
I met the problem of "no merged_nodups.txt", too. That is because I didn't use "-assembly" to produce "old style merged_nodups", as they said in the "--help".
I guess you can either,

Add "-assembly" when you restart to run juicer.sh .
Execute this command line under the working folder,
samtools view -@ ${ThreadsNum} -O SAM -F 1024 aligned/merged_dedup.*am | awk -v mnd=1 -f ${juiceDir}/scripts/sam_to_pre.awk > aligned/merged_nodups.txt
, modified from line #1929 of SLURM/scripts/juicer.sh .

I choose 2 to save my time.


Correctly use the return value of slices.Delete

Fixes: #1645

⚠ this line deletes i- i = 0 elements, this is probably not what we want

⚠ this line deletes i- i = 0 elements, this is probably not what we want

Thank you for picking up on that @Deleplace, fixed in https://github.com/google/osv.dev/pull/1646/commits/13fc97e133ca204dbd13b91e82857c04c07f4c33


Why some data in longbook_qa_eng were modified?

Hey great efforts on constructing InfiniBench! One quick question I've noticed:
In a recent upload to HF dataset https://huggingface.co/datasets/xinrongzhang2022/InfiniteBench/commit/f2fd8f04ea3af8304b88de2c58bd33887bcccdb8
You changed 20 questions and answers compared to the original commit in: https://huggingface.co/datasets/xinrongzhang2022/InfiniteBench/commit/bae90aaac350652b7c7292dcf556bdd52542929b
Consequently some numbers based on the previous / current dataset may not be directly comparable. May I ask what are the considerations behind this?
Thanks!

we correct some typos. Besides, we also update some Q&A to enhance quality, difficulty, and long dependency required by Q.


fix: required customer id and email error if not passed in confirm request

Type of Change

[x] Bugfix

Description

We used to receive a missing required field error if the customer_id or email_id is not given in the confirm request.The reasoning for this is that if the customer_id and email were not sent with the request, we would not be able to confirm.Therefore, this issue was resolved by getting it from the database or, if present, the request.
Motivation and Context

Fix missing required field error
How did you test it?

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed submitted code

closing this PR, as #1317 will solve this problem


Agent healthcheck history

Agent view should show healthchecks over time (like we show tasks in the home view)
We can save recent history in Redis, maintain a sliding window by pushing/popping from a queue

done as of 959a7bc


Pseudo elements like ::part and ::theme

Hi, is there a chance that the pseudo elements ::part and ::theme work in all browser?
It works for Chrome and opera but not in Firefox or Edge.
Thank you

There is a branch with some experimental work here that we did awhile back. We're not super happy with the performance impact of this approach and we're not sure right now if/when/how we're going to carry this forward.
One thing we think definitely makes sense and can be implemented with minimal performance impact is to use part attributes in an element's DOM and then have a system that can inject stylesheets into elements that target (only) these parts. This approach is not strictly a polyfill for ::part but it is inspired by and compatible with it and gives some of the same functionality. There's an example in that same branch here. Hope that helps.


fix: properly require 1 element in MailDataRequired type

Current type does not work with the latest versions of typescript. This updates the type so it remains an array, and not the intersection of an object and an array.

Fixes
A short description of what this PR does.
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the Contribution Guidelines and my PR follows them
[x] I have titled the PR appropriately
[x] I have updated my branch with the main branch
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added the necessary documentation about the functionality in the appropriate .md file
[x] I have added inline documentation to the code I modified

If you have questions, please file a support ticket.

Would appreciate seeing this released 🙂
Thanks!


Error Compiling

unhandledRejection .....\node_modules\capacitor-blob-writer\blob_writer.js:3
import {Capacitor, registerPlugin} from "@capacitor/core";
^^^^^^
SyntaxError: Cannot use import statement outside a module

Your compiler is misinterpreting interpret blob_writer.js as a CommonJS module when it is actually an ES module. It should be using blob_writer.umd.js, which is compatible with CommonJS. See package.json.

Please try v1.1.18.

tried v1.1.19
compiles
now getting files on android
But weird thing is happening, the files are almost twice the original size, and cannot open in native apps, i.e. pdf or image says invalid format.  I write to the datadirectory.documents

Strange. Try running the demo to see if the tests pass on your device.

FYI found the issue with doubling size and invalid format, it's a capacitor issue on Android/IOS.
when using capacitor it translates the fetch to use native, however in their wisdom the response is typed as Blob, but actually returns a Base64 encoded Blob, which is no use to anyone.
https://github.com/ionic-team/capacitor/issues/6241
Still in current version

Thanks.


useDefaults: 'empty' for undefined values

What version of Ajv you are you using?
6.7.0
What problem do you want to solve?
According to the readme doc With the option value "empty" properties and items equal to null or "" (empty string) will be considered missing and assigned defaults. useDefaults: 'empty' should work only if value is '' or null, but ajv creates a field/value even if in source object particular field does not exists
What do you think is the correct solution to problem?
do not create default value for undefined fields.

@epoberezkin the use case is:
const Ajv = require('ajv');

const ajv = new Ajv({
    allErrors: true,
    coerceTypes: true,
    removeAdditional: false,
    useDefaults: 'empty',
});

const obj = { a: '' };

const comp = ajv.compile({
    type: 'object',
    properties: {
        a: { type: 'number', default: 0 },
        b: { type: 'number', default: 0 },
    },
});
comp(obj);
console.log(obj);

Actual result:
{ a: 0, b: 0 }
Expected result:
{ a: 0 } // b is absent in source object and don't needed after validation
Thanks in advance

I was asking about an actual use case, not about the code sample. The way default: "empty" is defined, it is useful as it can be used, e.g. both for form submissions when empty fields can be present as empty strings and for API calls when the properties will be absent. What is the use case of the above?

@epoberezkin

API calls when the properties will be absent
I need to validate an object (from POST request). One thing what is needed to decline comparison with undefined properties only with '' (empty string) and null. Why it's needed: in the code determined some business logic if one key is absent do not use another one, however, ajv creates a key with a default value, and it's to hard check if that key is passed through API or created by ajv.

@akaNightmare you need to structure your schema differently to achieve it. The behaviour of the default keyword will not change to achieve what you describe.


persistBy: "create" not deleteAll data on relation

Hello
I found bug with { persistBy: "create" }
the item relation is not clear or deleteAll() before/after call order.api()
please review this bug from codesanbox
https://codesandbox.io/s/vuex-orm-relation-test-h6xmb?file=/src/App.vue
** item is exists after call order.api() **
   await Order.api().get("/orders.json", {
     persistBy: "create",
     persistOptions: {
       create: ["merchant_item"]
     }
   });

or

   await Order.api().get("/orders.json", {
     persistBy: "create",
   });

I thank when set "persistBy: "create"," or persistOptions
deleteALl() any data on relations so will be correct
Thank you.

@mean-cj thanks for opening this issue.
This is expected behaviour, and I wouldn’t consider this a bug. For create to have any effect on a model there needs to exist at least one record for the target relation. Since the reproduction example you’ve provided yields no relations in the second call, the normalisation omits the merchant_item altogether subsequently not invoking the create command.
In this scenario, you will almost always want to programmatically remove relations before creating parent models because Vuex ORM is unable to make that decision for you.
There was a similar discussion where an option could be made available to purge an entity if there are no records in the corresponding API request but this is something, at least for now, up to the developer to validate and handle.

thanks for your so much for your suggestion and your code.
what do you think if there are support deleteAll option on persistOptions
for the choice to clear all data relation before call this { persistBy }

what do you think if there are support deleteAll option on persistOptions for the choice to clear all relation data before call this { persistBy }

There will be support for this feature in the next iteration of Vuex ORM. For the time being you may have to resort to the more verbose approach.

Hello @cuebit
When i return with save() , the promise return only catch
return orders.save()
    Order.api().refresh()
        .then(({ response }) => {
          console.log('then')
        })
        .catch(() => {
          console.log('catch')
        })

I don't know why
thanks

I think action only catch because save() return PromiseValue is undefined
console.log( orders.save() )
Promise {<pending>}
__proto__: Promise
[[PromiseStatus]]: "resolved"
[[PromiseValue]]: undefined


Fix build warnings

💭 In the future we can consider alternatives to [Obsolete] for experimental features, but for now it seemed to make sense to clean up the build.

Codecov Report

Merging #227 into master will decrease coverage by <.01%.
The diff coverage is 0%.

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

@vancem I'm planning to move commit 5a66428 from this PR to a separate PR, and remove it from this one.

@sharwell any chance you could merge the commits?  Vance added me to a change comment, but I am having a hard time finding it.  WRT using obsolete as a means to indicate experimental, I am OK ingeneral with removing them.  There were a few that are there really only to support PerfView and we should come up with another strategy to help identify those (I saw the comments where y'all were discussing options).

@jeffschwMSFT The comment appears above but it's now collapsed since I removed the commit that contained the original change:

any chance you could merge the commits?

I can, but my preference is always to keep logically separate commits separate. For this PR, each commit is an independent unit that would successfully build and test.

I am going to pull out the hasProcessEvents variable since it is no longer used, but I will do that after merge.


Made def syntax more regular.

Currently, definitions have two different syntaxes: def <pat> = <expr> and def <id> <pat1> ... <patN> { <expr> }. Both forms of definitions can be used to define functions. (The second form can be thought of as a value <id> parameterized by <pat1> through to <patN>.)
Whenever I'm parameterizing a function, for example changing def myrange = range to def myrange a b { range a b }, I frequently forget to insert braces and remove equals sign, so I end up with the following currently invalid code: def myrange a b = range a b.
In order to reduce the likelihood of users making this mistake and ease describing the syntax, this pull request changes the second syntax for definitions to be def <id> <pat1> ... <patN> = <expr>.

Seems fine trying to unify it like this, does seem like a good change

@murisi I keep getting a stack overflow when running this branch after including the fresh_algos commit

@murisi I keep getting a stack overflow when running this branch after including the fresh_algos commit

Thanks. The stack overflow was caused by the pest parser infinitely recursing on vampir source code. This was in turn caused by the presence of unbounded recursion in the PEG (vampir.pest) which is sometimes triggered when source files are syntactically incorrect.
I've now fixed the syntax errors in the vampir source files (alu.pir and one or two others), and I've also fixed the unbounded recursion in the PEG by using a negative lookahead (to prevent the parser from retrying to parse the same string endlessly). The parser will now print out the incorrect source code fragment instead of overflowing.


This PR fixes the logic that is used to detect CI and skip these tests.

This piece of code had been copy/pasted from another test, and should have read "This test only runs on Travis", or rather the check would be that web-push-service-test is installed and running on the machine
I'm not for abandoning tests on CI yet, only if solution 3 of https://github.com/web-push-libs/web-push-php/issues/274#issuecomment-610746518