Please add Honolulu, Hi!

DISTRICT 1: Kymberly Marcos Pine<EMAIL_ADDRESS>DISTRICT 2: Heidi Tsuneyoshi<EMAIL_ADDRESS>DISTRICT 3: Ikaika Anderson<EMAIL_ADDRESS>DISTRICT 4: Tommy Waters<EMAIL_ADDRESS>DISTRICT 5: Ann Kobayashi<EMAIL_ADDRESS>DISTRICT 6: Carol Fukunaga<EMAIL_ADDRESS>DISTRICT 7: Joey Manahan<EMAIL_ADDRESS>DISTRICT 8: Brandon Elefante<EMAIL_ADDRESS>DISTRICT 9: Ron Menor<EMAIL_ADDRESS>

@cstrouse thank you for providing the list of recipients. Unfortunately, we will need a bit more information in order to proceed with this request. @mayumiishikawa, I'm going to close this one out, but please reopen the issue with an email, subject, and recipients (or the ones that @cstrouse has provided) and we will proceed forward!


Some Kotlin MPP publication fixes

I realized one problem with Default vs Jvm configuration for Multiplatform projects. The if conditional I put was not really the correct way.
Plus, the pom configuration like (scm, developers, license) were removed.
Approach w/ this PR
Create Default publication explicitly only for non-KMP (regular) modules
Keep pom configuration for all MavenPublications
With this change, calling publishAllPublicationsToBintrayRepository will call Default one for all modules except for the KMP one (apollo-api). In KMP modules, it will run all other publish tasks (no Default). Here is the dry-run output
For apollo-compiler
:apollo-compiler:jar SKIPPED
:apollo-compiler:generateMetadataFileForDefaultPublication SKIPPED
:apollo-compiler:generatePomFileForDefaultPublication SKIPPED
:apollo-compiler:javadoc SKIPPED
:apollo-compiler:javadocJar SKIPPED
:apollo-compiler:sourcesJar SKIPPED
:apollo-compiler:publishDefaultPublicationToBintrayRepository SKIPPED
:apollo-compiler:publishAllPublicationsToBintrayRepository SKIPPED

For apollo-api (notice no Default anymore)
:apollo-api:compileKotlinIos SKIPPED
:apollo-api:generateMetadataFileForIosPublication SKIPPED
:apollo-api:generatePomFileForIosPublication SKIPPED
:apollo-api:iosSourcesJar SKIPPED
:apollo-api:publishIosPublicationToBintrayRepository SKIPPED
:apollo-api:compileKotlinIosSim SKIPPED
:apollo-api:generateMetadataFileForIosSimPublication SKIPPED
:apollo-api:generatePomFileForIosSimPublication SKIPPED
:apollo-api:iosSimSourcesJar SKIPPED
:apollo-api:publishIosSimPublicationToBintrayRepository SKIPPED
:apollo-api:compileKotlinJvm SKIPPED
:apollo-api:compileJava SKIPPED
:apollo-api:jvmProcessResources SKIPPED
:apollo-api:jvmMainClasses SKIPPED
:apollo-api:jvmJar SKIPPED
:apollo-api:generateMetadataFileForJvmPublication SKIPPED
:apollo-api:generatePomFileForJvmPublication SKIPPED
:apollo-api:processResources SKIPPED
:apollo-api:classes SKIPPED
:apollo-api:javadoc SKIPPED
:apollo-api:javadocJar SKIPPED
:apollo-api:jvmSourcesJar SKIPPED
:apollo-api:publishJvmPublicationToBintrayRepository SKIPPED
:apollo-api:generateMetadataFileForKotlinMultiplatformPublication SKIPPED
:apollo-api:generatePomFileForKotlinMultiplatformPublication SKIPPED
:apollo-api:publishKotlinMultiplatformPublicationToBintrayRepository SKIPPED
:apollo-api:compileKotlinMetadata SKIPPED
:apollo-api:metadataMainClasses SKIPPED
:apollo-api:metadataJar SKIPPED
:apollo-api:generateMetadataFileForMetadataPublication SKIPPED
:apollo-api:generatePomFileForMetadataPublication SKIPPED
:apollo-api:metadataSourcesJar SKIPPED
:apollo-api:publishMetadataPublicationToBintrayRepository SKIPPED
:apollo-api:publishAllPublicationsToBintrayRepository SKIPPED

Now the error in tests look like this:
A problem occurred configuring root project 'testProject'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not resolve com.apollographql.apollo:apollo-api:1.4.0-SNAPSHOT.
     Required by:
         project : > com.apollographql.apollo:apollo-gradle-plugin:1.4.0-SNAPSHOT:20200316.212313-24 > com.apollographql.apollo:apollo-compiler:1.4.0-SNAPSHOT:20200316.212313-23
      > Unable to find a matching variant of com.apollographql.apollo:apollo-api:1.4.0-SNAPSHOT:20200316.212313-30:
          - Variant 'metadata-api' capability com.apollographql.apollo:apollo-api:1.4.0-SNAPSHOT:
              - Incompatible attribute:
                  - Required org.gradle.usage 'java-runtime' and found incompatible value 'kotlin-api'.
              - Other attributes:
                  - Required org.gradle.dependency.bundling 'external' but no value provided.
                  - Required org.gradle.jvm.version '8' but no value provided.
                  - Required org.gradle.libraryelements 'jar' but no value provided.
                  - Found org.gradle.status 'integration' but wasn't required.
                  - Found org.jetbrains.kotlin.platform.type 'common' but wasn't required.

So, when I tried in an iOS project, it was failing because it couldn't find iOS artifacts. I will try now with the latest SNAPSHOT.


Consider storing hashing functions for ranges of chunks instead of chunk references in metadata files

As I understand it, Icechunk current stores references to chunks with something like the following:

Chunk Index
Reference

0
3abc21

1
c14fe1

2
537c21

This will struggle to scale for very large datasets like those in ARCO-ERA5, which use 10s of millions of chunks.
Instead, what about storing records of writes with corresponding ranges of chunks, e.g.,

Chunk start
Chunk stop
Version

0
10000
v1

10000
11000
v2

Then raw chunks could be stored at /{version}/{chunk_id} or /{version}/{hash(chunk_id)}. This would keep metadata files quite small, so they could still be loaded into memory even for very large stores.
I think this would work well for the typical update patterns we encounter for weather/climate data, where we have O(1) writes per day, and which update contiguous ranges of chunks, e.g., "append one weather forecast" or "update the last few days of data with additional quality controls"

@shoyer I really like this approach, and I'd encourage people to continue this line of thinking. But there are a few details I want to bring up.
The complex scenario on writes is when you have distributed writers, writing concurrently on multiple different sessions. This would be a scenario where two or more users start different distributed writes, using, for example, the same parent snapshot_id.
In this scenario, we cannot let the written chunks overwrite each other, each writter needs the guarantee that their chunks will be safe in object store before the commit starts. So, as a minimum, we would need to give each writer a "safe" prefix (or hash function, encoding, etc.) to put their chunks in.
Then, at commit time, some of those chunks will need to be pointed from the manifest, and we don't initially know which writer writes which chunks, so we need to store it in the manifest.
One trivial way to do that would be to make the manifest a table like (I'm skipping over several details, like the node id, for clarity)

Chunk coord
Writer id or prefix or hash

0/0/0
abcd

0/0/1
ef01

0/0/2
2345

That is in fact a small optimization, because writer id would neew fewer bytes than chunk id. But not a huge optimization, we still need to list chunks.
Other approach would be to try to create an efficient way to encode the list of chunks coordinates that a writer wrote. With that we would go from one row per chunk to one row per chunk range. This is trivial for certain patterns of distributed writes, but not for the general case.
I wanted to add two extra pieces of context.

Each chunk id is 12 bytes, and we'll have some padding for alignment, and some inefficiencies, we have done no work to optimize this yet. Let's say 20 bytes. That's pretty small, a few hundred million chunks would fit in memory comfortably.
But, more relevant to your append weather data use case:

All the chunks for an array don't need to fit in a single manifest, we will split them
As you write new data at higher values of the time dimension, those chunks will go to new manifests as the previous ones grow beyond certain limit.
When you read data, if you read recent times, as is usually the case, you don't need to fetch and load every manifest, only the ones that contain chunks you need.
Also, we can optimize offline how the chunks are distributed across manifests. This can be done based on many different inputs:

You could think there is a process that searches clusters of initialized chunks and gathers them in the same manifest. This way, geographically constrained queries would need fewer manifests.
We could do the same on the time dimensions, or in time and space.
We could also optimize based on query statistics, and gather together in a manifest chunks that are usually queried together.


multipart response example/feature expansion?

i'm not sure how to handle redirects after a posted form fails my validation.  i have a sequence of error objects i need to respond with, preferably with posted input, along with a failure/success message (from session?), but i'm not sure how i'm supposed to do all that. All i see is a body string just like in the request, and i'm not sure how to use that for the above detailed needs. I wouldn't have known how to parse that body string from the request without that multipart example either. :)
ps. BTW, I used the multipart example even though i didn't need to upload a file, as i wasn't sure if a regular url-encoded form post request was supported, or how to handle it.
thanks

i'm going to close this for now, as i forgot/didn't realize that both form validation errors and old input should probably just be flashed to the session and retrieved from there, and would be outside the scope of the response object.

I need to set up an example for regular form-encoded bodies. It is a todo list item. Multipart is imo best avoided unless you need file uploads, though there is no harm in using it, it is just more complex under the hood.


Setup continuous deployment

I was thinking it might be a good idea to automate the process of deploying the website to Github Pages. I'm thinking that whenever a commit is made on the master branch, then a Github Action will run that will build and push to the gh-pages branch.

That would be great.


Two Step Loaders

Fixes #1011

Can we add a test or a demo view config that has a data URI as the URL for a JSON file? I dont see why it would cause an issue, but it is kind of an edge case, and it is still used in the portal, right?

Can we add a test or a demo view config that has a data URI as the URL for a JSON file? I dont see why it would cause an issue, but it is kind of an edge case, and it is still used in the portal, right?
Update: never mind, I see that the raster.json loader already kind of does this

I believe the Cypress OME test already does this, using options not url for raster.json


Title attribute broken

This is a regression from https://github.com/qTip2/qTip2/commit/e4d59b0b5d32d73e9231ebf910fbead056ed42e8
The problem is that the parenthesis for invalidContent were improperly removed, causing string types to start returning true.

Minimal repro is to make a qtip with {content: {text: 'some text', title: 'some title'}}

Fixed as of https://github.com/qTip2/qTip2/commit/727174cf1592beed782f9ed41fc3a5a3bee69a74


Inner and outer product

Hi
G5 has chosen inner and outer product of vectors

Thank you for letting us know. @yasmensarhan27
G4 has chosen the return of a diagonal.


Better looking pointer

Instead of a crude circle, this makes it look more like a pointer. I manually tweaked the numbers until it looked like a pointer :)

I think we should take the opportunity and assign each user a random color, because the pointers will be confusing quick:

Or leave it for another PR.

You read my mind, PR for that incoming ;)


[twitter:spaces] Invalid data found when processing input

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
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
Japan
Provide a description that is worded well enough to be understood
When trying to download a twitter spaces audio, I encountered the error "Invalid data found when processing input".
I tried several times, but I got this same error.
I was running yt-dlp version 2<IP_ADDRESS>006.
I would be grateful if someone would fix this issue.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] 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] Command-line config: ['-vU', '-f', 'ba', '-o', '%(uploader_id)s_%(upload_date)s_%(title)s_%(id)s.%(ext)s', '-P', '.', '--netrc', 'https://twitter.com/i/spaces/1eaKbrQbjoRKX']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[92315c037] (zip)
[debug] Python 3.10.4 (CPython arm64 64bit) - macOS-13.4-arm64-arm-64bit (OpenSSL 1.1.1u  30 May 2023)
[debug] exe versions: ffmpeg 6.0 (setts), ffprobe 6.0
[debug] Optional libraries: certifi-2021.10.08, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 938b91103cd7dc7544ea4eadd44d8488ec66e0ab237d0aee211baa5244b16fb9
yt-dlp is up to date<EMAIL_ADDRESS>[twitter:spaces] Downloading login page
[twitter:spaces] Downloading flow token
[twitter:spaces] Submitting JS instrumentation response
[twitter:spaces] Submitting username
[twitter:spaces] Submitting password
[twitter:spaces] Submitting account duplication check
[twitter:spaces] Logging in
[twitter:spaces] Extracting URL: https://twitter.com/i/spaces/1eaKbrQbjoRKX
[twitter:spaces] 1eaKbrQbjoRKX: Downloading GraphQL JSON
[twitter:spaces] 28_1664225270709256192: Downloading legacy API JSON
[twitter:spaces] 28_1664225270709256192: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] 1eaKbrQbjoRKX: Downloading 1 format(s): 0
[debug] Invoking hlsnative downloader on "https://prod-fastly-ap-northeast-1.video.pscp.tv/Transcoding/v1/hls/uoJ7s877sh5vLzDFNKzftjMMb1_Jd7m5i-ePKC63ZK0LnMLjztwQQCNGMZu7WHDdBYkKHM7slVLUMtGLkQhxvw/non_transcode/ap-northeast-1/periscope-replay-direct-prod-ap-northeast-1-public/audio-space/playlist_16761126492889887124.m3u8?type=replay"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 125
[download] Destination: ./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a
[download] 100% of    4.46MiB in 00:00:06 at 705.21KiB/s
[FixupM4a] Correcting container of "./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a' -map 0 -dn -ignore_unknown -c copy -f mp4 -movflags +faststart 'file:./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.temp.m4a'
[debug] ffmpeg version 6.0 Copyright (c) 2000-2023 the FFmpeg developers
  built with Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
  configuration: --prefix=/opt/homebrew/Cellar/ffmpeg/6.0 --enable-shared --enable-pthreads --enable-version3 --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libaribb24 --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librist --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libsvtav1 --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libspeex --enable-libsoxr --enable-libzmq --enable-libzimg --disable-libjack --disable-indev=jack --enable-videotoolbox --enable-audiotoolbox --enable-neon
  libavutil      58.  2.100 / 58.  2.100
  libavcodec     60.  3.100 / 60.  3.100
  libavformat    60.  3.100 / 60.  3.100
  libavdevice    60.  1.100 / 60.  1.100
  libavfilter     9.  3.100 /  9.  3.100
  libswscale      7.  1.100 /  7.  1.100
  libswresample   4. 10.100 /  4. 10.100
  libpostproc    57.  1.100 / 57.  1.100
[mov,mp4,m4a,3gp,3g2,mj2 @ 0x124633930] Format mov,mp4,m4a,3gp,3g2,mj2 detected only with low score of 1, misdetection possible!
[mov,mp4,m4a,3gp,3g2,mj2 @ 0x124633930] moov atom not found
file:./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a: Invalid data found when processing input

ERROR: Postprocessing: file:./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a: Invalid data found when processing input
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3439, in process_info
    replace_info_dict(self.post_process(dl_filename, info_dict, files_to_move))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3621, in post_process
    info = self.run_all_pps('post_process', info, additional_pps=info.get('__postprocessors'))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3603, in run_all_pps
    info = self.run_pp(pp, info)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3581, in run_pp
    files_to_delete, infodict = pp.run(infodict)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/common.py", line 24, in run
    ret = func(self, info, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/common.py", line 129, in wrapper
    return func(self, info)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/ffmpeg.py", line 882, in run
    self._fixup('Correcting container', info['filepath'], [*self.stream_copy_opts(), '-f', 'mp4'])
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/ffmpeg.py", line 863, in _fixup
    self.run_ffmpeg(filename, temp_filename, options)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/ffmpeg.py", line 374, in run_ffmpeg
    return self.run_ffmpeg_multiple_files([path], out_path, opts, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/ffmpeg.py", line 329, in run_ffmpeg_multiple_files
    return self.real_run_ffmpeg(
  File "/usr/local/bin/yt-dlp/yt_dlp/postprocessor/ffmpeg.py", line 367, in real_run_ffmpeg
    raise FFmpegPostProcessorError(stderr.strip().splitlines()[-1])
yt_dlp.postprocessor.ffmpeg.FFmpegPostProcessorError: file:./tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a: Invalid data found when processing input

same issue here but youtube video

@PikuZheng open a new issue with verbose log please

Could you pls give output of ffprobe tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a

I think the 'container': 'm4a_dash' hack doesn't work for every stream. Need to revert to forcing ffmpeg for all spaces downloads, even if not live

I have to rename the file to .aac in order for ffprobe to read anything, and then I get:
[aac @ 0x55e120cbac40] Format aac detected only with low score of 1, misdetection possible!
[aac @ 0x55e120cbac40] Estimating duration from bitrate, this may be inaccurate
Input #0, aac, from '20230601_tomeru_ikinone_1eaKbrQbjoRKX__.aac':
  Metadata:
    HydraParticipants: []
    TIT3            :<PHONE_NUMBER>.284
    JSONMetadata    : {"HydraVersion":4,"ntp":3894606003.284}
    id3v2_priv.com.apple.streaming.transportStreamTimestamp: \x00\x00\x00\x00\x86\xa9\x9d\xfa
    HydraAudioLevel : [46]
  Duration: 00:06:22.18, bitrate: 97 kb/s
  Stream #0:0: Audio: aac (LC), 48000 Hz, stereo, fltp, 97 kb/s

@pukkandan
Thank you for your reply.

Could you pls give output of ffprobe tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a.

Output of ffprobe tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a is as below.
ffprobe version 6.0 Copyright (c) 2007-2023 the FFmpeg developers
  built with Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
  configuration: --prefix=/opt/homebrew/Cellar/ffmpeg/6.0 --enable-shared --enable-pthreads --enable-version3 --cc=clang --host-cflags= --host-ldflags= --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libaribb24 --enable-libbluray --enable-libdav1d --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librist --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libsvtav1 --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libspeex --enable-libsoxr --enable-libzmq --enable-libzimg --disable-libjack --disable-indev=jack --enable-videotoolbox --enable-audiotoolbox --enable-neon
  libavutil      58.  2.100 / 58.  2.100
  libavcodec     60.  3.100 / 60.  3.100
  libavformat    60.  3.100 / 60.  3.100
  libavdevice    60.  1.100 / 60.  1.100
  libavfilter     9.  3.100 /  9.  3.100
  libswscale      7.  1.100 /  7.  1.100
  libswresample   4. 10.100 /  4. 10.100
  libpostproc    57.  1.100 / 57.  1.100
[mov,mp4,m4a,3gp,3g2,mj2 @ 0x145705490] Format mov,mp4,m4a,3gp,3g2,mj2 detected only with low score of 1, misdetection possible!
[mov,mp4,m4a,3gp,3g2,mj2 @ 0x145705490] moov atom not found
tomeru_ikinone_20230601_あ_1eaKbrQbjoRKX.m4a: Invalid data found when processing input

Also, is the file playable?

Yes. I could play the file with Elmedia Player.

@bashonly
I run the command with--downloader ffmpeg and no error occurred. Thank you.

@miminashii @bashonly can you write a complete guide on how to download twitter spaces now? it seems you guys know what to do but i dont see it documented anywhere. this is what i get right now

@swyxio
You're running an old version, and now you need to be logged in to access Spaces (yt-dlp can't workaround this)

Update yt-dlp, yt-dlp -U
pass cookies with your command, e.g. --cookies-from-browser firefox, or use --username and `--password to log in
add --downloader ffmpeg to your command to counteract #7536


gnuplot

Is possible to add a gnuplot output direct to terminal?

First you should maybe set up the shell integration or at least make sure you can run the show command. See http://extraterm.org/guide.html#shell-integration
Once you have show you can use it to show an image file in the terminal or pipe an image file into it. I haven't used gnuplot myself, but you could write a small script to run it and then immediately run show to display the image.

no show image but show .plot source code

show will not convert .plot to an image. You have to use gnuplot first to make and image and then you can use show to show it.

If gnuplot is installed, I guess this wrapper would work?
https://www.npmjs.com/package/gnuplot

@nigels-com With regards to displaying different file formats inside Extraterm, the idea is to have different viewers. There already are a number of them but they need more work, also the module/extension system needs to be more developed before it is easy to support something like gnuplot files directly.
This issue does demonstrate the usefulness of having lighter weight "filters" which can convert a file to a more common file format (e.g. a PNG image) for display by the built in a image viewer.


Add unit tests

Coverage increased (+0.7%) to 93.333% when pulling c800d9af361455f44bcb32cfd55228cc4b6bfbad on maretskiy:add-unit-tests into 7e4baa6cd04ca027d82aa7bf2d935bae5150efde on seecloud:master.


[Snyk] Upgrade lit from 2.7.2 to 2.7.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade lit from 2.7.2 to 2.7.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.
The recommended version was released a month ago, on 2023-06-01.

Release notes

Package name: lit

2.7.5 - 2023-06-01

2.7.4 - 2023-05-04

2.7.3 - 2023-04-26

2.7.2 - 2023-04-05

    from lit GitHub release notes

Commit messages

Package name: lit

06288ad Version Packages (#3932)
c3672fc [labs/virtualizer] Add support methods to work around ResizeObserver loop limit exceeded errors (#3929)
5d68be3 [infra] Add thank you to community authored changes. (#3921)
bfbf1c7 Bump json5 from 2.2.0 to 2.2.3 in /packages/benchmarks/generator (#3547)
c836b23 Bump jszip from 3.6.0 to 3.10.1 in /packages/benchmarks/generator (#3317)
7cb2bb2 Bump minimatch from 3.0.4 to 3.1.2 in /packages/benchmarks/generator (#3487)
c77417b Bump qs from 6.10.1 to 6.11.2 in /packages/benchmarks/generator (#3927)
c811835 Bump @ xmldom/xmldom from 0.7.5 to 0.7.11 in /packages/labs/cli-localize (#3928)
3bafeb7 Bump ansi-regex from 4.1.0 to 4.1.1 in /packages/benchmarks/generator (#3097)
ef318cd Bump terser from 4.8.0 to 4.8.1 in /packages/benchmarks/generator (#3153)
6755c70 Bump minimist from 1.2.5 to 1.2.8 in /packages/benchmarks/generator (#3701)
a7e17ad [labs/context] Make `@ provide` and `@ consume` support optional and private properties. (#3733)
feded34 [labs/virtualizer] Removing some crufty code in ScrollerController (#3909)
cc72139 Replace $` with code block to fix changelog (#3922)
5bfe62a fix(labs/gen-utils): js highlight extension link (#3880)
f6387e3 [reactive-element] Make decorators accept `ReactiveElement` from elsewhere (#3917)
52f1632 Bump webpack from 5.75.0 to 5.76.3 (#3760)
5f1ec31 Bump nunjucks from 3.2.3 to 3.2.4 (#3824)
33ba3ea Bump yaml from 2.2.1 to 2.2.2 (#3840)
0312e86 Revert "[reactive-element] Reusing HasChanged type" (#3916)
f8d7285 [labs/ssr] Allow rendering of non-array iterables (#3905)
5f44d4e Minor change for internal linter (#3915)
440890b [reactive-element] Reuse HasChanged type (#3864)
7932f7d [labs/react & labs/ssr-react] Coordinate props for server rendering and hydration (#3885)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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


Fix for exit code problems

Currently when Cerberus is not returning a correct exit code based on the violations it identifies. As a result of this, it was quite hard for developers to consume Cerberus in their build pipeline . This PR reduces the extra effort that our consumers are currently putting in parsing the output of Cerberus to fail or pass the build.
This was first expressed during a demo and was captured as an issue
https://github.com/philips-software/cerberus/issues/35

If all the commands now return the no. of violations encountered by them, this deviates from the original behaviour of always returning zero. Should we upgrade the version of Cerberus from 1.x.x to 2.x.x?

@dsvellal  We can do a major release, as Cerberus5.0 not a micro release, I do not see a problem. But currently AFAIK no one was relying on exit code since it was always success, we can now do an announcement in the release .


Move codegen factories into RedwoodExtension

[ ] CHANGELOG.md's "Unreleased" section has been updated, if applicable.

This branch is like two weeks old. Not sure why I didn't send it first time around.


My CAS Stop at forceauthentication

iam trying to use phpcas lib for codeigniter https://github.com/eliasdorneles/code-ig...as-library
but when iam trying to testing it on controller it stuck on success login view and its doesnt redirect back to my apps.
here is the log and the code
code:

$this->load->library('cas');
  $this->cas->force_auth();
  $user = $this->cas->user();
  print_r($this->cas);

  print_r($user);

log:
C0C1 .START phpCAS-1.1.3 ****************** [CAS.php:470]
C0C1 .=> phpCAS::client('2.0', 'sso.xxxx.com', 443, '/casadfs') [Cas.php:55]
C0C1 .| => CASClient::CASClient('2.0', false, 'sso.xxxx.com', 443, '/casadfs', true) [CAS.php:373]
C0C1 .| | Starting a new session [client.php:600]
C0C1 .| <= ''
C0C1 .<= ''
C0C1 .=> phpCAS:ConfusedetNoCasServerValidation() [Cas.php:59]
C0C1 .<= ''
C0C1 .=> phpCAS::forceAuthentication() [Cas.php:74]
C0C1 .| => CASClient::forceAuthentication() [CAS.php:893]
C0C1 .| | => CASClient::isAuthenticated() [client.php:869]
C0C1 .| | | => CASClient::wasPreviouslyAuthenticated() [client.php:975]
C0C1 .| | | | no user found [client.php:1111]
C0C1 .| | | <= false
C0C1 .| | | no ticket found [client.php:1028]
C0C1 .| | <= false
C0C1 .| | => CASClient::redirectToCas(false) [client.php:878]
C0C1 .| | | => CASClient::getServerLoginURL(false, false) [client.php:1128]
C0C1 .| | | | => CASClient::getURL() [client.php:361]
C0C1 .| | | | | Final URI: http://localhost/sbis/trunk/src/app/login/testSSO2 [client.php:2686]
C0C1 .| | | | <= 'http://localhost/sbis/trunk/src/app/login/testSSO2'
C0C1 .| | | <= 'https://sso.xxxx.com/casadfs/login?service=http%3A%2F%2Flocalhost%2Fsbis%2Ftrunk%2Fsrc%2Fapp%2Flogin%2FtestSSO2'
C0C1 .| | | Redirect to : https://sso.xxxx.com/casadfs/login?service=http%3A%2F%2Flocalhost%2Fsbis%2Ftrunk%2Fsrc%2Fapp%2Flogin%2FtestSSO2
C0C1 .| | | exit()
C0C1 .| | | -
C0C1 .| | -
C0C1 .| -
iam stuck at this log the CAS didnt print my $user = $this->cas->user();

Please try some of our examples and report any errors. We have little change to remotely debug any custom code on top of our library....
It looks like your user is redirected to the cas server but never "returns" with any ticket....

i have tried using your example https://github.com/apereo/phpCAS/tree/master/docs/examples
the result is same iam stuck at success notification

Log In Successful
You have successfully logged into the Central Authentication Service.

For security reasons, please Log Out and Exit your web browser when you are done accessing services that require authentication!

it doesnt redirect back to my example web but when i check the cookies i got two cookies SESSION and CASTGC. CASTGC is the ticket right?
other apps working fine with this CAS Server but i cant get the CAS setup/mod if there was any mod because there's no documentation on other apps CAS client.

the server CAS changing the host thats why its become an error


refactor: Move aws to topic folder

Task:

Move aws folder to topic folder
Update the https://github.com/tungbq/devops-basic#topic table to point to new location of aws

Similar task with #118, and PR #125

Done


test: Update criterion 0.3 -> 0.4

This fixes the following warning when running "cargo test":
warning: the following packages contain code that will be rejected by a future version of Rust: criterion v0.3.5

Nice! 🚀


addition of pgaudit workflow

Description
Adding a new github action workflow to test JDBC tests after enabling pgaudit extension.
Engine changes PR link: https://github.com/babelfish-for-postgresql/postgresql_modified_for_babelfish/pull/441
Issues Resolved
Issues while testing of JDBC tests with pgaudit extension enabled.

[BABEL-5269]:

Problem: Some crashes and diff files with the error as undefined objectId or classId.
RCA: Main issue was to over-write the address for the case T_CreateTableAsStmt. In this case we are setting the address via bbfSelectIntoUtility_hook and passing it further from there only.
But due to improper check we are also over-writing that address with a random initialised variable in ProcessUtilitySlow.
Fix is to pass the correct address once, so we can set address in bbfSelectIntoUtility_hook but instead of passing it there we can set the pointer to address which is allotted in hook and pass that address in ProcessUtilitySlow function itself.

[BABEL-5233]:

Problem: there were some files which were giving diff file upon running with pgaudit as: "ERROR (Message: cache lookup failed for relation 24553)".
RCA: we were not initialising the cache when we enter ENRgetSystableScan() function for table object type.
Fix is to change the dialect to tsql and then initialise the cache and then again change the dialect to earlier dialect via introducing a hook.

[BABEL-5281]:

Problem: Diff files with the error unexpected command tag "???"
RCA: mainly this issue arises when we call the identity() function via select into query. Upon debugging found that we
were not setting the object type while making the identity node (AlterTableStmt).
Fix was to set the object type to object_table while making the identity node.

Test Scenarios Covered
List
[Pranav Jain] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
44639

Relevant Lines:
60145

💛  - Coveralls

Do we really need to run PGAudit With all the tests???

Do we really need to run PGAudit With every PR??? Would really like this to be a manual workflow. or a scheduled one.

@shah-nirmit This will most likely be a schedule one.

Please add the engine and extension PR links in description.

Also as a follow up we should discuss whether  to add support to install pgAudit extension via sp_execute_postgresql


Remove version: GeoGebra.GraphingCalculator version 6.0.861

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://download.geogebra.org/installers/6.0/graphing/GeoGebraGraphing-Windows-Installer-6-0-861-0.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 923)

URL: https://download.geogebra.org/installers/6.0/graphing/GeoGebraGraphing-Windows-Installer-6-0-861-0.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 923)

URL: https://download.geogebra.org/installers/6.0/graphing/GeoGebraGraphing-Windows-Installer-6-0-861-0.exe
Response status code does not indicate success: 404 (Not Found).
(Automated message - build 900)

[Policy] Reset Labels
[Policy] Reset Feedback

@wingetbot run


{'success': False, 'code': 602, 'message': 'Signature verification failed!'} #1

The futures API works fine for some private methods, e.g. 'open_positions'.
However, when using other methods, e.g. 'assets' or 'cancel_order' I get the error described in the header.
I gave my API key all available rights.

check this -> https://github.com/makarworld/pymexc/issues/1

The MEXC Futures API is currently not operational.
Information on API availability can be found here -> https://t.me/MEXCAPIsupport


Setting permissions in Mariadb with "grant options" doesn't work.

Description
See screenshots in ticket #326891 in enterprise support.
Dbeaver generates the syntax with grant grant. It should be Grant all.
DBeaver Version
Dbevar enterprise Version <IP_ADDRESS>403110838
Operating System
Windows 11
Database and driver
Database version: 10.11.7-MariaDB-1:10.11.7+maria~deb12 on Debian 12 server
Steps to reproduce
Try giving general permission or in a database with grant options
Additional context
No response

Hello @buhlerax
Thanks for the bug report.
A ticket was created on the internal board, and this case will be solved there.


Some Static controls in a dialog cannot be specified, but print_control_identifiers can print them out.

Some Static controls in a dialog cannot be specified, even if print_control_identifiers can print them out.
But the button controls on the same dialog can be specified.
Why?
the error massage:
MatchError: Could not find 'Static2' in 'dict_keys(['Button', 'OK', 'OKButton', 'Button0', 'Button1', 'Button2', 'CancelButton', 'Cancel'])'

When print_control_identifiers , the different about these 2 categories of contols is the Static controls have ''(empty) titles or strange title such as 'title1 \r\n title2 \r\n'.

I think I found the reason, the static control's visible maybe be set to False, because I
use child_window(title_re='^title.*', visible_only=False) can get the control's identifier.
The stranger is I can see the static control on the window, even if it is set invisible.


Selection not visible in code blocks

The selected text color is the same as the code block background color:

This makes it very difficult to select text in a code block because there's no visual feedback.

I thought I had fixed this in the release of November 12th. @icholy Could you update your theme (Options > Appearance > Themes > Manage), just to make sure that you're on the newest version?

I updated and am still experiencing this issue.

Could you

activate the default theme,
close Obsidian,
delete .obsidian/themes/Solarized.css in your Vault,
restart Obsidian,
select Solarized again?

I did that and I even tried directly copying the css from the repo into .obsidian/themes/Solarized.css. The selection color and block color are still the same for me.

FWIW, I worked around this issue by adding
:root {
    /* ... */
    --base4: #e6dfca;
}

.theme-light {
    /* ... */
    --text-selection: var(--base4);
}

So that selected text is a different color than code blocks.

That's so weird. Is this in legacy editor, live preview or reading mode? And does that code block have a syntax applied? So
```json

instead of
```

?

This is what a selection in reading mode should look like:

Ah, that's it. It only happens in live preview mode.

Can confirm, it also happens to me, but only in edit mode. I have seemingly the latest version (there does no appear to be a way to check, but I updated via obsidian theme settings and then restarted the editor with no changes).

I've enabled live editor in my Obsidian to test this and can confirm what you were seeing. I've added a selector specifically for selections in live editor mode. Could you check if it works now?

@harmtemolder that fixed it. Thanks!

@icholy Thanks for confirming (and for reporting the issue in the first place)


UNSUPPORTED device HmIp-SMI55-2

Description of problem:
When adding the HomematicIP indoor motion sensor for 55 light switch frames to the CCU3, no entity is created in Home Assistant.
There should be support for the "HmIp-SMI55", but I noticed, the description of the device I received today has the name "HmIp-SMI55-2". I guess the device is the same as the "HmIp-SMI55" but with a "-2" at the end. Maybe we just need another definition for "HmIp-SMI55-2"?
Traceback or log (if applicable):
2021-11-13 21:57:38 DEBUG (SyncWorker_0) [pyhomematic.devicetypes.generic] HMGeneric.__init__: device_description: 00319D89993A72 : {'TYPE': 'HmIP-SMI55-2', 'SUBTYPE': 'SMI55', 'ADDRESS': '00319D89993A72', 'RF_ADDRESS': 5619087, 'CHILDREN': ['00319D89993A72:0', '00319D89993A72:1', '00319D89993A72:2', '00319D89993A72:3', '00319D89993A72:4'], 'PARENT': '', 'PARENT_TYPE': '', 'INDEX': 0, 'AES_ACTIVE': 1, 'PARAMSETS': ['MASTER', 'SERVICE'], 'FIRMWARE': '1.2.0', 'AVAILABLE_FIRMWARE': '0.0.0', 'UPDATABLE': True, 'FIRMWARE_UPDATE_STATE': 'UP_TO_DATE', 'VERSION': 4, 'FLAGS': 1, 'LINK_SOURCE_ROLES': '', 'LINK_TARGET_ROLES': '', 'DIRECTION': 0, 'GROUP': '', 'TEAM': '', 'TEAM_TAG': '', 'TEAM_CHANNELS': [], 'INTERFACE': '', 'ROAMING': 0, 'RX_MODE': 0}
2021-11-13 21:57:38 DEBUG (SyncWorker_0) [pyhomematic._hm] RPCFunctions.createDeviceObjects: created 00319D89993A72 as UNSUPPORTED device for HmIP-SMI55-2
2021-11-13 21:57:38 DEBUG (SyncWorker_0) [pyhomematic._hm] RPCFunctions.createDeviceObjects: adding to self.devices_all
2021-11-13 21:57:38 DEBUG (SyncWorker_0) [pyhomematic._hm] RPCFunctions.createDeviceObjects: adding to self.devices

Home Assistant version (if applicable):
2021.11.3
Problem-relevant configuration.yaml entries and steps to reproduce:
homematic:
  interfaces:
    hmip:
      host: <IP_ADDRESS>
      port: 2010
      resolvenames: "json"
      username: !secret user_homematic
      password: !secret password_homematic
    groups:
      host: <IP_ADDRESS>
      port: 9292
      resolvenames: json
      username: !secret user_homematic
      password: !secret password_homematic
      path: /groups
  hosts:
    ccu3:
      host: <IP_ADDRESS>
      username: !secret user_homematic
      password: !secret password_homematic

Setup HomematicIP Integration with Home Assistant
Add device HmIP-SMI55-2 to the CCU3
Device is not shown as entity in Home Assistant.

Here is the GIST-paste of the device attributes: https://gist.github.com/Ex0danify/546353cbba9725a387151e5536d12053

I compared the current code for HmIP-SMI55 with the device parameters you provided and the "new" HmIP-SMI55-2 device seems to be identical to the existing one. The only parameter I couldn't find was PRESS_SHORT from the HelperEventRemote class. Instead there is PRESS_LONG_START. I'm not sure though if the original HmIP-SMI55 device has that parameter and if it even matters. Maybe @danielperna84 can shed some light on this.

When will this change be available in Ha?

@Meldel007
Probably the next HA release in February.


A11y_AzureToolsforVSCode_AzureFunction_Selectalocationfornewresources_ScreenReader: The NVDA announcing incorrect list markup for "Recommended" and "Other" list items, when control is in collapsed state.

Actual:
When the "Recommended" and "Other" list items are in collapsed state, the NVDA announces incorrect list markup. For "Recommended" list item it announces "chevron-right Recommended 3 of 3" and for "chevron-right Other 2 of 3". But there are only 2 list is present when all controls are in collapsed state.
This issue is also observed with JAWS.
Expected:
The NVDA should announce correct list markup. For "Recommended" list item it announces "Recommended 1 of 2 collapsed " and for "Other 2 of 2 collapsed"
GitHubTags:#A11yMAS;#A11yTCS;BM_AzureToolsforVSCode_Win32_Aug2021;#Azure Tools for VS
Code;#A11ySev2;#WCAG1.3.1;#DesktopApp;# Win10;#NVDA;#JAWS;#Element:Header;
Environment Details:
Visual Studio Code
Azure Function
Operating System: Windows 10 Pro 21H1
Screen Reader: NVDA(2021.1),JAWS(2021.2107.12 ILM)
Steps to Reproduce:

Launch the Visual studio code.
TAB to on "Extension" button and search for "Azure Functions" and select it using ENTER key.
"Azure Function" screen should open. TAB to "Install" button and hit ENTER key to install it.
"Azure" button will appear in navigation pane. TAB to it and select it.
Start NVDA.
TAB to any Subscription and expand options using Shift+F10 then TAB to "Create Function App in Azure" control and select it.
Edit some name for new function and hit ENTER key, then select "Powershell 7.0" as a "Runtime stack" and hit ENTER key.
"Select a location for new resources" edit control will appear. TAB to "Recommended" list item and collapsed it using ENTER key if the control is in expanded state and do same for the "Other" list item.
Observe the announcement by screen reader for the "Recommended" list item.

User Impact:
The screen reader users will get incorrect information about the number of list items.
“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”

The existing grouping is a custom implementation for our extension. We want to use VS Code's grouping, which should be fully accessible, but unfortunately it's not exposed to extensions yet. Tracking issue here: https://github.com/microsoft/vscode/issues/74967

We will track this bug using the provided tracking bug id https://github.com/microsoft/vscode/issues/74967

Fixed by https://github.com/microsoft/vscode-azuretools/pull/1165

@12-shweta The work has been merged into the vscode-azuretools repo. Can you try to install the latest vsix to verify the fix?  Please address any questions to MicroFish91 as ejizba is no longer on this project anymore.  Thanks.

Verified the bug and the issue is fixed.
Hence closing the bug.

Github Tags:#Closed;

GitHubTags:#BM_AzureToolsforVSCode_Win32_Aug2021;


git annex works?

i have installed git annex on server and client, using http://redmine-git-hosting.io/how-to/install-addons/
ok, when i push files i see error
`# git annex sync --content
commit
On branch master
Your branch is ahead of 'origin/master' by 1 commit.
(use "git push" to publish your local commits)
nothing to commit, working tree clean
ok
pull origin
ok
copy bin (checking origin...) (to origin...)
SHA256E-s42106880--b9e1d42e98d20aae7d98f34f63f175e63578c895f3ac4c74c7e2d3d777c03e38
42,106,880 100%   82.39MB/s    0:00:00 (xfr#1, to-chk=0/1)
rsync: open "/home/git/repositories/annex.git/annex/tmp/SHA256E-s42106880--b9e1d42e98d20aae7d98f34f63f175e63578c895f3ac4c74c7e2d3d777c03e38" failed: Permission denied (13)
git-annex: /home/git/repositories/annex.git/annex/objects: createDirectory: permission denied (Permission denied)
git-annex-shell: recvkey: 1 failed
rsync error: syntax or usage error (code 1) at main.c(1196) [sender=3.1.2]
rsync failed -- run git annex again to resume file transfer
failed
push origin
Counting objects: 8, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (6/6), done.
Writing objects: 100% (8/8), 1.11 KiB | 1.11 MiB/s, done.
Total 8 (delta 1), reused 0 (delta 0)
Notifying Redmine about changes to this repository : 'x-test/annex' ...

Fetching changesets for 'annex' repository ...  [success]

Looking for additional global post-receive hooks...

Global hooks found : ["hooks/post-receive.d/mail_notifications"]

Looking for additional local post-receive hooks...

No local hooks found
remote:
Calling additional post-receive hooks...
Executing extra hook 'hooks/post-receive.d/mail_notifications'
multimailhook is disabled

To ssh://<IP_ADDRESS>/annex.git
edebc0a..9aaf9ee  git-annex -> synced/git-annex
bd12ee6..ca2afd4  master -> synced/master
ok
git-annex: sync: 1 failed
`
in documentation

By default git annex sync only syncs the metadata about your files that is stored in git. It does not sync the contents of files, that are managed by git-annex. To do that, you can use git annex sync --content

How i can using redmine-git-hosting with git annex? i see annex using rsync, but ssh server denied these operation ;( i need upload big file to repo, share and storage in redmine. Standalone git give me error when i push big files.

As discussed in https://github.com/jbox-web/redmine_git_hosting/issues/732#issuecomment-532362560, issues related to Redmine < 4.0 or severely outdated issues are being closed to help clean up the issue tracker.
If this issue is still relevant to you and you are running Redmine >= 4.0, please open a new issue including all new relevant information.


Auto flipping of disoriented Mammography images on stack viewport

Context
The feature is designed to flip the images in the stack viewport, which are initially stored in a flipped orientation. The images are flipped upon loading and when the viewport is reset. It relies on ImageOrientationPatient or PatientOrientation, and ImageLaterality or FrameLaterality to identify the necessary flipping. Primarily, this feature is aimed at MG images that are stored in an incorrect orientation. It is capable of correcting images through vertical and/or horizontal flips, but it does not handle images requiring rotation for correction.

Changes & Results
The activation of the feature is handled through CustomizationService. A function that accepts the metadata instance and returns whether auto-flipping is needed or not is provided in the CustomizationService.
A sample customization
function getCustomizationModule() {
	return [
		{
			name: 'default',
			value: [
				{ id: 'orientationCorrectionCriterion', criteria: isOrientationCorrectionNeeded }
			]
		}
	]
}

A demo recording of the feature
https://github.com/user-attachments/assets/58aabeff-4529-4d12-8ad5-0c3276eacead

Testing

Checklist
PR

[x] My Pull Request title is descriptive, accurate and follows the
semantic-release format and guidelines.

Code

[x] My code has been well-documented (function documentation, inline comments,
etc.)

Public Documentation Updates

[x] The documentation page has been updated as necessary for any public API
additions or removals.

Tested Environment

[x] OS: Windows 11 Pro
[x] Node version: 18.17.0
[x] Browser: Chrome 127.0.6533.120 (Official Build) (64-bit)

Cc : @Ouwen

@sedghi let us know if there are changes needed to this PR. @Adithyan-Dinesh-Trenser i would rebase to master in the meantime

CC'ing @wayfarer3130 since he has worked on a lot of MG hanging protocols recently

@wayfarer3130
So what is the decision on this? Do we need it in OHIF upstream?


HTTP Services all 404

Hello,
This may be more of an NGINX issue, but I'm trying to proxy my internal services (just HTTP at this point)(Sonarr/Radarr/Lidarr/etc.) to my server that's running the services, but all show the 404 page.
I'm trying to access, for example:
my.site/emby
My config for this is:
location /emby/ {
proxy_pass http://<IP_ADDRESS>:8096; # Local emby ip and non SSL port

proxy_hide_header X-Powered-By;
proxy_set_header Range $http_range;
proxy_set_header If-Range $http_if_range;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header Host $host;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;

#Next three lines allow websockets
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection "upgrade";
}

I have a working config on another machine (just command line/config files only) where this works.
For other services, I had to use:
proxy_pass http://localip:8800/sabnzbd/
Which appends a "/sabnzbd" after the Port number (Sonarr/Radarr/Lidarr/Sabnzbd require this).
My questions are:

For the 1st example (emby), is there something I'm missing to be able to get this to work? I can provide my other config (for the Server section) if needed.
For the 2nd example (Sonarr/Radarr/Lidarr), is there a way to append that URL piece after the Port?

As an addendum to this, setting these up as sub-domains instead of the folder structure does work (forwards correctly).

I believe you are having the same problem as issue #40


CreaatCreate Tshirt

Jsjdjsjdjdj
Changes

Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[x] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[x] To the best of my knowledge, all proposed changes are accurate.

.....

Jsjssj

Jjdjdjd

helllo

I like it


tinymce editors for comments missing lists and tables

the tinyMCE editors appear to be standardized everywhere except lists and tables are missing in the Comments area on the write plan page. this is only happening on dmptool stage, not roadmap stage.

this is related to https://github.com/DMPRoadmap/roadmap/issues/1594.
We weren't happy with the resolution for that one. My suggestion is to update the notifications page so that its Tinymce editor allows tables and lists. We can just rely on Super Admins not to use that functionality since it will not display properly. That should become evident pretty quickly if they do use it.

transferred to roadmap


Updating email addresses and other things.

My name is Jayson Lane and I approve of this pull request.

👍


add terratest to the supported tools

Is your feature request related to a problem? Please describe.
I'm always frustrated by managing versions of my terraform and tofu tooling. As terratest is also being used, this is not supported yet in tenv.
Describe the solution you'd like
Terratest is another tool being used within the terraform / tofu space. It would be very beneficial to also manage it using tenv.
Describe alternatives you've considered
Maintain terratest versions on my own.
Actually this sounds like a go version manager, though (sort of).
Additional context
N/A
REF: https://github.com/gruntwork-io/terratest

We have the similar request about tflint / trivy: https://github.com/tofuutils/tenv/issues/210
Do you think, that terratest requires anything more than just a version update? Please, describe your case a little bit more

Well, as updated above it's more a go version manager then.
AFAICT, it's just about pulling the proper version and maybe set ENV variables depending on how the installation is applied.
Terratest is based on go test and the terratest code versions are maintained within go.mod/go.sum.
I would just be helpful to support GO version management within tenv instead of incorporating asdf.

In order to leverage terratest (go get..), go is a pre-requisite. As mentioned above this request might rather sound like a go version manager backed into tenv. It would just allow us to use a single tool for managing dependencies.

Your point is taken 😃
I'll look into gvm / goenv.
Thank you!


python311Packages.jaxlib-bin: mark as broken on x86_64-darwin

Description of changes
Fails during the pythonImportsCheckPhase with:
/nix/store/1qn2d4m2489bl5xkab4m4rd1g1rpin6h-python-imports-check-hook.sh/nix-support/setup-hook: line 10: 28017 Illegal instruction: 4  /nix/store/5qpssbvkzfh73xih07xgmpkj5r565975-python3-3.11.9/bin/python3.11 -c 'import os; import importlib; list(map(lambda mod: importlib.import_module(mod), os.environ["pythonImportsCheck"].split()))'

cc @samuela
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


Project Quick-Start - Missing info on README

Hello @Mortrest
Please update the related section on README: how the users can use the product.
Also, mention a recommended NodeJS version.
https://github.com/admin-dashboards/nextjs-argon-dashboard#-quick-start

Added.

Closing. Ty @Mortrest


[WIP] Tch4lla/results store

Resolves #105
WIP
What changed

Created a pinia store using quick dummy data.
State is an array of objects.
Actions will get the results and toggle whether a selected result is active or not

Testing instructions
Navigate to resultsStore.ts in the stores folder

Is this still WIP? If so, can you put [WIP] at the beginning of your PR title? Also don't forget to run yarn lint --fix before you push. 😎

Is this still WIP? If so, can you put [WIP] at the beginning of your PR title? Also don't forget to run yarn lint --fix before you push. 😎

Yes ma'am. I thought I had added the WIP, but I guess I didn't have it in the brackets. I'll try to find out what the yarn lint issue is, as I've been running into some odd behavior with it.

@all-contributors bot please add @Tch4lla for code


indentHTML not detecting self-closing img tags

The third master-rule should include self-closing img tags. Otherwise it skips an indentation on the tag.
{
langs: "html",
name: "link|br|input|meta|img",
startToken: [/<(link|br|input|meta|img)/i],
endToken: [/>/],
advance: true
},

Thanks. I added hr and img to that rule. #4


Ip Range

I want to use PiVPN to access all kind of IP hardware on a remote LAN. Mainly to renew firmware and to update programs. The installation was a piece of cake. Great script!
But the ip range I get from the raspberry is not the ip-range that I want to access. I tried to change the range in the server.conf but that stopped everything. How can I adjust the ip-range? I have looked at the bridge mode but doesn't know if this applies to my problem.
thanks

PiVPN Issue Template

Console output of curl install.pivpn.io | bash

Console output of pivpn add or pivpn add nopass

Console output of pivpn debug

Issue

Hello,  when opening issues would be of great use providing the info requested in the template it helps a lot any1 that tries to help you otherwise the troubleshooting would be more of a trial and error and guessing what could be wrong instead of looking to looking to the exact config lines and understand what's wrong.
regarding the issue, it is not related to IP range!
OpenVPN server needs to be aware of other subnets existence which is done in the setup unless you have set up something else.
please check your server config file and pay special attention to this lines
your local subnet
push "route <IP_ADDRESS> <IP_ADDRESS>"
and confirm if they exist and if the IP addresses are properly set up!
if it doesn't help, please provide the pivpn debug command output please!

@redfast00 seems that this issue can be closed since there's no more answers/questions.


The description for the System.Buffers namespace in the .NET API browser is missing some text

Here is the description from The .NET API Browser (https://docs.microsoft.com/en-us/dotnet/api/?view=netcore-3.1) "The namespace contains types used in creating and managing memory buffers, such as those represented by and ."  Here is what the text should be: "The System.Buffers namespace contains types used in creating and managing memory buffers, such as those represented by Span and Memory." (https://docs.microsoft.com/en-us/dotnet/api/system.buffers?view=netcore-3.1).
Browser: Microsoft Edge beta - Version 79.0.309.56 (Official build) beta (64-bit)

This issue has come up before, where xref-styled text doesn't show up in the summary page.
/cc @mimisasouvanh

OK – Is it going to be fixed?
Ben Ellett
From: Genevieve Warren<EMAIL_ADDRESS>Sent: Thursday, March 26, 2020 4:15 PM
To: MicrosoftDocs/feedback<EMAIL_ADDRESS>Cc: Ben Ellett<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/feedback] The description for the System.Buffers namespace in the .NET API browser is missing some text (#2330)
This issue has come up before, where xref-styled text doesn't show up in the summary page.
/cc @mimisasouvanhhttps://github.com/mimisasouvanh
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/feedback/issues/2330#issuecomment-604734082, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHIBC2WHVRRZXDGZG3IEGXLRJPOWBANCNFSM4KB2APNQ.

I find this issue inother namespaces, too.

System.Buffers
System.Numerics
System.Drawing
System.Threading
System.Windows.Documents


operator-sdk test local ./test/e2e --no-setup --namespace test failing to run

What did you do?

Running end to end tests using following command:
operator-sdk test local ./test/e2e --no-setup --namespace test-namespace-25c86144-7275-475f-9c66-a3a511fba572 --go-test-flags "-v -timeout=15m"
Logs can be found at https://storage.googleapis.com/origin-ci-test/pr-logs/pull/codeready-toolchain_member-operator/26/pull-ci-codeready-toolchain-member-operator-master-e2e/9/build-log.txt

What did you expect to see?
While running end to end tests, for operator having go modules enabled, looks like operator sdk couldn't able to identity go.mod file which is default mode, due to which it's assuming it doesn't exists and appending -mod=vendor" for go test.
What did you see instead? Under which circumstances?
If I run go mod vendor, before running tests, it's working fine. but Ideally it should be able to resolve go modules.
Environment

operator-sdk version:
operator-sdk version: v0.8.1, commit: 33b3bfe10176f8647f5354516fff29dea42b6342

go version:
go1.12.5

Kubernetes version information:
v3.11.0+0cbc58b

Kubernetes cluster kind:
minishift

Possible Solution
Maybe worth to add flag to pass to detect environment if won't be able to detect automatically dep manager.
code references

https://github.com/operator-framework/operator-sdk/blob/master/cmd/operator-sdk/test/local.go#L221
https://github.com/operator-framework/operator-sdk/blob/master/internal/util/projutil/project_util.go#L101

Additional context
operator-sdk build quay.io/codeready-toolchain/member-operator is also failing

Hello. I've looked at the code and identified the problem. The issue you are having is present in the v0.8.1 release, but has been fixed in master. We are preparing to release v0.8.2 release, so I will make sure the fix gets into that release and will notify you when it is ready.

thank you @AlexNPavel

@dipak-pawar this issue will be fixed in v0.9.0 rather than a patch release v0.8.2. We only support using vendor/ in v0.8.x. Ability to use go modules without vendoring was a feature added after v0.8.0 was released.
v0.9.0 will be released very soon.


UNIX permissions causing unwanted failures

/kind bug

Which area this bug is related to?

/area ci
What versions of software are you using?
Operating System: N/A
Go Pkg Version: N/A
Bug Summary
Describe the bug:

Changes in UNIX permissions in commits can cause build scripts and CI actions to fail. This is due to script executions using the property of being executable, example in shell: ./build.sh
To Reproduce:

Though this is typically done by unintentional actions (e.g. actions done on a non-UNIX OS), this can be directly reproduced in a UNIX-based system by running the following:
chmod 644 <path-to-script>

or
chmod a+rw <path-to-script>

Expected behavior

Scripts should be able to run without relying on the executable UNIX property.
Any logs, error output, screenshots etc? Provide the devfile that sees this bug, if applicable.

N/A
Additional context

Any workaround?

In the current state, review on UNIX permissions and corrections when needed is required to avoid this issue.
Suggestion on how to fix the bug

All script execution calls should be changed from directly executing the script file to passing the script file on call of the appropriate shell defined by the scripts shebang, e.g. in the case of a bash script ./build.sh would become bash ./build.sh.
[Edit by @thepetk]
Target date: 28 Apr 2023

Update: I've created as an example the devfile-api#1107, where I've checked the .github/workflows/ dir and updated the direct scripts execution.

UPDATE: I've updated the documentation (tried to add bash wherever we mention that someone has to run a script locally) and updated the PRs after the first reviews.


open_resource() does not allow mode='rt'

In Python 3, mode='rt' is synonymous with mode='r' but Flask disallows it thinking it is not a valid mode for reading. While this is true in Python 2, Flask could (at least) provide a more meaningful error message, if not just allow it.
A simple solution might change these lines in flask/helpers.py:
    def open_resource(self, resource, mode='rb'):
        """..."""  # truncated for example
        if mode not in ('r', 'rb'):
            raise ValueError('Resources can only be opened for reading')
        return open(os.path.join(self.root_path, resource), mode)

...to...
    def open_resource(self, resource, mode='rb'):
        """..."""
        if mode == 'rt':
            mode = 'r'
        if mode not in ('r', 'rb'):
            raise ValueError('Resources can only be opened for reading')
        return open(os.path.join(self.root_path, resource), mode)

A heavier solution might use io.open() (and also allow encoding, but see #1740):
    def open_resource(self, resource, mode='rb', encoding=None):
        """..."""
        if mode not in ('r', 'rb', 'rt'):
            raise ValueError('Resources can only be opened for reading')
        return io.open(os.path.join(self.root_path, resource), mode=mode, encoding=encoding)

Expected Behavior
Calling open_resource() with mode='rt' should open a file in text mode (at least in Python 3).
from flask import Flask

app = Flask(__name__)
with app.open_resource('README.md', mode='rt') as f:
    print(f.read())

Actual Behavior
A misleading exception is raised.
Traceback (most recent call last):
  File "mwe.py", line 5, in <module>
    with app.open_resource('README.md', mode='rt') as f:
  File "/home/goodmami/repos/OMW/py3env/lib/python3.6/site-packages/flask/helpers.py", line 1002, in open_resource
    raise ValueError('Resources can only be opened for reading')
ValueError: Resources can only be opened for reading

Environment

Python version: 3.6.7
Flask version: 1.0.2
Werkzeug version: 0.14.1

Starting work on this now.

PR Created


Need a dubbo 2.7.3 example

We don't provide support for dubbo2, Currently we only provided integration fury with dubbo3

Got it.
发自我的iPhone
------------------ Original ------------------
From: Shawn @.>
Date: Fri,Nov 17,2023 0:30 PM
To: fury-project/dubbo-serialization-fury @.>
Cc: dragon-zhang @.>, Author @.>
Subject: Re: [fury-project/dubbo-serialization-fury] Need a dubbo 2.7.3example (Issue #14)
We don't provide support for dubbo2, Currently we only provided integration fury with dubbo3
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

The integration for dubbo2 is not complex, you can integrate it with yourself, or use fury in your application code


[MatMenu] MatMenuTrigger ExpressionChangedAfterItHasBeenCheckedError aria-expanded: null

What is the expected behavior?
Error should not occur
What is the current behavior?
Repeatedly and quickly clicking on menu trigger button causes error
What are the steps to reproduce?
StackBlitz
Click on menuTrigger repeatedly -> error shows in console
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 7.1.0
Angular Material 7.1.0
Chrome Browser
Is there anything else we should know?
ChangeDetection: OnPush
ERROR
Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'aria-expanded: true'. Current value: 'aria-expanded: null'.

I am seeing this error happening as well. Any word on a fix?

Same error here with 7.2.2 and 7.3.0 . Does not happen when being outside a *ngFor but inside I'm reliably  able to reproduce the issue.

Still an issue with 8.0.0.

Here is a workaround that fixed it for me:  use the menuOpened and menuClosed Outputs to set the disabled property of the trigger.
<button
  #menuTrigger
  [matMenuTriggerFor]="addContactButton"
  (menuOpened)="menuTrigger.disabled = true"
  (menuClosed)="menuTrigger.disabled = false"
  i18n>+ Add New</button>

Here is a workaround that fixed it for me: use the menuOpened and menuClosed Outputs to set the disabled property of the trigger.
<button
  #menuTrigger
  [matMenuTriggerFor]="addContactButton"
  (menuOpened)="menuTrigger.disabled = true"
  (menuClosed)="menuTrigger.disabled = false"
  i18n>+ Add New</button>

Actually, you just have to subscribe to menuClosed EventEmitter to get it "fixed", at least in my case.
<button #menuTrigger [matMenuTriggerFor]="menu" (menuClosed)="(true)">+</button>

Not sure why because if I subscribe directly to menuClosed in MatMenuTrigger directive I still have this issue.

Here is a workaround that fixed it for me: use the menuOpened and menuClosed Outputs to set the disabled property of the trigger.
<button
  #menuTrigger
  [matMenuTriggerFor]="addContactButton"
  (menuOpened)="menuTrigger.disabled = true"
  (menuClosed)="menuTrigger.disabled = false"
  i18n>+ Add New</button>

Actually, you just have to subscribe to menuClosed EventEmitter to get it "fixed", at least in my case.
<button #menuTrigger [matMenuTriggerFor]="menu" (menuClosed)="(true)">+</button>

Not sure why because if I subscribe directly to menuClosed in MatMenuTrigger directive I still have this issue.

I need aria-expanded=false when menu is not opened. What is the fix for this?

When matMenuTrigger is used in component with OnPush change detection strategy then aria-expanded is not changing. I suppose it is caused because menu trigger has default change detection strategy.


docs: ファイル名に関するルールを追加

close #310
Type of Change:
ドキュメントの改善
Details of implementation (実施内容)
#310 ( https://github.com/approvers/OreOreBot2/issues/185#issuecomment-1177514433 ) で指摘があったため、ファイル名に関するルールを追加しました。

Windows OSで使えない計9個の記号について追加
あまり使うべきではない機種依存文字・特殊文字についても追記

Codecov Report

Merging #314 (40d129d) into main (f4f53e4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 40d129d differs from pull request most recent head 849b2d4. Consider uploading reports for the commit 849b2d4 to get more accurate results

@@           Coverage Diff           @@
##             main     #314   +/-   ##
=======================================
  Coverage   68.60%   68.60%
=======================================
  Files          46       46
  Lines        3182     3182
  Branches      300      300
=======================================
  Hits         2183     2183
  Misses        699      699
  Partials      300      300

Flag
Coverage Δ

unittests
68.60% <ø> (ø)

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

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 f4f53e4...849b2d4. Read the comment docs.


UPDATE sakrajda.eu

Link to GTMetrix results: https://gtmetrix.com/reports/sakrajda.eu/aoVnV2bi/
- domain: sakrajda.eu
  url: https://sakrajda.eu
  size: 5.6

Sorry Kev, I cleaned up the website just when you have updated it, and I was too late to edit the previous request. :)

No problem. About to push the update.


Notifications are not pushing while the app is running in backround (only in Android 4.x devices)

Hi,
Initially i was testing my app in android 5.x(Lollipop) devices. I got all my notifications pushed successfully while my app is in background and foreground also. But when i run my app in android 4.x(Kitkat) devices it is receiving notifications only when my app is in foreground and it does not receives while my app is in background. Any suggestion please?

@GopalakrishnanK Can you provide me the full payload you are sending to GCM?

@macdonst Please find my full payload which i'm sending to GCM.
{
message : "30% Offer on each buy!!!",
title: "Lifestyle Offer!!!!",
image	: "notification",
notId : "b3c31373b4e09028161aebac7885e40e",
info: "Lifestyle Offer!!!!",
style: "picture",
picture: "http://36.media.tumblr.com/c066cc2238103856c9ac506faa6f3bc2/tumblr_nmstmqtuo81tssmyno1_1280.jpg",
summaryText: "30% Offer on each buy!!!",
collapse_key: "do_not_collapse",
foreground: true,
ad_id: "b3c31373b4e09028161aebac7885e40e"
}
Thanks.

@GopalakrishnanK do you put that in data or notification?

@macdonst I'm putting it in "data" like following.
_msg = {
"data":{
message : "30% Offer on each buy!!!",
title: "Lifestyle Offer!!!!",
image : "notification",
notId : "b3c31373b4e09028161aebac7885e40e",
info: "Lifestyle Offer!!!!",
style: "picture",
picture: "http://36.media.tumblr.com/c066cc2238103856c9ac506faa6f3bc2/tumblr_nmstmqtuo81tssmyno1_1280.jpg",
summaryText: "30% Offer on each buy!!!",
collapse_key: "do_not_collapse",
foreground: true,
ad_id: "b3c31373b4e09028161aebac7885e40e"
}
}
Thanks!!!

@GopalakrishnanK I will try and check on this as soon as I can load Android 4.x on a device. Might take until Wednesday.

@macdonst sure. Thanks.

@GopalakrishnanK I installed Android 4.4. on my Nexus 5 and I can't reproduce this issue. I can receive messages when the app is in the foreground or background.

@macdonst Thanks. But can you please suggest me what could be possible reasons for this.? I'm confused. Anyways thanks for your help.

@GopalakrishnanK can you run adb logcat and reproduce the issue? I'd like to see what the device says.

@macdonst Sure.
Thanks.

@macdonst thanks for your help. It is working now actually my problem is 4.x has problem with other plugin. I've fixed it.


TopBar with ngRepeat rendered items

It seems, that if you render out the menu items (LI with A tags) in a topbar from an array, the topbar does not close when you click the links.
I've forked the TopBar demo to reproduce the issue
http://plnkr.co/edit/CdLKfLTZNf0HZqIBJju5

+1


Enhancement: Dev-9439 API Error Message Display

Enhancement: API error message display
https://websupport.cdc.gov/projects/DEV/issues/DEV-9439
Error message added for when there is an unsuccessful API request.
Testing Steps
Scenario: Upload dev-9439-config.json and open the Dashboard Preview
Expected: A red warning box will appear with an error message. This message can be closed if the "X" is clicked on or it will close on it's own in 5 seconds.
.

Self Review

I have added testing steps for reviewers
My changes generate no new warnings
New and existing unit tests are passing

Screenshots (if applicable)

Additional Notes

@joshlacey @Matt-DiPaolo this works, I'm wondering if we want to update the styles on the alert to stretch the full dashboard though? If you decide to update those styles consider also creating a story to show the api error message.

Should I make the existing Alert component full width everywhere it is used or just this API Error for the filters?


Gateway API on EKS not working as expected

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I'm not able to configure the gateway.
My Cilium configuration seems ok:
cilium-cli config view | grep -w "enable-gateway-api"
enable-gateway-api                                true
enable-gateway-api-secrets-sync                   true

I followed this guide and when I create the gateway along with the httproute it creates a loadbalancer:
kubectl apply -f https://raw.githubusercontent.com/cilium/cilium/HEAD/examples/kubernetes/gateway/basic-http.yaml
gateway.gateway.networking.k8s.io/my-gateway created
httproute.gateway.networking.k8s.io/http-app-1 created

kubectl get svc
NAME                        TYPE           CLUSTER-IP       EXTERNAL-IP                                                                    PORT(S)        AGE
cilium-gateway-my-gateway   LoadBalancer   <IP_ADDRESS>    k8s-default-ciliumga-fa85db4e7d-83a59527c5457ff3.elb.eu-west-3.amazonaws.com   80:30311/TCP   4s

The HTTP route seems to be properly configured:
kubectl describe httproutes.gateway.networking.k8s.io http-app-1 | grep -A20 'Status:'
Status:
  Parents:
    Conditions:
      Last Transition Time:  2023-08-15T08:32:55Z
      Message:               Service reference is valid
      Observed Generation:   1
      Reason:                ResolvedRefs
      Status:                True
      Type:                  ResolvedRefs
      Last Transition Time:  2023-08-15T08:32:55Z
      Message:               Accepted HTTPRoute
      Observed Generation:   1
      Reason:                Accepted
      Status:                True
      Type:                  Accepted
    Controller Name:         io.cilium/gateway-controller
    Parent Ref:
      Group:      gateway.networking.k8s.io
      Kind:       Gateway
      Name:       my-gateway
      Namespace:  default
Events:           <none>

However, the gateway never gets ready:
Status:
  Conditions:
    Last Transition Time:  2023-08-14T16:22:53Z
    Message:               Unable to create Service resource
    Observed Generation:   1
    Reason:                NoResources
    Status:                False
    Type:                  Accepted
    Last Transition Time:  2023-08-14T16:22:53Z
    Message:               Address is not ready
    Observed Generation:   1
    Reason:                ListenersNotReady
    Status:                False
    Type:                  Programmed
  Listeners:
    Attached Routes:  1
    Conditions:
      Last Transition Time:  2023-08-14T16:22:53Z
      Message:               Listener Programmed
      Observed Generation:   1
      Reason:                Programmed
      Status:                True
      Type:                  Programmed
      Last Transition Time:  2023-08-14T16:22:53Z
      Message:               Listener Accepted
      Observed Generation:   1
      Reason:                Accepted
      Status:                True
      Type:                  Accepted
    Name:                    web-gw
    Supported Kinds:
      Group:  gateway.networking.k8s.io
      Kind:   HTTPRoute

Even though the curl command reaches the envoy service, it returns a 404:
curl --fail -s http://<IP_ADDRESS>/details/1 -vvv
* processing: http://<IP_ADDRESS>/details/1
*   Trying <IP_ADDRESS>:80...
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80
> GET /details/1 HTTP/1.1
> Host: <IP_ADDRESS>
> User-Agent: curl/8.2.1
> Accept: */*
>
< HTTP/1.1 404 Not Found
< date: Mon, 14 Aug 2023 16:36:18 GMT
< server: envoy
< content-length: 0
* The requested URL returned error: 404
* Closing connection

Am I missing something on EKS? Note that I'm using a Kyverno workaround to set the annotations properly.
Cilium Version
1.14.0
Kernel Version
I'm not able to ssh to the instance right now but it's the latest bottlerocket AMI. I'll try to enable SSM.
Kubernetes Version
1.27
Sysdump
cilium-sysdump-20230814-183832.zip
Relevant log output
No response
Anything else?
I'm writing a blog post about Cilium with Gateway API on EKS. My code is here if you want to look at it.
Code of Conduct

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

I just tried again in order to try that with Amazon LInux instances (kernel version: 5.10.184-175.749.amzn2.x86_64). I got exactly the same behavior. The gateway is not properly configured, the status Programmed=False.

Same behaviour. Created dupe issue for this. Here is a resources
Gateway with status
apiVersion: gateway.networking.k8s.io/v1beta1
kind: Gateway
metadata:
  name: rest-external
spec:
  gatewayClassName: cilium
  listeners:
    - allowedRoutes:
        namespaces:
          from: All
      hostname: '*.myhost'
      name: web-gw
      port: 443
      protocol: HTTP
status:
  conditions:
    - lastTransitionTime: "2023-08-17T09:19:21Z"
      message: Unable to create Service resource
      observedGeneration: 2
      reason: NoResources
      status: "False"
      type: Accepted
    - lastTransitionTime: "2023-08-17T09:19:21Z"
      message: Address is not ready
      observedGeneration: 2
      reason: ListenersNotReady
      status: "False"
      type: Programmed
  listeners:
    - attachedRoutes: 1
      conditions:
        - lastTransitionTime: "2023-08-17T09:19:21Z"
          message: Listener Programmed
          observedGeneration: 2
          reason: Programmed
          status: "True"
          type: Programmed
        - lastTransitionTime: "2023-08-17T09:19:21Z"
          message: Listener Accepted
          observedGeneration: 2
          reason: Accepted
          status: "True"
          type: Accepted
      name: web-gw
      supportedKinds:
        - group: gateway.networking.k8s.io
          kind: HTTPRoute

Service
apiVersion: v1
kind: Service
metadata:
  annotations:
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
    service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
    service.beta.kubernetes.io/aws-load-balancer-ssl-cert: arn:aws:acm:us-east-1:123456789012:certificate/12345678-1234-1234-1234-123456789012
    service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "443"
  finalizers:
    - service.k8s.aws/resources
  labels:
    io.cilium.gateway/owning-gateway: rest-external
  name: cilium-gateway-rest-external
  namespace: gateway-api
  ownerReferences:
    - apiVersion: gateway.networking.k8s.io/v1beta1
      kind: Gateway
      name: rest-external
      uid: 66cbf2c8-67ff-44cd-aa66-862c922d8114
spec:
  allocateLoadBalancerNodePorts: true
  clusterIP: <IP_ADDRESS>
  clusterIPs:
    - <IP_ADDRESS>
  externalTrafficPolicy: Cluster
  internalTrafficPolicy: Cluster
  ipFamilies:
    - IPv4
  ipFamilyPolicy: SingleStack
  loadBalancerClass: service.k8s.aws/nlb
  ports:
    - name: port-443
      nodePort: 32762
      port: 443
      protocol: TCP
      targetPort: 443
  sessionAffinity: None
  type: LoadBalancer
status:
  loadBalancer:
    ingress:
      - hostname: actualhostname-1234567890.us-east-1.elb.amazonaws.com

@Smana I find another workaround on this issue. You need to slightly tune policy that you using (afaik also written by me)
      mutate:
        patchStrategicMerge:
          metadata:
            annotations:
              service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
              service.beta.kubernetes.io/aws-load-balancer-ssl-ports: "443"
              service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
          spec:
            loadBalancerClass: service.k8s.aws/nlb # This is fixes issue

Well, it doesn't seem to work on my side even though I defined the loadBalancerClass.
apiVersion: v1
kind: Service
metadata:
  annotations:
    external-dns.alpha.kubernetes.io/hostname: echo-mycluster-0.cloud.ogenki.io
    policies.kyverno.io/last-applied-patches: |
      mutate-svc-annotations.mutate-cilium-gateway-echo-gateway.kyverno.io: added /metadata/annotations
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
    service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
  labels:
    io.cilium.gateway/owning-gateway: echo-gateway
  name: cilium-gateway-echo-gateway
  namespace: echo
spec:
  allocateLoadBalancerNodePorts: true
  clusterIP: <IP_ADDRESS>
  clusterIPs:
  - <IP_ADDRESS>
  internalTrafficPolicy: Cluster
  ipFamilies:
  - IPv4
  ipFamilyPolicy: SingleStack
  loadBalancerClass: service.k8s.aws/nlb
  ports:
  - name: port-80
    nodePort: 30131
    port: 80
  type: LoadBalancer

I have this error in the cilium-operator logs:
cilium-operator-76c4f97d54-mtlbg cilium-operator level=error msg="Unable to create Service" controller=gateway error="Service \"cilium-gateway-echo-gateway\" is invalid: spec.loadBalancerClass: Invalid value: \"null\": may not change once set" resource=echo/echo-gateway subsys=gateway-controller

I managed to get it work using only Classic LB. I had to configure the AWS LoadBalancer Controller accordingly with the option enableServiceMutatorWebhook set to false.
kubectl get svc -n echo cilium-gateway-echo-gateway -o yaml | head -n 10
apiVersion: v1
kind: Service
metadata:
  annotations:
    external-dns.alpha.kubernetes.io/hostname: echo-mycluster-0.cloud.ogenki.io
    policies.kyverno.io/last-applied-patches: |
      mutate-svc-annotations.mutate-cilium-gateway-echo-gateway.kyverno.io: added /metadata/annotations
    service.beta.kubernetes.io/aws-load-balancer-backend-protocol: tcp
    service.beta.kubernetes.io/aws-load-balancer-scheme: internet-facing
...

GATEWAY=$(kubectl get gateway echo-gateway -n echo -o jsonpath='{.status.addresses[0].value}')

curl http://$GATEWAY |jq
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2491  100  2491    0     0   101k      0 --:--:-- --:--:-- --:--:--  105k
{
  "host": {
    "hostname": "aa4ae59f99b1b4d20b74f35a06aa7ab7-228497968.eu-west-3.elb.amazonaws.com",
    "ip": "::ffff:<IP_ADDRESS>",
    "ips": []
  },
  "http": {
    "method": "GET",
    "baseUrl": "",
    "originalUrl": "/",
...

Well I don't know what's the recommendations here, as the Classic LB are deprecated, I guess I have to wait for this issue to be addressed.

@Smana It works with my workaround policy. But you need to apply policy first and then recreate the gateway (or just underlying service)

@sergeyshevch thank you I just tried one more time and indeed that works fine. thank you!

@sergeyshevch thank you I just tried one more time and indeed that works fine. thank you!

Did anyone managed to get http to https redirect working with the configuration mentioned in this issue? I tried the redirect http route and getting infinite redirect. Any ideas?

@lucidprogrammer , I think you're facing this issue https://github.com/kubernetes-sigs/gateway-api/issues/1185


Add replicas and shards to bgp index

Description
BGP index needs to be adjusted based on the number of ES nodes.
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Alina Militaru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Option to keep JSON  not specified in JSON-schema

An boolean config option to keep other parts of the JSON while updating.

Overwrite parts of the JSON not mentioned in JSON-schema
Keep parts of the JSON not mentioned in JSON-schema

new JSON is optionally merged with "old JSON" now


Power level able to go below 0?

I'm not sure if thats what we want or like, if it should flatten off at 0.
I mean, no power is no power, right?

This should be fixed now. Can you confirm?


Added Coveralls Action

@gep13 change made :+1:

@gavinhenderson your changes have been merged, thanks for your contribution 👍


[npm] Could you release a beta version of the next set of features into npm

Hi,
the version in npm is about four months old. You have since added some new features and made improvements in this repository, but we, the library users, can't yet take advantage of those.
I can appreciate that releasing a npm version might be a a burden, maybe due to documentation concerns for instance. But could you consider releasing the changes with a beta tag for instance? In that case you wouldn't need to complete the release, those of us who would want to use the improvements would get to do so and maybe you'd even get some beta testing out of us.
Thanks for the library and effort.

@Jokinen
Hi, l'm working on it, and there still are a few script errors in IE now, I'll publish one after fixing them, thank you:)


Adding "indexing" to End Double Counting

We have a double counting problem.
The root of our double counting problem is this: A single sentence might have two subjects that are the same.
For example: "One friend went to the store, but the other friend went home."
The extracted triples are:

friend-went-to-store
friend-went-home
friend-went-home

This sentence has two subjects that are "friend," but they refer to different friends.
We can fix this issue by looking at the index of friend. The index of the first mention of friend is 2, and the index of the second friend is 10, thus indicating that these are different subjects even though they look the same.
To see index do: token.i

@stephbuon validate changes


Loading Progress

Can we have more methods as we have onPdfError()
There should be methods including progress onStartLoading, onLoadedSuccess so we can show and hide progressbar

Hey you can track the progress in onPdfLoadProgress by implementing status listener.
 binding.pdfView.statusListener = object : PdfRendererView.StatusCallBack {
                override fun onPdfLoadStart() {
                    Log.i("statusCallBack","onPdfLoadStart")
                }
                override fun onPdfLoadProgress(
                    progress: Int,
                    downloadedBytes: Long,
                    totalBytes: Long?
                ) {
                    //Download is in progress
                }

                override fun onPdfLoadSuccess(absolutePath: String) {
                    Log.i("statusCallBack","onPdfLoadSuccess")
                }

                override fun onError(error: Throwable) {
                    Log.i("statusCallBack","onError")
                }

                override fun onPageChanged(currentPage: Int, totalPage: Int) {
                    //Page change. Not require
                }
            }

If you need further help, feel free to join https://discord.gg/z9qQSHMxSR for faster help.

It works like a charm. Thank you
If anyone having issue, please consider different versions have different override methods


Sampling Vanilla VAE

Hi,
First, thanks for all the shared work !
I have a question concerning the sampling function in the Vanilla VAE. Why do you sample from a normal distribution (0,1) and not from a normal distribution with the learned parameters mu and sigma ? Since when we train the network we decode from the latent space over this distribution isnt more meaningful to sample from this distribution ? Maybe is there something I didnt get.
Thank you again

I am not sure ideally it should be scaled like below.
mu/var comes out of the encoder network.
final_samples = mu + (sqrt(var) * samples)

I have the same question. @AntixK

When you do sampling, it means you did not have "input" images, you only sample from some distribution, here is a standard normal distribution. In other words, when sampling you did not have encoder, only use decoder, so you did not have this mu and var.


Audio is not seekable

As a user of the kalita-js front end, I am unable to seek within the audio file.
However, the cause of the problem is not in the front end, but in the fact that the server does not support seeking as the request header Range is ignored.
Please add support for audio seeking.


Texlab, cargo builds from git

Hi,
First of all, nice project :)
I'm a part-time Texlab user, and normally update by building directly from their git repo using
cargo install --git https://github.com/latex-lsp/texlab.git --locked

I looked at integrating this into nvim-lspupdate but quickly noticed that this would be incompatible with the default behaviour of combining multiple installs into a single command with the %s expansion.
Based on this, I think it would be nice to have an alternative way to run install commands that can install only one package at a time. I can think of two ways of doing this:

Make separate commands (e.g. cargo vs cargogit), and for each command register whether it has the ability to concatenate multiple packages into a single command line.
Register for each package whether it may be installed in the same command line as other packages or not. This would allow any extra flags such as --git in my example to be added to the package config.

What are your thoughts on this, would you be interested in supporting this approach?

Hello,
I think the biggest problem right now, is not the command itself (which you already can override and/or create additional commands), but the fact that you cannot override the config. Whatever is the config for that LSP (and right now, for texlab is the empty string), you are stuck with it.
I think I should just allow end users to override that as well, and then they can update the config on their end. Combined with adding and/or overriding commands, this should give all the freedom needed to experiment on their own.
The problem of concatenating packages still remains, but I don't see how that would affect you in this particular case, since you only have one package. Oh, maybe you use multiple cargo based LSPs? Otherwise, if I open up config, then you could override texlab to say cargo|https://github.com/latex-lsp/texlab.git and the cargo command to say cargo install --git %s --locked and that should do it (or introduce a new, cargogit command as you suggested).
Until then, I'm gonna add the package name for texlab right now, it's a tiny change anyway.

Hello,
I think the biggest problem right now, is not the command itself (which you already can override and/or create additional commands), but the fact that you cannot override the config. Whatever is the config for that LSP (and right now, for texlab is the empty string), you are stuck with it.
I think I should just allow end users to override that as well, and then they can update the config on their end. Combined with adding and/or overriding commands, this should give all the freedom needed to experiment on their own.
The problem of concatenating packages still remains, but I don't see how that would affect you in this particular case, since you only have one package. Oh, maybe you use multiple cargo based LSPs? Otherwise, if I open up config, then you could override texlab to say cargo|https://github.com/latex-lsp/texlab.git and the cargo command to say cargo install --git %s --locked and that should do it (or introduce a new, cargogit command as you suggested).
Until then, I'm gonna add the package name for texlab right now, it's a tiny change anyway.

Done, hope this helps: https://github.com/alexaandru/nvim-lspupdate/commit/d450e9cc9d1f5df486c4482b169a80d6faf84cad

Done, hope this helps: https://github.com/alexaandru/nvim-lspupdate/commit/d450e9cc9d1f5df486c4482b169a80d6faf84cad

There's one problem though with the above commit: it only does an install, failing to do what the package name implies: update :)
Looking at the manual https://doc.rust-lang.org/cargo/commands/cargo-install.html I see there is an -f flag to force a reinstall. So perhaps, that is the way to "install or update" with a single command, achieving this project's goals? Also, it seems lke the --locked flag works against that, so I'd remove that. Of course, end users can override and add it if they need it, but for the purpose of this package (lsp update), the goal is to update stuff, not to keep it locked to a particular version. Thoughts?

There's one problem though with the above commit: it only does an install, failing to do what the package name implies: update :)
Looking at the manual https://doc.rust-lang.org/cargo/commands/cargo-install.html I see there is an -f flag to force a reinstall. So perhaps, that is the way to "install or update" with a single command, achieving this project's goals? Also, it seems lke the --locked flag works against that, so I'd remove that. Of course, end users can override and add it if they need it, but for the purpose of this package (lsp update), the goal is to update stuff, not to keep it locked to a particular version. Thoughts?

cargo install will update the package if an update is available; if the latest version is already installed then it's a noop. So just using cargo install should be sufficient. The only time this might not be enough is if a user has updated their rust toolchain and really wants their packages to be rebuilt with the newer version, but tbh I don't think this is a likely use case.

cargo install will update the package if an update is available; if the latest version is already installed then it's a noop. So just using cargo install should be sufficient. The only time this might not be enough is if a user has updated their rust toolchain and really wants their packages to be rebuilt with the newer version, but tbh I don't think this is a likely use case.

Excellent, thank you very much. Then it is a perfect fit :)

Excellent, thank you very much. Then it is a perfect fit :)

Please let me know if the solution implemented works for you and if it's ok to close this ticket. Thank you :)

Please let me know if the solution implemented works for you and if it's ok to close this ticket. Thank you :)

Oops sorry! Yes, this is working for me. Thanks!

Oops sorry! Yes, this is working for me. Thanks!

Awesome, thank you very much! :) Cheers!

Awesome, thank you very much! :) Cheers!


Review runbooks site

This story has been raised to prompt a documentation review of the new documentation sites:
https://runbooks.operations-engineering.service.justice.gov.uk/#moj-operations-engineering-runbooks
The want to ensure that content is as expected.
Create a snagging list for any updates/fixes.

Reviewed. Few updates. We can go live with that site.


Add utf-8 encoding specification to writing

Issue #122 - added utf-8 encoding specification when writing report

Were you able to test this on a Windows machine?

FYI there are Windows tests that run in CI (only on main branch), so there could be a little test implemented to make sure this doesn't happen again but not absolutely necessary.

FYI there are Windows tests that run in CI (only on main branch), so there could be a little test implemented to make sure this doesn't happen again but not absolutely necessary.

Is this test in ci.yaml what you are referring to?
https://github.com/nasaharvest/openmapflow/blob/d96c9f1f942fb03a1d085cd6a9a1e791067dc1c8/.github/workflows/ci.yaml#L54-L101

Yes @bhyeh