Leading and trailing blanks lost when opening in Excel.

We used xlsxio to create a xslx with a text cell containing some leading or trailing blanks (e.g.: " a b "). Reading it back with xlsxio works as expected and delivers the exact same string (" a b ") for that text cell. But, opening it in Excel trims the blanks off of that cell (leaving only "a b"). Creating the same content using Excel, the content is not changed after saving and loading (always yielding " a b "). Apparently the result can even be read by xlsxio, also producing the correct content (" a b ").
We found out that Excel automatically adds an attribute xml:space="preserve" to text-nodes containing leading or trailing blanks. We then changed in function xlsxiowrite_add_cell_string in file xlsxio\lib\xlsxio_write.c the line
write_cell_data(handle, NULL, "<c t=\"inlineStr\"" STYLE_ATTR(STYLE_TEXT) COLNRTAG "><is><t>", "</t></is></c>", "%s", value);

to
write_cell_data(handle, NULL, "<c t=\"inlineStr\"" STYLE_ATTR(STYLE_TEXT) COLNRTAG "><is><t xml:space=\"preserve\">", "</t></is></c>", "%s", value);

That appears to have fixed the issue.
We 'd be glad if you fixed that in the repository, too. (Maybe you could

check automatically cell by cell, if the additional attribute was needed,
add a parameter bool preserve_space to xlsxiowrite_add_cell_string,
add another function xlsxiowrite_add_cell_string_preserving_space

or something like that.)
Kind regards.

Automating the addition of the attribute only for problematic cells, we encountered some surprises. The following appears to work for most cases:
Before the definition of function xlsxiowrite_add_cell_string add:
static int has_leading_or_trailing_blanks(const char* value)
{
  if (NULL == value)
  {
    return 0;
  }
  if (isspace(*(unsigned char*)value))
  {
    return -1;
  }
  if ('\0' == *value)
  {
    return 0;
  }
  size_t Length = strlen(value);
  /*
  The cast to unsigned char* is required to avoid a crash on platforms where char is signed.
  The crash occurs when a text cell contains a multi byte character
  such as ß in utf8 (c3 9f) at its end (as in "...ß").
  */
  return isspace(((unsigned char*)value)[Length - 1]);
}

Replace the line from the original post with the following block:
{
  if (has_leading_or_trailing_blanks(value))
  {
    write_cell_data(handle, NULL, "<c t=\"inlineStr\"" STYLE_ATTR(STYLE_TEXT) COLNRTAG "><is><t xml:space=\"preserve\">", "</t></is></c>", "%s", value);
  }
  else
  {
    write_cell_data(handle, NULL, "<c t=\"inlineStr\"" STYLE_ATTR(STYLE_TEXT) COLNRTAG "><is><t>", "</t></is></c>", "%s", value);
  }
}

The checks for leading and trailing spaces are obiously not sufficient for multi byte characters that count as spaces (if something like that exists).
A performance drawback of this automatic approach is that it has to scan all strings without a leading space to discover their length...

Thanks for trying all this.
But shouldn't I just always add xml:space="preserve"?
The XMLs are compressed anyway, so a recurring phrase like that won't bloat the xlsx file...

That sounds reasonable, especially considering the overhead involved with checking for the last character.
On the other hand, when I look at the strings that Excel stores (in shared strings), only those texts have the attribute that need it. I don't know where else this would matter (Perhaps, the auto trimming behaviour of cells without this attribute is desired in some circumstances?). And then, for large tables of (small) texts (that usually wouldn't need the attribute) avoiding the attribute might still make a considerable difference in size. So we tried to stick as close to the observed Excel behaviour as possible.
You might consider offering different flavours of the text adding function, simultaneously:

xlsxiowrite_add_cell_string: as before,
xlsxiowrite_add_cell_string_adaptive: with automatic attribute (incurring strlen-overhead) and
xlsxiowrite_add_cell_string_preserving_space: always add attribute without checking but incurring (limited) size overhead.

I have now added xml:space="preserve" to header and text cells to preserve spacing.
In xlsxio_write.c I defined a macro need_space_preserve_attr(value) which currently always returns non-zero, but in the future we could change this to detect if spase preservation is needed.
This change will cause the sheet XML file to grow in size, but since it's a recurring pattern the ZIP compression will compress this well.
For the test file generated by the example application this means 369 bytes extra (from 29435 to 29804 bytes = 1,25% increase).

Fixed in release 0.2.25


Security Error: Graphviz layout returned error code -1

Describe the bug
When attempting to generate documentation for my solution I get the Graphviz layout returned error code -1 error and noting except the folder get created
To Reproduce
Steps to reproduce the behavior:

Run the .exe file from the 1.2.4 self-contained version
Select any solution
Error occurs

Expected behavior
Solution documentation is created
Screenshots

I have the same problem.
Since the target framework ".NET Core 3.1" is out of support, I upgraded the projects to .Net 8 and all the NuGet packages to the latest versions.
The upgrade processes are easy.  There are some small fixes but there is no too much trouble. The only exception is "Rubjerg.Graphviz" NuGet package.  When using V1.1.0 of the package the projects are compiled and working fine.  However, if upgrading the package to V2.0.0, I got the "Security Error" very similar to the karolfilipowicz described.
I hope someone can help to figure out this mystery and (maybe) upgrade the projects to .Net 8.

Think I have identified the problem (reported it on the Rubjerg.Graphviz github repo) and have a workaround to make it work. Also updated my solution to .Net 8, had a few successful tests. Will continue testing, and if no further issues are found I'll release an updated version in the coming days

Reme,
Thanks to get back to me for this issue.
While you are working on the code.  I have a suggestion as following:
In our environment, the PowerDocu.GUI is required to "Run as Administrator" when click on the button, "Update your existing set of connector icons".
The problem is the cached icons are written to the Resources\ConnectorIcons folder under the program installation location.  I have a quick fix to change the line (ConnectorHelpers.cs, line 18) to
private static readonly string folderPath = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData) + @"" + AssemblyHelper.GetApplicationName() + @"\ConnectorIcons";
Where the new function AssemblyHelper.GetApplicationName() is implemented in AssemblyHelper.cs as:
public static string GetApplicationName()
{
return Assembly.GetEntryAssembly().GetName().Name;
}
This will create the icon cache in the folder under C:\Users(UserName)\AppData\Roaming and users won't need "Run As Administrator" to perform the function.

From: Rene Modery @.>
Sent: Thursday, January 18, 2024 5:08 AM
To: modery/PowerDocu @.>
Cc: CyberArcadian @.>; Comment @.>
Subject: Re: [modery/PowerDocu] Security Error: Graphviz layout returned error code -1 (Issue #128)
Think I have identified the problem (reported it on the Rubjerg.Graphviz github repo) and have a workaround to make it work. Also updated my solution to .Net 8, had a few successful tests. Will continue testing, and if no further issues are found I'll release an updated version in the coming days
—
Reply to this email directly, view it on GitHubhttps://github.com/modery/PowerDocu/issues/128#issuecomment-1898449250, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATTBA3PPMDFGE2RZDWOHIYTYPENDVAVCNFSM6AAAAABBGNDBMGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJYGQ2DSMRVGA.
You are receiving this because you commented.

The Graphviz issue should be fixed as part of today's 2.0.0 release. Can you confirm this?
Will test the recommendation for the connectors next

Rene,
I Confirmed the Rubjerg,Graphviz version 2.0.2 works correctly with PowerDocu v2.0.0 without exception or errors.  Thanks a lot.

Replaced the location for the connector icons as recommended by you (https://github.com/modery/PowerDocu.Common/commit/ab7646e3af9ebebc215b878dd109fa1a4d938aaa), will be part of the next release

Hi @modery! So sorry to get back to you so late. Unfortunately I get another error now but still related to Graphviz

I believe it happens for all flows.

Replaced the location for the connector icons as recommended by you (modery/PowerDocu.Common@ab7646e), will be part of the next release

This is now live in 2.0.1


US-3: More in-depth results of AI models and documents

As an application approver, I want to be able to click on results on the dashboard, including AI models, to access more in-depth result pages, so that I can perform a thorough analysis and facilitate the application approval process.
Linked issues:

[x] #12
[x] #13
[x] #14

Acceptance criteria:

[x] The user shall be able to click on each model to see more in depth responses, and have the opportunity to send information back to NorKart
[x] The user shall be able to see the documents uploaded, and open them individually

This is done


Keywords section of the metadata

Hello,
I've been wondering for a while where do the metadata keywords appear on a dataset's GBIF page after publishing? I can see the geographic description & taxonomical keywords (when available) but not the general keywords from the 'Keywords' metadata section.
Thanks for your help!

@SophiePamerlon Thank you for your question. I think they aren't displayed at the GBIF portal

Are they used in the Schema.org representation of the dataset?

Unfortunately, I can't answer this, and it's not an IPT scope. Closing this


feat: support root components outside layout

Now it's impossible to add component which not rerender when change layout
Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
With this prop in config we can add components under root nuxt instance like this

Resolves: #7753, #2603
nuxt.config.js
Array variant
  ...
  rootComponents: [
<EMAIL_ADDRESS>  ],
  ...

Object variant
  ...
  rootComponents: {
    global<EMAIL_ADDRESS>  },
  ...

And use it in components
{
  methods: {
    method() {
      const { global } = this.$nuxt.$rootComponents;
      if (global) {
        global.someMethod();
      }
    }
  }
}

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly. (PR: nuxt/docs#2118)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[x] All new and existing tests are passing.

Add fixture, update config snapshot and fix 2 tests in cli

Codecov Report

Merging #7768 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #7768   +/-   ##
=======================================
  Coverage   68.83%   68.83%
=======================================
  Files          90       90
  Lines        3821     3821
  Branches     1034     1034
=======================================
  Hits         2630     2630
  Misses        968      968
  Partials      223      223

Flag
Coverage Δ

#unittests
68.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/config/src/config/_app.js
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Hi @Kolobok12309 thanks for PR ❤ Actually with nuxt3 it is possible to define top-level components via app.vue. Since nuxt2 is feature freeze, closing but can see benefit of avoiding re-render by layout changes. If it is making critical issues, we can think of a workaround for #7753

Hi @pi0, ok, when released nuxt3 with vue3 try feature.
Is not critical moment, so we can wait nuxt3.
Mb someone else will ask for this feature :D

i just mount manually that component in middleware once.

Wait, are you telling me that nuxt re-renders all layout components on route change? Isn't that the whole point of layouts, that they don't re-render, only the route specific data in  should re-render?
damn 😮


Can't select items

The Script doesn't work on Ubuntu 22.04 LTS.
I can't select any item in the menu.

@reg3xp try 20.04 please


Unable to checkout repo with this action workflow

I am trying to checkout repo i github enterprise with this workflow, but i am getting the error as
  Retrieving the default branch name   Not Found
while i looked some other third party workflows, it worked.
eg: https://github.com/peter-murray/workflow-application-token-action
could you please help us here
Regards,
Atib

Can you please setup a public repository that reproduces the problem? We can't help you with the information you provided. We use this action ourselves in conjunction with actions/checkout, so it definitely works.

Hello @gr2m
Thanks For you reply.
https://github.com/iamatib/checkout-demo/actions
I have setup here, could you please have a look.
The github app has "read" permission to all repo's.
I am trying checkout a private repo from the this repo https://github.com/iamatib/checkout-demo
could you please have a look.
Thanks in Advance.
Regards,
Atib

You are trying to checkout iamatib/checkout-test from iamatib/checkout-demo, but the token that gets created only has access to the current repository unless you set owner and/or repositories. In your case, try this
  on: [workflow_dispatch]

  jobs:
    hello-world:
      runs-on: ubuntu-latest
      steps:
        - uses: actions/create-github-app-token@v1
          id: app-token
          with:
            app-id: ${{ vars.APP_ID }}
            private-key: ${{ secrets.PRIVATE_KEY }}
+           repositories: 'checkout-test`
        - name: Checkout Repository
          uses: actions/checkout@v3
          with:
            token: ${{ steps.app-token.outputs.token  }}
            repository: iamatib/checkout-test

Hello @gr2m
Thank You, I thought when we give permissions for repo's in Github App, it should already be authenticated to run api's.
Yeah, now i  got it why it was not working.
Will close the Issue.
Regards,
Atib

Hi @gr2m
could we also rotate this private key in timely manner? is the rotation possible ?
Thanks in Advance.
Regards,
Atib

is the rotation possible ?

I don't think it's possible to programmatically change a GitHub Apps private keys unfortunately. Repository secrets can be rotated though.

Thank You again..!


Feature: Multimarkdown support

Would it be possible to add multimarkdown support?
Things like footnotes are not rendered properly.
Thanks
For multimarkdown info/reference see.
http://fletcherpenney.net/multimarkdown/features/
https://rawgit.com/fletcher/human-markdown-reference/master/index.html

You can enabled some of multimarkdown's functionarity in OmniMarkupPreviewer's setting like the picture above.


Ocelot.Configuration.File.FileReRoute class is not supported

ocelot new source code ,Ocelot.Configuration.File.filereroute is  missing,which  has been replaced by Ocelot.Configuration.File.FileRoute.

I get Reference to type 'FileReRoute' claims it is defined in 'Ocelot', but it could not be found. Suggestions?


DNS rebinding vulnerability found in etcd dependency

Package
Affected versions
Patched version

go.etcd.io/etcd(Go)
< 3.4.0
3.4.0

DNS rebinding vulnerability found in etcd 3.3.1 and earlier. An attacker can control his DNS records to direct to localhost, and trick the browser into sending requests to localhost (or any other address).
We need to upgrade the dependency.

@wonderflow but latest version is v3.3.27+incompatible

@hzliangbin  I  think >3.3.1 is fine

@wonderflow already fix by commit,  issue could be closed

@hzliangbin  Thanks


Update webpack-dev-server dependency

https://www.npmjs.com/advisories/725
Details
Security advisory, needs version >=3.1.6
We are currently at 2.*
Lots of the dependencies needed to be updated to match peer dependencies as well to satisfy this.
Acceptance Criteria

Update the dependencies, verify everything still works

note: this is only when you run npm run dev a web server locally to debug your web pack app with live-rebuilds of the code. It's a command meant to run on a developer's local machine, and is not related to the final code that ends up on a CDN.


Header Content Count field should convert the value into 0 if given anything apart from valid data(numeric value)

Bug report
Title
Header Content Count field should convert the value into 0 if given anything apart from valid data(numeric value)
Preconditions
1. framework Version. Master

Steps to reproduce
1. step1: enter any string value in header content count and save

Expected result:
after saving it should convert that string into 0
Actual result:
same string is getting saved

Fixed in master


Alert

Closing this Draft PR.
Created new draft pr: https://github.com/nagra-opentv/react-native-skia/pull/152


[WFCORE-562] Wrong order of elements in jboss-cli.xml

BZ: https://issues.jboss.org/browse/WFCORE-562

Windows Build 941 is now running using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c

Linux Build 1256 is now running using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c

Core - Full Integration Build 519 is now running using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c

Windows Build 941 outcome was SUCCESS using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c
Summary: Tests passed: 3052, ignored: 95 Build time: 0:18:29

Linux Build 1256 outcome was SUCCESS using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c
Summary: Tests passed: 3052, ignored: 95 Build time: 0:19:23

Core - Full Integration Build 519 outcome was SUCCESS using a merge of 98a3774a2ff221ac02d93d01e353685ce1cb427c
Summary: Tests passed: 2859, ignored: 381 Build time: 0:48:42


Closing unsaved file that was modified by another application causes large slowdown and freeze

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.56.2
OS Version: Linux Mint 20.1 x86_64

Steps to Reproduce:
Real-life example.

Open a small JSON file (3kb is enough).
Use for example palette command "Format document".
Don't save the file.
Modify the same file using another editor.
Try closing the unsaved file in VS Code.
CPU usage should jump to 100% (single thread).
After 10~20 seconds VS Code will unfreeze.

Can you follow this wiki and post the results back here to understand more about why performance is so bad for you: https://github.com/Microsoft/vscode/wiki/Performance-Issues
Thanks!


[UWP] .net native compiled project won't run (Entry Point Not Found error)

When targetting .net standard 2.0 for a windows uwp application (Windows IoT core, ARM targetting Windows 10 build 16299). The project compiles normally however when run on a target devices generates the error "Entry Point Not Found".
Please refer to the discussion on the MSDN forum here
When the same project is compiled in debug modus (and thus without .net native toolchain), the application runs normally on the target device.
It is difficult to determine in more detail what the exact cause is of this error.

cc @lt72 @sergiy-k

@tijoytom, is this the same issue with Windows SDK and p/invokes that you investigated earlier?

Could be , it looks similar.
@bderuijsscher can you please share the verbose build log?

Gladly, however how do i achieve a more verbose build log than what is rendered by default in the output window of visual studio?
Regards

Tools -> Options-> Build and Run -> Change msbuild output verbosity

buildlog-verbose.txt
Thanks, please find the verbose build log in the attachment. I have removed references to my username and projectname by replacing them with [USERNAME] and [PROJECTNAME]
I hope this helps to diagnose the possible cause of the error.

Ok , Looks like the same issue ,  what nuget packages are you using ?
P/Invoke 'FindFirstFileEx' resolved to 'FindFirstFileEx!api-ms-win-core-file-l1-2-0.dll' from 'SupportedAPIs.xml'. (TaskId:394)
P/Invoke 'FormatMessage' resolved to 'FormatMessage!api-ms-win-core-localization-l1-2-0.dll' from 'SupportedAPIs.xml'. (TaskId:394)

This project uses:
Microsoft.Azure.Devices.Client (1.4.0)
Microsoft.NETCore.UniversalWindowsPlatform (6.0.1)
Newtonsoft.Json (10.0.1)
Serilog (2.6.0)
Serilog.Formatting.Compact (1.0.0)
Serilog.Sinks.Http (4.2.1)
Serilog.Sinks.Rollingfile (3.3.0)
Serilog.Sinks.Trace (2.1.0)
System.Collections (4.3.0)
And a IoTDMClientLib library as indicated by the Azure Device Management docs.
Updating the Nuget packages above to their newest versions yields the same results. Furthermore, if you refer to the original poster of this problem on the MSDN forum (see reference link in initial description), there is a sample project that also generates the runtime error as mentioned here. The question therefore remains if the problem relates to the nuget packages used.

I'm having exactly the same problem. When compiling project, I received these warnings:
Unresolved P/Invoke method 'kernel32.dll!CreateFile'
Unresolved P/Invoke method 'kernel32.dll!SetFileValidData'
But it came from my imported lib not Nuget.

hello @tijoytom,
after some further evaluation using a blank base project and adding the mentioned Nuget packages one by one, It appears that the Microsoft.Azure.Devices.Client package is the offending package.
When adding this library, the error will be generated.
Since the original poster of the problem (MSDN forum) uses another package, I suspect the underlying problem is in one of the libraries/components used in these packages.

@tijoytom, is there a workaround for this issue?

Hello, the developers of the Microsoft.Azure.Devices.Client have released an update to the package to fix the problem. Please refer to the discussion in this issue
Apparently, it should also be possible to include the newest version of  nuget package Pinvoke.Kernel32 in the project as a workaround, however I have not yet tested this approach.

@sergiy-k  No workaround since the offending PInvoke comes from a nuget package , Yes , there is a workaround if the app can be compiled from command line . Anyways @AArnott  have published the new package with the fix.
@bderuijsscher Let me know if you still hit issue after using the new package

You can check your project.assets.json file (in your intermediates directory) to see if PInvoke.Kernel32.dll is being brought into your compilation. If it is, then yes, installing the latest PInvoke.Kernel32 package into your project to effectively upgrade the one you're getting transitively should workaround the Windows SDK bug that causes this issue.

Yes, the upgraded package of Azure Devices Client solved the problem, and @AArnott thanks for the advice for other situations for which this error might occur.

Hi everyone,
my app is facing the same problem. When compiling app, I got these warnings:
MCG : warning MCG0007: Unresolved P/Invoke method 'kernel32.dll!CreateFile' for method 'xxx.CreateFile(System.String, System.UInt32, System.UInt32, System.IntPtr, System.UInt32, System.UInt32, System.IntPtr)'. Calling this method would throw exception at runtime. Please make sure the P/Invoke either points to a Windows API allowed in UWP applications, or a native DLL that is part of the package. If for some reason your P/Invoke does not satisfy those requirements, please use [DllImport(ExactSpelling=true) to indicate that you understand the implications of using non-UWP APIs.
So, I think it might be the reason that caused crash when compiling with .net native. I've tried the workaround of @AArnott but Pinvoke.Kernel32 doesn't not have CreateFile method.
Here is my project: https://bitbucket.org/wpdev01/demobuilderror

@khanhnb Waht you are  seeing is not the same as the original problem that was discussed in this thread.
CreateFile is not a valid API for UWp , you should use "CreateFile2"

Microsoft Azure IoT 1.7.0 has released a new package without the PCLCrypto dependency (no workaround is required).
For our LTS branch (1.6.*), please follow @AArnott's instructions here and in https://github.com/Azure/azure-iot-sdk-csharp/issues/315#issuecomment-373558193
@jkotas @tijoytom FYI: This issue is also tracked internally by the Windows SDK team.

@tijoytom : you mean that my app only works in debug mode and fails when compiling .NET native is because of using invalid API for uwp (actually it's using some ws2_32.dll and kernel32.dll APIs)? I'm sorry that lacking of information in output window makes me misunderstand it.

@khanhnb said:

Pinvoke.Kernel32 doesn't not have CreateFile method

Not yet, that's true. And even when it does, it won't be available to UWP apps for the reason @tijoytom gave. PInvoke.Kernel32 also doesn't have CreateFile2, but that can certainly be contributed and it will be available to both UWP and desktop apps.
But as you're already getting CreateFile entrypoint breaks, it sounds like you have the P/Invoke coming from someplace else (ws2_32.dll perhaps? although I don't know what that is), so you're bringing in a non-UWP compliant binary.

@khanhnb Not a bug , you can't use all ws2_32 API in UWP . You need to stick to UWP approved APIs

@khanhnb I am closing this issue , since the original issue is fixed. If you need help , please open another issue with details.

Hi, I'm facing the same issue and the proposed solutions did not work for me. This is the exception when the app starts using the .net native compiler:
Exception thrown at 0x77384184 (ntdll.dll) in [MyAppExe]: 0xC0000139: Entry Point Not Found.
I'm building with Visual Studio 15.6.1.
Here are the nuget packages I think are relevant:
Microsoft.NETCore.UniversalWindowsPlatform 6.0.8
NETStandard.Library 2.0.1
PCLCrypto 2.0.147
PInvoke.BCrypt 0.5.134
PInvoke.Kernel32 0.5.134
PInvoke.NCrypt 0.5.134
PInvoke.Windows.Core 0.5.134
Validation 2.4.18
Please let me know if you have any other suggestions. Thanks.

Hi @jernelson7,
I am quite sure it is caused by one of the packages. You would be best off eliminating the packages one by one to determine which is the offending package. Alternatively you could build a new project and adding the packages one by one.
In the discussions I followed there were some mentions of a certain cryptography libraries, so possibly the pclcrypto is the offending package.
As soon as the package that causes this error is known one of the MSFT people should be able to assist in the proper solution.
I hope this helps, regards

@bderuijsscher I will give that a shot. I was under the impression from this discussion that updating to the latest nuget version of PInvoke/PCLCrypto/etc should resolve the issue, but I have done that and it is not resolved.

Update: I am able to reproduce this easily with a small test app. It has a Uwp project which references a .NET Standard 2.0 project. When I add PCLCrypto 2.1.17-alpha-g5b1e8dff8c to the .NET Standard project, it crashes immediately on startup with the same error.

@jernelson7 Thanks for reporting the issue , it's the same FormatMessage issue that was discussed in this thread. I am not sure why the new package did not fix it  @AArnott would know.
Mean while here is a workaround

Find "SupportedAPIs-x86.xml"    on your machine , it's likely in "..Program Files (x86)\Windows Kits\10\App Certification Kit" .
Rename it to say "SupportedAPIs-x86.dat" . Recompile your app and you should be good.

@jernelson7 you must install the latest pinvoke.kernel32 package to apply the workaround. That is where the workaround lies... Not pclcrypto.

@tijoytom Thanks, that does resolve the problem, but of course it needs to compile on other machines besides my own (build server, other developers, etc.), so it's not a very practical solution.
@AArnott you can see in my test app that I'm not referencing PInvoke directly, only PCLCrypto. It looks like PCLCrypto is bringing in PInvoke v0.5.126. Can you please push a new version that references 0.5.127?

@jernelson7 Yes, I understand you wouldn't naturally have a direct reference to PInvoke.Kernel32, but that is the standard nuget way of bumping up a dependency library's version for your app, so it's perfectly normal.
I do have work on my PCLCrypto backlog to update PCLCrypto soon to resolve some other issues. At that time, I expect I'll bump up the PInvoke dependency version as well so that it works out of the box as you say.


Public Safety: Pedestrian injury

Description: Pedestrian injury hospitalization per 100,000 residents
Logic:
Source(s): https://a816-dohbesp.nyc.gov/IndicatorPublic/VisualizationData.aspx?id=2093,4466a0,101,Summarize
Year(s):

[ ]

Geographies:

[ ] PUMA
[ ] Borough
[ ] City

Race and ethnicity breakdown:
Denominator:
SME: DOHMH
Questions:  What does the note "Contains numerous measures" mean?
Roadblocks:

once we have final CD to PUMA crosswalk we should be good to go.

It doesn't seem like the there is an API so the .csvs will have to be downloaded one by one. I can approach the same way I did traffic injuries, where I write put the source data in the repo, write the logic, and then finally and then route them through data library once I have the logic working. Let me know if that's not a good approach @AmandaDoyle

Also the portal only has 2016 data which is curious?

Dealing with PUMA that combine multiple CD's by averaging, let me know if that's not correct


rename id column in address table

If we will just copy id of feature into address table (just like our former storage strategy in CityDBv4),
then we need at least to rename the column name as "feature_id".
address table

Good catch. I'd rather say the foreign key constraint to feature is wrong and should be removed. Like in v4, the address table has its own sequence for the id column which is used to populate the value. Also the appearance tables in v5 have similar foreign keys to feature although they have their own sequences. These foreign keys should be removed as well.

According to the off-line discussion at TUM last year, the foreign key constraint to feature should be kept. https://github.com/3dcitydb/3dcitydb/issues/105. The sequence address_seq is wrong and must be removed.

Thanks @yaozhihang for pointing to this. In this case, id should not be renamed in my opinion because an address is an feature and does not reference a feature.

Also good catch @clausnagel, the foreign key to the feature table should be removed from the appearance

We might add gmlid to the address table, because its missing in the current schema.

According to the offline-discussion on 10.03.23, we decided to drop the address table again. This ticket is not relevant anymore. So close it.


Fix Header for all breakpoints

Fix #20
Test URLs:

Before: https://main--airlessco--hlxsites.hlx.page/
After: https://feature-header--airlessco--hlxsites.hlx.page/

This PR will break some things on the body of the pages - another PR with review to fix all of that will be forthcoming.
There were conflicts in the global styles and header styles that couldn't be resolved, and reworking the body blocks.


fix(hmr): duplicated modules because of query params mismatch (fixes #2255)

Description
This PR fixes #2255 by only appending ?import when it should be.
I passed explicitImportRequired property to do the check on server side, not to duplicate isExplicitImportRequired function to client/client.ts.
fixes #2255
Additional context
?import is used to differentiate import './foo.png' and fetch('./foo.png'). The ideal way is to use Accept header but unfortunately requests by import uses Accept: */* and cannot be used.

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

I considered about including ?import in path property, but I didn't reach a conclusion.
One thing I can say is that introducing explicitImportRequired property has less implication to existing implementations than including ?import in path property.

@sapphi-red lets mark the new field as @internal or @experimental so we can merge this fix in 3.1, and we can discuss later with more time in team meeting


Request/thought: Client agnostic way of adding arguments to runnables

I'm not sure if this is within the scope of the language server, but something cool would be the ability to add custom arguments to runnables without the client doing anything (by reading a json file on project root if it exists maybe). This would atleast prevent having to implement it for every client because tbh it is a desirable feature.

I think that's https://github.com/rust-analyzer/rust-analyzer/pull/5954 ?
cc https://github.com/microsoft/language-server-protocol/issues/1239

Sorry for being so late, yes that is what i was thinking of.
Closed as #5954 is already merged

Ah, meant to link https://github.com/microsoft/language-server-protocol/issues/944
I wish there was more motion on this feature -- it is one of the best feature of RA. If, on neovim side, you'd wish to add first-class support for it, I'd be happy to implement anything on VS Code side.
The main thing this feature is missing at the moment is configuration persitance and editing. As a user, I want to be able to get the "run" ccommand for a specific test, then edit it, save it, and use the edited command across restarts.


[AlertDialog] forceMount causes document body scroll-locked even when AlertDialog is closed

Bug report
Current Behavior

I want to animate AlertDialog entering and closing, therefore I was planning on using forceMount prop.
Document body is scroll-locked when applying forceMount to AlertDialog's Portal sub-component, even when the dialog is closed.

Removing forceMount from Portal successfully fixed the behaviour, therefore I'm convinced there's an issue between forceMount and scroll-locking behaviours.
Expected behavior

Document body should not be scroll-locked when the AlertDialog is in closed state.
Reproducible example
Codesandbox repro
Suggested solution

Conditionally apply the scroll-locked behavior on open, instead of on mount of AlertDialog.
Additional context

Your environment

Software
Name(s)
Version

Radix Package(s)
@radix-ui/react-alert-dialog
1.1.1

React
n/a

Browser
n/a

Assistive tech
n/a

Node
n/a

npm/yarn
n/a

Operating System
n/a

Turns out I can use CSS-only animations for my use-case.
Hence, that bug is non-blocking to me ! 👌

This happens even dont use Portal.
The pointer-event: none is like a default property on radix

+1
This makes animating a dialog in and out extremely annoying requiring second state to track animating in and out. Force mounting should not be blocking scrolling/clicking if the state is closed.


Remove NON AD specific cmdlets

There are several cmdlets that are useful in domain environments but are not for managing a domain environment.

Following functions are not specific to AD

Add-UserToLocalGroup
Get-LocalGroupMember
Add-DomainGroupToLocalGroup
Set-ComputerName

see commit 8923494a6a90d3251c80d61a100ea94ec4c420a7


Remove HostNetwork from attachment Pod

What this PR does / why we need it:
Removes HostNetwork from attachment Pod.
HostNetwork requires a more privileged security policy -
for example in the case of SCC or PSS.
There doesn't seem to be a relevant reason to trade security here.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
None

There is no way to turn of networking on a pod

I believe we can achieve something close to that with Multus - IIRC there's some mechanism to specify the network (--> CNI plugin) to be used instead of the default pod network. With that, we can use a bogus (or at least, a dummy) CNI plugin that doesn't consume of the actual default pod network.

/lgtm

Well at this point, we have 1 attachment pod per VM, regardless of how many disks are hot plugged, so there is no need for host network anymore. So feel free to remove that.

yes agreed.
As Alexander has pointed out, the host network on the attachment pod was simply an optimization for the attachment pod. That's it.

/hold cancel
SCC was removed.


Fix: condition in tx simulation

Looks like another Boolean logic mistake. Probably after someone asked to refactor it in the PR comments.
Fixes https://github.com/safe-global/safe-react/pull/4003#issuecomment-1171715025

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 37.631%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/helpers/TxModalWrapper/index.tsx
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/safe/components/Transactions/helpers/TxModalWrapper/index.tsx
1
57.32%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4104

Relevant Lines:
9904

💛  - Coveralls

Looks good, merge it in the RC


Workflow Compensation

Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[ x] Specification
[ x] Schema
[ ] Examples
[ ] Usecases
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:
Adds compensation capabilities to workflow language
Special notes for reviewers:
Additional information (if needed):

Easier way to read the new "Workflow Compensation" section in the pr branch - https://github.com/tsurdilo/specification/blob/compensation/specification.md#Workflow-Compensation

@manuelstein @jorgenj mind taking a quick look whenever you have some free time? I'd like to get this in cause next hands-on video will be on this subject ;)

here is a video that describes compensation and how to use it in SW per this pr - https://www.youtube.com/watch?v=Ea8yR-EkrZQ&feature=youtu.be
maybe it helps with reviewing it.
I would like to get this in before out next meet if possible. If there are no further comments on this PR I'd like to go ahead and push it and we can improve for next release if needed.

Will go ahead and merge as this is important to move forward and there seem to be no comments on it.


Temp

Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Tests performed by the developer:
Tips for QA testing:

The Hummingbot Foundation just started the governance process for the community to approve or reject Pull Requests through Pull Request Proposals.
For more information about how PRP votes work, check this page: https://hummingbot.org/governance/prp/
We want to know if you still have interest in having your Pull Request merged into the main codebase.
If so, please check the following on your Pull Request for our team to be able to open it for community voting:

Your Pull Request is targeted to the development branch
Your branch is up to date with the current development branch
All GitHub checks have passed
There is no merging conflicts
The Pull Request have a clear description of what it does

If you have any questions or need any help, feel free to reach us out on the #dev-general channel on our Discord (https://discord.gg/HAgZ79nzU4)

If you have no interest in merging this PR to our code-base or don't reply to this comment in 30 days, the Foundation team will close this PR.
You can still create a new PR any time you want!
Thank you for contributing to Hummingbot Client!

senamakel Foundation would need you to create a PR Proposal to community for voting. This is to merge PR or not. Please contact me if you have questions

@phbrgnomo @mifeng This PR has a very bad name and no description at all. Also it is pointing to the master branch. It should be removed.


Add support for exiv2 v0.27.1

The BasicError class was not being included because we were not
requiring a header that's present in v0.27.1.
I have tested this on exiv versions 0.24 through to 0.27.1 and all work fine :)

Just checking and this should actually also work on older versions of exiv2 as well, the error.hpp header is available for older versions, this change is required as previously  image.hpp would require iptc.hpp which would require error.hpp but this is no longer the case in the newest version so we must be explicit.

Thanks all, will release this as v0.0.9 :)


Progress on API?

Is there any progress on the API?
I would like to fetch the latest post from my ghost blog and show it on an external site.
Documentation is still empty:
http://api.ghost.org/v0.1-alpha/docs

Hi @4Digits, please see this issue comment


The tooltip in the Visual Studio shows an error

The tooltip for Linq.select' will show an error.
(Linq.select will display the tooltip as expected.)

The following is for copy and paste.
testCase "test"
  <| fun _ ->
      let xs = [ 1 .. 5 ]
      let ys = xs |> Linq.select' (*)
      Expect.equal ys [ 1; 4; 9; 16; 25 ] "select with index"

I used Visual Studio 2022 and installed this library (Funtom.Linq v0.0.3) with Nuget.
The test itself passes, so there is no problem with the runtime.

Hi, @eagle-k
Linq.select' can be obtained with value and index as parameters.
Therefore, your code will be expanded as follows:
let xs = [ 1 .. 5 ]
let ys = xs |> Linq.select' (fun value index -> value * index)

In other words, you need to specify [ 0; 2; 6; 12; 20; ] as the expected value.
I recommend that you modify the test code as follows:
testCase "test"
  <| fun _ ->
    let xs = [ 1 .. 5 ]
    let ys = xs |> Linq.select' (*)
    let expects = [| 0; 2; 6; 12; 20; |]
    for i = 0 to expects.Length do
      Expect.equal ys[i] expects[i] "select with index"

OR
testCase "test"
  <| fun _ ->
    let source = [ 1 .. 5 ]
    let actual = source |> Linq.select' (*) |> Linq.toArray
    let expect = [| 0; 2; 6; 12; 20; |]
    Expect.equal actual expect "select with index"

Thanks,
midoliy

Sorry for the confusion, the test case is wrong.
The main issue is that the tooltip for select' is not displayed correctly in the IDE.
I am getting internal errors in both Visual Studio 2022 and Visual Studio Code.
There is no problem at runtime.

I was able to confirm the error on my end as well.

It seems that there is a problem with F# Language Sever.
It is difficult to fix it on the library side, so I will report it to the Visual Studio team.


_id Formats

I noticed that the _id format for colors was in a UUID style. How are you doing this and how can I change it to use the Mongo ObjectId format (56a90a02a8ab459af30cd324) or the Meteor _id format (G7yZxw6vnaY8QZtsg)?

This is not something big. For that, you need to use Meteor's random package. Just add the Random package to the context.


Add package servant-static-th

This PR adds the package servant-static-th.

Thanks!

The tests failed, I'm guessing some extra-source-files are missing?
test/Spec/ServerSpec.hs:1:1: error:
    Exception when trying to run compile-time code:
      test/test-dir: getDirectoryContents:openDirStream: does not exist (No such file or directory)
    Code: createApiAndServerDecs "FrontEndApi" "testDirServer" testDir

@bergmark Thanks for the heads up.  You're right, I think it's the missing extra-source-files.
I've fixed this is v<IP_ADDRESS> on Hackage.
For future reference, is there somewhere public I can see the tests you're referring to?

Thanks!
No we don't publish the results anywhere, I've thought about just scp:ing it somewhere but I haven't gotten around to it.

Ah, okay, I see.  Thanks for letting me know about the error though!


[-] FO : Correct IE8 syntax error

In Internet Explorer version 8 and older, having a property called "default" (technically a reserved word) defined as a bare word throws a syntax error and halts script execution. If you happen to be using CCC, this will break all Javascript on your entire site. Wrapping the property name in quotes fixes the issue.

Hi,
Thank you for your PR. I believe this indeed needs to be fixed in default theme too
https://github.com/PrestaShop/PrestaShop/blob/1.6/themes/default-bootstrap/js/modules/blockwishlist/js/ajax-wishlist.js#L316
Regards

Good catch. I've submitted a PR to that repo as well.

Many thanks !

Hi,
thank you for your contribution.
Best regards.


Ungrinchify! Merged pull requests are rendered even if no gifts are sent in the current year

Fixes #1717

🎅

! <3


Add no longer actively maintained message on plugin site

Today the website don't have a message telling that the plugin is no longer actively maintained. So, if the user don't check the Github repo, he'll never know that the plugin is not being maintained.

Archiving the repository would also be great: https://github.com/blog/2460-archiving-repositories


.github/actions/env/package.json

Before submitting the PR, please make sure you do the following

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

This new workflow doesn't seem to add anything.


tips and tricks added about random theme using Z-shell

Summary of the Pull Request
Simply i added trick for random theme for Ubuntu distro in WSL terminal by using Z-shell and other things i provided in code .it changes theme every single time when you open another terminal window that's all.
## References

Z-shell

PR Checklist

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

I like the idea. But this does not seem to really fit in with Windows Terminal trick. It's more about zsh itself. Don't get me wrong. I love zsh and use it almost everyday. But you can zshell anywhere you want, right? Be it iTerm2, Gnome Terminal. Konsole, etc. You know what I mean.

I like the idea. But this does not seem to really fit in with Windows Terminal trick. It's more about zsh itself. Don't get me wrong. I love zsh and use it almost everyday. But you can zshell anywhere you want, right? Be it iTerm2, Gnome Terminal. Konsole, etc. You know what I mean.

Yup I understand but in readme Microsoft terminal member said add tips and tricks and this trick can help others that's why added what do you think ?

Hey @krishnadevz
Thanks for the contribution! However, I think I tend to agree with what @skyline75489 said. I think we intended that doc to contain tips & tricks specific to the Windows Terminal, not just general commandline tips and tricks. This is certainly a neat little tip, but I'm not sure it's really at home here.
Thanks!


When deleting a file, an error will be reported

[vite] Internal server error: /src/pages/admin/index.vue has no corresponding SFC entry in the cache. This is a @vitejs/plugin-vue internal error, please open an issue.
      at getDescriptor (/Users/morelearn/project/vite/vite-vue3/node_modules/@vitejs/plugin-vue/dist/index.js:4068:11)
      at Context.load (/Users/morelearn/project/vite/vite-vue3/node_modules/@vitejs/plugin-vue/dist/index.js:4639:28)
      at Object.load (/Users/morelearn/project/vite/vite-vue3/node_modules/vite/dist/node/chunks/dep-5c642f9e.js:43728:50)
      at processTicksAndRejections (internal/process/task_queues.js:93:5)
      at async transformRequest (/Users/morelearn/project/vite/vite-vue3/node_modules/vite/dist/node/chunks/dep-5c642f9e.js:59344:24)
      at async /Users/morelearn/project/vite/vite-vue3/node_modules/vite/dist/node/chunks/dep-5c642f9e.js:59538:32

Only after the vite.config.ts is resaved can it be reloaded normally

Can you try v0.10.0?

The error is still there

Because you delete file that is using by your browser, vite is serving file through middleware, so it's using that file, not like webpack is serving bundled files.
As this is not related to this repo, I will close this now. You can open issue in vite's repo.


fix: GetAtt functions may include "." and cdk functions do not

Line 245 in the template contains a Fn::GetAtt with 'Compliance.Type' and it is not handled correctly in any language.
The tests are being left out of this PR because not all issues have been resolved yet. The template is included for context.
Fixes #
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Is there any way we can remove the . in the IR instead? Or would that break something else?

I'm afraid this could have some issues with our ability to separate on that . in the future if we don't already. I feel like it may be safer to do this for now imo

I agree that we should be handing this in IR and it also makes me realize that there's a bunch of stuff we're handing in individual languages that should probably be done in IR instead. Anyway, fixed it here.


Translation broken? v4.5.2

Hi,
i just wanted to migrate to qBittorrent using your theme. I'm running qBittorrent on K8s & cloning the repo to /theme using /theme/webui as alternate webui path.
But this is how it looks like:

#18 fixes the theme colors, but not the translations

#18 fixes the theme colors, but not the translations

I cant understand. how to fix this ?

#18 fixes the theme colors, but not the translations

check #22
already been 2 months ago. still fix?

same problem here

Seeing this on v4.5.4

This post seems to address what's needed https://github.com/qbittorrent/qBittorrent/issues/19631
Seems like the translation files have to be compiled and bundled.


Image as Planes using PNG's when exported as .glb leads to a grey plane.

Describe the bug
I'm new to Blender and WebAR and am trying to import png into Blender 2.8 using image as planes, have changed Blend Mode to Alpha Blend. When i export it into a .glb all i get is a grey plane. Any help would be appreciated. Thank you :)

Hello,
Blender 2.80 Images as Plane creates a node tree that is not supported by glTF exporter. You can find the doc of what is supported for glTF here : https://docs.blender.org/manual/en/latest/addons/io_scene_gltf2.html
But Images as Plane has been updated recently to create a node tree that is compatible with glTF exporter.
You can find here alpha version of future 2.81 : https://builder.blender.org/download/
--> Your issue is solved in recent blender build


Experiment for with different approach to contact manipulation

Support endpoints /global-component-contacts/ and
/release-component-contacts/.

The output has the format that looks very usable to me. Unfortunately, it does not seem to work correctly for inheritance of release component contacts.
I tried with almost empty database, with a single global component and single release component. Define a contact for the global component –<EMAIL_ADDRESS>as Build_Owner. Then define a contact for the release component –<EMAIL_ADDRESS>as QE_Owner. Visit /release-component-contacts/ API. Only the contact from release component is included, while there should be both (one defined directly on release component, the other inherited from global component).
If I delete the contact from release component, the release component contact API will start showing the inherited one.
It looks like the contacts are inherited correctly only if the release component has no contacts on its own.

@lubomir Yes. It is what I understood the requirement. I thought if release component has any contact then use its contact information and don't care the global component's information. If rc don't have contact information at all, then use global component.
So the correct understanding should be if global component has any contact role information that rc don't have, this contact role information should be corresponding rcs' contact role info, right?

The inheritance should be the same as is now: release component takes all contacts from global component, and overwrites it with anything it has defined on its own. So your second paragraph is correct. I'm sorry if the original task description was unclear.

@lubomir Thanks. I tried to implement as you said in 14bf5c556f54fdd7. But I changed the model structure back. Because I found in original model structure, it looks more convenient in cloning release, changing contact at one place for all global components and release components as the task mentioned, etc.

I'm not sure how this is supposed to be used. The original proposal included identifiers for each component contact that were unique. In this version when I visit /global-component-contacts/, I get a page that contains contacts with identifiers 1 to 20, but for each id there are multiple contacts. How is the user supposed to pick one which they want to edit?
For release components it seems to work the same.
I tried to put some examples in PDC-912 to hopefully better illustrate the original intention. This is not a hard requirement if something better is found.

@lubomir Thanks for the review and update.
I think the output should not like you said " for each id there are multiple contacts.". Each id's contact should be exactly same. This id is role contact's ID, the contact and role should be same for one id. The difference is component (and component-id for release component). Fro release component, the component should be release component that is connected to this contact role directly and the release component which don't have this role's contact directly but its global component is connected to this role contact.
For the question to edit contact, if user change a role contact's contact. Then it take effect to all global component and release component that are connected to this role contact. For example, curl -H 'Content-Type: application/json' "http://<IP_ADDRESS>:8000/rest_api/v1/release-component-contacts/35/" -X PATCH -d '{"contact": {"mail_name": "IdM - Red Hat Directory Server"}}'.
The solution you suggested is each release component and each global component its own contact and role, even some of them have same contact and role, but they don't share contact and role as current. One record in db table is (contac_id, role id, release component/global component id), NOT (role contact id, release component/global component id) any more. In your example global-component-contacts/1234, the 1234 is id of record (role contact id, global component id).
When we change role or contact information, it will only affect this specific release/global component, not affect other components. Currently if you change contact or role in one role contact record, it will affect all related components. In your suggestion, if we want to replace contact at one place, we will change Contact model content directly rather than RoleContent. Is the understanding correct? Sorry to bother again. Thank you very much for the patience and help.

@lubomir I think again, maybe I understood above is incorrect. It looks difficult to show inherited information. If we separate global component contact in table with record item being (role id, contact id, gc id) and release component contact with record item (role id, contact id, rc id), It is not a property id to show inherited rc contact when list rc contacts. Because it is from gc contacts table. Is it possible to keep old model structure and manipulate role contact, and global components and release components are all connected to role contact?


Add to Homebrew

I think it would be super cool to install this via Homebrew. Don't you think?

If you don't mind, I'll close this issue since it seems to be stale.
If there are any new developments, please feel free to reopen.

:+1: Agreed.


Support Vault 2023

Fixed with commit https://github.com/coolOrangeLabs/powerflc-explorer-extension/commit/493f4535586a0b3a29993e4e3cc5b3243e4f5de0


implement bulk indexing

it might be nice to have bulk indexing.

please consider the pull request pull #7


[Feature Request]: ROS like modules to create any kind of agents both for software operations and hardware operations

Is your feature request related to a problem? Please describe.
I was exploring ROS for fun and found that it has many modules that we can use out-of-the-box to build things faster. We can build similar thing around AutoGen which allow us to pick, integrate and run.
Describe the solution you'd like
For software example, suppose I want to use RAG some where in my code so instead of implementing RAG from scratch we can have different modules that we can just apply and solve problem (RAG which just need to specify basic parameters to work).
This is just a thought excited to see what comes out ...
Additional context
No response

Thanks @mr-shitij . @randombet please create a roadmap issue for RAG when you're ready and include @mr-shitij in the loop.


Build error in C ++ projects

I am building a library for ESP32S2. The main project for ESP32S2 is written in C ++.
While compiling the library, I get the error:
ota_os_interface.h: 272: 22: error: expected unqualified-id before 'delete'
As I understand it, this is due to the fact that in C ++, delete is a keyword. Therefore, it cannot be used as a variable name.
https://github.com/aws/ota-for-aws-iot-embedded-sdk/blob/a1794596f0ba94e5f422cb064e7c60e8218f1485/source/include/ota_os_interface.h#L272
After changing the name, the assembly of the library was successful and the library does not cause problems.

I'm facing the same issue.

Thank you for reporting this issue, we are working on fixing this.

Closing this issue as the fix is merged. Thanks!


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106511

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106572

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106914

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/107065

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/118085


Fixing bytes-related TypeError upon report uploading when using python3

When trying to submit a coverage.xml file, under Python 3.5.0 and using a git-installed codacy-coverage, the following exception is raised:
Traceback (most recent call last):
  File "VENV_PATH/bin/python-codacy-coverage", line 9, in <module>
    load_entry_point('codacy-coverage==1.1.0', 'console_scripts', 'python-codacy-coverage')()
  File "VENV_PATH/lib/python3.5/site-packages/codacy/__init__.py", line 6, in main
    return reporter.run()
  File "VENV_PATH/lib/python3.5/site-packages/codacy/reporter.py", line 140, in run
    upload_report(report, CODACY_PROJECT_TOKEN, args.commit)
  File "VENV_PATH/lib/python3.5/site-packages/codacy/reporter.py", line 110, in upload_report
    message = json.loads(r.content)['success']
  File "/usr/lib64/python3.5/json/__init__.py", line 312, in loads
    s.__class__.__name__))
TypeError: the JSON object must be str, not 'bytes'

This commit fixes this issue by adding the six library as a dependency and implements a different behaviour if Python3 is being used.

Mh, it would seems that #7 already deals with this issue in a much smarter way (using Request's Response.text attribute)
Sorry for the noise, closing this issue.


basic drawing tools

changes in the readme file. Adding some instructions for the basic drawing tools

resolve issue #30


Check Script and Unit Testing

Are components present
Check Script: Fast Automated Test

tangential to pycubed.py updates issue

Do the components work
Sensor script: Fast Manual Test

IMU reads realistically
MAG reads realistically
Light sensors read realistically
Send PWM commands
Activate burn wire
Camera script: Fast Automated Test
Basic camera SPI communication: to mainboard and back
Radio script: Fast Automated Test
need to have 2 boards connected
Radio beacon send
Radio receive message

System check script merged


Worker kill not worked

Hi. I wrote simple test code:
		$task = new \application\Test();
		\Amp\Loop::repeat(100, function($reference) use($task){
			\Amp\Loop::disable($reference);
			$pool = \Amp\Parallel\Worker\pool();
			echo "Get worker";
			$worker = $pool->getWorker();
			try {
				echo 'enqueue task';
				$promise = $worker->enqueue($task);
				$canceller = new \Amp\TimeoutCancellationToken(1000);
				$id = $canceller->subscribe(function() use($worker){
					echo "Killing";
					$worker->kill();
				});
				echo "Waiting";
				$result = yield $promise;
				echo "done";
				$canceller->unsubscribe($id);
			} catch(Throwable $exception) {
				echo $exception->getMessage();
			}
			echo 'Result is:' . $result;
			if(!$worker->isRunning()) {
				echo "Worker not running";
			}
			\Amp\Loop::enable($reference);
		});
		\Amp\Loop::run();

and Test is:
class Test implements Task
{
	/**
	 * @inheritDoc
	 */
	public function run(Environment $environment)
	{
		sleep(30);
		return 'ok';
	}

}

I expect what worker will be killed after 1 second and "enqueue" promise will fail. But instead of that promise succeeds after 30 seconds.

Fix in 1.0.3 of amphp/process.


Update Dockerfile

Bump go version to 1.15.2
This bump is specifically to support bare importing of the time/tzinfo package into main per this issue: https://github.com/wtfutil/wtf/pull/997
Go versions prior to 1.15 would error out when attempting to import _ "time/tzinfo".

Thanks!! :+1:


How do I turn off tooltips on hovering over pagination items?

Hello!
I'm struggling with disabling tooltips on hovering over pagination items.
This can be seen in examples for pagination:
https://react-bootstrap-table.github.io/react-bootstrap-table2/storybook/index.html?selectedKind=Pagination&selectedStory=Standalone Pagination Total&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
When you hover over any item the tooltip appears showing either the number of the page, or the Title for Next/Previous/First/Last items.
Is there a way to disable it?
What options should I provide to make this happen?
I tried augmenting options with for example nextPageText: null with no luck
 const options = {
  custom: true,
  totalSize: products.length,
  nextPageText: null,
  prePageText: null,
};

@paroxyzm I will add a prop to allow developer to disable the title on the button, thanks!

@paroxyzm RELEASE NOTE
Please upgrade react-bootstrap-table2-paginator to 2.1.2 and add disablePageTitle as true in pagination config.
Thanks

Thank you!


CPU utilization and Memory usage by host missing on frontend pods

^^ Provide a general summary of the issue in the title above. ^^
Description
In the APM Summary dashboard, CPU utilization and Memory usage by host missing on frontend pods, though this data is present for the backend ones - see the screenshot:

Expected Behavior
Expected to see the charts for the frontend pods as well
[Troubleshooting]
Here's a suspected failure on Puma middleware tracing try from the NR agent debug log:
[2022-08-03 23:04:09 +0000 frontend-f45dc9c7f-l6wfn (1)] DEBUG : Creating Ruby Agent worker thread.
[2022-08-03 23:04:09 +0000 frontend-f45dc9c7f-l6wfn (1)] DEBUG : Creating New Relic thread: Worker Loop
[2022-08-03 23:04:09 +0000 frontend-f45dc9c7f-l6wfn (1)] DEBUG : Connecting Process to New Relic: /bundle/ruby/2.7.0/bin/puma
[2022-08-03 23:04:09 +0000 frontend-f45dc9c7f-l6wfn (1)] DEBUG : NameError : wrong constant name rails - when accessing config key agent_enabled
[2022-08-03 23:04:10 +0000 frontend-f45dc9c7f-l6wfn (1)] ERROR : Exception during Transaction#finish
[2022-08-03 23:04:10 +0000 frontend-f45dc9c7f-l6wfn (1)] ERROR : NoMethodError: undefined method `name' for nil:NilClass
[2022-08-03 23:04:10 +0000 frontend-f45dc9c7f-l6wfn (1)] DEBUG : Debugging backtrace:
/bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction/abstract_segment.rb:231:in `force_finish'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction/abstract_segment.rb:89:in `finalize'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction/tracing.rb:52:in `block in finalize_segments'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction/tracing.rb:52:in `each'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction/tracing.rb:52:in `finalize_segments'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction.rb:550:in `commit!'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/transaction.rb:529:in `finish'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/instrumentation/middleware_tracing.rb:112:in `call'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/configuration.rb:252:in `call'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/request.rb:77:in `block in handle_request'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/thread_pool.rb:340:in `with_force_shutdown'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/request.rb:76:in `handle_request'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/server.rb:441:in `process_client'
  /bundle/ruby/2.7.0/gems/puma-5.6.4/lib/puma/thread_pool.rb:147:in `block in spawn_thread'
  /bundle/ruby/2.7.0/gems/newrelic_rpm-8.9.0/lib/new_relic/agent/tracer.rb:414:in `block in thread_block_with_current_transaction'

Steps to Reproduce
This is from a running staging environment:
Account: 1521341
The https://app.development-0... service.
Your Environment
NR Ruby agent: 8.9.0
Ruby v. 2.7.6
Rails v <IP_ADDRESS>
For Maintainers Only or Hero Triaging this bug
Suggested Priority (P1,P2,P3,P4,P5):
Suggested T-Shirt size (S, M, L, XL, Unknown):

Tried to run with the fix of parent.name if parent - no success.
The log now says:
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] DEBUG : Creating Ruby Agent worker thread.
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] DEBUG : Creating New Relic thread: Worker Loop
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] DEBUG : Connecting Process to New Relic: /bundle/ruby/2.7.0/bin/puma
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] DEBUG : NameError : wrong constant name rails - when accessing config key agent_enabled
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] WARN : Segment: Controller/Rack/****::Application/call was unfinished at the end of transaction. Timing information for this segment'sparent  in Controller/health/index may be inaccurate.
[2022-08-04 11:55:22 +0000 frontend-75f56499fc-scqbc (1)] WARN : Segment: Ruby/Thread/153680 was unfinished at the end of transaction. Timing information for this segment's parent Middleware/Rack/ActionDispatch::Static/call in Controller/health/index may be inaccurate.

Resolved the issue setting the thread.tracing to false .

Thanks for letting us know about this issue. Having thread tracing enabled isn't expected to have that kind of impact on reporting, so it's possible there may be a bug in there somewhere. I have a couple of questions about your application to see if we can get a better idea of the issue. I'm going to re open this issue because I'd like us to investigate this behavior.

Are you using the infinite tracing version of the ruby agent?
You mentioned not having this problem on your backend servers, is thread tracing enabled there?
I see the parent of the thread segment in your log snippets is coming from ActionDispatch, how are you using threads in your application in general (and if there is a difference with the frontend vs backend, that would be great to know those differences)
Do you have any constants listed in the config option autostart.denylisted_constants?
Are you missing other unscoped metrics, or is it just those ones? If you aren't sure, one way to check this that should be pretty straightforward is to navigate to the metrics explorer and search for supportability/invoke_remote_serialize/analytic_event_data, select count in the dropdown next to the metric and see if it is present. This is an unscoped metric that I would expect to always be present for almost all applications. Here is a screenshot of me searching for that metric on a test app I fired up briefly to help explain what I mean.

Thank you!

Thanks for letting us know about this issue. Having thread tracing enabled isn't expected to have that kind of impact on reporting, so it's possible there may be a bug in there somewhere. I have a couple of questions about your application to see if we can get a better idea of the issue. I'm going to re open this issue because I'd like us to investigate this behavior.
* Are you using the infinite tracing version of the ruby agent?

* You mentioned not having this problem on your backend servers, is thread tracing enabled there?

* I see the parent of the thread segment in your log snippets is coming from ActionDispatch, how are you using threads in your application in general (and if there is a difference with the frontend vs backend, that would be great to know those differences)

* Do you have any constants listed in the config option `autostart.denylisted_constants`?

* Are you missing other unscoped metrics, or is it just those ones? If you aren't sure, one way to check this that should be pretty straightforward is to navigate to the metrics explorer and search for `supportability/invoke_remote_serialize/analytic_event_data`, select count in the dropdown next to the metric and see if it is present. This is an unscoped metric that I would expect to always be present for almost all applications. Here is a screenshot of me searching for that metric on a test app I fired up briefly to help explain what I mean.  ![Screenshot](https://camo.githubusercontent.com/b8c8dce0dc9952c6108f58853a6dda3513806f8b9c4bb75fabf17bc46ef8cf5c/68747470733a2f2f692e6962622e636f2f59447679514b502f6d6574726963736a70672e6a7067)

Thank you!

Hello!

I'm not using the infinite tracing version of the ruby agent
Yes, thread tracing was enabled at the backend pods as well
There is Puma on the frontend  pod, running with 1 process and 2 threads, and on the backend we have no web-server, only the DelayedJob with 32 workers
Yes, autostart.denylisted_constants: "rails::console"
Yes, the supportability/invoke_remote_serialize/analytic_event_data metric is displayed

Glad to be of some help!


internal/telemetry: report go version, git commit

This improves the version command and reports the go version in telemetry.
When provided no ldflags:
~/C/a/i/spicedb ❯❯❯ go build ./cmd/spicedb
~/C/a/i/spicedb ❯❯❯ ./spicedb version
spicedb 9679839c2d9c-dirty

This uses the new build info in Go 1.18, but degrades gracefully.

@jzelinskie image build is still failing?


wip: py bindings

@scsmithr I noticed that I ended up having to duplicate a lot of code from the CLI. This is probably fine for a POC, but it may be a good idea to extract this shared logic into it's own crate. That'd make it a lot easier to have additional libraries too (such as rust or JS).

@scsmithr I noticed that I ended up having to duplicate a lot of code from the CLI. This is probably fine for a POC, but it may be a good idea to extract this shared logic into it's own crate. That'd make it a lot easier to have additional libraries too (such as rust or JS).

Duplicating the code for now seems reasonable to me. I'd rather duplicate now and figure out what we can abstract out later than try to foresee everything upfront.

a few comments:

There's a couple todo's still in here for if you try to run a sql query with multiple statements
this is using just (a better make) as the command runner. I'm not super familiar with xtask so i just used this as it was the easiest for me to get up & running with. We can leave it, or convert it to xtask, no strong preferences on my end.
once we have pypi setup, I'll go ahead & set up the CI for this.  see #1186

a few comments:
* There's a couple `todo`'s still in here for if you try to run a sql query with multiple statements

Looks reasonable to me, we can follow up with more prs.

* this is using [just](https://github.com/casey/just) _(a better make)_ as the command runner. I'm not super familiar with xtask so i just used this as it was the easiest for me to get up & running with. We can leave it, or convert it to xtask, no strong preferences on my end.

I would prefer xtask, but we can leave this for now. I'm not familiar with python workflows yet, so we can keep just and use it for the time being.

* once we have pypi setup, I'll go ahead & set up the CI for this.  see [Chore: CI/CD for python bindings #1186](https://github.com/GlareDB/glaredb/issues/1186)

🚀


Remove EscapePath

Was a no-op and failed when input string was null (e.g. when no KeyFile is used).

Hm, seems that my Regex is a bit rusty, it's not a no-op.
However there is still the exception that gets raised when the input string is null. I'll create a separate PR for this.


Add eleventy-sanity-blog-boilerplate to starters

As per question on twitter I went ahead a made a starter for using 11ty with a headless CMS. The data-fetching/page-generation-part of it should be relatively easily adaptable for other API sources as well. It also demonstrates how to wrap 11ty in a monorepo.

Thank you!


wowza cloud

Hello
Im trying your lib and im having problems trying to send a rtsp stream to wowza cloud using a trial demo account. I get audio but no image.... see some issues related that say its fixed but... maybe is broken again?
Gracias!!

note that rtmp works, this only happens on rtsp

Fixed: https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/7c838102729d66c32edbcbfe4330bb06219af779
Let me know if all is working for you. If not report me logcat connection.

Thank you!!!!, sadly still not able to view, but have to say that i'm not sure is your fault or a wowza issue. Sadly i was not able to found another online service that supports rtsp to test
Video shows "static" on preview window but is not able to show on webplayer, and streams sometimes goes to paused.... extrange because in rtmp works without any issue
Logcat on error
2021-02-05 00:55:21.184 32114-32114/com.pedro.rtpstreamer E/DisplayService: RTP Display service create 2021-02-05 00:55:21.194 32114-32114/com.pedro.rtpstreamer E/DisplayService: RTP Display service started 2021-02-05 00:55:21.332 32114-25261/com.pedro.rtpstreamer E/C2AllocatorIon: >>>SEC mapUsage Mapper1 2000000 2021-02-05 00:55:21.332 32114-25261/com.pedro.rtpstreamer E/C2AllocatorIon: >>>SEC C2AllocatorIon::mapUsage ion_is_legacy++++++++ 2021-02-05 00:55:21.332 32114-25261/com.pedro.rtpstreamer E/C2AllocatorIon: >>>SEC mapUsage	0, 2000000 2021-02-05 00:55:22.897 32114-25270/com.pedro.rtpstreamer E/DisplayService: RTP service destroy
in verbose
2021-02-05 00:56:07.985 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 315 2021-02-05 00:56:08.032 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Video, size: 248 2021-02-05 00:56:08.032 32114-25276/com.pedro.rtpstreamer I/BaseSenderReport: wrote report: Audio, packets: 258, octet:<PHONE_NUMBER>-02-05 00:56:08.064 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 271 2021-02-05 00:56:08.064 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 260 2021-02-05 00:56:08.065 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 298 2021-02-05 00:56:08.144 32114-25261/com.pedro.rtpstreamer D/BufferPoolAccessor: bufferpool2 0x78d3c96e20 : 5(25600 size) total buffers - 3(15360 size) used buffers - 1198/1203 (recycle/alloc) - 5/2331 (fetch/transfer) 2021-02-05 00:56:08.145 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 261 2021-02-05 00:56:08.146 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 258 2021-02-05 00:56:08.225 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 265 2021-02-05 00:56:08.225 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 281 2021-02-05 00:56:08.226 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 265 2021-02-05 00:56:08.305 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 275 2021-02-05 00:56:08.306 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 296 2021-02-05 00:56:08.385 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 282 2021-02-05 00:56:08.385 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 266 2021-02-05 00:56:08.386 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 285 2021-02-05 00:56:08.466 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 264 2021-02-05 00:56:08.467 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Audio, size: 265 2021-02-05 00:56:08.542 32114-25276/com.pedro.rtpstreamer I/BaseRtpSocket: wrote packet: Video, size: 248

Can you try with default example to discard rtsp module problems?
I think that your problem is related with display mode but it is weird that rtmp is working

Sorry i did not mentioned but im testing with your sample app to discard code problem in my side.
Yep rtmp works ok

I mean using Default RTSP or OpenGL RTSP buttons instead of display rtmp or rtsp

yep sorry, i tried that, default RTPS and also custom to play with setings to see if found something but same behaviour all the time.
Video Thumb shows a image but webplayer no, and also static panel goes sometimes to paused >>> loading >>> normal
screeshot
https://ibb.co/NVXwwrV

I will check it.
Anyway if I open stream with vlc it is working for me.
You can use HLS url for it

thank you!

sorry, you also see the pause>>loading>>normal behaviour¿ or is just me

Yes, I see that sometimes but I can still reproduce stream in VLC without problem

ok!!!! yes i tested and able to see.... but stops :(... even on custom settings with low res and bitrate and im under a "conexion de fibra", rtmp works much better.
thank you for taking a look


Cannot find module 'angularfire2/interfaces'.

I`m trying to add offline capabilities to my ionic application using angularfire2-offline, but I get the folowing error when I run ionic serve?
Typescript Error
Cannot find module 'angularfire2/interfaces'.
node_modules/angularfire2-offline/database/database.d.ts
import { FirebaseListFactoryOpts, FirebaseObjectFactoryOpts } from 'angularfire2/interfaces';

node_modules/angularfire2-offline/database/interfaces.d.ts
import { FirebaseListFactoryOpts } from 'angularfire2/interfaces';
node_modules/angularfire2-offline/database/list/afo-list-observable.d.ts
node_modules/angularfire2-offline/database/list/emulate-query.d.ts

My packacge.json:
"angularfire2": "^4.0.0-rc.2",
"angularfire2-offline": "^4.2.4",
"firebase": "^4.3.0",
"ionic-angular": "3.6.0",
I've already tryed to remove dependencies and adding them agein, but the error persists
I appreciate your help

Hi @matheus-fatguys, if you set angularfire2 from ^4.0.0-rc.2 to 4.0.0-rc.0 it should work. 👍

Thank you! I've done that and now i works.
Thank you.


intellij-ultimate is starting to bug out!: ""[12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '+'""

i had 0 knowledge of this tool existing on intellij, but i just found about 6 thousand warnings  in my logs.(and i dump caches every couple days) below is a few examples. please let me know how to fix this problem asap, thanks
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '+'
2024-08-08 00:23:06,175 [12784750]   INFO - #c.i.i.p.m.MarketplaceRequests - Cannot load data from https://plugins.jetbrains.com/api/icon?pluginId=org.dochub.idea.arch (statusCode=403)
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '+'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,175 [12784750]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,176 [12784751]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'
2024-08-08 00:23:06,176 [12784751]   WARN - com.github.weisj.jsvg.cF - Identifier starting with unexpected char '/'

Hi, it isn’t obvious from the logs (as IntelliJ uses a obfuscated version of the library), but those are emitted by the CSS parser. I suspect that some plugin you have installed uses unsupported CSS features in one (or more) of their SVG files. Most likely sibling selectors and escaped special characters in id selectors. Though without knowing the SVG file causing the warning there is not much I can do here.

Closing this for now. Feel free to reopen if you have additional context.


Update 19.2 code samples to note crdb_internal.force_retry() does not require root anymore

See https://github.com/cockroachdb/cockroach/issues/39243
Estimated scope of work here is:

Update any 19.2 code samples that make use of this function, since some of the comments note the old behavior (they can't just be copypasta'd directly from 19.1 in at least 1-2 cases)
Update 19.1 code samples when and if  https://github.com/cockroachdb/cockroach/pull/39246 is backported in an upcoming 19.1.x point release
Verify that our 19.2 docs (and 19.1.x as applicable) do not make mention of this function needing to be the root user (I don't think this is mentioned outside code samples, but IDK)

FYI: The entry for force_retry in the functions doc doesn't mention privileges: https://www.cockroachlabs.com/docs/v19.2/functions-and-operators.html#system-info-functions.
All of those function descriptions are really unhelpful though. That's a separate issue.


Fixed #35428 -- Increased parallelism parameter for the ScryptPasswordHasher.

Trac ticket number
ticket-35428
Branch description
I modified the parallelism value of the ScriptPasswordHasher to the value recommended by OAWSP. For other options, increasing N=2^x would exceed the specified limited memory, so I did not want to increase the parameters of maxmem, so I only increased the parallelism value.
I think the best value is this option.
N=2^14 (16 MiB), r=8 (1024 bytes), p=5
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

The release note should be for 5.1, as this won't be backported, and should not be described as a bug fix, as this is a security hardening.

The release note should be for 5.1, as this won't be backported, and should not be described as a bug fix, as this is a security hardening.

Thank you for letting me know :)


chore(test): Unification of error messages for better end to end test…

…ing.

Pull Request Test Coverage Report for Build 86

23 of 24 (95.83%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.5%) to 95.313%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app.js
4
5
80.0%

Totals

Change from base Build 85:
-1.5%

Covered Lines:
93

Relevant Lines:
97

💛  - Coveralls


Am I doing something wrong: asmcrypto way slower than other libs with sha256

Hi.
I made a little benchmark to compare asmcrypto with two other popular node.js crypto libs and my tests show that it's way slower than them, which is unexpected considering that performance is the main point of your lib:
let asm = require('asmcrypto.js');

start = process.hrtime();

for (let i = 0; i < 100000; i++) {
    asm.SHA256
        .hex('abcdefghijklmnopqrstuvwxyz')
    ;
}

diff = process.hrtime(start);

console.log((diff[0] * 1e9 + diff[1]) / 1e6 + 'ms');

let shajs = require('sha.js');

start = process.hrtime();

for (let i = 0; i < 100000; i++) {
    shajs('sha256')
        .update('abcdefghijklmnopqrstuvwxyz')
        .digest('hex')
    ;
}

diff = process.hrtime(start);

console.log((diff[0] * 1e9 + diff[1]) / 1e6 + 'ms');

let crypto = require('crypto');

start = process.hrtime();

for (let i = 0; i < 100000; i++) {
    crypto
        .createHmac('sha256', '')
        .update('abcdefghijklmnopqrstuvwxyz')
        .digest('hex')
    ;
}

diff = process.hrtime(start);

console.log((diff[0] * 1e9 + diff[1]) / 1e6 + 'ms');

On my system:
asmcrypto: ~650ms
sha.js: ~370ms
crypto: ~300ms
Is there something I'm doing wrong here? I'm already thinkng that other crypto libs are extremely slow (for comparison, PHP hash function execute that test in ~70ms - yes, seventy!) and was hoping asmcrypto would solve that issue.

OS, Browser, Version? Did you check it with other browsers?

@jochenonline, no browser, server-side, nodejs 9.8.0.

Why don't you use the internal crypto functions?
https://nodejs.org/api/crypto.html#crypto_crypto_createhash_algorithm_options

@jochenonline, I do. But I was assuming your implementation would be faster.

It was... until 2014 when Crome and FF released WebCrypto API support.

@ericmorand : asmcrypto is asm.js / WebCrypto is binary. So on server side any binary implementation will beat asmcrypto.

Ooooh, sorry, I missed that part. Thanks for your clarification and patience.


redo(ticdc): limit memory usage in applier

What problem does this PR solve?

Issue Number: ref #8056
What is changed and how it works?

Limit total memory usage in reader.
Add memory quota in redo applier.

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`Fix the issue that redo apply may cause OOM`.

/run-all-tests

/run-all-tests

/run-all-tests

/merge

/merge

/run-all-tests

/merge

/run-integration-test

/run-verify


FOGL-6180.v2

We have integrated Unit Tests on CI and ran the Jenkins Job.
Below is the link for the Unit Tests Report -->
http://jenkins.dianomic.com:8080/job/fledge-iot-plugins-tester/job/fledge-south-dht11/view/change-requests/job/PR-20/1/
Note: unit tests were run against develop branch of the fledge repository.


experiments: Atomic load benchmark - atomic value vs unsafe pointer

I've benchmarked atomics which we will use in cluster.
The winner is unsafe pointer.
Benchmark result:
goos: darwin
goarch: amd64
pkg: scylla-go-driver/experiments
cpu: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
BenchmarkLoadAV
BenchmarkLoadAV-8   	691782937	         1.614 ns/op
BenchmarkLoadUP
BenchmarkLoadUP-8<PHONE_NUMBER>	         0.4589 ns/op
PASS

Cheers for the benchmark, this is the right approach.
It also shows that it costs 1 NS so there is also room for convenience :)

For convenience you can include github.com/uber-go/atomic in the benchmark

Benchmark with uber:
goos: darwin
goarch: amd64
pkg: scylla-go-driver/experiments
cpu: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
BenchmarkLoadAV
BenchmarkLoadAV-8       	864799339	         1.216 ns/op
BenchmarkLoadUP
BenchmarkLoadUP-8<PHONE_NUMBER>	         0.4040 ns/op
BenchmarkLoadUberAV
BenchmarkLoadUberAV-8   	946084138	         1.198 ns/op
BenchmarkLoadUberUP
BenchmarkLoadUberUP-8<PHONE_NUMBER>	         0.5388 ns/op
PASS
New benchmark has shown uber package is as fast as sync.atomic.
When we look into uber implementation we can see it wraps sync.atomic, so basically these things are the same.

Yes it is there no way around it the API is easier to work with.


xmtp_envelope.proto rename/split

This is the file rename/split I proposed in #14. The rationale is that envelope is really message_api concept, and that the file itself is getting long. The proposal is to split it into a message.proto with all the message/header types, and content.proto with all the message content types.
This should be transparent for go clients, but ts clients will need to adjust their imports since the xmtpEnvelope export is now split into message and content imports. That's a bit of friction for sure. Worth it?
If we go with this I'll follow up with xmpt-js update promptly

@dmccartney, @nakajima this change may be somewhat disruptive to the swift and dart dependencies. I will update xmtp-js, to change any xmtpEnvelope imports from @xmtp/proto to be either message or content as appropriate. Wondering if something like that will be needed for the swift or dart clients.


Gain

We currently have linear gain messages in 0.4.  Some of the new messages proposed for 0.5 call for gain in dB.  This is potentially confusing.
Maybe those messages should be /gaindB?

/gaindB makes sense in relation to ADM, since the gain element has an attribute "gainUnits" to set the units to dB. The default is linear.

/channel/1/gain -60
or
/channel/gain 1 -60
———
/channel/*/gain -60

Lawo: Normalized gain solves min & max; second message creates more opportunity for failure

Is there a standardisation for how the normalised gain maps to the gain in dB though? If a sender wants to set an object to a specific value in dB, or indeed adjust by an amount in dB, how would that sender know how to achieve this?
If the gain was in dB could it not be up to the receiver to set min/max thresholds?

issue with /gaindB is the [-INF] value to transmit as a float. Trade of could be to define a corresponding very low value like -120.0 dB

This works if your gain range is between -inf and 0dB. However, if an application wanted to apply positive gain, the linear representation would be above 1.0. Currently the ADM-OSC specification is expecting a value between 0.0 and 1.0.

Lawo's argument about min and max (above) is that /gain 0. represents the system's minimum gain and /gain 1. represents maximum gain.  So, if your gain slider goes from -80db to +12db, then that's what 0.0-1.0 will get.
ADM does define a gain of 1.0 as no change.  This is the default value for gain. Gain is just a multiplier, so to get +12db the gain would be 4.0.
Either way we have some complexity. If gain 1.0 is "top of the fader" then we don't really know what gain we're getting. But if gain 1.0 == 0 dB, then we don't know if higher gain is possible.  I would tend to stick with ADM here. There's no reason ADM-OSC can't support gain > 1.0, but a specific renderer might not support that. (L-ISA doesn't.)

@mzed - I think the solution is that the equations in @madees answer above should be the standard for converting between linear and logarithmic gain. The equations should probably be referenced directly in the ADM OSC specification.
In the specification the 'max' value shouldn't be 1.0 (as it currently is), but could be either higher or undefined (?). What is defined by those equations is that a value of 1.0 is 0dB (or unchanged) and 0.0 is -inf. It's up to the client to define the maximum value it accepts.

Yes, this makes sense to me.  ADM-OSC supports linear gain as positive floats. Gain dB can be derived with the above equations.
Maybe we need a /maxGain query?


Use correct hostnames in integration tests and verify in preflight

Fixes #424
We were using incorrect hostnames in some integration tests.
In KET v1.2.x we were setting the Calico hostname flag instead of relying on the OS reported one. Fo Calico upgrade to work correctly the planfile node name should be the correct hostname

LGTM!


⚠ ClamAV has degraded performance

In eb73a1a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 64 ms

Resolved: ClamAV performance has improved in 970f64a.


Minio local

@fpapon doesnt it miss the addition of the dependency in the documentation module to ensure the reactor is built in order and the doc is up to date?

@rmannibucau good catch, I will update

done


Add util::wif module

Extract all WIF code into a separate module.
Accompanies https://github.com/rust-bitcoin/rust-bitcoin/pull/221.

Closing, see IRC and https://github.com/rust-bitcoin/rust-bitcoin/pull/183


Animation.Messenger missing sends

I only see one Animation.Messenger.send only triggered once, instead of once per animation.
Steps

I have a list of 3 items, each with an Animation.Messenger.State
I call Animation.interrupt to start the animation on each item, with an Animation.Messenger.send to notify me when an animation is done.
I use Animation.Messenger.update for updates on each item.
I only see Animation.Messenger.send only triggered once, instead of once per item.

The use case is that I want to remove the item from a list when the animation is done.
I'm not sure if I'm using the library incorrectly, here is a gist of the problem:
https://gist.github.com/trotha01/0daa217c07bf05d859c8dcfe88229009
If you run the gist, we see that "start" is logged to the console 3 times, but "done" is only logged once.
Any help is appreciated, thank you!

Hello!
So, this one was sneaky: on line 112 switch == to /=.

Thank you for the quick response! Sorry for the bad code! Love the library 💖


[SPARK-37433][SQL] Uses TimeZone.getDefault when timeZoneId is None for ZoneAwareExpression

What changes were proposed in this pull request?

Calling timeZoneId.get on Option[String] leads to NoSuchElementException: None.get. This PR matches the value of Option[String] and uses TimeZone.getDefault.toZoneId when zoneId is None, this avoid unexpected exceptions to users.
Why are the changes needed?

Calling .get on Option variable never been a good idea. We can either use a default value or choose to throw a meaningful exception. In this case, TimeZone.getDefault will be a good fit for a default value.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Tested Locally and via Unit Test.

Can one of the admins verify this patch?


FRITZ!DECT 210 wrong temperature value

Can u provide more infos pls? Screenshots?

Please see below
homebridge-config-ui-x v4.41.2
Homebridge v1.3.4
Node.js v14.17.6
Npm v6.14.15
homebridge-fritz-platform v6.0.14
FRITZ!Box 7590, FRITZ!OS: 07.28
FRITZ!DECT 210 v04.17

Can u post your config.json pls

Hello,
homebridge-fritz-platform v6.0.15 does not solve the problem. Wrong temperature value is still displayed.
Regards!

Am 17.09.2021 um 07:15 schrieb seydx @.***>:
Can u post your config.json pls
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SeydX/homebridge-fritz-platform/issues/230#issuecomment-921504870, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATAQUEFMYNJW2XZDHG3IZELUCLFIXANCNFSM5EEPDAJQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@Norbert-Witowski
Can u post your config.json pls

Hello, please see attachment. Regards!

Am 19.09.2021 um 21:29 schrieb seydx @.***>:
@Norbert-Witowski https://github.com/Norbert-Witowski
Can u post your config.json pls
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/SeydX/homebridge-fritz-platform/issues/230#issuecomment-922524257, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATAQUEHLV5XDTIUU5PJCMJ3UCY227ANCNFSM5EEPDAJQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hello, please see attachment. Regards!

I can not see any attachment, lol

Hello, your support is still welcome. Gruß, Norbert Witowski!

Am 21.09.2021 um 09:31 schrieb @.***:

Am 21.09.2021 um 00:11 schrieb seydx @.***>:
Hello, please see attachment. Regards!
I can not see any attachment, lol
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


fix: don't use external buffers on platforms without support

Fixes https://github.com/duckdb/duckdb/issues/9172

Thanks for the PR! LGTM - is the change to the Makefile intentional?

Thanks for the PR! LGTM - is the change to the Makefile intentional?

Yep, it's intentional. Without it, running make debug will first build a copy slowly, then another in parallel

Thanks !


GitHub actions

In this case is it not just a windows package madness issue, i.e. if we use a windows action with chocolatey or whatever we should be good? The only thing is that I don't have a proper windows box atm

Sounds good, don't let me hold you up.

Sounds good, don't let me hold you up.

Ok