Google/Plus import of contacts fails

There is a "typing error/mismatch" issue with importing contacts from a Google+ user.
https://github.com/miled/wordpress-social-login/blob/063587671500919348df3cac0d6d6b85af0ec339/hybridauth/Hybrid/Providers/Google.php
The database won't accept NULL values for profileURL or photoURL but these are being passed that way in the database UPDATE.
Looks very ugly with all these DB errors on the page for a user to see.

Yes, and while the user gets an activation email and their profile shows up in the WP admin under Users, they only get these errors when trying to log in and the user does not show up in WSL's Users list.

So... is there a fix in sight?
Generally, I'd assume it was core functionality and a pretty basic thing to be have this sort of stuff working properly. What should my expectations be??

I am watching this repository for some time and I doubt there will be any fix.
There is unfortunately never any response by miled and there are 12 not integrated pull requests.

Hi,
the plugin author seems to be busy in recent months. I am using the import contact feature and got similar error messages in the server log. I have made small changes to the data type definition for the import to work. You may have already done this as a work around. here is the link to my solution just in case someone is reading this thread looking for help. Import User Contacts with WordPress Social Login.
The article is written for someone who is not a developer and who knows little about this plugin. apologies if it is boring to you and states the obvious.
Some Facebook contacts may not allow export of their email address. As a result the email addresses are missing from the returned data. Google+ contacts may actually be from email or Android phone. so email and URLs could be missing too.
The easy solution is to accept empty values, and allow user to login. What do you think? I will create a pull request for @miled to consider merging into the plugin.
also as a reminder, PHP error messages should be turned off on a production server for security concerns.
best.
Oliver


mobile: switching C/C++ tests to not using yaml

This requires a minor envoy wrapper for listener code used in E-M tests.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

cc @phlax because I keep changing things =P


!! Improve Touch/Click Behavior to Distinguish Click From a Drag Operation

Using a touch device or a device with a trackpad (MacBook etc.) clicking on a slider element doesn't always is detected as a click but drags the slider instead. This can be frustrating for users because the drag is too faint to notice and it appears as if the click is broken.
What is Expected?
The slider should check for the click duration to distinguish a click from a drag operation.

There's  a workaround to solve that behavior by setting the threshold option to something greater than 0, like 20px. If "touch distance" will be lower than this value then swiper will not move:
swiperOptions: {
    ...
    threshold: 20,
},

Hope it helps.

Awesome! Maybe there should be a link to the swiper documentation to understand all the options?


Upload should be transactional

With a dependency expat-sys = "=2.1.0", cargo build -v shows:
unable to get packages from source

Caused by:
  failed to unpack package `expat-sys v2.1.0`

Caused by:
  failed to unpack `expat-sys-2.1.0/testdata/largefiles/recset.xml` into `/home/simon/tmp/multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/expat-sys-2.1.0/testdata/largefiles/recset.xml`

Caused by:
  corrupt deflate stream

I’m just guessing what happened here, but I think that when I hit CTRL-C in https://github.com/rust-lang/cargo/issues/2173 I interrupted cargo publish in the middle of the upload, and crates.io got a partial (corrupted) tarball. It should instead have considered this upload failed and not published that version.

Hm yes this is quite bad! I think it's basically the same as #2173 though (e.g. the same solution), so I'm gonna close in favor of that. Hoping to tackle it this week though!

It’s sort of related, but there are at least two sorts of failures to protect against (and that might required different measures):

Verification is accidentally skipped (without --no-verify) and a crate with a build error is published
Upload is interrupted in the middle and a partial tarball (that can not be extracted, or even can but is missing some files) is published

Ah well I'm gonna post a PR soon to fix both of these, so it doesn't matter too much.


Filter SQL results in lieu of a search clause.

Jenkins Build Summary
Built from this commit
Built at 20171231 - 21:03:20
Test Results

Build Type
Result
Status

clang.debug.unity
970 cases, 1 failed, t: 380s
FAIL :red_circle:

coverage
970 cases, 1 failed, t: 510s
FAIL :red_circle:

clang.debug.nounity
968 cases, 1 failed, t: 349s
FAIL :red_circle:

gcc.debug.unity
970 cases, 1 failed, t: 429s
FAIL :red_circle:

gcc.debug.nounity
968 cases, 1 failed, t: 357s
FAIL :red_circle:

clang.release.unity
969 cases, 1 failed, t: 468s
FAIL :red_circle:

gcc.release.unity
969 cases, 1 failed, t: 499s
FAIL :red_circle:

The unit test is failing because the manual filter threw off the LIMIT clause's results. I have to rethink this.

I rewrote the SQL in the original instead of making a manual filter. It passes unit tests and improves performance.

How big is a full history sqlitedb these days? Is it even still possible?


TextEditorDecorations are not visible for empty lines

VSCode Version: 1.7.2
OS Version: Ubuntu 16.04.1 LTS

I would like to display github review comments as text-decorations in visible editors. Unfortunately the decorations are not shown for empty lines. Is there a reason to not display them in this case?
IMHO the before and after elements of the decorations should be visible.

related: #8003

By design the before/after annotation have to be before or after something. The empty range is not really valid.

I would just like bump this, as I am working on a new release of GitLens and would love to be able to have decorations on blank lines.
See below:

The missing lines do belong to a commit and it would be great to not break the visuals either :)

This is resolved as of https://github.com/Microsoft/vscode/issues/30133

@alexandrudima You mentioned that your work on this

Yes, this should already work.


BitChute failures

Checklist

[X ] I'm reporting a broken site support
[ X] I've verified that I'm running youtube-dl version 2021.03.03
[ 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

PASTE VERBOSE LOG HERE
[tsingi@shadow channels]$ youtube-dl --verbose https://www.bitchute.com/video/6UXYHAyU0Nha/
[debug] System config: ['--prefer-free-formats']
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.bitchute.com/video/6UXYHAyU0Nha/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.03.03
[debug] Python version 3.9.2 (CPython) - Linux-5.10.19-200.fc33.x86_64-x86_64-with-glibc2.32
[debug] exe versions: ffmpeg 4.3.2, ffprobe 4.3.2
[debug] Proxy map: {}
[BitChute] 6UXYHAyU0Nha: Downloading webpage
[BitChute] 6UXYHAyU0Nha: Checking video URL
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://seed307.bitchute.com/9EVMKBtR18V5/6UXYHAyU0Nha.mp4'

ERROR: Did not get any data blocks
  File "/usr/bin/youtube-dl", line 33, in <module>
    sys.exit(load_entry_point('youtube-dl==2021.3.3', 'console_scripts', 'youtube-dl')())
  File "/usr/lib/python3.9/site-packages/youtube_dl/__init__.py", line 474, in main
    _real_main(argv)
  File "/usr/lib/python3.9/site-packages/youtube_dl/__init__.py", line 464, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 2054, in download
    res = self.extract_info(
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 798, in extract_info
    return self.__extract_info(url, ie, download, extra_info, process)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 805, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 837, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 871, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 1678, in process_video_result
    self.process_info(new_info)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 1962, in process_info
    success = dl(filename, info_dict)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 1901, in dl
    return fd.download(name, info)
  File "/usr/lib/python3.9/site-packages/youtube_dl/downloader/common.py", line 366, in download
    return self.real_download(filename, info_dict)
  File "/usr/lib/python3.9/site-packages/youtube_dl/downloader/http.py", line 352, in real_download
    return download()
  File "/usr/lib/python3.9/site-packages/youtube_dl/downloader/http.py", line 322, in download
    self.report_error('Did not get any data blocks')
  File "/usr/lib/python3.9/site-packages/youtube_dl/downloader/common.py", line 165, in report_error
    self.ydl.report_error(*args, **kargs)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 627, in report_error
    self.trouble(error_message, tb)
  File "/usr/lib/python3.9/site-packages/youtube_dl/YoutubeDL.py", line 589, in trouble
    tb_data = traceback.format_list(traceback.extract_stack())

[tsingi@shadow channels]$ youtube-dl --version
2021.03.03

Description

Bitchute is failing to download consistently on some videos and not on others.  i.e. if it fails it continues to fail or it works first time.
The dump I provided was the third such problem I've had today.
Sometimes BitChute videos just fail to publish properly but for this failure I CAN watch the video online.
I have no guess to offer.

One other thing I forgot to mention.
This is awesome software.  Well done.

Seeds are not always populated.  Bitchute may or may not be doing this intentionally to prevent abuse.   Happens outside ytdl too.

Good enough.


requires-read-access: Fix activating-item

activating-item needs to look at the new requires-read-access prefix. Otherwise, it's never going to be true on affected pages (right now if you look at Dashboard, it does not work).
cc @gib @sandersonet

👍 Thanks for this @krallin

Thank you @krallin!


Having the serial port open prevents video from streaming

Steps to repo;

Connect the HDMI2USB
Connect to the serial port using flterm
Try and get video from the video device

On newer Ubuntu versions you are the following output while trying to get video;
[19325.775956] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19330.773331] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19335.770452] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19340.767697] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).

 The kernel output will look like this;
[19424.301449] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19429.298562] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19434.295781] uvcvideo: Failed to set UVC probe control : -110 (exp. 26).
[19443.310507] usb 2-1.1: reset high-speed USB device number 24 using ehci-pci

After the reset line, the video will start streaming in.
From the point you have gotten video you can have both the serial and video running. Stopping / starting the programs (the video or serial) has no effect.
It works first time if you do;

Connect the HDMI2USB
Try and get video from the video device
Connect to the serial port.

The gstreamer pipeline dies with the following error;
obtained NetClientClock from host <NetClientClock object at 0x7f80747bd240 (GstNetClientClock at 0x1d98290)>
waiting for NetClientClock to sync…
starting pipeline
playing
Received Error-Signal
Error-Details: #13: gstv4l2object.c(3274): gst_v4l2_object_set_format_full (): /GstPipeline:pipeline0/GstV4l2Src:videosrc:
Call to S_FMT failed for MJPG @ 1280x720: Input/output error

Connect the HDMI2USB
Connect to the serial port using flterm, close flterm
Try and get video from the video device
See -110 errors
See the reset
Video starts working

Connect the HDMI2USB
Start the video stream, stop video stream.
Connect to the serial port using flterm, close flterm
Try and get video from the video device
See -110 errors
See the reset
Video starts streaming again.

It is likely the #148 issue is related here.
It is also likely this will be fixed by #149.

This issue was moved to timvideos/HDMI2USB-fx2-firmware#14


Put Models in specialized collections

Is your feature request related to a problem? Please describe.
I sometimes choose the wrong model for the type of request I make
Describe the solution you'd like
I would like to type "EditModel." and get only models that are valid for an edit endpoint, or "ImageModel." and get valid Image models, etc.
Describe alternatives you've considered
I see they're labeled, but it'd be nice if it weren't possible to supply an invalid model to my request object.
Additional context
I just started playing with the library today, it's very helpful in general. If this request is approved, I'd be happy to help implement it.

Unfortunately, OpenAI documentation is unclear about which models are supported with edit endpoint. I raised this with OpenAI team. We can implement it whenever they introduce new documentation. I will close the issue until then. Thanks for reporting.
https://community.openai.com/t/is-edit-endpoint-documentation-incorrect/23361/4


Python types

I got too excited and forgot to break my commits up by section. Oh well.
Whenever you have a chance @ozan

@aleksharma12 looking great! Done reviewing for now, my biggest question is around how much we should clarify that we are speaking about a specific implementation of the language.

I think putting a paragraph at the top that says "Hey; we're assuming you're using the CPython implementation." Otherwise, I feel like we'd have to constantly be inserting "depending on the implementation" everywhere through this chapter, and that's not as helpful to a student trying to understand the “approximate“ cost of all these operations.
But it is a good point, yes.

Great, yeah this is a good amount of clarification. Looks good to merge from my end!


Any links seems to be start redirected to this repo instead actual tweet

Any links seems to be start redirected to this repo instead actual tweet.
Wrong thambnail and redirect as it seems.

Can confirm. Just started happening a bit ago.

Oops, I didn't notice the issue you opened up until now, but yes I pushed a broken change and it was up for a few minutes before I reverted it. The tweet in question appears to load again. I pushed a major change to how HTTP routing is handled (moving from itty-router to hono) and I broke something about it. If you find any other problematic URLs please let me know! If the issue is resolved then we can close this.

Alright, I tried few new links and it still the same.
https://fxtwitter.com/i/status/1723124989011124275/
https://fxtwitter.com/i/status/1723125398815522940/
https://fxtwitter.com/i/status/1723125576800895155/
https://fxtwitter.com/i/status/1723125886843785552/

Ah, I see, the issue is the trailing slash. I have now deployed a fix that actually should work!

Alright, I can confirm it works for new link used but not existing.
Looks like some cached one be showing same.

Yeah, unfortunately Discord's cached links will not clear. If you clear the trailing slash (or add a query string, etc) you will be able to reset an individual embeds to a working state.

Yeah, I believe so.
Alright, issue itself be clear so let me close ticket now.


Travis builds on Linux fail due to dependency issues

From @ryanbrandenburg on Tuesday, October 10, 2017 12:33:18 PM
https://travis-ci.org/aspnet/WebSockets/jobs/278439693. This appears to have been caused by a change in Travis images rather than within this repo.
Copied from original issue: aspnet/WebSockets#205

We don't Travis anymore :)


Remove some extra back-ticks in documentation

I noticed what I assume are some extra back-ticks in the documentation.

Yep, but this should be placed into docblocks inside source code


refactor(generated api): generated api now uses async / await

Rewrite the generated api code to use async / await instead of Promises
fix #96

Coverage decreased (-0.03%) to 97.695% when pulling f9e970f23882b8a5420d1171109b428175bdcb79 on feature/async-generated-api into b0d8ab834254bdb0699f85455a6680fbc9d8e5d1 on develop.


Add Scorecard action

Before submitting
Please complete the following checklist when submitting a PR:

[ ] All new functions and code must be clearly commented and documented.

[ ] Ensure that code is properly formatted by running make format.
The latest version of black and clang-format-14 are used in CI/CD to check formatting.

[ ] All new features must include a unit test.
Integration and frontend tests should be added to frontend/test,
Quantum dialect and MLIR tests should be added to mlir/test, and
Runtime tests should be added to runtime/tests.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

@mlxd Is this something we still want in the repo?


Clawback claim not possible with 0 XCH

What happened?
The receiver of a clawback transaction currently needs to have XCH in their wallet in order to claim the transaction.

This wallet does not contain any XCH, to it cannot claim this transaction in the GUI. Note that it is not possible to enter zero fee.  Zero fee is allowed elsewhere in the GUI, so it should probably be allowed here as well.
Version
1.8.2rc3
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output or stacktrace
No response

Fixed in 1.8.2rc6.


[BUG] Crash if permission denied

Describe the bug
I had a directory in my music collection with read permissions but without execute and it caused checkrr to crash.
time="2024-10-18T17:29:41-07:00" level=fatal msg="lstat /music/They Might Be Giants/The Pamphlet/wav/04 Fortnight.wav: permission denied /music/They Might Be Giants/The Pamphlet/wav"

❯ ls -l /media/mp3/They\ Might\ Be\ Giants/The\ Pamphlet
total 16532
-rw-rw-r-- 1 drich mp3 4429965 Nov 11  2021 '01 Helicopter of Elves.mp3'
-rw-rw-r-- 1 drich mp3 5782999 Nov 11  2021 '02 Buckle Down, Winsocki.mp3'
-rw-rw-r-- 1 drich mp3 3974464 Nov 11  2021 '03 There Will Be Sad.mp3'
-rw-rw-r-- 1 drich mp3 2708864 Nov 11  2021 '04 Fortnight.mp3'
drwxrwsr-x 1 drich mp3       0 Aug 23  2023  __MACOSX
-rw-rw-r-- 1 drich mp3   23819 Nov 11  2021  cover.jpg
drwxrwsr-- 1 drich mp3     246 Aug 23  2023  wav

To Reproduce
Steps to reproduce the behavior:

Create a directory in your collection with permissions 0774
Run the scanner and tail the log
See it crash when it tries to read that directory

Expected behavior
While it should probably throw an error, it shouldn't crash.
Desktop (please complete the following information):

OS: MacOS
Browser: N/A
checkrr Version: 3.4.0

Additional context
FWIW, I'm probably not going to use checkrr to scan my music collection where it found this error. Several thousand of my mp3 files cannot be scanned by checker. For some reason it doesn't seem them as mp3 files. I wish there was a way to validate the magic number of files within the checkrr docker image. It doesn't have the file command, which is how I would normally do it. I'm not sure what the go module uses.

The go code is using h2non/filetype for magic number detection. As for the container, it's based on alpine so it should be as simple as apk add file once you get into a shell in the container.

Can you provide the stacktrace for the crash? I can't seem to reproduce it on macOS or nixos even with 0000 as the perms:
[nix-shell:~/src/checkrr]$ ./checkrr -c test-noarr.yaml

            _|                               _|
   _|_|_|   _|_|_|       _|_|       _|_|_|   _|  _|     _|  _|_|   _|  _|_|
 _|         _|    _|   _|_|_|_|   _|         _|_|       _|_|       _|_|
 _|         _|    _|   _|         _|         _|  _|     _|         _|
   _|_|_|   _|    _|     _|_|_|     _|_|_|   _|    _|   _|         _|
Checkrr version development
 Commit:
 Built On:
 Built By:
INFO[0000] Using config file: test-noarr.yaml
INFO[0000] Next Run: 2024-10-18 21:00:00 -0600 MDT
[GIN-debug] [WARNING] Creating an Engine instance with the Logger and Recovery middleware already attached.

[GIN-debug] [WARNING] Running in "debug" mode.     Notifications Connected=false Startup=true
INFO[0053] File "test/Movies/test-movie.mp4" is not a recognized file type  FFProbe=false Type=Unknown
INFO[0053] Couldn't find a target for file "test/Movies/test-movie.mp4". File is unknown.  Unknown File=true
+---------------------+-------------+
| Files Checked       |           1 |
| Hash Matches        |           0 |
| Hashes Mismatched   |           0 |
| Submitted to Sonarr |           0 |
| Submitted to Radarr |           0 |
| Submitted to Lidarr |           0 |
| Video Files         |           0 |
| Audio Files         |           0 |
| Text or Other Files |           0 |
| Unknown Files       |           1 |
| Elapsed Time        | 18.184488ms |
+---------------------+-------------+
INFO[0053] Next Run: 2024-10-18 21:01:00 -0600 MDT
^C
[nix-shell:~/src/checkrr]$ ls -la test/Movies/test-movie.mp4
---------- 1 aetaric users<PHONE_NUMBER> Oct 18 20:54 test/Movies/test-movie.mp4

I set the directory to 0000 and got a fatal and a exit code of 1, is that what you mean by "crash"?

Patched this. checkrr will bail on that path and resume execution now.

            _|                               _|
   _|_|_|   _|_|_|       _|_|       _|_|_|   _|  _|     _|  _|_|   _|  _|_|
 _|         _|    _|   _|_|_|_|   _|         _|_|       _|_|       _|_|
 _|         _|    _|   _|         _|         _|  _|     _|         _|
   _|_|_|   _|    _|     _|_|_|     _|_|_|   _|    _|   _|         _|
Checkrr version development
 Commit:
 Built On:
 Built By:
INFO[0000] Using config file: test-nologs.yaml
WARN[0000] No logging config found. Forcing standard out.
INFO[0000] Next Run: [2024-10-22 15:20:00 -0600 MDT]
[GIN-debug] [WARNING] Creating an Engine instance with the Logger and Recovery middleware already attached.

[GIN-debug] [WARNING] Running in "debug" mode.   Notifications Connected=false Startup=true
WARN[0034] An error occurred walking the tree for [test/Movies]. Please correct this before the next run.
ERRO[0034] Error encountered in checkrr run: [lstat test/Movies: permission denied]  run=failed
+---------------------+------------+
| Files Checked       |          0 |
| Hash Matches        |          0 |
| Hashes Mismatched   |          0 |
| Submitted to Sonarr |          0 |
| Submitted to Radarr |          0 |
| Submitted to Lidarr |          0 |
| Video Files         |          0 |
| Audio Files         |          0 |
| Text or Other Files |          0 |
| Unknown Files       |          0 |
| Elapsed Time        | 14.48423ms |
+---------------------+------------+
INFO[0034] Next Run: [2024-10-22 15:21:00 -0600 MDT]


Considering to use gRPC for each service connection

I am in the process of examining whether gRPC can be used for exchanges between the various services.
Using it with Go would give me a significant advantage, but otherwise I am carefully choosing the software architecture as it would be more difficult to implement due to unfamiliarity.

Useful article.

https://github.com/ymmt2005/grpc-tutorial
https://dev.to/techschoolguru/the-complete-grpc-course-protobuf-go-java-2af6

research summary

https://zenn.dev/e_kaikei/scraps/d757d82c29acb9

First, the logger embedded in the Echo's router must be integrated into the gRPC server.


fix: Keyboard goes down when shifting to saved mode.

Fixes #214
Changes: Keyboard goes down when shifting to saved mode.

@adityastic @anhanh11001 please review.

@iamareebjamal please review.


Add ServerSpec support

We need serverspec based tests for post convergence verification.

We're using Inspec now


Multiple thread HTTP POST

Hello, When I tried to HTTP POST with multiple threads after a time program crashes, how can I handle this problem?

Thanks for reporting this crash.
Could you please provide a bit more context?

Which version CPR are you using?
Does it always happen in a deterministic manner?
Please provide a small sample project or unit test, so we can reproduce and fix the issue.

Closing it for now.
Feel free to reopen in case this issue still persists.


refactor(ui): multi-level modals

Description:
Support multiple modals in a stack.
Motivation and Context:
Make for a better UX by allowing for multiple modals to be stacked on top of each other.
How Has This Been Tested?
Try all modals.
Types of changes:
UX improvement
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-1.6%) to 19.098% when pulling 99ccf825c624a137ef22fae53bae7e2518513fc2 on mrfelton:refactor/multi-level-modal into 0931d9c3d08de42d57b0e23b877b52e802cd13f9 on LN-Zap:next.

Coverage decreased (-1.6%) to 19.098% when pulling 99ccf825c624a137ef22fae53bae7e2518513fc2 on mrfelton:refactor/multi-level-modal into 0931d9c3d08de42d57b0e23b877b52e802cd13f9 on LN-Zap:next.

Made these updates @korhaliv - ready for re-review

@mrfelton This is starting to look not bad at all. But there is an issue. If you hit escape it still closes all levels of the popups

Yeah, re the escape issue - I'd like to address in a separate PR since there are already issues with that outside of this PR


More public API?

It would be really nice if the rustfmt package had more internal formatter data available so that anyone could make their own (non-CLI) frontend[^1]. Because right now, the only way to use FormatReport I could find is FormatReportFormatter, which simply prints the report to the stdout.
[^1]: My use case is an automated code review bot that comments on all improperly formatted lines rustfmt has found

Just noticed that the package hasn't been published for 7 months. I will check if the current version provides better access.

To be fully transparent, I'd advise against trying to utilize rustfmt as a lib. There's only a handful of cases where folks attempt to do this and it's a royal pain due to how dependent rustfmt is on the compiler internals.
It's an interesting sounding use case though, so FWIW I'd suggest running the rustfmt bin (like a user or IDE would) with use the json emit mode, perhaps even in a docker container, and then have your bot work with the json output. I'm not necessarily adamantly opposed to increasing the visibility from crate to public on a few things (eg. the FormattingErrorMap), but also generally not too keen on attempting to support more lib-based use cases which will become increasingly less viable.

There's a JSON mode? I have to try it out then. Thank you


thrift_proxy network filter fails coverage build

Description:
I'm running code coverage on the envoy source, branched off master at fcc68f1165d0343891d3ce14c2952019fe403743.
Command
BAZEL_TEST_OPTIONS="--jobs=3" test/run_envoy_bazel_coverage.sh
Output
The coverage build fails with the following error:
ERROR: /home/kyle/src/platform/envoy/source/extensions/filters/network/thrift_proxy/BUILD:297:1: undeclared inclusion(s) in rule '//source/extensions/filters/network/thrift_proxy:auto_transport_lib':
this rule is missing dependency declarations for the following files included by 'source/extensions/filters/network/thrift_proxy/auto_transport_impl.cc':
  'bazel-out/k8-dbg/bin/source/extensions/filters/network/thrift_proxy/_virtual_includes/protocol_lib/extensions/filters/network/thrift_proxy/binary_protocol_impl.h'
  'bazel-out/k8-dbg/bin/source/extensions/filters/network/thrift_proxy/_virtual_includes/protocol_lib/extensions/filters/network/thrift_proxy/compact_protocol_impl.h'
Target //test/coverage:coverage_tests failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 675.865s, Critical Path: 88.76s
INFO: 184 processes: 184 local.
FAILED: Build did NOT complete successfully

Version info
bazel version
Starting local Bazel server and connecting to it...
Build label: 0.19.1
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Mon Nov 12 15:32:30 2018<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Hm. auto_transport_lib depends on protocol_lib which depends on compact_protocol_lib and binary_protocol_lib which have the purportedly missing dependencies in their hdrs declarations.
It's certainly possible that I did something that's safe in bazel.  I wonder if this is some change in bazel 0.19.1?

According to https://docs.bazel.build/versions/master/build-ref.html#actual_and_declared_dependencies it's not safe. The rationale is that since the lib directly depends on those headers, it should have an explicit dependency in the build file so that somebody doesn't indirectly break the dependency change by changing protocol_lib.
I honestly don't understand why I'm seeing this but nobody else seems to. :/ Could the coverage build be passing different arguments to bazel?

I guess that makes sense. It probably just happened to work until 0.19.1. Once I reproduce it I'll rejigger the build files to fix it.

FWIW, I think I fixed it by adding the headers to the exports of protocol_lib. I'm not sure offhand if that's considered a good approach, though, and I eventually gave up on the gcov build because it was taking too long anyway lol.

This doesn't happen in non-coverage builds?

Hmmm. I just built all the UT for thrift_proxy.
bazel test  --jobs=4  //test/extensions/filters/network/thrift_proxy:*
...
//test/extensions/filters/network/thrift_proxy:app_exception_impl_test   PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:auto_protocol_impl_test   PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:auto_transport_impl_test  PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:binary_protocol_impl_test PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:buffer_helper_test        PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:compact_protocol_impl_test PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:config_test               PASSED in 0.9s
//test/extensions/filters/network/thrift_proxy:conn_manager_test         PASSED in 1.3s
//test/extensions/filters/network/thrift_proxy:conn_state_test           PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:decoder_test              PASSED in 0.9s
//test/extensions/filters/network/thrift_proxy:framed_transport_impl_test PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:header_transport_impl_test PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:integration_test          PASSED in 52.7s
//test/extensions/filters/network/thrift_proxy:metadata_test             PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:route_matcher_test        PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:router_ratelimit_test     PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:router_test               PASSED in 2.1s
//test/extensions/filters/network/thrift_proxy:thrift_object_impl_test   PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:translation_integration_test PASSED in 28.8s
//test/extensions/filters/network/thrift_proxy:twitter_protocol_impl_test PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:unframed_transport_impl_test PASSED in 0.7s

You'd think that would trigger the issue. So, what the heck is the gcov build doing?

This fails: bazel test  --jobs=4 --spawn_strategy=standalone   //test/extensions/filters/network/thrift_proxy:*
ERROR: /home/kyle/src/platform/envoy/test/extensions/filters/network/thrift_proxy/BUILD:152:1: undeclared inclusion(s) in rule '//test/extensions/filters/network/thrift_proxy:auto_protocol_impl_test_lib':
this rule is missing dependency declarations for the following files included by 'test/extensions/filters/network/thrift_proxy/auto_protocol_impl_test.cc':
  'bazel-out/k8-fastbuild/bin/source/extensions/filters/network/thrift_proxy/_virtual_includes/protocol_lib/extensions/filters/network/thrift_proxy/binary_protocol_impl.h'
  'bazel-out/k8-fastbuild/bin/source/extensions/filters/network/thrift_proxy/_virtual_includes/protocol_lib/extensions/filters/network/thrift_proxy/compact_protocol_impl.h'
INFO: Elapsed time: 21.903s, Critical Path: 21.63s, Remote (0.00% of the time): [queue: 0.00%, setup: 0.00%, process: 0.00%]
INFO: 5 processes: 5 local.
FAILED: Build did NOT complete successfully
//test/extensions/filters/network/thrift_proxy:app_exception_impl_test (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:binary_protocol_impl_test (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:buffer_helper_test (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:compact_protocol_impl_test (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:config_test      (cached) PASSED in 0.9s
//test/extensions/filters/network/thrift_proxy:conn_manager_test (cached) PASSED in 1.3s
//test/extensions/filters/network/thrift_proxy:conn_state_test  (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:decoder_test     (cached) PASSED in 0.9s
//test/extensions/filters/network/thrift_proxy:framed_transport_impl_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:header_transport_impl_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:integration_test (cached) PASSED in 52.7s
//test/extensions/filters/network/thrift_proxy:metadata_test    (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:route_matcher_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:router_ratelimit_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:router_test      (cached) PASSED in 2.1s
//test/extensions/filters/network/thrift_proxy:thrift_object_impl_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:translation_integration_test (cached) PASSED in 28.8s
//test/extensions/filters/network/thrift_proxy:twitter_protocol_impl_test (cached) PASSED in 0.6s
//test/extensions/filters/network/thrift_proxy:unframed_transport_impl_test (cached) PASSED in 0.7s
//test/extensions/filters/network/thrift_proxy:auto_transport_impl_test  PASSED in 0.6s

Executed 1 out of 21 tests: 20 tests pass and 1 fails to build.
FAILED: Build did NOT complete successfully

Seems to be the opposite of what's indicated here?

--spawn_strategy strategy
This option controls where and how commands are executed.
standalone causes commands to be executed as local subprocesses.
sandboxed causes commands to be executed inside a sandbox on the local machine. This requires that all input files, data dependencies and tools are listed as direct dependencies in the srcs, data and tools attributes. This is the default on systems that support sandboxed execution.

I'm not able to reproduce this on either MacOS or Linux, but the doc seems pretty clear that dependency-only libraries aren't supported. I believe #5039 will fix it by removing the protocol_lib and transport_lib targets.

It's kind of sad. I feel like there's a difference between a library whose sole purpose is to aggregate the functionality of other libraries, and one which simply happens to use those other libraries in its implementation. You're a lot less likely to "accidentally" remove dependencies in the former case.


Switch to SIMDJSON OnDemand API

https://github.com/simdjson/simdjson/releases/tag/v0.9.2

#9 upgrades the SIMDJSON to v0.9.1

#11 schedules upgrade to SIMDJSON to v0.9.2


Chart refactor

Since there's still an open issue in the main repo and I've the feeling this chart has been dropped, I've decided to rework it completely.
The main differences are:

different values for server and agent
HPA
it works (seriously, the original didn't work for me because of this issue)
server is deployed using a StatefulSet, allowing the usage of PV (through volume claim template), resolving #2
a custom image can be specified, useful when an image with a custom plugin should be used
I've no longer seen #1, maybe just because of dkron upgrade (3.1.6)

Known issues:

no health check specified, always because of this (help wanted)

Let me know what are your thoughts, I'm open to feedback available to make any changes

Sorry for the mega-delay here @FedeBev, thank you very much for the work! I had not prioritized the K8s work until now, going to test this and merge if it works.

What kind of problem are you facing @victorcoder? I just tried a rolling update from version v3.1.8 to v3.1.10 and everything went good except for a slight leader election delay.
Do you mind sharing your values and some details about your cluster?
I've just fixed the missing newlines and cleaned up the sample values file

I just do a helm install helm upgrade and the rolling is too fast for the leader election to happen correctly, when done rolling there's no leader in the cluster, all nodes ends up as followers.
Are you tweeking some other options?

No additional tweek on my configuration.
This also happens to me, but after a few seconds after the end of the rollout a new leader is elected again and everything goes fine.
We can try to tune the rollout strategy, but taking into account the issue about the kubernetes discovery, I don't think we can achieve something better with helm. An operator would solve the issue, but it's a huge effort.

@FedeBev what about a liveness, readiness check?

@vcastellm in the  issue about the kubernetes discovery I've mentioned before, you can find why I wasn't able to use those.
I don't see any other possible way right now. I guess we should change something about how dkron starts or request a PR to the kubernetes discovery library.
In my opinion, the point is that dkron should be able to discover itself when the pod is running (/health returns 200), NOT when is ready. This way the k8s discovery works and we can implement a useful /health and ready probe.

Understood, thanks, I will take a look on how we can improve the /health endpoint

Keep me posted, I'd be glad to help.
Dkron could become a core component for a project I'm working on  however our platform relies entirely on K8s and this is a big issue

I'm also very interested in this fix. @vcastellm @FedeBev anything that I can do to make this happen? Also, is this not better than the currently available chart?

@Espina2 the chart is working on my cluster in a dev environment, but it's still very young and there's still a long way to go before being production ready.
There's an issue with the k8s discovery that makes you dkron cluster unavailable due to master election during the rolling upgrade.

hello! also highly interested in this. Has anyone come up with a way to work around the discovery issue? any thoughts on how to proceed?

Merged this PR and added some changes, I'll take over from here.


assertion failed: !ret.is_empty()

STR
$ cargo new assertion
$ cd assertion
$ vim Cargo.toml
$ cat Cargo.toml
[package]
name = "assertion"
version = "0.1.0"
authors = ["me"]

[lib]
crate-type = ["dylib"]  # <- important bit

$ RUST_BACKTRACE=1 cargo build --target x86_64-unknown-linux-musl
thread '<main>' panicked at 'assertion failed: !ret.is_empty()', src/cargo/ops/cargo_rustc/context.rs:335
stack backtrace:
   1:     0x5562af320300 - sys::backtrace::tracing::imp::write::h777bcf35f5f62316pru
   2:     0x5562af32839b - panicking::default_handler::_$u7b$$u7b$closure$u7d$$u7d$::closure.43419
   3:     0x5562af328008 - panicking::default_handler::he9bc2fc9631cb5acdZy
   4:     0x5562af30ad1c - sys_common::unwind::begin_unwind_inner::h180f7a790497cb1fegt
   5:     0x5562aee5578f - sys_common::unwind::begin_unwind::h11943618306376478202
   6:     0x5562aeeae8c4 - ops::cargo_rustc::context::Context<'a, 'cfg>::target_filenames::h5376a53660062a8f5Nl
   7:     0x5562aef2bf33 - ops::cargo_rustc::compile::hbb14b05fcf244699gcp
   8:     0x5562aeebc765 - ops::cargo_rustc::compile_targets::h7c5bb61b1a33fadd61o
   9:     0x5562aeeb1d09 - ops::cargo_compile::compile_pkg::hc537479810426ef9qLh
  10:     0x5562aeeaebf0 - ops::cargo_compile::compile::h1fc23cb180e812aeDFh
  11:     0x5562aede63ff - call_main_without_stdin::h12386465949200514573
  12:     0x5562aedd76d0 - execute::h38ff5dff31c326besMc
  13:     0x5562aedd4344 - call_main_without_stdin::h8984535069057422791
  14:     0x5562aedd13ca - main::h5458bbdcd8f4a2307ca
  15:     0x5562af327c64 - sys_common::unwind::try::try_fn::h9504535173218613138
  16:     0x5562af31f63b - __rust_try
  17:     0x5562af3276ee - rt::lang_start::h86595346586db82djRy
  18:     0x7f8418a8f61f - __libc_start_main
  19:     0x5562aedd0d28 - <unknown>

The x86_64-unknown-linux-musl target doesn't support building dylibs, so this configuration won't work, but it should display an error message instead of panicking.
cc @alexcrichton

Wow I'm kinda surprised it took this long for this to surface...
Should be an easy enough bug to fix!

I think my PR solve the problem, but I'm not sure is the correct way, any comments about it?

@jespino I guess, we can just use try! here:
LibKind::Dylib => {
    let (prefix, suffix) = try!(self.dylib(unit.kind));
    ret.push(format!("{}{}{}", prefix, stem, suffix));
}

Output:
$ cargo build --target x86_64-unknown-linux-musl
error: dylib outputs are not supported for x86_64-unknown-linux-musl
$

The same can be done for StaticLib, I guess. @alexcrichton thoughts?

This is currently being implemented in https://github.com/rust-lang/cargo/pull/2437 I believe, but I think we only want to use try! here if there are 0 filenames, otherwise we should just ignore the ones that don't work.


Add moderatehatespeech.com capabilities to CM

https://moderatehatespeech.com/
https://www.reddit.com/r/redditdev/comments/xdscbo/updated_bot_backed_by_moderationoriented_ml_for/
https://moderatehatespeech.com/docs/
Potentially as another provider for the sentiment analysis rule or as a separate rule entirely.
Would provide another signal for comment toxicity. Things to think about WRT where to add...
MHS results only provide a confidence score (0 to 1) which doesn't mesh well with sentiment score (-1 to 1) mostly because sentiment results tell positive/negative and their score is just "how confident is this toxic" -- seems to point towards an independent rule.
On the other hand, requiring a separate rule to be configured/added introduces more complexity. But it also allows for more granularity...

Perhaps a hybrid:
Add a rule for MHS
For SA if the criteria is for detecting negative intent then get MHS result and apply confidence as a weighted score

Added in 0.13.0


[Snyk] Security upgrade golang from 1.10 to 1.21.11

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 2 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

awstesting/sandbox/Dockerfile.test.go1.10

We recommend upgrading to golang:1.21.11, as this image has only 89 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Integer Overflow or Wraparound SNYK-DEBIAN9-SQLITE3-307593
  501

Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
  343

Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
  343

Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
  343

Out-of-Bounds SNYK-DEBIAN9-GLIBC-356506
  343

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)


Remove Signature['Args'] when there are no arguments

I would like to see... 🙋♀🙋♂
In v0.4.1, the codemod uses the empty object {} to mean that there are no arguments. This assumption had been made to ease the initial development.
interface Signature {
  Args: {};
};

Glint, on the other hand, recommends omitting the Args key:

Args represents the arguments your component accepts. Typically this will be an object type mapping the names of your args to their expected type. If no Args key is specified, it will be a type error to pass any arguments to your component.

Confirm this recommendation with the Glint team. Afterwards, update the codemod so that we may get:
interface Signature {};

Why and how 💬
The how is unclear at the moment. There will be a need to refactor code first.

Would be great to have this fix, this is the one issue I'm running into with the codemod.
https://github.com/datafruits/datafruits/pull/1150
Thanks for making this!

@mcfiredrill Appreciate your feedback! I'll try to work on it later this week/next week.

I asked the TypeScript/Glint team some months ago, and received a reply that having Args: {} is correct when there are no arguments.

writing Args: {} is completely correct (it's the default value if Args is unspecified)
it's just that typescript-eslint doesn't like {} as a type, because it comes with some footguns

https://discord.com/channels/480462759797063690/786312479620726804/1149362992169893898


Auto snippet matches alpha and \alpha

Hey, just wanted to say I really appreciate this plugin. In math mode, typing alpha turns into \alpha and typing \alpha turns into \\alpha. Is this intended? I would expect typing \alpha to not trigger the snippet and return \alpha (unaffected).

No, that's not intended. However, I can't reproduce.
Does \alpha always expand for you?

I realized I was using a different plugin of the same name LOL


Project feedback features

Portfolio: finish mobile version

My recent work of the portfolio
Project list
About me
Feedback form
Footer

Below correction mad

Use the correct font size
Change h1 tags to h6
change my section from display:flex to grid


How can I display the n-grams of rouge?

How can I display the n-grams of rouge? I need it for my thesis.
and does anyone know how to use a different evaluation?

I have another question
Is there a list of stop words?


feat: add timezonedb 2023.2 release

update timezonedb with release 2023.2

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


README: add AdGuard Home to project list

added AdGuardHome
sorted and reformated the markdown table

How does Adguard use quic-go?

See https://github.com/AdguardTeam/AdGuardHome/issues/3955

Sorry for the delay!


Added README.md for sync-tool

Added README.md for sync-tool local setup

@maheshkumargangula - Can you please review and merge this PR for adding README.md ??
cc : @coolbung

@GauravTekdi - Thanks for the contribution.


Hide characters

Is it possible to hide the tokens in the editor?
(This)

(Instead of this)

It can be easily done with CSS:
.cm-formatting {
	display: none;
}


Slack use of "empty(expression)"

If targeting PHP 5.3 * 5.4, I don't think you'll want to use expressions in "empty()":
PHP Fatal error:  Can't use method return value in write context in /var/www/html/kanban/app/Integration/SlackWebhook.php on line 51
if (! empty($this->config->get('integration_slack_webhook_channel'))) {
Pulling the expression out into a variable fixes it.

Thanks, it's fixed in master.


Migrate claim extended fields

Overview
Facility claims can contain a facility_type and processing_types.
Following moving to the new taxonomy for these values, existing types
are invalid.
The following invalid values are dropped: "Other", "Boarding", "Denim
Services". All other existing values are mapped to the equivalent
taxonomy values. Extended Fields are created for the processing_types
and facility_type.
Extended fields are also created for other applicable claim fields.
Connects #1648
Demo

Notes
I pulled the current facility_production_types and facility_type values from production and created a mapping to the new system, with the assistance of the OAR team.
Because we are filtering out / removing some invalid values, and because we need to update the actual claim object for the facility_type and processing_types fields, it wasn't possible to directly use the existing scripts for processing without a lot of edits, and it didn't make sense to rewrite them for this very specific one-off use, so I ended up copying over a lot of the code and editing where needed in the migrations file.
As part of this migration, I checked out out an earlier version of the develop branch before the latest version of the taxonomy was applied, ran resetdb, and created several claims. Then I ran migrations on the current version of develop and on this branch, and confirmed that the migrations ran.
Testing Instructions

Manual create old-style claims (or see notes for an optional alternate testing path).

Run ./scripts/server, login, and claim two facilities. Approve the claims as an admin. Fill out the claim fields for name, native language name, parent company, number of workers, and product types.
Run ./scripts/manage shell_plus, then (use the appropriate claim ids):

f = FacilityClaim.objects.get(id=1)
f.facility_production_types = ['Cut and Sew / RMG', 'Boarding', 'Denim Services', 'Fusing', 'Marker Making', 'Pattern Grading', 'Sample Making', 'Trimming/Embellishment', 'Warehouse']
f.facility_type = 'Cut and Sew / RMG'
f.save()

f2 = FacilityClaim.objects.get(id=2)
f2.facility_production_types = []
f2.facility_type = 'Other'
f2.save()

 * Confirm in the shell that the claims match the structure of existing claims.

Run ./scripts/manage migrate.
View the claim details and confirm that for claim 1 the new values match the expected mapping, and for claim 2 no values are present.
View the claimed facilities' detail pages and confirm that the extended field values match the expected taxonomy mapping.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Hold off on reviewing - I've realized a key piece is missing.

This is ready for review now.

The build broke because we merged a PR with a new migration. Rebasing and fixing the numbering and the dependency array should be all that is needed.

Thank you for reviewing!


Invalid syntax

hello,
i have tried doing everything as is in the readme,
pip install deepface
composer require astrotomic/php-deepface

however, when i do basic
$deepface = new \Astrotomic\DeepFace\DeepFace();
$deepface->extractFaces(   img_path: storage_path('test.jpg'), );

i get error
  Symfony\Component\Process\Exception\ProcessFailedException  The command "'/usr/bin/python3' '-c' 'import json; import sys; from deepface import DeepFace;  try:     result = DeepFace.extract_faces(         img_path = "storage/test.jpg",         target_size=[224,224],         enforce_detection=True,         anti_spoofing=False,         detector_backend="opencv",         align=True,         grayscale=False,     );      print(json.dumps(result, default=str)) except ValueError as e:     print(json.dumps({"error": str(e)}), file=sys.stderr)'" failed.

Exit Code: 1(General error)

Working directory: /home/forge/...

Output:
================

Error Output:
================
  File "<string>", line 1
    import json; import sys; from deepface import DeepFace;  try:     result = DeepFace.extract_faces(         img_path = "storage/test.jpg",         target_size=[224,224],         enforce_detection=True,         anti_spoofing=False,         detector_backend="opencv",         align=True,         grayscale=False,     );      print(json.dumps(result, default=str)) except ValueError as e:     print(json.dumps({"error": str(e)}), file=sys.stderr)
                                                             ^^^
SyntaxError: invalid syntax.

Hi, I noticed that error too. I opened a PR (#16) to fix it.

@verybigelephants I suggest to clone my branch, the issue you mentioned is fixed and I updated the package to include recent changes introduced in v0.0.93 of deepface.
You need to manually include it to your project though.

There's a new release: https://github.com/Astrotomic/php-deepface/releases/tag/0.4.0


Error loading Omni Sharp

I have the latest versions of unity-mono develope- the plug-in, I downloaded everything yesterday.
After doing all the steps and opening VSC I get this error:
"Error: Cannot start Omnisharp because Mono version >=4.0.1 is required"
I'm on Mac.

You need to install the mono framework (sdk).
http://www.mono-project.com/download/


Ability to use space instead of ctrl-space

I know this was somewhat discussed in #1, but you sent a link to an add-on that, a few years later, no longer exists. I tried a number of add-ons for shortcut management but none addressed the issue. Thanks!

I changed the shortcut for exiting iteration mode since to avoid the conflict. If you want to change the shortcuts, you'll have to use Chrome, because despite the mission statement, the switch to webextension did not add better shortcut handling (yet?) while taking away the existing possibilities.
Now to the title of this issue, using only space will not be possible, even in Chrome, unless I implement a lot of logic to handle this. But I won't because there are so many scenarios where this can interfere with websites and other extensions. While a broken extension would be the least of the problems, websites where space doesn't work any more would be a huge and it would be very hard for a user to figure out where the problems come from.


About Newsletter Plugins

hi
i need plugins for newsletter so what plugins available can help me about that

If it not here or on plugins.cakephp.org - you can ask in the IRC channel or just build one (with the help of the community even maybe).


Websockets do not work when MITM is enabled

Title says it all. You can test this out pretty easily by starting a browser using a running proxy and navigating to https://www.websocket.org/echo.html. The page loads just fine but the websocket demo won't actually work properly.
In looking at the browser logs, it appeared that the websocket upgrade is failing and a 400 is returned. I poked around in the code and it appears to be a couple of issues:

The proxy strips out the Connection and Update headers if the proxy is not in transparent mode. These are required headers for the websocket upgrade sequence and should not be stripped out in this case.
If the proxy has transparent mode enabled, the websocket upgrade works but the socket does not transmit/receive any data afterwards. This appears to be a result of the way in which the proxy handles HTTP requests; the connection is left open after the upgrade is completed but no bytes are transferred from the client to the server.

I believe that the fix for this is to switch the connection to tunneling after the upgrade completes. I've got a local version of this working, but the proxy is rather complex and I doubt that I've implemented my solution in the best way possible.

There's some guys with a start on websocket support over here:  https://github.com/verygoodsecurity/LittleProxy/pull/53/files

I also think some of their connection handling flow and fixes look promising to incorporate. Unfortunately, they don't publish artifacts, but we are free to incorporate their stuff.

Wow, that would be cool to add in. They did a much more clean job than I did locally, I never even thought about trying to muck with the Netty pipeline stuff. @mrog would you be willing to use what they've got? Your fork seems to be more active and it's being used by browserup (which is what I'm actually using anyway). If so, I can try to open a PR with their change.

They have some heavy alterations, which I think we actually want, that you may or may not need to grab to integrate this.   From my perspective, feel free to pull in more of their changes if needed to be as compatible as needed to try this out. Thoughts @mrog ?

If they have a good solution, I'm all for using it.  I don't have the time to look at this in detail right now, but I'm happy to accept pull requests (hint, hint).

So, quick update. I got the changes from the verygoodsecurity fork added and mostly working, but there are still instances in which connections will hang. I spent a long time trying to figure out what exactly is causing this condition but it was pretty slow going and I didn't get far.
I started working on a toy implementation of a Netty proxy just to get more familiar with this whole process and after doing so I think there is an issue with the verygoodsecurity approach. It does not appear to be generally regarded as a good idea to remove handlers from the pipeline after the channel has been initialized.
The verygoodsecurity code will dynamically remove the HttpUpgradeHandler from the pipeline when the request is not a websocket, and I think it causes some timing-sensitive issues when a message comes in before the pipeline is updated.
I think it might be better to simply have the handler conditionally decide not to take an action based solely on the state of the current flow. For instance, when the Upgrade request is returned with a 101, we can set a state variable and have the handler do something like this:
if(!websocketRequestUpgraded){
  ctx.write(msg, promise);
} else {
  // do whatever we need to do for websockets
}

Thoughts? I'm still pretty new to Netty and I could be misinterpreting advice online.

I'm not an expert in this area by any means. Conceptually, though, if I imagine that you have active handlers running on a channel, it does seem a bit odd and maybe error prone to tinker with the handlers while the channel is active.
Your approach seems safe in principle and doesn't seem like it would have a lot of downside.

Please try again with xyz.rogfam:littleproxy:202.  Note that the proxy must be operating in transparent mode in order for the websocket upgrade to work.
See https://github.com/mrog/LittleProxy/pull/63

To add some more detail to my previous answer, here is the state of WebSocket support in LittleProxy 2.0.2 with MITM in transparent mode:

The only cases that don't currently work are when a client attempts to tunnel via HTTP CONNECT to a non-ssl-enabled endpoint, hence the issues with ws:// and http://.  That issue is described in more detail here: https://github.com/mrog/LittleProxy/issues/71

FYI for those interested in Websockets support... I submitted a PR which fixes some significant issues with it: https://github.com/LittleProxy/LittleProxy/pull/308


Fix: parse error during stack diff

Fix #597
Bug introduced during upgrade to CDK to 2.148 mangleLikeCloudFormation and fullDiff requires the template content in JSON format instead of the file paths.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi @k7k0 , thank you for your contribution to the Landing Zone Accelerator!
I'm pleased to say this work has been integrated into the code base and will be available shortly in the v1.10.0 release. I'll close this PR in the meantime and update the associated issue once that release is publicly available.
Thank you for your support of the LZA!

Too good @k7k0 and @erwaxler
Appreciate it 🚀


Add an option to disable time stamp element generation in WSSecurity

The module works great, until I use it to access charge point web services.
I'd like to add an option to disable the timestamp generation in WSSecurity XML.
When I use this module to access charge point web service, their service is not able to process the timestamp.
The service wsdl is here, https://webservices.chargepoint.com/cp_api_4.1.wsdl

I'm closing this PR as there was no activity for more than a year now...


Correct usage of packet.NewPacket() method

Hello!
I have an issue with packet.NewPacket() method, but I don't know whether it is a bug or method misuse. Below is code snippet, which results in segmentation violation error.
package main

import (
	"github.com/intel-go/nff-go/packet"
)

var pkt, err interface{}

func main() {
	pkt, err = packet.NewPacket()

}

Here is the error:
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x44 pc=0x4eba47]

runtime stack:
runtime.throw(0x75519d, 0x2a)
	/usr/local/go/src/runtime/panic.go:774 +0x72
runtime.sigpanic()
	/usr/local/go/src/runtime/signal_unix.go:378 +0x47c

goroutine 1 [syscall]:
runtime.cgocall(0x4ee8f0, 0xc000058e60, 0xc0000c41f0)
	/usr/local/go/src/runtime/cgocall.go:128 +0x5b fp=0xc000058e30 sp=0xc000058df8 pc=0x439bdb
github.com/intel-go/nff-go/internal/low._Cfunc_allocateMbufs(0x0, 0xc0000b6240, 0x1, 0x0)
	_cgo_gotypes.go:401 +0x4d fp=0xc000058e60 sp=0xc000058e30 pc=0x4e415d
github.com/intel-go/nff-go/internal/low.AllocateMbuf.func1(0x0, 0xc0000b6240, 0xc0000c41e0)
	/home/secret/go/pkg/mod/github.com/intel-go/nff-go@v0.9.2/internal/low/low.go:679 +0x97 fp=0xc000058e98 sp=0xc000058e60 pc=0x4e4547
github.com/intel-go/nff-go/internal/low.AllocateMbuf(0xc0000b6240, 0x0, 0xc000058f50, 0x43c4cf)
	/home/secret/go/pkg/mod/github.com/intel-go/nff-go@v0.9.2/internal/low/low.go:679 +0x39 fp=0xc000058f00 sp=0xc000058e98 pc=0x4e43c9
github.com/intel-go/nff-go/packet.NewPacket(0x0, 0x8e000, 0xc00008c058)
	/home/secret/go/pkg/mod/github.com/intel-go/nff-go@v0.9.2/packet/packet.go:834 +0x50 fp=0xc000058f38 sp=0xc000058f00 pc=0x4e5000
main.main()
	/home/secret/go/src/github.com/MyProject/main.go:10 +0x26 fp=0xc000058f60 sp=0xc000058f38 pc=0x4e52c6
runtime.main()
	/usr/local/go/src/runtime/proc.go:203 +0x21e fp=0xc000058fe0 sp=0xc000058f60 pc=0x4630fe
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1357 +0x1 fp=0xc000058fe8 sp=0xc000058fe0 pc=0x48b7b1

My environment params are:

go1.13.4 linux/amd64
gcc version 7.4.0 (Ubuntu 7.4.0-1ubuntu1~18.04.1)

I don't think this function would work correctly before NFF-Go framework is at least initialized, maybe it should be even started to work correctly.

@AlekseyKanaev thanks for pointing this out - we should update the documentation to reflect the situation better. Packets are allocated in the memory which is addressable after the NFF-Go is initialized. As a reference example, please look into https://github.com/intel-go/nff-go/blob/master/examples/generate.go It shows how to do the whole flow correctly.

@AlekseyKanaev thanks for pointing this out - we should update the documentation to reflect the situation better. Packets are allocated in the memory which is addressable after the NFF-Go is initialized. As a reference example, please look into https://github.com/intel-go/nff-go/blob/master/examples/generate.go It shows how to do the whole flow correctly.

Thanks!!! That example helped me to resolve the issue.


Cant add second anchor point to first cotrol point in Path2D

There is no option to add second anchor point in the first point of Path2D - while it is possible to do it on Path3D. Also editing both is quite different and incoherent to me - including shortcuts for doing same things.

Path does not allow curves, why would somebody want 2 pivots in the first or last point, now the shortcuts and behaviors of both editors is coherent

Closing as old then.


Travis isn't configured properly

1- No need to add coverage if the tests are not written yet.
2- Incorrect directory path for js files (jslint)
For now better to use Codesniffer for code quality.

@Rohit599 You should work on this first and add the codesniffer so it will be easy for me to review.


[add-resource] New Plugin: Admin Frontmatter Yaml

Add a new Plugin to the Grav Repository, the project details: grav-plugin-admin-frontmatter-yaml.
In short, it lets adds a functional to contain non-localized header blocks in the frontmatter.yaml file. This is allows you to avoid duplication of common values, which are saved out in a single file.
The most frequent use - for sites with content in many languages.

I've added this to GPM, but you need to create a release before it will show up.  Check the learn docs for details.
https://learn.getgrav.org/16/advanced/grav-development#theme-plugin-release-process
Cheers

Thanks.
I created tag and release (grav-plugin-admin-frontmatter-yaml/releases)

It's showing in GPM now.. thanks.


Secure etcd for non-GCE distros.  Etcd-in-Pod for apiserver.

For GCE, etcd listens to clients on localhost, and does not listen to peers.
On all salt distros, it appears to use localhost for the client port (4001).
(There is still some possible cleanup to do in cluster/saltbase/salt/kube-apiserver/default to hardcode this.  See zmerlyn's comment.)
On some non-salt distros,  it accepts clients on the public address of the master node.
Some people may want this since they are running flanneld too.
I suggest that we solve this by running an apiserver-only etcd instance in a pod with the apiserver (the way we do for skydns).  @ArtfulCoder is working on this.  Then etcd will be secure because it is on the localhost of the Pod's netns.
On CoreOS/flannel-based distros, there will be multiple un-related etcd server-pools.  It is up to the user or the distro author to secure these etcds.  Examples here are libvirt-coreos, rackspace.
Once we move to replicated apiserver, we can setup TLS credentials for the peer-to-peer communication between the etcd-in-a-pods by generating certs and setting flags per example 3 in https://github.com/coreos/etcd/blob/master/Documentation/security.md

#5011 is the one targeting for v1.0
cc/ @lavalamp who is a different proposal beyond v1.0

@erictune When I looked into this just a few days ago, I found etcd listening on all interfaces. The salt configuration is confusing (some of it says <IP_ADDRESS> and some says <IP_ADDRESS>) and #4654 by @davidopp also makes it look like we are listening on all interfaces on GCE.

Yeah, I just figured this out now.  Files an issue about confusing salt config.
#6081

@roberthbailey Is this the case when running etcd2? I think we default to localhost across the board, but this may have more to do with the salt scripts.

@kelseyhightower Our salt config specifically sets the bind address for etcd2 to <IP_ADDRESS>. But with etcd in a pod, we won't need to use salt to pass the command line flags any longer.

@roberthbailey Ok. While it maybe obvious I'll just say it anyway. localhost in the pod is different from localhost on the host. So you might still need to set etcd2 to listen on <IP_ADDRESS> in the container if you are not using --net=host (docker). You'll also need to advertise the IP of the docker hosts for peer and client communication -- see: https://github.com/coreos/etcd/blob/master/Documentation/docker_guide.md

We are running etcd in a pod with net=host
I believe localhost would be the same as the underlying host.
Sent from my iPhone

On Apr 2, 2015, at 1:15 PM, Kelsey Hightower<EMAIL_ADDRESS>wrote:
@roberthbailey Ok. While it maybe obvious I'll just say it anyway. localhost in the pod is different from localhost on the host. So you might still need to set etcd2 to listen on <IP_ADDRESS> in the container if you are not using --net=host (docker). You'll also need to advertise the IP of the docker hosts for peer and client communication -- see: https://github.com/coreos/etcd/blob/master/Documentation/docker_guide.md
—
Reply to this email directly or view it on GitHub.

@ArtfulCoder OK, that clears things up. Yeah just set to localhost then or use etcd2 defaults.

#6417

Obsolete


x86 calling conventions broken for floating point returns

The x86 lifter currently uses XMM0 as the floating point return register, but the win64 and SYS-V calling conventions specify it should be the full-width ZMM0. This PR addresses this issue, but this breaks some of the unit tests (so it has been un-merged).
However, this may be revealing an issue in the analysis, since the ZMM0 register is apparently being included in the default return (clobber?) register sets for functions.

The ZMM0 register gets included in the return register set for unknown functions, causing cascading issues:

======================================================================
FAIL: test_binary___raw (generated.default.unit.TestBinaryNinja)
----------------------------------------------------------------------
# This is caused by zmm0 being included in the clobber set of the unknown function sub_f00302 being called
<---18 same lines--->
+ Function: f00060 HLIL line: int32_t rax_3
+ Function: f00060 HLIL line: int32_t zmm0_1
+ Function: f00060 HLIL line: rax_3, zmm0_1 = sub_f00302("Hello World!\n")

* This is caused by zmm0 being clobbered by any of the preceding function calls
<---1 same lines--->
- Function: f00060 HLIL line: int32_t var_b8 = 42.4199982f.d
?                                                ^ ---------
+ Function: f00060 HLIL line: int32_t var_b8 = 0x4229ae14
?                                              ++  ^^^^^

* This is a continuation of the zmm0 clobber from above
<---3 same lines--->
- Function: f00060 HLIL line: int32_t var_108 = sub_f00302("Hello World!\n")
?                                               ^^^ --- --------------------
+ Function: f00060 HLIL line: int32_t var_108 = rax_3
?                                               ^^^

* These are the same effects caused by zmm0 being implicitly clobbered by unknown functions as above:
<---33 same lines--->
+ Function: f00060 Instruction: f000ff HLIL->LLIL instruction: None
+ Function: f00060 Instruction: f000ff HLIL->MLIL instruction: None
+ Function: f00060 Instruction: f000ff HLIL->MLILS instruction: []
+ Function: f00060 Instruction: f000ff HLIL->LLIL instruction: None
+ Function: f00060 Instruction: f000ff HLIL->MLIL instruction: None
+ Function: f00060 Instruction: f000ff HLIL->MLILS instruction: []
+ Function: f00060 Instruction: f000ff HLIL->LLIL instruction: rax#6, rcx#6, rdx#4, rsi#5, rdi#8, r8#3, r9#3, r10#3, r11#3, xmm0#3, xmm1#3, xmm2#3, xmm3#3, xmm4#3, xmm5#3, xmm6#3, xmm7#3, xmm8#3, xmm9#3, xmm10#3, xmm11#3, xmm12#3, xmm13#3, xmm14#3, xmm15#3, zmm0#5, mem#12 = call(0xf00302, stack = rsp#2 @ mem#11, params = rdi#7, rsi#4, rdx#3, rcx#5, r8#2, r9#2)
+ Function: f00060 Instruction: f000ff HLIL->MLIL instruction: rax_3#6, zmm0_1#2, mem#3 = 0xf00302("Hello World!\n") @ mem#2
+ Function: f00060 Instruction: f000ff HLIL->MLILS instruction: ['"Hello World!\\n"', '0xf00302', 'rax_3#6, zmm0_1#2, mem#3 = 0xf00302("Hello World!\\n") @ mem#2']
<---5 same lines--->
- Function: f00060 Instruction: f0012d HLIL->LLIL instruction: [rbp#1 - 0xb0 {var_b8}].d = zmm0#3.xmm0.d @ mem#15 -> mem#16
?                                                                                ^
+ Function: f00060 Instruction: f0012d HLIL->LLIL instruction: [rbp#1 - 0xb0 {var_b8}].d = zmm0#6.xmm0.d @ mem#15 -> mem#16
?                                                                                ^
- Function: f00060 Instruction: f0012d HLIL->MLIL instruction: var_b8#1 = zmm0#3:0.d
?                                                                               ----
+ Function: f00060 Instruction: f0012d HLIL->MLIL instruction: var_b8#1 = zmm0_1#3
?                                                                             ++
- Function: f00060 Instruction: f0012d HLIL->MLILS instruction: ['42.4199982f', 'var_b8#1 = zmm0#3:0.d', 'zmm0#3:0.d']
?                                                                   ^ -------                     ------------------
+ Function: f00060 Instruction: f0012d HLIL->MLILS instruction: ['0x4229ae14', 'var_b8#1 = zmm0_1#3']
?                                                                 ++  ^^^^^                    ++
<---23 same lines--->
- Function: f00060 Instruction: f00188 HLIL->MLILS instruction: ['"Hello World!\\n"', '0xf00302', 'rax_3#6, mem#3 = 0xf00302("Hello World!\\n") @ mem#2', 'var_108#1 = rax_3#6']
+ Function: f00060 Instruction: f00188 HLIL->MLILS instruction: ['rax_3#6', 'var_108#1 = rax_3#6']

The ZMM0 register appears in extraneous ϕ nodes in MMLIL_SSA:

======================================================================
FAIL: test_hlil_arrays (generated.default.unit.TestBinaryNinja)
----------------------------------------------------------------------
# Strangeness here caused apparently by having an extra ϕ node for zmm0, leading to rax escaping from the loop iteration expression, leading to a void function being decompiled as returning rax

<---36 same lines--->
+ Function: 100003730 HLIL line:     rax = a + 1
- Function: 100003730 HLIL line: for (uint64_t a = 0; a u< 3; a = a + 1)
?                                                                  ^^^^
+ Function: 100003730 HLIL line: for (uint64_t a = 0; a u< 3; a = rax)
?                                                                 + ^
+ Function: 100003730 HLIL line: return rax
+ Function: 100003730 HLIL line: uint64_t rax

* Looking at MMLIL_SSA, they diverge here:
  35 @ 100003749  xmm15#1 = ϕ(xmm15#0, xmm15#2)
  36 @ 100003749  zmm0#1 = ϕ(zmm0#0, zmm0#2)
  37 @ 100003749  mem#1 = ϕ(mem#0, mem#2)
  vs
  35 @ 100003749  xmm15#1 = ϕ(xmm15#0, xmm15#2)
  36 @ 100003749  mem#1 = ϕ(mem#0, mem#2)
* For some reason this causes:
   4 @ 100003829  return rax:0
  vs
   4 @ 100003829  return
* And the outer loop's iter clause:
100003749      uint64_t rax
100003749      for (uint64_t a = 0; a u< 3; a = rax)
10000381b      {
...
100003815          rax = a + 1
100003815      }
100003829      return rax
  vs
100003749      for (uint64_t a = 0; a u< 3; a = a + 1)

* The rest of these are because 100003815 maps to multiple MLILS, whereas it used to map to nothing:
<---11 same lines--->
+ Function: 100003730 Instruction: 100003815 HLIL->LLIL instruction: rax#4 = rax#3 + 1
+ Function: 100003730 Instruction: 100003815 HLIL->MLIL instruction: rax#4 = rax_12#3 + 1
+ Function: 100003730 Instruction: 100003815 HLIL->MLILS instruction: ['1', 'rax#4 = rax_12#3 + 1', 'rax_12#3', 'rax_12#3 + 1']
<---2 same lines--->
- Function: 100003730 Instruction: 10000381b HLIL->MLILS instruction: ['1', 'a#3 = rax#4', 'rax_12#3', 'rax_12#3 + 1']
?                                                                      -----                   --- ^^^^^^^^^^^^^^^^^
+ Function: 100003730 Instruction: 10000381b HLIL->MLILS instruction: ['a#3 = rax#4', 'rax#4']
?                                                                                ^

* And this is because the return is not void, even though it should be:
<---1 same lines--->
- Function: 100003730 Instruction: 100003829 HLIL->MLIL instruction: return
+ Function: 100003730 Instruction: 100003829 HLIL->MLIL instruction: return rax#1:0
?                                                                           +++++++
- Function: 100003730 Instruction: 100003829 HLIL->MLILS instruction: ['return ']
+ Function: 100003730 Instruction: 100003829 HLIL->MLILS instruction: ['rax#1:0', 'return rax#1:0']
?                                                                      +++++++++++        +++++++

This is available for testing here: https://github.com/Vector35/arch-x86/tree/test_full_width_fp_return_register

I believe this was fixed in 3.6.4705 in conjunction with https://github.com/Vector35/binaryninja-api/issues/3191


ENH: add SQL Expression tools

Codecov Report
Base: 91.11% // Head: 89.83% // Decreases project coverage by -1.28% :warning:

Coverage data is based on head (ea0d3b6) compared to base (77eefd8).
Patch coverage: 83.48% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
- Coverage   91.11%   89.83%   -1.29%
==========================================
  Files          20       22       +2
  Lines        1081     1299     +218
==========================================
+ Hits          985     1167     +182
- Misses         96      132      +36

Impacted Files
Coverage Δ

redbird/sql/expressions.py
83.41% <83.41%> (ø)

redbird/sql/__init__.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add workflow start and complete time

Please add start and complete time to workflow CRD.

Commit a2f034a adds startedAt, finishedAt, and message to the NodeStatus object. Also be aware that the status field has been renamed to phase.


Uncaught (in promise) Repo not found for: E:\projects\my-project

When selecting my-project -> right click -> Github Issues  nothing happens.
I opened up the console and i see error:
Uncaught (in promise) Repo not found for: E:\projects\my-project

In my package.json i set up the repository field.
"repository" :
  { "type" : "git"
  , "url" : "https://github.com/.. my-project"
  },

I restarted atom. Still doesn't work.
Is there something more i should be doing ? Some other configuration? Thanks 😄

If you run git remote show origin inside the repo, what is the output?

@r24y i think is a good idea if you close this issue.

I don't have it anymore - and i don't know how to replicate it either. Thanks :)


scannet_200_insseg

Hi, thanks for your great work! when running unscene3d_dino_2d3d.sh, it reports "No such file or directory: 'data/ScanNet/scannet_200_insseg/val.txt'". How can I get scannet_200_insseg?

I am facing the same issue.

Hey guys,
Really fair point, I forgot this bit from the README, but generally we followed the logic of all scannet instance segmentation preprocessing methods. You can find the original script here, but if you pull this repo again, I updated the README and the necessary preprocessing scripts in this repo as well.
Please let me know if you still miss something and thanks for flagging it.
Cheers,
David

嗨，感谢您的出色工作！运行 unscene3d_dino_2d3d.sh 时，系统报告“没有这样的文件或目录：'data/ScanNet/scannet_200_insseg/val.txt'”。我如何获取 scannet_200_insseg？

请问您解决了吗

我遇到了同样的问题。

请问您解决了嘛

Please check my previous answer and preprocess the instance segmentation dataset according to this guideline.
Closing the issue as solved with updated scripts.

我遇到了同样的问题。
你好请问解决了吗

Dear @zyjhxd,
As I mentioned please refer to this link for preprocessing. Yes, it has been solved.
Also, please use english here as many users, including myself, can't read and speak Chinese and using a translator might lead to misunderstandings.
Cheers,
David

Okay, thank you for your prompt response. But whether I use the previous script or the updated script to run the second instruction The same error occurs in scripts/unscene3d-dino-2d3d.sh as shown in Figure 1 (I have produced the processed file as shown in Figure 2 using the Python scannet200_insseg.py command). Is there any solution?

I think the only thing is missing your train/val/test splits text file. Can you check in the parent folder of your screenshot?

But at the end of my file, there are three files that separate the datasets

hmm, I can't seem to see this image (also previous comment figure 2 is still in updating stage)

The parent folder is shown in the picture, with only one folder.

The image of the previous comment is as follows

right this looks correct. so if you run the original script, the file (val.txt) should be found as the paths are correct (assuming the here shown data folder lives in the root repo path which is not shown on this screenshot).

But I run the script When generating pseudo labels with scripts/unscene3d.dino-2d3d.sh, there is an error message stating that the file train.exe cannot be found. However, this file does exist in this path, as shown in the figure. What is the reason for this?


Tag

Hi,
The last commit has been over a year ago. Can we expect a tag? Thank you!

Hi, @maniankara @karolmajek is it possible to have a tag against the latest commit https://github.com/moosefs/moosefs-csi/commit/62bb6055987c164f1a354c09b6eab59b6c8ea302 ? Thank you!

Hello @antoinetran,
We are working on the update for the MooseFS Containter Storage Interface driver (hence no tag yet), but the changes are not yet ready to be published. We will publish them once ready.
Piotr

@antoinetran Please check out the latest release we have pushed to the repository. Thanks


Network Access Required When Validating id-token

why is network access required to do firebase_auth.verify_id_token(token)?
Isn't it possible to do this without accessing Google APIs?

Token is verified against a public key certificate that must be fetched from a remote API. It's done only once, and then cached for up to 24 hours.


Cannot read property 'onTokenRefresh' of undefined

I have used the plugin before, now that I am trying to use it in a new and fresh project i get this error. I am probably missing something very basic I just can't recall.
Tried adding:
<plugin name="cordova-plugin-firebase" spec="0.1.22" />
to the config.xml file. before adding a platform and building/serving
also, tried installing using the cordova cli, it always seems to install properly but when i go on the console for chrome "window" doesnt show "FirebasePlugin" which tells me its definetly missing.
O yea, I also added the google-services.json file from firebase in the root folder.
Thanks!

it wont be available until after deviceready has been fired. see http://cordova.apache.org/docs/en/7.x/cordova/events/events.html#page-toc-source
if you have other js errors or you try to run an ios/android build in the browser (instead of a browser build) it may never fire


[requires.io] dependency update on master branch

Current coverage is 60.11% (diff: 100%)

Merging #25 into master will increase coverage by 0.10%

@@             master        #25   diff @@
==========================================
  Files            61         61
  Lines          1973       1973
  Methods           0          0
  Messages          0          0
  Branches        158        158
==========================================
+ Hits           1184       1186     +2
+ Misses          750        749     -1
+ Partials         39         38     -1

Powered by Codecov. Last update 26ad702...f0df48d


Warning: componentWillReceivedProps has been renamed and is not recommended for use

Issue
componentWillReceivedProps has been renamed and is not recommended for use
Expected Behavior
It should work without any warnings ?
Code
<DatePicker
              style={{width: 200}}
              date={this.state.date}
              mode="date"
              placeholder="Select Date"
              format="YYYY-MM-DD"
              confirmBtnText="Confirm"
              cancelBtnText="Cancel"
              customStyles={{
                dateIcon: {
                  position: 'absolute',
                  left: 0,
                  top: 4,
                  marginLeft: 0
                },
                dateInput: {
                  marginLeft: 36
                }
                // ... You can check the source to find the other keys.
              }}
              onDateChange={(date) => {this.setState({date: date})}}
            />

Also I got another warning that it has been merged to DatePickerIOS and will be removed in the future. Does that mean we cannot use it for Android anymore ?

datepicker.zip
Try the attached file
I created pull request to merge with master

@asibrahim1985 Thanks. But another warning still presents.
"Warning: DatePickerIOS has been merged with DatePickerAndroid and will be removed in a future release. It can now be installed and imported from '@react-native-community/datetimepicker' instead of 'react-native'. See https://github.com/react-native-community/react-native-datetimepicker"

why this plugin is not updating?

Any help with this warning?


[CMR] M3-400: Fixes for Table Styling

Description
This fixes the hover state for the up arrow and borders for dark mode.
Type of Change

Non breaking change ('update', 'change')

Note to Reviewers
The hover state for the rows is a bit off in dark mode since there isn't a provided color for that yet.

Looks good for the intended scope of this PR, but I'm observing a mismatch in the widths of the entity header and the table:

@DevDW good catch!


psdnOCEAN Interview

[ ] Send questions to fla1white @ NewOrder Network

[ ] Film video interview

[ ] Edit footage


Order checkout complete fail il phone number is incorrect

Hello,
If I have a malformed phone number for the delivery or billing address, and I don't have validation via the Symfony form, I get an error at the complete checkout step, when i payment is captured.
request.CRITICAL: Uncaught PHP Exception libphonenumber\NumberParseException: "The string supplied did not seem to be a phone number." at PhoneNumberUtil.php line 1673 {"exception":"[object] (libphonenumber\NumberParseException(code: 1): The string supplied did not seem to be a phone number. at /apps/sylius/vendor/giggsey/libphonenumber-for-php/src/PhoneNumberUtil.php:1673)"}
In my example, phone number was a simple hyphen.
Solution must be to catch NumberParseException in PayPlug\SyliusPayPlugPlugin\Creator\PayPlugPaymentDataCreator, formatNumber function :
public function formatNumber(string $phoneNumber, ?string $isoCode): array
{
    try {
        $phoneNumberUtil = PhoneNumberUtil::getInstance();
        $parsed = $phoneNumberUtil->parse($phoneNumber, $isoCode);

        if (!$phoneNumberUtil->isValidNumber($parsed)) {
            return [
                'phone' => null,
                'is_mobile' => null,
            ];
        }

        $formatted = $phoneNumberUtil->format($parsed, PhoneNumberFormat::E164);

        return [
            'phone' => $formatted,
            'is_mobile' => PhoneNumberType::MOBILE === $phoneNumberUtil->getNumberType($parsed),
        ];
    }
    catch (NumberParseException $e) {
        return [
            'phone' => null,
            'is_mobile' => null,
        ];
    }
}

Hello,
Thank you for bringing this issue to our attention.
We will review this issue and investigate the proposed solution to ensure it resolves the problem effectively without introducing any new issues. Our team will need some time to thoroughly test and verify the fix.
We will update you as soon as we have more information or a resolution.

Hello, this issue has been solved in the version 1.9.3.
Thanks for taking the time to report it.


Returning default value from database after inserting new object

When I insert a new object un DB I would like to get all default values that I didn't specified to be stored in my C# class.

This does not only apply to inserts, but also to updates under specific circumstances.


when i change laravel controller file, but it do not change in container of php-fpm,workspace,nginx

Description:

when i change laravel controller file, but it do not change in container of php-fpm,workspace,nginx.
Expected Behavior:

Context information:
Output of docker version
docker 3.0.1

System info: Mac, Windows or Linux. Include which disto/version
mac 11.1

Steps to reproduce the issue:

docker build php-fpm nginx workspace mysql mongo
docker up -d php-fpm nginx workspace mysql mongo
visit url
everything is fine now
when i edit php file. the changed file not change in container

anyone encounter this problem??

same problem here.
After rebuild php-fpm container, I check phpinfo message to make sure opcache is not enabled, and code change on host is fine.But after a while , any change made in host doesnt work in container,I mean every change, restart php-fpm container change nothing. Seems like a opcache issue but I check the php-container opcache.ini, its disabled

I thought I was going crazy; seems to work for a while then I have to kill all the docker processes and restart docker for my changes to appear. Its become more fighting with docker than actual development time

I thought I was going crazy; seems to work for a while then I have to kill all the docker processes and restart docker for my changes to appear. Its become more fighting with docker than actual development time

me too. Thank you for your reply, now I just need restart Docker instead of rebuild container every this problem occurs.

@glitterlip - I'm going to try out the docker sync setup that comes with laradock and see if that makes any difference

Related docker issue: https://github.com/docker/for-mac/issues/5046

Related docker issue: docker/for-mac#5046

should be this problem.

i back docker version。

same problem here.
After rebuild php-fpm container, I check phpinfo message to make sure opcache is not enabled, and code change on host is fine.But after a while , any change made in host doesnt work in container,I mean every change, restart php-fpm container change nothing. Seems like a opcache issue but I check the php-container opcache.ini, its disabled

it is not about opcache。i had checked。file not change。i back docker version <IP_ADDRESS>。


Raw user delete

Goal
Delete user data in Snowflake being sourced from other systems.
Implementation Decisions
This is an extension of #75 and should be deployed after #75 is merged.
Not Perfect
There are a lot better ways that we can do this. I have concerns about this solution in regards to developer workflows, maintenance load, and duplicate code. However, we have to start with an accounting of all the things that we need to do and what our ultimate solution should do. By doing this first we get that full list of tasks and we can check off the project as meeting our requirements very early in the process. This should give us some space to figure out the right long-term solution.
Deployment steps

[ ] Database migrations?
[ ] Secrets?
[ ] FOR POCKET DEVELOPERS ONLY - Post in #changelog:

Write a #changelog message using our best practices if this PR (potentially) impacts users. Describe the 'why' and 'what'. @mention relevant teams. Link to this PR.

References
Documentation:

Project doc

@kennylong @kirill-demtchouk sorry. I forgot to push the newest changes. I just saw that in my git history. It looks like I hopped into our meeting without hitting enter on the git push. 🙈


Images with different heights break a vertical carousel

====================================================================
http://jsfiddle.net/fmo50w7n/322/
====================================================================
Steps to reproduce the problem

Have a Vertical carousel with images.
All images should have different height.(Big difference = great discrepancy)
additionally: enable centermode.

====================================================================
What is the expected behaviour?
When the slider moves, the center slide should stay in the center.
====================================================================
What is observed behaviour?
As long as the height difference of those images is small the only problem is that the current slick slide is a little of center. As soon as the difference get to big the center Slide is no longer visible and the track element starts to bug out and goes of screen completely.

Hey @Fansana, firstly, thanks for making the issue. What you're saying definitely makes sense. Unfortunately support for vertical mode is pretty limited at this stage.
Right now, I think the height of the track is calculated by taking the height of the first slide and multiplying slidesToShow and the track moves by the height of the current slide, which is set to auto and adjusted to slide content--the upshot is there's no support currently for dynamically centering of variable height slides.
Setting a fixed height for the slide might help your situation though. Hope that helps!
Related to #1803 and #1854


switching tabs is not smooth

cause of OnTabSelectedListener and below code, switching tabs is not smooth
       viewPager.setCurrentItem(tab.getPosition());
       InterfaceManager.shared().a.setShowLocalNotifications(showLocalNotificationsForTab(tab));
       for (int i = 0; i < tabs.size(); i++) {
                ((BaseFragment) tabs.get(i).fragment).setTabVisibility(i == tab.getPosition());
       }

I can't replicate this as of 4.7.1


turtl-server: init at 2018-11-05

Motivation for this change
Run your own turtlapp server. See github.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Fits CONTRIBUTING.md.

@raquelgb any updates on this?

If you are still interested in working on this, please open a new PR.


Add OpenTF migration to the roadmap

We should plan the migration from Terraform to OpenTF, now that HashiCorp has changed its licences and OpenTF has been created.

As OpenTofu has not been widely adopted by the community, the decision is made to keep Terraform.


Checkov Integration Setup Documentation Is Out of Date

Since the implementation of the redirect from bridgecrew.cloud to prismacloud, the documentation for setting up an API token is incorrect. Please update to include the new direction on acquiring an API token.
Documentation: https://marketplace.visualstudio.com/items?itemName=Bridgecrew.checkov
Broken Integration link: https://www.bridgecrew.cloud/integrations/api-token
It's not immediately clear whether or not checkov is now paywalled (beyond the 7 day free trial), and it's excruciating to have to wait for a "Palo Alto Networks specialist to reach out to me" before I can get an account and API token setup. I hope this isn't the case.
Thanks,
Alex

Hi @AlexAtkinson thanks for pointing this out! We'll update this listing.
Bridgecrew has been merged into Prisma Cloud by Palo Alto Networks and so to use this extension, you will need a valid Prisma Cloud Access Key. Checkov by itself is still open source to use.

We appreciate the feedback, @timoa. Certainly this is not the experience we want you to have. Rest assured that Checkov as an open source tool is and will remain open source and well supported.
In regards to Bridgecrew, the Bridgecrew site has had a deprecation banner up for a few months announcing the deprecation of bridgecrew.cloud. This marketplace documentation was missed, but we'll get it updated as well.

@tsmithv11 Shouldn't you have change this extension's entry on VS Code extensions page switched from 'FREE' to Trial /Paid, or maybe have the extension removed altogether from there until this change is complete?
The current status is misleading.

This is still a problem

Given the current circumstances it might be a proper place to ask for alternative recommendations.

The new Prisma Cloud plugin doesn't require an API Key and uses Checkov: https://marketplace.visualstudio.com/items?itemName=PrismaCloud.prisma-cloud
The Checkov extension will be moved to deprecated in the future.

I've found that the Prisma extension doesn't work
And shouldn't you have deprecated the Checkov extension at least 6 months
ago?
Seriously people, get your ""stuff" " together!
On Tue, 1 Oct 2024, 17:36 Taylor, @.***> wrote:

The new Prisma Cloud plugin doesn't require an API Key and uses Checkov:
https://marketplace.visualstudio.com/items?itemName=PrismaCloud.prisma-cloud
The Checkov extension will be moved to deprecated in the future.
—
Reply to this email directly, view it on GitHub
https://github.com/bridgecrewio/checkov-vscode/issues/141#issuecomment-2386478169,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARAYJJP4I6SKJC4O36TK6WDZZLFPLAVCNFSM6AAAAABPDCWEFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGQ3TQMJWHE
.
You are receiving this because you commented.Message ID:
@.***>