process.cwd() gives wrong path

I have opened a StackOverflow question too. Initially I thought it was weird issue with node or I was doing something wrong. But then when I did the same with vanilla javascript I did not run into the issue.
I am trying to create a simple module called creator that creates a folder layout with help of mkdirp. This is my first time writing a reusable package in node. Thought I'd give it a try with webpack too. For now, just for this issue, I am logging __dirname and process.cwd() from the creator inside a function called create. When I run the module alone, I get the correct paths:
export function createLayout(config) {
  console.log(__dirname, process.cwd());
}
// E:\projects\creator E:\projects\creator

When I do an npm link to another project called consumer, and call this function there I get '/'.
var createLayout = require('creator').createLayout;
createLayout();
// '/' '/'

Which results in various issues I mentioned in stackoverflow.
However if I write the export file in simple js ...
module.exports = {
  createLayout: function() {
    console.log(__dirname, process.cwd());
  }
}

I get the following log:
E:\projects\creator\dist E:\projects\consumer

Here's my webpack.config for the creator:
module.exports = {
  entry: [
    './index.js'
  ],
  output: {
    filename: 'creator.js',
    path: __dirname + '/dist',
    library: 'creator',
    libraryTarget: 'umd'
  },
  module: {
    loaders: [
      {
        test: /\.js$/,
        exclude: /node_modules/,
        loader: 'babel'
      }
    ]
  },
  externals: {
    fs: 'fs'
  }
};

Please help me in understanding what I am doing wrong here. Thanks

You may be setting the path twice. __dirname is a nodejs reference to your project path, and while you load your files, you use your function, and webpack takes this and does something like /__dirname/dist/__dirname/__dirname (proccess.cwd() ). That's my impression :smiley:

So: Try change your entry point where __dirname is, and then your function to be [name]/[id]/[__dirname] in your output without invoking it through your console.log

@ev1stensberg Thanks for you reply, your solution works, and another solution from the stackoverflow also worked: setting target: 'node' in config. I am closing this issue as resolved.
Thanks to all of you.


Fix floating point exception with tf.unravel_index

This PR tries to address the issue raised in #40204 where
tf.unravel_index caused floating point exception when any one
element is 0 in dims.
The issue is that indices in tf.unravel_index should make sure
it is not out of boundary compared to dims.
This PR fixes the issue by adding a check before hand, though Eigen.
This PR fixes #40204.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Thanks @ezhulenev for the review. The PR has been updated. Please take a look.


Uncaught exception when trying to remove pagination of deleted message

When a message with pagination is removed before the bot tries to remove the pagination reactions, an exception will be raised and displayed in channel:

To reproduce this, issue the !tags command and remove the message before the bot removes the pagination.
Although this will be a rare event, since it needs moderator removal, we probably want to suppress it anyway so it doesn't show up like above in public channels.

I think this issued is fixed, or at least I did achieve to reproduce it, either by using !helps or !tags commands.

I've verified just now that the bug is still present.

Duplicate of #770 and fixed in #771.


Axis colour is not correct

One of the axis colours is attached with the text_color_(not axis_color_). Is it intended?
https://github.com/leonardvandriel/cvplot/blob/e997368c6a3963159b8ae12dfa2c0f3c04d46083/src/cvplot/figure.cc#L650

Thank you for reporting this issue. It is fixed in #32


November month being detect as december by typo

Hi there,
On the follow line about the november month, I saw that has a dicember lost inside. Is it maybe a typo?
https://github.com/kermitt2/grobid/blob/8e4ce03fc91a75d0185fd61f3a480444df21933e/grobid-core/src/main/java/org/grobid/core/engines/DateParser.java#L206

Hello!
Ah yes it's a typo sorry!
Lesson learnt: never write any manual rules, create a classifier for everything ;)
thanks for reporting.

so fixed with commit 2d3e11eafcf7e5768ff48681b6af757dc4b4e84a
thanks again @rlucca !

I close this issue given the committed fix.


java logo request

Icon Request

Icon name:  java logo
Tags of this icon:
Use case:
Screenshots of similar icons:

@NikhilM01 The icon you proposed has been completed and launched. Hope you like this project and Star our repo to support our efforts.


[11.x] Improve model make command

This PR changes the -p flag to generate a policy instead of making a pivot model.
Currently the make:model command accepts various short flags to create model related files: -f, -m, -r, etc., so using the -p for the policy generation would improve consistency.
For me, it's a constant struggle, because I always run the make:model -f -m -p command and as a result I get a pivot model instead of a policy which was not my intention.
Also, the pivot generation would be consitent as well: --pivot and --morph-pivot with no short options.

I think it's fine.


Automatically run cloud tests

We don't run the cloud tests on every check-in because they take too long, but we should be running them automatically on some schedule so we catch bugs early rather than late.

I started setting this up but never turned it on because they kept failing for me.  I will resurrect the branch.

Someone should take this on -- this is becoming urgent


Install rsync before guest.pull()

@mruprich reported that after running installability test the tmt
fails on missing rsync when pulling out artifacts.
After investiation, it turned out, installability for rsync removes
rsync from the test machine, but tmt still expects it to be around.
This tries to fix the problem and make sure rsync is still installed
on the machine before trying to pull out the artifacts.
Todo:

[ ] review comments
[ ] fix the issues with current tests
[ ] add a new test covering the problem

Signed-off-by: Miroslav Vadkerti<EMAIL_ADDRESS>

Not sure whether this check should go to the internal executor code. Should we include this rather under pull() and push() methods? We already have one similar code here:
https://github.com/psss/tmt/blob/4630cc7d573abb4bb814ae796b030fd91ccd8542/tmt/steps/prepare/__init__.py#L101

@psss ack, sounds like a better place, will move it there

we  can try again the review @psss @lukaszachy

/packit test

Looks like there is a syntax error:
Sorry: IndentationError: expected an indented block (__init__.py, line 18)

@thrix, please, have a look.

wtf, what does this autopep8 module do? :D

LGTM, let's hit it


Doesn't get anything on iOS

Hello,
Although the code at https://jsfiddle.net/kartagis/pjdqp3ku/6/ loads up the images instantly on Android, it stalls at ActivityIndicator.show() on iOS, resulting in not showing any images.

Should be permission issue.
If a call to getLibrary returns permission error, you should call requestAuthorization.
Something like this:
cordova.plugins.photoLibrary.getLibrary(
  function (library) { },
  function (err) {
    if (err.startsWith('Permission')) {
      // call requestAuthorization, and retry
    }
    // Handle error - it's not permission-related
  }
);

Isn't this what --variable PHOTOLIBRARY_USAGE_DESCRIPTION='' does?
On 26 January 2017 at 11:08, viskin<EMAIL_ADDRESS>wrote:

Should be permission issue.
If a call to getLibrary returns permission error, you should call
requestAuthorization.
Something like this:
cordova.plugins.photoLibrary.getLibrary(
function (library) { },
function (err) {
if (err.startsWith('Permission')) {
// call requestAuthorization, and retry
}
// Handle error - it's not permission-related
}
);
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/terikon/cordova-plugin-photo-library/issues/35#issuecomment-275329291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABiHXnBXcuREdzPPNMtxopao1i0fkxgVks5rWFR-gaJpZM4LtQDV
.

--

@kartagis all that does is set the message that will be shown on permission prompt. I havent tried this plugin yet but I think @viskin is right. Try querying for permission again.

@viskin @cozzbie aye, it was permission issue. After I use requestAuthorization, how do retry reading the library? getLibrary again?

I tried requestAuthorization, and called getLibrary in the callback, but it's way too slow on iOS.

Wow, 20sec...

Is it time getLibrary method runs, or also retrieving thumbnails?
How many photos are on your device?

1- It's only the time getLibrary runs
2- There are 1185 photos on my device
On Fri, 27 Jan 2017 at 21:29, viskin<EMAIL_ADDRESS>wrote:

Wow, 20sec...

Is it time getLibrary method runs, or also retrieving thumbnails?
How many photos are on your device?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terikon/cordova-plugin-photo-library/issues/35#issuecomment-275737959,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABiHXiIOHn4NFNe7yBr3DPhMw4sdmw4Kks5rWjedgaJpZM4LtQDV
.

--

Moving to the other issue, as this is no more relevant


Incorrect Canonical URL tag

When selecting the Blog as the main page the canonical tag is assigned incorrectly, I have been able to check this by checking the source code of my website, this is the tag that applies: <link rel="canonical" href="https://site.com/all">
If I change the home page to the Tags or FoF Pages the canonical tag is correct, applying: <link rel="canonical" href="https://site.com/">

I'd like to vote up this issue.


BAU: Correct instructions for validating the ID token

Why
We do not generate an auth_time claim, and never have.
What
Replace the incorrect instructions about auth_time with correct ones about iat
Technical writer support
How to review

NB at time of merge CI check:
https://github.com/govuk-one-login/authentication-tech-docs/actions/runs/8068447634/job/22041243084?pr=179
Was failing. This is unrelated to this change and I've confirmed we're introducing no dead links here.


Add web Icon

Add power_settings_new Icon (again?)

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[x] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[x] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] All new and existing tests passed

Description

Add new Icon (web) from materials UI using the Icon generator
Also add the power_settings_new again (added by #37 ) again to icongen.config.js, because it was missing there

Please update the screenshot regarding the icons.
@kotielnikov can we exclude this particular snapshot? This is a quite moving part. (We could also keep it and include it in the icongen).

"npm run build:demo" will be enough to update the screenshots?

@FlorianRappl no we can't. it tests all the components. I'm going to replace the library with other more simple and faster solution later.
to update screenshots now:
npm run build:demo
npm run test:visual
npm run test:visual-update

But again, I will improve the process soon

It was the wrong icon, so I will close this PR and open another with the correct icon and updated screenshots.


Update simple example and README.

Codecov Report

Merging #79 (a2d9604) into master (780a4df) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a2d9604 differs from pull request most recent head abb3a5f. Consider uploading reports for the commit abb3a5f to get more accurate results

@@           Coverage Diff           @@
##           master      #79   +/-   ##
=======================================
  Coverage   94.64%   94.64%
=======================================
  Files           1        1
  Lines         168      168
  Branches       31       31
=======================================
  Hits          159      159
  Misses          5        5
  Partials        4        4

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Porting to Godot 4

Trying to develop a plan for Godot 4.

Heyo, just to keep track of this here as well:
https://github.com/CaptainProton42/GodotHoloPlayNative/commit/9cf76cd3592b5ceb036ccd29264eea5f0826851d moved a lot of the rendering that was previsously done in OpenGL using framebuffers to the Godot VisualServer (using Viewports and TextureRects).
This means that the main things that are being done "outside" of Godot rendering-wise are creating a separate window (using GLFW) and compiling and running the ligthfield shader from HoloPlayCore (using OpenGL and some native Windows calls such as wglMakeCurrent to draw to the window).
I haven't looked at Godot 4 yet that closely but I've heard that it'll have support for multiple windows so this part could be moved to Godot as well. Compiling the lightfield shader in-engine would require support for GLSL shaders in Godot 4 though.

I have verified that this demo works in Godot Engine https://github.com/fire/looking-glass/.
It's a shader that converts 45 images to Looking Glass Format.
I wasn't able to solve the 45 images generation problem. It's blocking me.

https://github.com/fire/looking-glass/blob/master/godot_demo/looking_glass.gdshader
Is this the shader file? That's really cool. How did you figure the shader out?
I'm a bit afraid of including a plain text lightfield shader due to licensing which means I hope to be able to import the GLSL code from the DLL in Godot 4.
Regarding generating the 45 views: I used the VisualServer to create the 45 cameras (or less for most settings since 45 views are pretty heavy) and viewports and let the engine do the rest. You could then use the Viewport textures in your shader. PROJECTION_FRUSTUM works really well for the cameras as it doesn't distort the focal plane.

It was derived from https://github.com/fire/looking-glass/blob/master/linquilt2native.py

Can you provide some sample code that does this and outputs to my quilt inputs?
I have trouble converting the 45 images to a ImageArray2d, so being able to match my inputs is really helpful.

The second bit usb access to:

get_calibration_from_eeprom.py: Get the display calibration values (in the form of a JSON string) from a Looking Glass.

Can you provide some sample code that does this and outputs to my quilt inputs?
I have trouble converting the 45 images to a ImageArray2d, so being able to match my inputs is really helpful.

This is the code in the GDNative library that creates the cameras/viewports:
https://github.com/CaptainProton42/GodotHoloPlayNative/blob/1b1de0d24cc414e7409779a9621313349ff2ce86/src/godot_holoplay/HoloPlayVolume.cpp#L476-L508
I'll try to outline it a bit: Using VisualServer, I create up to 45 cameras, viewports and texture rects. Each camera renders a view to a viewport whose texture is displayed on a texture rect. By adding all texture rects in a grid to a parent canvas item I essentially create the quilt where each view is updated by one camera/viewport. The parent canvas item is also attached to a viewport. The texture of this root viewport will be the quilt you want.
There is also the update_cameras() function which sets the projections for all cameras:
https://github.com/CaptainProton42/GodotHoloPlayNative/blob/1b1de0d24cc414e7409779a9621313349ff2ce86/src/godot_holoplay/HoloPlayVolume.cpp#L513-L543
This article was really helpful for getting the correct projections: https://docs.lookingglassfactory.com/keyconcepts/camera

The second part is usb access to:

get_calibration_from_eeprom.py: Get the display calibration values (in the form of a JSON string) from a Looking Glass.

I think the easiest solution to this is just writing a wrapper for the HoloPlayCore SDK in GDNative. I'll probably try to do that this week as first step of getting it to run in Godot 4.

Godot 4 development has been moved to https://github.com/CaptainProton42/GodotHoloPlayExtension using the new extension system.


feat: forward amount u64 on contract call

Summary
Enable forwarding amount of size up to u64 in contract calls.
Motivation
Before, we were forcing the amount to be an Immediate12, which doesn't work for many use cases where we would need a u64. Passing a bigger number to the amount being forwarded would cause an overflow.
closes: #174

Can you provide some more context of what this changes is supposed to be for?

Can you provide some more context of what this changes is supposed to be for?

@adlerjohn  Just updated the description to do that!


17.03 user $PATH doesn't have /run/wrappers/bin, sudo fails

Issue description
I have switched to NixOS 17.03, and sudo no longer works. I have rebooted.
# ollie at nixos-desktop in ~ [15:23:22]
→ nixos-version                                                                                ~
17.03beta501.acb7958 (Gorilla)

# ollie at nixos-desktop in ~ [15:24:29]
→ ls -al /run/booted-system                                                                                ~
lrwxrwxrwx 1 root root 91 Mar 22 15:17 /run/booted-system -> /nix/store/7h5fcyzrksna72v33432zzargfbh76ai-nixos-system-nixos-desktop-17.03beta501.acb7958

# ollie at nixos-desktop in ~ [15:24:33]
→ sudo                                                                                ~
sudo: /run/current-system/sw/bin/sudo must be owned by uid 0 and have the setuid bit set

# ollie at nixos-desktop in ~ [15:24:34]
→ /run/wrappers/bin/sudo -i                                                                                ~

[root@nixos-desktop:~]# logout

# ollie at nixos-desktop in ~ [15:24:43]
→ echo $PATH                                                                                ~
/home/ollie/bin:/var/setuid-wrappers:/home/ollie/.nix-profile/bin:/home/ollie/.nix-profile/sbin:/home/ollie/.nix-profile/lib/kde4/libexec:/nix/var/nix/profiles/default/bin:/nix/var/nix/profiles/default/sbin:/nix/var/nix/profiles/default/lib/kde4/libexec:/run/current-system/sw/bin:/run/current-system/sw/sbin:/run/current-system/sw/lib/kde4/libexec

Let me know what else is useful.

I'm currently working on fixing this, the previous fix was not enough.

I'm currently working on fixing this, the previous fix was not enough.

Fixed in e82baf043e25abbb354c7ab4415a40a4155df398

This still doesn't work for me:
# ollie at nixos-desktop in ~ [14:59:47]
→ nixos-version                                                                                ~
17.03.889.9626bc7db7 (Gorilla)

# ollie at nixos-desktop in ~ [15:00:08]
→ ls -l /run/booted-system                                                                                ~
lrwxrwxrwx 1 root root 91 Apr  4 14:55 /run/booted-system -> /nix/store/kq0mlmgh939li6l5jhicj1xjhky2b7kr-nixos-system-nixos-desktop-17.03.889.9626bc7db7

# ollie at nixos-desktop in ~ [15:00:13]
→ ls -l /run/current-system                                                                                ~
lrwxrwxrwx 1 root root 91 Apr  4 14:59 /run/current-system -> /nix/store/9avb0phhk79zg865b1g5wml81y7ihl91-nixos-system-nixos-desktop-17.03.889.9626bc7db7

# ollie at nixos-desktop in ~ [15:00:14]
→ sudo                                                                                ~
sudo: /run/current-system/sw/bin/sudo must be owned by uid 0 and have the setuid bit set

# ollie at nixos-desktop in ~ [15:00:16]
→ /run/wrappers/bin/sudo                                                                                ~
usage: sudo -h | -K | -k | -V
usage: sudo -v [-AknS] [-g group] [-h host] [-p prompt] [-u user]
usage: sudo -l [-AknS] [-g group] [-h host] [-p prompt] [-U user] [-u user] [command]
usage: sudo [-AbEHknPS] [-C num] [-g group] [-h host] [-p prompt] [-u user] [VAR=value] [-i|-s] [<command>]
usage: sudo -e [-AknS] [-C num] [-g group] [-h host] [-p prompt] [-u user] file ...

Doh, I just found an export PATH in my .zshrc!


keymap not being set

Hello
I have another issue that I've noticed for a number of my tests but haven't really looked at it until now.
During customize, I use --plugin L10n:host but the keyboard on the new Pi isn't set to a British format as the host is (so i get a '#' where I should get '£', etc).  Pretty sure I had this on earlier versions running on Mint and I'm still getting it on 11.4 running on a Pi with Bookworm 32 bit.
I've pulled the following out from the logs, hope it helps...
2024-02-16 21:07:20 > Command Line: /usr/local/bin/sdm --customize --showapt --plugin L10n:host --plugin

2024-02-16 21:07:20 * Plugins selected:
2024-02-16 21:07:20    * L10n
2024-02-16 21:07:20        Args: host

2024-02-16 21:07:21 > Run Plugins Phase '0'
2024-02-16 21:07:21 > Run Plugin 'L10n' (/mnt/sdm/usr/local/sdm/plugins/L10n) Phase 0 with arguments: 'host'
2024-02-16 21:07:21 * Plugin L10n: Start Phase 0
2024-02-16 21:07:21 > Plugin L10n: Keys/values found:
2024-02-16 21:07:21    host:
2024-02-16 21:07:21 > Plugin L10n: Read L10n configuration from host system
2024-02-16 21:07:21 > Plugin L10n: Load Localization (L10N) settings from running system
2024-02-16 21:07:21 > Plugin L10n:   Keymap:       gb
2024-02-16 21:07:21 > Plugin L10n:   Locale:       en_GB.UTF-8
2024-02-16 21:07:21 > Plugin L10n:   Timezone:     Europe/London
2024-02-16 21:07:21 > Plugin L10n:   WiFi Country:
2024-02-16 21:07:21 > Plugin L10n: Save L10n configuration
2024-02-16 21:07:21 * Plugin L10n: Complete Phase 0

2024-02-16 21:07:52 > Run Plugins Phase '1'
2024-02-16 21:07:53 > Run Plugin 'L10n' (/usr/local/sdm/plugins/L10n) Phase 1 with arguments: 'host'
2024-02-16 21:07:53 * Plugin L10n: Start Phase 1
2024-02-16 21:07:54 % Plugin L10n: Keymap will be set during FirstBoot
2024-02-16 21:07:54 > Plugin L10n: Disable keyboard-setup service; will be re-enabled during FirstBoot
2024-02-16 21:07:55 > Plugin L10n: Set locale to 'en_GB.UTF-8'
2024-02-16 21:08:19 > Plugin L10n: Set timezone to 'Europe/London'
2024-02-16 21:08:21 * Plugin L10n: Complete Phase 1

2024-02-16 21:21:49 > Run Plugins Phase 'post-install'
2024-02-16 21:21:49 > Run Plugin 'L10n' (/usr/local/sdm/plugins/L10n) Phase post-install with arguments: 'host'
2024-02-16 21:21:49 > Run Plugin 'disables'

$  sudo journalctl -b -1 | grep FirstBoot
Dec 11 01:24:46 GatewayPi systemd[1]: systemd-firstboot.service - First Boot Wizard was skipped because of an unmet condition check (ConditionFirstBoot=yes).
Dec 11 01:24:46 GatewayPi systemd[1]: first-boot-complete.target - First Boot Complete was skipped because of an unmet condition check (ConditionFirstBoot=yes).
Dec 11 01:24:55 GatewayPi systemd[1]: Starting sdm-firstboot.service - sdm FirstBoot System Configuration...
Dec 11 01:24:55 GatewayPi systemd[1]: Started sdm-firstboot.service - sdm FirstBoot System Configuration.
Dec 11 01:24:55 GatewayPi root[670]: sdm FirstBoot: Start sdm FirstBoot Service
Dec 11 01:24:55 GatewayPi root[673]: sdm FirstBoot: Start Configuration with configuration file '/etc/sdm/auto-1piboot.conf'
Dec 11 01:24:55 GatewayPi root[676]: sdm FirstBoot: Configuration with configuration file '/etc/sdm/auto-1piboot.conf' Completed
Dec 11 01:24:55 GatewayPi root[679]: sdm FirstBoot: Set WiFi Country to 'GB'
Dec 11 01:25:12 GatewayPi root[976]: sdm FirstBoot: Waiting 10/120 seconds for system time synchronization
Feb 17 21:57:49 GatewayPi root[1092]: sdm FirstBoot: System time synchronization achieved
Feb 17 21:57:49 GatewayPi root[1098]: sdm FirstBoot: Run Generated FirstBoot Script '/etc/sdm/0piboot/010-configure-keyboard.sh'
Feb 17 21:58:00 GatewayPi root[1552]: sdm FirstBoot: Run Generated FirstBoot Script '/etc/sdm/0piboot/050-disable-bt.sh'
Feb 17 21:58:00 GatewayPi root[1557]: sdm FirstBoot: Disable bluetooth using /etc/modprobe.d/blacklist-sdm-bluetooth.conf
Feb 17 21:58:02 GatewayPi root[1579]: sdm FirstBoot: Skip Custom FirstBoot Scripts
Feb 17 21:58:05 GatewayPi root[1603]: sdm FirstBoot: Regenerate SSH Host Keys
Feb 17 21:58:13 GatewayPi root[1634]: sdm FirstBoot: set boot behaviour to B1
Feb 17 21:58:18 GatewayPi root[1680]: sdm FirstBoot: Re-enable getty@tty1
Feb 17 21:58:20 GatewayPi root[1705]: sdm FirstBoot: System will restart in 20 seconds
Feb 17 21:58:42 GatewayPi root[1728]: sdm FirstBoot: System restarting now
Feb 17 21:58:43 GatewayPi systemd[1]: Stopped sdm-firstboot.service - sdm FirstBoot System Configuration.

And the /etc/sdm/0piboot/010-configure-keyboard.sh file contains:
#!/bin/bash
source /etc/sdm/sdm-readparams
SUDO_USER=$myuser raspi-config do_configure_keyboard "$keymap" nonint
systemctl enable keyboard-setup

The image is Bookworm lite.  I wonder is this is something that worked on Bullseye but which has changed under Bookworm.
Thank you

Ack! A bug! Will fix.

If you want to fix ahead of the next release coming out, you can do:
sudo sed -i 's/\\$keymap/$value/'  /usr/local/sdm/plugins/L10n

Update will be out in the next couple of days.

Yep, that's working.
Thanks

Thanks for verifying!


#30 Create DeleteItemFragment Class

Create DeleteItemFragment Class for Delete Pop-up

The DeleteItemFragment Class was changed to deleteDialog that is created inside MainActivity class


Threshold values per field: fuzzy matching for some, exact for others

First of all, thank you for fuse.js!
Description
For each document in my index, I have some fields of which some are text, others are numbers or IDs. For the text fields I'd like to search with a certain (non-zero) threshold value, but for the ID fields I only want exact matching (i.e. threshold = 0).
So for example, with
{
  "name": "John Doe",
  "internalId": "63B5JOX13",
  "externalId": "Y45HNXXZ"
}

searching on "Jo" should only match on the name field.
Describe the solution you'd like
Perhaps we could extend the functionality of the threshold argument to the constructor? Now it accept a Number, but if it were to also accept a dictionary, we could specify thresholds per field, like so:
new Fuse(collection, {
  threshold: {
    name: 0.45,
    internalId: 0,
    externalId: 0
  }
});

If there are more fields than are specified in the dictionary, Fuse could rely on a fallback threshold option, say defaultThreshold.
I have not yet looked into the internal workings of Fuse, so I am not sure how realistic this is, but if it's of interest, I could draft  up a pull request? Let me know what you think?

@UserJHansen im running into a similar situation. from what i'm experiencing, you need to enable extended search to use that exact match feature and as soon as you do, the whitespace in the name field is treated differently per the docs

White space acts as an AND operator, while a single pipe (|) character acts as an OR operator.

https://fusejs.io/examples.html#extended-search
i have the exact same thing where i need to provide exact match on an ID but i need fuzzy search on a title-like string (like a blog post title)

actually nevermind, i think my issue may have been misuse of minMatchCharLength, where in my example, setting minMatchCharLength to 3 would prevent me from matching strings like "how to ..." with a query of "how to" (i dont totally understand this one but fixed without it)


added email matcher example

@bgildson looks good, good job!
There are only two things to be changed:

at line 6 set the author to your name.
at line 26 make the comment more clear (like "at symbol").

done

@bgildson ok its merged. Thank you very much for your contribution!
If you want you can leave some feedback about what you think of this library and things that can be improved.
You should now also have a valid pull request for hacktober fest. If not, let me know.
Cheers, and good coding 😊


nixos/redis: ensure conf var and conf run dirs exist

Description of changes
...prior to touch'ing those files
I found this when tracing through a deploy-time error that I'd thought was due to a config problem:
$ cat /var/lib/redis-mastodon/redis.conf
include "/run/redis-mastodon/nixos.conf"

# cat /run/redis-mastodon/nixos.conf
cat: /run/redis-mastodon/nixos.conf: No such file or directory

$ bash -x -e /nix/store/99b55pxjcj6achbxmswqd5d310r3lnm0-redis-mastodon-prep-conf
+ touch /var/lib/redis-mastodon/redis.conf /run/redis-mastodon/nixos.conf
touch: cannot touch '/run/redis-mastodon/nixos.conf': No such file or directory

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] Fits CONTRIBUTING.md.

Shouldn't these be created by RuntimeDirectory and StateDirectory?

@ajs124 Do you mean that you expect RuntimeDirectory and StateDirectory to be creating these directories already, meaning the explicit mkdir in this patch is not needed? Because I'm not seeing that behavior 🤔


feat(mysql): Metrics for mysql

Which problem is this PR solving?
https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1176
Short description of the changes
Add the metric db.client.connections.usage to mysql according to this: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/semantic_conventions/database-metrics.md#connection-pools

@haddasbronfman does the above comment make sense to you? if not please let me know so i can make sure to clarify

I opened this issue: https://github.com/open-telemetry/opentelemetry-specification/issues/2958
until it is resolved, I think I can't finish this PR.


remove clusterlogging from community operators

Remove cluster logging from community operators

cc @SamiSousa

@jcantrill If cluster logging and elasticsearch are being removed from community, how will they be available on an openshift cluster?

@jcantrill If cluster logging and elasticsearch are being removed from community, how will they be available on an openshift cluster?

via red hat certified operators.

/lgtm
We are currently in freeze for changes to community-operators. We will put this in hold until the freeze ends.
/hold


Form.TextArea loses is Lato font, even in your website https://react.semantic-ui.com/collections/form/

It was happening in my code but I checked the website https://react.semantic-ui.com/collections/form/ with chrome and safari and find out that the same happens.
The textarea font is changed from Lato to Arial.
I found a bug but it was closed from 2018,
thanks!

I have to style my component in order to look alike the others
<Form.TextArea
style={{ fontFamily: 'Lato' }}
/>


Terminal title/description setting localized strings need tweaking

https://github.com/microsoft/vscode/blob/e7fed4a7f9de12b393e474098efb0f0173e74854/src/vs/workbench/contrib/terminal/common/terminalConfiguration.ts#L13-L22
These need to change to make sure we're not including non-translatable strings in the "". For example:
'`${cwd}`: ' + localize('cwd', "the terminal's current working directory")
...
'`${separator}`: ' + localize('separator', "a conditional separator ({0}) that only shows when surrounded by variables with values or static text.", '`" - "`')

create an issue for the editor


Is it possible to control parameter of the presets with web MIDI ?

Hi, thanks for your great work.
I implemented Butterchurn into my experimental website.
It listens to MIDI input and visualizes the played sound.
I wonder if it would be possible to control the parameters of the presets with web MIDI (or at all).
Here is the site if you want to have a look: http://webmidi.handmadeproductions.de

Hey, thanks! Love seeing new integrations, thanks.
Well, it would be technically possible, but it will be hard and require a decent amount of knowledge of how MilkDrop presets work.
If you want to learn more about MilkDrop presets, I would start here: http://www.geisswerks.com/hosted/milkdrop2/milkdrop_preset_authoring.html
Butterchurn presets are just JSON files, so you could technically setup a MIDI controller, modify the preset JSON based on input and then pass that new preset to butterchurn.
Every preset has a baseVals section that defines the initial values for presets.  The problem is that the presets can also write to these values on every frame, so the baseVals is really just the starting value many times. So it is up to the preset whether it is updating certain values each frame or just sticking with the value from baseVals, so that might make it look like the starting value isn't really doing anything.
Let me know if you give it a try, thanks!

thanks for your explanations. i will give it a try, but i will need some time. and i will let you know.
without knowing enough - would it be possible to write the midi values every frame too?

No, thats not currently possible. Also that would lead to some weird behaviours, but I suppose that still might be better than having no affect.

Going to close this for now, but feel free to get in touch if you start working on integrating MIDI controls in anyway, thanks!


Updated examine_sim to use mujoco

examine_sim was still using mujoco_py which is no longer a dependency. I've updated it to use mujoco.

Thanks @andrearosasco  for the PR. It has been merged.


Pagination click issue

config: SwiperConfigInterface = {
// a11y: true,
direction: 'vertical',
slidesPerView: 1,
keyboard: false,
mousewheel: true,
scrollbar: false,
navigation: false,
pagination: {
el: '.swiper-pagination',
type: 'bullets',
clickable: true,
hideOnClick: false,
},
};
There are five div with different background image and no any other content to swipe, everything works fine but only when click the pagination buttons, sometimes it goes to wrong image, such as when I click the fourth page, the swiper shows the second images.
Did anyone know how to fix this?

I added a click listener in the page such as :
this.renderer.listen('document', 'click', (event) => {
console.log(event.target);
}
and write the onIndexChange function:
onIndexChange (index: number) {
console.log('goto index:' + index);
}
the log shows that:

the last 2 logs, there is only onIndexChange event but no click event, but actually I click the third pagination icon.
further I click the image on the page, I found that sometime click the image will change the index, like this:

Does anyone knows what is the reason and how to solve it?

What you are describing does not make much sense and its not happening in the example app. So maybe you could try to reproduce your issue in Stackblitz. With that little information there is nothing shown that could explain the behavior you are seeing. So would need to see the whole code and preferably in Stackblitz.

https://stackblitz.com/edit/angular-utp6tv?file=src%2Fapp%2Fapp.component.ts
@sconix above is the Stackblitz, the same behavior happened. when you click the icon of the pagination, it will goes to the wrong slide.

@sconix  OK, I think I find the root course, I did not set the threshold so it was the default value which was 0. After set the threshold to 50 as the example, everything works fine.
At first I thought it was because the 'noSwiping' config, and I set
noSwiping: true,
noSwipingClass: 'swiper-slide',
This avoided the behavior of click the image which change the image.
Anyway, thanks very much for answer me such soon.


Reconnect to MQTT Broker

Hi,
I've noticed that sometimes the device loses connection to the MQTT broker and reading through the code I don't see any attempts to try to reconnect. This makes it so you have to restart the ESP to regain connection. Is there any work being done to detect disconnects and attempt to reconnect?
Thanks for a great project!

Digging deeper, perhaps the MQTT library is supposed to handle this automatically?
What I'm seeing sometimes after leaving the device for a some time and coming back I get the following errors.
E (36555749) esp-tls: [sock=50] select() timeout E (36555750) TRANSPORT_BASE: Failed to open a new connection: 32774 E (36555750) MQTT_CLIENT: Error transport connect

Hi,
Yes, the component that handles MQTT will attempt to reconnect when it detects disruptions. If you are experiencing frequent disconnections you might have some connectivity issues.
Is the ESP far from the AP? Do you perhaps have it behind something that could cause interference, something metal?

It's about 40cm from my AP since I'm still doing some work on this on my desk. It's also not in any enclosure yet.
I'll do some more debugging and see if I can figure out why it's dropping the connection. FWIW I still just have it running over USB from my desktop and watching the serial output. Not sure if there's something that causes it to drop when my machine sleeps.

on sleep, it switches to the standby rail and it possibly confuses the WiFi chip.

Makes sense. I guess we can close this one until I've tested it over longer perioid with a stable power source (not my desktop).
Thanks!


docker-compose.yml sends users to a web page that doesn't exist

# DO NOT USE THIS IN PRODUCTION - it is for development only due to insecure
# settings and the use of "runserver" versus "gunicorn".
# See https://takahe-server.readthedocs.io/en/latest/installation.html for more

This link is a 404.

ok now that I play with it more, I see there's a lot more going on with the docker-compose.yml. I'll go play with it in my corner and see if I can work up something 1. that works and 2. might be a little more elegant. Assuming both aspirational goals are met I'll offer it up for PR consideration.

Sure - always happy to have some improvement in there. I don't use compose locally, so I can't maintain it very well.


Github releases

Hi @SUPERCILEX, thank you for this extension. This is not a bug report .
I'm maintaining an AUR package for this and it would be very helpful if you used the github releases so I, and other package maintainers, can get notifications when a new version is released.
Thanks!

Pretty sure I do: https://github.com/SUPERCILEX/gnome-clipboard-history/releases/tag/1.3.2

Maybe the you just forgot the latest release. Your latest tag is 1.3.3. Thanks for the quick response!

Oh, my bad lol. Just forgot, fixed.


Add option to configure icon position

Hey I would like an option to change where the icon is. Like if you use the the Raised mod with just raises the hotbar a bit the icons looks weirdly off.

A option that lets you set the Y offset is appreciated, adding on to above.

yep please look into this @Roundaround

I want it too (it will be cool to prevent it to overlaps with other mods HUD)


Add Support for TopK Filter Commands

#11

@dengliming can we update from master? ( it will trigger CI )

@dengliming sure.

@filipecosta90 sure.

Thank you =) I tried to push it to your master but was not able to do so ( on your other branch I was able )

@filipecosta90 Wrong Operation. Sorry about that. I will submit a new PR from upstream. dimiss this one. : )

@filipecosta90 Sorry for the wrong operation. Reopen now.


bug: repo branch is changed permanently when an addon does git checkout

Bug
When an addon runs on an app, fetches the repo for that app and changes the git branch to a different one, that change is permanent. Future addon runs will use that new branch. Even the Tackle UI shows the new branch instead of the one the user specified.
Steps to reproduce

Create an application in the Tackle 2 Hub with a repo url and a branch main.
Run an addon that fetches the repo and checkouts a different branch or creates a new branch.
Once the addon is finished, go back to the Tackle 2 Hub UI and look at the application. The branch will have changed.

Context
Currently working around this issue in Move2Kube addon by detecting the current branch before checking out a new branch
https://github.com/konveyor/addon-move2kube/blob/main/cmd/utils.go#L114-L126
and then checking out the original branch once the transformation is finished.
https://github.com/konveyor/addon-move2kube/blob/main/cmd/utils.go#L148-L151

cc @jortel

@HarikrishnanBalagopal I figured running what I had over here would demonstrate the behavior that you were describing, but I didn't see that.

Did I miss something?


clientv3: do not retry on modifications

/cc @heyitsanthony

lgtm; the leaky goroutines here probably aren't a big deal

@heyitsanthony I am merging this. We will resolve the potential leaky routine in another pr.


It doesn't work when renaming image names using MD5 is disabled and you want to keep the original image names and resource directory names

It doesn't work when renaming image names using MD5 is disabled and you want to keep the original image names and resource directory names

1.3.5
0.15.7
Windows10
When I disable rename image names with MD5 and want to keep the original image names vs resource directory names, it doesn't work, it always renames image names with resource directory names using MD5 which is very frustrating. I hope you can fix this error in time, thank you very much for your excellent extension. 👍😜

Because such a file path looks very incomprehensible😒

Hi, I cannot see a screencast etc in your issue. If you are trying this setting when downloading images, keep in mind, that this setting only applies to inserted attachments, not to uploaded ones. This behavior was implemented on purpose.  There is no 100% reliable way to find the original file name on a server.

I can understand what you are saying.
But I have some ideas.

It is recommended to use date and time stamps or incremental numbers for image file naming, maybe it will be better to understand.
Suggest keeping the name of the original resource directory and not base64 encoding the name of the original resource directory, this would also be better.
It is suggested to provide an option to allow users to turn off auto-replacement to local image files (only download network image files, but not auto-replace to local image files, so as to keep it updated with online image resources in real time).
Also I will upload screenshots to you again.

I have turned off MD5

Because such a file path looks very incomprehensible😒

I can understand what you are saying.

You obviously do not. Sorry. Or it is just me can not understand you English.
See my message above.

This behavior was implemented on purpose. There is no 100% reliable way to find the original file name on server.

I once had an idea to name files after anchor (if it is not empty, which happens pretty often in md fies), but it also requires some amount of time to implement

I can understand what you are saying.
But I have some ideas.

suggest to use hour, minute, millisecond or timestamp or incremental numbers to name the image files, maybe it will be better to understand.
suggest to keep the name of the original resource directory instead of base64 encoding the name of the original resource directory, that would also be better.
Suggest providing an option to allow users to turn off auto-replacement to local image files (i.e., only download web image files, but do not auto-replace to local image files to keep the online image resources updated in real time) 👍.

MD5 is used for encryption, but as a unique identifier I don't think it's the most ideal solution
MD5是用於加密的，但是作為唯一標識我認為不是最理想的方案

MD5 is used for encryption, but as a unique identifier I don't think it's the most ideal solution
I know the most ideal way to rename files is to use: hour, minute, and second and to time to milliseconds, because many similar software also use this batch file naming scheme, you may consider whether to use such a scheme.
Of course there are also schemes like GUID or UUID etc....
MD5是用於加密的，但是作為唯一標識我認為不是最理想的方案
我認識最理想的方式是使用：時分秒並將時間精確到毫秒的方式來重命名文件，因為很多類似的軟件也是採用這種批量文件命名的方案，您可以考慮是否採用這樣的方案。
當然還有GUID或者UUID這種方案等等。 。 。

suggest to keep the name of the original resource directory instead of base64 encoding the name of the original resource directory, that would also be better.

It not base64, it is called URI

MD5 is used for encryption, but as a unique identifier I don't think it's the most ideal solution

1 MD5 is for hashing, not for encrypting
2 You are free to implement your own solution. Good luck.

I do not know JavaScript and other web technologies, or I could also create a similar extension

There is actually a simpler implementation of file naming:

if the file does not exist in the directory then direct downloading;
if the file exists in a directory then download it for the file name + 1;
The name of the file for a batch download, e.g:
myflle.jpg
myflle01.jpg
myflle02.jpg
myflle03.jpg
...

Holy cow.... Are serious? :D Or it is just trolling.

Because such a file path looks very incomprehensible

Btw , you can use wikilinks mode. It does not require URI  in URL.  Thanks for your profound thoughts and advices ;) Will close this later.


added attributes to the url input field

these attributes prevent iOS from using autocorrect and autocaps

awesome thanks

released in 1.0.2 (and https://060f9c8fa729ff66a38c2475cc3a781c92ddaa89.htmlb.in)


RxNetty and 2.x vs. old-2.x

I had been tracking the RxNetty based version of Zuul that was recently moved to the "old-2.x" branch. Is RxNetty no longer on the roadmap? Is the intention for Zuul 2 to provide some sort of non-blocking semantics - either through Netty/RxNetty or the Servlet 3.1 API?
Thank you

Hey David. An RxNetty-based Zuul is still on the roadmap. I moved the 2.x branch as that was an early prototype, and we needed to go back and change some of the interfaces, so wanted to flag that that branch was ending.
I've been working on the "zuul2-bridge" branch, and will at some point soon, use that as the new 2.x, although we'll probably just be building snapshots for a while, until sure that the interfaces have settled down.
One of the intents of the zuul2-bridge branch was to separate out the zuul-core module from being dependent on netty or servlets, and then make the filter interfaces and FilterProcessor the same for operating on either RxNetty or blocking Servlets.
thanks

Thanks for the update - I'll go ahead and close this.
We're definitely interested in a non-blocking zuul-like gateway and so will be keeping an eye on this project and how it develops.


Create devcontainer.json

Summary
Briefly describe the changes and the goal of this PR. Make sure the PR title summarizes the changes effectively.
Changes
List the major and minor changes.

Major Changes

...

Minor Changes

...

Motivation
Why are these changes necessary? How do they improve the cookbook?
Self Review Checklist

[ ] Is the writing easily skimmable?

Sections have informative titles.
Key takeaways are upfront.
Short paragraphs and topic sentences are used.

[ ] Is the writing quality high?

Sentences are simple and unambiguous.
Demonstrative pronouns are avoided or clearly referenced.
No left-branching sentences.

[ ] Is the content universally helpful?

Terminology is specific and avoids jargon.
Provides solutions to common problems.
Code examples are general and exportable.

[ ] Is the content consistent?

Styling and formatting align with existing documentation.
Consistent use of punctuation and case.

When contributing a new example, make sure to add a new entry for it in registry.yaml to render it on the cookbook website.
Note: For additional guidelines on writing good documentation, check out What Makes Documentation Good.

I don't believe we need a devcontainer for this repo. Feel free to resubmit with an explanation though if you believe this is necessary.


[FPGA] Enhancement of Xilinx OpenCL backend

Description
This PR provides enhancements for the generated code that target Xilinx FPGAs.
In particular two features are introduced:

The ability to instantiate multiple compute units for the same kernel.
The transparent addition of the __attribute__((xcl_pipeline_loop(1))). (1 is the default initiation interval value (II) used by Xilinx HLS tools).

Problem description

With regards to the first feature, we have seen up to 2.5x performance improvement when fitting multiple compute units on the FPGA. In particular for the DFT and 65536 input size, we managed to fit up to 12 kernels on the Xilinx KCU1500 FPGA. The problem that did not allow this flag to work was due to an inability in the OCLCodeCache to parse the --nk computeDft:2 as a flag in the FPGA configuration file.
This flag is tested with SDAccel compiler xocc and resulted in 2.5x speedup for DFTDynamic.

Regarding the second feature, TornadoVM currently supports the loop unrolling for Intel FPGAs. This PR has added the addition of the Xilinx OpenCL pipelining attribute for loops, as this attribute can contribute to up to 2.2x performance increase. The XIlinx OpenCL loop unrolling attribute did not result in any performance change.

Backend/s tested

[x] OpenCL
[ ] PTX

OS tested

[x] Linux
[ ] OSx
[ ] Windows

Did you check on FPGAs?
If possible, check your changes on FPGAs.

[x] Yes
[ ] No

How to test the new patch?
I tested the PR in the Intel FPGAs to ensure that it does not break the code generation. In addition, I tested it on the Xilinx KCU1500 FPGA using SDAccel 2018.2 and on emulation mode with Vitis 2020.2. Here is the output of the execution with the generated code:
$ tornado --printKernel --debug --igv -Ds0.t0.device=0:1 uk.ac.manchester.tornado.examples.dynamic.DFTDynamic 512 tornado 1

__kernel void lookupBufferAddress(__global uchar *_heap_base, ulong _frame_base, __constant uchar *_constant_region, __local uchar *_local_region, __global int *_atomics)
{

  __global ulong *_frame = (__global ulong *) &_heap_base[_frame_base];

  // BLOCK 0
  _frame[0]  =  (ulong) _heap_base;
}  //  kernel

Initialization time:  759578609 ns

__kernel void lookupBufferAddress(__global uchar *_heap_base, ulong _frame_base, __constant uchar *_constant_region, __local uchar *_local_region, __global int *_atomics)
{

  __global ulong *_frame = (__global ulong *) &_heap_base[_frame_base];

  // BLOCK 0
  _frame[0]  =  (ulong) _heap_base;
}  //  kernel

[DEBUG] JIT compilation for the FPGA
Warning: -Dtornado.opencl.userelative was set to False. TornadoVM changed it to True because it is required for FPGA execution.
__attribute__((reqd_work_group_size(64,1,1)))
__kernel void computeDft(__global uchar *_heap_base, ulong _frame_base, __constant uchar *_constant_region, __local uchar *_local_region, __global int *_atomics)
{
  long l_11, l_30, l_32, l_31, l_10, l_9;
  float f_28, f_27, f_26, f_25, f_24, f_7, f_6, f_15, f_13, f_23, f_22, f_21, f_20, f_19, f_18, f_17, f_16;
  ulong ul_12, ul_14, ul_0, ul_2, ul_34, ul_1, ul_33, ul_3;
  int i_8, i_29, i_35, i_5, i_4, i_36;

  __global ulong *_frame = (__global ulong *) &_heap_base[_frame_base];

  // BLOCK 0
  ul_0  =  (ulong) _frame[3];
  ul_1  =  (ulong) _frame[4];
  ul_2  =  (ulong) _frame[5];
  ul_3  =  (ulong) _frame[6];
  i_4  =  get_global_id(0);
  // BLOCK 1 MERGES [0 5 ]
  i_5  =  i_4;
  // BLOCK 2
  // BLOCK 3 MERGES [2 4 ]
  f_6  =  0.0F;
  f_7  =  0.0F;
  i_8  =  0;
  __attribute__((xcl_pipeline_loop(1)))
  for(;i_8 < 512;)
  {
    // BLOCK 4
    l_9  =  (long) i_8;
    l_10  =  l_9 << 2;
    l_11  =  l_10 + 24L;
    ul_12  =  ul_0 + l_11;
    f_13  =  *((__global float *) &_heap_base[ul_12]);
    ul_14  =  ul_1 + l_11;
    f_15  =  *((__global float *) &_heap_base[ul_14]);
    f_16  =  (float) i_8;
    f_17  =  f_16 * 6.2831855F;
    f_18  =  (float) i_5;
    f_19  =  f_17 * f_18;
    f_20  =  f_19 / 512.0F;
    f_21  =  sin(f_20);
    f_22  =  cos(f_20);
    f_23  =  f_22 * f_15;
    f_24  =  fma(f_21, f_13, f_23);
    f_25  =  f_7 - f_24;
    f_26  =  f_21 * f_15;
    f_27  =  fma(f_22, f_13, f_26);
    f_28  =  f_6 + f_27;
    i_29  =  i_8 + 1;
    f_6  =  f_28;
    f_7  =  f_25;
    i_8  =  i_29;
  }  // B4
  // BLOCK 5
  l_30  =  (long) i_5;
  l_31  =  l_30 << 2;
  l_32  =  l_31 + 24L;
  ul_33  =  ul_2 + l_32;
  *((__global float *) &_heap_base[ul_33])  =  f_6;
  ul_34  =  ul_3 + l_32;
  *((__global float *) &_heap_base[ul_34])  =  f_7;
  i_35  =  get_global_size(0);
  i_36  =  i_35 + i_5;
  i_5  =  i_36;
  // BLOCK 6
  return;
}  //  kernel

Task info: s0.t0
	Backend           : OpenCL
	Device            : xilinx_u50_gen3x16_xdma_201920_3 CL_DEVICE_TYPE_ACCELERATOR (available)
	Dims              : 1
	Global work offset: [0]
	Global work size  : [512]
	Local  work size  : [64, 1, 1]
	Number of workgroups  : [8]

Total time:<PHONE_NUMBER>2 ns

Is valid?: true

Validation: SUCCESS

Provide instructions about how to test the new patch.

Currently in OCLDeviceContext
    @Override
    public boolean isPlatformFPGA() {
        return getDevice().getDeviceType() == OCLDeviceType.CL_DEVICE_TYPE_ACCELERATOR
                && (getPlatformContext().getPlatform().getName().toLowerCase().contains("fpga") || getPlatformContext().getPlatform().getName().toLowerCase().contains("xilinx"));
    }

    @Override
    public boolean isPlatformXilinxFPGA() {
        return getPlatformContext().getPlatform().getName().toLowerCase().contains("xilinx");
    }

I guess it should be the following to avoid duplication:
    @Override
    public boolean isPlatformFPGA() {
        return getDevice().getDeviceType() == OCLDeviceType.CL_DEVICE_TYPE_ACCELERATOR
                && (getPlatformContext().getPlatform().getName().toLowerCase().contains("fpga") || isPlatformXilinxFPGA);
    }

    @Override
    public boolean isPlatformXilinxFPGA() {
        return getPlatformContext().getPlatform().getName().toLowerCase().contains("xilinx");
    }

Also, probably the accelerator device check must be duplicated in isPlatformXilinxFPGA