Feature/supertest fixes #22

This fixes #22

Ready for y'all to tear apart

Ideally (IMHO) npm run lint would be run as part of the build, but baby steps

@faceleg This looks nice, I'll further review in the morning.
One thing I would like to see is a move towards the org standard that is happening; best seen in the level store https://github.com/senecajs/seneca-level-store. I'd rather not bring gulp in right now as we are trying to keep the testing (with lab) and the linting (eslint) super simple. We are following Standard's rules for the repo updates we are doing.
Regardless, the supertest stuff looks pretty epic :D I'll take a look in detail first thing in the morning, but good work :+1:

I'll remove the gulp bits, no problem

@faceleg Awesome :D I'm a gulp user myself but have to keep it super simple for org repos for new contributors. If you check the level store you will see it's eslint too, it will make yours smaller but it's more or less the same rules. As I said though, super work! This is why I love Open Source :D

IMHO the gulpfile makes it even easier for new contributors, as the deps are installed with the npm test pretest command, husky automatically sets up the git hooks, and the linting / tests are run ON COMMIT, so there is less friction for new devs as they don't actually need to do anything - their computer tells them if they've done it all right, in terms of tests passing / linter linting.
But I'll be happy conforming to whatever standards or opinions you "The Project" requires, and won't be so presumptive again :)

I'll make another feature branch based on this to work on #21

@faceleg No no!!
 Sometimes org rules take precedent and that's ok but other times, like with supertest, we never even considered it. We love having people contribute, especially improvements. We love when people reach out and take ownership like this; we positively encourage it!

In addition, if you do have any opinions or thoughts on org wide issues we would love to hear them, stuff like refining building and testing is welcome; we even have a special place set up for it here,
https://github.com/senecajs/contrib
If you want to add to the issues there we can get the ball rolling and a discussion going. As I said sometimes ideas won't fit.... but other times they do :smile:

Opened https://github.com/senecajs/contrib/issues/34

@faceleg Awesome :dancer:

Related: senecajs/contrib#35

Soooo the tests fail in node 0.10 because "Promise" is not defined.
Options:

Remove use of promises
Use bluebird in dev
Remove node 0.10 from .travis.yml

@mcdonnelldean @rjrodger

Is option 2 going to work for you all?
@rjrodger @geek @mcdonnelldean

Does it need to use promises?  If it does, then bluebird sounds like a plan.

I can't argue that it must use promises. Given a choice between a promise and a callback, I will choose promise everytime.

@geek @faceleg Was it using promises before this? How was it doing it previous? I've heard good things about bluebird so if we need it I'm :+1: I just want to see what was it before? Sorry, ramping up.
Is it just because of node .10 ?

The relevant file (test/example.js) didn't use any form of async.
I made the current change so the tests can get the app variable, which supertest needs in order to be able to test an express app.
If one runs it naked, it'll start listening on port 3000 as it did before.
Prior to my change, the example.js file did not export anything, as any tests were done using bash.
I've used bluebird for a good long time, it's great. I originally intended just to use native promises for this simple implementation, but as Promise doesn't exist in 0.10, I had to drop bluebird in there.
The alternative is to refactor this portion to use callbacks instead of promises, but I humbly submit to you that promises are so much superior to callbacks in terms of readability composability and maintainability that we should be promoting their use wherever possible.

Cheers! for the response. Let me give this all a look over tomorrow.
Thanks a mill for the work so far :D On light review it all looks awesome :D

No problem, supremely happy to have the chance to contribute

@faceleg First off, Awesome work! This is a really nice PR.
The only thing it needs is a good lint, there are some semi-colons and additional spaces in there. If you can get these knocked out and hook up linting to be forced on npm test that would be awesome, we can get it merged for ya :D

When I run standard against the repo I get hundreds of lines of output, including for files I haven't touched.
I'll fix it, but it'll take time.

Witness: https://gist.github.com/faceleg/460b0e1d4146cb38876c

Only do the ones you touched, for the others, ignore them with a .eslintignore you don't need to do the whole rep :D

:P

Shall I close this?

@faceleg No no, can you make it remergable?

@faceleg can you update to be mergable?

Doing so now

@geek ready to go

Thanks @geek!

@faceleg should be published tomorrow, thank you


False positive resource leak in Java

I have this code using a try-with-resources and Infer still generate a warning :
I am using Infer version v0.9.5.
            try (
                    // Let's anticipate up to 80 characters on top of those already read.
                    ByteArrayOutputStream out = new ByteArrayOutputStream(end - pos + 80) {
                        @Override
                        public String toString() {
                            int length = (count > 0 && buf[count - 1] == CR) ? count - 1 : count;
                            try {
                                return new String(buf, 0, length, charset.name());
                            } catch (UnsupportedEncodingException e) {
                                throw new AssertionError(e); // Since we control the charset this will never happen.
                            }
                        }
                    }
            ) {

                while (true) {
                    out.write(buf, pos, end - pos);
                    // Mark unterminated line in case fillBuf throws EOFException or IOException.
                    end = -1;
                    fillBuf();
                    // Try to find LF in the buffered data and return the line if successful.
                    for (int i = pos; i != end; ++i) {
                        if (buf[i] == LF) {
                            if (i != pos) {
                                out.write(buf, pos, i - pos);
                            }
                            pos = i + 1;
                            return out.toString();
                        }
                    }
                }
            }

RESOURCE_LEAK ERROR HIGH
resource of type ...cache.lru.StrictLineReader$1 acquired to out by call to new()
at line 151 is not released after line 180

line 151 is ByteArrayOutputStream out = new ByteArrayOutputStream(end - pos + 80) {
Looks like this is a false-positive, is there an issue with Infer ?

+1 as I have faced a similar false positive.

Infer will treat every class implementing Closeable as resource by default. However, for some reason, ByteArrayOutputStream does not need to be closed. So we model this behavior here: https://github.com/facebook/infer/blob/master/infer/models/java/src/java/io/ByteArrayOutputStream.java.
However, when doing
 new ByteArrayOutputStream(end - pos + 80) {
   ...
 }

The code actually creates a new subtype of ByteArrayOutputStream and the analysis does not know that this new subtype does not need to be closed, in spite of implementing Closeable.
So at the end, the issue is more in the design of the Java API in which some implementations of Closeable need to be closed, some other don't, and with on way do distinguish between the two case except by manually modeling the corner cases, which is not possible for anonymous classes.

Thanks for the explanation, what is unexpected for me is this ByteArrayOutputStream is opened in a try-with-resources block so it should be closed.
try (
  new ByteArrayOutputStream(end - pos + 80) {
    ...
   }
) {
  ...
}

Even if ByteArrayOutputStream the try-with-resources will close it, right ?

Okay, I did not see the try was from a try-with-resources. Do you still get the leak if you remove the line:
   throw new AssertionError(e);

?

The simples case that I could find is:
import java.io.ByteArrayOutputStream;
import java.io.FilterOutputStream;
import java.io.IOException;
import java.io.OutputStream;

public class Main {

    public static void main(String[] args) throws IOException {
        ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream();
        try (DummyFilterOutputStream os = new DummyFilterOutputStream(byteArrayOutputStream)) {
        }
    }

    static class DummyFilterOutputStream extends FilterOutputStream {
        DummyFilterOutputStream(OutputStream outputStream) {
            super(outputStream);
        }
    }
}

This produces the following error:
Main.java:13: error: RESOURCE_LEAK
   resource of type `com.company.Main$DummyFilterOutputStream` acquired to `os` by call to `new()` at line 12 is not released after line 13
  11.           ByteArrayOutputStream byteArrayOutputStream = new ByteArrayOutputStream();
  12.           try (DummyFilterOutputStream os = new DummyFilterOutputStream(byteArrayOutputStream)) {
  13. >         }
  14.       }
  15.

However as soon as you replace DummyFilterOutputStream with FilterOutputStream it works without any errors.
Tried using:
Infer version v0.12.1

This also happens when explicitly closing the stream in a finally block.
try {
    os = new DummyFilterOutputStream(byteArrayOutputStream);
} finally {
    os.close();
}

Produces:
Main.java:16: error: RESOURCE_LEAK
   resource of type `com.company.Main$DummyFilterOutputStream` acquired to `os` by call to `new()` at line 14 is not released after line 16
  14.               os = new DummyFilterOutputStream(byteArrayOutputStream);
  15.           } finally {
  16. >             os.close();
  17.           }
  18.       }

I have seen this behavior in the wild and with a simpler test case:
    try (final FileInputStream fis = new FileInputStream(f)) {
      while ((bytesRead = fis.read(buffer)) >= 0)
        checksum.update(buffer, 0, bytesRead);
      return checksum.getValue();
    }

Infer says:

RESOURCE_LEAK: resource of type java.io.FileInputStream acquired by call to FileInputStream(...) at line 421 is not released after line 425. Note: potential exception at line 422

I have faced a similar false positive:
#0 src/main/java/edu/jiangxin/apktoolbox/file/core/EncoderConvert.java:59: error: Resource Leak   resource of type java.io.FileInputStreamacquired by call toFileInputStream(...) at line 59 is not released after line 59.   57.      58.           // TODO see http://akini.mbnet.fi/java/unicodereader/   59. >         try (BufferedReader reader = new BufferedReader(   60.                   new InputStreamReader(new FileInputStream(srcFileFile), srcEncoder));   61.                   BufferedWriter writer = new BufferedWriter(
https://raw.githubusercontent.com/jiangxincode/ApkToolBoxGUI/infer-out/report.txt

The false-positive looks like affects every try-with-resources, just another example:
  resource of type `java.util.zip.ZipFile` acquired to `zipFile` by call to `ZipFile(...)` at line 168 is not released after line 168.
**Note**: potential exception at line 178
  166.           Path destFile = getDestFile().toPath();
  167.           Path tmpZip = Files.createTempFile( destFile.getParent(), null, null );
  168. >         try ( ZipFile zipFile = new ZipFile( getDestFile() );
  169.                 ZipOutputStream out = new ZipOutputStream( Files.newOutputStream( tmpZip ) ) )
  170.           {

(Infer 1.1.0)


Cannot download https://www.youtube.com/watch?v=qevIIQHrJZg

Checklist

[ x] I'm reporting a broken site support
[x ] I've verified that I'm running youtube-dl version 2021.04.17
[ x] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ x] I've searched the bugtracker for similar issues including closed ones

Verbose log
youtube-dl.exe --verbose "https://www.youtube.com/watch?v=qevIIQHrJZg"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=qevIIQHrJZg']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.04.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] qevIIQHrJZg: Downloading webpage
[debug] Default format spec: best/bestvideo+bestaudio
[debug] Invoking downloader on 'https://r1---sn-8xgp1vo-cvnl.googlevideo.com/videoplayback?expire=1618695084&ei=TP96YJuRMJqYhgbg5oCoCQ&ip=<IP_ADDRESS>&id=o-ALygxFavfjmmDmu6leVxkzVZqOEdaDM1SjGjvBrkgmYm&itag=22&source=youtube&requiressl=yes&mh=rW&mm=31%2C29&mn=sn-8xgp1vo-cvnl%2Csn-ab5l6ndy&ms=au%2Crdu&mv=m&mvi=1&pcm2cms=yes&pl=17&initcwndbps=1652500&vprv=1&mime=video%2Fmp4&ns=Adh6ClK7gdvqj1pdZ19HYf8F&cnr=14&ratebypass=yes&dur=869.842&lmt=1618601127502676&mt=1618673077&fvip=1&fexp=24001373%2C24007246&c=WEB&txp=5516222&n=InZogzFPsglCFq&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Ccnr%2Cratebypass%2Cdur%2Clmt&sig=AOq0QJ8wRQIgQ-U-1UDMMp3lR3oHHb7NnOswk1gAefIG1Mmj-Wpp_scCIQChXLWyJbYAf8VPHa6HmIG1IF1ycE7Ov6c9BS3eTjf0UQ%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpcm2cms%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRgIhAJNpFSysXzppqUJfI0j9UPm5rFKvYqqf9SJwGxVQfKtPAiEA1Dg5-ruJJK0GpF-ETxWZrA_tDylql9OC95Y2GI1mnTU%3D'
[download] Destination: This Unstoppable Robot Could Save Your Life-qevIIQHrJZg.mp4
[download]   0.2% of 175.22MiB at  1.96MiB/s ETA 01:29         [download] Got server HTTP error: Downloaded 374907 bytes, expected 183726924 bytes. Retrying (attempt 1 of 10)...

ERROR: Did not get any data blocks
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\__init__.py", line 475, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\__init__.py", line 465, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 2060, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 799, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 806, in wrapper
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 838, in __extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 872, in process_ie_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 1683, in process_video_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 1967, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 1906, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\downloader\common.py", line 366, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\downloader\http.py", line 352, in real_download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\downloader\http.py", line 322, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\downloader\common.py", line 165, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 628, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp_5l7wxqx\build\youtube_dl\YoutubeDL.py", line 590, in trouble

Description
I tried multiple variations replacing  the '/' and '?' characters
>youtube-dl.exe "https:%2F%2Fwww.youtube.com%2Fwatch%3Fv%3DqevIIQHrJZg"
[generic] https://www.youtube: Requesting header
WARNING: Could not send HEAD request to https:%2F%2Fwww.youtube.com%2Fwatch%3Fv%3DqevIIQHrJZg: <urlopen error no host given>
[generic] https://www.youtube: Downloading webpage
ERROR: Unable to download webpage: <urlopen error no host given> (caused by URLError('no host given',))

also
>youtube-dl.exe "https://www.youtube.com/watch%3Fv=qevIIQHrJZg"
[generic] watch?v=qevIIQHrJZg: Requesting header
WARNING: Could not send HEAD request to https://www.youtube.com/watch%3Fv=qevIIQHrJZg: HTTP Error 404: Not Found
[generic] watch?v=qevIIQHrJZg: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

The link is active
https://www.youtube.com/watch?v=qevIIQHrJZg

Server side Youtube issue for format 22. Select different format.


Latest version 4.41.5 freezes on compile

Bug report

What is the current behavior?
Just updated to the latest webpack, 4.41.5 and on compile my Typescript app now freezes. On displaying progress I see that it outputs this:
70% finish module graph FlagDependencyExportsPlugin
And gets stuck at this point. No errors. Any ideas why? How can I output errors?
If the current behavior is a bug, please provide the steps to reproduce.
Run a typescript compile task using a config file:
npx webpack --config webpack.config.js --entry ./source/ts/Main.ts --display-error-details --progress

What is the expected behavior?
I expect it should compile without issues as per previous release, 4.41.4.
Other relevant information:
webpack version:  4.41.5
Node.js version: v6.11.2
Operating System: MacOS High Sierra 10.13.6

Please create reproducible test repo

Same problem, freezes on 70% finish module graph FlagDependencyExportsPlugin - 100cpu.
Switched back,  4.41.4 work well.

Please create reproducible test repo


tour: "underlying array" instead of "slice"?

Context: https://tour.golang.org/moretypes/10
"The default is zero for the low bound and the length of the slice for the high bound."
Should not this be
"The default is zero for the low bound and the length of the underlying array for the high bound."

No, slices can have a length that is smaller than the slice's capacity.  The next slide has some useful context:
https://go.dev/tour/moretypes/11
See also the slice forming examples in:
https://go.dev/tour/moretypes/7

Ah, I see the confusion now.
Duplicate of golang/tour#182


Register tmpfiles.d config on RHEL7

RHEL7 uses systemd-tmpfiles to populate /var/run/.  This is necessary
for /var/run/postgresql/ to exist so postgresql startup succeeds.

It may be better for upstream to package the equivalent into the RPM, but as they don't presently, this is needed.

Thx for the fix
this looks indeed as a better solution


Reload web pages when Atom reloads

Reload web pages when Atom reloads and put in a config setting.

+1, and it would be nice to integrate with one of the session management packages.


Course Schedule

!! Preliminary !!
Morning

Open Science overview (30 - 45 min) (must)
Toolkit basics - xradar/command line tools/xarray, LROSE (GUI, CLI), BALTRAD (1 to 1.5 hours)

Research (30 minutes)

xradar
Py-ART/wradlib
LROSE
Visualization (tools, colormaps, …)

Operations (30 minutes)

Pyrad
LROSE
BALTRAD

Sample workflows

Xradar → Py-ART + wradlib
GPM-API + wradlib
LROSE wind tool?
BALTRAD GUI

What aspects of the radar processing chain where each package is useful; make a flowchart of data flow?
See this for example: https://github.com/openradar/asean2024-pyrad-course/blob/main/presentations/pdf/01_open_meteorological_radar_software_constellation.pdf
Afternoon

Some set of curated datasets
GPM - ground based radar comparison
NEXRAD - potential for multidoppler
xarray/datatree/zarr + dask (cloud/arco format) Colombian Sigmet radar data - ARCO!
OPERA data? Focused on Italy?

Preliminary Time Schedule (tbd)
09:00 - 09:15 Welcome and getting started
09:15 - 09:30 Community weather radar software and Open Science
09:30 - 10:00 xradar Introduction
10:00 - 10:30 Coffee break
10:30 - 11:00 Py-ART/wradlib
11:00 - 11:30 LROSE
12:00 - 12:30 BALTRAD (?, maybe move to first slot in afternoon)
12:30 - 13:30 Lunch break
13:30 - 14:00 Pyrad/BALTRAD/LROSE General Operational Aspects (?)
14:00 - 15:30 Groupwork/Workflows
15:30 - 16:00 Coffee break
16:00 - 18:30 Groupwork/Workflows
18:30 - 19:00 Open slot, discussion, evaluation

@openradar/erad2024-team
I am looking at the Preliminary time schedule. With @wolfidan we were planning a general 30 min (could be reduced to 20 min) introduction (in the morning or early afternoon) and a group work focused on QPE.  Could we set the 20 min slot right after the Py-ART/wradlib intro? We do not need a specific slot in the afternoon then.

Resolved by #83


Outdated compatibility in setting description

Describe the bug
Same as tile
To Reproduce
Steps to reproduce the behavior:

Go to 'Unc0ver'
Click on 'Settings'
Click on the info button
Outdated compatibility info

Screenshots

And more
Device (please complete the following information):

iOS Version: iOS 13.4
iOS Device: iPhone8,2
unc0ver Version: 3.6.2+1.gb9c17cb

Place an "x" between the brackets if true:

[x] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

I also made a post about this.
Sent with GitHawk


ipynb files on february updates

Type: Bug
The february versions of VS Code (Version: 1.76.x) don't work with Jupyter ipynb files. I tried both versions and with none of them I was able to edit and save Jupyter files.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.76.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I tried both versions and with none of them I was able to edit and save Jupyter files.

@vladimirrav do you get any notifications or prompts when you try to save files? What happens when you try to edit the files?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

It says "The operation is taking longer than expected" and the file is never saved. Some files I cannot open using the version 1.76.x, so I kept the latest 1.75 version in order to keep working with the files. Sometimes i clicked on save nothing happens and the "not saved mark" is still visible next to the filename.

The unsaved file mark next to the filename:

Clicking to close the tab:

After some minutes after clicking save in an unsaved file and trying to close the VS Code window it shows this message:

These files have SQL code and markdown;
After reopening the file it is still unsaved:

Sometimes I get this editor error when opening the file:

Error message that says the file couldn't be saved:

Loading a file throws error:

There is this message of detecting a kernel, maybe it is a clue:

When I downgrade VS Code to version 1.75.1 it works fine and none of those errors happen.

System information the tests were performed
Version: 1.76.2 (system setup)
Commit: ee2b180d582a7f601fa6ecfdad8d9fd269ab1884
Date: 2023-03-14T17:55:54.936Z
Electron: 19.1.11
Chromium: 102.0.5005.196
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045
Sandboxed: No

@bpasero do you know how this happens and what we can do to troubleshoot the long saving?

@rebornix we block the shutdown / close of a window if we see that there are working copies that are pending to be saved. In other words, if a save operation has not finished, we refuse to just quit, because there is a potential for data loss.
Here I think it would be helpful to get the logs from a code --verbose run isolated to this issue, ideally:

run code --verbose
open a Jupyter notebook
make a change
save it (this should not complete given the issue here)
send us the logs

In the logs I unfortunately see no evidence of an attempted save happening. The notebook is resolved as indicated by this message:
[stored file working copy] doCreateModel() file:///c%3A/Users/User/Documents/SQL/DATAIND-1265.ipynb

And I see a bunch of changes to the notebook, but never a save.
There is a backup being scheduled for the notebook because it turned dirty based on the changes. I do not see that operation succeed though which would explain the modal dialog you see that backups takes longer.
Again, the backup method also relies on the snapshot method implemented by notebooks, so something must be severely broken there that it never returns.

I captured the screen when the errores happen
Trying to open a file that was closed:
Log: VS Code - Open file.txt
Screen record: https://youtu.be/ZNT8zXw68Gw
Result: Keeps loading and never opens
Edit and save:
Log: VS Code - Edit and save file.txt
Screen record: https://youtu.be/cdvgCCewcj8
Result: multiple errores
Notice: For the file to be open in the editor of the VS Code 1.76 I had to open it in the 1.75 version, then install the newer version for it to be in the editor.

There is a spinning "Detecting Kernels" message in the notebook, could that have an impact?

I have no idea, it only happens in the 1.76.x versions, maybe it crashes and cannot find a kernel to render the file.

We have seen multiple reports on file not able to save and sometimes NPE, so it's likely the same issue that we fail to resolve notebook text model properly. Need some efforts to dig into this.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Bug in Biometry for IOS causing GetBiometryKind detection to always return None

There's a bug in DW.Biometry.iOS.pas which means GetBiometryKind will return None for phones with TouchID.
Because the TLAContext is being created within this function, the required .canEvaluatePolicy() is never called on it so it always return the default value of None regardless of the hardware available.
Wrapping the check in a .canEvaluatePolicy() fixes the call.
Fix:
class function TPlatformBiometric.GetBiometryKind: TBiometryKind;
var
LContext: LAContext;
begin
LContext := TLAContext.Create;
if LContext.canEvaluatePolicy(cBiometryMode, nil) then
begin
case LContext.biometryType of
LABiometryTypeFaceID:
Result := TBiometryKind.Face;
LABiometryTypeTouchID:
Result := TBiometryKind.Touch;
else
Result := TBiometryKind.None;
end;
end
else
Result := TBiometryKind.None;
end;

Fixed


'Github Page' --> 'GitHub repository'

Context (Environment)
Visit https://nyt-library-demo.herokuapp.com/
Expected Behavior

'GitHub' should be spelled with a capital 'H' in the footer and on document pages
'page' should be lowercase in the footer. Or better yet, that should read: "repository"

Actual Behavior

GitHub is spelled 'Github' in the footer, and at the top of the Feb 2020 update.
Github 'Page' is capitalized in the footer. Refers to Page rather than repository.

To Reproduce

View footer on any page of
https://nyt-library-demo.herokuapp.com/
and
View link near the top of
https://nyt-library-demo.herokuapp.com/the-library-team/community-updates/community-update-february-2020

Additional Information

(Might be helpful to have a separate Issue type for typos or content suggestions.)
Possible Solution

Also, remove extra space and comma in "has confused newcomers, ,"
https://nyt-library-demo.herokuapp.com/the-library-team/community-updates/community-update-february-2020

Thanks for opening the issue! Since those pages are archives of emails sent out, it seems best not to edit them after they've been sent (even if that means leaving in some of my typos 😬 ). Regarding the other string changes, I'll make some updates and they should go out next time we deploy the public site.


doc /type/Kernel

That's a dup of https://github.com/perl6/doc/issues/503, but I don't really want to close this because I like smaller tickets. maybe https://github.com/perl6/doc/issues/503 should be split into 4 tickets (one of which is right here).

Initial version pushed with df06cbc0709ec26848321eb . Comments welcome!

Although… I think we'll discuss it all in #503, so we can close this for now.


Automatically link existing POI on an event page upon creation

Motivation

When I post an event at a location already stored in Integreat and use the map function, the location information in the event article does not link directly to the map entry.
Proposed Solution

Upon entering the location data for an event, the location is automatically linked to the already existing POI. The users can click on the link and view the location on the map.
Alternatives

User Story

As a municipality CMS user, I want the location information in the event article to be directly linked to the map entry, when the entry already exists, so that I need not enter any additional text and links in the editor form everytime.
Additional Context

Design Requirements

None

If you want to change how something looks in the app, please open an app issue in https://issues.tuerantuer.org/projects/IGAPP/


Add some performance advice.

About TT munchers, internal rules, and push-down accumulators.

Great changes once again, thanks!


camel-test/camel-testng: Add #exchangeProperty(String) to TestSupport…

… and deprecate methods which redirect to deprecated methods of org.apache.camel.builder.Builder

Thanks for merging. :)


feat(pingback url)

Can we have a function like healthchecks, that allow a ping back url to check is a script run for instance?

what do you mean?
if you wanna let the server actively initiates some event notifications, you can try feed feature.

Don't know if you check, but healthecks.io monitor a "service/site" whatever... that you give an interval and in that interval something should make a call to say the service is up... for instance in a cronjob...

ic, i will check it.


No package id found

Hello, I inegrated the testfairy plugin into my project and set it up. It is all working good, except I keep being spammed in the logs with this error:
No package ID 24 found for ID 0x245a3c5c.
Invalid ID 0x00000001.
Any idea on how I can fix this ?

Hi @bogdanvelescantt ,
Thanks for reaching out. Can you post how you include testfairy_flutter in your pubspec.yaml file?

I added this to the dependencies:
testfairy_flutter: 2.1.0

Thank you. I'll try to reproduce and see what I can do. 👍

Thank you. I'll try to reproduce and see what I can do. 👍

Thanks for taking the time. Let me know if you need more info.

The problem is now solved. Can you please update to the latest version and try again?

Thanks for fixing it, but I now get this error after updating to 2.2.1:
../flutter_sdk/.pub-cache/hosted/pub.dartlang.org/testfairy_flutter-2.2.1/lib/src/network_logging.dart:100:20: Error: The parameter 'f' of the method '_TestFairyHttpClient.authenticate' has type 'Future Function(Uri, String, String?)?', which does not match the corresponding type, 'Future Function(Uri, String, String)?', in the overridden method, 'HttpClient.authenticate'.

'Future' is from 'dart:async'.
'Uri' is from 'dart:core'.
Change to a supertype of 'Future Function(Uri, String, String)?', or, for a covariant parameter, a subtype.
Future f(Uri url, String scheme, String? realm)?) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:1744:12: Context: This is the overridden method ('authenticate').
void set authenticate(Future f(Uri url, String scheme, String realm)?);
^
../flutter_sdk/.pub-cache/hosted/pub.dartlang.org/testfairy_flutter-2.2.1/lib/src/network_logging.dart:106:20: Error: The parameter 'f' of the method '_TestFairyHttpClient.authenticateProxy' has type 'Future Function(String, int, String, String?)?', which does not match the corresponding type, 'Future Function(String, int, String, String)?', in the overridden method, 'HttpClient.authenticateProxy'.
'Future' is from 'dart:async'.
Change to a supertype of 'Future Function(String, int, String, String)?', or, for a covariant parameter, a subtype.
Future f(String host, int port, String scheme, String? realm)?) {
^
org-dartlang-sdk:///third_party/dart/sdk/lib/_http/http.dart:1855:12: Context: This is the overridden method ('authenticateProxy').
void set authenticateProxy(
^

FAILURE: Build failed with an exception.

@bogdanvelescantt That's a new change in Flutter api. It should fix itself after running flutter upgrade and rebuilding the project.


doctrine/dbal 3 support

Would it be possible to support doctrine/dbal 3?
"doctrine/dbal": "^2.5|^3.0"

https://github.com/bosnadev/database/pull/51

@wimski pull request is merged!


Update Ed Res with latest AWIPS Tips entry

Not sure why the very last line looks different...I'm assuming some whitespace changed or something (I didn't actively change it...)?


Supporting multiple hostnames per backend service

I've looked through all the documentation and it appears that the ingres rules only support a single hostname per rule. Is this correct or is there a way to route multiple domains to a single backend service? I tried to fudge this by how I would like it to run, like this
  - host:
      - example.com
      - www.example.com
      - someotherdomain.com
    http:
      paths:
      - backend:
          serviceName: exampleservice
          servicePort: '443'

but as you would expect
spec.rules.host in body must be of type string: "array"
Is there an alternate way to list a number of different hostnames?

You can repeat it multiple times. That should work.
  - host:
      - example.com
    http:
      paths:
      - backend:
          serviceName: exampleservice
          servicePort: '443'
  - host:
      - www.example.com
    http:
      paths:
      - backend:
          serviceName: exampleservice
          servicePort: '443'
  - host:
      - someotherdomain.com
    http:
      paths:
      - backend:
          serviceName: exampleservice
          servicePort: '443'

Thanks for your fast response. That is ultimately how I am implementing it, however I also enabled the health check on the backends, which means the configuration it generates is probing the backend for each hostname because it generates a separate hostname for each. It is probably overkill and just relying on a health / readiness check on the kubernetes layer might be better.

You can achieve this with Voyager. Set a backend name and then add frontend rules that matches the extra domains. That will result in something similar.

@tamalsaha thank you for your great help 👍 - that suggestion worked (I didn't know you could set a specified backend name and add additional rules like that). :-)

:) Can you suggest how do we document it? This is pretty advanced feature and usually used by folks who have prior experience with HAproxy.

Actually documenting this in the form of an example would be very helpful. What I did based on your suggestion was define an explicit backend name and manually crafted the acl and use_backend for the additional hostnames. What I was looking for in the documentation that I couldn't find was essentially how to do this. You are right, this is my prior haproxy experience kicking in here.
I'd suggest defining a primary hostname as per all the other docs, but for additional hostnames that you may want

Add the extra hostnames on the tls section to enable the ssl binding for the certificate
Define an explicit backend name
Define frontend rules for the ports that it would run on

I imagine it might make sense to reverse the ordering of this to list the backends first, but this is how I
had it, which worked. Note my backend is using SSL on the webservers also, which requires an annotation on the service to work, so maybe document it with port 80 on that bit.
  frontendRules:
    - port: 80
      rules:
      - acl acl_additional.example.com  hdr(host) -i additional.example.com
      - use_backend examplebackend if acl_additional.example.com
    - port: 443
      rules:
      - acl acl_additional.example.com  hdr(host) -i additional.example.com
      - use_backend examplebackend if acl_additional.example.com

  rules:
  - host: example.com
    http:
      paths:
      - backend:
          name: examplebackend
          serviceName: exampleservice
          servicePort: '443'

One bit that is missing is there appears to be a http -> https redirect rule that is added by the default configuration, which I haven't done here, and probably should add. I think if this isn't a common use case, then at least documenting this is good. Otherwise it might be good if host could be an array to generate this config automatically.


Add Pabitra LENKA to the Contributors list

I added my ID and name. Please add me to the list. Thank you.

Congratulations ... Enjoy GitHub!


dnload_raw operation requires the ability to sudo -u hdfs

The eggo cli tool to download a dataset on a CDH cluster uses Hadoop Streaming basically as a job scheduler to dnload files into HDFS.
The CLI here:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/cli/datasets.py#L33
And the code here:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/operations.py#L38-L84
The mapper script is here:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/resources/download_mapper.py
The user creates a tmp HDFS directory to receive the data:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/operations.py#L40
The MR job then runs and downloads the data using curl into that directory, but runs as user "yarn".
After the dataset is dnloaded, we create the final output directory:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/operations.py#L78
And ideally we'd just move all the data there.  However, all the data is owned by user "yarn", so it causes lots of permissions problems downstream.  Instead, we chown all the data here:
https://github.com/bigdatagenomics/eggo/blob/c0e980f6581e85d4687de625af2957906d446c22/eggo/operations.py#L79-L81
(note: this chowns is to user ec2-user, but easy to change to whatever the current user is)
which requires the sudo capability.
Any way around this?  cc @tomwhite

There are a couple of options:

Set dfs.permissions.enabled to false, so that permission checking is disabled.
Enable the LinuxContainerExecutor so that containers run as the user that submitted the job.

The second is preferable from a security point of view. See the following for more info:

https://hadoop.apache.org/docs/current/hadoop-yarn/hadoop-yarn-site/SecureContainer.html
http://www.cloudera.com/content/www/en-us/documentation/archive/cdh/4-x/4-3-0/CDH4-Security-Guide/cdh4sg_topic_18_3.html

So anyone that wants to use the dnload tool on their cluster has to set up the LinuxContainerExecutor?  Do you think this is an overly-restrictive environment?  Should I just not be using MapReduce to dnload the datasets?


Terminus Update to version 1.0.157

Terminus Update to version 1.0.157.  Terminus has been significantly updated and the new program is now called tabby.

91a573bfcf2a9d2aba9d5572294bdc1bbe2c8ec5


Skip TX Validation during Composition with ?validate=false

See also https://github.com/CounterpartyXCP/counterparty-core/issues/2649

done here https://github.com/CounterpartyXCP/counterparty-core/pull/2650


Ecto.Query.API.type/2 translate integer as bigint in PostgreSQL

Environment

Elixir version (elixir -v): 1.6.4
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): PostgreSQL 10.3
Ecto version (mix deps): 2.2.9
Database adapter and version (mix deps): 0.13.5
Operating system: Ubuntu

Current behavior
I have a table with integer[] type in PostgreSQL. In migration I define the column as add :some_field, {:array, :integer}. When building a query I use type(^value, {:array, :integer}) and the generated SQL fragment is ?::bigint[]. When combine with array operators and functions, PostgreSQL raises error about type mismatch.
The reason is that Ecto treat integer as bigint in database level here. There's a related issue #2343 .
I know that the column type in migration and field type in schema are 2 things. But type/2 has to concern data type transformation in both application level and database level. So it's not wise to force type/2 to treat integer as bigint. It's very confusing, and people can't specify integer type in PostgreSQL.
Expected behavior
I suggest 2 solutions.
Solution 1: let type/2 to support all PostgreSQL integer types (smallint, integer and bigint).
Solution 2: let type/2 don't concern types in database level.
Personally I prefer solution 2. It's more clear to separate types in application level (schema and type) and database level (migration). If the data is the right type in application level, it works fine in most case. Users can also explicitly write ::type with fragment if necessary.

type/2 may perform casting on interpolated inputs and therefore it must
concern itself with Ecto types. The solution here is indeed to use a
fragment.
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Director of R&D

Yes type/2 should perform casting on interpolated inputs. But the issue is that it also generate ::bigint[] in SQL (when we use {:array, :integer}). I don't think it's an expected behaviour because of 2 reasons.

Ecto type :integer translates to PostgreSQL type bigint is wrong and confusing. Especially in the migration :integer means PostgreSQL type integer.
User can not specify PostgreSQL integer with type/2, unless he manually use ::. It makes type/2 less useful.

Given my use case as an example. I have a samples table which has a filter_ids column, the type is integer[]. In the query I want to use array operator @>, so I write code like this:
# Explicit specify type
fragment("? @> ?", sp.filter_ids, type(val, {:array, :integer}))

# Use the same type as filter_ids
fragment("? @> ?", sp.filter_ids, type(val, sp.filter_ids))

In both form, the generated SQL is like s0.filter_ids @> $1::bigint[]. Because PostgreSQL is explicit about types when calling operators and functions, the SQL will raise error. The solution is either changing column type to bigint[] or using fragment("? @> ?::integer[]").
Actually I think type/2 can't guess the database level type correct. For example I can have {:array, :map} in schema and the database level type is jsonb. So my opinion is, if it can't guess right, just don't guess.


make created folder name more readable from "yo liferay-theme:layout"

Feature enhancement:
Currently 'yo liferay-theme:layout" generates a folder with -layouttpl. To me it looked like a typo and if the tpl extension is still something needed in folder name then I was expecting something like -layout-tpl.

I don't know much about DXP but I think it might depend on the naming pattern based on what I see in git grep layouttpl (here is a probably equivalent GitHub search).
Some sample hits:
https://github.com/liferay/liferay-portal/blob/09230c4f8d5d08045c305c359ffb359d509320d0/portal-kernel/src/com/liferay/portal/kernel/model/Plugin.java#L32
https://github.com/liferay/liferay-portal/blob/ff9beb2b4c2c62ff14faf39f77d6039ebe292f8a/modules/apps/marketplace/marketplace-service/src/main/java/com/liferay/marketplace/util/ContextUtil.java#L61
https://github.com/liferay/liferay-portal/blob/65059440dfaf2b8b365a20f99e83e4cdb15478aa/modules/sdk/gradle-plugins/src/main/java/com/liferay/gradle/plugins/tasks/DirectDeployTask.java#L93
https://github.com/liferay/liferay-portal/blob/65059440dfaf2b8b365a20f99e83e4cdb15478aa/tools/servers/tcat/LiferayDeployerFactory.groovy#L52
https://github.com/liferay/liferay-portal/blob/ff9beb2b4c2c62ff14faf39f77d6039ebe292f8a/portal-impl/src/com/liferay/portal/tools/PluginsSummaryBuilder.java#L501
https://github.com/liferay/liferay-portal/blob/ff9beb2b4c2c62ff14faf39f77d6039ebe292f8a/portal-impl/src/com/liferay/portal/tools/PluginsGitSvnSyncer.java#L317
At least some of those make it look like changing the folder name would be a breaking change. I am going to close this in the spirit of spring-cleaning, and if my analysis turns out to be mistaken we can re-open it (FWIW, I agree that the name isn't super readable).


Which Repo is Active? (This or AngularJS fork)

Hey there, I'm interested in helping out. I was just curious which repo is under active development?

I moved the contents of that fork here.

Excellent, so the angular-port branch on this Repo is the place to go? K, got it.


Example of how to get and refresh tokens from outside SDK

Hi,
We are providing ML webservices which are key-based authentication to customers who donot necessarily use Azure ML services or Azure. Now they want to have token based authentication.
In the document it says that

If you use a service principal to get the token, and want it to have the minimum required access to retrieve a token, assign it to the reader role for the workspace.

You'll need to request a new token after the token's refresh_by time. If you need to refresh tokens outside of the Python SDK, one option is to use the REST API with service-principal authentication to periodically make the service.get_token() call, as discussed previously.
We strongly recommend that you create your Azure Machine Learning workspace in the same region as your Azure Kubernetes Service cluster.
To authenticate with a token, the web service will make a call to the region in which your Azure Machine Learning workspace is created. If your workspace region is unavailable, you won't be able to fetch a token for your web service, even if your cluster is in a different region from your workspace. The result is that Azure AD Authentication is unavailable until your workspace region is available again.
Also, the greater the distance between your cluster's region and your workspace region, the longer it will take to fetch a token.

Our workspace and AKS clusters are in the same region.
Could you provide examples of how to use this service from external users perspective? And also as a service provider what we need to do ?
Thanks.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7bb9b6a1-2c4a-651e-cab3-56f82f8fc97b
Version Independent ID: ffbf9800-c465-90c9-e685-6c871ac3e9b1
Content: Configure authentication for models deployed as web services - Azure Machine Learning
Content Source: articles/machine-learning/how-to-authenticate-web-service.md
Service: machine-learning
Sub-service: core
GitHub Login: @cjgronlund
Microsoft Alias: cgronlun

@chengyu-liu-cs With respect to using the webservice endpoints deployed from Azure ML you can use auth or key based authentications. Some examples on using these endpoints are listed in our documentation page to consume web services. From the end

@RohitMungi-MSFT
Thanks. I was asking for example of refreshing tokens from outside python sdk, particularly for this quote:

You'll need to request a new token after the token's refresh_by time. If you need to refresh tokens outside of the Python SDK, one option is to use the REST API with service-principal authentication to periodically make the service.get_token() call, as discussed previously.

@chengyu-liu-cs Thanks for the clarification. The REST API examples which are in preview are available here to get the token which can help list the workspaces, runs, models and other artifacts. Some samples with ADAL library are also available here.
With respect to AKS webservice authentication with tokens the current way is to use the SDK or the az ml service get-access-token option.
@cjgronlund The note in this section of the document needs to be reviewed for options to get the token with REST API.

@RohitMungi-MSFT So the only options are to use the SDK or the az ml service get-access-token option. In the case we choose az ml service get-access-token, what we need to do as a webservice provider and how our customer gets authenticated to fetch tokens ?  In the command, there is not any parameter to provide secrete/username/password assuming our customer doesn't have Azure subscription and account? I guess from service provider side we need to create a service principal. If yes, can you provide details here, e.g., for what we need to create the service principal?

@chengyu-liu-cs You can use the Azure CLI document which details how to create and assign the required roles to the service principal.

@chengyu-liu-cs We hope the above steps to use service principal helped to proceed further, We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply.

@RohitMungi-MSFT
I was occupied by other tasks and was not able to test the solution yet until now.
When a SP is created, APP_ID should be specified. What should APP_ID be ? Should it be the AKS cluster where AKSwebservice deployed or something else ? I think it should not be subscription, nor resource group.

@chengyu-liu-cs The app_id is the output of the SP create command. This is documented in the link mentioned above.
The appId and tenant keys appear in the output of az ad sp create-for-rbac and are used in service principal authentication. Record their values, but they can be retrieved at any point with az ad sp list.

@RohitMungi-MSFT
I just did the experiments/tests. Here is summary. Please correct me if my understanding is wrong.
I create a SP for the Azure ML workspace by running these commands
az login
az account set -s $SUB_ID
az ad sp create-for-rbac --sdk-auth --name $SP_NAME --scopes $WORKSPACE_RESOURCE
az role assignment create --assignee APP_ID --role Reader
az role assignment delete --assignee APP_ID --role Contributor
this line above does not work somehow. I double checked from portal that this SP has Contributor role instead of Reader role. Error message shown below

(No matched assignments were found to delete)

az ml service get-access-token --name $SERIVCE_name  --workspace-name $WORKSPACE_NAME --resource-group $RESOURCE_GROUP
This line above can fetch tokens, expiration and others.
But this solution has some issues:

To refresh/fetch tokens, a user must have an Azure account to login to run az ml service get-access-token --name $SERIVCE_name  --workspace-name $WORKSPACE_NAME --resource-group $RESOURCE_GROUP as running az login --service-principal -u <app-url> -p <password-or-cert> --tenant <tenant> requires tenant_id. The particular customer is not using Azure but on-premiss infrastructures. (how to solve this)
Somehow SP's Contributor role can not be removed. This is very risky for us as our customer can do many things to the Azure ML workspace with the SP. For example, with the SP, I can run az ml computetarget create amlcompute --name XXXX --workspace-name $WORKSPACE_NAME --resource-group $RESOURCE_GROUP to create a new compute cluster. (So the role must be Reader, how to solve it)
Even if a SP only has a Reader role, a lot of read type commands can be executed for example az ml endpoint realtime list to get all existing endpoints deployed by the workspace. This is another risk. Is there a way to assign a SP to a particular ML AKSservice only?

Is there other ways? This solution is really clumsy and not secure.

@chengyu-liu-cs Due to the current limitation with AKS deployments the process to acquire tokens from the deployed endpoint might be cumbersome and as mentioned it might not be feasible to share the details with your customer to get the tokens from the above methods.
A workaround or a secure way to solve the issue could be to expose an API from your application to share the token with your customer where they can use it against the endpoint instead of following the steps above. Since the SDK supports getting the tokens the exposed application can do this in the backend against your subscription and share the token to the customer whenever they request it. This allows your customer to only access your deployed service instead of running any other unwanted commands and it also obfuscates their exposure to Azure cloud.
With respect to the issue where the contributor role cannot be removed for your SP your AAD settings needs to be reviewed by your AAD admin on why the delete of contributor role is not working for the SP. If the issue persists after they have reviewed the settings we would recommend to raise a support issue from the portal against this resource or SP where our service engineer could advise what is wrong with your subscription.
There are some built-in roles that can be used for AKS that can be assigned but this token needs to be tested after your contributor access issue is resolved against your deployed service.


typo fixed.

readme.md url fixed
Signed-off-by: Kunal Kushwaha<EMAIL_ADDRESS>
What this PR does / why we need it:
fixes the broken link in client-go readme file.
https://github.com/kubevirt/kubevirt/pull/2551#discussion_r311431901
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
https://github.com/kubevirt/kubevirt/pull/2551#discussion_r311431901
Special notes for your reviewer:
Release note:

NONE

/ok-to-test

Travis is green, no need to wait for functests for this doc fix.


create legacy formula for ArangoDB 2.x

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same formula update/change?
[x] Have you built your formula locally prior to submission with brew install <formula> (where <formula> is the name of the formula you're submitting)?
[x] Does your submission pass brew audit --strict --online <formula> (after doing brew install <formula>)?
[x] Does your submission adhere to Versions "Acceptable Formulae" guidelines?

The new ArangoDB 3.x is not compatible (for database and Foxx apps) with the 2.x version.

@BrewTestBot test this please

@ilovezfs can you merge this as well, now that arangodb 3.0 is available?

Apologies, will re-queue the CI build.

@fceller this repository is @DomT4's realm and and his favorite repository, so we'll need to wait on his approval.

and his favorite repository

It's definitely not 🙈.

@DomT4 @ilovezfs sorry, I was not aware of the realm separations :-)

@DomT4 @ilovezfs it is now all green

:+1:
@DomT4 ?

Seems fine. Thanks for working on this @ilovezfs & thank you again for your contributions to Homebrew @fceller. Merged in https://github.com/Homebrew/homebrew-versions/commit/4182f2609c7646c69c53f5cf590b117ceae6d262.


fix(Designer): Additional changes for aliasing with split on

Requirement Checklist

[ ] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes or features)
[ ] Docs have been added or updated (for bug fixes or features)

Type of Change

[x] Bug fix
[ ] Feature
[ ] Other

Current Behavior
Currently the aliasing for spliton outputs is not applied to schemaAlias which causes a missing alias property in dynamic schema calls on PAuto side.
New Behavior
The change introduces a plug-in from manifest service to handle the spliton outputs conversion - the function is optional and falls back to existing logic if not provided.
Impact of Change
Unblocks PAuto, should have no effect on Logic Apps

[ ] This is a breaking change.

Screenshots or Videos (if applicable)

Closing, PA has forked designer


Github contribution not loading in the Fun Facts section

Possible reasons:

[ ] Beta mode of activity overview. It may not allow pubic requests: But it does show some information. It just doesn't load the graph. Maybe the end point is partially open (which is a bit unlikely, pointing towards issue within the existing js code)
[ ] Issue with the existing js code

Resolved in 4327c6de5471fd66fcf3fedf05ec4b3e942cffd6


Also need to add gvm user to sudoers, else ICMPV4 error results

Thank you so much for this script!
The install was successful, and it almost worked for me first time once Postgres had stopped whirring with all the test data, but I found that my scans had a status of Interrupted as soon as they started.  The logs showed error:
failed to open ICMPV4 socket: operation not permitted
When comparing the script with the official Greenbone instructions, I noticed that the instruction to add the gvm group to sudoers was missing:
sudo visudo
...
allow users of the gvm group run openvas
%gvm ALL = NOPASSWD: /usr/local/sbin/openvas
I don't think you'll be able to add this to the script (but would be happy to be proved wrong) - do you think you could add it to the steps in the README?   It's what I had to do to get it to work.
Thanks so much again!

Hello and thank you for your kind words! We are glad the script worked for you. We have taken into consideration your comments and we'll review the requested changes on the main script once we find a suitable time frame. Thank you for contributing!

In the newest commit, we've added a configuration for /etc/sudoers to allow the gvm group users to run OpenVAS: https://github.com/Kastervo/OpenVAS-Installation/commit/a587fdce0e4472a4e2d5562068cac617cdfaa16b


keyring not decrypting keys right-after a soft reload; needed to do a PATCH to the plugin to decrypt keys.

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
Kong Enterprise <IP_ADDRESS>
Current Behavior
From the documentation it seems the key material is persisted following a soft reload BUT not loaded to memory Automatically. I guess the patching of plugin acting as a 'trigger' to load the persisted keys to memory.
Documentation:

Note that encryption keys are held only in memory, and do not persist past a restart of the Kong process (e.g., running kong restart). Because of this limitation, you must export the keyring following its initialization. Otherwise, if all Kong nodes in a cluster restart simultaneously, any sensitive fields written with the keyring become unrecoverable. Key material still persists after a soft reload of Kong (i.e., kong reload).

Expected Behavior
Since it's a soft-reload the keyring material should load to memory automatically and any encrypted key should decrypt automatically.
Steps To Reproduce
1. Generate RSA key pair and enable keyring in kong.conf in cluster mode
2. Add a value to an encrypted = true field of a plugin and save
3. Test the encryption and decryption (using postman or other means); works
4. Do a soft reload /usr/local/bin/kong reload --vv
5. Test again; fails
6. Open the plugin in Point 2, do a patch request (click Submit changes button or disable/enable)
7. Do test again; works.

Anything else?
Following this documentation to setup the keyring (cluster mode)

@pravynandas I wasn't able to produce this, by step 5 Test again; fails do you mean send a GET request to admin API? And what are the symptons of failure do you observe?
Could you also share the kong error log, if it's available.

I have configured a static route with GET method and associated a plugin which includes a custom field marked as "encrypted = true".
Before the kong reload, I was able to save a string to this field and able to verify its encrypted value "$ke$1..." in the database.
Upon a kong reload, the key cannot be decrypted without opening the plugin at least once and hit Submit changes button. Alternatively I was able to make the decryption work by disabling and enabling the plugin.
The only thing I noticed in the error log is that, the stored key is retrieved in its encrypted form ($ke$1) but not its is decrypted value following kong reload.
Hope that helps!?

Hi, Any update on the above issue? Is this considered as product a bug? Thanks,

Thank you for your response. We are in the process of escalating this issue
to the point of contact at Kong.
On Tue, Jun 21, 2022 at 7:14 AM Hans Hübner @.***>
wrote:

Hello @pravynandas https://github.com/pravynandas and @shanexguo
https://github.com/shanexguo,
from what you describe, this looks indeed as if it could be a bug. We
noticed that you are using Kong Enterprise, however, and this is the Kong
open source repository. Have you been in touch with your sales rep or
customer success engineer at Kong about this issue yet? If not, getting in
touch through the Enterprise Edition channel will be the best route to get
this issue looked into.
Kind regards,
Hans
—
Reply to this email directly, view it on GitHub
https://github.com/Kong/kong/issues/8943#issuecomment-1161604579, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7I4NOZKCVWG4RFZKZVVTDVQGP2PANCNFSM5YVPPF3A
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
King regards,
Praveen


Allowing CUDA callback to set the future directly from an OS thread

Setting CUDA future directly from the stream callback

This was takien directly from @sithhell's branch. I assume he's fine with it ;)


Adding stories to existing session

Is there a way to add stories to an estimation session already in progress?

This would be useful when you split a story

Any updates on this? my scrum master will buy me a burrito if the gets this feature


Iron Kingdoms Update

General update, including roll templates for NPC attacks, Virtuoso checkboxs, collapsing weapons, and the derived attributes on every page

Thank you!

Some of the changes you made broke the style on the sheet, if you could fix that and then pull again it would be much appreciated.

Its suppose to be the exact same thing I use on my custom sheet I use in the game I run.  Mable I copied it wrong.  Will resubmit.


service discovery via service ennvars

find services via service environment variables
instead of relying on kube-dns.

closes #96


[Docs] Simplify PR template

PR Details
Simplified the PR template to reduce the amount of nonsense people have to deal with.

Description has been removed, most issues could be explained entirely through PR Details while more complex ones would just split it into paragraphs anyway.
Motivation and Context is fairly obvious from the title or description of the PR.
Added a link to the documentation about PRs, for both newcomers and maintainers to hopefully guide them into titling and labeling PRs appropriately.

Related Issue
None.
Motivation and Context
See what I mean, this is just busywork :)
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have built and run the editor to try this change out.

It looks good.
I was also looking into it. When I do docs related PRs, it could be even simplier but there can't be multiple templates for PRs if I am right, so anything simplified is good 👍

Thanks for checking @Jklawreszuk and @VaclavElias I'll merge this in then :)


Feature Request: Drop cell into another cell

Hello again! At some point I'll be putting folders into my table - any suggestions for dragging a cell onto another cell with existing functionality, or places to start to make this possible in your library would be helpful.
I figure additional delegate methods (or a separate delegate) might look something like:
func canDragHoverOverIndexPath(_ indexPath : IndexPath) {
// return true if folder / hoverable element, false otherwise to reorder.
}
func didDropDragItemIntoIndexPath(_ indexPath: IndexPath, fromSourceIndex sourceIndexPath : IndexPath) {
// delete the source index path and update data.
}

That's a good question. I think you'd have to make some substantial changes to the reordering logic. Currently the code is only looking for a new "destination row" as you drag cells around, but to allow hovering, it would have to also check for a "hover" state and update things accordingly. Probably would need to add something like var hoveredRow: IndexPath? to the ReorderContext to keep track of that.
To detect the hover state, it might be as simple as adding a check to updateDestinationRow() before calculating the proposed destination row – i.e., if the dragged cell is almost centered over one of its adjacent neighbors within some tolerance, set that as the hovered row instead of updating the destination row. You'd also need to add logic to endReorder() to handle the case where one cell is dropped into another (delete the source row from the table, notify the delegate, animate the snapshot view away, etc).
Those delegate methods look like a good place to start. You might also want one that fires when a cell is hovered/un-hovered, so the cell can be visually highlighted in some way.

Thanks! I'll definitely need folders at some point, and I don't see a better option than extending this library so far. My high level understanding is that there is a blank cell below the hovered cell that is switching depending on where the dragged cell is located. I might be wrong on that tho.
If that is the case, I assume that I could call my delegate function for a destination row: canDragHoverOverIndexPath - and if it returns true, I'd just delete the blank cell from the table and store the blank cell's reference in a local class. Then I'd set isHighlighted = true on the destination row. This could be problematic however if deleting the blank cell scrolls the tableView, but I'm getting ahead of myself.
Anyway, thanks again for your thoughts on this. I think they will make more sense once I do a side-by-side comparison of the code with your notes.

You're right, there is a blank spacer cell inserted in the tableview beneath the location you're dragging. I don't think you'd want to remove that spacer when hovering over another row though. That would cause your content to keep shifting up and down as you dragged cells around.
As an example, check out the behavior when reordering apps on the iOS home screen. When hovering one app over another to create a folder, there's still a blank space where the original app used to be.

Good feedback. I'll implement it this way. Thanks!

No problem! Let me know if you're able to get it working.


Instrument with WIPAC Telemetry Prototype

closes https://github.com/WIPACrepo/MQClient/issues/12

Note: Since MessageGeneratorContext is not simply a generator function, we can't rely on WIPAC Telemetry to fully auto-instrument iterations, enters, and exits.

To see these changes in action, go to: https://app.circleci.com/pipelines/github/WIPACrepo/MQClient-RabbitMQ?branch=update-base-for-telemetry

@dsschult Can you take a look at recv()/MessageGeneratorContext's instrumentation? In particular, I want your opinion on whether __next__() should be an event or a span.

Event

pros: simple; is grouped inline with the containing span
cons: only has an initial timestamp; no duration

Span

pros: has an enter and exit timestamp; easily see duration which is nice for detecting bottlenecks
cons: more complex; with more iterations this may get out of hand

Instrumenting send() and recv_one() are straight forward.

@dsschult Can you take a look at recv()/MessageGeneratorContext's instrumentation? In particular, I want your opinion on whether __next__() should be an event or a span.

Event

pros: simple; is grouped inline with the containing span
cons: only has an initial timestamp; no duration

Span

pros: has an enter and exit timestamp; easily see duration which is nice for detecting bottlenecks
cons: more complex; with more iterations this may get out of hand

Instrumenting send() and recv_one() are straight forward.

Never mind, it needs to be a span in order to make a link to the producer. Also, it's possible that not each iteration's incoming message is from the same producer, so documenting that is extra important.

@dsschult I've finished what I planned to do. A quick note on two things:

I had to add callbacks to the consumer methods. This was the cleanest solution I found that enables linking to the producer via a carrier object.
The span for MessageGeneratorContext.__next__() documents the raised exception: StopIteration. It's not a "real" exception, but it's recorded the same. I'm not sure if we want to treat this differently.


Request to 'http://localhost:1234/api/messages' failed: [500] Internal Server Error

Request Internal Server Error

You  need to update your botbuilder to new version.

@mohammadamer, Please ensure your using BotBuilder SDK version 3.9+. Apologies for the trouble.


Paula, Tom and Chris didn't start in October

As seen in the data folder structure, they didn't start work on October:

Logs:
AAChris.1.0-Logs.zip
AATom.1.0-Tom.zip
AAPaula.1.0-Logs.zip

Paula depends on Chris which depends on Olivia. Tom also depends on Olivia. I need to have the logs of Olivia to know what happened.

Olivia was working well, as well as Bruce. Candles where up to date on all periods, showing on the charts. The logs are no longer available.


Feature/rjm task exclusion

Hi Luke,
Please enjoy this PR.
Thanks,
Robin

Current coverage is 65.39% (diff: 49.01%)

No coverage report found for master at 0956046.

Powered by Codecov. Last update 0956046...595e9f4


Readme: Update the code example for client.send_audio

I tried the code as is, and I got 400 error:
Client error: 400 b'{"type":400,"title":"Bad Request","invalid_parameters":[{"reason":"Invalid type parameter provided","name":"stream_url"}]}'

According to the docs, stream_url accepts an array of strings.
Changing the stream_url to a list works.

Pull Request Test Coverage Report for Build 275

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 98.328%

Files with Coverage Reduction
New Missed Lines
%

nexmo/init.py
4
99.0%

Totals

Change from base Build 273:
-0.7%

Covered Lines:
294

Relevant Lines:
299

💛  - Coveralls

Pull Request Test Coverage Report for Build 275

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 98.328%

Files with Coverage Reduction
New Missed Lines
%

nexmo/init.py
4
99.0%

Totals

Change from base Build 273:
-0.7%

Covered Lines:
294

Relevant Lines:
299

💛  - Coveralls

Pull Request Test Coverage Report for Build 275

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 98.328%

Files with Coverage Reduction
New Missed Lines
%

nexmo/init.py
4
99.0%

Totals

Change from base Build 273:
-0.7%

Covered Lines:
294

Relevant Lines:
299

💛  - Coveralls

Pull Request Test Coverage Report for Build 275

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.7%) to 98.328%

Files with Coverage Reduction
New Missed Lines
%

nexmo/init.py
4
99.0%

Totals

Change from base Build 273:
-0.7%

Covered Lines:
294

Relevant Lines:
299

💛  - Coveralls

I don't think my change caused the coverage to drop, only readme.md was modified.

Thanks for catching this @Mariatta


[backport] Fix decorator usage in tests

Backport of #2820

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 8622385ee9125890714c8f59885708c8c61bc02d, target branch v7) succeeded!

LGTM


Reinforcement Learning - Where Art Thou?

Wonderful book; learning a ton! Question: In first chapter, you explain the three types of learning (supervised, unsupervised and reinforcement). Usually the third is not listed. So, I search your text for other material on RL but found none. Future chapter in next edition? Future book? In your other resources, are there links about RL with a scikit-learn style? Love Karpathy's blog on "Pong from Pixels".

Thanks for the nice words, Richard, I really appreciate it, and I am happy to hear that you like it overall -- despite the "missing" reinforcement learning chapter(s) :).

Usually the third is not listed. So, I search your text for other material on RL but found none. Future chapter in next edition? Future book?

Yeah, I had a hard time to meet the max. page limits by the publisher, and I thought that a reinforcement learning chapter was a bit out of scope. Sure, an intro chapter would be a great idea for a second edition (alternatively, I was thinking on separate chapters on recurrent neural nets and ConvNets). To be honest though, I'd say that Reinforcement Learning would really be a better off as a separate, future book :). I thought it would be a good idea mentioning it in the intro though for the sake of completeness (i.e., letting readers know that it exists :P).
PS: I just wanted to mention Andrej Karpathy's blog, but I see that you already stumbled upon it. Coincidentally, there's an online course on reinforcement learning that just started 2 days ago, maybe that's a useful resource: https://onlinecourses.nptel.ac.in/noc16_cs09/preview
(It's based on R. S. Sutton and A. G. Barto. Reinforcement Learning - An Introduction. MIT Press. 1998.)
Cheers,
Sebastian


Optuna Rock

Description
Provides an optuna rock.

This PR was used as base for ROCK development. The work was completed in this PR https://github.com/canonical/katib-rocks/pull/20
Closing.


[Fix #3120] Fix #new visibility modifier scopes

Blocks passed to {Class,Module,Struct}.new create a new scope, in a
similar way to Klass.class_eval. This was a regression introduced by my
PR #3060 where I forgot about these method calls.
Before submitting a PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it)
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

👍


Error installing in Foxy

My colcon build exits on errors:
`Starting >>> bluespace_ai_xsens_mti_driver
--- stderr: bluespace_ai_xsens_mti_driver
/usr/bin/ld: cannot find -lxscontroller
/usr/bin/ld: cannot find -lxscommon
/usr/bin/ld: cannot find -lxstypes
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/xsens_mti_node.dir/build.make:201: xsens_mti_node] Error 1
make[1]: *** [CMakeFiles/Makefile2:78: CMakeFiles/xsens_mti_node.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
Failed   <<< bluespace_ai_xsens_mti_driver [1.33s, exited with code 2]
Summary: 0 packages finished [1.47s]
1 package failed: bluespace_ai_xsens_mti_driver
1 package had stderr output: bluespace_ai_xsens_mti_driver
`
And apparently these libraries don't exist anywhere else on the internet.
Did anybody else face this problem? How do I solve it?

You are likely missing the "build xspublic" step mentioned in the readme (
https://github.com/bluespace-ai/bluespace_ai_xsens_ros_mti_driver) that
needs to happen before colcon build.
On Sat, Oct 8, 2022, 1:57 PM AndroAdi @.***> wrote:

My colcon build exits on errors:
`Starting >>> bluespace_ai_xsens_mti_driver
--- stderr: bluespace_ai_xsens_mti_driver
/usr/bin/ld: cannot find -lxscontroller
/usr/bin/ld: cannot find -lxscommon
/usr/bin/ld: cannot find -lxstypes
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/xsens_mti_node.dir/build.make:201:
xsens_mti_node] Error 1
make[1]: *** [CMakeFiles/Makefile2:78: CMakeFiles/xsens_mti_node.dir/all]
Error 2
make: *** [Makefile:141: all] Error 2
Failed <<< bluespace_ai_xsens_mti_driver [1.33s, exited with code 2]
Summary: 0 packages finished [1.47s]
1 package failed: bluespace_ai_xsens_mti_driver
1 package had stderr output: bluespace_ai_xsens_mti_driver
`
And apparently these libraries don't exist anywhere else on the internet.
Did anybody else face this problem? How do I solve it?
—
Reply to this email directly, view it on GitHub
https://github.com/bluespace-ai/bluespace_ai_xsens_ros_mti_driver/issues/15,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADJX55D2OHSMP2XXGWIVNLWCEV6JANCNFSM6AAAAAARAFDJPE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Imported Schema Definition dropped from gen/srv/srv/csn.json

Hello,
I have two imported schemas (provider) in my CDS (wrapper) project. Both the imported schemas are different services exposed by other CDS projects. The wrapper schema has its own layer of database and filtering/massaging of underlying services. So essentially the wrapper CDS project exposes its own services that builds on top of the underlying services. In addition to that, partial entities from the underlying schema is also exposed with filtering.
the underlying project exposes a full service. the wrapper application should only use "a view" of the underlying service. In future I have plans to build other projects that utilize other aspects of the underlying service.
When testing locally all the 3 perform correctly . the wrapper CDS provides results from its own implementation and also exposes the underlying services (deployed on BTP and accessed via destination service). The problem arises when wrapper is deployed in BTP. The csn.json in gen/srv/srv does not contain the full definitions . the csn.json has only 2 definitions (1. wrapper 2. one of the underlying service). The second underlying service is not present in the csn.json of gen/srv/srv.
Just to note, all 3 definitions (wrapper and 2 underlying) are present correctly in db/csn.json. Just to tryout, I rebuild the entire thing without the CDS-DBM / CDS-PG. This csn.json in srv has the correct definitions.
Hence I am assuming this is coming from cds-dbm.
Please help
Thank you

I did attempt to debug and find the root cause. Unfortunately, I cannot provide the screenshots and the content in this public forum but if you would like to see it I can join a private session or connect with you directly to explain the services or share the content. The error is observed in cds build that internally is relying on cds-dbm. Since this is working locally.
I appreciate the effort you are putting into this. If you can guide me towards how to debug the build itself or the design I can attempt to troubleshoot. I am a new comer to nodejs world. I come from ABAP world with decent java understanding. But I feel comfortable playing around the nodejs debugging with guidance.


feat: add option to set headers

Thank you for the PR!
Mind updating the README.md file with the new option and bumping the minor version?


How to add  Support for XinFin Netowrk

feature
Hi Everyone
i want to to know what needs to be done to support XinFin network on graph-node.
https://www.xinfin.org/

The biggest question here is: Is XinFin network EVM compatible?
If yes, it's pretty straight-forward: Are there any JSON-RPC endpoints? You can run a local-graph node and pointing to that endpoint and try it out.
If no: This is way more complicated.

Thanks for reply
I can see xinfin does have JSON-RPC endpoints
https://apidocs.xinfin.network/
https://apidocs.xinfin.network/docs/#xinfin-apis
Can you suggest what and where changes needs to be made.

If you are running a local graph-node you just need to set the rpc URL e.g. on this line in the docker-compose:
https://github.com/graphprotocol/graph-node/blob/bcb5b44a099095ceb0917bbc4fd97641084eaba2/docker/docker-compose.yml#L20
If this chain is EVM compatible this shouldn't require any changes in graph-node, so closing this issue. Thanks!


[Bug] [Worker] Match wrong pid on linux server

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
If you deploy DS on linux server, cancel process instance, then it's possible to get error log:
[ERROR] 2023-07-27 23:14:54.887 +0800 org.apache.dolphinscheduler.server.worker.processor.WorkerTaskKillProcessor:[196] - kill task error
org.apache.dolphinscheduler.common.shell.AbstractShell$ExitCodeException: kill: (26483): Operation not permitted
kill: (558): No such process
kill: (1493): No such process

What you expected to happen
No error log.
How to reproduce
Ditto.
Anything else
That is because in ProcessUtils.getPidsStr, only match pids on macos and windows.
        if (SystemUtils.IS_OS_MAC) {
            String pids = OSUtils.exeCmd(String.format("%s -sp %d", TaskConstants.PSTREE, processId));
            if (null != pids) {
                mat = MACPATTERN.matcher(pids);
            }
        } else {
            String pids = OSUtils.exeCmd(String.format("%s -p %d", TaskConstants.PSTREE, processId));
            mat = WINDOWSATTERN.matcher(pids);
        }

If run on linux server, will get pids string like: sudo(6279)---558_1497.sh(6282)---sleep(6354), but if use windows matcher,  it will match "6279 558 1497 6282 6354", expected matching results is "6279 6282 6354".
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I'll fix it


[BUG]: environment-gpu.yml can't create environments with newest conda and pip install doesn't work too!

Installing solaris using conda
conda update conda
conda env create -f environment-gpu.yml

results in
visionosl:solaris$ conda env create -f environment-gpu.yml
Collecting package metadata (repodata.json): done
Solving environment: -
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed                                                                                \

# >>>>>>>>>>>>>>>>>>>>>> ERROR REPORT <<<<<<<<<<<<<<<<<<<<<<

    Traceback (most recent call last):
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/exceptions.py", line 1079, in __call__
        return func(*args, **kwargs)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda_env/cli/main.py", line 80, in do_call
        exit_code = getattr(module, func_name)(args, parser)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda_env/cli/main_create.py", line 111, in execute
        result[installer_type] = installer.install(prefix, pkg_specs, args, env)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda_env/installers/conda.py", line 32, in install
        prune=getattr(args, 'prune', False), update_modifier=UpdateModifier.FREEZE_INSTALLED)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/core/solve.py", line 117, in solve_for_transaction
        should_retry_solve)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/core/solve.py", line 158, in solve_for_diff
        force_remove, should_retry_solve)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/core/solve.py", line 281, in solve_final_state
        ssc = self._run_sat(ssc)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/common/io.py", line 88, in decorated
        return f(*args, **kwds)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/core/solve.py", line 808, in _run_sat
        should_retry_solve=ssc.should_retry_solve
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/common/io.py", line 88, in decorated
        return f(*args, **kwds)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/resolve.py", line 1318, in solve
        self.find_conflicts(specs, specs_to_add, history_specs)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/resolve.py", line 347, in find_conflicts
        bad_deps = self.build_conflict_map(specs, specs_to_add, history_specs)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/resolve.py", line 507, in build_conflict_map
        root, search_node, dep_graph, num_occurances)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/resolve.py", line 369, in breadth_first_search_for_dep_graph
        last_spec = MatchSpec.union((path[-1], target_paths[-1][-1]))[0]
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/models/match_spec.py", line 481, in union
        return cls.merge(match_specs, union=True)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/models/match_spec.py", line 475, in merge
        reduce(lambda x, y: x._merge(y, union), group) if len(group) > 1 else group[0]
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/models/match_spec.py", line 475, in <lambda>
        reduce(lambda x, y: x._merge(y, union), group) if len(group) > 1 else group[0]
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/models/match_spec.py", line 502, in _merge
        final = this_component.union(that_component)
      File "/home/visionosl/anaconda3/lib/python3.7/site-packages/conda/models/match_spec.py", line 764, in union
        return '|'.join(options)
    TypeError: sequence item 0: expected str instance, Channel found

`$ /home/visionosl/anaconda3/bin/conda-env create -f environment-gpu.yml`

  environment variables:
                 CIO_TEST=<not set>
  CONDA_AUTO_UPDATE_CONDA=false
                CONDA_EXE=/home/visionosl/anaconda3/bin/conda
         CONDA_PYTHON_EXE=/home/visionosl/anaconda3/bin/python
               CONDA_ROOT=/home/visionosl/anaconda3
              CONDA_SHLVL=0
            DEFAULTS_PATH=/usr/share/gconf/ubuntu.default.path
           MANDATORY_PATH=/usr/share/gconf/ubuntu.mandatory.path
                     PATH=/home/visionosl/anaconda3/bin:/home/visionosl/anaconda3/condabin:/home
                          /visionosl/bin:/home/visionosl/.local/bin:/usr/local/sbin:/usr/local/b
                          in:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
       REQUESTS_CA_BUNDLE=<not set>
            SSL_CERT_FILE=<not set>
            XDG_SEAT_PATH=/org/freedesktop/DisplayManager/Seat0
         XDG_SESSION_PATH=/org/freedesktop/DisplayManager/Session0

     active environment : None
            shell level : 0
       user config file : /home/visionosl/.condarc
 populated config files : /home/visionosl/.condarc
          conda version : 4.8.2
    conda-build version : 3.18.9
         python version : 3.7.4.final.0
       virtual packages : __cuda=10.1
                          __glibc=2.23
       base environment : /home/visionosl/anaconda3  (writable)
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/visionosl/anaconda3/pkgs
                          /home/visionosl/.conda/pkgs
       envs directories : /home/visionosl/anaconda3/envs
                          /home/visionosl/.conda/envs
               platform : linux-64
             user-agent : conda/4.8.2 requests/2.22.0 CPython/3.7.4 Linux/4.15.0-70-generic ubuntu/16.04.6 glibc/2.23
                UID:GID : 1000:1000
             netrc file : None
           offline mode : False

An unexpected error has occurred. Conda has prepared the above report.

Environment Information
Running this on a fresh installation of anaconda
$ conda list anaconda$
# packages in environment at /home/visionosl/anaconda3:
#
# Name                    Version                   Build  Channel
anaconda                  2019.10                  py37_0

$ conda info

     active environment : None
            shell level : 0
       user config file : /home/visionosl/.condarc
 populated config files : /home/visionosl/.condarc
          conda version : 4.8.2
    conda-build version : 3.18.9
         python version : 3.7.4.final.0
       virtual packages : __cuda=10.1
                          __glibc=2.23
       base environment : /home/visionosl/anaconda3  (writable)
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/visionosl/anaconda3/pkgs
                          /home/visionosl/.conda/pkgs
       envs directories : /home/visionosl/anaconda3/envs
                          /home/visionosl/.conda/envs
               platform : linux-64
             user-agent : conda/4.8.2 requests/2.22.0 CPython/3.7.4 Linux/4.15.0-70-generic ubuntu/16.04.6 glibc/2.23
                UID:GID : 1000:1000
             netrc file : None
           offline mode : False

Installing solaris using pip
$ sudo add-apt-repository ppa:ubuntugis/ppa
$ sudo apt-get update
$ sudo apt-get install python-numpy gdal-bin libgdal-dev python3-rtree
$ pip install solaris

results in gdal error
Failed to build gdal
ERROR: torchvision 0.5.0 has requirement torch==1.4.0, but you'll have torch 1.3.1 which is incompatible.
Installing collected packages: pyparsing, numpy, kiwisolver, cycler, matplotlib, shapely, gdal, torch, pillow, torchvision, urllib3, pyproj, decorator, networkx, pyyaml, imageio, scipy, PyWavelets, scikit-image, opencv-python, affine, click-plugins, attrs, munch, cligj, fiona, termcolor, absl-py, markdown, grpcio, werkzeug, tensorboard, gast, mock, tensorflow-estimator, h5py, keras-applications, astor, keras-preprocessing, tensorflow, rtree, snuggs, rasterio, pytz, pandas, geopandas, mercantile, supermercado, rio-cogeo, imgaug, opencv-python-headless, albumentations, solaris
    Running setup.py install for gdal ... error
    ERROR: Command errored out with exit status 1:
     command: /home/visionosl/anaconda3/envs/dl/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-hosc2bjy/gdal/setup.py'"'"'; __file__='"'"'/tmp/pip-install-hosc2bjy/gdal/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-otn602ug/install-record.txt --single-version-externally-managed --compile --install-headers /home/visionosl/anaconda3/envs/dl/include/python3.7m/gdal
         cwd: /tmp/pip-install-hosc2bjy/gdal/
    Complete output (62 lines):
    running install
    running build
    running build_py
    creating build
    creating build/lib.linux-x86_64-3.7
    copying gdal.py -> build/lib.linux-x86_64-3.7
    copying ogr.py -> build/lib.linux-x86_64-3.7
    copying osr.py -> build/lib.linux-x86_64-3.7
    copying gdalconst.py -> build/lib.linux-x86_64-3.7
    copying gdalnumeric.py -> build/lib.linux-x86_64-3.7
    creating build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/gdal.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/gdalconst.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/osr.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/gdalnumeric.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/gdal_array.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/gnm.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/__init__.py -> build/lib.linux-x86_64-3.7/osgeo
    copying osgeo/ogr.py -> build/lib.linux-x86_64-3.7/osgeo
    Fixing build/lib.linux-x86_64-3.7/gdal.py build/lib.linux-x86_64-3.7/ogr.py build/lib.linux-x86_64-3.7/osr.py build/lib.linux-x86_64-3.7/gdalconst.py build/lib.linux-x86_64-3.7/gdalnumeric.py build/lib.linux-x86_64-3.7/osgeo/gdal.py build/lib.linux-x86_64-3.7/osgeo/gdalconst.py build/lib.linux-x86_64-3.7/osgeo/osr.py build/lib.linux-x86_64-3.7/osgeo/gdalnumeric.py build/lib.linux-x86_64-3.7/osgeo/gdal_array.py build/lib.linux-x86_64-3.7/osgeo/gnm.py build/lib.linux-x86_64-3.7/osgeo/__init__.py build/lib.linux-x86_64-3.7/osgeo/ogr.py
    Skipping optional fixer: ws_comma
    Fixing build/lib.linux-x86_64-3.7/gdal.py build/lib.linux-x86_64-3.7/ogr.py build/lib.linux-x86_64-3.7/osr.py build/lib.linux-x86_64-3.7/gdalconst.py build/lib.linux-x86_64-3.7/gdalnumeric.py build/lib.linux-x86_64-3.7/osgeo/gdal.py build/lib.linux-x86_64-3.7/osgeo/gdalconst.py build/lib.linux-x86_64-3.7/osgeo/osr.py build/lib.linux-x86_64-3.7/osgeo/gdalnumeric.py build/lib.linux-x86_64-3.7/osgeo/gdal_array.py build/lib.linux-x86_64-3.7/osgeo/gnm.py build/lib.linux-x86_64-3.7/osgeo/__init__.py build/lib.linux-x86_64-3.7/osgeo/ogr.py
    Skipping optional fixer: ws_comma
    running build_ext
    gcc -pthread -B /home/visionosl/anaconda3/envs/dl/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I../../port -I../../gcore -I../../alg -I../../ogr/ -I../../ogr/ogrsf_frmts -I../../gnm -I../../apps -I/home/visionosl/anaconda3/envs/dl/include/python3.7m -I/home/visionosl/anaconda3/envs/dl/lib/python3.7/site-packages/numpy/core/include -I/usr/include -c gdal_python_cxx11_test.cpp -o gdal_python_cxx11_test.o
    gcc -pthread -B /home/visionosl/anaconda3/envs/dl/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I../../port -I../../gcore -I../../alg -I../../ogr/ -I../../ogr/ogrsf_frmts -I../../gnm -I../../apps -I/home/visionosl/anaconda3/envs/dl/include/python3.7m -I/home/visionosl/anaconda3/envs/dl/lib/python3.7/site-packages/numpy/core/include -I/usr/include -c gdal_python_cxx11_test.cpp -o gdal_python_cxx11_test.o -std=c++11
    building 'osgeo._gdal' extension
    creating build/temp.linux-x86_64-3.7
    creating build/temp.linux-x86_64-3.7/extensions
    gcc -pthread -B /home/visionosl/anaconda3/envs/dl/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -I../../port -I../../gcore -I../../alg -I../../ogr/ -I../../ogr/ogrsf_frmts -I../../gnm -I../../apps -I/home/visionosl/anaconda3/envs/dl/include/python3.7m -I/home/visionosl/anaconda3/envs/dl/lib/python3.7/site-packages/numpy/core/include -I/usr/include -c extensions/gdal_wrap.cpp -o build/temp.linux-x86_64-3.7/extensions/gdal_wrap.o -std=c++11 -I/usr/include/gdal
    cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
    extensions/gdal_wrap.cpp:4075:1: error: ‘VSIDIR’ does not name a type
     VSIDIR* wrapper_VSIOpenDir( const char * utf8_path,
     ^
    extensions/gdal_wrap.cpp:4123:38: error: ‘VSIDIR’ was not declared in this scope
     DirEntry* wrapper_VSIGetNextDirEntry(VSIDIR* dir)
                                          ^
    extensions/gdal_wrap.cpp:4123:46: error: ‘dir’ was not declared in this scope
     DirEntry* wrapper_VSIGetNextDirEntry(VSIDIR* dir)
                                                  ^
    extensions/gdal_wrap.cpp:4124:1: error: expected ‘,’ or ‘;’ before ‘{’ token
     {
     ^
    extensions/gdal_wrap.cpp:3225:7: warning: ‘char* get_aligned_buffer(char*, GDALDataType)’ defined but not used [-Wunused-function]
     char* get_aligned_buffer(char* data, GDALDataType ntype)
           ^
    extensions/gdal_wrap.cpp:3244:13: warning: ‘void update_buffer_size(void*, char*, char*, size_t)’ defined but not used [-Wunused-function]
     static void update_buffer_size(void* obj, char* data, char* data_aligned, size_t buf_size)
                 ^
    extensions/gdal_wrap.cpp:3299:5: warning: ‘int GetUseExceptions()’ defined but not used [-Wunused-function]
     int GetUseExceptions() {
         ^
    extensions/gdal_wrap.cpp:3305:6: warning: ‘void UseExceptions()’ defined but not used [-Wunused-function]
     void UseExceptions() {
          ^
    extensions/gdal_wrap.cpp:3322:6: warning: ‘void DontUseExceptions()’ defined but not used [-Wunused-function]
     void DontUseExceptions() {
          ^
    extensions/gdal_wrap.cpp:3370:13: warning: ‘void ClearErrorState()’ defined but not used [-Wunused-function]
     static void ClearErrorState()
                 ^
    error: command 'gcc' failed with exit status 1
    ----------------------------------------
ERROR: Command errored out with exit status 1: /home/visionosl/anaconda3/envs/dl/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-hosc2bjy/gdal/setup.py'"'"'; __file__='"'"'/tmp/pip-install-hosc2bjy/gdal/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-otn602ug/install-record.txt --single-version-externally-managed --compile --install-headers /home/visionosl/anaconda3/envs/dl/include/python3.7m/gdal Check the logs for full command output.

Also tried installing GDAL before pip install solaris
sudo add-apt-repository ppa:ubuntugis/ppa && sudo apt-get update
sudo apt-get update
sudo apt-get install gdal-bin
sudo apt-get install libgdal-dev
export CPLUS_INCLUDE_PATH=/usr/include/gdal
export C_INCLUDE_PATH=/usr/include/gdal
pip install GDAL

results in same gdal error.

@dudeperf3ct,
It looks like the first issue is a conda error unrelated to solaris (see the related issue opened on conda's GitHub repo here), and the latter two are GDAL install errors which are, unfortunately, very common (to the point of solving them almost being a rite of passage). I'd recommend you try installing a slightly older version of conda and re-trying the conda install. Closing this issue since it's not directly solaris-related.

Hey @nrweir, I tried downgrading conda to version 6 but still gdal error persists. What version of conda is recommended?

If we can't install solaris by conda and pip, is there any other way?

Conda's current release is at 4.8.2 - I'm currently running conda 4.7.12 and haven't had the problems you indicated.
I'm not sure it's accurate to say that solaris can't be installed by conda/pip - I believe you need to resolve your conda/GDAL install issues, which aren't really related to this project, at which point you'll be good to go. I can sympathize with the challenge of getting that working, though. Best of luck!


Date picker in filter not working in Safari Desktop

Describe the bug
Not showing date picker at all
Screenshots
.

With checking MDN of input[type=date], looks like Safari doesn't support showing a native date picker for now.
So, I think this is not an issue for this library.

Besides, if you want to track the issue for Safari, you can check https://bugs.webkit.org/show_bug.cgi?id=119175.


Remove duplicate entry

remove duplicate code db.Sequelize = Sequelize;

It's not duplicated. One is exporting the Sequelize class and the other the instance.

sorry :(