Snap package crashes while discarding changes

I have run into an issue where VSCode editors will have changes in them and crash when trying to discard the changes.  I initially tried uninstalling and clearing the snap cache, but the open editors persisted.  I then ran the application with the verbose flag received the following output when trying to discard changes:
(code:93979): GdkPixbuf-WARNING **: 12:00:54.352: Cannot open pixbuf loader module file '/usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders.cache': No such file or directory

This likely means that your installation is broken.
Try running the command
  gdk-pixbuf-query-loaders > /usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders.cache
to make things work again for the time being.

(code:93979): Gtk-WARNING **: 12:00:54.352: Could not load a pixbuf from icon theme.
This may indicate that pixbuf loaders or the mime database could not be found.

(code:93979): GdkPixbuf-WARNING **: 12:00:54.352: Cannot open pixbuf loader module file '/usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders.cache': No such file or directory

This likely means that your installation is broken.
Try running the command
  gdk-pixbuf-query-loaders > /usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/loaders.cache
to make things work again for the time being.
**
Gtk:ERROR:/build/gtk+3.0-24RAPQ/gtk+3.0-3.22.24/./gtk/gtkiconhelper.c:493:ensure_surface_for_gicon: assertion failed: (destination)

So I tried the recommended suggestion, but the folders only existed to /usr/lib/ so the command failed.  I then created folders up to /usr/lib/x86_64-linux-gnu/gdk-pixbuf-2.0/2.10.0/ and was able to run the above command.  This fixed the issue for me and I was able to discard the changes in the editors.
Now I'm not all that familiar with GTK libraries, but aren't snap packages supposed to come with all of the dependencies OOTB?  I don't see why it would need a file on my host machine.  This only started happening 3-4 weeks ago.

VSCode Version: 1.47.3 (snap)
OS Version: Manjaro 20.0.3

Steps to Reproduce:
I am a little unsure of how to reproduce it honestly.  I think it happens when the files that are open and have changes in the buffer are no longer mounted.  This happens when an sshfs session detaches because of hibernation or something else.

Does this issue occur when all extensions are disabled?: Yes
Lmk if you need any more info.  Thanks

/duplicate of #100940
Also check comment https://github.com/microsoft/vscode/issues/100940#issuecomment-651225395 if workaround described here helps.


Timeline not scrolling to right based on time

First of all, Great work. Hi, I am using React Player as my video source, based on the lib, I am setting the timeline time from the react player output. My auto scroll prop of react-timeline-editor is true, but the timeline not scrolling to the right with respect to the time from the video. Please help.

First of all, Great work. Hi, I am using React Player as my video source, based on the lib, I am setting the timeline time from the react player output. My auto scroll prop of react-timeline-editor is true, but the timeline not scrolling to the right with respect to the time from the video. Please help.

AutoScroll is used for scrolling while dragging. Scrolling by time when playing is another set of logic.
Therefore, added TimelineState.setScrollLeft attribute, allow custom scrollLeft on demand.
Released version 0.0.5.
You can refer to this example：demo

@xzdarcy Thanks i will try

work like a charm, Thank you for your effort


Add option to uninstall_operator to avoid ns cleanup

Short description:
Adds an option to uninstall_operator method to avoid complete ns cleanup
More details:
Currently when we uninstall an operator using the uninstall_operator function, we clean up the operator namespace completely. This is a problem if the operator is installed in openshift-operators, as it will delete any other cluster-scoped operator installed. This PR adds an option to avoid this behavior and just clean up the specific target operator, without affecting the previous behavior.
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/verified


Vickel: Watch tfmlaw.com

Vickel requests the watch of the watch_keyword tfmlaw\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

tfmlaw\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Cody Gray in Charcoal HQ


Optimizer

Hi, great job! I have a question about implementation. Why are two optimizers used instead of one since all the settings of the two optimizers are exactly the same？

No specific reason ... I guess it was created while building the code hierarchy, but you can just use one.


[repo] update chart-testing-action

Release action is acting up. Scattered GH reports aren't conclusive, but they all mention issues with arguments, so hopefully updating to the final 1.0.0 and using the config: syntax will clear that.

@rainest CI failed.


Fix double connect from benchmark

While testing this for my prod system, I was wondering why it'd connect twice, write twice, etc. Tracked it to the benchmark function.

Oh, I see a "do not modify" on this file but I don't see the mentioned file in the other repo.

Wondering if removing the Semantic Logger dependency was the right one. Tried to remove the dependency for anyone that does not want that dependency, but I realize now that their is too much logging code being duplicated in Net::TCPClient. Also Semantic Logger is moving on and the logging file in this gem is falling behind.
Debating just adding [Semantic Logger]( Semantic Logger back as a dependency to this gem, and removing the logging file.
Any thoughts?

I think it would be OK. I have a slight preference not to do my logging in another thread, and I don't see an option on SL for that. I'm building a plugin for fluentd, and I can't control the logging of the system, so there's some risk of out of order logging (or some threading bug since their logger can feed back into the system? Maybe? would have to think about it.).
But that's not the end of the world and some strategically placed flushes would solve it anyway.

Sounds like I should keep SemanticLogger as a soft / optional dependency. Will take extra work to write some tests for the built-in logging class. Thank you for the pull request.
Semantic Logger is an ideal solution for completely removing the logging implementation in fluent: https://github.com/fluent/fluentd/blob/master/lib/fluent/log.rb
Moving all logging to a separate thread allows logging to occur without slowing down the program at all since the message is just dropped into a queue and the application moves on. This becomes more of an issue when we start writing to multiple destinations at the same time such as NOSQL, bugsnag, NewRelic etc.
Sequencing is guaranteed since the order they go into the queue is that same as they are written to the multiple destinations.
For example, Semantic Logger uses Net::TCPClient to write Syslog messages when the TCP protocol is being used: https://github.com/rocketjob/semantic_logger/blob/master/lib/semantic_logger/appender/syslog.rb


Auth

Not a good implementation, do let me know the changes needed !

In the future, consider using clear commit messages. It will be beneficial when reviewing your work later on.

Merging


⚠ SpeakUpConference.com has degraded performance

In 26daec3, SpeakUpConference.com (https://www.speakupconference.com) experienced degraded performance:

HTTP code: 200
Response time: 14362 ms

Resolved: SpeakUpConference.com performance has improved in 53890f6 after 18 minutes.


feat: reports source of ESC errors when possible

Tested successfully - output from testing an invalid "asay" command:
2022-4-9T115532 (E)    Errors in file res://game/rooms/room02/esc/button.esc
Invalid command detected: asay
Command implementation not found in any command directory

Thanks!


Horizontal Rule

'insertHorizontalRule' (mentioned in the documentation) didn't work for me. Then I changed it to the following and it worked fine.

var btnsGrps = jQuery.trumbowyg.btnsGrps;

$('.simple-editor').trumbowyg({
btns: ['viewHTML',
'|', 'formatting',
'|', btnsGrps.design,
'|', 'link',
'|', 'insertImage',
'|', btnsGrps.justify,
'|', btnsGrps.lists,
'|', 'horizontalRule']
});

Hi! Browser? Version?
Thanks :)

insertHorizontalRule is the execCommand function
horizontalRule is the button name for the toolbar.
I was make a mistake in the documentation ?

OK. I think you talk about this line : https://github.com/Alex-D/Trumbowyg/blob/gh-pages/documentation.html#L385
It's a mistake, i'll fix it very soon. Sorry for the flood and thanks for report!

Tested in Google Chrome, Lastest Version 40.0.2214.111 m. I think so, you made a mistake in the documentation. Thanks.

Fixed now : https://github.com/Alex-D/Trumbowyg/blob/gh-pages-v2/documentation.html#L413
Soon published (when v2.0.0 will be stable).

Publised!
http://alex-d.github.io/Trumbowyg/documentation.html#button-pane


Fix river network direction

Currently a lot of the lines in the river network are not properly aligned. Needs to be fixed.

resultatene var ganske lovende, men ikke 100%
nå settes min testing på pause
au revoir!


Handling #getFilteredChildren incompatibility in new junit

ParentRunner on JUnit master (link https://github.com/junit-team/junit/blob/master/src/main/java/org/junit/runners/ParentRunner.java#L422) changed its signature for getFilteredChildren from a List to a Collection. Ran into this attempting to integrate burst into some Square code, and I'm assuming our version of junit there overrides the pinned version of junit used here, 4.11.
This change should be safe for current JUnit version and future JUnit versions (newest is 4.12).
@dlubarov @JakeWharton

LGTM, thanks Paul!


[Data-657] carto implement pose marker

Implement an optional marker for the robot's current location on the jpeg map endpoint

https://viam.atlassian.net/browse/DATA-657

Testing with apriori map loading. Using the feb_11_large dataset and updating/localizing with feb_11_small.
initial map(only feb_11_large)

using updating mode

using localization mode

fine with removing the original trajectory method with this PR. The submap generator still relies on the surfaces from that function so would need to pull those out

@JohnN193 I don't think this is worth the effort right now, since we can't really see the trajectory anyways, and as much as the jpeg image improvements are not going to be pursued in favor of implementing the generalized map interface, I think that removing the faded trajectories is equally not worth the time & effort


Minor documentation update

Call out distinction between original and current SQL engines.
Signed-off-by: MaxKsyunz<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #728 (1b9a8a4) into main (8f1b31d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #728   +/-   ##
=========================================
  Coverage     97.76%   97.76%
  Complexity     2880     2880
=========================================
  Files           276      276
  Lines          7077     7077
  Branches        447      447
=========================================
  Hits           6919     6919
  Misses          157      157
  Partials          1        1

Flag
Coverage Δ

sql-engine
97.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[EOS-11067] Move allow CAPA access to IMDS instead of keos-installer

The only component that has access to IMDS (AWS) must be CAPA

LGTM


Leakage

Follow up to #775, using numpy

Yes, that's the idea.

Ok i'll close #775

can you re-open this PR? I will apply it again since github seems to have reverted it along with #789


have problems in installing display

when i try luarocks install display,
it shows:
Installing https://raw.githubusercontent.com/torch/rocks/master/display-scm-0.rockspec
Missing dependencies for display scm-0:
   async >= 1.0 (not installed)

display scm-0 depends on async >= 1.0 (not installed)
Installing https://raw.githubusercontent.com/torch/rocks/master/async-scm-1.rockspec
Cloning into 'async'...
remote: Counting objects: 210, done.
remote: Compressing objects: 100% (184/184), done.
Rremote: Total 210 (delta 19), reused 172 (delta 19), pack-reused 0eceiving objects:  91% (192/210), 403.80 KiB | 95.00 Receiving objects:  93% (196/210), 403.80 KiB | 95.00 KiB/s
Receiving objects: 100% (210/210), 403.80 KiB | 95.00 KiB/s, done.
Resolving deltas: 100% (19/19), done.
nmake LUA_BINDIR=E:/torch/distro-win/./install/bin  LUA_LIBDIR=E:/torch/distro-win/./install/lib  LUA_INCDIR=E:/torch/distro-win/./install/include

Microsoft (R) 程序维护实用工具 12.00.21005.1 版
版权所有 (C) Microsoft Corporation。  保留所有权利。

makefile(1) : fatal error U1000: fatal error U1000: Syntax error: missing in macro call ")"
Stop.

Error: Failed installing dependency: https://raw.githubusercontent.com/torch/rocks/master/async-scm-1.rockspec - Build error: Failed building.

I'm currently using VS2013 on Win 10 x64.
update:
I tried VS2015, the same ...

It seems async's rockspec is not windows ready, you can try to git clone async project, modify the rockspec file for windows. Here is a reference from torch7 project. You may have to use && to combine commands since luarocks on windows may not recognize commands in multiple lines.


Remove release milestone github action

How to categorize this PR?

/area dev-productivity
/kind enhancement
What this PR does / why we need it:
We would like to test Tide milestone functionality this release (ref). Thus, this PR removes the release milestone Github action to be able to test Tide properly.
We could revert the change, if the test was not successful.
Special notes for your reviewer:
PR has to be merged manually because Tide is not allowed to change Github workflows.
This is the corresponding PR to enable milestone check for release v1.55 in prow.
Release note:

NONE

Merging manually...


format examples

It would be nice if several people could review this PR, e.g., @trixi-framework/developers
TODO

[x] update style guide

I updated this PR based on the recent main. I also added the workaround for the weird formatting of binary operators everywhere. Some stochastic last reviews would be nice before merging this when tests pass.
There are some instances where the arguments x, t are on separate lines when defining boundary conditions - but that's necessary given our line length limit we use right now. The other cases with t, alone on a separate line are fixed.

Looks alright to me, we loose some "nice" formatting like
coordinates_min = (-1.0, -1.0) # minimum coordinates (min(x), min(y))
coordinates_max = ( 1.0,  1.0) # maximum coordinates (max(x), max(y))

which becomes
coordinates_min = (-1.0, -1.0) # minimum coordinates (min(x), min(y))
coordinates_max = (1.0, 1.0) # maximum coordinates (max(x), max(y))

but apart from that this looks fine to me.

Looks alright to me, we loose some "nice" formatting like
coordinates_min = (-1.0, -1.0) # minimum coordinates (min(x), min(y))
coordinates_max = ( 1.0,  1.0) # maximum coordinates (max(x), max(y))

which becomes
coordinates_min = (-1.0, -1.0) # minimum coordinates (min(x), min(y))
coordinates_max = (1.0, 1.0) # maximum coordinates (max(x), max(y))

but apart from that this looks fine to me.

Personally, I actually  prefer the formatted one.

Does someone know why some of the tests fail? Some of them even quite considerably (the actual error differs by one order of magnitude to the expected one)... We having similar issues also in other PRs, but strangely not the same tests fail.

Does someone know why some of the tests fail? Some of them even quite considerably (the actual error differs by one order of magnitude to the expected one)... We having similar issues also in other PRs, but strangely not the same tests fail.

I have no clue... 😞

Since CI also fails in other PRs, I think we should go ahead and merge this PR to avoid merge conflicts with other works. Okay for you, @sloede?

Since CI also fails in other PRs, I think we should go ahead and merge this PR to avoid merge conflicts with other works. Okay for you, @sloede?

I think we should wait until all tests have run, but yes, I agree that if this is just the same errors as elsewhere, we can merge

Does someone know why some of the tests fail? Some of them even quite considerably (the actual error differs by one order of magnitude to the expected one)... We having similar issues also in other PRs, but strangely not the same tests fail.

So for the elixirs using the error-based timestepping I remember that there where actually different errors depending on the machine. This time, I can actually reproduce the errors on my machine and we have differences for other examples using CFL-based timestepping.
In principle, we should be able to find the point where things went wrong, right...?

Yes, you're right. Did some update of one of the time stepping dependencies introduce the change we see here?


Use OS::delay_usec() to avoid using deprecated usleep() on Linux

usleep(3) was declared obsolete in POSIX.1-2001 and removed in POSIX.1-2008. nanosleep(2) was recommended to be used instead.
OS::delay_usec() internally uses nanosleep().
This also uses large number separators for improved readability.

See https://github.com/ioquake/ioq3/pull/652.

Thanks!


package.json "main" key has wrong file name

Describe the bug
The file path for "main" in package.json should be ./lib/index.js
Steps To Reproduce
- create a directory for reproduction and `cd` into it
- `yarn init -y`
- `yarn add @aws/clickstream-web`
- put the following in `index.js`:

const clickstream = require('@aws/clickstream-web')
console.debug(clickstream)

- run `node index.js`

Observe that the program crashes.

Expected behavior
The program outlined in the reproduction steps should successfully run to completion.
The fix is to update "main" to point to ./lib/index.js
Clickstream-Web Version
0.10.0
Node version
v20.11.0
TypeScript version
N/A
Relevant log output
<details>
<summary>Log Messages</summary>

INSERT LOG MESSAGES HERE

```
Is this a regression?
Yes
Regression additional context
No response
Browser
N/A
Additional context
Here is the output of the reproduction program:
node:internal/modules/cjs/loader:452
      throw err;
      ^

Error: Cannot find module<EMAIL_ADDRESS>Please verify that the package.json has a valid "main" entry
    at tryPackage (node:internal/modules/cjs/loader:444:19)
    at Module._findPath (node:internal/modules/cjs/loader:715:18)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1130:27)
    at Module._load (node:internal/modules/cjs/loader:985:27)
    at Module.require (node:internal/modules/cjs/loader:1235:19)
    at require (node:internal/modules/helpers:176:18)
    at Object.<anonymous> (/Users/harrison.marshall/Developer/reproduce-clickstream-issue/index.js:1:21)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Module.load (node:internal/modules/cjs/loader:1207:32) {
  code: 'MODULE_NOT_FOUND',
  path<EMAIL_ADDRESS>  requestPath: '@aws/clickstream-web'
}

Version 0.10.1 fixes this issue, please try again.


[KYUUBI #2427][FOLLOWUP] Flaky test: deregister when meeting specified exception

Why are the changes needed?
https://github.com/apache/incubator-kyuubi/issues/2427
master nightly log :
04:23:04.585 Executor task launch worker for task 0.0 in stage 2.0 (TID 2) ERROR Executor: Exception in task 0.0 in stage 2.0 (TID 2)
org.apache.spark.SparkArithmeticException: Casting 2.147483648E9D to "INT" causes overflow. To return NULL instead, use 'try_cast'. If necessary set "spark.sql.ansi.enabled" to false to bypass this error.
	at org.apache.spark.sql.errors.QueryExecutionErrors$.castingCauseOverflowError(QueryExecutionErrors.scala:95) ~[spark-catalyst_2.12-3.3.0-SNAPSHOT.jar:3.3.0-SNAPSHOT]

[SPARK-39007][SQL] Use double quotes for SQL configs in error messages
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

local test

Codecov Report

Merging #2568 (0f9fb28) into master (270a572) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2568   +/-   ##
=========================================
  Coverage     63.72%   63.72%
  Complexity       73       73
=========================================
  Files           377      377
  Lines         18099    18099
  Branches       2444     2444
=========================================
  Hits          11533    11533
  Misses         5467     5467
  Partials       1099     1099

Impacted Files
Coverage Δ

...ache/kyuubi/operation/KyuubiOperationManager.scala
83.60% <0.00%> (-1.64%)
:arrow_down:

.../engine/spark/session/SparkSQLSessionManager.scala
80.95% <0.00%> (+1.19%)
:arrow_up:

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 270a572...0f9fb28. Read the comment docs.

thanks, merged to master

Checked today's build spark-master and it was successful.

https://github.com/apache/incubator-kyuubi/actions/runs/2288458282

thanks @cxzl25 for the double check


Enable creation of PermissionError and ResourceError Exceptions

Context
We extensively use ichiban/prolog for our okp4/okp4d protocol, and we have reached a point where there is a need to standardize the error representation of our custom predicates. This includes throwing exceptions such as permission_error and resource_error.
Issue
At present, the ichiban/prolog library allows custom predicates to raise exceptions such as TypeError and DomainError. However, it lacks the capability to raise PermissionError and ResourceError exceptions since the relevant functions are private (engine/exception.go#L294). Additionally, the NewException function, while seemingly useful, cannot capture the private current context (varContext), which is important.
Solution
A straightforward solution would be to make the functions responsible for creating exceptions like PermissionError and ResourceError accessible to developers by making them publicly available. However, I trust your judgment for making the final decision on the solution.
Thx
PS: I would be delighted to contribute if needed. Just let me know.

I was sure that the developers will need TypeError and DomainError but unsure about other errors.
So I've been waiting for the requests. Let's add PermissionError and ResourceError If you need them!
I'll do it when I've got time (been busy moving) or you can make a PR and I'll happily merge it!

Thank you for the contribution, @ccamel! I merged the PR and released v1.2.0.


[MB-1573] Implement lookup for CubicFeetCrating

Description
The lookup for CubicFeetCrating finds the length, width, and height of the crate for a given service item, and multiplies them to get the volume.
Reviewer Notes

I had to preload Dimensions when setting up lookups to make sure that I could access the field in the new lookup. If there's a better way to do this, let me know.
I've been going back and forth about moving the thousandth of an inch to feet conversion to unit.ThousandthInch. I haven't so far because there are a few potential numerical pitfalls here and I wanted everything in one place so those would be easier to spot, but I'm open to refactoring that out.

Setup
I don't think there's any code that uses this lookup yet, so just make sure tests pass:
make server_test

References

Jira story for this change

Error: This repo is not allowlisted for Atlantis.

Messages

:book:
:link: MB-1573

  Generated by :no_entry_sign: dangerJS against b5e9129f95b6a5a09b012add2753e5c2f329f33b


Added a General tab page with XAML layout and controls according to spec

Created a proof of concept General settings tab page with XAML layout. Fontsizes, margins and paddings are a 1:1 copy of the official Windows 10 settings app.
Summary of the Pull Request

References

PR Checklist

[] Applies to #xxx
[] CLA signed. If not, go over here and sign the CLA
[] Tests added/passed
[] Requires documentation to be updated
[] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

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

I would do a next PR to start stylizing stuff so we are consistent between navview items

do a next PR to start stylizing stuff so we are consistent

Yeah good idea.. especially for the margins and paddings. Will pick that up

We can merge this in and do a PR for that

You may want to not refer to it as a Tab page.  The TabView is a specific thing, and this uses the Navigation View or Split View instead. :)


Can't Build by JDK 21 on Gradle 8.4

Current Behavior
Welcome to Gradle 8.4-20230928234018+0000!
Here are the highlights of this release:

Compiling and testing with Java 21
Faster Java compilation on Windows
Role focused dependency configurations creation

Starting a Gradle Daemon, 3 incompatible Daemons could not be reused, use --status for details

Configure project :aidl
WARNING: The option setting 'android.enableParallelJsonGen=true' is experimental.
The current default is 'false'.
WARNING: We recommend using a newer Android Gradle plugin to use compileSdk = 34

This Android Gradle plugin (8.1.0) was tested up to compileSdk = 33 (and compileSdkPreview = "UpsideDownCakePrivacySandbox").
You are strongly encouraged to update your project to use a newer
Android Gradle plugin that has been tested with compileSdk = 34.
If you are already using the latest version of the Android Gradle plugin,
you may need to wait until a newer version with support for compileSdk = 34 is available.
To suppress this warning, add/update
android.suppressUnsupportedCompileSdk=34
to this project's gradle.properties.
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':server:compileDebugKotlin'.

Unknown Kotlin JVM target: 21

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

Expected Behavior
Build Successful
Context (optional)
No response
Steps to Reproduce
./gradlew :manager:assembleDebug
Gradle version
8.4-20230928234018+0000
Build scan URL (optional)
No response
Your Environment (optional)
No response

Same here

it is not a bug. Kotlin still not support JDK 21

it is not a bug. Kotlin still not support JDK 21

Okey,I know, thanks

See https://github.com/gradle/gradle/issues/25574#issuecomment-1761314551

Hi, you can check out this guide https://androidstrike.com/fix-we-recommend-using-a-newer-android-gradle-plugin-to-use-compilesdk-34/

8.7 still not java 21?

I'm not sure if this has been solved yet, But I just had this issue today and came across this post.
I resolved my issue by changing the Gradle JVM to Amazon Corretto version 21.0.2 and was able to compile with minimal to no errors.


:Implement easymesh translation for radio subdoc

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


Lower arithmetic.int_ops.iu_to_s to LLVM

In principle this should be a noop, but we could check for overflow and panic?

Related to https://github.com/CQCL/hugr/issues/1702


Added HeadSpin

Added HeadSpin

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

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/testing-other-browsers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Who will be able to merge this request?

Docs Build status updates of commit 9623cd5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/testing-other-browsers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 60c09b2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/testing-other-browsers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Reproducible error in RbVmomi::VIM::finalizer

I've been getting NotAuthenticated errors autorunning specs with guard as it's one of the rare scenarios where GC can clean up RbVmomi while stdout/stderr are still synced.
If you add a begin block with a debug point in the rescue block to the finalizer, the following script should reproduce the problem:
require 'rbvmomi'
RbVmomi::connect(
  #<args>
).close
sleep 3
puts 'connected and closed'

An error should get caught when the script is cleaning up.

True. Either option would be more efficient. I not sure about suppressing all exceptions that could occur in the finalizer, as there may be a small chance we want to see other errors that could occur? Another option could be to update a flag attribute (say logged_in?) on each call to #connect and #close and then pass it to the finalizer. That would avoid the extra overhead and avoid supressing errors. So I'd say either suppress only NotAuthenticated, or pass in a flag to indicate whether is session might be dangling.

Also wanted to say thanks for making this project available. Our hosting service will be a whole lot easier to automate because of it.
Let me know if you'd like me to close this request now or, if you want, make the changes you deem best.

I think catching NotAuthenticated would be best. It'd be great if you could update or resubmit the pull request with that change.

Cool stuff. Here's the updated changes then.


Scrollable buttons

How to make buttons scrollable?

Do you mean rotary scrolling?

Yes I mean that

According to the current code design, there are some difficulties in achieving the rotation, and it is also true to realize the rolling with the fingers.

Ok, any suggestion to implement that?

If you want to implement a scrollable menu, the sliding interaction should be received by a UIScrollView. If you use a swipe gesture, you need to implement the effect of inertia deceleration. The problem with UIScrollView is that there is no way to scroll indefinitely.
My idea is

Upper layer - UIScrollView receives touch events and forwards click events
Middle Tier Container View - Contains all buttons, calculates the rotation angle according to ScrollView, and implements animation
The lower layer is the container View

Use the x+y of the ScrollView's offset to calculate a detla value, which is passed to the middle layer View, which is used to rotate all the buttons.

@zsy78191 thank you, let me try it...

You are welcome


An error occurs when starting the queue: The process has been signaled with signal "11

Laravel Version: 5.7.24
PHP Version: 7.2
Database Driver & Version: MYSQL: 5.7.24-0ubuntu<IP_ADDRESS>

Description:
An error occurs when starting the queue php artisan queue:listen:
Symfony\Component\Process\Exception\ProcessSignaledException  : The process has been signaled with signal "11".
`[2019-02-01 13:19:11][1] Processing: App\Jobs\CreateOrUpdateWebhooks
Symfony\Component\Process\Exception\ProcessSignaledException  : The process has been signaled with signal "11".
at /mnt/c/Users/User/Documents/kedrapps/kedr-cross-sell-and-up-sell/vendor/symfony/process/Process.php:426
422|             usleep(1000);
423|         }
424|
425|         if ($this->processInformation['signaled'] && $this->processInformation['termsig'] !== $this->latestSignal) {

426|             throw new ProcessSignaledException($this);
427|         }
428|
429|         return $this->exitcode;
430|     }

Exception trace:
429|         return $this->exitcode;
430|     }

Exception trace:
1   Symfony\Component\Process\Process::wait()
/mnt/c/Users/User/Documents/kedrapps/kedr-cross-sell-and-up-sell/vendor/symfony/process/Process.php:235
2   Symfony\Component\Process\Process::run(Object(Closure))
/mnt/c/Users/User/Documents/kedrapps/kedr-cross-sell-and-up-sell/vendor/laravel/framework/src/Illuminate/Queue/Listener.php:175`

This generally means there was a fatal error in the spawned process (the queue worker).  There should be an error in the log file that shows the actual error.
If you run the queue worker with queue:work instead of queue:listen it won't spawn a process so you should see the actual error message instead of what you are seeing now.

@yuloh Thank you for your reply. I tried to start the queue with the command - "php artisan queue:work". And after that another error is displayed - Segmentation fault (core dumped). What does this mean? Logs are empty.

That is most likely an error in your install of PHP itself or one of your extensions.  It's very unlikely that it's specifically related to Laravel.  A good place to start would be disabling any unnecessary PHP extensions.
Since this issue tracker is for issues with Laravel you will probably have better luck getting help with one of these resources: https://github.com/laravel/framework/blob/0682ecbf65346b4fd7e5d9b3b3f8aa2be527232c/.github/ISSUE_TEMPLATE/2_Support_question.md

@improck as a quick, since you seem to be on Ubuntu, you may find a crash file in /var/crash. This one you could debug yourself (google is your friend) and it may hint quickly which extension is the problem.

@GhostCoding1 It's fantastic!!!
After removed sentry library, no errors. You saved a lot of time for me!!! Now everything works without errors. Many thanks for your recommendation. And thanks to everyone who decided to help)))

@GhostCoding1 @improck would you mind sharing which exact Sentry version you did have installed before removing it?

@GhostCoding1 Of cource. Version: "sentry/sentry-laravel": "1.0.0-beta2". Probably need to inform the developers of the library?


Pool Destroy

Storage pool destroy (stop) feature.

https://github.com/Hari-Kiri/virest-storage-pool/pull/51


Complexity or Simplicity

I have been trying the whole day to use this without any success. I am using this in an ASP.Net aspx page in which I am including js files using script tags.
Getting/downloading the json files is like climbing Mount Everest without oxygen since nothing in the documentation explains this clearly and then the task of finding how to load these json files is like finding a needle in haystack. It would help a  lot  if there was a large zipped file that included all the json files and the link to this zipped file was provided in Getting Started section of documentation.
BUT, then I tried the older jQuery globalize version and it was too simple to use and I had a running page in under 2 minutes.
So my question is: What would I gain by using this ultra-complex library rather than the original jquery globalize which had me up and running in 2 minutes?  Even if I do succeed in using this library, and leave the project on completion, then I am not sure how easy/difficult maintenance of globalize in this page would be when a new developer takes over. When I used the jquery globalize plugin, I only downloaded 2 small files - jquery-glob.min.js of 12KB size and jQuery.glob.hi-IN.min.js of 1.3KB size. So, may be I am missing something which is there in newer version even though its extremely complex.

Why Globalize? has our design goals. The first three bullets are valid only for 1.x. In summary, that means to you:

better i18n quality and coverage due to CLDR
better control: modular features, i.e., load the code you need; no embedded data, i.e., use the CLDR version you want, the slice you want, load it the way you want, you can even override the data with business specific updates.
better efficiency: https://github.com/globalizejs/globalize-compiler (see the README of https://github.com/rxaviers/todomvc/tree/react-globalize/examples/react-globalize for some numbers)

The easiest way to get started is https://github.com/globalizejs/globalize/tree/master/examples/app-npm-webpack
Was some of this info hard to find? Please, help us improve our docs/education. Thanks


Is RxJS retry patched?

I've noticed that anywhere where I could normally update binded attributes no longer works when I add retry (other than the initial attempt).
Is this the intended behaviour?
I have to use NgZone but I would prefer not to.

@JiaLiPassion may be #1194 is related?

will close for now, please reopen if there is future information.


STM32MP1 Series support

Add basic support of the Cortex-M4 core for the  STM32MP1 processor.
Provide support for the stm32mp1 DK2 discovery board.
This first step allows to run and debug the Hello_world sample.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13794   +/-   ##
=======================================
  Coverage   52.26%   52.26%
=======================================
  Files         307      307
  Lines       45410    45410
  Branches    10504    10504
=======================================
  Hits        23734    23734
  Misses      16887    16887
  Partials     4789     4789

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 cd65ed8...1ac85f9. Read the comment docs.

recheck

Looks good to me, one minor comment is the commit order in the PR seems odd.  The board commit should be the last one.

Looks good to me, one minor comment is the commit order in the PR seems odd. The board commit should be the last one.

I have changed the commit order and re-push commits.
Thanks

Update done to fix rebase issue on master (compilation failed due to https://github.com/zephyrproject-rtos/zephyr/commit/4da0f8b7965b962e3ac7fc0183d327b275adae18).
Impact in soc/arm/st_stm32/stm32mp1/linker.ld to remove (OPTIONAL)

Don't take into account false Kconfig fail:
reason is that error reports a ext/hal/st/stm32cube/stm32mp1xx/README:51 text which describes the (https://github.com/zephyrproject-rtos/zephyr/pull/13794/commits/9f63849214f76edb154d815cac92df7df1c11923) commit that updateq definition in ext/hal/st/stm32cube/stm32mp1xx/drivers/include/stm32mp1xx_ll_rcc.h to suppress this error check.

Hello Kumar,
Thank for the merge!
Arnaud
From: Kumar Gala<EMAIL_ADDRESS>Sent: vendredi 19 avril 2019 19:06
To: zephyrproject-rtos/zephyr<EMAIL_ADDRESS>Cc: Arnaud POULIQUEN<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [zephyrproject-rtos/zephyr] STM32MP1 Series support (#13794)
Merged #13794https://github.com/zephyrproject-rtos/zephyr/pull/13794 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/zephyrproject-rtos/zephyr/pull/13794#event-2288627566, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGCNTA2XHTSIMCANYRNBZPDPRH3WTANCNFSM4G2J5BXQ.


Oversized regions dropped in remapped chromosomes

Bug: If a chromosome that has been remapped (e.g. chromosome 23 to chromosome X) contains regions that are larger than the size threshold, they are filtered with the original chromosome name, leading in those regions being dropped from the results.
The bug seems to be due to the region beds being in the original chromosome mapping, and the summary statistics that the regions are built from being in the mapped chromosome. When the summstats are filtered using the original chromosome beds, the chromosomes do not match, which filters the whole region out.
This was seen in R9, where M13_ARTHROSIS_KNEE misses the complete chrX:95-105MB region, even though there is signal (https://r9.finngen.fi/pheno/M13_ARTHROSIS_KNEE).
@mkanai @Fedja what do you think, what would be the best way to fix this? We could

remap also the beds, but then they'll be in a different chromosome than before (intentional or not?)
not remap in the region selection, and remap the chromosomes using e.g. sed after region selection
Something else?

 I think it's essentially filter_sumstats bug which does not map chromosomes to integers
https://github.com/FINNGEN/finemapping-pipeline/blob/b54125e79d0085d33632dcad93cf19849197f276/python/make_finemap_inputs.py#L148-L151
while generate_bed does.
https://github.com/FINNGEN/finemapping-pipeline/blob/b54125e79d0085d33632dcad93cf19849197f276/python/make_finemap_inputs.py#L196-L197
So, the most straightforward fix would be just replacing (df.chromosome == f.chrom) with (df.chromosome.map(CHROM_MAPPING_INT) == f.chrom) in L150.
Bigger question might be whether we want to keep integer chromosomes in a bed file... I don't remember why I forced so (maybe pybedtools restrictions?), but am generally fine to introduce a breaking change so that we can make it consistent with sumstats. In this case, I think the fix would be simply removing CHROM_MAPPING_INT mapping from L197.
@Lipastomies Do you mind investigating whether this change works and would affect any other subsequent pipelines that perhaps use .bed files?
Thanks!

I don't think we use them anywhere. The beds are not used later in the pipeline, and I'm not aware of any other pipelines using them.


Added command for hiding the phpunit window.

This is an example of phpunit output directed to a text buffer to support a vim-mode user's workflow. When running phpunit for the first time, the current pane is split down and output is placed in a new text buffer. That text buffer is reused for subsequent phpunit runs unless it's closed.
Currently, the original output window is commented out. If this looks like a useful addition, I'll clean up the code and make output to the window or to a text buffer configurable.
Other to-dos:

auto-save the text buffer output to a configurable filename so the buffer can be closed without confirmation.
add support for opening the file and jumping to the line of the failing test as is supported in the original output window

Yeah looks good to me :+1:  You mentioned a little clean up, I'll let you do that before I close the pull request.

Great will probably have it done this weekend. The current output window is completely commented out, so there's definitely some cleanup to do!

Ok, finally got around to finishing. Most of the time went to updating the tests. Since this is a plugin for working with unit tests, I felt that it should have its own tests working. Coffeescript nuances and learning Jasmine really slowed me down.

Looks good.


Add an accessibility testing checklist

What do people think of this as a first draft of things to check when changing the frontend UI?
As it says in the docs, this isn't supposed to be a comprehensive list of accessibility checks (that's the WCAG AA list) - it's a set of things which are particularly relevant to TDR, and which we'll want to catch at the implementation step.

@techncl I've added the A11y project link that you shared, and updated the ARIA links.

@Lholbourn Do you have any feedback on the checklist so far, based on the testing you've done recently?

@Lholbourn Do you have any feedback on the checklist so far, based on the testing you've done recently?

I'm finding the checklist really useful so far! Really straight forwards and easy to understand. That's reminded me I need to add the instructions for how to block the CSS loading within Chrome! I have been thinking should we state which type of screen reader/browser combination is preferred?


Prevent Amazon Bedrock models fine-tuning

Title: Prevent Amazon Bedrock models fine-tuning
What is this?
Change in SCP to prevent the creation of custom models jobs in Amazon Bedrock.
Why is this important?
Models fine-tuning could lead to expensive costs. Amazon Bedrock pricing needs some understanding of how AI/ML works. Until we get a broader experience using the service we should prevent the use of a potentially expensive feature such as model fine-tuning.
How to implement it?

[x]  add bedrock:CreateModelCustomizationJob in SpaDenyCreationOfCostlyResources
[x]  Merge the change into main branch
[x]  validate the behavior
[x]  change sample SCP provided with SPA
[x]  change workbook on preventive controls

behaviour validated after tests on Oct-24


Prisma generate option --allow-no-models doesn't work.

Bug description
npx prisma generate --allow-no-models doesn't work in v5.16. It works in v5.15
How to reproduce
Repository
https://github.com/MansurAliKoroglu/prisma-bug
Steps

Run npm i to install packages.
Run npx prisma generate --allow-no-models and see it doesn't work
Edit package.json file and change prisma packages to version 5.15.0
Run npm i to install packages.
Run npx prisma generate --allow-no-models and see it works.

Expected behavior
It should work in v5.16 too.
Prisma information
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "sqlite"
  url      = env("DATABASE_URL")
}

No code needed.
Environment & setup
OS: Kubuntu 22.04
Node: 20.15.0
Prisma Version
5.16

Hi @MansurAliKoroglu, thanks for informing us about this issue.
I've fixed this regression in https://github.com/prisma/prisma/pull/24741, and added more tests to avoid similar issues in the future.
This fix will be part of next week's stable release, Prisma 5.17.0.
Cheers 👋🏻


Fix typo of Authentication in navigation menu

I'm sorry to be "that kind of person" but somebody has to do it

@tajchert: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

I'm sorry to be "that kind of person"

Ahah no need to apologize, thanks a lot for catching and reporting it! 🙏

Oh man THANK YOU for being that kind of person :). I can't unsee it now. Thanks for fixing it!


DMN Editor Import Java Datatype disabled

I try to use a domain model from a maven dependency within a kjar generated by the archetype.
When I open the kjar project with DMN Editor plugin, navigate to the Data Types Tab, the Import Java classes Button is disabled.
I've created a project to reproduce the problem under MBoegers/kjar-with-domain-from-dependency
org.kie.version: 7.73.0.Final
plugin version: v0.21.0
Error from Developer Tools
dmn-loader.js:2 Error: Internal error.
    at e.callback (DmnEditorEnvelopeApp.js:2:517269)
    at e.receive (DmnEditorEnvelopeApp.js:2:519025)
    at Object.receive (DmnEditorEnvelopeApp.js:2:514342)
    at e.receive (DmnEditorEnvelopeApp.js:2:521554)
    at eventListener.eventListener (DmnEditorEnvelopeApp.js:2:520714)
(anonymous) @ dmn-loader.js:2
Promise.catch (async)
(anonymous) @ dmn-loader.js:2
Pl @ dmn-loader.js:2
t.unstable_runWithPriority @ dmn-loader.js:2
Yo @ dmn-loader.js:2
Bl @ dmn-loader.js:2
(anonymous) @ dmn-loader.js:2
R @ dmn-loader.js:2
w.port1.onmessage @ dmn-loader.js:2

Hi @MBoegers!
To enable the Import Java Classes button, you can refer to my blog post in kie blog
Please keep in mind this feature is designed to work with kogito (https://kogito.kie.org/), which is not your case (as you are using the 7.x kie version.
Despite the feature wasn't tested with the 7.x stream, it should work. Pay attention when using the @KieActivator annotation. That could require importing kie-api dependency in your project.
Let me know if face any issues.

Hi @yesamer the new popup provides better information, thanks for that improvement.
I can not make it work with 7.x nor with 8.x..
My process:

create project with ´mvn archetype:generate -DarchetypeGroupId=org.kie -DarchetypeArtifactId=kie-kjar-archetype -DarchetypeVersion=8.26.0.Beta´ (for 7.x I used archetype version 7.73.0.Final)
add Person class with public getter/setter to ´src/main/java/io/..´
add class with Annotation ´org.kie.api.project.KieActivator.KieActivator´ in same package as Person
run mvn package
add Person.dmn to src/main/resources

The Button stays disables, but with the new popup message. I've updated my reproducer, maybe you can see where I've missed something.
VSCode version: 1.70.2, Language Support for Java by Red Hat v1.9.0, Red Hat Business Automation Bundle v0.22..0

@MBoegers It was able to reproduce the issue, it's a bug on our side. I'll provide a fix in the same PR.

@MBoegers PR updated. That fix would be part of the next 0.23.0 release.

@MBoegers 0.23.0 version has been released. Can you please check and eventually close this issue? Thx!

@MBoegers feel free to reopen if the issue is not fixed!

Hi, sorry for not responding here, whole was family was attacked by flue.
I can verify that the button is enabled at every time and the dialog provides expected Java classes, at least for KIE 8 ;)
Thanks for the fix!


Browserify errors aren't verbose

When browserify has an error I don't get enough info. I get this:

I would like to get the same as with running browserify directly:

Oh, boy. That's actually not what I was thinking. I didn't even look at the output from direct browserify. It doesn't really help me figure out where the erroneous require call is.

I can't see where browserify is showing more information than the grunt plugin. I am closing this issue for now, feel free to reopen if there is still an issue for you.


Adding flag disablesubPathExpr

Thanks this looks good. Can you also update the gds / bloom license example? https://github.com/neo4j/helm-charts/tree/dev/examples/bloom-gds-license

Done


Create a hierarchy for builtin objects

There is currently a hierarchy for objects found in the user directory.
There should be a separate hierarchy for objects that are built into Talon.
Once such a hierarchy exists the cache_builtin command should emit such objects.

Object hierarchy was restructured to make a separate hierarchy for builtin objects unnecessary.


ReadProcessMemory

hello,Can you write an example of ReadProcessMemory? I don't know how to handle the garbled data returned by  final lpNumberOfBytesRead = calloc<IntPtr>();     final lpBuffer = wsalloc(byte);     SetLastError(0);     ReadProcessMemory(hProcess, Pointer.fromAddress(lpBaseAddress), lpBuffer,         byte, lpNumberOfBytesRead);

Unfortunately I don't have the bandwidth to document all the Win32 APIs.
But if you can show me a simple permissively licensed example written in C, ideally from the Microsoft docs, I'll see if I can convert it for you.

Unfortunately I don't have the bandwidth to document all the Win32 APIs.
But if you can show me a simple permissively licensed example written in C, ideally from the Microsoft docs, I'll see if I can convert it for you.

thx~

I'm interesting


User Preferences Feature

Idea

With new features coming out, it would be nice to allow users to have their own preferences

For example, if a user does not like using the @ feature then they can toggle it.

So, basically we can implement specific preferences and create general ones and label theme as themes to reduce the complexity of changing the settings
Another idea might be to display preferences so if they want to save their preferences to make it easy to switch between them
Preferences will be tied to users, to create some kind of thing to work with this, might just allow the server to spin up it's own database to store it's data.

When a conversation dies (or time passes), data entries can be removed from the database to reduce the load on the model that ollama is running

Possible Solution

Slash commands to set user preferences
Examples:

/set mention off (something like that, I am not sure how slash commands work yet!)
/set message-style embed (or normal)
/set stream true (or false as this is a very slow feature)
/set threads 8 (by default it will be 4)
etc ...

It would also be nice if we could do it in bulk, but that might not be very easy or user-friendly

Likely the resolution for this is to provide a generic slash command to set all of these

/set theme coding
/set theme general
/set theme images

Update

Refer to #12 for skeleton structure to create slash commands
This is the first step of however many (hopefully of 3 or 4) to allow users to have a preference

Concurrently running API and client with the npm run start command may result in the above issue with deleting and registering commands.

Since we've added Docker to the project, start has been removed, but it can be readded back into the project for instances where people might not want to run Docker for some reason... or we just make people run the application in docker... who knows for now. Better friendly than not.
Issue above still persists even on development run. It seems like the commands are registered then deleted immediately, even if embed is registered after the deletion... odd. I will try to do some investigating when I can.

A fix for this has been implemented and will be in the next release for user preferences!


FAQ: Redundant 'Bots or 'Bot Usefulness

Often that ivar (@ivarbot:hveem.no) & mijris (@mijris:chat.weho.st) are both in a room, raises FAQs.  But they offer useful functions.  When one is unreachable, the other performs.  And, each may have modules the other does not have, or they offer different information.  And other 'bots have different functions.  ¶Where the objection is «too many 'bots», users could mentallt ignore them, or use Matrix or IRC client functions to ignore them.

s/mentallt/mentally

Title: Too Many Bots


#622 Make deprecations and feature flag more explicit in compile output and addressing some boring warnings

Thanks for your pull request.  Please review the following guidelines.

[x] Title includes issue id.
[x] Description of the change added.
[x] Commits are squashed.
[x] Tests added.
[ ] Documentation added/updated.
[x] Also please review CONTRIBUTING.md.

I am not sure if 30.secondsor 30 seconds (with import scala.language.postfixOps) preferable.

I agree 30 seconds is elegant but 30.seconds is less difficult for the compiler to figure out
Postfix operator notation is unsafe and should not be used.
Scala style recommendation

Since the scala style guide recommendation is clear here, I would vote we move away from Postfix. I think for test code though postfix improves readability in some cases and I would prefer to optionally continue to use it there.

@sebady tests, by nature, are DLSs :)
the magic of DSL is built upon implicits, postfix and magnet pattern
Don't conclude that postfix is evil
It's only a matter of balance between readability and safety
IMHO 30.seconds is still very readable ;)

Sorry for chiming in late. Agree with 30.seconds.

@akara updated and rebased


Use email keyboard type for email field when logging in

The email keyboard provides easier access to characters like @ and .com

Can I work on this?

Sure. If you have questions, don't hesitate to ask.

👍 This is a great starter task


BABYlONJS

In the context of 3D layout design, there is a method available for the TransformNode class that allows you to hide all meshes contained within it. Specifically, you can use the "setEnabled" method to disable rendering of the TransformNode and all its child meshes. This effectively hides the meshes from view within the 3D layout.

is there any command that can hide the TransformNode in this module?

Hi @dheeju143,
I haven't been using this node myself for a long time, but it seems from my code that I already have implemented something similar for lights in the past.
So it should be possible to do something similar for meshes.  I have quickly created an "enable-mesh" branch in this repository, that implements your feature (see here).   I have very few time lately, so would be nice if you could test it and let me know whether it works.

You can install my branch from Github - when you have git installed - via this command (from within your .node-red directory):npm install bartbutenaers/node-red-contrib-ui-babylonjs-3d#enable-mesh

Restart Node-RED.
Refresh your browser screen.
 Thank you for your valuable time bartbutenaers.

please view the folder structure was there in png and name "shop_center_4_0"
class: mesh has been updated and works well but I am actually looking for the class:- TransformNode to disable it. It was actually like a group of meshes that you can disable with TransformNode. I tried to use update_TransformNode as a command but that was not implemented i think so.

@dheeju143,
If you reinstall my branch again, you should have some basic support for tranform nodes...
You can find an example flow on the readme page (see new Transform nodes section).
When you create a mesh, you can specify the parent transform node in the input message:

If you haven't created the transform node in advance, you will get obvious this error:

Here you can see the result of the demo flow:

Note that the transform node can currently only be applied as a parent to meshes.  I thought that BabylonJs also supports it for some lights, but I have no time to read about that.  So if you want it, it would be nice if you could look it up and summarize in this discussion where it can be used.
As soon as you have tested this (and it works fine), I will merge it into the main branch afterwards...

getting error for create,update,remove transformnode
BabylonJS supports creating, updating, and removing TransformNodes, even when they are parented to other TransformNodes. For example, you can create a TransformNode that bundles a complete mesh under its group, which allows for fast processing.
The TransformNode class, which includes the "isEnabled" attribute, can be used to represent the floor-to-floor mapping structure in a building. This allows you to easily visualize and manipulate the layout of the building at different levels.
PFA VIDEO how was can use transformnode
https://user-images.githubusercontent.com/86141162/219623586-8c3a35b5-6475-49a3-a2f0-958b1ee8d160.mp4

Ah damn, I had updated the readme page but seems I have forgotten to push my changes...
It is now available in the enable-mesh branch.
You can also use an update_mesh message with a parent property, to link meshes to a transform node afterwards.
But I am not going to implement the entire BabylonJs interface.  This ui node is made for the old Node-RED AngularJs dashboard is end of life for already quite some time.   When I have free time, I will focus on migrating to Flexdash.  That is the reason why I am not going to spend too much of my (very limited) free time anymore in these old ui nodes.
But of course pull requests are always welcome.

Thanks I will look into it

@bartbutenaers I look into the 3D on which is sounds good!!
i would like you ask more adding on features on below would be great.

X,Y,Z axis view representation in 3d drawing.
do we have any annotations where we can click on object to and view zoom position or get the customize text to view it.

I look into the 3D on which is sounds good!!

Do you mean that you have tested my changes, and it is working correctly so I can publish it on npm?

X,Y,Z axis view representation in 3d drawing.

Do you mean something different compared to the current axes viewer possibilities?

do we have any annotations where we can click on object to and view zoom position or get the customize text to view it.

I see here in my code that I have added at the time being a gui control for text.  Which means for the command create_gui_control and payload.type textBlock and some text inside payload.defaultValue.  But of course this is static text.  Don't see anything like you want at first sight.  But perhaps you can find something that we can use...

@bartbutenaers  Yeah I will find out and let you know if I can have anything of annotations

@bartbutenaers  Yes the updated code is working fine you can publish it on npm

@bartbutenaers  any sample flow is there for GUI control for text block  would be great!

Thanks for testing!  I had forgotten that this node is not available on npm, so I have simply merged the "enable-mesh" branch into the master branch.
About the GUI control for text block.  I saw this evening that there were some bugs and some required features were missing.  So I have added them on a new "textblock-extension" branch.  However it doesn't work for some reason.  And unfortunately I don't have enough free time to digg further into this.  The problem is that the linkWithMesh does not work: I link the textbox to a specified mesh, and then it should appear on top of that mesh (and also track the mesh if it moves).  However it doesn't appear on top of the mesh.  I don't know why...
This was my test flow that I created tonight for this new branch:
[{"id":"a12627a6.c0b228","type":"ui_babylon_js","z":"6de43ccfaf136eb5","name":"BabylonJs widgets","group":"ac886c83.c74f1","order":2,"width":"12","height":"9","folder":"","filename":"","outputField":"payload","actions":[],"showBrowserErrors":true,"startupCommands":"[{\"command\":\"create_camera\",\"type\":\"arcRotate\",\"name\":\"my_arcRotate_cam\",\"position\":{\"x\":100,\"y\":100,\"z\":100},\"targetPosition\":{\"x\":0,\"y\":0,\"z\":0},\"active\":true},{\"command\":\"create_mesh\",\"type\":\"box\",\"name\":\"my_box\",\"meshOptions\":{\"width\":30,\"height\":30,\"depth\":30},\"position\":{\"x\":0,\"y\":0,\"z\":0}}]","_mcu":{"mcu":false},"x":890,"y":260,"wires":[[]]},{"id":"f878b6fb.09a598","type":"inject","z":"6de43ccfaf136eb5","name":"Create text block","props":[{"p":"payload"}],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","payload":"{\"command\":\"create_gui_control\",\"type\":\"textBlock\",\"name\":\"my_textblock\",\"defaultValue\":\"hello babylonjs\",\"height\":\"30\",\"width\":\"120\",\"targetName\":\"my_box\",\"targetOffsetX\":10,\"targetOffsetY\":50,\"horizontalAlignment\":\"right\",\"resizeToFit\":false,\"textHorizontalAlignment\":\"center\",\"textVerticalAlignment\":\"center\",\"outlineWidth\":2,\"outlineColor\":\"red\",\"color\":\"white\",\"fontSize\":30}","payloadType":"json","_mcu":{"mcu":false},"x":660,"y":260,"wires":[["a12627a6.c0b228"]]},{"id":"ac886c83.c74f1","type":"ui_group","name":"Gui control demo","tab":"956f2124.271f6","order":2,"disp":true,"width":"12","collapse":false},{"id":"956f2124.271f6","type":"ui_tab","name":"BabylonJs Widgets","icon":"dashboard","disabled":false,"hidden":false}]

ok Nice. meanwhile i will also check the Flow and test it.


Enable Sound Position Indication not working[BUG]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Enable enablesoundpositionindication
set agoraAudioEffects.SetRemoteVoicePosition

Expected behavior
Spatical Audio should work
Desktop (please complete the following information):

OS: W10
Browser: Edge

Additional context
It seems enablesoundpositionindication is not available on webSDK but it's available on react native, is there is any way to implement the same on webSDK as well? Or I'm missing something?

No, the current implementation of Spatial Audio is not supported in WebGL.  It is is indicated in the README Wiki.
Please wait for future releases.

hi @icywind , since webGL version is using webSDK, can i implement that by my own by adding some jslib and updated agora libs?

Yes of course. @ragavendranbala , however, the feature is WIP right now.  You will expect to see it in two weeks.


d3.conf.example from pkg installer is just ruby-jss.conf

what's installed does not match https://raw.githubusercontent.com/wiki/PixarAnimationStudios/depot3/d3.conf.example

additionally, nothing is present in /Library/Ruby/Gems/2.0.0/gems/depot3-3.0.15/data/


It returns DateTime or Timestamp values in a wrong format

The problem is olingo does not return the date/time or timestamp columns in a correct format as defined in Odata standard (or maybe Microsoft standard). I mean my front-end library (i.e. OpenUI5) expects some value like <d:ValidFrom>/Date(1579075200000)/</d:ValidFrom> for Edm.DateTime column but what olingo returns is <d:ValidFrom>2020-01-15T08:00:00</d:ValidFrom> which is not accepted by OpenUI5!
What is your suggestion?
1- Is there a way that I force Olingo to return data/time or timestamp in the format that Microsoft defined or not? (i.e. /Date(timestamp)/)
2- Is there a way that OpenUI5 accept the date\time in a format like yyyy-MM-ddTHH:mm:ss ?

In XML it returns this type, In $format=json it will returns timestamp


[Feature Request] Display mail_updated time in local time

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Currently the sensor.mail_updated displays the time in UTC. Is it possible to have the time displayed in the local timezone instead?

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

No this is how the timestamps in Home Assistant work.
What are you attempting to do that this is causing an issue?

I know that is how it stores timestamps.
However, it is displayed in UTC in the frontend as well.

This is how it displays in a entity card:

Is that's the Mail and Packages card you're using?

Apologies. I didn't realise I was not using the Mail and Packages card. All good now.


No GBR on WS Chat for approver after Take Control + Approve/Forward from admin in OldDot

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: N/A
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL: N/A
Issue reported by: @saracouto
Slack conversation (hyperlinked to channel name): #expense
Action Performed:

Create an OldDot workspace, invite Submitter + Approver A, enable Advanced Approvals, set approval asl Submitter > Approver A
As submitter, create expense in your WS chat and submit to Approver A
As admin, open the report and take control
As admin, approve/forward back to Approver A in OldDot

Expected Result:
Approver A should see GBR on Submitters WS chat for the report
Actual Result:
Approver A doesn't see GBR on Submitters WS chat for the report
Workaround:
Approver A can go find the report in WS chat, but only if they know it's waiting on them
Platforms:
All
Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub

Looks like this one has been unassigned for a while, so I will work on re-triaging this to confirm it still exists!

Not overdue

I just triaged this and we do seem to show the approver the GBR on the submitters workspaces chat now!
I submitted a report from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Then, as<EMAIL_ADDRESS>I took over the report in OldDot and approved and forwarded it back to<EMAIL_ADDRESS>
I then signed into<EMAIL_ADDRESS>and see the GBR in the LHN of the workspace chat to action the approval.


Deploy terraform code in relation to #16235 in production

DevSecOps Issue
Summary
Terraform needs to be updated to include the changes found in #16235: Add Environment variable "JAVA_OPTS" with value ""-Dfile.encoding=UTF-8" to all env terraform files.
Some background from James Gilmore:
#16235 makes the deployed environments match the local one with respect to the default encoding so that any characters after decimal 127 actually display. The issue was that I believe the prior devops teams was manually applying terraform changes somehow but I'm not clear on how. I added the env variable to the staging env through the azure portal for testing but AFAIK it has not been applied to production.
The update to the .tf files was deployed to prod with the 10/17 deployment (#16251), but that doesn't appear to have updated the terraform in prod to include the env variable.
Environment

[ ] Local
[ ] Dev
[ ] Stage
[x] Prod

Priority

[ ] Critical - affecting prod systems
[x] Major - blocking major functionality, deployment, etc

This is blocking the ETOR Newborn Screening project from moving to production with their CA pilot, which is slated for Nov 18.

[ ] Minor - improvements, bug fixes
[ ] Nice-to-have - feature request

Blocks the following

List any blockers

None known

Link any Github relevant issues

#16235

Blocked by the following

Link to any tickets that add context, etc

Contact
Team:
Slack Team Channel:
Slack Contact (Full Name):
Technical Team Lead:
How to Reproduce
Screenshots, links, etc. for context
How to Test
Definition of Done

[ ] Add environment variable "JAVA_OPTS" with value ""-Dfile.encoding=UTF-8" to all env terraform files in production (a previously manual process by DevOps)

Context Links
Git Repo:
Relevant Code Links:
Azure:
Site URL:
Other:
Notes

Duplicate of Arnej's ticket #16528 .
Closing this one.


Tag vocabulary

These changes create a new endpoint, /api/tags-vocabulary analogous to the existing endpoint /api/dependencies-vocabulary. Because there is no file like dependencies.txt in the index, this is achieved by looking at the Fields of the IndexReader and picking out the Terms of the Field with the same name as the config for posTagTokenField. These are returned (in Lucene's orthographic UTF-8 order) to the user as a JSON array of strings.
Tests for this new endpoint are included, as well as tests for the /api/dependencies-vocabulary endpoint and the /api/corpus endpoint.

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

@danebell , I'm wondering if this information should be included in the JSON file introduced in #255 .  Thoughts?

@danebell , I'm wondering if this information should be included in the JSON file introduced in #255 . Thoughts?

depending on size of the list, makes a lot of sense to me, fwiw, else it could also go into a separate file like tags.txt or something.  If these types of files seem like they will proliferate (which we should try to semi-prevent prob), we can put then in an index_info subdir (not deps prob, bc it's directly used by the index, but others).

I'd be even happiest if we could have a complete list of possible tags from the annotator of choice. That way it'd be complete and invariant even when new documents are added. Whether we keep it in the JSON or in a separate tags.txt file doesn't make much of a difference to me -- I think there are few enough tags to fit comfortably in the JSON. More relevant than size is that there might be additional discrete collections users might want, and we should have a policy that works for all of them.
Can we try to enumerate all the things we might want to store this way? Beyond dependencies and tags, I can think of NER labels (e.g. B-ORG) , chunk labels (e.g. B-NP), and semantic role labels (e.g. A0). Anything else come to mind? Is this an open-ended list? I can conceive of someone wanting spell check or similar for raw/words/lemma, but obviously that would be much more cumbersome.

@danebell, this may be completely off the mark, but you may want to check the file TagSet.scala in recent versions of processors for some of them.

I like where this is going. I think that a full list of potential tags is great, and I like the idea that it would be invariant, but I don't know that that is possible (see below).
I'd also like info on what annotator (type and version) was used, so we still know after the fact.
we could add something to the config in the index section that specifies which fields get their vocabs listed out, and the indexwriter builds a vocab for them?
That said, this solution would not be based on the annotator, instead it would be based on the corpus.... i think that with all the different options for parsers out there we likely won't be able to have the corpus-invariant version...
What else... I still plan to add the size of the index (in sentences if not docs)... thoughts?

@danebell, this may be completely off the mark, but you may want to check the file TagSet.scala in recent versions of processors for some of them.

That's helpful, thanks. I'd prefer it if we could get it from the annotator itself upon creation, so that if Processors (or some equivalent) takes on new languages or syntactic models, the process of collecting them would be the same.

I'd also like info on what annotator (type and version) was used, so we still know after the fact.
Good idea. I think that'd be helpful. And it could be added to the /api/buildinfo endpoint, which currently returns

{
  "builtAt": "2021-03-23T14:58:14+0000",
  "name": "odinson-core",
  "gitUncommittedChanges": "true",
  "gitHeadCommit": "27dc4a50f08b4d853aaf44bc1a76d039ac513788",
  "gitCurrentBranch": "singleEventExtractor",
  "scalaVersion": "2.12.10",
  "version": "0.3.2-SNAPSHOT",
  "sbtVersion": "1.2.6",
  "gitHeadCommitDate": "2021-03-22T15:17:00-0400"
}

we could add something to the config in the index section that specifies which fields get their vocabs listed out, and the indexwriter builds a vocab for them?
Very good idea! I like that solution a lot. Much like the setting for which fields to save.

That said, this solution would not be based on the annotator, instead it would be based on the corpus.... i think that with all the different options for parsers out there we likely won't be able to have the corpus-invariant version...
:-( That makes sense, but it's disappointing. Anyway, a corpus-level solution is still pretty good.

What else... I still plan to add the size of the index (in sentences if not docs)... thoughts?
I guess the index size will have to be updated as new docs are added, right? That information seems useful for a human to have readily available. No other ideas right now about what else to add.

we likely won't be able to have the corpus-invariant version...

I don't think we should even try for that.

I guess the index size will have to be updated as new docs are added, right?

For what it's worth, we already have an endpoint that computes the sentence count on-the-fly.

@danebell , can you address the conflicts in OdinsonController?

we likely won't be able to have the corpus-invariant version...

I don't think we should even try for that.

Can you say more about that? My motivation is that if you're correcting the POS tags on a small corpus, you might conceivably not have all the tags in a given scheme (or semantic roles or entities or whatever) extant in the index. I don't mean that the list should be invariant from corpus to corpus, but rather that adding additional documents should ideally not change the range of values allowed for tags (or whatever field).

So if I can briefly summarize my understanding, the OdinsonIndexWriter currently writes a copy of the Vocabulary upon close. What's being proposed is that what gets written in addition (e.g. tags, chunks, etc.) to this should be specified in the core config. (Incremental indexing will soon mean that we'll need to overwrite these collections by manually deleting the existing files and writing them as if new upon every close call.)  Once this is accomplished, the tags-vocabulary endpoint can stay in place but be implemented as merely recalling the contents of these file. Maybe we should consolidate these into a single vocabulary endpoint to which one passes a string specifying the vocabulary (including dependency) desired.

per meeting: caching can happen in diff places, but look at how to cache an entrypoint/function through scala play

Okay, @myedibleenso & @BeckySharp, now caching is ready for the tags-vocabulary endpoint, and the timeout is configurable. I set it to 1 hour by default.

  Thanks for adding them.

Looks like some backend tests are now failing after the change of formats to some of the fields used in the controller.


[Bug]: the spread source model are not projected

Describe the bug
for type's sourceModels of spread, it seems the type is not projected when compiling with a versioned service.
model with projection:

source model is not projected:

Reproduction
playground
Checklist

[X] Follow our Code of Conduct
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Feature/recompile gil stylesheet

saw syntax error in gil less, which caused some styling issues
re-compile less fixed it

@pbugni
Paul, I have rebased this branch.  Can be merged upon review.


Warning messages of the maven plugin are not providing enough details to enable efficient investigation

When applying a scan for licenses using the maven LicenseScout plugin by starting of  with the sample configuration a typical maven output contains numerous warning and info messages like:

[WARNING] No license text for license ID: 'NLIL'

Comment: NLIL seems to be a place holder license identifier for "no license information found" in  licenses.xml#L6235

[WARNING] Cannot find provider with ID: ''

[WARNING] License not found by URL: https://www.eclipse.org/legal/epl-2.0/, http://www.gnu.org/copyleft/gpl.html, http://www.gnu.org/licenses/lgpl.html

[WARNING] Neither license name nor license URL mapping found for name/URL: Eclipse Public License - v 2.0 / https://www.eclipse.org/legal/epl-2.0/

[WARNING] Parameter buildUrl not configured
[WARNING] Parameter licenseReportHtmlUrl not configured
[WARNING] Parameter licenseReportCsvUrl not configured
[WARNING] Parameter licenseReportTxtUrl not configured

[INFO] No license mapping found for URL: http://www.jooq.org/inc/LICENSE.txt

Bug report
Expected Behavior - What where you expecting to happen?

Warning and Info Levels should be used consistently (see above example 1-5 vs. 6)
Warnings should be classified into

Problems in configuration of the project to be scanned by LicenseScout (missing properties, configurations in the pom.xml)
Problems or inconsistencies in existing configuration files (e.g. NLIL license text not configured)
Missing entries into specific license configuration files, e.g. name or URL mappgins
(including the file name + maven coordinates configured for the specific license scan

Warnings should always refer to the location in the analysed project (dependencies)

Current Behavior - What happens?
Warnings are generated, but in order to identify the next steps to clarify the problems the source code of the plugin or even java debugging of the maven build process is required to identify the affected project and license scout artifacts.
Possible Solution

Check for Log Level to be used: Info/Warning/Error
Check for common Warning format: e.g.
 : ; Configuration File

Thanks for the report @rmlekus - as this is just an issue with the logging and not with the core functionality of the plug-in, I'd flag the issue as a feature request. Anyway, it is a good point and we will work on standardize the logging that LiceseScout outputs. Of course, PR are welcome for this issue if you have a solution in place!

What we really need is a use case based documentation, to explain (not only) the mysteries of URL mapping. First point is of course that for a specific project a specific configuratin is required that needs to be maintained by that project. However, for instance 'https://www.eclipse.org/legal/epl-2.0/' can be safely mapped to a license identifier, while 'http://www.gnu.org/licenses/lgpl.html' can not, because the link does not include a version number and there are multiple versions of the LGPL.
Mapping 'http://www.jooq.org/inc/LICENSE.txt' to a dedicated license is risky, because the project may change its licence by changing the contents of the file the link points to, and then the mapping becomes wrong.
These are the issues that should be explained in such an extended documentation.

Another note: to research why licenses are not detected, it is also possible to run the LicenseScout with logging set to level DEBUG. This will give extensive information on what is going on during the license scan. You will get output like this:
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/org/apache/velocity/shaded/commons/io/IOCase.class
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/LICENSE.txt
[DEBUG] Checking file for licenses: META-INF/LICENSE.txt
[DEBUG] Result licenses: [Apache License]
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/NOTICE.txt
[DEBUG] Checking file for licenses: META-INF/NOTICE.txt
[DEBUG] Result licenses: []
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/pom.xml
[DEBUG] Checking POM file: /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/pom.xml
[DEBUG] Checking POM file: /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/pom.xml
[DEBUG] Checking POM file: /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/pom.xml
[DEBUG] License name: Apache License, Version 2.0
[DEBUG] License URL: http://www.apache.org/licenses/LICENSE-2.0.txt
[DEBUG] parsePackedJarArchive(): processing /velocity-engine-core-2.0.jar!/META-INF/maven/commons-io/commons-io/pom.properties


Add error code for DEV-19 not supported by Manager

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

If the Manager does not support the DEV-19 message, it should return the following in the Commit level acknowledgment:
MSA-1 = CR (Enhanced mode Commit Reject)
ERR-3 (HL7 Error Code) = 200 (Unsupported Message Type)
ERR-4 (Severity) = E

Auto-close didn't work as a # was missing before the issue number. Closing manually.


Temperature sensor hasn't been implemented yet

@hendrikE can you please write down the ideas you had over the weekend here?

Temperature sensor stuff has now been uploaded to the temperature branch. But isn't functional yet.
A quick overview of how I think this was going to work:
Use generate_heatmap.py to generate an image representing temperatures across the map.
A thermometer (Assets/Thermometer.png) would then be filled up to represent the temperature.

An update to this. Got a basic working sensor on the temperature branch (video here) Still needs integrating into the rest of the GUI

Integrated into master with #34


Document "failsafe" behavior for multirotor and fixed wing

cases to cover:

loss of RC

with GPS
without GPS

"KILL" switch activation

It seems this needs a state diagram created by code inspection.

The intent is that this is documented only at the user level by QGC. Any amibiguities in the UI are issues against QGC.


vbuild can't find babel when installed globally using yarn

If I start with a new node install, 7.1.0 for example:
nvm install node
Install yarn first:
npm -g install yarn
and install vbuild using yarn:
yarn global add vbuild
now inside a project, try to run using:
vbuild --dev
ERROR in multi client
Module not found: Error: Can't resolve 'babel' in '/Users/neves/Documents/git/paliari-e-city-nfe'
 @ multi client

ERROR in (webpack)-hot-middleware/client-overlay.js
Module not found: Error: Can't resolve 'ansi-html' in '/Users/neves/.config/yarn/global/node_modules/webpack-hot-middleware'
 @ (webpack)-hot-middleware/client-overlay.js 25:15-35
 @ (webpack)-hot-middleware/client.js?reload=true
 @ multi client

ERROR in (webpack)-hot-middleware/client-overlay.js
Module not found: Error: Can't resolve 'html-entities' in '/Users/neves/.config/yarn/global/node_modules/webpack-hot-middleware'
 @ (webpack)-hot-middleware/client-overlay.js 40:15-39
 @ (webpack)-hot-middleware/client.js?reload=true
 @ multi client

Now if I install vbuild using npm, it works:
npm -g install vbuild
vbuild --dev

Really weird, I try to reproduce this, but ran into another issue:
nvm install 7.1.0
yarn global add vbuild
vbuild
-> fish: Unknown command 'vbuild'
yarn global remove vbuild
yarn global add vbuild
-> now it works

For Can't resolve 'babel' I haven't encountered this problem 😅

Did you try with a brand new node installation?

Yep, I did nvm install 7.1.0

Do you know docker? Here is a reproducible error using docker for isolated env: (works fine if installed using npm -g i vbuild)
docker pull node
docker run -it --rm node /bin/bash
# inside docker bash
npm -g i yarn
yarn add vbuild
vbuild init todo
cd todo
npm run dev

ERROR in multi client
Module not found: Error: Can't resolve 'babel' in '/todo'
@ multi client
ERROR in (webpack)-hot-middleware/client.js?reload=true
Module not found: Error: Can't resolve 'strip-ansi' in '/root/.config/yarn/global/node_modules/webpack-hot-middleware'
@ (webpack)-hot-middleware/client.js?reload=true 95:14-35
@ multi client
ERROR in (webpack)-hot-middleware/client-overlay.js
Module not found: Error: Can't resolve 'ansi-html' in '/root/.config/yarn/global/node_modules/webpack-hot-middleware'
@ (webpack)-hot-middleware/client-overlay.js 25:15-35
@ (webpack)-hot-middleware/client.js?reload=true
@ multi client
ERROR in (webpack)-hot-middleware/client-overlay.js
Module not found: Error: Can't resolve 'html-entities' in '/root/.config/yarn/global/node_modules/webpack-hot-middleware'
@ (webpack)-hot-middleware/client-overlay.js 40:15-39
@ (webpack)-hot-middleware/client.js?reload=true
@ multi client

Looks like https://github.com/yarnpkg/yarn/pull/1679/files#diff-a7748843fbda5167d065231e3a2e7a1fR39 is the reason, yarn changed its directory for globally installed modules.

Fixed in v3.2.3

I used yarn global add vbuild today and nothing installed. npm i -g vbuild worked fine.

@jazoom maybe you should update yarn?

Oops, sorry. I thought your post above was saying it was fixed in vbuild. I didn't notice that was the version for yarn.


[modify] cms layout and part : root_owned true

Coverage decreased (-11.1%) to 49.153% when pulling 579e7cfaef5ffd71d2fa2d5ae55fd91f1dd4cec4 on fix/cms-root-contents into 5a435fc14bef418444af57e0c6fccdb58909894c on master.


Add stateful uploads

This changes the packfile endpoints to accept batches of files, staging them until they're explicitly completed. Dodges various limitations in the server leaking file descriptors, and the browser saying no to exceedingly large uploads.
Accomplished with minimal drama by implementing an intermediary placers which is very simple, and modifying the existing Packfile placer to consume the result. Does not implement orphan ticket cleanup.

Fails CI:
        r = api_as_admin.post('/upload/label', files=data.files)
>       assert r.status_code == 400

E       assert 500 == 400

I'm going to claim that fixing this is blocked by #66 and that we should modify the test with a comment.

LGTM with debug prints removed and the CI test passing, but I'd like to have @rentzso take a look at that integ test failure as these are new tests he wrote as part of yesterday's PR.

Agreed, I'm still not sure how to run unit tests locally and Travis doesn't show the API logs.

to run the tests locally:
0) stop fly/fly

create a python virtual env
activate the env and run core/bin/install.sh (edited)
connect to the default docker env
run the script core/bin/runtests.sh integration in daemonized mode
the script creates two docker containers
to get the full logs you can run:
docker logs <uwsgi_container_id>

@ryansanford should be able to give you some help if something is not working

I'd throw the cleanup under /packfile as well

There should be docs as part of this PR that it is up to the client to call the clean-packfiles endpoint when using the packfile upload feature.

@kofalt Note the use of uuid.uuid4() when creating download tickets.

LGTM, with comments.


createDirectory not working for Samsung devices

The Following code doesn't create sampleFolder for Samsung devices
UsbFile dataFile = root.createDirectory("Data"); UsbFile sample=dataFile.createDirectory("sampleFolder");

I have tested the application on Sony Xperia Device it works fine but it doesn't work on Samsung devices.
@magnusja Kindly let me know if I am doing something wrong.

Hey there,
issue #33 already discusses this problem with Samsung devices. I really don't know what is happening there. Can you tell which Samsung devices and which version of Android are affected?

I have tested it on Samsung Note 4(Android 6.0.1), Samsung J7 prime(Android 6.0.1) and Samsung Tab 2(probably android 6.0.1). Even USB light blink. I have also debugged the code it shows that FatDirectory has correct entries but there is no folder created on USB. And one important thing is when i create directory from root directory object it works but when I create directory from any sub folder of root it doesn't work.

@magnusja I have also read the all comments on issue #33. Thanks for sharing. Yes I think my issue is same. So now what do you think? How we can fix this issue?

I don't know. I guess I need to get one of these Samsung devices. Can you tell me the cheapest option on where this issue occurs?

I have tested only on these three devices. There are also cheaper devices than these where this issue may produce. I just searched craigslist you can find the same model i am using in 150$. i.e Samsung Note 4. This link might help you https://sfbay.craigslist.org/sby/mob/6133525277.html

I have exactly the same problem. I'm testing on a tablet Samsung Galaxy Tab S2, model SM-T719 and a phone Galaxy S6, model SM-G920F. Both have Android 7.

Hey there,
sorry to hear that. I have very limited time at the moment and still did not buy a Samsung device. So I cannot give an estimate when this will be fixed.
In the meanwhile, I would suggest trying to debug the issue yourself.

It's not urgent enough to do this during my work hours and unfortunately I don't have much time after work for this at this moment. I just wanted to check status, no worries.


dropdown - autofocus first element incompatible with mouse over

http://boosted.orange.com/components/#btn-dropdowns
Auto-focus first list element for keyboard user is good, but this selected element need to be update when user is browsing with mouse.
2 way for fixing : best way is to have hover/focus state (use the same color as buttons), low cost solution is to manage selected element both with mouse over and keyboard selection

fixed by commit https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/commit/abf7166e301078b3ddd6cbbafec720549388ef50


Add pageview-props extension to script

I just added my custom pageview property, so the script tag looks like this:
<script defer="" data-domain="xxx.me" data-api="https://plausible.io/api/event" event-creator="6494126149978adbb4433382" src="https://plausible.io/js/plausible.outbound-links.file-downloads.tagged-events.hash.js?ver=1.3.4" id="plausible"></script>
Even with every option enabled, it seems the required pageview-props script extension is not added. Related documentation: https://plausible.io/docs/custom-props/for-pageviews
I tried finding the part where the WordPress settings are added, and the final script name is build, without success :D
I will happily provide a PR if you could send me in the right direction.

I found it and made it working 🎉.
Just for completeness, I will add my example script to add any acf field as pageview prop:
// Add ACF Field as prop to pageviews
add_filter('plausible_analytics_script_params', function($params) {

	// Get field value
	$value = get_field('awesomefield');

	// Exit early if empty
	if (empty($value)) {
		$params;
	}

	// Add prop to script tag
	$params .= " event-awesomefield='$value'";
	return $params;
});

This feature (and your PR) will be added in 2.0.0, which is the current release we're working on.
Thanks for your contribution!


Download file fails if spaces in names are exist

Hello All,
We found out files with spaces in names cannot be downloaded or deleted due to one-directional UrlEncode on clients without of UrlDecode on servlets. So symbols ' ' are being translated to '+' and file cannpt be found.
I have two solutions:

Add UrlDecode to FileDownloadServlet and FileDeleteServlet. It works while we don't try to download file with symbol '+' in name. It's always translated back to ' ' during decoding, so file name becames incorrect again.
Use java.util.Base64.getUrlEncoder. It works always correctly but URLs are going to be not human-readable.

I have both solutions working locally, so can make Pull request for any of them.

Hi, @myznikov , you can make PR and I will merge it :)
I don't use this project atm and don't have time to improve it.

I was not sure what of two alternative soulutions should be pushed.
If you do not want to make a decision too, I prefer to use base64 encoding then.


Do not add IHttpContextAccessor to DI by default

Issue: https://github.com/aspnet/HttpAbstractions/issues/473
Depends on: https://github.com/aspnet/HttpAbstractions/pull/475
@DamianEdwards @Tratcher @davidfowl @rynowak

Hi @pakrym, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Why is the ci failing?

@davidfowl  https://github.com/aspnet/HttpAbstractions/pull/475 not in yet

:up: :date: HttpAbstraction PR was merged

:shipit:


Refactor API to be more pythonic

Is there any scope to refactor the API/ReadMe so that its use is more pythonic, for example currently it is tricky to read / understand what and where it is doing the processing:
for file in bfd.Files(paths).iter():
    market_count += 1
    for market in file:
        update_count += 1

Small change to declare the files first before iterating:
files = bfd.Files(paths)

for file in files.iter():
    market_count += 1
    for market in file:
        update_count += 1

This could be improved further by utilising python's builtins __iter__ removing the requirement to call iter entirely.
files = bfd.Files(paths)

for file in files:
    market_count += 1
    for market in file:
        update_count += 1

bflw processing can stay the same
files = bfd.Files(paths)

for file in files.bflw():
    market_count += 1
    for market in file:
        update_count += 1

Is there any reason why the mutable flag isn't available at the Files level?

Hey, I've actually gone backwards and forwards on this a few times already and I've never been really happy with it.
The challenge is exposing different options between the different outputs. Ie, no mutable implementation for bflw, and soon stream_id on bflw.
What I'm currently leaning towards is just exposing a different Files constructor under the bflw package, so we could have
files = bfd.Files(paths, mutable=True)

# or for bflw

files = bflw.Files(paths)

Then i can just put the implementation specific parameters on their own Files constructor, and completely integrate the iterator onto that object and everything gets a bit simpler. The downside is it could be weird to have both a bfd.Files object and a bflw.Files object in one codebase, but I doubt this would actually be a common occurrence outside testing scripts

Yeah I think that is fine, I see this library becoming a 'core' library with (hopefully) lots sitting on top which will do their own abstraction as required.
Maybe it's the naming of Files that I find confusing, as it's a bit more than that.

Closed in #6


1.4.0.Beta1 announcement

Thanks a lot! Added one sentence as a teaser for the upcoming tracing blog post. Btw. we also got staging support for the website now, @Naros (not sure whether you've seen it yet; details in the README).


FOOTER: Add Link to Whitepaper

Please add "ThreeFold Whitepaper"
under the Developers column
http://whitepaper.threefold.me/

done at staging