Layers is undefined in<EMAIL_ADDRESS>

According to documents, I can customize a CompositeLayer with following codes:
import {CompositeLayer, IconLayer, TextLayer} from 'deck.gl';

class LabeledIconLayer extends CompositeLayer {
  // TODO
}
LabeledIconLayer.layerName = 'LabeledIconLayer';

However, I got this error message when I use it (with deck.gl@6.4.10):

Uncaught (in promise) TypeError: Class extends value undefined is not a constructor or null.

The null refers to CompositeLayer.
On the other hand, if I want to use any properties (such as 'defaultProps') inside IconLayer or other layers, I got error message like this:

Uncaught (in promise) TypeError: Cannot read property 'defaultProps' of undefined

Docs： https://deck.gl/#/documentation/developer-guide/writing-custom-layers/composite-layers?section=collection-layers

However, I didn't get related upgrade tips from Upgrade Guide https://deck.gl/#/documentation/overview/upgrade-guide

They are defined<EMAIL_ADDRESS>Can you double check your dependency and bundling configs.

@Pessimistress I checked the package I used in my project again, and restricted it to ~6.3.1 version (which is v6.3.9 installed actually), but following usage would still got errors like:
import {ArcLayer} from 'deck.gl';

index.js:10 Uncaught (in promise) TypeError: Class extends value undefined is not a constructor or null

I also print the package itself out and got following result (most of layers are undefined inside deck.gl, however, CompositeLayer is valid inside it, is there ant ideas why it could happen?):
import * as testDeckGl from "deck.gl";
console.log(testDeckGl);

Result:

I noticed there are some usages different from previous version, such as I should import<EMAIL_ADDRESS>rather than 'deck.gl' if I want to use ScatterplotLayer? Would it be the reason?
Thanks.

deck.gl re-exports all the classes from submodules. Looks like your submodules (e.g. @deck.gl/layers) are not properly installed. You can try clear your node_modules and reinstall all dependencies.

I removed my node_modules folder and reinstalled, but it seems doesn't work if I only declare deck.gl in package.json.
The strange thing is that ArcLayer is exported but remains undefined, which means corresponding package<EMAIL_ADDRESS>isn't installed rightly. However, when I open the deck.gl's node_modules folder inside node_modules folder, @deck.gl/layers is there with version 6.3.9.
I need to separately install these submodules and import from them rather than 'deck.gl' to solve my problem currently. Got no idea what's wrong with my environment that causes the wrong settings of deck.gl. :-(

It seems submodules not included in<EMAIL_ADDRESS>will be exported as undefined with specific Babel environment. I suspect it's caused by deck.gl's export or module.export coding style.
I will reopen it if I locate a more specific reason.

I have the same issue as described above (same deck.gl version as well).
This error occurred when I added CompositeLayer so as to compose two of the layers.
I have no idea on how to solve this issue. If you have any suggestions,  let me know

I have the same issue as described above (same deck.gl version as well).
This error occurred when I added CompositeLayer so as to compose two of the layers.
I have no idea on how to solve this issue. If you have any suggestions, let me know

@kfragkedaki Hi, there. Currently, I have no idea why it would happen either, but I solved it temporarily by importing sub modules like<EMAIL_ADDRESS>and<EMAIL_ADDRESS>separately.
I used them in my project with:
import DeckGL from<EMAIL_ADDRESS>import { ScreenGridLayer } from<EMAIL_ADDRESS>
Maybe you can try it the same way. I still suspect it's caused by deck.gl's export or module.export coding style.

Hi again, I solved this problem by using the following piece of code. It seems that I had to create aN object of LayerText class and include its properties when using it on deckgl's layers.
import { TextLayer, ScatterplotLayer, CompositeLayer } from "deck.gl";

export const layerText = data => {
  class LayerText extends CompositeLayer {
    renderLayers() {
      const scatterplot = new ScatterplotLayer(
        this.getSubLayerProps({
          id: "icon-eta",
          data: this.props.data,
          radiusScale: 6,
          opacity: 1,
          getPosition: d => d.coordinates,
          getFillColor: [4, 171, 193]
        })
      );

      const text = new TextLayer(
        this.getSubLayerProps({
          id: "text-distance",
          data: this.props.data,
          getPosition: d => d.coordinates,
          getText: d => d.distance,
          getColor: [255, 255, 255]
        })
      );
      return [scatterplot, text];
    }
  }
  LayerTexT.layerName = "LayerText";

  return new LayerText({
        id: `icontext-eta`,
        data
   });
}

<DeckGL
  layers= {[
     layerText
  ]};>
   // TODO
</DeckGL>
}

@kfragkedaki I suspect we met different problems. I think the problem you faced is how to use Layers correctly, but I just couldn't import them from deck.gl directly?
And one more thing, is it typos that you pass an array contains function into layers property rather than an array contains layers (since layerText in your code is a function, which should be executed to return the exact layer instance)?


include a timestamp in the location data

we need to see the time that the location data was initially created and when last it was modified
http://healthsites.mashastudio.net/map.html
https://github.com/konektaz/healthsites/wiki/Core-attributes-for-Health-facilities

the service is not there yet, and it is related with ticket #258


No Character limit (minimum and maximum) and taking alphanumeric characters as well in Creating Control Station from 'Create User'.(Company Panel)

Describe the bug
Every field in Control Station has some manner of writing
i.e Remote IP Address- XXX : XXX : XXX : XXX (where X- 0-255)
Remote Port Address- XXXX (where X is a number 0-9)
Device ID- alphanumeric Value.
Contact Number- Must be 10 digit
User Display Name- Only alphabet
But for every field of  Control Station has No Character limit (minimum and maximum) and taking alphanumeric characters as well.
To Reproduce
Steps to reproduce the behavior:
1.Enter in the address bar-pulseptt.com
2.Login With Id and Password.
UserName-pulse123
Password-123456789
3.Click On User Management and Click on Create New user and Select Control Stations.
Expected behavior
Every field of Control Station must be written according to the requirement of the field.
Desktop (please complete the following information):
OS: Windows 10
Browser: Version 100.0.4896.60 (Official Build) (64-bit)
Version: [<IP_ADDRESS>]

When Validation error occurs then pop up with specified field which is causing error must be displayed on pop-up. But it is displaying 'ValidationError: 2 errors occurred' which is confusing for the user to understand.
Ex- If error occurs in 'Remote IP Address' field then it must specified it with error in 'Remote IP Address error'
Note- Changed the format of  -Remote IP Address- XXX : XXX : XXX : XXX (where X- 0-255) in Describe the bug

If I am writing Remote IP Address- <IP_ADDRESS> i.e values like 000,001,002,003 till 009 then it gives an ip error. But Ip can be written from 0-255, so those values from 000 to 009

must also be included.


Adding MSL 4.0.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

The OpenModelica Standard Language 4.0.0 is maintained in the branch OM/maint/4.0.x.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Feature: Code Review Check handle commits made by version bump bots

Is your feature request related to a problem? Please describe.
Discussion started at https://github.com/electron/electron/pull/35741#pullrequestreview-1118960274
Just oppening the discussion about the commits made automatically by some bots that, depending on its access rights and on its complexity, can not represent a big security risk.
We have some situations where it is not reasonable for the project to review every single commit made by the bot, since it is a very frequent and automatic job. But, by the other hand, allowing an ignore list for bots also leaves the door open to other projects excluding bots that might be more susceptible to compromise.
Describe the solution you'd like
The Code Review Check could handle, somehow, commits made by version bump bots, that just updates the version of the project itself, not of its dependencies.
Describe alternatives you've considered
I've consider to allow the user to exclude commits made by the bot, but this has the turnaround mentioned above of leaving the door open to other projects excluding bots that might be more susceptible to compromise.
Additional context
Add any other context or screenshots about the feature request here.

I think this is doable, the approach would be to check if the authors of a change were bots. Previously we used User.Login (API), but I think we should use User.Type instead


Explicitly document Microsoft tenant's limitation on device code and Microsoft Graph

Previously in https://github.com/MicrosoftDocs/azure-docs-cli/pull/3153, we didn't explicitly call out Microsoft tenant has such limitation, as we think https://github.com/Azure/azure-cli/issues/22629 is a common problem and we don't want external customers to be bothered with Microsoft internal business.
Now we are still seeing internal tickets created for this issue, so let's explicitly call out that this issue happens for Microsoft internal users.

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

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

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

Looks good to me.


Only include working platforms in released versions?

I'm not sure of the best way to handle this.  This is a little tricky and depends on whether the plugin is intended for use with Intel XDK exclusively or for general usage.
Somehow, we need to let users know which platforms will work in the Intel XDK.
For example, the iOS platform will not work until Intel XDK adds support for cordova-ios 4.0+, which will not happen until sometime after cordova-ios 4.0 is released (at the time of this writing it has not been released).
One way this could be done is via the list of supported platforms in plugin.xml.
Another way could be by updating the information in README.md.

Mentioning @xmnboy for visibility and input.

This plugin will work with both, either through cordova CLI or Intel XDK but both has iOS version dependency of ‘>=4.0.0’ Thanks for bringing the issue, our iOS compatibility check needs to be integrated which was missing. We will be adding below compatibility check in plugin.xml  :
<engine name="cordova-ios" version=">=4.0.0-dev" />

We tested with the above in IntelXDK locally and were not able to build for iOS using this update. So this seems to be a valid approach for compatibility check, any thoughts? We are also in process of updating ReadMe.md having supported platform details and will be doing so shortly

Adding the engine compatibility flag is a good step for the general case.
If README.md also includes a compatibility section, this seems like it would cover the issue to me.
@xmnboy - thoughts?

Ben, see @tony-- comment about adding to the README.md

Added to README

Leaving open actually in case of other edits

Thanks for the updates.  Suggested edit:
iOS (For Intel XDK the plugin will not work for iOS as it has dependency on cordova v4.0.0 which has not been released yet.)
-->
iOS (For Intel XDK the plugin will not work for iOS as it has dependency on cordova-ios v4.0.0 which has not been released yet.)

We have updated the Readme.md file with the mentioned suggestion (cordova -> cordova-ios)

Thanks


fixed error on first run and added guessing for default "perls" direc…

Fixes #10
Fixed error on first run and added guessing for default "perls" directory.

Thanks, I've implemented an alternate approach in https://github.com/Grinnz/perldoc-browser/commit/9f48bebd438dcaaf0c506073788138db803e9dc1


Maybe some bug in hbb2obb_v2 function in ./mmdet/core/bbox/transforms_rbbox.py?

Hello,
Thanks for your excellent framework for rotation detection. And I have experiments the configs, such
as retina_obb, roiTransformer_obb etc...And I can reach, even surpass, the performance of the original paper.
But I find a possible bug in hbb2obb_v2 function in transforms_rbbox.py, the
ex_heights = boxes[..., 2] - boxes[..., 0] + 1.0
ex_widths = boxes[..., 3] - boxes[..., 1] + 1.0
may be replaced by
ex_widths = boxes[..., 2] - boxes[..., 0] + 1.0
ex_heights = boxes[..., 3] - boxes[..., 1] + 1.0
Am I right?
I check the code of hbb2obb, it seems right.
So on the one hand, I don't know why define v2 of hbb2obb; on the other hand, why the function works well?...

+1 The same question.

Hello,
Thanks for your excellent framework for rotation detection. And I have experiments the configs, such
as retina_obb, roiTransformer_obb etc...And I can reach, even surpass, the performance of the original paper.
But I find a possible bug in hbb2obb_v2 function in transforms_rbbox.py, the
ex_heights = boxes[..., 2] - boxes[..., 0] + 1.0
ex_widths = boxes[..., 3] - boxes[..., 1] + 1.0
may be replaced by
ex_widths = boxes[..., 2] - boxes[..., 0] + 1.0
ex_heights = boxes[..., 3] - boxes[..., 1] + 1.0
Am I right?
I check the code of hbb2obb, it seems right.
So on the one hand, I don't know why define v2 of hbb2obb; on the other hand, why the function works well?...

why added +1.0 ??


Clarify comments around handling poll_apic

Given that most of our backings halt in kernel mode instead of user mode, the previous doc comment was rather misleading.

Superseded by #114


Add ability to revoke recorder tokens

Coverage increased (+0.26%) to 96.73% when pulling 0e9127f0b962bc8bf31918589e9355d0efb1076f on token-revocation into 8c50e7d1b10b63186547e03fa5208d633b124295 on master.


-implement the up/down keys for the gallery view/slide modal

Fixes #715
Were any Django, import pipeline, table_schema, or dictionary files modified? Y/N

If YES:

Database used: opus3_test_200903
All Django tests pass: NA
FLAKE8 run on modified code: NA

Were any JavaScript or CSS files modified? Y

If YES:

JSHINT run on all affected files: Y
Tested on Chrome, Firefox, Safari, Edge, and iOS: Y/NA
(Remember to test all browser sizes)
Tested for race conditions (with delayed API calls if applicable): NA

Description of changes:
-Catch the up and down arrows when the modal is open; in gallery view, the up/down will move the binoculars up or down one row and update the modal w/the new opusID; in table view, the up/down will do the same as next/prev and move one up or one down, then update the modal.
-if there are no more rows up or down, the up/down arrow will do nothing.
Known problems:

Issues:

Binoculars stay on the gallery view after some changes in select metadata menu.

Steps to reproduce the issue:

Start OPUS freshly, and go to gallery view.
Click on any thumbnail to open the metadata modal, and then close it.
Go to select metadata menu and add something like volume id, and then save.
Binoculars will stick to the thumbnail we clicked at step b with metadata modal closed. Hitting left/right arrows won't do anything and up/down arrows will move the scrollbar. Need to click on any thumbnail again to open the metadata modal and resume the correct behavior of binoculars.

In console, error message: metadataboxHtml: in each, observationData may be out of sync with colLabels;

Steps to reproduce the issue:

Add some API delay.
In gallery view, open up a metadata detail modal, and hold the down arrow key and it will move down to trigger initInfiniteScroll load.
When spinner is still running, click edit in the modal and delete a metadata col. At the same time, keeping hitting down key maybe for about 3 to 4 times, then we can see the error message in the console. Most of the time, all metadata info will still be loaded fine, but sometimes it will have a blank right pane without any metadata info.

re: 1.	Binoculars stay on the gallery view after some changes in select metadata menu - that has been resolved.
re: 2.	In console, error message: metadataboxHtml: in each, observationData may be out of sync with colLabels;   This issue is in production code and is an unrelated bug.  It is easy to dup in production code w/out any delays by using the table view, edit metadata, etc, similar as described above.


createPredicton returns an iterable object, but no data is returned

I tried to run the example in the readme file ( replacing the url and chatId)
"flowise-sdk": "^1.0.9",
node version: 20.15.0
SO: Windows
// changed this because the original code breaks. I'll made another issue about this.
import pkg from 'flowise-sdk';
const { FlowiseClient } = pkg

const flowise = new FlowiseClient({ baseUrl: 'http://localhost:3000' });

async function main() {
  const completion = await flowise.createPrediction({
    chatflowId: '<id>',
    question: "hello",
    streaming: true
  });

  for await (const chunk of completion) {
    console.log(chunk);
  }
}
main();

In the console, the completion object look like this
{
  [Symbol(Symbol.asyncIterator)]: [AsyncGeneratorFunction: [Symbol.asyncIterator]]
}

But the for await loop gets nothing and the script ends.

Just for clarifying, everything works OK calling it using Curl.


Created Program for Vector Representation of General Sparse Matrix

Added Program for Vector Representation of General Sparse Matrix.
Hope you like it :-)

Please star this repository to show your support, in the meantime I am reviewing your code.

Ok sure. Thanks for reviewing.


not possible to get accessories defined outside homebridge-mqtt

Is it possible in any way to get hold of, and control, other accessories, like lamps etc, that was not configured in the scope of homebridge-mqtt, but exists on the same homebridge, or same Home?

@tvillingett:
Short answer: no it isn't possible
There are 3 different layers:

Homekit with access to all accessories.
Homebridge has only access to all accessories defined in the installed plugins.
Plugin has access to the hown accessories.

The MQTTAPI could be added to layer 2 (homebridge), this depend on the homebridge-developers. I don't think Apple will be willing to add a MQTT API to Homekit.

I get it, not really feasible...

I'm a bit late to the party, but a (very) inconvenient way is to use the hidden API of homebridge in the insecure mode. E.g. http://:51826/accessories
If someone is interested I can share my blueprint for toggling Hue lamps via a Sonoff T1 that sends a custom MQTT message

You don't even need this plugin to expose the API. Just start homebridge with the -I flag. ;)
Correct, every device is exposed via the raw HomeKit protocol/API (HAP).
See e.g. https://github.com/nfarina/homebridge/issues/1219#issuecomment-288243015 and, https://github.com/oznu/hap-client or https://developer.apple.com//homekit/specification/ for more information about the HAP

I see. Well, if in insecure mode, it would be great if
topic: homebridge/to/get payload: {"name": "*"}
would return every accessory in the homebridge instance.

If any developers for the mqtt plugin would like inspiration, homebridge-config-x (Homebridge GUI plugin) does exactly this when Homebridge is run in insecure mode. Then a new tab is available with a graphical layout showing all accessories regardless of plugins, with states and the ability to manipulate them.

Now the HAP-nodered node for Node-Red does this and works great along with homebridge-mqtt.
Sent with GitHawk

I've just had a go with this: Home(iPadOS)->homebridge+websockets->homebrew program->wifi->flashed Sonoff->heater and it works. But if I want to extend this to lots of devices, it seems that there is just one global namespace. So I could have lounge-heater and office-heater, but I can't have 'heater' on its own within the scope of a room? Not sure how Siri could cope with all this. (Fwiw the homebrew program->wifi->flashed sonoff bit has been working for ages with a web browser control. The browser has a location hierarchy structure)

Sorry - wrong place for post.


"There are no active source control providers"

Issue Type: Bug
I was happily working along, changing code, checking it in. I restarted VS Code, and now it cant' find git.
It had no problem finding git before (located in /usr/local/bin/git). but sometimes it finds and sometimes it doesn't. Its non-deterministic. I've tried starting it from macos finder, and starting it from the command line. Both situations it could or could not find git.
VS Code version: Code 1.27.0 (7b9afc4196bda60b0facdf62cfc02815509b23d6, 2018-09-05T05:22:06.672Z)
OS version: Darwin x64 16.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2700)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (1.32GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron .

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

EditorConfig
Edi
0.12.4

vscode-docker
Pet
0.1.0

vscode-database
baj
1.5.2

solargraph
cas
0.17.6

dart-code
Dar
2.18.0

cpptools
ms-
0.18.1

Go
ms-
0.6.89

gradle-language
nac
0.2.3

ruby
reb
0.20.0

java
red
0.30.0

vscode-arduino
vsc
0.2.20

vscode-java-debug
vsc
0.12.1

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.8.0

vscode-maven
vsc
0.10.0

vim
vsc
0.16.3

JavaScriptSnippets
xab
1.7.0

vscode-open-in-github
ziy
1.3.5

Looks like my email attachments never made it into the Issue. Here they are.

The /usr/local/bin/git comes from brew.
All xcode licenses are accepted.  (that used to be a cause of this issue)

In Code's devtools Console, what does process.env.PATH return?

<EMAIL_ADDRESS>/bin:/usr/local/sbin:~/flutter/bin:/Users/jstillwell/.nvm/versions/node/v6.3.0/bin:/Users/jstillwell/.rbenv/shims:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/Applications/Wireshark.app/Contents/MacOS:~/Library/google-cloud-sdk/bin/:~/Code/depot_tools:~/Library/Android/sdk/platform-tools/:~/go/bin"
This seems to happen with some specific projects.
It doesn't seem to matter if I start vscode from Finder, or via the command
line tool.
I checked the path value for both a working and non-working project, and
they both have the same path.
Thanks again for the help, btw.
On Mon, Sep 10, 2018 at 1:19 AM João Moreno<EMAIL_ADDRESS>wrote:

In Code's devtools Console, what does process.env.PATH return?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-419827690,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_riTLkujR4cAIx_fKbKGEsfRRiRcgks5uZiCPgaJpZM4Wb6H2
.

This seems to happen with some specific projects.

Any way we can maybe use this to try to find out what is it that causes the issue? Which projects work and which don't?

What if you open a Flutter project with code --disable-extensions?

That won't disable the built-in extensions?
On Wed, Sep 12, 2018 at 1:18 AM João Moreno<EMAIL_ADDRESS>wrote:

What if you open a Flutter project with code --disable-extensions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420556156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rtKui2wWypNgevnkgZILOkkM6ekqks5uaMNVgaJpZM4Wb6H2
.

Sorry, it has no affect. "there are no source control providers" when I
open a problem project.
On Wed, Sep 12, 2018 at 9:05 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

That won't disable the built-in extensions?
On Wed, Sep 12, 2018 at 1:18 AM João Moreno<EMAIL_ADDRESS>wrote:

What if you open a Flutter project with code --disable-extensions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420556156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rtKui2wWypNgevnkgZILOkkM6ekqks5uaMNVgaJpZM4Wb6H2
.

Wait, let me take that back.
code --disable-extensions didn't disable the extensions in the project.
even from the project directly.
but code -disable-extensions . did.
And with extensions disabled, the git built-in extension did start to work.
So I'll track down which extension is conflicting with git.
On Wed, Sep 12, 2018 at 9:07 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

Sorry, it has no affect. "there are no source control providers" when I
open a problem project.
On Wed, Sep 12, 2018 at 9:05 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

That won't disable the built-in extensions?
On Wed, Sep 12, 2018 at 1:18 AM João Moreno<EMAIL_ADDRESS>wrote:

What if you open a Flutter project with code --disable-extensions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420556156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rtKui2wWypNgevnkgZILOkkM6ekqks5uaMNVgaJpZM4Wb6H2
.

Yeah, its the dart extensions. I'll take it to them to find out why its
stopping the git extension.
On Wed, Sep 12, 2018 at 9:10 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

Wait, let me take that back.
code --disable-extensions didn't disable the extensions in the project.
even from the project directly.
but code -disable-extensions . did.
And with extensions disabled, the git built-in extension did start to work.
So I'll track down which extension is conflicting with git.
On Wed, Sep 12, 2018 at 9:07 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

Sorry, it has no affect. "there are no source control providers" when I
open a problem project.
On Wed, Sep 12, 2018 at 9:05 AM Jason Stillwell<EMAIL_ADDRESS>wrote:

That won't disable the built-in extensions?
On Wed, Sep 12, 2018 at 1:18 AM João Moreno<EMAIL_ADDRESS>wrote:

What if you open a Flutter project with code --disable-extensions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420556156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rtKui2wWypNgevnkgZILOkkM6ekqks5uaMNVgaJpZM4Wb6H2
.

Thanks for the help.

@joaomoreno I'm the author of the Dart/Flutter extensions. I don't think there's anything in here that should be able to affect Git so I'm not sure how to troubleshoot this (it doesn't seem to happen locally for me). If there's anything I can do that might help track this down, please let me know.

Thanks for the response.
Seems to be fixed since I added "git.ignoreMissingGitWarning": false to my settings.

No idea.
@dragonfax You haven't showed us the output of Git: Show Git Output?

It doesn't show up when the git extension isn't active.
On Wed, Sep 12, 2018 at 1:06 PM João Moreno<EMAIL_ADDRESS>wrote:

No idea.
@dragonfax https://github.com/dragonfax You haven't showed us the
output of Git: Show Git Output?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420779483,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rmre33H6Ccd_inA1i3Ai4asz02gqks5uaWlcgaJpZM4Wb6H2
.

When everything is working. this is what we've got.
Looking for git in: /usr/local/bin/git
Using git 2.14.1 from /usr/local/bin/git
> git rev-parse --show-toplevel
> git config --get commit.template
Open repository: /Users/jstillwell/Code/dragonfax/XXXXXX
> git status -z -u
> git symbolic-ref --short HEAD
> git rev-parse master
> git rev-parse --symbolic-full-name master@{u}
> git rev-list --left-right master...refs/remotes/origin/master
> git for-each-ref --format %(refname) %(objectname) --sort -committerdate
> git remote --verbose
> git check-ignore -z --stdin

On Wed, Sep 12, 2018 at 2:29 PM Jason Stillwell<EMAIL_ADDRESS>wrote:

It doesn't show up when the git extension isn't active.
On Wed, Sep 12, 2018 at 1:06 PM João Moreno<EMAIL_ADDRESS>wrote:

No idea.
@dragonfax https://github.com/dragonfax You haven't showed us the
output of Git: Show Git Output?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420779483,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rmre33H6Ccd_inA1i3Ai4asz02gqks5uaWlcgaJpZM4Wb6H2
.

The command doesn't appear in the command palette at all? Or, you run it and it doesn't show up?

You know. frustratingly, even when I put "git.ignoreMissingGitWarning": true back into my settings.json. The Git extensions continues to work
fine. Even with the problem projects.
In other words, I can no longer replicate the issue.
It could have been some legacy thing. Since I don't remember doing it, I
probably enabled ignoreMissingGitWarning more than a year ago, possibly
several years.  Having disabled it (the default value) at least once, has
fixed everything going forward.
On Thu, Sep 13, 2018 at 12:39 AM João Moreno<EMAIL_ADDRESS>wrote:

The command doesn't appear in the command palette at all? Or, you run it
and it doesn't show up?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/58017#issuecomment-420913391,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4_rjevaTdtCKKql_2WSBpVq43VHPFqks5uaguhgaJpZM4Wb6H2
.

Too bad. Let me know if it comes back.


Create new RPC method: update_query_prices()

Resolves #1157
Graph-node operators may want to apply regular query pricing updates based upon their system and market analysis. This PR adds an endpoint that allows one with access to update the query_price for any existing deployment on the node.

@fordN does this PR still make sense?


Add WORLD env var to download save via HTTP, and other fixes

Allow -e WORLD=http://xxxx/world.zip to download archived world and unpack
Allow 'adventure' and 'spectator' game modes
Allow /mods and /config mounts for shared Forge mods without using /data
Quote many strings in case of blank values
Let VERSION take lowercase options, and set to LATEST if blank or invalid
Enable command blocks by default (needed for adventure worlds)
Allow DIFFICULTY to be given numerically
default to empty banned-players and banned-ip files to avoid error message

The code changes look great. I'm fixing up the CircleCI build (always due to a stale Cassandra version) and wanted do my due diligence and sanity test your PR.

Fixes #48


Question: youtube-dl creates -lots of- .Frag files

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.01.16. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.01.16

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Question:
I use this command (*) to download the best dash quality of some youtube videos:
youtube-dl -f 137+140 http://www.youtube.com/watch?v=3JZ_D3ELwOQ
*Source: http://unix.stackexchange.com/questions/272868/download-only-format-mp4-on-youtube-dl/272934
Sometimes this command creates lots of .Frag-files in the home folder - and sometimes not. It happens with large videos and shorter ones but not consistent. Crawling the readme gave no hints on that. So can anybody explain why youtube-dl creates this fragments?
cheers
red

Some videos served in fragments. In this case frag files are created.

I'm also curious about this. Sorry if i ask a bit more here. Just want to know, Then when a video served in fragments, it will be downloaded as separated frag files first by youtube-dl? or is it possible to download it with no frag files?

@linglung See #9937

Can you set maybe a run time working dir for youtube-dl to use till its ready to copy the finished concatenated file whatever the main env $CWD was when the initial command was run?

@brokendash See #10993. Before that is implemented, you can use --exec

Ahhh exactly what I'm looking for!  Thanks Sir... :-)


Adding Graphite port and instructions

Allow users to import Graphite data.

Hey @googanhiem, could you please explain why you choose 2004 , when the default port for Graphite is 2003 ?

@denisgolius oops, I just did a quick google search and read something about 2004. Don't mind which port is listed.


Edit influxd upgrade docs

Closes #1680.

[ ] Tests pass (no build errors)
[ ] Rebased/mergeable

(Consider editing https://github.com/influxdata/influxdb/blob/master/cmd/influxd/upgrade/upgrade.go#L118-L124 as well.)


OOM issue

I am getting OOM in 40 GB available GPU memory(P4 instance) after increasing number of layers and heads to 4. This commands I am using torchrun --nproc_per_node=1 hydra/train/train.py --model_name_or_path lmsys/vicuna-7b-v1.3  --data_path data/sharegpt/raw/  --bf16 True  --output_dir ckpts  --hydra_num_heads 4  --hydra_num_layers 4. I tried with 1 head and 1 layer, it is running fine. Also I tried with precomputed data, still it is showing OOM error. What is required configuration to run this?

You can try decreasing the micro_batch_size. Could you also post the command you use for pre-computed data.

Closing for now but feel free to open at any time @ShivangiAg


Fix fine-grained progress bars

Commit 3cbd533da3eda9162da8955ead6218a022b0604b shadowed the floating-point
fill variable with the integer number of entirely-filled characters in the
progress bar. We still need the floating-point progress to pick the fine-grained
progress character: after that commit, (fill * n) % n is always 0, so we
always use the last fine-grained character.
Fix this by using a different name for the integer progress.
While I'm there, try to make the function easier to read:

Use str::repeat.
Rename some variables and add comments.
Remove the unnecessary pct variable.
Skip the selection of a fine-grained character when we have exactly one.
Use fill.fract() instead of modulo to discard the integer part of progress.
This should make it difficult to reintroduce the same bug.

Fixes #206

Thanks!


Fix the mangling and AST reconstruction of @isolated(any)

The fix in #71591 was not complete.

@swift-ci Please smoke test


Nativescript/Angular app UI is freezing

I recently converted an old NS6 app to NS8, all the latest libraries.  Noticed in some areas the UI is really slow but even worse, now the UI is freezing upon the very first click, my Login button.  Is there any way I can capture any type of logs that might expose why this is occurring?  It literally just freezes/stops and I have to restart the emulator with ns -avd myavdname -wipe-data

Update! - Switching AVD from Nexus 4 to Nexus 6 appears to have resolved the freezing and Livesync is working now also.  No idea the root cause, perhaps some compatibility issue somewhere in the pipeline.

This has been happening to me as well. Once this freeze happens, all the NativeScript apps within the emulator stop working properly. For me there are 2 kinds of freezes: All the tap bindings stop working, but scroll views and buttons are still functional, and the second kind is when everything stops working. I am yet to catch anything in adb logcat that might be related.

@nikoTM : I am seeing the same behavior, NA app not really freezing but tap events not being processed. However, in my case it only appears to happen in the simulator. Have you found the source of the issue? Or you still think bug in this library?

@cjohn001 I wasn't able to find the cause, but a workaround is to restart the emulator, not the process but the device, just holding down the power button and clicking restart within the emulator. After a restart it never happens again. I don't think this is happening on real devices, I feel like it's an emulator issue but I am not sure.

@nikoTM : Thanks for the feedback. Unfortunately, after a restart it takes only some clicks till the app is going in the same state again.


Enterprise Login Management

Description:
With the B2B Organization Management(#13658) feature an organization should be capable of sharing applications with other organizations to provide B2B services. To provide this, organizations should have the option to share an application with another organization, allowing users of that organization to login to the application to benefit from the services it provides.
The ability to configure applications such that they can be shared with organizations and to authenticate users of organizations that the application has been shared with are requirements that need to be addressed to allow an application to be shared as a resource across organizations.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Renaming Zeit.Hyper to Vercel.Hyper + fixes

ZEIT recently changed name to Vercel Inc. I also took this opportunity to add tags, change the homepage, expand the description and actually add switches that make it install silently.
This solves issue https://github.com/microsoft/winget-pkgs/issues/1436.
Microsoft Reviewers: Open in CodeFlow

The pipeline incorrectly counts a deletion as a second manifest:
****************************************************************************************
Pull Request Validation
****************************************************************************************
Status: Started
Added the following files:
File: manifests/Vercel/Hyper/<IP_ADDRESS>0.yaml
Deleted the following files:
File: manifests/Zeit/Hyper/<IP_ADDRESS>0.yaml
Status: Completed
##[error] Result: Failed
##[error] The pull request contains more than one manifest file.
##[error]Pull Request Validation Failed

****************************************************************************************
Pull Request Validation
****************************************************************************************
Status: Completed
##[error] Result: Failed
##[error] One or more errors occurred. (The pull request contains more than one manifest file.) The pull request contains more than one manifest file.
Finishing: Validate Pull Request

Unfortunately you will have to split this into two pull requests or play around with git commands as described in https://stackoverflow.com/questions/433111/how-to-make-git-mark-a-deleted-and-a-new-file-as-a-file-move

Closing in favor of #1506


Bump version to 2.2.8

Release v2.2.8

We have an outdated entry in NOTICES.txt - golang.org/x/sys-0.3.0 should be golang.org/x/sys-0.8.0. Otherwise LGTM.

Good catch - fixed


Inheritance principle?

Evaluating the current status of the eeg_sleep set with the EEG-bids-validator returns the expected:
~/Desktop/bids-validator/bin/bids-validator eeg_sleep/
This dataset appears to be BIDS compatible.
        Summary:              Available Tasks:        Available Modalities:
        40 Files, 890B        sleep
        1 - Subject
        1 - Session

For testing reasons, I moved the sub-01_task-sleep_eeg.json to the root directory and renamed it to: task-sleep_eeg.json. According to the inheritance principle in the BIDS specification, this *_eeg.json file should now be valid for all subjects that do not have their own *_eeg.json. However, when evaluating the dataset again, I get:
 ~/Desktop/bids-validator/bin/bids-validator eeg_sleep/
The directory eeg_sleep/ failed an initial Quick Test.
This means the basic names and structure of the files and directories do not comply
with BIDS specification. For more info go to http://bids.neuroimaging.io/

This seems like a bug, especially because evaluating ds001 works out fine --> and there we have the same situation that gives rise to an error in eeg_sleep (defining the *_"modality".json at the top level and using the inheritance principle)

when I remove the two task-matchingpennies files at the root level (or when I use .bidsignore), the error still appears. What makes you think it is due to inheritance errors?

when I delete sub-?? and replace those with sub-01 from eeg_sleep (and update participants.tsv), then the validator passes. This is with the top-level task files in place, so it does not seem an inheritance issue.

What makes you think it is due to inheritance errors?

It was a first guess, do I maybe have a wrong idea about the inheritance (see below) ??

this fixed it

Mmhh yes, I figured that fix out as well, but it seems more like a tweak to be honest ... due to the inheritance principle, these files should not be needed:
create mode 100644 eeg_matchingpennies/sub-05/eeg/sub-05_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-06/eeg/sub-06_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-07/eeg/sub-07_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-08/eeg/sub-08_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-09/eeg/sub-09_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-10/eeg/sub-10_task-matchingpennies_eeg.json
create mode 100644 eeg_matchingpennies/sub-11/eeg/sub-11_task-matchingpennies_eeg.json

... because at the root, we have task-matchingpennies_eeg.json, which should be the default for all subjects unless I specify a more concrete *_eeg.json at the subject level.
By adding all these *_eeg.json files for each subject, there is no gain in terms of information. It's just cluttering the directories, isn't it? All the information is already present in the task-matchingpennies_eeg.json at the root.

agreed. I also encountered it just now for a channels.tsv file. The error is not EEG specific (and I am not good enough as javascript programmer), I suggest you report it on https://github.com/INCF/bids-validator/issues


ROCm 1.7.0

As ROCm 1.7.0 uses DKMS, the image needs to be pulled together differently. We bump into a "There is no device can be used to do the computation". The docker-instance is run as root, 1.6.4 worked fine and 1.7.0 works fine on the bare metal.
What are we missing?

This is still not solved in Docker.
As a reference. When running docker from Gitlab CI, you need to add the DRI Devices to the gitlab-runner config:
[[runners]]
  name = "<name>"
  url = "<domain>"
  token = "<token>"
  executor = "docker"
  [runners.docker]
    image = "rocm/rocm-terminal:latest"
    privileged = false
    disable_cache = false
    devices = ["/dev/kfd", "/dev/dri"]
    volumes = ["/cache"]
    shm_size = 0
  [runners.cache]

This is the config for Gitlab CI, that is in line with that. It says privileged = false.

You're misunderstanding. I put the solution as a reference. There is no problem!!

Aha, thanks @VincentSC :-)


regression: npm run dev --port $PORT

npm run dev --port 4000 spawns the dev server on port 3000

Thanks for fixing this here .. just linking it #58


List of API changes for app developers (e.g. files.cat)?

Is there any documentation with a list of the API changes suitable for app developers.
For example ...I just got bitten by the change of files.cat just started returning Buffer instead of a Stream. I know the change was announced back in October in https://github.com/ipfs/interface-ipfs-core/pull/162 but that is in development, so its only now with the 0.27 that it bites app developers.
Is there a full list of API changes from 0.26 to 0.27 so we can check for other issues?
The only other one I know of is that app developers who use WebRTC should switch their config to websockets star.
Have I missed some list ?

We've been announcing new releases -- https://github.com/ipfs/js-ipfs/issues/1095 -- through all of our most used channels (IRC, Github, Twitter and Reddit). These announcements serve as a changelog and a feature highlight so that devs know beforehand what to expect on next release and have time to update.
We also invite everyone that is using js-ipfs to tell us how and where so that we can run the tests before any release. If you have a strong test suite and your codebase always passing on master, then you just have to add it to the list -- https://github.com/ipfs/js-ipfs/blob/master/RELEASE.md -- and I'll let you know which tests failed so that you know what you need to pay attention to.
With regards to the files.cat returning a buffer. That was actually one feature request you submitted back at https://github.com/ipfs/interface-ipfs-core/issues/164 and got accepted with https://github.com/ipfs/interface-ipfs-core/pull/162, that PR shows the diff between the old and new API, you can find the changes there.
As for your last question, I believe you didn't miss anything else :)

Hi David  - yes, no issues with having made that request and it being changed, it just that I didn't realize it had changed (even though I made the request - I think partly I missed it because Issue #164 was closed before the fix was actually done).
#1095 gives a good list of NEW features - which is great, I can add the ones I need incrementally. BUT  can I suggest adding a section on each release that highlights things that have (intentionally) changed, I'm sure it would help app developers find the things that break previously working code. Especially since we can't actually make the change in our code even when the fix is done, only when the release gets pushed.

I suggest adding a section on each release that highlights things that have (intentionally) changed

I like that 👍


Generic Field Types (JSON)

Enhanced the field types to support generic types and not only strings. Changes are backwards compatible.

Coverage decreased (-0.8%) to 96.429% when pulling 7d7081bae98c84c5454ba6da98aa03a12642551e on cbratschi:master into 7ac41957a43069ef440ef89b66841425cb0ca26e on victorparmar:master.

Hello again and thank you for contributing :)


Forcing the reauthentication of a user in T. Accounts

Add support for the parameter "prompt=login" in the request to the authorization endpoint to force a user authentication even if the user has an active session.
http://openid.net/specs/openid-connect-core-1_0.html#AuthRequest
Some critical applications have lower session lifetime that T. Accounts and can require the user to authenticate again.
Best regards.

This is already supported. Actually, we are about to add support for prompt=none. Please follow issues at  https://github.com/TDAF


remove storybook deps

In GitLab by @psimyn on Dec 14, 2017, 05:34

[ ] move component/docs code to static files
[ ] remove stories folder
[ ] remove dependencies

In GitLab by @psimyn on Dec 29, 2017, 14:49
closed


editor.mouseWheelScrollSensitivity set as 0 will disable scrolling completely

https://github.com/Microsoft/vscode/issues/46578
We should probably only allow values > 0
Related: https://github.com/Microsoft/vscode-sublime-importer/issues/14

Values < 0 are often used to simulate an OS feature, namely inversed scrolling. So I suppose we could ignore just the value 0.


Add number of driftignore lines to telemetry

Description
Following #441, we want to add the number of rules (ignore empty lines, invalid lines and comments) in the driftignore to telemetry.

back to product investigation, closing.


Mongo DB 3.x support

When will add support for Mongo 3.x new authentication model?

What exactly "authentication model" do you mean?

Hi Serge,
In Mongo DB 3.08, they changed the “model” for authenticating users connecting with the data base.
What you will find is that DBeaver cannot connect to Mongo 3.08 or later. In my Spring application, I had
to use updated MongoDB Java driver, and I had to change my Spring code to use the “MongoClient” to
set the credentials (username, password, plus connection info) for the DB connection.
This is all documented on the Spring site. If you need some more information, let me know, I can go back
and look up again. Shouldn’t be too big a deal. Would be nice if I can continue to use DBeaver with Mongo DB.
Thanks.
-Thomas
From: Serge Rider<EMAIL_ADDRESS>Sent: Saturday, February 27, 2016 9:18 AM
To: serge-rider/dbeaver
Cc: Thomas Bishop
Subject: Re: [dbeaver] Mongo DB 3.x support (#224)
What exactly "authentication model" do you mean?
—
Reply to this email directly or view it on GitHubhttps://github.com/serge-rider/dbeaver/issues/224#issuecomment-189648213.

Hi @thomas-bishop,
Will Mongo DB change his authentication model on each next versions ? Comparing with previous, is there only "model" that you talked above?

Hi,
I can’t say for sure what MongoDB team will do, but I would not think they will be changing this again for awhile.
I don’t know the details motivating the change, but I would think there was a good reason for doing it.
 Some other tools already incorporate the needed changes.
-Thomas
From: Carlos Alberto Junior Spohr Poletto<EMAIL_ADDRESS>Sent: Tuesday, March 01, 2016 11:08 AM
To: serge-rider/dbeaver
Cc: Thomas Bishop
Subject: Re: [dbeaver] Mongo DB 3.x support (#224)
Hi @thomas-bishophttps://github.com/thomas-bishop,
Will Mongo DB change his authentication model on each next versions ? Comparing with previous, is there only "model" that you talked above?
—
Reply to this email directly or view it on GitHubhttps://github.com/serge-rider/dbeaver/issues/224#issuecomment-190786883.

I'll try to adapt DBeaver for Mongo 3.x in the next version.
Hopefully they have some backward compatibility.

Hi Serge,
How do I get this update. I’ve been unable to use DBeaver for Mongo for some time now.
Thanks.
-Thomas
From: Serge Rider<EMAIL_ADDRESS>Sent: Friday, March 11, 2016 1:39 PM
To: serge-rider/dbeaver
Cc: Thomas Bishop
Subject: Re: [dbeaver] Mongo DB 3.x support (#224)
Closed #224https://github.com/serge-rider/dbeaver/issues/224.
—
Reply to this email directly or view it on GitHubhttps://github.com/serge-rider/dbeaver/issues/224#event-586790079.

@thomas-bishop you need wait for 3.6.2 release
You can see approximate release dates in milestone details.
https://github.com/serge-rider/dbeaver/milestones

Okay. Looks like it is behind schedule, so please let me know when it is released.
Thanks!
-Thomas
From: infectormp<EMAIL_ADDRESS>Sent: Friday, March 11, 2016 3:27 PM
To: serge-rider/dbeaver
Cc: Thomas Bishop
Subject: Re: [dbeaver] Mongo DB 3.x support (#224)
@thomas-bishophttps://github.com/thomas-bishop you need wait for 3.6.2 release
You can see approximate release dates in milestone details.
https://github.com/serge-rider/dbeaver/milestones
—
Reply to this email directly or view it on GitHubhttps://github.com/serge-rider/dbeaver/issues/224#issuecomment-195535979.

3.6.2 will be released around March 14.
There was a bad date in the milestone info (dunno how it happened).

Thanks Serge! Looks good.
From: Serge Rider<EMAIL_ADDRESS>Sent: Friday, March 11, 2016 4:32 PM
To: serge-rider/dbeaver
Cc: Thomas Bishop
Subject: Re: [dbeaver] Mongo DB 3.x support (#224)
3.6.2 will be released around March 14.
There was a bad date in the milestone info (dunno how it happened).
—
Reply to this email directly or view it on GitHubhttps://github.com/serge-rider/dbeaver/issues/224#issuecomment-195563455.


improvements to ephemeral messaging

puts it inline with the SendMessage API.
also starting to lay the ground work for the send message api to support response URLs from things like slash commands.

lgtm


docs(css-reset): Adjust docs to fix things that were not rendering properly

Small update to colours docs too
Removed instructions for installing packages, as that belongs in getting started docs, not styleguidist docs
Also updated a Tooltip snapshot...

@lzcabrera Unless the PR is something really impactful and complicated, I think that if you review and "approve", then you can just go ahead and merge it in right away. No need to wait for other reviewers I think for normal PRs.
If you have comments of things that should be changed, then please mark it as "request changes" instead of approving it so the submitter knows to change something.
@theetrain does that make sense?


Invalid error message

In this message it indicates that the queue name is not provided, but it is actually the queue length
https://github.com/kedacore/keda/blob/ce639ecc577d922c6b55f02494e0d973fc2722cb/pkg/scalers/rabbitmq_scaler.go#L81

Closed in #355
thanks @joskfg!


[pulsar-test] enable pooled netty allocator for unit-tests

Motivation
It fixes #11750 which has flaky test that fails because test checks if consumed payload is allocated in direct-memory and it doesn't give a guarantee to allocate message in direct-memory as default allocator is not pooled and therefore, sometime client accepts messages in non-pooled (heap-memory) allocator and test fails to validate message buffer type.

It looks like there's a test which isn't compatible with pooled allocator. This test is failing:
Error:  Tests run: 7, Failures: 1, Errors: 0, Skipped: 6, Time elapsed: 4.129 s <<< FAILURE! - in org.apache.pulsar.broker.service.TransactionMarkerDeleteTest
Error:  testMarkerDelete(org.apache.pulsar.broker.service.TransactionMarkerDeleteTest)  Time elapsed: 0.018 s  <<< FAILURE!
java.lang.UnsupportedOperationException: direct buffer
	at io.netty.buffer.PooledUnsafeDirectByteBuf.array(PooledUnsafeDirectByteBuf.java:226)
	at org.apache.pulsar.broker.service.TransactionMarkerDeleteTest.testMarkerDelete(TransactionMarkerDeleteTest.java:122)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)

It can be fixed by changing line 108 in TransactionMarkerDeleteTest from ByteBuf payload = PooledByteBufAllocator.DEFAULT.buffer(0); to ByteBuf payload = PooledByteBufAllocator.DEFAULT.heapBuffer(0);.

I pushed the fix to TransactionMarkerDeleteTest to this PR since it would be nice to get this PR merged asap to improve CI stability.

I hope that this change won't introduce instability due to other leaks in the tests

I agree. one of the issue with  -Dpulsar.allocator.pooled=true is test starts leaking due to test-failure or missed to close/cleanup broker/client then we will see memory failure soon. So, I was thinking to revert back -Dpulsar.allocator.pooled=false and keep direct-memory check in if condition but before that this PR is merged now. let me create a PR and we can merge it if that makes sense.


Capture Java Thread ID

When running an application within a Web Container (i.e. Jetty), is there a way to capture the Java Thread Id in which the transaction was running on? I am looking to take transactions that have been identified as slow in PinPoint and trace them back to our Jetty logs using the Java Thread Id. This would be useful in tracking down any logs we have within our application to help further track down performance issues.

Hi @Ryner51
While we don't capture thread ids, you might find this useful.
What it does is add transaction ids and span ids to your log so that you can correlate what's shown in pinpoint back to your log.


Possible memory leak in scene geometry data structures

I suspect a memory leak in OSPRay's scene geometry handling. Hope you can help.
It would be great if you could take a look at the attached minimal test case, which reproduces the issue. I originally noticed a constantly growing memory footprint in my application code, which uses OSPRay for rendering an animation with changing scene geometry. I then managed to boil it down to this very simple and self-contained test case.
The test code creates a "mesh" GeometricModel consisting of 10,000 triangles. In a for-loop, it then repeatedly wraps the GeometricModel in a Group, wraps the Group in an Instance, which is then inserted into a new World. Then all objects get released at the end of the loop and the process repeats.
During each loop iteration, the memory footprint of the running process grows by several megabytes, which can be observed in the OS task manager, suggesting that the world's internal data structures do not get properly released. Note that the minimal example does not involve any rendering, just setting up the world.

Tested with OSPRay master branch and v2.1.0.
ospcommon 1.3.1
Embree v3.9 and v3.10
Operating system: macOS 10.15.4

test_case.zip
EDIT:
I should mention that I already made my own unsuccessful attempt to debug the problem (by inserting a couple of print statements into the OSPRay code). But I could not identify the origin of the memory leak. I got the impression that all OSPRay class destructors get properly called upon releasing the World object. So the actual problem may lie deeper, perhaps inside the Embree library or the code building the acceleration data structures.

Thank you for the info, we have done some experiments and can confirm that indeed there exists problem with memory leaking. Please expect a fix soon.

Thank you.
While you are at it, I would like to ask you to take a closer look at this particular code block:
https://github.com/ospray/ospray/blob/master/ospray/common/Data.h#L291-L302
I am not 100% certain, but during my own bug search, I got the feeling that the ad-hoc Data object created by this code may leak. Note that this problem is not related to the memory leak reported above, but it should probably be fixed as well.
The problem occurs when assigning a single OSPMaterial to the material field of a GeometricModel. It can be worked around by first wrapping the material in a Data array of length 1 before assigning it to the material field.

Confirmed with

ospray master c902dca37f165e01365e51b6bba107dcc53a6842,
openvkl 0.10
rkcommon 1.4.2
embree 3.11.0

compiled with gcc10 on ubuntu 18.04
I have project which is very similar to the minimal example provided by @miroslawpawlowski, the problem seems to be the same.
Valgrind finds:
Indirect leak of 240 byte(s) in 5 object(s) allocated from:
    #0 0x7f90b05febb7 in operator new(unsigned long) (/usr/local/lib64/libasan.so.6+0xadbb7)
    #1 0x7f90ae3dce6d in rtcSetSharedGeometryBuffer (<embree_root>/lib/libembree3.so.3+0x205e6d)
    #2 0x7f90a6389d27 in ospray::Mesh::commit() (<my_project_root>/ospray/lib/libospray_module_ispc.so+0x129d27)


Add Cloud Vision API

The Google Cloud Vision API recently went to beta.  It would be great to add API support:
https://cloud.google.com/vision/docs/
There's an existing NPM module here:
https://github.com/tejitak/node-cloud-vision-api

Check it out! https://github.com/GoogleCloudPlatform/gcloud-node/pull/1136 :)

Great to see this coming along!


Media Resolver health check requires ListAllBuckets permission

The health check uses the ListAllBuckets method to check if the S3 credentials are correct.
This requires access to all buckets of the account. In order to follow the Principle of Least Privilege, the health check should try to put a file in the exact s3bucket and in the s3path as specified in the media-storage config map.

Hello @pascal-airy, I just opened a PR for this issue, the health check now list objects in the s3bucket.
I don't think it is a good Idea to try to upload an object every time the heath check runs.
The only issue of this approach is that this only tests the read privileges.

Thank you @juan-sebastian !
You are right that uploading is not necessary but could you only list the objects in the s3path because I want to limit each customer's access to their own folder.

Ok I can do that 👍


OSError in s2s_pipeline.py, line 42

nltk.data.find('tokenizers/punkt_tab')
throws an OSError on my system
Thanks,

This should fix it: https://github.com/huggingface/speech-to-speech/pull/29
Please reopen if it didn't.
Thank you for raising.


Two different versions

Hello. Dear admin; @SoftCreatR thanks for imei.
Ubuntu 20.04 installed on my vps server. I also installed the imagick program with apt. (8: <IP_ADDRESS> + dfsg-2.1ubuntu11.1)
then I installed imei. installation was smooth. but he still sees the old imagick program on the site.

I will be glad if you help have a nice day.

Though it's not mentioned anywhere, it's advised to uninstall any previously installed ImageMagick version first to avoid such issues.
As soon as you made sure, that there's only one Imagemagick version installed (preferably the one that has been installed using IMEI), you may also compile the Imagick PHP module from source, e.g.:
apt install php-dev
cd /usr/local/src
git clone https://github.com/Imagick/imagick.git
cd imagick
phpize
./configure
make
make install

Thank you. 👍
Regards, Greetings from Turkey.


[BUG] Access is Denied

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
I started my computer. and WingetUI autoupdate started. all good up to now. but the installer gave me every time "access is denied" and i started screaming. good. after that, i started looking if some file was using the files inside wingetui, like last time #1125 but no. after some more screaming i went to the installation folder. I right-clicked, selected Properties, then Security... and I noticed that I didn't have any permissions for that folder. EVEN AS AN ADMINISTRATOR.  Then I right-clicked on the entire folder, chose "Take Ownership," and in the end... the program updated without any issues.
Steps to reproduce the issue
i don't really know. it was a problem with the folder, not the program itself..
WingetUI Log
I cannot provide it, the program crashes (Not Responding) when I try to export the file. And the "copy to clipboard" function is not working.

Package Managers Logs
I cannot provide it, the program crashes (Not Responding) when I try to export the file. And the "copy to clipboard" function is not working.

Relevant information
Nothing strange
Screenshots and videos

WingetUI can't be installed in a directory that does require administrator preferences due to the requirements of certain components of the package managers.
Please install WingetUI on the default location

@marticliment
Thanks for the information. I adjusted the NTFS permissions to enable write access.


Finish Post UX.

From morpho-app created by neema-orual: Radiant-Industries/morpho-app#4
Embeds, buttons (like, repost, reply), post stats, blocked post and not found post UX

Progress

Big post (when you click on a post to open it) UX done-ish.
Image embed UX should work, at least for thumbnails.

Still need to finish/link up the actual image viewer.

Link card UX should work (even open in browser on click).
Blocked/deleted quote posts as well as quotes of text-only posts probably working
Working on logic for post embeds with their own images/link cards, looks like it needs to ask for the full post from the ref to get the embed content.

UX done except for label and content filtering.
Bunch of behaviour still needs implementing.

Note to self, go back and get all the facets stuff working.

Facets mostly working except positioning is sometimes incorrect (see https://github.com/morpho-app/Morpho/issues/31).
Content filtering initial implementation working (note lack of genitals):

Still to do:

display non-filtering labels on posts (plus more stuff in the filtering backend)
show stats, who liked, etc. on right-click/long-press
Wellness Features: https://github.com/morpho-app/Morpho/issues/21


chore(main): release 0.1.2

:robot: I have created a release beep boop
0.1.2 (2023-06-21)
Bug Fixes

change default cov id (#44) (b344521)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/socialgene/sgnf/releases/tag/v0.1.2 :sunflower:


Managed Instance applicability

The current info at top of page indicates that this doesn't apply to MI, but there is a link from the MI auditing doc below that points to this page. ("For further details about the hierarchy of the storage folder, naming conventions, and log format, see the Blob Audit Log Format Reference.")
https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/auditing-configure
Can we clarify the applicability here? This doc is also positioned in the ToC under SQL DB only, so it may need to be moved to shared docs.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f4adb3ca-e7b5-648c-5389-904af0a1af87
Version Independent ID: f086acf8-2414-2a4f-3e1a-9ab787d894be
Content: SQL Database audit log format - Azure SQL Database
Content Source: articles/azure-sql/database/audit-log-format.md
Service: sql-database
Sub-service: security
GitHub Login: @DavidTrigano
Microsoft Alias: datrigan

@dcbrown16 The same document mentions there are some inherent differences between how auditing is done in Az SQL DB and MI.
https://docs.microsoft.com/en-us/azure/azure-sql/managed-instance/auditing-configure#auditing-differences-between-databases-in-azure-sql-managed-instance-and-databases-in-sql-server
This is the reason both have been kept separate and discussed separately.
Hope this helps.

I'm not asking about that page, I'm asking about the page I reported this on.
https://docs.microsoft.com/en-us/azure/azure-sql/database/audit-log-format
The problem is that this page is marked as only for SQL DB and DW, but it also includes info relevant to MI, so it needs to be marked and positioned in the ToC as so.

Hello @dcbrown16,
Many thanks for your feedback.
We proceeded with your proposal and you should be able to see MI within 24 hours in the section applies to of the documentation SQL Database audit log format.
Best,
David T.
Program Manager
Azure Data Security

@NavtejSaini-MSFT you can go ahead and close this issue

@DavidTrigano Thank you for the change. Please share the PR if created for this issue.
@dcbrown16 We will go ahead and close this issue for now.

@NavtejSaini-MSFT PR #117557


Update README.md

clarify the arguments returned in the 'gridster-resized' callback, and make it easier for users to copy/paste example code and have it work out of the box.

Thank you!

Of course -- thanks for your work maintaining!


Slow Compile on Pages

Just installed it on a fresh install (first use of 5.2) - and every file change is taking ages to re-load in the browser (30 secs +). I expect slow loading when editing .scss files - however it is taking this long to reload every single file edit - i.e. single.php. Seems like compile is forced on every page reload - even though I have ensured that define('WP_ENVIRONMENT_TYPE', 'development'); is not set. Any ideas?

Please check here https://github.com/bootscore/bootscore/issues/252
If you are using LocalWP, this comment might be helpfull https://github.com/bootscore/bootscore/issues/252#issuecomment-1332001484

Closed as duplicate https://github.com/bootscore/bootscore/issues/252


Add Dassault Rafale from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of LuizChacon, to add Dassault Rafale to CKAN.
Please direct questions about this pull request to LuizChacon.

part of #1860


<IP_ADDRESS> shapeblue1 patch

PR for purpose of checking commits only

@blueorangutan package

@weizhouapache BO doesn't work on forks yet, we'll need to manually create pkg and run smoketests in the lab.


angular-messageFormat select statement bug

I have the following in my template
<span>
   <!--i18n: Gender fun-->
   {{model.gender, select,
         male   {He gave me a cactus}
         female {She gave me a cactus}
         other  {They gave me a cactus}
   }}
   <!--/i18n-->
</span>

And I am getting an error that says 'other is a required option' with stacktrace:
at http://localhost:1080/static/lib/angular/angular.min.js:6:434
    at new MessageSelectorBase (http://localhost:1080/static/lib/angular/angular-messageFormat.js:102:11)
    at MessageSelectorBase.SelectMessage (http://localhost:1080/static/lib/angular/angular-messageFormat.js:165:23)
    at MessageFormatParser.ruleSelectKeyword (http://localhost:1080/static/lib/angular/angular-messageFormat.js:577:21)
    at MessageFormatParser.run (http://localhost:1080/static/lib/angular/angular-messageFormat.js:451:12)
    at interpolate (http://localhost:1080/static/lib/angular/angular-messageFormat.js:891:14)

I tried copy-pasting the example from the docs for the messageformat pr and got the same result.

@chirayuk

I should mention also that the declarative plurals are always going to other. It looks like the MessageFormatParser objects are somehow borked and this.choices returns {}. I am on ea9fd82.

Yup.  This is a bug.  I'll be fixing it in the next day or two.


[Docs] Typo in the 32-bit support Post-Install README

https://github.com/secureblue/secureblue/blob/live/docs/POSTINSTALL-README.md#32-bit-support

or run an ocassional application in Wine

PRs welcome 😄


Tried to handle a committed transaction in the pool but its nonce doesn't match the head of the queue for that sender

Client was just running, I did nothing.
coda_crash_report_2020-01-25_14-25-59.143175.tar.gz

Dup of #4298. Closing.


Outlook encrypt attachments sample

Q
A

Bug fix?
no

New feature?
no

New sample?
yes

Related issues?
no

What's in this Pull Request?
This sample showcases how to use event-based activation in an Outlook add-in when the user composes an email or appointment/meeting request. It demonstrates how to run tasks based on events that fire when certain data changes when the user:

adds an attachment to an email or appointment/meeting request
adds recipients or distributions lists as required or optional attendees in a meeting request
changes the start or end date or time in an appointment/meeting request
adds a notification message to the item when a user newly composes an email or appointment/meeting request, instructing the user to open the Task Pane for further information (for this sample, it shows instructions and details for this sample!)

@elegault can you please add the .vscode folder to the gitignore file so the VS Code files drop out of the PR? Thanks!

@elegault can you please add the .vscode folder to the gitignore file so the VS Code files drop out of the PR? Thanks!

Hi @davidchesnut: Sure thing. But doing that doesn't seem to make any changes that I can push, and excluding the .vscode folder creates a modification to the .gitignore file at the root of the PnP-OfficeAddins repo. Should I delete all of the files in the .vscode folder in this PR instead?


Provide Miner Statistics

Description
Miners need a method to determine their success/attempts. This allows for monitoring and feedback to ensure a miner is properly working. The initial idea was to use balances to track the progress, however with the Goal of FAT integration, it would be a lot of code duplication at this phase.
This ticket should serve as the discussion page for designing the statistics needed to provide a miner.
Steps

Design the spec
Create the issues associated with implementing the design

Conditions of Satisfaction

A set of issues that detail the statistics/data needed to support a miner
The issues should be categorize by milestone and tagged appropriately

Fortunately we don't need to track balances at all to do this, miners can calculate this entirely from scanning through/grading the OPRs via the list of winners. We just have to check the payout address of the 10 winning oprs to determine how well they did historically.
Things I would like to track and eventually have visible to miners:

How many OPRs there are total
A full breakdown of where the miners landed in the range (ie top10, top50, or below). This is the meat of the matter and should let you know if your miner is competitive, whether competition has grown or shrunk, etc. For the top10 we should provide 10 different points but for the rest a percentage is probably enough.
Overall payout (can be calculated dynamically from all the wins)

How we actually track this is still up in the air. We can do either by payout address or by miner id (later did), not sure which one is better.

Since payout address will later be resolved from the miner's DID, I think we should track by the miner-ID
Agree with the stats you've listed out so far. Those should be sufficient for now, just to show we have mining and grading working

I'm having a hard time figuring out how exactly the format of an api response would look for #2 in the bullet list. I'd suggest a simple counter?. This is what the popular battle royale games like Fortnite do their for performance reporting for a given user under the hood. Feels pretty similar to our mining.
{
    ...
    placements {
        "0": 1,
        "1": 2,
        "2": 0,
        "3": 1,
        ....
        "50": 10,
        "50+": 102
    }
    ...
}

I think this would be pretty simple, and I'd avoid using percentages for performance, as they can get drowned out by volume. Eg, if the miner wins 10% of the time for months, and then starts losing, the percentage will be slowly dragged down. If we use hard counters, the user can make the determinations from the raw data.
I think it would also be beneficial to be able to query for a counter for a given minerid at a given height. So a miner can eval block ranges.

Yeah I imagine we could structure it similar to how prometheus does histogram buckets. Where each bucket is cumulative (i.e. # of submissions with rank < 5, # of submissions with rank < 50, etc).
{
	"results": {
		"block_range": {
			"start": 0,
			"end": 2000,
		},
		"placements": {
			"1": 5,
			...
			"10": 100,
			"25": 200,
			"50": 300,
			"100": 400,
			"250": 500,
			"500": 600,
			"1000": 700,
			"2500": 800,
			"5000": 900,
			"10000": 1000,
			"+Inf": 1500,
			"count": 1500,
			"sum": 125000,
		},
		"total_rewards": 1337,
	}
}

The "count" would be total submissions, and the "sum" would be the cumulative sum of all ranks so that you can do sum / count to get the average rank for that block range.
And I think it might be wise to include the total rewards rather than let the user compute that themselves. Although rewards are fixed for now, maybe in the future it becomes something like bitcoin where reward amount follows some curve. Or maybe that's never going to happen? Idk, just thinking out loud

I have no objection to being cumulative.
Would the total rewards be in PNT?

I imagine it'll be denominated in PNT. Not sure if there are any plans to allow for automatic conversion of rewards to a different asset type, or if that will always be up to the user to do at a later time

Since all the data remains intact we can add very elaborate stats in the future, but specifically for milestone 1, I'd like to suggest the following two API calls:
result:
input: block height (default = latest), which checks the oprs contained in that block (the winners are set in height+1 or in grader memory for the latest)
returns:

payout the amount of PNT you won
position a list of all the OPRs you submitted that block, with entry hash, miner id, position as int with 0 being the #1 winner

AND
performance
input: choice of how many blocks to go back in range, or the option of something like 1 block, 6 blocks (hour), and 144 blocks (~day)
returns: three separate cumulative stats in the format of:

count total number of OPRs submitted during that range
payout the amount of pnt you won during that range
placement similar to sam's excellent list from above, this would be a breakdown of how many wins by 1-10, top50
hash power count of how many of your OPRs are above average difficulty and below average difficulty

miners should also receive feedback via console of:

how many OPRs they submitted last block
how many of those won
pnt payout

I imagine it'll be denominated in PNT. Not sure if there are any plans to allow for automatic conversion of rewards to a different asset type, or if that will always be up to the user to do at a later time

that definitely smells of wallet feature!

I agree with those two calls. 👍

result:
input: block height (default = latest), which checks the oprs contained in that block (the winners are set in height+1 or in grader memory for the latest)

to do this we need to parse/grade the whole mining chain every time a request is made right? or add the database functionality?
since in order to validate the current block, you need winners from previous, and so on

to do this we need to parse/grade the whole mining chain every time a request is made right? or add the database functionality?
since in order to validate the current block, you need winners from previous, and so on

validation happens when pegnet is running normally. every factom minute 1 it calls GetEntryBlocks which queries and verifies all eblocks, saving the oprs to global variable OPRBlocks. GetPreviousOPRs(height) gets the most-recent block from that list, so all the necessary data is cached.
the only thing we'd need to add in that regard would be getting the live winners for the currently-being-built OPR, which we could set in the Grader's run() (where it also sends the alerts)

We should really get some organization into objects vs globals. It's hard to find all these things.

For hash power I think we should also have a placement thing. We want to know if we are in or out of the top 50, as the top 50 make it into the grading (right?).
So since graded only grades the top 50, and hash power has a list of all OPRs, we can have that list have larger buckets past 50. Since we want to get into the top 50, where graded we want into the the top 10. So having granularity near those milestones is what we need.
{
	"results": {
		"block_range": {
			"start": 201000,
			"end": 2012500,
		},
		"graded": {
			"1": 5,
			...
			"10": 100,
			"25": 200,
			"50": 300,
			"+Inf": 1500,
			"count": 1500,
			"sum": 125000,
		},
		"hash_power": {
			"1": 5,
			...
			"10": 100,
			"25": 200,
			"50": 300,
                        "100": 400,
			"250": 500,
			"+Inf": 1500,
			"count": 1500,
			"sum": 125000,
		},
		"total_rewards": 1337,
	}
}


java.lang.AssertionError: data were not fully read, check your serializer

The problems described in #515 still seem to exist in version 1.0.9.
Env:

Java(TM) SE Runtime Environment (build 1.8.0_73-b02) 64-Bit
Windows 7

Details:

The stack traces are the same as in the referenced ticket.
The db probably got corrupted after trying to write to the db with no free memory left on the used file system.

1.0 branch is no longer supported, sorry :(


Add sailing mode like Al dente. Also the magsafe support, when battery is at set battery level, it will turn green and stop charging.

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

My MagSafe light is green and charging is limited to 80%. I think that may already be included.

I think what @Confidack is asking for is to let the battery fluctuate between say 70% and 80% in order to avoid trickle charging.

+1

Issue template is not decorative.


Fix inconsistency in multi-attribute values in user info

Describe the improvement
In earlier user response was similar to
{ "sub"<EMAIL_ADDRESS>"groups": "Internal/everyone,dev", "given_name": "new", "family_name": "user", "email"<EMAIL_ADDRESS>}
But with new configurations, it will be
{ "sub"<EMAIL_ADDRESS>"groups": ["Internal/everyone","dev"], "given_name": "new", "family_name": "user", "email"<EMAIL_ADDRESS>}
multi-valued claims will return as a JSON array.
Public git issue - https://github.com/wso2/product-is/issues/6575
Suggested Labels:
migration

Added to docs with https://github.com/wso2/docs-is/pull/2991, https://github.com/wso2/docs-is/pull/2997


Display of creating new account with Ledger

Problem
Currently, users cannot create new account with ledger in Google Chrome. However, they may not know it and create account as usual. Finally, after they made a deposit, they cannot recover the account and lose tokens.
Expected Behavior
From the beginning, let users know the Google Chrome is not supported for Ledger in this page.

Steps to reproduce

Click create new account
Set up an ID
Select Ledger hardware wallet
Connect ledger device
Make a deposit to the single-used funding address
Cannot recover the account in Google Chrome

there's an update that will solve this problem, enabling Chrome again. cc @MaximusHaximus who will link here the PR


Ticket#7/forgot password

@mosefer Code Review:

when resetting redirect to a confirmation page with a larger text;

confirmation page should display should email, something like "We emailed password reset instructions to<EMAIL_ADDRESS>have a button that takes you to the login page
redirect you to the login after the existing timeout

similar behavior after the password was reset
add cypress test to validate UI handling of once the token/url is used it's not valid anymore
allow users to configure timeout without specifing PT prefix
db.changelog-master.xml - changeset 2 has not be released yet - roll changeset 3 into 2
consider enhancing DataMigrationDBIT.groovy to test creation of  password_reset_token table
on system settings please add help icon with a quick explanation when this url will get used and why user had to specify it
implement cypress tests for the system and email configuration tests
add backend integration tests for email and system settings

@mosefer thanks for taking care of all the items, few items were forgotten:

on the password reset page - do not go to the error page if a user enters a non-existent email; display the same message is when the token is not correct; will need a cypress test for that as well
do not error if the email and system settings are not configured; redirect to the page that explains that this feature is not yet enabled and encourage to reach out to the SkillTree admin
display/set default value in the Settings > System > 'Password Token Expiration' input field
decrease redirect timeout period; try 10 seconds if it feels too short please try 15 seconds
got the following vee-validate error while running cypress tests with the console:

VM10710 vee-validate.esm.js:317 Uncaught (in promise) Error: [vee-validate] Validating a non-existent field: "#3". Use "attach()" first.
    at createError (VM2292 vee-validate.esm.js:317)
    at Validator._handleFieldNotFound (VM2292 vee-validate.esm.js:3695)
    at Validator.validate (VM2292 vee-validate.esm.js:3274)
    at ScopedValidator.validate (VM2292 vee-validate.esm.js:2747)
    at fn (VM2292 vee-validate.esm.js:2393)
    at later (VM2292 vee-validate.esm.js:230)

@sudo-may addressed requested enhancements, please take a look at FeatureService.isPasswordResetFeatureEnabled, after adding the email sender test, I'm concerned that may be too slow to do every time a user requests a password reset?  It also strikes me that someone could maliciously hammer that endpoint and cause stability issues with the underlying mail server. Thoughts?

@mosefer

please change /isFeatureSupported endpoint to /public/isFeatureSupported


Update index.md

I could be wrong, but pretty sure it's meant to be common_languages instead of common_locales

No, I think locales is right
common_locales is country codes like en_GB -> United Kingdom
common_languages is language codes like en -> English

Hmm, though the one above looks wrong...
This is confusing docs as they both refer to language when one is locale...

If you could amend this PR to make that aspect of the docs more clear and correct the one above, that would be super helpful!

@dhensby in the absence of that improvement, shall we just merge this?

We can make the change when we merge by hand :P


Agglomerative Clustering

Adding agglomerative clustering plot and heuristics
Added relevant test modeling idealized outcome, reference baseline_images directory

@hcho1111 please fix issues, make sure the CI is green and tag the team for review.