Service invocation example using Java  and HTTP

Signed-off-by: Pravin Pushkar<EMAIL_ADDRESS>Description
Service invocation example using Java and HTTP client
Issue reference
We strive to have all PRs being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: NA
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] The quickstart code compiles correctly
[x] You've tested new builds of the quickstart if you changed quickstart code
[x] You've updated the quickstart's README if necessary
[x] If you have changed the steps for a quickstart be sure that you have updated the automated validation accordingly. All of our quickstarts have annotations that allow them to be executed automatically as code. For more information see mechanical-markdown. For user guide with examples see Examples.

Also, please rebase your branch with master.


The Tasks section in the profile page has a blue “next” button even if there are no more tasks to see.

I would like to work on this issue!


Highlight word on seed phrase keyboard

Make autosuggestion background wider and more noticeable colour, suggested word bigger and medium font https://zpl.io/V4omEJX
Add animation of appearing autosuggestion background and word from keyboard.
Do not show autosuggestion background if there's no suggested word.

checked


Provide ES Module distribution

It would be very helpful if this library used ES Modules in the source code, or an ES Module distribution were provided, so that the code could run in the browser without bundling. Not everyone uses webpack or rollup or other bundlers, and requiring these tools to run the code is not ideal for all users. The code should run in the browser with as little modification as possible, which it does not do currently.

It would also be helpful to not rely on non-standard import functionality like importing json with ES Modules. Bundlers will take care of this, but I do not use the bundlers and the code should be able to run with as little modification as possible. For example, sdk.ts imports WalletContract as json. This is non-standard JavaScript and is not implemented by any ES Module implementation as far as I know


Protect from peers without supported transports

What
Trying to run compliance tests from the kad router module. I've tried to port the new swarm API but forgot to add a transport. The tests ended up blowing up instead of failing gracefully.
How to test
peerOne = new Peer(Id.create(), [multiaddr('/ip4/<IP_ADDRESS>/tcp/8090')])
peerTwo = new Peer(Id.create(), [multiaddr('/ip4/<IP_ADDRESS>/tcp/8091')])
swarm = new Swarm(peerOne)
swarm.dial(peerTwo, {}, function (err) {
  console.log(err);
});

This just work and display the error.

Nice catch! :D
(for some weird reason, github didn't show this two PR you did on swarm in the last couple of days, sorry for have kept you waiting!)


client-side dart:io library

This issue was originally filed by @seaneagan

It would be nice to be able to use the existing server-side dart:io library on the client as well.
Instead of having the dart:html library include the HTML 5 File API directly, the dart:io library could be implemented in the client as wrappers for the HTML 5 API.
If certain operations are not supported by the HTML 5 File API, then an UnsupportedOperationException can be thrown when attempting to use them.
The client side Dart VM could fill in holes that are not available in the HTML 5 File API.

This isn't planned. Hopefully if/when Dart has configuration/platform configurable libraries, this can be built by the community.

I'm interested in looking at this issue :) By compiling subset of dart's code to wasm.
Tools like Emscipten's emcc can compile c to wasm. Emcc has a file system api that adapts many of c's stdlib calls to webassembly, please see https://emscripten.org/docs/api_reference/Filesystem-API.html . Example, obscuring filesystem calls as IndexedDB.
Is there any chance that a subset of dart-lang c code can be compiled to wasm? Then used to make dart:io for web?
Does anyone think this is possible? I tried looking into this recently but am unfamiliar with the ninja build system, and want to get an opinion and recommendations on where to start thanks.


Does it works with multi ADs or domains

hi Seji,
thank for your solutions. it helps me a lot!
i wonder if it works with multi domains, multi DC, 2 domains for my case?
thanks.

Hi,
No, unfortunately multiple domains are currently not supported / implemented.


plugin fails when running with sshfs

using sshfs to mount a remote nova-compute / at ~/remote in my VM and trying to run hotsos against it produces the following error:
ubuntu@ganso-bastion:~/git/hotsos$ sudo ./hotsos.sh ~/remote/
INFO: analysing sosreport at /home/ubuntu/remote -Traceback (most recent call last):
  File "/home/ubuntu/git/hotsos/plugins/system/main.py", line 5, in <module>
    PluginRunner()()
  File "/home/ubuntu/git/hotsos/core/plugintools.py", line 226, in __call__
    inst()
  File "/home/ubuntu/git/hotsos/plugins/system/pyparts/checks.py", line 114, in __call__
    self.sysctl_checks()
  File "/home/ubuntu/git/hotsos/plugins/system/pyparts/checks.py", line 99, in sysctl_checks
    if value != actuals[key]:
KeyError: 'net.ipv4.neigh.default.gc_thresh1'

hotsos:
  version: development
  repo-info: a8f45b7
juju:
  version: 2.9.14
  machine: '21'
  charms:
    - neutron-openvswitch-291
    - nova-compute-334
  units:
    local:
      - neutron-openvswitch-0
      - nova-compute-0
  potential-issues:
    - desc: there is no Juju machined running on this host but it seems there should
        be
      origin: juju.machines
      type: JujuWarning
kernel:
  cpu:
    smt: disabled
  memory-checks: no issues found

At first I thought it was because of "analysing sosreport at /home/ubuntu/remote" so I edited hotsos.sh and replaced DATA_ROOT with the path, despite observing that it is pretty much what the code does and the sosreport path message doesn't seem to be relevant, and it gave the same error.
SSHFS works fine and I can access the remote filesystem at ~/remote

The cause of the error is [0].
After changing that value to the path of the sshfs mount, hotsos ran successfully, however, it ran with a lot of data missing when compared to running in the live system itself, and a lot of data (such as kernel version) was from my VM instead of the nova-compute node.
I believe that is because it is invoking BinCmd in my VM. Right now, I see that the commands such as sysctl_all can either be run as a command in the live system or as a file that its path in this particular case would exist in a sosreport. Therefore there would need to be an alternative to access this data through a file in the system such /run, /proc, etc, which are available in the sshfs mount.
[0] https://github.com/dosaboy/hotsos/blob/a8f45b7a4ffa3bd5a0651a5dbdf5f0d99eb806bb/core/cli_helpers.py#L307

as a complementary note, setting up sshfs required copying the authorized_keys of the ubuntu user in the nova-compute node to the /root user, because sshfs needs to be run as root. The known_hosts data would also need to be copied, but I did not need to because I tested ssh'ing to it beforehand and it got added to the known_hosts.

I made the minimum required changes for this to work, and this is the performance result:
running in the host:
real    0m10.293s
user    0m9.104s
sys     0m1.821s
running remotely:
real    1m33.523s
user    0m12.645s
sys	    0m2.931s
Performance is terrible. I guess we will need to find another way to re-using hotsos assets and sharing the code. We can close this bug.


Match to allow Scala style matching on a range of values.

It would be nice to extend Match functionality to allow a range of possible values to match on. An example being the following case in Scala...
def toBool(choice: String): Boolean = choice match {
    case "Y" | "YES" | "T" | "TRUE" | "1" => true
    case "N" | "NO" | "F" | "FALSE" | "0" => false
    case _ => false
  }

Currently this would require multiple 'caze' statements and is a bit more verbose. EG:
Match
	.caze("Y", o -> true)
	.caze("YES", o -> true)
	.caze("T", o -> true)
	.caze("TRUE", o -> true)
	.caze("1", o -> true)
    .caze("N", o -> false) ... etc

Maybe allowing...
Match.caze("Y", "YES", "T", o -> true)

Or if this is not possible, something like...
Match.cazeIn(new String[] { "Y", "YES", "T", "TRUE" }, o -> true)

See PR #315

Now we are able to write:
Match
    .when(1).then(i -> i)   // before: when(1, i -> i)
    .when(2).then(() -> 1)  // before: when(2, ignored -> 1)
    .whenIn(3, 4, 5).then(i -> i)
    .whenIn(6, 7, 8).then(() -> 1)
    .whenNull().then(n -> 1)
    .whenNull().then(() -> 1)
    .apply(1000);

In the meantime the Match API emerged. Instead of using it as function (as shown above), we now may use it as Monad by capturing the value to be matched at the beginning with 'of'.
The new when().then() / whenIn().then() syntax has also to be added to the Match.of() branch of the API. Then we will also able to write:
Match.of(1000)
    .when(1).then(i -> i)   // before: when(1, i -> i)
    .when(2).then(() -> 1)  // before: when(2, ignored -> 1)
    .whenIn(3, 4, 5).then(i -> i)
    .whenIn(6, 7, 8).then(() -> 1)
    .whenNull().then(n -> 1)
    .whenNull().then(() -> 1)
    .get(); // .orElse(...) .orThrow(...) etc.

Note:
Because of ambiguities we cannot use when(null).then(...). Therefore whenNull().then(...) was introduced.
TODO: whenNull().then(...) should be simplified to whenNull(...), where arguments are Supplier and Function.

@martincooper First shot of Match rework finished.
Also added a whenTypeIn(class_1, ..., class_n).then(...). Removed whenNull, it is not necessary any more.
The generic type bounds are currently cluttered. Will do this next.
Unit tests and javadoc + the website have to be updated before the release. I think end of next week is realistic.

Great, thanks!

@martincooper I'm afraid, I can't push the current master to maven central. The Match rework isn't finished yet. At least we have to cover the new API with unit tests. I hacked it from the brain to my git fork. Also, I'm on a trip for a few days and can go back to my notebook within the next week.
We have to do it right, publishing a version before makes no sense.
But luckily I have now vacation (4 weeks) and more time to work on Javaslang.

That's fine. No rush. Better to do it right. I'm on holiday for a couple of weeks too so won't have much time to pick up the update for a while anyway.
Thanks
Enjoy your hols. :)

Thanks! I will put it all together the next days.
Enjoy your hols, too :)


permit existing PIL images to be encoded directly

I'd like to be able to run blurhash on images in memory (i.e: not from a file on disk)
This patch should be a non-breaking change, that also allows PIL images to be passed in and encoded directly, with colorspace conversion if necessary.
An example flow would be:

Open image
Resize to make blurhash quicker
Run blurhash

Of course, apologies!... should be done.

Thank you very much! I'll publish a new version to PyPI soon.

I've released a new version: https://pypi.org/project/blurhash-python/1.2.0/


Simulate Input focus or make its style trigger via prop

I'm creating a date picker using Material-UI Input (TextField) as the base. When the textfield is clicked, naturally the date picker pops up and when it's being manipulated the input losses focus - not a wanted behaviour. I'd like to keep the Material-UI's focused style until needed.
Here's example code, if there's a simple and elegant workaround - I'll be happy to hear it
<TextField
        label="Date"
        onFocus={()=>this.setState({focused: true, showDatePicker: true})}
        inputRef={(input) => { this.dateInput = input; }}
        onBlur={ () => this.setState({showDatePicker: false})}
        value={this.state.date ? this.state.date.format('DD-MM-YYYY') : ''}
        className={classes.textField}
/>
 {(this.state.focused || this.state.showDatePicker) &&
  <DayPickerSingleDateController
           date={this.state.date}
           focused={this.state.focused}
           onDateChange={date => this.setState({ date })}
           onFocusChange={({ focused }) => this.setState({ focused })}
           onOutsideClick={() => this.setState((prevState)=>(!prevState.showDatePicker && { focused:
           false }))}
           ...
 />
}

I'd like to keep the Material-UI's focused style until needed.

@jwm0 We solve the problem like this with the Select component: https://github.com/mui-org/material-ui/blob/5dd0aedb184c62fd0ded0edf8fbe2d16eacb2a22/packages/material-ui/src/Select/SelectInput.js#L113-L124

I'm creating a date picker using Material-UI

@jwm0 We are happy to hear it :), don't forget to benchmark the alternatives:

https://github.com/dmtrKovalenko/material-ui-pickers
https://github.com/chingyawhao/material-ui-next-pickers


[fzf-tmux] fish: Unknown signal '-SIGUSR1'

Category

[ ] fzf binary
[x] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[x] fish

fzf-tmux emits errors in the tmux pane it creates under fish 2.6.0:
- (line 1): function: Unknown signal '-SIGUSR1'
function __trap_handler_-SIGUSR1 --on-signal -SIGUSR1; kill; end
^
from sourcing file -
        called on line 109 of file /usr/local/Cellar/fish/2.6.0/share/fish/functions/trap.fish

in function 'trap'
        called on standard input
        with parameter list 'kill -SIGUSR1 -44257 EXIT SIGINT SIGTERM'

- (line 1): function: Unknown signal '-44257'
function __trap_handler_-44257 --on-signal -44257; kill; end
^
from sourcing file -
        called on line 109 of file /usr/local/Cellar/fish/2.6.0/share/fish/functions/trap.fish

in function 'trap'
        called on standard input
        with parameter list 'kill -SIGUSR1 -44257 EXIT SIGINT SIGTERM'

It looks like it could be a quoting problem related to this line, but haven't confirmed.
Note: this error output is normally obscured by the FZF window, however you can see it briefly before FZF's UI appears, or by pressing Ctrl-Z from within FZF.

I just installed fish 2.6.0 on my macbook, and fzf-tmux works as expected. Maybe a configuration problem?

To be clear, the script still works in that fzf itself launches and exits normally, despite the error, and the text is not necessarily visible on screen unless you suspend FZF.

Maybe a configuration problem?

I'll report back on whether I can reproduce this on a clean installation. (Won't be able to get to it in the next few days though.)

So I guess the trap command is inconpatible in fish. Then we should explicitly execute the commands with bash. Oh, it's frustrating that fish always makes things more complicated.

Can you check if this patch fixes the issue?
diff --git a/bin/fzf-tmux b/bin/fzf-tmux
index 5ec0dc4..c3834f7 100755
--- a/bin/fzf-tmux
+++ b/bin/fzf-tmux
@@ -146,6 +146,7 @@ cleanup() {
   fi

   if [ $# -gt 0 ]; then
+    trap - EXIT
     exit 130
   fi
 }
@@ -170,21 +171,22 @@ for arg in "${args[@]}"; do
 done

 pppid=$$
-trap_set="trap 'kill -SIGUSR1 -$pppid' EXIT SIGINT SIGTERM"
-trap_unset="trap - EXIT SIGINT SIGTERM"
+echo -n "trap 'kill -SIGUSR1 -$pppid' EXIT SIGINT SIGTERM;" > $argsf
+close="; trap - EXIT SIGINT SIGTERM $close"

 if [[ -n "$term" ]] || [[ -t 0 ]]; then
-  cat <<< "\"$fzf\" $opts > $fifo2; echo \$? > $fifo3 $close" > $argsf
+  cat <<< "\"$fzf\" $opts > $fifo2; echo \$? > $fifo3 $close" >> $argsf
+  cat $argsf
   TMUX=$(echo $TMUX | cut -d , -f 1,2) tmux set-window-option synchronize-panes off \;\
     set-window-option remain-on-exit off \;\
-    split-window $opt "$trap_set;cd $(printf %q "$PWD");$envs bash $argsf;$trap_unset" $swap \
+    split-window $opt "cd $(printf %q "$PWD");$envs bash $argsf" $swap \
     > /dev/null 2>&1
 else
   mkfifo $fifo1
-  cat <<< "\"$fzf\" $opts < $fifo1 > $fifo2; echo \$? > $fifo3 $close" > $argsf
+  cat <<< "\"$fzf\" $opts < $fifo1 > $fifo2; echo \$? > $fifo3 $close" >> $argsf
   TMUX=$(echo $TMUX | cut -d , -f 1,2) tmux set-window-option synchronize-panes off \;\
     set-window-option remain-on-exit off \;\
-    split-window $opt "$trap_set;$envs bash $argsf;$trap_unset" $swap \
+    split-window $opt "$envs bash $argsf" $swap \
     > /dev/null 2>&1
   cat <&0 > $fifo1 &
 fi

Can you check if this patch fixes the issue?

Yes, it fixes the issue. :+1:


Remove test folder

test and Makefile are useless for a colorscheme and we actually do not use that function anymore.

Yeah. In the current state these don't really do much. I'm not sure if tests are completely useless for colorschemes, but since this one uses colorbuddy, there's less reason to do (even though there are some custom settings here, but pretty basic).
I've opened a PR to remove these in case @maaslalani also agrees :)

It makes sense to remove these, good call!


Fix/monitoring&logging

Added better log messages to VRFRouteConfiguration and Layer2NetworkConfiguration
Try to fix frr-exporter not correctly terminating sometimes
Reduced exported routes from netlink neighbors endpoint

Looks like frr-exporter netlink neighbors are now broken

Looks like frr-exporter netlink neighbors are now broken

Still missing ^^


Use decimal numbers for endpoint/execution IDs

As discussed some time ago on the dev mailing list.

My counter proposal is to follow @ok2c and increase to 10. This makes by default possible to fit multiples of Integer.MAX_VALUE and still retain a not too long string. Everything else will need to scale. Having no padding as @carterkozak proposed requires less operations in the JVM, but the will leave logging like a flush left text, hard to follow with the eye.


Rails 5.1

this adds tests for rails 5.1.
unfortunately, controller testing changed quite heavily (they're integration tests by default) so those won't run on anything < 5 anymore. not sure how you'd like to approach this.
also makes some other adjustments for which i was too impatient to create separate PRs:

only install growl if running on macOS
gitignore gemfiles/*.lock
add test-unit gem to rails 3.2 gemfile because it got removed from core ruby since 2.2+
updated schema dump (mainly ruby syntax)
code climate integration according to 1.0 docs
made config class compatible with older rails versions (currently only works with active support 4)
upgraded rspec-rails because of changes in newer rake versions
remove version-lock from acts-as-taggable-on

alright. 4.2 and 5.1 tests are green. supporting earlier versions is pointless pain, in my opinion, as they're all eol.
@Bodacious, still there?

@glaszig Thanks for doing this. I'll get back to you ASAP with these PRs.

hello?

This gem being actively maintained anymore?


merge Crafting recs fix

Fix crafting recs moving stuff between farming and crafting in large blocks

Fix #25


assets: improve local development and deployment strategies

Currently, unpkg is being used to fetch the latest updates from the new build system. While this is an ok solution for prototype testing, it makes development much harder as npm publish needs to be run prior to checking if the changes have indeed been done. I saw that the webpack-dev-server runs on it's on port. Maybe it's possible to use that local deployment in development (e.g. through env var) and save the hassle of changing the URL manually.

Yep, that's planned :)

Is this already fixed @inyono?

Well, kind of, as described in https://github.com/serlo-org/athene2#development. You have to change the url only in a single place. An environment variable or something similar would be a further improvement

Closed because of inactivity. Having to comment in a single line is good enough, imho


Go fullscreen on mobile when scrolling down

http://www.html5rocks.com/en/mobile/fullscreen/

Been searching the webs still no results


python3Packages.screenlogicpy: 0.4.3 -> 0.5.0

Motivation for this change
Update to latest upstream release 0.5.0
Change log: https://github.com/dieselrabbit/screenlogicpy/releases/tag/v0.5.0
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 142861 run on x86_64-linux 1

2 packages built:

python38Packages.screenlogicpy
python39Packages.screenlogicpy


Floating Point Exception

Hello,
I'm having a problem running WHAM on some smaller BAM files. Every time I try to run it the WHAM.err file returns the following:
INFO: WHAM-BAM will using the following fasta: ../ABL1_cds.fasta
INFO: OpenMP will roughly use 24 threads
INFO: target bams:
S2.bam
INFO: gathering stats for each bam file.
At which point it exits out and prints "Floating Point Exception" to the terminal.
The reads are aligned with BWA-MEM, and duplicates marked with Picard. Any idea what would be causing this issue? My BAM file is relatively small, could this be causing the problem?
Thanks for your help.
-Ana

@akkellogg Sorry for the late reply, i'm just returning from holiday. What type of data are you working with?  Exome or genome?

@zeeev We think we pinpointed the problem. We were aligning the reads to a custom reference and we think the issue was with that reference, since aligning to a different reference seems to have solved the problem.


Fixes immediate closing bug

Fixes #72

@ArtOfCode- Your requested changes were made, could you merge this?

@Taeir done, thanks for the reminder!


Issue with transitions on Safari

I faced some weird animations on Safari 8.0.8 with MacOS. Digging into the DOM did show me that there is a max-height value for accordion-item-body which is adding up to something close to 100px on my click and then less than a second changes back to none. Any ideas?

thanks for reporting, and the GIF!
Seems like safari animates the switch to max-height: none; as if setting max-height: 0 first; looks the same in 9 (testing in browserstack)
Made a demo here: http://codepen.io/psimyn/pen/zBWQjr
Hover to expand, then click (and hold) should cause flicker. The commented line of CSS in the pen is possibly an option (unset the animation when setting max-height: none

Yes, that's exactly what happened. Seems like Safari has a different interpretation of none value in max-height.
By the way, looking at the code it seems max-height is only used to keep the integrity of animations. Why not just calculate the height as it seems to work well with Safari as well.

I'll have a look at switching to height. Main advantage of using max-height is you can make it work without JS, but since nothing else does at the moment that doesn't seem worthwhile.
If you want to submit a PR for changing to height would be good, otherwise I'll have a look over the weekend

Yes, give me so some time and I will try to figure it out today or tomorrow.

Some progress on this issue?
Thank you very much.

I've submitted a PR (#93) which should fix this issue

@p0o Have you fix this issue?

I'm still having this issue on Safari mobile.

https://github.com/daviferreira/react-sanfona/commit/aa3224eff630e9aa8f9a4adb4a4610c0e28330cf


Do not navigate to the NFT screen or Crowdloan screen

Describe the bug
Do not navigate to the NFT screen or Crowdloan screen
To reproduce the behavior
Steps:

Go to the NFT tab
Click on Send fund to open the Send fund popup
Then close send fund popup
Switch to other account
Click on NFT tab or Crowdloan menu

Actual: Do not navigate to the NFT tab/ Crowdloan screen

Version test: v1.1.7-0
Validation passed


Merge Config and ConfigUtil

We may be able to merge Config and ConfigUtil, making the overall class work more akin to how the other classes in MainApp work, such as Settings.

We are close to implementing final features, going into the bugfixing phase, and wrapping up the project, so closing this idea.


[Question]: include_text parameter in index.as_query_engine method

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
Hi, I don't quite understand the include_text parameter in index.as_query_engine method. What's the difference between when it is set to be True or False please? Thanks a lot!

This is specific to knowledge graph indexes.
If it's true, the chunk where a matching triplet is found will also be sent to the LLM
If false, then only the matching triplets are sent to the LLM

This is specific to knowledge graph indexes.
If it's true, the chunk where a matching triplet is found will also be sent to the LLM
If false, then only the matching triplets are sent to the LLM

Thanks a lot!


Disable drag feature for individual node?

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Feature request
[X ] Documentation issue or request

What is the current behavior?
I am not sure if this is possible but been looking for a way to disable drag or drop on certain nodes. Like if a user does not have write permissions on certain nodes. If this is possible is there any documentation on how to use it?
Expected behavior:

Minimal reproduction of the problem with instructions:

Version of affected browser(s),operating system(s), npm, node and angular-tree-component:
Other information:

Hi @kingbe01, it's possible to just enable/disable dragging for specific nodes. If you take a look into the docs for the dragging there is the option to set functions for allowDrag and allowDrop. In those functions you get the node and can test some node properties and enable/disable it based on that.
In our application we get a bool from the api if a node is draggable. If you want to base this on permissions you probably need to add a type property to the nodes and in the function check if the user has permission for that type.


FIX: Compatibility with DALI 0.18.0

Since DALI 0.16.0, nvJPEGDecoder() was renamed to ImageDecoder().

@TE-PongsakornUchupala Thanks for your contribution!


Additional playbooks

Adding the following playbooks

MDTI DATA host info
MDTI Automated Triage Custom
Article Newsletter

@KwachSean  can you resolve the conflict it is showin above in the readme file. not allowing to merge? if doesn't you might have to delete this pull request and submit a new one

adding additional changes to the playbooks


Improve the transposition table

PR not ready to be merged.
Added:

Clear function to be able to clear the transposition table of all entries.

Thanks for the PR!
I'm going to close this since the PR is a bit old now and there appear to be compilation errors with it.


[Feature Request]: Add Heap Data Structures and Algorithms

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to contribute by adding heap data structures (min-heap, max-heap) and associated algorithms (heap sort, heapify, etc.) to the package.
@UTSAVS26, please assign this task to me. I believe this would be a useful enhancement to the pysnippets package.
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Full Name
BENAK DEEPAK
Participant Role
GSSOC

Can u assign me this task i want to contribute to this


printf() and other var_args do not handle null terminated byte slices gracefully

The byte slices need to be correctly sized as strings (NullTerminatedByteSlice) before they can be printed. Otherwise you get random characters after the last byte.

Actually this might not be a bug. The functions populating the buffers (like fread) are not responsible for adding the NULL character on the end.


[FR] Automatically set FileClass

Use case:

Create a new file
In YAML frontmatter (new Properties UI, version 1.4 of Obsidian) add a property / field, for example 'project:'
As a value for this property, type a link to an existing file
Expected behavior: the selected file automatically gets the FileClass 'Project'

(Inspired by Tana where certain nodes / field are automatically associated to a supertag)

I would love to see this feature being implemented.
(But I have no idea how feasable this is)
Tag VS property field for defining a type of a note remains a disputed area, but especially with the new properties manager and having backlinks in the metadata, I guess there are a substential number of people using now properties for that.
In my perspective, having added the relevant metadata fields when setting a type by a property would be a major QOL update when using obsidian,
Thank you so much for the work you are doing and have done so far!

you could try this setup based on a gloabl fileclass containing a select field filled with the list of fileclasses
https://www.youtube.com/watch?v=yRt_9fDRd_A&t=390s

Hey,
I got this working with Query based FileClass settings.
With help out of the community I created this query for dataviewJS:
dv.pages().where(p => dv.func.contains(p.type, dv.fileLink("name of internal link")))
Just create queries for fileclasses and it works 👍
So thank you for implementing this mdelobelle!
But I would still hope for a native implementation, since I think this is fundamental useful ;)
Setting up queries is cumbersome and it would be nice to just have an option to quickly connect the fileclass with a link.
I too wonder if it gets heavy on the performance when checking a lot of queries in the background (?).
I don't have any programming knowledge, but was thinking how it could work:
Could the plug-in create a list of paired field values and fileclasses?
So setting up a "special" field (e.g. "type") that has the ability to set fileclasses.
In a fileclass (e.g. fileclass_countries) have an option to set a paired internal link (e.g. "[[countries]]").
It then adds this information to the list:

if type:: [[countries]] -> fileclass: fileclass_countries

Now when one opens a file, if it has the "special" field ("type") it will check the list and if there is a match with an internal link, it applies the right file class.
Thoughts on this?
I would love this feature, but I'm already more than happy with metadatamenu as is it ;) So just an idea.
As always thanks for putting so much work in the plug-in!


As a MacroMaker user I want to use a keyboard shortcut to start/stop recording so that my recorded session won't be affected by opening MacroMaker window

I would like to record a 6-steps mouse click macro in a fullscreen app and then play it on MacroMaker for 50 times.
Currently that is not possible as after the last step I had to switch to MacroMaker window (using Alt+Tab) to stop the recording and this will also be recorded as a macro in my MacroMaker session making it not possible to reproduce the recorded session more than once.
Would be good to have a keyboard shortcut (customisable key combination) to start/stop the recording so the case above would be possible.

I will look into that. Glad to see that someone actually found this little tool useful :)


[vtk] Added opengl feature.

Describe the pull request
Adds opengl feature.
Change description of feature: all.

Which triplets are supported/not supported? Have you updated the CI baseline?
all, no.
Does your PR follow the maintainer guide?
yes.

Updated.

Thanks for your update.
Needs to test the features now. I will update the test results once finished.

The feature tests are blocked by icu
warning: make-4.3-1 is up to date -- skipping
error: could not open file /var/cache/pacman/pkg/libgdbm-1.18.1-3-x86_64.pkg.tar.zst: Child process exited with status 127
error: could not open file /var/cache/pacman/pkg/gdbm-1.18.1-3-x86_64.pkg.tar.zst: Child process exited with status 127
error: failed to commit transaction (cannot open package file)

So we need to fix this issue first.

@larshg
The problem caused by icu has been resolved.
Could you please merge the latest commit to this PR?

Isn't a rebase the preferred way?

I will update this on my local to test features.
You won't merge it any more.

Hi @larshg
Sorry for the long delay.
Could you please look into the following problems?
The features build failed on x64-windows triplet:
The feature all is broken due to this:
CMake Error at CMake/vtkDownload.cmake:8 (message):
  Attempted to download Module_MomentInvariants when VTK_FORBID_DOWNLOADS is
  ON
Call Stack (most recent call first):
  CMake/vtkModuleRemote.cmake:142 (vtk_download_attempt_check)
  Remote/MomentInvariants.remote.cmake:5 (vtk_fetch_module)
  Remote/CMakeLists.txt:9 (include)

The feature qt, atlmfc, openvr ,paraview is broken due to this:
CMake Error at ThirdParty/hdf5/CMakeLists.txt:26 (message):
  An external MPI-aware HDF5 requires that VTK be built with MPI support as
  well.

The feature python is broken due to this:
CMake Error at CMake/vtkModule.cmake:4140 (message):
  Could not find the Python3 external dependency.
Call Stack (most recent call first):
  Utilities/Python/CMakeLists.txt:49 (vtk_module_find_package)

The feature mpi is broken due to this:
CMake Error at CMake/vtkModule.cmake:959 (message):
  The VTK::IOParallelNetCDF module requires the disabled module VTK::mpi.
Call Stack (most recent call first):
  CMakeLists.txt:251 (vtk_module_scan)

All of those features seems to be unrelated to the opengl features? So those features has stopped working since VTK 9 was first added by @Neumann-A ?
I don't really have the time nor knowledge to look into these errors :(

@larshg
Thanks for your instant reply.
opengl is also broken due to this
-- Found OpenGL: opengl32  found components: OpenGL
-- vtkDICOM: Building vtkDICOM as a Remote VTK Module
CMake Error at ThirdParty/hdf5/CMakeLists.txt:26 (message):
  An external MPI-aware HDF5 requires that VTK be built with MPI support as
  well.

I will check this on the master later.

-- vtkDICOM: Building vtkDICOM as a Remote VTK Module
CMake Error at ThirdParty/hdf5/CMakeLists.txt:26 (message):
An external MPI-aware HDF5 requires that VTK be built with MPI support as
well.

VTK needs to pass -DVTK_GROUP_ENABLE_MPI=YES which it does:
if("mpi" IN_LIST FEATURES)
    list(APPEND ADDITIONAL_OPTIONS
        -DVTK_GROUP_ENABLE_MPI=YES
    )
endif()

Maybe we need to also pass -DVTK_USE_MPI=ON

I have just rebased on master an rebuild vtk[opengl] but didn't encounter any errors?

/azp run

@larshg
Feature opengl is broken on x64-windows:
-- vtkDICOM: Building vtkDICOM as a Remote VTK Module
CMake Error at ThirdParty/hdf5/CMakeLists.txt:26 (message):
  An external MPI-aware HDF5 requires that VTK be built with MPI support as
  well.

After add -DVTK_USE_MPI=ON, the error is like this:
CMake Error at F:/11399/vcpkg/scripts/buildsystems/vcpkg.cmake:252 (_add_library):
  Target "IONetCDF" links to target "MPI::MPI_C" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3328 (add_library)
  IO/NetCDF/CMakeLists.txt:10 (vtk_module_add_module)

CMake Error at F:/11399/vcpkg/scripts/buildsystems/vcpkg.cmake:252 (_add_library):
  Target "IOParallelNetCDF" links to target "MPI::MPI_C" but the target was
  not found.  Perhaps a find_package() call is missing for an IMPORTED
  target, or an ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3328 (add_library)
  IO/ParallelNetCDF/CMakeLists.txt:14 (vtk_module_add_module)

CMake Error at F:/11399/vcpkg/scripts/buildsystems/vcpkg.cmake:252 (_add_library):
  Target "IOMINC" links to target "MPI::MPI_C" but the target was not found.
  Perhaps a find_package() call is missing for an IMPORTED target, or an
  ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3328 (add_library)
  IO/MINC/CMakeLists.txt:15 (vtk_module_add_module)

CMake Error at F:/11399/vcpkg/scripts/buildsystems/vcpkg.cmake:252 (_add_library):
  Target "exodusII" links to target "MPI::MPI_C" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3328 (add_library)
  ThirdParty/exodusII/vtkexodusII/CMakeLists.txt:300 (vtk_module_add_module)

CMake Error at F:/11399/vcpkg/scripts/buildsystems/vcpkg.cmake:252 (_add_library):
  Target "IOExodus" links to target "MPI::MPI_C" but the target was not
  found.  Perhaps a find_package() call is missing for an IMPORTED target, or
  an ALIAS target is missing?
Call Stack (most recent call first):
  CMake/vtkModule.cmake:3328 (add_library)
  IO/Exodus/CMakeLists.txt:18 (vtk_module_add_module)

Do you install it with other features enabled as well? I'm on x64-windows and I don't encounter those errors?

Ah, maybe I haven't got an MPI-aware HDF5 installed...

This is my hdf5:

So this error is probably also present if you just try to install vtk without opengl feature?

I just try to install vck[opengl]:x64-windows, and it will fail due to the errors.

On a clean machine or do you have multiple features installed from hdf5 (specially the parallel feature which adds MPI).

I try to build it on a clean machine and there are no any errors on x64-windows.
Have you tested it on other triplets?

No, I work only on x64-windows with dynamic libraries.
I thought this is why we have CI's running various systems/configs :-)

Thanks for your update. I will continue to test other triplets.

Feature opengl has passed with the following triplets:

x86-windows
x64-windows
x64-windows-static

Note: Other features are not changed.

Thanks for the hard work!


The .fit() method passes unsized batches to a custom loss function.

The .fit() method passes unsized batches to a custom loss function if the dataset size is not a multiple of the batch size.
This happens twice, then everything goes fine. But if you need to use the batch size to normalize or reshape data, then an error occurs.
import numpy as np
from tensorflow import keras
import keras.backend as K

inp_data = np.full((1000, 8), [0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8])
out_data = np.full((1000, 4), [0.2, 0.4, 0.6, 0.8])

inp = keras.layers.Input((8,))
x = keras.layers.Dense(32, activation='relu')(inp)
out = keras.layers.Dense(4, activation='sigmoid')(x)

model = keras.Model(inp, out, name='My_model')

def my_loss(x, y):
    if x.shape[0] is None: print("Batch size = None!", x.shape)
    return K.sum(K.square(x-y), axis=0)   # /x.shape[0] <---ERROR, because None!

#model.compile(optimizer='adam', loss='mean_squared_error')
#log = model.fit(inp_data, out_data, epochs=3, batch_size=50)   # OK! 1000/50=20
#log = model.fit(inp_data, out_data, epochs=3, batch_size=30)   # OK! 1000/30=33.33~ -> 34

model.compile(optimizer='adam', loss=my_loss)
#log = model.fit(inp_data, out_data, epochs=3, batch_size=50)   # OK!
log = model.fit(inp_data, out_data, epochs=3, batch_size=30)    # Batch size = None!

"""
Epoch 1/3
Batch size = None! (None, 4)
Batch size = None! (None, 4)
34/34 [==============================] - 1s 2ms/step - loss: 0.0961
Epoch 2/3
34/34 [==============================] - 0s 2ms/step - loss: 0.0268
Epoch 3/3
34/34 [==============================] - 0s 2ms/step - loss: 0.0055
"""

Hi @Ybisalt ,
I have tested the given code with Keras 3.3.3v and it executes fine. Please note that I have changed the code K.sum to keras.ops.sum and same for K.square also. Please refer to attached gist.

I have tested the given code with Keras 3.3.3v and it executes fine. Please note that I have changed the code K.sum to keras.ops.sum and same for K.square also.

No! Same problem here. You didn't notice "Batch size = None! (None, 4)" line in the last output log.
The number of batches without size depends on the order in which the method is called. Try just one run of the fit() method:
log = model.fit(inp_data, out_data, epochs=3, batch_size=30)    # Batch size = None!

x.shape is the "static shape" of x. It is often not a number. It can be None. If you want the actual number value, you must use keras.ops.shape(x), e.g.
keras.ops.sum(keras.ops.square(x - y), axis=0) / keras.ops.shape(x)[0]

Hi @Ybisalt -
I am not able to reproduce this issue in lates keras(3.5.0) version. Attached gist for the reference.
Are you still able to reproduce this issue ?


more clear readme with a diagram and general fixes

much more brief and concise than before, hope you like it.

I see that you also corrected a few cases of bad grammar in there. Hah, thanks! Anyway, I left you a few comments.

I have created a wiki page for more detailed description of ProxyCommand in general and ssh -W. I was thinking of just linking to that from the README. I have added a diagram like yours as well. Can you comment on that?
I'm sorry for turning your PR's down. I really do appreciate the feedback, and the need for documenting this. I just really want the first thing people see to mostly be an overview, especially seeing that this is not isolated to sshmux, but instead a fairly normal SSH feature.


03-matrix-multiplication.py problems

When I comment out this line https://github.com/openai/triton/blob/db6935c300508cc1f3933059c70c87ba4f0221ee/python/triton/testing.py#L141, triton becomes very time-consuming? Why

You can try do_bench_cudagraph

Related #404


[android] server error code : 400

Report
Plugin Version
6.1.2
On what Platform are you having the issue?
android
What did you do?
apk install  + android studio results with server code 400.
{"data": "Error connection to server: 400", "status": "failure", "type": "onInstallConversionFailure"}
What did you expect to happen?
no 400 error
Please provide any other relevant information.
it all works fine with ios. there are 2 versions of app for ios and android specifically.

Hi @genseric.  Be sure you integrated the SDK correctly, with all the permissions needed, and you added the app to your AppsFlyer account.
for more help, please contact our support team<EMAIL_ADDRESS>


Animate the Sign-In Link Pages

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

cc @shawnborton
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0179f7696509701054
Upwork Job ID:<PHONE_NUMBER>656201728
Last Price Increase: 2023-05-02

Job added to Upwork: https://www.upwork.com/jobs/~0179f7696509701054

Triggered auto assignment to Contributor Plus for review of internal employee PR - @eVoloshchak (Internal)

not overdue

not overdue

not overdue

started this

not overdue

@shawnborton could I get the lottie files for these (magic ball, magician, locked safe)? While I'm working on the animated backgrounds sizing problems, these seem straightforward since they're statically sized

Safe: https://www.dropbox.com/s/rvg9lp3fj63wgms/Expensify-Safe-Slow-022323-kjs-1.json?dl=0
Code: https://www.dropbox.com/s/ln5w64pxlv8pvva/Expensify-Magic-Code-121222-kjs-1.json?dl=0
Link: https://www.dropbox.com/s/s2yb7qvmkxvommc/Expensify-Magic-Link-120922-kjs-1.json?dl=0

not overdue

not overdue

not overdue, not prioritizing

^

^

^

@shawnborton could you confirm if the design here is still correct? I'm going to get this sorted to delegate out

Wow, blast from the past!
I think those designs are correct but honestly I'm not even sure how you access them, as I haven't really run into any of these designs in the wild before.
Perhaps @cristipaval can help confirm if these designs are still relevant or not, as I think you helped implement them back then?
cc @Expensify/design for visibility, too.

yes the above designs are still the same in NewDot. But there is one more page for when the magic code expired

@cristipaval how do you access the designs from above though? The images from the first comment on this issue.

@shawnborton I'm unsure I understand what you mean by accessing the design. Are you asking how I open the pages as a user in NewDot, or you're asking how the App loads the images? (they are bundled in the App)

Ah, I meant how do those pages get accessed as a normal user. My understanding was that most of those cases were just directly handled in the left area of the sign in page now?

These pages are opened when the user clicks on the magic link received in the email. The link opens a new browser tab with one of these pages, according to the sign-in state that the user has on the browser. Please see the screen recordings below
https://github.com/Expensify/App/assets/18078393/734fc8cd-45e9-49a3-9103-de100c117c64
https://github.com/Expensify/App/assets/18078393/9cf4ea8d-cc9d-4907-9bd9-fb3a2af8b530

Right on - that's what I was looking for!
Magic Code:

Lottie file here
Magic Link:

Lottie file here

cc @Expensify/design for vis, so you can see how dope these are

Yeah those are absolutely sick!

Should be able to get this rolling again, I need to double check where we're at with these and get assets then I'll have a contributor take it on

Let's get this moving- Making this External!
See here for assets https://github.com/Expensify/App/issues/18262#issuecomment-1639104282

Proposal
Please re-state the problem that we are trying to solve in this issue.
Animate the Sign-In Link Pages
What is the root cause of that problem?
New Feature
What changes do you think we should make in order to solve the problem?

There are different sign pages that needs to be animated like hmmm not here page, invalid magic page etc,
What alternative solutions did you explore? (Optional)
N/A

Updated proposal

Updated Proposal with test branch and example, of changes I will make to the files to implement animations in Sign-in Link Pages @shawnborton @grgia

@eVoloshchak if you're around, I think we can get this assigned today

Updated Test Branch with assessts as provided above, but its not working maybe need Lottie converted file. @shawnborton

do you remember how we've been getting Lottie converted files previously?

Yeah, we've been converting them from .json to .lottie in https://app.lottiefiles.com/?utm_medium=web&utm_source=login-main-nav - sign in with credentials in 1P, upload the JSON, download the .lottie: https://stackoverflowteams.com/c/expensify/questions/17073

Amazing, thanks for the refresher!

@shawnborton are you able to link the converted files ?

assigning BugZero for tracking

Let me know if these work: Animations.zip
FWIW I think we already have the Safe animation in product in Settings > Security.

bump for C+ review @eVoloshchak, but low priority :)

Three proposals were posted in a 5-minute timeframe with @tienifr's being the first one. This is a feature request, so a proposal cannot contain a lot of details, only the general approach (this is demonstrated by 3 proposals sharing the same approach, i.e. "replace Icon with Lottie assets").
I think even the initial @tienifr's proposal (We can replace the use of static Icon in those pages with the Lottie that uses the above assets.) is valid, the proposal looks good to me

🎀👀🎀 C+ reviewed!

Thank you @eVoloshchak, assigning @tienifr

Hmm I followed the instructions that @roryabraham had provided. @roryabraham - any ideas why this one might be so laggy?

I don't think we have a good animation for that particular case, so we can leave that one as-is for now.

any ideas why this one might be so laggy?

Unfortunately not really if you already used the LottieFiles "optimized dotlottie" file. It might be the case that automated tools can only do so much, and optimizing lottie animations beyond that requires a good amount of requisite skill in AfterEffects that I don't have.
The only other thing you might try is switching Lottie to use the canvas renderer instead of svg - I have seen that provide smoother animations before, but not consistently.

The only other thing you might try is switching Lottie to use the canvas renderer instead of svg - I have seen that provide smoother animations before, but not consistently.

Is that something that you do in the code, or via LottieFiles? I can't seem to find anything like that in LottieFiles.

switching to the canvas renderer would be done in the code. It might also require a patch for lottie-react-native to enable that, as it's web-specific.
Also you might try setting different values for the renderMode flag, also a code-level optimization

posted some additional thoughts in slack here

Can someone help me understand the exact issues we're having with the animation? I just need more information to take back to the animator to help them better understand what they need to change. Curious for your thoughts too @roryabraham

ok, so I tried the animation in a Lottiefiles preview and it seems to be running just fine. Going to try again in our app.

Seems to be running fine for me (albeit on a different page) 🤷🏼
https://github.com/Expensify/App/assets/47436092/afe80b57-07aa-4ffc-a49c-5c5b7b0ca414
@tienifr try again?

Oh, I'm looking on Chrome though. I see @tienifr said the problem was on Safari

Confirmed, the problem is with the lottie file itself - even on lottiefiles' preview it doesn't run at all in Safari 😞

Looks similar to this issue, which frankly is a bit over my head.

@shawnborton do we have access to the original AfterEffects files for these animations anywhere? After importing them into AfterEffects from the Lottie json files, they seem a bit nonsensical. I'm wondering if exporting and reimporting them is mangling they AE project?

Sorry, no update from me. I also don't have a super clear plan-of-action, was just going to open the animations in AfterEffects and watch youtube videos to try and figure out how to optimize it / figure out why it's not working well in Safari.

Maybe we can merge as-is for now and then follow up to fix the specific Safari issue for the magic code animation?

Sure, @eVoloshchak if you continue with the review and ignore the issue with the crystal ball animation on Safari we can move forward

Payment Summary:

Contributor: @tienifr $250 paid via Upwrok
Reviewer: @eVoloshchak $250

@eVoloshchak can you complete the checklist and request payment when you get a chance?

@eVoloshchak can you complete the checklist

@garrettmknight, according to https://github.com/Expensify/App/issues/43718#issuecomment-2166872712, it was decided to remove the animations added by this PR, so checklist is not applicable

Still open for payment

$250 approved for @eVoloshchak


T-Shirt Design Contest

http://cambridgeenergyalliance.org/winit has a logo. We need an initial t-shirt design concept.

Closing this because it will be included as part of what Christine is doing in https://github.com/codeforboston/cambridge_energy_app/issues/45


3.0 Shader Language: render_mode diffuse_burley is broken

windows, 8ed1a28, nvidia (i doubt that it is driver related tho)
(two screens below)

How is that part of the issue template not relevant enough for you to keep it?

Operating system or device, Godot version, GPU Model and driver (if graphics related):

Oh, this was fixed by tagcup some weeks ago, closing.


Userled driver: Implement getall by checking actual pin state

Summary
The include/nuttx/leds/userled.h interface already defines the ULEDIOC_GETALL command for retrieving the current state of the LEDs. The current implementation directly returns the state as it was previously set.
However, this does not necessarily reflect the actual state of the LEDs. Hardware may be broken or there might be a wrong configuration in software. We would like to change this command so it reads the actual pin state of the pins that control the LEDs, which is a more accurate representation of the LED state. This would require an additional board_userled_getall() function to be defined.
This is still open for discussion. We want a way to check the actual LED state to verify that the LED on our board is correctly controlled and that the LED is mounted without short circuits. We want to do this from user board logic or a user application. The existing ULEDIOC_GETALL command seems like a good fit, but only provides the state to which the LED should have been set, not its actual state. If the original functionality needs to be preserved, maybe the proposed functionality can be implemented as a separate command. We would just like to hear the opinion of the community. We are also open to other suggestions to implement this kind of functionality.
FYI @cisvanmierlo
Impact
Boards and applications that previously used the generic userled driver implementation would need to implement a board_userled_getall() function. This then needs to read the logic level of the pins that control the LEDs. Exact implementation might vary between MCUs and boards.
Testing
I will test on S32K144EVB as soon as possible. I just wanted to start the discussion already.

I agree with this modification, but why don't call it just "userled_ledget" instead of "userled_ledgetall"? Notice that userled_ledset is able to set all LEDs, but we don't call it userled_ledsetall. Just to keep things uniform/equivalent.

Thanks for your feedback, @acassis. I used "userled_ledgetall" because the IOCTL command is named ULEDIOC_GETALL, so I was trying to keep it consistent.
There are also IOCTL commands for setting a single LED (ULEDIOC_SETLED) and for setting all LEDs (ULEDIOC_SETALL). If in the future maybe a ULEDIOC_GETLED command (to get the state of a single LED) is to be added, then the naming will get confusing if I named it "userled_ledget" instead of "userled_ledgetall".
Naming is unfortunately not really consistent between userled_lower.c and userled_upper.c, which makes this a bit of a mess. I just felt that naming it "userled_ledgetall" would avoid most of this confusion and keep it somewhat consistent with the existing IOCTL commands and function names (userled_led and userled_ledset).
I think it would be more clear if we renamed all userled_ functions; "userled_supported" can remain as it is, but "userled_led" should become "userled_setled". "userled_ledset" should become "userled_setall". Then the new one can become "userled_getall"?

Thanks for your feedback, @acassis. I used "userled_ledgetall" because the IOCTL command is named ULEDIOC_GETALL, so I was trying to keep it consistent.
There are also IOCTL commands for setting a single LED (ULEDIOC_SETLED) and for setting all LEDs (ULEDIOC_SETALL). If in the future maybe an additional ULEDIOC_GETLED command (to get the state of a single LED) is to be added, then the naming will get confusing if I named it "userled_ledget" instead of "userled_ledgetall".
Naming is unfortunately not really consistent between userled_lower.c and userled_upper.c, which makes this a bit of a mess. I just felt that naming it "userled_ledgetall" would avoid most of this confusion and keep it somewhat consistent with the existing IOCTL commands and function names (userled_led and userled_ledset).
I think it would be more clear if we renamed all userled_ functions; "userled_supported" can remain as it is, but "userled_led" should become "userled_setled". "userled_ledset" should become "userled_setall". Then the new one can become "userled_getall"?

@jarivanewijk I agree! You can, please do that in a new PR.

I just opened another PR to address these concerns. After that PR gets merged I will continue work on this PR.


please help parse-server mongo It is reset every 1st day

please help parse-server mongo It is reset every 1st day

Please fill the issue template and reopen the issue


Storage test enhancements

closes https://github.com/Azure/azure-sdk-for-cpp/issues/614
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

@katmsft I'd like to know this this test case was disabled. I didn't find any related issue.

@katmsft I'd like to know this this test case was disabled. I didn't find any related issue.

@katmsft also, let's add LAT back and see if it works now

@katmsft also, let's add LAT back and see if it works now

Looks like these uncommented tests are still failing.
Refreshing the run since the build is gone:

/azp run cpp - storage

Looks like these uncommented tests are still failing.
Refreshing the run since the build is gone:

/azp run cpp - storage

/azp run cpp - storage

/azp run cpp - storage

@Jinming-Hu This PR is nearing 30 days old.  If this PR is abandoned please close it.  If the PR is still valid please work to get it submitted ASAP.

Moving the milestone to March since this doesn't seem ready for the Feb release.

This PR is now really old with no recent activity. Please feel free to re-open when you plan to work on it.


Incompatibility with select2-bootstrap-5-theme. Clear button not appearing.

When using select2-bootstrap-5-theme package  for theming, clear button doesn't appear. This is clearly a compatibility issue between these two package since DAL in isolation works fine and same for select2-bootstrap-5.
I don't know if it's DAL issue or select2-bootstrap-5 issue so I created a issue there too.
Steps to reproduce:

Setup DAL in a new Django project.
Add select2-bootstrap-5 css.
Set "data-bootstrap-5" and "data-allow-clear" in widget's data attributes.

autocomplete.ModelSelect2(
    url='<View url>',
    attrs={
        'data-theme': 'bootstrap-5',
        'data-allow-clear': True
    },
)

I created a sample repository that shows the issue.

Ok to close @Alireza-Farahani ? Looks like the issue was related to script order - in the cross-post, it looks like this was resolved. https://github.com/apalfrey/select2-bootstrap-5-theme/issues/70

Yep. I'm going to close this. However as a side question, @shapiromatron what do you suggest as the best solution to scripts order? Subclassing DAL classes, putting 'select2-bootstrap-5' css files at the end of '' in base template or another way I'm not aware of?

Hey @Alireza-Farahani; I'd just change the import order in html, and not get too fancy with subclassing DAL classes. In fact, the response from @apalfrey on the other issue seemed entirely correct - https://github.com/apalfrey/select2-bootstrap-5-theme/issues/70#issuecomment-1222057988


Custom dataset training clarification

Hi team 👋!
First of all great project! I'm super excited to see that you used Detectron2 as framework of choice.
I'm trying to train my own model using custom dataset in COCO format. And for now I have 2 questions:

Do I need to provide annotations for all tasks? Thats how I understand those guidelines. I'm mostly interested in instance segmentation task:

coco/
  annotations/
    instances_{train,val}2017.json
    panoptic_{train,val}2017.json
    caption_{train,val}2017.json
    # evaluate on instance labels derived from panoptic annotations
    panoptic2instances_val2017.json
  {train,val}2017/
    # image files that are mentioned in the corresponding json
  panoptic_{train,val}2017/  # png annotations
  panoptic_semseg_{train,val}2017/  # generated by the script mentioned below

Do I need to train on 8x A100. I understand that you needed that much power when you trained from scratch, but if I use your checkpoint will 1x A100 be sufficient?

Hi @SkalskiP, thanks for your interest in our work. Please find the answers to your questions below. I have tried to be as comprehensive as possible.

Annotations:

As we state in the paper, we only use panoptic annotations during our joint training process. Therefore, you need only the panoptic annotations for training the model.
However, we evaluate our model on the respective task annotations (task belongs to {panoptic, instance, semantic}). So, during data preparation, we prepare annotations for all tasks.
Custom Dataset Training: If you only want to train and test for the instance segmentation task, you can change the task sampling probabilities in the config files and set the default task as instance during testing. Note that we derive all labels from panoptic annotations in the dataset scripts, so prepare your files accordingly.
https://github.com/SHI-Labs/OneFormer/blob/fea6b22e4b4cba00646abcc5872005b034dae83b/configs/coco/Base-COCO-UnifiedSegmentation.yaml#L46-L48 https://github.com/SHI-Labs/OneFormer/blob/fea6b22e4b4cba00646abcc5872005b034dae83b/configs/coco/oneformer_R50_bs16_50ep.yaml#L57

Training Resources:

I haven't tried training on a single GPU myself. I suggest training on at least 2 or 4 A100 GPUs because the hyper-parameters are set for multi-GPU training. Also, you might encounter OOM on a single GPU.

Feel free to re-open if you face any issues.

Hey, I am trying to train a custom dataset for instance segmentation which is in the coco data format. I am able to train the model, it lacks bbox coordinate while inference, thus unable to infer the results.


Bug in ReferencedObject::init

Hello, I think there is an error in ReferencedObject::__init__. I found it like this:
import sbol2

participation = sbol2.Participation('p', participant='p1')
print(sbol2.ReferencedObject(participation, 'aa','bb', '1', '1', [], 'aaa'))
print(participation.participant) #prints none

I think I found the error, posting a PR soon.

This is fixed now. I don't understand what is the use-case for the if statement. When is self._sbol_owner None, and why not assign the property in that case and add an empty list instead?
I noticed that there is other similar if statements in the file, maybe it would be good to review them.


Add example, setup instructions. (#5)

Codecov Report

Merging #6 into master will decrease coverage by 1.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage   65.99%   64.55%   -1.45%
==========================================
  Files           6        6
  Lines         347      347
==========================================
- Hits          229      224       -5
- Misses         89       92       +3
- Partials       29       31       +2

Impacted Files
Coverage Δ

client.go
70.1% <0%> (-5.16%)
:arrow_down:

Continue to review full report at Codecov.

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


Unhandled promise rejection on lock timeout

Examine the following code:
const AsyncLock = require("async-lock");
const lock = new AsyncLock({ timeout: 5000 });

lock.acquire("test", function(done) {}); // hold the lock open
const promise = lock.acquire("test", function() {}); // try to acquire the busy lock
promise.then(() => console.log("success"));
promise.catch(() => console.log("fail"));

What do you expect the program to output? I was expecting "fail", but the reality was:
$ node test.js
fail
(node:20581) UnhandledPromiseRejectionWarning: Error: async-lock timed out
    at Timeout._onTimeout (/home/vwoo/shared/autocomplete/node_modules/async-lock/lib/index.js:164:17)
    at ontimeout (timers.js:436:11)
    at tryOnTimeout (timers.js:300:5)
    at listOnTimeout (timers.js:263:5)
    at Timer.processTimers (timers.js:223:10)
(node:20581) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:20581) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

It is very strange to me that we have both the "fail" message as well as an unhandled promise rejection error. I'm not sure why and I'll try to take a peek inside the library to understand (just documenting my findings here).

One thing that jumps out at me is that the library seems to have two different ideas of when it should be in "promise" mode or not.
/**
 * Acquire Locks
 *
 * @param {String|Array} key 	resource key or keys to lock
 * @param {function} fn 	async function
 * @param {function} cb 	callback function, otherwise will return a promise
 * @param {Object} opts 	options
 */

//...

	if (typeof (cb) !== 'function') {
		opts = cb;
		cb = null;

		// will return a promise
		deferred =  new this.Promise(function(resolve, reject) {
			deferredResolve = resolve;
			deferredReject = reject;
		});
		deferred.catch(function() { console.log('wtf3') })
	}

The jsdoc claims that it is the presence of the cb function that triggers promise mode, which, hey, fair enough. However, elsewhere in the code:
		// Callback mode
		if (fn.length === 1) {
			var called = false;
			fn(function (err, ret) {
				if (!called) {
					called = true;
					done(locked, err, ret);
				}
			});
		}
		else {
			// Promise mode
			self._promiseTry(function () {
				return fn();
			})
			.then(function(ret){
				done(locked, undefined, ret);
			}, function(error){
				done(locked, error);
			});
		}

It seems the library also uses the signal of whether the fn executing function accepts a done callback parameter or not to determine if it is promise mode. This seems like a potential cause of this issue.

Oh, maybe I don't have a bug, I forgot that promises fork in the chain - I should have written
promise.then(() => console.log("success")).catch(...
Closing for now!


python run_CIL.py error

File "<string>", line 32     def single_image_random_dot_stereograms(depth_values, hidden_surface_removal=True, convergence_dots_size=8, dots_per_inch=72, eye_separation=, mu=, normalize=True, normalize_max=, normalize_min=, border_level=, number_colors=256, output_image_shape=[1024, 768, 1], output_data_window=[1022, 757], name=None):                                                                                ^ SyntaxError: invalid syntax
I am encountering this issue, when I try to run the code in python3.7, tensoflow-gpu 1.14, cuda-10, ubuntu 18. Any pointers?

Try running the suggested versions please ! I haven’t updated it :(
On Tue, Jul 16, 2019 at 8:03 PM Varun Jammula<EMAIL_ADDRESS>wrote:

File "", line 32 def
single_image_random_dot_stereograms(depth_values,
hidden_surface_removal=True, convergence_dots_size=8, dots_per_inch=72,
eye_separation=, mu=, normalize=True, normalize_max=, normalize_min=,
border_level=, number_colors=256, output_image_shape=[1024, 768, 1],
output_data_window=[1022, 757], name=None): ^ SyntaxError: invalid syntax
I am encountering this issue, when I try to run the code in python3.7,
tensoflow-gpu 1.14, cuda-10, ubuntu 18. Any pointers?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/carla-simulator/imitation-learning/issues/80?email_source=notifications&email_token=ABFUYEKZJCYD56BAWSVKJVDP7ZHUTA5CNFSM4IEIX4JKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7S7FGA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFUYENFLGA5A75BFXR4QOLP7ZHUTANCNFSM4IEIX4JA
.

It is an issue when CARLA is imported before tensorflow. Just need to be careful  about the import order.

i also have this problem, how did you solve it? @varunjammula


Queued Notifications with Dependency Injection can have problems with Serialization

Laravel Version: 5.8.9
PHP Version: 7.3.3
Database Driver & Version: SQLite 3

Description:
Queued Notifications that use Dependency Injection can not be serialized when there is a Service or Class with Closures.
This will make the queue job fail miserably, with PHP saying "Serialization of Closure is not allowed"
Steps To Reproduce:
Create a Queueable Notification with a Service containing a Closure, like the UrlGenerator that ships with Laravel.
namespace App\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Routing\UrlGenerator;
use Illuminate\Notifications\Notification;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;

class UnserializableNotification extends Notification implements ShouldQueue
{
    use Queueable;

    /**
     * The URL Generator, that uses Closures.
     *
     * @var UrlGenerator
     */
    protected $url;

    /**
     * LoginAttemptNotification constructor.
     *
     * @param bool $remember
     * @param string|null $intended
     * @param Config $config
     * @param Url $url
     */
    public function __construct(UrlGenerator $url)
    {
        $this->url = $url;
    }

    /**
     * Get the notification delivery channels.
     *
     * @param mixed $notifiable
     * @return array
     */
    public function via($notifiable)
    {
        return ['mail'];
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param mixed $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)
    {
        return (new MailMessage)
            ->greeting('If you see this...'))
            ->line('... some magic worked because this is not serializable');
    }
}

I suspect this happens being done because the Queue creates the object payload using the serialize() function, and the Notification Class doesn't know how to deal with serializing instances that include Closures, meaning, using Dependency Injection in the __construct can become very troublesome.
Are we bound to use app() inside the notification for better certainty of not crashing the queue job?

Definitely can't serialize closures. It's a PHP shortcoming. It's not just Notifications and it's not just laravel.

@devcircus - there are packages like https://github.com/opis/closure that do allow for it.
But is this a bug, or a framework feature request? I'll let Dries decide...

Definitely not a bug. It's been discussed many many times. It would be a great improvement.
Same with caching routes with a closure. It rears it's head in many places.

I think it's indeed best that we discuss this in ideas further to tackle all the concerns before proceeding with this.


Duplicate comment section

Describe the bug
A clear and concise description of what the bug is.
i can see two utterances comment section below the post
To Reproduce
Steps to reproduce the behavior:

open a post page, scroll down to the comment section.
navigate to another page, say About, then scroll down.
two utterances are there.

Expected behavior
A clear and concise description of what you expected to happen.
one comment section
Screenshots
If applicable, add screenshots to help explain your problem.

Platform:
macOS firefox/safari

OS: [e.g. iOS, Android, macOS]
Browser [e.g. chrome, safari, firefox]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

No worries, I'll fix it tomorrow.

btw, i think there is too much blank space after the comment section.

Bug fixed (I have no idea what I was doing)


Reduce defect/enhancement/task confidence threshold to 80%

Coverage remained the same at 25.73% when pulling 569b004c88c830277902858ea53af036af281f23 on marco-c:reduce_confidence_threshold into fd1011b16ed84d4887ea0b8041165152dded1d57 on mozilla:master.


Repo sync for protected CLA branch

The pull request is created from main637855886740266921 to main to fix git push error for protected CLA branch

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


Not Indexing full svg path while making model fromSVGPathData

Trying to make model with simple below svg path
M249.286 106.372L471.068 327.86
H471.068 27.505 27.505 L249.286<PHONE_NUMBER>0000001Z

This is the preview of svg path.

I am making model with bolow code.
const svgPath=`M249.286 106.372L471.068 327.86
H471.068 27.505 27.505 L249.286<PHONE_NUMBER>0000001Z`

const makerModel=makerjs.importer.fromSVGPathData(svgPath);

Due to some reason it is not indexing some of lines.
const svg=makerjs.exporter.toSVG(makerModel);
Response of final svg becomes like below

Final svg output
<svg width="221.782" height="221.488" viewBox="0 0 221.782 221.488" xmlns="http://www.w3.org/2000/svg"><g id="svgGroup" stroke-linecap="round" fill-rule="evenodd" font-size="9pt" stroke="#000" stroke-width="0.25mm" fill="none" style="stroke:#000;stroke-width:0.25mm;fill:none"><path d="M 0 0 L 221.782 221.488 L 0 0 Z" vector-effect="non-scaling-stroke"/></g></svg>
It should give same output as input.
Code reproduced link
### Tasks

@danmarshall
Is this something that MakerJs is designed to handle, or could it potentially be considered for inclusion in a future release?
Your insights would be greatly appreciated. Thank you!

The parser code probably isn't robust enough to handle this case. PRs are welcome.
Meanwhile, if you check, then uncheck the "Minify output" button in the svg path inspector you can get a standardized output:
M 249.286 106.372 L 471.068 327.86 H 471.068 H 27.505 H 27.505 L 249.286 106.372 Z


Allow adding/removing of queues of a user

Hello,
at the moment, in order to add user to a queue one needs to state all queues the user is in. When a user has multiple queues, the command gets excessively long:
elis> user change -q 53 -q 4142 -q 4143 -q 4144 -q 4145 -q 4146 -q 4147 -q 9037 -q 14147 -q 14148 -q 16065 -q 16068 -q 16070 -q 16804 -q 16814 -q 20099 -q 21332 -q 21334 -q 21335 -q 21337 5632

Suggested solution:
create a command elisctl user queues, which would allow for add, remove and list subcommands.

related to https://github.com/rossumai/elisctl/issues/77


Suggest that you document the data interface

...it has required some software archeology to figure out how this interface works.  A small amount of documentation that describes how to add new data sources would be greatly helpful.
Thanks!
Mark

I agree! :+1:


[Bug]: bulkWalkCmd does not return the last value before the MIB end

Expected behavior
The hlapi.asyncio.bulkWalkCmd should return all values.
Actual behavior
The last value of the MIB is not returned.
Detailed steps
Perform a hlapi.asyncio.bulkWalkCmd to the whole MIB.
Expected result: All values are returned
Current result: The last value before the MIB end is missing
When using hlapi.asyncio.walkCmd all values are returned.
Python package information
6.0.9
Operating system information
Linux
Python information
3.11.8
(Optional) Contents of your test script
No response
Relevant log output
No response

Thanks for the report.
Can you verify the fix in release 6.0.10 works for you?

The fix works, thank you.


Vertical alignment is not consistent across file types

I have my line-height fairly small in the tree view (20px), so this is more noticeable, I think, than the default layout.  Notice that in the screenshots below, the .jsx file's text is bottom-aligned, while the .html file is properly middle-aligned.  This roughly is caused by the different font-sizes of the icons.

Bit more to it than that, I'm afraid. There's actually a myriad of different font families, each of which are using noticeably different sizes for each glyph. The manually-set font-sizes are there to try normalising the size discrepancies between the different fonts.
So, short of manually pick through every font we're using and compile them into one neatly-organised superfont (which... I really wouldn't recommend for all the obvious reasons), we're reduced to this constant tennis match batting line-heights and font-sizes around. :-(

@zyklus This will be resolved in V2, hopefully. Please subscribe to the relevant task-list for updates.


Task is overwriting gradlew file, dirtying the repository

Question, Bug, or Feature?
Type: Bug
Enter Task Name: Gradle
Environment

Server - Azure Pipelines or TFS on-premises?

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Organization: TheAvantGardeGroup
Project: AML Controlli
Build definition: AML Controlli-BE-Development.
Build ID: 20220110.9

Agent - Hosted or Private:

Hosted, but I can't find the queue name
Issue Description
Gradle task overwrites gradlew file, resulting in a dirty repository that reckon refuses to tag.
I have a complex build pipeline made using templates, detailed in a subsequent post, where the complexity is mainly about parameterization and reuse of same pipeline.
It can be summarized as

Login to Maven
Login to ACR
Prepare SonarCloud
Run Gradle (clean publish)

clean
build Java
build Test
run JUnit
run Jib (build Docker)
Tag the repository with reckon
Push

Publish SonarCloud analysis
Publish coverage reports and unit tests reports
Logout from ACR

When Gradle runs, it executes the tasks descripted at 2nd level. My build file is fairly complex, so I'll be providing log
The expected result is that Gradle never touches the repository. Assuming the gitignores are set up correctly, Gradle should build in the ignored folder(s) and keep repository clean. This happens locally, as reckon can tag the repository
            - task: Gradle@3
              displayName: Publish with Gradle
              inputs:
                gradleWrapperFile: gradlew
                gradleOptions: -Xmx10240m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8
                options: -Pci=true -PbuildId=$(Build.BuildId) -Preckon.stage=FINAL -Preckon.scope=${{parameters.versionScope}}
                jdkVersionOption: 1.11
                jdkArchitectureOption: x64
                publishJUnitResults: true
                sqAnalysisEnabled: true
                sqGradlePluginVersionChoice: specify
                sqGradlePluginVersion: 3.2.0
                testResultsFiles: '$(System.DefaultWorkingDirectory)/build/test-results/**/TEST-*.xml'
                tasks: clean publish
              env:
                IT_ORBIT_FILE_REPOSITORY_AZURE_STORAGE_CONNECTION_STRING: ${{variables.testStorageAccountConnectionString}}

What happens is that:

Reckon Gradle plugin complains about repository in dirty state, refuses to work, the pipeline fails
I have investigated with a simple git status command and looks like gradlew script is modified

Note: the error in the pipeline execution is a feature of Reckon Gradle Plugin, but shouldn't be triggered
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
logs_6074.zip
Troubleshooting
I have "wrapped" the above Gradle task with a couple of git status commands. After the Gradle task, looks like gradlew script is modified.
This is a problem in my scenario.
Locally, the gradlew script is never overwritten.
I guess this is a problem with Azure DevOps Gradle task
Error logs
Downloading https://services.gradle.org/distributions/gradle-7.3.3-bin.zip
...........10%...........20%...........30%...........40%...........50%...........60%...........70%...........80%...........90%...........100%

Welcome to Gradle 7.3.3!

Here are the highlights of this release:
 - Easily declare new test suites in Java projects
 - Support for Java 17
 - Support for Scala 3

For more details see https://docs.gradle.org/7.3.3/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

FAILURE: Build failed with an exception.

* Where:
Script '/home/vsts/work/1/s/gradle/jib.gradle' line: 13

* What went wrong:
A problem occurred evaluating script.
> Cannot release a final or significant stage without a clean repo.

* 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

BUILD FAILED in 1m 4s
Error: The process '/home/vsts/work/1/s/gradlew' failed with exit code 1

Full azure-pipelines.yaml
parameters:
  # true to publish a Docker artifact to the Registry, false to keep the build local
  - name: publish
    type: boolean
    #
  - name: versionScope
    type: string
    default: patch
    values:
      - major
      - minor
      - patch
  - name: versionStage
    type: string
    default: SNAPSHOT
    values:
      - SNAPSHOT
      - FINAL

stages:
  - stage: Build
    variables:
      # Build
      vstsFeed: ....
      buildArtifactName: aml-controlli-be

      # SonarCloud parameters
      sonarCloudOrganization: regtechopenproject
      sonarCloudProjectKey: TheAvantGardeGroup_aml-controlli-be
      sonarCloudProjectName: aml-controlli-be

    jobs:
      - job: Build
        displayName: Build application

        pool:
          vmImage: ubuntu-20.04
        steps:

          - checkout: self
            persistCredentials: true

          - task: MavenAuthenticate@0
            inputs:
              artifactsFeeds: 'AVG_RTOP_Feed'

              # Docker ACR Login
          - task: Docker@2
            displayName: Login to ACR
            inputs:
              command: login
              containerRegistry: AVG Orbit Registry

            # SonarCloud Prepare Analysis
          - task: SonarCloudPrepare@1
            displayName: SonarCloud Prepare Analysis
            inputs:
              SonarCloud: 'SonarCloud'
              organization: $(sonarCloudOrganization)
              scannerMode: 'Other'
              extraProperties: |
                # Additional properties that will be passed to the scanner,
                # Put one key=value per line, example:
                sonar.exclusions=**/*.bin, build/**, build-extra/**
                sonar.projectKey=$(sonarCloudProjectKey)
                sonar.projectName=$(sonarCloudProjectName)

          - script: git status
            displayName: Git status
            condition: succeededOrFailed()

          - ${{ if eq(parameters.publish, false) }}:

              - task: Gradle@3
                displayName: Build with Gradle
                inputs:
                  gradleWrapperFile: gradlew
                  gradleOptions: -Xmx10240m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8
                  options: -Pci=true -PbuildId=$(Build.BuildId) -Preckon.stage=${{parameters.versionStage}} -Preckon.scope=${{parameters.versionScope}}
                  jdkVersionOption: 1.11
                  jdkArchitectureOption: x64
                  publishJUnitResults: true
                  sqAnalysisEnabled: true
                  sqGradlePluginVersionChoice: specify
                  sqGradlePluginVersion: 3.2.0
                  testResultsFiles: '$(System.DefaultWorkingDirectory)/build/test-results/**/TEST-*.xml'
                  tasks: clean build
                env:
                  IT_ORBIT_FILE_REPOSITORY_AZURE_STORAGE_CONNECTION_STRING: ${{variables.testStorageAccountConnectionString}}

          - ${{ if eq(parameters.publish, true) }}:

              - task: Gradle@3
                displayName: Publish with Gradle
                inputs:
                  gradleWrapperFile: gradlew
                  gradleOptions: -Xmx10240m -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8
                  options: -Pci=true -PbuildId=$(Build.BuildId) -Preckon.stage=${{parameters.versionStage}} -Preckon.scope=${{parameters.versionScope}}
                  jdkVersionOption: 1.11
                  jdkArchitectureOption: x64
                  publishJUnitResults: true
                  sqAnalysisEnabled: true
                  sqGradlePluginVersionChoice: specify
                  sqGradlePluginVersion: 3.2.0
                  testResultsFiles: '$(System.DefaultWorkingDirectory)/build/test-results/**/TEST-*.xml'
                  tasks: clean publish
                env:
                  IT_ORBIT_FILE_REPOSITORY_AZURE_STORAGE_CONNECTION_STRING: ${{variables.testStorageAccountConnectionString}}

          - script: git status
            displayName: Git status
            condition: succeededOrFailed()

          - task: SonarCloudPublish@1
            displayName: Publish SonarCloud Quality Gate
            inputs:
              pollingTimeoutSec: '300'

          # Publish Artifacts

          - task: PublishCodeCoverageResults@1
            displayName: Publish code coverage results (test)
            condition: succeededOrFailed()
            inputs:
              summaryFileLocation: '$(System.DefaultWorkingDirectory)/build/reports/jacoco/test/*.xml'
              pathToSources: $(System.DefaultWorkingDirectory)/src/main/java
              failIfCoverageEmpty: false

          # Finally logout from Docker
          - task: Docker@2
            condition: succeededOrFailed()
            displayName: Logout from ACR
            inputs:
              command: logout
              containerRegistry: AVG Orbit Registry

Hi @djechelon could you please clarify which files has been changed? There is a general error 'The process '/home/vsts/work/1/s/gradlew' failed with exit code 1', but as I see there are no mentions of the file changed...
Is there some report generated probably to check it?

Hi @anatolybolshakov, the error is displayed in the Gradle log. Except for the line number, which is attributed to the jib.gradle file, the error is
2022-01-10T11:11:09.5668543Z FAILURE: Build failed with an exception.
2022-01-10T11:11:09.5671608Z
2022-01-10T11:11:09.5672275Z * Where:
2022-01-10T11:11:09.5673538Z Script '/home/vsts/work/1/s/gradle/jib.gradle' line: 13
2022-01-10T11:11:09.5675614Z
2022-01-10T11:11:09.5676146Z * What went wrong:
2022-01-10T11:11:09.5683930Z A problem occurred evaluating script.
2022-01-10T11:11:09.5685007Z > Cannot release a final or significant stage without a clean repo.
2022-01-10T11:11:09.5690336Z

Cannot release a final or significant stage without a clean repo.
To better understand this, we need to inspect my Gradle pipeline. To summarize, I use a plugin named reckon, which I already linked, that creates git tags when publishing the artifact. As you see, I opened an issue to the project itself, but let's focus here.
If I run gradle clean publish locally, Reckon plugin will generate a git tag after all the previous steps are done. On Azure Pipelines, Reckon complains that the repository is in a dirty state, which can't be reproduced locally. This is a design choice of the project, apparently. The plugin won't generate a tag for a stable version until the repository is fully committed.
So I tried to investigate more myself, and added another tasks who will check for git status at the next step. Step 8, to be exact
Git status reports that
2022-01-10T11:11:10.2311801Z HEAD detached at f80aab9
2022-01-10T11:11:10.2312701Z Changes not staged for commit:
2022-01-10T11:11:10.2313337Z   (use "git add <file>..." to update what will be committed)
2022-01-10T11:11:10.2313961Z   (use "git restore <file>..." to discard changes in working directory)
2022-01-10T11:11:10.2314513Z 	modified:   gradlew

Interesting is the step 6, which is another identical git status step. It reports that the repository is clean.
From this information, steps 6 to 8, I have evidence that somewhere during the Gradle task the gradlew file is modified. This concludes that I have a problem using the Reckon plugin, which by design doesn't allow a dirty repository.
However, this doesn't prove that Azure Gradle task is overwriting my gradlew script. It could be somewhere during the Gradle execution.
Please believe me that I don't run gradle wrapper as part of the pipeline.
Please understand that I am not sharing my build.gradle script only to save both from a few dozen minutes of explanations (me) and reading (you), we don't have such national security secrets in Gradle, I have not invented a super-unpatented algorithm I want to keep away from foreign eyes. No, while I could paste it at once, I am particularly lazy in answering for clarifications. And there is no code reference to gradlew in my Gradle build file.
It could be possible to reproduce the problem in a smaller repository, something I could push both to my Devops private organization and Github. A super-simple Spring Boot Hello project with the two git status tasks.
But I hope I have provided enough information for now to continue investigation. I am confident that the logs I have provided reveal useful information for maintainers.
Thanks for your precious help.

Further investigation
This could be a Linux-Windows problem with Gradle, Git and reckon plugin, and not with Azure DevOps itself.
In the source code of the Gradle script from Azure DevOps, I have found the above code that made me suspicious
https://github.com/microsoft/azure-pipelines-tasks/blob/3547164d2de4506d296b0242ba66c0d84adaf904/Tasks/GradleV3/gradletask.ts#L124-L140
In the code shown above, the runner ensures that gradlew file is executable by forcing a chmod. This is correct, as people committing only from Windows won't ever notice that their gradlew (sh) file is not executable. Windows doesn't support execution attribute, and Gradle for Windows relies on gradlew.bat, which happens exactly to be what I run from Powershell.
If you get a single developer running Linux in your team, you'll spot this. Also, depending on who of us created the test repository first, the problem could be discovered or not. I could have created the test repository under Windows and reproduce the problem, you could be running on Linux, create a test repository and reply "what the hell djechelon, I can't reproduce", and you'll be damn correct!!
So, setting the chmod looks like sufficient to trigger a Git change in the working copy, and sufficient for reckon plugin to detect. This because I created the gradlew under Windows, which doesn't set any particular attribute. Creating the wrapper the first time in Linux sets the correct permissions.
Gradle task for Azure DevOps doesn't apparently overwrite the gradlew file, but is touching it.
Solution found from the usual place is to run git update-index --chmod=+x <file> and commit. At this point, chmodSync won't have any effect.
Conclusion
This is not a problem with Azure DevOps and for this reason the issue might be closed. However, please appreciate the effort put into this investigation and its findings. This was critical for our pipeline process unless our team wanted to fork the reckon plugin project.
I would have suggested to amend the code in the Gradle task in order to warn about the chmod if the file has insufficient permissions. Azure DevOps could check for the chmod and warn the user "we are chmodding the gradlew file to make it executable", however I couldn't find a Nodejs function to synchronously check for executable flag.

@djechelon thanks for investigation! Let us leave this open for the chmod warning then - to give heads up on this in pipeline logs.


Provide dependency info in each kamelet's catalog page

Kamelets are now open for usage in standalone, but when we try to use them in a standalone app without some automatic dependency resolver like camel-kamelet-main it's cumbersome to know which extra component dependencies we need to put in pom.xml.
I think it's useful to provide a section in each catalog page that tells which components the kamelet uses. Does it make sense?
If it makes sense I can work on it.

Sure, it makes sense. You can also think to add specific sections about using them in particular environments (jbang?).

I now noticed that it's fixed months ago. Thanks @oscerd !


Don't include leading zeros on numbers

Is there much utility to this?

Yes, it's fails when u use the sass-linter

Closing for the same reasons as https://github.com/twbs/bootstrap-sass/pull/1013#issuecomment-183412699 (unless I'm really misunderstanding how the Sass port works), without taking any opinion on pros/cons of leading zeros.


Rename memory settings

Fixes #167

Deprecate max_mem in favour of allowed_mem, which now also includes reserved mem.
Replace required_mem with projected_mem, which now also includes reserved mem.
Update the documentation and add a new diagram showing the relationship between memory settings.

I'd like to merge this soon unless you have any objections @TomNicholas

Looks good to me! Really like the new diagram.