Send message to specific client

How to send a message to a specific client? For example, user refill account with money and when transaction with bank is successful, bank send request to my server payments handler with user ID parameters, amount etc.. I want to notify client of successful transaction.

This is not a bug report and I won't help you with 101 stuff


Feature: Configurable output format in connector SDK

Feature description
The destination connector is responsible for deciding how a record gets encoded and sent to the 3rd party system. This means a user generally can't control what the connector produces (unless that connector implements different options like the S3 connector, see option format).
We should add functionality to the SDK that controls what gets produced by the connector. The idea is to tap into the sdk.Record struct and provide a function that marshals it into a byte slice (e.g. Record.Bytes()). The SDK could then inject an option in the connector config that controls the behavior of this method. For example the user could choose that Bytes() returns the JSON representation of the whole record, a Kafka Connect compatible JSON record, or just the payload. Connectors that produce raw data would then be expected to use the Bytes() method to marshal the record.
Note that connectors that mutate entities in place only work with structured data and would still be able to manually inspect structured data.

This is about being a nicety in the SDK. Connector developers would be able to call a function and then all of the format changes could be made on their behalf. They wouldn't have to program all of that themselves. This is a fairly big issue.

push to the next release


Write a Swift program to find area of a cuboid

Description
Write a Swift program to find area of a cuboid
Area of cuboid = 2 × (Length × Width + Length × Height + Width × Height)
Length : 2
Width  : 3
Height : 4

Area : 52

How to contribute

Save the solution in program/FindAreaOfACuboid.swift file
Add FindAreaOfACuboid.swift file in find-area-of-a-cuboid folder

!assign


Precedence on the Sort Features

I'm trying to fix the Sort annotation and appoint it here:
https://github.com/jakartaee/data/issues/94
Currently, we have three ways to define Sort:

query by method (Static order)
Pageable and Sort (dynamically)
Sort annotation (Static order)

Once thrown, an exception is not a good API for developers; it might be better to define precedence.
The first two are more accessible, and we have samples in the market. So, the query by method and appends from the question dynamically.
We could add the Sort annotation as optional and third in the precedence.

query by method
Pageable and Sort on parameters
Sort annotation (optional by vendors)

I agree with Markus, and it does not look intuitive; furthermore, we need a case for it. Another option would be to remove it and wait until we collect feedback from a vendor who does it first.

The first question to answer with respect to establishing a precedence is whether the higher precedence is defined as replacing the lower precedence criteria, or whether the higher precedence is defined to sort by that criteria first before sorting by the lower precedence criteria only if needed.
Here's an example that will illustrate.  You could have:
Pageable p = Pageable.ofSize(10).sortBy(Sort.asc("price"));
products.findByNameLikeAndPriceBetweenOrderByName(pattern, minPrice, maxPrice, p);

If we say that sort criteria from Pageable has higher precedence than sort criteria from OrderBy, in the above example it could mean either of:

Sort only by price because the sort criteria from Pageable replaces sort criteria from OrderBy, or
Sort first by price and for any which have the same price, then sort by name because the sort criteria from Pageable is evaluated first, and the sort criteria from OrderBy is only evaluated when the higher priority criteria is not able to fully determine the order on its own.

Let's determine this question first, and then we can consider what precedence to assign to each of the 4 ways to specify sort criteria.

This one looks like an append to me:
products.findByNameLikeAndPriceBetweenOrderByName(pattern, minPrice, maxPrice, p);

Thus, sort by name and what is on the pageable, so price.

I think I said there are 4 different ways to do sorting above. There are actually 5:

OrderBy query-by-method name pattern
@OrderBy annotation (often referred to as the sort annotation)
Sort on Pageable parameter
Sort as method parameter
@Query with an "... ORDER BY ..." in it.

To simplify this, I suggest that it continue to remain an error to combine the first with the second (having multiple static OrderBy), or the third with the fourth (having multiple dynamic Sort) on a single method.
Then the problem reduces to determining order of append for:

static OrderBy
dynamic Sort
static @Query with an "... ORDER BY ..." in it.

Ignoring the Sort annotation.
How does it work in Spring and Micronaut?
@graemerocher @odrotbohm @mp911de

In Spring Data, there's a "static" sort component, defined in either the manually declared query or in the method name. That usually gets pushed down into a cached instance of the store's query abstraction. The dynamic one, handed in as method parameter, then augments that query instance, i.e. gets appended to the statically defined sort, to build up the query that's brought to execution eventually. There's no sort annotation or the like.
That said, we usually see queries defined without any static sort reference and users declaring default methods without a Sort to then call the actual query method with some default. Also, we generally reject query methods that declare both a Sort and a Pageable. If you declare a query to be a paging one, the Sort you want to apply needs to be handed in as part of the Pageable.
HTH

That's good, it sounds like Spring Data behavior is consistent with what I was expecting.  @Query with ORDER BY in it can just be treated as another form of static, so then a method can have at most one static mechanism and at most one dynamic mechanism for specifying sort criteria, with the static taking precedence and the dynamic being appended on to that.  I think that will be easy to explain in the spec in a way that makes sense.

Markus, I have the same feelings about the @Query
@odrotbohm @graemerocher what do you think about it?

Let me add a bit more opinion from my side: Not having Sort support for SQL @Query queries results in not having pagination support for these which imposes quite some constraints on users.
Also, our current thinking goes into the direction that we could well specify the WHERE part in a query as this wouldn't impose limitations on pagination and sorting augmentation for queries.
Another possibility is giving users a mechanism to augment/rewrite a query. Clearly, these limitations apply only to particular stores.
As a way out, @Query could serve as a meta-annotation marker. We can leave the actual @Query handling to implementation modules which effectively turns @Query into implementation details instead of part of the spec.

I found the following in Spring Data documentation giving examples where @Query can be combined with Sort,
  @Query("select u from User u where u.lastname like ?1%")
  List<User> findByAndSort(String lastname, Sort sort);

  @Query("select u.id, LENGTH(u.firstname) as fn_len from User u where u.lastname like ?1%")
  List<Object[]> findByAsArrayAndSort(String lastname, Sort sort);

For the sake of users who are already doing that, I think it does make sense for the spec to allow it to continue to be valid.  I like the idea of allowing for Sort/Pagination with @Query that is limited to the WHERE clause and doesn't already contain an ORDER BY clause in it.  I think we can come up with some spec language that identifies the limitation and makes it the application's responsibility if trying to combine @Query with other sorting to write their query in such a way that an ORDER BY clause can be appended without requiring the provider to parse query language.  I've made an attempt at this in commit https://github.com/jakartaee/data/pull/117/commits/bfbefedcc25672e65f7df202d98211397169609a to pull #117.

@njr-11, what do you think about putting several samples and their precedence on it?
I guess it will get clear the proposal:
List<User> findByNameOrderType(String name);

@Query("my native query here order by name")
List<User> findByNameOrderType(String name);

@Sort("age")
@Query("my native query here order by name")
List<User> findByNameOrderType(String name);

@Query("my native query here order by name")
List<User> findByNameOrderType(String name, Pageable pageable);

@Sort("age")
List<User> findByNameOrderType(String name, Pageable pageable);

Can someone elaborate what the point of the @Sort annotation is? Either you have a query defined explicitly, in which you better define the sort directly in the query (second method). Or you use query derivation, in which you'd include all static query information in the query method name (method one)? I don't quite get which problem methods three and five solve.

@otaviojava - that's an excellent idea to add examples.  I did so here.
@odrotbohm - I think those examples were written quickly and have a few errors in them.  I cleaned that up when adding it to the pull, so they'll hopefully make sense now.  There's a separate issue for discussing the topic of why we have the multiple mechanisms in the first place and whether we should keep them all, so I'll leave that sort of discussion to that issue. This issue here is only to document the precedence of what we currently have.

I've commented on the other ticket. I'd vote to get to a decision whether to introduce the annotation in the first place as you (we) wouldn't want to spend time specifying and thinking about precedence of programming model means that might not even make it.

Can we make it at least optional on the spec?
Thus, Spring and Eclipse JNoSQL do not need to implement and are still compatible with the spec.

We could add language that says something along the lines of: "MappingException is raised when the database is incapable of ordering with the requested sort criteria."  That way, Jakarta Data providers for any NoSQL databases that can't do sorting (or some aspect of it) are compliant with the spec by raising the exception.


Change user agent name for native app

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 12 (100.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1047

Relevant Lines:
1047

💛  - Coveralls


Fixes if modified since being deleted by other kap components

fixes https://github.com/razee-io/Kapitan-core/issues/24


Not able to run "make" in freeflyer_build/native

Hello. I faced no issue in installation of dependencies.
But I am facing this issue while running make -j2 in my build directory
Kindly help.
make_error.docx


Generate BookId instead of (Key Book) for persistent's type aliases

Currently if you have a type alias type BookId = Key Book that you Capture "book-id" BookId it'll be rendered as (Key Book), and you have to replace it manually, like at https://gitlab.com/k-bx/meetup/blob/master/backend/meetup/src/Meetup/GenerateElm.hs. Having a BookId would be nicer.

I've run into this issue as well. Looks like the URL referenced above has moved to https://gitlab.com/k-bx/meetup/-/blob/master/backend/meetup/src/Le/GenerateElm.hs
And the behavior from the persistent side is described for example at https://www.yesodweb.com/book/persistent#persistent_closer_look_at_types

It seems this is already possible by defining something like myAlteration from elm-bridge and then call it from servant-elm's defElmOptions like so:
generateElmModuleWith
    defElmOptions { urlPrefix = Static $ baseURL
                  , elmAlterations = myAlterations }
    [...]
   ...

... where the alteration looks like:
import           Elm.TyRep as ET
import qualified Elm.Module as Elm

myAlterations :: ET.ETypeDef -> ET.ETypeDef
myAlterations = Elm.recAlterType myTypeAlterations

myTypeAlterations :: ET.EType -> ET.EType
myTypeAlterations t = case t of
  ET.ETyApp (ET.ETyCon (ET.ETCon "Key")) (ET.ETyCon (ET.ETCon "Book")) ->
    ET.ETyCon (ET.ETCon "BookId")
  _ ->
    Elm.defaultTypeAlterations t

But then you still need to specify the type of BookId that matches the database backend, such that elm-bridge can generate the type aliases and serialization functions...


Separate interface package system_modes_msgs

Separated msg and srv types into seperate package system_modes_msgs. This seems to be the ROS 2 way anyway, was requiring a hack beforehand, and seems necessary to get the windows build working. (I have a working windows build based on this change now)

Codecov Report

Merging #74 (beda889) into master (2917384) will decrease coverage by 0.76%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   32.63%   31.87%   -0.77%
==========================================
  Files          14       14
  Lines        1048     1048
  Branches      754      754
==========================================
- Hits          342      334       -8
- Misses        132      136       +4
- Partials      574      578       +4

Impacted Files
Coverage Δ

system_modes/include/system_modes/mode_manager.hpp
100.00% <ø> (ø)

system_modes/include/system_modes/mode_monitor.hpp
100.00% <ø> (ø)

...ystem_modes/include/system_modes/mode_observer.hpp
100.00% <ø> (ø)

system_modes/src/system_modes/mode_manager.cpp
17.29% <0.00%> (-1.13%)
:arrow_down:

system_modes/src/system_modes/mode_monitor.cpp
26.98% <ø> (ø)

system_modes/src/system_modes/mode_observer.cpp
20.00% <ø> (ø)

system_modes/include/system_modes/mode_impl.hpp
81.48% <0.00%> (-3.71%)
:arrow_down:

system_modes/src/system_modes/mode_inference.cpp
35.96% <0.00%> (-1.09%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


WebJob background file change handler lack tracing

WebJob has file watcher on app_data\jobs folder and handle them.  We have no tracing at all when that happens.

Hi
Kudu will continue to run in Azure App Service. However, this repo will no longer be maintained.
If the problem persists and is related to running on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Empty sys.jobs table when running queries

I,
when a run a query (long or short) I see nothing in the sys.logs table:
cr> select * from sys.jobs;
+----+---------+------+
| id | started | stmt |
+----+---------+------+
+----+---------+------+
Crate release 0.56.3
Brgds

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/39035204-empty-sys-jobs-table-when-running-queries?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github).

Did you set stats.enabled to true?
See https://crate.io/docs/reference/sql/system.html#jobs-operations-and-logs
and https://crate.io/docs/reference/configuration.html#collecting-stats

Hello,
Thanks for your anwser, it’s working fine now.
Brgds
Cdt
De : Mathias Fußenegger<EMAIL_ADDRESS>Envoyé : mardi 8 novembre 2016 10:04
À : crate/crate
Cc : RIVIER Francois DTSI/DSI; Author
Objet : Re: [crate/crate] Empty sys.jobs table when running queries (#4393)
Did you set stats.enabled to true?
See https://crate.io/docs/reference/sql/system.html#jobs-operations-and-logs
and https://crate.io/docs/reference/configuration.html#collecting-stats
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/crate/crate/issues/4393#issuecomment-259082073, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV63Tt4fbxJXLPJ0t6J3XF-T4C6smqHSks5q8DrtgaJpZM4KrY8h.

Ce message et ses pieces jointes peuvent contenir des informations confidentielles ou privilegiees et ne doivent donc
pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu ce message par erreur, veuillez le signaler
a l'expediteur et le detruire ainsi que les pieces jointes. Les messages electroniques etant susceptibles d'alteration,
Orange decline toute responsabilite si ce message a ete altere, deforme ou falsifie. Merci.
This message and its attachments may contain confidential or privileged information that may be protected by law;
they should not be distributed, used or copied without authorisation.
If you have received this email in error, please notify the sender and delete this message and its attachments.
As emails may be altered, Orange is not liable for messages that have been modified, changed or falsified.
Thank you.


Run unit tests in a container locally

Summary
In a recent discovery, it was found that a hashing implementation we were using had behavior that varied across operating systems locally. In other words, our unit tests were failing if they were ran on Linux but were passing on MacOS. We should come up with a way that we avoid environment skew and gain confidence in our changes prior to opening PR's.
Proposed Solution
Update our local make test-unit target to run the unit test suite inside of a container and thus avoid environment skew. This will avoid issues where the tests pass locally but fail in CI.

This relates to #423 and was originally motivated by the non-determinism we saw in the hashing implementation.
Due to the high startup costs to the unit tests, and maintaining this solution over time, it does not seem likely that we would like to implement this. The SDK team also fielded a similar issue and decided not to containerize their unit tests.


Executables in package losing execute permissions after update

OS
MacOS Sonoma 14.6.1 (23G93)
Programming Language
8.0.403
VPK Version
0.0.869
Library Version
?
What happened?
As part of the C# DOTNET application files, we also bundle some third party executables inside a libs directory.
On MacOS, upon first installation of the program, the permissions for the executables appear to be set correctly, however when a delta update is received, the executable flag is removed.
The executables are not being changed as part of the update release.
In this example, I am demoing with Exiftool.
Before update (if installed for all users):
-rwxr-xr-x@ 1 root  wheel   319K Oct  4 10:03 exiftool
Before update (if installed for current user only):
-rwxr-xr-x@ 1 my_user staff   319K Oct  4 10:03 exiftool
After update (in both installation scenarios):
-rw-r--r--  1 my_user  staff   319K Nov  4 13:07 exiftool
Relevant log output
No response

Velopack does not currently persist/preserve the file mode on MacOS when you run vpk pack, but this feature is being worked on in #63.
The current behavior is to just chmod 755 anything that looks like it could be a mach-o executable.
I gather exiftool is not actually a binary? Is it a shell script or similar? If you think it is a valid binary, can you please upload an example and let me know how it was compiled so I can modify the detection code?

Velopack does not currently persist/preserve the file mode on MacOS when you run vpk pack, but this feature is being worked on in #63.
The current behavior is to just chmod 755 anything that looks like it could be a mach-o executable.
I gather exiftool is not actually a binary? Is it a shell script or similar? If you think it is a valid binary, can you please upload an example and let me know how it was compiled so I can modify the detection code?
Perhaps a workaround could be to set 755 on all the files we extract until the proper solution is available?

Thanks for the super quick response!
The exiftool package can be downloaded here.
Apologies, you're right, the exiftool file is not a binary, it's a perl script.
Happy to work around by applying permissions in the meantime, would you recommend using the App Hooks to automate that?

App hooks also don't work on Mac unfortunately. Also, I think modifying the file mode will require root and/or might even break your app bundle signature.
I think it might be good enough for Velopack to set all files to 755 for now, until a better solution can be implemented.


Origin support fixes

Origin support added to the application does not obtain the proper path from the registry values. As I do not use the application this bug is open pending community feedback regarding correct path. liborigin simply needs updated with proper key string in order to function correctly.

There doesn't seem to be anything about custom installation directories in the registry. However, there is a file called "Client_Log.txt" in ProgramData which keeps track of all the activities done through the client, including changing the installation directory. The custom libraries can found by using that.
BTW, since Origin logs all changes, it also logs mistakes; including directories which can hold unrelated data. This can be hazardous if it is not handled correctly. It is a clunky solution, but it is the best I could find.
(FYI, Origin has a download cache which stores installers, but the GUI provides an option to keep or remove them.)

@sam97 I actually do not use Origin as most of my stuff is on Steam or GoG. I can check this out via a VM when I get some time. I will have to check if they have some free games I can install similar to Steam though.

No worries, take your time. I have exams coming, too.
Here's something free.

Thank you for the link, I will give it a look.


added alternative call curve coloring

Added additional coloring mode for call curves for better discriminability.
New globally applied option is available via context menu of calls:

Summary of Changes

References and Context

Test Instructions

/format

/format

/format


error when importing module

I am using VMware vSphere PowerCLI6.0 Release 3 and copied the Vi-Module to the modules folder
ProgramFiles(x86)/VMware/Infrastructure/vSpherePowerCLI/Modules
when I run the Import-Module Vi-Module -Force -Verbose I get the attached errors - should this work with powercli? or am I missing something?
Thanks

Check the $PSVersionTable variable, you have to upgrade the PowerShell to
V5 or remove ", Vi-Sdrs.ps1" from Vi-Module.psd1 file and lose SDRS related
functions.
The choice is yours.
https://ps1code.com
On 17 Oct 2017 9:26 PM, "lorriEDw"<EMAIL_ADDRESS>wrote:

I am using VMware vSphere PowerCLI6.0 Release 3 and copied the Vi-Module
to the modules folder
ProgramFiles(x86)/VMware/Infrastructure/vSpherePowerCLI/Modules
when I run the Import-Module Vi-Module -Force -Verbose I get the attached
errors - should this work with powercli? or am I missing something?
Thanks
[image: screen shot 2017-10-17 at 2 11 37 pm]
https://user-images.githubusercontent.com/28764879/31681676-f556c6ba-b345-11e7-88f3-db327a526d38.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rgel/PowerCLi/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGpFRf2CJbCtHM0J4PlJrCaYFDnXvTnXks5stPADgaJpZM4P8m3e
.

Duplicate Issue. Using PS Version 5.1, also removed Vi-Sdrs.ps1 script. Does not seem to matter, seems there is a valid error in the Expand-VMGuestPartition Function.

It seems you had Copy-Pasted the RAW file from Github, try "Clone or download" button from the main PowerCLi repo page.

The download was taken from the main PowerCLi page. But you gave me another idea to copy-paste from the RAW file from Github. That worked, the code is error free. Thanks.

Happy scripting
https://ps1code.com
On 14 Nov 2017 15:13, "prisoner107"<EMAIL_ADDRESS>wrote:

The download was taken from the main PowerCLi page. But you gave me
another idea to copy-paste from the RAW file from Github. That worked, the
code is error free. Thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rgel/PowerCLi/issues/4#issuecomment-344254732, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGpFRQtFk-4otVmQAuLX7DM5nM8SirXSks5s2ZHfgaJpZM4P8m3e
.

without  Vi-Sdrs.ps1

If I had Copy-Pasted the RAW file from Github import module had been done.

This is known issue, sometimes files would be damaged while downloading from the GitHub :-(
There are two download methods: copy-paste raw files or download a whole repository zip file

The reason is your enterprise proxy or antivirus software.
Try to use different download method or download outside your organization.

I found that in Vi-Module.psm1 there was a  ANSI character â€“ that is displayed as - in UTF-8 encoding in 3 places. Once I replaced â€“ with - it worked as expected.


[BUG] Updating the copyright year at the footer of the website to 2022 from 2021

Description of the bug
Since the copyright year in the license is updated to 2022 we should also update it in the website's copyright.
Please add screenshots (if applicable)
No response
Add any other context about the problem here
No response
Code of Conduct

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

I would like to do this

Looks like there's a problem with the build or code 👀 I'll see what I can do and if it's easy to fix. Thanks.

Looks like there's a problem with the build or code 👀 I'll see what I can do and if it's easy to fix. Thanks.

Okay cool

Go to src/_includes/partials/footer.html.
In line 94-96, replace the current code with the following code:

<small>Copyright © {{ settings.name }} <script>document.write(" " + new Date().getFullYear())</script>. Made with ❤ by

That's it! Now just make a PR and we'll check it. :)

Thank you so much : )


Add beamDeps argument to rebar3Relx

Motivation for this change
Providing compiled dependencies via beamDeps means faster rebuilds, since the dependencies don't have to be recompiled every time and can be cached. The first commits are from #122633 which needs to merged first.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Result of nixpkgs-review pr 123384 at 6341e922 run on x86_64-linux 1

1 package marked as broken and skipped:

lfe_1_2

1 package skipped due to time constraints:

tests.trivial

5 packages built successfully:

elixir_ls
erlang-ls
lfe (lfe_1_3)
rebar3
relxExe

12 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near lib/attrsets.nix:212:41:
    |
212 |   nameValuePair = name: value: { inherit name value; };
    |                                         ^

warning: unused-argument
Unused argument: rebar3WithPlugins.
Near pkgs/development/beam-modules/erlang-ls/default.nix:1:38:
  |
1 | { fetchFromGitHub, fetchHex, stdenv, rebar3WithPlugins, rebar3Relx, buildRebar3
  |                                      ^

warning: unused-argument
Unused argument: writeText.
Near pkgs/development/beam-modules/rebar3-release.nix:1:11:
  |
1 | { stdenv, writeText, writeScript, erlang, rebar3WithPlugins, openssl,
  |           ^

warning: unused-argument
Unused argument: stdenv.
Near pkgs/development/beam-modules/erlang-ls/default.nix:1:30:
  |
1 | { fetchFromGitHub, fetchHex, stdenv, rebar3WithPlugins, rebar3Relx, buildRebar3
  |                              ^

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/development/tools/build-managers/rebar3/default.nix:47:5:
   |
47 |     buildPhase = ''
   |     ^

warning: unused-argument
Unused argument: writeScript.
Near pkgs/development/beam-modules/rebar3-release.nix:1:22:
  |
1 | { stdenv, writeText, writeScript, erlang, rebar3WithPlugins, openssl,
  |                      ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/build-managers/rebar3/default.nix:51:5:
   |
51 |     installPhase = ''
   |     ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/beam-modules/rebar3-release.nix:85:5:
   |
85 |     meta = {
   |     ^

warning: name-and-version
Did you mean to pass pname instead of name to mkDerivation?
Near pkgs/development/beam-modules/rebar3-release.nix:40:5:
   |
40 |     name = "${name}-${version}";
   |     ^

Near pkgs/development/beam-modules/rebar3-release.nix:41:12:
   |
41 |     inherit version;
   |            ^

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near lib/attrsets.nix:212:41:
    |
212 |   nameValuePair = name: value: { inherit name value; };
    |                                         ^

warning: license-missing
Package is missing a license.
Near pkgs/development/beam-modules/rebar3-release.nix:85:5:
   |
85 |     meta = {
   |     ^

warning: unused-argument
Unused argument: enableDebugInfo.
Near pkgs/development/beam-modules/rebar3-release.nix:17:3:
   |
17 | , enableDebugInfo ? false
   |   ^

I'm essentially good to merge this. Let's leave a couple more days open though for more people to manifest and test.
@gleber if you have a moment and you want to give this a try perhaps?


Soft Links to not properly convert for users of Team Explorer Everywhere

When converting a repo that has soft link files, they do not get the attributes preserved in git.  Further executable attributes also do not get preserved in git.
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (64-bit)
Note: If you want to force git-tfs to use another version of the tfs client library,
set the environment variable GIT_TFS_CLIENT with the wished version (ie: '2013' for Visual Studio 2013,...)

I believe this issue is related to #748 - where in the project would the file be annotated as a junction or soft-link?

@dgtlrift did you get any work around for this

We are trying to clean up old/stale issues. As this issue hasn't seen any updates (or progress with pull requests) in quite a while, the issue will be closed. In case further information is provided, we can revisit that and reopen the issue again.


[Support]: Adding different 3D arrow models to the Virtual Tour Plugin and changing their vertical position

What are you trying to do?
Hi,
I am developing an web application using this library and the Virtual Tour Plugin, but I have not been able to customize the 3D arrow models and give specific values to their vertical position. Can you provide information on how I can do this?
Best Regards
Library Version
3.4.0-psv5.4.0
What operating system are you using?
macOS
What browser are you using?
Chrome

Hi! Thanks for your issue, but I have to close it because it's related to the main library photo-sphere-viewer by Damien Sorel. Consider to ask him on his repo.
Thanks

Hello again
In the main library we customized the 3D arrow and put our own model and it worked very well. After forking the main library and putting the arrow we customized, we want to use the arrow we customized through this React library. How can we give a path the library we forked in this library.

Hey! The only way Is to fork also my repo and use It as local lib.
Of course Is not really comfortable for the mainteinment of your project. I
suggest to make a PR on the main lib with a custom prop or method to change
the Arrow model.
This should be a nice improvement for photo-sphere-viewer.
Thanks
Il gio 28 dic 2023, 20:22 Mete Çözer @.***> ha scritto:

Hello again
In the main library we customized the 3D arrow and put our own model and
it worked very well. After forking the main library and putting the arrow
we customized, we want to use the arrow we customized through this React
library. How can we give a path the library we forked in this library.
—
Reply to this email directly, view it on GitHub
https://github.com/Elius94/react-photo-sphere-viewer/issues/42#issuecomment-1871433956,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADRXUU72O23AZI4C3EYTNK3YLXBIRAVCNFSM6AAAAABATC3FLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZRGQZTGOJVGY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hey,
I'm sorry to have taken up your time. I don't think I'm talented enough to contribute to an open source project yet, but I will definitely try to realize this idea in the future. I'm interested in the idea of being able to fork and use both repos, would you mind explaining the milestones of this process in general?


Discord link is expired

https://discord.gg/6KaKCKds ->

Ah, I just saw this issue. There has been no activity on the Discord unfortunately so if you have any questions, please message me on X (https://x.com/rudrankriyam) or LinkedIn (https://www.linkedin.com/in/rudrank/)


docs: Update folder download documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 86.552%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
4174

Relevant Lines:
4671

💛  - Coveralls


Inconsistencies in Django project template urlpattern regex

Perhaps this is intentional, but the Django App Project template's urls.py (for the Django project) has the following:
url(r'^contact$', app.views.contact, name='contact'),
url(r'^about', app.views.about, name='about'),
# ...
url(r'^logout$', ...)

Is there a reason any r'^about' isn't r'^about$'?
Similarly, in the Polls Django Web Project template, there's an added r'^seed':
url(r'^about', app.views.about, name='about'),
url(r'^seed', app.views.seed, name='seed'),

Should that be r'^seed$'? As it stands, requests to /abouttime and /seedsareinterestingthings are picked up by these patterns.
I'm happy to make a PR if they should be changed for consistency.

Looks like oversight. We'll take your PR if you have time to fix it. Thanks!

Done, #4038


feat: add IsForbiddenErr error checking for 403s

Codecov Report
Base: 52.43% // Head: 35.91% // Decreases project coverage by -16.51% :warning:

Coverage data is based on head (ebd3e33) compared to base (e0fcd03).
Patch coverage: 83.33% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #264       +/-   ##
===========================================
- Coverage   52.43%   35.91%   -16.52%
===========================================
  Files          51       51
  Lines        4621     4627        +6
===========================================
- Hits         2423     1662      -761
- Misses       1670     2664      +994
+ Partials      528      301      -227

Flag
Coverage Δ

2.1.4
35.91% <83.33%> (+0.62%)
:arrow_up:

<IP_ADDRESS>
?

2.2.2
?

2.3.3
?

<IP_ADDRESS>
?

2.4.0
?

<IP_ADDRESS>
?

2.5.1
?

<IP_ADDRESS>
?

2.6.0
?

<IP_ADDRESS>
?

2.7.0
?

<IP_ADDRESS>
?

2.8.0
?

<IP_ADDRESS>
?

3.0.1
?

<IP_ADDRESS>
?

3.1.0
?

<IP_ADDRESS>
?

community
35.91% <83.33%> (-2.84%)
:arrow_down:

enterprise
?

integration
35.91% <83.33%> (-16.52%)
:arrow_down:

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

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Append to csv/file on update

Is there a way to instead of running the script and it downloading and saving output to a file each time, we can just append the latest tweets into the file?  Some way of comparing what we have versus what we're missing?
Thanks

Hi and thanks for the hint, now as now you can use the since and until args to filter out a specific date range and so use the until arg of the day before of the latest tweet. The day before because you may not have been able to download all the tweets of the "latest day".
Until we don't provide a solution for your situation I suggest you to proceed in that way, hope this helps.


The search term "(Pantun" returns a blank page

Not sure if this should be considered an issue because I don't think the term "(Pantun" should even be in the database. The term "(Pantun" exists in the database now because I want to keep track which pantun is part of a collection. For example, the collection of Pantun Sarawak. If you search for "pantun sarawak", the API will return pantuns that are part of the collection.
For future improvements, we can implement search by tags and the collection names will be the tag names.
With that being said, I do believe this is a bug. I haven't tested for any other punctuations except the comma (,). The comma works well.
If anyone find other punctuations that error, please reply to this issue, thanks!

The ( is part of special characters for regular expressions. Therefore, it must be escape using backslash before adding to the RegExp function.
See #2 for solution.

Yup, looked it over and everything looks great, merged!


Update CrowdSourcedScience.netkan

A fat lot of good the console does now.

#rebuild

#rebuild

#rebuild

#rebuild

Your latest release ( https://github.com/DuoDex/CrowdSourcedScience/releases ) does not contain any zip files, which we really require to install an addon from. Please fix your release and observe that the netkan-bot will index it.


Add description to builder config

inspect-builder
Given I run pack create-builder my-builder -b builder.toml with the following field in the builder.toml file
description = "A short description of my builder"

When I run pack inspect-builder my-builder then I observe the following field in the output
Desciption: A short description of my builder

suggest-builders w/ description
Given a suggested builder has been created with a description
When I run pack suggest-builders
Then The desciption field in the output will be populated with the configured description
suggest-builders w/o description (Backwards Compatibility)
Given a suggested builder has an empty description
When I run pack suggest-builders
Then The current hard-coded description will be printed

@ameyer-pivotal  Validated cases 1 and 3, but don't know how to modify the suggested builders.  Would I just docker pull them, add a description, and then run pack create-builder?  I also don't want to mess with them as they are the "canonical" builders.

That's a good point... @sclevine @ekcasey Are y'all OK deferring part 2 acceptance and just validating it when we create new builders? It only looks at remote builders and we don't want to mess with the published ones just yet.

This story is accepted

done on master, will ship in v0.2.0


Remove deprecated file-based watchdog and graceful shutdown triggers

What this PR does / why we need it:
Remove deprecated file-based watchdog and graceful shutdown triggers. The PR which did the change is now more than two years old (https://github.com/kubevirt/kubevirt/pull/3212), and there is no chance that a <0.29 KubeVirt deployment is still around.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
A preparatory cleanup to make restarting  the work on something like https://github.com/kubevirt/kubevirt/pull/3311 easier.
Release note:
NONE

/cc @davidvossel
/cc @xpivarc

/retest

/hold
while this change is closed in itself, there are still a lot of old code paths left.

/remove-lifecycle rotten


Fix docker ext release

The docker extension release action is also responsible for updating the base image used by the docker extension, due to a bug that was introduced when manual trigger was added to this github action the automatic updating part didn't work. This patch fixes it

Tested this by manually running the github action
https://github.com/headlamp-k8s/headlamp/actions/runs/4486610733/jobs/7889256019


Can you please support datasets without multiple choice and personality?

Hello,
You've done good work. But most datasets out there don't have multiple candidates or personalities. Can you please have an option to support this kind of datasets--datasets that only contain the history of the conversation? Thank you.

Could definitely try.
What would your preferred input format be? See example_entry.py for the current format.

I think a popular format is a .txt file where each line corresponds to a single dialogue, within each turn is separated by some delimiter (either a tab-token '\t', end of token '', etc...).
For example (sorry for the very boring conversation):
Hey, how are you doing?</t>I'm good how about you?</t>Prety well, what's new?</t>Nothing much.
Hey! Sup?</t>Not much, you?</t>

Hello, @sshleifer. Can you please support this kind of data?

Unlikely. I think we might eventually support not passing persona, and you can pass num_candidates=1, but I don't see huge value in supporting different raw data formats, given how many different ones there are.

So, an example entry would look like this?
EXAMPLE_ENTRY = {
    "utterances": [
        {"candidates": [
            "I'm good how about you?"
        ],
        "history": [
            "Hey, how are you doing?"
        ]
        },
        {"candidates": [
            "Prety well, what's new?"],
         "history": [
            "Hey, how are you doing?",
            "I'm good how about you?"]
         },
        {"candidates": [
            "Nothing much."],
         "history": [
             "Hey, how are you doing?",
            "I'm good how about you?",
            "Prety well, what's new?"
            ]
         },
    ]
}

@sshleifer ?

LGTM

Okay 👍 Thanks!

I actually decided to go with changing this line:
sequence = [[bos] + list(chain(*persona))] + history + [reply + ([eos] if with_eos else [])]
to:
sequence = [[bos]] + history + [reply + ([eos] if with_eos else [])]
And to make the following changes:
Deleting: for _ in range(args.personality_permutations): and other stuff related to pesona.
def get_data_loaders(args, tokenizer):
    """ Prepare the dataset for training and evaluation """
    personachat = get_dataset(tokenizer, args.dataset_path, args.dataset_cache)

    logger.info("Build inputs and labels")
    datasets = {"train": defaultdict(list), "valid": defaultdict(list)}
    for dataset_name, dataset in personachat.items():
        num_candidates = len(dataset[0]["utterances"][0]["candidates"])
        if args.num_candidates > 0 and dataset_name == 'train':
            num_candidates = min(args.num_candidates, num_candidates)
        for dialog in dataset:
            for utterance in dialog["utterances"]:
                history = utterance["history"][-(2*args.max_history+1):]
                for j, candidate in enumerate(utterance["candidates"][-num_candidates:]):
                    lm_labels = bool(j == num_candidates-1)
                    instance, _ = build_input_from_segments(history, candidate, tokenizer, lm_labels)
                    for input_name, input_array in instance.items():
                        datasets[dataset_name][input_name].append(input_array)
                datasets[dataset_name]["mc_labels"].append(num_candidates - 1)
                datasets[dataset_name]["n_candidates"] = num_candidates

Should that work?

After a lot of testing, it seems like when num_candidates=1 is set and the actual size of the candidates list in the dataset is 1, there's an error relating to the size of tensors during the evaluation of the model (validation). Training is alright. @sshleifer

Closing, reporting problem at #40.

Hey @dimeldo  Were you able to train it finally by passing empty personalities? Also, how did you resolve the num_candidates issue?
@sshleifer


No Mapping Found for http request error

My pom.xml is the same, but for some reason I had to add an asterisk to
<context:component-scan base-package="com.in28minutes.*" />
in todo-servlet.xml (Step 12)
http://stackoverflow.com/questions/28316187/org-springframework-web-servlet-pagenotfound-nohandlerfound-no-mapping-found-for
I have Luna Service Release 2 (4.4.2)

Hi Mark,
Interesting find. Can you give me the specific version of Spring MVC & Spring you are using?
Regards,
Ranga


Updating WildFly to 11.0.0.Final and to latest Lucene micro fix

https://hibernate.atlassian.net/browse/HSEARCH-2927
https://hibernate.atlassian.net/browse/HSEARCH-2928

Hmm, is CI down or what? The build didn't trigger automatically...

Jenkins, test this please

Merged, thanks!

@Sanne I am backporting right now, but I see you didn't include 5.6 in the versions we should backport HSEARCH-2927... is it on purpose? I know it's old, but since it's the last version supporting ORM 5.1...


Update docker hub 2.4.x tags along with latest?

Hi,
I just noticed that the 2.4.x tags on docker hub are quite a way behind latest.
It looks like you've adopted the openldap version for the container, seeing as there's several 2.4.x tags. As changes here aren't factored into the docker hub tag, do you think it'd make sense to update those tags along with latest?

FYI I've forked this project with intent to provide better tagging on Images. Take a look at https://hub.docker.com/r/jlkinsel/openldap/ - open to comments/suggestions.


Assign Power Apps per app baseline license

seems this part is being missed from above instructions

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

ID: 0f1f9ece-05b6-907c-9d52-5a19f5bf9872
Version Independent ID: 9eaed30a-4c07-ef03-e3b4-d3db9989a760
Content: About Power Apps per app plans - Power Platform Admin center
Content Source: power-platform/admin/about-powerapps-perapp.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

We will be updating the licensing documentation in the future to address.


修正错别字

但果 -> 如果

你好，这里原文确实是有however的意思在，是不是改成类似于“但是，如果。。。那就实在是太糟糕了”更通顺一些？
However, it would be very bad if the statement balance was not equal to the sum of the transactions plus the previous statement balance (an error in the sums), or if a transaction was charged to you but not paid to the merchant (disappearing money). Such problems would be violations of the integrity of the system.

你好，这里原文确实是有however的意思在，是不是改成类似于“但是，如果。。。那就实在是太糟糕了”更通顺一些？
However, it would be very bad if the statement balance was not equal to the sum of the transactions plus the previous statement balance (an error in the sums), or if a transaction was charged to you but not paid to the merchant (disappearing money). Such problems would be violations of the integrity of the system.

我觉得你的改法更好一些，也符合对原文的翻译，我以为这里是写错了字呢

确实是写错了：）
好的，那我就关闭这个 PR，直接修改了哈，谢谢发现这个问题！


Request: Inline actions within text

Summary
There are quite a few scenarios where it makes sense to have an action displayed as a link on a part of the text in a TextBlock. As an example, GitHub is showing this as I write this issue:

In that screenshot, contributing is just a link, but in the context of Adaptive, card authors may want to hook it to a Submit or ShowCard action (or even Http action in Outlook.)
Asks

Microsoft Teams
Outlook

Shipped in 1.2!

Is there a sample of this?  I saw the release notes but couldn't figure out how to do this.


Add two CICE setting files of ice_in for initial/continueous restart

Provide two setting ice_in files for initial and continue restart in CICE of TP5.
If setting tr_pond_lvl  = .false., it will switch off melt pond as the albedo_type uses constant.

Add the explain comment at the head.

@Jipingnersc  Actually it seems that there is no need to have two ice_in files for "restart" and "initial".
You can always have, runtype  = 'restart'.  Then in the case of initialization from climatology, the script "expt_preprocess.sh" will be called with "--INIT" which will modify ice_in file by changing "runtype"   to  'initial' and copy it under SCRATCH

@Jipingnersc  Actually it seems that there is no need to have two ice_in files for "restart" and "initial".
You can always have, runtype  = 'restart'.  Then in the case of initialization from climatology, the script "expt_preprocess.sh" will be called with "--INIT" which will modify ice_in file by changing "runtype"   to  'initial' and copy it under SCRATCH

@Jipingnersc Actually it seems that there is no need to have two ice_in files for "restart" and "initial".
You can always have, runtype = 'restart'. Then in the case of initialization from climatology, the script "expt_preprocess.sh" will be called with "--INIT" which will modify ice_in file by changing "runtype" to 'initial' and copy it under SCRATCH
Thanks Alfati. Great, I tested as your mentioned without the special initial setting of ice_in. The concerned script can automaticly work so this branch is not necessary at present.

@Jipingnersc Actually it seems that there is no need to have two ice_in files for "restart" and "initial".
You can always have, runtype = 'restart'. Then in the case of initialization from climatology, the script "expt_preprocess.sh" will be called with "--INIT" which will modify ice_in file by changing "runtype" to 'initial' and copy it under SCRATCH
Thanks Alfati. Great, I tested as your mentioned without the special initial setting of ice_in. The concerned script can automaticly work so this branch is not necessary at present.


Use ScriptureReference more consistenly (in text collection extension)

There's multiple ways to represent scripture references being used in our code.

The VerseRef class from @sillsdev/scripture
The ScriptureReference type from scripture-util
Creating an object in place that kinda looks like ScriptureReference

Ideally we would use VerseRef everywhere, but since that poses some challenges we've decided to not do that for now. Instead we will use ScriptureReference everywhere, consistently. Being consistent allows for an easier change to VerseRef when it is time to do that.

Ideally we would use VerseRef everywhere, but since that poses some challenges we've decided to not do that for now.

Can you tell me what the challenges are? Since I'm the current maintainer of @sillsdev/scripture I'd like to know what needs improving?

I think @tjcouch-sil can tell you more about it, but IIRC it has to do with VerseRef being a class and not a type


Update multi tasks docs

Motivation and context

Updated styles on create multi tasks form. Margin seems to be too large, reduced it a bit

Updated documentation text (there was some weird description about underscores if task has name with several words, but cvat doesnt have such restrictions, we can use spaces) + some minor fixes
Updated documentation screenshots with new UI

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have created a changelog fragment
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

New Features

Enhanced user interface for task name fields with clearer tooltip instructions.
Updated documentation for creating multi-tasks and annotation tasks, improving clarity and usability.

Bug Fixes

Updated visual references in documentation to ensure relevance and clarity.

Documentation

Improved instructions and visuals in the "Create multi tasks" and "Create annotation task" manuals.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.09%. Comparing base (c9754a9) to head (9d240c3).
Report is 15 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8516      +/-   ##
===========================================
- Coverage    74.34%   74.09%   -0.26%
===========================================
  Files          394      396       +2
  Lines        42216    42774     +558
  Branches      3896     3897       +1
===========================================
+ Hits         31387    31693     +306
- Misses       10829    11081     +252

Components
Coverage Δ

cvat-ui
78.72% <ø> (-0.02%)
:arrow_down:

cvat-server
70.10% <83.33%> (-0.37%)
:arrow_down:


Add Javadoc on many Spring Cloud Core project sites

Go to the home page of Spring Cloud Task project. In the documentation table on the right, the API links points to the Javadoc for Spring Cloud Task. This is what I would expect.
Yet in most of the other Spring Cloud projects I've seen so far, the API links point to a wrong resource (either the the associated Github repo or the Reference documentation). For example, go to Spring Cloud Security. The API links for releases 1.1.4, 1.1.5 and 1.2.1 point to the Github repo. The API links for 1.2.2 and 1.2.3 point to the Reference documentation.
Similar behavior for (e.g. Spring Cloud Config, Spring Cloud Netflix, Spring Cloud AWS, etc.
The convention for working projects (throughout the Spring ecosystem) seems to be that javadoc is hosted at https://docs.spring.io/, e.g.: https://docs.spring.io/spring-boot, https://docs.spring.io/spring-cloud-task. For the errant sites above, there is no such javadoc, e.g.: https://docs.spring.io/spring-cloud-config, https://docs.spring.io/spring-cloud-netflix, https://docs.spring.io/spring-cloud-aws, etc., all produce 404s.
It would be helpful if all spring projects generated and deployed javadoc to docs.spring.io, with correct links to them on their associated project sites.

Spring Cloud Core projects (Build, Netflix, Commons, Config, Bus, Release, Aws, Cloudfoundry, Consul, Contract, Gateway, Security, Sleuth, Vault, Zookeeper, Pipelines) do not publish documents to http://docs.spring.io.
We're thinking of setting links to javadoc.io like this: https://www.javadoc.io/doc/org.springframework.cloud/spring-cloud-commons/1.2.3.RELEASE
Spring Cloud Task and Spring Cloud Stream are not part of core and have separate project teams, you'll need to submit separate issues there.

SC Task and SC Stream publish API artifacts already (like most Spring projects), so there is no need to open issues  for those projects.


feat(scaffolder): add first class citizen support for zod

Hey, I just made a Pull Request!
https://github.com/backstage/backstage/issues/26933

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Not stale

@benjdlambert @Rugvip PTAL?

Not stale, been busy for KubeCon.

@schultzp2020 just to recap, we were chatting how we can get a zod schema from Record<string, ZodType> right?
Can we not just do something like:
const zod = z.object(
  Object.fromEntries(
    Object.entries(input).map(
      ([k, v]) => [k, v(z)]
     )
  )
)

Obviously for both input and output schemas, but that should allow us to use zod-to-json-schema to generate the view model, but all validation just goes through the zod object.

@schultzp2020 just to recap, we were chatting how we can get a zod schema from Record<string, ZodType> right?
Can we not just do something like:
const zod = z.object(
  Object.fromEntries(
    Object.entries(input).map(
      ([k, v]) => [k, v(z)]
     )
  )
)

Obviously for both input and output schemas, but that should allow us to use zod-to-json-schema to generate the view model, but all validation just goes through the zod object.

Thank you for the idea! I haven't thought of that.

@benjdlambert the PR is ready for review.

@benjdlambert After further investigation, I believe overloading createTemplateAction is not a good idea. The generic types differ significantly, causing numerous type errors. In the long term, maintaining an overloaded function would likely be far more challenging. Instead, it would be better to properly deprecate the old functionality and encourage the use of the new function.


fix 3.2 based unit tests

@kwilcox @daf @brianmckenna our Travis-CI tests are constantly failing against 3.2 builds starting from https://travis-ci.org/geopython/OWSLib/builds/81882404:

https://travis-ci.org/geopython/OWSLib/jobs/89142187
https://travis-ci.org/geopython/OWSLib/jobs/89142188

Any idea what the issue could be?

Hi @tomkralidis, it looks like the coverage library has an issue with Python 3.2, according to what I'm reading here: menegazzo/travispy#20
Supposedly 3.7.1 runs on Python 3.2 OK - but I'm not sure how or if we want to conditionally install coverage based on Python versions.  Thoughts?  @ocefpaf ?

I see the some u'' uses in some places to declare unicode.  I believe that was re-introduced in Python 3.3.
BTW:  There is an unwritten rule that Python 3 starts at 3.4.  I think is no one will be hurt if you stop testing against 3.2.

Should we remove 3.2 and 3.3 tests?

I'd say yes.
On Fri, Nov 13, 2015 at 1:28 PM, Tom Kralidis<EMAIL_ADDRESS>wrote:

Should we remove 3.2 and 3.3
https://github.com/geopython/OWSLib/blob/master/.travis.yml#L5-L6 tests?
—
Reply to this email directly or view it on GitHub
https://github.com/geopython/OWSLib/issues/291#issuecomment-156509924.

Done.  Let's see how the tests go.

Funny thing that I found out because of this issue.  You can:
conda create -n PY1 python=1
But you cannot:
conda create -n PY32 python=3.2
conda create -n PY31 python=3.1

(Python 3.3 is still alive and breathing, but barely.)


Grafana ChargeLevel Dashboard: Usable battery alterantes between usable battery and battery

Describe the bug
The usable battery graph in the ChargeLevel Grafana dashboard alternates between usable battery and battery. The reason is a wrong SQL statement which let the usable battery value be the battery value if usable battery is null.
Expected behavior
The usable battery values should be connected to each other directly
How to reproduce it (as minimally and precisely as possible):

Go to the Charge Level Grafana Dashboard
Go to a timespan of max 3 hours, where you drive while having a lower usable battery than battery value, to clearly see the issue
You see the usable battery graph alternating between usable battery and battery

Screenshots
current situation:

Solution
Replace coalesce(usable_battery_level, battery_level) in the second line of the SQL-statement with usable_battery_level and set Stacking and null value --> null value to connected
Then you will get the following:

TeslaMate version: 1.21.2
Type of installation: Docker

If I remember correctly, the SQL query is written this way for historical reasons. Initially, the usable battery range was not recorded, so this fallback was built in. By now, it should be no problem to adjust the query, though.

Is there really an issue if you set null value to connected? I don't think so, I think if there are only null values, there is no 'usable battery' line in the diagram, so wouldn't be an issue, even if usable battery was not recorded earlier. But I am not 100% sure.


[6.0 🍒][Dependency Scanning] Emit diagnostics on scan query failure during initialization or cycle detection

Explanation: https://github.com/swiftlang/swift/pull/71907 was an important fix that moved the diagnostics collected during dependency scanning to be associated with an individual scan query, instead of scanner overall. This meant that multiple concurrent scans launched by libSwiftScan clients would have isolated diagnostic domains. However, upon query failure, either when constructing a scanning instance (e.g. un-recognized scanner argument) or upon encountering and diagnosing a dependency cycle, the new per-scan diagnostic state was lost and the scanner query returned an error.
This means that scanner clients (SwiftDriver) would no longer see diagnostics from such failures. With this change, on dependency scan query failure, we ensure any diagnostics that the compiler has accumulated along the way are still produced to the client, by constructing a hollow output object that contains just enough state to also carry the diagnostic error payload.

Scope: This change primarily affects Explicit Module Builds (not default, opt-in with -explicit-module-build) which were already going to fail. On such failures the client would simply get an error from the dependency scanning query without additional context, and will now produce scanner-emitted diagnostics.

Risk: Low. Due to the scope of this change being largely limited to compilation tasks which were already going to fail, and only on opt-in Explicit Module Builds

Original PR: https://github.com/swiftlang/swift/pull/75111

Issue/Radar: rdar://130897498

Reviewed By: @cachemeifyoucan

@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test

https://github.com/swiftlang/swift-driver/pull/1659
@swift-ci test


How to train with CUDA in AWS

I tried to use GPU in AWS and failed.
I used g2.2xlarge with Amazon Linux AMI with NVIDIA GRID GPU Driver, which I think the easiest way to use cuda.
The commands I tried:
#install Anaconda
wget https://3230d63b5fc54e62148e-c95ac804525aac4b6dba79b00b39d1d3.ssl.cf1.rackcdn.com/Anaconda-2.2.0-Linux-x86_64.sh
bash Anaconda-2.2.0-Linux-x86_64.sh -b
echo 'export PATH=$HOME/anaconda/bin:$PATH' >> .bashrc
echo 'export PYTHONPATH=$HOME/anaconda/lib/python2.7/site-packages:$PYTHONPATH' >> .bashrc
source .bashrc
conda update conda -y
#install chainer
pip install chainer
sudo apt-get install git
git clone https://github.com/pfnet/chainer.git
pip install -r chainer/chainer/requirements/cuda-requirements.txt
python chainer/examples/mnist/train_mnist.py --gpu 1
The error I encountered:
Traceback (most recent call last):
File "chainer/examples/mnist/train_mnist.py", line 40, in
cuda.init(args.gpu)
File "/home/ec2-user/anaconda/lib/python2.7/site-packages/chainer/cuda.py", line 119, in init
device  = Device(device)
pycuda._driver.LogicError: cuDeviceGet failed: invalid device
Could you help me?
I think it is better to release the official chainer AMI for GPU instances.

Hi.
Did you try device whose id is 0? If not, I recommend you try --gpu 0 instead of 1.

It worked! I should have noticed that id starts from zero. Thank you.

Great! Thank you for trying Chainer!


Unable to change path of index.android.js

I need to change path of index.android.js, By default it access the file from root of project created (created by react native), I want to put it in asset folder of android structure.

I think you should override this method https://github.com/facebook/react-native/blob/master/ReactAndroid/src/main/java/com/facebook/react/ReactActivity.java#L60 and return appropriate path to your index in new location.

I already tried this, but as you suggest, I'll again check it out. I'll get back to you soon.

Yes its working, there was some problem in path last time.


Update application-gateway-troubleshooting-502.md

In Line 44, please update it to this. Adding relevant documentation for app gateway subnet NSG configurations.

Check NSGs associated with the application gateway subnet. Ensure that communication to the backend isn't blocked. For more information, please refer to https://learn.microsoft.com/en-us/azure/application-gateway/configuration-infrastructure#network-security-groups

@greg-lindsay
Can you review the proposed changes?
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

#sign-off


Tolerate not being able to connect to server?

Hi,
I'm interested in being able to deploy the diskover client (i.e. the worker and the crawlers) separately from the diskover-web stack (redis, ES, and the PHP app), but as far as I can tell, the worker bots currently have no way of tolerating the server going away by e.g. queueing up POSTs and trying again on a timer or when the network environment changes. Is that true? Would there be plans to change this?
Thanks!

diskover can be separate from diskover-web. diskover requires Elasticsearch and Redis, but those also could be installed on separate hosts or in their own docker container. diskover-web requires just php and ideally nginx or apache. diskover-web also needs a connection to that same Elasticsearch where the indices are that get created by diskover. worker bots (which are different than diskover tree walk client), can be on any host in your network, they are not required to be on diskover or diskover-web host. They just require connections to the same redis and elasticsearch as diskover.py.


Nick: Watch travancoreanalytics.com

Nick requests the watch of the watch_keyword travancoreanalytics\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

travancoreanalytics\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Organizer

@arnavgrg  Giving this issue to you so that you can implement a consistent user logic for all the different user classes.


Test 1

Proposed changes
Summarize your changes here to communicate with the maintainers and make sure to put the link of that issue
Types of changes/litmus/pulls
What types of changes does your code introduce to Litmus? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices applies)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have read the CONTRIBUTING doc
[ ] I have signed the commit for DCO to be passed.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added necessary documentation (if appropriate)

Dependency

Please add the links to the dependent PR need to be merged before this (if any).

Special notes for your reviewer:

/merge

/merge

/merge-please

/merge-please


add docs to webapp on replicating results locally

From @MattHJensen on June 23, 2015 16:34
answer: 'how can I get the same code from the web app on my local machine?'
This will interact with: https://github.com/OpenSourcePolicyCenter/Tax-Calculator/issues/284
Copied from original issue: OpenSourcePolicyCenter/webapp#216

From @talumbau on July 17, 2015 1:59
This should be a blurb that discussed how one can do conda install -c ospc taxcalc. Maybe a link to the getting started guide, or general info about conda.

@MattHJensen, this might be just the text to put in the footer, now that we have a flat-block for the footer in #39. Do you think that's a reasonable place to put such a blurb?

@MattHJensen This is the blurb I came up with for the results footer. Let me know if you want to go with this for the next release or if you prefer a different blurb. OSPC.org still has the lorum ipsum in the results footer, so this text would replace that

This looks good to me and then I might tinker before the 10/13 release.

@MattHJensen and @talumbau,
Are the "pre-built packages" going to include the 2009 PUF file as used by the TaxBrain webapp?  If not, isn't the phrase "want to run this code on your machine?" a little misleading?  I think most people would read that phrase as "want to run this analysis on your machine?".  I know that is not what it says --- "code" is not "analysis", but I think more than a few people will read it that way.  And unless the 2009 PUF is downloadable along with the code, they will be very disappointed.

What if we drop the header? Actually, I had initially viewed this footer as a place to put some methodological footnotes for the tables; particularly a discussion of our expanded income measure and a discussion of calendar year vs. fiscal year. I could add those above the content starting "these results were generated...."  if we didn't have the header.

We don't have a great answer here until we have a publicly available data set. This should happen fairly soon, but we'll want to have more extensive documentation than what would just be at the bottom of the TaxBrain results page.


KeyError: 'HTTP_AUTHORIZATION'

Hello,
Tested locally, but in the Bot framework channel emulator I see only 500 error in post requests.
<IP_ADDRESS> - - [09/Aug/2017 19:48:49] "POST /api/messages HTTP/1.1" 500 -
[2017-08-09 19:48:49,332] ERROR in app: Exception on /api/messages [POST]
Traceback (most recent call last):
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/microsoftbotframework/msbot.py", line 77, in _message_post
valid_token = self._verify_token(request)
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/microsoftbotframework/msbot.py", line 110, in _verify_token
authorization_header = request.headers['Authorization']
File "/Users/a.mosyagin/.virtualenvs/hybrid_env/lib/python3.6/site-packages/werkzeug/datastructures.py", line 1349, in getitem
return unicodify_header_value(self.environ['HTTP' + key])
KeyError: 'HTTP_AUTHORIZATION'
<IP_ADDRESS> - - [09/Aug/2017 19:48:49] "POST /api/messages HTTP/1.1" 500 -
What do you think?
Thx for answer.

Hi Anton,
When using the simulator you need to disable the JWT validation. You can do this by setting the config value verify_jwt_signature. The simplest way to pass it as a argument to the MsBot object.
bot = MsBot(verify_jwt_signature=False)

You can also set it in the environment vars (best option) or config.yaml file.
I am going to leave this issue open as I am going to add a check to see if the emulator is been used and only give a warning if the auth validation fails or isn't present. The other option is to implement the full emulator JWT validation.


Randomly-generated MS2 ions in only one scan were assigned to fragments of features.

Dear Developer,
Thank you for providing such a convenient tool for MS data. We are having a problem in the process of using MZmine.

My operating system and version: Windows 10 Pro 21H1
My MZmine version: 3.4.16
In LC-MS/MS analysis, there are some ions appearing by chance in MS1 or MS2 spectra. These ions  have no correlation with components in the sample and usually exist for one scan. These random ions will be detected by "Mass detection". In "ADAP resolver", the one-scan random ions of MS1 can be filtered by "peak duration" and "RT wavelet". However, in MS2 ions assigned by "MS/MS scan pairing", the one-scan MS2 ions will be assigned to features. We tried to dispose the MS2 ions with "ADAP resolver" and the one-scan MS2 ions can be filtered. While in the exported mgf file, the MS2 ions of features still included with the  one-scan ions.
For example, for a feature at m/z 487.14, a random ion at m/z 79.4189 appeared in one MS2 scan. In the exported mgf file, this ion was in the list of MS2 fragments of the procursor at m/z 487.14.

The m/z tolerance was set as 0.05 Da. The noise levels of MS1 and MS2 were respectively set as 50 and 10. “RT tolerance” was set as 0.2 min in "MS/MS scan pairing". For MS1, “Min highest intensity” and “group intensity threshold” were respectively set as 500 and 100. “Min highest intensity”, “group intensity threshold” and “min group size in # of scans” were set as 500, 50 and 4. The file of MZmine cannot be uploaded on this webpage for "Something went really wrong, and we can't process that fire. Try again".

Thank you  for reading this letter. We would appreciate it if you could kindly help to sovle this problem.

Hi yuhan,
you are right these are just noise signlas. Most of them should be filtered by the Mass detection step, if you set the appropriate nosie level. As you described, you set the noise level to 10 for the MS2 scans, the signal intensity of that signal ist >10, so it is still in your data.
See this article: https://mzmine.github.io/mzmine_documentation/module_docs/featdet_mass_detection/mass-detection.html#how-do-i-determine-the-noise-level-in-my-data
I suggest to start with the wizard and then check the documentation for to optimize the workflow.


Request: Export Prometheus Metric for backups

Description
We create backups in PostgreSQL via action create-backup as a shell script in crontab.
It would be nice to have a way to monitor via Prometheus if backup failed or not as well as a backup counter.

Dear @amandahla, thank you for the request!
Please find my reply in VM charm: https://github.com/canonical/postgresql-operator/issues/356#issuecomment-1961322035
We have the same story here.


fix(EU): Driving Info Error

Safer method to get values.

@WillemD61 could you confirm this code works for you before I merge? This is to try resolve https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/issues/440

I tested it and it still works for me.
But I doubt whether this is going to solve #440 since the problem is not about getting the right fields out of the total response but getting a proper response in the first place. The response that  received was valid but totally empty, so you will still not get a value for the field totalPwrCsp.
Focus should be on getting a response with contents filled first. Once that is done the fields can be picked out.

Agreed something else is going on data wise.  This will just handle the exception in theory.


[TC-ACE-1.6] Script has to be updated as per test plan changes

Description:
Below mentioned steps has to be updated in YAML script:

Step 2 - Label has to be updated as per the latest test plan.
Step 5 - Label and values has to be updated as per the latest test plan.

@KishokG fixed in PR 31312 verify it and close the ticket.

@manjunath-grl Issue not resolved in this PR https://github.com/project-chip/connectedhomeip/pull/31312


fix: Delete app keys associated with project while deleting project

This PR will delete app-keys associated with project when project is undergoing deletion.

Codecov Report
Base: 31.69% // Head: 35.36% // Increases project coverage by +3.67% :tada:

Coverage data is based on head (61c016b) compared to base (8b75d32).
Patch coverage: 42.68% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #680      +/-   ##
==========================================
+ Coverage   31.69%   35.36%   +3.67%
==========================================
  Files          95      110      +15
  Lines       11588    12508     +920
==========================================
+ Hits         3673     4424     +751
- Misses       7586     7683      +97
- Partials      329      401      +72

Impacted Files
Coverage Δ

api/server/v1/error.go
0.00% <0.00%> (ø)

api/server/v1/marshaler.go
9.96% <0.00%> (-0.27%)
:arrow_down:

api/server/v1/validator.go
0.00% <0.00%> (ø)

internal/cache.go
0.00% <0.00%> (ø)

query/search/search.go
44.44% <ø> (-1.34%)
:arrow_down:

server/metadata/database.go
0.00% <0.00%> (ø)

server/metadata/key_encoder.go
38.57% <0.00%> (-7.20%)
:arrow_down:

server/metrics/quota.go
31.37% <0.00%> (-7.66%)
:arrow_down:

server/middleware/forwarder.go
0.00% <ø> (ø)

server/middleware/measure.go
0.00% <0.00%> (ø)

... and 60 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


The type or namespace name 'global_asax' does not exist in the namespace 'ASP'

Having a .cshtml file inside App_Code give errors on Visual Studio about the global_asax doesn't exist on namespace ASP.
this is used to have some razor function available application wide.
Compilation and application works ok, it's just the errors that are annoying.

@Bartmax Are you using MVC 5 or MVC 6?

Oops, you are right, I'm using MVC 5. sorry.

@Bartmax I have the same issue (http://stackoverflow.com/questions/28513532/mvc-5-c6-asp-global-asax-does-not-exist-in-the-namespace-asp)
Did you find a solution?


Using a multiplexer for button inputs

Hello, I want to use a multiplexer (4 pins for selecting the readable channel + 1 pin that reads the channel) to read the button states from my wheel.
I tried poking around and implemeting something but I can't figure out how the button values are stored in the "d" or "buttons" variable.
Are they stored in a bit fashion or some other way?
Any help would be appreciated :)

Every bit of 32-bit variable stores state of button. 1=pressed, 0=not pressed.
0b00000000000000000000000000000001 = button #1 is pressed
0b00000000000000000000000000000010 = button #2 is pressed
and so on.

Amazing stuff. I managed to implement it in my own way but this way with bitRead and bitWrite is so much sleeker. I have everything working, from HX711 brake(which im probably going to replace), to reading the angle and writing the force effect to an esp32 that's running FOC code for the stepper! Only thing, I tried Forza Horizon 5, and it reads all the buttons etc but the force feedback doesn't work, so i run Forza Emuwheel to get the feedback working, Have you seen anything similar?

In my case, Forza Emuwheel did not work with Forza. The only solution I could find was GIMX.


Unit tests for exec.go

Coverage increased (+5.6%) to 83.905% when pulling 17fe4832254e155f079942d0af4b7da0bf62594e on amshinde:test-exec-go into 111c1e3376e6a1669e060b751987a4f2a6e15c33 on clearcontainers:master.

Coverage increased (+5.6%) to 83.905% when pulling bbdfa17cb7eb3d7e0747d0b78aef03612eff1796 on amshinde:test-exec-go into 111c1e3376e6a1669e060b751987a4f2a6e15c33 on clearcontainers:master.

Coverage increased (+5.6%) to 83.905% when pulling bbdfa17cb7eb3d7e0747d0b78aef03612eff1796 on amshinde:test-exec-go into 111c1e3376e6a1669e060b751987a4f2a6e15c33 on clearcontainers:master.

I've tried restarting the semaphore build a few times, but it is consistently failing.

@jodh-intel thanks, I'll take a look at it today.

Coverage increased (+5.7%) to 87.43% when pulling f52b2e07d492f3aa43ed69fc13c101f9a23fba50 on amshinde:test-exec-go into 8a21d77e75884d3094244618db812fd01cadad07 on clearcontainers:master.

lgtm

Coverage increased (+5.7%) to 88.331% when pulling 75014c0cdfdc0c073d3d2abbe1f9c91347d9df1b on amshinde:test-exec-go into 55849b738faff3d4f92e2489879c9784ac59518d on clearcontainers:master.

@amshinde great thanks !
LGTM


allow sending network traffic usage for app metrics

What is this change about?
Include network traffic usage in the when sending App metrics.
What problem it is trying to solve?
Stakeholders can't observe the network traffic usage for a particular app.
What is the impact if the change is not made?
cf tail -c metrics <app-guid> -f  will not show rx_bytes and tx_bytes.
How should this change be described in diego-release release notes?
Container network traffic is now being sent to the logging stack.
Please provide any contextual information.

https://cloudfoundry.slack.com/archives/C01ABMVNE9E/p1683732484532129
https://github.com/cloudfoundry/cf-networking-release/issues/64

Tag your pair, your PM, and/or team!

https://github.com/JVecsei1

With this feature, users that deploy this version of diego-release would see significant increase in their foundation's total emitted metrics. This is not always ideal when the cost of consuming those metrics goes up significantly after upgrading to this version. One suggestion would be to move to a model where operators can choose what app-metrics to consume. We can always default to what's being emitted currently and allow the option for anyone to overwrite the default value.

With this feature, users that deploy this version of diego-release would see significant increase in their foundation's total emitted metrics. This is not always ideal when the cost of consuming those metrics goes up significantly after upgrading to this version. One suggestion would be to move to a model where operators can choose what app-metrics to consume. We can always default to what's being emitted currently and allow the option for anyone to overwrite the default value.

Very good point!
We implemented a feature flag for turning on/off these metrics, see

https://github.com/cloudfoundry/garden-runc-release/pull/294
https://github.com/cloudfoundry/guardian/pull/417/commits/e87a8d4bb577828795ea3379718939a98b32c175


Ensure memory leak detection only runs as a direct dependency

This addon should only detect memory leaks in addons and apps that directly depend on this package.
If ember-cli-example-addon installs ember-cli-memory-leak-detector, we should not run memory leak detection in consumers of ember-cli-example-addon (unless that consumer app also directly depends on ember-cli-memory-leak-detector).

This is now implemented, but more work is needed to get the automatic ES class name detection working in addons.


fix enabled property warning

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/22206
PROPOSAL: https://github.com/Expensify/App/issues/22206#issuecomment-1621360328
Tests

Run the dev app.
Go to any chat.
Tab composer to open the keyboard.
Hide the keyboard by dragging down.
Verify that the warning has gone away.

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

Offline tests

N/A
QA Steps

Run the dev app.
Go to any chat.
Tab composer to open the keyboard.
Hide the keyboard by dragging down.
Verify that the warning has gone away.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

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

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

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Web
N/A

Mobile Web - Chrome
N/A

Mobile Web - Safari
N/A

Desktop
N/A

iOS
https://github.com/Expensify/App/assets/8579651/a8dc2cb9-89ee-488f-8697-fe11ebc54bfa

Android
https://github.com/Expensify/App/assets/8579651/bb807db3-74dc-400c-84cb-d9329f34435c

Wait for a second, doesn't this change the behavior? If this.props.enabled is undefined then enabled should be true because that should be the default value. Now if you are using this.props.enabled directly it's not true anymore
I've used a weird syntax for this :neutral_face:  const {enabled = true} = this.props; but in the PR for react-native  is a version with improved syntax const enabled = this.props.enabled ?? true;
This may be the proper solution to resolve this problem

But it changes the behavior. If the this.props.enabled is undefined then enabled should be true. It's not here anymore

Ah, yes sorry 😄 Yeah I think we could create a new patch and replace the current one with the code from react-native PR. To make sure that the changes here are the same as the proposed one for the upstream fix

Should I create a PR or maybe @ntdiary can continue with this one?

LGTM

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

🚀 Deployed to staging by https://github.com/robertjchen in version: 1.3.39-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/francoisl in version: 1.3.39-11 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


DELETE Repository is still possible event if permission is set to DENY

I have set the "Delete Repository" permission to DENY, but it is still possible for the given user to delete a repository.
Screenshot of permission setting:

How I'm doing the delete operation:

I would expect this not to work ... but it works.
Am I missing something?
Thank you in advance!

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

ID: 4253dae1-49d7-5785-41d5-ce70bc4c9010
Version Independent ID: 7de01552-ecb6-5594-3294-cbfb41cb4477
Content: Set Git repository permissions - Azure Repos
Content Source: docs/repos/git/set-git-repository-permissions.md
Product: devops
Technology: devops-code-git
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@Christian-e-S - If you've set a specific permission to Deny for a user, and that user still can perform the associated task - then something else must be going on.  I recommend that you see what other groups the user is a member of. A few things to check:

Use az devops security permission show to list the user's permissions.
Troubleshoot permissions

Since this is not a document issue, if you continue to have issues, I reccomend you post your issue on our Developer Community page.


Invalid status code: 1 Error

I have this when I try to run the action on a fork Invalid status code: 1
Error: Invalid status code: 1

Thanks for the report!
First of all, indeed it could show some more logs and information on why it fails.
Do you think you can give it a try? As you already have the fork, you can try locally to add some more logs :)
Speaking of local, does it run there for you? You can either do that by running the vscode debugger, or a simple npm run test-debug that will execute the test, running the action.
I'm sorry for the lack of documentation, it's still a fresh project but I will work on it!

I added a few more logging with a9e3c4e1ab5acaaad3b016f0df02de3dc54f9a4d
Can you please update your fork and try again to run the action manually? :)

I added a few more logging with https://github.com/Balastrong/chess-stats-action/commit/a9e3c4e1ab5acaaad3b016f0df02de3dc54f9a4d
@iib0011  can you please update your fork and try again to run the action manually? :)

It works well in vscode but not in github

still no logs

Thanks :)
I did some investigation and looks like it fails on the commitFile function.
https://github.com/Balastrong/chess-stats-action/blob/a9e3c4e1ab5acaaad3b016f0df02de3dc54f9a4d/src/main.ts#L59
Feel free to open a PR if you can manage to fix it :)

Okay

I've got some good news, a potential cause was that I changed the file name from hardcoded README.md to a new input passed via the env variables.
However, the commit action still had the hardcoded filename.
@iib0011 can you please update your fork now and let me know if it works?

Okay. I'll check it
Le dim. 31 juil. 2022 à 08:43, Leonardo Montini @.***>
a écrit :

I've got some good news, a potential cause was that I changed the file
name from hardcoded README.md to a new input passed via the env variables.
However, the commit action still had the hardcoded filename.
@iib0011 https://github.com/iib0011 can you please update your fork now
and let me know if it works?
—
Reply to this email directly, view it on GitHub
https://github.com/Balastrong/chess-stats-action/issues/5#issuecomment-1200368719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBJ6LCPABLBNH5IL4GXNKTVWYVATANCNFSM55AOJ65A
.
You are receiving this because you were mentioned.Message ID:
@.***>

It works now. Great
On Sun, Jul 31, 2022 at 1:46 PM Ibrahima Gaye Coulibaly <
@.***> wrote:

Okay. I'll check it
Le dim. 31 juil. 2022 à 08:43, Leonardo Montini @.***>
a écrit :

I've got some good news, a potential cause was that I changed the file
name from hardcoded README.md to a new input passed via the env variables.
However, the commit action still had the hardcoded filename.
@iib0011 https://github.com/iib0011 can you please update your fork
now and let me know if it works?
—
Reply to this email directly, view it on GitHub
https://github.com/Balastrong/chess-stats-action/issues/5#issuecomment-1200368719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOBJ6LCPABLBNH5IL4GXNKTVWYVATANCNFSM55AOJ65A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add various classifiers, async execution pipelien, video stream and suspicion detection classes

Adds a bunch of new things for Vigilancia backend. This PR finishes following tasks:

Added Inception classifier.
Added Event detector classifier.
Added Unusual activity detector classifier.
Added video stream class to standardize opencv interface.
Added suspicion detection class and combined all classifiers as a single function call.
Added async class to use python multiprocessing and threading for parallel execution of all classifiers.
Other changes in data_downloader.sh, vgconf, main.py files.

@Yash9060 @yash97 @vrajesh978 check this out. This finishes backend classifiers implementation of Vigilancia. Do we need anything else on classifiers part?