Question: Why is there an msb3073 error?

I installed it in the build Microsoft.Windows.CsWinRT An MSB3073 error occurred in the .Net5.0 project of nuget package:
命令“C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe @"projpath\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp"”已退出，代码为 -1073741511。

@Haceau-Zoac That seems to be a dependency missing error.  Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe.  Also what version of Windows are you running on?

@Haceau-Zoac That seems to be a dependency missing error.  Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe.  Also what version of Windows are you running on?

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

And my windows version is windows7 service pack 1

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

And my windows version is windows7 service pack 1

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

I remembered, I deleted the CsWinRT NuGet package, I went to install it.

@Haceau-Zoac That seems to be a dependency missing error. Do you get any error if you just run C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe. Also what version of Windows are you running on?

I found out i don't have C:\Users\Administrator.nuget folder!

I remembered, I deleted the CsWinRT NuGet package, I went to install it.

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that.  Note the \ after Administrator as I expect .nuget to be a folder under it.

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that.  Note the \ after Administrator as I expect .nuget to be a folder under it.

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that. Note the \ after Administrator as I expect .nuget to be a folder under it.

I will check.
Thanks!

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator.nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that. Note the \ after Administrator as I expect .nuget to be a folder under it.

I will check.
Thanks!

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator\ .nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that. Note the \ after Administrator as I expect .nuget to be a folder under it.

I entered the command .\cswinrt in the C:\Users\Administrator\.nuget\packages\microsoft.windows.cswinrt\1.1.0 folder using PowerShell 7-preview, and it has no output.
I entered the command .\cswinrt "F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", it has no output, too.
I also entered the command .\cswinrt @"F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", it outputs: ```text
ParserError:
Line |
1  |  .\cswinrt @"F:\DOTNETYES\Toolbox\obj\Debug\net5.0-windows\Generated F …
|              ~
| No characters are allowed after a here-string header but before the end of the line.
I entered `cswinrt` using `cmd.exe`, He popped up a dialog box titled `cswinrt.exe - 无法找到入口` with the content:
```text
无法定位程序输入点 CreateFile2 于动态链接库 KERNEL32.dll 上。

And I entered cswinrt @"F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", He popped up a dialog box, too.
I checked the CreateFile2 document (https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfile2), and its Minimum supported client is Windows 8 [desktop apps \| UWP apps], so should I use Windows 8 or later?

@Haceau-Zoac Also one other thing to check is if C:\Users\Administrator\ .nuget\packages\microsoft.windows.cswinrt\1.1.0\cswinrt.exe exists and if so try manually running that. Note the \ after Administrator as I expect .nuget to be a folder under it.

I entered the command .\cswinrt in the C:\Users\Administrator\.nuget\packages\microsoft.windows.cswinrt\1.1.0 folder using PowerShell 7-preview, and it has no output.
I entered the command .\cswinrt "F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", it has no output, too.
I also entered the command .\cswinrt @"F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", it outputs: ```text
ParserError:
Line |
1  |  .\cswinrt @"F:\DOTNETYES\Toolbox\obj\Debug\net5.0-windows\Generated F …
|              ~
| No characters are allowed after a here-string header but before the end of the line.
I entered `cswinrt` using `cmd.exe`, He popped up a dialog box titled `cswinrt.exe - 无法找到入口` with the content:
```text
无法定位程序输入点 CreateFile2 于动态链接库 KERNEL32.dll 上。

And I entered cswinrt @"F:\**projpath**\obj\Debug\net5.0-windows\Generated Files\cswinrt.rsp", He popped up a dialog box, too.
I checked the CreateFile2 document (https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfile2), and its Minimum supported client is Windows 8 [desktop apps \| UWP apps], so should I use Windows 8 or later?

@Haceau-Zoac It does look like we had an indirect dependency on CreateFile2 which is not available on Windows 7.  We typically test on Window 10, but I did a quick run on Windows 8 and the CsWinRT tool does seem to run on it.  So I would suggest trying out Windows 8 or later.  Given this is the issue, I am going to go ahead and close the issue, but feel free to comment here or open a new issue if you still run into issues with Windows 8 or later.

@Haceau-Zoac It does look like we had an indirect dependency on CreateFile2 which is not available on Windows 7.  We typically test on Window 10, but I did a quick run on Windows 8 and the CsWinRT tool does seem to run on it.  So I would suggest trying out Windows 8 or later.  Given this is the issue, I am going to go ahead and close the issue, but feel free to comment here or open a new issue if you still run into issues with Windows 8 or later.


Refactor timeout handling

Right now, connection timeouts are held in RedisChannelHandler and CommandExpiryWriter. Especially when setting the timeout, we need to propagate the timeout into CommandExpiryWriter to ensure that the connection timeout gets considered. By introducing CommandListenerWriter, we wrap CommandExpiryWriter and that requires further casting.
We should ideally store the timeout in a delegate/state object so that we can simplify the setTimeout code.
Worth considering: Timeouts in multi-connection arrangements (Cluster, Master/Replica) could benefit as well from the state object to share a single configured timeout. Individual connections use that timeout value until setTimeout gets called on the individual connection object.

Yes, I think it makes sense to refactor timeout propagation.
However, I don't know for sure where this state object should be held, because current places suit not so good for accessing and mutating timeout. Some kind of singleton object to hold it not so good either because it suites bad in case of multiple shards/sources.
@mp911de maybe you know the most appropriate place to keep such data?


(GSSOC'21) Modified Bubble sort with improved time complexity

I'm a GSSOC 2021 participant. I checked the C++ folder and noticed that this algorithm was missing. Please assign this issue. I'll write a clean, well explained, and concise code for the same.

I am a GSSOC'21 participant and want to work on issue.
I can provide the code in C++ with proper comment description .
please assign me this issue.

Hi I am GSSoC'21 participant
i would like to contribute in Modified Bubble sort with improved time complexity
i will write the clean and optimized code , by keeping best case time and space complexity in mind and
with  brief commenting and algorithm.
so i would like @bhav09  to please assign this Issue to me,
i will make this algorithm more adaptive :)

Hi I am GSSoC'21 participant
i would like to contribute in Modified Bubble sort with improved time complexity
i will write the clean and optimized code , by keeping best case time and space complexity in mind and
with brief commenting and algorithm.
so i would like @bhav09 to please assign this Issue to me,
i will make this algorithm more adaptive :)

@bhav09 sir i can use C++/JAVA :)

Greetings sir/Mam,
I would like to contribute the code for Bubble Sort Algorithm #933 in C++/CPP.
The code is ready. I have added the code , along with proper explanation of the algorithm , test cases , time complexity and space complexity of the same, as per the guidelines.
Kindly assign it to me.
This is regarding GSSOC'21.
Regards,
Hitain Kakkar

You can open a separate issue for the same, where you can propose to implement in the language you aspire to do.

You can open a separate issue for the same, where you can propose to implement in the language you aspire to do.

@bhav09 sir i have opened this issue please go through it ,
and if you found all ok then please assign this issue to me so that i can start contributing :)


Add more initialization methods

Hello. Currently, the default initialization for the linear layers is a random tensor with std=0.02.
I suggest adding more initialization methods such as Xavier (Glorot) and Kaiming (He).
http://proceedings.mlr.press/v9/glorot10a/glorot10a.pdf
https://arxiv.org/pdf/1502.01852.pdf
I have also previously experienced that initializing the cores so that the reconstructed matrix has expected Frobenius norm of 1 is sometimes helpful. Not sure if it's useful to be added though.

I guess with the new release (Thanks @JeanKossaifi 😄) there is no need for new initializations as PyTorch’s gaussian Xavier and Kaiming initializations will work by default.
Example:
import tltorch
import torch

tt_tensor = tltorch.TensorizedMatrix.new((4, 4),(3, 3), rank='same', factorization='TT')
torch.nn.init.kaiming_normal_(tt_tensor)
torch.nn.init.xavier_normal_(tt_tensor)

Also I think the init module is currently redundant—as it’s only a duplicate of the TensorizedMatrix.normal_ methods—and could be removed.


#158 more things ticked off the list, check the issue for progress

Current coverage is 47.05%

Merging #164 into master will not affect coverage as of 09a9417

@@            master    #164   diff @@
======================================
  Files           18      18
  Stmts          289     289
  Branches         0       0
  Methods          0       0
======================================
  Hit            136     136
  Partial          0       0
  Missed         153     153

Review entire Coverage Diff as of 09a9417

Powered by Codecov. Updated on successful CI builds.


InheritAnnotations from parent field

Table of Contents
Link to GitHub issues it solves
closes #51
Description
InheritAnnotations from parent field
Contributing has more information and tips for a great pull request.

@zeobot set lc to pass


How to use the library into Objective-C

I trying to use the library into Objective-C, there is a way to do it? Dou you have some guide?
Thanks.

I have the same problem, have you finded a way?

Thank you for your answer, I just found this outdated objective-c alternative and I'm trying it.

@ciccioska Perfect 👍

@ciccioska did it work?


add whitenoise to upgrade

Improve upgrade page see https://forum.image.sc/t/omero-web-upgrade-to-django-3-2-issue/68344/9

Is this temporary since it's an extra step for the user? Would adding this pin in omero-web suffice? (with a new release of omero-web)

whitenoise is currently not a dependency of omero-web. It requires an extra installation step e.g. https://omero.readthedocs.io/en/stable/sysadmins/unix/install-web/walkthrough/omeroweb-install-centos7-ice3.6.html
That's why it was also added to the upgrade page
Including it in omero-web will definitely simplify the installation.

Ah. Do you know the reasoning for only having pip install whitenoise in the docs and not in the repo itself?

By not in the repo, do you mean a dependency of omero-web?
I do not know the reason why it was not included in the first place.
This is certainly a candidate to introduce similar to what we do with "django-cors-headers recently

Closing this PR, whitenoise is now a dependency of omero-web. This included in 5.16.0


Gradle's configuration cache could not be enabled

If project has org.gradle.unsafe.configuration-cache=true then build fails:
1339 problems were found storing the configuration cache, 50 of which seem unique.
- Task `name` of type `com.likethesalad.tools.resource.locator.android.task.ResourceLocatorTask`: cannot serialize object of type 'com.android.build.gradle.internal.lint.AndroidLintAnalysisTask', a subtype of 'org.gradle.api.Task', as these are not supported with the configuration cache.
... and much more ...
  See https://docs.gradle.org/7.3.3/userguide/configuration_cache.html#config_cache:requirements:task_access

Hi @danaimset, I see. I just checked and it seems like this feature is incubating, as of today's latest Gradle version, so it can get major changes on next versions that might break whatever I might do right now to avoid this issue. So I prefer to wait until it's stable.
Also, to be honest the error message you've shared doesn't provide much info on what could be done to avoid it, it mentions that an object of type 'com.android.build.gradle.internal.lint.AndroidLintAnalysisTask' (which is not a Stem's class but instead comes from the Android Gradle plugin) cannot be serialized. Though there's two things that don't seem to be right, the first one is that there doesn't seem to be such a type defined inside the ResourceLocatorTask class, and the second one is that the class that cannot be serialized doesn't belong to Stem or its internal tools, but it actually comes from the Android Gradle plugin itself, so I'm left with no clues to get to the root cause of this. Probably the error message is not accurate enough due to the stage of this Gradle's feature, so for now what I an do is wait until it's stable.

It's been a while now and there's an 8+ version of AGP available, as well as better support in Stem for 8.0.0+ AGP api changes, I'm not sure if the AGP has sorted this issue yet, although this issue has been open for quite some time now and there's no action I can take to sort it from within Stem. So I'm closing this.


[docs-verifier] False positive broken link when heading contains xref

This run contains a false positive for the following link:
Error: In file 'docs/fundamentals/networking/telemetry/metrics.md': Invalid link: '../../../core/diagnostics/metrics-collection.md#create-a-custom-collection-tool-using-the-net--api'.
However, the link is correct and the build itself is clean.
This is the heading itself - notice that it contains an xref link.
## Create a custom collection tool using the .NET <xref:System.Diagnostics.Metrics.MeterListener> API

FYI @Youssef1313


simple fix for the security group description to make it more appropriate for the EB module

what
Change the description from: "Security Group for EKS cluster" to "Security Group for the EB environment", which seems more appropriate for the EB module.
why
Minor fix to make the description for the security group created more appropriate for the context of this module.
references
fixes #207

/test all

/test all

I tested this and if consumers are using the default security group description, this change will cause the security group to be replaced. The security group can't be replaced without setting security_group_create_before_destroy to true. But once this is done the security group can be replaced affecting the EB app: it will perform a regular environment update, replacing instances, etc.


App stalled when failing to connect to flora ble device

Hello,
This may relate to #9 .
As BLEClient::connect is a blocking call, "floraClient->connect(floraAddress)" never returns if the device can't be reached.
The application therefore goes in hibernate but doesn't operate.
To workaround this, one could scan the network when initializing the bluetooth and then ignore the devices that could not be found at scan time. What do you think ?

In general it would be better to scan all visible devices and then report data for all of them, instead of using fixed list of BTLE addresses.

I'll check the PR, thanks for the suggestion!


Support ordinal scales on line and stacked area charts

Hi! It would be nice if the x values are not limited to dates only.

Hi @ndabAP , thanks for the proposal.
This is a use case that seems to be common, although we haven't find it in our day to day work.
I know @biovisualize is working on something similar in a fork of Britecharts, and we are willing to port that over soon.
Some questions that come to my mind would be:

Should we keep the two scales inside the charts or should we extract them so it's another configuration of the chart?
In case we keep them inside, should we automatically detect the right scales?

I just discovered this chart library while I was looking for an easy to use, nice looking set of charts. I consider to use it for my project but the date limitation is the only thing that's blocking me.
Regarding scales: Since you can use most charts out of the box, automated scale detection would be something I expect.

Thanks for the feedback @ndabAP !
I am curious about your use case, what kind of data do you need to chart that has a line/stacked area for ordinal data? I guess I would try a bar chart, but maybe is not a good choice?

It's completely user driven. My users can compose charts and therefore there should be no limitation. But when I speak for myself, I'm a runner and I love to see my distance and duration visualized.
Thanks a lot for taking the time and the investigation.

Also for us, I need to compare hourly data week over week or month over month...  this is most easily done by stacking them on the line chart so you can see which week/line deviates from the rest.
Currently it is weird to set them all to the first week of the year and then try to hide the axis.  :(

Is there anything new about it? I would like to contribute in implementing this, but it would be good to know if there was already done some work on this.

Hi @SoerenHenning,
Nobody is working on this at the moment. We are in the middle of a general build system update and refactor into ES modules.
Please, feel free to investigate this! If you could write something about your approach here it would be ideal. We could discuss it before you write any code.
Thanks for your contribution!!


[ADD] Add Media Queries for Responsive Design

Add Media Queries for Responsive Design
Description
Enhance the responsiveness of the website by implementing media queries in a separate CSS file and linking it to the main application file.

Task Details

File to Create:

Create a new file named responsive.css in the Codebase/frontend/public directory.

Add Media Queries:

Use media queries to ensure the application layout adjusts gracefully for different screen sizes, including:

Mobile devices.
Tablets.
Desktops.

Define styles for breakpoints (e.g., max-width: 768px, max-width: 1024px) to optimize the design for various devices.

Link the CSS File:

Link the responsive.css file to the main application file App.jsx located at Codebase/frontend/src/App.jsx.

Testing:

Verify that the website is responsive across a variety of devices and screen sizes.
Test critical components and pages for proper layout adjustments.

Follow Pull Request Guidelines:

Please refer to the Pull_request_template.md file to create your Pull Request for this change.
Ensure code quality and test thoroughly before submitting the Pull Request.

I am interested in contributing to this issue.


Intellisense/import Code Actions don't work with React JSX Transform unless React is in scope

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

VS Code Version: 1.70.0
OS Version: Mac OS 12.4

Steps to Reproduce:
I created a reproducible demo using npx create-expo-app -t expo-template-blank-typescript. The only change I made was adding the basic React component and adding it to App.tsx without the import.

Clone the reproducible demo I created https://github.com/nhuesmann/vscode-react-jsx-transform-issue
cd <my-repo-you-cloned>
Install dependencies: npm i or yarn
code --disable-extensions .
Open App.tsx
Notice the error on line 8 with the <Test /> component
Click on the component and use cmd + . to bring up Code Actions
Observe VS Code's suggestion: import 'React' from "react"
Accept the suggestion
Once again use cmd + . (while clicked on <Test />) to bring up Code Actions
Observe VS Code's suggestion of Add import from "./Test"

The second suggestion should be suggested initially, since React's JSX Transform does not require React to be in scope. VS Code will not do JSX import suggestions unless React is in scope.
Related issue: #41762
Temporary workaround: https://github.com/microsoft/TypeScript/issues/41762#issuecomment-758535009

The expected behavior is only enabled if you set the jsx option to indicate that you're using a newer version of React that doesn't need the import, react-jsx

The expected behavior is only enabled if you set the jsx option to indicate that you're using a newer version of React that doesn't need the import, react-jsx

I initially had it set to react-native because I was extending Expo's config. I just changed it to react-jsx as you suggested. The custom React components I've made are now detected, however it will not detect React Native components still. Maybe we could benefit from a separate option, i.e. react-native-jsx? I'd rather not choose between having to have React in scope to get React/React Native autocompletion vs leaving React out of scope but not having any React Native import autocompletion.

Is this a "real" behavior, or a quirk of Expo's build system? The React Native docs do not show examples without the React import. It's quite explicit:

Because JSX is included in the React library, it won’t work if you don’t have import React from 'react' at the top of your file!

I just checked, Expo is using a slightly tweaked version of @babel/plugin-transform-react-jsx under the hood. Not sure, but it might have to do with Expo's build system. I feel like it'd be weird of them to mention supporting the JSX transform if it broke some feature of React Native + TS. Maybe I'll create an issue/question in Expo's repo. Do you have any suggestions otherwise?

@andrewbranch thoughts on making auto-import just not care about React being in scope? Seems like this might be causing more problems than solutions.

@RyanCavanaugh I saw that this was closed automatically. If you guys decide to make any changes to this behavior and reference this issue, will I be notified even if it's closed?

Yes

@nhuesmann I know I said 'yes' above but I'm not sure if the above PR actually triggered a notification, so I'll mention you just to be sure 😉


npm run github-deploy (dev and prod) fail

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Reporting a Bug

What is the current behavior?
Errors when using github-deploy

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via

Clone the repository
install NPM packages with npm install
run npm run github-deploy:prod
(if you run github-deploy without prod flag, you get tslint-loader errors that I think are red-herrings for the real problem)
Errors will occur while compiling main and polyfills.
The errors indicate that a module is not found at node_modules/typescript/lib/package.json

What is the expected behavior?
Github deploy should not error

What is the motivation / use case for changing the behavior?
it is a bug

Please tell us about your environment:

Angular version: 2.4.6
Browser: Not applicable
OS: Windows 10

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

ERROR in ./src/polyfills.browser.ts
Module build failed: Error: Cannot find module 'C:\_Dev\Github\dbosley-info\node_modules\typescript\lib\package.json'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at setupTs (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\instance.ts:162:29)
    at Object.ensureInstance (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\instance.ts:84:24)
    at compiler (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\index.ts:40:22)
    at Object.loader (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\index.ts:14:18)

ERROR in ./src/main.browser.ts
Module build failed: Error: Cannot find module 'C:\_Dev\Github\dbosley-info\node_modules\typescript\lib\package.json'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at setupTs (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\instance.ts:162:29)
    at Object.ensureInstance (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\instance.ts:84:24)
    at compiler (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\index.ts:40:22)
    at Object.loader (C:\_Dev\Github\dbosley-info\node_modules\awesome-typescript-loader\src\index.ts:14:18)

Not related to tslint
This is caused by the gh-pages packages that overrides (or one of the packages dependencies override) some functionality in the prototype of a core primitive (I think Array, not sure).
This causes an unexpected behaviour and awesome-typescript-loader get's unexpected results from an array function call.
The fix is simple, in webpack.github-deploy.js move:
const ghpages = require('gh-pages');

down just before the call that uses it
const ghpages = require('gh-pages');
ghpages.publish(webpackConfig.output.path, options, function(err) {
           if (err) {
             console.log('GitHub deployment done. STATUS: ERROR.');
             throw err;
           } else {
             console.log('GitHub deployment done. STATUS: SUCCESS.');
           }
         });

I'll post a PR shortly

@shlomiassaf wow, thank you for this simple solution! I'll give it a shot and report back with my results.
I hacked at this for a few hours the other day but was unable to figure out the problem.

Yes it's not an easy one to catch, need to debug webpack for that...


Added a new examples : vertical sliders and mouse-owning sliders

These gui elements provide examples on how to extend the library.
They can be dropped as-is into a project, and the "owning" variants of the sliders solve #255 without changing the library API.

@odecaux Thanks for the addition! I will review it more carefully asap!


Unable to mark a model property as Required in portable class library

Hi,
I have just configured Swashbuckle for the first time and everything works very well except for trying to specify "Required" properties. I know that you can do this with annotations but my models are stored in a portable class library meaning I cannot actually reference the required attribute. I cannot move the models into a standard class library because the consumers of the API are mobile devices and having a portable project enables easy export/import.
I have tried using a portable version of data annotations (https://github.com/ryanhorath/PortableDataAnnotations) to try and get around this but as I kind of expected this did not work (the properties marked by the portable required attribute still showed as optional in the Swagger documentation).
Is there any workaround I could use to mark properties of a portable model as required?
Thanks,
Luke

Hi, I'm having the same issue here. Any work arounds? thanks !

Your best option here would be to create a custom ISchemaFilter (see readme) that get's applied to every  complex Schema that's generated. In that filter, you can reflect over the type for properties that are annotated with the PortableDataAnnotation and then update the Schema's "required" property accordingly. Here's an example to get started with:
public class PortableAnnotationsSchemaFilter : ISchemaFilter
{
    public void Apply(Schema schema, SchemaRegistry schemaRegistry, Type type)
    {
        var publicProperties = type.GetProperties(BindingFlags.Instance | BindingFlags.Public);

        schema.required = publicProperties
            .Where(p => p.GetCustomAttribute<RequiredAttribute>() != null) // this would be the portable attribute version
            .Select(p => p.Name)
            .ToList();

        // NOTE: If you're serializing property names to camel case, you'll need to reflect that here
        // in the list of required properties
    }
}


Channels break if genres aren't selected

The following JSON breaks:
{"channels":[{"audioOnly":false,"genres":"SELECT Genres","logo":"logo.png","url":"stream.ts","name":"7 Brisbane","number":"1","service":"","splashscreen":""}],"modified":1478997754126,"possibleGenres":["ANIMAL_WILDLIFE","ANIMAL_WILDLIFE","ARTS","COMEDY","DRAMA","EDUCATION","ENTERTAINMENT","FAMILY_KIDS","GAMING","LIFE_STYLE","MOVIES","MUSIC","NEWS","PREMIER","SHOPPING","SPORTS","TECH_SCIENCE","TRAVEL"]}

Because the genres property is "SELECT Genres", which is the button text, but doesn't correspond to actual genres. A null or undefined case should be handled.
This should be incorporated into a test suite by loading things into a ChannelDatabase and trying to verify the data.

Now the app should handle empty genres before adding.


WIP: Convert all remaining consumers of Chef::REST to Chef:ServerAPI

:construction:
Also clobber the json auto expansion guff. Fixes #2737, fixes #3518
:construction:

cc @lamont-granquist

holy awesomeness...

I'm gonna do a follow up PR to add deprecations to Chef::RESTand any json_create methods, but I think this should be ready to review, @chef/client-core (travis failures are ipv6 and kitchen, which are known busted and being worked on)

it turned into a bit of a monster, happy to split up if need be…

@danielsdeleo addressed your requests

LGTM now :+1: Not sure what's going on with appveyor, we should fix that but it seems unrelated to this patch.

rebase to fix appveyor failure

ping @chef/client-core

okay one nit to pick other than that :+1:

:+1:


Merge cherrypicking samples view changes

Closes GPL769
Changes proposed in this pull request:

Integrate view access with GPL842

Convert to draft, as it happens we might need to recover the workflow for uwh and ewh, in order to use the new view.

Is related with https://github.com/sanger/unified_warehouse/pull/200

Tests:

[ ] Running the 'positives on site report' before deploying this to see if it alters the performance.
[ ] Deploy
[ ] Running the 'positives on site report' after deploying this to see if it alters the performance.

Time before deploying without the new view:

Time after deploying using the view:


Expose an unlimited refresh token to consumer call of MSAL

Library

[x]<EMAIL_ADDRESS>
Description
Hi,
I already talked about this issue with Juan Balmori Labra from MS Outlook team and Matt Geimer from MS SSO team and they suggested to post here.
I am dealing with the authentication for an Office JS add-in in Outlook.
The Office JS API provides a function to get a SSO assertion token for Microsoft Graph API, then the On-Behalf-Of flow provides a nested Graph token which contains a refresh token inside.
This refresh token lifetime is also unlimited, which is a key feature since it will be used by the server hosted by my company (out of Outlook process) to manipulate calendar appointments for its clients.
My problem is that I read that the AppSource certification process of the Office JS add-in requires a non-SSO fallback authentication.
So I thought about using msal-browser version 2 which implements the Authorization Code flow in a SPA, however there are two issues:

the refresh token is not exposed,
the refresh token has a limited lifetime of 24 hours.

I read in another issue that the refresh token was exposed in the previous version of authentication library ADAL.
Would it be possible that MSAL exposes also the refresh token to consumer call, and that it never expires?
Thank you,
Julien.
Source

[ ] Customer request

@Deymoun The refresh tokens in MSAL.js v2 are short-lived as a security consideration for browsers, and MSALs by design do not return refresh tokens to applications. Can you provide more details about why you need the refresh token?
You can see a sample Office Addin using MSAL.js v2 here: https://github.com/OfficeDev/PnP-OfficeAddins/tree/master/Samples/auth/Office-Add-in-Microsoft-Graph-React

Hi @jasonnutter my company would need the refresh token to access its clients' calendar anytime after a single user interaction.
The goal is to not have the clients reopen the Office taskpane every day to send again the Graph token, but rather they open only once the Office taskpane and send only once the refresh token so that my company can take care of the calendars in a job, months or years after this original setup.

@Deymoun With MSAL.js, if your app needs an access token, it should invoke acquireTokenSilent and MSAL will handle getting a new token using the refresh token (or from the cache). As noted, these refresh tokens are only good for 24 hours. Furthermore, refresh tokens acquired via MSAL.js v2 must be used client-side and cannot be used server-side. Given we do not have plans to remove these restrictions, it sounds like MSAL.js may not be the right library for your scenario, and you may instead want to look at MSAL.net, MSAL Node, etc. depending on your backend auth stack.

Hi @jasonnutter thanks for your suggestions, however both MSAL.NET and MSAL Node do not expose refresh token (proof for MSAL Node)
Would there be another solution than implementing the complex solution developed by another consumer of MSAL ?

@Deymoun The decision was made by all MSALs, not just javascript, to not expose the refresh token. As @jasonnutter described, when your application needs an access token it should call acquireTokenSilent which will fetch tokens from cache or attempt to renew using the refresh token. If you use a server side library such as msal.net or msal-node the refresh token lifetime will be much longer than 24 hours (weeks or even months) so your users will not need to reauthenticate frequently.
Closing as this feature will not be picked up but feel free to open a new issue if you have further questions. Thanks!

@tnorling @jasonnutter
Hi to the both of you. I'm pretty sure I know the answer to this per the documentation, but I wanted to get clarification:
MSALJS 2.0 (for the browser) doesn't respect the refresh token settings as specified in the User Flow Policy in Azure AD B2C, correct? Meaning that if you have your refresh token set to expire in 90 days, the library will regardless expire your session and require interaction after 48 hours (I believe there's a 24 hour grace period in which you could return to a site using this library and it would renew your session) right?


Godot 3.2.2b4 Produces PopupMenu Error

Version
v4.2.1 & MASTER
Description
Trying the latest Beta 4 release of 3.2.2 produces the following error on startup.
SCRIPT ERROR: _ready: Trying to assign value of type 'PopupMenu' to a variable of type 'MenuButton'.
At: :15
The TEST panel no longer functions correctly because of this error.
To reproduce: Take a working project with Godot 3.2.1 and open it with the new 3.2.2b4
Note
This might also be broken with other 3.2.2 releases, although I have not tested those.
Possible Fix
GUI.TSCN contains to variables of type MenuButton they are
Menu & ViewMenu
However, the type of the return value from get_popup() is of type PopupMenu
If you can confirm that MASTER branch is for 3.2.x I can make a patch, test it and make a pull request. Otherwise, I can do it on the 4.2.1 branch.

Resolved by 90c2b6c233bf70dd49d826b0ad29a297a06b35cd


fix for Bug NVFMSA-5854: added client support for action profiles/sel…

…ectors
Signed-off-by: Koushik Kopanati<EMAIL_ADDRESS>

Can you please add the sample o/p of group and member dump here?

for action profile:
[root@localhost ipdk.recipe]# sudo  ./install/bin/p4rt-ctl dump-entries br0
Table entries for bridge br0:
table=ingress.ipv4_host headers.ipv4.dst_addr=0x01010101 actions=action_profile_member_id(1)
table=ingress.ipv4_host headers.ipv4.dst_addr=0x01010102 actions=action_profile_member_id(2)
for action selectors:
root@wp28:/home/admin2/drop8/ipdk-recipe# sudo  ./install/bin/p4rt-ctl dump-entries br0
Table entries for bridge br0:
table=ingress.ipv4_host headers.ipv4.dst_addr=0x01010101 group_id(1)
table=ingress.ipv4_host headers.ipv4.dst_addr=0x02020202 group_id(1)
table=ingress.ipv4_host headers.ipv4.dst_addr=0xffffffff group_id(1)
table=ingress.ipv4_host headers.ipv4.dst_addr=0xe0000001 group_id(1)

Namrata asked to review for Tofino.
This should work for Tofino if it works for other targets. Ok to commit.

Namrata asked to review for Tofino. This should work for Tofino if it works for other targets. Ok to commit.

I have a question we are still using sizeGet() here, will it not cause issue with tofino
as you mentioned in https://github.com/ipdk-io/stratum-dev/pull/50

As a rule, the changelist comment should consist of a short heading followed by a more complete (and self-contained) description of the change.
The best commit heading provides a quick answer to the question, "what is the purpose of this commit?" Imagine that another developer has done a git log --oneline and is scanning the output to see what kind of changes have been made or whether a particular problem has been fixed. How would these stack up?
5652048 Fix build error when invoking grpc_cpp_plugin (#95)
e85698a Basic Pull Request gating (#94)
bdf819b Update krnlmon reference point and fix build issues (#93)
b162b35 Introduce daemon support for infrap4d for DPDK target (#89)
b5bb543 Set INSTALL_RPATH in libraries and executables (#86)
387b234 Provide extra control over dependencies setup (#87)
efb248b Address bandit warnings in p4rt-ctl (#90)
5ce8e5c Minor setup_env.sh maintenance (#88)
1167611 Add requirements text file for infrap4d (#91)
8ed693b Use find_package to locate grpc_cpp_plugin (#85)
6ef575b Integrate krnlmon into infrap4d instead of stratum (#84)
4d4c739 Update README file to reflect recent changes (#83)

The best commit comments give the reviewer the context they need to understand the changes and assess the correctness.
Google's Writing good CL descriptions document is an excellent guide to writing commit comments. I highly recommend it.
Also as a rule, you should limit references to resources that are not available to the reader.  NVFMSA-5854 is an internal Jira ticket number: useful for traceability at the project level (an argument for including it in the comment body) but too much arcane detail for the one-line summary.


Todo card?

Im not sure but wouldn't it be great wit a card that rendered checkboxes for the tasks, implementing the services Update, Complete and maybe even new Task?
I have no Idea of how to do, but something like the Shoppinglist is what I'm talking about.
//Lasse

Sorry, this isn’t question for the integration which is backend only. A card would be frontend coding which I don’t do and would not reside in this repository. I’m happy to provide a link to a card that someone created, or include an example using existing card types such as the Entities card.

Ahh that seems obvious when you explin it that way. Please all links are welcome :-)

The bottom of this page shows how to use Markdown to create a list from the output I create. Not sure if you can add checkboxes and service calls as well - https://github.com/black-roland/homeassistant-microsoft-todo

Hey, I created a custom card for the integration, https://github.com/fixtse/o365-card.
Currently, it only provides an easy way to present the data from the sensors in the dashboard, I hope it helps 😁


./command.sh -i fail

in ubuntu 16.04, run ./command.sh -i fail.
install docker-compose with sudo apt install docker-compose, docker-compose version is
docker-compose --version
docker-compose version 1.8.0, build unknown

fail log as followed
start xhost successfully..
start to init pulseaudio server
pulseaudio server build and start failed.. the reason is Version in "/home/eric/github/deepin-wine-docker/pulseaudio-server/docker-compose.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a version of "2" (or "2.0") and place your service definitions under the `services` key, or omit the `version` key and place your service definitions at the root of the file to use version 1.
For more on the Compose file format versions, see https://docs.docker.com/compose/compose-file/, and ..

Sorry！Due to personal reasons, I don't have time to deal with the problem. I will give you a solution during this time.


Add prometheus metrics

Draw inspiration from https://docs.dask.org/en/latest/setup/prometheus.html. Exposing gc metrics in particular seems a good idea.

I love this.

What about also exposing time spent processing requests?

Might also be use to count socket and file system reads.

I wonder if it’s appropriate to use Python audit hooks for that kind of performance tracing.

We needed to start capturing metrics urgently to make sense of some user reports, so I rushed something into main. Needs docs and refinement.

The socket and file usage can be observed through a generic node_exporter. I don’t think we need the Tiled application to do anything special there.
Still needs docs.

Old gist that is relevant: https://gist.github.com/danielballan/1bfb0576781c5671e3b97af2ebafcfa1

Evidently podman-compose does not support --scale. It seems to silently ignore it, as reported here: https://github.com/containers/podman-compose/issues/267


⚠ Bofrost.lu has degraded performance

In 8ccbac6, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1623 ms

Resolved: Bofrost.lu performance has improved in 7ba567b after 20 minutes.


[Bug] unable to install release 0.2.1 on windows

Search before asking

[X] I searched the issues and found no similar issues.

Component
Other
What happened + What you expected to happen
This issue is reported by a workshop user.
When trying to install the following
pip install data-prep-toolkit-transforms==0.2.1  data-prep-toolkit-transforms-ray==0.2.1
Getting the following error : https://gist.github.com/Alirezasedd/b9949fee563facb28aae34082517e151
Basically deepsearch-glm release not found
ERROR: Could not find a version that satisfies the requirement deepsearch-glm==0.21.0 (from data-prep-toolkit-transforms) (from versions: 0.1.0, 0.2.1, 0.2.2, 0.2.3, 0.3.0)
ERROR: No matching distribution found for deepsearch-glm==0.21.0

This is reported on windows (native install with anaconda).  I am able to install it successfully on Ubuntu 24.02.
Reproduction script
pip install data-prep-toolkit-transforms==0.2.1  data-prep-toolkit-transforms-ray==0.2.1
Anything else
No response
OS
Other
Python
3.11.x
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

You can use these steps to use data-prep-kit on windows.
dataprep toolkit transforms on windows
We can run data prep transform recipes on windows if we use wsl on windows.

Install wsl on windows

wsl --install Ubuntu-24.04

NOTE: Steps below can also work on linux (redhat/fedora or ubuntu).

Install miniconda for python

Execute these commands on wsl or linux shell.
wget https://repo.anaconda.com/miniconda/Miniconda3-py310_24.7.1-0-Linux-x86_64.sh
chmod a+x Miniconda3-py310_24.7.1-0-Linux-x86_64.sh
./chmod a+x Miniconda3-py310_24.7.1-0-Linux-x86_64.sh

To activate conda's base environment in your current shell session:

# Activate conda for current shell
eval "$(/home/shivdeep/miniconda3/bin/conda shell.bash hook)"

Create a python3.11 environment using miniconda

conda create -n data-prep-kit-1 -y python=3.11

# activate the new conda environment
conda activate data-prep-kit-1
# make sure env is swithced to data-prep-kit-1

## Check python version
python --version
# should say : 3.11

#Install jupyter lab
pip3 install jupyterlab

Add gcc/g++ using miniconda

# Install c/cpp toolchains
conda install gcc_linux-64
conda install gxx_linux-64

Now the set up should be ready.

Activate virtualenv

conda activate data-prep-kit-1

Then you can try your commands
pip install data-prep-toolkit-transforms==0.2.1  data-prep-toolkit-transforms-ray==0.2.1

@dolfim-ibm @shivdeep-singh-ibm
there was a confusion about this.  I was asking about native windows install.  Not WSL2 on windows.
I am hoping with the new docling release integration, we can install on windows natively (currently being tested)


Fix Tooltip vulnerable to XSS

You have to omit the last / when you create a project, but the engine will fill it in and display something like that:

The solution would be to just escape the project name for the tooltip.

Now it looks like this:


Signed releases for downstreams

Hi! I package this project for Arch Linux.
With the recent policy change, PyPi broke the reproducibility of downstreams: https://blog.pypi.org/posts/2023-05-23-removing-pgp/
As signatures are now no longer available on the platform, builds of projects from there are failing. This is a super sad state of affairs for downstream transparency and verification and we'll have to work through quite a lot of tickets such as this one: https://bugs.archlinux.org/task/79083
Do you plan on providing signatures for tarballs on github going forward? If so, doing a release would unblock me on that topic.
Thanks for you consideration!

FWIW, I will switch to signed git tags for the time being. Do note, that for some downstreams this may not be an option.

Hi! Thank you for letting me know about this change! It's indeed a concerning move on PyPi's part.

Do you plan on providing signatures for tarballs on github going forward?

Sure, I'm open to it. I will include this in the release process for the next release.

If so, doing a release would unblock me on that topic.
FWIW, I will switch to signed git tags for the time being. Do note, that for some downstreams this may not be an option.

I hadn't planned on doing a new release anytime soon, but from what I understood, this is no longer blocking you since the signed git tags worked for you.

https://github.com/geopy/geopy/releases/tag/2.4.0 has just been released with a signature attached to the GitHub release. This step is now also incorporated into the release checklist: 4fd273a


Revert Heapster version in deployment files to 1.5.4

What this PR does
Reverts change introduced in PR #2068. Heapster is during the release process, but it's not yet tested and therefore the image is not yet publicly available. This fixes reference deployments.

/lgtm
/approve


feat(cachedComputedFrom): decorator for computed properties with caching

cachedComputedFrom is just a caching-enabled version of computedFrom.
cachedComputedFrom implements temporary caching for repeated fast access to the computed property while it's dependencies remain unchanged.
I did some profiling and cachedComputedFrom was more performant where it needed to be, i.e. when the the computed property is the same if it's dependencies remain unchanged.

@jdanyow what are your thoughts on this PR?

Perhaps something like this should be the default behavior?

@EisenbergEffect Possibly.
Initially, I thought of augmenting the computeFrom decorator to have a toggle for caching but I don't think that's a good idea just because it's giving one thing too much to do.
I think there should definitely be 2 different behaviours, since it's not clear the sort of functions people use as their getters, they could well be impure and be expected to cause a myriad of side effects every time they are invoked. It's always good to have a choice

What about just making the default to not cache unless specified. (I would like it to be the other way, but as you stated people could currently be doing some additional logic in their getters)
@computedFrom("") -> not cached
@computedFrom("", true) -> cached

@jdanyow What if we aliased computedFrom to be computed and then we added the cached mechanism using a decorator called pure?

What if you want to specify a completely different object for the caching mechanism?
@computedFrom("firstName", "lastName")
@cached -> original implementation
@cached("cachingMechanism")
get fullName(){//returns concat};
Analyzes a different object to determine whether or not the value needs to be refreshed.
(maybe first name has changed but you don't want to propogate the changes until last name has also changed)

@EisenbergEffect hey, so whatever you guys decide is best I'd be more than happy to implement it. The intention of this pull request really was both to get feedback on my solution to a problem I faced while creating an aurelia app but also to get my foot in the door with regard to making a habit of contributing to aurelia. It's not clear to me what needs contribution but I hope this is the beginning.

Awesome! Thanks for being so willing to join in and help. @jdanyow Do you have any thoughts on this?

Definitely a solid idea.  I think this would make a nice plugin.  Couple thoughts/suggestions:

Using string.split to parse the expression is probably ok for 99% of the real-world scenarios.  computedFrom supports any expression that works in binding (eg foo[0].bar.baz)- not sure if that's something you wanted to support as well.
You could exchange the .every and .map for simple for-loops to avoid extra closures
I'm wondering if it would be cleaner/more-performant to put the caching logic in the ExpressionObserver instead.  There may be some logic still needed in the decorator to check for the presence of the observer when the property is being accessed and forward the call to the observer's getValue method (assuming it has caching).

Did this ever ended up being implemented anywhere (as a plugin or otherwise)?

@Truffula thanks for notifying about this. I didn't know we had a PR for this. The current state of this plugin does not guarantee it works in all scenarios, as dependencies are simply retrieved via index/keyed operator [].
There's a plugin here that gives you the cached computedFrom behavior:  https://github.com/bigopon/aurelia-deep-computed (I created this)


Fix Hyrax::VisibilityPropagator error for remote files

In UTK, we were getting a Hyrax::VisibilityPropagator::NullVisibilityPropogator error for remote files that was triggered by a misspelling.  Conveniently, this led to the question of why is it even hitting the NullVisibilityPropogator in the first place?  Turns out, the FileSet that comes in did not have a parent because of the logic in Bulkrax.
This PR swaps the logic so the FileSet should have a parent prior to hitting the Hyrax::VisibilityPropagator

As part of this test, did you confirm that remote filesets still work? @kirkkwang

@ShanaLMoore
I used this CSV:
remote_file.csv
And this was the result.  The visibility of private matches on both the Attachment and its FileSet.

Is that what you're referring to?

@kirkkwang Did this also resolve the log_profile issue?

@ShanaLMoore
I haven't seen that error pop up when i was doing my tests, so perhaps it did fix it?  Unsure though.

It should resolve it, because depositor was also nil because of no parent work going into the visibility job.


Unknown npm issue in "Plugins" page - v4.65.1

Describe The Bug
After upgrading the homebridge-config-ui-x to v4.65.1, when searching for a plugin on the "Plugins" page, an error showed up on the bottom right screen. There were no error descriptions within the pop-up. Downgrading to v4.65.0 seems to fix the issue.

Logs
[12/18/2024, 12:06:17 AM] [HB Supervisor] OS: Linux 6.8.0-1017-raspi arm64
[12/18/2024, 12:06:17 AM] [HB Supervisor] Node.js v22.12.0 /opt/homebridge/bin/node
[12/18/2024, 12:06:17 AM] [HB Supervisor] Homebridge Path: /var/lib/homebridge/node_modules/homebridge/bin/homebridge
[12/18/2024, 12:06:17 AM] [HB Supervisor] UI Path: /opt/homebridge/lib/node_modules/homebridge-config-ui-x/dist/bin/standalone.js
[12/18/2024, 12:06:21 AM] [Homebridge UI] Homebridge UI v4.65.1 is listening on :: port 8581
[12/18/2024, 12:06:22 AM] [HB Supervisor] Starting Homebridge with extra flags: -I -P /var/lib/homebridge/node_modules --strict-plugin-resolution
[12/18/2024, 12:06:22 AM] [HB Supervisor] Started Homebridge v1.8.5 with PID: 14771
[12/18/2024, 12:06:23 AM] Loaded config.json with 0 accessories and 1 platforms.
[12/18/2024, 12:06:23 AM] Loaded 0 cached accessories from cachedAccessories.
[12/18/2024, 12:06:23 AM] ---
[12/18/2024, 12:06:23 AM] No plugins found.
[12/18/2024, 12:06:23 AM] Loading 1 platforms...
Setup Payload:
X-HM://--------------
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:
┌────────────┐
│
└────────────┘

[12/18/2024, 12:06:23 AM] Homebridge v1.8.5 (HAP v0.12.3) (Homebridge 81BE) is running on port 51032.
[12/18/2024, 12:06:51 AM] [Homebridge UI] Failed to search the npm registry - "" - see https://homebridge.io/w/JJSz6 for help.
Config
No response
Homebridge UI Version
4.65.1
Homebridge Version
1.8.5
Node.js Version
22.12.0
Operating System
Ubuntu / Debian (or a variant)
Environment Info
Using hb-service
Raspberry Pi Model
Raspberry Pi 4 B

#2290 Should resolve this, plus update to v4.65.2

Yep it has been fixed, thank you!

Yep it has been fixed, thank you!


Test Python docstring in markdown files

And experiment with tested snippets

@kevinlacaille looks good to me!

This is really helpful, I'm absolutely going to refer to this in https://github.com/planetlabs/planet-client-python/pull/941 - thanks!


Fixed small typo in data types definition

Category

[x] Content fix

Related issues

fixes #7014

What's in this Pull Request?
Very small typo was raised in #7014
I've added the missing quotation mark.

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

File
Status
Preview URL
Details

docs/schema/field-element-list.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 804316d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/schema/field-element-list.md
:bulb:Suggestion
View
Details

docs/schema/field-element-list.md

Line 156, Column 365: [Suggestion-disallowed-html-tag] HTML tag 'FieldName' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.
Line 212, Column 257: [Suggestion-disallowed-html-tag] HTML tag 'ErrorText' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

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


docs: add documentation and migration instructions for new composability API

Closes #3809
Closes #1536, as it mostly documents the output of it.
Closes #418, since RouteRefs are the output of that, and now they're finally documented somewhere and actually work 😅
Right now this adds a big blob of docs in a single file to the plugin section, as I think it's too early to start rewriting existing documentation for how to create plugin and use them in the app. The first thing that needs to happen is to migrate all open source plugins, which should be able to happen quite quickly, and in a backwards compatible way. At that point it will be possible to migrate apps, and also to actually start moving documentation around.

@freben I left out details on how to build custom stuff on top the new APIs for now, just opting to mention it and let people ask if interested. Intending to add that in later, but I'm thinking it's better if we can get through migration of plugins and apps first.

@freben I left out details on how to build custom stuff on top the new APIs for now, just opting to mention it and let people ask if interested. Intending to add that in later, but I'm thinking it's better if we can get through migration of plugins and apps first.


Correct isCacheable logic

The current logic seems to be broken since 4e75e081f4502ca798097f3ba521e129e5798fc3
return !cc.isNoCache() || !cc.isNoStore();
got replaced with
return cc.forall(c -> c.isNoCache() || !c.isNoStore());
(flipped isNoCache boolean)

Add a test so it doesn't regress in the future? 👼

Added in #83b93ee77

Released


No print_summary for aaf?

Why is there is no print_summary function for AalenAdditiveFitter like there is for CoxPHFitter? Wouldn't it make the API of lifelines more consistent if they both had print_summary's?

Hey there, yes however I'm not sure what it would print out! The CoxPH summary is clear (scalar parameter's + their statistics), but the AAF, we don't have scalar parameters, but a time series of parameters.
What would you like to see in the print_summary?

Hey there, yes however I'm not sure what it would print out! The CoxPH summary is clear (scalar parameter's + their statistics), but the AAF, we don't have scalar parameters, but a time series of parameters.
What would you like to see in the print_summary?

I'm new to survival analysis so I don't really know what a print_summary of AAF should look like.
The R package timereg has documentation which states

summary.aalen Prints summary statistics
Description
Computes p-values for test of significance for nonparametric terms of model, p-values for test of
constant effects based on both supremum and integrated squared difference.
Returns parameter estimates and their standard errors.

Maybe that kind of summary would work for lifelines as well?

https://github.com/CamDavidsonPilon/lifelines/issues/234 related


Extract http status for version check via curls '--write-out' switch

Summary
Resolves https://github.com/radekg/terraform-provisioner-ansible/issues/159

Resolved with https://github.com/radekg/terraform-provisioner-ansible/pull/161.


Reduced spec for 0.1.

Hmm, any ideas, @wmdietlGC ? The internal CLA tool tells me that I can accept code from various @google.com email addresses, but it says "no" for yours :(


Update sbt-web-scalajs to 1.1.0

Updates com.vmunier:sbt-web-scalajs from 1.0.11 to 1.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.vmunier", artifactId = "sbt-web-scalajs" } ]

labels: sbt-plugin-update, semver-minor

superseded by #873


[google_sign_in] Execution failed for task ':google_sign_in_android:compileDebugJavaWithJavac'.

Steps to reproduce
Add google_sign_in as a dependency in pubspec.yaml.
Compile the sample with an Android target.
Expected results
Should compile and run.
Actual results
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':google_sign_in_android:compileDebugJavaWithJavac'.

Could not resolve all files for configuration ':google_sign_in_android:androidJdkImage'.
Failed to transform core-for-system-modules.jar to match attributes {artifactType=_internal_android_jdk_image, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
Execution failed for JdkImageTransform: /home/sir/Android/Sdk/platforms/android-34/core-for-system-modules.jar.
Error while executing process /home/sir/software/android-studio/jbr/bin/jlink with arguments {--module-path /home/sir/.gradle/caches/8.11/transforms/aedd227c5079b41feccfe72afa723f97-aac45aa3-e84a-4f4c-8c45-811070aa1ddf/transformed/output/temp/jmod --add-modules java.base --output /home/sir/.gradle/caches/8.11/transforms/aedd227c5079b41feccfe72afa723f97-aac45aa3-e84a-4f4c-8c45-811070aa1ddf/transformed/output/jdkImage --disable-plugin system-modules}

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

BUILD FAILED in 2m 43s
Error: Gradle task assembleDebug failed with exit code 1
Code sample
Code sample
// Copyright 2013 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

// ignore_for_file: avoid_print

import 'dart:async';
import 'dart:convert' show json;

import 'package:flutter/foundation.dart';
import 'package:flutter/material.dart';
import 'package:google_sign_in/google_sign_in.dart';
import 'package:http/http.dart' as http;

// import 'src/sign_in_button.dart';

/// The scopes required by this application.
// #docregion Initialize
const List<String> scopes = <String>[
  'email',
  'https://www.googleapis.com/auth/contacts.readonly',
];

GoogleSignIn _googleSignIn = GoogleSignIn(
  // Optional clientId
  clientId: .............,
  scopes: scopes,
);
// #enddocregion Initialize

void main() {
  runApp(
    const MaterialApp(
      title: 'Google Sign In',
      home: SignInDemo(),
    ),
  );
}

/// The SignInDemo app.
class SignInDemo extends StatefulWidget {
  ///
  const SignInDemo({super.key});

  @override
  State createState() => _SignInDemoState();
}

class _SignInDemoState extends State<SignInDemo> {
  GoogleSignInAccount? _currentUser;
  bool _isAuthorized = false; // has granted permissions?
  String _contactText = '';

  @override
  void initState() {
    super.initState();

    _googleSignIn.onCurrentUserChanged
        .listen((GoogleSignInAccount? account) async {
// #docregion CanAccessScopes
      // In mobile, being authenticated means being authorized...
      bool isAuthorized = account != null;
      // However, on web...
      if (kIsWeb && account != null) {
        isAuthorized = await _googleSignIn.canAccessScopes(scopes);
      }
// #enddocregion CanAccessScopes

      setState(() {
        _currentUser = account;
        _isAuthorized = isAuthorized;
      });

      // Now that we know that the user can access the required scopes, the app
      // can call the REST API.
      if (isAuthorized) {
        unawaited(_handleGetContact(account!));
      }
    });

    // In the web, _googleSignIn.signInSilently() triggers the One Tap UX.
    //
    // It is recommended by Google Identity Services to render both the One Tap UX
    // and the Google Sign In button together to "reduce friction and improve
    // sign-in rates" ([docs](https://developers.google.com/identity/gsi/web/guides/display-button#html)).
    _googleSignIn.signInSilently();
  }

  // Calls the People API REST endpoint for the signed-in user to retrieve information.
  Future<void> _handleGetContact(GoogleSignInAccount user) async {
    setState(() {
      _contactText = 'Loading contact info...';
    });
    final http.Response response = await http.get(
      Uri.parse('https://people.googleapis.com/v1/people/me/connections'
          '?requestMask.includeField=person.names'),
      headers: await user.authHeaders,
    );
    if (response.statusCode != 200) {
      setState(() {
        _contactText = 'People API gave a ${response.statusCode} '
            'response. Check logs for details.';
      });
      print('People API ${response.statusCode} response: ${response.body}');
      return;
    }
    final Map<String, dynamic> data =
    json.decode(response.body) as Map<String, dynamic>;
    final String? namedContact = _pickFirstNamedContact(data);
    setState(() {
      if (namedContact != null) {
        _contactText = 'I see you know $namedContact!';
      } else {
        _contactText = 'No contacts to display.';
      }
    });
  }

  String? _pickFirstNamedContact(Map<String, dynamic> data) {
    final List<dynamic>? connections = data['connections'] as List<dynamic>?;
    final Map<String, dynamic>? contact = connections?.firstWhere(
          (dynamic contact) => (contact as Map<Object?, dynamic>)['names'] != null,
      orElse: () => null,
    ) as Map<String, dynamic>?;
    if (contact != null) {
      final List<dynamic> names = contact['names'] as List<dynamic>;
      final Map<String, dynamic>? name = names.firstWhere(
            (dynamic name) =>
        (name as Map<Object?, dynamic>)['displayName'] != null,
        orElse: () => null,
      ) as Map<String, dynamic>?;
      if (name != null) {
        return name['displayName'] as String?;
      }
    }
    return null;
  }

  // This is the on-click handler for the Sign In button that is rendered by Flutter.
  //
  // On the web, the on-click handler of the Sign In button is owned by the JS
  // SDK, so this method can be considered mobile only.
  // #docregion SignIn
  Future<void> _handleSignIn() async {
    try {
      await _googleSignIn.signIn();
    } catch (error) {
      print(error);
    }
  }
  // #enddocregion SignIn

  // Prompts the user to authorize `scopes`.
  //
  // This action is **required** in platforms that don't perform Authentication
  // and Authorization at the same time (like the web).
  //
  // On the web, this must be called from an user interaction (button click).
  // #docregion RequestScopes
  Future<void> _handleAuthorizeScopes() async {
    final bool isAuthorized = await _googleSignIn.requestScopes(scopes);
    // #enddocregion RequestScopes
    setState(() {
      _isAuthorized = isAuthorized;
    });
    // #docregion RequestScopes
    if (isAuthorized) {
      unawaited(_handleGetContact(_currentUser!));
    }
    // #enddocregion RequestScopes
  }

  Future<void> _handleSignOut() => _googleSignIn.disconnect();

  Widget _buildBody() {
    final GoogleSignInAccount? user = _currentUser;
    if (user != null) {
      // The user is Authenticated
      return Column(
        mainAxisAlignment: MainAxisAlignment.spaceAround,
        children: <Widget>[
          ListTile(
            leading: GoogleUserCircleAvatar(
              identity: user,
            ),
            title: Text(user.displayName ?? ''),
            subtitle: Text(user.email),
          ),
          const Text('Signed in successfully.'),
          if (_isAuthorized) ...<Widget>[
            // The user has Authorized all required scopes
            Text(_contactText),
            ElevatedButton(
              child: const Text('REFRESH'),
              onPressed: () => _handleGetContact(user),
            ),
          ],
          if (!_isAuthorized) ...<Widget>[
            // The user has NOT Authorized all required scopes.
            // (Mobile users may never see this button!)
            const Text('Additional permissions needed to read your contacts.'),
            ElevatedButton(
              onPressed: _handleAuthorizeScopes,
              child: const Text('REQUEST PERMISSIONS'),
            ),
          ],
          ElevatedButton(
            onPressed: _handleSignOut,
            child: const Text('SIGN OUT'),
          ),
        ],
      );
    } else {
      // The user is NOT Authenticated
      return Column(
        mainAxisAlignment: MainAxisAlignment.spaceAround,
        children: <Widget>[
          const Text('You are not currently signed in.'),
          // This method is used to separate mobile from web code with conditional exports.
          // See: src/sign_in_button.dart
          textButton(
            onTap: _handleSignIn,
          ),
        ],
      );
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        appBar: AppBar(
          title: const Text('Google Sign In'),
        ),
        body: ConstrainedBox(
          constraints: const BoxConstraints.expand(),
          child: _buildBody(),
        ));
  }
}```

</details>

### Screenshots or Video

<details open>
<summary>Screenshots / Video demonstration</summary>

[Upload media here]

</details>

### Logs

<details open><summary>Logs</summary>

```console
[Paste your logs here]

Flutter Doctor output
Doctor output
/home/sir/software/flutter/bin/flutter doctor --verbose
[✓] Flutter (Channel master, 3.27.0-1.0.pre.25, on Ubuntu 22.04.3 LTS 6.8.0-48-generic, locale en_US.UTF-8)
    • Flutter version 3.27.0-1.0.pre.25 on channel master at /home/sir/software/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c667c97c3f (5 weeks ago), 2024-10-11 17:55:42 -0400
    • Engine revision b39fec6d97
    • Dart version 3.6.0 (build 3.6.0-334.2.beta)
    • DevTools version 2.40.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/sir/Android/Sdk
    • Platform android-35, build-tools 34.0.0
    • ANDROID_HOME = /home/sir/Android/Sdk
    • Java binary at: /home/sir/software/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.25.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2024.2)
    • Android Studio at /home/sir/software/android-studio
    • Flutter plugin version 82.1.3
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /home/sir/software/android-studio
    • Java version OpenJDK Runtime Environment (build 21.0.3+-12282718-b509.11)

[✓] VS Code (version 1.85.1)
    • VS Code at /usr/share/code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] VS Code (version 1.86.0-insider)
    • VS Code at /usr/share/code-insiders
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Linux (desktop)         • linux         • linux-x64      • Ubuntu 22.04.3 LTS 6.8.0-48-generic
    • Chrome (web)            • chrome        • web-javascript • Google Chrome 117.0.5938.149

[✓] Network resources
    • All expected network resources are available.

• No issues found!
Process finished with exit code 0

This has been fixed on the master channel but in the meantime there's a solution you can use in https://github.com/flutter/flutter/issues/156304#issuecomment-2397707812
Closing from here as this has already been fixed.
Thank you


Feature Request/Question: Lift the explicit type requirement in assertions for participation in CFA

Suggestion
🔍 Search Terms
explicit type, assertion, CFA
✅ Viability Checklist

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
As stated in #32695, functions using asserts require explicitly typed/annotated like so:
declare let x: unknown;
const aFoo = a.literal("foo");
aFoo(x); // error
// Assertions require every name in the call target to be declared with an explicit type annotation.(2775)
//  input.ts(2, 7): 'aFoo' needs an explicit type annotation.

const _aBar = a.literal("bar")
const aBar: typeof _aBar = _aBar;
aBar(x); // works
let test1: "bar" = x

declare let y: unknown;
a.string(y) // works
let test2: string = y;

namespace a {
  export function literal<T>(t: InferLiteral<T>){
    return function(v: unknown): asserts v is T {
      if (v !== t) throw new Error()
    }
  }

  export function string(v: unknown): asserts v is string {
    if (typeof v !== "string") throw new Error();
  }
}
type InferLiteral<T> =
  | (T extends string ? T : string)
  | (T extends number ? T : number)
  | (T extends boolean ? T : boolean)

The reason stated in the PR is "This particular rule exists so that control flow analysis of potential assertion calls doesn't circularly trigger further analysis."
My question is, umm, what does this mean in more layman terms? My impression was it's to not allow recursive usage but looks like that's not the case because this compiles... So not sure what the above statement means
  namespace a {
    export function literal<T>(t: InferLiteral<T>){
      return function(v: unknown): asserts v is T {
+        let _aLol = a.literal("lol")
+        let aLol: typeof _aLol = _aLol;
+        let x = {} as unknown;
+        aLol(x)
+        let test: "lol" = x;
        if (v !== t) throw new Error()
      }
    }

Also I understand it's a "design limitation" but, excuse my ignorance, is it really that hard to lift it? Because it's quite annoying to make make two variables for the same function then annotated the other, makes me think "Eh why can't the compiler do this for itself" haha. Maybe lift the restriction in some scenarios?
📃 Motivating Example
💻 Use Cases
I was writing a fail-fast parser that basically composes assertion functions something like this... But I have to use the mentioned workaround. Even if this is too much of a feature request, an explanation why the requirement exists would make me feel less annoyed when I redeclare and annotate assertions :P
Playground (Hit on run to see the ParseError with message "At Person.age: Expected a number")
namespace a {
  export const string: Asserter<string> =
    (v, p) => invariant(typeof v === "string", "Expected a string", v, p)

  export const number: Asserter<number> =
    (v, p) => invariant(typeof v === "number", "Expected a number", v, p)

  export const object =
    <O extends { [_ in string]: Asserter }>(tO: O):
      Asserter<{ [K in keyof O]: O[K] extends Asserter<infer T> ? T : never }> =>
        (v, p) => {
          invariant((v): v is object => typeof v === "object", "Expected an object", v, p);
          for (let k in tO) {
            (tO[k] as any)((v as any)[k], `${p}.${k}`)
          }
        }

  export class ParseError extends Error {
    constructor(message: string, public actual: unknown, public path: string) {
      super(`At ${path}: ${message}`)
    }
  }

  function invariant(test: boolean, message: string, actual: unknown, path: string): void
  function invariant<T, U extends T>(test: (v: T) => v is U, message: string, actual: T, path: string): asserts actual is U
  function invariant<T>(test: (v: T) => boolean, message: string, actual: T, path: string): void
  function invariant(test: boolean | ((v: unknown) => boolean), message: string, actual: unknown, path: string) {
    if (typeof test === "function" ? !test(actual) : !test)
      throw new ParseError(message, actual, path);
  }
}
type Asserter<T = unknown> = (v: unknown, path: string) => asserts v is T

const _aPerson = a.object({ name: a.string, age: a.number })
const aPerson: typeof _aPerson = _aPerson;

let person = { name: "Devansh", age: true } as unknown;
aPerson(person, "Person");
let test: string = person.name

This rule exists so that the feature could be added without drastically impacting performance.
Let's say you had some sequence of calls
let x = `e`;
fn1(x);
fn2(x);
fn3(x);
fn4(x);
let m = x; // <-

where all of fn1 .. fn4 had non-explicitly-declared types. If you wanted to query the type of m (independently, like you would in an editor), today there is only one step required -- compute the type of e. This can be safely done because the rule says that the calls to fn1 et al. cannot legally resolve to an assertion signature.
Without the rule, we would have to compute types for all fn1 et al., because anyone one of them could resolve to an assertion signature.
The worst-case scenario here could look something like
declare function mut1(x: unknown): asserts x is (n: unknown) => asserts n is (m: unknown) => m is string;
function nightmare(a: unknown, b: unknown, c: unknown, n: unknown) {
  mut1(a);
  a(b);
  b(c);
  c(n);
  const a = n;
}

where the computation of n, because it's an argument to c, requires a full evaluation of every inbound reference to an identifier used in its initialization. The only rule that "gets it right" without effectively computing the side effects of the entire function body for any arbitrary identifier reference is the one that says the target must be explicitly-annotated, because at that annotation boundary we can stop the control flow analysis.

Thanks a lot for the explanation! I'm convinced that it indeed is a hard problem to solve without impacting the performance, I did think this for a while and it's true that requiring an explicit annotation is the only solution that "gets it right". Although I think I have an improvement on that.
So if I understand it correctly I think when you said "today there is only one step required -- compute the type of e" you were simplifying it bit but this is what actually happens (assuming some things) -

User does a quickinfo query for type of x
TS starts looking for function calls that pass x as a (direct argument) in the lines above it
Finds fn4, checks if it's annotated (which is probably easy to check as I'd be in the ast or some quickly derived tree of that); nope it isn't, continue
Finds fn3 does the same thing; nope it's not annotated, continue
Finds fn2 does the same thing; okay it's annotated, resolve it's type to see if it's making an assertion; okay it is, grab the asserted type
Stop no need to go upward and show the asserted type (TS stops even if fn1 was annotated and makes an assertion but we don't care as whatever assertion fn2 would be making would be the final -- you see it's an "assertion" :P -- it might be asserting an impossibility because of what fn1 is asserting and we'd show red squiggles on fn2 (afaik) but that's an entirely different operation which we aren't concerned about when making a quickinfo query)

Okay we saved ourselves from resolving types of fn4 and fn3 which is great. So essentially, the explicit annotation requirement acts as some sort of a "first pass" as checking if a function is annotated is cheap.
But the problem here is it's too much of a requirement, if we want a marker for "this function might be making an assertion" one could introduce a syntax for it -- say as assertion
So basically this...
let aFoo = a.literal("foo") as assertion

would be equivalent to writing this...
let _aFoo = a.literal("foo")
let aFoo: typeof _aFoo = _aFoo

And now instead of "Assertions require every name in the call target to be declared with an explicit type annotation" it'd be "Assertions require call target to be annotated with 'as assertion' or every name in the call target to be declared with an explicit type annotation"
Afaict this would not add much cost, in fact checking if a function asserts via "as assertion" would be faster than checking if all call targets are explicitly annotated (not sure tho maybe both would be just flags on the node)
And if I were to make a critic: using explicit type annotation as a first pass for assertions is a bit cheating as both have no correlation as such, it took me a while to get that the ultimate goal is to have a first-pass that is cheap to check
I think this is a pretty new feature and people aren't leveraging it like I'm doing -- I already found myself redeclaring 6 assertions and the project isn't even complete yet :P -- ultimately it just makes the complier look a little stupid, the explicitness requirement for a first-pass is very much valid, I totally get it, but it would be nice if there was a syntax to aid it.
What are your thoughts?
(Imagine if I got this all wrong hahaha xD)

@RyanCavanaugh senpai wdyt? Also you haven't labelled it anything yet.

I definitely want this, but since it's apparently too hard to do it, could this at least be a request for a "quick fix" that gives you some idea what you should be doing?  https://github.com/microsoft/TypeScript/issues/34596#issuecomment-548084070


feat: migrate to nestjs 9

Breaking Changes:

Migrated to Morpheus module to NestJs 9
Used new Configurable Module Builder. This means that now we register modules using register/registerAsnyc instead of forRoot/forRootAsync
Removed deprecated env vars

NEO4J_SCHEME
NEO4J_HOST
NEO4J_PORT
NEO4J_USERNAME
NEO4J_PASSWORD

Other changes

Added e2e tests
Migrated CLI app to NestJs as well
Added cspell as a pre-hook
Updated readme to include the new register/registerAsnyc examples
Updated readme to explain the config options https://github.com/marianozunino/morpheus/issues/23
Added a minimal example of a NestJs project using Morpheus.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use jsdoc Boolean over boolean

JSDoc isn't case sensitive: https://github.com/jsdoc3/jsdoc/issues/1046, but it doesn't mean we shouldn't try to reduce variance.

We’re not really using jsdoc here, or else we’d be linting for it.


chore: prepare release

Motivation
Prepare the version numbers and overview of the changes for a release.

For history puposes: I pushed a commit to adapt effective time of release to 1100Z before merge.


Allow more granular control over the ApplicationMaxBufferSize

Today the ApplicationMaxBufferSize is a single setting per server, we should consider a callback model where this can be configured per connection. That would give more control over which connections get to buffer more incoming data than other connections.
We could also consider 2 limits:

Anonymous users and Authenticated users

The default limits are there to protect against unauthenticated users from blowing up server buffers but if users are authenticated the limits might be purely driven by your application. Also, since users are authenticated you have a way to audit and disallow misbehaving users.
Related to https://github.com/aspnet/SignalR/issues/2266

Seems reasonable. I wonder if we want to give full control of the PipeOptions to this callback

No I don’t want us to expose that type.


Input pattern?

Is it possible to use an user patterns file for TesseractEngine.Process?

Should be, though I haven't tried it myself.
On 3 Jul 2016 16:29, "invegat"<EMAIL_ADDRESS>wrote:

Is it possible to use an user patterns file for TesseractEngine.Process?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract-samples/issues/2, or mute the
thread
https://github.com/notifications/unsubscribe/AAPzyPapcYn2smv82ZxipGc6BfsCT2axks5qR8dSgaJpZM4JD4SW
.


int default support?

Hello, it looks like you only have support for default for strings. Is this correct? If not can you please document an example of how to use it? Thanks

Hi @yanfali, defaults for all types are supported.  Please see the TestEnvWithDefaultWhenMissing function in env_test.go for a complete list.


How to call node api and electron api?

How to call node api and electron api?

I haven't tried it. Probably by calling from WASM back out to javascript and from there into the node and electron APIs. If you figure it out feel free to send a PR with a small example.

I get this in wasm-bindgen repo example:
use wasm_bindgen::prelude::*;

#[wasm_bindgen(module = "/defined-in-js.js")]
extern "C" {
    fn name() -> String;

    type MyClass;

    #[wasm_bindgen(constructor)]
    fn new() -> MyClass;

    #[wasm_bindgen(method, getter)]
    fn number(this: &MyClass) -> u32;
    #[wasm_bindgen(method, setter)]
    fn set_number(this: &MyClass, number: u32) -> MyClass;
    #[wasm_bindgen(method)]
    fn render(this: &MyClass) -> String;
}

// lifted from the `console_log` example
#[wasm_bindgen]
extern "C" {
    #[wasm_bindgen(js_namespace = console)]
    fn log(s: &str);
}

#[wasm_bindgen(start)]
pub fn run() {
    log(&format!("Hello from {}!", name())); // should output "Hello from Rust!"

    let x = MyClass::new();
    assert_eq!(x.number(), 42);
    x.set_number(10);
    log(&x.render());
}

Mybe need to define all node and electron rust api. I can't do it.

You can try the "electron_sys" crate.

You can try the "electron_sys" crate.

thanks


Prune repo of unused things

Such as two of the image_const nodes (we only need one, and it should arguably live in tools).
Or even the lane_detector, which we are scraping for the future road_detector node aka agencynet

Ehh, I agree with the first part of this (redundant stuff), but the whole point of buzzmobile was to potentially have various ways of doing things, and use it as a testbed for stuff.

I'd argue those can happen in branches but master should be cleaner

Depends, there's value in being able to test multiple things at once,
though I guess if you're smart enough with git you can merge non-branches
into each other to get that effect. Worth weighing in mind in any case.
On Wed, Oct 19, 2016 at 12:44 AM Raphael Gontijo Lopes <
<EMAIL_ADDRESS>wrote:

I'd argue those can happen in branches but master should be cleaner
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gtagency/buzzmobile/issues/39#issuecomment-254709147,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEz4AFessdkNetlfTpRwU0Y1NNmfeXzMks5q1aA1gaJpZM4KahZ3
.


gh-2471 Fix hotfix release merging issue

Related Issue

Resolve #2471

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@e0efd74). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #2472   +/-   ##
==========================================
  Coverage           ?   75.85%
  Complexity         ?      185
==========================================
  Files              ?       11
  Lines              ?      584
  Branches           ?       50
==========================================
  Hits               ?      443
  Misses             ?      100
  Partials           ?       41

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 e0efd74...be6b0dd. Read the comment docs.


Created several initial docs to get us started

Please provide feedback, and I'm happy to make updates!

Thanks for your efforts @geekygirldawn ! I just added some comments on top of @juliusv, nothing too serious


az devops login fails miserably

I am using az 2.0.69 but this fails just as miserably with 2.0.70.
See attachment
ttt.txt

I suspect I am running into this issue
https://github.com/99designs/aws-vault/issues/304
would be nice if someone came up with a fix for this, to make azure-cli a bit more lovable.

I suspect I am running into this issue
https://github.com/99designs/aws-vault/issues/304
would be nice if someone came up with a fix for this, to make azure-cli a bit more lovable.

fwiw, I was running a vnc session with twm when I was doing this, and following the xinitrc method to set up gnome-keyring-daemon per https://wiki.archlinux.org/index.php/GNOME/Keyring#xinitrc_method helped solve the issue in my case (I was able to do az devops login from my vnc)


Array with red text uneditable, but shows it's value on JSON export, which renders the SOL invalid when importing the edited JSON

headHave array shows up as red text. Trying to edit the 15th or 16 array and importing it back will cause the resulting .sol file to appear corrupted to the application.
S7_SHARED.sol.zip

I took a look at this in Firefox 115 and not sure i can replicate the issue. I opened the file and went to the headHave array. I didn't see anything in red text.

You said to try editing the 15th or 16th array, i assume you mean integer as in the children within the array? I edited the 15th item from a 0 to a 1 in the array and saved it.

I also re-saved the file prior to editing it and compared it to the original. Without editing, the file saved exactly the same as the original.

With the edit, the only difference was one byte (the value i changed).

At this point I don't see any reason why the application would think it's correct. If possible, please give me more details on how to replicate the issue. Step by step exactly would help. Thanks.

Chrome 120 with guest profile so it's a brand new browser profile every time it's launched.
Steps:

Use "Open" to open S7_Shared.sol
search for "headhave"

Expected:
I can expand the array and show its content
Actual:
Red text on array headHave and clicking the triangle shows nothing. secHead (I think that is called?) also behaves the same

Regarding the app data corruption, I strongly think that the app itself has a bug preventing that only array element to be set as 1 and not detect some sort of corruption. Editing other elements in the array does work and the app won't detect the edited SOL as corrupted.


Documentation may become more intuitive if setup steps are provided for each platform + spatial alignment combination

Users experienced difficulties understanding what they would need getting started. There was a cognitive disconnect when initially viewing the dependencies list for the various spatial alignment strategies. It was suggested to have different step by step instructions for each spatial alignment strategy.

Try and assess this at the same time that we look into basic setup steps improvements.

we've had a lot of documentation changes. Lets reopen this as needed based on user feedback


Time is always 0 in DASM and perf results

I am currently following the tutorial for snitch cluster at https://pulp-platform.github.io/snitch_cluster/ug/tutorial.html and reached the debugging/benchmarking step. However, when trying to analyze the performance, it seems that the tstart and tend metrics are always 0. This value persists from the DASM file to the text, csv, and json results from the other steps for benchmarking.

The command I ran: bin/snitch_cluster.vlt sw/apps/blas/axpy/build/axpy.elf
Resulting files is attached.
trace_hart_00000000.dasm.txt
hart_00000000_perf.json
perf.csv
event.csv
Running on docker on Linux, amd64:
$ uname -a
Linux b0f761a4bb94 6.5.0-1004-oem #4-Ubuntu SMP PREEMPT_DYNAMIC Fri Sep 15 19:52:30 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Tested running on

axpy in tutorial
blas/axpy from repo

Things I tried changing:

Adding back the patch from https://github.com/pulp-platform/snitch/blob/d026f47843f0ea6c269244c4e6851e0e09141ec3/util/patches/riscv-isa-sim/fesrv.patch

Thank you for opening the issue.
This is a known problem with Verilator simulation as it does not log simulation timestamps.
The Python scripts could be extended to use the cycle count instead of the absolute timestamp as a workaround.
If you could provide such a feature, I would be willing to merge it.

Do you know if there are any known simulators that are able to provide wall time statistics? I saw that Banshee was another tool mentioned but that seemed to lean towards instruction accuracy instead of cycle accuracy. On IIS there seemed to be other tools like Questa too.

How can one get this license? Is it offered within the Docker installation?


PARQUET-2250:[C++][Parquet] Expose column descriptor through RecordReader

This change exposes the column descriptor through the RecordReader.

@emkornfield could you take a look? Thanks!

@wgtmac, @emkornfield would you mind merging this? Thanks!

Rebased, @wgtmac, @emkornfield would you mind merging this? Thanks!


Basic PDF layout issues

Setting aside the edge-case bugs we're tracking in #15 , we have a few hopefully more straightforward issues with the size and layout of things in the PDF:

Header boxes overflow

CBPS date boxes too big and crowded

┆Issue is synchronized with this Asana task

very weird that some buildings print three links in the footer:

and some only two (which matches the website afaik):

Unless there is some building-specific switch here that I missed until now?

okay, with media queries, I can get the 3 link case buttons to not overflow:

Done in 53c65bd but not tagged in a commit

Closing for now. Anything more specific should get a new issue


Link error in the README

Describe the issue:
Link error in the README of Codespace and Dev Container.

Suggestion:
Change links to :
Codespace : https://codespaces.new/pinecone-io/pinecone-rag-demo-azd
Dev Container : https://vscode.dev/redirect?url=vscode://ms-vscode-remote.remote-containers/cloneInVolume?url=https://github.com/pinecone-io/pinecone-rag-demo-azd
@hemarina and @cwaddingham for notification.

Fixed.


Add basic documentation to the nsinit directory to spawn containers

Problem
nsinit lacks basic getting started documentation.
Solution
Add basic documentation to the nsinit directory that explains how to get a simple busybox container up and running and the available options that one can pass.

#dibs We're writing a PR to improve the doc.

@resouer thanks!!!

@crosbymichael I have a problem to understand how the nsinit  read the configration file.
I didn't find any code in libcontainer or nsinit that have loaded the "container.json" file.
The whole libcontainer project is searched and only find "container.json" was reminded in README and Dockerfile.
So I hope you can do me a favor to tell where do the nsinit load the "container.json" into "cli context".
Thank you very much.

@wonderflow the config file should be passed through --config, nsinit exec will parse and load it,  your command from README.md, container.json won't be available, nsinit would get config from function getTemplate().
https://github.com/docker/libcontainer/pull/493 is fixing the doc.

@hqhq Thanks for your explanation.

@crosbymichael
Another question is what is "nsinit init" used for?
I've also ask the question here.
Hope to get some help, Thanks!


Update functions/Invoke-DbaDiagnosticQuery.ps1

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[X] Documentation
[ ] Build system

Purpose

Update example to reflect current parameters. The value for -SqlInstance edited.

😏


Add TIFF Export filetype

Improves export times by a significant amount compared to PNG.
Build: https://github.com/illusion0001/trdrop/releases/download/d97e8e7/trdrop-release-d97e8e7.7z

Partially resolves #105

Partially resolves #105


Add new assertions for fill order validations and using new FillOrder…

… type from bumped utils
This depends on this being merged in first so that the updated utils can be installed: https://github.com/SetProtocol/set-protocol-utils/pull/37/files
Solves:
https://trello.com/c/tEql7IVp
https://trello.com/c/I8HsBAbv

Tests still failing

Failing cuz the assertions were wrong lol