Create LICENSE

I said (c) Monarch Initiative, if that is OK with everybody?

@pnrobinson Peter, I think it is better if we assign Justin, Deepak and Tiffany to review this pull request.


Modify X-Rays to be reusable in other apps.

Pulls out backend code into lib.rs.
Adds ability to mount with a prefix.
Some simplifications in the client.

@wally-the-cartographer merge

Merge requested by authorized user SirVer. Merge queue now has a length of 1.


fix(worktree): didn't show git branch inside worktree directory

I've noticed that vim-gitbranch plugin didn't show the git branch name inside of worktree directory. And I found out that the cause was  the path inside of .git file or to be precise the gitdir content inside of .git file already give the full path, so the return value when .git is a file is something like this:
/home/username/worktree-directory/home/username/git-repo/.git/worktrees/worktree-directory/HEAD

which doesn't exist.
I omitted the path so that this only return the content of gitdir. Please let me know if there's a case of gitdir doesn't have a full path to the HEAD file, I only test this with git worktree after all.
My git version by the time of this PR: 2.34.1

The content of .git in submodule is normally a relative path to the file in the root git repository. The content of modules/oniguruma/.git in https://github.com/stedolan/jq is gitdir: ../../.git/modules/modules/oniguruma.

I see, I'll close this for now. I'm still not sure how to handle absolute path and relative path. Thank you for your response!


You don't have sufficient permissions to use the requested API

When I click on the Click here to create a new project with the following API's enabled link in the documatation I get the error
You don't have sufficient permissions to use the requested API
I am a superadmin.  Any workaround to this?

Pretty sure the link is always broken. Have you tried manually creating a project and enabling the APIs?


Feature/2/footer

Engineers Checklist
What this PR does?

List of updates for features, enhancements, or bugfixes

How can it be tested?

Please provide step-by-step guide on how to test
If web or mobile, screenshots will be very helpful in addition

Where is it documented?

List JIRA tickets

Backend Updates

[ ] Updated Postman Documentation

[ ] Requests
[ ] Examples
[ ] Tests

[ ] Updated Tasks on JIRA
[ ] Tested API updates in dev environment
[ ] CI Checks Passed

Worked on the alignment


Improve documentation: Update Readme and create a wiki section (2022.1)

Describe the issue
Missing installation process details for 2022.1
Expected behavior
We are expecting to see a clear explanation about the installation process of 2022.1 and avoid confusions with 2021.4, for example, the kernel environment and so on. Could you (@adrianboguszewski and @Debskij) help us with that?
Please Update the readme file and correlated installation files if it is necessary. Could you create a wiki page with a description of this process?

Is this related to #468?

These are tasks that should have been complete before merging the 2022.1 to main. We can discuss on Wednesday.


switch main development branch to master

Changing main development branch to master before release.
After initial release, development will be more git-flowy

Do you have an ETA for the first release yet?

I don't... my side-project has taken more time than I originally thought.
Definitely before the end of the year... but I want to target something closer to the start of December.
I originally thought I could release something this month, but life got in the way :roll_eyes:.


Update docs to support 1.22

Issue #, if available:
Description of changes:
Bring in changes from #1560 and launch 1.22 on the website
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold


fix: windows issue

Explanation
Fix windows issue.
Related issue
Fixes #1140

/cherry-pick release-0.2


feat: esbuild 0.20.x

Upgrades esbuild to ^0.20.0. Please not that according to the support policy, minor version upgrades of esbuild might include breaking changes. See the esbuild release for details: https://github.com/evanw/esbuild/releases/tag/v0.20.0
To stay on a different version of esbuild, see https://github.com/mrgrain/cdk-esbuild?tab=readme-ov-file#tsjs-how-can-i-use-a-different-version-of-esbuild

@mergifyio backport v4


Need delay setting for showing all Intellisense and popup windows

Type: Feature Request
VSCode would have a MUCH better user experience if all Intellisense and related editing pop ups would have and honor delay setting.
It is SO distracting and frustrating to have to keep pressing the Escape key "all the time" to dismiss these popups.  95% of the time, we developers are just typing our code and do not need a popup every 5 seconds while typing.
PLEASE greatly improve the usability of the product.
Even with the below settings my editing experience is always poor because I have to keep pressing Escape just to see my code!
"editor.hover.delay": 500,
"editor.quickSuggestionsDelay": 2000,
"errorLens.delay": 3000,
"workbench.hover.delay": 1000,
This happens when I use VSCode for C#, Dart and Flutter
VS Code version: Code 1.84.2 (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:51:52.184Z)
OS version: Windows_NT x64 10.0.19045
Modes:

These settings will help you: editor.quickSuggestions and editor.suggestOnTriggerCharacters

@jrieken this didn't help an did not address the request.
Why did you close this?

this is NOT a question.  This is a feature request.

/open
Software Devloper
Contact if you have any questions

From: Long Time Developer @.>
Sent: Friday, November 24, 2023 11:57:39 PM
To: microsoft/vscode @.>
Cc: Subscribed @.***>
Subject: Re: [microsoft/vscode] Need delay setting for showing all Intellisense and popup windows (Issue #198905)
this is NOT a question. This is a feature request.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode/issues/198905#issuecomment-1825495719, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXPWG62SS4JA3TTJ5IQNYN3YGB4SHAVCNFSM6AAAAAA7XD4MIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVGQ4TKNZRHE.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


nixos/nginx: add commonServerConfig option

Description of changes
My usecase is that I want to set a common set of headers for all server without repeating them or knowing which virtual hosts exist. This basically mirrors virtualHosts.extraConfig without setting it on every virtualHost
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

The NGINX module is too complicated to provide simple ad-hoc options IMHO, I'd accept an approach where you would have the capability to attach "common server config" to a group of servers rather than all servers.

If you know what you are doing then it isn't, for example I am doing:
commonServerConfig = ''
  more_set_headers "Server: Internal Server Error";
'';

commonServerConfig."*" = {...}; and for example, virtualHosts.partOfGroup = "*" or something.

How isn't that making the module even more complicated? Also that is not solving my problem where I want to attach some common config to every virtualHost without touching every virtualHost.

The NGINX module is too complicated to provide simple ad-hoc options IMHO, I'd accept an approach where you would have the capability to attach "common server config" to a group of servers rather than all servers.

If you know what you are doing then it isn't, for example I am doing:
commonServerConfig = ''
  more_set_headers "Server: Internal Server Error";
'';

Yeah, but if you know what you are doing: we should rather consider discussing how to make the NGINX module extensible for power users rather than having all power users keeping go at it.
Otherwise, it will just become increasingly inscrutable without proper planification and discussion on how to make it grow, you were not interested into the maintenance of NGINX, so I don't think it's responsible you just add stuff without caring about the long run.

commonServerConfig."*" = {...}; and for example, virtualHosts.partOfGroup = "*" or something.

How isn't that making the module even more complicated? Also that is not solving my problem where I want to attach some common config to every virtualHost without touching every virtualHost.

This is generic in the subsets of virtualHosts you want to reach, you can have a special string to have the subset := all virtual hosts.

Yeah, but if you know what you are doing: we should rather consider discussing how to make the NGINX module extensible for power users rather than having all power users keeping go at it.

The only option that comes straight into mind is to convert it to freeform settings but based on my experience with the rather picky and slightly rusty config parser is nginx that will not be that easy.

Otherwise, it will just become increasingly inscrutable without proper planification and discussion on how to make it grow

That sounds like RFC level planning and after the last few ones that went nowhere after closing to one year, I am not interested in that.

you were not interested into the maintenance of NGINX, so I don't think it's responsible you just add stuff without caring about the long run.

Somewhat but not so much to majorly refactor it because it will break and debugging won't be easy.

This is generic in the subsets of virtualHosts you want to reach, you can have a special string to have the subset := all virtual hosts.

Do we have something similar anywhere else? I can't think of something right now. Maybe we should just convert everything under virtualHost into freeform? Maybe that is more manageable?

@RaitoBezarius I just tried to use nixos-modules with a friend that isn't using my nixpkgs fork and he ran into this option being undefined.
I can't think of way to rewrite this without causing an infinite recursion, since I need to access the attrNames of virtualHosts before I can fully evaluate the inside of it.
That's the diff I tried:
diff --git a/modules/nginx.nix b/modules/nginx.nix
index 49d8904..55647d3 100644
--- a/modules/nginx.nix
+++ b/modules/nginx.nix
@@ -99,10 +99,6 @@
           zstd_types application/x-nix-archive;
         '';

-        commonServerConfig = lib.mkIf cfg.setHSTSHeader /* nginx */ ''
-          more_set_headers "Strict-Transport-Security: max-age=63072000; includeSubDomains; preload";
-        '';
-
         package = lib.mkIf cfg.quic.enable pkgs.nginxQuic; # based on pkgs.nginxMainline

         recommendedBrotliSettings = lib.mkIf cfg.allCompression true;
@@ -157,7 +153,11 @@

               quic = lib.mkIf cfg.quic.enable true;
             };
-          };
+          } // (lib.mkIf cfg.setHSTSHeader (map (virtualHost: {
+            virtualHost.extraConfig = /* nginx */ ''
+              more_set_headers "Strict-Transport-Security: max-age=63072000; includeSubDomains; preload";
+            '';
+          }) (lib.attrNames cfg.virtualHosts)));
       };
     };

If you have an alternative to this PR, please let me know.

Didn't know that you could extend submodule type options that easily. Solved my problem differently https://github.com/SuperSandro2000/nixos-modules/commit/fc5c44de21d1695e726f92be53d6d9ed64971575


runtime: TestCgoExecSignalMask flaky on s390x builder due to EAGAIN from fork/exec

Seen a few times on the s390x builder:
https://build.golang.org/log/a1f9b5a3f3653242254a132003fbe000ce7ab8df
https://build.golang.org/log/8c8066abee8ba3c074a55593f936c4cff930f0b8
https://build.golang.org/log/a1c7bfaac1333926ec21f1ecbcc605872b582b99

Let's just ignore EAGAIN.  I'll send a CL.


Resolve peer dependencies from current project

🔍 Search Terms
peer local file
✅ 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 isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
When a dependency is specified in a library's peer dependencies, and it is installed in the current project, it should be resolved from the current project's node_modules, not from the library's node_modules, when file:// is used for testing the library locally.
📃 Motivating Example
Say you have a project "test" and a library "foo". "foo" has "react" in its peer dependencies, "test" has react and @types/react installed. With this change, you can now test "foo" in "test" with file://../foo without needing to delete its node_modules.
💻 Use Cases

What do you want to use this for? Library development
What shortcomings exist with current approaches? It leads to incorrect type errors
What workarounds are you using in the meantime? Deleting the library's node_modules

I tested that this doesn't work with typescript@next

I'm not quite sure what you're getting at here. Some code examples might help?


Secret value with binary data shouldn't be show as input & only show size

Steps:

Go to Secret: Create
Add from file
Select a small image file

Results: Value is trying to display info.  We shouldn't try to display binary data, only the size of the file.  Not sure if my data is even being saved properly in this case now?

Fixed


Made Friend.presence_type optional

Friend.presence_type only gets returned if the user is online.

pulling this in for now although I want a better solution down the line


[TASK] Update Wallet Address component & add to  UI Kit

Description
To ensure consistency with the update of the Chain Connector component, please update the Wallet Address component according to figma file. We should add this component to webb UI kit as it is repeatedly used across applications.l
Figma Link
Screenshot

Closed on https://github.com/webb-tools/webb-dapp/pull/1489.


Add test to check that a base user can stop their server

Signed-off-by: Luca Giorgi<EMAIL_ADDRESS>

test: https://opendatascience-jenkins-csb-rhods.apps.ocp-c1.prod.psi.redhat.com/job/rhods-execute-tests/105/


Remove unnecessary/incorrect serial getty service enabling in marketplace images

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Remove enabling of<EMAIL_ADDRESS>in marketplace images
Change Log

Remove enabling of<EMAIL_ADDRESS>in marketplace images
Does this affect the toolchain?

YES
Associated issues

#6207

Links to CVEs
Test Methodology

Pipeline build id: https://dev.azure.com/mariner-org/mariner/_build/results?buildId=423366&view=results

merged via #6233


Add DropwizardMockitoMocks

Add DropwizardMockitoMocks and test
Add DropwizardMockitoContext, used as a return value in
DropwizardMockitoMocks#mockDropwizardEnvironment

Question: I could not think of a decent name for the method that returns a DropwizardMockitoContext so it is just named mockDropwizardEnvironment as opposed to just mockEnvironment for the one that returns the Environment. Can you think of a better name?
Fixes #31

So the mockDropwizardEnvironment is really setting up the context correct?  Couldn't you name it mockDropwizardContext instead of environment?

So the mockDropwizardEnvironment is really setting up the context correct? Couldn't you name it mockDropwizardContext instead of environment?

Both of them set up a completely mocked Environment. The only difference is that the latter returns the "context" object which just makes it easier to grab out individual pieces, e.g. the MetricRegistry. And the context object is our object, not something from Dropwizard. But mockDropwizardContext is probably a better name, though it seems to imply that there is an official DropwizardContext, when in reality there isn't.

So the mockDropwizardEnvironment is really setting up the context correct? Couldn't you name it mockDropwizardContext instead of environment?

Both of them set up a completely mocked Environment. The only difference is that the latter returns the "context" object which just makes it easier to grab out individual pieces, e.g. the MetricRegistry. And the context object is our object, not something from Dropwizard. But mockDropwizardContext is probably a better name, though it seems to imply that there is an official DropwizardContext, when in reality there isn't.

What about mockDropwizardPlumbing haha or just mockDropwizard?

I don't like that GitHub doesn't seem to have a simple reply in thread mechanism...anyway. Yeah, naming is hard. I am letting this marinate while doing other things, so maybe a decent name will percolate to the surface...

I went with your mockDropwizard suggestion 🤘


[SPARK-25200][YARN] Allow specifying HADOOP_CONF_DIR as spark property

SPARK-25200 - https://github.com/apache/spark/pull/22289
Replicates the behaviour of setting HADOOP_CONF_DIR, but as a spark config spark.yarn.conf.dir

Going to repush to this repo so that tests run

Thanks for the swift review @robert3005 ! I made the changes


triger an alert even it finds 0 hits

Hi,
is there any possible to generate an alert even it finds 0 hits or 0 matches.
because even there is no traffic i want to generate an alert
Thanks.

Yelp/elastalert is no longer maintained. Please use jertel/elastalert2.
https://github.com/jertel/elastalert2


Zio json

Add support to zio-json json library - Fix #395

Thank you!
I'm extending your PR to trigger CI and add some doc.
https://github.com/jwt-scala/jwt-scala/pull/396

I have activated CI in a different branch and fixed a few issues, could you create some doc in the following directory?
https://github.com/jwt-scala/jwt-scala/tree/master/docs/src/main/mdoc
Documentation to test your doc locally is here: https://github.com/jwt-scala/jwt-scala/blob/master/CONTRIBUTING.md

ok I'll do it in a few days, thanks for the links

#398 fixed

thank you!


Form is not destroyed when updating reduxForm({name})

Are you submitting a bug report or a feature request?

bug report
What is the current behavior?

If you type in the sandbox when this.state.form is article, you see the content in the input.
When you click the button, you change the form property passed to the reduxForm() decorator, basically you change from one form to another.
Type to the input again.

The field's this.context._reduxForm property doesn't have the updated context with the new form name, and in the redux devtools it's visible that the form is not destroyed after the update.
What is the expected behavior?
I expected that the field's context is updated along with the form's context. The form should be destroyed, because the form's (dynamic) name is changed.
Sandbox Link

https://codesandbox.io/s/mj7xjyw5yx
What's your environment?

react: 16.3.2
redux: 4.0.0
redux-form: 7.4.2
Other information

I haven't found any related issue to this particular one. I of course found ones with dynamic form names, but it's more about how user can set the form name dynamically, not about forms not working when updated. I might miss something, any help is appreciated.

The form still reads value from corresponding from correctly; using redux devtool to dispatch:
{
  type: '@@redux-form/CHANGE',
  meta: {
    form: 'vacancy',
    field: 'title',
    touch: false,
    persistentSubmitErrors: false
  },
  payload: 'Test Value for vacancy form'
}

yield a change in vacancy form.
The problems are:

the form doesn't update its action creator(or whatever) and still dispatch using its initial form name
the form doesn't update to read value when form name prop change.

I've seen the same behaviour, even when calling using destroy() [and confirming that the @@redux-form/DESTROY is actioned]. The only way I can move from one form to another is to put my SPA into a state where no form is displayed temporarily.


gh-450: Workaround for failing ASAN on macOS

There already are a bunch of other macOS workers, and this is the only one where we're seeing this issue, so I assumed it's some machine-specific issue and not something general to all macOSes.
I don't think there's a downside to setting the env var at the builder level, but that would also be something that represents a divergence from most users' setups.

I don't think there's a downside to setting the env var at the builder level, but that would also be something that represents a divergence from most users' setups.

The env var is only set for ASAN buildbots, which is not a typical user setup.

I don't think there's a downside to setting the env var at the builder level, but that would also be something that represents a divergence from most users' setups.

The env var is only set for ASAN buildbots, which is not a typical user setup.

right. I can bring it back then :)


RE: /installing-airbrake/installing-airbrake-in-a-rails-application

The link at at the bottom of the page goes to a 404. I'm trying to find out if there is a config option of some kind I can put into a app.json file so review apps get Airbrake installed in review apps, but don't send error emails. I just want a place to store the error so I can see the context for each error without combing through the logs.

Hey @nextekcarl thanks for pointing that out. We'll update the doc.
About what you're trying to achieve, I think I need more info. What kind of application is this? eg: Rails, JavaScript, PHP, etc
If you want to report errors to Airbrake but not get error emails sent to you, you can change your email notification settings. No extra setup would be required too, you'd just install Airbrake normally and tweak your email settings separately.

Thanks.
It is a Rails app using the Heroku Pipeline. We have staging and production set up and sending emails currently. But Review Apps are throw away environments that are set up based upon the settings in he app.json file. What I'd like is a way to set up Airbrake:free when we set up a Review app (which happens automatically when we open a PR) but I don't want the emails to be sent out for that review app (it goes to the account that owns the Heroku Pipeline). Since it is set up automatically, the only fix I can see right now is to manually turn off emails notifications every time we open a PR and a Review app gets built as the default behavior is to send those notifications.
Are there any options we can add to the app.json file to control the notifications of a Review App when it is set up?

There is no way to change your personal user notification settings via an API or our gem. The only current way is via the web interface.
I'd be happy to log it as a feature request though - let me know.

That would be really useful, thanks for the quick response.

Certainly. Please let us know if you have any other suggestions or questions. You can always reach us at<EMAIL_ADDRESS>


Incorrect version indication on Windows platform

The latest v0.3.0 on Windows -

Not more reproducible for new beta version.


Highlight Full Name when registered as a nickname

Is your feature request related to a problem? Please describe.
If a character's display name is a subset of a nickname, only the display name will be highlighted. This appears to only be the case when whitespace is used in the nickname. For example, the display name is just the first name, but the character's full name is used as a nickname to ensure it is also highlighted. See below example.
Describe the solution you'd like
I would like Dialogic to use a greedy Regex when searching for nicknames or display names. This would allow the longest available name to be selected even when it contains a shorter name that is a subset of another.
Describe alternatives you've considered
I have considered using the Display Name to contain the full name, but this looks very awkward when a character's full name is not how a player or their avatar would likely refer to a character. Just feels oddly formal.
Additional context
Example:

Display Name: John
Nicknames: John Smith, Johnny
Result: Hello, I am John Smith. Call me Johnny.
Expected Result: Hello, I am John Smith. Call me Johnny.

Hey, thanks for the report.
I will look into this, it's a regex issue, and possibly also an issue of sorting these options by length first.

If you point me to the correct area to look at, I'd be more than happy to look into this and make a contribution.


SST by day original: how to display / transform?

Hi Kelly,
The original data for the sea-surface temperature sst icon in icons to data - Google Sheet was by day with SST_min and SST_max:
SST_1985-2019_FK_CRW_NOAA - Google Sheets

Date
SST (Min)
SST (Max)

1/1/1985
24.41
25.15

1/2/1985
24.12
25.13

...

We have a couple questions about this:
Q1. Similar to all other indicators, do you want this summarized by year? If so, how? For instance, we could use the Min, Max and Mean (Mean_SST = AVG(Min_SST,Max_SST)) for the entire year and plot this as 3 lines in the figure. @superjai already summarized this data here:
SST_BY_YEAR_1985-2018_FK_CRW_NOAA - Google Sheets

Year
Min SST
Mean SST
Max SST

1985
25.67
26.39
27.11

1986
25.79
26.52
27.25

...

Q2. Do you want to keep this SST indicator by day? If so, do you want to plot one (ie Mean_SST = AVG(Min_SST,Max_SST)) or more like already here:
marinebon.org/fk-esr-info/sanctuary-waters.html

Kelly would like in single figure daily with Min, Max; not Avg.


add LuaJIT port

@pgrabas,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@pgrabas, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

I forgot to add, LuaJIT port conflicts with existing Lua port, so one should not have installed both.

The change looks good, but the patch does not apply correctly now (git and vcpkg say that it did, but src/msvcbuild.bat is only partially modified) and static build fails. Could you regenerate it once more?

I have regenerated the patch. Applying now works as should.

LGTM, thanks for the PR and thanks @codicodi for the review!

Is there a way to tell vcpkg to compile LuaJIT as C++? @pgrabas @ras0219-msft @codicodi
Thank you for your time.
P.S.
I cannot thank you guys enough for this entire project!

I don't think there is any way to do it without modifying build scripts. Do you have any issues with interoperability with c++?


created item readers

Current coverage is 13.85%

Merging #29 into master will decrease coverage by -5.09% as of 16df0be

@@            master     #29   diff @@
======================================
  Files           54      62     +8
  Stmts          586     801   +215
  Branches        20      41    +21
  Methods          0       0
======================================
  Hit            111     111
  Partial          0       0
- Missed         475     690   +215

Review entire Coverage Diff as of 16df0be

Powered by Codecov. Updated on successful CI builds.


docs: OT1 docs are not marked as being only for OT1

overview
If I perform a Google search (as many people will) for "opentrons pipette" I get as the top API result a page entitled "Liquid Handling — Opentrons API v2.0" which I would expect to be what I want to use for the OT2. In fact it is the OT1 API, but the only indication of this is in the URL: https://docs.opentrons.com/ot1/pipettes.html
expected behavior
Put a big warning on all these pages that they are for the OT1 only.

@theosanderson and other users: I also stumbled across this confused google result. For people who are looking for the API v2 (BETA, may be subject to change, currently for the OT-2 robot), see this documentation: https://docs.opentrons.com/v2/new_protocol_api.html .

Note that even though this ticket is marked as closed, we probably won't deploy the docs change until the next release so we don't accidentally deploy documentation only valid in e.g. 3.16.0 and confuse everybody.


Use iteration for callsign generation - Fixes #466

Fixes #466
Use iteration rather then recursion for callsign generation.  There is no stop condition in case all callsigns are used so it's possible this would cause a freeze.
I think this also fixes a bug where duplicate callsigns were added to the list of callsigns.

Works! :+1: It does eventually cause that freeze, but I added a line to remove the callsign from prop.aircraft.callsigns when an aircraft is removed, so that callsign is made available to subsequent new aircraft.
Combined, it eventually still gets gunked up, but presumably just from there being like 500 departures waiting on the ground for taxi. lol
On my system, it now runs for about 58 real-time hours before dying. Somehow I have a feeling that will be sufficient... :laughing:


Displaying Map on full screen

UI Enhancement to Map:
Displaying Google Map on full screen, with 'Public Dataset'  button as an overlay. An info icon(i) will be
displayed at the top right corner, which tells user what green trail and purple circle represents.

Please assign this enhancement to my name.

This would be a good enhancement, however, we do not have this feature as part v1.0 of safepaths release. So, it would be great if you can start adding this feature here Private Kit Repo

@tremblerz

we do not have this feature as part v1.0

If the code is already done and working, I don't see a reason to not include it.
@ArchanaNair
Please provide a screenshot (as requested in PR).

New PR raised for this after updation in latest UI,
PR No:
https://github.com/PrivateKit/private-kit/pull/12

Maps are no longer part of Safe Places.


[Feature Request] Floating Legend In Chart Area

I am considering using Chart.js because I have found it very simple to use and the ease of integrating live  customization with events and the '.update()' function. The one exception is the legend seems like it only can form itself on the borders of the chart area. You can't really specify a position and have it form inside the chart area like most other graphing utilities.
I understand there could be some overlap and z-order issues with onHover/onClick events that could probably be the reason it hasn't been tackled yet. I think i'm going to try tackling this a bit this weekend, but curious if there is a real reason it hasn't been implemented yet.
Also: Here is a really basic plugin i wrote to give background color customization (background and chart area). Is there really no innate ability to do this?  I'd strongly consider adding something similar also since this is clearly needed core functionality if the chart is converted to PNG or copy & pasted where it could be converted to 'rgb' from rgba. Plus a '0,0,0,0.0' default just seems bad when most charts have white backgrounds so would expect a default of '255,255,255,0.0'.
    //Background color plugin
    Chart.pluginService.register({
        beforeDraw: function (chart, easing) {
            var ctx = chart.chart.ctx;
            var chartArea = chart.chartArea;

            ctx.save();

            //Fill entire canvas
            ctx.fillStyle = chart.config.backgroundColor || 'rgba(255,255,255,0.0)';
            ctx.fillRect(0, 0, chart.chart.width, chart.chart.height);

            //Fill chart area
            ctx.fillStyle = chart.config.options.chartArea.backgroundColor || 'rgba(255,255,255,0.0)'
            ctx.fillRect(chartArea.left, chartArea.top, chartArea.right - chartArea.left, chartArea.bottom - chartArea.top);

            ctx.restore();
        }
    });

@ChaseLewis the z-order and hit test issues are the likely reason that this has not been tackled yet.

I want this a lot

I'm also looking for this at the moment, doing some work where the design brief has the example of the legend floating in the top-right over the chart area itself.
I suppose it could be done with the html legend stuff, but some built-in support would be grand.

https://www.chartjs.org/docs/latest/configuration/legend.html#position

When using the 'chartArea' option the legend position is at the moment not configurable, it will always be on the left side of the chart in the middle.


UserDefaults observation not trigger

Hi ! I just stumbled across your post and tried your project since I use the same technique to observe user defaults changes in our app and it's not working anymore.
So I downloaded your project, ran it and nothing was outputted. Although I think the two lines you mention should appear:
Changed from: Optional(false) to Optional(true)
Changed from: Optional(true) to Optional(false)

I wonder if it's something general for all apps with UserDefaults. To let you know, when using a key initial when registering the observer the initial value is properly received. But that's the only one.
 I would love to have your thoughts on that one.

False alarm, there is just a missing assignment on Storage.isFirstLaunch in your implementation to output something at launch.


[iOS] Handle error when fetching thumbnail

One of my videos can't generate a thumbnail and it causes the error Domain=PHPhotosErrorDomain Code=3303, resulting in fetchThumb never calling back.
We should handle this case probably the same way as when imageData is nil.
I'm attaching the video I have issue with.
https://github.com/fluttercandies/flutter_photo_manager/assets/11507974/b2d3d444-072a-4e50-8060-ef61e172fe88

Your video only lasts 2 seconds and seems unrelated.

This is the video that I can't fetch thumbnail from. Add it to your library and call fetchThumb on it, you should get an error and no callback.


When a test fails, the stand alone web driver process is not shut down

In our project we want to run all e2e tests, create a report and show it to the developer with all the successful and failed tests. We find this gulp task super useful but we are having trouble with one thing. When any test fails, the web driver process does not get terminated and the gulp task never ends. To mitigate this we're using this hack:
Sample gulpfile.js
var gulp = require('gulp');
var angularProtractor = require('gulp-angular-protractor');

gulp.task('run-e2e', function(callback){

    /* Hack to get gulp-angular-protractor to terminate the web driver process even if a test fails */
    var webDriver = require('../node_modules/gulp-angular-protractor/gulp-angular-protractor/web-driver.js');

    gulp.src(['./e2e/**/*.feature'])
        .pipe(angularProtractor({
            'configFile': 'protractor.config.js',
            'args': ['--baseUrl', baseUrl],
            'autoStartStopServer': true
        }))
        .on('error', function (e) {
			/* Use same url as in config */
            webDriver.webDriverStandaloneStop('http://localhost:4444/wd/hub', callback);
        })
        .on('end', function () {
            callback();
        });
});

gulp.task('create-e2e-report', ['run-e2e'], function() {/* omitted */});
gulp.task('open-e2e-report', ['create-e2e-report'], function() {/* omitted */});

Looking at the source (gulp-stream.js line 63) it looks like this is by design so there might be a use case that I don't understand. Would it be possible to always stop the web driver process, even if exit code > 0, or will that cause other issues?
Thanks for the great work!
Regards
Jesper

I am having a different weddriver issue. Not sure, could turn out to be a misconfiguration on my part... I have chromedriver setup globally on my dev machine but while trying to run the tests; I get the following error:  ```
.../node_modules/protractor/lib/driverProviders/local.dp.js:42
throw new Error('Could not find chromedriver at ' +
^
Error: Could not find chromedriver at .../node_modules/protractor/selenium/chromedriver.exe
at LocalDriverProvider.addDefaultBinaryLocs_ (.../node_modules/protractor/lib/driverProviders/local.dp.js:42:15)
at LocalDriverProvider.setupEnv (.../node_modules/protractor/lib/driverProviders/local.dp.js:59:8)
at Runner.run (.../node_modules/protractor/lib/runner.js:335:31)
at process. (.../node_modules/protractor/lib/runFromLauncher.js:32:14)
at emitTwo (events.js:87:13)
at process.emit (events.js:172:7)
at handleMessage (internal/child_process.js:686:10)
at Pipe.channel.onread (internal/child_process.js:440:11)
[launcher] Runner Process Exited With Error Code: 1
[13:08:40] gulp-angular-protractor - Webdriver standalone server is stopped
I can run `protractor` on same directory with  any issues. A hack would be to copy all requirements (chromedriver?) to the folder it's expecting it but I don't think that would scale... Did you encounter same problems? @jesperoastrom

Bernard.

@jesperoastrom
I don't remember this case: maybe when the plugin shall not start (and so stop) automatically ?
I will fix it
Many thanks for your feedback

@ojengwa
Can your create a separated issue with a concret example (a zip file with your configuration)
Many thanks!


unsupervised arg in training_utils.train confusing

https://github.com/deepinv/deepinv/blob/c5532cf903df85cd396f9ca629e101cbb12bb6fb/deepinv/training_utils.py#L73
When training in an unsupervised fashion (e.g. with loss.MCLoss or loss.EILoss for doing Equivariant Imaging), but using a dataset with paired measurements and ground truth for doing evaluation, unsupervised should be set to False in order to load the ground truth, which is a bit confusing.
May I suggest renaming unsupervised=False to measurements_only=False or flip the bool and name it paired_measurements=True?
Thanks

To be fixed in #147

Resolved in #181


Update UG (3 of ?)

Closes #230

Codecov Report

Merging #266 into master will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #266   +/-   ##
=========================================
  Coverage     82.80%   82.80%
  Complexity     1564     1564
=========================================
  Files           139      139
  Lines          4101     4101
  Branches        537      537
=========================================
  Hits           3396     3396
  Misses          598      598
  Partials        107      107

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/chopchop/ui/MainWindow.java
78.57% <100.00%> (ø)
9.00 <0.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 875dee1...9bf22f9. Read the comment docs.


Change "Undo" Feature Algorithm

While working on the "Hint" feature I added an "undo_move" method to the Board class. The Game class should be updated to use this method to undo a move, rather than storing a copy of the Board instance after every move, which requires more memory and time.
The parameters for "undo_move" are the peg, jump, jumped peg, and peg's original location. In order to know these last two parameters, we should add a stack data structure to store each move as an attribute of the Game instance. Then to undo, we just pop from this stack and pass the move to board.undo_move
STEP 2
After completing this, I can use the moves stack to compare the users move against an optimal solution that is calculated when user requests a hint. Since this method can be slow (if there are many pegs on the board), it would be nice to prevent unnecessary calls to it. Now that we are tracking user moves, we can compare each move to the optimal solution, and if they match, we don't need to call "solve" again...just the next move in the solution.

Undo feature has been updated and is working.
Next Step: Step 2 (see above)

Implemented feature to store results of an optimal solution when it is obtained, and then maintain it for as long as the user continues to follow the list of optimal moves. Tested and it is working: it does not make a new call (and therefore doesn't take any time) to display a hint as long as you continue to follow the hints
Closing this issue


Appstudio update test-component-pac-nilq

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-nilq-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
16 seconds
init

✅ Succeeded
35 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
24 seconds
inspect-image

✅ Succeeded
26 seconds
deprecated-base-image-check

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
34 seconds
sbom-json-check

✅ Succeeded
18 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
10 seconds
show-summary

Pipelines as Code CI/test-component-pac-nilq-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
26 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
10 seconds
inspect-image

✅ Succeeded
11 seconds
deprecated-base-image-check

✅ Succeeded
49 seconds
clamav-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
19 seconds
label-check

✅ Succeeded
8 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


BUG: python - c “import pandas” crashing python.exe on exit

I usually run my python code in an interpreter. There running import pandas works fine and I have been able to use pandas functions regulary.
However, I am not able to use pandas from cmd line: for instance the below crashes python after half a second (it crashes when exiting):

python -c "import pandas"

This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.
I can import other modules fine this way. Any idea how to solve this? Pandas is the only one with this behavior.
My config is: Win32 / Python 2.7.5 / pandas 0.12.0 / GCC 4.7.2 (I also tried 0.13.0 dev and experience the same issue)
See stackoverflow question for more info: http://stackoverflow.com/questions/18293838/python-c-import-pandas-is-crashing-python-exe-on-exit

I am encountering this issue with Pandas 0.18.1 on Python 2.7 32-bit for Windows.  I am installed via pip, and have tried uninstalling, reinstalling, and upgrading.  @lorifan, are you also installed via pip?

@superlou yes, I am using pip to install pandas. I would suggest to discard your current setup and re-install everything from scratch. That's how I got around it in the end.

Seems like an inelegant fix, but I uninstalled all of Python 2.7 as much as possible on Windows, then reinstalled all packages using unofficial builds, and it seems like the error is went away.


fetch tags via commitId

Note: Please use sites such as http://stackoverflow.com for general questions about JGit usage, not issues in this project. Issues should be used for problems with snippets and suggestions of missing snippets. Snippets from good answers on stackoverflow can then be included here, naturally.

No actual comment, just the issue-template, closing for now as it seems to be spam/miss-clicking.


v0: rename package to pyuavcan_v0

Do not merge until tested.

@pavel-kirienko Not sure what's wrong here, the twine upload is giving a 403 forbidden - is there a separate token/password for each pypi package that you need to update?

is there a separate token/password for each pypi package that you need to update?

Yes, that's the preferred way because it securer. However, your new configuration will create a new package becase currently there is no package named pyuavcan_v0, so I guess we need an unscoped token for that. I just created one:
pypi-AgEIcHlwaS5vcmcCJDQ4NTllNThjLTg2OTQtNDdkMS1hZDQ3LWFlOTU2MzkzNjUwMwACJXsicGVybWlzc2lvbnMiOiAidXNlciIsICJ2ZXJzaW9uIjogMX0AAAYg_7YgINwm7mj9Dm7E8dijLWWgqk71iSZkSuG7t0IkCTY

Please tell me your PyPI username so that I could add you as a co-maintainer there. After the first package is published we should replace that unsafe token with a constrained one.

AppVeyor looks sad. I think it needs an .appveyor.yaml with build: off in it.

Ok then. I added back the original appveyor (which is already disabled).

Nice. Let's wait for the GUI tool test and then it should be good to merge.

It seems that the dsdl_files directory isn't being installed with the package.

Maybe the repository is not cloned recursively?

You're right. I cloned recursively for the tests but not for the publishing job!

Alright. @pavel-kirienko The gui tool boots up now: https://github.com/UAVCAN/gui_tool/pull/51
(I published a new version, 1.0.0.dev34)


chore: update to 1.0.0 beta.2.2

Update test suite to include excplicit capabilities

wrong repo , sorry


Total Number of Combinations

I want to add the function that gives number of combinations for picking x objects at a time from y different objects.
Sample problem:- Number of combinations for picking 4 objects at a time from 6 objects.
Solutions:- There are 15 different combinations available.

Good idea, I've assigned it to you.

@Ritu1611 There should also be a permutations problem generator. Would you like to pick that up as well or should I add it as an issue for somebody else?

I will do that. I was thinking to link this with probability problems. I have already added the function and a pull request can you see that.

@lukew3  I am new to this. Can you tell me whether I am done with this combinations function or else I have to do something more.

Sorry, I just forgot to close the issue. Your combinations pr was good.


Fix cupyx.scipy.sparse._base tests for SciPy 1.11

Part of #7664.

/test head

LGTM! Remaining head test failures will be fixed in separate PRs.


Defined service for ubuntu

I am sorry about the mess of commits. I want to know what tool are used to verify the code.

I use puppet-lint to do some basic code syntax and quality checking.

Ok, Thank you.


Update Help link once Help pages are posted on WordPress site

Once it is ready and live, we should update the link in the header that says, "Need help making a submission? Read our submission workflow instructions ." since those instructions seem pretty outdated and not very helpful for users.

This issue depends on https://github.com/eclipse-pass/main/issues/690

Will update this ticket once I have the WordPress site link

Moving this to blocked, waiting for link from wg.

Link is confirmed: https://pass.jhu.edu/guide

instructions link updated in stage and prod to https://pass.jhu.edu/guide


Dependency cycle after update to 8.4.0

Describe the Bug
After updating the module from 8.3.0 to 8.4.0, I get a dependency cycle on all Debian family nodes, for any repo I manage:
Found 1 dependency cycle:
(Exec[apt_update] => Class[Apt::Update] => Package[gnupg] => Apt::Key[Add key: 5B7C3E5A735BCB4D615829DC0BDDA991FD7AAC8A from Apt::Source theforman_buster] => Apt::Setting[list-theforman_buster] => File[/etc/apt/sources.list.d/theforman_buster.list] => Class[Apt::Update] => Exec[apt_update])\nTry the '--graph' option and opening the resulting '.dot' file in OmniGraffle or GraphViz

Expected Behavior
No dependency cycle.
Steps to Reproduce

Update the module
Start an agent run

Environment

Version 8.4.0
Platform Debian 10 / Ubuntu 20.04

Additional Context

After adding back the lines

  # required for adding GPG keys on Debian 9 (and derivatives)
  ensure_packages(['gnupg'])

in init.pp the cycle was gone.

I do all the repository setup in a separate Puppet stage, executed before the main stage.

Same issue here.
Reverting #1020 as suggested by @dhs-rec fixed it.

@dhs-rec @tlebars
Hey, have put a pr up for this issue if you'd like to take a look and possibly pull it down onto one of your machines to check that it fixes the issue:
https://github.com/puppetlabs/puppetlabs-apt/pull/1039

@david22swan, I replaced my workaround with your fix and it's still working.

That's it merged and a release has been cut so it should be good now.


docker/docker#8513 : Add a test for trailing carriage return in output of 'docker exec -it

This just adds the test, still needs a fix. See docker/docker#8513

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "8513-extra-carriage-return" git@github.com:frankcarey/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

ping @frankcarey can you sign-off your commit?

I'm having trouble setting up the tests correctly for testing TTY, I looked at the docker_run.. tests and tried using the pty package, but not having much luck.
Without the pty, I get an error about "using tty connection without tty" (my first commits)
Then after adding it, I don't get an error, but the test fails because it's not getting a response. I assume I need to block somehow until I get the response to test it, but it's not obvious how to do that, so any feedback or guidance is very welcome.  I'll push what I have so far.  This isn't ready to merge yet and I'll rebase this all down to a final commit and signoff on that once it's ready.

@frankcarey if you don't mind I'm closing this for now for the following reasons:

Although the testcase is very valuable, there is no bugfix yet for it so we can't merge this PR.
The git situation makes it impossible to merge. I suggest you copy the file with the new test to /tmp, do a git fetch $remote && reset --hard $remote/master where $remote is the name of the docker/docker remote (you can check in git remote -v). Then you can copy the file back from /tmp, and do a git commit -asm 'Add a test for ...'

Further recommendations:

use gofmt -w -s on the files you modified. You can set up a hook in your editor or in git.
run make validate binary locally before pushing
when you update your PR feel free to squash your updates into one commit, and force push with git push -f, this will update this PR and rerun the tests on the CI.

If you have a fix, feel free to ping us to reopen this PR so you can update it. If you push to your fork and this PR is closed, you will have to open a new PR.
If someone else has a fix, please make sure to include @frankcarey's test case.


[Discussion] Move styled-media-query package to the SC organization

TL;DR: I'd like to hear what do you think about moving styled-media-query to the organization?
I know media queries are trivial in styled-components, also a simple utility function can be a media query, but some people rather to not write their own utility functions.
Its API is quite readable compared to normal confusing max-width and min-width and it converts units (px to rem, em to px, etc) with built-in utils.
https://github.com/morajabi/styled-media-query/
For the next version, I may make use of theme for breakpoints and change the API a little bit.
Let me know what do you think about moving this, how it can be better, or why you hate/love it! 😉
/cc @styled-components/core

I feel like that's a package that's perfectly fine to live in your personal GitHub—there's nothing really to be gained from moving it to the organization imo. :blush:

At the time of writing the first paragraph of this issue, I felt like you, but wrote it anyway, what can go wrong! Thanks :)


Update: Add cid attribute to column fragment

Description
Update: Add cid attribute to column fragment
Proposed Changes

Add cid attribute to column fragment for unique column references
(like in visualizations)
Add nanid as core dependency

Screenshots
License
I confirm that this contribution is made under an MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Can we make a ticket for supporting this in the webservice


Make pivottable work better in IE8

In IE8 i got 'ghost' items (-1) in the field list.
I have changed line:
1045: for (i=0; i<shownAttributes.length; i++) {	// work around IE8   for (i in shownAttributes) {
May be it is better to change all 'for in' combinations.

Do you see this issue with any of the examples here http://nicolas.kruchten.com/pivottable/examples/ ? If so, which ones and which OS are you using?

Also, please note that IE8 is not currently really supported: see #100


Add time-tracking extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi, thank you for your feedback!
I've changed the following:

start & stop commands are now no-view commands
the list command now has a loading state
actions now have icons
added a confirmAlert to the delete action

In addition to the changes requested, I also added a "Copy Duration" action to the list.


Bug: Mobile view - context filter button has max-w-max, breaks out of container on long repo names and removes the cancel button

Describe the bug
Long repo names cause the context filter button to break out of its container and lead to unintended horizontal scroll. Also, the cancel button is inaccessible when the component reaches the right margin.

Steps to reproduce

Go to https://insights.opensauced.pizza/hot/repositories/filter/JoshuaKGoldberg/expect-no-axe-violations on mobile
Look at the header filter component and swipe sideways

Affected services
insights.opensauced.pizza
Platforms
Mobile
Browsers
Chrome
Environment
Production
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Nice catch @Themesses :clap: Feel free to give it a shot :rocket:
I think the take action is available :thinking:

.take

@Themesses checking in to see if you have questions

@Themesses checking in to see if you have questions

@bdougie thanks for checking in. Just opened a PR. Apologies for the delay.


[Mobile] Context Menu Remains Visible After Text Input or Paste Action

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
10.8.5
Steps to reproduce
Scenarios:
Scenario 1: Entering New Text

Type text in the input field.
Select a portion of the text or double-tap it to display the context menu.
Start typing new text.

Scenario 2: Using the "Paste" Action

Type text in the input field.
Select a portion of the text or double-tap it to display the context menu.
Tap the "Paste" option in the context menu.

Expected results
The context menu should close automatically after new text is entered or the "Paste" action is performed.
Actual results
The context menu remains visible, obstructing the user experience.
Additional Context
Similar as https://github.com/singerdmx/flutter-quill/issues/2111

Screenshots / Video demonstration
Actual result
Actual result
Expected results
Expected result

Is this a regression or related to #2026?

@EchoEllet I'm not certain. The original issue (#2111) was marked as resolved after PR #2116, which was merged after the PR you referenced (#2026).

The magnifier feature has some other regressions that have not been solved yet. We will need to try older versions.

If you can specify the version to test on, I will do it.

Try with 9.4.0, ensure to remove the caret (^) to force using this version:
flutter_quill: 9.4.0

Or:
$ git clone --depth 1 --branch v9.4.0 https://github.com/singerdmx/flutter-quill.git
$ (cd flutter-quill/example && flutter run)

The issue does not reproduce on this version


remove duplicate vite config

README.md gives the exact same config twice - guessing this was left over from a rewrite:
import { comlink } from 'vite-plugin-comlink'
export default {
  plugins: [
    comlink()
  ],
  worker: {
    plugins: [
      comlink()
    ]
  }
}

fixed


[ES Currency] Incorrect value returned for compound currency entity in ES-MX

Describe the bug
For a compound currency entity, the entity and units are identified correctly, but the value is missing a decimal.
"15 dólares y 15 centavos" Returns units = "Dólar" but value = "1515" instead of "15.15"
To Reproduce
Create an es-mx LUIS app and test the above text.
Expected behavior
type: "money"
units: "Dólar"
value: "15.15"
Sample input/output
type: "money"
units: "Dólar"
value: "1515"
Additional context
This only affects es-mx.
In es-es, the same example produces "15.15"

We can't replicate this issue. The correct output is produced in both es-ES and es-Mx.
Where did you see this behaviour and in which version of the packages?
You can test the current behaviour using the SimpleConsole project in the repo.

I originally saw the behavior while testing a LUIS app in the Azure web interface.
To determine whether it was a Recognizers issue or related to LUIS post-processing, I tested es-ES in SimpleConsole and got the expected output, but I don't know how to test es-MX:
If I just set DEFAULT_CULTURE to 'es-mx' in sample.py, the behavior appears to default to en-US. (It recognizes '15 dollars' as a currency but not '15 dólares'.)

@srandal, you should use the .NET version of SimpleConsole. LUIS utilizes the .NET packages.

Good to know -- thanks


Update private-link.md

added limitations and link to Next steps section.

@duongau
Can you review the proposed changes? Note that the validation warning must be resolved before merge.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @Court72, I'm going to close this PR and raise a private one since this is a request from a PM. I need to make some changes to fix broken article reference path.

#please-close


debian package has wrong version number

influxdb 2.x on debian stable:
sudo apt-get update && sudo apt-get dist-upgrade && sudo apt-get clean
Hit:1 http://mirror.netcologne.de/debian bullseye InRelease
Hit:2 http://mirror.netcologne.de/debian bullseye-updates InRelease
Hit:3 http://security.debian.org/debian-security bullseye-security InRelease
Hit:4 https://repos.influxdata.com/debian bullseye InRelease
Hit:5 https://packages.grafana.com/oss/deb stable InRelease
Reading package lists... Done
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Calculating upgrade... Done
The following packages were automatically installed and are no longer required:
  linux-image-5.10.0-12-amd64 linux-image-5.10.0-13-amd64
Use 'sudo apt autoremove' to remove them.
The following packages will be upgraded:
  influxdb2
1 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/89,4 MB of archives.
After this operation, 35,2 MB disk space will be freed.
Do you want to continue? [Y/n]
dpkg-deb: error: parsing file '/tmp/dpkg-deb.ZyqsmK/control' near line 2 package 'influxdb2':
 'Version' field value 'v2.3.0-1': version number does not start with digit
Traceback (most recent call last):
  File "/usr/share/apt-listchanges/DebianFiles.py", line 124, in readdeb
    output = subprocess.check_output(command)
  File "/usr/lib/python3.9/subprocess.py", line 424, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['dpkg-deb', '-f', '/var/cache/apt/archives/influxdb2_v2.3.0-1_amd64.deb', 'Package', 'Source', 'Version', 'Architecture', 'Status']' returned non-zero exit status 2.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/bin/apt-listchanges", line 323, in <module>
    main(config)
  File "/usr/bin/apt-listchanges", line 104, in main
    pkg = DebianFiles.Package(deb)
  File "/usr/share/apt-listchanges/DebianFiles.py", line 358, in __init__
    parser.readdeb(self.path)
  File "/usr/share/apt-listchanges/DebianFiles.py", line 127, in readdeb
    raise RuntimeError(_("Error processing '%(what)s': %(errmsg)s") %
RuntimeError: Error processing '/var/cache/apt/archives/influxdb2_v2.3.0-1_amd64.deb': Command '['dpkg-deb', '-f', '/var/cache/apt/archives/influxdb2_v2.3.0-1_amd64.deb', 'Package', 'Source', 'Version', 'Architecture', 'Status']' returned non-zero exit status 2.
dpkg: error processing archive /var/cache/apt/archives/influxdb2_v2.3.0-1_amd64.deb (--unpack):
 parsing file '/var/lib/dpkg/tmp.ci/control' near line 2 package 'influxdb2':
 'Version' field value 'v2.3.0-1': version number does not start with digit
Errors were encountered while processing:
 /var/cache/apt/archives/influxdb2_v2.3.0-1_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

issue is solved in the meantime

This is happening with the centos/rhel packages too
The version in the packages for 2.2.0 and 2.3.0 are:
2.2.0
v2.3.0
This makes it not able to upgrade.
`# yum install influxdb2-2.3.0.x86_64.rpm
Loaded plugins: fastestmirror, priorities
Examining influxdb2-2.3.0.x86_64.rpm: influxdb2-v2.3.0-1.x86_64
influxdb2-2.3.0.x86_64.rpm: does not update installed package.
Error: Nothing to do
rpm -Uvh influxdb2-2.3.0.x86_64.rpm
error: Failed dependencies:
influxdb2 < 2.1.0 conflicts with (installed) influxdb2-cli-0:2.3.0-1.x86_64`


Settings IP for sdp candidate

If you use Docker, the IP of the container is added to the sdp and clients located on the host's local network cannot connect. Can you add to the settings so that you can specify which other IP addresses to add to the sdp? otherwise it turns out that the container will only work for the local host

Hmm i havent seen this issue and run ion-sfu in a container locally. Are you opening the containers UDP port range?

If you run it in a container and try to connect via a local network from another device the sdp does not have the IP host on which the container is running
Example:
Local IP <IP_ADDRESS>
Container IP <IP_ADDRESS>
if connect from <IP_ADDRESS> then the sdp address equal <IP_ADDRESS>

Example settings in janus-gateway
https://github.com/meetecho/janus-gateway/blob/master/conf/janus.jcfg.sample.in#L267

gotcha! if youd like, you can create a PR with this functionality

@lxShaDoWxl please let me know if #184 solves your issue. thanks for surfacing this!

Yes, this solves the problem. Thanks!


Include suburb announced/date information in map

Consider marking suburbs in dropdown with flag saying whether they have been announced or not
Show announced date for the suburb if applicable

I’ll do this today

See PR.  Caveat - I'm not a javascript guy


Bug: CEB: Filter Custom Data Download By Year

Test Case
TC344
Steps to Reproduce

Step
Action
Expected
Status

1
Select year
The system displays the year component
Pass

2
Enter a year outside of the valid range         Valid date range: 1996 to current year
The system displays an error message
Fail

Actual Results

The system allows a user to apply filter criteria for 1995. The valid range is 1996 to the current year

The error message needs to be updated to display "Enter year(s) between 1996 and this year"

Requirements
User Story
As a CAMPD custom data download user I want to enter filter criteria for time period so that my dataset contains my desired years
Acceptance Criteria
Given that a user has selected compliance emission based
When the filter criteria page is displayed
Then the user has the option to select year
Given that a user has selected year
When filter panel is displayed
Then the user can enter one, multiple, or a range of  years
Given a user has entered filter criteria for year
When the user applies the filter
Then the system will ensure that the data is valid
Given that a user has selected  year
When the criteria is applied
Then the system ensures the criteria is in the YYYY format
And displays error message FV-102 if not
Given that a user has selected  year
When the criteria is applied
Then the system ensures the criteria is between 1996 to current year
And displays error message FV-105 if not
Given a validation has failed
When the error message is displayed
Then the error message is displayed in accordance with the USWDS validation component
Given that a user has applied filter criteria for year
When the criteria is applied
Then the system displays a filter tag
Given that a user has applied filter criteria for year
When any  year criteria is cleared
Then the system clears the criteria from the filter

https://epacamd.testquality.com defect D70.

Defect D70 resolved


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in 2cb013e after 6 minutes.


Export Button 414 (Request-URI Too Large)

Summary
When I was getting 414 (Request-URI Too Large) on datatable load/draw. I add the data table option 'sServerMethod' => 'POST', and it started working. After that Search and ordering started working too.
But when I try to do export CSV/EXCEL/PDF I get 414 (Request-URI Too Large). I even tried to add method in ajax call ->ajax(['url'=>route($this->route),'method'=>'post']) But still export uses get method.
Btw I am using DatatablesServiceProvider
System details

Ubuntu 14
PHP 7.1
Laravel 5.1
Laravel-Datatables 6

Follow the steps I posted on this blog to shorten the url. If you are using buttons plugin v2.0, you can use minifiedAjax() which basically uses the same technique on the blog.

Thanks for this, one thing I noted is when I copied the script in blog my column search and ordering stopped working. for this I changed the code a bit,
   function minifiedAjax(data) {
        for (var i = 0, len = data.columns.length; i < len; i++) {
            if (! data.columns[i].search.value) delete data.columns[i].search;
            if (data.columns[i].searchable === true && (data.columns[i].search === undefined || data.columns[i].search === "")) delete data.columns[i].searchable;
            if (data.columns[i].orderable === false) delete data.columns[i].orderable;
            if (data.columns[i].data === data.columns[i].name) delete data.columns[i].name;
        }
        delete data.search.regex;
    }

I am using yajra/laravel-datatables-oracle ^6.0
Regards,

@yajra I'm also facing this same issue in Export option. Look the blog link you have shared was now available now - getting 404 error. Could you please help me to fix this issue?
@karmendra Kindly let me know how did you fixed this issue

@poovarasudev  I have the same problem, have you found any solution?

@mikesocorecc Try using postExcel or postCsv button instead


Panic when using image annotation on Handle<Image> field

Using the #[asset(image(...))] on an image field causes the app to panic with "Only asset collection fields holding an Image handle can be annotated with image".
Here is an example which panics when run (with the necessary asset):
use bevy::prelude::*;
use bevy_asset_loader::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .init_collection::<MyAssets>()
        .run();
}

#[derive(AssetCollection, Resource)]
struct MyAssets {
    #[asset(path = "my_image.png")]
    #[asset(image(sampler = nearest))]
    my_image: Handle<Image>,
}

This is expected and I will edit the documentation to mention that the image annotation requires using a loading state.
The problem is that to change things like the sampler, I need to take the asset from the asset collection. For that, it needs to be loaded. But without a loading state, I cannot run code at the point where the asset is loaded.

I opened #230 to look into supporting the image annotation without a loading state, but I consider this issue closed with the updated documentation in #231


⚠ Bofrost.es has degraded performance

In a6e56f4, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1531 ms

Resolved: Bofrost.es performance has improved in 8ea7c33 after 13 minutes.


Refactor: Improve performance and memory management

Lots of small changes aimed to improve code performance, especially in debug build.

Thanks. Appreciate all the work put into this.


make tests compatible with spark 3.3

and run fast tests nightly against 3.3
tested locally against both 3.3 and 3.4

build


Question about disparity image.

Hello.
In KITTI2012 data_stereo_flow\training\disp_noc\000000_10.png, I saw the maximum raw value is 17873. Is it the real disparity? Does it need any transformation?
Thanks a lot.

Yes. The image format is PNG 16bit. If you divide them by 256, you get the value that you are familiar with (i.e. PNG 8bit).


Unable to support images in full-screen mode. Use the mouse to click the buttons left and right to navigate to the previous and next image. . .

Unable to support images in full-screen mode. Use the mouse to click the buttons left and right to navigate to the previous and next image. . .
For example:

Currently there is no such function in the latest version, it is very inconvenient to use, I hope it can be added.
There is no such function in the latest version at present, it is very inconvenient for users who like to use the mouse to navigate, hope it can be added.
Also found two errors:

When I turn off the navigation thumbnails at the bottom, the left and right arrow key navigation doesn't work anymore.

After closing the navigation thumbnail at the bottom, the length and width of the pop-up image should increase, but the actual situation is that the length and width of the pop-up image have not increased. Please fix this bug, thank you very much!

Thank you for the very good image extension 👍😜
無法支援全螢幕模式下的圖片使用滑鼠點擊按鈕左右導航到上一張與下一張圖片。 。 。
例如：

目前最新版本中沒有這個功能，對於喜歡使用鼠標來導航的用戶來說使用起來非常不方便，希望可以加入它。
另外發現二個錯誤：

當我關閉底部的導航縮略圖後，左右方向鍵導航不起作用了。

關閉底部的導航縮略圖後，彈出圖片的長寬尺寸應該增大才對，但是實際的情況是彈出圖片的長寬尺寸沒有增大。請修復這個錯誤，非常感謝您！

感謝您提供的非常優秀的圖像擴展👍😜

An example demonstration is as follows:
OS: Win10
Obsidian: 1.4.16
The effect is as follows:

If developers are willing to add this feature, the user experience will be greatly improved.
如果開發者願意加入該功能，那麼使用者體驗將會大大提升。

same as #86


Support for dotenv

I was going to ask this in Discord but decided to do it here in case others have the same question.
dotenv is a popular mechanism of specifying project-specific environment variables without accidentally adding secrets to git history. It relies on git ignored .env file with environment variables.
Backstage has it's own configuration reading API. In theory, dotenv and Backstage config should not be in conflict as long as dotenv is loaded before Backstage config. At the same time, adding secret environment variables directly to app-config.development.yaml and git ignoring app-config.development.yaml would serve the same functionality as dotenv.
What are your thoughts on how app-config.development.yaml should be used and the role of .env if any?

You can use app-config.local.yaml and app-config.development.local.yaml! They are ignored by default, and you can put secrets straight up as values.
For example, I use the following in app-config.local.yaml to have a separate file with some custom entity definitions for development, as well as Google auth setup:
catalog:
  locations:
    ... standard stuff

    - type: file
      target: /Users/me/dev/backstage/dev-components.local.yaml
      rules:
        - allow: [User, Group]

auth:
  providers:
    google:
      development:
        clientId: <numbers>.apps.googleusercontent.com
        clientSecret: <secret-key>

Perhaps a "local development configuration" section in the docs would help with visibility?

This is exactly what I was hoping to find out.

Perhaps a "local development configuration" section in the docs would help with visibility?

I'll add the docs - if you assign the issue to me.

Sorry forgot to point to the existing point for the feature: https://backstage.io/docs/conf/writing#file-resolution
Thinking the local development setup could be more clearly explained there, but maybe also that we should mention it in the getting started bit

I've been using https://direnv.net to handle this use case. I've found this easier then creating a custom app-config.local.yaml, which requires duplicating a bunch of sections from the app-config.yaml. Having dotenv support would essentially be the same result, but something build in. My 2 cents anyways 😄

@andrewthauer do you have an example of something that requires duplication? From my experience values are either easy to override with a single value, or tricky to configure through environment variables anyway.
Not saying we can't add support for loading in env files, but trying to stay away from too many solutions for the same issue if it doesn't provide additional value.

@Rugvip - I'm mostly using environment variables for just supplying a value for an existing $secret in the main config file.
So, instead of having to replicate the full config path like this:
# app-config.local.yaml
auth:
  providers:
    google:
      development:
        clientId: 'XXXXXX'
        clientSecret: 'YYYYYY'

I can just do:
# .envrc or .env
export GITHUB_ACCESS_TOKEN="XXXXXX"
export GITLAB_ACCESS_TOKEN="XXXXXX"

However, I use direnv for all kinds of things so this is natural and works without having to add dotenv support :)

@andrewthauer well the first option lets you type less characters if you're typing them out :grin:
I think it's a pretty minor difference anyway, and would rather help make the existing alternatives more visible. Especially since there's plenty of stuff in config that would be pretty tedious to configure through env, like the custom component location I have in my example

Agee 💯. Was just sharing an alternative view 😀.

@taras did you manage to update the docs for this?

@Rugvip Currently, local configuration is mentioned in passing in "Configuration" -> "Writing Backend Configuration" -> "File Resolution".
I looked at a few places where local configuration could be mentioned.
For example, some options are,

In "Getting Started" -> "Running Backstage Locally" -> after "Clone and Build"
In "Getting Started" -> "Development Environment"
In "Getting Started" -> "App Configuration" -> add new page "Local Configuration"

With that said, having so many places where this could go suggests that now might be a good time to reorganize the Getting Started section.
Thoughts on where it should go? or reorganizing the Getting Started section?

@alund, do you have a preference? 📓 👑

@benjdlambert If you mean me, then no 😎

Ok in that case maybe #2 @taras?

@benjdlambert 👍

To me, there is still a need for dotenv as backstage still recommends to use environment variables and there is no easy way to test those unless you package environment variables in docker variables or kubernetes config map. What do you think? @Rugvip

+1

I would still very much recommend using app-config.local.yaml with inline secrets for any local customizations, overriding the environment variables that might be ther by default. It's a lot more powerful and lets you do deeper customizations than you could ever do with environment variables.
We're very hesitant to introduce additional concepts and files into the config system, since it increases complexity and the risk of confusion.

I'll also mention again that using a combination of the preferred catalog-info.local.yaml and something like direnv can easily simulate native dotenv support. At least for local development.

I believe this has been a community desire for some time. Since using variables through app-config.yaml is not so simple.
I myself have already requested support for the use of this strategy and have not been able to get clear examples of effective use.

I ended up just using a small yarn plugin like
import { readdirSync, readFileSync } from 'node:fs';

import { parse } from 'dotenv';

import { Plugin } from '@yarnpkg/core';

const plugin: Plugin = {
  hooks: {
    setupScriptEnvironment(project, scriptEnv) {
      if (scriptEnv.NODE_ENV !== 'production' || scriptEnv.READ_ENV_FILES === 'true') {
        const dotenvFiles = readdirSync(project.cwd).filter(file => /\.env.*/.test(file));
        Object.assign(scriptEnv, ...dotenvFiles.map(file => parse(readFileSync(file))));
      }
    },
  },
};

export default plugin;

and it worked well for me.

https://github.com/webark/yarn-plugin-dotenv
Pushed it up if anyone wants to use it. (it seems like the general plugin installation path is just a download from GitHub direct 🤷 )

@webark Could you present the structure that was your Backstage?

Congratulations on the initiative to develop something for this purpose @webark!
But I keep getting the variable as undefined when using your library/package.

Are you on the discord server? It might be easier to chat there.
I was using dotenv cli to load up a ".env" and ".env.local" file at the root of the project.

You also can't have the "NODE_ENV" set to "production".
Im on the discord under "webark". Feel free to ping me in the support channel if you are still having issues.

For direnv (which I highly recommend as a general tool), you can setup it up using the following:

Install direnv (e.g. brew install direnv on mac)
Hook into your shell (e.g. add eval "$(direnv hook zsh)"  to your .zshrc file)
Reload your shell
Create an .envrc file in the root of your backstage app with something like:# .envrc
dotenv # this will load an `.env` file

Create a standard .env file or use an existing one
Run direnv allow to tell direnv to accept the .envrc file. Required anytime the file changes. This will also reload your shell with the .env variables in your shell
Run yarn dev and with the environment variables ready.

NOTE: direnv is a directory switcher, so anytime you `cd` into a directory with a `.envrc` file in the directory structure it will find it and execute the commands automatically. It also has many other useful features beyond this which you can explore and use for other purposes.

yea. Some people used that. And even though you can put it in a tools version for people that use asdf, I still wanted something that was a "no configure" strategy. but yea. Every one I know that uses it has good things to say.


Fixed typos in wstring/dstring alias statements

wstring is an array of wchar not dchar.
dstring is an array of dchar not wchar.

@mtrenkmann good catch. why did you close the PR?

Sorry for closing, was my first PR .. reopened

Sorry for closing, was my first PR .. reopened

No worries, thanks a lot!


Simplify implementation with flask.after_this_request

Hi, I was looking into a solution for setting Cache-Control headers in Flask, preferably with view function decorators and stumbled on this project. I had a look into your implementation and was a bit overwhelmed by the complexity and layers.
After some more looking around I cooked up my own version, illustrated here:
https://gist.github.com/soxofaan/4a646f0ac24208cd097c7ab6ea21cc85
It's just around 40 LOC, one single function (the decorator), no need for something like init_app, no use of pseudo-globals like flask.g or other hidden coupling, just a direct flask.after_this_request call. Feature-wise it comes very close to what Flask-CacheControl already supports I think.
I'm putting it here if somebody would be interested.
Or it could serve as inspiration for a next version of Flask-CacheControl

Tbh, I'm completely puzzled by why I didn't take the short path of directly registering response modifiers via  flask.after_this_request from the start. Not a clue.
Thanks a lot @soxofaan for pointing it out. Removing the g detour made things a lot simpler. It's part of the 0.3.0 release that just went out.
Again, thanks!

Great! And already with a new release? That's a fast response time :smile:


if serviceConnection?.Service is null, try to reconnect

 public async override Task<IEnumerable<InAppBillingProduct>> GetProductInfoAsync(ItemType itemType, params string[] productIds)
        {
            if (serviceConnection?.Service == null)
            {
                throw new InAppBillingPurchaseException(PurchaseError.ServiceUnavailable, "You are not connected to the Google Play App store.");
            }

Above code or other api functions throws exception very often because of although ConnectAsync function is used before and connected=true returned. Google api just randomly disconnect you in the middle of the process. thats why below issues created earlier.
it would be better to try to reconnect before every other api function attempt.
https://github.com/jamesmontemagno/InAppBillingPlugin/issues/197
https://github.com/jamesmontemagno/InAppBillingPlugin/issues/195

I am having the same problem, I am running the example in this same repository but the serviceConnection always appears null, when I try to make a purchase I get the same error: You are not connected to the Google Play App store. Any idea what might be happening?

I have an alternative nuget, you can try if you want. I reprogrammed from the scratch and i don't see this error any longer.

Would recommend trying 4.0 which uses the new Android APIs.


[Fix] "Keys" tab: add npub; fix typo

Tab "Keys" could show full npub and give a preview of nsec (not just 'copy').
And there is typo in "key has been copied"

Then, the "key has been copy" popup doesn't disappear automatically


Set of utilities

For example, broadcast date/time, log and resend on subscribe last value, store history, etc

Viewer - add broadcast time/date feature
$SYS/broker/time?

Blob exchange sever - send URL with mqtt/udp
Use for picture exchange
Time server
History store server - use blob exchange as graphics storage
Logger server - use history server as base?


How to use kfac to train two probabilistic models jointly?

In my application, I need to jointly optimize two probabilistic models. They contribute to two different terms in the final loss function.
I am wondering what would be the recommended pattern of using kfac ?
More specifically, does it make sense to invoke  kfac_jax.register_normal_predictive_distribution twice (for the two probabilistic models respectively) ?
Thanks in advance!

Perhaps a more concrete example of my question:
How train a VAE using kfac-jax?

Thanks!


"A State" Exploration for AWS marketplace listings

https://github.com/slackernews/slackernews/compare/70-a-state-aws-marketplace?expand=1


Backfeeding comments and reactions returns error

I noticed today that backfeeding comments and reactions to my posts crossposted to Mastodon hasn't been working since September, it seems.
I use Known (1.5) on my website and using brid.gy for backfeeding which has been working flawless. It appears that around September something changed somewhere, and my Mastodon page on brid.gy shows Error exclamation marks on responses:

When I click through on the error link, I get the message that no log is found:

I notice it says the last webmention was sent 4 months ago, which seems incorrect, as I've sent webmentions just a few days ago (at least I should've). I didn't change anything in Known, but I am running on a shared server, so potentially something changed with the underlying server settings, but I'm unsure how to check for my webmentions working.
Any good pointers?

I used webmention.rocks to check if webmentions are sent correctly from my site, and they show up correctly there. From that it seems like webmentions work on the site. Not sure what to check next.

Hi @voss, sorry for the trouble! Looks like your site's webmention endpoint is failing on these webmentions and returning HTTP 500 with this message:
Something went wrong.

base64_encode(): Argument #1 ($string) must be of type string, array given

Hi @snarfed,
That's super helpful, thank you!

Welcome!  Do you have a URL where the error message you quoted in https://github.com/snarfed/bridgy/issues/1668#issuecomment-1947387905 shows up in a readable way, or did you find it in your logs? I'm trying to raise the issue with my hosting provider, so see if it's an issue that came about due to upgrading software on the shared hosting, and providing them with an example will probably be helpful.
Thank you!


Can't Install on Ubuntu 16.04.1

My npm install always returns with an error and all the workarounds I found haven't worked.
Clone Log:
sudo git clone https://github.com/GordyD/3ree.git
Cloning into '3ree'...
remote: Counting objects: 603, done.
remote: Total 603 (delta 0), reused 0 (delta 0), pack-reused 603
Receiving objects: 100% (603/603), 120.38 KiB | 0 bytes/s, done.
Resolving deltas: 100% (287/287), done.
Checking connectivity... done.

Install Log:
sudo npm install
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>Babel's CLI commands have been moved from the babel package to the babel-cli package
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.

><EMAIL_ADDRESS>install /var/nodejs/3ree/node_modules/phantomjs-prebuilt
> node install.js

sh: 1: node: not found
<EMAIL_ADDRESS>/var/nodejs/3ree

.....tree....
pm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>No repository field.
npm ERR! Linux 4.4.0-31-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the phantomjs-prebuilt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs phantomjs-prebuilt
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls phantomjs-prebuilt
npm ERR! There is likely additional logging output above.

@JessicaHergert  It looks like nodejs isn't installed on your machine.  Try to install it with sudo apt-get install npm. It should say its already installed if its present already, in which case you probably don't have the path to the node command in your PATH, or specifically the root user's PATH since you're running the command as root (sudo).

@snobear As the headline reads, this issue refers to installing on Ubuntu 16.04.1, it also says that node.js is installed at version 4.2.6 in the log I posted, where it reads "npm ERR! node v4.2.6".
nodejs -v
v4.2.6

Right - thats why I suggested sudo apt-get install...apt-get is the package manager for Ubuntu.
Anyhow, looks like its installed.  I bet the node command isn't in the root user's path.  You could fix that, BUT I would recommend that you do all these commands without sudo, as a general recommendation. You should be able to fire it up the project as a regular user.

@snobear to be clear about the command thing: the command node is already taken in ubuntu 16.04.1 - the command is "nodejs"
also without sudo npm can't really install any of these things at all.
but even if I try to install phantomjs manually beforehand, it still fails.


H/gather layer

Gather layer implemented
The output of the test cases match the output of onnx examples provided in https://github.com/onnx/onnx/blob/master/docs/Operators.md#Gather

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- model_compiler/python/nnir.py  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- model_compiler/python/nnir.py  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- model_compiler/python/nnir.py  2

See the complete overview on Codacy


Allow disabling PSPs

Otherwise Cilium helmrelease fails to install on<EMAIL_ADDRESS>note: I'd add this here in the transition period and remove it later, once the flags are true for all the apps (imho)

How to run CI
Comment on this PR with:
/run cluster-test-suites

Wait before merging. We can also add to drop the removed flag

@vxav i think we can drop the flag completely, it doesn't have to be based on the k8s version. Because that TTLAfterFinished  feature that cleans finished jobs is on by default since 1.23

Done in https://github.com/giantswarm/cluster-vsphere/pull/118/commits/fa3eb4a64eb18667a21270068cb234bb75303fe7

Update: We will implement something like in this PR https://github.com/giantswarm/cluster-azure/pull/179 Please wait.


Minor CMB2 macro typo on pathfinder-neceros character sheet

For the CMB2 button it says @{CMB} instead of @{CMB2} causing the check to roll the wrong line.

This pull request should fix this issue.

#1822 now holds the fix.


Unable to complete build_redtail_image.sh script

Hi There,
I've run the build_redtail_image.sh script to install the TensorRT-<IP_ADDRESS>.Ubuntu-16.04.4.x86_64-gnu.cuda-9.0.cudnn7.1.tar.gz file. The install fails when it gets to the setup tools stage.
How do I rectify this?
I've listed the output on screen when it fails
Thanks a lot in advance :)
Output log******
Reading state information...
0 upgraded, 0 newly installed, 0 to remove and 41 not upgraded.
Collecting setuptools
Downloading https://files.pythonhosted.org/packages/2f/8e/38259f4a44944a92068d5ff77230511a4c685604b47a81318f9e5cf2cc24/setuptools-49.2.0.zip (2.2MB)
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "setuptools/init.py", line 21, in
import setuptools.version
File "setuptools/version.py", line 1, in
import pkg_resources
File "pkg_resources/init.py", line 1380
raise SyntaxError(e) from e
^
SyntaxError: invalid syntax
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-Re_B_G/setuptools/
You are using pip version 8.1.1, however version 20.1.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
The command '/bin/sh -c apt-get update && apt-get -y --no-install-recommends install         ant         bzip2         ca-certificates         ccache         cmake         curl         genromfs         git         gosu         iproute         iputils-ping         less         lcov         libcudnn7=<IP_ADDRESS>-1+cuda9.0         libcudnn7-dev=<IP_ADDRESS>-1+cuda9.0         libeigen3-dev         libopencv-dev         make         nano         net-tools         ninja-build         openjdk-8-jdk         patch         pkg-config         protobuf-compiler         python-argparse         python-dev         python-empy         python-numpy         python-pip         python-serial         python-software-properties         rsync         s3cmd         software-properties-common         sudo         unzip         xsltproc         wget         zip     && apt-get -y autoremove     && apt-get clean autoclean     && pip install setuptools wheel     && pip install 'matplotlib==2.2.2' --force-reinstall     && pip install cpp-coveralls     && pip install jinja2     && rm -rf /var/lib/apt/lists/{apt,dpkg,cache,log} /tmp/* /var/tmp/*     && cd /opt && curl http://www.eprosima.com/index.php/component/ars/repository/eprosima-fast-rtps/eprosima-fast-rtps-1-5-0/eprosima_fastrtps-1-5-0-linux-tar-gz?format=raw | tar xz eProsima_FastRTPS-1.5.0-Linux/share/fastrtps/fastrtpsgen.jar eProsima_FastRTPS-1.5.0-Linux/bin/fastrtpsgen     && ln -s /opt/eProsima_FastRTPS-1.5.0-Linux/bin/fastrtpsgen /usr/local/bin/fastrtpsgen     && mkdir -p /usr/local/share/fastrtps && ln -s /opt/eProsima_FastRTPS-1.5.0-Linux/share/fastrtps/fastrtpsgen.jar /usr/local/share/fastrtps/fastrtpsgen.jar' returned a non-zero code: 1

Any updates on this? I am running into the same issue.

Look at my pull request:
https://github.com/NVIDIA-AI-IOT/redtail/pull/157
It fixes the docker issues. There were a few commands that needed changing in the Dockerfile.kinetic file.

Thank you - I've merged the fix to master.


Add a Gitter chat badge to README.md

georgigriffiths/seneca-flow now has a Chat Room on Gitter
@georgigriffiths has just created a chat room. You can visit it here: https://gitter.im/seneca-flow/Lobby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 95.285% when pulling 8c7adf927e261e8bdfc36aaf329e1bb155091865 on gitter-badger:gitter-badge into ca91288ca86230619dfe04993514e4abd61375f7 on georgigriffiths:master.


bemenu: fix build against upcoming ncurses-6.3

On ncurses-6.3 with extra printf() annotations gcc now detects
use of user input in place of format strings:
lib/renderers/curses/curses.c:234:9:
  error: format not a string literal and no format arguments [-Werror=format-security]
  234 |         mvprintw(0, 0, menu->title);
      |         ^~~~~~~~

Result of nixpkgs-review pr 144307 run on x86_64-linux 1

1 package built:

bemenu

@ofborg eval

Ah never mind, basic eval checks is a github action. Anyway, the eval failure was unrelated to this PR and should be resolved now.


Toolbar causes CSRF validation to fail

This seems to be similar to #5024.
I have a DRF APIView that acceps data via POST. The CSRF check for that view fails. If I set CMS_TOOLBAR_HIDE = True in my settings, everything works fine.
At first glance, I think that the bug is here: https://github.com/divio/django-cms/blob/60d7202b73609735015504503ff1215ae1e69d77/cms/middleware/toolbar.py#L32-L33
Maybe the logic of this check should rather be if toolbar_hide: return False? On the other hand, this would then not show the toolbar on pages with AppHooks, I think.
Maybe the better course of action is to ensure that the toolbar does not interfere with the request in the first place?

https://github.com/tomchristie/django-rest-framework/issues/3951

I'm closing this issue, since the real problem seems to be in DRF, and it looks like Tom is already on it :)

Cool, thanks for looking into it @rfleschenberg


FilePicker is not responsive

Category
[ ] Enhancement
[ ] Bug
[X] Question
Version
Please specify what version of the library you are using: 3.12.0
Expected / Desired Behavior / Question
The FilePicker component is not usable on mobile devices. Is there a plan to implement a responsive interface for this component?
I would expect it to behave the same way as the native SharePoint Image web part where the menu can be opened as a floating panel when you click 'Select source'.

Observed Behavior
The left navigation menu is still visible on mobile which is occupying most of the screen width leaving very little space for the right-side panel.

Hi @jacksonv1lle,
Did you have the chance to test with latest version (3.15.0)?
I've tested and it's responsive as expected from my side:

Can you confirm it?


404 page

Hello,
First thank you for the starter.
My question is the 404 page doesn't seem to be working, if visit /404 it's fine, but any other test just gives the server error page. Any help or am I just possibly missing something?
Thank you for your time.

Hey @PatrickWitmer, could you create an example reproducing the error? It can be a repository or a CodeSandbox.
Testing with our website (running the latest version of the theme), seems to be ok.

I did some testing @jpedroschmitz, and it works on a test netlify deploy, but not my nginx cloud server.
Here's a live example

Hum, sorry, but I don't have experience with Nginx.
If it's running ok on Netlify it's probably an error with your Ngnix config and not something with Gatsby or the theme.

Do you think it could be a pathing issue with it being hosted on a subdomain?

@jpedroschmitz I appreciate your time.