How to tag datasets to create a collection of datasets?

The Living Norway team would like to have 'data collection pages' in our hosted portal. A data collection page is a collection of datasets of certain topics, i.e 'Education', 'Nature index'
In order to get the datasets that belong to a collection we need to tag the datasets in some way so we can query the gbif api to get the datasets for a given collection in the Living Norway network. Do you have any suggestions on how to do that?
It would also be nice to list all the collection tags, to get the data collections that exists in the the Living Norway network. Is there a way to achive this?

I'm not entirely sure I understand, but there is currently no dataset search. Once available it will be based on the existing APIs which are documented here
https://www.gbif.org/developer/registry#datasetSearch
Topics as you mention sound like it could be the same as keywords in the dataset search API
Unfortunately there is no search support for datasets grouped by networks.
https://github.com/gbif/registry/issues/338
So for networks we have a few options:

implement https://github.com/gbif/registry/issues/338
have a client/middleware/custom solution for filtering (double the code to maintain and slower if many entries). For this I see following options:
2.1. Pull down all network datasets to the browser and populate a data table of a sort for client side filtering.
2.2. Use a proxy dataset search that consults the network and then sets a root scope of x datasets (could be thousands in VertNets case), then the normal API could be used, but would probably net perform well.
2.3. Create a graphql endpoint that do server side filtering instead of using the proper API. This would spare the frontend from extra code, but would be a half-baked solution that only allowed for a subset of filters.
you write your own solution based on APIs
Manually keeping a simple markdown table up to date (no search beyond ctrl+f)

I would far prefer option 1, but that is easy for me to say as I wouldn't be the one implementing it.

Thank you @MortenHofft for your reply and the different approaches to this. One question - could it be a good idea to rely on machine tags for this? And query the api for datasets with a certain machine tag? We must then manually maintain a list of the machine tags we are using for our collections.

It could, but there are 2 dataset search APIs. One based on postgresql v1/dataset and one based on ElasticSearch /v1/dataset/search. The former supports machineTags the latter do not. But the latter supports facets and free text queries. Even if we extended the dataset/search endpoint with machineTag search functionality, then you still wouldn't be able to edit the machineTags. That would require new functionalities in the registry backend. So using MachineTags isn't a simple option, but other than that you are right - it is also an option

Can a machine tag be added to datset by us if we have the correct permissions to the registry? Or is machine tag something that is generated? I see that we can add keywords/tags to the datasets.

MachineTags can only be created/edited/deleted by admins I'm afraid.
Doesn't keywords fit your use case quite nicely? The examples 'Education' and 'Nature index' sounds like keywords. What is you concern with those?

@MortenHofft Thank you for your reply. I also think that keywords will fit our case. I was just asking about machine tags to ensure we choose the best solution. I was not fully aware of what a machine tag is. Sorry for that :-)

Honestly I'm not sure what is the best solution either, but keywords sounds like it would work for what you describe you need. And best of all it doesn't require any extra development so you can start using them today.
MachineTags was introduced before my time in GBIF, but I believe the original plan was to be able to give access (to say you as a developer) to edit any machineTag within a namespace. But that was never implemented and it is currently admin only. The suggestion to scope machineTags was brought up in another issue recently.
The challenge I can see with keywords is that it require you to edit the dataset archive, whereas tags and machineTags can be edited in the registry/API. Which might be convenient in some cases. But frustratingly tags and machineTags are not exposed in dataset/search. So hence my suggestion to use keywords - it seems the simplest solution right now.


Add GOPATH/bin to PATH for etcd-postsubmit job

Missing $GOPATH/bin in PATH is causing failing ./scripts/test_lib.sh: line 240: gotestsum: command not found
Hence the junit xml files are not being generated.

/cc @mkumatag


Low Priority Question

Context
I have custom field for various asset details like MAC address and IP Address. I am trying to search for an asset by IP address I end up getting multiple results.
For Example, if i search for an IP like <IP_ADDRESS>, I will get results for <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, etc.
Is there a way make the search more exact or to search by a custom field? I looked through the commands and docs and don't see anything of that sort. If there's not, maybe I'll look into doing a PR or something. It would be very useful.

Currently SnipeitPS does not support searching direcly with specific custom field. Only option is  first get all assets or sub set of assets and filter results with powershell.
Custom fields are little tricky. to compare custom fields you cannot compare field directly , instead comparing to fields value property will work.
So you can filter your results with custom files by using  syntax like:
Get-SnipeitAsset | Where-Object { $_.custom_fields.'your custom field name'.value -eq 'somevalue'}

use Get-SnipeitAsset -all  or other filter if needed.


Duplicated annotation: annotation(Dialog(enable = true, group = "Inhibitor Arc"));

https://github.com/lochel/PNlib/blob/fb0bb36dfb4e8ae738423ee4e4d8905d88da0c1e/PNlib/Components/IA.mo#L5

Has been fixed
Thank You.


update documentation for using apply

I'm submitting a

[ ] bug fix
[ ] feature
[x] documentation addition

What is the current behaviour?
N/A
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
What is the expected behavior?
N/A
What is the motivation / use case for changing the behavior?
Issue #470
Update documentation to make it more clear on ways to run manifests in comtrya. More specifically, the usage of the -d and -m flags.
Please tell us about your environment:
Version (comtrya --version): v0.8.9
Operating system: macOS 15.0

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 26.19%. Comparing base (b574682) to head (70fb776).
Report is 51 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #471      +/-   ##
==========================================
- Coverage   28.59%   26.19%   -2.40%
==========================================
  Files          85       84       -1
  Lines        2301     2317      +16
==========================================
- Hits          658      607      -51
- Misses       1643     1710      +67

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


AttributeError: 'NoneType' object has no attribute 'GetPath'

在安装完成所有部分后，
我在运行 python3 scripts/rsl_rl/train.py --task Template-Isaac-Reach-Kinova-v0 --num_envs 4096 这行命令的时候，弹出Isaacsim软件并卡住，

过一会产生如下报错：

[INFO]: Time taken for scene creation : 513.928256 seconds
Traceback (most recent call last):
File "/home/user/RL/isaacLab.manipulation/scripts/rsl_rl/train.py", line 119, in
main()
File "/home/user/RL/isaacLab.manipulation/scripts/rsl_rl/train.py", line 74, in main
env = gym.make(args_cli.task, cfg=env_cfg, render_mode="rgb_array" if args_cli.video else None)
File "/home/user/下载/yes/envs/isaaclab/lib/python3.10/site-packages/gymnasium/envs/registration.py", line 802, in make
env = env_creator(**env_spec_kwargs)
File "/home/user/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/envs/manager_based_rl_env.py", line 75, in init
super().init(cfg=cfg)
File "/home/user/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/envs/manager_based_env.py", line 108, in init
self.scene = InteractiveScene(self.cfg.scene)
File "/home/user/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/scene/interactive_scene.py", line 105, in init
self._add_entities_from_cfg()
File "/home/user/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/scene/interactive_scene.py", line 404, in _add_entities_from_cfg
asset_cfg.spawn.func(
File "/home/user/IsaacLab/source/extensions/omni.isaac.lab/omni/isaac/lab/sim/spawners/from_files/from_files.py", line 145, in spawn_ground_plane
collision_prim_path = prim_utils.get_prim_path(
File "/home/user/.local/share/ov/pkg/isaac-sim-4.0.0/exts/omni.isaac.core/omni/isaac/core/utils/prims.py", line 631, in get_prim_path
return prim.GetPath()
AttributeError: 'NoneType' object has no attribute 'GetPath'
2024-07-25 08:22:33 [521,184ms] [Warning] [carb] [Plugin: omni.sensors.tiled.plugin] Module /home/user/.local/share/ov/pkg/isaac-sim-4.0.0/extscache/omni.sensors.tiled-0.0.3+106.0.0.lx64.r/bin/libomni.sensors.tiled.plugin.so remained loaded after unload request
2024-07-25 08:22:33 [521,188ms] [Warning] [carb] [Plugin: omni.spectree.delegate.plugin] Module /home/user/.local/share/ov/pkg/isaac-sim-4.0.0/kit/exts/omni.usd_resolver/bin/libomni.spectree.delegate.plugin.so remained loaded after unload request
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,189ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Annotators' for removal
2024-07-25 08:22:33 [521,190ms] [Warning] [omni.graph.core.plugin] Could not find category 'Replicator:Core' for removal
2024-07-25 08:22:33 [521,465ms] [Warning] [carb] Recursive unloadAllPlugins() detected!

Please check if your network proxy is functioning properly. Additionally, please ensure that your hardware system meets the requirements for Isaac.


Mass publish sites management command

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
Closes #730
Closes #805
What's this PR do?

Renames the trigger_pipelines command to mass_publish and refactors it to run batched sets of tasks to:

run prepublish actions (but only if specifically requested)
sync content to github
merge git content to the appropriate branch
trigger the concourse pipeline
filters to source=ocw-import by default unless overridden

Renames some functions to hopefully make the code more consistent and less confusing
Does not try to update youtube metadata for videos imported from legacy OCW (unless on production).

How should this be manually tested?

Use the same AWS_, CONCOURSE_,  but notGIT settings as RC (use a personal org)
Import some ocw sites: manage.py import_ocw_course_sites --bucket ocw-to-hugo-output-qa --limit 2
Run manage.py mass_publish live --filter 1-1 --chunks 1  --prepublish
The git 'preview' and 'release' branches of matching sites should be merged from main, and the concourse pipelines should be triggered.  They will fail if pointing to your local org repo but that's ok.
Once testing is done, from a heroku shell run  manage.py import_ocw_course_sites --bucket ocw-to-hugo-output-qa --limit 2 again to set back to the correct org repo.

Codecov Report

Merging #801 (7560cee) into master (341be69) will increase coverage by 3.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #801      +/-   ##
==========================================
+ Coverage   91.56%   94.64%   +3.08%
==========================================
  Files         190       94      -96
  Lines        7135     2670    -4465
  Branches     1126      583     -543
==========================================
- Hits         6533     2527    -4006
+ Misses        523      141     -382
+ Partials       79        2      -77

Impacted Files
Coverage Δ

content_sync/api.py

content_sync/decorators.py

content_sync/pipelines/base.py

content_sync/pipelines/concourse.py

content_sync/tasks.py

websites/views.py

content_sync/backends/github.py

websites/admin.py

main/utils.py

content_sync/admin.py

... and 86 more

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 341be69...7560cee. Read the comment docs.

Just rebased and fixed all the conflicts, ready for another look


Don't pre-allocate account locations in the sparse ledger

This PR builds upon #14520, attempting to fix the potential 'gappy allocations' caused by the current protocol. In particular, a pair of transactions in a block where

the first creates a new account, but the transaction failed
the second creates a different new account, and the transaction succeeded.

Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

!ci-build-me

!ci-nightly-me

!ci-build-me

!ci-nightly-me

!ci-build-me

Breaking change; closing.


FileName validation?

There are numerous warnings about using the FileName property of IFileInfo without validation but there is no indication on how to do that validation. Is there an example that shows how to properly do a validation on the FileName?

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

ID: 9a5cfa38-0e6c-4346-dafc-84ea5c513fcf
Version Independent ID: c6787f0f-3e9a-6efb-0e6e-5ccb957d3f4a
Content: File uploads in ASP.NET Core
Content Source: aspnetcore/mvc/models/file-uploads.md
Service: unspecified
GitHub Login: @ardalis
Microsoft Alias: riande

Hello @KevinBurton!
Thanks for reporting. We have a few issues to address on this topic (it's due for an overhaul/rewrite I believe). Those issues are tracked by https://github.com/aspnet/Docs/issues/5331.
I'll add this to the list. Thanks again for reporting!

After spending almost all day on this, I had working samples but it did not work in my code. Finally, I discovered that if you remove the ApiController attribute, it works.


-Fixed typos in checkpoints missing .tgz (eg. 'BioGPT-QA-PubMEDQA-BioGPT-Large.tgz')

-Checkpoint links were missing '.tgz' extensions

I actually fixed all of the checkpoint links in the README.md . :)


Analytics Apim Artifact names need to be corrected according to the common naming convention.

Description:
Please follow the "Standards and Guidelines for Analytics Artifacts" documentation and rename the artifact name accordingly.

By Sajith Dimal from https://wso2.org/jira/browse/ANLYAPIM-145

These artifacts are not relevant to new SP based analytics. Stream names and attributes are already in the correct format. Hence closing.


docker: 18.09.1 -> 18.09.2

Motivation for this change
CVE-2019-5736
@GrahamcOfBorg test docker
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Upstream does some weird git-bundle gymnastics in order to get the correct runc revision, but I'm pretty sure I got it right. Verification would be appreciated: https://github.com/docker/docker-ce/commit/e4931e664feac6fa8846f3f04268a0cc98822549#diff-9f3e85b3e5dd57d45832939603ad323eR25

Thanks for this PR @srhb. Do you know the impact this CVE has on nix installations of docker? My intuition is that it should be minimal because the nix store is usually mounted as read-only, but I'm not 100% sure.

@carlosdagos I too believe we're immune, but at the time I had no way of actually testing the bug, so better safe than sorry. :)


Deleted MIEngine libraries

Hi
I am a developer for Tizen App.
During working, we got one issue.
In Visual Studio 2022 Community Preview (17.3.0 Preview 1.1), we tried to install https://marketplace.visualstudio.com/items?itemName=tizen.VSToolsTizen2022 (<IP_ADDRESS>)
After completing to installation, I find out the one issue.
“Visual Studio Tools for Tizen” has Tizen MIEngine internally
When a user installs “Visual Studio Tools for Tizen” vsix, “Visual Studio Tools for Tizen” tries to install embed MIEngine together.
Normally, After completion of the installation, we can see both “Visual Studio Tools for Tizen” and “Tizen MIEngine” libraries.
There is no issue till Visual Studio 17.2.3
But On the 17.3.X preview version, After installation the vsix, we can see both “Visual Studio Tools for Tizen” extension libraries but we can not see Tizen_MIEngine libraries.
As we analyzed, we find out the below steps.
Install “Visual Studio Tools for Tizen” extension libraries
Install Tizen_MIEngine libraries.
Execute vsfinalizer.exe
After that, Tizen_MIEngine libraries were deleted => It’s the issue which we ask you.
I also update in detail vsfinalizer.exe’s command lines.
Executing: “C:\Program Files\Microsoft Visual Studio\2022\Preview\common7\ide\vsfinalizer.exe” -Operation Modify -SKU Community -InstallationID 975af454 -InstallationName VisualStudioPreview/17.3.0-pre.1.1+32519.111 -InstallationVersion 17.3.32519.111 -InstallationWorkloads Microsoft.VisualStudio.Workload.CoreEditor,Microsoft.VisualStudio.Workload.NetCrossPlat,Microsoft.VisualStudio.Workload.ManagedDesktop,Microsoft.VisualStudio.Workload.NativeDesktop,Microsoft.VisualStudio.Workload.VisualStudioExtension -InstallationPackages
We make a workaround to debug it on the preview version(17.3.x), too
Workaround
src/MIDebugPackage/source.extension.vsixmanifest // Tizen MIEngine project
We changed from  to
src/Tizen.VisualStudio.VsixSetup/source.extension.vsixmanifest // Visual Studio Tools for Tizen
We changed from  to
src/Tizen.VisualStudio.VsixSetup/Tizen.VisualStudio.VsixSetup.csproj // Visual Studio Tools for Tizen
Removed the lines

$(GetVsixSourceItemsDependsOn);AddToVsixSources

<_Contents Include=“$(OutputPath)\DeployedContents**” />

Tizen\VisualStudio%(_Contents.RecursiveDir)
MSBuild

We can reproduce it in the sample projects also using the following steps,

Create a sample dependent C# vsix project from the project wizard, change the option in source.extension.manifest file to allusers=true (e.g - )and generate vsix.
Create a sample main C# vsix project, change the option to allusers=true in source.extension.manifest file(e.g - ), copy the dependent vsix generated above, add dependency as embedded VSIX, and create main VSIX.
example code,
<Dependencies>
<Dependency Id=“Microsoft.Framework.NDP” DisplayName=“Microsoft .NET Framework” d:Source=“Manual” Version=“[4.5,)” />
<Dependency d:Source=“File” DisplayName=“VSIXProject1” Id=“VSIXProject1.61d8d5f9-ff1b-483f-8e85-71594cb37f6b” Version=“[1.0,<IP_ADDRESS>)” d:InstallSource=“Embed” Location=“VSIXProject1.vsix” />
</Dependencies>

Installation in Visual studio 2022 17.2.3 version
-> Both the main and dependent VSIX is installed.
Installation in Visual Studio 2022 Preview 17.3.1 version
-> Only the main VSIX is installed, the dependent VSIX is not installed.
For the Official version like community or professional,
we need to have AllUsers=“true”
If  there is no AllUsers=“true” on the vsix, we can not install the vsix.
But for preview version like community preview (17.3.x)
we should not have AllUsers=“true”
If there have AllUsers=“true” on the vsix, we can not debug tizen.net app
Could we use  AllUsers=“true” on the preview version, too?
Thanks

We are waiting for your help.

Moving to https://github.com/microsoft/VSExtensibility/issues/95


Dispose ConfigurationReloadToken

Merge of https://github.com/aspnet/Extensions/pull/588

@HaoK So based on Chris's comments, it sounds like we shouldn't do this.

Yeah I guess the issue is that we shouldn't dispose the underlying cancellation token after it fires, but we maybe should dispose the ConfigurationReloadToken as part of ConfigurationRoot's dispose:
https://github.com/aspnet/Extensions/blob/master/src/Configuration/Config/src/ConfigurationRoot.cs#L124
Thoughts?

Those roots would only be disposed at the end of the application, no? Yes that'd be a good idea, but I don't see how it ties back to the original bug report.

I'm not sure what you are referring to in the original bug, you mean "Possible CTS leak in ConfigurationReloadToken / ConfigurationRoot" ?  If its not safe to dispose the CTS after firing, then the only thing we can do is dispose it when the root is disposed.

They implied there was a memory accrual for non-disposed tokens during the lifetime of the app. It doesn't look that that was ever confirmed.

Configuration leaking was addressed via https://github.com/aspnet/Extensions/issues/861 so yeah that was  reported in other issues

Closing this for now, issue remains open for tracking/discussion


Make 'return' (and later 'next'/'last'/'redo') bind to the mainline, not to the quasi/macro

Suggestion via conversation with @vendethiel++, who finally convinced me this is a good idea.
Here's some code to discuss around:
macro r42() {
    quasi {
        return 42;
    };
}

func foo() {
    r42();
    return -1;
}

According to my previously held view, the return 42 would get injected by the mainline, but then fail at runtime because at that point the surrounding routine that bound that return (the macro r42) has already exited.
This issue suggests changing the semantics so that return 42 actually returns from foo. This is done in practice by return binding later, at macro expansion time. No prob, since a quasi does not contain 007 code.
By the way, once we have #325, this should apply to those control-flow keywords as well.
(@vendethiel describes the above behavior as return being dynamic, but I don't think that's quite it. It's still a lexical binding, it's just that it's done a bit later, at macro expansion time. This is in analogy with #216 and macro calls, by the way. The whole thing can be justified, or at least hand-waved, because return is a statement form. In other words, this is a counterargument against making return a listop, as #300 suggests. (Because then return would just be a normal identifier, and there'd be no good reason for it to bind so late.))

@vendethiel describes the above behavior as return being dynamic, but I don't think that's quite it. It's still a lexical binding, it's just that it's done a bit later, at macro expansion time.

I did mean "dynamic at compile-time" (please don't make this dynamic at runtime!), I think Let Over Lambda's term of "sublexical" is applicable here.

Same as in https://github.com/masak/007/issues/159#issuecomment-233127155 ?

Yes -- you have better memory than I do.

Probably not a big deal (since various dataflow steps will have to happen after macro expansion anyway), but implementing this issue means that you can't know things statically about unexpanded source with macros in it, since any given macro might contain control-flow primitives.

since any given macro might contain control-flow primitives.

It was already (meant to be) possible before, with exceptions.

Fair point, although (correct me if I'm wrong), if you're not doing any CATCH on the mainline end, your code will either run to completion (because no exception) or die and be caught somewhere further up the stack. Whereas here, mysteryMacro() might mean that the code breaks out of your loop, or your case statement, or returns from your function.
Again, no big deal. It just means that until we've expanded fully, we don't necessarily have the full control-flow graph.


Run not working, returning error message

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
Immediately when starting when Run is enabled or when trying to reenable Run.
✔ Expected Behavior
Run to activate
❌ Actual Behavior
Error message and Run not working
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22621.0
IntPtr Length: 8
x64: True
Date: 2023-06-12 17:16:05
Exception:
System.IO.FileNotFoundException: Could not find file 'C:\Program Files\PowerToys\modules\launcher\Images\app.dark.png'.
File name: 'C:\Program Files\PowerToys\modules\launcher\Images\app.dark.png'
at Microsoft.Win32.SafeHandles.SafeFileHandle.CreateFile(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options)
at Microsoft.Win32.SafeHandles.SafeFileHandle.Open(String fullPath, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable1 unixCreateMode)    at System.IO.Strategies.OSFileStreamStrategy..ctor(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable1 unixCreateMode)
at System.IO.Strategies.FileStreamHelpers.ChooseStrategyCore(String path, FileMode mode, FileAccess access, FileShare share, FileOptions options, Int64 preallocationSize, Nullable`1 unixCreateMode)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share)
at System.Windows.Media.Imaging.BitmapDecoder.SetupDecoderFromUriOrStream(Uri uri, Stream stream, BitmapCacheOption cacheOption, Guid& clsId, Boolean& isOriginalWritable, Stream& uriStream, UnmanagedMemoryStream& unmanagedMemoryStream, SafeFileHandle& safeFilehandle)
at System.Windows.Media.Imaging.BitmapDecoder.CreateFromUriOrStream(Uri baseUri, Uri uri, Stream stream, BitmapCreateOptions createOptions, BitmapCacheOption cacheOption, RequestCachePolicy uriCachePolicy, Boolean insertInDecoderCache)
at System.Windows.Media.Imaging.BitmapImage.FinalizeCreation()
at Wox.Infrastructure.Image.ImageLoader.Initialize(Theme theme)
at PowerLauncher.App.<>c__DisplayClass19_0.b__0()
at Wox.Infrastructure.Stopwatch.Normal(String message, Action action)
at PowerLauncher.App.OnStartup(Object sender, StartupEventArgs e)
at System.Windows.Application.<.ctor>b__1_0(Object unused)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
Other Software
No response

i can't say that i got this same error, but i can confirm that Run is not working for me either.

/dup #19871
This might be caused by some older files not being uninstalled correctly.
Would you mind:
1 - Uninstall PowerToys.
2 - See if the PowerToys install directory (default: C:\Program Files\PowerToys) is still there. If it is, delete it manually.
3 - Install PowerToys.
Please let us know if this works for you.


Consolidated response to open issues

Closes #48 (3760e42)

Closing due to staleness.
If this is wanted by the current repo owners, let me know and I'll be happy to re-submit it as a new PR.


Bumped version of material asset binary serializer

PR Details
Fixes material being black when loading older project files.
The binary change was introduced with 2f96f724438e62c1d5716b3c4d70518499d1a4d8
Description

Related Issue

Motivation and Context

Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks (sorry, I should have caught this one)


Address book

Allow to store accounts and auto-complete when generating new transactions.

small addition:
All own accounts added to the app should also be available when sending transactions to allow fast transfer between own accounts.


Nullable dates are parsed as 01/01/1970 instead of staying null when fetched from the GraphQL API

Describe the bug
My document has a date field, non-required with a null default value. When fetched through the GQL API, I end up having a 01/01/1970 date instead of a null value.
To Reproduce

Create a blueprint collection with a nullable date
Insert an object in the collection with the date set as null
View/Edit the document from the generated CRUD on client side

Expected behavior
The parsed values shoudl detect the date is null and keep it that way instead of new Date(null)

@Fen747 the response from GraphQL returns a timestamp, is that timestamp 0 in your network response?
In the serialise() we do run this check: https://github.com/bluelibs/bluelibs/blob/2a3c1daf228706b77682a92ca5f51edad70bf752/packages/x-bundle/src/graphql/scalars/Date.scalar.ts


[ART]: Kitchen Sink Model

Description of what will be added or updated
Kitchen Sink model by Synuk, contains only the .fbx model file.
Additional Information
Trello Card: https://trello.com/c/3SuzyRgd

 Sink

Model added with PR #45


Allow anonymous endpoint

Allow to create an endpoint as anonymous.
An anonymous endpoint does not get added
to the network service records.

Signed-off-by: Alessandro Boch<EMAIL_ADDRESS>

LGTM


WHERE clause not effective for SHOW metadata queries

WHERE clauses are not yet effective for certain SHOW metadata queries. SHOW TAG KEYS for example.

works for SHOW TAG KEYS and SHOW MEASUREMENTS. Does not work for SHOW FIELD KEYS but that makes sense, as it would require a full scan of all points.


operator camel-karavan-operator (3.20.0)

Signed-off-by: Marat Gubaidullin<EMAIL_ADDRESS>Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description of the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/retest

/merge possible

@J0zi @mvalarh I have just set minKubeVersion: 1.22.0 and now Operator test failed and 4.8-deploy-operator-on-openshift still checking. How could I fix that?

/retest-required

/retest

/merge possible

/merge possible


Test PPE preview on public repo

Docs Build status updates of commit 50a70c7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/master-data-services/add-a-group-master-data-services.md
:warning:Warning

Details

docs/master-data-services/add-a-group-master-data-services.md

Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Value out of range for 'ms.date': '03/01/2023'. Must be a date in format MM/DD/YYYY, no more than 30 days from today.
Line 24, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'brokenlink.md'.
Line 26, Column 1: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/sql/master-data-services/master-data-services-overview-mds' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 2, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: 'description: "Add a Group (Master Data Services)"'
Line 7, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: 'ms.technology: master-data-services'
Line 10, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '  - "groups [Master Data Services], adding"'
Line 11, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '  - "adding groups [Master Data Services]"'
Line 16, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '# Add a Group (Master Data Services)'
Line 26, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '[Test 2](https://docs.microsoft.com/en-us/sql/master-data-services/master-data-services-overview-mds)'
Line 53, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '-   [Assign Functional Area Permissions (Master Data Services)](../master-data-services/assign-functional-area-permissions-master-data-services'
Line 53, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: '-   [Assign Functional Area Permissions (Master Data Services)](../master-data-services/assign-functional-area-permissions-master-data-services.md)  '
Line 53, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: 'missions (Master Data Services)](../master-data-services/assign-functional-area-permissions-master-data-services.md)  '
Line 56, Column 1: [Suggestion: sensitive-language] Term 'master' is sensitive and should not be used in content or code. Usage: ' [Security (Master Data Services)](../master-data-services/security-master-data-services.md)  '

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Congresses served in

On multiple occasions, I've found myself looking through the legislatiors-historical data to find members who served in X Congress. As best I can tell, you have to use the dates of the terms to figure that out, which can be difficult as Congresses don't always start on Jan 3. Could there be an array field with the numbers of Congresses that a legislator served in (ie [113, 114, 115])?

See #185 for an earlier long discussion about this. :)


AIX requires EXTDLDFLAGS to specify an entry point

EXTDLDFLAGS is defined as "-e$(TARGET_ENTRY)" on AIX, so replace "$(TARGET_ENTRY)" with an actual entry point name.  As of now, AIX is the only platform that defines EXTDLDFLAGS, so this change won't affect any other platforms.

Looks good, thanks!
I'll try to use mkmf when I get some time, so we don't need to tweak so much this helper, and it should help portability.


Bower support

Can you add Bower support too?
How do you add this lib through bower?
bower search Leaflet-WFST  gives no results.

done
>bower search Leaflet-WFST
Search results:

    Leaflet-WFST git://github.com/Flexberry/Leaflet-WFST.git

you also can install this lib with bower directly from repo

thanks


How to correctly release ID3D12Resource for IDXGISwapChain3::ResizeBuffers

Hi I'm new to rust, but experienced with win32/directx.
I am trying to make a resizable swap chain from a window using IDXGISwapChain3::ResizeBuffers
I get an error in the validation layer DXGI ERROR: IDXGISwapChain::ResizeBuffers: Swapchain cannot be resized unless all outstanding buffer references have been released. [ MISCELLANEOUS ERROR #19: ]
I have working c++ code which calls ID3D12Resource::Release on buffers which are obtained from the IDXGISwapChain3::GetBuffer function before making the call to IDXGISwapChain3::ResizeBuffers.
What would be the equivalent of a Release call in rust? because the api doesn't mention Release

AddRef and Release are tied to Clone and Drop respectively. Implicitly or explicitly droppings your buffers should be sufficient.

Thanks for your quick response,
In this case I don't have references to buffers that I keep myself, they are created implicitly along with the IDXGISwapChain3  so I think I must need to explicitly drop them in this case as the ownership is with the swap chain and I want to retain the swap chain itself.
I am not sure how to explicitly drop the internally owned buffer resources. They can be obtained through the function call IDXGISwapChain3::GetBuffer I have tried passing to std::mem::drop but still get the same validation error.
  for i in 0..FRAME_COUNT {
      let buffer: ID3D12Resource = self.swap_chain.GetBuffer(i).unwrap();
      std::mem::drop(buffer);
  }

Thanks again for your help, it has given me more to look into.

Have you created render target views for any of the buffers? I believe they'll hold a reference.

Yeah, I have references to render target views which I was also clearing.
I have been able to successfully call ResizeBuffers as long as I don't use the render target views in a command list, so I'm just following back from there now to make sure the waits are being correctly handled and the command lists are no longer in use on the gpu.
I think it should be ok from here on out, at first I wasn't entirely sure how the COM objects interacted with rust lifetimes but it's much clearer now.
Thanks for the help.

The issue seems to be with using a transition barrier, each frame the swap chain buffer leaks references. This issue is also present in the direct3d12 sample of the repository.
If you build run the app through visual studio upon closing D3D Live objects are reported in the debug console.
D3D12 WARNING: 	Live Object at 0x000001F4E2D4D020, Refcount: 1318. [ STATE_CREATION WARNING #0: UNKNOWN]
D3D12 WARNING: 	Live Object at 0x000001F4E2D4CD70, Refcount: 0. [ STATE_CREATION WARNING #0: UNKNOWN]
D3D12 WARNING: 	Live Object at 0x000001F4E2D4DFD0, Refcount: 1318. [ STATE_CREATION WARNING #0: UNKNOWN]

The 2 with large ref counts in the samples case leak 2 references per frame. one barrier for transitioning present > write and then write > present.
I am able to successfully resize the swap chain (and make draw calls, clear) without using the transition (with validation warnings spewing out)
The command lists and allocators have been reset and I have also tried to create new command lists but that also doesn't seem to help to drop the refs.
I am not sure at this point whose responsibility it is to drop the memory for the transition. I am using the same function the sample in this repo uses.
fn transition_barrier(
    resource: &ID3D12Resource,
    state_before: D3D12_RESOURCE_STATES,
    state_after: D3D12_RESOURCE_STATES,
) -> D3D12_RESOURCE_BARRIER {
    D3D12_RESOURCE_BARRIER {
        Type: D3D12_RESOURCE_BARRIER_TYPE_TRANSITION,
        Flags: D3D12_RESOURCE_BARRIER_FLAG_NONE,
        Anonymous: D3D12_RESOURCE_BARRIER_0 {
            Transition: std::mem::ManuallyDrop::new(D3D12_RESOURCE_TRANSITION_BARRIER {
                pResource: Some(resource.clone()),
                StateBefore: state_before,
                StateAfter: state_after,
                Subresource: D3D12_RESOURCE_BARRIER_ALL_SUBRESOURCES,
            }),
        },
    }
}

The transition in the barrier is std::mem::ManuallyDrop so this makes sense the references might leak, I just haven't figured out how to call manually drop myself because I am new to rust and there are quite a lot of new concepts this is covering.
I will continue digging and see where I get.

OK I have managed to get it working, but it doesn't quite feel right. I can manually drop the transition immediately after pushing the resource barrier into a command list:
let mut barrier = transition_barrier(
    &queue.swap_chain.GetBuffer(bb as u32).unwrap(),
    D3D12_RESOURCE_STATE_RENDER_TARGET,
    D3D12_RESOURCE_STATE_PRESENT,
);
self.command_list[bb].ResourceBarrier(1, &barrier);
let _: D3D12_RESOURCE_TRANSITION_BARRIER = std::mem::ManuallyDrop::into_inner(barrier.Anonymous.Transition);

This cleans up all references and when closing the app no D3D Live Objects are being reported.
The extra book-keeping is not necessary with barriers in c++. I am now just not 100% sure how to handle this correctly, it works right now with no issues or any validation errors but I'm not sure when the drop should actually be called. After the transition is passed to command list? after the command list is submitted on CPU? or after command list is submitted on the GPU?
As the sample for direct3d12 has this leak issue it would be useful to have a correct implementation of where to drop the transition.

I am going to close this issue no I have investigated a bit more.
For each barrier transition added to a command list I push a barrier (move) into a vec. Once the command list has been completed on the GPU the barriers are removed from the vec and then manually dropped.
This will ensure all in flight resources on the GPU will retain a reference and allows to decouple CPU resource management from the GPU.

Ran into this same issue and don't think this should be closed but perhaps be something reevaluated by @kennykerr on the code gen side? The ManuallyDrop here is pretty dangerous and it took me a full day to figure out what was going on in my application. I know this API is unsafe in general, but this is a pretty sharp edge to cut yourself on.

Yeah I agree, I spent a good couple of days debugging this.
I am new to rust and it was the first time I encountered ManuallyDrop so wasn't sure if it was just me who was confused and not sure how to deal with it.
I think the intention is that once you create a transition you must manually drop after it has been consumed on the GPU, so keeping a reference to the resource until you know it is safe to drop it makes sense.
I am not sure if this can be deduced or automatically handled because of how internally d3d12 works, but I think the ideal situation for a user would be that when a D3D12_RESOURCE_BARRIER is passed to a ID3D12GraphicsCommandList the command list takes ownership of the resource ref and then once the command buffer is Reset then it drops the resources that were attached to transitions.
I am doing this already myself, you can see in the implementation here... my CmdBuf keeps track of in_flight_transitions.

Thanks for the reminder. I was planning on looking into this issue more closely and if the issue remains open, I won't forget about it. 😉

@damyanp / @robmikh any insights you can share about the ownership semantics expected by D3D would be much appreciated.

I think the intention is that once you create a transition you must manually drop after it has been consumed on the GPU, so keeping a reference to the resource until you know it is safe to drop it makes sense.
This was something I checked the docs about but they didn't shed much clarity on the matter. One would imagine that D3D would increment the COM pointer internally and hold onto it, but perhaps wistful thinking. I any case I made the assumption the related resource would be kept alive by other means and drop it immediately. Works for now, but something I'll need to revisit for sure.

Cool stuff btw! Definitely will give me some inspiration for my project (porting egui to windows-rs). Thanks for sharing. 😄

Thanks for the reminder. I was planning on looking into this issue more closely and if the issue remains open, I won't forget about it. 😉

No problem! I figured it was something that'd be of interest and just slipped through the cracks. I noticed that it seems all of Damyan's sample ports are hit by this issue too, so I feel somewhat validated in my struggle to track this down now. Hopefully you find a nice solution in the end. 🤞
Thanks for all the great work on windows-rs and C++/WinRT by the way. You're making Windows programming a lot more fun. 😁

D3D12 is fundamentally unsafe here and relies on the programmer to ensure that resources are kept alive while the GPU is still using them.  It does not increase the reference counts for resources passed to it through resource barriers, or for any other APIs that record command lists or create descriptors.
So, in terms of COM reference counts, D3D does not take ownership of the resources passed in to these APIs.  Ideally, the projection would allow the app to just pass in an unsafe raw pointer to D3D.  IIRC, that was possible in an earlier version of windows-rs with the ABI types?

Thanks Damyan,

Ideally, the projection would allow the app to just pass in an unsafe raw pointer to D3D.

That is consistent with how the windows crate uses ManuallyDrop to put the ownership and decision-making here in the caller's hands.

Ideally, though, the transition barrier struct itself shouldn't own the reference.  It should be possible to fill out the struct without an addref.

Yes, unfortunately there's no metadata to distinguish between structs that want to own their fields and structs that don't. If we can convince the Win32 metadata folks to add something like that, I could plumb it through to Rust.

There isn't really anything left for me to do here. Perhaps @sotteson1 or @riverar want to tackle this in the metadata as I suggested above.

Reopening this as I had the idea to introduce a Borrowed<T> type that can be used to solve this and provide lifetime semantics more in line with Win32.

There isn't really anything left for me to do here. Perhaps @sotteson1 or @riverar want to tackle this in the metadata as I suggested above.

I don't know of a way for the scraper to know which structs own their fields and which don't. I can see the benefit, but I don't know where I would get that kind of information.

WinRT structs definitely own their fields while Win32 structs traditionally don't and just hold on to raw pointers/handles without implying that those resources will be freed when the struct goes out of scope. I've got a new Borrowed<T> type that I think should fix this for Win32 structs (WinRT structs already do the right thing) and also solve a few other scenarios that require borrowing owned resources. Borrowed is basically a lifetime-aware borrow-friendly ABI-compatible version of ManuallyDrop. It also comes in handy for interop when directly working with the ABI for some API.

Think we'll have to manually curate a list to track these.
Is this what you're effectively looking for in metadata?
// https://docs.microsoft.com/windows/win32/api/d3d12/ns-d3d12-d3d12_resource_transition_barrier
public struct D3D12_RESOURCE_TRANSITION_BARRIER
{
+ [Borrowed]
  public ID3D12Resource pResource;
  public uint Subresource;
  public D3D12_RESOURCE_STATES StateBefore;
  public D3D12_RESOURCE_STATES StateAfter;
}

// https://docs.microsoft.com/windows/win32/wmdm/wmdm-prop-config
public struct WMDM_PROP_CONFIG
{
  public uint nPreference;
  public uint nPropDesc;
+ [Owned]
  public unsafe WMDM_PROP_DESC* pPropDesc;
}

I don't think we should try to encode this in metadata. We should instead just assume that all Win32 structs don't own.

This would also solve #907.

It would be great if anyone who is interested in this issue to take a look at https://github.com/microsoft/windows-rs/pull/1788 which I believe would address this.


Add kerberos authentication support

Please add kerberos auth support by adding the net-ssh-krb gem and allow gssapi-with-mic auth method.
https://github.com/cbeer/net-ssh-kerberos
You will probably need to add something like this to your connection options.
https://github.com/nwops/kitchen-kerberos/blob/master/lib/train/transports/kerberos.rb#L74

This request is related to this jira issue and @joshcooper 's #107 .
We currently have released a Puppet forge module to address this very issue. But #107 should solve the issue. Partially anyway.

Even better solution is to support external ssh executables, filed as BOLT-174

Support for the gssapi-with-mic authentication method was added in https://github.com/puppetlabs/bolt/commit/2ad71b7a9a59bdcb4562ce6ebea4f997949e5dc6 and will be released on 0.7.0 later this week. Due to GPLv2 licensing issues, bolt does not have a hard dependency on net-ssh-krb gem. However, if you manually install the gem (on the controller node), then bolt will make use of it. You may need to update/modify your ~/.ssh/config to ensure the gssapi-with-mic authentication method is in your PreferredAuthentications option.


Prepare for 2.1.1 release

Codecov Report

Merging #83 (52aa4fb) into master (ccc1454) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   37.79%   37.79%
=======================================
  Files          41       41
  Lines        1061     1061
  Branches      141      141
=======================================
  Hits          401      401
  Misses        529      529
  Partials      131      131

Impacted Files
Coverage Δ

packages/walletconnect-v1/src/types.ts
86.66% <ø> (ø)

packages/walletconnect-v1/src/wc-signer.ts
78.57% <ø> (ø)

packages/walletconnect-v1/src/wc-wallet.ts
56.81% <100.00%> (ø)

Continue to review full report at Codecov.

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


awatcher: init at 0.2.7

Description of changes
awatcher is an alternative activity and idle watcher for ActivityWatch. This can be useful for some folks in certain occasions where aw-watcher-window  - https://github.com/NixOS/nixpkgs/blob/755b915a158c9d588f08e9b08da9f7f3422070cc/pkgs/applications/office/activitywatch/default.nix#L58
doesn't really work, for example when using KWin on wayland + other neat features.

Note: Since awatcher doesn't belong to the ActivityWatch org I thought it should probably be put into it's own separate pkgs/by-name/aw/awatcher/ dir.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Output of nix-shell -p nixpkgs-review --run "nixpkgs-review pr 321249":
1 package added:
awatcher (init at 0.2.7)

@applePrincess Thanks for the feedback!  Should I close this then or? Not sure what it is about the v3.0.0, the current master beta is v3.0.0. but they haven't yet released a v3.0.0 version hence why I opted for the latest one. Might just have been an accidental release, not sure.

 Should I close this then or? Not sure what it is about the v3.0.0, the current master beta is v3.0.0. but they haven't yet released a v3.0.0 version hence why I opted for the latest one. Might have been just an accidental release, not sure.

It happens, don't worry. Actually, your PR is even the third one #315864 :p
Sorry for your duplicated work. I guess we can keep both PRs open until we get more information, maybe @aikooo7 could explain?

Sooo, out of curiosity I checked my local awatcher repo and I found a locally cached v3.0.0 tag so maybe they really created it and deleted it? Possibly to bring more features? Not really sure, however #287858 builds without any issues for me so ig okay?

 Should I close this then or? Not sure what it is about the v3.0.0, the current master beta is v3.0.0. but they haven't yet released a v3.0.0 version hence why I opted for the latest one. Might have been just an accidental release, not sure.

It happens, don't worry. Actually, your PR is even the third one #315864 :p Sorry for your duplicated work. I guess we can keep both PRs open until we get more information, maybe @aikooo7 could explain?

 Should I close this then or? Not sure what it is about the v3.0.0, the current master beta is v3.0.0. but they haven't yet released a v3.0.0 version hence why I opted for the latest one. Might have been just an accidental release, not sure.

It happens, don't worry. Actually, your PR is even the third one #315864 :p Sorry for your duplicated work. I guess we can keep both PRs open until we get more information, maybe @aikooo7 could explain?

At the time 3.0.0 existed, if it doesn't anymore, I will change my pr soon

Should be good to go now :)

Alright, let's close this PR since the other one is older. Thank you for your work


[fix] #195 No entries written to C:/Windows/system32/drivers/etc/hosts

It's code generate invalid cmd file like:
>>"C:/Windows/system32/drivers/etc/hosts" echo <IP_ADDRESS>  lesson1  # ...

which is not needed because we have already checked the write permissions on the line:
119: if !File.writable_real?(@@hosts_path)

I can confirm that this patch worked for our team as well in Windows 10, which was not otherwise usable. It would be nice to have this included in a release.


ChemicalJS and more

This pull optimizes images, updates packages, and adds chemicalJS

ChemicalJS doesn't have gzip compression rn


EKS only returning I | missing_scope in pod log

Hey,
Installed following this guide:
https://medium.com/better-programming/monitor-your-kubernetes-resources-with-kubewatch-d40ecf420f28
(I also tried helm but got permission errors for listing services).
I used an app and generated an xoxb- token.
Added the app to the channel.
However all I get in the log is:
2020-12-21 11:08:35.811485 I | missing_scope
2020-12-21 11:08:35.842232 I | missing_scope
2020-12-21 11:08:35.894755 I | missing_scope
2020-12-21 11:08:35.924684 I | missing_scope
2020-12-21 11:08:35.974846 I | missing_scope
2020-12-21 11:08:36.007731 I | missing_scope
2020-12-21 11:08:36.054510 I | missing_scope
2020-12-21 11:08:36.088262 I | missing_scope
2020-12-21 11:08:36.136822 I | missing_scope
2020-12-21 11:08:36.170395 I | missing_scope
2020-12-21 11:08:36.217120 I | missing_scope

What should I do? Since this is not informative to what is broken.
Might be related to #123 , but no information there how to fix it

This is caused by your bot permission, you can check your slack bot permission

These are the required scopes


Fix problems with shell init when logging in via ssh

Load session-init in shell init files (/etc/profile for bash, /etc/zshenv for zsh).
Remove -u in session-init because it would enable -u when it is loaded as part of a shell init config file.
Supersedes #140.
Ping @azuwis

TBH I can not think of a reason to not load session-init on ssh login.


GH-43187: [C++] Support basic is_in predicate simplification

Rationale for this change
Prior to https://github.com/apache/arrow/pull/43256, this PR adds a basic implementation that does a linear scan filter over the value set on each guarantee. This isolates the correctness/semantics of is_in predicate simplification from the binary search performance optimization.
What changes are included in this PR?
SimplifyWithGuarantee now handles is_in expressions.
Are these changes tested?
A new unit test was added to arrow-compute-expression-test testing this change.
Are there any user-facing changes?
No.

GitHub Issue: #43187

@bkietz any ideas on the failing checks? The errors don't appear to be relevant, but lmk if I should look into them.

Failed CI unrelated


Docker build failing due to fairseq

#24 10.46   Downloading fairseq-0.12.1.tar.gz (9.6 MB)
#24 10.56      ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 9.6/9.6 MB 103.0 MB/s eta 0:00:00
#24 11.12   Installing build dependencies: started
#24 13.47   Installing build dependencies: finished with status 'done'
#24 13.47   Getting requirements to build wheel: started
#24 13.59   Getting requirements to build wheel: finished with status 'error'
#24 13.60   error: subprocess-exited-with-error
#24 13.60
#24 13.60   × Getting requirements to build wheel did not run successfully.
#24 13.60   │ exit code: 1
#24 13.60   ╰─> [16 lines of output]
#24 13.60       Traceback (most recent call last):
#24 13.60         File "/venv/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
#24 13.60           main()
#24 13.60         File "/venv/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
#24 13.60           json_out['return_val'] = hook(**hook_input['kwargs'])
#24 13.60         File "/venv/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
#24 13.60           return hook(config_settings)
#24 13.60         File "/tmp/pip-build-env-ksd6aas7/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 327, in get_requires_for_build_wheel
#24 13.60           return self._get_build_requires(config_settings, requirements=[])
#24 13.60         File "/tmp/pip-build-env-ksd6aas7/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 297, in _get_build_requires
#24 13.60           self.run_setup()
#24 13.60         File "/tmp/pip-build-env-ksd6aas7/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 313, in run_setup
#24 13.60           exec(code, locals())
#24 13.60         File "<string>", line 27, in <module>
#24 13.60         File "<string>", line 18, in write_version_py
#24 13.60       FileNotFoundError: [Errno 2] No such file or directory: 'fairseq/version.txt'
#24 13.60       [end of output]

https://github.com/rsxdalv/tts-generation-webui/actions/runs/9616242480/job/26525293745#step:6:2951

Fixed


Use is_alive in favour of isAlive for Python 3.9 compatibility.

 Issue by tirkarthi
Sat Oct 24 14:34:20 2020
Originally opened as https://github.com/codelucas/newspaper/pull/853

Fixes #804

tirkarthi included the following code: https://github.com/codelucas/newspaper/pull/853/commits

solved


Show lifecycle status for API

User story

As an API user
I want to be able to see the lifecycle status of an API
so that I can determine how safe it is to start using an API

Definition of done

[ ] Once owner has added a lifecycle status for API, it needs to be shown in

[ ] Catalog card view
[ ] Catalog table view
[ ] API profile

[ ] User can filter catalog by lifecycle status

Wireframe

API cards showing 5 lifecycle statuses:

from @bajiat 's feedback, we should replace the tag for "private" APIs. The reason is that an API may or may not be private, but it would always have a lifecycle status. so that tag should be placed first in API card

API lifecycle status can be seen as colored labels in both card and table view.
From both view, the APIs can be filtered by one/more lifecycle statuses.
[future implementation suggestion: filter by private and public APIs]

In API profile, the status appears on top of the page.
On transiting from one status to another, the time of change and transit from status appears as information (alt. can also be shown as tooltip on hover)

Default use for undefined
Primary use for development
Success use for production
Info use for design
Warning use for testing
Danger use for deprecated

Follow up task to filter API catalog by Lifecycle Status

@brylie based on @bajiat 's feedback, am changing the color of "Deprecated" Status to Red. Red would work as a caution to API consumer, so that they use this API in their own risk. The status "Unknown" can be shown in grey.
Am updating other lifecycle wireframes in related issues.


userData command line option

This change-set allows the setting of the userData path to be passed in as a command line option. This path controls to which folder localStorage data (and possibly other things) is stored as is critical if you want to run multiple clients on the same machine.
The option can be passed in as follows:
NODE_ENV=development ./node_modules/.bin/electron . --userData /Some/path/here

Keep in mind, on Windows, there is a different syntax there to set the environment variable.
Also, keep in mind with the above command you are bypassing all our dev "sugar" (watchers, sass compiler, etc...) that is otherwise present with npm start. Unfortunately, in order to pass a command line option to 'npm start' and then have it ultimately be passed into the 'electron' process would require some tweaks to our npm scripts. Since it's unlikely you would need to run a second client concurrently and develop on it, we're going to leave that alone for now.

The way to do it on Windows is
set node_env=development&&electron . --userData pathToDirectory

It's worth noting you have to have electron installed globally for that to work.

On windows, this can be done via:
set node_env=development
node_modules\electron\dist\electron.exe . --userData pathToDirectory

An additional point to consider is that starting the app this way will not automatically kick off the various watcher processes which compile your resources (e.g. Sass to CSS). So, if you'll be making any changes to the source when running the client this way (e.g. you pull in some updates), you'll want to run npm run build first before launching with the command above.


CC77368: Corrected links to images

Hello, @yuridio,
This proposed file change comes from https://github.com/MicrosoftDocs/EMDocs.it-it/pull/17 .
Could you review this contribution and help to merge if agreed?
Many thanks in advance.

@v-makoud : Thanks for your contribution! The author, @yuridio, has been notified to review your proposed change.

@v-makoud - You should pull this change over to the private repo so you can stage the change first. You can use these steps to pull the commits over to your local copy of the private repo.

Created a new PR here: https://github.com/MicrosoftDocs/EMDocs-pr/pull/786
@v-makoud: please-close

#please-close


Npx: Create npx_add_npx_to_command.py

Add npx to command
Prefixes npx to command name to run locally installed npm commands, also corrects misspelled npx commands.

Example

jest
npx jest

npx jeet
npx jest

jeet
npx jest

I am not sure how to write tests for the code, any help would be appreciated.

Thanks for contributing, @KartikSoneji! Apologies for the delay.
This is looking good. Regarding tests, you can base off of any existing test and write unit tests for match and get_new_command. Take the following as an example:
https://github.com/nvbn/thefuck/blob/11b70526f72339bcce0beefa1665c74bb06e18b3/tests/rules/test_git_commit_add.py#L1-L38


Fixed MacOS build

Fixed some errors on MacOS:

microhttpd.h not found
incompatible pointer types passing 'int (void *,...)instead ofenum MHD_Result (void *, ...)`.
Disabled error "implicit use of sequentially-consistent atomic may incur stronger memory barriers than necessary [-Werror,-Watomic-implicit-seq-cst]"

... except that compilation fails on my macos


Fixed Swap Nodes in mutate_rows per issue #41

Swapped based on state, row[0], then sorted.  As per our conversation in issue #41 thread.

Is this telling me that I need to edit ./docs/tutorial/index.rst ?

Yes, the test output changed because of the updated mutation function, so the inline tests in the docs need to be updated.

So sorry I had completely missed this got opened and reviewed!

Feel free to nudge me in the future :)


truTV giving ERROR: Unsupported URL

Checklist

[x ] I'm reporting a broken site support issue
[x ] I've verified that I'm running youtube-dl version 2020.02.16
[x ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x ] I've searched the bugtracker for similar bug reports including closed ones
[x ] I've read bugs section in FAQ

Verbose log

youtube-dl https://www.trutv.com/shows/hot-ones-the-game-show/season-1/episode-1/beware-the-bucket -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.trutv.com/shows/hot-ones-the-game-show/season-1/episode-1/beware-the-bucket', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.02.16
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg 4.2.2
[debug] Proxy map: {}
[generic] beware-the-bucket: Requesting header
WARNING: Falling back on generic information extractor.
[generic] beware-the-bucket: Downloading webpage
[generic] beware-the-bucket: Extracting information
ERROR: Unsupported URL: https://www.trutv.com/shows/hot-ones-the-game-show/season-1/episode-1/beware-the-bucket
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\YoutubeDL.py", line 796, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpuz77jgzq\build\youtube_dl\extractor\generic.py", line 3350, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://www.trutv.com/shows/hot-ones-the-game-show/season-1/episode-1/beware-the-bucket

Description

Error when trying to download from TruTV. It seems as if it defaults to the generic extractor instead of using the specific one. TBS seems to still work, which has the same player and URL structure.

So, did a little bit of exploring on this. It's maybe do-able, but you need an authentication token. When changing the regex in trutv.py, you'd want the regex to parse as
series_slug = hot-ones-the-game-show
clip_slug = season-1/episode-1/beware-the-bucket
id = None
The API link yielded by the extractor would be https://api.trutv.com/v2/web/series/clip/hot-ones-the-game-show/season-1/episode-1/beware-the-bucket
Unfortunately, visiting it gives {"message":"Missing Authentication Token"}

I was able to download videos on trutv at 1080p using the m3u8 method. You can download videos this way for now until it is fixed.

Open trutv video and login to TV provider if needed
Ctrl + Shift + i and search for m3u8. Reload video and page if it doesnt show up
There will be a master.m3u8 and a 'layer' link. Copy the link address of the master.m3u8 and execute on youtube-dl. This should give you the options to the videos up to 720p.
To get 1080p, copy the link address for the 'layer' one instead, then change the number after the word 'layer' (this appears twice) to 7, which is 1080p and the highest possible resolution. I tried 8 and above but they don't give you a higher resolution than 1080p. Numbers lower than 7 are just resolutions that are downloadable from copying the master.m3u8 link.

@nhlfarm does browser matter?  I'm trying chrome and not having any luck.  https://www.trutv.com/shows/tacoma-fd/season-1/episode-1/on-the-hot-seat-explicit

@nhlfarm This does not work on videos with ads, it just grabs up to the first ad.

@roadzy I don't think browser matters. I just tried downloading the tacoma fd episode and I was able to get the 'layer' link on both chrome and firefox: https://i.imgur.com/ef1LtB7.jpg
After that, I just copied the link from chrome by right clicking -> Copy -> Copy link address. Then, I changed the two times 'layer3' appears into 'layer7' and executed it on youtube-dl: https://i.imgur.com/VkoKoRV.png

@ThatNerdyPikachu I am able to download full episodes with the method I posted above. I have ublock origin on both chrome and firefox, and thought that maybe it was able to give a layer link that bypasses the ads when downloading with youtube-dl. But then I turned off ublock origin on chrome, then got the m3u8 layer link: https://i.imgur.com/t7Lkg8I.png
I executed it on youtube-dl and was still able to download the full episode: https://i.imgur.com/ey8Sbuq.png
I haven't had any instance where youtube-dl only downloads up to the first ad; I've only downloaded the full episodes. If you need any additional info I am glad to help

@nhlfarm seems to work except, I keep getting this:
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:0; previous: 55544263, current: 11171171; changing to 55544264. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:0; previous: 55544264, current: 11174174; changing to 55544265. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:0; previous: 55544265, current: 11177177; changing to 55544266. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224276, current: 5459487; changing to 27224277. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224277, current: 5460511; changing to 27224278. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224278, current: 5461536; changing to 27224279. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224279, current: 5462559; changing to 27224280. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224280, current: 5463583; changing to 27224281. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224281, current: 5464607; changing to 27224282. This may result in incorrect timestamps in the output file.
[mp4 @ 0x3bda9c0] Non-monotonous DTS in output stream 0:1; previous: 27224282, current: 5

@roadzy I got that error before too, and I believe it's because it's downloading the video using ffmpeg, which is the default HLS downloader. However, I was still able to download the episodes fully, although the fps was not maximized (i.e. 29.970000 fps). I would download and get the same errors you're getting (non-monotonous DTS), which would cause the videos to be like 28.84415 fps for example.
The fix that worked for me (which should also work for you as well) was to choose the HLS downloader as HLS native instead. You just have to add "--hls-prefer-native" without the quotations into the command and you should download the video using HLS native, as seen in the example of my previous post. I didn't get any errors after doing this and was able to download them fully at 29.970000 fps all the time.

@nhlfarm what are you doing to get the full 24 minutes?  I'm only getting to the first ad break.  Mine is like 10 minutes.  I've tried multiple episodes.

@roadzy I'm literally just doing the exact steps of what I posted above, and I'm able to get the full episodes. I haven't encountered the error that you and nerdypikachu are getting where only up to the first ad break is downloaded. I have all the full Adam Ruins Everything episodes downloaded and tried multiple episodes of tacoma pd and haven't downloaded any 10 minute ones; only the full episodes. Have you tried it with/without adblock or with a different browser, or updating youtube-dl and ffmpeg? I would try those and also try using the master link and downloading the 720p version and see if it downloads the full episode for you.

@nhlfarm just curious... can you try this and see if you get the whole clip or just 12 minutes like i am?
youtube-dl --hls-prefer-native https://tve.cdn.turner.com/trutv/c52b7d45944f5d3a7c115e6584e230f4/layer7/layer7_bk.m3u8?hdntl=exp=1587546364~acl=%2Ftrutv%2Fc52b7d45944f5d3a7c115e6584e230f4%2F*~hmac=0ff4fb37759d472ae5350f3403d0a54769cd96369c44dc8ad1b5feab739dd7d5

@roadzy I just tried to execute that but got HTTP Error 403: Forbidden, probably because the session already expired since I am trying it 15 hours after you posted it. Instead, I downloaded and uploaded a random episode of tacoma, with screenshots of every step I did:

Selected tacoma episode. Opened network activity with ctrl+ shift + i and searched for 'm3u8,' then played the episode. Copied layer link: https://i.imgur.com/8MGTTY5.png
Changed the 2 instances of 'layer3' to 'layer7' then executed on youtube-dl: https://i.imgur.com/yFoTZSk.png
(The first command I executed was the one you provided, which gave the HTTP 403 error)
After it finished downloading, this is what my cmd looked like: https://i.imgur.com/EtoUifw.png
It downloaded a ~900 MB episode.
I opened the .mp4 file on VLC and this is what I got: https://i.imgur.com/BzJuRyB.png
Notice the codec info: 1920x1080, 29.970000 fps, and it's 24:32 long, which is the full episode.
I uploaded it to mega: /file/OJc0VYZD#FxTBAMEjrc3fNkdgd0dw0E4kuAfvJeJNK6yQ-9t8EPk
(add that after mega.nz)

@nhlfarm I figured it out.  The issue was with ffmpeg.  When you ran --hls-prefer-native it would fail over to ffmpeg.  From the first day I started using youtube-dl, it has always just used ffmpeg.  I didn't know there was another option.  So digging further, I found out I needed pycrypto to get the native to work.  Once it switched to the native, 24 minute episode.  Also has the DTS issue with fox using ffmpeg and it also fixed that too!  What is it using to encode?  Should I always be using hls-prefer-native ?  Thanks for all your help!

@roadzy For me, since I started using youtube-dl it has defaulted to using the native HLS downloader, which is what was always used to download videos for me, and I never got any problems. When trying to download from trutv is when I realized youtube-dl could also use ffmpeg to download videos, but I kept getting errors unlike with native HLS. It's weird that your youtube-dl defaults to using ffmpeg instead of native HLS, maybe it's an outdated version. But I'm glad to hear you got it working and are able to download the full episodes. When using the '--hls-prefer-native' I am not 100% sure of what it is using to encode, I just know that it's amazing and I will always use it over ffmpeg. So I just add '--hls-prefer-native' to all of commands now, and I recommend you do too, since you said your youtube-dl defaults to ffmpeg.

@nhlfarm I've had the same issue with TruTV and FFmpeg's HLS downloader.  The downloaded files also played incorrectly.  After switching to the native HLS implementation it downloaded without HLS errors and it played back perfectly.
I wonder the status of the pull request that fixes TruTV?  (https://github.com/ytdl-org/youtube-dl/pull/24466)

I wonder the status of the pull request that fixes TruTV?  (https://github.com/ytdl-org/youtube-dl/pull/24466)

I'll test my PR with the native HLS mode and report back.


More Stats?

Any Chance we can get some more Stats?
For example active Users or Users currently plotting? Right now there are around 1700 Users and only 300 new plots being plotted. Even when we would assume that 300 Users are plotting 1 plot each (what probably isn't true, I'm currently making 5 at a time) the amount of active users would be realy low at around 17% of total users.

Building that in dashboard is not priority in the next days, (preparing for official pooling protocol).  But I just peeked into the database and there are ~50 different users currently plotting


A workaround for piped input

Hi there!
First of all: this is a cool project, thank you! Unfortunately once I tried to pipe some predefined commands into it as part of a script, it broke:
echo "board list" | 3llo

Because apparently piping also emits an EOF character, which is not handled by 3llo. In case this happens I simply replaced it with an exit command, but maybe you want to change the behavior for 3llo to keep running instead. I don't know how to do this though.
That's it, would be cool if we could get it to accept commands through piped input. Thanks!

Thanks for working on this @maurobringolf, I will take a look at it later this week.

Closed via https://github.com/qcam/3llo/commit/93e1ccdb80985d75ff48bc5fa7d381e989bde2e6. With that introduced, instead of piping an echo command, one can just specify 3llo --init "board list".

Thanks a lot, that is indeed a preferable solution 👍


Add expenseId to TransactionType to add link to individual expense ge from all transactions view

This pull request adds expenseId to Transaction type.
this pr to close this issue : https://github.com/opencollective/opencollective/issues/1590

Seems to me it's not following the specs from @Betree

I think it's a good idea to introduce transaction.expense as @Betree  suggested. Exposing transaction.expense.id is more elegant and future proof than transaction.expenseId.
I'm not sure for the Dataloader, getExpenseForViewer does an important job at filtering the information. This is required and I'm not sure if that is appropriate to have this behavior in a Dataloader.

@znarf I updated my pr we now have all the expense object filtered through getExpenseForViewer. I suggest wrapping category,attachment,privateMessage in the new expense field, what do you think?

I suggest wrapping category, attachment, privateMessage in the new expense field

In theory yes, but in practice we need to keep compatibility, so it it should stay like that for now.
We can start trying to get rid of it in the frontend, but this should be another PR.

@znarf Working on it.


Cannet get my Gmail mails

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
K-9 should be able to fetch Gmail emails just like Thunderbird does (proposing a first time authentication popup to allow Thunderbird to read the mail).
Actual behavior
Authentication failed for .....@gmail. Update your server settings.
Steps to reproduce
1.Setup a Gmail account
2.Read the Gmail Inbox
Environment
K-9 Mail version: 5.600
Android version: 5.0 on Galaxy Note 3
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
Please take some time to retrieve logs and attach them here:
Can you please update your wiki for Android devices that don't have the pgrep tool like my Galaxy Note 3 :
$ adb shell pgrep k9
/system/bin/sh: pgrep: not found
$ adb shell ps | grep k9
u0_a241   4793<PHONE_NUMBER> 125896 ffffffff 00000000 S com.fsck.k9

Here is what log I get :
$ adb logcat -d --pid=4793 | tee k9-log_issue#4099.txt
- waiting for device -

Therefore, the file k9-log_issue#4099.txt is empty :
$ du -b k9-log_issue#4099.txt
0	k9-log_issue#4099.txt

Isn't that a duplicate of https://github.com/k9mail/k-9/issues/655 ?