TIME token does not display HH:MM:SS all the time.

If the videos on the result list are shorter then 1 hour then the display goes like MM:SS. However if the video is longer it displays as HH:MM:SS.
This breaks my custom layout as I put the TIME token at front. Can this be fixed in the next release?
Regards,
Piotr

Please check the extended version of custom_layout_format, which allows padding, alignment, colors and fixed-width formatting.
Example:
  custom_layout         => 1,
  custom_layout_format  => [
        {width => 3, text => "*NO*.", color => 'bold'},
        {width => '60%', text => "*TITLE*"},
        {width => '20%', text => "*AUTHOR*", color => 'yellow', align => 'right'},
        {width => 8, text => "*TIME*", color => 'blue', align => 'right'},
  ],

Tried to use the syntax from your example like this:
  custom_layout_format  => [
                            {width =>  8,       text => "*TIME*", align => 'right'},
                            {width => 11,       text => "*ID*"},
                            {width => 11,       text => "*PUBLISHED*"},
                            {width => '80%',   text => "*TITLE*\\n"},
                           ],

but the output I got from the youtube-viewer --custom-layout "long run" was empty. Then I pressed Enter to see the next page of results and I was the following output:
ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)ARRAY(0x560a83b9d830)

=>> Select one or more videos to play (:h for help)
>

It looks like there is something wrong.

Make sure to have latest master branch installed, as this feature was just added in https://github.com/trizen/youtube-viewer/commit/c3074371a9aa17ee840dddad12c147acbc144bf4.

OK. I was using script for updating the youtube-viewer which checks version on the github page:
lynx -dump github.com/trizen/youtube-viewer | grep "Version" | perl -ne 's/.*Version v*(\d+\.\d+\.\d+).*/$1/; print;' | sort | uniq | tail -1)

Since that version did not change, then I did not update the youtube-viewer. Maybe it is wise to update the version number each time there is change in master branch?
Now when I forced the update everything works as you described. :)
Many thanks!!!

No problem. I'm glad it works now. :)


Add tagging/untagging for documents

Add editable field that allows adding/removing tags from document (position of red rectangle in the right bottom corner):

Proposed syntax for tags is #tag_name1, #tag_name2, by pressing Enter editing of tags is ended.

@StaffEngineer I would like to look at this. Please can you assign this to me?

Instead of occupying some space on canvas better UX would be to have a button that opens modal for document tag editing.

This can be re-opened if someone wants to work on it.
Fuzzy search on native platform is enough for now.


add zig build system for luv

This is the first step in allowing simple cross compilation for luv and luvi via zig.
This code needs to be compared against the respective Makefiles and CMakeLists before this can be merged.
This is marked as a draft PR as some features (specifically tests) need to be able to pass successfully on all the supported platforms before this can be any semblance of a good way to build luv.
At the moment:
tests/test-signal's SIGINT test is not working,
the built luajit executable is being flagged by UBSan

the built luajit executable is being flagged by UBSan

I assume this is being worked around via disable_sanitize_c = true? Or is it still triggering UBSan checks with that?

I assume this is being worked around via disable_sanitize_c = true?

It should be, but yes, its still emitting ub1 instructions and I didn't take the time to figure out why.

Seems like it would be nice to try adding some more architectures to CI

I'll play around a bit with making github actions workflows for some of the major ones (linux64, macos64, win64/32) once I can get it all working correctly and passing tests.

I figured out why luajit was being ubsan'd (the library itself needed no-sanitize-c), however, something in libc's atexit is causing a segfault, and I can't figure out why that would be happening. It seems to be happening with both luajit and PUC, so its likely an issue with building libuv.


rosdep: Added exiv2-dev

Please add the following dependency to the rosdep database.
Package name:
exiv2-dev
Package Upstream Source:
https://www.exiv2.org/
Purpose of using this:
Development files for exiv2.
Distro packaging links:
Links to Distribution Packages

Debian: https://packages.debian.org/sid/libexiv2-dev

REQUIRED

Ubuntu: https://packages.ubuntu.com/focal-updates/libexiv2-dev

REQUIRED

Fedora: https://packages.fedoraproject.org/pkgs/exiv2/exiv2-devel/

IF AVAILABLE

Arch: https://archlinux.org/packages/extra/x86_64/exiv2/

IF AVAILABLE

Gentoo: https://packages.gentoo.org/packages/media-gfx/exiv2

IF AVAILABLE

macOS: https://formulae.brew.sh/formula/exiv2

IF AVAILABLE

Alpine: https://pkgs.alpinelinux.org/package/edge/community/x86_64/exiv2-dev

IF AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages?channel=24.05&show=exiv2&from=0&size=50&sort=relevance&type=packages&query=exiv2

OPTIONAL

openSUSE: N/A

IF AVAILABLE

rhel: https://centos.pkgs.org/9-stream/centos-crb-x86_64/exiv2-devel-0.27.5-2.el9.x86_64.rpm.html

IF AVAILABLE

There's a lint warning that suggests an OpenSUSE name for it, can you check that?

Thanks for spotting it. Added.

This now needs a rebase to fix conflicts; once that is done it will be good to merge.

@clalancette I'm on phone only now so rebasing would be difficult. Could you squash merge? Also, CI timed out, can you rerun it?


MongoDB adapter doesn't recognise FKs to mysql table correctly

Sails version<EMAIL_ADDRESS>Node version: 8.9.4
NPM version: 5.6.0
DB adapter name:  sails-mongo
DB adapter version<EMAIL_ADDRESS>Operating system: macOS 10.13.3

Hello,
I have a reference from my mongo model ( Client ) to a mysql model via user: { model: 'User', required: true } and User is in mysql database. I then want to create a Client and supply {user: 1} the adapter throws
Error: Invalid replacement foreign key value provided for association (user).  Cannot interpret 1 as a Mongo id.
REPO:  git@bitbucket.org:mp3por/sails-bug-2.git
Please advice :)

This also doesn't work if the id of the linked mysql model is UUID.
I managed to hack it for UUID via  user : { type:'string'} and for ID via user : { type: 'number' }.

@mp3por thanks for the heads up- I think I see why this is happening

Will look into it ASAP.  Your workaround makes sense in the mean time (of course it means you won't be able to rely on .populate())

Here's the code in sails-mongo that's specifically wrong:
https://github.com/balderdashy/sails-mongo/blob/18d0f31df0da0659c93bd5f59cdc4d422ab596a3/lib/private/machines/private/reify-values-to-set.js#L88-L99
@mp3por Also in the mean time, if you have a moment to take a pass at a PR, I'd be happy to review and hopefully merge.  I think what we need there is to add another disjunct checking if the foreign key is pointed at another model known to sails-mongo, and if so, skip the check.
(Also, you'll notice in that code that there's another workaround hack you could use-- though you shouldn't have to -- this is definitely something that should work out of the box)

Another possible workaround without changing the Model Structure (NOT a fix):
Setting dontUseObjectIds: true for the Model (which is stored in MySQL) seems to fix the issue.

@maheshwarishivam :+1:  thanks for the follow-up!
@streleck this is related to the other issue we were talking about this morning.  (If you've still got that link handy, would make sense to backlink here)

@maheshwarishivam  I'm using<EMAIL_ADDRESS>and I have the same issue. Does that label closed means it is already available in new version?

node v8.12.0
npm 6.4.1
<EMAIL_ADDRESS><EMAIL_ADDRESS>ubuntu 16.04 LTS

@ghost - The referenced issue was Closed because it was a duplicate of this one, however it has a lot of useful info so it's definitely still worth checking out.   @maheshwarishivam  Thanks for this workaround!


translating zh-cn & make some fix

good to go?

good to go?

done.
But I still have some doubts
en-us has duplicate ID "error_len_array"
one in 1635 row and one is 1719,could I delete one ?
in row 1715 ,there is a id  "error_"  i think is a wrong copy of ”error_len_array“(row 1719)


[Feature]: Define Echart ToolTip Functions In Javascript Files Instead of Inside Chart Widget

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
I found the thread requesting the ability to pass a function to the echarts option object for the formatter function. I was able to get this working but only by defining an anonymous function in the Custom Echarts Configuration input in the chart widget. This is not ideal because the tooltip function I need to implement is quite large and putting it in that little text box reduces reusability because it has to be defined on demand and reduces readability.
Example:
I have a javascript object called GetAuthObj and a tooltip function called getToolTip defined in this object. When I define my echart configuration I set the formatter option to GetAuthObj.getToolTip but appsmith does not pass the tooltip function correctly. It passes a function that takes no args and causes echarts to error when trying to show the tooltip.
See the image for my config object and the output when running the echart:

And the get tooltip function:

This however works:

I am running the appsmith application in an iframe if that makes a difference.
Why should this be worked on?
This promotes re-usability and readability for echart functionality.

@cch264 can I suggest simply returning the entire chart configuration as a function and that way you can control any other functions within it that you need to call?

Yes that is an option, previously I was passing the chart configuration as an object to the echart widget. But this does not work with formatter functions I get the same error as before. See below. I am defining the chart option as an empty object while waiting for auth credentials to load. Then I set the a property on the javascript object and then bind that property to the widget.

Trying out a function that returns an chart config object same issue:

This function returns an options object and the chart loads without issue but the tooltip functionality is still broken
See the return code
// Specify the configuration items and data for the chart 		var option = { 			legend: { 				top: 50, 				selectedMode: false, 			}, 			tooltip: { 				trigger: 'axis', 				axisPointer: { 					type: 'cross', 				}, 				formatter: this.getToolTip 			}, 			grid: { 				left: 100, 				right: 100, 				top: 100, 				bottom: 200 			}, 			dataZoom: [ 				{ 					type: 'inside', 					start: 0, 					end: 100 				} 			], 			yAxis: { 				type: 'value', 				max: 100 			}, 			xAxis: { 				type: 'category', 				data: xAxisLabels, 				axisLabel: { 					interval: 0, 					rotate: 70 				} 			}, 			series 		}; 		return option;
It appears the issue is using an object property to pass a function to the formatter property because when I switch to using an anonymous function this works fine. I think the issue still stands because using an anonymous function is not the best solution and is very confusing because my code is valid JavaScript but does not work in this context.

Going to change the feature request to Define Echart ToolTip Functions Using Named Functions


Added missing semicolon

@K232 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Improve config file handling

A couple of things that could/should be improved:
Must have:

Replace https://github.com/SwissOpenEM/LS_Metadata_reader/blob/main/configuration/setgetconfig.go#L20 with os.UserConfigDir() which is platform independent

Nice to have:

Using viper to parse configs, seehttps://github.com/spf13/viper and in the ingestor https://github.com/SwissOpenEM/Ingestor/blob/main/internal/core/config.go#L40

Hey, good point! I have changed the config overall to now allow flags as an alternative and incorporated os.UserConfigDir()
About the viper part I have not yet implemented that, might do that depending on how the interface will finally look like.


Java agent is added twice to launch command and doesn't record correctly

A user tried to map this repo: https://github.com/shai-almog/NetflixCloneServer
They were unable to connect to the server for remote recording. They also noticed that the -javaagent arguments were added twice to the launch command in IntelliJ, so that might be relevant to the recording issue.

@brikelly Unfortunately, I can't confirm this.
I'd at least the logs at Help > Show Log in ... and any error message, which might have been shown.
I installed the latest version of the AppMap plugin, cloned the repository and then launched com.codename1.demos.netflixclone.NetflixcloneApplication with the AppMap run button. This launched a server on localhost:8080.
Then, I started remote recording with URL http://localhost:8080, made a simple web request and then stopped it again.
This AppMap was created:

It's in fact adding two Java agents:
-javaagent:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56.plugins/intellij-appmap/appmap-agents/appmap-agent.jar
-javaagent:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56/lib/idea_rt.jar=44407:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56/bin

The 1st is ours and AFAIK the 2nd is part of IntelliJ's Maven or run configuration handling.
The launched command line was this:
/usr/lib/jvm/java-8-openjdk/bin/java -Dappmap.config.file=/home/jansorg/self-employment/customers/AppLand/sources/NetflixCloneServer/appmap.yml -Dappmap.output.directory=/home/jansorg/self-employment/customers/AppLand/sources/NetflixCloneServer/target/appmap -javaagent:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56.plugins/intellij-appmap/appmap-agents/appmap-agent.jar -javaagent:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56/lib/idea_rt.jar=44407:/home/jansorg/.local/share/JetBrains/Toolbox/apps/IDEA-C/ch-7/223.8617.56/bin -Dfile.encoding=UTF-8 -classpath /usr/lib/jvm/java-8-openjdk/jre/lib/charsets.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/cldrdata.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/dnsns.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/jaccess.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/localedata.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/nashorn.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/sunec.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/sunjce_provider.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/sunpkcs11.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/ext/zipfs.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/jce.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/jfr.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/jsse.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/management-agent.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/resources.jar:/usr/lib/jvm/java-8-openjdk/jre/lib/rt.jar:/home/jansorg/self-employment/customers/AppLand/sources/NetflixCloneServer/target/classes:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-data-jpa/2.2.2.RELEASE/spring-boot-starter-data-jpa-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-aop/2.2.2.RELEASE/spring-boot-starter-aop-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-aop/5.2.2.RELEASE/spring-aop-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/aspectj/aspectjweaver/1.9.5/aspectjweaver-1.9.5.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-jdbc/2.2.2.RELEASE/spring-boot-starter-jdbc-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/com/zaxxer/HikariCP/3.4.1/HikariCP-3.4.1.jar:/home/jansorg/.m2/repository/org/springframework/spring-jdbc/5.2.2.RELEASE/spring-jdbc-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/jakarta/activation/jakarta.activation-api/1.2.1/jakarta.activation-api-1.2.1.jar:/home/jansorg/.m2/repository/jakarta/persistence/jakarta.persistence-api/2.2.3/jakarta.persistence-api-2.2.3.jar:/home/jansorg/.m2/repository/jakarta/transaction/jakarta.transaction-api/1.3.3/jakarta.transaction-api-1.3.3.jar:/home/jansorg/.m2/repository/org/hibernate/hibernate-core/5.4.9.Final/hibernate-core-5.4.9.Final.jar:/home/jansorg/.m2/repository/org/jboss/logging/jboss-logging/3.4.1.Final/jboss-logging-3.4.1.Final.jar:/home/jansorg/.m2/repository/org/javassist/javassist/3.24.0-GA/javassist-3.24.0-GA.jar:/home/jansorg/.m2/repository/net/bytebuddy/byte-buddy/1.10.4/byte-buddy-1.10.4.jar:/home/jansorg/.m2/repository/antlr/antlr/2.7.7/antlr-2.7.7.jar:/home/jansorg/.m2/repository/org/jboss/jandex/2.1.1.Final/jandex-2.1.1.Final.jar:/home/jansorg/.m2/repository/com/fasterxml/classmate/1.5.1/classmate-1.5.1.jar:/home/jansorg/.m2/repository/org/dom4j/dom4j/2.1.1/dom4j-2.1.1.jar:/home/jansorg/.m2/repository/org/hibernate/common/hibernate-commons-annotations/5.1.0.Final/hibernate-commons-annotations-5.1.0.Final.jar:/home/jansorg/.m2/repository/org/glassfish/jaxb/jaxb-runtime/2.3.2/jaxb-runtime-2.3.2.jar:/home/jansorg/.m2/repository/org/glassfish/jaxb/txw2/2.3.2/txw2-2.3.2.jar:/home/jansorg/.m2/repository/com/sun/istack/istack-commons-runtime/3.0.8/istack-commons-runtime-3.0.8.jar:/home/jansorg/.m2/repository/org/jvnet/staxex/stax-ex/1.8.1/stax-ex-1.8.1.jar:/home/jansorg/.m2/repository/com/sun/xml/fastinfoset/FastInfoset/1.2.16/FastInfoset-1.2.16.jar:/home/jansorg/.m2/repository/org/springframework/data/spring-data-jpa/2.2.3.RELEASE/spring-data-jpa-2.2.3.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/data/spring-data-commons/2.2.3.RELEASE/spring-data-commons-2.2.3.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-orm/5.2.2.RELEASE/spring-orm-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-context/5.2.2.RELEASE/spring-context-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-tx/5.2.2.RELEASE/spring-tx-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-beans/5.2.2.RELEASE/spring-beans-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/slf4j/slf4j-api/1.7.29/slf4j-api-1.7.29.jar:/home/jansorg/.m2/repository/org/springframework/spring-aspects/5.2.2.RELEASE/spring-aspects-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-jersey/2.2.2.RELEASE/spring-boot-starter-jersey-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-json/2.2.2.RELEASE/spring-boot-starter-json-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.10.1/jackson-databind-2.10.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/core/jackson-core/2.10.1/jackson-core-2.10.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jdk8/2.10.1/jackson-datatype-jdk8-2.10.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/datatype/jackson-datatype-jsr310/2.10.1/jackson-datatype-jsr310-2.10.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/module/jackson-module-parameter-names/2.10.1/jackson-module-parameter-names-2.10.1.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-tomcat/2.2.2.RELEASE/spring-boot-starter-tomcat-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/apache/tomcat/embed/tomcat-embed-core/9.0.29/tomcat-embed-core-9.0.29.jar:/home/jansorg/.m2/repository/org/apache/tomcat/embed/tomcat-embed-el/9.0.29/tomcat-embed-el-9.0.29.jar:/home/jansorg/.m2/repository/org/apache/tomcat/embed/tomcat-embed-websocket/9.0.29/tomcat-embed-websocket-9.0.29.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-validation/2.2.2.RELEASE/spring-boot-starter-validation-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/jakarta/validation/jakarta.validation-api/2.0.1/jakarta.validation-api-2.0.1.jar:/home/jansorg/.m2/repository/org/hibernate/validator/hibernate-validator/6.0.18.Final/hibernate-validator-6.0.18.Final.jar:/home/jansorg/.m2/repository/jakarta/annotation/jakarta.annotation-api/1.3.5/jakarta.annotation-api-1.3.5.jar:/home/jansorg/.m2/repository/jakarta/ws/rs/jakarta.ws.rs-api/2.1.6/jakarta.ws.rs-api-2.1.6.jar:/home/jansorg/.m2/repository/org/springframework/spring-web/5.2.2.RELEASE/spring-web-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/core/jersey-server/2.29.1/jersey-server-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/core/jersey-common/2.29.1/jersey-common-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/osgi-resource-locator/1.0.3/osgi-resource-locator-1.0.3.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/core/jersey-client/2.29.1/jersey-client-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/media/jersey-media-jaxb/2.29.1/jersey-media-jaxb-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/external/jakarta.inject/2.6.1/jakarta.inject-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/containers/jersey-container-servlet-core/2.29.1/jersey-container-servlet-core-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/containers/jersey-container-servlet/2.29.1/jersey-container-servlet-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/ext/jersey-bean-validation/2.29.1/jersey-bean-validation-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/ext/jersey-spring5/2.29.1/jersey-spring5-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/inject/jersey-hk2/2.29.1/jersey-hk2-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2-locator/2.6.1/hk2-locator-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/external/aopalliance-repackaged/2.6.1/aopalliance-repackaged-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2/2.6.1/hk2-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2-utils/2.6.1/hk2-utils-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2-api/2.6.1/hk2-api-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2-core/2.6.1/hk2-core-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/hk2-runlevel/2.6.1/hk2-runlevel-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/class-model/2.6.1/class-model-2.6.1.jar:/home/jansorg/.m2/repository/org/ow2/asm/asm/7.1/asm-7.1.jar:/home/jansorg/.m2/repository/org/ow2/asm/asm-analysis/7.1/asm-analysis-7.1.jar:/home/jansorg/.m2/repository/org/ow2/asm/asm-commons/7.1/asm-commons-7.1.jar:/home/jansorg/.m2/repository/org/ow2/asm/asm-tree/7.1/asm-tree-7.1.jar:/home/jansorg/.m2/repository/org/ow2/asm/asm-util/7.1/asm-util-7.1.jar:/home/jansorg/.m2/repository/org/glassfish/hk2/spring-bridge/2.6.1/spring-bridge-2.6.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/media/jersey-media-json-jackson/2.29.1/jersey-media-json-jackson-2.29.1.jar:/home/jansorg/.m2/repository/org/glassfish/jersey/ext/jersey-entity-filtering/2.29.1/jersey-entity-filtering-2.29.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.10.1/jackson-annotations-2.10.1.jar:/home/jansorg/.m2/repository/com/fasterxml/jackson/module/jackson-module-jaxb-annotations/2.10.1/jackson-module-jaxb-annotations-2.10.1.jar:/home/jansorg/.m2/repository/jakarta/xml/bind/jakarta.xml.bind-api/2.3.2/jakarta.xml.bind-api-2.3.2.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-web/2.2.2.RELEASE/spring-boot-starter-web-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter/2.2.2.RELEASE/spring-boot-starter-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot/2.2.2.RELEASE/spring-boot-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-autoconfigure/2.2.2.RELEASE/spring-boot-autoconfigure-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/boot/spring-boot-starter-logging/2.2.2.RELEASE/spring-boot-starter-logging-2.2.2.RELEASE.jar:/home/jansorg/.m2/repository/ch/qos/logback/logback-classic/1.2.3/logback-classic-1.2.3.jar:/home/jansorg/.m2/repository/ch/qos/logback/logback-core/1.2.3/logback-core-1.2.3.jar:/home/jansorg/.m2/repository/org/apache/logging/log4j/log4j-to-slf4j/2.12.1/log4j-to-slf4j-2.12.1.jar:/home/jansorg/.m2/repository/org/apache/logging/log4j/log4j-api/2.12.1/log4j-api-2.12.1.jar:/home/jansorg/.m2/repository/org/slf4j/jul-to-slf4j/1.7.29/jul-to-slf4j-1.7.29.jar:/home/jansorg/.m2/repository/org/yaml/snakeyaml/1.25/snakeyaml-1.25.jar:/home/jansorg/.m2/repository/org/springframework/spring-webmvc/5.2.2.RELEASE/spring-webmvc-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-expression/5.2.2.RELEASE/spring-expression-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/com/h2database/h2/1.4.200/h2-1.4.200.jar:/home/jansorg/.m2/repository/org/projectlombok/lombok/1.18.10/lombok-1.18.10.jar:/home/jansorg/.m2/repository/commons-io/commons-io/2.6/commons-io-2.6.jar:/home/jansorg/.m2/repository/org/springframework/spring-core/5.2.2.RELEASE/spring-core-5.2.2.RELEASE.jar:/home/jansorg/.m2/repository/org/springframework/spring-jcl/5.2.2.RELEASE/spring-jcl-5.2.2.RELEASE.jar com.codename1.demos.netflixclone.NetflixcloneApplication

Thank you @jansorg. I was able to get this application mapping as well, but only if I used Java 11. When I use Java 17 (Eclipse Temurin JDK) then remote recording fails.
I will close this issue now, as it appears that the "double agent" is a red herring, and also that there's a separate issue doing remote recording with Java 17.


Merge master -> staging

It’s better to use real merge commits rather than squashing to merge stuff between long-running branches. It makes it easier to track what changes are included into the branch. It also works better with conflict resolution (though, normally, in the master -> staging workflow there should be no conflicts unless hotfixes are applied, and even then it’s usually clear).


[Feature Request] Want to control LLM model selection through configuration file

Describe the solution you'd like
Create model configuration file for UI (React).
Why the solution needed
After conducting a Proof of Concept (PoC) and determining that a specific model works well enough, we want to prevent actual users from using other models (as higher accuracy models are more expensive).
Implementation feasibility
Are you willing to collaborate with us to discuss the solution, decide on the approach, and assist with the implementation?

[x] Yes, I am able to implement the feature and create a pull request.
[ ] No, I am unable to implement the feature, but I am open to discussing the solution.

Related: #505


Deployment fails when two container apps reference the same DAPR component

Hi,
I ran into this issue while trying to build a simple DAPR pubsub demo. When i deploy two container apps that use the same DAPR component, the deployment of the 2nd container app fails. This happens on a brand new deployment as well as a new revision.
I tried the reddog container apps example to see if i was doing something wrong, but it runs into the same issue currently.

Seems to be the case for one of my deployments as well.
After successive failures trying to deploy a second container that referenced the same components o the first one, I tried deploying the exact same container to a new environment, and it succeeded.

Thanks for reporting. Currently due to the scoping of components at the Container App level, the naming of the component itself must be unique per Container App. This is an issue that is currently present in the reddog app as well.
i.e. If App A - Component A and App B - Component A, you will get an error. You would have to change the name of Component A in App B to Component B. This is not ideal for components that are shared. We are working on changes to the service that will eliminate this limitation and allow a component to be shared across multiple Container Apps with the same name.  @renatolfc @roelvandegrint

Thanks for reporting @roelvandegrint was facing the same issue here and it kept saying that the provisioning failed but I didn't know why

Note: when updating, the app doesn't change at all and the deployment hangs. Not sure if this is an issue with the CLI or not and with this, but I resolved it by adding a random identifier that redeploys the container app itself each time.
const randomStr = Math.random().toString(36).substring(2, 7);
const containerAppName = `container-${randomStr}`

@kendallroden Thanks for the explanation. Just to be sure I understand correctly. That means in the mean time we cannot do things like pubsub that need to share the same component, right? I don't know of any workaround but since you said 'not ideal' I thought maybe there is?

@roelvandegrint a workaround that worked for me for things like pubsub was to specify a "pubsub component" for each service, while the components only differ in their names (as @kendallroden mentioned).
"Under the hood" both dapr components pointed to the same (azure service bus) endpoint.
For example:
Service A -> Component A
"dapr": {
  "enabled": true,
  "appId": "service-a",
  "appPort": 80,
  "components": [
    {
      "name": "pubsub-service-a",
      "type": "pubsub.azure.servicebus",
      "version": "v1",
      "metadata": [
        {
          "name": "connectionString",
          "value": "",
          "secretRef": "servicebusendpoint"
        }]
    }]
}

Service B -> Component B
"dapr": {
  "enabled": true,
  "appId": "service-b",
  "appPort": 80,
  "components": [
    {
      "name": "pubsub-service-b",
      "type": "pubsub.azure.servicebus",
      "version": "v1",
      "metadata": [
        {
          "name": "connectionString",
          "value": "",
          "secretRef": "servicebusendpoint"
        }]
    }]
}


Warning PropTypes

Version
RN 0.48.1, react 16.0.0-alpha.12, react-native-router-flux 4.0.0-beta.21
Getting warnings when using this module:
Expected behaviour
This modules need to be updated to make compatible with react native latest version.
Actual behaviour
Warning: PropTypes has been moved to a separate package.
Warning: checkPropTypes has been moved to a separate package.
Warning: React.createClass is no longer supported. Use a plain JavaScript class instead.
 I think it has been fixed. I have no warnings on this

The problem disappeared for me after upgrading to React 16.0.0


Fix broken netlify build due to array.at()

If the build was only failing due to using array.at() which was added in node v16.6.0, then this should fix it and close #6.
Don't use Vercel myself. I presume you can set the node version similarly there.

Setting NODE_VERSION doesn't seem to work.
Since this is a serverless function, it appears to be related to AWS_LAMBDA_FUNCTION_VERSION; but it currently only supports up to nodejs14.x.
So I'll fix it with a hard reset, but thanks anyway for contributing!

Ah, then I misunderstood you. I thought you were saying the build was failing, not at run time. But then why not always use the static adapter for this demo site? Are you actually using a lambda for anything? If not, let the browser run your code since most support array.at() (see MDN docs linked above).

Ah, then I misunderstood you. I thought you were saying the build was failing, not at run time. But then why not always use the static adapter for this demo site? Are you actually using a lambda for anything? If not, let the browser run your code since most support array.at() (see MDN docs linked above).

Hmm... mainly I don't quite understand how to set the function to execute only with the browser. write it after if(browser) { ?
Maybe it's possible to set the toc component to be called only when browser === true.

Exactly, or wrap it in onMount/afterNavigate. That's what I did in svelte-toc.

👍 Thanks for the suggestion, now the toc is only displayed on the browser side.


Update glob dependency

Newer version of glob supports follow: true.
It should close #27 and it's needed for https://github.com/ember-cli/ember-cli/issues/4137.

Thank you!
@joliss - Can we have a release for this?

Yup, out in v0.2.7. Thanks @btecu, thanks @rwjblue.


Allow chose queue type when the job is scheduled

By default, rq-scheduler allowed to choose the queue type when the run process of rq-scheduler was started. These queue type was used for all scheduled jobs. You weren't able to choose an specific queue type for each job, they used the same queue type.
This PR allows choose the queue type of each job when it is scheduler. The param queue_class it's stored as job metadata when the job is scheduled.
Then when de job is queued the rqscheduler checks if the job has an especific queue_class defined and uses it. If there is no queue_class in the meta data of the job then the queue_class of the main rq-scheduler is used.

Could you please provide a test for this?

@selwin Test added.
The test test_create_job_with_depends_on is failing, but it is not related with this changes.

Can you also document this feature so other people know how to use it?

@selwin I've updated the Readme with document of this feature


Directors timeline broken due to Content Security Policy issue

https://www.apache.org/history/directors.html gives a JavaScript error as the Content Security Policy (on my Firefox 109.01 / macosx browser at least) prevents the https://www.gstatic.com/charts/loader.js script from loading. As a result, the timeline is not displayed, the page body is blank.
I'll fix that by adding a copy of that script to our site content, the licensing in that script looks ok for that.

Embedding the script doesn't work as it's loading a few other scripts. It seems like the content-security-policy header for that page is controlled by infra, I have created https://issues.apache.org/jira/browse/INFRA-24184 to ask for adding gstatic.com there.


GH-43878: [Go][Release] Remove Go related codes from our release scripts

Rationale for this change
The Go implementation is moving to apache/arrow-go from go/ in apache/arrow.
What changes are included in this PR?
Remove go related release scripts from apache/arrow
Are these changes tested?
The source verification is tested via archery
Are there any user-facing changes?
Yes, Go will not be releases as part of Apache Arrow anymore.

GitHub Issue: #43878

@zeroshade does this urls will have to point here https://pkg.go.dev/github.com/apache/arrow-go/go/v18 once we release?
https://github.com/apache/arrow/blob/18319d30e558af72f72ea58c7d7996e781777643/docs/source/index.rst?plain=1#L107
https://github.com/apache/arrow/blob/18319d30e558af72f72ea58c7d7996e781777643/r/_pkgdown.yml#L79
Is there any way to point to latest so we don't have to bump the release on arrow when we release arrow-go?

@github-actions crossbow submit verify-rc-source-*

Ubuntu 20.04 issues are unrelated:

https://github.com/apache/arrow/issues/44173

How about creating https://arrow.apache.org/go/ as the canonical Apache Arrow Go website?
We can link to https://pkg.go.dev/github.com/apache/arrow-go/v${LATEST} from the URL.

How about creating https://arrow.apache.org/go/ as the canonical Apache Arrow Go website?

How can I do this?

We can use .asf.yaml:

The ADBC case: https://github.com/apache/arrow-adbc/blob/fc51e60087f835810a01775bf303577462c4df04/.asf.yaml#L39-L41
The Apache Arrow FlightSQL adapter for PostgreSQL case: https://github.com/apache/arrow-flight-sql-postgresql/blob/5d2f0d4994519879c33e3c791d4ee562264397f8/.asf.yaml#L38-L40

With the publish: configuration in apache/arrow-go's .asf.yaml, we can deploy the asf-site branch content to https://arrow.apache.org/go/ .

https://github.com/apache/arrow-go/issues/139

@kou are you ok for me merging this? The docs, once deployed with the release, will point to https://arrow.apache.org/go/  but this can be merged as is so we can unblock removing the go code from this repo.

Yes!


Parameters/Rules Governing Polls (Feedback Needed)

Okay, there are a lot of different topics/issues open about this and I hate to add another, but the reason I am creating this is because I would like to take action on getting an agreed-upon (by the community) set of  rules/parameters governing how polls should be conducted.
Right now, we are basically in the "Wild Wild West" with our poll system. Someone creates a poll, and then everybody gets all upset when the poll isn't to some set of specifications which this community has never set and agreed upon. People get upset if the poll isn't the "right" duration, contains the "right" options, is based on the "right" voting weight, etc etc.
So why don't we just go ahead and create at series of polls to govern at least the basic rules for how future polls should be conducted? Each poll question can be a 14-day poll, can take into account the users balance/mag, have an "Abstain" option (all things people generally seem to accept/prefer currently.
I think we could start with the basics. Things we may have discussed/agreed-upon here in GitHub but never formalized with a community vote. Here are some examples:

Should all polls contain an "Abstain" option?
What duration should an actionable poll be? ("Actionable" meaning, the poll is regarding a decision the community is being asked to make, resulting in some sort of action being taken such as Whitelist Polls, Polls approving "X", etc)
Should blockchain/neural network problems which affect magnitude invalidate all active polls?

Things of that nature. I think if we can get all these in front of the community and get a vote on them, we can then create a procedure or tutorial on the Gridcoin website dictating how polls should be conducted. This way instead of everybody arguing about the legitimacy of polls constantly, we can just have a series of simple rules in place that dictate when a poll is simply invalid and must be redone in order to be actionable.
Thoughts? Do you have additional items you'd like to see the community weigh in on relating to polls? My hope is that we can put to bed a lot of individual open issues on polls by going this route and simply letting the community decide on parameters/rules/etiquette (whatever description you're most comfortable with).

Hey all -- I'm going to kick off one of these polls today (will start with the "Abstain" option question since it's so straightforward). Still hoping for feedback or ideas for additional poll governance rules we can work toward here.

If @jring-o 's poll passes this can probably be closed

https://github.com/gridcoin-community/Gridcoin-Tasks/tasks might be better place for this discussion.
Let me also link to the relevant poll: https://github.com/gridcoin-community/Gridcoin-Tasks/issues/227


Move from Trollop to Optimist for CLI option passing

Trollop has been renamed to Optimist. Let's use the new name.

I've submitted similar PR on this repo https://github.com/dougbarth/jsonlint/pull/1 wasn't sure which one is actually maintained. link from https://rubygems.org/gems/jsonlint took me there. Hope one of these is merged at some point in near future.

Thanks for your input, we've gone ahead and merged it in.

@Aeternitaas Oh, thanks! I can't seem to find the merge here, has something changed?

Ran into this "error" today, would love to see some action on this PR!

Another PR moved the gem to optimist. Thanks for the effort!


Adding the Content-Disposition header to attachments

Related to #24
I guess since the job is already done it doesn't hurt to add this, but if you prefer not then I have no objections.

In this spec https://tools.ietf.org/html/rfc2183 you can see this
   MUA may use whatever presentation method it deems suitable.

If add this it should be made as an optional but it will add complexity to email creation. I wanted to remove all headers definitions from users

@minosiants as you can see, the header is optional and for now is only used when adding an attachment. I think it can be left like that for now and expanded latter if needed.
Or, if you prefer, we can just close this.

We can keep it open and if we see that it is useful I will merge.
Thanks a lot for your contribution :thumbsup: . I think your are THE FIRST who is using this lib

I think your are THE FIRST who is using this lib

Well I am using it because someone recommended it to me, so no 😉


[Bug]: Excessive underscores in test function names

Is there an existing issue for this?

[X] I have searched the existing issues

Is there a StackOverflow question about this issue?

[X] I have searched StackOverflow

What happened?
For example, in https://github.com/android/nowinandroid/blob/607c24e7f7399942e278af663ea4ad350e5bbc3a/core/data/src/test/java/com/google/samples/apps/nowinandroid/core/data/repository/OfflineFirstUserDataRepositoryTest.kt
Check all codebase and use maximum of 2 underscores per test in the format: given_when_then
Relevant logcat output
No response
Code of Conduct

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

I can take this

@dturner  I am struggle with some decision

ForYouViewModel
Original test naming is expect_condition, like stateIsLoadingWhenFollowedTopicsAreLoading, I don't understand the reason, but I prefer using direct statement like whenFollowedTopicsAreLoading_thenStateIsLoading
ListToMapMigration, IntToStringMigration
I never heard naming convention in test start with capitalize, I would like to change ListToMapMigration_should_migrate_bookmarks into listToMapMigration_shouldMigrateBookmarks
given_then format
I am not sure which one for given_then format is better, offlineFirstAuthorsRepositoryAuthorsStream_isBackedByAuthorsDao or offlineFirstAuthorsRepository_authorsStream_isBackedByAuthorsDao, since in other case I will choose offlineFirstAuthorsRepository_syncDeletesItems_markedDeletedOnNetwork and offlineFirstAuthorsRepository_syncPullsFromNetwork, it would be more consistency with second one, but meanwhile it break the given_when_then format
NewsResourceDaoTest
newsResourceDao_filters_items_by_topic_ids_or_author_ids_by_descending_publish_date test with long condition expression would looks like newsResourceDao_filtersItemsByTopicIdsOrAuthorIdsByDescendingPublishDate , but this change doesn't make the name more readable, any advice?

I think that looks fine @Kenny50

This could be a lint 🤓 and pretty easy to write too.
But I'm not sure if this would fit in such a "app architecture showcase" repository.
An other simpler solution is to simply use Kotlin's backticks for method names and use spaces.
@Test
fun `given X when Y then Y`()

@dturner let us know what's the best approach here.

Although, there already is a DesignSystemDetector custom lint, so, this might not be a bad idea to add one for test method conventions after all?

Backticks will not work on Android instrumented tests, bit they work fine on regular unit tests.


Missing Type=SharedVoicemail in the Syntax section

fixes #5216

Docs Build status updates of commit e03974f:
:clock10: Full build: the average full build time is 12 min(s) 35 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit e03974f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 52e4452:
:clock10: Full build: the average full build time is 12 min(s) 35 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 52e4452:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

agree with @dariomws

SharedVoicemail needs to be defined for the -Type parameter in the parameter description. It looks like @dariomws covers that in his PR.

Docs Build status updates of commit aa73f79:
:clock10: Full build: the average full build time is 12 min(s) 36 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit aa73f79:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 7d2a70c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit c2ff393:
:clock10: Full build: the average full build time is 12 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit c2ff393:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 9439ecb:
:clock10: Full build: the average full build time is 12 min(s) 38 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 9439ecb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@dariomws - I will go ahead and merge this and would you please update the doc directly with your updates? Thank you.

Docs Build status updates of commit 2f7146c:
:clock10: Full build: the average full build time is 12 min(s) 39 sec(s), this is based on the last 1000 build(s) for this repository.

@dariomws - I will go ahead and merge this and would you please update the doc directly with your updates? Thank you.

@kenwith sure I will do.

Docs Build status updates of commit 2f7146c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/New-CsAutoAttendantCallableEntity.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


MB-13774-base-update-handlers-office

Jira ticket for this change
Summary
This ticket adds the base update handlers for moving expenses, progear weigh tickets, and weight tickets in the office app. This includes:

handlers that returns a basic payload
happy path unit tests
wires up api.go

This does not include (because they will be added in a later sprint):

error handling
unhappy path tests

Note to reviewer:

All error handling for the endpoints have been commented out to speed up the process of finishing these handlers in the next sprint. It is made to save time for the next ticket and I would like to leave them there.

Setup to Run Your Code

💻 You will need to use 2 separate terminals to test this locally.
Terminal 1
Make sure swagger files are generated
make swagger_generate

Terminal 2
Run tests
go test ./pkg/handlers/ghcapi --testify.m <TEST NAME> -v

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Fails

:no_entry_sign:
Yarn Audit Issues Found:
high - loader-utils is vulnerable to Regular Expression Denial of Service (ReDoS) via url variable
Package loader-utils
Patched in >=2.0.4
Dependency of @trussworks/react-file-viewer
Path @trussworks/react-file-viewer > pdfjs-dist > worker-loader > loader-utils
More info https://github.com/advisories/GHSA-3rfm-jhwj-7488
high - loader-utils is vulnerable to Regular Expression Denial of Service (ReDoS)
Package loader-utils
Patched in >=2.0.4
Dependency of @trussworks/react-file-viewer
Path @trussworks/react-file-viewer > pdfjs-dist > worker-loader > loader-utils
More info https://github.com/advisories/GHSA-hhq3-ff78-jv3g
2 high vulnerabilities and 0 critical vulnerabilities found

Messages

:book:
:link: MB-13774

  Generated by :no_entry_sign: dangerJS against 23d93c84e332a1b2c0ea9e863bc853d518954db2


feat: add http call logging

httpx lets you print the http metthod and url  (without query params) and response if HTTPX_LOG_LEVEL is set to debug
        status = f"{response.status_code} {response.reason_phrase}"
        response_line = f"{response.http_version} {status}"
        logger.debug(
            'HTTP Request: %s %s "%s"', request.method, request.url, response_line
        )

but it doesn't print headers or the body which you may want to do.
This merge request adds some logging.
I wasn't sure how to allow it via environment variable or boolean but settled on adding/using a logger
setting types.HTTP_CALL_LOGGER.setLevel(logging.INFO) logs the http method and url with any query params
setting types.HTTP_CALL_LOGGER.setLevel(logging.DEBUG) logs that plus headers/cookies/body
P.S. Do you know of any more comprehensive openapi examples to test changes on? Especially ones that let you connect to a test server to try out commands.

Hey @rtaycher, sorry for the long delay on response here. I haven't had much time for this project lately so I've been focused only on bug fixes 😬. Adding instrumentation in here is a great idea which I'll definitely move forward with, though I'll probably set up loguru and use the built in way to enable/disable for libraries since it has a lot of nice features.
As far as an end-to-end OpenAPI test server, I don't know of one that exists in the open anywhere. I'm actually working on one right now though as a way to integration-test this generator (and some other, future projects).

Hey @rtaycher, sorry for the long delay on response here. I haven't had much time for this project lately so I've been focused only on bug fixes 😬. Adding instrumentation in here is a great idea which I'll definitely move forward with, though I'll probably set up loguru and use the built in way to enable/disable for libraries since it has a lot of nice features.
As far as an end-to-end OpenAPI test server, I don't know of one that exists in the open anywhere. I'm actually working on one right now though as a way to integration-test this generator (and some other, future projects).

No problem. I actually meant to contribute a little bit more or at least fix my PRs but couldn't find time/got distracted. Thanks for fixing and merging the docstring branch, for introducing me to an interesting new library(loguru), and as always this project.

Gonna close this MR since it probably won't end up merging in (see my previous notes). I know where to find it to help set up loguru in the future.


PS Remote Play - Automate package updates via AU

This changeset implements an update script that utilizes the Chocolatey Automatic Package Updater Module, to assist with keeping this package up to date.
To test:

choco install au
Execute .\update.ps1.

ps-remote-play.nuspec and chocolateyinstall.ps1 should be updated as needed, and a new package that consumes the updated files will be automatically packed.
The package is also currently outdated (latest version as of now is v<IP_ADDRESS>0). I've tested the script locally against the updated version, and it appears to work as I'd expect.

Thank you

This changeset implements an update script that utilizes the Chocolatey Automatic Package Updater Module, to assist with keeping this package up to date.
To test:

choco install au
Execute .\update.ps1.

ps-remote-play.nuspec and chocolateyinstall.ps1 should be updated as needed, and a new package that consumes the updated files will be automatically packed.
The package is also currently outdated (latest version as of now is v<IP_ADDRESS>0). I've tested the script locally against the updated version, and it appears to work as I'd expect.

Trying to adapt this to other packages to save time, do you have any advice on fixing the below for my imazing repo?
Exception: C:\Program Files\WindowsPowerShell\Modules\au\Public\Update-Package.ps1:210
Line |
 210 |  … ion $Latest.Version)) { throw "Invalid version: $($Latest.Version)" }
     |                            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Invalid version: <IP_ADDRESS>


Can't import @excalidraw/excalidraw in Node environment

Just adding import Excalidraw from '@excalidraw/excalidraw'; into a Next project throws an error:
ReferenceError: self is not defined
    at Object.<anonymous> (/Users/gaearon/p/proto2/node_modules/@excalidraw/excalidraw/dist/excalidraw.min.js:2:313)

Apparently, coming from here:
!(function (e, t) {
  // ...
})(self, function (e, t) {

I get that the component might not support SSR, but ideally it would still be possible to import.

Never tried.
That self think comes from Webpack it looks like.
/cc @ad1992

There are a couple of issues when importing in node

self comes from webpack which can be fixed by adding globalObject: "this" in webpack config
Since Excalidraw supports only client-side rendering so window not defined error will start coming up.
Once 1, 2 are fixed, the next error is document not defined which comes due to using style-loader for injecting style tags in js, hence we can't use it if we want to support server-side rendering.
So we then need to have two separate builds one for node and one for the browser.

We should update in the readme as well to import dynamically when component mounted unless we support server-side rendering

In 2023, is this still true that importing from @excalidraw/excalidraw on Node.js fails in all circumstances?
I'm actually not as interested in the <Excalidraw> component - I'm interested in the utilities, specifically:

loadFromBlob()

What I'm trying to do looks approximately like this:
import { readFileSync } from 'node:fs';
import { loadFromBlob } from '@excalidraw/excalidraw';

<Excalidraw
  initialData={(function readInitialDataFromSvg() {
    const svg = readFileSync('./diagram.excalidraw.svg');
    return loadFromBlob(new Blob([svg]), null, null);
  })()}
/>

So that we can load an editor with a diagram from an *.excalidraw.svg file (the initialData prop can also take a promise)
Or... maybe there's an easier way of reading embedded scene data in an SVG in from a local file on the server...?

Seems like if I try it like this in my environment, I'm running into this (unhelpful) error when trying this out in Gatsby, something about eval, although I am guessing this is something that webpack is adding... 🤔
Error in function eval in ../../data/blob.ts:

Couldn't load invalid file

I was getting a similar error whilst running next build. Followed another thread from here and moved my Excalidraw wrapper into my components folder instead of my pages folder in case anyone runs into this issue.


Ug description

Codecov Report

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

@@            Coverage Diff            @@
##             master     #228   +/-   ##
=========================================
  Coverage     52.43%   52.43%
  Complexity      568      568
=========================================
  Files           114      114
  Lines          2523     2523
  Branches        309      309
=========================================
  Hits           1323     1323
  Misses         1107     1107
  Partials         93       93

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 8faba2b...2059b96. Read the comment docs.


Fixed #31117 -- Let TestDbCreationTests use a separate db

This lets TestDbCreationTests properly clean up by not only calling create_test_db, but also destroy_test_db.
Since we cannot call destroy_test_db on the 'default' database, which is used by other tests, and since calling create_test_db on a database that is already created is fragile in any case, this also changes the test to use the 'unused' database that is introduced in this PR.
Additionally, this changes the restoration of the connection settings to restore just the MIGRATE setting, rather than the entire dict, since that prevents issues when the same dict is referenced from multiple places (as detailed in #31117).
Before this PR can be merged, https://code.djangoproject.com/ticket/31055 must be fixed (work underway in #12201). Until then, this PR contains a hack to disable checking entirely.
I am not entirely happy with the name of the new database. I called it 'unused', since it is normally unused by testcases (e.g. not used directly), but can be used by testcases that need to go through the full create/destroy cycle. Any suggestions?
Also, when this database will be used for multiple tests (e.g. I also need it for #12166), I'm afraid there might be concurrency issues when both tests end up running at the same time. Is there any standard way to deal with this? Maybe simply add a lock for using this third database?

I did some experiments to figure out why this bug did not trigger in Jenkins, but now force pushed those experiments away and continued them in #12248. Compared to my first push, I fixed some flake8 errors, added another fix for a similar issue in backends.sqlite.test_creation and made the workaround for #12201 a bit more thorough. With that, all tests seem to be working again now on Jenkins.

(cross-post from https://code.djangoproject.com/ticket/31117)
This PR can now again run the testsuite completely successfully. Only things open are:

How to prevent concurrency issues accessing this new database when multiple testcases use it? Normally, this is handled by making the testrunner make clones of the database, but I can't see how that would work here. Any suggestions?
What name to use for this new database? 'unused' does not seem too great.
The Jenkins database configuration needs to be changed to configure this new database. Is there anything I can do for that?
https://github.com/django/django/pull/12201 should be merged first.

@matthijskooijman Thanks for this patch, however IMO it's too invasive, we can get a proper isolation by creating a test connection more carefully (as a copy of connections[DEFAULT_DB_ALIAS]), see #12256. We should be able to use the same approach in tests/backends/base/test_creation.py.

Thanks for this patch, however IMO it's too invasive,

Probably, yeah. It seemed easy since I was experimenting with an extra db already for #12166, but was not so easy as I had hoped. For that other PR, I might be able to reduce the tests and remove the need for an extra db as well.
As for improving isolation for this test, I think your #12256 looks good, and the same approach will probably work for TestDbCreationTests as well. Using a new connection object, it is probably safe enough to call both create_test_db and destroy_test_db. One point of attention is that create_test_db modifies django.conf.settings.DATABASES as well, which is not isolated by creating a a new connection (which is why I would consider calling destroy_test_db as well).
Regardless, would you maybe consider merging 283eac71afd08e1d01e78a62c6f134d359d758f7 anyway, even when no additional db will be used? Most parts of that commit simply make testcases more self-contained and less reliant on the test configuration, without complicating the tests in any way (some parts of the commit are a bit more elaborate and might be better to drop if not directly used).

Regardless, would you maybe consider merging 283eac7 anyway, even when no additional db will be used? Most parts of that commit simply make testcases more self-contained and less reliant on the test configuration, without complicating the tests in any way (some parts of the commit are a bit more elaborate and might be better to drop if not directly used).

It makes sense at first glance, but let's do this separately because it's not strictly related with an isolation issue.


feat: integrate replicated security

Closing this as no further work has been done since our team's forum post last November.


Parse (....) just like $....$

When I write latex, I'm used to using (....) for inline math. It's a low-importance quality-of-life change, but how hard would it be to add this notation into the franklinjl parsing?
I'd be happy to do the work myself for what it's worth.
Thanks!

Hello @RaphaelArkadyMeyerNYU,
If you get Franklin off the branch pmath-i986 you can do the following:
julia> using Franklin
julia> Franklin.enable_pmath()
julia> fd2html(raw"""
       * old way $x=y$
       * new way \(x=y\)
       """) |> println

old way \(x=y\)

new way \(x=y\)

julia> Franklin.disable_pmath()
julia> fd2html(raw"""
       * old way $x=y$
       * new way \(x=y\)
       """) |> println

old way \(x=y\)

new way (x=y)

So you'd just call enable_pmath() before calling serve.
I likely won't put this into master though as I'm refraining from putting new stuff in Franklin lest it should break other things, the new stuff should ultimately all go in Xranklin (which is more or less done bar gaps in docs which I'm struggling to find time to fill).
I hope this will be somewhat helpful to you though


why can't i set expiration on RAtomicLong?

I'm using Redisson 3.15.3
In my attached example you see that i try to create (or get an existing) RAtomicLong with a given name and expiration. Everything works apart from the yellow marked line where i try to set the expiration. In this concrete case expiration is 15 and unit is TimeUnit.MINUTES. But the expire() call always returns false.
I'm wondering if i do something wrong or what might be common causes for expire to return false (so i could investigate those further)?

I'm closing this


feat: Allow desc to set snippet description

Setting the description with dscr is still supported.
Access to the description is done through description or dscr.
Closes #997

Damn, cleanest PR yet👌👌👌👌👌

Thank you!!

That was quick, thanks :smiley:


sigs.k8s.io/apiserver-runtime is not compatible with<EMAIL_ADDRESS>or later

A compile error occurs when using these versions in go.mod:
k8s.io/apiserver v0.25.0
k8s.io/client-go v0.25.0
k8s.io/klog v1.0.0
sigs.k8s.io/apiserver-runtime v1.1.1
sigs.k8s.io/apiserver-runtime/pkg/builder/rest
vendor/sigs.k8s.io/apiserver-runtime/pkg/builder/rest/provider.go:60:10: cannot use parentPlumbedStorageGetterUpdaterProvider{ } (value of type parentPlumbedStorageGetterUpdaterProvider) as type "k8s.io/apiserver/pkg/registry/rest".Storage in return statement:
parentPlumbedStorageGetterUpdaterProvider does not implement "k8s.io/apiserver/pkg/registry/rest".Storage (missing Destroy method)
The Storage interface defined in k8s.io/apiserver v0.25.0
type Storage interface {
// New returns an empty object that can be used with Create and Update after request data has been put into it.
// This object must be a pointer type for use with Codec.DecodeInto([]byte, runtime.Object)
New() runtime.Object
// Destroy cleans up its resources on shutdown.
// Destroy has to be implemented in thread-safe way and be prepared
// for being called more than once.
Destroy()

}

This sounds like a bug in k8s.io, not a bug in the Go toolchain or libraries.  It should be reported in the k8s.io bug tracker.  Please comment if you disagree.


[Linux/X11] Check if required xkb functions exist before using it.

Apparently soname major means nothing, and half of the methods are missing on the ancient distros like Ubuntu 16.04. I wonder if we should do it for all wrapped libraries.
Fixes https://github.com/godotengine/godot/issues/74197

Yeah soname major only conveys breakage of backwards compatibility, but not forward compatibility. Adding new methods doesn't break backwards compatibility so they don't bump the major; but indeed that doesn't give us the guarantee that we can use these new methods on older versions of the lib.
I don't know if all libraries provide a reliable way to check a version string / number, but maybe we should do this:

Sync our headers with the lowest version of the library which we can use (i.e. which provides the API we need)
Validate that the dlopen'ed library's version is higher than this

Thanks!

Cherry-picked for 4.0.1.


[llvm-dis] Invalid instruction with no BB

Hello,
I get the following error if I call llvm-dis after running crab-llvm on the c code below:
llvm-dis: error: Invalid instruction with no BB (Producer: 'LLVM5.0.2' Reader: 'LLVM 5.0.2')
Information to reproduce the bug:
Crab-llvm: Docker build (i.e. with apron)
$ crabllvm --version
LLVM (http://llvm.org/):
  LLVM version 5.0.2
  Optimized build.
  Default target: x86_64-unknown-linux-gnu
  Host CPU: sandybridge

example.c:
#include <stdio.h>

unsigned a;
unsigned x;

int main(int argc, char const* argv[])
{
    a = *(unsigned *) argv[1];
    unsigned b = a;
    unsigned c = 1;

    x = 32;
    x ^= a * 3;

    if (b == 0x55787855)
        printf("SUCCESS\n");
    else
        printf("FAILURE\n");

    printf("0x%x", x);
    return 0;
}

How to reproduce:
$ crabllvm.py example.c --crab-dom=pk --crab-track=arr --crab-add-invariants=all -o test.crab.bc
[omitted]
$ llvm-dis test.crab.bc
llvm-dis: error: Invalid instruction with no BB (Producer: 'LLVM5.0.2' Reader: 'LLVM 5.0.2')

The error occurred with the following domains:

pk
term-int
term-dis-int
rtz

Have a nice day :)

Thanks for reporting this problem. I've committed a fix for this.
I'll close this issue for now. Let me know if you encounter more problems.


WFLY-10843 PropertyReplacingBeansXmlHandler should handle errors simi…

…larly to what we do in Weld.
JIRA - https://issues.jboss.org/browse/WFLY-10843
No automated test attached as the issue is about seeing a warning in a log (where the app keeps running).

Just a note - tested with @rsvoboda repo with MP TCK and the warning is gone.


Connection failure if the subnet mask is <IP_ADDRESS>

Describe the bug
It seems that the app does not work if the LAN has a subnet mask of <IP_ADDRESS> (i.e. the last 2 series of number both belongs to the LAN).
e.g. When a device is allocated <IP_ADDRESS>, and the other device is <IP_ADDRESS>, both devices belongs to the LAN, but they cannot connect with each other.
To reproduce
Use a router that support choosing a subnet mask of <IP_ADDRESS>, connect two devices to the network and try to send files through the app.
Expected behavior
Could connect successfully.
Screenshots
No response
Desktop (please complete the following information)
No response
Smartphone (please complete the following information)
No response
Additional context
No response

Thanks for your reply. This method does work for me.
However, it is still not convenient to check my IP address all the time, especially when the router is set to DHCP mode. At the same time, at school or at workplace, subnet mask of <IP_ADDRESS> or even <IP_ADDRESS> are note rare.
Could you help fixing the problem?
Thanks so much!

This app has 2 methods to discover other devices:

Send a basic http request to each member: This only works for <IP_ADDRESS>. On larger subnets this is not practical because there would be just too many requests.

Send a broadcast (multicast to be exact), this is implemented in v1.7.0 and should work in any subnet.

The second solution is faster and more general so I am wondering why this does not work on your side.

I think this has something to do with the multicast IP address. It seems that you are using <IP_ADDRESS>/24, so it might only work with subnet mask of 24?

244.0.0.x is reserved for localhost as far as I know. You could try to change the multicast address to 244.0.1.x or something else

You can checkout the multicast logs in
About -> Debugging -> Multicast

Refer #175

Hi,
Thanks for your reply. I have tried clicking the sync button and it does not work as #175 .
It seems that it is not showing other IP segments (my other device has an IP address of <IP_ADDRESS>).

my pc can receive UDP from my phone, but my phone cannot receive udp from my pc.

Closed in favor of #525


WIP: Flb/excelparser add properties

Description

Type of change

[ ] Bug fix.
[ ] New feature.
[ ] Documentation update.
[ ] Test update.

Checklist

This checklist can be used as a help for the reviewer.

[ ] Is the code easy to read and understand?
[ ] Are comments for humans to read, not computers to disregard?
[ ] Does a new feature has an accompanying new test (in the CI or unit testing schemes)?
[ ] Has the documentation been updated as necessary?
[ ] Does this close the issue?
[ ] Is the change limited to the issue?
[ ] Are errors handled for all outcomes?
[ ] Does the new feature provide new restrictions on dependencies, and if so is this documented?

Comments

A CI test fails: https://github.com/emmo-repo/EMMOntoPy/actions/runs/5132278193/jobs/9233367487?pr=612
That you have to remove well-formulated documentation because some arguments are commented for the moment is not what we want.
If mkdocs warnings should be treated as errors, they should be triggered when running pre-commit, not only on github. Furthermore, the error message shown on github gives no give any clue about that the option to turn off the strict mode is called warnings_as_errors. Even worse, setting warnings_as_errors=false still trigger the error...


[NPO] NPO live doesn't download anymore since version 2015.03.09

Test URL: http://www.npo.nl/live/npo-2
It doesn't work anymore unfortunately, while previous versions of youtube-dl worked fine.
 I haven't seen it before. Why include it as a videostream?
By using youtube-dl.exe -f 1089 http://www.npo.nl/live/npo-2 downloading still works though.

This will be fixed in the next version. Thanks for the report.

Confirmed. Thank you, dstftw.


Can't deploy: Invalid argument - setsockopt(2) (Errno::EINVAL)

Checklist

[x] Upgrade Jets: Are you using the latest version of Jets? This allows Jets to fix issues fast. There's a jets upgrade command that makes this a simple task. There's also an Upgrading Guide: http://rubyonjets.com/docs/upgrading/
[x] Reproducibility: Are you reporting a bug others will be able to reproduce and not asking a question. If you're unsure or want to ask a question, do so on https://community.rubyonjets.com
[ ] Code sample: Have you put together a code sample to reproduce the issue and make it available?  Code samples help speed up fixes dramatically. If it's an easily reproducible issue, then code samples are not needed. If you're unsure, please include a code sample.

My Environment

Software
Version

Operating System
macOS Mojave v10.14.4

Jets
2.3.1

Ruby
2.5.3

Expected Behaviour
I've tried to deploy my app using AWS_PROFILE=writtio jets deploy
Current Behavior
The deploy fails:
Traceback (most recent call last):
        40: from /Users/celicoo/.rvm/gems/ruby-2.5.3/bin/ruby_executable_hooks:24:in `<main>'
        39: from /Users/celicoo/.rvm/gems/ruby-2.5.3/bin/ruby_executable_hooks:24:in `eval'
        38: from /Users/celicoo/.rvm/gems/ruby-2.5.3/bin/jets:23:in `<main>'
        37: from /Users/celicoo/.rvm/gems/ruby-2.5.3/bin/jets:23:in `load'
        36: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/exe/jets:14:in `<top (required)>'
        35: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/cli.rb:5:in `start'
        34: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/cli.rb:28:in `start'
        33: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/base.rb:27:in `perform'
        32: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/base.rb:38:in `dispatch'
        31: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/thor-0.20.3/lib/thor.rb:387:in `dispatch'
        30: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/thor-0.20.3/lib/thor/invocation.rb:126:in `invoke_command'
        29: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/thor-0.20.3/lib/thor/command.rb:27:in `run'
        28: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/main.rb:21:in `deploy'
        27: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/deploy.rb:33:in `run'
        26: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/deploy.rb:85:in `build_code'
        25: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/commands/build.rb:27:in `build_code'
        24: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/code_builder.rb:38:in `build'
        23: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/code_builder.rb:38:in `chdir'
        22: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/code_builder.rb:41:in `block in build'
        21: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/code_builder.rb:341:in `package_ruby'
        20: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/code_builder.rb:352:in `build_lambda_layer'
        19: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/lambda_layer.rb:10:in `build'
        18: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/lambda_layer.rb:66:in `replace_compiled_gems'
        17: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-2.3.1/lib/jets/builders/gem_replacer.rb:11:in `run'
        16: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:15:in `run!'
        15: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:30:in `run'
        14: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:30:in `each'
        13: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:33:in `block in run'
        12: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:33:in `each'
        11: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/check.rb:35:in `block (2 levels) in run'
        10: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/exist.rb:18:in `check'
         9: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/exist.rb:18:in `inject'
         8: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/exist.rb:18:in `each'
         7: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/exist.rb:19:in `block in check'
         6: from /Users/celicoo/.rvm/gems/ruby-2.5.3/gems/jets-gems-0.2.0/lib/jets/gems/exist.rb:34:in `url_exists?'
         5: from /Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:1379:in `request_head'
         4: from /Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:1455:in `request'
         3: from /Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:909:in `start'
         2: from /Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:920:in `do_start'
         1: from /Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:943:in `connect'
/Users/celicoo/.rvm/rubies/ruby-2.5.3/lib/ruby/2.5.0/net/http.rb:943:in `setsockopt': Invalid argument - setsockopt(2) (Errno::EINVAL)

Solution Suggestion
Not sure. I've confirmed that the credentials are correct and that the user has enough permissions.

I guess I should ask this question in the http.rb repository.

Updating to Catalina fixed it.


Add tests for over-long request and response bodies

It would be useful to have some tests for what happens when the body is longer than that Content-Length says, e.g. Content-Length: 1 and body "Hello, World!"
I guess if we get an overly-long message body on a connection we should close that connection (don't process any requests/responses remaining on that connection) and log an error. However, I don't know what we actually do.

In the case of HTTP1, hyper enforces the content-length, and returns back just H as the body. If the connection is being kept-alive, then the following read will expect the bytes to be part of a new request. In the example case, it would return a parse error since ello, World! isn't a valid start line. If this were a server request, it would send back a 400 and close the connection. If it were a client response, the connection pool will have noticed the parse error and simply drop the connection.


How to load MP3 FILE?

HOW TO LOAD MP3 File?
When I import MP3 file In JS File,  and I did install url-loader in webpack/base.js and install url-loader --save,  It happened a lot of errors;

Hi, did you figure out how to load those mp3 files?

After importing the file inside your code like this:
import myMp3File from '../assets/myMp3File.mp3';
You will also have to edit the webpack config file called base.js like this:
Find the line under rules and above the word file-loader and change it from:
test: /\.(gif|png|jpe?g|svg|xml)$/i,
to:
test: /\.(gif|png|jpe?g|svg|xml|mp3)$/i,
in order to tell file-loader to also include files with the .mp3 extension.

I ended up figuring that out myself. Also, it took some time before the browser could actually load the sounds. I had changed the extensions in the base.js file, restarted my node application and after 5 minutes or so Firefox stopped giving me the errors and started working. I also read somewhere that Firefox doesn't like to work with .mp3 files, so I converted my audio files to .ogg and added that extension in base.js.
Anyways, just thought that might help someone out later

Further, you can also set up multiple file types for sound like this:
import mySoundFileMp3 from '../assets/sounds/mySoundFile.mp3';
import mySoundFileOgg from '../assets/sounds/mySoundFile.ogg';

and then
this.load.audio('mySoundFile', [mySoundFileMp3, mySoundFileOgg]);

and then whichever one is supported/preferred by the specific browser will be used.


java extractor: Fix NullPointerException on certain compilations with -sourcepath [phab:D993]

Created by jvg at 2016-08-01 19:38:37:

schroederc accepted this revision at 2016-08-01 19:42:34

jvg closed via commit bd502d99c0605f52bb268e109598487327abaceb at 2016-08-01 19:47:10


Casper test for realm creation

[x] Endpoint for confirmation_code
[x] Casper test
[x] Reordered the test numbers to make realm creation the first test.
[x] Move development URLS into separate file

Automated message from Dropbox CLA bot
@hackerkid, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott I am still working on this code. I need to add casper test for verifying the confirmation code and creating the realm. I will ping you when that is done.

Cool, sounds good!

@timabbott Casper test is done! Please check.

@timabbott I have also reordered the tests and made the test for realm creation as first one. Test no 8 was missing so I fixed it in the commit. I hope that's fine.

Can you make the renumbering its own commit at the start of the branch?

(I would like to be able to merge that immediately to minimize the merge conflict risk)

@timabbott Done :+1:

Can you update that commit to fix the numbers in our documentation examples:
tabbott@zaset:~/zulip$ git grep test-js-with-cas
README.dev.md:./tools/test-js-with-casper 10-navigation.js
docs/testing.md:You can run this with `./tools/test-js-with-casper` or as
docs/testing.md:`./tools/test-js-with-casper 05-settings.js` to run a single test file

@timabbott Updated the files where the tests are mentioned.

Cool, merged that commit.  It looks like the "Make send_registration_completion_email return the Confirmation object." commit has the indentation wrong, and that is fixed in the following commit.
Can you fix that?  In general, when you fix an issue in the code, you'll want to do a git rebase -i to fix it in the commit that introduced the issue, not just fix it in the latest commit.  Otherwise, the state could be broken if we merge only part of the branch at a time (and also it breaks using git bisect to debug problems).

@timabbott Sorry. Updated the code :+1:

This generally looks great, thanks for writing this @hackerkid!  I posted a couple small comments; once you've fixed those (and rebased this), let me know and I'll merge.

@timabbott Rebased and updated the code.

Thinking about the new URL a bit more, I think a cleaner and more clearly correct implementation might be to put the confirmation_key fetch route in a new file zproject/dev_urls.py, and have that file only be included from zproject/urls.py if settings.DEVELOPMENT.  As a sidenote, I think then you don't need Http404.
Sorry I keep thinking of things in multiple passes!

@timabbott Done. Please check whether I did the migration of dev URLS correctly.

-# These URL are made available only in the development environment
+# These URLs are available only in the development environment

This hunk should be in the first commit.

@timabbott Done :+1:

Yeah, just fixed that in master, and reviewed and merged this.  Thanks @hackerkid!


Expand "What did we do?" in installation guide

Hey there,
This looks promising! Our team wants to adopt, but we're having trouble understanding what exactly is going on in the installation. For example, the guide mentions that the esbuild watcher is no longer needed - but we'd like to understand why?
Essentially, laying out "what we just did" in even more detail so folks like us can adopt and feel informed about the impact of the decision.
I'm happy to open a PR for the README if you provide the brief details!
Thanks,
Anthony

Hi Anthony,
Very cool to hear you're interested in adopting LiveSvelte for your team!
Basically what happens during the setup is we go from using the standard Elixir esbuild package to using esbuild directly as a node_module. The standard Elixir esbuild package works great for simple projects with phoenix hooks, but to use LiveSvelte we need a more complex setup.
From the official docs: https://hexdocs.pm/phoenix/asset_management.html#esbuild-plugins

Phoenix's default configuration of esbuild (via the Elixir wrapper) does not allow you to use esbuild plugins. If you want to use an esbuild plugin, for example to compile SASS files to CSS, you can replace the default build system with a custom build script.

That's basically what's going on here. LiveSvelte follows the official documentation in this regard and that's by design.
The standard Elixir esbuild watcher is no longer needed as we create a new watcher that does the same thing:
node: ["build.js", "--watch", cd: Path.expand("../assets", __DIR__)]
Let me know if that makes sense or things are still unclear, happy to explain more!
Also, PR's are welcome as you said to improve the documentation :)

@woutdp Thank you, totally makes sense! Your response really cleared things up for me, so I weaved it into the docs – take or leave :)
https://github.com/woutdp/live_svelte/pull/111


Stub repo empty

Hi, first let me say that this project is really great. It looks like the repo for the php stubs (https://github.com/pinepain/php-v8-stubs) is empty so it is not possible to install the stubs through composer. I have played a bit with the extension and build a small lib (https://github.com/apioo/psx-v8) where I would like to use these stubs as composer dev dependency.

Hi, thanks for the hint and sorry for later reply.
I planned to git subsplit stubs to separate repo and then update php-v8 repo, but was a bit busy with php-v8 itself. Later today I will publish stubs and add them to packagist.

Fixed. Now you can just run composer require --dev pinepain/php-v8-stubs to require php-v8 stubs to your project.

Great thx!


Test with newer JDKs

Airlift tests currently won't pass on >8. https://travis-ci.org/airlift/airlift/builds/594032306
This is a much longer story than just travis configuration.


extend physical node detection across partitions

Original author: Jim Phillips (@jcphill)
Original issue: https://charm.cs.illinois.edu/redmine/issues/885

For GPU mapping with partitions it would be good to have something like CmiGetPesOnPhysicalNodeGlobal to tell if another partition's PE is competing for the device.

Original date: 2017-02-01 18:01:56

Is this still desired? We should assign it to someone if so.

Original author: Jim Phillips (@jcphill)
Original date: 2017-02-05 03:36:05

Yes, it is still desired.  NAMD has a work-around in the form of a +devicesperreplica  command line option, but it would be good to automate this instead.

Original date: 2017-10-11 20:17:05

Michael, putting this on you as the the GPU support lead.


Error running 0.4.1

Hello,
I tried to install it in lubuntu running pihole 5 but it fails like this
`   ┌──────────────────────────────────────────┐
│       π-hole 5 list tool  v0.4.1         │
└──────────────────────────────────────────┘
https://github.com/jessedp/pihole5-list-tool
Do not hit ENTER or Y if a step seems to hang!
Use CTRL+C if you're sure it's hung and report it.

Traceback (most recent call last):
File "/usr/local/bin/pihole5-list-tool", line 11, in
sys.exit(main())
File "/usr/local/lib/python3.6/dist-packages/ph5lt.py", line 92, in main
docker = utils.find_docker()
File "/usr/local/lib/python3.6/dist-packages/utils.py", line 74, in find_docker
result = subprocess.run(["docker", "inspect", "pihole"], stdout=subprocess.PIPE, stderr=subprocess.DEVNULL, text=True, check=True)
File "/usr/lib/python3.6/subprocess.py", line 423, in run
with Popen(*popenargs, **kwargs) as process:
TypeError: init() got an unexpected keyword argument 'text'`

Would you mind trying 0.4.2?

Would you mind trying 0.4.2?

How?

Install it.
pip3 install pihole5-list-tool --upgrade

Install it.
pip3 install pihole5-list-tool --upgrade

Works, thanks


os: support runtime poller with os.File on Solaris

The os test TestPipeThreads is failing on the Solaris builder, suggesting that for some reason os.File values are not working with the runtime poller.  The code looks OK to me.  I don't know why it is not working.  Filing this issue to record the problem.

I haven't looked at the original change yet, but pipes are bi-directional on Solaris which tends to trip people up with things that "work" on Linux.
I don't know if this helps, but from pipe(2):

NOTES
Since  a  pipe is bi-directional, there are two separate flows of data.
Therefore, the size (st_size) returned by a call to fstat(2) with argu-
ment  fildes[0] or fildes[1] is the number of bytes available for read-
ing from fildes[0] or  fildes[1]  respectively.  Previously,  the  size
(st_size)  returned  by  a call to fstat() with argument fildes[1] (the
write-end) was the number of bytes available for reading from fildes[0]
(the read-end).

When I get a moment, I'll see if I can figure something out.


Put the logo in and update the palette

Color
Main
Shades

Orange
#E47701
#FE762A #FE5E01 #FE982A

Green blue
#00A398
#6BC4A6 #20686C

White backgound
#FCFCFC

I have requested an svg logo

Where is the logo gonna be made available? The wiki? Or do we have to export from Figma ourselves?

It's in the design team Google drive linked on the wiki. SVG now available


question on 1.6.4 version laser io

Describe the feature you'd like to have added
technically changed but pls make the requirement for patchouli not a hard requirement pls
Additional information
pls change the laser io to not force require patchouli pls

I'll make the change in the next version and test it.

thank you abd dude keep up the good work ure mods are awesome


coin select: add coin selection strategy option to all on-chain RPCs

Comment from https://github.com/lightningnetwork/lnd/pull/8378:
          Yeah, I think that makes sense. Though it feels like that would be a different PR, as we'd probably also want to add such new parameters to the existing RPCs that currently use the config-level strategy (`FundPsbt`, `EstimateFee`, `SendCoins`, `SendOutputs`).

Originally posted by @guggero in https://github.com/lightningnetwork/lnd/pull/8378#discussion_r1452286904
Currently, the coin selection strategy can only be configured globally, via the --coin-selection-strategy=[largest|random] config/CLI option.
But because the strategy might be something a user wants to decide upon for every spend, a similar option should be added to all on-chain RPCs such as the ones mentioned in the original comment above.

@guggero I found there is no CLI command for the SendOutputs RPC in the code and Lightning Engineering API Docs - SendOutputs RPC Call. Is it okay to submit a PR for adding this feature + configuring the coin selection strategy option on the CLI level?

I don't think that is needed. That would require you to specify the pk scripts on the command line, which is kind of awkward. I think in the CLI the sendcoins command should cover all requirements. Or why exactly do you think we'd need a new command?
Also, please keep in mind that reviewing PRs, especially from new contributors, takes a lot of time from the maintainers. So if you really want to support the project, I'd suggest to also start looking into testing and reviewing pull requests from other people instead of writing more code.

Got it @guggero! Thanks for clarifying :)


Support for Reference type as Value?

Hi, we are trying to use fling in one of our project. We are wondering does fling support storing reference(or list of references) as value in the Key-Value pair? Thanks!

Might want to consider https://github.com/mrallen1/fling fork, since this repo isn't likely to get much attention


Internalise FrameBuffer and expose via IFrameBuffer

Prereqs:

[x] https://github.com/ppy/osu-framework/pull/5324

Adds the IFrameBuffer interface.

Internalises FrameBuffer and adds a method to create frame buffers via IRenderer.

Changes parameters of frame buffer creation to avoid ES30 references.

vNext
FrameBuffer is no longer exposed
Frame buffers may be created via the IRenderer and stored as an IFrameBuffer. Be aware that the parameter types have also changed!
class MyDrawNode : DrawNode
{
-   private readonly FrameBuffer myFrameBuffer = new FrameBuffer(new[] { RenderbufferInternalFormat.DepthComponent16 }, All.Nearest);
+   private IFrameBuffer myFrameBuffer;

    public override void Draw(IRenderer renderer)
    {
        base.Draw(renderer);

+       myFrameBuffer ??= renderer.CreateFrameBuffer(new[] { RenderBufferFormat.D16 }, TextureFilteringMode.Nearest);
    }

    protected override void Dispose(bool disposing)
    {
        base.Dispose(disposing);
        myFrameBuffer?.Dispose();
    }
}

Only a 16-bit depth format is supported for the time being, but more formats can be provided upon request.

Applied a bit of conflict resolution, but I don't think I broke anything. Changes look sane.


How to set up scaling sparse models to realistic sizes?

Can I ask what transformation matrix can be used to set the reconstructed sparse model to the size of the real world?

SfM only reconstructs camera poses up to an unknown scale. Recovering metric scale requires additional information like GPS.


I load a checkpoint showing GPU is not enabled.

I trained a model with GPU. However, when I tried to load this model on another one without GPU support, it popped up the error that GPU is not enabled. It seems the model file encodes the context information. So to this end, what is a generally good solution? Should I copy everything from GPU to CPU and then save the model? or maybe I think there could be a parameter that can control this. After all, not every computer can use GPU. Looking forward to good advice.

@horserma , you are wright, save funtion in ndarray contains the context information, which will bring pullze some time.
@tqchen @piiswrong how about change the design that save the model with ctx=mx.cpu() by default?

Saving NDArrays with ctx=cpu sounds good.
Some symbols however behaves differently on cpu vs gpu. I'm not sure if this will cause problems.

@tornadomeet @piiswrong I agree, I think it would be a good idea to separate context info and parameter values. Then we always load the parameters into cpu memory and initialise the model based on the specific context.

This is not yet changed in spec. For temp solution, what we do is always explicitly copy to cpu before serialization


Add Catalan translation

Thanks for the contribution!
Since I can't speak Catalan, I can't proofread the translation, of course - but the metadata looks all right. Don't forget to update the LINGUAS file by adding ca! 😊

Done! Thanks @daPhipz for the reminder!

Thank you both! :)


Pre-Live: Check if a new icon is required for a new feature being released

When the wallet is loading, the image in the right corner should correspond with the message being displayed. So a message about Gifting should have a corresponding image with the gift icon, a message about Donation should have image with donation Account.
@summikhan11

each image doesn't really stay for that long, a generic gear one is enough.

Summi's comment to be kept in mind: "For special scenario we can have dedicated animations with static content." Abhilash and whichever dev would be leading that feature, need to check whether loading image needs to be changed or not.

Need a LN logo for Hexa. We can get rid of "2" if we want. @summikhan11

https://xd.adobe.com/view/ec019ed7-5d7f-4e9f-a73d-ba75f254b45c-b952/screen/d790b1bc-61f7-4183-9d4c-96ca7be05049/

should we need an animation @ASN-BitHyve

should we need an animation @ASN-BitHyve

Lets do that when lightning stage 3 is ready and we're making a lasting change in the logo.


Catch invalid format exception: #501

Simple PR to better handle the InvalidFormatException which gets thrown if you try to commit offsets with a String rather than a Long with Rest Proxy.  This is for https://github.com/confluentinc/kafka-rest/issues/501 and now returns the 400 error code and suppresses the long stack trace being output into the log.

@confluentinc It looks like @MarkC0x just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

cc @rigelbm


Manually adding docs for 0.18.2 and 0.18.3.

Done in a similar manner to #2282.
/cc @bjwatson

LGTM AFAICT.


Update to Tesseract Source

What are the odds we could get an update to the compiled tesseract source? Tesseract 5.5 was released just this last month.

Nothing in particular, I just figured the model has probably improved in accuracy over time.