error installing on macos 10.14.x

In order of execution, the instalation fails on latest macos 10.14.x with the following issues
Step 8/11 : ADD target/scala-2.11/search-flight-spark-streaming-assembly-0.1.0.jar $SPARK_HOME/libs/search-flight-streaming.jar
ERROR: Service 'streaming-spark-dev' failed to build: ADD failed: stat /var/lib/docker/tmp/docker-builder288103139/target/scala-2.11/search-flight-spark-streaming-assembly-0.1.0.jar: no such file or directory

Step 3/6 : ADD target/search-flight-hadoop-batch-0.1.0-jar-with-dependencies.jar $HADOOP_HOME/libs/search-flight-batch.jar
ERROR: Service 'batch-hadoop-dev' failed to build: ADD failed: stat /var/lib/docker/tmp/docker-builder770657281/target/search-flight-hadoop-batch-0.1.0-jar-with-dependencies.jar: no such file or directory

Step 4/6 : ADD target/search-flight-storm-streaming-0.1.0-jar-with-dependencies.jar $STORM_HOME/libs/search-flight-streaming.jar
ERROR: Service 'streaming-storm-dev' failed to build: ADD failed: stat /var/lib/docker/tmp/docker-builder383930762/target/search-flight-storm-streaming-0.1.0-jar-with-dependencies.jar: no such file or directory

hello,
thanks for your contribution
I will check

Hi, @Chabane
Thanks for quick reply.
I hope you can reproduce

I have to investigate for 0.14.x
It works with OSX 10.13.6
$ git clone https://github.com/Chabane/bigdata-playground
$ cd bigdata-playground
$ cd streaming/spark && sbt clean assembly && cd ../..
$ cd docker
$ docker-compose -f dev/streaming-spark.yml up -d
Building streaming-spark-dev
Step 1/11 : FROM openjdk:8-jre
8-jre: Pulling from library/openjdk
741437d97401: Pull complete
34d8874714d7: Pull complete
0a108aa26679: Pull complete
eea0f08c7492: Pull complete
ca74b3d90c15: Pull complete
f11a643c4a85: Pull complete
05eafed73b65: Pull complete
Digest: sha256:2216ccda45993afaa42b3b269aa1ae8832cb2fabbf8207f63d528fdc7c8a087b
Status: Downloaded newer image for openjdk:8-jre
 ---> ed287c436e66
Step 2/11 : VOLUME /tmp
 ---> Running in 88436c00ffe6
Removing intermediate container 88436c00ffe6
 ---> 8b4ae4f0795f
Step 3/11 : ENV SPARK_VERSION spark-2.4.0-bin-hadoop2.7
 ---> Running in 18f69569d784
Removing intermediate container 18f69569d784
 ---> 43ec961a3688
Step 4/11 : ENV SPARK_HOME /usr/local/spark
 ---> Running in 3185f165f176
Removing intermediate container 3185f165f176
 ---> d45e0984fe65
Step 5/11 : RUN curl https://archive.apache.org/dist/spark/spark-2.4.0/$SPARK_VERSION.tgz -o $SPARK_VERSION.tgz;     tar xzf $SPARK_VERSION.tgz -C /usr/local/;
 ---> Running in f8d5c2aeea09
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  217M  100  217M    0     0   9.8M      0  0:00:22  0:00:22 --:--:-- 9401k
Removing intermediate container f8d5c2aeea09
 ---> 904bc6c8ec6b
Step 6/11 : RUN cd /usr/local && ln -s $SPARK_VERSION spark
 ---> Running in 1afb5821bc35
Removing intermediate container 1afb5821bc35
 ---> d4d112977ae5
Step 7/11 : ADD hbase-site.xml $SPARK_HOME/conf
 ---> 855d1f6f0aaf
Step 8/11 : ADD target/scala-2.11/search-flight-spark-streaming-assembly-0.1.0.jar $SPARK_HOME/libs/search-flight-streaming.jar
 ---> 7912943b00bd
Step 9/11 : RUN sh -c 'touch $SPARK_HOME/libs/search-flight-streaming.jar'
 ---> Running in 596e323e7cd5
Removing intermediate container 596e323e7cd5
 ---> 42bcd548a74e
Step 10/11 : WORKDIR $SPARK_HOME
 ---> Running in 5e91d58ce3fa
Removing intermediate container 5e91d58ce3fa
 ---> 09217c527d4e
Step 11/11 : CMD ["bin/spark-submit", "--class", "com.mitosis.Main", "--master", "local[*]", "--files", "/usr/local/spark/conf/hbase-site.xml", "./libs/search-flight-streaming.jar"]
 ---> Running in 05cea3bcbca9
Removing intermediate container 05cea3bcbca9
 ---> d16a3d45765a

Successfully built d16a3d45765a
Successfully tagged dev_streaming-spark-dev:latest
WARNING: Image for service streaming-spark-dev was built because it did not already exist. To rebuild this image you must use `docker-compose build` or `docker-compose up --build`.
Creating streaming_spark_dev ... done


add feature "serde_support" to ContextBuilder

The tetra_config can be loaded with ron from the filesystem, after adding "serde_support" feature annotation.

This is my first public pull request, I hope I did everything right.

Thanks for the PR!
In terms of the code, this looks fine - I just need to have a bit of a think about whether I want to commit to ContextBuilder only ever containing serializable types (as to remove this later would be a breaking change and probably make people sad 😅). Leaning towards merging it, though.

Given it some more thought, and I think this is okay - if I was going to drastically change the type of stuff that was in ContextBuilder, it'd probably be part of a larger breaking change anyway. I've also considered just renaming it to Settings (similar to Quicksilver and Coffee, I think).
The only thing I might add is a little disclaimer saying that the available fields may change from release to release. I wouldn't want people using this for their game's public-facing config files without realizing they might need to take that into account when updating Tetra. I would probably recommend using this for easy tweaking during development, and then migrate to a game-specific config file if you want more stability for a proper release.
Anyway, that's a very long winded way of saying - thanks for the PR 😛


CB-1514 Fix CRN-based authentication in redbeams

No additional comments. I take it most of this code was copied/adapted from elsewhere in the codebase?

Yeah, from core.


HDDS-8743. Recon - Expose API for hide/show flag for heatmap feature on Recon UI.

What changes were proposed in this pull request?
This PR is to expose an API "api/vi/features/disabledFeatures" that  provides list of all disabled features including HeatMap (unless user provides implementation of "org.apache.hadoop.ozone.recon.heatmap.IHeatMapProvider" interface.)
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8743
How was this patch tested?
This patch is tested using writing new Junit test case under "TestFeaturesEndPoint" class.
GET API "api/vi/features/disabledFeatures"  response. Currently below response contains only one feature which is in disabledFeature list, but in future, this list may have more depending on if a feature is in Beta phase or not implemented.

[
  "FEATURE1"
]

@sumitagrawl @dombizita  Pls review.

@devmadhuu For enum class, we need change...

For below points :
- enum is created with private constructor, else change type to class instead of enum - As discussed, no change needed.

@sumitagrawl please remove unnecessary details from the commit message when merging PRs.
Compare:
commit 9e44b503af1256f3f9a3ca91bce9d20331d2acf1
Author:     devmadhuu<EMAIL_ADDRESS>AuthorDate: Tue Jun 6 13:18:41 2023 +0530
Commit:     GitHub<EMAIL_ADDRESS>CommitDate: Tue Jun 6 13:18:41 2023 +0530

    HDDS-8743. Recon - Expose API for hide/show flag for heatmap feature on Recon UI. (#4822)

to:
commit 9e44b503af1256f3f9a3ca91bce9d20331d2acf1
Author:     devmadhuu<EMAIL_ADDRESS>AuthorDate: Tue Jun 6 13:18:41 2023 +0530
Commit:     GitHub<EMAIL_ADDRESS>CommitDate: Tue Jun 6 13:18:41 2023 +0530

    HDDS-8743. Recon - Expose API for hide/show flag for heatmap feature on Recon UI. (#4822)

    * HDDS-8743. Recon - Expose API for hide/show flag for heatmap feature on Recon UI.

    * HDDS-8743. Recon - Expose API for hide/show flag for heatmap feature on Recon UI.

    * HDDS-8743. Added ASF License header in Feature.java.

    * HDDS-8743. Added heatmap provider config doc in Recon.md file.

    * HDDS-8743. Fixed review comments.

    * HDDS-8743. Fixed review comments.

    * HDDS-8743. Fixed review comments.

    * HDDS-8743. Fixed review comments.

    * HDDS-8743. Fixed review comments.

    ---------

    Co-authored-by: deveshsingh <devmadhu<EMAIL_ADDRESS>


echo stuff

change echo output to std err since angsd sends stuff there. makes output easier to follow.
i.e.: >&2 echo "stuff"

maybe have a flag before angsd-wrapper echoes so we know what it is i.e.: ANGSD-WRAPPER: some message.
this may just be a preference thing for me, but i think it makes it easier to debug

👍 Yeah that makes sense.
On Mon, May 18, 2015 at 4:11 PM Tyler Kent<EMAIL_ADDRESS>wrote:

maybe have a flag before angsd-wrapper echoes so we know what it is i.e.:
ANGSD-WRAPPER: some message.
this may just be a preference thing for me, but i think it makes it easier
to debug
—
Reply to this email directly or view it on GitHub
https://github.com/arundurvasula/angsd-wrapper/issues/72#issuecomment-103246702
.

i fixed these i think


Multiple bookmarks can't be saved in their own folder

This is a Bug Report
Description
I often use the function CTRL+SHIFT+D to add multiple booksmarks in their own folder (e.g. if I do a research and want to add several tabs as bookmarks together in an own folder). Without the adon it works fine. If the adon is active, the new folder gets created in the designated folderpath (the "Default Bookmark Folder" folderpath), set by the adon, but all the bookmarks that should be inside the new subfolder, are in the "Default Bookmark Folder" as well, and not in the subfolder.
Steps to reproduce the problem

Open several tabs
Select multiple tabs
Press CTRL+SHIFT+D
Enter new foldername
Have a look inside your bookmark library, and search for the newly created subfolder in the "Default Bookmark Folder" folderpath.
Check if the bookmarks from your selected tabs are inside the subfolder.

Expected behavior
The bookmarks of the selected tabs should be created inside the subfolder, created from the firefox prompt, that is shown wenn you press CTRL+SHIFT+D
Specifications

Version of the add-on: 3.1.0

Firefox edition: Firefox (regular edition)
Firefox version: 114.0.2
Firefox language: German

Platform/OS: Windows
Platform/OS version: 11 Pro 22H2

Additional context
I use the implemented bookmark function.

I saw another strange behaviour regarding this topic:
When I add a new bookmark folder, the folder gets created in my "default-folder-path" all new bookmarks included. But directly after that, all the bookmarks from within this folder are getting moved directly in the "default-folder-path", leaving an empty bookmarkfolder that was just created.


Makefile Needed

This would be a much easier resource to pull from if there were a makefile for general use - we don't all have QT. Let me know if you want me to make one; I'd be glad to contribute.

That would be great! Thank you!


Easier adding of wikimedia_commons=* tag

People on mobile usually use Wikimedia Commons App to upload pictures and metadata to Wikimedia. The app provides Copy the wikitext to the clipboard button, which produces something like this in copy/paste buffer:

[[File:Kaptol_Square,_Zagreb,_Croatie.jpg|thumb|other_optinal_metadata...]]

OSM wikimedia_commons=* tag expects the values to be in the format:

wikimedia_commons=File:Kaptol_Square,_Zagreb,_Croatie.jpg

Current workflow is:

create picture in Wikimedia Commons App and click Copy the wikitext to the clipboard in it
switch to Every Door, navigate to wikimedia_commons field and paste value
edit the pasted value to erase [[ at the beginning, as well as everything after first |

That third step is very time consuming (and error prone if user sits on backspace key a fraction of second too long).
It would be great if EveryDoor could automate it -- e.g. if it detects the value starts with [[ it should simply extract the text until first |, and use that as a raw tag value for  wikimedia_commons=* OSM tag.
(related commons issue: https://github.com/commons-app/apps-android-commons/issues/4371)

You're very in luck: done in 3.1, released later today :)


Not downloading best thumbnail on some videos

Checklist

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

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-o', '%(upload_date)s - %(title)s.%(ext)s', '--cookies', 'cookies.txt', '-f', 'best', '--write-sub', '--all-subs', '--embed-subs', '--write-thumbnail', '--embed-thumbnail', '--add-metadata', '--write-description', '-v', 'https://www.youtube.com/watch?v=Q_9Fx3zztjE']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2020.02.16
[debug] Python version 3.8.1 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg N-95171-g6ca3d34ff8, ffprobe N-95171-g6ca3d34ff8
[debug] Proxy map: {}
[youtube] Q_9Fx3zztjE: Downloading webpage
[youtube] Q_9Fx3zztjE: Downloading video info webpage
[youtube] {18} signature length 102, html5 player vflsFV4r3
[youtube] {22} signature length 106, html5 player vflsFV4r3
[youtube] {137} signature length 106, html5 player vflsFV4r3
[youtube] {248} signature length 102, html5 player vflsFV4r3
[youtube] {299} signature length 102, html5 player vflsFV4r3
[youtube] {303} signature length 106, html5 player vflsFV4r3
[youtube] {399} signature length 106, html5 player vflsFV4r3
[youtube] {136} signature length 102, html5 player vflsFV4r3
[youtube] {247} signature length 106, html5 player vflsFV4r3
[youtube] {298} signature length 106, html5 player vflsFV4r3
[youtube] {302} signature length 106, html5 player vflsFV4r3
[youtube] {398} signature length 106, html5 player vflsFV4r3
[youtube] {135} signature length 106, html5 player vflsFV4r3
[youtube] {244} signature length 106, html5 player vflsFV4r3
[youtube] {397} signature length 106, html5 player vflsFV4r3
[youtube] {134} signature length 106, html5 player vflsFV4r3
[youtube] {243} signature length 106, html5 player vflsFV4r3
[youtube] {396} signature length 102, html5 player vflsFV4r3
[youtube] {133} signature length 106, html5 player vflsFV4r3
[youtube] {242} signature length 106, html5 player vflsFV4r3
[youtube] {395} signature length 102, html5 player vflsFV4r3
[youtube] {160} signature length 102, html5 player vflsFV4r3
[youtube] {278} signature length 106, html5 player vflsFV4r3
[youtube] {394} signature length 106, html5 player vflsFV4r3
[youtube] {140} signature length 106, html5 player vflsFV4r3
[youtube] {249} signature length 106, html5 player vflsFV4r3
[youtube] {250} signature length 106, html5 player vflsFV4r3
[youtube] {251} signature length 106, html5 player vflsFV4r3
[info] Writing video description to: 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.description
[info] Writing video subtitles to: 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.en.vtt
[info] Writing video subtitles to: 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.ja.vtt
[youtube] Q_9Fx3zztjE: Downloading thumbnail ...
[youtube] Q_9Fx3zztjE: Writing thumbnail to: 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.jpg
[debug] Invoking downloader on 'https://r6---sn-vgqsrnee.googlevideo.com/videoplayback?expire=1582973033&ei=CexZXqGeH8nCkwa4lbGQDQ&ip=<IP_ADDRESS>&id=o-APbkXBO-yrCD3Gpu8YTpsWLVxM5uKjQhPxqhj_UtgA59&itag=22&source=youtube&requiressl=yes&mm=31%2C26&mn=sn-vgqsrnee%2Csn-qxoedn7d&ms=au%2Conr&mv=m&mvi=5&pl=20&pcm2=no&initcwndbps=2156250&vprv=1&mime=video%2Fmp4&ratebypass=yes&dur=396.178&lmt=1580051912833389&mt=1582951350&fvip=6&fexp=23842630&c=WEB&txp=5535432&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cpcm2%2Cvprv%2Cmime%2Cratebypass%2Cdur%2Clmt&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=ABSNjpQwRAIgNZaxgF0VKukJmhvs6n3wiawi73LKWY1jAGaLoI9p_sACIE5EGidBfn7tUE6i76kOpf1ZrF7dMdGFdQC1YZhZkgb0&sig=ADKhkGMwRQIhAJAY1iez6JtosdNTrOUVLQ_hyPk0uXTCnbpyXK_HGiI1AiBs38D2KHrpoWqclrXKHpSS5MjCMOF1ZmmSj0fgukRpeQ=='
[download] Destination: 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4
[download] 100% of 71.93MiB in 00:03
[ffmpeg] Adding metadata to '20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4'
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4" -c copy -metadata "title=TMT | 사나의 TMI가 궁금하다면? 💖 TOO MUCH TWICE EP.01" -metadata "date=20200122" -metadata "description=TMT | 사나의 TMI가 궁금하다면? 💖 TOO MUCH TWICE EP.01

Spotify
http://open.spotify.com/album/4EG4E0Uk0HpJBgOMgXI26z

iTunes/Apple Music
https://music.apple.com/us/album/feel-special/1480249009

TWICE Official YouTube: http://www.youtube.com/c/TWICEonAir
TWICE Official Facebook: http://www.facebook.com/JYPETWICE
TWICE Official Twitter: http://www.twitter.com/JYPETWICE
TWICE Official Instagram: http://www.instagram.com/TWICETAGRAM
TWICE Official Homepage: http://TWICE.jype.com
TWICE Official Fan's: http://fans.jype.com/TWICE

ⓒ 2020 JYP Entertainment. All Rights Reserved" -metadata "comment=TMT | 사나의 TMI가 궁금하다면? 💖 TOO MUCH TWICE EP.01

Spotify
http://open.spotify.com/album/4EG4E0Uk0HpJBgOMgXI26z

iTunes/Apple Music
https://music.apple.com/us/album/feel-special/1480249009

TWICE Official YouTube: http://www.youtube.com/c/TWICEonAir
TWICE Official Facebook: http://www.facebook.com/JYPETWICE
TWICE Official Twitter: http://www.twitter.com/JYPETWICE
TWICE Official Instagram: http://www.instagram.com/TWICETAGRAM
TWICE Official Homepage: http://TWICE.jype.com
TWICE Official Fan's: http://fans.jype.com/TWICE

ⓒ 2020 JYP Entertainment. All Rights Reserved" -metadata "purl=https://www.youtube.com/watch?v=Q_9Fx3zztjE" -metadata "artist=TWICE" "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.temp.mp4"
[ffmpeg] Embedding subtitles in '20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4'
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4" -i "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.en.vtt" -i "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.ja.vtt" -map 0 -c copy -map "-0:s" -map "-0:d" "-c:s" mov_text -map "1:0" "-metadata:s:s:0" "language=eng" -map "2:0" "-metadata:s:s:1" "language=jpn" "file:20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.temp.mp4"
Deleting original file 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.en.vtt (pass -k to keep)
Deleting original file 20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.ja.vtt (pass -k to keep)
[atomicparsley] Adding thumbnail to "20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4"
[debug] AtomicParsley command line: AtomicParsley "20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.mp4" --artwork "20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.jpg" -o "20200122 - TMT _ 사나의 TMI가 궁금하다면 💖 TOO MUCH TWICE EP.01.temp.mp4"

Description

Doesn't download the best thumbnail on some videos. This is the video I used in the above https://www.youtube.com/watch?v=Q_9Fx3zztjE it downloads this thumbnail https://i.ytimg.com/vi/Q_9Fx3zztjE/hqdefault.jpg instead of this one https://i.ytimg.com/vi/Q_9Fx3zztjE/maxresdefault.jpg
But when I download this video https://www.youtube.com/watch?v=nHKVfEOqoR0 it downloads the maxres version https://i.ytimg.com/vi/nHKVfEOqoR0/maxresdefault.jpg

Can you link what this is a duplicate of I can't find it.


Dockerize models training

let's wrap model training in deployable docker container

@h1alexbel the puzzle #153 is still not solved.

@h1alexbel the only puzzle #153 is solved here.


Logs might be present as a reference to a ELK or Graylog systems

@rultor release, tag=0.10.0

@rultor release, tag=0.10.0

@dgroup OK, I will release it now. Please check the progress here

@rultor release, tag=0.10.0

@dgroup Done! FYI, the full log is here (took me 5min)


Migrate ESRPCodeSigning to v5

example https://github.com/microsoft/kiota-dotnet/pull/320

WIP
We are migrating the pipeline from microsoftgraph to msazure. We will configure v5 in the new pipeline

Thanks for the update. Once the migration is completed, can you please delete or archive the pipeline in microsoftgraph? This way it won't appear as infringing anymore.

Great call out @baywet. We will do that. cc: @KenitoInc.

Hey @SteveMutungi254. Gentle ping on this upgrade. We have a security item that requires deleting the current service connection you are using for ESRP Code Signing

@Onokaev, @KenitoInc will do this.

Note: the migration work has started here
https://github.com/microsoftgraph/entra-powershell/blob/6716b83f169e627047f8c80b54a0d5f5e880c02f/.azure-pipelines/common-templates/esrp/codesign-nuget-migrate.yml#L16
I believe the older file will eventually be replaced/removed and we can proceed with the deletion of the old connection.
https://github.com/microsoftgraph/entra-powershell/blob/6716b83f169e627047f8c80b54a0d5f5e880c02f/.azure-pipelines/common-templates/esrp/codesign-nuget.yml#L17

Resolved by #1067


Add zip to the list of accepted extra package file types.

R: @chamikaramj

Update documentation of the option ?
https://github.com/apache/beam/blob/master/sdks/python/apache_beam/options/pipeline_options.py#L604

Thank you, updated the documentation. PTAL.

LGTM


generate methods after @Override

completion after @Override helps users to easily override methods from super class, for example:
class MyList extends List {
    @Override
    |<C-x><C-o>
}

will show menu like:
|- ...
|- void add(Object arg1) {
|- Object get(int arg1) {
|- Object set(int arg1, Object arg2) {
|- ...

Yeah, nice idea. We need something like this and much more :smile: . Will add it in todo list.

it great work ,awesome

@artur-shaik after completion and type ENTER, seems to be needless empty line?

@mattn added some extra condition to prevent that.

when after completion,enter works well for me,maybe it is due to my maps
inoremap <silent> <buffer> <CR> <C-r>=MyEnterfunc()<Cr>
function MyEnterfunc()
    if pumvisible()
        return "\<esc>a"
    elseif getline('.')[col('.') - 2]=="{"&&getline('.')[col('.')-1]=="}"
        return "\<Enter>\<esc>ko"
    else
        return "\<Enter>"
    endif
endf

hi @artur-shaik can you make the completion like
protected java.lang.Object java.lang.Object.clone() throws java.lang.CloneNotSupportedException {}

@wsdjeg it might be so, if you press Space it will add close bracket at the same line, and if you press Enter it will add new line and bracket. What do you think?

great, it but it will affected by mapping
if i remove my my inoremap for EnterandSpace,it works very well

and here is result of enter and space ,it seems space also add o new line and will add a space at the end of the method
space

enter

hi @artur-shaik after change my MyEnterFunc(). the plugin works well ,and there is no problem with typping Enterafter completion,but  space seems not word
here is my MyEnterFunc()
function MyEnterfunc()
    if pumvisible()
        return "\<c-y>"
    elseif getline('.')[col('.') - 2]=="{"&&getline('.')[col('.')-1]=="}"
        return "\<Enter>\<esc>ko"
    else
        return "\<Enter>"
    endif
endf


Adding continuous time value-iteration ADP method

Summary
Continuous-time (CT) value-iteration (VI) adaptive dynamic programming (ADP; a.k.a. approximate DP) is implemented [1].
Some parameters are borrowed from the previous code written by T. Bian (in R) [2].
Notes
How to obtain the infimum (minimum)?
Note that the infimum of the approximate Hamiltonian [1, Eq. (16)] and the approximate optimal input [1, Thm. 2] are obtained by numerical optimisation in this implementation. On the other hand, the original implementation [2] relies on the structure of a specific function approximator to perform the optimisation.
This is a (potentially critical) limitation of the original paper.
How to run the code?
Run main/continuous_time_vi_adp.jl.
An example of simulation result

Training progress

adp.V̂.param = [0.0, 0.0, 0.0]
i = 1
adp.V̂.param = [0.6991429619303489; 0.4005297738730431; 0.14493613208036282]
adp.V̂(x) = Polynomial{true, Float64}[0.6991429619303489x₁² + 0.4005297738730431x₁x₂ + 0.14493613208036282x₂²]
i = 2
adp.V̂.param = [0.8777618743585718; 0.40893790123571194; 0.23829955870488628]
adp.V̂(x) = Polynomial{true, Float64}[0.8777618743585718x₁² + 0.40893790123571194x₁x₂ + 0.23829955870488628x₂²]
i = 3
adp.V̂.param = [0.9396217654811783; 0.3500294314726259; 0.32908420114968484]
adp.V̂(x) = Polynomial{true, Float64}[0.9396217654811783x₁² + 0.3500294314726259x₁x₂ + 0.32908420114968484x₂²]
i = 4
adp.V̂.param = [0.9689124227258321; 0.283789244824134; 0.4226847336535787]
adp.V̂(x) = Polynomial{true, Float64}[0.9689124227258321x₁² + 0.283789244824134x₁x₂ + 0.4226847336535787x₂²]
i = 5
adp.V̂.param = [0.98724708467294; 0.22191049831107348; 0.5172381029219096]
adp.V̂(x) = Polynomial{true, Float64}[0.98724708467294x₁² + 0.22191049831107348x₁x₂ + 0.5172381029219096x₂²]
i = 6
adp.V̂.param = [1.0008302225178511; 0.166501881304451; 0.6093622899080263]
adp.V̂(x) = Polynomial{true, Float64}[1.0008302225178511x₁² + 0.166501881304451x₁x₂ + 0.6093622899080263x₂²]
i = 7
adp.V̂.param = [1.0112731880714034; 0.11828864573556205; 0.6952782075797261]
adp.V̂(x) = Polynomial{true, Float64}[1.0112731880714034x₁² + 0.11828864573556205x₁x₂ + 0.6952782075797261x₂²]
i = 8
adp.V̂.param = [1.018699323879966; 0.07816716984803262; 0.7714270710492228]
adp.V̂(x) = Polynomial{true, Float64}[1.018699323879966x₁² + 0.07816716984803262x₁x₂ + 0.7714270710492228x₂²]
i = 9
adp.V̂.param = [1.0229635945512883; 0.04694227707866673; 0.8352098668933136]
adp.V̂(x) = Polynomial{true, Float64}[1.0229635945512883x₁² + 0.04694227707866673x₁x₂ + 0.8352098668933136x₂²]
i = 10
adp.V̂.param = [1.0242378402667032; 0.024679553235220295; 0.8856126280041667]
adp.V̂(x) = Polynomial{true, Float64}[1.0242378402667032x₁² + 0.024679553235220295x₁x₂ + 0.8856126280041667x₂²]
i = 11
adp.V̂.param = [1.0231516780659369; 0.01040933897948123; 0.923321580504069]
adp.V̂(x) = Polynomial{true, Float64}[1.0231516780659369x₁² + 0.01040933897948123x₁x₂ + 0.923321580504069x₂²]
i = 12
adp.V̂.param = [1.0205905389583578; 0.0024068450854986614; 0.9502476179259722]
adp.V̂(x) = Polynomial{true, Float64}[1.0205905389583578x₁² + 0.0024068450854986614x₁x₂ + 0.9502476179259722x₂²]
i = 13
adp.V̂.param = [1.0173925637241983; -0.001240126660707691; 0.9687872753732119]
adp.V̂(x) = Polynomial{true, Float64}[1.0173925637241983x₁² - 0.001240126660707691x₁x₂ + 0.9687872753732119x₂²]
i = 14
adp.V̂.param = [1.0141606607135396; -0.002178849613238054; 0.9812189081177559]
adp.V̂(x) = Polynomial{true, Float64}[1.0141606607135396x₁² - 0.002178849613238054x₁x₂ + 0.9812189081177559x₂²]
i = 15
adp.V̂.param = [1.011234784530039; -0.0016124251794424668; 0.9894000097929393]
adp.V̂(x) = Polynomial{true, Float64}[1.011234784530039x₁² - 0.0016124251794424668x₁x₂ + 0.9894000097929393x₂²]
stop_conds_dict = Dict{Symbol, Bool}(:max_iter => 0, :w_tol => 1)

After training

Notice
The algorithm seems poorly work with other basis function, which is not trivial to obtain the optimiser of approximate Hamiltonian and optimal input.
It will be checked with more clear test codes.
References
[1] T. Bian and Z.-P. Jiang, “Value Iteration, Adaptive Dynamic Programming, and Optimal Control of Nonlinear Systems,” in 2016 IEEE 55th Conference on Decision and Control (CDC), Las Vegas, NV, USA, Dec. 2016, pp. 3375–3380. doi: 10.1109/CDC.2016.7798777.
[2] [https://github.com/JinraeKim/nonlinear-VI](A forked repo of the implemented code by T. Bian)

@hnlee77
Welcome to review this PR.


Epic - Web Accessibility and Internationalization

Linked Stories:

https://github.com/jccjcat/mywebclass-simulation/issues/9
https://github.com/jccjcat/mywebclass-simulation/issues/17

Test the color contrast and alternative text for images
Test the language support and keyboard navigation


License please?

See http://choosealicense.com for help.

Hi bro,
Can I use your code for my project?
I need your permission.
Please check this: https://choosealicense.com/licenses/mit/
Thanks,
Bony

Hi @dalb8 and @bonyhardian ,
I added LICENSE file.


Move tutorial *.cocci to tutorial/

Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

ACK, merging


Updated black version in requirements.txt

I noticed the black version in continuous_integration.yml is different from the version mentioned in requirements file:
https://github.com/keras-team/keras-io/blob/master/.github/workflows/continuous_integration.yml
This is causing differences in formatting of python file done locally vs the formatting check happening as part of the workflow run and causing the check for black to fail
Hence updating requirements file to match version mentioned in continuous_integration.yml

LGTM, thanks


Try the new algolia search

WIP, but it means I can get the a11y bug on search passed

The feel is nicer for sure, will try on mobile when I get a PR build made
Trade-offs:

This uses a bunch of new react things I've never heard of
Introduces CSS vars into the app ( I had to do this eventually anyway for code samples)
Used to ship the JS/CSS for search async, it isn't anymore ish

TODO:

Need to see how having all this bundled affects DLs
Need to verify if it affects running without JS
Need to see what it does re:dark mode

I've been exploring #928 instead


Added annotationFontColor, sumFormat for Pie chart styling

Added annotationFontColor Theme/Styler attribute
Allow the Chart annotations to use a different Font color than the chart font color. This is particularly useful in the Pie chart case with darker colors where a lighter font color looks better but you don’t want to change the chart legend/sum font colors.
In order to avoid breaking any existing cases the AbstractBaseTheme.getAnnotationsFontColor() is set to use getChartFontColor() so this change won't have any affect unless either a custom Theme with a getAnnotationsFontColor() override or an explicit call is made to Styler.setAnnotationsFontColor().
Added sumFormat PieStyler attribute
In order to allow custom “Sum” formats within Pie graphs added a sumFormat attribute to the PieStyler if non-null this acts as a Formatter to calculate the string to be displayed as the sum.
If null (default) then the current behaviour will hold and the sum will be calculated as a number using the PieStyler DecimalFormat.
Within the Formatter the sum value will be passed as a Double.
So for example if the sum for a Pie is 101 then calling.
PieStyler.setSumFormat("Total %.1f GB")
will cause the sum to be displayed as Total 101.0 GB

I'll get to this soon hopefully


fix: clear the session of a signin but non-existent user

When I login with the user and password, then bind the WeCom user.
The session stores the WeCom username, But link operation will not create a new user, normally it should be the original username, at this point, can't logout and can't login

 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.


Questions about the loss curve in pretrain

Hi, Could you please share the pretrain and SFT loss curve of internVL series?
I can not get a make-sense loss curve when pretraining the model, the loss seems to be stuck at 3. Following the instruction here, adapter and ViT are unfreezed while training.
By the way, I have a mount of training data in pretrain stage.

Hi, you can see the SFT training curve of InternVL-Chat-V1-5 here: https://huggingface.co/OpenGVLab/InternVL-Chat-V1-5/tensorboard. Regarding the pretraining curve, I can briefly explain that my loss is around 1.8.
Could you please provide more information about your experiment, such as which language model you used, the batch size, the data, etc.? These factors can significantly affect the loss.

@czczup Hi! I try some hyper params these days, this is my newest pretraining  results:

As above, the loss seems to be stuck at around 2.5.
Currently, my training setting is blow:
  --max_dynamic_patch 12 \
  --drop_path_rate 0.1 \
  --bf16 True \
  --num_train_epochs 1 \
  --per_device_train_batch_size 2 \
  --gradient_accumulation_steps 64 \
  --learning_rate 1e-4 \
  --weight_decay 0.05 \
  --warmup_ratio 0.03 \
  --lr_scheduler_type cosine \
  --dynamic_image_size True \
  --use_thumbnail True \

I used 64 gpus for training, which means that the global batchsize reaches 2*64*64 = 8192
I used about 1M Chinese English image-text pairs which are from open-sourced data such as laion、COCO.
Could you please give some suggestions? Thanks so much!

The pretrain loss of language models of different sizes can vary significantly. For example, the loss of a 2B model is much higher than that of a 26B model.

Yes, this phenomenon is observed during the pre-training phase, and loss decreases very slowly in the later stages of pre-training.

Thanks!


Integration tests for get information about specific catalog versions.

Add Integration tests for get information about specific catalog versions.
Update Integration tests for catalog client.
Relates-To: OLPEDGE-1218.
Signed-off-by: Bautista, Iryna<EMAIL_ADDRESS>

Codecov Report

Merging #124 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #124   +/-   ##
======================================
  Coverage    82.4%   82.4%
======================================
  Files          39      39
  Lines         787     787
  Branches      120     120
======================================
  Hits          649     649
  Misses        104     104
  Partials       34      34

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 73ac3d5...e1ca3af. Read the comment docs.


cranelift: Remove is_pic predicate from x86 backend

👋 Hey,
This PR removes the is_pic predicate from the x86 backend. It isn't used anywhere as a predicate form. We always use the shared flags setting.
It also has a negated form, which was the only remaining use of PredicateNode::Not.

...it's also a bit worrying that the CI didn't pick that up?!

I was worried about CI for a moment after seeing your comment too. But both remaining uses of PredicateNode::Not are in a macro, in branches which were only used by the now-deleted predicate!(!is_pic). Since that macro is never expanded with input matching those cases, the compiler didn't see anything wrong.
We should delete both !$a:ident cases from macro_rules! predicate now, but fortunately the only harm in leaving them there is confusion during maintenance later.

Ah, good!


@MortezaHatamiKia I think  they will not approve this, can you publish it as a new fork/project?


[gltfpacker] Feature request: add support for unlit materials (KHR_materials_unlit extension)

Hi,
In order to test the models from the glTF-Sample-Models more easily I've made a small set of scripts to download the latest models and run them through gltfpacker: https://github.com/TimvanScherpenzeel/gltfpacker-sample-model-test. It is very much hacked together but it does the job, perhaps parts of it can also be useful to you too.
I haven't spotted any bugs or unhandled cases except for a model with unlit materials (using the KHR_materials_unlit extension). I think the objective of the extension is in line with the goals of gltfpacker.
Original

gltfpacker

UnlitTest.zip
Kind regards,
Tim

Oh, this is actually a bug - gltfpack tried to support unlit materials but clearly didn't do this correctly :)


Destination directory /etc/exim4 does not exist

using vagrant ubuntu 1804, I encountered this error:
Install mozilla/sops...
  live done | msg: OK (29335186 bytes)
Enable dm_crypt module...
  live done
Enable compress on logrotate...
  live done | msg: line replaced
Configure timezone...
  live done | msg: executed `/usr/bin/timedatectl set-timezone UTC`
Configure ntp client and restart it...
  live done
Set up exim4 conf...
  live failed | **msg: Destination directory /etc/exim4 does not exist**
restart ntp (via handler)...
restart cron (via handler)...
restart fail2ban (via handler)...

- Play recap -
  live                       : ok=11   changed=9    unreachable=0    failed=1    rescued=0    ignored=0

Replaying it again, here's the log

PLAY [prod] **************************************************************************************************************************************************

TASK [Gathering Facts] ***************************************************************************************************************************************
[DEPRECATION WARNING]: Distribution Ubuntu 18.04 on host live should use /usr/bin/python3, but is using /usr/bin/python for backward compatibility with prior
 Ansible releases. A future Ansible release will default to using the discovered platform python for this host. See
https://docs.ansible.com/ansible/2.9/reference_appendices/interpreter_discovery.html for more information. This feature will be removed in version 2.12.
Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ok: [live]

TASK [nickjj.fail2ban : ensure fail2ban is installed] ********************************************************************************************************
ok: [live]

TASK [nickjj.fail2ban : ensure fail2ban is configured] *******************************************************************************************************
ok: [live] => (item=etc/fail2ban/jail.local)
ok: [live] => (item=etc/fail2ban/fail2ban.local)

TASK [nickjj.fail2ban : ensure fail2ban starts on a fresh reboot] ********************************************************************************************
ok: [live]

TASK [common : Upgrade packages] *****************************************************************************************************************************
ok: [live]

TASK [common : Install packages] *****************************************************************************************************************************
ok: [live]

TASK [common : Install mozilla/sops] *************************************************************************************************************************
ok: [live]

TASK [common : Enable dm_crypt module] ***********************************************************************************************************************
ok: [live]

TASK [common : Enable compress on logrotate] *****************************************************************************************************************
ok: [live]

TASK [common : Configure timezone] ***************************************************************************************************************************
ok: [live]

TASK [common : Configure ntp client and restart it] **********************************************************************************************************
ok: [live]

TASK [common : Set up exim4 conf] ****************************************************************************************************************************
fatal: [live]: FAILED! => {"changed": false, "checksum": "38fce86296304f3db942244f1e945b648f075041", "msg": "Destination directory /etc/exim4 does not exist"}

PLAY RECAP ***************************************************************************************************************************************************
live                       : ok=11   changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

Hello, and thank you for the report.
I don't have a ubuntu 1804 at hand, but I found that exim4 package was not installed at all, while being required.
Could you pull latest version and test it? I have done so in a local Debian installation, and it is working on my side.
Thank you

Hello, and thank you for the report.
I don't have a ubuntu 1804 at hand, but I found that exim4 package was not installed at all, while being required.
Could you pull latest version and test it? I have done so in a local Debian installation, and it is working on my side.
Thank you

It worked thanks

It worked thanks


Add Lurk spec and update Readme

Adds the combined language and circuit spec, which I've named lurk-spec.md, but would like feedback on what it should be called, or if we should keep a more detailed language spec separately.

When running the REPL, should users run the exact same cargo run command as bin/lurkrs? If so, do we need to keep the latter?
Does running in release mode run into the same integer overflow problem noticed by @huitseeker? If so, we'll need to change the fcomm tests to use the Lurk dev profile.

Does running in release mode run into the same integer overflow problem noticed by @huitseeker? If so, we'll need to change the fcomm tests to use the Lurk dev profile.

It's not a problem as much as how overflow checks work by default in Rust.
https://doc.rust-lang.org/rustc/codegen-options/index.html#overflow-checks

When running the REPL, should users run the exact same cargo run command as bin/lurkrs? If so, do we need to keep the latter?

Late reply, but bin/lurkrs is very handy in development. Actually installing with cargo install --path . from time to time is a good idea too — since that gets you a 'good' executable with the right path, etc. Having that available is handy, for example, when a development version is broken or when comparing dev vs previous behavior.


Container resource

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #455
Special notes for your reviewer:
Release note:

This change is

/ok-to-test

/retest

I suggest you create a simple webhook that annotates incoming knservices
with the annotation. This can possibly be setup for the ns you configure
resourcequota on.. that was it is fully decoupled from kf. You could maybe
write a generic annotator wehook that can add annotationa from config to a
configurable resource :)
On Thu, Oct 17, 2019, 3:44 PM Dan Sun<EMAIL_ADDRESS>wrote:

@yuzisun commented on this pull request.
In config/default/configmap/inferenceservice.yaml
https://github.com/kubeflow/kfserving/pull/454#discussion_r336258466:

@@ -70,6 +70,10 @@ data:
]
}
}

deployment: |-
{

   "queueSideCarResourcePercentage": "0.1"

Are you suggesting make Knative to have a global config for the
queue-proxy resources like the storage initializer defaults we did here? We
definitely do not want user to set this annotation on InferenceService
themselves, users do not understand what queue-proxy and they have no idea
how much resources are needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/454?email_source=notifications&email_token=AA2CTBP3T2BE2RKQNFE325DQPDTGJA5CNFSM4JBRTA42YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCIMGAOY#discussion_r336258466,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2CTBOTQYGXF26JWQST2CTQPDTGJANCNFSM4JBRTA4Q
.

I do not think that will work, kfserving creates the knative service resource and if we add another webhook to mutate the service it will cause reconciliation loops. we already have this issue  because knative itself has service mutator, I am also not a fan of keep adding webhooks to walk around issues. Are we ok with removing the config and set the opinioned queue proxy limit in kfserving ?

Are we ok with removing the config and set the opinioned queue proxy limit in kfserving

I am okay would most prefer if we just had a set of annotation key value pairs in config that we apply to the knsvc we create - that allows it to be configurable and allows future annotations to be added (or removed/modified) by individual operators (person setting up the cluster). Also may want to only add the annotation if it not already present.

Are we ok with removing the config and set the opinioned queue proxy limit in kfserving

I would most prefer if we just had a set of annotation key value pairs in config that we apply to the knsvc we create - that allows it to be configurable and allows future annotations to be added (or removed/modified) by individual operators (person setting up the cluster). Also may want to only add the annotation if it not already present.

This may be taking it a bit too far Rakesh. You're talking Dynamic Config here. Can we start simple with an impl detail (not exposed) that sets a sane default request value (10m?)

Well - it should still not set it if it is already set.
On Fri, Oct 18, 2019 at 10:20 AM Ellis Bigelow<EMAIL_ADDRESS>wrote:

Are we ok with removing the config and set the opinioned queue proxy limit
in kfserving
I would most prefer if we just had a set of annotation key value pairs in
config that we apply to the knsvc we create - that allows it to be
configurable and allows future annotations to be added (or
removed/modified) by individual operators (person setting up the cluster).
Also may want to only add the annotation if it not already present.
This may be taking it a bit too far Rakesh. You're talking Dynamic Config
here. Can we start simple with an impl detail (not exposed) that sets a
sane default request value (10m?)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/454?email_source=notifications&email_token=AA2CTBKUKRZC2NJVX7HTHMDQPHV5XA5CNFSM4JBRTA42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBVGYTA#issuecomment-543845452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2CTBPU5QHXSDYBU5O6DJTQPHV5XANCNFSM4JBRTA4Q
.

--
Rakesh Kelkar

Are we ok with removing the config and set the opinioned queue proxy limit in kfserving

I would most prefer if we just had a set of annotation key value pairs in config that we apply to the knsvc we create - that allows it to be configurable and allows future annotations to be added (or removed/modified) by individual operators (person setting up the cluster). Also may want to only add the annotation if it not already present.

/retest

/retest

/retest

/retest

Hi @yuzliu
I checked E2E tests, That's strange. The deployment is not created for the kfservice:
[root@jinchi1 ~]# kubectl get inferenceservice -n kfserving-ci-e2e-test -o yaml
apiVersion: v1
items:
- apiVersion: serving.kubeflow.org/v1alpha2
  kind: InferenceService
  metadata:
    creationTimestamp: 2019-10-21T02:39:23Z
    generation: 1
    name: isvc-tensorflow-test
    namespace: kfserving-ci-e2e-test
    resourceVersion: "6382"
    selfLink: /apis/serving.kubeflow.org/v1alpha2/namespaces/kfserving-ci-e2e-test/inferenceservices/isvc-tensorflow-test
    uid: fd9d2e0e-f3ab-11e9-8f8d-42010a8e008b
  spec:
    default:
      predictor:
        tensorflow:
          resources:
            limits:
              cpu: "1"
              memory: 1Gi
            requests:
              cpu: 100m
              memory: 200Mi
          runtimeVersion: 1.14.0
          storageUri: gs://kfserving-samples/models/tensorflow/flowers
  status: {}
kind: List
metadata:
  resourceVersion: ""
  selfLink: ""
[root@jinchi1 ~]#
[root@jinchi1 ~]# kubectl get deployment -n kfserving-ci-e2e-test
No resources found.
[root@jinchi1 ~]# kubectl get route -n kfserving-ci-e2e-test
No resources found.

And I guess that's not caused by resource limites.
[root@jinchi1 ~]# kubectl get nodes
NAME                                                  STATUS    ROLES     AGE       VERSION
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1   Ready     <none>    35m       v1.13.11-gke.5
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-k0f2   Ready     <none>    35m       v1.13.11-gke.5
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-xdcv   Ready     <none>    35m       v1.13.11-gke.5
[root@jinchi1 ~]# kubectl describe nodes  gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1  |grep -i cpu
 cpu:                        4
 cpu:                        3920m
  Namespace                  Name                                                              CPU Requests  CPU Limits    Memory Requests  Memory Limits
  cpu                        1469m (37%)       10146m (258%)
[root@jinchi1 ~]# kubectl describe nodes  gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1  |grep -i memory
  MemoryPressure                False   Sun, 20 Oct 2019 19:55:45 -0700   Sun, 20 Oct 2019 19:19:19 -0700   KubeletHasSufficientMemory      kubelet has sufficient memory available
 memory:                     15398568Ki
 memory:                     12699304Ki
  Namespace                  Name                                                              CPU Requests  CPU Limits    Memory Requests  Memory Limits
  memory<PHONE_NUMBER> (12%)  7104202Ki (55%)
  Normal  NodeHasSufficientMemory  36m   kubelet, gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1     Node gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1 status is now: NodeHasSufficientMemory

the contoller works fine:
[root@jinchi1 ~]# kubectl get pod -n kfserving-system
NAME                             READY     STATUS    RESTARTS   AGE
kfserving-controller-manager-0   2/2       Running   1          22m

I think the controller logs can help us, but seems I do not have premission to get logs...

Thanks for looking @jinchihe! I see cpu limit is already 258% and looks like we only have 4 cpus. I think we can call kubernetes python client to get the logs if we do not have permission, it should be helpful for debugging when tests fail.

Hi @yuzliu
I checked E2E tests, That's strange. The deployment is not created for the kfservice:
[root@jinchi1 ~]# kubectl get inferenceservice -n kfserving-ci-e2e-test -o yaml
apiVersion: v1
items:
- apiVersion: serving.kubeflow.org/v1alpha2
  kind: InferenceService
  metadata:
    creationTimestamp: 2019-10-21T02:39:23Z
    generation: 1
    name: isvc-tensorflow-test
    namespace: kfserving-ci-e2e-test
    resourceVersion: "6382"
    selfLink: /apis/serving.kubeflow.org/v1alpha2/namespaces/kfserving-ci-e2e-test/inferenceservices/isvc-tensorflow-test
    uid: fd9d2e0e-f3ab-11e9-8f8d-42010a8e008b
  spec:
    default:
      predictor:
        tensorflow:
          resources:
            limits:
              cpu: "1"
              memory: 1Gi
            requests:
              cpu: 100m
              memory: 200Mi
          runtimeVersion: 1.14.0
          storageUri: gs://kfserving-samples/models/tensorflow/flowers
  status: {}
kind: List
metadata:
  resourceVersion: ""
  selfLink: ""
[root@jinchi1 ~]#
[root@jinchi1 ~]# kubectl get deployment -n kfserving-ci-e2e-test
No resources found.
[root@jinchi1 ~]# kubectl get route -n kfserving-ci-e2e-test
No resources found.

And I guess that's not caused by resource limites.
[root@jinchi1 ~]# kubectl get nodes
NAME                                                  STATUS    ROLES     AGE       VERSION
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1   Ready     <none>    35m       v1.13.11-gke.5
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-k0f2   Ready     <none>    35m       v1.13.11-gke.5
gke-zing-presubmit-e2e-4-default-pool-0beb0b76-xdcv   Ready     <none>    35m       v1.13.11-gke.5
[root@jinchi1 ~]# kubectl describe nodes  gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1  |grep -i cpu
 cpu:                        4
 cpu:                        3920m
  Namespace                  Name                                                              CPU Requests  CPU Limits    Memory Requests  Memory Limits
  cpu                        1469m (37%)       10146m (258%)
[root@jinchi1 ~]# kubectl describe nodes  gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1  |grep -i memory
  MemoryPressure                False   Sun, 20 Oct 2019 19:55:45 -0700   Sun, 20 Oct 2019 19:19:19 -0700   KubeletHasSufficientMemory      kubelet has sufficient memory available
 memory:                     15398568Ki
 memory:                     12699304Ki
  Namespace                  Name                                                              CPU Requests  CPU Limits    Memory Requests  Memory Limits
  memory<PHONE_NUMBER> (12%)  7104202Ki (55%)
  Normal  NodeHasSufficientMemory  36m   kubelet, gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1     Node gke-zing-presubmit-e2e-4-default-pool-0beb0b76-0zr1 status is now: NodeHasSufficientMemory

the contoller works fine:
[root@jinchi1 ~]# kubectl get pod -n kfserving-system
NAME                             READY     STATUS    RESTARTS   AGE
kfserving-controller-manager-0   2/2       Running   1          22m

I think the controller logs can help us, but seems I do not have premission to get logs...

Thanks for checking! @ellis-bigelow @yuzisun do you happen to know how to check the controller logs in the cluster that we run e2e tests? This pr keeps failing e2e tests because of this error
name = 'isvc-tensorflow-test', namespace = 'kfserving-ci-e2e-test'
Timeout_seconds = 600
   def wait_for_kfservice_ready(name, namespace='kfserving-ci-e2e-test', Timeout_seconds=600):
       for _ in range(round(Timeout_seconds/10)):
           time.sleep(10)
           kfsvc_status = KFServing.get(name, namespace=namespace)
           status = 'Unknown'
           for condition in kfsvc_status['status'].get('conditions', {}):
               if condition.get('type', '') == 'Ready':
                   status = condition.get('status', 'Unknown')
           if status == 'True':
               return
>       raise RuntimeError("Timeout to start the KFService.")
E       RuntimeError: Timeout to start the KFService.

However, I was able to start a kfservice in my local k8s cluster. I'd love to check the e2e cluster if I have permissions.

@yuzliu, can you help print out the status object of inference service test instance or underlying knative service after the test failure ? hopefully the error message is propagated there.

/retest

The e2e tests failed because of the knative webhook:
  {"level":"error","ts":1571669305.8146658,"logger":"kubebuilder.controller","msg":"Reconciler error","controller"
:"kfserving-controller","request":"kfserving-ci-e2e-test/isvc-tensorflow-test","error":"Internal error occurred: admission webhook \"webh
ook.serving.knative.dev\" denied the request: mutation failed: expected 0.1 <= 0.1 <= 100: spec.template.metadata.annotations.queue.sidec
ar.serving.knative.dev/resourcePercentage"

it seems that setting DefaultQueueSideCarResourcePercentage = 0.1 is too low.

/retest

@jlewi, what are Kubeflow's rules on getting some of our members access to the CICD Clusters? We'd love to be able to debug more efficiently her.

@jlewi, what are Kubeflow's rules on getting some of our members access to the CICD Clusters? We'd love to be able to debug more efficiently her.

@ellis-bigelow I granted Dan, me and another team member CICD access here: https://github.com/kubeflow/internal-acls/pull/181/files. :)
I also figure out why it worked in my local k8s cluster(using knative 0.7) but failed in e2e(using knative 0.9).
There is a range check added for QueueSideCarResourcePercentageAnnotation on July-28 which is after 0.7 was released: https://github.com/knative/serving/commit/52a2451c0839422981a96719c47ed6e8aba39799#diff-7f8909ce9aa68bb686c8723cfddc0367R49.
This check ensures QueueSideCarResourcePercentageAnnotation to be larger than 0.1 and smaller or equal to 100. So my default value 0.1 failed the check. I updated the default value to 0.2 and the e2e passed!

/retest

@ellis-bigelow I addressed new comments and passed e2e! Hope this PR is finally good to go!

/lgtm
/approve


After update V1.07 iammeter doesnt work

Hi,
I upgraded V1.07, after reboot i received this error.:
My iammeter  iammeter W3080T 3 phase meter.
Firmware Ver.: | ATVi.75.98.1@MX1290
Logger: homeassistant
Source: custom_components/iammeter_modbus/init.py:189
Integration: IamMeter Modbus (documentation, issues)
First occurred: du. 3:21:28 (4 occurrences)
Last logged: du. 3:25:24
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/config/custom_components/iammeter_modbus/init.py", line 143, in async_refresh_modbus_data
update_result = self.read_modbus_data()
^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/iammeter_modbus/init.py", line 173, in read_modbus_data
return self.read_modbus_holding_registers()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/iammeter_modbus/init.py", line 189, in read_modbus_holding_registers
inverter_data.registers, byteorder=Endian.BIG
^^^^^^^^^^
AttributeError: type object 'Endian' has no attribute 'BIG'

I deleted, re add, the integration not helped, i rebooted my W3080T but cant help me.
My operation system is Debian 11, HA in docker, HA version 2023.9.1
I downgrade v1.06,v1.05 but cant work again.
Can you help me please solve this problem?
Thank You Zsolt

Here is the solution, BUT I don't dare to update, v1.07
1, Remove the integration from HA
2, Remove the integration from HACS
3, Reboot HA.
4, Install integration in HACS
5, Add integration HA.
7, = WORK AGAIN this version v1.0.6
I don't know upgrade v1.0.7 or not.

because "requirements": ["pymodbus"] version is lower than needed,should fix in next iammeter_mobus release.
or change it in file manifest.json:
"requirements": ["pymodbus>=3.6.6"]

fixed in released version 1.0.10.


get_filtered_columns_in_relation returns empty string rather than empty list during parsing

Describe the bug
The get_filtered_columns_in_relation macro is designed to return an array of columns. When dbt is parsing the project (execute is False), the return value is an empty string (''), rather than an empty list ([]).
This can cause issues when the result is used in code and other macros that validates its arguments, such as the warning message in the (deprecated) surrogate_key macro.
It is expected that the macro returns the same data type, regardless of the execution mode.
Steps to reproduce

Initialize a dbt project
Create models/foo.sql:

{% set my_columns = dbt_utils.get_filtered_columns_in_relation(from=ref('bar')) %}
{% if my_columns is not string %}
  select 'success!'
{% else %}
  {{ exceptions.warn("Invalid type of 'my_columns'")}}
{% endif %}

Create models/bar.sql:

select 'other' as col

dbt run

Expected results
The model to be created without a warning.
Actual results
A warning that appears during the parsing phase:
Invalid type of 'my_columns'

Screenshots and log output

System information
packages:
  - package: dbt-labs/dbt_utils
    version: 0.8.4

Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[x] other (specify: netezza)

The output of dbt --version:
Core:
  - installed: 1.1.2
  - latest:    1.3.2 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Plugins:
  - netezza: 1.1.0 - Could not determine latest version

Additional context
This is where the issue is: https://github.com/dbt-labs/dbt-utils/blob/main/macros/sql/get_filtered_columns_in_relation.sql#L11
Are you interested in contributing the fix?
Sure!

Hey @joemirizio, I think this is a safe change to make! Give it a whirl and let's see how the CI goes - but if we release it and it turns out that it interferes with some existing behaviour somewhere we might have to revert in a speedy patch release.
I assume that

the warning message in the (deprecated) surrogate_key macro.

is not the only place you're seeing difficulty? I'm not that concerned about a deprecated macro having a bad time... everyone's meant to stop using it anyway!


Fixed #24294 -- Updated staff_member_required decorator

Updated staff_member_required decorator to work similarly to
login_required decorator -- handle both no-args and with-arguments
calls. Added
test_staff_member_required_decorator_works_with_argument() test.

buildbot, test this please

Buildbot, test this please.

merged in 08572e8d125da204fab230be5d1fdcc5d8d22256, thanks.


Fix firefox focus bug.  Fixes #63

Don't update model if search string hasn't changed.  (See comments in code).
Fixes #63

Perfect thanks :+1:

You're welcome.


Editor line spacing is too low

Description
Here is Xcode (on the left) and CodeEdit (on the right) comparison:

To Reproduce
Check self-explanatory screenshot
Expected behavior
CodeEdit line spacing should match Xcode one
Version information
CodeEdit: 0.0.1(11) (Git commit https://github.com/CodeEditApp/CodeEdit/commit/8911e11435ff1c24db1a9fa095c548be0c7723f7)
macOS: 13.1
Xcode: 14.2
Additional context
No response

Looks like this has been implemented, it just needs a preferences option to configure.
We could also increase the default value to closer match Xcode
Code that needs prefs:
https://github.com/CodeEditApp/CodeEdit/blob/8911e11435ff1c24db1a9fa095c548be0c7723f7/CodeEdit/Features/CodeFile/CodeFileView.swift#L72

I'll implement the preferences option

@lukepistrol  feel free to assign.

@ben-p-commits Let’s go with a range stepper field of 0.75-2 with a 0.05 increment for the settings control.

@austincondiff - do we want a default value of 1.2, as per the previous constant binding used?

It looks like Xcodes default is 1.45. See if that looks good and if so, let's go it it.

I wonder, do we instead want to go with a point value instead of multiplier?
VS Code handles this in the most complex way possible...

Wow, that is painful from VSCode...

Already published a PR using multiples, but happy to play with explicit point values- but I think multiples are more easily understood by users.

@vitalyster , @austincondiff , @lukepistrol - this has merged in #955 - can be closed.

Tested at b11127cd0020:

default line spacing still the same and too low
Settings shows 1,45 and allow increase to 2,00, reasonable spacing matching Xcode is 1,8-1,9, are you sure these numbers are correct?
Changes are not applied to opened tabs: I need to change value, then close editor tabs and reopen to apply changes.

I came up with that value by taking a screenshot of Xcode, throwing it in Figma, overlaying text with the same font and size, and playing with the line height. 1.45 seemed to match up correctly. Now I suppose it is possible that Xcode and Figma calculate this differently but I'd have to see for myself.


Change public key and private key prefixes

The fully qualified string serialization of a public or private key needs a prefix before the base58 encoded key data. This prefix currently includes a base prefix "EOS" followed by a string specific to the particular key type (currently "K1" and "R1").
There are a few problems with this approach.
First, there is no way for a user to immediately distinguish between a public key and a private key. The string representations of public/private keys in BitShares and Steem (and one version of string representations of public/private keys that EOS must support for backwards compatibility reasons) does not have this problem because a public key (e.g. EOS6cwVAGKJauz4Nk6eMNHPbSWV4MU2LEQTZu9F1zFHjvkh47BXZU) is clearly distinguishable from a private key (e.g. 5JYpogarMyZyLFGduXPs36Jpkwyy3agpsmDn7bVGkXqB2KX36Qu) just by quickly glancing at it. Not being able to quickly distinguish the two types of keys has security implications to users because they are more likely to accidentally copy and paste and send their private key to someone (or the public internet) when they intended to send their public key.
Second, the way we currently match keys to the particular key type means that we could not use certain prefixes when we wanted to add new key types in the future. For example, if a hypothetical key type with the prefix "EOSK12" was added, it would never be considered valid by our system because the matcher would match it with the "EOSK1" prefix, fail since it isn't a K1 key, and not try any of the other prefixes in the list. This is a bug in the current code that needs to be fixed.
Third, it is not clear where the prefix ends and the Base58 string beings. The prefix should really end with a character that is not a valid character in Base58.
The proposed redesign of our representation of public keys, private keys, and signatures is as follows:
Public keys will be represented as EOS.<key_type_id>.<base_58_encoding_of_public_key_data>.
Private keys will be represented as PRI.<key_type_id>.<base_58_encoding_of_private_key_data>.
Signatures will be represented as SIG.<key_type_id>.<base_58_encoding_of_signature_data>.
<key_type_id> in the above representations are strings like "K1", "R1", and whatever new key types we add in the future.
For backwards compatibility reasons we still need to support public keys and private keys without prefixes. These will continue to default to the K1 keys.

Fixed via PR #2441


Make it more module-like

At the techtalk tonight, there was a proposal to make this more dry by making this more thin, i.e. just do what the tauri action needs, and nothing more. (let the release action be external)
having an implementation example would help

The release is optional. Also I've discussed this with Jacob and we agreed that it's good to have the release option since the action ensures the build passes before making a release, simplifying workflows.


Builtin function or method in the Sentry PHP SDK to check if Sentry is initialized.

Problem Statement
Builtin function or method in the Sentry PHP SDK to check if Sentry is initialized.
Solution Brainstorm
Something like :
\Sentry\isSentryInitialized():bool

What would your use-case for such a function look like?


Slick refresh not reinitiating all events

I have several custom functions that run on , init, breakpoint, afterChange and beforeChange
and none of them are reinit if I use slick('refresh')
in this demo on init I added dots ul class dots-green ,   once you click the refresh button the class is not applied again
https://jsfiddle.net/Themezly/s0ose0mj/
I was under the impression that refresh is refreshing the slider instance. Or ?

There is an on('reInit') event for refreshes.

@kenwheeler , try it please, I cant get it to work , getting
Cannot read property 'apply' of undefined

if this is what you had in mind , it does not work for me
$(".refresh").on('click', function(e) {
  e.preventDefault();
  $slider.slick('refresh');

});

$slider.on('reInit', function(event, slick) {
		console.log('reinitalized');
});

https://jsfiddle.net/Themezly/s0ose0mj/2/

Yeah theres definitely some shenanigans there.

Any progress on this?

Can we get a hint for a fix or suggestion? I need to resell the theme soon and cant do this without the fix.

I'm having this issue as well.. I solved it with an if in the destroy function prototype:
I added an if statement around the _.cleanUpEvents(); function call
if(!refresh) {
    _.cleanUpEvents();
}

Looks like this fixes it for me, but I don't know if there are possible side-effects!

@kino90 , what line bud , where did you add the check ?

@danyj Around line 850, look inside Slick.prototype.destroy

On my radar

PRs obv welcome ;)

I see.
This "if" is used to tell the script to skip the cleanup of the events, it won't fire "init" again

@kino90  seems like you are leading us in right dir , the cleanupevents should do reinti, maybee
_.reinit();
needs to be reapplied in one of the cleanup methods

@leggomuhgreggo  did you had a chance to take a look at this ?

this one is still very annoying, cant do any refresh , reint or setOption, it is not reinitiating default events
anyone please advise. I tried many things in 11 months and just cant get it to work right. Even a hint is much appreciated.

if(!refresh) {
     _.cleanUpRows();
}

Well this function contains an empty() function so you could try to add this on line 906.

@danyj -- In response to https://github.com/kenwheeler/slick/issues/2678#issuecomment-270200432 -- According to the readme, and peeking in the code, "refresh" isn't a method name that can be called, rather it is a boolean param.
These options appear to work for me (v 1.8.1):
// $slider is a jQuery element that has had the slick slider applied to it
// Setup the event handler...
$slider.on('reInit', () => { console.warn('reinit!'); });
// option 1 - the most clear, but undocumented
$slider.slick('reinit');
// option 2 - explained in the readme, but not so clear
$slider.slick('setOption', {}, true); // the last param is `refresh`

Relevant function: https://github.com/kenwheeler/slick/blob/v1.8.1/slick/slick.js#L2111
Since it seems that any of the methods can be called by .slick(methodName), it would be nice if some more of them were documented.


Failed to follow the installation instructions

Hi There
I tried to get to know the script-languages-container-tool but failed to follow the installation instructions. Specifically the second Step:
"Then you can use the Python package itself or install the starter scripts which allow to run exaslct within a docker image: exasol_script_languages_container_tool.main install-starter-scripts --install-path $YOUR_INSTALL_PATH"
Could you please elaborate on the two options? How to use the Python package itself? How to install the started scripts?
Thanks,
floriaso

Hi @floriaso,
Thank you for your question. You have two options to use the container-tool. You can install it as Python package, then you can call use it via
python3 -m exasol_script_languages_container_tool.main <command> ...

You can either already use the command build, export, ... directly.
Or, alternatively, you can install starter scripts. These starter scripts work standalone without the package installed. We use them in other GitHub Repositories, when we want to provide a CLI for users without them installing another Python. These starter scripts start a bootstrap Docker Container and run the container-tool inside the container.
For example, in the https://github.com/exasol/script-languages-release/ Repository we already installed the starter scripts and you only need to call ./exaslct .... in the root of the Repository.
I hope this helps. Please let me know, when you have more questions.

@tkilias, do we need to update documentation (e.g. add a link to a user guide)? If not, I think you can close this ticket.

@redcatbear I think, we should clarify this in the documentation a bit. I will create a follow-up ticket.


Speeding up docs build in CI using caching.

FOR ISSUE #32687
Description
This speeds up the 'Docs Build' step of the docs build job by caching the results of building the docs from prior actions.
This kind of 'skips' the step 'build docs' altogether (in the successful case where there is a cache hit) and doesn't really provide any improvement to speeding up building the docs in case they were modified.
I didn't know if that's what you guys were going for but I thought it could be as a decent amount of the contributions made don't affect building the docs directly. In those cases it speeds up the action by eliminating building the docs entirely. (Drawback here is another element present in the cache fields.)
I was also thinking of other approaches to speed things up. If this isn't sufficient / what you're looking for I can find a different approach and improve it further! Give me any feedback you have I'll go back to work on this.
Verification
I tested the action on my own fork, which showed that it cached correctly and provided an elimination of the 'build docs' step in case of cache hit. It might take two runs to store the cache properly but barring that it seems to be a success.

This is close but not quite right.
We want to build able to incrementally build using the cache, like we can with our scons build cache.

I took your feedback and tried to improve it further. I think I've got it pending your review but I just wanted to provide some explanation behind the edits + broad strokes bullet points:

It turns out Sphinx already implements caching implicitly! The reason why it didn't manifest itself in building the docs for comma was that the build directory would be deleted on every build. That is where the automatically generated caches (or pickles) (.doctrees) were stored. I stopped that behavior and now it runs much faster.
I also added in parallelism at the sphinx-build call, but when I had this undergo testing on my fork it didn't change anything about the performance (probably because parallelism is already done through the github action). This can be removed to clean up the code.
I went through the route of looking at some optimizations that could affect the theme but I figured those were design decisions and I didn't want to intrude.
I changed the paths of some of the directory alias definitions to more closely follow definitions I saw in the docs.

When undergoing testing on my fork, this seems to have resulted in the same results with faster compilation time.
Let me know if you have any more feedback!

Sounds good, I'll review once you mark it as ready for review.

I went through the route of looking at some optimizations that could affect the theme but I figured those were design decisions and I didn't want to intrude.

We basically got it setup and haven't touched it since, so design changes are welcome!

I see a whole bunch of warnings and errors; are those not new?
https://github.com/commaai/openpilot/actions/runs/9589108324/job/26442355846?pr=32766

They seem to be the same warnings / errors that were present in the sample you showed in the issue:
https://github.com/commaai/openpilot/actions/runs/9454981147/job/26043750673
However I was worried about that. I was going to try to debug a little bit to see if any of those were related to my implementation before reviewing.

Just found a serious issue @adeebshihadeh, going to have to work on this a bit more. Sorry for prematurely setting up the PR!

No worries, take your time :)
You can also share your new docs on Discord and ask people to poke around for bugs

All right, fixed the issue! When I test the generated html through local hosting, everything seems equivalent. (Compared it side by side with the docs available online). I'll make a post on the discord for anyone to take a look as well but otherwise I think this is ready!

It looks like this now fixes the docs build system to build incrementally, but I don't see how it caches in CI to take advantage of that?

Initially, when I started this PR I thought the solution to speeding up 'build docs' lay somewhere in the github action. Once you mentioned incrementally building the cache I gave up speeding anything up through modifying the CI implementation and pivoted to trying to fix sphinx-build's behavior. (And why it wasn't retrieving its cache for incremental builds)
It seems like fixing sphinx-build has enabled the 'build docs' step to reach the target time (even though I don't do any explicit caching in the CI) because sphinx automatically builds a cache in the form of the doctrees directory, which is present in the build directory that the github action outputs. Now that we're not removing the build directory on every iteration of running the action, we get the CI-level caching for free.
A lot of this is stuff I've learned in the past day or so, but I believe it to be correct. I'm no expert so please provide any feedback / correct me if I'm completely wrong.


know database table name

in this table definitions, whats my table name in sqlite? is this User? or userTable?
@DataClassName("User")
class UserTable extends Table{
  IntColumn get id =>integer().autoIncrement()();
  ...
}

When not explicitly overridden, we use the snake_case of the class name, which would be user_table in your example. You choose a custom table name by overriding tableName:
@DataClassName("User")
class UserTable extends Table{
  @override
  String get tableName => 'users'; // table will now be called users
  IntColumn get id =>integer().autoIncrement()();
  ...
}

For more information, see the docs on tableName. If you need to read the table name from Dart, you could use something like
@UseMoor(tables: [UserTable])
class Database extends _$Database {
  // constructor + schema version getter omitted
  String get sqlNameOfUserTable => userTable.actualTableName;
}

@simolus3 i wish documentation better than with current version :)
for this issue i don't have any question, thanks


Split out event generation to separate Go package

As suggested by @mweagle [1], it would be nice to split out the Lambda event generation in event.go into a separate Go package so it can be used within other applications as a library.
[1] https://twitter.com/mweagle/status/935345253451644928

Thanks for considering this and I'm happy to help PR/review/etc.

This is already available as a separate Python module in this repo. I haven't seen a lot of interest in this feature. So closing it for now. Feel free to re-open if you are interested in working on this :)


Error while trying to run node

Hi,
I installed sharp by:
npm install sharp
Then when I try to run the nodejs program, I get the following error:
Error: The module '/opt/captivePortal/node_modules/sharp/build/Release/sharp.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 46. This version of Node.js requires
NODE_MODULE_VERSION 57. Please try re-compiling or re-installing
the module (for instance, using npm rebuild or npm install).
at Object.Module._extensions..node (module.js:681:18)
What should I do?
Please let me know what further info is needed to narrow down the issue.
Thanks

My best guess would be that there are two conflicting versions of Node on this machine.


Update hololens2-purchase.md

Updated link for "Buy HoloLens 2 - SHI International Corp." to point to https://www.shi.com/Product/38709629/Microsoft-HoloLens-2

Old link is a 404 error on the vendors' website - Updated link for "Buy HoloLens 2 - SHI International Corp." to point to https://www.shi.com/Product/38709629/Microsoft-HoloLens-2


bearer token proposal

Add bearer token auth to the service broker.

@rthallisey I updated the docs to clarify the questions section, and I tried to answer your question about AuthProvider.


Render passes are very inefficient; re-renders entire canvas every frame!

The goal here should be to only re-render the small part of the canvas that has changed (been drawn on) between each frame.
Here are two helpful blog posts about smoothing (and other things) on Android:
https://medium.com/square-corner-blog/smooth-signatures-9d92df119ff8 (see the "Surgically Invalidate" section)
https://medium.com/square-corner-blog/smoother-signatures-be64515adb33 (see the "Responsiveness" section)
Note that the same principles apply to iOS - but the methods will differ, slightly. For example, in one project I used the UIView object's setNeedsDisplayInRect: and specified only the rect/space representing the space between the last two touches to be re-rendered.

I've started working on this.
Here's a first iOS implementation: https://github.com/crossfield/react-native-sketch-canvas/commit/7ef9bb4e77bc169b7bef6aeff239879bc6f58dfb
It dramatically improves the performance.

I've implemented it first Android as well: https://github.com/crossfield/react-native-sketch-canvas/commit/1b14386bb9088f1466463742c10b65cc4d425e66
Now I need to restore the line smoothing which I lost by doing the incremental drawing.

Line smoothing is back:

iOS: https://github.com/crossfield/react-native-sketch-canvas/commit/d774c8a314ea749c2bf08c63e0ca3fab14acae59
Android: https://github.com/crossfield/react-native-sketch-canvas/commit/b6aff9f2e878fdbb32356358d8687ff7ad0252ae

@jeanregisser This is VERY amazing! Would you mind making a PR into this repo? @terrylinla has been good about reviewing & merging what gets submitted (although sometimes it takes a few weeks).
Also I'll test this in our app soon. :)

I have merged into repo and published to npm! It's really awesome!!

It looks awesome! Making a PR is welcome!

I'll send a PR as soon as I can!

Thanks a lot!! I appreciate your contribution!


Move __swiftPMEntryPoint() to its own file.

Tidying up the entry point code and moving the SwiftPM-specific entry points to their own file. Renames internal functions to not mention SwiftPM if they also support the general entry point mechanism.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test


Using asciimatics as a progress bar placeholder

Is it possible to use asciimatics as a progress bar placeholder? What I mean is say I have
status = GetActivityStatus() #returns True when done
while not status
    #show asciimatics display
    status = GetActivityStatus()

#When the loop exits the asciimatics display is exited automatically

I can't seem to figure out if this is possible from the examples.
Thanks!

The problem that you've got is that I've yet to expose a method to draw the next frame, so you need to reverse the logic and put your status into an Effect.  For example:
from asciimatics.effects import Print
from asciimatics.renderers import BarChart
from asciimatics.scene import Scene
from asciimatics.screen import Screen
from asciimatics.exceptions import ResizeScreenError, StopApplication
import sys
from random import randint

# Simple progress indicator - from 0 to 100
progress = 0

def get_progress():
    global progress
    if randint(0, 100) > 10:
        progress += 1
    return progress

class ProgressBar(Print):
    def __init__(self, screen):
        super(ProgressBar, self).__init__(
            screen,
            BarChart(
                7, 60, [get_progress],
                char=">",
                scale=100.0,
                labels=True,
                axes=BarChart.X_AXIS),
            x=(screen.width - 60) // 2,
            y=(screen.height - 7) // 2,
            transparent=False, speed=2)

    def _update(self, frame_no):
        global progress
        if progress >= 100:
            raise StopApplication("100% complete")
        else:
            super(ProgressBar, self)._update(frame_no)

def demo(screen):
    scenes = []
    effects = [
        ProgressBar(screen)
    ]
    scenes.append(Scene(effects, -1))
    screen.play(scenes, stop_on_resize=True)

while True:
    try:
        Screen.wrapper(demo)
        sys.exit(0)
    except ResizeScreenError:
        pass

@erturpin was that helpful?  Can I close this down now?

with no response in 2 weeks, I'm assuming that this can now be closed...

Thanks was helpful for me


I can  resolve the issue in log in and sign up.

It is taking the email address even if the format is not complete or in the correct format.

Thankyou for assigning me .


ref #114: Add cache clear command

Q
A

Branch
6.2.x

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Fixed issues
chameleon-system/chameleon-system#114

License
MIT

This is a feature, not a fix - that's OK here as the feature is completely independent, but please adjust the PR description :-)

Thanks @UlrichKu


Add documentation documentation regarding High Concurrency Sessions

To assign to Amr

Done

High concurrency sessions will be handled by #78


[FLINK-10325] [State TTL] Refactor TtlListState to use only loops, no java stream API for performance

What is the purpose of the change
Refactor TtlListState to use only loops instead of java stream API to exclude any performance impact
Brief change log

Refactor TtlListState.updateTs() to use loop
Refactor TtlListState.collect() to use loop
auxiliary methods in TtlUtils

Verifying this change
existing tests
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Hi @azagrebin ,does for loop perform better than stream api ?  And I have not found an existing test for the performance of TtlListState, may be we should add one to verify this change ?

Hi @Aitozi, loop does perform better than stream api, but the gain by using loop seems so little with the operations like update and collect. However, for the simplicity, I prefer the  stream api. What do you think? @azagrebin

@Clarkkkkk the TtlListState is somewhat performance critical and, thus, it makes sense imo to improve it as much as possible. One of the problems was also that we queried the timestamp for each element when updating it. With Andrey's changes this won't happen.

@tillrohrmann Thanks for your reply. Sounds reasonable.


examples/text/html_label.py exception (with fix)

Describe the bug
Running the test html_label.py, I get an exception at line 475 of https://github.com/pyglet/pyglet/blob/v<IP_ADDRESS>/pyglet/text/layout.py
Changing the code from:
self.element.place(layout, x, y, z)
to:
self.element.place(layout, x, y)
Fixed the issue
System Information:
Windows 10, x64
How To Reproduce
cd examples/text
python html_label.py

Added a fix in PR #736


Implement filter logic for state- Emissions filter logic

Definition of Ready

[x] Does this meet the INVEST criteria?
[x] Does this have a clearly defined user?
[x] Does this have an estimate?
[x] Does this have acceptance criteria?
[x] Does this have sufficient context for the specific functionality being developed (design, business rules, etc.)
[x] Does this have a dev link been provided in the design?
[x] Is this traceable to a feature, epic, or PI?

User Story
As a custom data download user I want the filter logic to function for state so that I can avoid producing a query with no results
Acceptance Criteria
Given that a user has selected an emissions subtype
When the functionality is displayed
Then the user is required to apply filter criteria for time period first
AND the optional filters are greyed out until time period criteria is applied
Given that a user has selected an emissions subtype on the custom data download
When filter criteria is applied for time period
Then the filter logic is engaged
AND the state filter is narrowed based on the selected time period and subsequent selections
Given that the filter logic is engaged
When the state filter panel functionality is displayed
Then non applicable selections are removed from the combo box
Given that the filter logic is engaged
When filter criteria is applied for state
Then the optional filters are narrowed based on the applied state filter criteria
Given that the filter logic is engaged
When the state filter tag is removed
Then the optional filters are narrowed based on applied criteria
Development Notes
Does this Story require 508 Testing?

[ ] Implementing a new authoring tool that will produce code / content for the application
[ ] Implementing a 3rd party widget that will be part of the application
[x] Adding new / updating custom functionality
[ ] Adding new / updating a page template
[ ] Adding new / replacing existing functionality (unless the replacement has already passed 508 testing)
[ ] Full Site Review prior to a scheduled EPA 508 review
[ ] Planning a video that will appear on the application
[ ] Producing a video that will appear on the application
[ ] Producing Training materials
[ ] Producing documents

Context
(User journeys, design mockups, architectural diagrams, business rules, data element definitions)
Testing
Test Case: TC432
Tester: @JanellC
Definition of Done

[x] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[x] Has the code been unit tested and peer reviewed?
[x] Have the functional tests been executed?
[x] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Is the user story ready to be deployed to the test/staging environments?

TC432 failed @tlageman-epa . I logged Bug: Implement filter logic for state- Emissions filter logic #2253 to track the changes

Bug #2253 resolved. This ticket is ready for 508 testing @aprematta


Legacy resources show up as root (for project level) or Root (for cluster level) resource when searching a cluster

Reproducible on v2.6.0-rc9:

enable legacy options in rancher
provision a downstream cluster
search the cluster for catalogs or alerts
results show multiple sections, and is unclear to user which is correct

Expected Result:
section that the search belongs to should be clear to the user.
Additional Context:
It appears that root maps to the project level for legacy resources, and Root is for the cluster level.
Potential Solution:

use the information from additional context to label the information more appropriately:

root could be Legacy Project and Root could be Legacy Cluster (other suggestions for new names are welcome)

The legacy pages should not show up in the search at all, so we'll hide them.

on 2.6-head (a6e7905):

none of the legacy features show up in search now


How to config es index generated by date? (version3.2.6)

skywalking 3.2.6.
How to config es index generated by date in the skywalking-collector? Because in the product env, the data is huge amount, we need deleting data by date cyclely, Its very simple and efficiency to delete old data.

Is this config will be work? I just try.

We don't have such mechanism, even in 5.x, we only provided data delete, but not es index sharding by time bucket. In 6, I think @peng-yongsheng proposed to do so.

That config is for 5.x only. Especially in RC and beta2.

These codes are existed in the version 3.2.6, and it seems just only to define the expired time of trace date in the es, rather than the index generated by time according to the code.

I don't remember we have that. :) Sorry. more than 1 year ago.


I can't get the interface's props to load properly in the monorepo environment w/o relative path

Bug Report

🔎 Search Terms

monorepo
import
export
interface
type annotation
absolute path
relative path
paths on tsconfig.json

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about monorepo

⏯ Playground Link

GitHub Public Repository for this case w/ pnpm.
https://github.com/Sn-Kinos/monorepo-interface
It issues on Yarn workspace too.
💻 Code

// We can quickly address your report if:
//  - The code sample is short. Nearly all TypeScript bugs can be demonstrated in 20-30 lines of code!
//  - It doesn't use external libraries. These are often issues with the type definitions rather than TypeScript bugs.
//  - The incorrectness of the behavior is readily apparent from reading the sample.
// Reports are slower to investigate if:
//  - We have to pare too much extraneous code.
//  - We have to clone a large repo and validate that the problem isn't elsewhere.
//  - The sample is confusing or doesn't clearly demonstrate what's wrong.

With above image, Type annotation for AppObjectWithInterface and AppObjectWithType on apps/app/index.ts didn't work properly.
🙁 Actual behavior

LibraryType on apps/ shows it any.
LibraryInterface on apps/  does not includes paths and absolute.

🙂 Expected behavior

LibraryType on apps/ shows properly.
LibraryInterface on apps/  includes paths and absolute.

Typescript obviously can't know if you're running pnpm or not, so please provide a repro that doesn't depend on that tool

Typescript obviously can't know if you're running pnpm or not, so please provide a repro that doesn't depend on that tool

Can I set up monorepo without pnpm? I did it also with Yarn Workspaces, But It didn't works. 😢

I also published npm branch that make monorepo with npm. And as I wrote in the text, the same problem occurred in Yarn, so I don't think this is a problem caused by a specific package manager... 🤔

@RyanCavanaugh May I ask if there is anything else I can do to resolve?

@Sn-Kinos I'd like you to provide a repro that doesn't require running tools that aren't TypeScript. TypeScript can only see the layout of files on disk so whatever problem you have is related to that layout of files, and should be reproducible


Parallel Block Merge mental breakdown

Just run Parallel Block Merge Sort yourself and view the carnage.

seems to be caused by the changes to Highlights.java

Cause identified: Highlights.clearAllMarks() was not synchronous. I have no clue how that fixes this.

Ah yes, threading without a GIL to keep you safe. always fun

clearAllMarks() is used by binary insertion sort which is used by parallel block merge, so I guess that's how it's relevant
still no clue how this alters the sort's behavior; the Highlights functions only modify Highlights properties


No response from oidc/rp/providers call

Hi,
I followed all instructions to run RP example project but the oidc/rp/providers call (when I click on "Entra con SPID" button) always responses with no data. The federation (<IP_ADDRESS>:8000) and cie provider (<IP_ADDRESS>:8002) work properly but I don't have any response in this case.
I've registered the relying party and created new profile on the OIDC Federation Entity Administration (http://<IP_ADDRESS>:8000/admin/).
I'm trying with the latest version of both projects spid-cie-oidc-nodejs and spid-cie-oidc-django.
Are there other tips I must follow?

In the log you can see the output of the trust chain, can you past it here?
anyway you have a CLI command to fetch, RP side, all the configured OPs in your settingslocal.py
examples/federation_authority/manage.py fetch_openid_providers --start -f

Hi,
I updated the configuration readme see here.
Make sure you have correctly created public.jwks.json private.jwks.json trust_marks.json files in the examples/express-react-relying-party directory.
The reason there are no providers is that some error occurred during trust chain validation.
If the problem persist we can schedule a screen sharing session to debug the issue.

You can also read the log files in the logs directory (it gets populated after starting the server on <IP_ADDRESS>:3000)

I've already created these files in examples/express-react-relying-party directory.
As suggested, I found the error reading the log files! Here I read what is the problem: the sub configuration. I had configured RP sub <IP_ADDRESS>:3000 and not <IP_ADDRESS>:3000/oidc/rp. Changing this field I solved.
Thank you!


removed unused methods

Current coverage is 74.77%

Merging #6 into master will increase coverage by +1.31% as of 0a2cebb

@@            master      #6   diff @@
======================================
  Files           27      27
  Stmts          569     563     -6
  Branches         0       0
  Methods        146     143     -3
======================================
+ Hit            418     421     +3
  Partial          0       0
+ Missed         151     142     -9

Review entire Coverage Diff as of 0a2cebb

Powered by Codecov. Updated on successful CI builds.


v2.5.1 - Error: Unknown weatheralerts alert level: Rip Current Statement

Describe the bug
card states Error: Unknown weatheralerts alert level: Rip Current Statement
Integration state attributes
No response
Screenshots

Card version
2.5.1
Installation type
Home Assistant Community Store (HACS)
Card Configuration
type: custom:meteoalarm-cardentities:  - entity: sensor.santa_rosa_coastalintegration: weatheralertshide_when_no_warning: truescaling_mode: headline_and_scaleoverride_headline: false
Integration type
weatheralerts
Integration configuration
YAML (configuration.yaml)
Integration Configuration
No response

Thank you for pointing that out!


[JN-569] Autofill survey stable ID

DESCRIPTION
This adds autofill for survey stable IDs during creation
TO TEST:  (simple manual steps for confirming core behavior -- used for pre-release checks)
Create a new survey and verify that as you type a name, the survey stable ID is automatically generated.
Confirm that once you make a single edit to the stable ID, it no longer autofills based on changes to the name field.

@breilly2 you're right, ideally it'd be something like this:

We'd probably have to store the portal and study prefixes in the DB somewhere and retrieve those. I'll hold off on merging this for the moment, it wouldn't be much effort to future-proof this to avoid headaches/confusion down the line.

I'm going to merge this because it's a UX improvement over what we have, and doesn't make the issue of uniqueness worse than it already is. I filed that as a future improvement: JN-596


fix: revert scroll dropdown to top of modal to prevent option list f…

…rom being cut off
This reverts commit 01375b196b96dd5e1e21602a3a6ab6f26651d68d.

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Variables Not Showing

Describe the bug
Variables... just don't show
Project Link
Don't have one.
To Reproduce
Steps to reproduce the behavior:

Try to run really simple project
Variables say goodbye

Expected behavior
I expected variables to show
System Information

OS: Windows 11 23H2
Python version: 3.13.0
Pygame version: 2.6.1

Variable monitors are not yet supported. I would like to potentially add support in the future though. See the currently supported blocks.
Currently, if you use a show variable block, it will be printed out to the console. That might help as a temporary workaround if you are trying to see what values variables have.