API-1581 async sdk

Abstracts Apache HttpClient behind SDK-specific interfaces
Adds pluggable HTTP client "connectors" using ServiceLoader
Implements Apache HttpClient as a "connector"
Implements okhttp HttpClient as a "connector" (for use with Android SDK)
Modifies request POJOs to use title casing for property accessor methods
Modifies request POJOs to create ApiRequest<T> implementations
Adds overloads to *Api classes to accept request POJOs and ApiRequest<T> implementations
Adds *ApiAsync classes for invoking SDK operations asynchronously

This PR will break the ApiResponse<T> class from the previous version.

The class has been turned into an interface.
Members to get the request information have been removed.
References to Apache-specific types have been refactored away, e.g. Map<String, String> instead of Header[].
Some of the other members have been renamed to be more consistent with request objects, e.g. getResponseObject() to getBody() and getResponseHeaders() to just getHeaders()


allow empty password in client.yml

fixes #374

Codecov Report
Base: 66.10% // Head: 66.08% // Decreases project coverage by -0.01% :warning:

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

:exclamation: Current head ce392de differs from pull request most recent head b7a18bd. Consider uploading reports for the commit b7a18bd to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #434      +/-   ##
==========================================
- Coverage   66.10%   66.08%   -0.02%
==========================================
  Files          35       35
  Lines        3720     3721       +1
==========================================
  Hits         2459     2459
- Misses        908      909       +1
  Partials      353      353

Impacted Files
Coverage Δ

cmd/subscribe.go
29.87% <0.00%> (-0.20%)
:arrow_down:

client/config.go
73.33% <100.00%> (ø)

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.

Love it!


when hovers are not sticky, alt/opt key should make them temporarily sticky

Version 1.68 introduced the feature of holding the ALT (OPT on Mac) key down in order to make hover texted locked so that you can move the text over it, and then magnify, select text, etc. from the hover. But then this:

Note that this feature only applies outside of the editor because editor hovers can always be moused over unless specified otherwise via the editor.hover.sticky setting

Setting 'editor.hover.sticky' to false is very good at preventing accidental clicks on a popup hover text, when you really mean to click on the line above the thing causing the hover (sometimes the hover text shows up at exactly the wrong time, and then you end up going to MDN or Can I Use, etc.). I would like it if I could keep 'editor.hover.sticky' as false, but press the ALT or option key down during a hover for when I actually do want to click on the link in the hover text, or when I do want to zoom in on it or copy something from it.

I guess IHoverService should move into platform to help fix this.

Bump!
I absolutely love the ability to set "editor.hover.delay": 0, "editor.hover.sticky": false, makes the IDE feel really responsive. My only pet peeve is that I can't copy-paste stuff from the hover popups to look up docs or whatever. Holding Alt sounds like a great solution. :)

Hi @brad-provident thank you for filing this issue. This is a good point, so you want to be able to hold alt in order to lock the hover when the sticky setting is false.

@aiday-mar Yes, correct.

Hey all, I reverted the PR which fixed this issue because it introduced a regression and it required some more polishing. I will make a new PR and we will iterate on it before merging again.

I had to revert PR #236356 because it causes #236583


Class helper functions - Support for generic classes

I noticed that functions like attr.has, attr.fields, attr.fields_dict doesn't work as i expected when i pass in class with generic type defined:

from attr import frozen, evolve, mutable, has, fields, fields_dict, Attribute

T = TypeVar("T")

@frozen
class A(Generic[T]):
    a: int
    b: T

has(A) => True
has(A[str]) => False (should be True)

fields(A) => Sequence[Attribute]
fields(A[str]) => TypeError: Passed object must be a class (should be Sequence[Attribute])

fields_dict(A) => Mapping[str, Attribute]
fields(A[str]) => TypeError: Passed object must be a class (should be Mapping[str, Attribute])```

Will be taking a look at this soon.

+1
This is quite annoying when I'm using cattrs' hook factories... e.g. the snake_case to camelCase cattrs example doesn't work with generic classes

first support has been merged as part of #1079


fix(sidenav): Fix renderer calls to use beta1 api

Taking a look, but I'm going to push this as a new PR to let Travis run.

LGTM, I'll merge if Travis passes the other PR.

Merged https://github.com/angular/answers-app/commit/0568af1aa52d35a867b257aa950101a17a60510e


⚠ Bofrost.at has degraded performance

In 3b6fa1e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1639 ms

Resolved: Bofrost.at performance has improved in a2fd5e4.


structured open fine resolution approaches

Adds a set of approaches to opening the fine-res data based on our current priorities.
Refactored public API:

Removes loaders.open_precomputed_fine_resolution because this is now an approach under loaders.open_fine_resolution

Significant internal changes:

loaders.open_fine_resolution now takes an approach argument in a backward compatible way.

[ ] Tests added

Might be easier to pair program on this since I'm asking for some pretty extensive changes.

Or split the code that actually computes the budget versus I/O etc?

I think this is the key. Let's define a consistent interface for what the output of I/O and merging should be (e.g. what variable names are expected).  You can use a protocol to do this as I have done elsewhere in this module.
class MergedData(Protocol):
    tendency_of_yada_due_to_yada: xr.DataArray
    temperature: xr.DataArray
    ...

class Budget(Protocol):
    Q1: xr.DataArray
    Q2: xr.DataArray

def compute_budget(data: MergedData) -> Budget:
     pass

def _open_blah(blah, blah, blah) -> MergedData:
     ....

def open_blah(blah, blah, blah, approach):
    merged = _open_blah(blah, blah, blah)
    return XarrayMapping(compute_budget_with_approach(merged, approch))


[Create Child] - remove childrenService from the component, move that logic to the state

Follow the link: https://github.com/ita-social-projects/OoS-Frontend/issues/1440

issue with branch


Reuse query engine for multiple PrismaClient instances

Problem
One of the most tricky parts of using Prisma Client is database connection handling.
While modern databases like Fauna DB or serverless Aurora don't require a long-running connection anymore and therefore no long-running process that keeps the connection up, the databases currently supported by Prisma (in particular MySQL and Postgres) have the concept of long-running connections, which are expensive to establish and once established should be reused.
In order to ensure optimal performance with these databases, the query engine, which powers Prisma Client, manages a connection pool. The default size of that connection pool is being determined by the number of cores times 2, as it turned out in internal performance tests, that that is the optimal number for the best performance.
All this works well, as long as there is only one Prisma Client and therefore query engine instance.
And usually, as you have your application and database under control (should at least), you can ensure, that there is just one Prisma Client instance, which holds the query engine with the connection pool.
However, as soon, as you have multiple Prisma Client instances running, this can get really slow. It can even be a memory leak if you instantiate a new Prisma Client per incoming API request.
This situation, that too many Prisma Clients have been instantiated has occurred numerous times in the community, see https://github.com/prisma/prisma/issues/1983 https://github.com/prisma/prisma-client-js/issues/632 https://github.com/prisma/prisma-client-js/issues/723 and https://github.com/blitz-js/blitz/blob/canary/examples/store/db/index.ts
In other words: You can easily shoot yourself in the foot because you need to manage state and managing state is always nasty.
How could Prisma Client help here?
One possible solution: Instead of instantiating a new query engine process per PrismaClient instance, we would have a "named singleton" pattern.
That means, that for a particular combination of args, we reuse the query engine instance and make that accessible under a name, e.g. "QE1". This way it doesn't matter, how often you instantiate PrismaClient, all instances will reuse the same query engine and therefore the same connection pool, with all the connections already warmed up.
This could also help us in the situation of clusters: https://github.com/prisma/prisma-client-js/issues/632  we could enable this singleton pattern across the whole cluster out of the box so that the node cluster can share the same connection pool. Building something like that as a user would be a lot of overhead and a non-trivial task to do.
What are disadvantages of this approach?
Usually, a singleton is considered an antipattern. In this case it could be considered "ok" though, as we're talking about resources that can be shared (and maybe should be shared) across instances.
Another big problem here is: You might indeed have a serious memory leak in your application, which would now not be visible anymore, as it "just works".
One could see this approach as "wrong", as you can always say: "Just instantiate  one Query Engine!".
Closing thoughts
We have this problem because the query engine is attached as a sidecar to the Node.js process. If we would run the query engine as a separate process as you do with a database, we would not have this problem of "unintentionally spawning too many query engines". However, the ergonomy and developer experience of Prisma Client would then greatly suffer.

One area where having multiple PrismaClient instances is needed is multitenancy. Right now we're able to create a new client for each tenant. If we're forced to use a singleton, I think it's important to ensure there's another path to multitenancy. Either the ability to programmatically add new datasources at runtime, the ability to change the schema/database at runtime, or the ability to opt out of the singleton client. Just wanted to call this out as I suspect moving to a singleton internally would break this functionality. I may be misunderstanding something though, so feel free to tell me I'm wrong. :)

One area where having multiple PrismaClient instances is needed is multitenancy. Right now we're able to create a new client for each tenant. If we're forced to use a singleton, I think it's important to ensure there's another path to multitenancy. Either the ability to programmatically add new datasources at runtime, the ability to change the schema/database at runtime, or the ability to opt out of the singleton client. Just wanted to call this out as I suspect moving to a singleton internally would break this functionality. I may be misunderstanding something though, so feel free to tell me I'm wrong. :)

The above is not real code, but was a former colleague trying to imagine what an API could work like. This does not exist and is also not a high priority for us right now.


Add hover functionality

I want to add hover functionality on the @mention tag added after selection and show an overlay with desired properties.
Add onHover on mention tags.

Yah, hover and onclick would be wonderful

@GitRGoodman onclick is already implemented. Just FYI. I may implement onhover soon.

@kovalforpublic I can't get it to work then, may I ask for an implementation exemple? (Sorry I know this might not be the right place to ask but I can't find a reference)

@GitRGoodman it's ok for me (but you can create a stackoverflow question and I'll transfer my answer there, for integrity purposes). Since people have questions I probably will add an example into readme.
this is an example I've added to readme.
    window.addEventListener('mention-clicked', (event) => {console.log('hovered: ', event)}, false);

So, you just need to listen for window:mention-clicked (or "mention-hovered" in the near future ;) )
I also created a [PR](https://github.com/afry/quill-mention/pull/227/) for "mention-hover" event. Please, you and @abhishekmatta999 feel free to add reactions so the holder of the project would have more incentive to merge it.
 And thanks for raising the issue about unclear documantation for mention-click/hover event 👍
  have a nice day

Thank you very much, correct me if I'm wrong tho, this way I wont' be able to get ant data from the mention click right?

Thank you, very informative and quick answer. Really a lifesaver

PR was merged, now hover functionality is available, so the issue can be closed.


Prebid.org site example shows old Adapter setup

@caseybryan
Can we please update the example on: http://prebid.org/dev-docs/getting-started.html to include the new Rubicon Project Fastlane adapter to keep it consistent with the documentation for 0.6. It shows "zonesize" which will not work.

Thank you for the update another publisher found an error in the documentation release notes on
https://github.com/prebid/Prebid.js/releases/tag/0.6.0. It shows the params not in brackets.
Current:
var adUnits = [{
code: '/9968336/header-bid-tag-0',
sizes: [[300, 250], [300, 600]],
bids: [{
accountId: "123",   //Required - String id
siteId: "456",      //Required - String id
zoneId: "789",      //Required - String id
sizes: [15]         //optional - array[number] of size key
}]
}];
It should really show:
var adUnits = [{
code: '/9968336/header-bid-tag-0',
sizes: [[300, 250], [300, 600]],
{
bidder: 'rubicon',
params: {
accountId: "1234",          //String - required
siteId: "1234",             //String - required
zoneId: "1234",           //String - required
sizes: [15],                //Array[Number] - optional

I am still seeing Publishers seeing that example and getting errors. Any update on when we can get that removed from: http://prebid.org/dev-docs/getting-started.html

@mkendall07 if no one else is working on this, I'll take it.

@rmloveland thanks!

This should be addressed by https://github.com/prebid/prebid.github.io/pull/31
Note that the (many!) uses of the rp_ Rubicon params in the live examples area of the site will be addressed in https://github.com/prebid/prebid.github.io/issues/29 -- I wanted to manage the scope on this to at least get something out today.
@caseybryan the examples you referenced should be fixed; please re-open this issue if not.


Broadcasting arrays on mathematical operations

Summary
In NumPy it is possible to perform a mathematical operation on arrays of different shapes. See NumPy documentation at https://docs.scipy.org/doc/numpy-1.15.0/user/basics.broadcasting.html. However, in NumSharp, I get a NumSharp.IncorrectShapeException when I try this.
(I assume that this functionality is just not implemented yet, but IMHO it is a very important feature when manipulating data.)
Minimum example
static void Main(string[] args)
{
    int n = 5;
    NDArray arr_Nx3 = np.arange(0, 3*n).reshape(n, 3);
    NDArray arr_1x3 = new int[] { 1, 2, 3 };
    NDArray result = arr_Nx3 + arr_1x3; // This throws a NumSharp.IncorrectShapeException

    NDArray expectedResult = new int[,]
    {
        { 1, 3, 5 },
        { 4, 6, 8 },
        { 7, 9, 11 },
        { 10, 12, 14 },
        { 13, 15, 17 },
    };
}

Expected result
When doing addition with a Nx3 array and a 1x3 array, I expect a Nx3 array (as described in the NumPy documentation). I expect this kind of array broadcasting to happen for all applicable operations (at the very least addition, subtraction, multiplication, division).

This issue has been implemented in the new release of NumSharp 0.11.0-alpha2 and available on nuget.


MySQL schema support

[x] SQL file to create database in MySQL format
[x] Configuration can be specified in zeebe.toml as createSchema = "/CREATE_SCHEMA_MYSQL.sql"
[x] Configuration can be specified in environment as SIMPLE_MONITOR_EXPORTER_JDBC_CREATE_SCHEMA
[x] Test created (not running by default as it requires living MySQL/MariaDB)
[x] If createSchema = "-" specified, will not create any schema at all

@huksley sorry for the late response and thank you very much for your contribution. I had no time to look at it I will try to do this on monday.

Sure, let me know how we can further expand exporter and not so simple anymore monitor app.
I`ve found that H2 is pretty much unusable even in local docker-compose setup
https://github.com/huksley/zeebe-kafka-camunda/blob/master/docker-compose.yml

Since we merged #42 I will close this


Run external tests periodically

To ensure that we catch errors in external modules early.
This runs the tests every third day at 4:47. The time is randomly chosen as is best practice for cron jobs.
GitHub Actions schedule syntax: https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#schedule

Visualization of the schedule: https://crontab.guru/#47_5_1/3__

We already have dependabot PRs quite often to catch test-external failures. Even if it is not designed for that, it's already a kind of cron job, is it really necessary?

Yeah, currently PR:s like Dependabot becomes the first ones to catch these errors, making most of their failures false negatives.
Better to run dedicated tests for the external repositories, so we can more easily spot whether they are the cause or the PR:s are the cause.


[magic-enum] Fix Sha512

Fixes #40429. Resolve the following issue:
error: Missing Neargye-magic_enum-v0.9.6.tar.gz and downloads are blocked by x-block-origin.
error: File does not have the expected hash:
url: https://github.com/Neargye/magic_enum/archive/v0.9.6.tar.gz
File: /home/anderswind/projects/vcpkg/downloads/Neargye-magic_enum-v0.9.6.tar.gz.4160505.part
Expected hash: 6154c816446e115f3b164df79ab8d8088eb76b632ee3fdc82ea17cc7ae8d04652c83e5cc587c2c4b334889904b101ba08a04c5837103af260768e93df17cc263
Actual hash: fa3792e9c69c57e437d6325b7659cf25e6cbdda3326ffeaf0411d9838822095b15306f322ad2ebd5ec68b905ef2aed08880d2507e7d8a8559ab3c5be6c8ce99c

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version.~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Duplicate of #40430.


Library should implement the confidence argument if opencv is installed

It would be great if we could implement the "confidence" parameter for pyautogui's screenshot functions. I've done this on my feature/confidence_level branch in my fork of the repository. However, it is based off of my Python3 version of the library in my feature/python3 branch.
My integration branch combines the two branches above. All unit and acceptance tests pass with python3.

PR #43 should do this.

Thanks! Merged.


Hitting / while entering portfolio entries jumps into search mode?

While / isn't a useful key to press (until #110 ) this does create a weird user experience.

closed by https://github.com/miguelmota/cointop/commit/fb79419c8415086af3e394d93e68b0330e7c3100


DOCTEST_CONFIG_USE_STD_HEADERS in MSVC++

It seems there's no need to include whole <sstream> header. Just <ostream> is enough.

Hey thanks for the PR! Good to reduce the problem a little. Do you happen to know why this is required though?
Also it seems that GitHub actions aren't running on your fork so I cannot verify the fix. Could you enable them for your repository?

I'm on to a different solution
See doctest docs for information.
After all macro, defined by the doctest, seems like a better solution.

Hey thanks for finding the option, now the error makes more sense!
In that case I think we should remove the local macro and just enable the flag by default for MSVC to avoid conflicting definitions and problems in other test files.
I think it should work if we add the following to the tests/CMakeLists.txt:
  elseif(MSVC)
    target_compile_options(Greeter PUBLIC /W4 /WX)
++  target_compile_definitions(GreeterTests PUBLIC DOCTEST_CONFIG_USE_STD_HEADERS)
  endif()

I totally agree. But we should somehow be explicit about defining this macro, as it does slightly change doctest's behavior.
When it's defined, overloaded by the doctest operator<<'s could run in some problems. Testing is required to know definitely, though.

Also looking at the source code it seems that it's only creating some forward declarations of std types, so as long as the code compiled before we should still be ok after defining the macro.

Ok, I see your point. So which's better, to create new pull request (branch) for the fix or to just add new commit here?

Just simply add the commit here, then the full discussion will be visible. I'll squash all commits before the merge so the main git's tree will remain clean.

Looks good, thanks for the fix!


Add NPM License Support from Package-Lock v2+

Package-lock, starting in version 2, optionally provides package licenses that should be incorporated into BOM generation.

@ansonallard could you kindly sign the commit by following the instructions below?
https://github.com/CycloneDX/cdxgen/pull/577/checks?check_run_id=17015695495


high cpu load mac os 10.13.6.

Hi,
I'm really happy that you support mac os 10.13.6. (mac 2011 :D,  8gb ram).
I've noticed that the wallet (daedalus 1.0.0) had an high cpu load on my machine. It synced but still %cpu was like 199%.  I closed the wallet and started again. After the restart the cpu load reads 3% and the machines runs smooth again.
Cheers

+1

Resolved in https://github.com/input-output-hk/cardano-node/pull/986


[] build failure

Host Environment

OS: Windows10
Compiler: MSVC142

To Reproduce
Steps to reproduce the behavior:
.\vcpkg install qwt:x64-windows
Failure logs
Starting package 1/20: freetype:x64-windows
Building package freetype[brotli,bzip2,core,png,zlib]:x64-windows...
-- Trying auto-select mirror...
-- Using cached I:/src/vcpkg/downloads/freetype-2.10.4.tar.xz
-- Skipping hash check for I:/src/vcpkg/downloads/freetype-2.10.4.tar.xz.
-- The default mirror is in Disaster Recovery mode, trying other mirrors...
-- Trying mirror cfhcable...
-- Using cached I:/src/vcpkg/downloads/freetype-2.10.4.tar.xz
-- Skipping hash check for I:/src/vcpkg/downloads/freetype-2.10.4.tar.xz.
-- Cleaning sources at I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source I:/src/vcpkg/downloads/freetype-2.10.4.tar.xz
-- Applying patch 0001-Fix-install-command.patch
-- Applying patch 0003-Fix-UWP.patch
-- Applying patch pkgconfig.patch
-- Applying patch brotli-static.patch
-- Applying patch fix-exports.patch
-- Using source at I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:108 (message):
Command failed: ninja -v
Working Directory: I:/src/vcpkg/buildtrees/freetype/x64-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
I:\src\vcpkg\buildtrees\freetype\config-x64-windows-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:319 (vcpkg_execute_required_process)
ports/freetype/portfile.cmake:30 (vcpkg_configure_cmake)
scripts/ports.cmake:136 (include)
Error: Building package freetype:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: freetype:x64-windows
Vcpkg version: 2020.11.12--unknownhash
Additionally, attach any relevant sections from the log files above.
///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
in I:\src\vcpkg\buildtrees\freetype\config-x64-windows-out.log
[1/2] cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean" "-DCMAKE_DISABLE_FIND_PACKAGE_HarfBuzz=ON" "-DFT_WITH_ZLIB=ON" "-DFT_WITH_BZIP2=ON" "-DFT_WITH_PNG=ON" "-DFT_WITH_BROTLI=ON" "-DCMAKE_DISABLE_FIND_PACKAGE_ZLIB=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BZip2=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_PNG=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BrotliDec=OFF" "-DCMAKE_MAKE_PROGRAM=I:/src/vcpkg/downloads/tools/ninja/1.10.1-windows/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=I:/src/vcpkg" "-D_VCPKG_INSTALLED_DIR=I:/src/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=I:/src/vcpkg/packages/freetype_x64-windows""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean" "-DCMAKE_DISABLE_FIND_PACKAGE_HarfBuzz=ON" "-DFT_WITH_ZLIB=ON" "-DFT_WITH_BZIP2=ON" "-DFT_WITH_PNG=ON" "-DFT_WITH_BROTLI=ON" "-DCMAKE_DISABLE_FIND_PACKAGE_ZLIB=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BZip2=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_PNG=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BrotliDec=OFF" "-DCMAKE_MAKE_PROGRAM=I:/src/vcpkg/downloads/tools/ninja/1.10.1-windows/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=I:/src/vcpkg" "-D_VCPKG_INSTALLED_DIR=I:/src/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=I:/src/vcpkg/packages/freetype_x64-windows""
-- The C compiler identification is MSVC 19.28.29335.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.28.29333/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Could NOT find ZLIB (missing: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
CMake Error at C:/Program Files/CMake/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:218 (message):
Could NOT find PNG (missing: PNG_LIBRARY PNG_PNG_INCLUDE_DIR)
Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:582 (_FPHSA_FAILURE_MESSAGE)
C:/Program Files/CMake/share/cmake-3.19/Modules/FindPNG.cmake:156 (find_package_handle_standard_args)
I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake:634 (_find_package)
CMakeLists.txt:216 (find_package)
-- Configuring incomplete, errors occurred!
See also "I:/src/vcpkg/buildtrees/freetype/x64-windows-rel/CMakeFiles/CMakeOutput.log".
[2/2] cmd /c "cd ../../x64-windows-dbg && "C:/Program Files/CMake/bin/cmake.exe" "I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean" "-DCMAKE_DISABLE_FIND_PACKAGE_HarfBuzz=ON" "-DFT_WITH_ZLIB=ON" "-DFT_WITH_BZIP2=ON" "-DFT_WITH_PNG=ON" "-DFT_WITH_BROTLI=ON" "-DCMAKE_DISABLE_FIND_PACKAGE_ZLIB=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BZip2=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_PNG=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BrotliDec=OFF" "-DCMAKE_MAKE_PROGRAM=I:/src/vcpkg/downloads/tools/ninja/1.10.1-windows/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=I:/src/vcpkg" "-D_VCPKG_INSTALLED_DIR=I:/src/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=I:/src/vcpkg/packages/freetype_x64-windows/debug""
FAILED: ../../x64-windows-dbg/CMakeCache.txt
cmd /c "cd ../../x64-windows-dbg && "C:/Program Files/CMake/bin/cmake.exe" "I:/src/vcpkg/buildtrees/freetype/src/827cda734a-239082ca76.clean" "-DCMAKE_DISABLE_FIND_PACKAGE_HarfBuzz=ON" "-DFT_WITH_ZLIB=ON" "-DFT_WITH_BZIP2=ON" "-DFT_WITH_PNG=ON" "-DFT_WITH_BROTLI=ON" "-DCMAKE_DISABLE_FIND_PACKAGE_ZLIB=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BZip2=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_PNG=OFF" "-DCMAKE_DISABLE_FIND_PACKAGE_BrotliDec=OFF" "-DCMAKE_MAKE_PROGRAM=I:/src/vcpkg/downloads/tools/ninja/1.10.1-windows/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_LINKER_FLAGS_RELEASE=" "-DVCPKG_LINKER_FLAGS_DEBUG=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=I:/src/vcpkg" "-D_VCPKG_INSTALLED_DIR=I:/src/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=I:/src/vcpkg/packages/freetype_x64-windows/debug""
-- The C compiler identification is MSVC 19.28.29335.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.28.29333/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Could NOT find ZLIB (missing: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
CMake Error at C:/Program Files/CMake/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:218 (message):
Could NOT find PNG (missing: PNG_LIBRARY PNG_PNG_INCLUDE_DIR)
Call Stack (most recent call first):
C:/Program Files/CMake/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:582 (_FPHSA_FAILURE_MESSAGE)
C:/Program Files/CMake/share/cmake-3.19/Modules/FindPNG.cmake:156 (find_package_handle_standard_args)
I:/src/vcpkg/scripts/buildsystems/vcpkg.cmake:634 (_find_package)
CMakeLists.txt:216 (find_package)
-- Configuring incomplete, errors occurred!
See also "I:/src/vcpkg/buildtrees/freetype/x64-windows-dbg/CMakeFiles/CMakeOutput.log".
ninja: build stopped: subcommand failed.
////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
How can i solve it?
Thanks in advance.

-- Could NOT find ZLIB (missing: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
-- Could NOT find PNG (missing: PNG_LIBRARY PNG_PNG_INCLUDE_DIR)
Is ZLIB and LIBPNG for x64-windows installed?
If they are installed try removing them and reinstalling...

-- Could NOT find ZLIB (missing: ZLIB_LIBRARY ZLIB_INCLUDE_DIR)
-- Could NOT find PNG (missing: PNG_LIBRARY PNG_PNG_INCLUDE_DIR)
Is ZLIB and LIBPNG for x64-windows installed?
If they are installed try removing them and reinstalling...

It works! Thank you!
I did install ZLIB and LIBPNG for x64-windows, but why missing?

It works! Thank you!
I did install ZLIB and LIBPNG for x64-windows, but why missing?


Add commit input to make it possible to use this action with third party PRs

Closes #23

I'm kinda new to making GitHub Actions, so I'm not actually sure how I can test this (either locally or by publishing the action under a new name from my fork of this repo)

I figured out how to publish this as a test action, but it doesn't seem to work. I suspect that publishing the annotations on a commit ID in the target repo causes them to just not get associated with anything, and thus lost.
I would suggest not merging this PR until it can be reworked by someone who understands the GitHub APIs better than I :)

I suspect that publishing the annotations on a commit ID in the target repo causes them to just not get associated with anything, and thus lost.

Yes that's correct, the same thing happened to me while I was working on https://github.com/yuzutech/annotations-action/commit/c707e6d690f646fe3d842b8e3abc1dfbc356da3a.

I would suggest not merging this PR until it can be reworked by someone who understands the GitHub APIs better than I :)

I took a look at other GitHub Actions but so far I have yet to find a working solution 😞


Can deploy this app to heroku?

I want to deploy this app to heroku.
Can you guide me or write on readme?

You should be able to deploy it as a Node.js app. Try this guide.

You can use node-js-getting-started code of heroku .
After add your code to project.
I was try but facebook block me (my IP diffrent that IP  region).

This can work (and we've done it in the past) but you can sometimes run into problems with facebook seeing logins from multiple IPs and temporarily blocking the account. You might be able to get around this using api.getAppState.

Hope this helped!  Let us know if you run into any problems 😄


Revamp of getting started guide - part 1

Hey, I just made a Pull Request!
Updating the getting started guide. Aim is to make the experience smoother. It shouldn't break existing references.
This is part 1 of getting started, so there's requirements which might not be immediately used in this doc.
:heavy_check_mark: Checklist

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

Sadly, accepting suggestions from the GitHub UI doesn’t sign off commits and then fails DCO. Sorry, didn’t mean to trap you into that 😅 The suggestion format is just so nice to show the difference.
I think you’ll be able to fix it locally by updating the branch and following the DCO instructions to amend the commits.


Testing Github App Integration with Slack

Description
Testing Github App Integration with Slack
Screenshots/testimonials
NA

Working Fine :+1:


Modal doesn't work in IE8

Although it is listed as "functional" in IE8, I did not find this to be the case. The modal would not fire as expected.

+1

Have you checked this with the version on master? I have now pushed a fix for the index file. This fixes it for me.
Can you please have another look at the issue?

Will not be handled, IE8 support will be removed.


Connecting with DAC?

Did you have any luck connecting using DAC?
I am getting:
Error:
Exception calling "Open" with "0" argument(s): "A connection was successfully established with the server, but then an
error occurred during the pre-login handshake. (provider: TCP Provider, error: 0 - The specified network name is no
longer available.)"

I can connect to DAC from SSMS with no problem.

How did you attempt a connection via the DAC? Did you just use port 1434?


CONFIG_ISM330DHCX cannot compile due to missing file

Describe the bug
When selecting CONFIG_ISM330DHCX and providing a device tree entry, the source will still not compile. The reason is this:
/home/schurig/d/proj/zephyr/drivers/sensor/ism330dhcx/ism330dhcx.h:19:10: fatal error: ism330dhcx_reg.h: No such file or directory
 #include "ism330dhcx_reg.h"
          ^~~~~~~~~~~~~~~~~~

The file is nowhere to be found:
schurig@desktop:~/d/proj$ cd zephyr/
schurig@desktop:~/d/proj/zephyr$ find -name ism330dhcx_reg.h
schurig@desktop:~/d/proj/zephyr$

Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows): doesn't matter
Toolchain (e.g Zephyr SDK, ...): doesn't matter
Commit SHA or Version used: v2.7.99-847-g81f1225040

On Oct, 2021-10-29 09:20, Gerard Marull-Paretas wrote ...

Make sure to run west update, the file is part of hal_st module.
../modules/hal/st/sensor/stmemsc/ism330dhcx_STdC/driver/ism330dhcx_reg.h

Oh, maybe the Kconfig help text, or a README.rst in the folder directory
should have said that?
My Dockerfile only has "west update cmsis hal_stm32" to not pull in the
world, or things that I don't need anyway.


rapids_cpm_package_details now validates required entries exist

Description
rapids-cmake now provides proper error message reports when the required json entry fields don't exist in either the root versions.json or the package override.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.

/merge


Lower oozie memory requirements for VM clusters

PR #121 raised the default memory allocation for oozie server process to 2G heap and 256M perm. This allocation is not viable on VM clusters with about 3-5G total memory. This PR overrides the memory settings and set them to half for VM clusters.

I'd like to see us test if 512MB will still meaningfully run Oozie (i.e. for HDFSDU). If so, reclaiming 1/2GB more on the VM headnodes would be appreciable.

Initial tests show hdfsdu is running fine on Oozie with 512M heap and 64M perm for oozie. I'll let it run some more to make sure all functions work OK.

This is good to review/merge.


onCallListener called but onRemoteStreamListener is not

I have the following code where the onCallListener callback if fired, but the onRemoteStreamListener is not:
    try {
      const connectyCubeSession = await createSession();
      ConnectyCube.init(...getConfig(connectyCubeSession.token));
      ConnectyCube.videochat.onCallListener = (session, extension) => {
      // this is called when an incoming call occurs
        store.dispatch(incomingCall(session, extension));
      }
      ConnectyCube.videochat.onRemoteStreamListener = (...args) => {
      // this call back is never called when an incoming call occurs
      }
      ConnectyCube.videochat.onSessionConnectionStateChangedListener = (...args) => {
        store.dispatch(connectionStateChange(...args));
      };
      const connectyCubeUserProfile = await logInToConnectycube(userCredentials);
      store.dispatch(userData({ connectyCubeUserProfile }));
      await connectToChat(connectyCubeSession, connectyCubeUserProfile);
    } catch (error) {

I am running this code when a HeadlessJS task is triggered in the background on Android. The session and extension are received by the onCallListener but for some reason a remote stream is never received.  I have tried moving the listeners to after the SDK is connected to chat, but it makes no difference. What could be the problem?
I'm using RN v 0.61.4 and "react-native-connectycube": "^3.0.1",
Thanks.

Hi @whenmoon do you accept the call in onCallListener ?

Actually that's a good point - this particular code is for a new user journey / call answer flow that is triggered by a notification button press.
So the call has to be accepted for the remote stream listener to be triggered?

@whenmoon correct
the real streams exchange happens after the remote party accepted the call

Ok got it - thanks that must be the problem. I'll close the issue then cheers.

hi @whenmoon i am also having the similar problem. Can you please share how you have solved it?


[Feat] Add animated cursor

Description
Enhance the website's interactivity and aesthetics by adding an animated cursor. A custom cursor can make the UI feel more engaging and provide a unique user experience.
Proposed Solution
Implement a smooth, animated cursor using CSS and JavaScript.
Offer fallback to the default cursor for unsupported browsers.
Optionally, customize the cursor based on user actions (e.g., hover effects or different pages).
This feature will make the site more dynamic and visually appealing.
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

@RachitSahu26 As of now, we do not need it. We will let you know in the future if there is a need. Please search for another issue and raise it, which I'm happy to assign to you.


ember deploy:list - undefined is not a function

I'm using ember-cli version 0.2.3.
'ember deploy:list' gives the following error:
user@linux:~/test-projects/testtoazure/config > ember deploy:list
version: 0.2.3
Could not find watchman, falling back to NodeWatcher for file system events.
Visit http://www.ember-cli.com/#watchman for more info.
undefined is not a function
TypeError: undefined is not a function
    at Class.module.exports.Task.extend.run (/home/eric/test-projects/testtoazure/node_modules/ember-cli-deploy/lib/tasks/list.js:26:19)
    at Class.module.exports.run (/home/eric/test-projects/testtoazure/node_modules/ember-cli-deploy/lib/commands/list.js:19:21)
    at Class.<anonymous> (/home/eric/test-projects/testtoazure/node_modules/ember-cli/lib/models/command.js:136:17)
    at lib$rsvp$$internal$$tryCatch (/home/eric/test-projects/testtoazure/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:489:16)
    at lib$rsvp$$internal$$invokeCallback (/home/eric/test-projects/testtoazure/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:501:17)
    at lib$rsvp$$internal$$publish (/home/eric/test-projects/testtoazure/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:472:11)
    at lib$rsvp$asap$$flush (/home/eric/test-projects/testtoazure/node_modules/ember-cli/node_modules/rsvp/dist/rsvp.js:1290:9)
    at process._tickCallback (node.js:355:11)
user@linux:~/test-projects/testtoazure/config >

do you have an index-adapter addon installed?

actually I ran into this myself right now. This happens when you try to run a command in an environment that is not specified in deploy.js.
I agree that this is annoying and we need to improve the error message in this case. thx for this issue.

closing in favor of https://github.com/LevelbossMike/ember-deploy-redis/issues/8

I seem to be getting this error no matter what I try even when my environment does exist (development)


Add history exclusion configuration

A rough draft PR to solve #516, by letting users configure which command lines get excluded from history. For example, users can configure reedline so that command lines starting with a space are not saved.
@sholderbach Let me know what you think, happy to revise this.

Doing the string match internally is certainly the minimally invasive change. How will this interact with the data amending as done for the sqlite history?

@sholderbach I don't think there's any special handling for SQLite history needed, because all the logic is happening in engine.rs. Perhaps I've missed something?

Starting the session without a normal command being run crashes, as soon as I try to add an entry that is skipped (with --features sqlite):

I think it tries to modify the last entry (and passes if it can mess with the last succesfully stored one)

Oops! Will take a look and try to add some tests, thanks.

Hm. Y'know, I'm not sure whether that's a bug. A case could be made that the demo is intentionally not handling that edge case well:
https://github.com/nushell/reedline/blob/076ce6f7fca5f3e1f63bbad9b9b940ac1463bd2a/examples/demo.rs#L130

I think this rather indicates even with ignoring error handling that https://docs.rs/reedline/latest/reedline/struct.Reedline.html#method.update_last_command_context would try to modify the last remembered entry.

Right. I don't see the problem but I'm probably missing something.

current status: I'm not really sure what the best way to implement this is. Might need to start storing the last command in memory instead of always looking it up in the DB. I've run out of steam to work on this for now but might come back to it another time; if anyone else wants to pick it up feel free.

Closed by #566


Reword some part of the french script to improve lipsync

Those rewordings are based on dubbing attempts on the latest animation to date
@remram44 this would introduce one conflicting line with your PR, I purposely didn't fix punctuation
@scribblemaniac

Any opinions on that?

Some changes make this closer to oral speech, some seems to make it farther. I am not sure what is the intent?

The intent is to make it closer to the lip synchronisation of the released animation on the 12th of June.
It is based on the changes made while recording our french dubbing. We expect other french dubbing attempts to benefit from those changes, hence this PR.
All meanings are retained and it's still a suitable subtitle for the english or russian versions.


Small changes required for an upcoming Atom release

Hi! Atom will soon be upgrading to a new version of Electron, which will cause some APIs to change.
You'll need to change this usage of require('remote').require('browser-window') to require('electron').remote.BrowserWindow.
There may be other similar small changes required. See this post on the electron blog for more information. And let me know if you need any additional help. Thanks!

Deprecation Cop message:

Use remote.BrowserWindow instead of remote.require("browser-window")
Object.require (C:\Users\jerone\AppData\Local\atom\app-1.12.0-beta2\resources\app.asar\src\electron-shims.js:54:12)
Object.<anonymous> (C:\Users\jerone\.atom\packages\dev-tools-themes\lib\dev-tools-themes.js:3:28)
Module.load (module.js:458:32)
tryModuleLoad (module.js:417:12)
Function._load (module.js:409:3)
Package.requireMainModule (C:\Users\jerone\AppData\Local\atom\app-1.12.0-beta2\resources\app.asar\src\package.js:741:27)


Support length in JSONPath

What happened: I run the following cmd kubectl get crd --selector="svcat=true" -o=jsonpath='{.items[0].spec.names.categories.length()
What you expected to happen: I expected the command to return the length of the categories list, as JSONPath has length function defined

Environment:

Kubernetes client and server versions (use kubectl version): Client: 1.20 - Server: 1.16

After discussing we recommend you use jq here. It will have much more features than are basic jsonpath implementation.
/close


Hello World project

Add Html And Css files.
Add linters.

🔍🚫 Hey, @jalal1379! I noticed that this is your second pull request for the same project and you deleted it before reviewing it. In the first pull request, the reviewer requested modifications to your code. Please avoid creating new pull requests for same project requirement.
Instead of creating multiple pull requests for the same feature, it's best to focus on a single pull request and iterate on it based on the feedback received. Well! you have done all requirements requested by the previous reviewer. Deleting this pull request is understandable if it was created unintentionally.
Remember, it's more efficient to consolidate all changes and updates into one pull request to maintain clarity and facilitate a smoother review process. Thank you for understanding! 🙏😊


HTML Generator

How do you feel about something like this:
function parseHtml(htmlString){
  var d = document.createElement('div');
  d.innerHTML = htmlString;
  return u(d).children();
}

var o = parseHtml('
New paragraph
');
u('body').append(o.first().outerHTML);

I did the work for this part already, it was quite different:
// Usage
u('
');

// Check in `.select()`
: /^</.test(parameter) ? this.select.html(parameter)

However, please read the [maybe] notes in Roadmap and new features before continuing with this work, as I'm not yet sure whether or not they are within the scope of Umbrella

And u('') ?

However that doesn't seem to be possible using insertAdjacentHTML, which is used internally and that I don't think it's a good idea to change

What do you mean with this?
And u('
')

u('

')

Yes that also seems to be the same, since it only takes into account whether it starts by < or not (so it does). However without changing all the methods for inserting html, as you said, it doesn't make much sense to implement this and .clone(), and I don't think it's a good idea to change them. Sorry

I do think this is a required feature, its useful to, for example, generate html dynamically at an ajax callback

But your example maybe all it needs

It is useful, but I think that the most useful part is what it's already there; an ajax call returning an html string can already be used since these methods would accept that string. While it'd be more useful if it could accept other things, I think that the cost for that (rewriting many parts of the core with sub-optimal methods) doesn't make it suitable.

Yes, I just tested it. This stores them as nodes in the current u() object:
var html = '
Bla

Bla
';
var lis = u(document.createElement('div')).html(html).children();

Then you can use appendChild, for example:
lis.each(function(li){
  u('ul').first().appendChild(li);
});

I'm ok with that, closing.

I opened it to implement what was suggested here. It should be implemented with some performance baked in though. See StackOverflow for an example.

Okay, it was added. The appendTo() and any other ___To are not welcome, but otherwise it's compatible with jQuery. Actually it's much better, since all of these work:
u('ul').append(el => `<li>${ el }</li>`, ['a', 'b', 'c']);

function greeting(){ alert("Hello world"); }

u('ul').append(function(el) {
  return u('
').html(el).on('click', greeting);
}, ['a', 'b', 'c']);


Generate Python typing compatible text signatures (and stubs)

My .pyx file uses a PEP-484 style type signature:
#cython: embedsignature=True
def f(path: str, a: int = 0, b: bool = True) -> typing.List[str]:
    return []

Cython 0.29.13 turns this into the docstring:
f(unicode path: str, a: int = 0, bool b: bool = True) -> typing.List[str]
(see also #1370 for "unicode")
When using mypy-stubgen, that docstring turns into:
def f(unicodepath: str, a: int = ..., boolb: bool = ...) -> typing.List[str]: ...
Which causes mypy false positives for Python code calling f with a named argument for b.
I think embedsignature should not output Cython-specific syntax, as that breaks tooling consuming these docstrings.
Ideally it'd also convert Cython signatures into the corresponding Python signatures; but it would be good enough for me if it just left PEP484 signatures as-is.

Alternate ideas:

Cython could maybe generate .pyi stubs itself?
I could enable binding=True to expose the signature to Python tooling. However the current mypy-stubgen only emits function declarations for what it considers a "C function":

def is_c_function(obj: object) -> bool:
    return inspect.isbuiltin(obj) or type(obj) is type(ord)

so all Cython functions with binding enabled get converted to a simple f: typing.Any. This should probably be fixed on the mypy side.

If generating .pyi stubs is not too complex, Cython could do it automatically, yes, similar to how it now generates the embedded signatures in AutoDocTransform. PR welcome.
I also agree that generating 'normal' Py3 annotations would be nicer than generating Cython syntax. PR welcome for that, too.

Looking forward to this functionality! I usually write all documentation in .pyx files, so it could be very nice to have automatic .pyi generation!

@cmpute PR welcome, as I wrote. See the AutoDocTransform.

Note that mypy has a tool called stubgen to generate stub files from runtime information.
For Cython, it suffers from the fact that a few special methods like __init__ and __call__ cannot expose their signatures as runtime type information. Apart from that, it should generally work with Cython.

Over in PR #5415, we started a discussion about mixing Cython's C style type declarations with Python type annotations, which I'm generally opposed to due to the potential confusion that this causes. Let's continue the discussion in this issue since it leaves the scope of a PR.
An example is:
    @classmethod
    def c03(type cls: type[Foo], unsigned int a: int) -> float: return a

where the Cython type declarations (and note that this only works in Cython .pyx syntax) are more specific and concrete than the Python type annotations, but the latter can be used to generate user defined text signatures and .pyi stub files. The main issue here are the two audiences of the declarations. Python type annotations are geared towards humans, whereas the Cython C type declaration is meant for the compiler.
@dalcinl wrote:

Remember that "Python type declarations" are actually just annotations.

… unless you use them as the way you declare types for Cython. In that case, they are type declarations, not just annotations. The fact that you can mix both in .pyx files is, well, limited to .pyx files. It doesn't help users who write their Cython code in .py modules to tell them that their life could be so much easier and cooler if they switched to non-Python syntax extensions and renamed their files. We are currently trying to discourage writing new code in Cython syntax and use Python syntax instead, simply because there is much more tooling available for writing Python code than for Cython code.
I don't think we should reject the mix of both as invalid syntax, but I also wouldn't want to present mixing both forms of type declarations as the best of both worlds. I don't want Python type annotations to be considered a second class citizen that is only looked at if there is nothing more convincing and can otherwise be happily ignored by the compiler. Any redundancy in code bares a potential risk of getting outdated, out of sync, and generally confusing. I consider duplicate type declarations something that Cython might warn about, not something to encourage.
@dalcinl suggested:

def foo(int arg: Annotated[int, MaxValueAllowed(42)]) -> Any:
    ...

In Python syntax, you would spell it as follows:
def foo(arg: Annotated[cython.int, MaxValueAllowed(42)]) -> Any:
    ...

I don't see why the two type declarations should be preferred over a single one here.

I don't see why the two type declarations should be preferred over a single one here.

Unless you are using #cython: annotation_typing=False in a pyx file.


Whenever I search for your project, I get pants.

Expected Behavior
DDG/Google search for "docker slim"
Receive link to this project.

Actual Behavior
Receive images of pants.

Steps to Reproduce the Problem
As above.
(On a more serious note, thank you for a cool project 👍 )

@mahalel This is a known problem and we are exploring all possible options to fix it including eliminating all slim pants and bringing back cargo pants in style :-)


Introduce Frontend Migrations instead of API versioning

Based on the discussion in https://github.com/CodeForPoznan/pah-fm/issues/426 and last meeting, we have to start doing something with breaking changes between API versions.
I also hereby declare that every Issue marked with the new Breaking API label is blocked until this ticket is closed.

1. One solution here is to version the API in a meaningful way and react differently depending on the version that's being used.
Effectively, we'd keep two kinds of the code:

a new one, which can use changed API
and an old one, which would be marked as obsolete but would still work functionally correctly and (most probably) will encapsulate logic needed for converting the old format to a new one.

DOCS: https://www.django-rest-framework.org/api-guide/versioning/#acceptheaderversioning

Versioning based on accept headers is generally considered as best practice, although other styles may be suitable depending on your client requirements.

2. Another solution is to use frontend migrations heavily. These migrations would be ran on users' browsers and would convert all drives stored locally to the newest version.
This solution has main, big advantage - no API versioning :)
Simple, mainstream flow of the endpoints would be ensured so we wouldn't have to bend in different ways perserving legacy code, just to make sure that some potential request is not rejected.
Migrations also fall in nicely into UX - user goes online, browser fetches latest version of itself (along with migrations), compares the version of store, sees the difference and shows a nice notification to the user - "Hey, there's a new version around! Refresh the page to see it live".
User refreshes website (simple!) and then the migrations are applied, store gets updated and is ready to be sent to the backend. Nice and simple. Also, very simple to test thoroughly with front-end unit tests (#414) so we can make sure that stuff doesn't break.
Same with backend API tests, we wouldn't have to keep around two versions of the tests for the same endpoint. Instead, we could easily go forward with releases.

Quick update here:
The problem with first solution is complexity. It doubles. That's bad.
Everything will end up duplicated - tests, endpoints, serializers. I don't think that we want that.
The problem with second solution is unknown territory. I found very little information about it, especially in case of vuex. The closest thing to migrations that I've tracked is using vuex with IndexedDB as a backend. IDB has an API that provides simplified interface to database migrations, but I think that it might be a massive overkill, which leads us to our own solution :tm:
I don't know much about this stuff, unfortunately, so I'm hoping to get some open discussion here.
@Bosmanfrx, can you help us with this?

Another update, this time from @Bosmanfrx :)
He suggested, that we can write custom handler for vuex-persistedstate that we use in pah in order to introduce migrations when gettng items with getItem.
Simplest possible example of that can be seen here: https://blog.pragmatists.com/data-persistence-in-evolving-web-application-efbf89ee7c19
However in order to go forward with this we have to have confidence in our newly written migration code, so we have to bootstrap frontend unit testing first. We have a ticket for it and as far as I'm aware that's a great starting point for extending the frontend code :)
https://github.com/CodeForPoznan/pah-fm/issues/414
So the rough plan would be:

Bootstrap unit testing side of code
Hook it up to travis
Find a way to attach the migrations to vuex or launch them at start
???

we'll discuss this more on next meeting.

Update the ticket minding the migration to React


Scheme validation

xsd scheme for Channels.xml and Products.xml
add resources to release executable
lxml dependency

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 28.57143% with 20 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

easy2homeassistant/xmlvalidator.py
33.33%
14 Missing :warning:

easy2homeassistant/easy2homeassistant.py
14.28%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

easy2homeassistant/easy2homeassistant.py
78.53% <14.28%> (-2.45%)
:arrow_down:

easy2homeassistant/xmlvalidator.py
33.33% <33.33%> (ø)


feat(reports): xlsx attachments

SUMMARY
Most people and departments in enterprise companies are addicted to spreadsheets. Currently our customers are heavily demanding Excel reports and, unfortunately, we have failed to persuade them to continue with CSV.
So here I am with a PR that adds the ability to attach XLSX files as report data and notifications (slack). If this one gets approved, I will also attempt to implement PDF in a similar way.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
Try sending an xlsx report or notification. I wasn't able to test slack since I don't use it actively but implemented it nevertheless for completeness.
Known Problems
Currently the first column is displayed like

It can be monkey patched but it should be fixed naturally once/if #19735 lands.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

Any update on this?

@cemremengu , I've posted this in superset slack (https://apache-superset.slack.com/archives/CCKHMGRRB/p1662748936923259) and there's some feedback. Hopefully get some 👀 on it. Thank you for creating this PR!

@cemremengu Thanks. Can you format a file with black to fix CI: Python Misc / pre-commit (3.8)?
pip install black
black superset/tests/integration_tests/reports/commands_tests.py

@cemremengu Can you also add Excel support in Download on explore page for consistency?

@cemremengu you forgot to add

to the file superset/reports/commands/execute.py
Otherwise, email reports look like this:

@cemremengu Thanks. Can you format a file with black to fix CI: Python Misc / pre-commit (3.8)?
pip install black
black superset/tests/integration_tests/reports/commands_tests.py

@EugeneTorap you mean replacing all occurrences of XLSX like  EXPORT_TO_EXCEL right ?

I think yes, need to replace all UI label from XLSX to Excel for name consistency.
@villebro @zhaoyongjie @michael-s-molina  What do you think?

Just to clarify @EugeneTorap should something like this be
<StyledRadio value="EXCEL">{t('Send as Excel')}</StyledRadio>
or
<StyledRadio value="Excel">{t('Send as Excel')}</StyledRadio>

@cemremengu Value must be XLSX
<StyledRadio value="XLSX">{t('Send as Excel')}</StyledRadio>

@EugeneTorap Fixed 2 of them, I agree it is better like this.

/testenv up

/testenv up FEATURE_ALERT_REPORTS=true

@cemremengu it looks like there was a PR by @lyndsiWilliams where some translations are moved into constants in one of the files. Would you mind rebasing off master to conform to this change? Sorry we didn't get this one merged first ;)

Hi @dpgaspar @villebro. Can you review the PR?

@EugeneTorap here is my attempt, still needs to be checked for runtime errors in a test-env though.

@cemremengu Btw, python integration tests are currently failing for this,
superset.reports.commands.exceptions.ReportScheduleUnexpectedError: argument of type 'method' is not iterable

@kakoni thank you for the tip!

@cemremengu did you get time to fix this ?

The test-mysql, test-postgres, and test-sqlite actions are failing due to this:

FAILED tests/integration_tests/reports/commands_tests.py::test_email_chart_report_schedule_with_csv - superset.reports.commands.exceptions.ReportScheduleDataFrameFailedError: Failed generating dataframe Input string must be text, not bytes

the pre-commit check is failing because of a linting error. You can probably just run pre-commit run --all-files and commit any changes to clear that up.

@rusackas can we please unblock this by answering @cemremengu's question in last comment.

Hi all! Sorry for not answering the last question, but this came up in my "PRs that haven't been touched in forever" audit, and  I didn't realize I was the one blocking things. I actually don't know the answer to the standing question, but if anyone/everyone wants to spin this up, let's at least start with a rebase. I'll convert this to Draft mode in the meantime, and ping @yousoph and @eschutho who may have interest in helping usher this along. Let's see if we can get it across the finish line!


Ensure esgateway secret provided by the user do not have ownerreferences added

Description
This change add a new test to make sure an OwnerReference is not being added to an esgateway custom secret
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

I'm sorry but I failed delete the branch after merging the pull request.


[WIP][SR-648] Add option to create statically linked binaries

This is a proposed fix for  SR-648 to produce static binaries with the package manager. If this is ok then I will add some test cases for it.
It allows static binaries to be created on supported platforms:
swiftc -emit-executable -static-executable -o main.static main.swift
It involves some linker magic to work around dlopen()/dlsym() not working on static binaries to fixup the conformance and metadata sections at startup

Adds -static-executable option to swiftc to use along with
-emit-executable

Only works on Linux for now, requires a stub to replace
libdl functions

Darwin doesnt support static binaries, see
https://developer.apple.com/library/content/qa/qa1118/_index.html

Stomping on system API with stubs like this feels brittle. If the binary is truly completely static, we have the full conformance table available statically, so the runtime could link against it. The registration stuff only exists to discover conformances from dylibs dynamically.

@jckarter Thanks for your comment, I have simplified the stub as the libdl functions are not actually used to find the conformance tables since the linker sets some symbols to point to them if linking statically and these are used instead. The registration stuff is then skipped entirely.
libdl is only needed to resolve the dlopen() etc however it won't actually work in a static binary and it gives the following message at link time:
warning: Using 'dlopen' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking

So I substituted them with some fatal error functions to catch any callers. I have no particular view as to linking libdl.a or using the stub functions as it will work either way although avoiding the warning message may help avoid user confusion

Do you know what is triggering that warning? It would be nice if, in the static library configuration, we didn't touch dlopen at all.

The warning is embedded in libdl.a:
$ objdump -s /usr/lib/x86_64-linux-gnu/libdl.a
In archive /usr/lib/x86_64-linux-gnu/libdl.a:

dlopen.o:     file format elf64-x86-64

Contents of section .text:
 0000 488b1424 e9000000 00                 H..$.....
Contents of section .gnu.warning.dlopen:
 0000 5573696e 67202764 6c6f7065 6e272069  Using 'dlopen' i
 0010 6e207374 61746963 616c6c79 206c696e  n statically lin
 0020 6b656420 6170706c 69636174 696f6e73  ked applications
 0030 20726571 75697265 73206174 2072756e   requires at run
 0040 74696d65 20746865 20736861 72656420  time the shared
 0050 6c696272 61726965 73206672 6f6d2074  libraries from t
 0060 68652067 6c696263 20766572 73696f6e  he glibc version
 0070 20757365 6420666f 72206c69 6e6b696e   used for linkin
 0080 6700                                 g.

Its caused by the use of libdl in ProtocolConformance.cpp. Since libswiftCore.a can be statically linked into a dynamic binary (-static-stdlib) It still needs to be able to read the conformance data from any dynamically loaded modules.
A possible solution, without creating libswiftCore.a and libswiftCoreWithoutDlopen.a, is:

Move the swift::_swift_initializeCallbacksToInspectDylib and related functions into a separate file and update the callers in ProtocolConformance.cpp and MetadataLookup.cpp to use it.

In this new file either use weak linking to the dl functions or possibly create two support libs e.g. libswiftSwiftDylibLookup.a and libswiftSwiftStaticLookup.a and link in the appropriate one at build time. This would be mirror the use of the libswiftSwiftOnoneSupport.a file

Point 1. may be worth doing anyway since there is some duplication of code between the two files and it would move all of the platform specific code with all of its #ifdefs into its own file making it easier to read ProtocolConformance.cpp

I agree, I like the idea of breaking the lookup code into its own .cpp file, letting the driver choose to link the fully-static or dynamic-loading version as needed by the build product.

I refactored all of the lookup code into SectionData.cpp/SectionData.h which tidied up ProtocolConformance.cpp and MetadataLookup.cpp. This is the first commit which I will open as a separate PR since it doesn't add or remove any functionality but is a worthwhile cleanup anyway
I did try creating 2 drivers for a DylibLookup.a and a StaticLookup.a however libicu still requires linking to libdl for dlopen() even though it doesn't use the functionality in my testing. Also trying to create two separate shared and static libs which contain different source files made the CMake files a mess of hard to maintain hacks to make it work correctly for platforms that can use it without adding extra complexity to platforms that can't.
So I went back to just modifying the lookup code for static binaries and adding in some extra checks if new sections are added without the appropriate symbols being set, and the stub just catches any calls to dl* functions and aborts as these would need to be fixed

If you're having problems getting the CMake scripts to look good, @erg might be able to help. We should look into fixing libicu upstream too. You might try reaching out to swift-dev to get more input on some of these issues. Hacking around these problems by providing our own definitions of these symbols seems to me like something that's only going to cause trouble down the line.

I agree that these libdl hacks arent the best solution.
I looked at libicu and it does have a configure.sh option
--disable-dyload        disable dynamic loading default=no

So it can run without plugins although I don't know what functionality this breaks. Of course libicu is configured by the distro package maintainers so apart from building it as part of stdlib (which is a rather heavy solution just to enable 1 build option) Im not sure what can be done here
The other fix needed is to remove the dladdr() from Errors.cpp for linux which I don't think works anyway (see SR-755)
I'll close this PR for now, I can always rework it if these issues are fixed and there is still a requirement for static binaries in the future

Thanks for looking into this, @spevans! To be clear, I think being able to emit static binaries would be awesome, and I hope I didn't discourage you by nitpicking too heavily. I doubt we rely on libicu's plugin loading feature. @dabrahams or @parkera would know for sure.

Foundation doesn't do anything like that directly, but I'm unsure if something we call in ICU does it on its own.


Fix position: sticky

.sticky-top is deleted after compilation.

No, it's not:
https://github.com/twbs/bootstrap/blob/f7b55da4509e62b559f1e8181cf4b03e964775a4/dist/css/bootstrap.css#L7796-L7803

.sticky-top is removed, leaving the wrong css code, if you use only the scss compiler


fix little grammar detail in flaskr/db.py

Just add a missing 'is' in the docstring in db.py, from the flaskr example.

If this request connected to the database

This should be: If this request is connected to the database.

It's correct as-is.

Now I see it; it was kind of confusing, like something was missing there.

Thanks for looking at it though. :-)


Add CodePen prototypes to each component

We added a CodePen prototype to the avatar component to showcase the real rendered version of the component. We should do the same for the rest of the components.

[x] Stack
[x] Button
[x] Forms
[ ] Media
[ ] App frame

Marking Media as complete:
https://github.com/thoughtbot/design-system/tree/master/src/media
https://codepen.io/thoughtbot/pen/LoNMxz

Adding App Frame Codepen here: #140


Failing unit test "ExposureWindowTest" if executed between 00:00 local time and 00:00 UTC

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Technical details

Device name: Xcode simulator / community build
iOS version: I tested with simulator iPhone 8 for iOS 15, but I think all are affected
App version: latest development version / branch release/2.14.x as of 29 Oct (commit = bdc8b885)

Describe the bug

Steps to reproduce the issue

Build for Test
wait till midnight (between 0:00 local time and and 00:00 UTC)
select unit test ExposureWindowTest
perform the unit test
see the test failing

Expected behaviour

The unit test should succeed.
Possible Fix

Additional context

Note: I guess there is an inconsistency with regard to local time vs UTC.
However, I have not analysed if the issue is with the system under test, or with the test expectation.

Internal Tracking ID: EXPOSUREAPP-10382

@ndegendogo Thanks. Internal Tracking ID: EXPOSUREAPP-10382

Has this been fixed @ndegendogo?


Typo: currnetPodIps

In pkg/controllers/netpol/policy.go you have the variable currnetPodIps but I guess you mean currentPodIps ;)

Actually that typo is present upstream as well, I just didn't notice it when moving things around https://github.com/cloudnativelabs/kube-router/blob/v1.3.2/pkg/controllers/netpol/policy.go#L109

But it was fixed later https://github.com/cloudnativelabs/kube-router/blob/master/pkg/controllers/netpol/policy.go#L110

Fixed and pushed


Aliased symbols not resolved in cljs

Aliased symbols are not resolved in cljs, a problem that can be traced to the following code, found at:  https://github.com/rbrush/clara-rules/blob/master/src/main/clojure/clara/rules/compiler.clj#L71-L81
(defn resolve-cljs-sym
  "Resolves a ClojureScript symbol in the given namespace."
  [ns-sym sym]
  (let [ns-info (get-namespace-info ns-sym)]
    (if (namespace sym)

      ;; Symbol qualified by a namespace, so look it up in the requires info.
      (if-let [source-ns (get-in ns-info [:requires (namespace sym)])]
        (symbol (name source-ns) (name sym))
        ;; Not in the requires block, so assume the qualified name is a refers and simply return the symbol.
        sym)
     ;; ...

(get-in ns-info [:requires (namespace sym)) doesn't function as wished, because the keys of the map represented by :requires are symbols, while namespace returns a string. The means that sym is always returned, which works in the case of fully-qualified symbols or referred symbols, but not in the case of aliased symbols.
Fix is simple:
(get-in ns-info [:requires (symbol (namespace sym))])

PR to follow.

PR submitted: https://github.com/rbrush/clara-rules/pull/114

Fix was merged.


get_data_types documentation

Hello,
In documentation, for get_data_types, schema_writing isn't an argument :

Regards,
Alexandre

This doc is corrected in v0.9.0 and it will be soon available.


Add release notes for patches

I think there are a couple of extra things that have to happen before these will show up. We need to update the Release Notes landing page to link to these and we need to update SUMMARY.md so the links show up in the navigation. I'm happy to do that on this branch. I'll also make a note to make this more clear in the Docs Best Practices guide, because it's definitely not right now.

Yeah, I actually did know that.  Thank you for finishing that! (I can certainly do it, if I get a chance to do it before you)

I just made the updates, I'm going to go ahead an approve myself (which is probably not the best practice.

Thank you!


Fix #23013: Add boder-top:0 in .list-group-flush .list-group-item

Fix issue mention #23013. I have added boder-top:0 in file _list-group.scss in class .list-group-flush .list-group-item this isssue only shows in chrome browser.

i think you misread the issue, we want the border to be visible not to get rid of the border
Your change gets rid of the border

@gijsbotje mistakenly i have removed border from top instead of bottom, currently there are border top as well as bottom which creating this issue, so i have remove border bottom to fix issue, i am also attaching screenshot of chrome, firefox and edge browser. Also i have remove double border from card header. Due to border-top in .list-group-item and border-bottom in .card-header

@mdo This PR is break the following card layout


Change envMap to os.Getenv

change all envMap to os.Getenv
and in the beginning do NOT load to map, bud just with dotenv.Load()

not necessary, dependency injection is better


Add bias towards expected solution, better suppression of trivial solution

Failure due to timed out HTTP request for uploading the last result. Merging.


SourceWithOffsetContext

As a part of work over issue #36 , SourceWithOffsetContext consumer source will be implemented and tested in this PR.
This souse is basically provides another way of passing consumed messages and committable offset to downstream. The main difference (as far as I understand) is that while CommittableSource passes CommittableMessage to the downstream, forcing consequent flows to handle both message and it's committable offset, this SourceWithOffsetContext allows to put committable offset to the source's context.
Also, some helpers like Committer static class with committing flows are implemented in this PR, because they are used to make testing simpler/cleaner.
There is one more test for this source type, but it will require one more producer flow implementation, so it will be added in one of the next PRs.

@IgorFedchenko have some merge conflicts here

Yes, each of my PRs adds new extension method to KafkaConsumer.cs file, so seems like we will have conflicts on each merge...

@IgorFedchenko ah, ok

After updating committer default settings in reference.conf file one of the tests failed because I was expecting 10 messages on the downstream, but there are less of them due to batchSize > 1 (so now expecting elemsCount/batchSize committable batches).


Docker joins with FQDN so we should find the nodes by FQDN

Thanks Andrea!
The CI failures are not related at all to your PR.
I will fix them in a separated branch.


Crash when changing Antialias Method in Preferences (Mac / RH 1.00)

Steps to Repro:

Click "Repetier Host-Mac" in menu bar

Go to Preferences -> 3D Visualization -> Antialiasing Method

Change the Antialiasing Method --> Crash

Device Info:
OS: Mac (OS 10.11.16 - El Capitan)
Repetier Host Version: 1.00
Call Stack:

Found the problem and solved it for 1.0.1 coming soon.


Write-Up 1 by Emily Miller

Grade:
Problem 1: 2/2
Problem 2: 0/2
Output of testing/test_all.sh, showing differences in incorrect output:
Testing problem1
Success testing problem1 case 1 in 0.00 seconds.
Testing problem2
Failure testing problem2 case 1, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 37.6
< Mean: 9.4
< Multiplication: 6142.5
< Geometric Mean: 8.85291493609189
---
> 37.6
> 9.4
> 6142.5
> 8.85291493609189
Failure testing problem2 case 2, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 39.0
< Mean: 9.75
< Multiplication: 256.0
< Geometric Mean: 4.0
---
> 39.0
> 9.75
> 256.0
> 4.0
Failure testing problem2 case 3, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 0.0
< Mean: 0.0
< Multiplication: 0.0
< Geometric Mean: 0.0
---
> 0.0
> 0.0
> 0.0
> 0.0
Failure testing problem2 case 4, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 4718.0
< Mean: 1179.5
< Multiplication: 1048788.0
< Geometric Mean: 32.0016173090261
---
> 4718.0
> 1179.5
> 1048788.0
> 32.0016173090261
Failure testing problem2 case 5, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 4.0
< Mean: 1.0
< Multiplication: 1.0
< Geometric Mean: 1.0
---
> 4.0
> 1.0
> 1.0
> 1.0
Failure testing problem2 case 6, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 628.0
< Mean: 157.0
< Multiplication: 625.0
< Geometric Mean: 5.0
---
> 628.0
> 157.0
> 625.0
> 5.0
Failure testing problem2 case 7, since there was a difference in output:
1,5c1,4
< Please enter four positive numbers
< Sum: 100.0
< Mean: 25.0
< Multiplication: 0.0
< Geometric Mean: 0.0
---
> 100.0
> 25.0
> 0.0
> 0.0


[BULK] - DocuTune remediation - Sensitive terms with GUIDs (part 30)

Applying sensitive terms with GUID changes as part of Content SFI and outlined in Overview - Writing content securely - Platform Manual. Changes are part of the Microsoft-wide SFI effort. Point of contact: @CelesteDG
DocuTune v<IP_ADDRESS>
CorrelationId: ac15aa43-4e2b-437f-ab1c-fdd7e79cd4db
#docutune

#docutune-review

LGTM


Make Stash and Controller Roles Exclusive

Closes https://github.com/paritytech/substrate/issues/3566
Prevents bonded accounts from becoming controllers. Likewise prevents controllers from bonding.

Not clear that we actually want to forbid this at the chain level, and certainly not during the Kusama run-in.

Looks good but, as mentioned, iced for now. If and when this is in the pipeline to be merged, it would be nice to use this PR to push in also:

let's make helper functions for is_stash and is_controller and is_bonded (which is basically the && of the previous two) as they are quite frequently used in the code and always manually checked by reading ::exists().
The test can probably be re-written using the already-funded accounts in mock.

Close for now as discussed with @joepetrowski


how w_resnet101 weights have been trained.

Hi,
thanks for your code sharing. I am not clear what is the difference between w_resnet101 weights and resnet101 weights ?
thanks

ResNet101 is the baseline network, W-ResNet101 is the network with the color invariant layer. Both have been pre-trained on Imagenet separately.
Hope this helps!


Minor release with docs update

Codecov Report

Merging #96 into master will decrease coverage by 0.13%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
- Coverage   16.05%   15.92%   -0.14%
==========================================
  Files          10       10
  Lines         847      854       +7
  Branches      115      116       +1
==========================================
  Hits          136      136
- Misses        705      712       +7
  Partials        6        6

Impacted Files
Coverage Δ

src/pricer/analysis.py
0.00% <0.00%> (ø)

src/pricer/campaign.py
0.00% <0.00%> (ø)

src/pricer/schema.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


Add solution to Amazon Pairwise Product Problem

Hey Rashika, It would be great if you can add few questions to that Amazon article. Below, Readme file there can be solutions directory which will contain the crowdsourced solutions.

Ok i will do so.


Plugin not working in background on iOS

Hi, i am encountering an issue with the plugin in my application, which prevents it from functioning correctly on background on iOS devices. I have followed the necessary steps to grant the required permissions, such as "Always" permission for location, physical activity permissions, and background update permissions.

When I minimize the app on a real device (iPad iOS 15.7 and iPhone 11 iOS 16.5), the plugin stops working and does not fire the  on('Location') event. I have configured the event to generate a log entry locally in text format and also to send an Ajax call to a web service to register the log in my database. However, none of these actions are executed when the app is in the background.
It is important to note that when using the Xcode simulator with a simulated running activity in the city, the plugin works correctly in the background. (iPhone 13 iOS 16.4)
I would appreciate any guidance or solution you can provide to address this issue and ensure that the plugin functions properly in the background on iOS devices. On Android device works perfectly.
Thank you for your assistance and attention to this matter!

    Sorry, I forgot to include the plugin configuration code:

    BackgroundGeolocation.configure({
        locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER,
        desiredAccuracy: BackgroundGeolocation.HIGH_ACCURACY,
        stationaryRadius: 1,
        distanceFilter: 1,
        debug: false,
        interval: 10000,
        fastestInterval: 5000
    });

   BackgroundGeolocation.on('location', function(location) {

        var eventosAnteriores = $("#eventos-entrada-salida").html();

        var htmlEntradaSalida = "\
            " + moment().format("HH:mm:ss") + "\
            "+location.latitude+ " - "+ location.longitude + "\
            ENTRADA\
            \
        ";

        $("#eventos-entrada-salida").html(htmlEntradaSalida + eventosAnteriores);

        cordova.plugin.http.sendRequest('url web service log register', {
            method: 'GET',
            serializer: 'json'
        }, function(data) {
        }, function(error) {
        });

        BackgroundGeolocation.startTask(function(taskKey) {

            cordova.plugin.http.sendRequest('url web service log register', {
                method: 'GET',
                serializer: 'json'
            }, function(data) {
            }, function(error) {
            });

            BackgroundGeolocation.endTask(taskKey);
        });
    });

Hello @tsanko95 could you finally make it work?
I do you have an example? I cannot make it compile in Android.
Please let us know. Thanks!

Might be a configuration issue, it a bug in the code here.
The following works for me in iOS:
https://github.com/IsraelHikingMap/Site/blob/f5480f43e0f03c104b1fbbffbdcbfe5fa0e74c68/IsraelHiking.Web/src/application/services/geo-location.service.ts

Hi @diegoarbelaez and @HarelM  finally i make it work and figured out why the plugin wasn't working for me. It turns out that I had misconfigured it by using "locationProvider: BackgroundGeolocation.ACTIVITY_PROVIDER". The "ACTIVITY_PROVIDER" is not designed for background mode, as it clearly states in the documentation.
After some troubleshooting, I found that the correct option to use for background mode is "DISTANCE_FILTER_PROVIDER". This is the best option for background tracking, and once I made this change, the plugin started working perfectly.
So, if you are facing similar issues with the plugin, make sure to use "DISTANCE_FILTER_PROVIDER" for background mode as recommended in the documentation. I hope this helps others who might encounter the same problem.
Thanks to all for your support!

@diegoarbelaez
If it works for you, I suggest tutorial be with cordova/ionic but without Angular/React libraries but with vanilla javascript, html and css. It would be helpful for beginners.

I'll be closing this issue now.
Feel free to open a PR to update the docs or the readme.


Update 0010-simple-classical-representations.md

Wrong spelling of 'system' corrected

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @ArushSamadhia Can you please sign the CLA?

Hi @ArushSamadhia Can you please sign the CLA?

Done

Thanks!


Finish Button

Is there a better way to access the Finish button?  I have to temporarily hide it.  Right now I'm having to do this:
Form.Controls[0].Controls[2].Controls[1].Visible = false;
Originally posted: 2014-07-01T13:27:36

If you can be satisfied with just disabling it, you can set the AllowNext property on the page to false. There is not a way to access the button in the WizardControl, but you could always use the WizardPageContainer and then supply your own buttons.
Originally posted: 2014-07-01T17:41:25


Go: Segmentation violation converting .db3 to .mcap

Description
I am trying to convert .db3 files to .mcap using the go command line tool but I am getting an error. I have tried both on MacOS and Linux (both ARM64)
Error I get:
./mcap convert multiple_files_0.db3 multi.mcap --directories /opt/ros/foxy/
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x695880]

goroutine 1 [running]:
github.com/foxglove/mcap/go/ros.collectMessageSchemas.func1({0x4000024012, 0x3e8}, {0x0, 0x0}, {0x915fa0, 0x40001c8180})
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/ros/ros2db3_to_mcap.go:30 +0x30
path/filepath.Walk({0x4000024012, 0x3e8}, 0x40001937b8)
	/usr/local/go/src/path/filepath/path.go:503 +0x68
github.com/foxglove/mcap/go/ros.collectMessageSchemas({0x4000140a80, 0x2, 0x2}, {0x4000119b10, 0x1, 0x1})
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/ros/ros2db3_to_mcap.go:29 +0x17c
github.com/foxglove/mcap/go/ros.DB3ToMCAP({0x916120, 0x4000134120}, 0x400011de10, 0x4000193d48, {0x4000140a80, 0x2, 0x2})
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/ros/ros2db3_to_mcap.go:160 +0x218
github.com/foxglove/mcap/go/mcap/cmd.glob..func2(0xb5e320, {0x400015c880, 0x2, 0x4})
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/mcap/cmd/convert.go:119 +0x4f8
github.com/spf13/cobra.(*Command).execute(0xb5e320, {0x400015c840, 0x4, 0x4})
	/home/bertaveira/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:860 +0x650
github.com/spf13/cobra.(*Command).ExecuteC(0xb5eaa0)
	/home/bertaveira/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:974 +0x424
github.com/spf13/cobra.(*Command).Execute(...)
	/home/bertaveira/go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:902
github.com/foxglove/mcap/go/mcap/cmd.Execute()
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/mcap/cmd/root.go:19 +0x30
main.main()
	/media/psf/Home/Documents/CFS/dv/random/mcap/go/mcap/main.go:21 +0x20

Version: main branch
Platform: MacOS and Linux (both ARM64)

Steps To Reproduce

I have tried with both a rosbag I have and a sample one supplied for testing and I get the same error. Rosbag here: https://github.com/ros2/rosbag2/blob/master/rosbag2_tests/resources/reindex_test_bags/multiple_files/multiple_files_0.db3
I then compile using make build and then try to convert it like so:
./mcap convert multiple_files_0.db3 multi.mcap --directories /opt/ros/foxy/

@bertaveira Thanks for the report. I wasn't able to reproduce this on a stock install of ros2 foxy (although I am on x64 -- I doubt this is architecture related), but I did just merge a fix for the bug that is causing this to panic, rather than report an error. Would you mind pulling down the latest and reporting what you see now?

Yes the Panic is fixed. However I now get an error related to message types that are not a part of the bag file I am trying to convert.
E.g: converting the sample bag linked above I get these errors:
./bin/mcap convert multiple_files_0.db3 multi-test.mcap
failed to convert file: failed to list provided directory /Users/bertaveira/ros2_foxy/install:/Users/bertaveira/Documents/CFS/dv/ROS_packages/cfs_msgs/install/cfs_msgs: lstat /Users/bertaveira/ros2_foxy/install:/Users/bertaveira/Documents/CFS/dv/ROS_packages/cfs_msgs/install/cfs_msgs: no such file or directory

Thanks. From your error I realized that the environment variable AMENT_PREFIX_PATH is holding a colon-separated list of paths, rather than a single filepath. I have just merged a fix to split the prefix path so all directories are searched. Can you try again?

Glad I could help. Since the last commit (d778d34) I can't build it. I get the same error in linux and MacOS:
$ make build
mkdir -p bin
go build -o bin/mcap -tags sqlite_omit_load_extension,netgo,osusergo -ldflags '-s -w -extldflags "-static"' .
# github.com/foxglove/mcap/go/cli/mcap/cmd
cmd/convert.go:116:7: undefined: prefix
make: *** [Makefile:13: build] Error 2

Hi, very sorry about that. Please try again now.

Yes it seems to have been fixed. It both compiles and is able to convert.
The file size is a bit suspicious since it is a 1/10 of the db3 file. It also complains about not finding some schemas that are in the directory supplied. Running multiple times in a row the error of missing schemas changes until it fully disappears.
I would close this issue since I have no way to test it for now. I will wait until the new release of foxglove with support for mcap arrives to test it further since the current one gives an error of initialisation opening the mcap files. Once I can test I'll report if anything comes up. Thanks for the quick fixes!

@bertaveira ok, interesting. That error about missing schemas sounds like a clear bug - I could maybe imagine different failures per run if the iteration over definitions were nondeterministic, but for it to go away sounds broken. I'll file an issue for that and investigate separately.
One thing you can do to test the contents of the file, is try running mcap info or mcap cat on it. The latter will print a line per message in the file, with the topic next to it. It won't parse message contents yet.
Thanks for trying the tool out!


Introduce nginx configuration for rate limiting

Thanks for contributing to the cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:
This change introduces rate limiting to API calls to the Cloud Controller by exposing the limit_req nginx configuration setting. The rate limiting proposed is applied per user, using nginx' $http_authorization variable.

An explanation of the use cases your change solves
It will prevent overloading the cloud controller with too many requests. It protects legit requests by applying the limit per-user.

[x] I have viewed signed and have submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[x] I have run all the unit tests using bundle exec rake

[x] I have run CF Acceptance Tests on bosh lite

Please note that the change was implemented by @petergtz and @jordanj.
@smoser-ibm & @suhlig from the Flintstone team

Hey @suhlig,
Thanks for the pull request.  We tested it out and were able to get it to successfully rate limit, and it passes our test suites when unconfigured.
Merged!
@thausler786
CAPI team member


💡[Feature]: I want to add a project in opencv projects

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I have worked on some of the open source projects i would like to add to this repo
Use Case
It will help for others to go through the opencv projects
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

can i know which project and some more details please.

The project is parking space detection and counter where we can identify the space available for parking a vehicle in a realtime image or in a video
And I also worked on a couple more projects in opencv


feat(tier4_perception_launch): test to skip detection by tracker output from camera-lidar-radar fusion

Description

Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

See https://github.com/autowarefoundation/autoware.universe/pull/5313 for update this PR.

May be replaced by https://github.com/tier4/autoware.universe/pull/964.