No image display on web interface.

Can't seem to figure out why I don't get the image displayed on the web interface. It is just blank. Everything else seems to work fine.   That all look like this....
app.js?id=1d007da1358979320604:2 Uncaught DOMException: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The HTMLImageElement provided is in the 'broken' state.
at h.Image._draw (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2347029)
at h.renderWithContext (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2337108)
at h.Image.h.Entity.draw (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2341057)
at h.addToStage (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2336043)
at h._callback (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2099528)
at h._animate (http://<IP_ADDRESS>:8080/js/app.js?id=1d007da1358979320604:2:2338182)
Thanks!

This is either a file permissions issue or an issue with the symlink. It looks like it generated predictions, so it was able to access the image file. I have seen this issue before so I think I can help you fix it.
A few questions:

What OS is this running on?
What is your docker version?
Please share your .env file

Thank you!  Windows 10 , docker 20.10.2, and my .env file looks like:
WATCH_FOLDER=D:\BlueIris\aiimput
WEB_INTERFACE_URL=http://<IP_ADDRESS>
WEB_INTERFACE_PORT=8080
How many days to keep in database (0 for unlimited)
DATA_RETENTION_DAYS=7
Set to true if the watcher is not picking up new image files quickly.
Unusual watch folders like network shares may require polling.
WATCH_USE_POLLING=false
Sets the PID and GID of the web server user so that it can access
the source files.
Change these to match your user ID. Run 'echo $UID' to get your ID.
On Windows this setting does not matter.
USER_ID=1000
GROUP_ID=1000
Turn on debugging to troubleshoot errors
APP_DEBUG=true
Change the Deepstack base url if you are hosting deepstack outside of docker.
DEEPSTACK_BASE_URL=http://deepstack:5000/

Try deleting the symlink and recreating it. The symlink is how the system maps images from your watch folder into the web app.
cd /path/to/last-watch-ai

rm src/public/storage

docker-compose run --rm artisan storage:link

You should get some output like "The links have been created". After that, try refreshing the detection event page again.

That fixed it. Thanks for your help.

Great, enjoy!
By the way, you might have to repeat this process if you ever change your watch folder.


use flaui and flauinspect in windows xp 32bit

hello
i want to automate old program in windows xp 32bit .
can i use FlaUI and flauinspect in windows xp 32bit?
thanks

This looks like a question, which would fit nice into https://github.com/FlaUI/FlaUI/discussions :)
Anyway FlaUIInspect 1.3 requires .NET Framework 4.7.1 to run, and I doubt it is possible to install it into Windows XP.
Note that installing .NET Framework 4.7.1 in a clean Windows 7 SP1 requires two updates: KB4474419 and KB4490628 (AFAIK they are not available for Windows XP)

Thank you for your answer
What is your suggestion so that I can do this in Windows XP?

@PersianCoders , I have no experience with using UIAutomation on Windows XP.
According to the official site it isn't available there out-of-the-box - some update should be installed first.
You can use older versions of FlaUI, which don't depend on newer versions of .NET Framework.
I guess, all this will require a lot of experiments...
If you manage to accomplish your task, please, post the solution here or in discussions.


Restore ktlint task dependencies

This PR restores the wiring of ktlint task dependencies to our codeQuality tasks and fix formating issues that went unnoticed.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test and merge

Your PR is queued because #16718 is being built.

OK, I've already triggered a build for you.


RDS cluster/instance deletion behavior (ability to verify when delete has completed)

When RDS cluster/instance CRs are deleted, those CR disappear immediately. Thus we don’t know if the deletion is completed.
Users should be able to see the CR when they are being deleted or deletion failed. If the CR is gone, that means the RDS resources are deleted.

totally agree with the proposed behaviour. Also it should be the same not only for DBs, but for every resource managed by ACK framework.
@a-hilaly please not this, that it is more general thing.


Dynamics form improvements

The current form layout presents some usability/flow issues, and the page overall needs some content edits as well.
https://prime.powerappsportals.us/siw/
Form

[ ] Look into better groupings for inputs (ex: primary and secondary contact are kind of jammed together and read L->R a bit strangely)
[ ] Confirm all fields are present
[ ] Confirm all options are present
[ ] Test submit the form from multiple devices
[ ] Test form validation and review error content

Moving to icebox. Not moving forward with Dynamics but these issues may apply to whatever we work on next.


Commands executed in a jail.
 There is no such problem when running go outside of a jail.

 We did not have any problem when our software was in C, and have been using FreeBSD jails for some time now.

Does every Go program fail in this was, or just the go tool itself?  For example, you should be able to use the go tool outside of a jail to build a simple hello, world program.  Does that program fail when run in a jail?
What kind of processor are you running on?
I actually don't see how that ktrace output is possible.  The Go runtime only ever passes 0xf and 0x10 to _umtx_op.  I don't know where the 0x11 is coming from.

Is the userland in your jail in sync with your kernel?

The architecture is amd64; the CPU is an Intel Core i5 750.
Both the userland and the base system and the jail are 10.2-RELEASE-p13.
I tried with a simple hello world program.

Building it out of the jail works. Running the generated executable in the jail works.
Building it in the jail results in the go runtime getting stuck the same way.

Finally I tried a hello world program using the C package:

package main

// #include <stdio.h>
//
// static void helloWorld() {
//     printf("hello world\n");
// }
import "C"

func main() {
        C.helloWorld()
}

Building it out of the jail, then running it in the jail results in it getting stuck.
Our software also uses C, and also gets stuck in the same situation.
Does the go runtime use C ? If yes, there could be something wrong with C when running in a jail.

The Go runtime itself does not use C.  However, many Go programs do use C.  Go makes system calls directly, including to _umtx_op.  _umtx_op is related to threads.  If the jail does not support threads, then no substantial Go program is going to work inside it.

There is afaik nothing in jails that prevent using threads in any way. I already ran multithreaded applications in jails without any problem.

Thanks.  I'm going to close this issue because I don't think there is anything we can do on the Go side to fix it.  Please reopen if there is something we can change in Go to make it work.


Unplugging the camera crashes the application, cannot catch exception

Required Info

Camera Model
R200

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu 16.04.1 LTS

Kernel Version (Linux Only)
4.4.0-53-generic

Build System
Makefile

Hello,
I'm planning on using the R200 camera in a system where the camera can be disconnected from the system while the application is running. When I unplug the camera from the usb, an exception is thrown in one of the capture threads:
terminate called after throwing an instance of 'std::runtime_error'
  what():  VIDIOC_DQBUF error 19, No such device
Aborted (core dumped)

This exception terminates my application, since it is unhandled in a separate thread. I would like to catch that exception and recreate the context to enumerate the connected cameras again.
Is such functionality on the roadmap for librealsense?

Yes, this should be addressed eventually. Exceptions from back-end thread are usually logged and should not escape the library, so it is a bug. However, responding to camera disconnect events is relatively low in priority in project road-map, so if you'd like to look into it and propose a fix it would be appreciated.

Hi lvretenar there are errors in uvc-v4l2.cpp source,in the poll method we should handle the error ,this error when VIDIOC_DQBUF returns EAGAIN error with no buffer available ,we should check for device connectivity and after freeing buffers if device not found, should call for stop() and destructor for subdevice and re-enumerate for connected devices.

For best support, post issues in the RealSense Community.
Hello,
Do you still need help with this issue?
Best regards,
Jesus
Intel Customer Support

Hi,
Yes, we would love this bug to be fixed! It is a real problem for us in our system.

@maciekd,
Development on the Master branch of librealsense has stopped and the R200 camera has been discontinued for some time now. All focus is now on librealsense 2.0 (a.k.a. SDK 2.0, development branch of librealsense) which will support the D400 series of cameras and the SR300.
Have you tried using the method recommended by souch55 above?
Regards,
Jesus
Intel Customer Support


Error during download: string indices must be integers

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
I encountered an error while attempting to extract audio from YouTube URLs within the web application. The application is deployed on a Google Cloud Compute Engine, utilizing uWSGI and Nginx as the web server. Developed with the Django framework, the web application allows users to obtain URLs from YouTube videos and convert them into downloadable MP3 audio files. Despite testing the application with various YouTube URLs, the same error consistently occurs for all of them. I am seeking assistance in resolving this issue to ensure smooth functionality of the application.
I am using,

yt-dlp version: 2023.07.06
Python version: 3.8
Django version: 4.2.3

debug.log:
[ERROR] [284082:uWSGIWorker10Core0] Error during download: string indices must be integers, URL: https://rr5---sn-qxoednee.googlevideo.com/videoplayback?expire=1691149584&ei=sJDMZP3yFPqcsfIPwui3yAM&ip=<IP_ADDRESS>&id=o-APBGVHuq0nMdjvWlUkAUvc9DohuNR9o-58SQMEuJsjMs&itag=22&source=youtube&requiressl=yes&mh=Vm&mm=31%2C26&mn=sn-qxoednee%2Csn-5goeenes&ms=au%2Conr&mv=m&mvi=5&pl=20&spc=UWF9f8agngY7rx9LkQr5IjngI5pIPoc&vprv=1&svpuc=1&mime=video%2Fmp4&cnr=14&ratebypass=yes&dur=136.115&lmt=1674102144627583&mt=1691127679&fvip=2&fexp=24007246%2C24362687&beids=24350018&c=ANDROID&txp=4532434&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Ccnr%2Cratebypass%2Cdur%2Clmt&sig=AOq0QJ8wRgIhAPPJty6BsMW4-TSw--3-4N4yPUphk2c_VBO2gMvZNSn_AiEAskMpkfSG6pSVLz9HhaRi_tjgqtRL81EnZHKKAgr15Ws%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl&lsig=AG3C_xAwRQIgKqM8MigMn8VqLs-G5iIROxU_9mMp42qAalsLgcKnbjwCIQDsZRVjc8YCBX2Um_hTMMAsljTF18D9rzSZiaObCE-aGA%3D%3D, Format: mp3
Could you kindly review the error information I've provided and offer any insights or solutions that could assist in resolving this issue? Your expertise would be greatly appreciated in ensuring the smooth functionality of the application.
Provide verbose output that clearly demonstrates the problem

[x] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out UTF-8 (No ANSI), error UTF-8 (No ANSI), screen UTF-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip) API
[debug] params: {'format': 'bestaudio/best', 'postprocessors': [{'key': 'FFmpegExtractAudio', 'preferredcodec': 'mp3', 'preferredquality': '192'}], 'progress_hooks': [], 'outtmpl': '/var/www/html/django-app/youtube_downloader/downloaded_files/%(title)s.%(ext)s', 'verbose': True, 'compat_opts': set()}
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.15.0-1037-gcp-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[youtube] Extracting URL: https://www.youtube.com/watch?v=XqZsoesa55w
[youtube] XqZsoesa55w: Downloading webpage
[youtube] XqZsoesa55w: Downloading ios player API JSON
[youtube] XqZsoesa55w: Downloading android player API JSON
[youtube] XqZsoesa55w: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id

Seems like a bug in your web app's code. Can you share a code example?

Seems like a bug in your web app's code. Can you share a code example?

https://github.com/TechExplorer27/youtube-mp3/blob/main/views.py

https://github.com/TechExplorer27/youtube-mp3/blob/main/views.py#L58
            ydl.params['outtmpl'] = f'media/{filename}'

That line is the problem. Do not mutate params after you have instantiated YoutubeDL. It is not supported by the API

@TechExplorer27 I can't reproduce the error with the updated code. Remove your error handling and post verbose log with traceback

@bashonly
I updated the code by removing error handling process.
But it doesn't show traceback in my log file.
This is the complete verbose output:
[youtube] Extracting URL: https://www.youtube.com/watch?v=XqZsoesa55w
[youtube] XqZsoesa55w: Downloading webpage
[youtube] XqZsoesa55w: Downloading ios player API JSON
[youtube] XqZsoesa55w: Downloading android player API JSON
[youtube] XqZsoesa55w: Downloading m3u8 information

it looks like your log is only showing output from stdout. the traceback and other debug information is sent to stderr.

Here's the full verbose output,
[debug] Encodings: locale UTF-8, fs ascii, pref UTF-8, out utf-8 (No ANSI), error utf-8 (No ANSI), screen utf-8 (No ANSI)
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip) API
[debug] params: {'format': 'bestaudio/best', 'postprocessors': [{'key': 'FFmpegExtractAudio', 'preferredcodec': 'mp3', 'preferredquality': '192'}], 'progress_hooks': [], 'outtmpl': 'downloaded_files/%(title)s_1691212807.mp3', 'verbose': True, 'compat_opts': set()}
[debug] Python 3.8.10 (CPython x86_64 64bit) - Linux-5.15.0-1037-gcp-x86_64-with-glibc2.29 (OpenSSL 1.1.1f  31 Mar 2020, glibc 2.31)
[debug] exe versions: ffmpeg 4.2.7, ffprobe 4.2.7, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.18.0, brotli-1.0.9, certifi-2023.07.22, mutagen-1.46.0, pyxattr-0.6.1, secretstorage-2.3.1, sqlite3-2.6.0, websockets-11.0.3
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[youtube] Extracting URL: https://www.youtube.com/watch?v=uoH321sVBeE
[youtube] uoH321sVBeE: Downloading webpage
[youtube] uoH321sVBeE: Downloading ios player API JSON
[youtube] uoH321sVBeE: Downloading android player API JSON
[youtube] uoH321sVBeE: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[youtube] Extracting URL: https://www.youtube.com/watch?v=uoH321sVBeE
[youtube] uoH321sVBeE: Downloading webpage
[youtube] uoH321sVBeE: Downloading ios player API JSON
[youtube] uoH321sVBeE: Downloading android player API JSON
[youtube] uoH321sVBeE: Downloading m3u8 information
[debug] Sort order given by extractor: quality, res, fps, hdr:12, source, vcodec:vp9.2, channels, acodec, lang, proto
[debug] Formats sorted by: hasvid, ie_pref, quality, res, fps, hdr:12(7), source, vcodec:vp9.2(10), channels, acodec, lang, proto, size, br, asr, vext, aext, hasaud, id
[info] uoH321sVBeE: Downloading 1 format(s): 251
ERROR: unable to create directory [Errno 13] Permission denied: 'downloaded_files'
Traceback (most recent call last):
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/utils/_utils.py", line 5149, in make_dir
os.makedirs(dn, exist_ok=True)
File "/usr/lib/python3.8/os.py", line 223, in makedirs
mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: 'downloaded_files'
Traceback (most recent call last):
File "/var/www/html/django-app/lib/python3.8/site-packages/django/core/handlers/exception.py", line 55, in inner
response = get_response(request)
File "/var/www/html/django-app/lib/python3.8/site-packages/django/core/handlers/base.py", line 197, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/var/www/html/django-app/youtube_downloader/youtube_downloader/views.py", line 60, in index
filename = download_file(url, 'mp3')  # Always convert to MP3
File "/var/www/html/django-app/youtube_downloader/youtube_downloader/views.py", line 36, in download_file
ydl.download([url])
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 3485, in download
self.__download_wrapper(self.extract_info)(
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 3460, in wrapper
res = func(*args, **kwargs)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1549, in extract_info
return self.__extract_info(url, self.get_info_extractor(key), download, extra_info, process)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1560, in wrapper
return func(self, *args, **kwargs)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1709, in __extract_info
return self.process_ie_result(ie_result, download, extra_info)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1768, in process_ie_result
ie_result = self.process_video_result(ie_result, download=download)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 2897, in process_video_result
self.process_info(new_info)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 3142, in process_info
if not self._ensure_dir_exists(encodeFilename(full_filename)):
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1866, in _ensure_dir_exists
return make_dir(path, self.report_error)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/utils/_utils.py", line 5153, in make_dir
to_screen(f'unable to create directory {err}')
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 1042, in report_error
self.trouble(f'{self._format_err("ERROR:", self.Styles.ERROR)} {message}', *args, **kwargs)
File "/var/www/html/django-app/lib/python3.8/site-packages/yt_dlp/YoutubeDL.py", line 981, in trouble
raise DownloadError(message, exc_info)
yt_dlp.utils.DownloadError: ERROR: unable to create directory [Errno 13] Permission denied: 'downloaded_files'
I tried changing the permissions of the folder but it din not fix.

You need to set the --output-template value so that when expanded it is a filename that your web application can create. Presumably the web application process (its user, not you or root) doesn't have permission to create a directory in its default directory.

Thanks! The issue was regarding the permission of the web application.


Broken color cycling for lava

If you try to convert caves tiles back, the color cycling for the lava becomes broken. Even with an 8-bit PNG, the tool will re-search and find a new index for each color in the image.
The game expects colors to be cycled to be in indices 1-31, and each color mapped to a specific index to make the cycling effect work. My suggestion is to add a direct conversion of 8-bit PNG images, where the tool will simply keep the same 1-1 palette indices of the palette in the image rather than remapping to one in the command line.

Here is an explanation of why this happens: the cycling effect fades in then out using the same color. This means duplicate palette entries exist. The tool will simply select the first matching index in the palette.

Here is an explanation of why this happens: the cycling effect fades in then out using the same color. This means duplicate palette entries exist. The tool will simply select the first matching index in the palette.

Ah, that makes sense. There are so many specific details to keep in mind to have the palettes fully working in the game :P

There are so many specific details to keep in mind to have the palettes fully working in the game :P

Yeah, to be fair though, they could have probably made it work without the fade out part if they coded tables instead of shifting everything by one (:

I imagine there should be a function for it, you don't even need the palette itself just have to decompress the raw image data and then RLE it.


fix: Return an error on publish if the publisher's state is not RUNNING

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes internal issue b/156525288.

Not sure if this is the best solution.
I didn't call onPermanentError() because the publisher may already be in a failed or terminated state.

Not sure if this is the best solution.
I didn't call onPermanentError() because the publisher may already be in a failed or terminated state.

This should be okay. ProxyService will handle this gracefully, please add onPermanentError.

I didn't call onPermanentError() because the publisher may already be in a failed or terminated state.

This should be okay. ProxyService will handle this gracefully, please add onPermanentError.

Done. PTAL at revised implementation in PublisherImpl.


Make pricelist an object instead of an array

Is your feature request related to a problem? Please describe.
Price updates are painfully slow with any decent sized pricelist and the bot is unresponsive while updating an entry.
Describe the feature you'd like
Change the pricelist to an object from an array to improve performance.
Describe alternatives you've considered
Alternatively, the pricelist.json file could be kept as an array, so users don't have to remake their pricelist and only convert the pricelist in memory to an array. This way saving to the file would have to be changed, which would make it a bit overcomplicated.
Additional context

I'd suggest mongodb instead. Everyone could get a free mongodb atlas account.


/dev/tty: Device not configured in pre-commit hook

I am using construction like this in my pre-commit hook:
exec </dev/tty

	read -n 1 -p "Are you sure, you want to continue? [Ny] " yesno

	case $yesno in
	[yY])
		echo "OK"
		;;

	*)
		echo "Doing nothing."
		exit 1
		;;
	esac

	exec <&-

If I run the pre-commit hook directly from my terminal, it works. If I run it with elixir_git_hooks configured like this:
config :git_hooks,
  verbose: true,
  auto_install: true,
  hooks: [
    pre_commit: [
      tasks: [
        {:file, ".githooks/pre-commit"}
      ]
    ]
  ]

It raises an exception saying that .githooks/pre-commit: line 22: /dev/tty: Device not configured
On line 22, is this: exec </dev/tty
Any idea how I could fix this error?
Thanks a lot. :-)

There is no input when running the hooks, so tty is not available. I guess something like this could be achieved using an Elixir module.
It's been a while since this issue, so if it's still relevant we can reopen this conversation and I can try to help you out with it.


Update dependency-injection-guidelines.md

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

If these additions are indeed accurate, the sentence has become too long and is probably better if changed to bullet points or split up and clarified.
Tagging @IEvangelist

If these additions are indeed accurate, the sentence has become too long and is probably better if changed to bullet points or split up and clarified.
Tagging @IEvangelist

Hi again @RyanLiu99 - I'm going to ask that @davidfowl takes a look at this too, as I'm not sure it is accurate.
Also, this is related to #22393

Hi again @RyanLiu99 - I'm going to ask that @davidfowl takes a look at this too, as I'm not sure it is accurate.
Also, this is related to #22393


Update LAB_11_SecuringAzureSQLDatabase.MD

Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Fixed a user interface change in the SQL lab.


update jest-junit dependency

Purpose

update jest-junit dependency

...

Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ x ] Other... Please describe:

dependency update
How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

build it to verify packages
What to Check
Verify that the following are valid

...

Other Information

related to this PR https://github.com/Azure/communication-ui-library/pull/1463

Waiting for https://github.com/Azure/communication-ui-library/pull/1731


Auto Testing multiple charts in specified folder

Implemented script from here: https://github.com/datreeio/helm-datree#testing-multiple-charts
for auto search all helm project under an specified folder, to not need to add for every single helm project an own config runtime.
Also added the line:
 echo "**Source path:** ${1}" >>"$GITHUB_STEP_SUMMARY"
to know which project was checked when multi helm project folders will be checked.
All other processes should be work as before, only helm is changed.

Hi @MVladislav thanks for this PR, looks great!
I left some comments, please have a look:)


how to add plug-ins?

I copy sample from your demo site. But it not work (not show drop-down menu on image button).
http://krestovskyisland.ru/index.html
what am I doing wrong?
<script src="js/trumb/plugins/upload/trumbowyg.upload.min.js"></script>
<script src="js/trumb/plugins/pasteimage/trumbowyg.pasteimage.min.js"></script>
        <script>
            /** Default editor configuration **/

            $('#editor').trumbowyg({

                plugins: {
                btnsDef: {
                    // Customizables dropdowns
                    image: {
                        dropdown: ['insertImage', 'upload'],
                        ico: 'insertImage'
                    }
                },
                btns: [
                    ['viewHTML'],
                    ['undo', 'redo'],
                    ['formatting'],
                    'btnGrp-design',
                    ['link'],
                    ['image'],
                    'btnGrp-justify',
                    'btnGrp-lists',
                    ['foreColor', 'backColor'],
                    ['preformatted'],
                    ['horizontalRule'],
                    ['fullscreen']
                ]
        }
});

Hi!
I think your have fail your copy/paste, because that's the code of the demo, and I think you've copy the wrong plugin key. https://github.com/Alex-D/Trumbowyg/blob/gh-pages/js/main.js#L9
The whole configuration used in demo is:
$('.editor').trumbowyg({
    btnsDef: {
        // Customizables dropdowns
        image: {
            dropdown: ['insertImage', 'upload', 'base64', 'noEmbed'],
            ico: 'insertImage'
        }
    },
    btns: [
        ['viewHTML'],
        ['undo', 'redo'],
        ['formatting'],
        'btnGrp-design',
        ['link'],
        ['image'],
        'btnGrp-justify',
        'btnGrp-lists',
        ['foreColor', 'backColor'],
        ['preformatted'],
        ['horizontalRule'],
        ['fullscreen']
    ],
    plugins: {
        // Add imagur parameters to upload plugin
        upload: {
            serverPath: 'https://api.imgur.com/3/image',
            fileFieldName: 'image',
            headers: {
                'Authorization': 'Client-ID 9e57cb1c4791cea'
            },
            urlPropertyName: 'data.link'
        }
    }
});

There is no documentation on How to create and add new plugins.

Check demo page code or in exemples folder.

@Alex-D
To create new plugin, need to check the source of existing plugins, which is somewhat time taking & hit - trial process. It would be better if there'll any docs for same.

Yep, there is some issues about that and there are open.
Here : https://github.com/Alex-D/Trumbowyg/issues/205
And here : https://github.com/Alex-D/Trumbowyg/issues/114
So I know, but I don't have the time yet. There is lot of simple exemples. If I wrote a doc, It should be at start a copy paste, so just read this code :)

I have written some short snippet to create a new plugin.

//basic template to create new plugin
(function($) {
  'use strict';

  $.extend(true, $.trumbowyg, { //extend `$.trumbowyg`
    //localization object
    // `foreColor` & `backColor` are new btns / functionality
    langs: {
      cs: {
        foreColor: 'Barva textu',
        backColor: 'Barva pozadí'
      },
      en: {
        foreColor: 'Text color',
        backColor: 'Background color'
      },
      fr: {
        foreColor: 'Couleur du texte',
        backColor: 'Couleur de fond'
      },
      sk: {
        foreColor: 'Farba textu',
        backColor: 'Farba pozadia'
      },
      zh_cn: {
        foreColor: '文字颜色',
        backColor: '背景颜色'
      }
    },

    plugins: { // required plugins object
      color: { // plugin name
        init: function(trumbowyg) { //init function, called when plugin is initialized.
           //`trumbowyg` editor instance.

          const btnDef = {
            fn: function(cmd) {
              //`fn` function is callback when button is clicked in editor.
              //`cmd` is button type clicked.
              // In this case, it could be `foreColor` or `backColor`

              /**
               * This command is used to execute command.
               * In this case, color is applied on selected text in editor.
               * trumbowyg.execCmd(cmd, #000000)
               */
            }
          };

          //adding buttons in editor
          trumbowyg.addBtnDef('foreColor', btnDef);
          trumbowyg.addBtnDef('backColor', btnDef);
        }
      }
    }
  });
})(jQuery);

@saggiyogesh Can you please help me out to create plugin for inserting table, when I hover on button it should show a grid structure (just like in ms-word)

https://github.com/Alex-D/Trumbowyg/tree/develop/plugins/table
There already exists a table plugin.

Am I missing something? I don't see anywhere in the docs (which are beautiful btw ;) ) how to actually add these plugins to my editor? I see they came in the /plugins folder in my npm module folder... how do I load/add them?

You can check here: https://github.com/Alex-D/Trumbowyg/tree/develop/examples/plugins

require('./../../../node_modules/trumbowyg/plugins/table/trumbowyg.table')

Hey Alex,
Could you please explain how to use a plugin?
The one I am trying to use is the table plugin.
I can't find anywhere an explanation on how to do it.
Thanks!

@lcompare
        <script src="../../dist/trumbowyg.js"></script>
        <script src="../../dist/plugins/table/trumbowyg.table.js"></script>
        <script>
            $('#editor')
            .trumbowyg({
                btnsAdd: ['table']
            });
        </script>

Well i tried and i dont have any additionnal table button showing up.


⚠ Turnos 24x7 has degraded performance

In 6536109, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 961 ms

Resolved: Turnos 24x7 performance has improved in 3c88eb2.


Added AnonymizeIP and OptOut Capabilities

I have added Support for AnonymizeIP and added OptOut Capabilities

Hi there @CeeEll!
Thanks for the PR, great work! Just a few comments.

Hey @cbrevik ,
thanks for your feedback.
Thats the better way of course ;)
I have integrated some changes and added a bit of documentation.

I had to merge it via command line, but your commits are in. Thanks for the PR @CeeEll!


Rework the interface of cranelift-module

The old design has a Module struct that wraps around a Backend implementation. Because the Backend is not exposed it is not possible to export extra methods to do things like deallocating specific functions or adding new predefined imports for the jit. It is also not possible to customise the behavior of existing methods to for example allow replacing an already defined function for hot code swapping. It was also not possible to create a module with a non-static lifetime (required for headcrab-inject) In the new design Module is instead a trait with ModuleDeclarations being a struct for some shared behavior regarding how declarations are handled. As the Module is fully exposed, it is possible to add new methlds to the api of every module and customize the behavior of existing methods. None of the previously mentioned use cases are implemented yet. I will leave that for a future PR.
Also fixes #1691

This looks cool - I want to spend some time reviewing it but I'm just about to leave for vacation, I'll get to it early next week.

Sorry for the delay - this is excellent work. The only functionality missing that we need in Lucet is an iterator across all function declarations. I have added two iterator getters to ModuleDeclarations here: https://github.com/bytecodealliance/wasmtime/commit/13c6bdd9bab2e8aaacad420a2f90b87624b8e6f3

Added your commit to this PR.


performLinearAnimation should return a callback on complete

Once the animation is finished, it would be ideal if it returned a callback.

Thanks! I'll be happy to accept PR :)

Ok, i'll try to get to it. I've worked around it for now using a setTimeout hack


Not functional with Ubuntu 16.04+

Howdy,
This is a fantastic utility for my 14.04 servers!  I've tried it on a minimal install of 16.04 but it no longer seems to function. The error is:
root@ubuntu:/home/adminux# ./kernel_cleanup
Linux-4.4.0-59-generic-x86_64-with-Ubuntu-16.04-xenial
Traceback (most recent call last):
  File "./kernel_cleanup", line 166, in <module>
    main()
  File "./kernel_cleanup", line 152, in main
    apt_cache, data = validate_and_mark(apt_cache, data)
  File "./kernel_cleanup", line 26, in validate_and_mark
    for linux_package, statuses in sorted(data.iteritems()):
AttributeError: 'dict' object has no attribute 'iteritems'

I don't know if that helps, but I'd dearly love to continue using this utility on upgraded servers.
Cheers,
Eric P.

The issue wasn't with Ubuntu 16.04 LTS (Xenial Xerus), it was with Python 3. I've updated it to work correctly in both Python 2 and Python 3.
I've also started on automated testing to help ensure this doesn't happen again.

Hey - thanks!  Much appreciated.
Eric
On 11 January 2017 at 20:19, Timid Robot Zehta<EMAIL_ADDRESS>wrote:

The issue wasn't with Ubuntu 16.04 LTS (Xenial Xerus), it was with Python
3. I've updated it to work correctly in both Python 2 and Python 3.
I've also started on automated testing to help ensure this doesn't happen
again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClockworkNet/kernel_cleanup/issues/1#issuecomment-272072369,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALS649P-dK0jZQHFDauEdeq0TwkOXkm4ks5rRam6gaJpZM4LhKLD
.


Document use of cancellation tokens for the dotnet-isolated worker

Update public MSFT docs for how to bind to cancellation token for the dotnet worker

So peppering all our service methods with "CancellationToken token = default" is paying off after all 😃


No option to customise CSS for Preview V2

Please select the area your Bug applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[ ] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[x] Misc (Choose this if your not sure)

Describe the bug
Dendron Preview v2 no longer has a method of editing CSS. This was previously an option for Markdown Preview Enhaced.
To Reproduce
Steps to reproduce the behavior:

Search for CSS after pressing F1
Only Dendron: Configure Graph Style (css) appears

Expected behavior
expected to see an option to customise CSS for Preview v2
Screenshots
N/A
Desktop (please complete the following information)

OS: Windows 10
Browser: Chrome
Version: 0.54.0

Additional context
Add any other context about the problem here.
Dendron Log file
Please attach the output of >Dev:Dendron: Open Logs here

Thanks for the ticket. We're planning on working on this in September

Related:

https://github.com/dendronhq/dendron/issues/1360
https://github.com/dendronhq/dendron/issues/1937


Improve accessibility

See https://github.com/ascoderu/opwen-webapp/issues/201

Merging for now. Let me know if you have any input @aidan-fitz and I'll address it in a future PR.


Timeout (499 Client Closed Request) issue introduced with v2.11.2 (working correctly with all previous versions)

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Hello there,
I had recent issues on my setup where Traefik is reverse proxying an Harbor registry.
Version 2.11.2 is closing connection abruptly after exactly 60 seconds when doing a docker push to the Harbor registry behind traefik.
All previous versions do not have this behaviour.
Docker registry shows :
level=error msg="response completed with error"  err.code=unknown err.detail="client disconnected" err.message="unknown error" http.request.contenttype="application/octet-stream" http.request.host=redacted http.request.method=PATCH http.request.remoteaddr=redacted http.request.uri="/v2/internal/redacted/blobs/uploads/41e9-9478-836fc5d5c93f" http.response.duration=1m0.452685338s http.response.status=500 http.response.written=89 vars.name="internal/lina" vars.uuid=ecdca202-8964-41e9-9478-836fc5d5c93f
Nginx proxy used in harbor shows:
upstream prematurely closed connection while reading response header from upstream
And Traefik logs shows :
traefik  | time="2024-04-21T19:12:25+02:00" level=debug msg="'499 Client Closed Request' caused by: context canceled"
I've made a lot of testings and pinpointed this issue to exact version v2.11.2.
What did you see instead?
To fix this issue I keep using v2.11.1 instead of v2.11.2.
What version of Traefik are you using?
Tested with  2.11.2 => has this issue
Tested with 2.11.1 => working correctly
Tested with 2.9.5 => working correctly
What is your environment & configuration?

entryPoints:
  https:
    address: ":443"
    http:
      tls:
        certResolver: default
        domains:
          - main: "*.redacted.domain"

serversTransport:
  insecureSkipVerify: true

providers:
  docker:
    endpoint: "unix:///var/run/docker.sock"
    exposedByDefault: true
    defaultRule: Host(`{{ default (.Name) (index .Labels "com.docker.compose.service") }}.redacted.domain`)
    network: "trafnet"

If applicable, please paste the log output in DEBUG level
traefik  | time="2024-04-21T19:12:25+02:00" level=debug msg="'499 Client Closed Request' caused by: context canceled"

I misread the release notes and thought that the readtimeout value was more permissive in this version than before when in was in reality changed from No timeout to 60 seconds.


usage-guide: fix s2n_mode comment about S2N_CLIENT

Resolved issues:
None
Description of changes:
According to examples in bin/s2nc.c s2n can act as a client. now.
So this comment about server only behaviour is no longer correct.
Call-outs:
None
Testing:
Generated markdown preview looks correct
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2274 into main will increase coverage by 42.15%.
The diff coverage is 84.00%.

@@             Coverage Diff             @@
##             main    #2274       +/-   ##
===========================================
+ Coverage   38.57%   80.73%   +42.15%
===========================================
  Files         164      265      +101
  Lines       12626    17875     +5249
===========================================
+ Hits         4871    14431     +9560
+ Misses       7755     3444     -4311


Work on Official Website.

There must be an awesome website for this project displaying all the features and giving links to clone this repo and steps to use it. I've deployed a Website (/website/src) here - https://twitch-live-chat-history.surge.sh
if You know You can help, Feel free to Contribute!

This Website will not display any chats. This website will be used officially for this project. This website must Display information about this project. Happy Contributing!

A TODO is present in the /website  directory.

Great @Jchann24 ! This PR helped the peoject a lot!

The updates have been deployed! https://twitch-live-chat-history.surge.sh

Great @Jchann24 ! This PR helped the peoject a lot!

Hope it helps! Cheers 🍻


Remove kexecbin package

Not needed anymore due to kexec and multiboot extension.

Codecov Report

Merging #123 into master will decrease coverage by 7.65%.
The diff coverage is 2.83%.

@@            Coverage Diff            @@
##           master    #123      +/-   ##
=========================================
- Coverage   49.56%   41.9%   -7.66%
=========================================
  Files          14      14
  Lines         569     673     +104
=========================================
  Hits          282     282
- Misses        259     363     +104
  Partials       28      28

Impacted Files
Coverage Δ

pkg/crypto/measure.go
0% <0%> (ø)
:arrow_up:

pkg/storage/blockdev.go
16.5% <0%> (-12.18%)
:arrow_down:

pkg/booter/bootentry.go
94.28% <100%> (ø)
:arrow_up:

pkg/bootconfig/zipconfig.go
52.23% <100%> (ø)
:arrow_up:

pkg/bootconfig/bootconfig.go
14.89% <2.85%> (-4.03%)
:arrow_down:

Continue to review full report at Codecov.

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


Select entities not displaying state anymore

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue: latest 4
Last working release (if known): I don't know, sorry! But I know it worked before.
Browser and Operating System:

Description of problem:

Select entities stopped being colored according to their states:
https://user-images.githubusercontent.com/29582865/174462371-01729645-9b32-4db8-8046-78dc223facfb.mp4
Javascript errors shown in the web inspector (if applicable):
N/A
Additional information:

This seems to still work for me.

Can I see the config?
Code near this did change recently (https://github.com/dermotduffy/frigate-hass-card/pull/684) so a bug may have sneaked in ...

Ah, I think it's broken if the options are overridden. Would guess you are doing that. Still investigating.

Confident I got it. Try it out and re-open if not!

@dermotduffy it worked, thank you so much!


[#55] Fix typos in tutorials

Fixes #55
Outline of Solution
Typo mistakes

[x] Typo in tutorial https://github.com/nus-cs2103-AY2324S2/forum/issues/396
[x] Typo in tutorial https://github.com/nus-cs2103-AY2324S2/forum/issues/374
[x] Typo in tutorial https://github.com/nus-cs2103-AY2324S2/forum/issues/352
[x] Typo in tutorial https://github.com/nus-cs2103-AY2324S2/forum/issues/404
[x] Typo in tutorial https://github.com/nus-cs2103-AY2324S2/forum/issues/438

Fixing Invalid intra-links

[x] Invalid link in ab3RemovingFields.md

@se-edu/summer-2024-pr-reviewers for review ... (in particular, cross-check against the issues listed to confirm all issues resolved)

Thanks for the PR, @NeoHW
I've cherry-picked the relevant commits onto the master branch.


Retries Tracing Question

I'm running envoy with jaeger native tracing
is it possible to get the number of retries of a request and store it in jaeger with a tag?
This is my current retry mechanism
                      retry_policy:
                        retry_on: 5xx,retriable-4xx,connect-failure,refused-stream,unavailable,cancelled,resource-exhausted,reset,gateway-error,retriable-status-codes
                        num_retries: 3
                        per_try_timeout: 6s
                        retry_back_off:
                          base_interval: "2s"
                          max_interval: "5s"
                        retriable_status_codes:
                          - 500
                          - 403

and I would like to see in jaeger UI how many times we retry with a tag or something

is it possible to use lua and have the retry counter with lua?


stat: reorganize examples to follow example_xxx_test convention

Fixes gonum/gonum#1385.
Please take a look.

The convention is xxx_example_test.go. This leave the test code next to the source code in the directory.

we have had a PR in gonum/plot that applied example_xxx_test.go.

I am ok with xxx_example_test.go, though.
this PR also migrated examples out of xxx_test.go into example_xxx_test.go (happy to migrate them into xxx_example_test.go)

we have had a PR in gonum/plot that applied example_xxx_test.go.

I missed that filename in the review. The convention is wide spread through the standard library.

I'll send a PR against gonum/plot shortly.

I'm not sure about this. Some of the tests reference the example tests, finding them becomes harder.

I am not sure I understand what you are refering to.

I don't care that much about having examples inside xyz_example_test.go files.


Bump organization management service version and auth rest version

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6747357444

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6747357444
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6747473228

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6747473228
Status: cancelled


Please add --add-host=[], --net options to docker build

This is a feature request to add the --add-host=[] (currently availble for docker run) to docker build also. This would be very handy.

@RoelVdP  Any reason to why you try to do this ?

@resouer Yes. In our somewhat complex network setup, we need to edit the host file. We found a workaround here; http://jasonincode.com/customizing-hosts-file-in-docker/ but surely there must be a better way. --add-host=[] would fix this. It also looks like (see other tickets) many people are looking for this. (Hence --add-host=[] was added to docker run, here it is available today.). Having the same for docker build would be fantastic, thank you

In general I'm +1 with adding support for things like resource management for the builds, however add-host is specifically something that could make a build work on one host and not on another, and you can't tell why by just looking at the source (Dockerfile).
If you need resolution to these hosts, I'd suggest setting up a DNS server to handle these requests transparently.
So basically, I would have to -1 this one for now, I think.

however add-host is specifically something that could make a build work on one host and not on another, and you can't tell why by just looking at the source (Dockerfile).

@cpuguy83 playing devils advocate here; having a special DNS on one host and not on another, would also break the build on one host; looking at the Dockerfile also wouldn't explain why it breaks on that host but succeeds on another?

A DNS server needs to be installed to cover the (fairly basic) functionality of adding a single host IP/name to the hosts file?

@RoelVdP My point is, if you want to modify /etc/hosts, do it as part of the build at the beginning of your RUN command.
If the fact that the build isn't working due to needing hosts is an exception to the rule, then yes, fire up a DNS server like github.com/tianon/rawdns to handle this scenario transparently to the build.
@thaJeztah I get your point. And having --add-host would probably be somewhat equivalent of firing up a DNS server... but a feature once added is abused... which makes me  (personally) less comfortable with this one in particular.

@cpuguy83 It is not possible to do it using RUN command/as part of the build! Docker makes this file R/O....

@RoelVdP The build container is the same thing as a docker run container.
/etc/hosts is mounted as read-write, and I checked just to make sure.
FROM busybox
RUN echo <IP_ADDRESS> foo > /etc/hosts; ping -c 4 foo

@cpuguy83 A nice one-line workaround you found there, but it's not static;
$ cat Dockerfile
FROM busybox
RUN echo <IP_ADDRESS> foo > /etc/hosts; cat /etc/hosts
RUN cat /etc/hosts
$ sudo docker build .
Sending build context to Docker daemon 2.048 kB
Sending build context to Docker daemon
Step 0 : FROM busybox
---> 4986bf8c1536
Step 1 : RUN echo <IP_ADDRESS> foo > /etc/hosts; cat /etc/hosts
---> Running in 22d95c996843
<IP_ADDRESS> foo
---> 5df6fa4f3b7d
Removing intermediate container 22d95c996843
Step 2 : RUN cat /etc/hosts
---> Running in d989b8e95b85
<IP_ADDRESS>     7f674915980d
<IP_ADDRESS>       localhost
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
---> 1232cf428f6b
Removing intermediate container d989b8e95b85
Successfully built 1232cf428f6b

Yep, you'd need to make your changes for each RUN command.

For anyone who's reading this; thanks to @cpuguy83 we now have the following workaround;
RUN echo ${IP} ${NAME} >> /etc/hosts; ${COMMAND}
The command HAS to be on the same line as the hosts addition (it will not be there the next RUN command), but at least this can be used for each command where needed. Definitely easier then the workaround on http://jasonincode.com/customizing-hosts-file-in-docker/
@cpuguy83 Thanks. While a nice workaround, it would still be great to have the better solution mentioned earlier?

Not that there isn't already sufficient justification for this in this thread, but I'm bumping into this big-time when trying to create a Dockerfile that builds a Java project which is dependent on a linked Docker DB container. When building the image, the Java build is kicked off which contains unit tests that need to be able to hit the DB... but of course, I can't link the DB container to the build container. So at a minimum, it'd be nice to be able to add the current host IP of the DB server to the build container's /etc/hosts — but better still would be to be able to establish the link at build time.

+1 we rely on some static hosts file entries for an internal VPN and adding echo "host IP" > /etc/hosts to every command that requires to know about the hosts is a major letdown.

I see the approach here as adding a host(s) as a build argument. It's really too bad we can't easily add a hosts to the Dockerfile as part of the build process. What's the advantage to adding a list of the hosts as a build argument, as opposed to providing a hostfile somewhere inside the Dockerfile, like I do for other resouces?

The posted solution above is the closest to this feature, just looks sort of icky and it might not work for every container:
http://jasonincode.com/customizing-hosts-file-in-docker/

In another thread I read a reference to HOSTALIASES. This works well for me.

--add-host=[] was previously added to docker run. Having the same option available for docker build would be great.

Does any one need --net option?
I think it is necessary for docker build to add it. Because in some case, I want to choose the net during docker build.

@xuxinkun +1
I just reached a few hours ago a situation where I need to explicitly set the --net option during build time. Do you know of any workaround?

@ramonsnir
I did not find a good way. Maybe the only way is to add --net option for build. I am coding on it. Later, I will try to make a pull request,

@xuxinkun Great. While at it, would you mind looking ad adding the --add-host=[] option also (this can maybe be taken integrally from docker run)?

@RoelVdP
OK. I will code it. I will try to make a pull request next week or later.

@cpuguy83 @duglin It seems to me this is another symptom that suggests making build have the same flags as run.
@RoelVdP As a workaround, you can today do docker run $runoptions builder-image | docker build -
assuming builder-image will output a tar context.

@tiborvass
how about this solution? add a --run-opts=[] option. u can use docker build --run-opts=['-net=bridge','--cpushares=1024 '] to achieve this.

Please add --net= option.
Since NAT, Portforwards  and such adds quite a bit of overhead and failure-scenarios we've disabled dockers default networking entirely (--bridge=none) and only allow --net=host to get networking in a container (otherwise we fill the conntrack pool within 15minutes even if we set it as high as 1M entries and effectively hit a DoS scenario), so the only way to get networking in a container is if you add --net=host to the run command. (i wish this could be selected as default).
Currently we can't build anything with docker build if i have an identical setup as we run in production since we can't tell the build-process to use "--net=host".

@cetex
As https://github.com/docker/docker/pull/16004#issuecomment-137591883 say, @tiborvass want to add this option, but not add it directly to docker build. He wants to make docker build have similar option as docker run. I suppose a solution as https://github.com/docker/docker/issues/10324#issuecomment-137907291.
If you want to use --net option right now, my pull request https://github.com/docker/docker/pull/16004 may help you.

I want --net, I don't see why those network parameters for "docker run" cannot be applied to "docker build". Actually I have problems with bridge everyday and eventually we basically always use --net=host for everything (most of my containers gossip each other, do leader election etc).

And I agree with cetex, --net=host should be default for docker run/build.

I'd prefer it if --net=host would be configurable on docker daemon commandline so you "select a system default" when starting the daemon. docker [run|build] --net= should only be used when you want to override the system defaults for some reason.

agree, --net=host should be allowed on build. building an image on our bare metal servers with our own apt repos is such major PITA for something so minutely trivial that I'm starting to rethink docker completely. what other gotchas for previously solved things will I run into?

+1 --net=host
"docker build" does not work for me when connected with VPN to my office,because the docker0 bridge is blocked. When running the container --net=host  solves the issue.

+1 --net=host
"docker build" does not work often times in various permutations of AWS EC2 VPC's as well, while docker run works perfectly fine on 1.8.1. The divergent networking behavior between docker build and docker run needs to be sorted.

+1 --net=host
Also, on a cloud vm.  run works with --net=host, but I  can't build a docker container.

@tiborvass
How exactly to do this workaround?

@bakanov docker run -v $(pwd):/context --net=host builder-image | docker build -t myimage - where builder-image sends a tar stream of /context to stdout.

1 This NEEDS to be implemented also in the dockerfile. It's totally absurd that I cannot set a host alias in a docker image during the build process. I dont see how I should have to supply arguments to the docker run command which should be internal to the container. It makes no sense to me.

@RoelVdP can you clarify if the work around you describe results in the /etc/hosts file being modified permanently in the container after the build is complete?  I can certainly see that it modifies etc/hosts within the context of a run command, but any changes I make are ultimately overwritten in the final built container

May be a good feature for Dockerfile will be smth like this:
REQUIREDHOSTS ['builddb', 'saltmaster', 'inetproxy']
Just naming the hosts to add during build and ensure their availability during run. Names only.
Such a feature will solve the initial problem and add the needed transparency avoiding possible human errors. Will it?

@jskrepnek Sorry cannot recall details. Been quite a while since we needed it.

+1 --net=host

+1 for --net=host or --add-host

+1

USER POLL
The best way to get notified of updates is to use the Subscribe button on this page.
Please don't use "+1" or "I have this too" comments on issues. We automatically
collect those comments to keep the thread short.
The people listed below have upvoted this issue by leaving a +1 comment:
@lwcolton

+1

Without this, how can you build this example from https://docs.docker.com/engine/examples/apt-cacher-ng/?
FROM ubuntu
RUN  echo 'Acquire::http { Proxy "http://dockerhost:3142"; };' >> /etc/apt/apt.conf.d/01proxy
RUN apt-get update && apt-get install -y vim git

# docker build -t my_ubuntu .

@ebuildy good point, this is a type in this docs, the RUN echo … should be down one line.

In this example, dockerhost cannot resolve, isnit?
I achieved this with --build-arg to give the container IP instead of a hostname, but I am really curious to see the doc example working, is it broken or I miss something?

@ebuildy it's broken, I did a tiny tiny PR to fix it (naively) https://github.com/docker/docker/pull/19580

why can't we also change hosts file inside the container? or a method to disable the hosts file management in the container, altogether..

Due to the issues with ufw on docker we use the setup described here https://github.com/docker/docker/issues/4737#issuecomment-191653053.
With this setup, only containers in our custom bridge from the IP range <IP_ADDRESS>/24 have access to the internet. During build we can not specify this bridge with --net=mybridge so we have no internet access and the build fails.
So big +1 for adding the --net option to build.

I have a different use case for docker build --net.  I want highly reproducible Docker image builds and would like to be able to disable networking entirely, so the only packages installed into the container are those provided explicitly and not random package downloads off the Internet.

Why not use something like the following in your Dockerfile?
FROM busybox

ARG HOSTS
ENV HOSTS ${HOSTS:-}

echo ${HOSTS} foo

Then just run…
docker build --build-arg HOSTS=<IP_ADDRESS>

I would like support for docker build net=none to be sure that a build was not using the internet.

To access apt-cacher-ng (or other services!) on a different network during docker build, we jump through a whole bunch of jinja2 hoops - something along the lines of

Set apt-cacher up to be accessible on '<IP_ADDRESS>/16
Have a header in dockerfile that updates the proxy to be at the "172.17" address for apt-cacher-ng
Do all the docker build stuff...
Have a footer in dockerfile that rewrites the proxy to be at the "real" address for apt-cacher-ng

(And the jinja2 entertainment comes in because apt-cacher-ng is a docker container elsewhere, and the address changes, so yay, more fun.  Whatever)
This does keep the dockefile Pure and Repeatable, but lord, at what cost?
So yeah, a huge +1 to having --net= as an option...

@franz-josef-kaiser If we do that, when the DNS of that domain changed, we have to rebuild the docker image. In a cluster of containers like in Kubernetes, the domain of a service will change a lot.

@wb14123 Valid concern. Above was meant as "in the meantime"/workaround. Additional note: When you run it as separate RUN statement in the end, then it's just a very quick container that you can rebuild very fast as the rest is already cached. Such a rebuild should just take some seconds.

+1

@franz-josef-kaiser I'm using your trick. But I think you meant:
RUN [ -n "${HOSTS}" ] &&

(Removed the bang !)
I'm using this to set a path to our enterprise Maven repository (building a Java project). The only purpose of this image is to provide an isolated build environment, so it's not a problem for me, that the IP could change.

+1

+1

+1

+1

Docker guys, this request is 1.5 years old, has a lot of discussion and people interested. When will very easy feature request be considered for implementation?

+1

+1

i think never XD

+1

the echo "ipaddress fullyqualified dns-name" >> /etc/hosts does not work if you build as non-root. /etc/hosts is only writeable by root. How is that fixed.
I do not like to rewrite all our buildscripts.

Hi, people interested in this issue, we are looking at https://github.com/docker/docker/pull/20987 again to implement this.

+1

+1

@tonistiigi @thaJeztah how does #27702 fix this?

Note that though setting the network at build time is now merged, then it has been possible to access the hosts on networks from the container to the internet (i.e container -> docker host -> osx (when on mac) -> whatever comes next).

I hit this issue today, one of the build script need to have specific domain name and IP mapping during the image building. Having --add-host in docker build would be handy in this case. And I don't think #27702 fully solved the request of this issue.

+1 , there needs to be --net options  for the docker build command.

@gauravkaila this feature is in 1.13, which will be released next week. Please try out the release candidates and let us know if it is working for you.

@twang2218 if you need new features, please don't comment on a closed, resolved issue as no one will read it, you need to open a new issue.

@justincormack The title of this issue is Please add --add-host=[], --net options to docker build, and only --net has been added to docker build, --add-host=[] is still missing, that is the reason I didn't open another issue, because it is the same issue, and it shouldn't be closed.

Just open a new issue. There were several issues for this previously, it is
not helpful to comment on closed issues, I only saw this by accident.
On 12 Jan 2017 11:44 a.m., "Tao Wang"<EMAIL_ADDRESS>wrote:

@justincormack https://github.com/justincormack The title of this issue
is Please add --add-host=[], --net options to docker build, and only --net
has been added to docker build, --add-host=[] is still missing, that is
the reason I didn't open another issue, because it is the same issue, and
it shouldn't be closed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/issues/10324#issuecomment-272143236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdcPCBi4v_TfDbg8SbHnxj6eDgc57AEks5rRhIagaJpZM4DWmzR
.

ok, I will open a new issue for this.

COPY hosts /root/hosts
ENTRYPOINT ["bin/sh", "-c", "cat /root/hosts >> /etc/hosts && exec /bin/bash"]
Seems to be working pretty well, if you need to add a list of hosts to your container's /etc/hosts

I'm facing now an issue that it could be solved with this feature. May be I'm wrong and it could be solved somehow else.
I'm building an image in my CI environment and in one of the commands I need to access a production service that is deployed in a private network. There is a public IP available, but I need that the configured ip in the image is the private one. I saw the --ad-host in the run command so i though that that could be a solution. Point the service to a host name and then redirect the hostname on build time and runtime to different ip's
But no --add-host uption for docker build...
May be there is another option to solve this?

@compains --add-host was added in v17.04 #30383

+1

theres a similar option for the service ?

+1000
We need to be able to set the custom dns from dockerfile

+1

+1

+1

+1

Hi,
If you want to specify hostname during build time, now it implemented under this pull request:
https://github.com/moby/buildkit/pull/1339
You can vote for it to be merged asap.


Fix docstring rst syntax

Should look better now.

Current coverage is 65.61% (diff: 100%)

Merging #1706 into master will not change coverage

@@             master      #1706   diff @@
==========================================
  Files            54         54
  Lines          6122       6122
  Methods           0          0
  Messages          0          0
  Branches       1210       1210
==========================================
  Hits           4017       4017
  Misses         1892       1892
  Partials        213        213

Powered by Codecov. Last update a76f0e5...cd9b9bf

Thanks!


Warn about strip extras by default

References #1613. In preparation for the next major release where --strip-extras will be the default, give a warning if the flag is not given and add a matching --no-strip-extras flag to allow forcing the existing behavior without the warning.
Contributor checklist

[ ] Included tests for the changes.
[x] PR title is short, clear, and ready to be included in the user-facing changelog.

Maintainer checklist

[x] Verified one of these labels is present: backwards incompatible, feature, enhancement, deprecation, bug, dependency, docs or skip-changelog as they determine changelog listing.
[x] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

I haven't written tests yet, because I'm not clear on the failures. I'm opening this PR for review and to see what the tests look like in the pull request.

@ryanhiebert check out #1955 where failures fixed.


[FLINK-8862] [HBase] Support HBase snapshot read

What is the purpose of the change
Flink-hbase connector only supports reading/scanning HBase over region server scanner, there is also snapshot scanning solution, just like Hadoop provides 2 ways to scan HBase, one is TableInputFormat, the other is TableSnapshotInputFormat, so it would be great if flink supports both solutions to ensure more wider usage scope and provide alternatives for users.
Brief change log

Create TableInputSplitStrategy interface and its implementations as abstraction logic for AbstractTableInputFormat
Update HBaseRowInputFormat and TableInputFormat
Add HBaseSnapshotRowInputFormat and TableSnapshotInputFormat
Extract 2 interfaces including HBaseTableScannerAware and ResultToTupleMapper
Add HBaseSnapshotReadExample

Verifying this change
This change is already covered by existing tests as follows, and new test cases has been added as well.
org.apache.flink.addons.hbase.HBaseConnectorITCase
This change added tests and can be verified as follows:

Manually create one snapshot for a specific HBase table, and use TableSnapshotInputFormat to do full scan.
Running existing HBaseReadExample to do full scan.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
For document, please visit JIRA ticket, a detailed design doc and class diagram have been attached.

@zentol I was thinking could you help me review the feature and code? Or is there anyone more appropriate for this PR? many thanks.

Thanks for the PR @neoremind.
At the moment, the community is busy working on the 1.5 release which means that PRs for 1.5 fixes have priority right now. Also a large contribution such as this one takes a lot of time to review. Unfortunately, I won't be able to review the PR in the near future. Best, Fabian

@fhueske Thanks for your response. I understand this case. Please take your time, hope this PR can be reviewed in the future and help people who needed. Thanks!

@neoremind
The Flink HBase connector resides in it's own repository nowadays, if this code change is still relevant, please open the PR in https://github.com/snuyanzin/flink-connector-hbase


ios Build Error - pinCache is inaccessible due to internal protection level

Hi,
I installed the module and got it up and running on Android without a problem. Switched over to my mac and did the pod install. Everything looked good. Did a clean and rebuild the workspace and I get two errors.
First is:
pinCache is inaccessible due to internal protection level in RNGiphyCache.swift at line 16 (GPHCache.shared.pinCache.diskCache.byeLimit = (Uint(limt)
Second is:
Module GiphyUISDK has no member named configure in RNGiphyUI.swift
I have use_frameworks! in my pod file and I also added the :modular_headers = true to the Pod File entry for good measure :)
Any thoughts???

I was able to get past this error by adding
pod 'PINCache', :modular_headers => true

to the Podfile then clean and rebuild


OR scraper down

The state may have re-re-designed the page again, but I have not looked.

I checked last night and the state web site broke. I'd emailed but they almost certainly have today off.


Redundant output from 'check --help'

The flag data that is in the long description for the the check command is redundant. It will get printed twice if --help is supplied.

What it currently looks like:
`❯ bin/opcap check --help
The 'check' command checks if OpenShift operators meet minimum
requirements for Operator Capabilities Level to attest operator
advanced features by running custom resources provided by CSVs
and/or users.
Usage:
opcap check [flags]
Example:
opcap check --catalogsource=certified-operators --catalogsourcenamespace=openshift-marketplace --list-packages=false'
Flags:
--catalogsource				specifies the catalogsource to test against
--catalogsourcenamespace	specifies the namespace where the catalogsource exists
--auditplan					audit plan is the ordered list of operator test functions to be called during a capability audit
--list-packages				list packages in the catalog
--filter-packages			a list of package(s) which limits audits and/or other flag(s) output
Usage:
opcap check [flags]
Flags:
--audit-plan strings              audit plan is the ordered list of operator test functions to be called during a capability audit. (default [OperatorInstall,OperatorCleanUp])
--catalogsource string            specifies the catalogsource to test against (default "certified-operators")
--catalogsourcenamespace string   specifies the namespace where the catalogsource exists (default "openshift-marketplace")
-h, --help                            help for check
--list-packages                   list packages in the catalog
--packages strings                a list of package(s) which limits audits and/or other flag(s) output
`


Can't run development mode with bun

Describe the bug

ts-node throws config:ts-node TypeError: module_1.Module._preloadModules is not a function.
With a workaround for the issue above oclif throws this warning warn: Requested module is not instantiated yet

To Reproduce
Steps to reproduce the behavior:

Create a new oclif project bunx oclif generate pe1ucaCLI
In bin/dev replace the first line with #!/usr/bin/env bun
In the same file comment require('ts-node').register({project})
Run bin/dev hello:world
See oclif doesn't find the example command

Expected behavior
oclif should run the commands from the source code.
Screenshots
Environment (please complete the following information):

OS & version: Ubuntu 22.04.3 LTS
Shell/terminal & version bash-5.1.16

Additional context
require('ts-node').register({project}) throws an error due to an issue with bun https://github.com/oven-sh/bun/issues/6227
Since bun has typescript as first-class citizen this dependency is not necessary.
But commenting this line is not enough, since oclif/core has another call to register
https://github.com/oclif/core/blob/49a48415b30056f762e864974263ad686e0ee291/src/config/ts-node.ts#L94
Commenting both of these lines makes oclif properly load the source code.
Before commenting the line in core
config loading core plugin from /home/pe1uca/Documents/pe1ucaCLI +0ms
config:ts-node registering ts-node at /home/pe1uca/Documents/pe1ucaCLI +0ms
config:ts-node ts-node path: /home/pe1uca/Documents/pe1ucaCLI/node_modules/ts-node/dist/index.js +0ms
config:ts-node TypeError: module_1.Module._preloadModules is not a function. (In 'module_1.Module._preloadModules(service.options.require)', 'module_1.Module._preloadModules' is undefined)
config:ts-node     at register (/home/pe1uca/Documents/pe1ucaCLI/node_modules/ts-node/src/index.ts:4:37)
config:ts-node     at /home/pe1uca/Documents/pe1ucaCLI/node_modules/@oclif/core/lib/config/ts-node.js:58:13
config:ts-node     at processTicksAndRejections (:13:78) +28ms
config:pe1ucaCLI loading IDs from /home/pe1uca/Documents/pe1ucaCLI/dist/commands +0ms
config:pe1ucaCLI found commands [] +11ms

After commenting it
config:ts-node registering ts-node at /home/pe1uca/Documents/pe1ucaCLI +0ms
config:ts-node ts-node path: /home/pe1uca/Documents/pe1ucaCLI/node_modules/ts-node/dist/index.js +0ms
config:ts-node tsconfig: {
config:ts-node   <tsconfig>
config:ts-node } +14ms
config:ts-node ts-node options: {
config:ts-node   <ts-node options>
config:ts-node } +1ms
config:ts-node determining path for /home/pe1uca/Documents/pe1ucaCLI/dist/commands +0ms
config:ts-node lib dir: /home/pe1uca/Documents/pe1ucaCLI/dist +0ms
config:ts-node src dir: /home/pe1uca/Documents/pe1ucaCLI/src +0ms
config:ts-node src commands dir: /home/pe1uca/Documents/pe1ucaCLI/src/commands +0ms
config:ts-node Found source file for /home/pe1uca/Documents/pe1ucaCLI/dist/commands at /home/pe1uca/Documents/pe1ucaCLI/src/commands +0ms
config:pe1ucaCLI loading IDs from /home/pe1uca/Documents/pe1ucaCLI/src/commands +0ms
config:pe1ucaCLI found commands [
<list of all commands in src>
] +12ms

With the fix above the warning warn: Requested module is not instantiated yet shows up, can't see something in my project affected by this, the commands run properly.
Seems related to this part of the code
https://github.com/oclif/core/blob/49a48415b30056f762e864974263ad686e0ee291/src/module-loader.ts#L19
Here's the log
warn: Requested module is not instantiated yet.
    at link (:1:11)
    at linkAndEvaluateModule (:1:11)
    at <anonymous> (/home/pe1uca/Documents/pe1ucaCLI/node_modules/@oclif/core/lib/module-loader.js:16:93)
    at processTicksAndRejections (:12:39)
module<EMAIL_ADDRESS>task: cacheCommand
plugin: pe1ucaCLI
root: /home/pe1uca/Documents/pe1ucaCLI
See more details with DEBUG=*
warn: Requested module is not instantiated yet.
    at link (:1:11)
    at linkAndEvaluateModule (:1:11)
    at <anonymous> (/home/pe1uca/Documents/pe1ucaCLI/node_modules/@oclif/core/lib/module-loader.js:16:93)
    at processTicksAndRejections (:12:39)
module<EMAIL_ADDRESS>task: cacheCommand
plugin: pe1ucaCLI
root: /home/pe1uca/Documents/pe1ucaCLI
See more details with DEBUG=*

Seems the issue with the requested module cause by trying to load a command with an import with a relative path.
import * as config from '../../config';
Removing this import makes the warning disappear.


How to add a new page

I'd like to create an off-topic page.
I created an _offtopic folder and add a few files under it. I also updated navigation.yml and created offtopic.html file.
When I want to access the files under _offtopic folder
{% include base_path %}

{% for post in site.offtopic reversed %}
  {% include archive-single.html %}
{% endfor %}

the page looks empty.
How can I fix this?

any luck? I also tried to rename a page. However if I rename it, the site can't be found anymore. I renamed the _portfolio dir,  navigation.yml and also _pages/portfolio.html. Anything else?

I have the same problem now... I wonder how did you guys manage to solve this? @xuewenyuan @abdullahyildiz

Aha! I found the solution. For those who will see this in the future, I need to update the collections and defaults fields in the _config.yaml file.

thanks for  your solution, I got it!

thanks for your solution, I got it! @fangzhou-xie


Add isBetween(min:max:) and clamped(min:max:) Comparable extensions

🚀
This PR adds two extensions to Comparable
isBetween(min:max:) returns whether or not a Comparable is bounded in a range.
clamp(min:max) limits a Comparable within the provided bounds.

Why are you calling it clamp @freak4pc ? 😊

@jdisho It's the common name for this kind of behavior:
https://developer.apple.com/documentation/swift/range/1786602-clamped
http://en.cppreference.com/w/cpp/algorithm/clamp

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

@freak4pc Ah good to know! Thanks! 👍

Hey guys :))
@freak4pc Nice extensions 💯
This is merged but just point few things, that maybe we can have as future improvements if you guys think they are valid to the discuss :))
About 7.isBetween(min: 6, max: 12) I don't know, I think this is a behavior that can be easily achieved with 6...12~=7 or (6...12).contains(7) But is still a valid extension :))
The clamped() extension, instead of using two parameters min and max, what do you guys think about using a signature with a range type like
 public func clamped(to range: ClosedRange<Self>) -> Self {
        return max(range.lowerBound, min(self, range.upperBound))
 }

// Used like "c".clamped(to: "e"..."g")

I think it can be a more clear API and that is a signature more like this Apple one
@freak4pc Maybe you already thought about those points, but any way, let me know if you think they are valid points to discuss :))
cc @SD10

Hey @LucianoPAlmeida .
ClosedCountableRange only supports Strideable types. Notice that all your examples are using Numerics. Using that method you won't be able to confirm a String is in range or that a Date is in range, etc. Comparable is much more wide in its coverage.
Hope this clears up the reasoning on not using a range type.


pkg/bisect: bisect release tags

Currently we process release tags one by one, which can be a real problem if there are many tags. We could bisect them instead.
The related code is here:
https://github.com/google/syzkaller/blob/356d821720a2d24a4cc96f8c0b2b7a11c8882190/pkg/bisect/bisect.go#L379-L389
@HerrSpace  FYI

I think this is by design (though not my design @dvyukov). I think the idea of this is, that crashes are somewhat regularly re-introduced. E.g. this can happen if a fixing commit was later reverted. If we bisect over [bug introduced, bug fixed, bug reintroduced] git bisect might find the first 'bug introduced' commit.
If we don't care about this we could always bisect the whole range between the oldest commit we support bisecting and HEAD. However I think what we are currently doing is not a bad idea.
As we have previously discussed, our current approach results in android bisections regularly timing out. Possible mitigations might include: not testing every x.y.z tag but x.y tags instead, skipping increasingly large number of tags when going further back in history, increasing job timeouts, ...

The initial linear search is by-design for major releases (x.y) and it seems to work fine.
If we always start with the oldest supported revision, we hit non-booting kernels with newer config and lots unrelated bugs.Overall there was high correlation between how old is the bug and successful bisection that points to the right commit:
https://github.com/google/syzkaller/issues/1051#issuecomment-477264585
I did the linear search for minor LTS releases (x.y.z) merely because we already had that code, we did not have binary search code, and it was unclear how well/bad it will work beforehand.
Taking into account the timeouts we see, I think some form of binary search is needed:

either start from the previous major release (X.Y) and then bisect minor ones in halves
or a more conservative approach if we think we can hit older versions of the same bug: try x.y.z-1, -2, -4, -8, ..., and either bisect the smaller range, or switch to major releases if we reach one.

The problem is no longer specific only to x.y.z versions. Here's e.g. a sample bisection log from upstream: https://syzkaller.appspot.com/x/bisect.txt?x=153007a0680000
For clang, there are currently ~ 23 releases that might potentially have to test (v5.4 -- v6.5), that's 8-9 hours. Given that we also spend 2 hours minimizing the kernel config, we barely have any time left for the actual bisection.

or a more conservative approach if we think we can hit older versions of the same bug: try x.y.z-1, -2, -4, -8, ..., and either bisect the smaller range, or switch to major releases if we reach one.

A related simple approach is:
a) Test 2 latest x.y.z releases, if we start from x.y.z (this is not relevant for upstream Linux).
b) Take 3 latest x.y releases.
c) Then take every 3rd x.y release.
Now that there are other bisection quality improvements in place, we can probably tolerate the quality of cause bisections over 2-3 releases.
For bisections with clang, this would be "v6.5", "v6.4", "v6.3", "v6.0", "v5.17", "v5.14", "v5.11", "v5.8", "v5.5", "v5.2" (10 releases, ~4 hours of bisection time).
Proper bisection over releases would of course give more precision on average, but the problem is that we don't have just "good" or "bad" verdicts, but also"skip". These skips, which we have to consider, would complicate the bisection code. Probably it's not really worth it.


Add ResponseCls class attr to HTTPConnectionPool

This attribute defaults to HTTPResponse, but developers can subclass HTTPConnectionPool and substitute their own HTTPResponse subclass, as long as it has valid from_httplib() and __init__() methods, with compatible method signatures.

I'm happy enough with this change.

Great idea, thank you! Important step in the right direction.

Can someone with permissions re-try the pypy tests? It looks like it failed a flaky test.

ACK

:shippedit:

:boom:


Binary release

XCode is not really intended for non-technical Mac users. Also it occupies a vast amount of disk space. I would imagine that many potential BTrain enthusiasts do not have it installed.
Would it be possible to make a binary release ?

Yes, this is a good idea, let me see what I can do!

Done - you can now find the binary for version 1.5 here.

I'm running Monterey (macos 12.6.3) and got this error from 1.5:
> open /Applications/BTrain.app/
The application cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-10825 "kLSIncompatibleSystemVersionErr: The app cannot run on the current OS version" UserInfo={_LSLine=3863, _LSFunction=_LSOpenStuffCallLocal}

I've never seen this message before, from any Mac app.
Would it help if I provided some kind of debugging info ?

Ah, you need to be on Ventura (13.x). Let me see if I can re-compile for 12.x, it will depend on the number of APIs that are not available for 12.x.

On Jan 31, 2023, at 10:33 PM, F.Baube @.***> wrote:
I'm running Monterey (macos 12.6.3) and got this error from 1.5:

open /Applications/BTrain.app/
The application cannot be opened for an unexpected reason, error=Error Domain=NSOSStatusErrorDomain Code=-10825 "kLSIncompatibleSystemVersionErr: The app cannot run on the current OS version" UserInfo={_LSLine=3863, _LSFunction=_LSOpenStuffCallLocal}
I've never seen this message before, from any Mac app.

Would it help if I provided some kind of debugging info ?
—
Reply to this email directly, view it on GitHub https://github.com/jean-bovet/BTrain/issues/29#issuecomment-1411536304, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIUKJDILNVU2B5VABXINWODWVH7UHANCNFSM6AAAAAAUKYVINM.
You are receiving this because you modified the open/close state.

My own Mac is not capable of running Ventura, so it is not an option.
"About This Mac" says "MacBook Pro (Retina, 13-inch, Early 2015)".

Ah, sorry to hear that! Unfortunately, I don't have the resources at the moment to make my code compatible with OS version before Ventura.


Fixes dependency issue with idna

Fixes #778

I will test this branch and see what my results are later tonight. Thanks for the quick turnaround @kevgliss!

I will test this branch and see what my results are later tonight. Thanks for the quick turnaround @kevgliss!


Aave Supply Alerts

@mikeshultz is going to disable the supply alerts that we currently have for Aave v1 in our Discord #ousd-bot channel. We will want to add the alerts back once we're monitoring Aave v2.

Specifically, we'll want to add v2 snapshotting, review any other events we might find of interest, and reenable the supply/borrow notifications.
Ref: https://github.com/OriginProtocol/ousd-analytics/pull/172

@sparrowDom Is this alert included in the new prometheus based monitoring/alerting system?

Yes the amount that is available to borrow via Aave is now done with Grafana&Prometheus. No need for a separate system implementing this


UI not loading list of users in organisation/1/settings/users page

List of users throws 404 error

Tested it by adding more users. It working fine

@shmiitg the problem was on my end!


Backbone demo application + tutorial

I created a separate sln for example web apps. This example uses the Todos app from the Backbone site (http://documentcloud.github.com/backbone/#examples-todos) with permission.
Tutorial is in the readme for now (https://github.com/christophercliff/cassette/tree/backbone/examples/Backbone), not sure where it should go.

Excellent, thank you! I'll get this merged in soon.
For now the tutorial in readme.md form is fine. For simplicity, I'll create a page on getcassette.net that links to tutorial readmes on GitHub.
Eventually we can move to having tutorials hosted directly at getcassette.net.


Polygon diagnoals cut polygon's outline during monotone partition.

[2024-06-09T07:13:38Z DEBUG triangulate_rs::app] current cursor position:(157,543)
[2024-06-09T07:13:38Z DEBUG triangulate_rs::app] Pushing point coordinate:(157,29)
[2024-06-09T07:13:39Z DEBUG triangulate_rs::app] current cursor position:(308,400)
[2024-06-09T07:13:39Z DEBUG triangulate_rs::app] Pushing point coordinate:(308,172)
[2024-06-09T07:13:39Z DEBUG triangulate_rs::app] current cursor position:(481,523)
[2024-06-09T07:13:39Z DEBUG triangulate_rs::app] Pushing point coordinate:(481,49)
[2024-06-09T07:13:40Z DEBUG triangulate_rs::app] current cursor position:(624,399)
[2024-06-09T07:13:40Z DEBUG triangulate_rs::app] Pushing point coordinate:(624,173)
[2024-06-09T07:13:41Z DEBUG triangulate_rs::app] current cursor position:(500,223)
[2024-06-09T07:13:41Z DEBUG triangulate_rs::app] Pushing point coordinate:(500,349)
[2024-06-09T07:13:42Z DEBUG triangulate_rs::app] current cursor position:(378,336)
[2024-06-09T07:13:42Z DEBUG triangulate_rs::app] Pushing point coordinate:(378,236)
[2024-06-09T07:13:43Z DEBUG triangulate_rs::app] current cursor position:(185,239)
[2024-06-09T07:13:43Z DEBUG triangulate_rs::app] Pushing point coordinate:(185,333)
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex4 is start vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex6 is start vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex5 is merge vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex3 is regular vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] interior is left to vertex3
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex1 is split vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex2 is end vertex
[2024-06-09T07:13:44Z DEBUG triangulate_rs::triangulate_2] vertex0 is end vertex

fixed with 1e066a3e45b6ae0115135a545a0e25fd8a7845f7


Add Debian files for packaging

We are building certain Python packages for a long time as Debian packages, but the workflow we used so far needs to change.
So the question is if you want to add these files to upstream. For future releases you would/should update the debian/control file for the changelog.
If you don't want to maintain this I am fine with that and we would do this only for our version.

@AlexanderWells-diamond , any thoughts on this?

I'm afraid I don't really understand what this PR is trying to achieve. Does Debian need different builds compared to other linux/unix platforms?

This PR would allow epicsdbbuilder to be built and installed using Debian ecosystem. If the above files are present, one is able to use dpkg-buildpackage to create a .deb package out of the Python source files, so you get something like python3-epicsdbbuilder.deb which then you can install via the Debian based package manager ( apt). In this case you don't have to rely or even need pip or PyPI (which in our case is not available inside our closed accelerator networks).
With more recent Python versions this will become more of an issue, as installing Python packages via pip system wide is dis-encouraged ( see PEP 668 ) and actually results in an error (which you can override by a command line parameter) for example in Ubuntu 24.04 (LTS).
One "downside" is that the uniqueness of the Debian package is defined by the debain/changelog file, so for a new version release, this would need to be updated. To give an example, here is the changelog file for the debainzed version of numpy.
In case you don't want to maintain the Debian flavour (by keeping the debian/changelog up to date), we can close thsi PR and we would probably set up a workflow similar to how Debian maintains its Python packages: Maintaining our own "Fork" (technically it will not be a fork), where we maintain the Debian related files in a separate branch

It looks like we currently don't have a CHANGELOG.rst in this project (such as pythonSoftIOC/CHANGELOG.rst.  Would the Debian build process with debian/changelog being a soft link to such a file, or would the formatting be incompatible?

There are strict formatting requirements for the debian/changelog. Each version has an entry such as this, where the first and last line is "critical" and in between is bascially free form text to list the changes:
package (1.1.0) stable; urgency=low

* Change 1
* Change 2

 -- Name<EMAIL_ADDRESS>Tue, 31 Oct 2023 14:26:45 +0100

so you cannot just include a .rst file.
I am also considering on creating PRs  to add debian files for:

pythonSoftIOC
epicscorelibs
setuptools_dso
pyepics
caproto

Overall I don't think I'm in favour of this and would prefer it to be maintained in a separate place.
This is largely because I'm not sure I'd ever remember to update such files when doing a release!

Ok, makes sense to me. Then we will set up our internal workflow for this (and maybe try to make it publicly available for those who would be interested)


Reformat code & documentation

Reformat JavaScript code to follow more modern coding standards. Also, documentation to be updated to use docco. High risk job but I feel it should be done for the full release.
Build process will need updated to reflect these changes to documentation generation, which will probably be maintained in the repository from now on.

Like iOrder, this will now also include conversion to CoffeeScript.

Build process has been updated, the JavaScript code has been converted to CoffeeScript and there appears to be no errors and basic functionality is working, documentation has been changed to be compatible with docco and initial generated docs have been added to version control.
All that is left is more intense testing to ensure the conversion worked in all areas. This might possibly involve comparing the compiled JavaScript with the most recent version of the old handwritten JavaScript files. Also, final draft of docs will need to be generated just before 1.0.0 is released.

Code/logic looks good after a quick comparison and I'm happy with the results. There may be bugs introduced but hopefully the testing of #49 will help uncover these.


chore: Add comments for properties that come from additional API endpoints

The Encharge and Dry Contacts models have properties that come from 2 different API endpoints. This adds comments to specify a primary API endpoint that returns the majority of the data, and comments to specify which properties come from other API endpoints. Referenced everything by their constant name in const.py

Codecov Report
Patch and project coverage have no change.

Comparison is base (c5d2f36) 0.86% compared to head (fd8e605) 0.86%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #14   +/-   ##
=====================================
  Coverage   0.86%   0.86%
=====================================
  Files          7       7
  Lines        230     230
  Branches      44      44
=====================================
  Hits           2       2
  Misses       228     228

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


Switch from winapi to windows-rs

I'm trying to compile this crate for ARM64EC. winapi doesn't seem to be developed any more and isn't going to be updated for that architecture, and the current PR #5 for the windows crate is woefully outdated.

Fixes #9


If logged in using account 1, clicking login email link sent to account 2 doesn't do anything

Reproduce:

login in the app with account 1
request a login code for an account 2 (on station.ruuvi.com)
click the email login link on the phone that has account 1 logged in on the app
the app opens but nothing happens and account 1 stays logged in

I would expect the account 2 to login. But that could cause some confusion for the user because the logout confirmation message would not be shown. So maybe safer and more simple would be just to show a note with an OK button:
"Please sign out before trying to sign in using a different account."
"Kirjaudu ulos ennen kuin koitat kirjautua sisälle eri tilillä."
I haven't tried to do this with iOS. Please try and create a similar ticket so that the user experiences will be similar.

1.6.15


idb2pat: incorporate fixes from @HongThatCong

this is the updated script that @HongThatCong referenced in #97

need to test this on py3, since i think there's some issues around str/bytes handling.

Hi !
I have add Python 3 support to idb2pat.py.
https://github.com/HongThatCong/idb2sig
Thanks


added updates to data copy procedure to include note on scc warning

This includes a note on the following:
Warning: would violate PodSecurity "restricted:latest": allowPrivilegeEscalation != false (container "example" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "example" must set securityContext.capabilities.drop=["ALL"]), runAsNonRoot != true (pod or container "database" must set securityContext.runAsNonRoot=true), seccompProfile (pod or container "example" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")

Using the article i was actually able to get rid of the security warning this way:
(venv) [ospng@lenovo-p720-rdo-10 adoption-db]$ oc run mariadb-client --override-type=strategic --overrides='{"apiVersion": "v1", "spec": {"containers": [{"name": "mariadb-client", "securityContext": {"allowPrivilegeEscalation": false, "runAsNonRoot": true, "runAsUser": 42434, "runAsGroup": 42434, "capabilities": {"drop": ["ALL"]}, "seccompProfile": {"type": "RuntimeDefault"}}}]}}' --image $MARIADB_IMAGE -i --rm --restart=Never --    mysql -h "$PODIFIED_MARIADB_IP" -uroot "-p$PODIFIED_DB_ROOT_PASSWORD" -e 'SHOW databases;'
If you don't see a command prompt, try pressing enter.
warning: couldn't attach to pod/mariadb-client, falling back to streaming logs: Internal error occurred: error attaching to container: container is not created or running
Database
cinder
glance
information_schema
keystone
mysql
neutron
nova
nova_api
nova_cell0
performance_schema
placement
pod "mariadb-client" deleted

But you can see somehow that creates a new warning which was not present previously:
warning: couldn't attach to pod/mariadb-client, falling back to streaming logs: Internal error occurred: error attaching to container: container is not created or running

And i had to rely on digging out the numeric mysql user ID from the container. So i think we should still just document that the warning is not problematic.


Fixed Previews and Added Contacts List UI

Quick Description of What this is about

I resolved issues with Screens not being previewed in Android Studio
Added new Contacts List UI based on #53

BottomSheet(Old)
Navigation(New)

@esthcarelle, I think this PR might disrupt some of the changes you made on #53 , Please have a look and let me know what you think I should Change

@cedricbahirwe I am wondering what was the point of working on my ticket ,If you already had it working and Re re re factored in this PR :O ?

Not, I did not have it working, I just found out this article: https://betterprogramming.pub/jetpack-compose-request-permissions-in-two-ways-fd81c4a702c, and I thought it was less complex than what we currently have,