Visual Studio Marketplace link in README points to a non-existing extension

The extension does not exist on the vsts marketplace. Is VSTS online not supported anymore?

@moddabiri Currently the project is marked as private on the Marketplace as we evaluate the extension.  We currently are not taking any new testers but will post any updates to this repo if things change.


fix: update to stacks.js 4.0.0 (removes username checking)

Try out this version of the Hiro Wallet - download extension builds.

scope

updates stacks.js deps to 4.0.0
fixes test issues and switches to node test environment

Where can we see the changes made to auth?
I can't think of any immediate dangers. Auth does a lot so if we can decouple any BNS features of it, then great. I don't fully understand why auth needs to look up/match BNS names in the first place.

I agree that name resolution should be decoupled from authentication. It's easy enough to expose functions to resolve names to addresses, I personally don't think it needs to be included in an authentication response.

Ok folks — the current deployment should work and NOT include username checks ⛵

I'm seeing errors and missing data after entering my password upon sign in here on the home screen:

Where can we see the changes made to auth?

✨ https://github.com/hirosystems/stacks.js/pull/1230

Is this PR still needed for the wallet to remove username checking now that https://github.com/hirosystems/stacks.js/pull/1230 has removed it on the app side?

It's happening 🔥 can't wait to see this one merged so we can finally add Hiro wallet support to Sigle!

in 0f3a2b9 I had to skip a test (not sure why this fails), if somebody wants to look into it. may be related to switching to node as a test environment.
in 0f3a2b9 I also added --forceExit to jest, because some tests are leaking (traceable w/ --detectOpenHandles, after updating jest to latest)

handing back off to #userx, feel free to merge, or just use as a test-app for 4.0.0 stacks.js w/ connect.
(needed for arbitrary message signing)

Yeah, I do like this! In general I'm a fan of supporting a more "virtual" auth flow. On ETH there are no auth checks, so you can "log in" as any address you want to an app. This has some nice benefits - like I can login to an app using my vault address while on my phone, which doesn't have that key. If an app wants to strictly verify ownership (which is a more edge use-case, like gating premium features) they can always use other APIs for that.

Before we merge, can we rebase this PR into 1 (or a few commits) 🙏🏼


Help to translate

Hi Frank, I saw your twitter post. I'm Brazilian and I would like to help you translate the circuit into pt-BR

Thank you so much! I will create a machine translation to kick things off, and then would love it if you could correct it. Thank you!

Here is the starting translation: https://github.com/praeclarum/CircuitTranslations/blob/master/pt.lproj/Localizable.strings
I am not currently doing region-specific, so if you could make the translation work for people in Portugal too, that would be helpful thanks!

Sure, is it possible have two portuguese translation, one for brazil and another for portugal?

OK, I split it up:

BR https://github.com/praeclarum/CircuitTranslations/blob/master/pt-BR.lproj/Localizable.strings
PT https://github.com/praeclarum/CircuitTranslations/blob/master/pt-PT.lproj/Localizable.strings

Thanks for helping!

Hi @praeclarum , sorry for delay.
I'm translating the best way, always checking in electronic books. And I would like to ask questions about some terms that I not sure about them, if you can send me a photo, schematic, detailed explanation.
DC Rail;
Fwd Voltage @ 1A;
Monospace;
Off Resistance (Ω);
On Resistance (Ω).

Thanks, I appreciate you taking the time!

DC Rail is often seen in the context of Operational Amplifiers. The "rail" refers to a maximum (or minimum) that the amplifier can achieve. They're DC because they are power and don't affect the signal.
Fwd Voltage @ 1A is the Forward Voltage (or Bias) of a Diode when 1A of current is traveling through it. https://en.wikipedia.org/wiki/Diode#Forward_bias
Monospace refers to a single width Font (all letters consume the same horizontal space). It's used in programming and looks like what a type writer produces.

On Resistance (Ω) is how much resistance a Relay provides when it is in the On state.


Kindly,
Julien

How about adding an 'after' decorator to go with it?

I was thinking about it and then thought it would perhaps not be useful ;) but I will add it

@girlwithglasses @jayphelps,
I was confused about how to deal with an asynchronous task and would like some help on it.
@before(functionToBeCalledBefore)

functionToBeCalledAfter synchronous: fine, and we just apply decorated one then
functionToBeCalledAfter asynchronous: I choosed to only support a Promise, any better approach ?

@after(functionToBeCalledAfter)

functionToBeCalledAfter is sync/async :  no matter, we just execute it
but problem is what if decorated function is doing something async, how to trigger something to tell decorator to execute its functionToBeCalledAfter function ?

@darul75 sorry about the delay! :frowning:
Can you describe some real world use cases that this helps solve?

Closing due to inactivity. Feel free to add some real world use cases and we could discuss further.


Still get deprecated warnings despite setting IceCube.compatibility = 12

Hi,
I've read through all of the related issues and was hopeful that adding this setting to config/initializers/ice_cube.rb would help, but in my case it didn't.
The warning in the log is:
ruby-2.1.3/gems/ice_cube-0.12.1/lib/ice_cube/parsers/hash_parser.rb:11:in `to_schedule'
We're using Ruby 2.1.3, Rails 4.1.6 and ice_cube 0.12.1 and not referencing start_date anywhere in our code.
I'd love to rid the log of the extra 'noise' any ideas/help would be much appreciated.
Jeff.

I can't reproduce your error or warning here (just cleaned up your console output so it's executable):
require 'active_support/time'
serialized_hash = {
    :start_time => {
        :time => Time.parse("2014-10-01 17:00:00 UTC"),
        :zone => "Eastern Time (US & Canada)"
    },
        :rrules => [
          {
            :validations => {
                :day => [
                    6
                ]
            },
              :rule_type => "IceCube::WeeklyRule",
               :interval => 1,
             :week_start => 0,
                  :until => Time.parse("2014-10-04 13:00:00 -0400")
        }
    ],
        :rtimes => [],
       :extimes => [
        {
            :time => Time.parse("2014-10-04 17:00:00 UTC"),
            :zone => "Eastern Time (US & Canada)"
        }
    ]
}

IceCube::Schedule.from_hash(serialized_hash)

It's possible you have an older serialized schedule stored with "start_date" instead of "start_time" as a hash key. See UPGRADING for info.


Dracula extension unavailable on marketplace

Originally reported in https://github.com/dracula/dracula-theme/issues/878
The official Dracula VS Code extension appears to have been unpublished from the marketplace and locked. cc @isidorn

Fixed by @madhurivadaligithub
The extension is again available. I will comment on the VS Code issue with more details since all the users are there. Thus closing this.


#1174 Replace invalid row value with nowrap

The value used for the "wrap" property of Stack component is not a valid one. It should be "nowrap".
Description

The "wrap" property of the Chakra UI Stack component is a short hand one for  the flexWrap property. There are only 3 valid options - wrap, nowrap and wrap-reverse. There is no "row" property. More information https://developer.mozilla.org/en-US/docs/Web/CSS/flex-wrap
Types of Changes

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

Change the value "row" to nowrap.

How to Test-Drive This PR

Add item to the cart
Open cart page

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[x] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Hi @mdenchev-aiopsgroup, thanks for informing us of the issue and for submitting a fix. The PR looks good. However, before we can merge, we will need all commits on the PR to be signed.

@mdenchev-aiopsgroup here is more info on signing commits if you're not already familiar: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits


Files Conflict: Replace Exisiting Button for all Conflicts

What's the Problem?
When copying some files with the same name to another folder the conflict windows suggests two options:

Generate new name
Replace exisiting file
You have to decide manually for each file whether to replace or generate a new name.

Solution/Idea
I wish to have an option for choosing once for all confilcts.
Alternatives
Just deciding manually for each file.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

A better UI is in the works, but for now you can:

select the option for one then right click and select "apply to all"
or press "Ctrl+A" before selecting an option


Add HCTeams.ORG (This is not the original HCTeams)

General:

[x] The pull request title is descriptive. (ex. Added Lunar Network, not Updated metadata.json)
[x] The pull request does not contain any unrelated commits. (ex. commits from previous pull requests)

Mapping Additions or Updates:

[x] Folder with appropriate name has been created / updated (must not include any non-alpha characters or whitespace).
[x] No changes were made to the file's formatting (4 spaces per tab).
[x] There are no syntax errors.
[x] There is no pre-existing mapping matching my id (check if there is an existing folder).
[x] My field values match your requirements.
You can view our patterns here: metadata.schema.json, or take a look below and complete the field checklist:

[x] id: a lowercase string, which should match the folder name (ex. myserver)
[x] name: a string (ex. MyServerPvP)
[x] description: hook between 16 and 80 characters (ex. Home of Competitive Minecraft PvP)
[x] addresses: an array with lowercase strings (ex. ["my.server", "your-server.com"])

You do not need to specify subdomains, Lunar Client services automatically detect them.

[x] primaryAddress: the primary address that people connect to with (please include the subdomain if required) (ex. mc.my.server)
[x] minecraftVersions: an array with Minecraft versions as strings _(ex. ["1.18._", "1.19.2"] - Must be versions or subversions supported by Lunar Client)*
[x] primaryMinecraftVersion: a Minecraft version as a string (ex. 1.19.2 - Must be a subversion supported by Lunar Client)
[x] primaryColor: a hexademical color code that primarily distinguishes the server (ex. #00FFFF)
[x] secondaryColor: a hexademical color code that accompanies the primaryColor of the server (ex. #FF0000)
[ ] primaryRegion: the primary region where your server operates in (ex. NA)
[ ] regions: a list of regions where you have servers located that service your players (ex. ["NA", "EU", "AS"])
[x] gameTypes: a list of games that describe the content on your server, must be a max of 3 listed (ez. ["PVP", "UHC", "HCF"])
[ ] crossplay: whether the server has support for Bedrock Edition players (through proxies such as GeyserMC)
[ ] (optional) presentationVideo: YouTube video ID (in slug) to server trailer / introduction (ex. 7EV4cPuJvXE)
[x] (optional) website: url of server website, must include URL schema (http:// or https://) (ex. https://www.your-server.com)
[x] (optional) store: url of server store, must include URL schema (http:// or https://) (ex. https://store.your-server.com)
[ ] (optional) wiki: url of server wiki, must include URL schema (http:// or https://) (ex. https://wiki.your-server.com)
[ ] (optional) merch: url of server merchanise site (if seperate from store), must include URL schema (http:// or https://) (ex. https://merch.your-server.com)

Socials

[x] (optional) twitter: username of twitter account without the @ (ex. MyServer)
[x] (optional) discord: invite link to discord (ex. https://discord.com/invite/4ceEJuH)
[ ] (optional) youtube: slug or username of youtube channel (ex. MyServer)
[ ] (optional) instagram: username of instagram account (ex. MyServer)
[ ] (optional) twitch: username of twitch account (ex. MyServer)
[ ] (optional) telegram: slug of telegram group (ex. MyServer)
[ ] (optional) reddit: slug of subdreddit wtih 'r/' (ex. MyServer)
[ ] (optional) tiktok: username of tiktok account (ex. MyServer)
[ ] (optional) facebook: slug of facebook page (ex. MyServer)

Compliance

[x] (optional) privacyPolicy: url to your Privacy Policy (ex. https://www.lunar.gg/privacy)
[x] (optional) termsOfService: url to your Terms of Service (ex. https://www.lunar.gg/terms)
[x] (optional) rules: url to your Rules (ex. https://www.lunar.gg/rules)
[ ] (optional) support: url to your Support Site (ex. https://www.lunar.gg/support)

Media:
Logo

[x] My logo is a png file.
[x] I have uploaded my logo to my server folder (ex. lunarnetwork) and named it logo.png.
[x] My logo has a transparent background and is square (1:1 aspect ratio).
[x] My logo is 512 pixels in width and height.
[x] My logo is my own property and complies with relevant copyright/privacy laws.

Background

[x] My background is a png file.
[x] I have uploaded my background to my server folder (ex. lunarnetwork) and named it background.png.
[x] My background has an aspect ratio of 16:9.
[x] My background is atleast 1920 pixels in width and 1080 pixels in height (can be bigger, but must remain the same aspect ratio).
[x] My background contains relevant content pertaining to the server, is my own property, and complies with relevant copyright/privacy laws.
[x] My background doesn't contain any markings, text, or logos (unless part of a build).

Wny bro 🥲
On Sat, Jul 27, 2024 at 10:11 PM Anakin @.***> wrote:

Closed #2449 https://github.com/LunarClient/ServerMappings/pull/2449.
—
Reply to this email directly, view it on GitHub
https://github.com/LunarClient/ServerMappings/pull/2449#event-13669948831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3YRP27NJ6AL5QVG2YCJKYLZOROPRAVCNFSM6AAAAABLRN7IHCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGY3DSOJUHA4DGMI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Astrum Operations🌐 Website: astrum.gg📧 Email: @.***
🐦 Twitter: twitter.com/Beachwayyy


Private migration failing from Pantheon 1.1 to Besu 1.4

From RC:

I believe i've identified an edge case related to private state migration, seems if the Besu node can't detect schema version i.e. version = 0. Besu still starts and private database migration won't run
Migration only runs if schema version is 1 and migration flag is set, otherwise node won't start... (PrivateStrorageMigrationService.java). The node I'm testing dates back to pantheon 1.1 and there's no DATABASE_METADATA.json in the database directory.

We need to check what is happening in this case.
Expected behaviour
• If no DATABASE_METADATA.json is present, the private database is assumed to be 1.0. If private transactions are present and the migration flag was set, the migration should happen.
• If no private transactions are present, migration shouldn't happen.

For clarification there is a DATABASE_METADATA.json file in the data directory I was looking in data/database.
However it looks like in Besu 1.4 private tx related items are being written to data/database whereas in Besu 1.3.x looks like it wrote to data/database/private.
The migration service sees the database as empty - so no migration occurs all I see in the PRIVATE_TRANSACTION segment is entries suffixed PGHEADMAP - which I believe are new in 1.4
The Besu 1.3.x PRIVATE_TRANSACTION segment in data/database/private contain keys suffixed LOGS and OUTPUT

In 1.3.x RocksDBPlugin.java createAndRegister
factory = new RocksDBKeyValueStorageFactory(configuration, segments);
privacyFactory = new RocksDBKeyValuePrivacyStorageFactory(configuration, segments)
separate storage path - factory = data/database, privateFactory = data/database/privacy
In Besu 1.4.x
factory =
new RocksDBKeyValueStorageFactory(
configuration, segments, RocksDBMetricsFactory.PUBLIC_ROCKS_DB_METRICS);
privacyFactory = new RocksDBKeyValuePrivacyStorageFactory(factory);
privacyFactory inherits the configuration from factory so same storage path = /data/database
Unless i'm missing something Besu 1.4.x won't see content in old privacy storage, just looking in new location...

We are investigating this and will see what we can do. In the meantime, you have the option of re-syncing your node. This should get you back up and running with the database populated correctly.

And thanks for all the details reporting this problem. It helps us a lot!

@magooster Could you confirm the Orion version that you are using in your setup?

Was originally using Besu 1.3.4/Orion 1.4.0 have switched to Besu 1.4.0/Orion 1.5.0

@magooster Sorry, I meant the version that you were use of before trying the migration.

Orion 1.4.0

I just upgraded Besu to 1.4.0 from Besu 1.3.4, did not change Orion.

Ok, so just to confirm:

You have this node since Pantheon 1.1.x, using private transactions
Your last update before Besu 1.4.0 was Besu 1.3.4

Correct?

Private transactions from Besu 1.3.1, don't recall running privacy with earlier versions. NB: This is just a test network I have running...

Lucas to test rolling back and resyncing.

We have identified the issue. Between version 1.3.4 and 1.3.5 there was a change in the way that the private database metadata was maintained and, unfortunately, users migrating from 1.3.4. to 1.3.5 with existing private transactions in their chain will lose the transaction execution data for transactions prior to 1.3.5.
At the moment, the best way to recover from this situation is to start a fresh node running +1.3.5 (preferably 1.4.0+) using the same Orion keys as the <= 1.3.4 node and resync the chain.
Since the PMTs are stored in the public chain, and all private transaction payloads are in Orion, resyncing the node will ensure that the private state is rebuilt. In the end, all private transaction execution information will be available again.
Let us know if you have any questions.

Thanks for the update - I assume this will be added to the migration notes.

I assume this will be added to the migration notes.

Sure! We are preparing docs and comms to help others that might face this issue.

We've added a page to the Wiki describing this issue and how to recover - https://wiki.hyperledger.org/display/BESU/Critical+Issue+for+Privacy+Users
The migration notes have also been updated.
Thanks for raising this issue and taking the time to provide additional information.  Have you got any further questions/comments on this before we close this?

All good thanks


gcc13.2 compile fail when using std::string to compare the json object

Description
Kindly refer to this link: https://godbolt.org/z/c3fzjnqzG,
okay with gcc 12.3
failed with gcc 13.1 and 13.2
Reproduction steps
refer to description
Expected vs. actual results
expect using gcc13.2 can be compiled successfully
    if (std::string("bar") == j["xxx"])
    {

    }

Minimal code example
#include <iostream>
#include <string>
#include <nlohmann/json.hpp>

int main()
{
    nlohmann::json j;

    if (std::string("bar") == j["xxx"])
    {

    }

    return 0;
}

### Error messages

_No response_

### Compiler and operating system

gcc12.3 and gcc13.1

### Library version

3.11.1

### Validation

- [ ] The bug also occurs if the latest version from the [`develop`](https://github.com/nlohmann/json/tree/develop) branch is used.
- [ ] I can successfully [compile and run the unit tests](https://github.com/nlohmann/json#execute-unit-tests).

This only happens with C++20 mode enabled, and appears to be a result of the updated C++ compare code.  It works in one direction but not the other.
This fails
if (std::string("bar") == j["xxx"])

This is good
if (j["xxx"] == std::string("bar"))

I can confirm I get the same behavior running on ubuntu 23.04 with gcc 13.2.
It looks like the issue is due to the c++20 template requirement for the operator==
template<typename ScalarType>
requires std::is_scalar_v<ScalarType>
bool operator==(ScalarType rhs) const noexcept
{
    return *this == basic_json(rhs);
}

If the 'requires' line is commented out it appears to use this version of the == operator for this case.
I believe the reason why it works when lhs and rhs are swapped is because of the operator value_t() definition which lets it work with the std::string == operator.
I'm not sure why this function isn't just setup to work with all compatible types which would work with the basic_json(rhs) conversion?

I did not write this code myself, and did not find the time to read into these operators. Help is more than welcome here!

Without having worked on the library details I cannot give good rationales, I decided to try out changing the constraint so operator== is called for the compatible types, like the equivalent catch - all constructor.
@@ -3694,9 +3694,11 @@ class basic_json // NOLINT(cppcoreguidelines-special-member-functions,hicpp-spec

     /// @brief comparison: equal
     /// @sa https://json.nlohmann.me/api/basic_json/operator_eq/
-    template<typename ScalarType>
-    requires std::is_scalar_v<ScalarType>
-    bool operator==(ScalarType rhs) const noexcept
+    template < typename CompatibleType,
+               typename U = detail::uncvref_t<CompatibleType>,
+               detail::enable_if_t <
+                   !detail::is_basic_json<U>::value && detail::is_compatible_type<basic_json_t, U>::value, int > = 0 >
+    bool operator==(CompatibleType rhs) const noexcept
     {
         return *this == basic_json(rhs);
     }

The minimal example compiles (with GCC 13.2.1 on Arch) and the tests pass on develop.
There might be some details that I've missing here, like unforeseen effects of this, so any input is appreciated. If this is okay I'll gladly make a PR for it.

thats becauseIn GCC 13.1 and 13.2, stricter checks might be in place that catch potential issues earlier. It could be related to the handling of non-existing keys in a nlohmann::json object. Accessing a non-existing key with the operator[] in nlohmann::json returns a nullptr, which may not have a direct comparison overload with std::string in later versions of the library or compiler, I ALSO HAV THE CODE REFER TO THIS
if u using .find(),this error wont occur in here
auto it = j.find("xxx");
if (it != j.end() && *it == std::string("bar")) {
}
if (j["xxx"].is_string() && std::string("bar") == j["xxx"].getstd::string()) {
}

I can confirm I get the same behavior running on ubuntu 23.04 with gcc 13.2.
It looks like the reason why this only appears for c++20 is the template requirement for the operator==
template<typename ScalarType>
requires std::is_scalar_v<ScalarType>
bool operator==(ScalarType rhs) const noexcept
{
    return *this == basic_json(rhs);
}

If the 'requires' line is commented out it appears to use this version of the == operator for this case.
I believe the reason why it works when lhs and rhs are swapped is because of the operator value_t() definition which lets it work with the std::string == operator.
I'm not sure why this function isn't just setup to work with all compatible types which would work with the basic_json(rhs) conversion?

When you write j["xxx"] == std::string("bar"), the operator== from the nlohmann::json library gets invoked, but due to the constrained template, it doesn't find a suitable match for a non-scalar right-hand side. This is different from std::string("bar") == j["xxx"], where the operator== for std::string comes into play, and the nlohmann::json object gets implicitly converted to std::string (if it can be), making the comparison possible.


Tweak formatting

I fixed a typo and removed the 🧠 from the title (looks better without it).

Thanks @cbrnr!
Would you be able to re-run the script (with the MNE sources placed in ../mne-python) for mne:main and add the updated stubs here?

Done. When building the stubs, I noticed that the following packages were still missing (even though I did pip install -e ".[dev]"):

pytest
eeglabio
edflib-python
pybv

It seems like these are not part of mne["full"].
Also, I wonder why edflib-python is still here. On main, I thought we'd replaced it completely with edfio...

Also, I wonder why edflib-python is still here. On main, I thought we'd replaced it completely with edfio...

The tool that adds the parameter default values imports mne; so I believe that if you still have an older version of MNE installed, it will use that one.
So what you should do before running this script is installing mne:main:
pip install -U ../mne-python
pip install ".[dev]"
python ./gen_type_stubs.py

I noticed that the following packages were still missing
It also seems we need to add mne[full,dev] to the dev dependencies in pyproject.toml here!

Thanks!
Regarding MNE versions, would it not make sense to generate stubs for the latest release (instead of main)?

Please feel free to take a stab at this

If you pull in main, the issues discussed above should be resolved


Add platform_type test for one of the 3 options that NCEI accepts

@benjwadams please review

Addresses https://github.com/ioos/cc-plugin-glider/issues/17

Merged, but consider using compliance_checker.base's check_has to simplify/refactor the code in future commits.


Patched #235

Fixes #235

Coverage increased (+0.06%) to 72.753% when pulling 04f77dfda08a49b96e74b59dc3496d3a5e02740e on feature/fix-downloads into 5659b3f3e726fb373b4e3de68b2d65da965ea785 on master.


please check preset "Unmarked Crossing"

It will currently used for tagging combination:
highway=crossing + crossing=traffic_signals
and also for
highway=crossing + crossing=unmarked
edit: talking about nodes...

#6004


Support All Aggregate Data

Thanks! Will take a look

Getting some exceptions with some of the data types
java.lang.IllegalAccessException: Class java.lang.Class<kotlin.reflect.jvm.internal.KClassImpl$Data$objectInstance$2> cannot access private final static  field androidx.health.connect.client.records.BasalBodyTemperatureRecord$Companion androidx.health.connect.client.records.BasalBodyTemperatureRecord.Companion of class java.lang.Class<androidx.health.connect.client.records.BasalBodyTemperatureRecord>

and
java.lang.IllegalArgumentException: Unsupported aggregation type BloodPressure_diastolic_avg

androidx/health/connect/client/impl/platform/records/AggregationMappings.kt defines which data types are available for aggregation retrievals, might have to change the reflection approach to each individual field that supports aggregation instead


Invalid multibyte sequence error on make l10n

A bit minor note: I did try switching to having Thai text as default based on this PR, and the issue still persists when using make l10n as you can see below:
holidays/locale/pot/TH.pot:53: warning: The following msgid contains non-ASCII characters.
                                        This will cause problems to translators who use a character encoding
                                        different from yours. Consider using a pure ASCII msgid instead.
                                        พระราชพิธีฉลองสิริราชสมบัติครบ 60 ปี พ.ศ. 2549
holidays/locale/pot/TH.pot:53: invalid multibyte sequence
holidays/locale/pot/TH.pot:53: invalid multibyte sequence
holidays/locale/pot/TH.pot:53: invalid multibyte sequence
holidays/locale/pot/TH.pot:53: invalid multibyte sequence
holidays/locale/pot/TH.pot:53: invalid multibyte sequence
holidays/locale/pot/TH.pot:53: invalid multibyte sequence

With this in mind, I'm considering continuing my approach on #946 instead, but will switch around the test cases to generate our desired result of having th as default and en_US as supported.
Originally posted by @PPsyrius in https://github.com/dr-prodigy/python-holidays/issues/955#issuecomment-1434250465

Hey @PPsyrius,
could you share your code - I'll try to reproduce the error and fix it.

@arkid15r Sure, here's the version of the code I used https://github.com/PPsyrius/python-holidays/tree/955_thailand_i18n_support

@arkid15r Found solution to this now, turns out on top of adding blank files, headers are also needed so that the reader would accept UTF-8 charset:
Here's one of the examples that finally get the script working for me:
TH.po

# Thailand holidays English translation.
# Authors: PPsyrius<EMAIL_ADDRESS>(c) 2023.
#
msgid ""
msgstr ""
"Project-Id-Version: Python Holidays 0.20\n"
"POT-Creation-Date: 2023-02-19 14:03+0700\n"
"PO-Revision-Date: \n"
"Last-Translator: PPsyrius<EMAIL_ADDRESS>"Language-Team: Python Holidays Translation Team\n"
"Language: en_US\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"

This issue can be closed down, IMO.

Hey @PPsyrius,
it's great you figured that out without my help!
Just FYI the misformatted .po file hadn't been included into https://github.com/PPsyrius/python-holidays/tree/955_thailand_i18n_support when I looked into it.
Anyway, it seems the error had to do with this specific header "Content-Type: text/plain; charset=UTF-8\n" absence.
Closing this out.


next/release

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

v2.3.0
Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


water sensors NBS FL

talked with David at FL today, they installed their sensors about a week ago, when they installed the nodes for rep one were backwards (cover into bare and bare into cover). they will switch them next week once rain has stopped. additionally, I asked him to download data from all node since one of them is not sending to the gateway.
the gateway at that farm is not recognizing the SD card, so it's sending data every 15 min, they will check on that too.
when they do the switch, they will notify me so that I can write down in here that date for us to update the data showing. (from day of install to date of transfer, data showing for bare rep one is supposed to be for cover rep one and vice versa).

Site is online.


Allow more actions in automates

Automates only allow to add tags to documents, and move them in a folder. It'd be great to have more actions possible. A few I could get use of are :

Rename document (it would require some kind of templating engine, and probably group captures when match is set to regex ?)
Add metadata
Set permissions on the doc
Send an email notification to someone

this more resembles full fledged workflows which will be added later.


Table expand row

On a table when a row is expanded and after the row's content is changed, the row is refresh and the row collapse. Would be nice if it doesn't collapse, if it save the state or to have a method where you can expand a row without clicking.

I implement a lighter table with element-ui's css with expand row.
If you need dynamic content in expanded rows, it may be better to use a lighter implementation.

HI @reverland
can you elaborate on what you mean by use lighter implementation?
I need to call an ajax request on row expand and show result on the expanded row.
Thanks

@rodwin I mean, write simple collapse tables by yourself...just use element-ui's css

The row was re-render, not the expanded row.
so just set row-key attribute for “in-place patch” (see https://vuejs.org/v2/guide/list.html#Maintaining-State for detail)

it doesn't work for me.


Brace completion doesn't take existing end brace into account

It always injects the end brace even if one already exists:
https://github.com/dotnet/vscode-csharp/assets/95136/d42481f1-6819-452d-acbf-8efc17045717

Unfortunately since extensions do not run inside the VSCode rendering process, VSCode does not allow us as much control over brace completion as VS does.  The existing support for customizing brace completion in VSCode is called language-configuration.json, but I don't think it allows us to fix this issue easily.
In this particular instance, while we can adjust the autoCloseBefore characters (see link above) to remove ) as a character to close before, that would result in an equally poor experience in a different area.  Consider the following example where the cursor is represented by $$
Console.WriteLine(GetString$$)
static string GetString()
{
    return "Hello";
}

Without ) as an autoCloseBefore character, typing ( would result in Console.WriteLine(GetString(), instead of Console.WriteLine(GetString()) and putting the cursor inside the nested parens.
I was also thinking if VSCode could have a conditional auto close before ) that only triggers if the close paren has a matching open paren.  But that also wouldn't solve the case in your video because ) is matched.
May need to discuss with the VSCode team to see if there is something we can do here.

May need to discuss with the VSCode team to see if there is something we can do here.

Can you open an issue on the vscode repository? This behavior is quite difficult to deal with.

Yes, I plan to do that, but I need to collect a few more details and minimal repro steps first.  I'll update this issue once I've done that.


Aspect ratio (mosaic) layout

Discussed in https://github.com/deckerst/aves/discussions/58

Originally posted by quantified1995 August 31, 2021
One of the best but rarely seen features of gallery apps is the ability to display photos in their original aspect ratio or what some call Mosaic layout. Some apps that employ this are Google photos, Samsung gallery, and recently the gallery x app on the Play store (no tagging or metadata system, definitely not feature rich). Would it be possible to allow photos to be shown this way? Would love to see it!

Example:

A really greatest feature <3


Metrics and reports

We need real time metrics on the console and logs that can be parsed to create a beautiful HTML JavaScript report including request and response time distribution, percentiles, number of errors and HTTP successes post run. If we can get real time graphs in the console that would be great and developers would love that.
Clients can use Influx and Grafana or something for real time visualization by piping the logs to the datastore. We document how to do this but it is out of our scope.

In fact I am not sure about console graphs. Tables with HTTP response codes and percentiles would be nice. I'd purposely leave out mean as load tests don't follow a normal distribution curve.

Are you familiar with the multiple reports available in Vegeta? I agree a single interactive web page with multiple ways to slice and dice the data, which updates in real-time, would be killer.

Have a look at the  Gating highcharts, they are beautiful and what stakeholders want to see.


WIP: Proof of concept—adding cloud auth via session_id

Putting up this WIP PR for visibility—it basically flushes out a proof of concept for cloud auth via the Authorization: Bearer {session_id} request header. Everything is hard coded for now, so still needs some work. All tests pass via npm run test:node against the Textile daemon running locally.
cc: @andrewxhill

closing in favor of #114 cc: @andrewxhill


[WIP] run rewriter in autogen

…en for it

Motivation

Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_KZ6qeUKwOnltBQ
→ https://go/builds/bui_KZ6qggyYZ3wO4l


Dropdown Animation Speed

Hey there - I'm looking for a way to speed up the dropdown animation and can't seem to find one. Can you please confirm there is no way to do this, and if there is, please let me know how I might do it? Thanks!

@AhmedLSayed9 Thank you!


🔥 Remove moj_slack.py

👀 Purpose

To remove the deprecated lib directory, in relation to https://github.com/ministryofjustice/operations-engineering/issues/3952

♻ What's changed

Removed lib/moj_slack
Removed bin/get_slack_support_messages - this was the only dependency but is not currently being run by anything
Removed tests

📝 Notes

There was no job using the upstream dependency, if we're not using it - we should get rid of it. We can always recover it from the history if we end up doing something similar in the future

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

Comparison is base (51a176e) 98.58% compared to head (0aae4b6) 98.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3976      +/-   ##
==========================================
+ Coverage   98.58%   98.66%   +0.07%
==========================================
  Files          63       59       -4
  Lines        5602     5483     -119
==========================================
- Hits         5523     5410     -113
+ Misses         79       73       -6

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


Default pin factory

Should we change the default pin factory from RPi.GPIO to pigpio in the upcoming v1.4 release?

https://github.com/RPi-Distro/python-gpiozero/issues/248#issuecomment-205959582
https://github.com/RPi-Distro/python-gpiozero/issues/248#issuecomment-219285355
And IMHO we'd need much more comprehensive unit-tests before switching the default Pin backend.

Indeed - also, now that backend can be switched with nothing more than an environment variable (which people could set in .bashrc if they want to use a particular backend all the time), there's less reason to move the default until we've got comprehensive tests.

now that backend can be switched with nothing more than an environment variable

Unfortunately that's not been documented yet :-/

Doh - definitely ought to have that in there somewhere ... maybe the pins chapter header. Maybe link to that from the Servo docs.

Probably not the best idea as it requires pigpiod to be running.


fixed usage of CancellationToken.Register

This PR adds proper cleanup of CancellationTokenRegistations by wrapping CancellationToken.Register() into a using.
Because of the missing deregistration, you could end up with some memory leaks because of the captured closure in all CancellationToken.Register() and/or execute the registered action multiple times, if the user, passes the same CancellationToken.
see: MSDN: How to: Register Callbacks for Cancellation Requests
I removed the nullability of CancellationToken because it adds unnecessary complexity.
General Feedback:
The idention style makes it hard to contribute, because there are different indentions (tabs and spaces) used across the code (even mixed up in the same file (e.g. ITextToSpeech.cs)). It would be nice, if you run a "apply codestyle" command on all files. It would remove trailing spaces as well. This will lead to cleaner diffs in pull requests.
See the difference:

:) I now have an editor config, so it should fix up a few things for style. I use VS defaults. I think swapping between VS/VSMac sometimes is the issue.


UWP: Process initialization failed with status 0xc0000139

From @CIPop on January 31, 2018 2:1
A very simple UWP application using SDK 16299 will fail if compiled (AOT) with the following error when the binary is loaded:
"Process initialization failed with status 0xc0000139"
Activating "https://docs.microsoft.com/en-us/windows-hardware/drivers/debugger/show-loader-snaps" then running the app in a debugger shows the following issue:
1725 3de8:3df4 @ 00384546 - LdrpProcessWork - ERROR: Unable to load DLL: "NTASN1.dll", Parent Module: "C:\WINDOWS\SYSTEM32\ncrypt.dll", Status: 0xc0000139
Copied from original issue: AArnott/PCLCrypto#137

This looks like a Windows OS bug to me. ncrypt.dll can't load another DLL it depends on sounds like a UWP sandboxing problem, perhaps. Not sure what PCLCrypto can do about it.

We may be able to workaround this by specifying explicit entry points for A/W methods such as FindFirstFileEx and FormatMessage.


Add support for isolated namespaces in #model_name

When the class is in an isolated namespace, the namespace has to be
defined to generate the correct route key in ActiveModel::Name.

Oops, I didn't see a PR was already open for this issue.

:grin:


[FEAT]: Disable workspace chat history (chat recording)

What would you like to see?
As an administrator, I would love to be able to fully disable the workspace chat:

I would like to be able to tell my users that their questions are anonymous and that they don't have to feel monitored.
This may not make much sense if the questions are sent to third-party APIs, but it does make sense in a local LLM scenario.

We can simply hide this UI, but we cannot disable chat history record keeping because otherwise the threads and historical content of users chats are lost and basically each message would be standalone or ephemeral

Hiding this UI would help. However to make this one trustable, re-enabling this UI at any point in time later on should not be possible for the given workspace.

We can make the option immutable - however, it would still have to come down to the user "trusting" that is enabled. Maybe there is a hint or detail we can add to multi-user chats they can discover to see if chats are visible/invisible to admins


Nuget Shield displays soft-deleted version

On NuGet you can mark specific Versions of your Package as "not listed" (aka soft-delete).
The NuGet shield displays the latest NuGet package, weather it is marked as listed or not on NuGet.
Since not listed NuGet packages are not visible to the public anymore the shield should display the latest package that is marked to be listed.
BR Matthias

Hi, thanks for reporting this.
Would you like to look into it and open a pull request?

@Daniel15 Do you know if your PR #945 fixes this?

It's possible! NuGet is a little bit inconsistent and some endpoints continue returning unlisted packages, whereas others don't. @matthiaslischka Do you have an example of a package that has an unlisted version?

Seems to be fixed. Tested with https://www.nuget.org/packages/CatlightApi/
Latest not listened version is <IP_ADDRESS>18
Actual listed version is 1.0.1
Works fine now. Think this issue can be closed.

Thanks. Since #945 isn't deployed yet, we should test on master too.

Still working post-deploy. Sweet!


feat: refactor ProjectGraph to have similar package inferrence like in EmberAppProjectGraph

What
ProjectGraph.discoverByEntrypoint() needs to be improved to infer the closest package like in EmberAppProjectGraph solution shown in #838

Currently is uses the root package.
What it should do is find up for the nearest package.

Why?

Fixes: #825 for plain js workspaces
Potential to help improve package meta data that can improve graph file resolution.

When?
Our internal projects would not benefit from this feature, this is why it should be a backlog item.

@wondersloth @chadhietala is this related to #964

Yes. Just a non-ember version. That entry point work should be refactored to the base class.


The site for searching logos is not found

languages.abranhe.com
It will be more helpful if you can maintain that site as an automated GitHub page. Then you will not need to worry about it at all. Just a suggestion. :wink:

Thanks, it was hosted on Netlify and I moved the domain to Vercel, I'll take care of fixing it. By the way the repo for the site is https://github.com/abranhe/programming-languages-logos-site

:+1:

This was a one-minute fix, just changing the DNS records and I totally forgot it, but it has been fixed now, therefore I am closing this issue.


Migrate Camera to a move based API

Thank you for your contribution!
We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:
Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[x] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[x] My changes were added to the CHANGELOG.md in the proper section.
[x] I have added necessary changes to user code to the Migration Guide.
[x] My changes are in accordance to the esp-rs API guidelines

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description
Similar to #2191, this PR migrates the Camera driver to move based API for the same reasons. Camera now accepts a DmaRxBuffer rather than a plain slice, providing flexibility with DMA.
This PR depends on #2213 and #2191 (I need to refactor it to expose the new View type).
This PR introduces a new DmaRxBuffer implementation called DmaRxStreamBuf. It's effectively the "circular transfer" version of DmaRxBuf but with an important difference, it doesn't create a circle of descriptors, which avoids the problem in #2021. It also provides a zero copy API and exposes EOFs (See #2033). There's more info on how it works in the doc of the struct.
To support this new DmaRxBuffer implementation, I've added an associated type View to DmaRxBuffer (and DmaTxBuffer) to allow limited but safe access to the buffer during the DMA transfer. This is what lets users read from the buffer during the transfer.
Now the choice of "one shot" or "circular" (or any other transfer type) is based on the dma buffer implementation rather than the driver, effectively achieving half of https://github.com/esp-rs/esp-hal/discussions/2035 .
I haven't provided a with_buffer/SpiDmaBus equivalent here because most people using this driver would typically just start one transfer and endlessly read from it for the rest of their application.
Users can easily create a wrapper if they have a more specific use case.
Testing
Ran the modified example on my devkit and it's even more reliable than before.
cc MabezDev for running the example on your ESP32-S3-EYE. (I don't think anyone else on the team has a ESP32-S3 + OV2640 devkit).

Let's start moving these buffer types out to submodules, the main dma module is big enough without these as it is.

I'll do this one in a separate PR, I'm scared of merge conflicts from doing it here.


ignore go.work files

for more detail please read https://go.dev/blog/get-familiar-with-workspaces

:boom: Acceptance test cs3ApiTests-ocis failed. Further test are cancelled...


Bug when using FileUpload twice inside a relation controller

Describe the bug
When using FileUpload field inside a relation controller, files are correctly saved but both fields display the same file/image. This is not happening outside the relation controller
To reproduce
Steps to reproduce the behavior:

Create two related models
Create both the resources and and relation manager
Insert two FileUpload fields in the form and fill them with two different files
In the relation manager, you'll see only the latter's preview. In the resource form, previews are displayed flawlessly.

Expected behavior
I expect two different previews, one for each FileUpload
Screenshots

Context

Package version: [1.9]
Livewire version: [e.g. 2.0.0]
Laravel version: [e.g. 8.37.0]
Browser: [e.g. Brave, Chrome, Firefox Dev, Safari]
Server OS: [Osx Valet 2.13.19]
PHP version: [7.4]
Database: [MySQL 5.7]

Confirm this is still presente in 1.9.6

Unfortunately, this is a Livewire bug (livewire/livewire#2583) which we must wait to be fixed. Apologies.

Hi, still no update on the Livewire bug. I'm going to think about this problem and implement an alternative solution in v2 that avoids the bug. Sorry about that!

Thanks for the update @danharrin! If you have some idea of the right direction I'll be glad to help!


[NFC] Add SILBuilderContext.

In an upcoming bug fix, I want to pass SILBuilderContext to a
utility. I could continue reusing SILBuilder, even though the utility
must set its own insertion point and debug location. However, this is
terrible practice that I don't want to perpetuate.
The lifetime of a SILBuilder should correspond to a single insertion
point and debug location. That's the only sane way to preserve debug
information in SIL passes.
There are various pieces of contextual state that we've been adding to
the SILBuilder. Those have made it impossible to use SILBuilder
correctly. I'm pulling the context out, so clients can begin using
better practices. In the future, we can make SILBuilderContext
polymorphic, so passes can extend it easily with arbitrary
callbacks. We can also make it self-contained so we don't need to pass
around pointers to an InsertedInst list anymore.

@swift-ci smoke test.

@swift-ci smoke test.


Bug: Workspaces does not work when ?redirectedFrom=%2F is in the URL

I tested this on beta and was able to reproduce it to happen after login redirects to the feed. TBH, I am not confident if the param is the issue or something else, but it is reproducible on beta and deploy previews.

Confirming that this works for me now.


Some users cannot resolve api.faforever.com

Describe the bug
From the forum:
updated to 1.6 today, after having not played for a good while, however when i try to log in i get the following message
java.util.concurrent.CompletionException: org.springframework.web.reactive.function.client.WebClientRequestException: failed to resolve ' api.faforever.com ' after 2 queries ; nested exception is java.net.UnknownHostException: failed to resolve ' api.faforever.com ' after 2 queries
To Reproduce
Some minority of users cannot resolve api.faforever.com using the webclient from spring
Expected behavior
Users should be able to easily connect to the api like they have with client 1.5.0
Wanna have the bug fixed quickly?
Visit Issue hunt...

Failing to resolve api.faforever.com is a system error and can basically only happen if they are offline or have a bad DNS configured (given that FAF or their DNS provider didn't mess things up). Either way, there is nothing the client can do other than maybe displaying a human-readable message that says "You're either offline or someone messed something up and you can't do anything about it" 😄

Yes I suspect that issue is either potentially a bug in how the NettyDnsResolver is resolving the name or the timeout value changed. As there were no users that had this issue in 1.5.0 when we used the restTemplate.
With the restTemplate we also used the apache http client which has its own resolver. So another soultion may be to use their async http client with the webclient. For now I tried just increasing the default timeout as that was the most obviouserror in the stack traces I had seen.
But this is definitely caused by either a change in a library or our code as it was not an issue previously.

Please set the milestone so that release notes are properly generated

Yes I normally go through and milestone everything before release


Soni Phonetic layout

changes for Santali is here again.

you need to update ime submodule which has soni layout

I did the changes.


sha256 dependency is deprecated

According to the repository for sha256, it is deprecated and suggested alternatives are sha.js or hash.js.

Thanks for this info. I will switch the dependency out with the next minor version. If you want, you can provide a pull request for this.

It looks like Webjars has already pulled down the sha256 jar.  I will look at make the change over the weekend to use sha.js or hash.js

Any update on this? Thanks!

The next version of this lib which will land in a few days will use another lib for this.


[Documentation] Expand documentation on Single region deployment

Single Region deployment currently does not provide description or definition about each of the component being deployed part of the automation.
Kindly assist to add the details.

@Mahesh-MSFT Looking for your help in this regard

this is tracked via this pr.


Fix non-overlay build

This PR has a single change which adds an if-def to app/app.c so builds with ENABLE_OVERLAY=0 don't blank-boot the radio after flash.

oh, sorry about that. I've edited the error, will be there when I next upload.
Thank you

Many thanks. I build with -flto and disabling the overlay is needed, that's how I discovered the bug.


additional dependencies needed for a clean setup of workbenches distros

Yikes.. that wasn't the cleanest of merges... must check my "process" for the next one!!


display wordpress-hosted images in modal

Similar to displaying wordpress-hosted images in the scene, please enable the same functionality in the modal window.

same idea as previous implementation, added the same validation to render_modal function


Missing dependencies: transform-es2015-modules-commonjs, transform-object-rest-spread.

When trying to run the start script, I get the following errors:
ReferenceError: Unknown plugin "transform-es2015-modules-commonjs" specified in <%app.home%>/.babelrc ....
ReferenceError: Unknown plugin "transform-object-rest-spread" specified in <%app.home%>/.babelrc...

I think someone forgot to include them in package.json.

Will be fixed with #65 (sort of related)


Mono/C# editor/editor_settings.cpp:151 - EditorSettings::_get - Property not found:

Godot version:
Godot_v3.2.1-stable_mono_win64
OS/device including version:
windows 10 geforce 920
intel core i7 5500U
msBuild
Issue description:
Create new project:
editor/editor_settings.cpp:151 - EditorSettings::_get - Property not found: mono/editor/editor_path_optional
Run:
editor/editor_node.cpp:5268 - An EditorPlugin build callback failed.
Steps to reproduce:
1.Create godot project on Godot_v3.2.1-stable_mono_win64.(Project created in earlier version is working)
editor/editor_settings.cpp:151 - EditorSettings::_get - Property not found: mono/editor/editor_path_optional
2.Create node2d
3.Add c# script to node2d.
4.Run project
editor/editor_node.cpp:5268 - An EditorPlugin build callback failed.
Minimal reproduction project:
Project that not works: Rocket
Rocket.zip
Project that works:
c#.zip

Duplicate of #36995. The "property not found" warning was fixed by #37210 (which will be available in Godot 3.2.2).


EvictionVeto: rename and complete documentation

The EvictionVeto.vetoes names are misleading in the sense that we will only take the result as a hint for eviction, nothing stronger.
Also, the current documentation does not indicate that the state is resolved when the mapping is written to the cache, not when it is a potential eviction candidate.
So this issue leads to:

A potential rename of the type and the method
Improved documentation that clarifies implementation should not have any time related factors in the decision that are expected to be evaluated during eviction itself.

wiki doc that needs to be updated too: https://github.com/ehcache/ehcache3/wiki/module.xa


worker name details issue

Hi there,
awesome work, congrats!
Installed with requirements in Python3.8
This is the issue:

Cheers

Should be fixed in v1.0.3

Thank you, it looks ok.
However, now I have a different issue, none of the jobs are showing up, except if I select "failed" jobs. Even with the "failed" checked, the jobs are showing incorrect time, it shows:
"created 8 hours ago". This is not the correct time, not even hours.
Using:
pip list |grep rq
rq                     1.9.0
rqmonitor              1.0.4
Do not want to take much if your time though. Only if you would like me to post the issues.
Cheers

Oh, I think it is not respecting the Time Zone, so now it is 3.13pm, but the queue "birth date" is 13-07-2021 07:13:36
From console:
date
Tue 13 Jul 2021 15:20:19 AWST - Australian Western Std Time (+8)
Hope this helps.


feat: create dropdown for docs sidebar

Description
Created a Dropdown for the Docs Sidebar, for all the subcategories of Docs in the sidebar.
Related issue(s)
Resolves #1299

@akshatnema I checked it closely for smaller screens and IMO we should have that cursor on dropdown menu in center as in this screenshot it can be seen it is a little shifted to left

@Mayaleeeee as this PR has some design changes so I suggest you should have also have a look reagarding design perspective

is it ready for final review?

@derberg just one comment hasn't been addressed other then that it is ready for review 🚀

@derberg just onehttps://github.com/asyncapi/website/pull/2328#issuecomment-1826800955 hasn't been addressed other then that it is ready for review 🚀

@sambhavgupta0705 The comment has been resolved in the latest commit.

@derberg you can now review this PR

I only see some header issue on mobile:

yes @derberg that is in the production as well so it is not due to this PR.
The issue regarding it has been already raised here https://github.com/asyncapi/website/issues/2427

I only see some header issue on mobile:

This particular issue is not part of this PR. This issue has been handled in separate PR.

/rtm

@asyncapi/bounty_team

/bounty


Use <button> for controls

Is your feature request related to a problem? Please describe.
<button> behaves a bit differently from , this would supersede https://github.com/Leaflet/Leaflet/issues/7368 and https://github.com/Leaflet/Leaflet/issues/7369 (these describe some of the problems with  controls).
Describe the solution you'd like
Use <button type="button"> over  for controls (and over  for markers?).
Additional context

Use buttons, not links, for controls

Agreed!

Originally posted by @mourner in https://github.com/Leaflet/Leaflet/issues/3210#issuecomment-73291846

we should use the button element here

This would be a breaking change we can make in Leaflet 2.

Originally posted by @jonkoops in https://github.com/Leaflet/Leaflet/issues/7368#issuecomment-976742275

Related PR: https://github.com/Leaflet/Leaflet/pull/6248
Reference: https://www.w3.org/TR/2016/WD-wai-aria-practices-1.1-20160317/examples/button/button.html

This change would require a CSS reset for buttons, something along the lines of:
button.leaflet-control {
  background-color: transparent;
  border: none;
  border-radius: 0;
  color: inherit;
  font: inherit;
  line-height: inherit;
  margin: 0;
  padding: 0;
  overflow: hidden;
  text-align: inherit;
  text-transform: none;
  -webkit-appearance: none;
     -moz-appearance: none;
          appearance: none;
}

(this is just a generic class I've been using before, if Leaflet will use a default border-radius for all controls for example there's probably no reason to reset that.)

I only want to document this: We should not remove the existing css classes because they are used for many custom controls

I've created a label to keep track of breaking changes like these.

Would be nice it those buttons had accessible name.
https://www.drupal.org/project/leaflet/issues/3232474

@Chi-teck The buttons would have accessible names as they have now. This issue is about the elements themselves.

https://www.drupal.org/project/leaflet/issues/3232474

An aria-label to give the popup close button an accessible name was fixed in #7794.


Monitor and maintain user feedback

As one way to measure response accuracy, we should monitor user feedback and aim to maintain current levels of positive feedback. (Using the existing LangSmith integration to do this is acceptable.)
Priority: P0 (must have for Phase 2)

Already being monitored by the AI/ML team after project hand over


Add support for $NEER on Transak

Is your feature request related to a problem? Please describe.
Transak has added support for $NEER. Please integrate $NEER fiat on-ramp on SubWallet to allow users to buy $NEER with fiat currencies.

https://docs.transak.com/

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-pr-1325-build-1.0.2-2023-05-08T07-27-03UTC.zip
Validation passed!


Add ruby language detection

What does this PR do?
Add Ruby language detection. No changelog as this code is still dormant.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Set the following config:
log_level: trace
process_config:
  process_collection:
    enabled: true
  language_detection:
    enabled: true

Run a ruby process (e.g. a long-running ruby foo.rb or an empty rails server) and verify that the process is detected as Ruby by looking for the following log line:
detected language ruby for pid <pid>

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Serverless Integration Tests failures are all of the type
<       "cold_start:false",
---
>       "cold_start:true",

which is ignorable per https://datadoghq.atlassian.net/wiki/spaces/SLS/pages/3104015888/Serverless+integration+tests+known+issues


Provide artifacts via CI/CD instead of dist folder (which should be included in .gitignore)

See lpp-scratch for more information.

Closing. I decided to rollback the framework implementation.


#2145 Add tests video recorder

Proposed changes
Add tests video recorder as standalone solution and plugin for JUnit ant TestNG.
It uses BiDi and ffmpeg.
Checklist

[X ] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command
[ X] I have added tests that prove my fix is effective or that my feature works
[ X] I have added necessary documentation (if appropriate)

Probably, the easiest way to get ffmpeg to work with gh actions is to actually install it and let the operating system resolve the dependencies itself.
- name: Install ffmpeg
  run: sudo apt-get install -y ffmpeg

In fact most of all Linux users will run into this issue. Especially on CI, where Docker images are cut as much as possible.

@BorisOsipov @asolntsev all tests are done. Except ios tests. Could you please take a look. Thanks in advance

@britka Why did you delete .idea/icon.svg?

@asolntsev sorry. My fault. Have already reverted changes

@britka Could you check links in readme file that lead to Selenium, for instance, this one returns 404:
Now the last versions of Selenium have [BiDi functionality](https://www.selenium.dev/documentation/webdriver/bidirectional/).

@britka Could you check links in readme file that lead to Selenium, for instance, this one returns 404:
Now the last versions of Selenium have [BiDi functionality](https://www.selenium.dev/documentation/webdriver/bidirectional/).

Thank you. I'll check

@britka Could you check links in readme file that lead to Selenium, for instance, this one returns 404: Now the last versions of Selenium have [BiDi functionality](https://www.selenium.dev/documentation/webdriver/bidirectional/).

@yevgeniy-mikhailov Fixed

@BorisOsipov @asolntsev rebuild recorder. Delete BiDi implementation and add by using webdriver take screenshot implementation. Updated documentation and tests.

@britka Thank you for the contribution. Video recorder has been released in Selenide 7.6.0.


Oficial docker image?

Is there a plan to have an official docker image for the CLI?

@omerlh this is not an official docker image. But might suit your needs https://github.com/patros/docker-devskim

:+1:
I would like to be able to provide this in our CI pipelines as well as allowing developers to include it in their IDE.

@jhulten what IDE do they use? There is already extensions for Visual Studio and VS Code.
And if you use my docker image you can "break" the build based on the exit code. I'm currently using it across about 200 repos.

We have some in VSCode, VS, Atom, and Sublime with a couple of JetBrains products in the mix.
Your image is great, but it will be easier to get through security if it is mainstreamed.
Good exit codes would also allow pre-commit hooks via https://pre-commit.com/

Oh, that's simple @jhulten. My Dockerfile and DevSkim are MIT licenced. Take a cut of them and build your own internal image. That way you have full control over it which should keep the security team happy.

We are now planning to make an official docker image.  We are currently working on completely rewriting the build pipeline, and this task relies on #86 to complete.

We hit some delays in getting the rest of our CI pipeline running. We've now completed that. This is still coming soon.

We hit some delays in getting the rest of our CI pipeline running. We've now completed that. This is still coming soon.

We are now also distributing devskim as a dotnet global tool. That may help in the meantime with a single line install (as long as you have .net core 3.1 installed)
dotnet tool install --global devskim
That will place the devskim cli on your path.

In case there is anyone following this who uses my docker image. I’ve switched it over to using the official build via dotnet tool install.

In case there is anyone following this who uses my docker image. I’ve switched it over to using the official build via dotnet tool install.

Quick update, the dotnet tool has been moved here:
https://www.nuget.org/packages/Microsoft.CST.DevSkim.CLI

Thanks for the heads up @daalcant


Support for SQL Always encrypted with Data export service

Hi All -
I am planning to use Data Export Service to replicate Dynamics data to Azure SQL database for some offline/reporting purposes. The method is outlined in below article. Following this article i was able to make it work without any issue.
Few of my entities has PII data and as per my companies security guide lines i have to make sure data is encrypted at rest and transit along with always encrypted.
I know encryption at rest (TDE) is default feature of Azure SQL however i still have below two question and this document does not mention anything about this -
Can i enable always encryption (column level encryption) and still do data replication
Can i change my connection string to include Encrypt=True and TrustServerCertificate=False in connection string in key vault and it will do encryption in transit as it say in MSFT docs below ?
https://docs.microsoft.com/en-us/azure/azure-sql/database/security-overview#transport-layer-security-encryption-in-transit
Any help around this will be very helpful.

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

ID: 3d51ad6f-b7a7-7abd-f1d1-1d7963dd3d49
Version Independent ID: 4612ff88-8457-2f49-8222-4fd110fd72f6
Content: Replicate data to Azure SQL Database using Data Export Service - Power Platform
Content Source: power-platform/admin/replicate-data-microsoft-azure-sql-database.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@rupeshprasad
Sorry for the late response. Please see if you can get your question answered here:
https://powerusers.microsoft.com/t5/Power-Apps-Community/ct-p/PowerApps1


Address compilation and clippy warnings

Use explicit dyn for trait in Differ definition.
Remove unnecessary lifetime in file_byte_iter by using '_
Replace ".expect" with "unwrap_or_else to avoid building the string
in the caller, even when it's not used.

Thanks @lpenz!


useNativeEvents should not be a part of the page object out of the box?

I've just observed the following in the codebase
https://github.com/Addepar/ember-classy-page-object/blob/714cb4523f7cbab01e7539b0ddfe7d81714ba919/addon-test-support/-private/page-object.js#L8-L9
I believe it should not be here, and if really needed, it should be called on the consumer test setup side.
The useNativeEvents( is used to enable the ember-native-dom-helpers, which was needed in the era of moduleFor* tests. Now, for the test suites using modern @ember/test-helpers, I don't think we need it anymore.
Also, useNativeEvents( is now removed in the v2-beta of the ec-page-objects, cause moduleFor test helpers are now removed completelly from @ember/test-helpers, so I think this invocation would break ember-classy-page-object under ember-cli-page-object@v2 when it's released.
So my guess is that we should remove it from here at some point, as a breaking change? @mixonic  what do you think?

Hm, yeah I can try to make the change and see how it impacts the large codebases at Addepar. Good catch, thank you!

Just in case useNativeEvents( accepts a single boolean argument. By default it's true.

I've removed it as part of my testing the v2-beta.   https://github.com/Addepar/ember-classy-page-object/pull/36


Potential Lua bytecode change

Hi there! Super cool project, I hope to make one of my own at some point.
I recently referenced your Lua bytecode work, and noticed that the source scripts had hardcoded function names and argument length. I had to work around these restrictions for a project I'm building, and thought you might be interested in the change for yourself.
Here's the relevant file in my code: https://github.com/strawdynamics/pdportal/blob/main/src/lib/util/luaBytecode.ts. This allows:

Calling any global function with a name <= 40 chars (after this point, something beyond the normal string bytecode change happens, but I'm not sure what)
Passing any length string argument (I've tested up to 100KB in a single call without issue)

Pretty cool! Thanks for the heads up. We don't need it here at the moment but it's really nice to know there's a more flexible solution and it might come in handy in the future.
pdportal is actually something I was thinking of making myself, I'm glad you're taking on the challenge so I don't have to :)

Sounds good, it's out there if you need it!


CirrusSearch output contains recommendations to not existing Wiki pages

Add check via id-title-mapping file

fixed in bdapro branch


error throwing for pip install ai4bharat-transliteration

building 'fairseq.libbleu' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for fairseq
Failed to build fairseq
ERROR: Could not build wheels for fairseq, which is required to install pyproject.toml-based projects
WARNING: Ignoring invalid distribution -orch (c:\users\admin\appdata\local\packages\pythonsoftwarefoundation.python.3.9_qbz5n2kfra8p0\localcache\local-packages\python39\site-packages)

Hi @DhivyaThamu
As you can see in the first line of the error, this is a problem related to FairSeq.
So please raise your issue here: https://github.com/facebookresearch/fairseq/issues/4880

Fairseq does not support windows. Use WSL 2 for windows.


[CI] Use force flag in Cilium install apply command

Unblocks #9519
Fixes: #9552

This change is

test-me-please

Coverage increased (+0.003%) to 45.37% when pulling 56627fbf95aa3b07266e0d5dd9ba98684e8ba244 on pr/ci-force-apply into f1644be68cd35eba6d96843c7e658b5a58112410 on master.

Nice!


Copilot keeps "finding" code that doesn't exist.

Type: Bug
I have cloned a dotnet/runtime repo and opened the src/coreclr directory in VS code.
I've asked Copilot:
@workspace can you please find all lambda expressions usage in source files in the directory "vm" and all of its subdirectories?
It has reported:

Here are the lambda expressions used in the "vm" directory and its subdirectories:
In loaderheap.cpp:
auto AddRangeWorker = [&](void* pStart, void* pEnd) -> bool {
// lambda body
};
In loaderallocator.cpp:
auto lambda = [&](void* pStart, void* pEnd) -> bool {
// lambda body
};
In vm/someotherfile.cpp:
auto lambda = [&](int a, int b) -> int {
// lambda body
};
These are examples of lambda expressions found in the "vm" directory and its subdirectories.

There are no such lambdas in the source files mentioned. And the vm/someotherfile.cpp doesn't even exist.
I've then told it:
The lambdas you have mentioned are not in the source, please include only existing ones.
It came up with another set, again non-existing ones.
Extension version: 0.23.2
VS Code version: Code 1.96.2 (fabdb6a30b49f79a7aba0f2ad9df9b399473380f, 2024-12-19T10:22:47.216Z)
OS version: Windows_NT x64 10.0.26100
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i9-13900K (32 x 2995)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
127.71GB (103.60GB free)

Process Argv
.\Program.cs --crash-reporter-id c93599cb-e413-443a-8b1c-69cfc6e95afc

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythonnoceb:30805159
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
dvdeprecation:31068756
dwnewjupytercf:31046870
nativerepl2:31139839
pythonrstrctxt:31112756
nativeloc2:31192216
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013
stablechunks:31184530
6074i472:31201624

Can you share a project that causes this to happen so I can try to reproduce the issue?

@mjbvz you can just run git clone https://github.com/dotnet/runtime, then open VS code and do "File" - "Open Folder". Select the src/coreclr subfolder from that repo. That's all I did.


update text about AdaptiveVideoTrackSelection as class doesnt exist anymore

in
https://codelabs.developers.google.com/codelabs/exoplayer-intro/#4
this part:
We are now streaming DASH while addressing bandwidth when selecting tracks. Create a breakpoint in the method AdaptiveVideoTrackSelection.updateSelectedTrack and start your app in debug mode. The breakpoint is hit each time a download of the next chunk is initiated and the track selector needs to make a decision for an appropriate video format.
is outdated. the class was renamed to AdaptiveTrackSelection in 2.4

Text has now been updated. And it only took 2 years! 😉 Thanks for filing this issue.


support k8s websockets

Closes: https://github.com/opendatahub-io/odh-dashboard/issues/1138
Description

Adds support for dynamic-plugin-sdk useK8sWatchResource using web sockets.
Note that this PR only adds support for web sockets but does not introduce any usage of web sockets to the application.
Backend:

Bump fastify to latest versions.
Add dependency @fastify/websocket.
Introduce a websocket proxy for route /wss/k8s.
Removed the work around for safeURLPassThrough as the related bug has been fixed in the dynamic-plugin-sdk.
Created a work around for https://issues.redhat.com/browse/RHCLOUD-28058

Return an empty response after successfully deleting a k8s resource.

Frontend:

Bumps dynamic-plugin-sdk and @openshift/dynamic-plugin-sdk-utils to latest versions.
This gives us access to the latest APIs which allow for static models to be used to bypass API discovery.
Isolates the dynamic-plugin-sdk redux store from our own by creating a custom redux context for ODH dashboards.
Add dev webpack proxy for websockets.

How Has This Been Tested?

Ad hoc testing with custom test page.
Smoke test current app.
Test Impact

None.
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

/hold Target branch is TBD.

/hold Target branch is TBD.

I would say that this should go to something like f/performance since that's the main goal of it, let me create a tracker to sum up all the performance work we have at the moment.
@andrewballantyne What do you think?

That works for me -- we should also talk about our merging strategy more -- I demoed a loose idea this morning but the process needs work. Things that are not feature roadmap items can't live in incubation forever. But that feature branch can be done now and we can work to improve the process behind the scenes.

That works for me -- we should also talk about our merging strategy more -- I demoed a loose idea this morning but the process needs work. Things that are not feature roadmap items can't live in incubation forever. But that feature branch can be done now and we can work to improve the process behind the scenes.

Yes, we can have a sprint for stability or something and then merge the feature branch, I'm open to any idea here.

First rundown seems ok, I'm gonna try to get some time today or tomorrow to give a deeper look just in case.


surelog expects +libext+ to be specified only once

I'm doing the zerosoc tutorial and build.py failed because synth didn't find lots of files that should be in ydir libraries
For what I could check it seems that the call to surelog includes the parameter +libext+ twice, once for each extension (.v, .sv), but the proper way to specify both extensions would be with +libext+.sv+.v
https://github.com/siliconcompiler/siliconcompiler/blob/47422a694524f2f7cc84eda56aad6c90d8c1da46/siliconcompiler/tools/surelog/surelog.py#L120-L121
I'm using surelog 1.26

Thanks for the bug report, fixed in the most recent commit!


This application failed to start because it could not find or load the Qt platform plugin "xcb" in "".

We have something that begins to launch now, but:
$ Downloads/Lector-ff03947-x86_64.AppImage
(...)
This application failed to start because it could not find or load the Qt platform plugin "xcb"
in "".

Reinstalling the application may fix this problem.

I've got no idea. CC @TheLastProject do you remember whether we've had a bug like that while bundling Pext?

Maybe Pext/Pext#99?

@TheLastProject that looks good, thanks!
@probonopd please add xorg-libxi to your conda packages.

Thanks @TheLastProject and @TheAssassin. Turns out that it is looking for the Qt plugins in an entirely wrong directory.
( cd squashfs-root/usr/conda/bin/ ; ln -s ../plugins/platforms/ . )  was needed to "fix" this. I don't assume that this is a Lector-specific issue, it's probably more a Qt-in-Conda thing?

The conda Qt packages are broken in many ways. Try to use PyQt5 from PyPI directly.

This is what I was using so far:
export APPNAME=lector
export PIP_REQUIREMENTS="-e git+https://github.com/BasioMeusPuga/Lector#egg=$APPNAME"
export CONDA_CHANNELS="conda-forge"
export CONDA_PACKAGES="pyqt;beautifulsoup4;xorg-libxi"  # Only use this if the package is in a Conda channel (e.g., conda-forge); can also be used for dependencies if the main application has no depends.txt

Much better now @TheAssassin. This should be documented, and possibly a bug be filed against Qt from conda-forge.


PoC: Support async .provides in fusion-test-utils

Partial PoC for https://github.com/fusionjs/fusionjs/pull/443

Closing this as it is not intended to land.  Proof-of-concept may be referenced or used in a future pull request.


Guzzle v3 causing issues with other packages requiring Guzzle v4

I'm trying to use VCR with another package which uses "guzzlehttp/guzzle": "4.2.2" but because VCR is using "guzzle/http": "3.*" I'm getting an error:
To set a CURLOPT_READFUNCTION, CURLOPT_INFILESIZE must be set.

I've spoken to the maintainer of the other package, omniphx/forrest, in some depth about it in issue 17 on his repository and full details of the errors we've managed to reproduce are available there.
Do you have any plans to upgrade to Guzzle 4 or 5, or is there a workaround you could suggest to solve the issue we're having?
Thanks

Just merged the "removal of Guzzle" branch. There are integration tests now with different Guzzle versions and Soap. Please let me know if there are problems.

Hello @adri since we solved the other issue (#63) we are now using PHP-VCR in our project :) I tried then to use it also for calls to the SalesForce API and I'm experiencing similar problems to the ones in this issue. I'm continuing on this issue because I think it's the same problem. If you prefer me to open another one I can do that.
This is what I'm recording now:

-
    request:
        method: POST
        url: 'https://test.salesforce.com/services/Soap/c/29.0/0DF20000000TuVr'
        headers:
            Host: test.salesforce.com
            Content-Type: 'text/xml; charset=utf-8; action=""'
        body: "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<SOAP-ENV:Envelope xmlns:SOAP-ENV=\"http://schemas.xmlsoap.org/soap/envelope/\" xmlns:ns1=\"urn:enterprise.soap.sforce.com\"><SOAP-ENV:Body><ns1:login><ns1:username>YYY</ns1:username><ns1:password>XXX</ns1:password></ns1:login></SOAP-ENV:Body></SOAP-ENV:Envelope>\n"
    response:
        status:
            http_version: '1.1'
            code: '500'
            message: 'Server Error'
        headers:
            Date: 'Tue, 03 Feb 2015 15:04:21 GMT'
            Set-Cookie: 'BrowserId=CZr-GXpiRN2d7TLherhWeQ;Path=/;Domain=.salesforce.com;Expires=Sat, 04-Apr-2015 15:04:21 GMT'
            Expires: 'Thu, 01 Jan 1970 00:00:00 GMT'
            Content-Type: text/xml;charset=UTF-8
            Content-Length: '287'
        body: '<?xml version="1.0" encoding="UTF-8"?><soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/"><soapenv:Body><soapenv:Fault><faultcode>soapenv:Client</faultcode><faultstring>SOAPAction HTTP header missing</faultstring></soapenv:Fault></soapenv:Body></soapenv:Envelope>'

Looks like the problem is related to some missing header. I've tried both with and without the headers matcher, but no joy. Do you have any suggestion? Thanks!

Maybe the authentication header. There has been work on this recently, are you sure you are on the latest vcr version?


Add ShouldNot to codespell ignore list as it's a function name in ginkgo

The code spell is failing due to the function name ShouldNot, used with ginkgo.
So adding it to ignore list.
Signed-off-by: Malay Kumar Parida<EMAIL_ADDRESS>

/retest

/cherry-pick release-4.12


Inline various trivial functions

CC @jf2048
Draft until

[x] https://github.com/gtk-rs/gtk-rs-core/pull/865 is finally merged
[ ] Checked effect on generated assembly
[ ] Checked effect on compile times
[ ] Checked effect on code size

@bilelmoussaoui Changes like in this PR in gtk4-rs are your job :)

Looks good to me but for such changes, having perf comparisons would be really nice. At some point we should really write benchmark to not blindly merge such changes.

Yeah, let's discuss at the next hackfest how we can somewhat automate such things. This was manually checked on various GStreamer plugins and some GTK application code with cargo asm. Performance was approximately the same but the generated assembly is considerably shorter and cleaner, and has fewer function calls.


\notin missing metrics

As mentioned here, the \notin symbol (U+2209, 8713 in decimal) is missing font metrics.  This seems to be an isolated instance, but maybe worth checking for other missing characters.
Once this is fixed, we should change the corresponding KaTeX test to check toBuild instead of toParse.

I should probably check that notin still works after my changes to 0x338.
  notin => [
    'Select(0u2208)','Copy()',
    'Select(0u2209)','Paste()',
    'Select(0u338)','Copy()',
    'Select(0u2209)',
#    'PasteWithOffset(-63,0)',
    'PasteWithOffset(831,0)',
    'RemoveOverlap()',
  ],

Indeed, the glyph is now messed up:  (via https://opentype.js.org/glyph-inspector.html)

The metrics for 8713 are still missing, too.  Perhaps we should remove this glyph altogether, and define a macro \notin -> \not\in (as in LaTeX)?

@edemaine thanks for checking those glyphs.  I'll work on this this weekend.

Here is how \neq and \notin are defined in LaTeX (base/fontmath.ltx):
\def\neq{\not=} \let\ne=\neq
\DeclareRobustCommand
  \notin{\mathrel{\m@th\mathpalette\c@ncel\in}}
\def\c@ncel#1#2{\m@th\ooalign{$\hfil#1\mkern1mu/\hfil$\crcr$#1#2$}}

Based on this, I find the following macro definitions work well in KaTeX:
defineMacro("\\neq", "\\not=");
defineMacro("\\notin", "\\not\\mskip1mu\\in");

But the latter one isn't quite perfect -- slightly off spacing before/after symbol.  So maybe easier to just fix the symbols after all.

Striving for the ability to use macros to define more things I think it is a good direction b/c modifying fonts takes considerably more time/effort.

OK, I'm making a new PR for the macro approach.

I'm going to put this on hold until that gets merged.

The master branch now features bad glyphs for \neq and \notin.  Luckily, they're no longer used by KaTeX.  But we should remove the glyphs.  I'll take a look.


--log_level argument doesn't change logging level

After the logger.py was added to allow easy logging configuration across the two modules, the log no longer follows the --log_level flag value.
Multiple attempts were made at attacking the problem from a different angle, but ultimately the issue persists.

fix will be included in pull request #9

auto close didn't work when I used #4 fixed in the PR description


SASS error interrupts watchFiles function

Not sure what I am missing, I am using the same gulpfile/folder structure and whenever there is a sass error it stops watchFiles function.

I experienced the same issue and solved it by using "on('error')" like the below example:
gulp.task("compileSass", function () {
    gulp.src('app/styles/**/*.scss')
        .pipe(maps.init())
        .pipe(sass({
            outputStyle: 'expanded'
        })
            .on('error', sass.logError))
        .pipe(maps.write("./"))
        .pipe(gulp.dest('app/styles'));
});

i am very new at this so i dont know exactly why it keeps running - but it does :). I am guessing it is because it runs sass.logError upon something going wrong but it does not write anything to the console.
take a look at this stackoverflow question for more info:
http://stackoverflow.com/questions/21080433/gulp-sass-watch-stops-when-invalid-property-name


Autocomplete doesn't work in latest Safari version

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

Go to https://teradata.github.io/covalent/#/components/chips
Start typing where it says autocomplete input is supported
Autocomplete is not showing, it just adds chips

What is the expected behavior?
Show autocomplete
Which version of Angular and Material, and which browser and OS does this issue affect?
Whatever the demo uses, latest safari version and macOS
Did this work in previous versions of Angular / Material?
No idea.

Which version specifically of safari does this happen?

I'm seeing it in Safari 10.0, the autocomplete dropdown doesn't work:

Native HTML5 input list isn't supported by Safari... I'm debating what we want to invest since angular-material2 is building this https://github.com/angular/material2/issues/117

@emoralesb05 10.0.1
@kyleledbetter Do you mean, you're just going to wrap TeraData components [Autocomplete] with Material 2?

true, md-autocomplete is coming so we might replace ours with that one inside md-chips.

@leolorenzoluis the goal of Covalent is to build on top of angular-material2 and add to it, but in a few cases like this we needed a component before they had it so if their version meets all needs we'll retire ours, or if we need extra features we'll wrap it as you suggest

@kyleledbetter That's awesome, I am in the stage of evaluating Teradata to be used as a UI component as Material 2 is lagging behind from 1. How is the support for UI components? Is this free or there will be enterprise grade support in the future?

No plans for enterprise support or commericalism. This is a traditional open source developed and community supported project that's free and licensed under MIT like everything it's built on

@kyleledbetter Just curious, you have more domain knowledge than me, but given that Material 1 has at least complete and stable API/components. Why doesn't Teradata just scrap existing Material 1 and just come up of a simple API such as for Autocomplete?

Hmm, then given Teradata is just building on top of Material 2, why won't Teradata just branch off Material 2 rather than having a separate repo?  All they have to do is branch from it, and more over you get the benefit of likely merging whatever API design you use now that are currently not available in M2?

Not sure what you mean by  just branch off Material 2 rather than having a separate repo < we include angular-material2 as a dependency just like any other node repo, just as we include angular2 itself. We don't want to replace any of those things or be responsible for maintaining that part of the code base. We want to build on TOP of all their hard work and add all the things missing to make a full-featured application.
Let's keep this issue focused on Autocomplete in Safari and head to our Gitter chat if you'd like to have philosophical debates about our architecture decisions :) https://gitter.im/Teradata/covalent


Base implementation for Utorg provider

Added https://utorg.pro provider via direct flow.
We faced several problems:

Top-up page is not designed for multiply on-ramp providers.
Some components (like ActivateAccount) rendering related to MoonPay availability. We didn't touch them.
We don't which ENV variable we should use to show button only for production wallet. Related with: https://github.com/near/near-wallet/issues/2355

@samkrew please open a ticket, so we can discuss design and implementation details outside of the technical discussion here

@stefanopepe https://github.com/near/near-wallet/issues/2365

Also, the moonpay has region based checking and the button is hidden / disabled if the current ip's country is not supported. Can / should this be added? This is the case in the SetupImplicit, BuyNear and CreateImplicitAccount components. cc: @agileurbanite

Not the best solution actually. We let user to choose residence on the first step. For example: EU customer in US.

@corwinharrell i'm hoping we can merge this PR along with the nearpay one as a stop gap to onboard the new fiat providers while other teams / contributors are working on the new ux for fiat onramps.  I don't see this as a concern given that the issue on the new ux should be mitigated fairly soon given the onboarding of new contributors.  Thoughts?

@agileurbanite I guess I wasn't aware that new UX was being designed for fiat onramps. Could you point me towards this so I can check it out?
If we do decide to include multiple onramps for this onboarding step in particular, I think we'll need to makes some additional changes here.
@samkrew In the design above, how can the user know what to expect when clicking the pre-defined dollar amount options? If there are multiple options here, say MoonPay and UTORG, how do you expect them to function?

For onboarding screen let's kill these buttons with dollar amounts as @corwinharrell suggested:

we will revisit the UX for the onboarding flow in a separate issue.

@samkrew do you mind taking care of that in this PR?


align_pseudo_comments - bug?

@larshp why 61 if the answer to all questions is 42? 😄
https://github.com/abaplint/abaplint/blob/aa3cab556dafadbfa0aacbdabf4770ba50ff43da/packages/core/src/rules/align_pseudo_comments.ts#L53
ok, let's be serious for a while. Run this example in the Playground.
REPORT zfoobar.

SELECTION-SCREEN BEGIN OF BLOCK range.
PARAMETERS p_adatfr TYPE dats DEFAULT sy-datum MODIF ID inc. "#EC NEEDED
PARAMETERS p_adatto TYPE dats DEFAULT sy-datum MODIF ID inc. "#EC NEEDED
PARAMETERS p_atimfr TYPE tims MODIF ID inc.                  "#EC NEEDED
PARAMETERS p_atimto TYPE tims DEFAULT sy-timlo MODIF ID inc. "#EC NEEDED
SELECTION-SCREEN END OF BLOCK range.

which version are you running, with/without indentation of selection-screen blocks?

The default one, but I turned off prefer_pragmas. As in original project I have syntax set to v702, so I can't use pragmas. But just disabling prefer_pragmas rule even on v758 gives the same result.

Another example. It's available on Abaplint Playground. Just disable prefer_pragmas for NEEDED
REPORT zfoobar.

CLASS lcl_test DEFINITION.
  METHODS foo
      RETURNING VALUE(result) TYPE zsome_class_name=>constant_structure-constant_value "#EC NEEDED
      RAISING zcx_some_exception.
ENDCLASS.

@larshp ?


Allow non-root containers to be deployed

Fixed: https://github.com/stardog-union/helm-charts/pull/9


Invitations

Backend:
Added an invitation table, with the invitation id (uuid v4), roadmapId and RoleType of the invitation and email of the invited.
Added routes for adding, patching and deleting invitations, as well as using a invitation. On use the invitation is deleted.
Frontend:
Route for joining is /join/:joiningLink. The joining link follows uuid v4 pattern. User needs to be logged in or register in order to join.

on successful join, user is redirected to the roadmap's dashboard page
on unsuccessful join, an error message is displayed - perhaps a better design should be asked for

Frontend can be tested by adding a team member as an admin, logging out and using the created invitation id in the joining route.

Added the fixes
Also came up with an unhandled corner case: what if admin adds multiple invitations in the same roadmap for the same email?

Should the previous invitation be deleted if a new one is added? Or treat it as a invitation role patch...
If multiple can exist, all invitations (with the same roadmap & email) need to be deleted once one is used

Should the previous invitation be deleted if a new one is added? Or treat it as a invitation role patch...

Deleting the previous ones would mean, that those links are then invalidated. I would say, that this is undesirable option.
Patching would be an option. It should also send the link, as that would be the expected outcome from clicking the "send link" button.
If we add expiration (probably will at some point), that should also be updated on patching the invite this way.
Patching would also have the benefit, that there will never be invitations for the same email&roadmap with different roles.
At the moment, there is no ui for updating, revoking, or even seeing, existing invitations. If this is added, the semantics of patching the invitations from there would likely be different than sending a new invite to existing invitee.
I'd say updating existing invite would be a good choice.


Add some test cases to verify that favorite sessions are displayed correctly on the  favorites screen

Currently, we can only check without any favorites. As mentioned in the comments, by injecting test data into UserDataStore, we should be able to test screens with favorite sessions.

👍


PC Gamer shows up

Hey there!
If I open SAM "PC GamerDigital" appears and I could manage its achievements. The "game" or magazine is not installed on my PC and neither in my library, but as soon as I open it in SAM it will be placed in my library.
Is this an issue I need to be worried about? As soon As I close Steam and start it again, the "game" disappears in my library.
It's this PC Gamer: https://www.pcgamer.com/
Cheers!

The functionality you're experiencing is how Steam handles free games/applications. They're only visible in your library when installed.
A more recent change causes games that "run" that use Steamworks (which is how SAM works) to show up briefly in the library even if not installed via Steam.


Instrumentation Instance

Adding new CRD InstrumentationInstance which will hold status for each active SDK. currently only created for eBPF SDKs.
Support notifications for errors in the new CR.
Add condition for healthy instances for specific source in the source endpoint.

The current design looks good, but my main concern is a potential flood of alerts on the frontend if there are many instrumented applications. I suggest adding batch-processing logic to mitigate this.

The current design looks good, but my main concern is a potential flood of alerts on the frontend if there are many instrumented applications. I suggest adding batch-processing logic to mitigate this.

Created a common batcher and modified the instrumented app and instrumentation instance to use it.


Update api.md - "Crate" > "Create"

What did you implement:
Docs: "Crate" > "Create"

How did you implement it:

How can we verify it:

Todos:

[ ] Write tests
[ ] Write documentation
[ ] Fix linting errors
[ ] Make sure code coverage hasn't dropped
[ ] Provide verification commands / resources
[ ] Enable "Allow edits from maintainers" for this PR
[ ] Update the messages below

Is this ready for review?: NO
Is it a breaking change?: NO

Thanks, @jacobsauerhoefer !