Resource Manager - List view option, bulk actions and usability

Description of problem
The current view of the folders and files in the new RM of DNN 9.8.0 is very difficult to visually navigate and manage. The actions are hidden (and not labeled, see #4384) and even relatively short file names are unnecessarily even more shortened, unless you hover on each one. Plus, every card is very bulky, so even a folder with a dozen files or little more becomes unnecessarily "long".
It doesn't help that files and folders have cards with the same background (and poor contrast with the text), and the image previews are mostly hidden (see screenshot 1).
Description of solution
Provide an option to switch to/set a more readable and compact list view, with already visible actions+labels, where the file/folder names are shortened only above 60-100 characters (or so) and the file type always visible.
Include a checkbox to select files/folders to perform actions, such as move and delete, on multiple files at once.
Ideally, the list view could also include "date added/last modified" columns.
Description of alternatives considered
Not ideal as a standalone alternative, but the "cards" view could also be improved by:

improving contrast and information hierarchy (when I have several folders or non-image files of the same type, their name is more important than the icon)
increasing file name limit before shortening (the font could be also smaller, once contrast is improved)
making extensions always visible for files
adding "order by" file type
making the action icons always visible; they could be moved on top instead of on the right side to make space for the filename and just slightly reduced in size; obviously add labels/tooltips to the actions, as suggested in #4384
widening the clipping of the file preview, where applicable
adding a selection checkbox to each card for bulk actions
taking more advantage of responsive styling and flexbox capabilities instead of absolute positioning and fixed widths/heights for the cards and their contents

(see screenshot 2; it's just a quick edit via browser inspector, I know it's visually more busy and can be improved, but again, I still consider the list view option much better)
Screenshots
Screenshot 1 (current view)

Screenshot 2 (modified view)

Additional context
A use case where all of this becomes a serious hindrance: a content editor, who needs to regularly upload and manage a significant amount of pictures or other files. They might need it for sharing or to perform regular massive content uploads/updates in another extension (eg. news articles or e-commerce catalogs). They realize they erroneously uploaded 100+ files, but all the jpegs should have gone into another folder, others should be deleted and a few should stay where they are. All the file names start with a similar string with an identifier at the end. All the previews look very similar. The editor now has to hover over each box to verify what file it is and perform the appropriate action one by one.
Affected browser

Doesn't seem to be a browser-related issue

I would vote that this definitely needs to stay open and get fixed.

I guess this issue doesn't get much traction as it is quite a lot. @donker would it be an idea to create several smaller items for this?

If someone wants to split this up into more granular issue it would certainly help for management yeah.

I'll take a crack at splitting this in to multiple issues... if a) no one else has started already and b) someone does a cleanup so its closer to standards after I am done hacking away.

@valadas and @Tychodewaard - Okay, done for now. Maybe someone could spend a few minutes to verify that I didn't lose anything in the split? Happy to revise things further if needed. Cheers!

@jeremy-farrance Thank you very much!

@Valeria-B I think @jeremy-farrance did a great job splitting up this issue. I propose to close this one as it has all been accounted for in the issues mentioned.

@Valeria-B I think @jeremy-farrance did a great job splitting up this issue. I propose to close this one as it has all been accounted for in the issues mentioned.

yes absolutely!

Awesome, I did not have the time to read all the new issues and their comments but, smaller items are usually better :) Thanks for that splitting up effort.


self-hosted: Document FunctionMode env vars

ServiceFunctionMode determines the mode in which the service runs based
upon the env variable "PULUMI_API_CONTAINER_MODE". If this value is not defined,
then it defaults to "FULL". If the value does not match a known type, it defaults
to "FULL".
Usage:
ServiceModeFull: Runs both the Pulumi API & Background Jobs
ServiceModeAPI: Runs only the Pulumi API
ServiceModeBackgroundJobs: Runs only background jobs
Signed-off-by: Steve Sloka<EMAIL_ADDRESS>

@stevesloka in the future if ya dont use a fork, we can run the tests on github


Runing code sample on SignalR and received HTTP Error 502.5 - Process Failure.

I am using VS2017 with IIS Express. Tried quite few of code samples on SignalR and all received HTTP Error 502.5. Am I missing steps or configurations. Thanks.

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

ID: 0dd2991d-c22d-ad0f-80b6-bd1b503db8f3
Version Independent ID: 5e3fbe1d-c97f-9940-4f0c-067d4b2112b1
Content: Get started with SignalR on ASP.NET Core
Content Source: aspnetcore/tutorials/signalr.md
Product: aspnet-core
GitHub Login: @rachelappel
Microsoft Alias: rachelap

You'll need to gather logs from IIS to figure out why the process isn't starting.

Hi BrennanConroy, I have found that the code samples works for version 2.1.0-rc-final of "Microsoft.AspNetCore.App", not for version 2.1.1 (which thrown HTTP Error 502.5). I have also notice that in the code sample. the version of Microsoft.NetCore.App is also 2.1.0-rc1. I tried to match the version to 2.1.1 for both packages but failed due following error:
uninstall-package : Cannot modify an evaluated object originating in an imported file "C:\Program Files\dotnet\sdk\2.1.300-rc1-008673\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.Sdk.DefaultItems.props".
At line:1 char:1

uninstall-package Microsoft.NetCore.App -version 2.1.0-rc1

  + CategoryInfo          : NotSpecified: (:) [Uninstall-Package], Exception
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PackageManagement.PowerShellCmdlets.UninstallPackageCommand

Any comments? Thanks.

What sample are you talking about? The sample for the Getting Started page uses version 2.1.1.
As for your error stackoverflow has an answer https://stackoverflow.com/questions/35855186/cannot-modify-an-evaluated-object-originating-in-an-imported-file


[whitelist.js] Make "paths" and "files" fields optional

So that we keep backwards compatibility.

Did you test it locally?
If not, you can do a npm install --save /path/to/repo/locally and run it on an old doc branch, to double check

I tested it locally, yes.


Error message upon load

Please remove unknown errors generally - improve error messages.
Remove this one which appears a couple of days.

Tx!

Error should not appear again.
Unknown errors removal - pending.


In a ip access-list configuration there is a highlighting problem with the argument "neq"

Hello,
When we write an udp sequence in an access-list, the argument used to match a specific port "eq" is highlighted normally but the argument "neq" seems to be unrecognized and the entire line become unlighted
You can see it on this screen capture

It is possible to fix this ?
Thanks a lot
Léo.

thanks for reporting :)


cannot  set skipCacheInvalidation option

in order to use the skipCacheInvalidation option, we need to pass it as a command line option parameter, but it no longer supported.
how can I still use this option?
also I think skipCacheInvalidation should be part of the config

This has been removed in 0.78.0. No cache invalidation for es modules


polybar not fully showing, it just shows the workspace number and time. (like its cut off or something)

https://imgur.com/a/xUnNmLo

https://imgur.com/a/xUnNmLo

Hello @ins0mniacdude,
I have no idea what's going on there. Did you manage to find a solution? If yes, then please share it in case anyone has the same issue again.

Hello @ins0mniacdude,
I have no idea what's going on there. Did you manage to find a solution? If yes, then please share it in case anyone has the same issue again.

no i did not find a solution sadly.

Same. Looks like mate-power-manager isn't working as expected.


Clarification needed on post and pre computation in code

Hello,
I've come across this segment of code and I'm trying to understand its intent and functionality:
argsort = solution.argsort()   loc = x_in[:, :, :2]  post = loc.gather(1, solution.view(bs, -1, 1).expand_as(loc)) pre = loc.gather(1, argsort.view(bs, -1, 1).expand_as(loc)) post = torch.norm(post - loc, 2, -1, True) pre = torch.norm(pre - loc, 2, -1, True)
What's post and pre means?
I try get it as a next point and last point of current point....
From the comments and variable names, it seems the goal is to compute the coordinates of the next (post) and previous (pre) nodes for each node based on the given solution. However, based on the behavior of loc.gather(...), it seems the current implementation does not achieve this. Instead, it looks like it's rearranging the nodes based on the solution order.

Hi Visterainer,
Thanks for your question. You are correct that the goal is to compute the coordinates of the next (post) and previous (pre) nodes for each node based on the given solution. We can achieve this by rearranging the nodes because the solution variable "solution" encodes the order information in a linked list format. If solution[i] = j it means that node j is the next node of node i (also node i is the previous node of node j). Thus rearranging the solution can compute the difference between the coordinates of the pair (a node, the next node) and the pair (a node, the previous node) during "post - loc" and "pre - loc".
You may find more info on the linked list format in the readme and the provided jupyter notebook. Thanks!

Thank you, yining043, for your detailed response.  I appreciate the time you took to clarify things for me.


Create codeql-analysis.yml

From https://github.com/excalidraw/excalidraw/security

Any idea what does it do?

GitHub security suggests it.. of doesn't harm.. it seems green

I still think adding this doesn't make much sense. It adds ~6min to our build for no reason, as we're definitely not be using this any time soon: https://securitylab.github.com/tools/codeql


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20240414153807110 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20240414153807110 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.001%) to 41.738%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
73.44%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
30720

Relevant Lines:
68888

💛  - Coveralls


v1 Provider type requires unstable_system but optional according to docs

🐛 Bug report
Current behavior
Trying to use Provider for SSR in my project that uses Typescript. The typing on Provider is stating that unsable_system is required. The getting started documentation states that it is optional. https://reakit.io/docs/get-started/#server-side-rendering
Expected behavior
I would expect to be able to use Provider without any props.

Thank you @stramel. It was fixed in 29c4456339306f020996c38276c2f9edf3f6697b and will be available in the next patch.


About multiple worksheets combine to a asset

Hello,this is a great asset,I'm using it in my new project,and I have a question.
Is it possible to combine multiple worksheets or xls of the same format into one ASSET? For example, I have a lot of data in an equipment table and I want to split it into 2 tables, but I need them to be combined in one asset

You can make 1 total data. Add 1 button combine to copy all data from single data

Add 1 button combine to copy all data from single data
What does the above mean?

using UnityEngine;
using UnityEditor;
using System.IO;
using UnityQuickSheet;

public static class ExcelFileProcessor
{
    static string modelDataPath = "Assets/Model/PropertyData"; //1.  you must gather your xls in a specify folder first
    [MenuItem("WebGL/Convert .xlsx to asset")]   //2. then you can use this menu to convert each xls to a unity asset
    public static void FileHandler()
    {
        var files = Directory.GetFiles(modelDataPath, "*.xlsx", SearchOption.AllDirectories);
        foreach (var item in files)
        {
            var filename = Path.GetFileName(item);
            if (filename.StartsWith("~")) continue; //2.1 XLS temp file always start with "~" , ignore them.
            CreatAndFillDataFromExcelFile(item);
        }
        Debug.Log($"{nameof(ExcelFileProcessor)}:  所有 excel 文件处理完毕 ！");
        AssetDatabase.Refresh();
    }
    [MenuItem("WebGL/删除转换好的数据文件")] // 3. you can delete those converted asset as you wish for some reason
    public static void AssetFileHandler()
    {
        var files = Directory.GetFiles(modelDataPath, "*.xlsx", SearchOption.AllDirectories);
        foreach (var file in files)
        {
            var filename = Path.GetFileName(file);
            if (filename.StartsWith("~")) continue;
            var assetFilePath = file.Substring(0, file.LastIndexOf(".")) + ".asset";
            //var assetFilePath = file.Substring(0, file.LastIndexOf(@"\")+1) + "Data.asset";
            AssetDatabase.DeleteAsset(assetFilePath);
        }
        Debug.Log($"{nameof(ExcelFileProcessor)}:  所有 asset 文件处理完毕 ！");
        AssetDatabase.Refresh();
    }

    const string sheetName = "Sheet1";
    const string assetsuffix = ".asset";
    private static void CreatAndFillDataFromExcelFile(string file)
    {
        var assetFilePath = file.Substring(0, file.LastIndexOf(".")) + assetsuffix;
        Sheet1 data = (Sheet1)AssetDatabase.LoadAssetAtPath(assetFilePath, typeof(Sheet1));
        if (data == null)
        {
            data = ScriptableObject.CreateInstance<Sheet1>();
            data.SheetName = file;
            data.WorksheetName = sheetName;
            AssetDatabase.CreateAsset(data, assetFilePath);
        }
        ExcelQuery query = new ExcelQuery(file, sheetName);
        if (query != null && query.IsValid())
        {
            data.dataArray = query.Deserialize<Sheet1Data>().ToArray(); // 4. Sheet1Data  should replaced by your data model.
            ScriptableObject obj = AssetDatabase.LoadAssetAtPath<ScriptableObject>(assetFilePath);
            EditorUtility.SetDirty(obj);
        }
    }
}

see commit in this code , you'd better make it fit you situation.
and combine those asset you can do it by yourself.  just operate arrays .


ModuleNotFoundError: No module named 'chardet'

Hi, I'm using pdftotree on Ubuntu with python3.6. I first installed python3 toolkit, then pdftotree. So when I import pdftotree, this error appeared
ModuleNotFoundError: No module named 'chardet'
So I think I should install chardet but I want to know which version should I install to be in sync with the application. Thank you for your feedback

Hey guys, this problem is provided by pdfminer on the utils script and exactly this part:
if six.PY3:
import chardet  # For str encoding detection in Py3
unicode = str
so, i installed chardet==3.0.4 and the problem is solved.

Thanks!

https://github.com/pdfminer/pdfminer.six/issues/213


Isn't Google Map Street View available in Flutter?

Hello
Is Google Map Street View available in Flutter? If not is there a plan to provide the plugin in the future? I really want to integrate Street View into my app.
Thanks.

Hello
Is Google Map Street View available in Flutter? If not is there a plan to provide the plugin in the future? I really want to integrate Street View into my app.
Thanks.

@BondarenkoStas Will there be any release soon?

Hi,
any news about streetview plugin?
Thanks.

@daddym85 Unfortunately no news yet. But, I needed that urgently so I developed my own plugin.

@azizk-araneux , is your plugin available?
Thanks

@azizk-araneux , is your plugin available?
Thanks

No, because it is not publish ready as I just created it for myself and it doesn't fulfill the guidelines.

@azizk-araneux Could you share your plugin code please?

I can share you the base code? Will that be okay?

Yes, of course. It would be great.
Thanks.
El jue., 26 dic. 2019 a las 7:05, Aziz Kaukawala<EMAIL_ADDRESS>escribió:

I can share you the base code? Will that be okay?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/33048?email_source=notifications&email_token=AAYYB4NJZ7V5F7YJKIT7U53Q2RCS3A5CNFSM4HOCYL6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHU7LTY#issuecomment-568980943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYYB4NBLRHCWLDGNXVCTVTQ2RCS3ANCNFSM4HOCYL6A
.

@azizk-araneux  Could you share your plugin code please?

Hello there ,
any news about the flutter plugin for  streetview?

I think you can do this. Try below code
    GoogleMap(
      ...
     mapType: MapType.normal,
    ),

@azizk-araneux Share street view plugin please

@pelli23 Could u share street view plugin please ?

Is there any update on this item?

any plugin yet ?

@daddym85 Unfortunately no news yet. But, I needed that urgently so I developed my own plugin.

Thanks

No, because it is not publish ready as I just created it for myself and it doesn't even fulfill the pub.dev's guidelines.

Do you pretend publish your plugin ?

guys we can use just webview for this senario or native code ,for me webview worked perfectly whenever i want streetview i passe latitude longtitude data to the javascript function

guys we can use just webview for this senario or native code ,for me webview worked perfectly whenever i want streetview i passe latitude longtitude data to the javascript function

Is this allowed with no limitations on how many calls can be done?

Has anyone tried using the SDKs for iOS and Android and run them as native code?

https://github.com/ahmedcs2600/Flutter-StreetView-
I have implemented native code(currently supports Android only for now).
You guys can use native implementation if it's helpful to you. Feel free to contribute and ask me for help :)

@google-admin any news on this???

@google-admin any news on this???

guys we can use just webview for this senario or native code ,for me webview worked perfectly whenever i want streetview i passe latitude longtitude data to the javascript function

Hi, can you show me source where can I find this solution, it looks good for me

Anything yet?

Hello everyone,
Flutter plugin flutter_google_street_view provides street view and support web, Android and iOS.
Try it!

Hello everyone,
Flutter plugin flutter_google_street_view provides street view and support web, Android and iOS.
Try it!

Perfect.


Implement more completely iterator method for hdb.

So that whole db can be iterated from start to end.

Thanks for the pull requests @m00natic. I didn't notice them until today - for some reason I didn't get an alert from github.

No problem.  I was intending to add tests too but haven't come around
yet.  Much of the code is battle tested within an internal tool that
required these additions.


[Feature] Add permissions on blips

Please could you make permissions on map blips for example show blips only for a specific jobs and don't have map full of blips. Thank you.

Not possible, blips can't have permissions. The only way is to do the perm check before adding the blips, but permissions are dynamics (group add/rm, special perms) where blips are not so dynamics. It would require to listen to every permissions changes and update ALL blips accordingly, in all case a rewrite of lot of the vRP code is necessary for this feature and performances will be impacted. So, it's not planned right now.

Ok, there is a simple way to do that maybe on item transformer we can show blip if have permissions ?


stack().unstack() not the same as original for datavars dependent on single coordinate of multi_index

What is your issue?
(See MVCE example)
The combination ds.stack().unstack() doesn't entirely give back the original ds, when there's a datavariable that only depends on a subset of coords of the multi-index used for stacking.

Is this on purpose? And if so, what's the rationale?
I would imagine that it could also be more memory efficient, when the original indexes x and y are kept that make up the multi-index (midx=[x,y]) after a stack() operation. Because then you don't have to express and thus repeat the values of dataarrays that only depend on a subset of the indexes that make up the multi-index.

MVCE
# xarray==2022.11.0
import xarray as xr

ds = xr.Dataset(coords={'x':[1,2], 'y':[3,4]})
ds['a'] = ds.x + 5
# <xarray.Dataset>
# Dimensions:  (x: 2, y: 2)
# Coordinates:
#   * x        (x) int32 1 2
#   * y        (y) int32 3 4
# Data variables:
#     a        (x) int32 6 7

ds_stacked = ds.stack(midx=['x','y'])
# <xarray.Dataset>
# Dimensions:  (midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (midx) int32 6 6 7 7

ds_unstacked = ds_stacked.unstack()
# <xarray.Dataset>
# Dimensions:  (x: 2, y: 2)
# Coordinates:
#   * x        (x) int32 1 2
#   * y        (y) int32 3 4
# Data variables:
#     a        (x, y) int32 6 6 7 7

Expected
ds_unstacked to be the same as ds.
Instead the variable a has now also become a function of coordinate y, but that's not entirely correct.
I.e., after ds.stack(), that the variable 'a' is still only dependent on the original coordinate 'x', which is just a part of the multi-index.
ds_stacked = ds.stack(midx=['x','y'])
# <xarray.Dataset>
# Dimensions:  (midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (x) int32 6 6 7 7

Maybe for clarity
# <xarray.Dataset>
# Dimensions:  (midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (midx.x) int32 6 6 7 7

Or maybe to save memory
Make a relation/difference between midx.x (repeated values of x due to stacking) and x (original unique values).
# <xarray.Dataset>
# Dimensions:  (midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (x) int32 6 7

Interesting! I don't think that when adding stack / unstack we were thinking that variables with only a subset of the stacked dimensions would be a common use case.
I guess it would be possible to add some option to stack only the variables that have all the dimensions to be stacked, and leave the other variables unchanged? However, one problem with keeping the original dimension coordinates is that we would have name conflicts between the single index coordinates and the multi-index coordinates.
In your expected example, the "x" coordinate is part of the multi-index but it doesn't have the same dimension "midx"? I find it rather confusing.

Hi @benbovy, thanks for the reply!
Made a mistake in expected, edited the example.
In terms of name conflicts, I don't know what the solution should be. It would need to know that the x of the multi-index refers to the same label but is, due to stacking, just a repeated version of the original x coordinate?

The last example in your comment is probably the most meaningful one:
# <xarray.Dataset>
# Dimensions:  (x: 2, midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (x) int32 6 7

To avoid name conflicts, we could just discard the original dimension coordinates x and y. Like here above, "x" becomes a dimension without coordinate. I that example, when unstacking we would retrieve the "x" dimension coordinate like in the original dataset.
(note: I think it is now possible to have a dimension "x" and a coordinate "x" with different dimensions, but I haven't checked).

But does this still work properly with broadcasting? For example, let's say there is another data variable b (midx) and an operation is done like ds_stacked['c'] = ds_stacked.a + ds_stacked.b. Then it should be that c (midx) and a (x) should be "repeated" to midx.x

I think it would keep things much simpler if we consider "x" and "midx" as two separate dimensions in the stacked Dataset, i.e., ds_stacked['c'] would result in a 2-d array (x, midx). There's no such thing like a "midx.x" dimension in Xarray.

There's no such thing like a "midx.x" dimension in Xarray.
I'm aware, just using it to communicate thoughts.

Perfectly fine with the solution proposed, as long as it works with unstack(). I would then expect something like
ds_stacked
# <xarray.Dataset>
# Dimensions:  (x: 2, midx: 4)
# Coordinates:
#   * midx     (midx) object MultiIndex
#   * x        (midx) int32 1 1 2 2
#   * y        (midx) int32 3 4 3 4
# Data variables:
#     a        (x) int32 6 7
#     b        (midx) int32 10, 20, 30, 40
#     c        (x, midx) int32 16, 26, 38, 48

ds_unstacked
# <xarray.Dataset>
# Dimensions:  (x: 2, y: 2)
# Coordinates:
#   * x        (x) int32 1 2
#   * y        (y) int32 3 4
# Data variables:
#     a        (x) int32 6 7
#     b        (x, y) int32 10, 20, 30, 40
#     c        (x, y) int32 16, 26, 38, 48


[KOGITO-5478] - Management console does not work on OpenShift

Many thanks for submitting your Pull Request :heart:!
Jira issue : https://issues.redhat.com/browse/KOGITO-5478
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've ran make before-pr and everything is working accordingly
[x] You've tested the new feature/bug fix in an actual OpenShift cluster
[x] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins test

/jenkins test

/jenkins test


The sample briefing book does not work in Internet Explorer 9 (IE9)

Open Internet Explorer and open the console (F12). If using a version later than 9, change the document mode to Internet Explorer 9 compatibility.
Visit the following site, an example of the Briefing Book application from Esri's GitHub Page:
http://<IP_ADDRESS>/briefingbook/default.htm
Test this link in a later version of Internet Explorer, or Chrome or Firefox, to verify that it is accessible to you.
Open the link in IE9 (or a compatibility version of IE9) and note that four RequestErrors appear, making the content inaccessible.

@aOrtego Has this been tested on a secured application? Per the requirements IE9 is only supported if the application is setup that way?

This issue is regarding Esri's sample application, so I don't know how it is being hosted.

@aOrtego The customer will need to deploy this app to their own web server, secured with SSL, using a trusted certificate authority. The sample app is not intended for their production use and we do not currently have it SSL enabled.

@aOrtego @conklinbd    Andrew and I had a few off thread conversation on this issue. The issue above is related to only viewing the application in IE9 not the authoring/viewing. We need to support either support viewing of the application in IE9, or update the documentation to reflect the app isn't supported for viewing only unless the app is secured with SSL (like we say for authoring).
@conklinbd I think this could be a bounce back.
I am closing this issue. A new issue will be opened related to this once it comes through Esri Support via triage, and we decide whether the fix is documentation OR app related.


Sound problems UE 5.1

Hello! When the Project Acoustics plug-in is enabled in the settings. When playing sounds that are positioned in 3d, 2d sounds (for example roomtones or VO) are interrupted or not played at all. What can be the reason? The spatialization plugin is also selected Project acoustics, but disabling it does not affect the situation. And there are no errors in log screen. Also sometimes when you switch the sound to binaural it stops spatializing, and sounds always in the center. If you return the value to panning, everything works. This applies to sounds in Soundscapes(Colours and sound pallet)

@Denis-Semenovykh It's pretty difficult to know exactly why this is happening, but the first thing I'd check is to take a look at the Attenuation settings attached to each of these sounds to see if there are shared properties between the different sound types.
Also, this might be a stretch, but we are aware of an issue in the Google Resonance plugin that when enabled (but not necessarily used), there can be some negative side effects that impact other spatialization plugins.  This has been fixed recently, but it may not be in your version of UE.  If you do have Resonance enabled in the plugins window, please disable the plugin, restart your project and then see if your behavior changes.
Thanks,
Noel

@NoelCross   Hello Noel! Thank you for your reply! But it didn't help me and didn't change the situation. After looking more I realized that everything happens while the character is moving, when playing footsteps sound, less obvious on the walk mode, but on the run mode very clearly when the footsteps become more frequent. All the surrounding sounds begin to interrupt. If you put the steps in panning mode, everything works fine. Another thing, if you play sounds through play notify in animations, if they are in binaural mode and Ignore Attenuation is not checked, the sound is not played, again if you switch to panning mode everything works.

And another one questions. I tried baking the data, on the pre-calculation everything is fine, the probes are aligned. But after baking, the coordinates of the bake zone in the ace file moved. I.e. probes and voxels in Acoustic Space are in a different place, not where they should be in the preliminary calculation. I baked not a large area locally, on my machine. Why could this happen?

@Denis-Semenovykh this last reply has a clue about the behavior that might indicate what the problem is.  The Acoustics Space actor has the ability to be placed in the scene with an offset other than 0,0,0.  If the offset is anything other than 0,0,0, then you would be getting the strange behavior that you describe in your last reply.  Check the Acoustics Space actor to ensure there is no translation applied.

@NoelCross Hello, Noel! Got it, I'll look into it, thanks! What about my previous comment?

@Denis-Semenovykh it's unclear why the problems you are seeing are specific to the Best quality setting for spatial reverb.  One other thing to try is to see if choosing Stereo Convolution gives you a different experience that contrasts with the Spatial Reverb option.
Also, does this behavior happen on multiple machines with different audio adapters?  I'm curious if your hardware is exposing multi-channel output which might be a condition that wasn't handled properly.  This doesn't make sense because the Good vs. Best quality setting shouldn't matter when it comes to the number of output channels.
I'm not sure if you are able to share a video illustrating the behavior you are experiencing.  Ideally, turn on the debug visualizations so we can see what parameters are being applied for each sound source.

@NoelCross Yes, this problem happens on all our different computers. We do not use multi-channel outputs. I would be happy to provide a video, but unfortunately I can't, because it's a closed project, of course I'll talk to our management, maybe we'll find some solution for this. At the moment, tell me what I need to pay attention to? And this problem happens when the character moves, if you stand still, then even in Best mode everything is fine.

@Denis-Semenovykh if you have a stripped down, generic version of the project with generic assets, that might be another way that we can investigate this issue.
When looking at the debug data, the arrival direction arrows from the sound sources is always a good indicator of what metadata is flowing through the system.  Also, looking at the wet/dry levels help give an indication of what the rendering DSP is attempting to render.  By turning on the debug visualizations, you might be able to identify why the rendering of a specific source is incorrect.

@NoelCross Hello! Choosing Stereo Convolution working fine, no bugs. This happens mostly with 2d sounds, roomtones and VOs. I recorded you audio, for more understanding, and brought out loud roomtone. The problems start when the character starts moving, and there are other 3d sounds that affect the 2d, if you stand there is nothing wrong. For example, you will hear when a source with music appears nearby, the roomtone starts to interrupt. Also there is an effect on VO, they are interrupted if you start moving. Nowhere does it show any errors.
https://github.com/microsoft/ProjectAcoustics/assets/90043919/4a1c7813-6425-4527-affc-373f6b3f0659

@Denis-Semenovykh I'm sorry to inform you that there is no Linux version of the plugin.  We only distribute versions of the binaries for Android and Windows in the UE Marketplace.  I assume that you are seeing linker issues with our runtime components.  If you are seeing something different than linker issues, I'm curious what build error you are seeing.

@NoelCros Error: Failed to cook /Game/Maps/Game/TrainStation/SLs/SL_Train_station_audio for platform LinuxServer. It imports class /Script/ProjectAcoustics.AcousticsSpace, which is in a module that is not available on the platform.

@NoelCross Wrote to you above, the errors we see. Is it possible to make linux dedic build without Project Acoustic? I.e. part of the classes should be placed in a module that has no platform restrictions. inside through preprocessor macros to divide the code so that if the platform is Windows or Android, the plugin is activated, and if another platform, the class is just empty. Is it possible to realize such a solution?

Agree with Denis that it would be great if you could at least compile on Linux by default. My solution was to make my own versions of some classes (e.g FAcousticsDesignParams, AcousticsInterpolationDisambiguationMode...) and add a lot of #if PLATFORM_LINUX to make the server builds to compile, but it won't be easy to upgrade PA the next time. :)

A lot of our code is incorporated into the game code so it's a bit hard to separate. But the big things (checking changesets) to get you started (have no idea if this is the best way):

Added a NeedsLoadForServer (returning false) in acousticdata/space so the server does not try to load the data.
Added defines before all usage of PA-code and includes
#if !UE_SERVER
#if !PLATFORM_LINUX
In the build.cs files added check for target platforms for the project acoustic plugins. E.g. in ProjectAcousticWwiseEditor.Build.cs

if (Target.Platform == UnrealBuildTool.UnrealTargetPlatform.Win64)
{
PublicDependencyModuleNames.AddRange(
new string[] {
"ProjectAcoustics",
"ProjectAcousticsWwise",
}
);
}
ProjectAcousticsBakeUI.Build.cs ->
if (Target.Platform == UnrealBuildTool.UnrealTargetPlatform.Win64)
{
PrivateIncludePaths.AddRange(
new string[] {
"../Source/ProjectAcoustics/Public"
// ... add other private include paths required here ...
}
);
        PublicDependencyModuleNames.AddRange(
            new string[] {
            "ProjectAcoustics",
            }
        );
    }

etc.

Added #include "AcousticsShared.h" in the acoustics .cpp files to make them compile in non-unity builds

Sooo, there's a start at least. I accidentally push to our main branch before I tested the linux-build so these fixes were made during a long evening with no design whatsoever... so I guess there might be better ways. =)

@NoelCross Hello! What do you think about it?

@Denis-Semenovykh When it comes to building on Linux servers, we haven't had a customer that requested this configuration, so it is not something that we've tested.  If the changes suggested above gets you unblocked, then I don't see a problem with integrating those changes into your project.
For the issue with the 2d sound dropping out when selecting the Best quality spatial reverb, I'm still suspecting that the reverb processing is overloading the system.  The Best quality setting can take up to 20% of a system, so there might be some contention in the system for resources even on a high end machine.  A good test would be to bring up the Unreal profiler and see if threads are getting starved for processing during gameplay.  Another option for debugging would be to come up with a minimal repro project that exhibits the same problem and doesn't illustrate any of the secrets that you don't want to show.  We could diagnose this simple test project to see if we can identify the root cause.

@Denis-Semenovykh we currently don't have plans to update the plugin anytime soon, so best to make the changes on your end and clearly mark in comments where the changes are.  I don't expect that there will be too many conflicts if we update the plugin in the future with the changes you need to get working on Linux servers.

@NoelCross Thank you for your comments! We will try to integrate this solution in our project. If there are any questions I will write. And I was honored to communicate with you! 🙂

Closing. Feel free to reopen if you need further assistance.


Add api to control thread for all index

Signed-off-by: lixinguo<EMAIL_ADDRESS>issue: #374
It offers api to control threads when query and build in all indexes. You can add build_thread_num and query_thread_num when creating config and it offers the default values if you don't add it.

Following code piece is repeated more than 10 times.
        if (CheckKeyInConfig(config, meta::QUERY_THREAD_NUM)) {
            omp_set_num_threads(GetMetaQueryThreadNum(config));
        } else {
            omp_set_num_threads(knowhere::DEFAULT_QUERY_THREAD_NUM);
        }

Suggest: add new files "omp.h" and "omp.cpp" under knowhere/common, and create an new API

SetOmpWithDefault(const Config& conf, const MetaType& type, const int64_t default_val)

Good catch, I thought Utils.h under common folder sounds more rational to me. We will not have too many omp related common functions


Design baraza's database.

The database for baraza should contain the tables and columns as follows:
(t1)users - stores information on registered users
A  date registered
RU first name
RU last name
RU phone # (primary key)
(t2)chiefs - stores information on registered chiefs
A  date registered
AD first Name
AD last Name
AD chief # (primary key)
AD phone #
(t3)conflicts - stores information on reported conflicts
A  conflict # (primary key)
RU nature of Conflict
RU details
RU name of victim/reporter
RU number of reporter (foreign key)
RU name of accused
RU number of accused
C  resolution passed
C  resolution passed
(t4) events/announcement - stores events and announcements posted by chief
C name of event/anouncement
C details
A date made
C event date
A event #
The letters to the left indicate who populates what field.
C for chief, RU for registered user, A for auto-filled, AD for administrator.
User registers from phone application. Info is saved in t1. On logging in, they see an option of reporting an issue. When they report an issue, this info is saved in t3. The other option is seeing a feed that is populated from t4.
Chiefs are registered from the server into t3 not from the application.
When chief signs in, they see conflicts and option for events and announcements. Conflicts are populated from t3. They are responsible for editing some columns. They populate events t4.
Task: Create SQL database from this information.
Deliverables: SQL database for baraza application.
NB: Still unsure about hosting the database and the application's information, open to suggestions. =)

This will be a relational database. Since initial plans are to host the app on heroku, it will be a Postgres database.


Login method being triggered when UALLoggedInAuthType exists in session

I'm not sure if this issue should be reported for EOSIO/universal-authenticator-library or EOSIO/ual-reactjs-renderer, since I haven't had time to completely debug where the issue comes from - but given the following conditions:

Using the ual-reactjs-renderer module
One (or more) authenticators are provided to <UALProvider ... />
The authenticator has shouldRender always set to true
The authenticator has shouldAutoLogin always set to false
The user has logged in previously, and UALLoggedInAuthType is set in local storage

The login method of the authenticator is being called on page load automatically (after the 250ms interval triggers). This can be reproduced with the conditions above by successfully logging in using UAL + provider, then refreshing the page. The greymass/ual-anchor provider illustrates this happening under these circumstances and on page load the login method is called our QR Code display pops up without user intervention.
I would expect that if shouldAutoLogin is set to false, on page load the login method of the authenticator would not automatically execute.

I agree and I've narrowed the issue down to ual-reactjs-renderer.  I will keep you updated when I open a PR with changes.

Quick update,
The process you noticed is intended in some cases to emulate user sessions elsewhere.  I'm planning on adding another option for authenticators in UAL to set an invalidate time to not reconnect on new browser sessions, so in your case, you could set the invalidate time to zero.

@aaroncox universal-authenticator-library and ual-reactjs-renderer have both been updated with a new published version (v0.2.0 for both).  These new versions support a new Authenticator method called shouldInvalidateAfter that returns the amount of seconds until the authentication will be invalid for logging in on new browser sessions.  In your case, you can set this method to return zero and ual-reactjs-renderer will not attempt to re-authenticate.
Let me know if you have any issues with this new method.

@bradlhart It appears to be working, I do have some oddities I'm still trying to deal with - but I'll open a new issue related to the next piece.


Update http4s-client, http4s-core, ... to 0.23.21

About this PR
📦 Updates

org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl

from 0.23.20 to 0.23.21
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #177.


Things which can be improved

Filtering selection by width or height - sometimes it is needed to exclude images which have smaller width or height, but we may have one or two images in this group we still want to be selected. It could be done manually but it can't be done now in the case where filter excluded them out. We can see this excluded images on the list but there is no way to select and include them manually other than removing filter and including all the unwanted rest of images. I think addon should exclude images according to filters like it does currently, but still it should allow the user to select manually such excluded images.

Sometimes one or more images can't be fetched (server issues, network issues, other errors) and in such cases the red rectangle will appear in the middle of properly fetched images (in the place where the missing image should be). This red rectangle has tooltip under contained full link to image but there is no option to refresh such image. It would allow addon to try to check this failed image once again without the need to check whole page again, but for now it is not possible to do this. Maybe it is this way due to filter which excluded image because addon couldn't get proper size, i don't know. The only way is to repeat operation of detecting everything on such site.

2. there is no option to refresh such image

Actually you can press F5 to fetch again. Image Picka uses a cache system so only broken images will be re-download. A better way is to use the Retry on failure option:
https://github.com/eight04/image-picka#retry-on-failure

Maybe relate: #301


Lidar attached to vehicle with SpringArm results in limited FOV

Hello,
I was following the lidar raycast sensor guide which suggests using the SpringArm attachment. Using SpringArm results in a non-rotating FOV limited to sight of the front-left and back-right sides of the vehicle as shown below:

This is what the lidar output of the same scene using the Rigid attachment looks like:

If I understand SpringArm correctly, it should cause the lidar sensor to avoid entering walls/obstructions and lag slightly behind the vehicle it's attached to rather than arbitrarily limiting the FOV like this. Is this behavior intentional?

@RyanHopkins7 indeed, the springarm provides a very specific pose, which will limit the FOV. I guess, this is not the right tool for your needs. Just use the rigid one.

@germanros1987 All I'm wondering is if this behavior is intentional. If it is, I'll close, but it did take me a little while to figure out the cause of this issue considering that the tutorial I mentioned used SpringArm without limiting the FOV. A note about this in the documentation might be helpful so people know what causes this.

Hold on a second, I may have misunderstood the first screenshot. Is the first image supposed to be taken from a bird-eye-view? I thought the points on the left and right side of the pointcloud were from a wall. Could you further explain the setup, please?

Yes, they were both taken from a bird's eye view using the same settings other than the attachment type. They were also taken in the same environment. The obstructions on the left and right in the first photo are not caused by walls.

@RyanHopkins7 please send over the sensor configuration/setup you have used for this so that we can reproduce it.
@DSantosO once we get the sensor configuration, could you try to reproduce?

This is the sensor configuration I was using:
lidar_cam = None
lidar_bp = world.get_blueprint_library().find('sensor.lidar.ray_cast')
lidar_bp.set_attribute('channels',str(32))
lidar_bp.set_attribute('points_per_second',str(300000))
lidar_bp.set_attribute('rotation_frequency',str(120))
lidar_bp.set_attribute('range',str(20))
lidar_bp.set_attribute('lower_fov',str(-40))
lidar_location = carla.Location(0,0,2)
lidar_rotation = carla.Rotation(0,0,0)
lidar_transform = carla.Transform(lidar_location,lidar_rotation)
lidar_sen = world.spawn_actor(lidar_bp,lidar_transform,attach_to=vehicle2,attachment_type=carla.AttachmentType.SpringArm)
lidar_sen.listen(lambda point_cloud: point_cloud.save_to_disk('sensor_output/lidar/f%.6d.ply' % point_cloud.frame))

@RyanHopkins7 Thank you for the information. I am able to reproduce your result and yes, it seems to be something weird with the output when SpringArm is the attachment type. I will look into it.

Hello,
Sorry for the late replay, we have looked into the specific of the case and there is couple of things to comment.
First, you need to take in account that the SpringArm is thought to have a better camera visualization so it should not be used in sensors like LiDAR. In this sense, please use always the Rigid mode.
Anyway, the SpringArm is doing something weird in your specific case because of the zero translation in the 'x' axis. This is a problem because this SprintArm is thought to see the car in 'third person' not directly in top of the car. We will add some safeguard about this problem soon.
Thank you very much for reporting the issue, I hope this information is helpful.

@DSantosO Thanks for getting back to me and I'm glad to help. If it's recommended not to use SpringArm for LiDAR, perhaps the documentation should be changed to demonstrate using the Rigid mode.

Hello @RyanHopkins7
Yes, that is mistake. Sorry about it. We will change it.
Thanks again!


Particle speed lost when using gravity

This Issue is about (✏ delete as applicable)

A bug in the API:

Phaser version(s): 2.7.2 +
Live example:
http://phaser.io/examples/v2/particles/click-burst#gv (change to version 2.7.2 or later in drop down (initial load works, you have to change it to something, but then if you change it back to latest it still is broken))
What should happen:
Particles should not stay in starting position
What happens instead:
Particle stay in starting position

This is because Emitter#gravity was changed to a Point in 2.7.2 and that change wasn't backwards-compatible. 😢 So it's not exactly a bug but it is a problem.
I think we could replace gravity with a setter that accepts either a number or a Point.
https://samme.github.io/phaser-examples-mirror/particles/click burst.html

@samme Ahh, that makes sense. I use typescript and it's still showing it as a number. I have the latest TS, so that might need an update too? For now I can obviously just use gravity['y'] to set the gravity without having TS show a compile time error.

I've corrected the type in the master branch.


Overwrite standard emoji in HTML?

Hi, is there a way to overwrite standard Emojis in HTML with Twemoji for a consistent and nicer look?
Like EmojiOne can here: https://demos.emojione.com/latest/class-convert.html
For example emojis inserted in to HTML like this...

I understand it won't change them in dev console but on the front-end!
Many thanks.

If I understand correctly what you are looking for, it's a bit like the ABC of this library since about ever
https://github.com/twitter/twemoji#dom-parsing
document.querySelectorAll('.convert-emoji').forEach(el => {
  twemoji.parse(el);
});


feat: Add new report, and alert type: webhook

SUMMARY
I add webhook as a new selectable report type.

By the use of webhook the user can post csv/png/pdf to any endpoint, with some informative parameters as:

name of the report
id of the report
report type (chart, or dashboard)
content id
content format

For security reasons I made a new optional parameter: WEBHOOK_SECRET. It's a secret string between the Superset and the webhook endpoint. In the case where WEBHOOK_SECRET is filled, Superset adds an "X-Webhook-Signature" parameter to the header of the post call, which hashes the json data to be sent. The receiving party can verify that the party sending the webhook is the real sender by hashing the received json data and comparing it with the sending party's "X-Webhook-Signature" parameter. If the two parameters do not match, the receiving party may reject the call because the sender is not the supposed host.

On the report log, we can get feedback about the successful, and not successful requests as well.

I'm happy to share my code with you guys!
TESTING INSTRUCTIONS

(optional) Fill the WEBHOOK_SECRET input parameter in the config file
On the Alerts & reports page, press the "+ Report" or "+ Alert" button.
On the notification panel, you can select Webhook as Notification method
Fill the webhook endpoint url
Fill all the mandatory input

By the result you will get the selectend chart/dashboard file on your webhook endpoint with some informative parameters as:

name of the report
id of the report
report type (chart, or dashboard)
content id
content format

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

Really cool idea @kistoth90!

I love the idea here. Approving the next CI run... ping me if it needs to be kicked again.

At this point I don't know what, or how should I fix the errors:

I tried to run the cypress on my desk (cypress-run-chrome) but I got constant "400 Bad Request: The CSRF token is missing." during the login call. I was able to log in manually by the test credentials.
the pre-commit works locally fine... i dont know why it want to refresh the source code during the CI run.

I would appreciate some assistance about, becauseI'm stuck.

For the Pre-commit issue, have you tried running ruff and black locally to remove the formatting blocks?

Hi @fisjac,
Yepp, the ruff, and Blacklist passed as well!
I made some changes by the guidance of dpgaspar. Let's hope the best! 🤞

Closing and reopening to try to get tests to kick off.

Hi @dpgaspar / @eschutho are there any blockers from merging this?

/testenv up FEATURE_ALERT_REPORTS=true

It sounds like this feature may be a good use-case for putting behind a feature flag. I can see that some superset administrators may not want to enable webhooks to their users, even if they have the alerts/reports feature on.


How can i define a function block!

Hello!
How can i define a function block, like blockly, thanks!
Operating System and Browser
e.g. Mac OS 10.11.6 Safari 10.0

@lixiaocheng18 - Could you be a little more specific with what you are attempting to do?
If you are trying to create a 'Custom Block' like in 2.0, we have not implemented that feature yet.

@lixiaocheng18 Here's the issue tracking 'Custom Blocks' https://github.com/LLK/scratch-blocks/issues/607 . Is that what you're looking for?


configure.ac: Use fixed version for stable branch

Having the version generated from a git tag could be useful for dev branches
and when people build from a git repository. But is not correct for released
tarballs, since attempting to execute the git command will fail and won't be
a version defined, i.e:
$ ./bootstrap
Generating file lists: src_vars.mk
fatal: Not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
$ grep -i version lib/sapi.pc
Version:
So after a branch is released, just use a fixed version number:
$ grep -i version lib/sapi.pc
Version: 1.3.0
Signed-off-by: Javier Martinez Canillas<EMAIL_ADDRESS>

I'm a bit confused. We use git and tags to get the version number. If you've building from git you can check out the appropriate tag, do the whole ./bootstrap && ./configure dance and the PACKAGE_VERSION will be set appropriately. So like you say for dev and builds from git this works as expected.
For the release tarballs we use the dist target from automake. What this does for us is package up the generated ./configure script which already has the PACKAGE_VERSION set. This works because I generate the release tarballs from a checked out git repo so people consuming the release tarballs don't have to.
In your example you run the ./bootstrap script which isn't packaged in our release tarballs. Is is possible that you're using the github generated Source.zip files instead of the signed tarballs?

@flihp my bad. I missed that you also have release tarballs in https://github.com/intel/tpm2-tss/releases. I thought there were only source code tarballs there, since that's the case for the tpm2-tools (https://github.com/intel/tpm2-tools/releases).
@williamcroberts I think you should also do the same and publish the release tarball generated by themake dist target.
I'll fix the packages to point the source to https://github.com/intel/tpm2-tss/releases/download/1.3.0-rc2/tpm2-tss-1.3.0-rc2.tar.gz instead of https://github.com/intel/tpm2-tss/archive/1.3.0-rc2.tar.gz as is the case now.
I'm closing this PR now, sorry for the noise.

@flihp there's a bug in the generated tarball though, the LICENSE file is missing. I'll send a PR for this.


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit 0c14661:
:clock10: Full build: the average full build time is 5 min(s) 44 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 0c14661:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

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


CITATION.cff validation and Zenodo parsing bug?

CITATION.cff parsing changed in Zenodo around a month ago, and it stopped accepting array values for license.
See this issue for workaround: https://github.com/zenodo/zenodo/issues/2515
Additionally, GitHub is complaining about parsing our CITATION.cff file; let's add a pre-commit validator that runs cffconvert to validate our CFF.

Resolved by PR #199  :point_up:


Fix Error message displays briefly after reopening the 2FA step 2 page

PR Format Expensify

Details

Fix Error message displays briefly after reopening the 2FA step 2 page
Note: 2FA feature can only be tested Online
Fixed Issues

$ https://github.com/Expensify/App/issues/28937
PROPOSAL: https://github.com/Expensify/App/issues/28937#issuecomment-1758517204
Tests

Go to Settings > Security > Two-factor authentication
Click on the Copy button
Click on the Next button
Enter an invalid code and check that an error message shows up,
click on the back icon
Click on the Next button again
The error message should not show

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to Settings > Security > Two-factor authentication
Click on the Copy button
Click on the Next button
Enter an invalid code and check that an error message shows up,
click on the back icon
Click on the Next button again
The error message should not show

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/20473526/8d59f9af-de53-4693-80c4-76186186795d

Android: mWeb Chrome
https://github.com/Expensify/App/assets/20473526/a31a80c7-d765-42ea-b9a2-aca9519b5b31

iOS: Native
https://github.com/Expensify/App/assets/20473526/a5604982-3be1-45c9-8118-d1fbe118acf3

iOS: mWeb Safari
https://github.com/Expensify/App/assets/20473526/8978c2d3-4f81-4696-bf6c-f40842ccd872

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/20473526/8eefd36b-23ab-4f7d-862e-eb7424c98936

MacOS: Desktop
https://github.com/Expensify/App/assets/20473526/5b2c2e2d-b8da-4274-941b-25b81095681e

Done it @s77rt

Reviewer Checklist

[X] I have verified the author checklist is complete (all boxes are checked off).
[X] I verified the correct issue is linked in the ### Fixed Issues section above
[X] I verified testing steps are clear and they cover the changes made in this PR

[X] I verified the steps for local testing are in the Tests section
[X] I verified the steps for Staging and/or Production testing are in the QA steps section
[X] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[X] I checked that screenshots or videos are included for tests on all platforms
[X] I included screenshots or videos for tests on all platforms
[X] I verified tests pass on all platforms & I tested again on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[X] I verified proper code patterns were followed (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I verified that this PR follows the guidelines as stated in the Review Guidelines
[X] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/16493223/de61d4f1-aab6-4b71-a885-658cdaf1f28e

Mobile Web - Chrome

https://github.com/Expensify/App/assets/16493223/eb3ce713-ab02-4f04-97fb-a9c8d5bcb22c

Mobile Web - Safari

https://github.com/Expensify/App/assets/16493223/af0e9349-4772-4970-90a7-315db10c0008

Desktop

https://github.com/Expensify/App/assets/16493223/6f662c14-a3c9-4981-ab1d-93fb28baf8ce

iOS

https://github.com/Expensify/App/assets/16493223/2ae9150e-e1f5-481e-8f54-758c0eb846fd

Android

https://github.com/Expensify/App/assets/16493223/7a087bf5-8290-4efa-8831-523335fd237a

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/grgia in version: 1.3.84-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.84-10 🚀

platform
result

🤖 android 🤖
skipped 🚫

🖥 desktop 🖥
skipped 🚫

🍎 iOS 🍎
skipped 🚫

🕸 web 🕸
skipped 🚫

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.84-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/grgia in version: 1.3.85-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Linux: VS Code blink and close itself after a few minutes

Since the upgrade the VS Code started to blink and close itself after a few minutes (usually 1 or 2 minutes).
Looking at the other topics I saw that using --no-sandbox makes VS Code stable again.
I'm currently using Ubuntu 20.04.
I didn't found any crash report to attach.
Version: 1.57.0
Commit: b4c1bd0a9b03c749ea011b06c6d2676c8091a70c
Date: 2021-06-09T17:18:42.895Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.8.0-55-generic snap

I think it should be closed as duplicate of #126007
Are you using snap installation?

/duplicate https://github.com/microsoft/vscode/issues/126019


Change ansible_sudo to ansible_become

RHEL subscribe playbook gets a permission problem when running with non-root user, this fix solves it.

@blomquisg please merge, this fix is needed to get the rhel-subscribe pr working.

LGTM :+1:

@miq-bot assign chessbyte


DDSaaS: StreamNative: Crawler Integration v1.0.0

What does this PR do?
This is an initial release PR of StreamNative integration including all the required assets.
Motivation

Crawler code for this integration has been committed in its respective repo
Pipeline created for this integration are available in our sandbox and would be shared separately with the required teams.
Samples for the pipeline review would also be shared separately with the required teams.
Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current Datadog behavior.

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Looks good overall, a few suggestions! Also passing this by the docs and agent team! Thanks for your work on this!

Created Jira card for Docs Team editorial review.

Thanks for all of the updates, we're running this by our docs team and PM.

@narendranandaniya-crest Can you please take this PR out of draft? We have reviews from all stakeholders, so once they're addressed, we're happy to approve and merge! Thanks!

Hi @apiazza-dd , normally we follow this path where once the without Assets PR is merged we will move the Assets PR in Ready for Review State.
As we have without Assets PR in open state, we have kept this Assets PR in draft state. But considering we have the required reviews and approvals, we can move the Assets PR out of draft for merging and close the without Assets PR since everything related to non assets PR is already covered in here. CC: @dj0well

/merge


The typechecker fails with an exception on functor applications

On:
val tricky : Set.Make(Int).t -> bool

the typechecker fails with:
gospel: internal error, uncaught exception:
        Invalid_argument("Ppxlib.Longident.flatten")
        Raised at Stdlib.invalid_arg in file "stdlib.ml", line 30, characters 20-45
        Called from Ppxlib__Longident.flatten_exn in file "src/longident.ml" (inlined), line 34, characters 22-33
        Called from Gospel__Typing.ty_of_core in file "src/typing.ml", line 101, characters 39-70
        Called from Gospel__Typing.val_parse_core_type in file "src/typing.ml", line 766, characters 8-25
        Called from Gospel__Typing.process_val in file "src/typing.ml", line 964, characters 18-49
        Called from Gospel__Typing.process_sig_item.process_sig_item in file "src/typing.ml", line 1329, characters 31-67
        Called from Gospel__Typing.process_sig_item in file "src/typing.ml", line 1361, characters 23-51
        Called from Gospel__Typing.type_sig_item in file "src/typing.ml", line 1366, characters 15-49

Proposal of fix:

add a new “Functor application not supported” error,
wrap all calls to Longident.flatten_exn to report that error if need be.


Status colors

I'm not exactly sure how to generalize this, but for our use-case, we're using colors to reinforce the availability status of an item and the ready-status of a hold request (e.g. green = available or ready).  Note that there can also be a recommendation to "not use colors to indicate/reinforce meaning"

do you have an example link for this?
in general we are avoiding relying on color in favor of text (eg: “available”, “not available”). availability can also be reinforced with the presence or absence of buttons.
as we design the request/hold process we can decide if color is required.

moved to https://github.com/NYPL/design-toolkit/issues/9


Greene measure

Had to modify Corpus to add a "sample" dataset option. The rest I left untouched i believe. I like to use import numpy as np and use np instead of numpy though.


Update curation to deal with selected_points being a set (rather than list).

See https://github.com/napari/napari/pull/1126
Not sure if this will cause any issues, but need to check the behaviour of cellfinder.train.curate.main with the napari master branch.

Fixed in 2afb401c16feb6cf5b9277f39ab048322ed6c861


Update consumers of IntegrationTesting after #996

We need to update consumers of IntegrationTesting after merging #996. There is a breaking change in 0.4.0 and while 0.3.0 is still on the feed, it will fall off soon. Also we want to use the new logging ;)
Just tracking the many PRs for this here. Will close when they're all merged.

[ ] https://github.com/aspnet/MusicStore/pull/769
[ ] https://github.com/aspnet/Entropy/pull/217
[ ] https://github.com/aspnet/IISIntegration/pull/349
[ ] https://github.com/aspnet/ServerTests/pull/67

I think that's all of them. /cc @Tratcher @natemcmaster

Reminder to unpin 0.3.0 of IntegrationTesting when this closes. /cc @pranavkm @Eilon

@pranavkm The package can be unpinned now!


Touch point gets stuck if an element is removed

If an item is removed during a gesture, a touch point does not end even after all fingers are lifted. Observed in the Infinity example.

The problem represents expected behaviour! The touch events are sent to the element where they started even when the element becomes removed. The removal however prevents the elements bubbling to the touchable manager of the view. See this discussion in Hammer.js GitHub.
From the zoomability point of view, this feature of touch events forces us to implement it always the same way: remove elements only after the last touch end.
How to prevent programmers falling into the same pit again?

How to prevent programmers falling into the same pit again?

An error message would be nice when the item of Touchable is removed during ongoing gesture.
However this would not remove the problem because removal of any element between touchev.target and the element of the item breaks the event flow.
There are ways to observe removal of HTMLElement, like MutationObserver but we should not use them just to print a warning.


backport [SPARK-27747][SPARK-27816][SPARK-28344]

backport 3 PRs to 2.4 to fix ambiguous self-join. This PR excludes the logical plan link which is not necessary in 2.4.
For more details about ambiguous self-join, please refer to https://github.com/apache/spark/pull/25107

The risk is high based on the code changes in this PR.

Thank you all for your opinion. Especially, thank you for making this PR, @cloud-fan . I'll remove Target Version: 2.4.5 for now. If we need this in branch-2.4, Target Version will be 2.4.6.

@cloud-fan, @tgravescs, @gatorsmile, I was wondering if you could say a little more about the risks you saw here.
We found some "surprising" results and consider taking this backport over on our palantir/spark fork. I just want to better understand why you decided against merging and see if we're willing to take risk. Were you afraid of false positives? Was it the amount of changes? I understand it's been a while since you thought about this.

The risk is that many commits need to be backported. And there are probably more now as we have new bug fixes of the self-join detection in 3.x.


Support aliasing in %%opts magic

Following discussion in issue #72 regarding a 'Global' entry in %%opts (an idea I wasn't fond of), here is a more flexible approach that I do think would be useful:
Store.alias(Set=[Overlay, RGB, Image])

Allowing you to do:
%%opts Set [aspect=2 fig_size=200]

The alias command should support strings to avoid needing to import the aliased elements and it could also support something like:
Store.alias(Global='all')

Though I still don't think I would want this to exist by default.

I've had another idea that doesn't replace the suggestion above but may alleviate the need for aliases in a fair number of many cases.
We want HoloViews to have smart, predictable behaviour that allows the user to be as succinct as possible. What I personally find is that one reason my %%opts definitions grow redundant  is that 1) I first set some plotting option on some element I am looking at 2) then I decide I want to overlay something (e.g an annotation, some points etc) 3) now I have to repeat the plot options I just specified  for Overlay. This is the sort of redundancy you might try to solve using the proposed %opts magic alias system.
@philippjfr may raise some technical objections that I haven't considered, but I can't see why the following wouldn't be sensible behaviour for OverlayPlot:
Before dispatching the elements in the overlay to their respective plot classes, collect the plot options of all the overlay elements (which you should be able to do in OverlayPlot afaik) using the following policy:

Iterate through the elements in z-order (i.e from the bottom to the top) and collect common plot options (i.e at least those on Plot or the plot options shared by all the element plot classes and ``OverlayPlot).

Collect the plot option values as you iterate, giving the plot options of the layers higher up in the overlay a higher precedence than those below.

Finally consult the plot options of the overlay itself for the highest precedence plot options. You should now have some sensible plot options for your OverlayPlot.

This seems to me to be quite consistent with the semantics of overlays in general and it means that if you set a plot option for an Image (say) and then decide to overlay some Points on top later on (without setting clashing plot options), you no longer need to repeat what you specified for Image again in a definition for Overlay.
The main worry seems to be what happens when plot options are inconsistent across the overlay. The policy I just described is to use z-ordering as a precedence but another would be to simply drop clashing (common) plot options.
Anyway, this idea might be shot down right away for technical reasons or maybe it just doesn't make sense for reasons I haven't thought of. If it does make sense and people like the idea, this might be filed as a separate (but related) feature request.
Lastly, I should note that I think this would mostly be a backward compatible change, simply because this redundancy is required if you want plot options to stick after turning your elements into an overlay. In addition, we could have another plot option called inherit on Overlay itself to enable/disable this behaviour.


Add a rosdep rule for libopenal-soft

add libopenal-soft rule for arch, debian, fedora, gentoo and ubuntu.

arch: https://www.archlinux.org/packages/extra/x86_64/openal/
debian: https://packages.debian.org/ja/sid/libopenal-dev
fedora: https://apps.fedoraproject.org/packages/openal-soft
gentoo: https://packages.gentoo.org/packages/media-libs/openal
ubuntu: https://packages.ubuntu.com/ja/xenial/libopenal-dev

This library is for playing spatial audio, and I am developing a ROS interface to it.

changed fedora entry to openal-soft-devel. Thank you for reviewing.


Not showing decimals

Describe the bug
I cannot see decimals even though this is set.
allowDecimals={ true }
decimalScale={ 2 }
decimalSeparator=","

Also on your site https://cchanxzy.github.io/react-currency-input-field/, the first example in which the decimals should work, is not working as expected.
To Reproduce
Steps to reproduce the behavior:

Create currency input
Set allowDecimals={true}
Try writing decimals in the 1. example  on https://cchanxzy.github.io/react-currency-input-field/
See error

Expected behavior
I would like to have input in which I can write decimals.

Hi, I believe this is because the decimalSeparator  is the same as the default groupSeparator (which will also likely be ,). I've added a fix to throw an error if that happens so it is more explicit.
Let me know if that helps.


User account - ORCID button works only when clicking on the green photo without its edges

https://user-images.githubusercontent.com/82588017/234232020-ba75df71-1901-4034-83f2-ade751ddc50a.mov

merged and working


fix(package.json): npm scripts using cross-env

hey @SofianD 👋🏼
Thanks for this contribution 🙌🏼
Could you explain to me what it fixed?

hey @SofianD 👋🏼
Thanks for this contribution 🙌🏼
Could you explain to me what it fixed?

Hi @tony-go
When running the old commands on Windows, an error occurs because Windows can only use recognized commands. And on Windows the environment variable syntax is different from that on systems using the POSIX standard.
Below is a copy of the error that occurs (in french, sorry):
><EMAIL_ADDRESS>test-only
> NODE_OPTIONS=--loader=esmock cross-env tap --test-regex='.spec.js' --no-check-coverage

'NODE_OPTIONS' n’est pas reconnu en tant que commande interne
ou externe, un programme exécutable ou un fichier de commandes.

The cross-env package allows us to ignore syntax issues, but as the examples in the documentation show, commands must start with the keyword 'cross-env' even if it's not said literally.
Otherwise, the command execution seems to be split into two parts (before and after the 'cross-env' keyword).
Feel free to tell me if I misunderstood something.


global_row_key_cache_test.TestGlobalRowKeyCache.functional_test

CQL2 has been removed. The test needs rewritten.

I forgot to close this when I ported the test to CQL3: https://github.com/riptano/cassandra-dtest/commit/23fe3446650abedd1c040580a8b19fbb2355e6fe


Fix taskdialog vb project

Summary
Propagate the changes made by @EatonZ for @RussKie
Related to #4067

I think there are other samples that need to be fixed too.

Good call @EatonZ . I filed issue #4069 to track that.

Thank you


tests(rpc): Add some RPC acceptance tests

Motivation
We need to add some RPC port and config tests. Close https://github.com/ZcashFoundation/zebra/issues/3165
Solution
Add two tests:

rpc_endpoint to test configuration and make sure everything is up and running.
zebra_rpc_conflict to test RPC port conflicts.

Review
Anyone can review.
Reviewer Checklist

[ ] New tests logic and code
[ ] New tests pass

Follow Up Work
Should we add a port hint for already in use RPC port as we did for other endpoints ? If so, we can open a ticket to do it.

I want to open a new PR in the foundation repo and close this one as all tests are passing locally while in the CI a lot of stuff seems to be failing (probably by using a fork to push). However i don't want to lose the open comments yet so let me know when it will be a good time to do this @teor2345

I want to open a new PR in the foundation repo and close this one as all tests are passing locally while in the CI a lot of stuff seems to be failing (probably by using a fork to push). However i don't want to lose the open comments yet so let me know when it will be a good time to do this @teor2345

Happy to do this after we've resolved the request question.

@oxarbitrage did you want to open the new PR now? I'm happy to approve it straight away.

Replaced by https://github.com/ZcashFoundation/zebra/pull/3641


文件写入成功回调

   let URL = NSURL(string: "https://raw.githubusercontent.com/onevcat/Kingfisher/master/images/kingfisher-1.jpg")!
   KingfisherManager.sharedManager.retrieveImageWithURL(URL, optionsInfo: nil, progressBlock: { (receivedSize, totalSize) in
        print(receivedSize,  totalSize)
    }) { (image, error, cacheType, imageURL) in
        print("success")
    }

猫神，我想在下载完成之后并且获取本地化的回调，但是我看源码中NSFileManager的回调直接传入的nil，这里有什么办法能够让我获取到这个存储完图片之后的回调吗？
因为我要获取当前图片的字节来判断是否支持查看原图模式，谢谢

KingfisherManager or the extension methods on UIImageView encapsulate most details of downloading and caching an image. So, instead of using the highly wrapped types, you may need to use the downloader and cache separately for your purpose. You can get the original data in this callback by using the download API in ImageDownloader.

对的，我也知道可以单独用ImageDownloader来获取下载成功的回调， 但是其他的逻辑比如缓存，存入本地，这些都是和你写的retrieveImageWithURL这里面就一模一样了。那就意味着我只能重新将缓存以及下载还有存沙盒都重新自己在方法中实现吗？

That's right.
Another possible workaround would to be using the ImageDownloaderDelegate, in which there is a delegate method to inform you about the downloading response. From there you might be able to build a list your own to hold the information you need. So you could keep using the KingfisherManager or extension methods.

OK, Thank you very much！
So would you tell me the reason why completionHandler doesn't provide original data or file path?

Most of users do not care about it, for them, it is just noise and implementation detail. So by hiding it could make the APIs of framework easier to use and understand.


Runtime error in timeloop+accelergy exercises

Hi,
Thanks for providing such useful exercises.
I can run the examples in timeloop/accelergy successfully, but encounter one runtime error in the timeloop+accelergy exercise as show below:
Problem configuration complete.
ERROR: key not found: arithmetic, at line: 0

Could you help to solve such issue, thanks in advance for your time and effort!
Best regards,
Haoran You

Thanks for the comment. Are you using docker to run these exercises? If you are installing everything natively, please make sure timeloop is update-to-date. We are not able to reproduce the same issue with the docker or most-up-to-date timeloop.

Ok, thanks for your reply, let me update and try again.

I am facing a similar issue. Please let me know how to resolve this. I am using the latest version of timeloop.

Thanks for bringing up this issue.
I think this is probably due to a typo in the readme, instead of arch/eyeriss_like-int16.arch.yaml, please try arch/eyeriss_like-int16.yaml.
You can also do ls arch/ to see the files names in the arch  folder.
We have updated the readme as well.

Thanks a lot.


Clean up pwrmgr test bench to drive external reset request

          This just feeds the internal reset request back into the DUT as external reset request, right? This may work to trick some of the existing tests, but it doesn't allow to verify the wait-for-external-reset feature, and a quick simulation shows failures in the following tests:

pwrmgr_reset
pwrmgr_stress_all
pwrmgr_sec_cm_lc_ctrl_intersig_mubi

which don't fail in the latest nightly regression.
So we need to create an issue to fix this and properly verify it.
Originally posted by @andreaskurth in https://github.com/lowRISC/opentitan/pull/22575#discussion_r1570803707

https://github.com/lowRISC/opentitan/pull/22575#discussion_r1571177536

A loop back path from internal request to external SoC reset request was added in the testbench as a temporary fix.
A more elaborate TB fix is needed to

make sure that TB can control the timing of the generated external reset request after it sees an internal reset request
TB would also need to test scenarios where the external reset request is driven asynchronously by the SoC, with a pending internal reset request in flight.
Help with coverage closure


Add install script

Developed to not only include setting-up the expected configuration
file in its location, but also add the program to an expected location
on the PATH for many terminal users.
Not a huge use case, but thought I would start to provide the
convenience, and we can always improve on it later.

I was actually thinking about this after reading this article and the comments thread: http://www.linux-magazine.com/Online/Blogs/Productivity-Sauce/Connecting-to-proXPN-VPN-on-Linux-with-the-proXPN-Bash-Client

So, would we be able to merge this?

Definitely! I just haven't had a chance to try it out yet. I'll do my best to make some time tonight.

Ah, ok, just making sure there were no comments to add to it.  Thanks!

Probably not, I looked it over quickly the other day and nothing jumped out. Giving it another look now.

Left a few small comments. This looks really great thanks! If it's not too much trouble, squashing down to a single commit would be awesome, but it's not a big deal either way.

Now that #1 is done, I think I'll cut another release as soon as this install script is ready to go. :100:

What do you think about changing the config paths to /usr/local/etc

Oops clicked the wrong button! Opening a new issue about possibly changing the install and config paths but that doesn't need to hold this up.

Try those!
I had a small idea and was going to do some merging, not only with the master updates that I already included, but also the code from #17 & #18 that I was going to build off of, but instead of trying to figure out that part of GitHub and wondering if that's stepping out of bounds here, I'll just let you continue those tasks.

Looks good thanks! :+1:


More versions?

is there any plans on makeing it compatabel with more mods?

Please read our FAQ at https://sinytra.org/docs/connector/faq