Quick Alchemy makes Medium sized bombs and elixirs for Tiny player character

Pretty much the title. Playing a sprite and when I quick alchemy the items are medium sized and make me encumbered.

Pretty much the title. Playing a sprite and when I quick alchemy the items are medium sized and make me encumbered.

Looking into this… rules wise I’m not sure that it works that way RAW. Nowhere do I find that Quick Alchemy makes smaller items or that consumables are size dependent.
Will keep looking but in the meantime I’ll work on making it an optional setting.

Thank you. Sprites are Tiny pcs so anything they make should also be tiny
or they wouldn't be able to use them.
On Sun, Nov 24, 2024, 1:13 PM Joseph Thomas @.***>
wrote:

Pretty much the title. Playing a sprite and when I quick alchemy the items
are medium sized and make me encumbered.
Looking into this… rules wise I’m not sure that it works that way RAW.
Nowhere do I find that Quick Alchemy makes smaller items or that
consumables are size dependent.
Will keep looking but in the meantime I’ll work on making it an optional
setting.
—
Reply to this email directly, view it on GitHub
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496147383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHQR6ZAIM6JLWVAEMAN2JGT2CIJNVAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGE2DOMZYGM
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Here's the specific rule entries.
https://2e.aonprd.com/Rules.aspx?ID=2164
https://2e.aonprd.com/Rules.aspx?ID=2165
On Sun, Nov 24, 2024, 2:58 PM Ryan Larkin @.***>
wrote:

Thank you. Sprites are Tiny pcs so anything they make should also be tiny
or they wouldn't be able to use them.
On Sun, Nov 24, 2024, 1:13 PM Joseph Thomas @.***>
wrote:

Pretty much the title. Playing a sprite and when I quick alchemy the
items are medium sized and make me encumbered.
Looking into this… rules wise I’m not sure that it works that way RAW.
Nowhere do I find that Quick Alchemy makes smaller items or that
consumables are size dependent.
Will keep looking but in the meantime I’ll work on making it an optional
setting.
—
Reply to this email directly, view it on GitHub
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496147383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHQR6ZAIM6JLWVAEMAN2JGT2CIJNVAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGE2DOMZYGM
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Personally, I agree with you and that’s how I would run it (have a sprite player in one of my games), however the consensus online including the PF2e discord is that those rules apply to permanent items not consumables, as consumables do not (in the rules) have a size. Otherwise everyone would just load up on tiny consumables that would be negligible for them.
So for now I’ll still make it optional but not the default setting.
Sent from my iPhone
On Nov 24, 2024, at 1:03 PM, captnobody61 @.***> wrote:

Here's the specific rule entries.
https://2e.aonprd.com/Rules.aspx?ID=2164
https://2e.aonprd.com/Rules.aspx?ID=2165
On Sun, Nov 24, 2024, 2:58 PM Ryan Larkin @.***>
wrote:

Thank you. Sprites are Tiny pcs so anything they make should also be tiny
or they wouldn't be able to use them.
On Sun, Nov 24, 2024, 1:13 PM Joseph Thomas @.***>
wrote:

Pretty much the title. Playing a sprite and when I quick alchemy the
items are medium sized and make me encumbered.
Looking into this… rules wise I’m not sure that it works that way RAW.
Nowhere do I find that Quick Alchemy makes smaller items or that
consumables are size dependent.
Will keep looking but in the meantime I’ll work on making it an optional
setting.
—
Reply to this email directly, view it on GitHub
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496147383,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHQR6ZAIM6JLWVAEMAN2JGT2CIJNVAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGE2DOMZYGM
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

—
Reply to this email directly, view it on GitHubhttps://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496204965, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB3C2GYMM4QIA4FTPLGYT732CIWINAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGIYDIOJWGU.
You are receiving this because you were assigned.Message ID: @.***>

Gotcha. So it's just foundry programing forcing all equipment to have a
size despite the rules?
On Sun, Nov 24, 2024, 5:32 PM Joseph Thomas @.***>
wrote:

Personally, I agree with you and that’s how I would run it (have a sprite
player in one of my games), however the consensus online including the PF2e
discord is that those rules apply to permanent items not consumables, as
consumables do not (in the rules) have a size. Otherwise everyone would
just load up on tiny consumables that would be negligible for them.
So for now I’ll still make it optional but not the default setting.
Sent from my iPhone
On Nov 24, 2024, at 1:03 PM, captnobody61 @.***> wrote:

Here's the specific rule entries.
https://2e.aonprd.com/Rules.aspx?ID=2164
https://2e.aonprd.com/Rules.aspx?ID=2165
On Sun, Nov 24, 2024, 2:58 PM Ryan Larkin @.***>
wrote:

Thank you. Sprites are Tiny pcs so anything they make should also be
tiny
or they wouldn't be able to use them.
On Sun, Nov 24, 2024, 1:13 PM Joseph Thomas @.***>
wrote:

Pretty much the title. Playing a sprite and when I quick alchemy the
items are medium sized and make me encumbered.
Looking into this… rules wise I’m not sure that it works that way RAW.
Nowhere do I find that Quick Alchemy makes smaller items or that
consumables are size dependent.
Will keep looking but in the meantime I’ll work on making it an
optional
setting.
—
Reply to this email directly, view it on GitHub
<
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496147383>,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/BHQR6ZAIM6JLWVAEMAN2JGT2CIJNVAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGE2DOMZYGM>

.
You are receiving this because you authored the thread.Message ID:
@.***
com>

—
Reply to this email directly, view it on GitHub<
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496204965>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AB3C2GYMM4QIA4FTPLGYT732CIWINAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGIYDIOJWGU>.
You are receiving this because you were assigned.Message ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2496279900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHQR6ZGJENCSRRK6UIE4RID2CJHXDAVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJWGI3TSOJQGA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

This should be fixed with v2.1.0

Thanks for the update and all the work on the module. Appreciate it!
On Tue, Nov 26, 2024, 8:15 AM Joseph Thomas @.***>
wrote:

This should be fixed with v2.1.0
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/releases/tag/2.1.0
—
Reply to this email directly, view it on GitHub
https://github.com/thejoester/pf2e-alchemist-remaster-ducttape/issues/2#issuecomment-2500774056,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHQR6ZGXMD3QWUNU3HYZFED2CRX53AVCNFSM6AAAAABSLO6YPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBQG43TIMBVGY
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


opentrack: 2022.3.0 -> 2023.3.0

Description of changes

Update to 2023.3.0
Add patch for a specific configuration, tracked by upstream PR: Fix build for wine without wrapper opentrack/opentrack#1754, as it's not NixOS specific.
Update X-Plane SDK URL
Add wine output support

Fixes #251257
Update of existing PR #267171
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

And please squash/rebase your commits. At least remove the reduntant ones (e.g. the one adding the platform).

Rebased on top on master and squashed commits for wine support.


Added "object" and "URLResponse" to default error, allow users to create more 'rogue' URL path

Other than that, the rest looks nice!

Simplified the implementation as you suggested. I didn't add fragment supports yet but you should really consider it! =D

Oh, the code I suggested drops path of baseURL :bow: (e.g. https://example.com/api)
How about appending path before creating NSURLComponents like below?
let URL = path.isEmpty ? baseURL : baseURL.URLByAppendingPathComponent(path)
guard let components = NSURLComponents(URL: URL, resolvingAgainstBaseURL: true) else {
    return .Failure(.InvalidBaseURL(baseURL))
}

AFAIK, HTTP servers don't handle URL fragment. Is that really necessary?

OK, turns out my test cases wasn't good enough =( I'll fix the issue as well as adding some test cases for it.

AFAIK, HTTP servers don't handle URL fragment. Is that really necessary?

OK nevermind, I think you're right :(

OK updated the PR as you suggested, and added tons of TERRIBLE TERRIBLE test codes :imp:
This makes sure how createTaskInURLSession() handles multiple / encounters, such as "https://example.com///foo//bar//".

Thanks :+1:


Remove console.log entries

Ticket No

Console.log and Indentation

Summary of changes

Removed console.log statement from DashboardFilter.tsx.
No changes have been made to the indentation as it seems to have the original indentation of the file and it looks fine in the codebase and different only on github.

How should this be tested? [Screenshots, Video or Type instructions]

Screenshot

Notes for reviewers

Notes for reviewers here

Out of scope

@Roland-Sankara , restrict this PR to only removing console.log entries and it should be good to go. Remove everything else apart from that. Some of the errors are actually useful for development - "No internet connection found. App is running in offline mode", etc. Can we explore something like this? The rule is conditionally disabled depending on the environment


Issue-Add extraEntropy

Automated Review Comments
Issues in index.d.ts:

(6, 'Line exceeds 80 characters')
Issues in index.js:
(2, 'Hardcoded string found')
(8, 'Hardcoded string found')
(9, 'Hardcoded string found')
(11, 'Line exceeds 80 characters')
Issues in index.ts:
(6, 'Line exceeds 80 characters')
(7, 'Hardcoded string found')
(8, 'Hardcoded string found')
(9, 'Line exceeds 80 characters')
Issues in package.json:
(3, 'Hardcoded string found')
(7, 'Hardcoded string found')
(9, 'Hardcoded string found')

I got it.


fix #18

Seems reasonable. Would you be willing to add a test similar to the example in #18?

Sorry for the delay. I have found FromSql is only required with Vec<Enum>, but I am not quite familiar with MySQL and SQLite, so I only added tests for postgres.

Thanks, looks great. I'll try to get it merged and released over the weekend


'Sealed override' is shown as 'virtual'

I tried to fix it but it looks like abandoning CodeDOM might be necessary. See https://github.com/ApiApprover/ApiApprover/pull/111.

abandoning CodeDOM might be necessary

Or forking it.

Once the event modifier stuff is in we could technically do the same trickery with methods

I'm not saying I like it but so far Code Dom has proven itself fairly flexible :)

Of course, with CodeDom and replacement magic you can achieve much more than just with CodeDom.

Added https://github.com/PublicApiGenerator/PublicApiGenerator/pull/152 for the method part. I have to finish 6 tests that came from the event modifier changes but other than that it seems that most cases are now addressed

PR is ready to go. In case nobody chimes in with thoughts I will merge it tomorrow evening or so

Added https://github.com/PublicApiGenerator/PublicApiGenerator/pull/155 for the properties part

I think this is reasonably addressed. Pretty sure there is an edge case that isn't covered yet but we can deal with those when the need arises.

Do you want to give beta 8 a stab? There is a known issue with a bit too much whitespace with the new modifer that I already sent a PR for. Other than that I hope this is it for 10.0.0


Dynamically create Judge model Secret resource

Description

Create Judge model Secret based on information provided in environment variables.
User doesn't have to manually create Secret with all the needed values.
Environment variables used for Secret creation:

JUDGE_API_KEY: Provide API key for Judge model endpoint
JUDGE_NAME: Name of the Judge model
JUDGE_ENDPOINT: Judge model serving endpoint
JUDGE_CA_CERT_FROM_OPENSHIFT: flag indicating that Judge model endpoint use the same certificate as OpenShift CA where the script is running (so that the certificate should be trusted), set to true

How Has This Been Tested?

Manually by running the test against OpenShift cluster with Judge model deployed in the same cluster.
Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/approve


change ownership's /var/log  #10

hotfix #10

LGTM, but create an issue to remove this hack, so we can discuss


Where are the embeddings for memory stored when memory is enabled?

While enabling memory, I want to know where are the embeddings stored and is there an option to specify the location for the database.
The code used is as followed:
memory=True,         embedder={             "provider": "cohere",             "config":{                 "model": "embed-english-v3.0",             "vector_dimension": 1024             }         },
The reference used is as below:
https://docs.crewai.com/core-concepts/Memory/
I want to make changes to the database manually and keep a check on the data stored in the database.

According to the documentation you can customise where your storage files live by setting the CREWAI_STORAGE_DIR. This gives you flexibility and allows you to have things where you want them and makes it easy for you to inspect the data.
You can read more about it under this section https://docs.crewai.com/core-concepts/Memory/#implementing-memory-in-your-crew


[Request]: build for linux sytems

Is your request related to a problem?
No
How is the feature request related to the problem?
No response
Describe your proposed solution
have ci/cd build for linux, at least a tar, or even better, build src rpms/ src debs if you cant be bothered to build for specific arch types
Describe alternatives you've considered
running in wine/ building locally
Additional context
No response

Due to the many direct calls to Windows function, at least at our current capacity we cannot offer linux builds. There are quite a lot of changes needs to be done on the back end to replace the Windows calls with linux one, and as all of our current devs are not a linux dev, this is quite difficult. We're sorry about this.
I'll left the issue open if someone wants to take on the challenge.


[vcpkg scripts] toolchain/osx: Fix CMAKE_HOST_SYSTEM_PROCESSOR condition with lower case arm64

Fix https://github.com/microsoft/vcpkg/issues/40474
CMake STREQUAL is case-sensitive, change ARM64 to arm64 https://cmake.org/cmake/help/latest/variable/CMAKE_APPLE_SILICON_PROCESSOR.html

There is an issue that it seems that this optimization has not taken effect:

The host dependency will still build
In port file, VCPKG_CROSSCOMPILING is still 1 (TRUE). So the port still requires being passed tools path for running.
In toolchain file, CMAKE_CROSSCOMPILING is set to empty. The package will build tools, and the passed tool path will be ignored.

Currently, the additional host dependencies could be avoided by specifying --host-triplet to same with target triplet. However, this method will fail if the host dependency does not support the target architecture, for example arm64.
Reference

#16111
#24100
https://devblogs.microsoft.com/cppblog/vcpkg-host-dependencies/

I see that the old line didn't make sense, but the new behaviour might not be desired either. Basically, I would suggest to let a cross build be a cross build.
At least for ports, there are no CPU-based short-cuts. When the target triplet is different from the host triplet, it is "host": true, !native, in manifest, and VCPKG_CROSSCOMPILING, in portfile. So when this line is reached for the target triplet, the host tools are already built.

I see that the old line didn't make sense, but the new behaviour might not be desired either. Basically, I would suggest to let a cross build be a cross build.

In that case, it will make "CMAKE_CROSSCOMPILING" true even if "--host-triplet" is specified.

In that case, it will make "CMAKE_CROSSCOMPILING" true even if "--host-triplet" is specified.

This particular toolchain file sets CMAKE_CROSSCOMPILING to OFF for a case which might looks like meant for target_triplet == host_triplet:
https://github.com/microsoft/vcpkg/blob/2f210a9c13fcf2b0c36b1f2187366feec37e3390/scripts/toolchains/osx.cmake#L44-L46
But vcpkg doesn't control CMAKE_HOST_SYSTEM_PROCESSOR... but CMake does, for macOS:  https://cmake.org/cmake/help/latest/variable/CMAKE_HOST_SYSTEM_PROCESSOR.html#macos-platforms
I can't tell how far one would get with running x64 build tools on arm64 osx but this would clearly remove the crosscompiling from targeting x64 osx.
And everything else could be subeject to a specific toolchain instead of vcpkg's generic one.

After not setting CMAKE_CROSSCOMPILING empty, x86-windows fails with 24 error ports and 76 cascaded ports.
SUMMARY FOR x86-windows
SUCCEEDED: 2138
BUILD_FAILED: 24
POST_BUILD_CHECKS_FAILED: 1
CASCADED_DUE_TO_MISSING_DEPENDENCIES: 76
It is basically be caused that these packages have different behavior depending on the variable CMAKE_CROSSCOMPILING.
Temporarily suspended modifications.

This is expected. The x86 windows hack hides the poor cross-build capabilities of several ports. The android triplets where the first real cross builds in CI.

This was intentionally changed in https://github.com/microsoft/vcpkg/pull/26240 with the expectation that arm64 macs can run x64 tools, and setting it to off can allow some ports to work that would not otherwise. It isn't clear to me that rolling this change back as this PR does will lead to a better user experience; in particular, it isn't clear that it fixes the original reported issue in https://github.com/microsoft/vcpkg/issues/40474 .
Is the real problem that we need to supply a native arm64 cmake?

When cross compiling, setting CMAKE_CROSSCOMPILING to empty leads confusion.

set(CMAKE_CROSSCOMPILING OFF CACHE STRING "")

sets CMAKE_CROSSCOMPILING to OFF, not "empty". The "" is the "doc string"

@AugP @vicroms @JavierMatosD @data-queue @ras0219-msft We aren't sure how this change fixes the problems described in #40474 and agree with the original design intent clearing CMAKE_CROSSCOMPILING at this time. That said, if it's better explained why this fixes the original problem, we would consider changing the existing CMAKE_CROSSCOMPILING design.

Speaking personally, I think if we were to make this change on macOS we should make similar changes for Windows, and that is going to break a lot of users. So we need a big carrot to go with that stick.

This is expected. The x86 windows hack hides the poor cross-build capabilities of several ports. The android triplets where the first real cross builds in CI.

Speaking personally, I think if we were to make this change on macOS we should make similar changes for Windows, and that is going to break a lot of users. So we need a big carrot to go with that stick.

As @dg0yt said, there are many ports depends on this hack for x86 windows. I'll try to fix these portability issues.

We aren't sure how this change fixes the problems described in #40474

I'll change this PR to fix the original issue.

Failed ports on x86-windows cross compiling

ampl-asl:x86-windows
aws-sdk-cpp:x86-windows
blas-test:x86-windows
clamav:x86-windows
cmake-user:x86-windows
cppmicroservices:x86-windows
dcmtk:x86-windows
discount:x86-windows
fastrtps:x86-windows
forge:x86-windows
gamedev-framework:x86-windows
halide:x86-windows
itk:x86-windows
kf5plotting:x86-windows
kf5sonnet:x86-windows
kf5syntaxhighlighting:x86-windows
kfr:x86-windows
lapack-test:x86-windows
libfreenect2:x86-windows
ois:x86-windows
openblas:x86-windows
opencc:x86-windows
plplot:x86-windows
qtactiveqt:x86-windows
salome-medcoupling:x86-windows
torch-th:x86-windows
vulkan-loader:x86-windows

Thanks to @JavierMatosD for testing this on an arm64 macOS device for me. It fixes things to have the intended behavior for him.

And it makes vcpkg build host tools for the arm64-osx host triplet but not reliably use them when targeting x64-osx, leaving more port bugs hidden.
(Yes, I still think these lines should be removed!)

@dg0yt Except the ports that can be disabled cross-compiling check, for the other ports that require tools to be compiled and run in-place , we will try make these depend on their own host ports. Subsequent fixes will be updated in the list above for the time being.

I don't believe this will happen.
This PR added a shortcut which wasn't needed in the past, and it shouldn't be needed in the future. Except for hiding poor port quality.


Minimize button in system menu still appear

Describe the bug
I want to hide minimize button so I use property ShowMinButton="False" to hide, minimize button on MainWindow is hidden but when I Right-click on titlebar, System Menu show up and Minimize Button stilll appear
Expected behavior
Minimize button in System menu is hidden (or disabled)
Screenshots

Environment:

MahApps.Metro version 1.6.5

@LuanPham97 this has now been fixed, and it will be released in the next version of MahApps.Metro v2.0

@LuanPham97 this has now been fixed, and it will be released in the next version of MahApps.Metro v2.0

Okay that’s great. Thank you!


Where are the settings stored on OSX ? Are multiple instances supported ?

@

Multiple instances are not yet supported.  For this app, I will have multiple logins built in instead of having to start a new instance of the app.  The app is an electron app, which uses Chrome.  Settings are stored in Chrome's Local Storage.


Extern type defined with generic parameters incorrectly generated in C++

This commit fixes #149

Automated message from Dropbox CLA bot
@rdegnan, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Automated message from Dropbox CLA bot
@rdegnan, thanks for signing the CLA!

Hi @rdegnan.  Thanks for taking a stab at this.  Looks like this change breaks some existing functionality, though.  When I try to build the sample app I see many errors like this:
/Users/atwyman/src/djinni/example/generated-src/cpp/item_list.hpp:13:10: error: use of class template 'std::vector'
      requires template arguments
    std::vector items;

Looks like your change is affecting the handling of template args for types like vector<> in a bad way.
For future reference, when making any changes, please make sure you can make all from the top-level of the repo (which can only run on a Mac due to the inclusion of iOS stuff).  That'll re-generate and build the example apps and test suite, and run the tests.  Note that you should also check in the resulting generated code.  That's not a normal best-practice for generated code, but in this case, reviewing the generated code for the example and test suite is the easiest way for a reviewer like me to see the net effect of any changes to the generator.


Add not found page and set up redirection

This PR adds a not found page and redirects to it if the user tries to navigate to a non-existent path.
Fixes #135.

Solid work @AlbertoPdRF! Thanks for this, merging in now.


phandangluu.ntucoder.net

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/hello-world-tutorial/modify
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Please provide details as stated above:
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.


How to pass env variables when starting a container?

I triesd:
pass as Hash - it is not working
Docker::Container.create('Image' => image_name, 'Env'=> { 'a' => '123'})
pass as String - it is not working
Docker::Container.create('Image' => image_name, 'Env'=> 'a=123')
What way is correct?

It should be 'Env' => ['KEY=value', 'KEY=value']


Jupyter + ML.NET | DataFrame | Suggestion on fetching columns

When using the DataFrame object, the current way to retrieve columns are by the df["ColumnName"]
It would be very neat to instead be able to get a property by executing df.ColumnName as a property. This may be possible to do by creating a generic version of a DataFrame, e.g.
var df = new DataFrame().ReadCsv(filePath)
I understand that the ReadCsv method currently is static, so not sure if this breaks a paradigm.

cc @pgovind - thoughts?

This may be possible to do by creating a generic version of a DataFrame

What generic would you imagine here?
public class Customer
{
    public string FirstName;
    public string LastName;
}

DataFrame<Customer> df = DataFrame<Customer>.ReadCsv(customerDataPath);
df.FirstName   // this gets me the FirstName column?

Is something like that what you are envisioning?

@eerhardt - yep exactly, something along those lines.
The DataFrame class can certainly be a copy-implementation of Pandas in .NET, but it would be really cool I think if we could make a bit more .NET friendly :)
Given that a large crowd that will be using ML.NET and DataFrame come from a .NET background, it would be so neat if we could leverage generic, LinQ and other C# syntax to work with the DataFrame.
I certainly can't answer whether that is possible from a performance stand-point though. I would be more than happy to try to help out as time permits.

Interesting. There is a good chance that the DataFrame API could eventually converge to something like this. This is the second time this has come up. First time is here. The DataFrame type seems to be in a space that is an intersection of data scientists and engineers, so there's always been this constant question of how strongly typed the API should be. The reasons I haven't made the APIs very strongly typed yet are:
a) I was targeting a natural DataFrame + Jupyter experience in .NET and we didn't have IntelliSense working in Jupyter yet. I assume this will happen in the future. At that point, there'd be a stronger case for this IMO.
b) I wanted to be conservative with the API initially. For ex: The datasets I've seen for most ML tasks seem to have many columns. On a notebook, defining a schema such as Customer but with say 15 columns didn't seem natural to me? My impression was that users might be turned off by all the code they'd have to type(especially without IntelliSense/code completion) just to read in a csv file. Without code completion, I figured that a weakly typed API was the way to go since it made code shorter and the intent clearer.
Having said all that, there are places where I realllllly wish I had more type information. The Merge/Join APIs or indexing(like you mention here) for ex. It'd also help with AppendRow and, I'm sure, in other places. I'll keep this in mind as we go along. A generic DataFrame derived from a base DataFrame will fit in nicely for sure with .NET for Spark scenarios.
Thoughts? Feedback? We definitely don't want the API to feel alien to .NET developers!

Thank you for your detailed and long answer @pgovind!
That all makes a lot of sense, and I can certainly see why the DataFrame is built as it is today.
With that said, it would be awesome to have the option for both, and that may not be in the road map as yet, and that's okay :) I was thinking about it last night, and it may be possible to achieve it without asking the user to specify the entire schema (which I agree can be a bit ugly).
We should be able to infer the schema based on the first row (if such row exist), and in theory create a new type based on that, and the inferred data types using reflection: https://docs.microsoft.com/en-us/dotnet/api/system.reflection.emit.typebuilder.createtype?view=netframework-4.8
I haven't tried that myself, so not sure if it actually would work.
When it comes to being ".NET friendly", I think the biggest ask I would have would be to enable LinQ queries on the columns or rows, to filter, select and project data.
Thank you for all your contributions to this framework!

We should be able to infer the schema based on the first row (if such row exist), and in theory create a new type based on that, and the inferred data types using reflection

@aslotte In the .NET notebook we wouldn't need to use reflection. We're already compiling code submissions, so we could, for example, introduce a magic command that generates and compiles this on demand, after which the generated type would be available for the duration of the notebook session. It might look something like this:
%%compile-dataframe --data c:\housing.csv --type MyData
MyData df = MyData.ReadCsv("c:\housing.csv");
IntColumn populationCol = df.population;

That's a great idea @jonsequitur

FYI, you can try this out by installing the Microsoft.DotNet.Interactive.ExtensionLab package in a .NET Interactive notebook:

@jonsequitur - any thoughts on contributing that directly to the Microsoft.Data.Analysis package? Then users don't need to install both?
https://github.com/dotnet/corefxlab/tree/master/src/Microsoft.Data.Analysis.Interactive


Dear Author, when will you make your code public?

Hello,
Thank you for your interest. Code is now released. I am Closing this issue.


Add support for doorsense to the activity module

Thanks for the PR! Can you also include sample request / response here if u have them?

Included in 0.11.0: https://github.com/snjoetw/py-august/releases/tag/0.11.0

@snjoetw. When I get add some docs with the sample responses in a follow-up PR
I also have some constants to add as well to enable some removal of strings in home assistant so I’ll try to find some time to get that in as well


WillDisplay doesn't get called?

IMPORTANT - BEFORE CREATING YOUR ISSUE PLEASE READ THE FOLLOWING:

PLEASE STATE THE VERSION NUMBER you are using.
v7.1.5
Are you using the latest version of JTAppleCalendar? Latest version is currently 7.1.5
YES
PROVIDE ENOUGH INPORMATION SO I CAN RECREATE THE PROBLEM.

I run the sample project you attached at issue 667
And I download the  sample project from the repo, it has same issue.
Any idea why?

For you to say that it is not getting called, you must have some expectation of when it gets called.
When do you expect it to be called?

I just want my calendar to scroll smoothly, since I need do some time-consuming jobs for the date before it calls cellForItemAt.

I understand the concern you have.
But i do not understand what you are trying to do in the code to fix your concern, and how is your fix not working.

I think what might be @pandaApe's issue is that when scrolling horizontally between calendar frames, cellForItemAt is not called for the very next cell coming into the view. My guess is this has something to do with tableView pre-fetching?
In the screenshot below, I am scrolling from one week to the next and the 13th is the next date to appear in the view frame, however, cellForItemAt is not called for that cell and is instead called for a cell two indexPath's later. As a result, my configureCell function is not called to update the incoming cell's backgroundView for the appropriate selectedPosition

See my JTAppleCalendarViewDelegate implementation code below:
func calendar(_ calendar: JTAppleCalendarView, willDisplay cell: JTAppleCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {
    let cell = cell as? CalendarDayCell
    cell?.configureCell(for: cellState, date: date)
}

func calendar(_ calendar: JTAppleCalendarView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTAppleCell {
    let cell = calendar.dequeueReusableCell(ofType: CalendarDayCell.self, for: indexPath)!
    cell.configureCell(for: cellState, date: date)

    return cell
}

func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
    let cell = cell as? CalendarDayCell
    cell?.configureCell(for: cellState, date: date)
}

func calendar(_ calendar: JTAppleCalendarView, didDeselectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
    let cell = cell as? CalendarDayCell
    cell?.configureCell(for: cellState, date: date)
}

@obianachebe
I am not sure how your calendar is not updating because i do not have your code.
Therefore i just added an example code you can follow to do range selection.
Download the github project and run the sample calendar attached to it. You'll see rangel selection as the 4th option. Let me know how it works.

@patchthecode I think my issue was actually because I'm using a calendar embedded in a UITableViewCell. For my use case, calling tableView.reloadData() updated the selection style correctly. Thanks for all the help and great work on this library!

Awesome. Glad it works. Closing this issue now.


Add support for GS/WH detection APIs

We need APIs that can query GS/WH arch at runtime (not from device ptr). This will allow us to better share code between the 2 archs. Furthermore, getting the grid size will also be needed. Note: we need support for grayskull, n150, n300 and galaxy grid sizes

not in need of this anymore. Please reopen if needed


/inheritance flag?

I see that the /inheritance option exists, but I don't see any documentation for it in this article, or else I'm misreading it.

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

ID: 0a990d09-b5b1-3aba-de57-0aec79c12784
Version Independent ID: 36d90c17-ba95-4ed6-de9d-f379dcad6c4d
Content: icacls
Content Source: WindowsServerDocs/administration/windows-commands/icacls.md
Product: windows-server-threshold
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Thanks for the feedback - I've updated the article.


async-37: Multilevel Rendering

Description

WIP

Type of change

[x] New feature in experimental code

References

Quadtree and Tiled rendering - Wikipedia
Quadtree and Octrees - Mike James
Working with Terrain - Sandbox
Terrain Rendering - Intel
Rendering of Large Textures for Real-Time Vizualization - Backes, et al
Bing Maps Tile System - Microsoft
Terrain in Battlefield 3 - EA (DICE)
Terrain Rendering in Far Cry 5 - Ubisoft

Terrain?
We reference terrain links because our images are "terrain sized". One of our test images is (300,000, 900,000) pixels. Often people with images that large are dealing with terrain. However, our rendering is much simpler than terrain rendering for three reasons:

Terrain is usually viewed in 3D with perspective. We are viewing top-down in 2D.
Therefore terrain is always rendering multiple resolutions, we do only when loading.
Terrain has varied elevations and geometry, our geometry is always perfectly flat.

With terrain they usually render in perspective, flying over the terrain. In that case, you are always rendering multiple levels of the quadtree. You render high resolution up close, and then progressively lower resolutions into the distance.
For us, we only render multiple resolutions when loads are in progress. Once things have "settled down" we are always rendering just a single level, a simple grid. So while aggressively zooming our rendering is terrain-like, but in the steady-state, ours is simpler, more image-like.
Also with terrain not only do you have geometry, that geometry also needs to be high resolution up-close and coarser in the distance. So again ours is simpler. Yet ours is not trivial since we do have to render multiple levels of the quadtree while loads are in progress, so that basic technique is the same.

This is somewhat working but has bugs. Kind of a cool glitch right now:

Starting to work somewhat but here is a glitch, still debugging:

Now seamless zooming in:

Adding Octree.get_children() so now it can be seamless zooming both in and out:

Now it will find the nearest ancestor, even if it's more distant than the parent. Here panning reveals a tile from a higher level which is then replaced as it loads in:

This is ready for review. I think I need to look more at paging into RAM. This is mostly for RAM int VRAM although in theory in can deal with both stages ->RAM and then RAM->VRAM.

This one is going more than one level up to find an ancestor:


to_clipboard wrong linebreaks on Windows

Copying dataframe with to_clipboard and Qt gives different results: when pasted to Excel there's one more empty line. It may be unexpected and overwrite important data.
When looking into clipboard with Free Clipboard Viewer we can see that Qt uses CR+LF linebreaks:

and pandas uses LF:

Maybe this is the source of problem.
(This isn't fixed by #9693)

you might need to specifier the separator, this is using read_csv under the hood. Pls investigate what would make this work.

This helps:
pd.DataFrame(["Hello world"]).to_clipboard(line_terminator="\r\n")

sorry, meant to_csv. excel=True already implies sep='\t', so not sure what you think should change here.

What? I do not undrrstand.) It also implies line_terminator to be '\r\n', cause we're going to paste data in Excel. But it is '\n' by default.

@Winand sorry, was distracted a bit.....yes it is whatever the default line_terminator is in to_csv, which is \n. This works with excel now, so are you suggesting this should default (on windows) to \n\r?

@jreback i mean:
if we say Excel=True, we assume that data in clipboard may be pasted into Excel. BUT. If line terminators are '\n' (by default) an unwanted empty line is pasted. So i suggest to set line_terminator to '\r\n' by default on Windows.

but this DOES work on windows. can you give a copy-pastable example where this does work.

Right now i have pandas 0.16.2 patched with #9693 (works the same as 0.17.0 with #9693)
import pandas as pd
pd.DataFrame(["Привет, мир"]).to_clipboard()

Ctrl+V in Excel:

And now with \r\n:
pd.DataFrame(["Привет, мир"]).to_clipboard(line_terminator='\r\n')

Ctrl+V in Excel:

I see that. Hmm, must have always been like that.
@sinhrks @TomAugspurger ever seen this? does it matter?

Haven't tried to_clipboard -> excel before, but just tested it on windows and I see the extra line.
Not sure what if anything this could break if we change the default. Presumably people on Windows would expect the default to be \r\n?

Confirmed the issue on my Windows also. And changing line sep sounds valid. I think we can use os.linesep (\r\n on windows) as default?

this is actually tricky because say you do to_clipboard, we don't know which platform its going to be pasted, though I suppose we can assume that its the SAME as the current platform.
Though I suppose non-windows platforms can handle the os.linesep correctly. can someone test this out?
e.g.
do these produce differing results on non-windows? (these work for me on macosx)
for term in ['\n','\r\n']:
    df.to_clipboard(line_terminator=term)

so is soln just to default line_terminator='\r\n'?

anyone want to do a PR for this?

i'll try today.

https://github.com/pandas-dev/pandas/commit/4a1a330343f31fc8ca62b98a84978475968cba9c
updated the vendored pyperclip. does this resolve?


fix: warehouse size validations

max clusters of 10 is a soft limit and can be lifted via support. this removes some validation on the max clusters to allow for these soft limit elevations.

@sfc-gh-bculberson Thanks for fixing this! Can you add a unit test with MinCount > 10 to ensure we don't introduce a regression later?

/ok-to-test sha=2d402ba665a2d979d3593ed3181261f184040f7d


editor bigger than it should

https://user-images.githubusercontent.com/88567622/198382616-a87a2d48-6363-4411-8994-79fb27bda837.mp4
I'm not sure but the solution might be add scrollbar in file, because it gets bigger so you can see all your files

I'll try to work on that. Might fix it today, maybe tomorrow.

Got it! Nice catch.


feat(docs): Add light documentation for Relay and Styled Components

Took me a minute to figure out all of the various pieces of this to setup in our app so figured I would come back and add some light docs.

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


Error with Travis build step

Hello guys
i have been looking for a utility like that one for a few weeks now .. i followed the instructions to the best i could but after many tries , i always come down with this error at the build process
Not so sure if it's a bug or something wrong i'm doing but any help will be appreciated
here's the log from the Travis build
Computing checksum with sha256sum
Checksums matched!
Now using node v7.9.0 (npm v4.2.0)
travis_time:end:15af49d8:start=1492564024861290434,finish=1492564028362466982,duration=3501176548
travis_fold:end:nvm.install
$ node --version
v7.9.0
$ npm --version
4.2.0
$ nvm --version
0.33.1
travis_fold:start:install
travis_time:start:05dec4d8
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g

<EMAIL_ADDRESS>postinstall /home/travis/build/godloth/Test-doc/node_modules/jsonpath
node lib/aesprim.js > generated/aesprim-browser.js

<EMAIL_ADDRESS>/home/travis/build/godloth/Test-doc
travis_time:end:05dec4d8:start=1492564029459106206,finish=1492564055571356505,duration=26112250299
travis_fold:end:install
travis_time:start:0cdb816b
$ npm test

<EMAIL_ADDRESS>test /home/travis/build/godloth/Test-doc
swagger-repo validate

Adding paths to spec
/home/travis/build/godloth/Test-doc/node_modules/swagger-repo/lib/index.js:121
throw Error('All paths should be defined inside ' + pathsDir);
^
Error: All paths should be defined inside spec/paths/
at Object.exports.bundle (/home/travis/build/godloth/Test-doc/node_modules/swagger-repo/lib/index.js:121:13)
at Command. (/home/travis/build/godloth/Test-doc/node_modules/swagger-repo/bin/swagger-repo.js:41:23)
at Command.listener (/home/travis/build/godloth/Test-doc/node_modules/commander/index.js:301:8)
at emitTwo (events.js:106:13)
at Command.emit (events.js:194:7)
at Command.parseArgs (/home/travis/build/godloth/Test-doc/node_modules/commander/index.js:615:12)
at Command.parse (/home/travis/build/godloth/Test-doc/node_modules/commander/index.js:458:21)
at Object. (/home/travis/build/godloth/Test-doc/node_modules/swagger-repo/bin/swagger-repo.js:85:4)
at Module._compile (module.js:571:32)
at Object.Module._extensions..js (module.js:580:10)
npm ERR! Test failed.  See above for more details.

travis_time:end:0cdb816b:start=1492564055576742665,finish=1492564057024598145,duration=1447855480

The command "npm test" exited with 1.
Done. Your build exited with 1.`

@godloth Travis fail with the following error:
Error: All paths should be defined inside spec/paths/

It's happening because you enabled splitSpec option (disabled by default):
https://github.com/godloth/Test-doc/blob/master/.yo-rc.json#L7
Solution 1
You can disable it by generating repo one more time and answer no (default) to this querion:

Split spec into separate files: paths/, definitions/  [Experimental]?

Solution 2
Or you should specify your spec here:

Please specify path to OpenAPI/Swagger spec(local file)?

And your spec will be automatically split into multiple files
Solution 3
You can manually split your spec into multiple files using these instructions:
https://github.com/godloth/Test-doc/tree/master/spec/paths
https://github.com/godloth/Test-doc/tree/master/spec/definitions
Please write to us if you will have problems with one of the proposed solutions.

Thanks for the complete answer @IvanGoncharov , i will test them out and get back to you to close the issue
thanks

@godloth Can I close this issue?

You can , thanks
Get Outlook for iOShttps://aka.ms/o0ukef

From: Ivan Goncharov<EMAIL_ADDRESS>Sent: Thursday, May 4, 2017 11:40:00 AM
To: Rebilly/generator-openapi-repo
Cc: godloth; Mention
Subject: Re: [Rebilly/generator-openapi-repo] Error with Travis build step (#9)
@godlothhttps://github.com/godloth Can I close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Rebilly/generator-openapi-repo/issues/9#issuecomment-299224044, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AVSThd1Nm_YCXsWsHGkZVYtDlk9AbaSZks5r2fFQgaJpZM4NBFpw.


Changed lines of modified files

Is this feature missing in the latest version?

[X] I'm using the latest release

Is your feature request related to a problem? Please describe.
Why:
By providing the start and end of changed lines (chars) one can check some validations that only apply to the changed lines.
Use cases:
on a pull request, a github action will only check to see if developer has added a new inline disabled rule for eslint and create a comment to make it visible to reviewers that the pr includes disabled rules.
Describe the solution you'd like?
I think the provided output should include the changed lines:
modified_files_details   |   array<changeInfo>    |  [{file:string, type:modified | deleted | added, changes:[{startLine:number, endLine:number}]}]

Ex.
[
   {file:'package.json', type: 'modified', changes:[{startLine:2, endLine:5}, {startLine:13, endLine:15}]},
   {file:'package.json', type: 'added', changes:[{startLine:20, endLine:21}]}
]

or we can use file names as keys:
modified_files_details   |  object    |  {fileName:[{ type:modified | deleted | added, changes:[{startLine:number, endLine:number}]}]}

Ex.
{
   'package.json': {type: 'modified', changes:[{startLine:2, endLine:5}, {startLine:13, endLine:15}]},
   'package.json': {type: 'added', changes:[{startLine:20, endLine:21}]},
}

Describe alternatives you've considered?
No response
Anything else?
No response
Code of Conduct

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

@ImanMahmoudinasab Thanks for your suggestion, given the complexity of managing the number of lines that can change in a Pull Request, at this time this wouldn't be possible without a more involved action so I'll recommend integrating a third-party solution which should be able to handle the complexity on a case-by-case basis.


Integration of custom-refresh-indicator and paginate_firestore

Hi friends,
What I to integrate custom-refresh-indicator and paginate_firestore, I think this feature will be very useful for a lot of devs, can somebody guide us on how to achieve this? Thanks in advance!

You should be able to wrap your paginate_firestore with refresh_indicator...

Hi, @eturkmenler I managed that, I needed a few more control over the stream it's fine now, the only issue now is that the author to update the dependency to latest firebase SDK ;)


Adding the structure of urlConfiguration in rewriteRuleActionSet

These changes have already been approved in issue https://github.com/Azure/azure-rest-api-specs/pull/7275/ and merged as a part of September milestone. However, because of some timeline issue, we reverted these changes in September milestone. This is targetted to be released as a part of December milestone.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Automation for azure-sdk-for-python

LGTM but could you please also run prettier to fix JSON formatting?
https://dev.azure.com/azure-sdk/public/_build/results?buildId=203587&view=logs&jobId=b56e9678-4dad-5642-289c-5530cc4dad11

/azp run automation - sdk

LGTM but could you please also run prettier to fix JSON formatting?
https://dev.azure.com/azure-sdk/public/_build/results?buildId=203587&view=logs&jobId=b56e9678-4dad-5642-289c-5530cc4dad11

@number213 : I am not sure why it is failing. I followed https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md to run prettier-fix for changed files and it worked fine.
Sample output:
D:\Aznet\swagger\dec19\azure-rest-api-specs>npm run prettier -- --write "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json

<EMAIL_ADDRESS>prettier D:\Aznet\swagger\dec19\azure-rest-api-specs
prettier "--write" "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json"

specification\network\resource-manager\Microsoft.Network\stable\2019-11-01\applicationGateway.json 211ms

@MikhailTryakhov These changes have already been approved in issue #7275 https://github.com/Azure/azure-rest-api-specs/pull/7275 and merged as a part of September milestone. Can you please approve?

I observed prettier check is failing..
@v-shhatt v-shhatt
I am not sure why it is failing. I followed https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/ci-fix.md to run prettier-fix for changed files and it worked fine.

Sample output:
D:\Aznet\swagger\dec19\azure-rest-api-specs>npm run prettier -- --write "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json
<EMAIL_ADDRESS>prettier D:\Aznet\swagger\dec19\azure-rest-api-specs
prettier "--write" "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json"
specification\network\resource-manager\Microsoft.Network\stable\2019-11-01\applicationGateway.json 211ms

Can one of the admins verify this patch?

@number213 @v-shhatt @MikhailTryakhov Prettier is failing in this build however it is passing locally on my system.
Sample output:
D:\Aznet\swagger\dec19\azure-rest-api-specs>npm run prettier -- --write "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json
<EMAIL_ADDRESS>prettier D:\Aznet\swagger\dec19\azure-rest-api-specs
prettier "--write" "specification/network/resource-manager/Microsoft.Network/stable/2019-11-01/applicationGateway.json"
specification\network\resource-manager\Microsoft.Network\stable\2019-11-01\applicationGateway.json 211ms

@abjai Please commit and push changes after running prettier

/azp run automation - sdk

@abjai Please commit and push changes after running prettier

@NullMDR : Please review the PR


Can not load dashboard

Describe the bug
When trying to get started get following message.
Error: Timeout waiting for strategy element ll-strategy-dashboard-mushroom-strategy to be registered
Versions

Mushroom-Strategy: 1.0.1
HACS: Core  2023.11.2
Supervisor      2023.11.3
Operating System   11.1
Frontend    20231030
Mushroom: 3.2.2
Home Assistant:

To Reproduce
Steps to reproduce the behavior:
Add text to empty dashboard vi raw configuration editor
strategy:
  type: custom:mushroom-strategy
views: []

Expected behavior
Dashboard to generate
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Tried to delete and reinstall

#90

Hi,
Prior to reporting I had done everything in #90.
Also deleted to and reinstalled. Restarted and reboooted HA.

Ok, then please upload your diagnostics file of HACS.
See https://hacs.xyz/docs/basic/diagnostics/

Mushroom Dashboard decided to appear.
Will leave things there for now.
Not sure what I did right.


Add Quilt

https://docs.quiltdata.com/api-reference/cli#quilt_disable_usage_metrics
https://github.com/quiltdata/quilt/blob/version-3.4.x/docs/API Reference/cli.md#quilt_disable_usage_metrics
disable via env var QUILT_DISABLE_USAGE_METRICS=true

Closed via 5f4e265


Add the TodoMVC example

I figured TodoMVC deserved a place under "Examples and boilerplates"

:+1:


Make sure description is not empty to avois spamming log

Avoid throwing IllegalArgumentException because description is empty.

+1


Failed at the<EMAIL_ADDRESS>install script 'node install.js'.

info postinstall<EMAIL_ADDRESS>4691 verbose unlock done using /root/.npm/_locks/request-7bb64398ccb405eb.lock for /opt/local/lib/node_modules/phantomjs/node_modules/request
4692 verbose about to build /opt/local/lib/node_modules/phantomjs
4693 info build /opt/local/lib/node_modules/phantomjs
4694 verbose linkStuff [ true,
4694 verbose linkStuff   '/opt/local/lib/node_modules',
4694 verbose linkStuff   true,
4694 verbose linkStuff   '/opt/local/lib/node_modules' ]
4695 info linkStuff<EMAIL_ADDRESS>4696 verbose linkBins<EMAIL_ADDRESS>4697 verbose link bins [ { phantomjs: './bin/phantomjs' }, '/opt/local/bin', true ]
4698 verbose linkMans<EMAIL_ADDRESS>4699 verbose rebuildBundles<EMAIL_ADDRESS>4700 verbose rebuildBundles [ '.bin',
4700 verbose rebuildBundles   'adm-zip',
4700 verbose rebuildBundles   'fs-extra',
4700 verbose rebuildBundles   'kew',
4700 verbose rebuildBundles   'md5',
4700 verbose rebuildBundles   'npmconf',
4700 verbose rebuildBundles   'progress',
4700 verbose rebuildBundles   'request',
4700 verbose rebuildBundles   'request-progress',
4700 verbose rebuildBundles   'which' ]
4701 verbose gentlyRm verifying that /opt/local is managed by npm
4702 silly isManaged /opt/local is not inside /opt/local/lib/node_modules
4703 silly isManaged /opt/local is not inside /opt/local/lib/node_modules
4704 silly isManaged /opt/local/bin is inside /opt/local
4705 silly gentlyRm /opt/local is managed by npm
4706 silly gentlyRm /opt/local/bin/phantomjs is under /opt/local
4707 verbose gentlyRm vacuuming /opt/local/bin/phantomjs up to /opt/local
4708 info install<EMAIL_ADDRESS>4709 verbose unsafe-perm in lifecycle false
4710 info<EMAIL_ADDRESS>Failed to exec install script
4711 verbose unlock done using /root/.npm/_locks/phantomjs-7b796d8c35016010.lock for /opt/local/lib/node_modules/phantomjs
4712 verbose stack Error<EMAIL_ADDRESS>install: node install.js
4712 verbose stack Exit status 1
4712 verbose stack     at EventEmitter. (/opt/local/lib/node_modules/npm/lib/utils/lifecycle.js:213:16)
4712 verbose stack     at EventEmitter.emit (events.js:110:17)
4712 verbose stack     at ChildProcess. (/opt/local/lib/node_modules/npm/lib/utils/spawn.js:14:12)
4712 verbose stack     at ChildProcess.emit (events.js:110:17)
4712 verbose stack     at maybeClose (child_process.js:1008:16)
4712 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1080:5)
4713 verbose pkgid<EMAIL_ADDRESS>4714 verbose cwd /home/admin/blecloud
4715 error SunOS 5.11
4716 error argv "/opt/local/bin/node" "/opt/local/bin/npm" "-g" "install" "phantomjs"
4717 error node v0.12.0
4718 error npm  v2.5.1
4719 error code ELIFECYCLE
4720 error<EMAIL_ADDRESS>install: node install.js
4720 error Exit status 1
4721 error Failed at the<EMAIL_ADDRESS>install script 'node install.js'.
4721 error This is most likely a problem with the phantomjs package,
4721 error not with npm itself.
4721 error Tell the author that this fails on your system:
4721 error     node install.js
4721 error You can get their info via:
4721 error     npm owner ls phantomjs
4721 error There is likely additional logging output above.
4722 verbose exit [ 1, true ]
4723 verbose unbuild lib/node_modules/phantomjs
4724 info preuninstall<EMAIL_ADDRESS>4725 info uninstall<EMAIL_ADDRESS>4726 verbose unbuild rmStuff<EMAIL_ADDRESS>from /opt/local/lib/node_modules
4727 verbose gentlyRm verifying that /opt/local is managed by npm
4728 silly isManaged /opt/local is not inside /opt/local/lib/node_modules
4729 silly isManaged /opt/local is not inside /opt/local/lib/node_modules
4730 silly isManaged /opt/local is not inside /opt/local/bin
4731 silly isManaged /opt/local is inside /opt/local
4732 silly gentlyRm /opt/local is managed by npm
4733 silly gentlyRm /opt/local/bin/phantomjs is under /opt/local
4734 verbose gentlyRm vacuuming /opt/local/bin/phantomjs up to /opt/local
4735 silly gentlyRm removing /opt/local/bin/phantomjs
4736 silly gentlyRm quitting because other entries in /opt/local/bin
4737 info postuninstall<EMAIL_ADDRESS>4738 verbose gentlyRm vacuuming /opt/local/lib/node_modules/phantomjs
4739 silly gentlyRm purging /opt/local/lib/node_modules/phantomjs
4740 silly gentlyRm quitting because other entries in /opt/local/lib/node_modules

The npm package wrapper around phantomjs is separately maintained at https://github.com/Medium/phantomjs

Exactly. We don't maintain the NPM package.


ZeroDivision error when max=0

When "max" is set to zero (can happen if there's nothing to process) the code returns a ZeroDivision error when update() is called.
The culprit seems to be in progress/__init__.py, like 156:
@property
def progress(self):
    return min(1, self.index / self.max)

I see there's already a fix in PR #72, closing this one.


Issue:

Experiencing the error message below after successful validation.
I find in C:\wamp64 - ssl auto config\vhosts\http\ an empty folder
and in C:\wamp64 - ssl auto config\vhosts\https\conf an httpd-ssl.conf
WampServer SSL Auto Config (v1.2.3)
22:50:50.62 : Parsed configuration file.
22:50:50.63 : Common 'logs' directory already exists.
22:50:50.64 : Script initialised.
22:50:50.66 : ---------------------------------------------
22:50:50.68 : Found WampServer installation path.
22:50:50.70 : Found WampServer Apache path.
22:50:50.71 : Found WampServer Apache installation(s):
22:50:50.74 :   'apache<IP_ADDRESS>'
22:50:50.77 : Found OS 'hosts' file.
22:50:50.80 : Found OS Apache service name.
22:50:50.81 : Validating Apache config file(s):

                        FAILURE

WampServer Apache Validation Error
File:  See below
Error: httpd.exe: Syntax error on line 597 of C:/wamp64/bin/apache/apache<IP_ADDRESS>/conf/httpd.conf: Include/IncludeOptional: No matches for the wildcard '*.conf' in 'C:/wamp64 - ssl auto config/vhosts/http', failing
Please correct the error in the stated configuration file and restart WampServer.

Press any key to exit.
Appreciate some guidance on this issue.

Hi @GerryGit
Could you please add your config file to this thread.
As this is not the first time you have run ssl_config.bat, did it work the very first time you ran it?
It looks like your [host-name].conf file is missing from the designated directory (being C:/wamp64 - ssl auto config/vhosts/http).
May you have accidently deleted C:/wamp64 - ssl auto config/vhosts/http/[host-name].conf ?
You should have a minimum of one [host-name].conf file in both the http and https directories, as shown below:

C:/wamp64 - ssl auto config/vhosts/http/[host-name].conf
C:/wamp64 - ssl auto config/vhosts/https/[host-name].conf

Let me know if they exist.

The folders do exist as follows:
C:\wamp64 - ssl auto config\vhosts\https\conf
httpd-ssl.conf contents:
Listen 443
SSL Cipher Suite:
SSLCipherSuite HIGH:MEDIUM:!MD5:!RC4:!3DES
SSLProxyCipherSuite HIGH:MEDIUM:!MD5:!RC4:!3DES
Enforce the server's cipher order.
SSLHonorCipherOrder on
SSL Protocol support:
SSLProtocol all -SSLv3
SSLProxyProtocol all -SSLv3
Pass Phrase Dialog:
SSLPassPhraseDialog  builtin
Inter-Process Session Cache:
SSLSessionCache        "shmcb:c:/Apache24/logs/ssl_scache(512000)"
SSLSessionCacheTimeout  300
and
C:\wamp64 - ssl auto config\vhosts\http
which is empty
Neither of them contain [host-name].conf files
I did not delete them. Is it possible to add them manually?

Hi @GerryGit
Could you please add the below listed files to this thread to assist in debugging.

config.ini (See "readme.md" here for more information)
ssl_config.log

Thank you.

Hi @GerryGit
As I haven't received a response regarding the above issue for over 30 days now I'm going to assume that your issue has been resolved.
If not, just open a new issue so we may continue to investigate the problem.
Thank you.


fix image urls

Corrected the image url in the README file

Before
After

Hey @shuklaritvik06 , thanks for creating this PR! Unfortunately I wasn't able to directly import this as there was an issue with our copybara config that is now fixed. That fix also solved the image URLs, so I'll close this PR. Thanks for highlighting this issue and hope you'll contribute more to this project later! :)

Hey @shuklaritvik06 , thanks for creating this PR! Unfortunately I wasn't able to directly import this as there was an issue with our copybara config that is now fixed. That fix also solved the image URLs, so I'll close this PR. Thanks for highlighting this issue and hope you'll contribute more to this project later! :)

Hey @biharygergo  Happy that this issue is resolved now, thanks to you btw, I got to know about the new tool copybara 🔥 I would surely love to contribute and explore this and google's other OSS projects :)


Improved JS bundling

Improved JS bundling.
Note that we should serve both:

[x] Separated js/wasm bundle (default)
[ ] Joined wasm in js file (via base64)

@syrusakbary what else do you need to push this over the line?


tbv2: fix pointer codegen

tbv2: fix pointer codegen
A previous change enabled running OIL tests with specific features enabled.
This highlighted that pointer code generation under TreeBuilder-v2 was very
broken. This change updates pointer code generation to work and enables the
skipped tests. All enabled tests need expected_json_v2 added to them due to
formatting differences.
Reformatted and rewrote the basic type handler that handles primitives and
pointers. Removed the reliance on features to decide whether to generate for
TreeBuilder-v2 as the intermediate features have been removed.
There were a couple of other changes needed to enable these tests on TBv2 that
aren't worth their own issues and PRs, I sneaked them in here.
Extra changes:

Added Pointer and Reference to TopoSorter so they generate
NameProvider instances. It might be worth visiting the graph differently
for NameProvider as it requires so many instances that others generators do
not. Will consider that in the future.
Follow typedefs when calculating exclusive size for a type.

Closes #458.
Test plan:

CI
Enabled previously disabled tests.

Stack created with Sapling. Best reviewed with ReviewStack.

-> #461
#460

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (4c047b5) 62.44% compared to head (03c81cb) 62.52%.

Files
Patch %
Lines

oi/CodeGen.cpp
66.66%
0 Missing and 3 partials :warning:

oi/type_graph/TopoSorter.cpp
75.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #461      +/-   ##
==========================================
+ Coverage   62.44%   62.52%   +0.07%
==========================================
  Files         123      123
  Lines       12021    12034      +13
  Branches     1958     1958
==========================================
+ Hits         7507     7524      +17
+ Misses       3569     3563       -6
- Partials      945      947       +2

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


Unable to set fsGroup parameter in a Pod that uses RWX PVC

Describe the bug
We were trying to install an application on our cluster through helm installation, and we found out that the fsGroup parameter in the pod Security Context was not being applied due to our PVC having RWX access mode (relevant trident documentation). this is causing a Permission denied error in our container, and the only workaround we have is to run the pod as root user, which we want to avoid. We require RWX mode for the PVC for our use case.
Essentially this is causing one of the directory's owners to change to root, but the container runs with user 1000. Then, we see 'Permission denied' error because the user 1000 does not have permission to execute commands in the directory.
Is there any way we can get around this issue? We have tried adding an initContainer to change the ownership and access of the directory, but I think this gets overwritten in the main container as we're still seeing the same error.
Environment
Provide accurate information about the environment to help us reproduce the issue.

Trident version: [e.g. 19.10]

$ tridentctl version -n trident
+----------------+----------------+
| SERVER VERSION | CLIENT VERSION |
+----------------+----------------+
| 23.07.0        | 23.07.0        |
+----------------+----------------+

Container runtime:

We are using cloudbees image: https://hub.docker.com/layers/cloudbees/cloudbees-cloud-core-oc/2.452.3.2/images/sha256-d92794c80d0c5281fb0c3195a24934944dec765376cdb8260d0af24fba5d0d6a

Kubernetes version: [e.g. 1.15.1]

'Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.0", GitCommit:"c2b5237ccd9c0f1d600d3072634ca66cefdf272f", GitTreeState:"clean", BuildDate:"2021-08-04T18:03:20Z", GoVersion:"go1.16.6", Compiler:"gc", Platform:"darwin/arm64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.8", GitCommit:"7061dbbf75f9f82e8ab21f9be7e8ffcaae8e0d44", GitTreeState:"clean", BuildDate:"2022-03-16T14:04:34Z", GoVersion:"go1.16.15", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes orchestrator: we are using on-prem Kubernetes, we used Kubespray to deploy

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

This works as designed from a K8s perspective. It is unrelated to Trident.
fsGroup is not supported for RWX PVCs. This design decision tries to avoid flapping permissions as you could have multiple pods with different fsGroup settings that mount the same RWX PVCs. In this case the permissions would be unpredictable as they would frequently change and only reflect the last pod that was (re)started.
Straight from the K8s code, anything that is not RWO is skipped and you'd see an appropriate log message "skipping fsGroup, only support ReadWriteOnce access mode": https://github.com/kubernetes/kubernetes/blob/release-1.22/pkg/volume/csi/csi_mounter.go#L419
If it absolutely must be RWX, init containers/processes or similar approaches are your only option, I think.

@ssolanki38 I hope the above comment helps answer your question. If not, please let us know.
I will be marking this issue as closed for the time being.


Respect the @internal trait concerning ValidationException messages

This Smithy PR adds @internal to enum EnumString. We don't respect the semantics of @internal when considering constraint trait ValidationException messages and as a result, the PR updating to Smithy 1.28.1 causes the corresponding protocol tests to begin failing.

I think the Smithy spec should clarify here that this is indeed how the trait should be honored. My reading of the spec does not imply this. I opened a consultation: https://github.com/awslabs/smithy/issues/1737.


Camera angle

Hi,
Can you please tell me what is the camera angle(thetaX,thetaY, thetaZ)? Is it camera eye or gaze? How can we get camera angle?

I don't understand your question. Please have a look at the documentation: you set the "angle" with dir.


Port Cropping library

They cropping library in Bodoko is quite useful for creating diagrams. We should definitely port that over: https://github.com/Kashomon/bodoko/tree/master/cropping

I am not sure what autocrop is supposed to do, might want to check on that

In fact, I don't think it's complete either, so I will leave it commented out


clarify license

There is no license mentioned anywhere. Can it be added?

This issue is a blocker for packaging in Fedora. Please add a license file.

Above Pull Request adds MIT license, as suggested.


update status.get.html to use new css and jquery

That's awesome!  Thanks for doing all this work!


[GameTest][feature] 4D location

This will be a breaking change. I'm not sure if it's good enough since it breaks other's code.
Constructor
const loc4 = new Location4(x, y, z, MinecraftDimensionTypes.overworld);
const loc4 = new Location4(x, y, z, world.getDimension("overworld");

Factory
const loc4 = player.dimension.getLocation(x, y, z);

Setter(enum/Dimension instance)
loc4.dimension = MinecraftDimensionTypes.overworld;
loc4.dimension = world.getDimension("overworld");

Getter(Dimension instance)
loc4.dimension.runCommand("say hi");

Access world
4D location should have all methods in class Dimension
world.getBlock(blockLoc4);
blockLoc4.block;

world.isEmpty(blockLoc4);
blockLoc4.isEmpty;

loc4.spawnEntity("minecraft:cow");
blockLoc4.spawnEntity("minecraft:cow");

loc4.runCommand("kill @e[r=8]");

Convert(according to NBT, 1:8 is only by default)
const netherLoc4 = overworldLoc4.toNether();

Teleport
entity.teleport(target.location4);

MinecraftDimensionTypes only returns dimension id (string btw)

@JaylyDev Do you know what getter and setter are?
const ow = {id: "minecraft:overworld"};
const n = {id: "minecraft:nether"};
const te = {id: "minecraft:the_end"};
class Loc4 {
  #dimension;
  set dimension(type) {
    if (["minecraft:overworld","minecraft:nether","minecraft:the_end"].includes(type)) {
      this.#dimension = type;
      return;
    }
    throw new Error;
  }
  get dimension() {
    switch (this.#dimension) {
        case "minecraft:the_end":
        return te;
        case "minecraft:nether":
        return n;
        default:
        return ow;
    }
  }
}
const Types = {
  overworld: "minecraft:overworld",
  nether: "minecraft:nether",
  theEnd: "minecraft:the_end"
};

const loc4 = new Loc4;

loc4.dimension = "minecraft:nether";
console.log(Object.is(loc4.dimension, n));

loc4.dimension = "minecraft:the_end";
console.log(Object.is(loc4.dimension, te));

loc4.dimension = "minecraft:overworld";
console.log(Object.is(loc4.dimension, ow));

I do think that once upon a time we did consider making location "4D" - but instead cleaved the API such that it's implicit what dimension is being operated in in a lot of cases (e.g., if you set location on an entity, that's based on a dimension object or the dimension the entity is currently within.) We did argue about this a good amount, but I think we're pretty far down the path that dimension is, for better or worse, separate from location.


[GEKIDOU] Bottom Tab Navigation

Implementing new bottom tab navigation for Gekidou

@avinashlng1080 anything in particular that you want me to look at ?

@avinashlng1080 anything in particular that you want me to look at ?

Hey @enahum  - nothing in particular at the moment.  Just wanted to share this PR so that we can see the progress made.

/release-note-none

@deanwhillier added you as I included the changes for the default Themes ;)

going to merge this.


Using twarc-network programatically

Hi,
I am using twarc from a custom code I implemented. I use it to extract tweets and store them into a database. Now, I would like to use some of those tweets to create a network graph using twarc-network. Instead of re-extracting the tweets from Twitter or reverse engineering the jsonl format to export my data to that format, is it possible to easily use twarc-network programmatically to be able to pass the tweets directly from my DB and generate the files?
Thanks,
David.

It's a bit awkward:
It's always advisable to keep the original json files twarc produces when retrieving - this way you can always run anything on top of those, but if you don't have those anymore, or you have some specific selection happening through your database, you can also use twarc-network manually too, by calling https://github.com/DocNow/twarc-network/blob/main/twarc_network/init.py#L83 directly (infile doesn't have to be a file, it can be an iterable) it expects to iterate over lines of text though, so you have to pass it strings it can parse as tweets.
It doesn't have to be exactly the same as the original json, but it has to resemble it - a tweet still has to have an author dict with a username in it for example. So the process could be: Retrieve rows from your DB, construct a dict that resembles the original json, turn it into a string and pass it to the function.
This is a bit awkward - so if you no longer have original json, and you only have DB records in some custom format, it's easier to rewrite your own https://github.com/DocNow/twarc-network/blob/main/twarc_network/init.py#L83 function so it runs through your rows correctly - that way everything else will work along with that.

It should not be a big problem, because even if I store my own version of the structure of the tweets I retrieve, I also store the original output of the twitter API just in case I needed it. And this seems to be the case. I will give it a try. If I find an easy way to do it, I will open a pull request to the repo to add instructions of how to use it programmatically.

@DavidHernandez if you can come up with a way to make twarc-network more useful programmatically while preserving the existing behavior that would be most welcome!


How to work with existing public key

Hello, it's clear from the docs how to work with public/private keys when generating them from scratch. I'm trying to figure out how to use the existing public key to pass the validateEvent check.
I went through the nostr-tools and @noble/secp256k1 packages and tried to simulate the creating of the keys.
import * as secp256k1 from '@noble/secp256k1'
import { utf8Encoder } from 'nostr-tools/utils'

const pubkey = secp256k1.utils.bytesToHex(
  utf8Encoder.encode('npub1yrmj7fe0t5tjffv8e28et7ur27zvrmucmeqytt3w2wdp2tp3la3shpy9pf')
)

// 6e7075623179726d6a376665307435746a66667638653238657437757232377a76726d75636d6571797474337732776470327470336c613373687079397066

Such constructed pubkey fails the validation pubkey.match(/^[a-f0-9]{64}$/).
How should I work with existing public key? Thank you!

Just found NIP19 which probably answers my question.

Just found NIP19 which probably answers my question.

@cypcz can you elaborate?

All functions expect public keys in hex format (32-bytes as 64 hex characters). To convert between npub and hex use the nip19 module.

understood! but if i just want to validate that the provided hex format public key is valid? sorry if it's a noob question

hexkey.match(/^[a-f0-9]{64}$/)

hexkey.match(/^[a-f0-9]{64}$/)

appreciate you!


Support filtered tree query

Currently the CTE  query build tree  in table scope and this make a huge performance impact when table is big.
In my situation, we seprate the records by user id which means the performance overhead will be reduced if we can build tree at the user level.

Maybe django-cte has a solution? https://github.com/dimagi/django-cte/blob/master/tests/test_recursive.py
I think there's some hope to avoid the optimization fence in PostgreSQL in the future and allow pushing down predicates into the CTE – avoiding this performance problem in the database engine itself isn't impossible but I think PostgreSQL still doesn't support it. I don't know about other database engines.
I'm not sure but I think that django-tree-queries isn't well suited when it comes to extending or customizing the CTE itself. The implementation of django-cte is probably a better fit when it comes to supporting more elaborate queries, I'm not sure.

Fixed by #66


We log failures in "after". Probably should do it "before" too.

More generally, is it correct to even run the after if an Exception is thrown in before?

:shipit:


Meeting Time (Frontend)

We should decide a regular time for meeting as frontend team. I think we can make our meeting before 3 pm on Monday. Is it okay for you?

Yes, I'm cool with that.


Powershell Values

Hey all,
I am looking to pull in values onto my dashboard based on results I get via Powershell. Not looking for anything that's too far-fetched (for now!), a simple proof of concept would be something like (Get-Hotfix).count, that returns a simple numeric result such as 45.
Anyone already doing this??
://mk

How about having your Powershell code post to you dashboard using Curl for windows? Alternatively, I believe there are methods in Powershell to use HTTP (such as: Invoke-WebRequest).
Unless I'm missing the gist of you question, gather the data you want using Powershell, then post the result to the dashboard using HTTP. If you need to do this periodically, run the Powershell script via a scheduled task. I do something similar using VBScript.
It may also be possible to run your Powershell script from Ruby using System or Exec commands and pass the data via a file dump. Ruby call a batch file, which runs a Powershell script and dumps data to a temp file, Ruby then reads the temp file data and posts to the dashboard. Have not done this myself but seems possible.
Tim

Martyn,
Somehuman is correct in using Invoke-Request.  Here is an example of pulling total # of VMs from Virtual Center via PowerCli and posting to Dashing via Rest.
`if ( !(Get-Module -Name VMware.VimAutomation.Core )) {
“C:\Program Files (x86)\VMware\Infrastructure\vSphere PowerCLI\Scripts\Initialize-PowerCLIEnvironment.ps1”
}
function Postto-Dashing {
<#
.Synopsis
Post text to a Dashing Widget
.DESCRIPTION
Long description
.EXAMPLE
Postto-Dashing -uri "http://<IP_ADDRESS>:3030/widgets/welcome" -text ”New message from Powershell” -authToken "Powershell"
.INPUTS
$uri,$text,$user
.NOTES
All Inputs are Mandatory
#>
[CmdletBinding()]Param
(
# Incoming Webhook
[Parameter(Mandatory=$true
)]
$uri,
# Body of message
[Parameter(Mandatory=$true
)]
$text,
# Username to post as
[Parameter(Mandatory=$true
)]
$authToken
)
$Payload = @{auth_token=$authToken;current=$text}
Invoke-RestMethod -Uri $uri -Method Post -ContentType 'application/json' -Body (ConvertTo-Json $Payload)
}
#Import Credential
$user = ''
$File = ''
$creds =New-Object -TypeName System.Management.Automation.PSCredential -ArgumentList $User, (Get-Content $File | ConvertTo-SecureString)
$vcenters = ('')
$vms = @()
foreach ($vcenter in $vcenters) {
Connect-VIServer $vcenter -Credential $creds
$vms += Get-VM | Where-Object {$_.PowerState -eq "PoweredOn"}
Disconnect-VIServer -Confirm:$false
}
$totalCPU =
$authToken = 'YOUR_AUTH_TOKEN'
$uri = ''
Postto-Dashing -uri $uri -text $vms.Count.ToString() -authToken $authToken`

For the Graph widget:
https://github.com/Smashing/smashing/wiki/How-To:-Setup-a-Graph
$DBURL = "http://dashboardserver:3030/widgets/"
$metric="yourmetric"
$cpuArray = @{x=1980;y=1323},@{x=1981;y=53234},@{x=1982;y=2344}
$jArray = ConvertTo-Json $cpuArray
$body = '{ "auth_token": "YOUR_AUTH_TOKEN", "points": ' + $jArray + ' }'
Invoke-RestMethod -UseBasicParsing $DBURL$metric -ContentType "application/json" -Method POST -Body $body


Highlight tiles that were just played

This is a very rudimentary solution to #4.
It adds a class played to the tiles on add_piece and then removes the played class from all tiles on on_move_accepted and on_played.
The code is probably not good enough given my very limited knowledge of Rust, but it might be of some help.

Updated + merged in fd69a0159cc0501f6d3a9f5cbd25dbdcf56854ba, thanks!
I ended up changing the color to make it more neutral, although I'm still not 100% happy with the color scheme – needing to distinguish between active + played + placed while not messing with the primary colors is tricky!


[FEAT REQ] Better way to render custom content based on the active anchor element

First of all, thanks for the improved v5 version. I am currently trying to implement it and it looks really good.
However, I noticed a difference to the v4 version that I cannot seem to find a workaround for:
Written in your documentation: https://react-tooltip.com/docs/examples/children
◕‿‿◕
<Tooltip id="my-tooltip">

This is a very interesting header

Here's some interesting stuff:

Some

Interesting

Stuff

</Tooltip>

Starting from this example, I would like to display the content of the hovered element inside this child component. Meaning something like this:
◕‿‿◕
<Tooltip id="my-tooltip">

This is a very interesting header

Here's some interesting stuff:

Some

{CONTENT}

Stuff

</Tooltip>

This basically allows a complete custom Tooltip by using the ReactTooltip as a kind of wrapper.  So, the display logic is used only inside the children.
In the v4 edition, I did it similar like this:

I added getContent={handleContent} to the ReactTooltip
Each anchor got a "data-tip" attribute
The handleContent method sets a react state (useState) which the children accesses.

Taking the example above, this becomes something like this:
const [tooltipContent, setTooltipContent] = useState('');
const handleContent = dataTip => {
        if (!dataTip || tooltipContent === dataTip) {
            return "";
        }
        setTooltipContent(dataTip)
    }

◕‿‿◕
<ReactTooltip id="my-tooltip"  content={handleContent}>

This is a very interesting header

Here's some interesting stuff:

Some

{tooltipContent}

Stuff

</Tooltip>

This worked great until v5.   The handleContent method is not called for each hover event on an anchor, just for some, otherwise just the plain data-tooltip-content data is displayed.  And this also is somewhat consistent to your documentation:

The tooltip children have lower priority compared to the content prop and the data-tooltip-content attribute. Useful for setting default content

But there has to be a way to use a dynamic children, not only for placeholder or default content, right?

Why do I want to use something like that:
I have a complex Tooltip component and a lot of data.  There are multiple DOM elements that show the exact same tooltip. So I could add the same data-tooltip-content to each repetitive element, which is a memory issue if the data-tooltip-content data is large.
The solution is:

Create a map like this:

{"entry1":"long content for entry1"}
{"entry2":"long content for entry2"}
{"entry3":"long content for entry3"}

Inside the data-tooltip-content, reference only the keys of the map, not the values:
anchor text

Store the map key ("entry1") inside the react state using the "content" (getContent) method of ReactTooltip

Inside the children, access the react state and translate the key to the content (map["entry1"])

Thanks for the suggestions!
Wrote a rough sketch over at #950 of how this could work. I had the idea of doing something like that in the past, but never came around to it. Might go over it next week to figure out the details. Any further suggestions are appreciated.

This is how I currently use v4. Is there any update as to when this will be available?

@wrgoto
We were just missing the docs. You can check #975 for a basic example on how to use it, and test it out with the beta release yarn add<EMAIL_ADDRESS>Let us know if everything works as expected, so we can proceed with the official release.

@gabrieljablonski, much appreciated! I'll upgrade and report back.

@gabrieljablonski Getting an error on our production environment while the beta works locally in development.
TypeError: (0 , a.jsx) is not a function
    at tS (react-tooltip.esm.min.js:1)
    at lb (react-dom.production.min.js:167)
    at i (react-dom.production.min.js:290)
    at uI (react-dom.production.min.js:280)
    at react-dom.production.min.js:280
    at uM (react-dom.production.min.js:280)
    at ux (react-dom.production.min.js:271)
    at uS (react-dom.production.min.js:268)
    at x (scheduler.production.min.js:13)
    at MessagePort.R (scheduler.production.min.js:14)

@wrgoto are you using nextjs 13? this is a known issue. check #972


Microsoft.Azure.Management.Storage CloudException: The 'Authorization' header is missing

I'm trying to create storage accounts programatically using the Microsoft.Azure.Management.Storage package along with a subscription management certificate, however I'm getting strange issues with authentication that leads me to believe the package is broken.
Here's how I'm trying to create the account:
var storageClient = new StorageManagementClient(new CertificateCloudCredentials(subscriptionId, managementCert));

var storageParams = new StorageAccountCreateParameters(accountType, location);
var createResult = await storageClient.StorageAccounts.CreateAsync(resourceGroup, accountName, storageParams);

I get the following error: Hyak.Common.CloudException: AuthenticationFailed: Authentication failed. The 'Authorization' header is missing.
StackTrace:
"One or more errors occurred." ---> System.AggregateException: One or more errors occurred. ---> Hyak.Common.CloudException: AuthenticationFailed: Authentication failed. The 'Authorization' header is missing.
   at Microsoft.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at Microsoft.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccess(Task task)
   at Microsoft.Azure.Management.Storage.StorageAccountOperations.<CreateAsync>d__29.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()

I've triple checked that my management cert is the correct one. Any ideas as to why I'm getting this? I would expect if I used the incorrect certificate I would get an access denied error. The error above leads me to believe the SDK didn't even include an auth header for some reason.

@tyjen
Which version of SDK do you use?
Do you might to upgrade toe the latest version of SDK and try again? https://www.nuget.org/packages/Microsoft.Azure.Management.Storage/7.2.0-preview

I also got this error. I then updated to the latest SDK (8.0-preview) and got a very similar error:
Microsoft.Rest.Azure.CloudException: 'Authentication failed. The 'Authorization' header is missing.'

Turns out this isn't meant to work. The error message could be greatly improved...

@Porges ,@tyjen
You may try obtaining a token based cloud credentials from the certificate. We are using AD App to do operations which has certificate associated. Below goes the code for HDInsightManagementClient. It may work for your scenario too.
        internal static async Task<HDInsightManagementClient> Get(string subscriptionGuid)         {             string token = await TokenFactory.GetAccessTokenUsingADAppCertificate("https://management.core.windows.net/");             TokenCloudCredentials tcc = new TokenCloudCredentials(subscriptionGuid, token);             return new HDInsightManagementClient(tcc);         } internal class TokenFactory     {         private static AuthenticationContext context = new AuthenticationContext($"https://login.windows.net/{Configurations.TenantId}"); //Tenant Id from portal         private static Lazy<IClientAssertionCertificate> _certProvider = new Lazy<IClientAssertionCertificate>(() => GetClientAssertionCertificate(Configurations.AzureADApplicationId));         internal static async Task<string> GetAccessTokenUsingADAppCertificate(string resource)         {             var result = await context.AcquireTokenAsync(resource, _certProvider.Value);             return result.AccessToken;         }         internal static IClientAssertionCertificate GetClientAssertionCertificate(string clientId)         {             string certIssuerName = Configurations.GetConfigValueByKey("CertificateIssuerName");             X509Certificate2 clientAssertionCertPfx = CertificateHelper.FindCertificateByIssuerName(certIssuerName);             return new ClientAssertionCertificate(clientId, clientAssertionCertPfx);         }     }

@tyjen , @Porges
Is your problem resolved? Do you still need any assistance from our team?

Yes, I believe the library should error-out early with an informative message. Passing CertificateCredentials to the StorageManagementClient simply can't work.

@Porges
Thanks for the reply!
Would you please confirm if your problem is resolved by "obtaining a token based cloud credentials from the certificate" ?
And since the code is auto generated by autorest from swagger, I will contact autorest team to see if they have any idea on this kind of issue, after the root cause if confirmed.

Closing per @blueww 's comments


[FLINK-36687][cdc-pipeline-connector/doris] Improve Doris Pipeline Connector Version to 24.0.1

Improve Doris Pipeline Connector Version to 24.0.1, Related release note https://github.com/apache/doris-flink-connector/issues/477
https://github.com/apache/doris-flink-connector/issues/499

I've bumped doris-flink-connector version in #3691 to support altering columns' type with default value. Would @JNSimba please take a look, too?


feat: migrate from janky custom manifest format to yaml

Implementation of #75.
Needs some cleaning up and implementation of all features

[ ] add attribute for non-standard plugin paths in repo
[x] add attribute for pinning plugin to specific commit
[x] Add tag that adds a warning to the plugin
[x] possibility to mark plugins as deprecated

Currently does not generate all plugins

should be ready now


Spanish layout is not added

Add the spanish layout to nationalizer

Hi
I don't have a spanish keyboard
Can you help and tell me which keys to replace?
Also, it's easy to add another language
Want to create pull request?


Doc: Add limitation file

Add LIMITATIONS.md to hold the limitations.

Coverage remained the same at ?% when pulling f05654db50c4f363334eb67f5814cd90f3d2257c on cathay4t:limitation into 0ef2e0f151661bc2bf981099a33c540a571111c6 on nmstate:master.

| I am not that sure why we need to manage this file here, we could have just pointed to a Jira report with appropriate filters.
Although the question still stands, I now see that it was one of my suggestions some time back.
I am just asking for a reasoning to this which will explain why we add it.
My main concern is that it will be forgotten and not removed/managed.

New patch uploaded.
Changes:

Add the Limitations section to README.md file with link to (https://nmstate.atlassian.net/issues/?jql=project %3D NMSTATE AND labels %3D Limitation ORDER BY priority DESC)


As a user, when I request a password reset, I should receive a reset email.

Alternatively, as an admin I should be able to directly reset/change a user's password.

This is tentatively fixed pending any further problems for users.


Add support for React 16

The library assumes that the componentDidUpdate method has three parameters. But the last one (prevContext) has been removed in the React 16.
Changelog: https://reactjs.org/blog/2017/09/26/react-v16.0.html#breaking-changes "componentDidUpdate lifecycle no longer receives prevContext param"
This should fix the problem with missing param...

Have you tried using this version in the Avocode app? This is modifying the core feature of the component.


Avoid errors if vendor-prefixed APIs do not exist (fix #2437)

Avoid errors in the generated vendor-prefix code if neither the
standard API nor the vendor-prefixed API exists. Instead, fall back
to undefined, which will give the Rust module a chance to run and
handle the error when it tries to use the API.
Fixes #2437.

Seems reasonable to me, thanks! Could you be sure to add a test for this as well?

Seems reasonable to me, thanks! Could you be sure to add a test for this as well?

Done!

Done!


Unnecessary join: foreign key same as property name

Describe the bug
I am migrating a project where the foreign key in the database is not suffixed with Id. For example, the database table looks like this:
CREATE TABLE "Profile" (
  "id" TEXT NOT NULL   ,
  "user" TEXT    REFERENCES "User"(id) ON DELETE SET NULL,
  PRIMARY KEY ("id")
)

The problem is that when you query this class like so:
em.find(Profile, { user: { $in: [userId] }, })

The code below in ObjectCriteriaNode will trigger. It will see that user is a non-scalar field and join in the User table, even though I'm really just trying to query on an indexed foreign key that happens to also be called user in the database.
export class ObjectCriteriaNode extends CriteriaNode {

  static create(metadata: MetadataStorage, entityName: string, payload: Dictionary, parent?: CriteriaNode, key?: string): ObjectCriteriaNode {
    const node = new ObjectCriteriaNode(metadata, entityName, parent, key);
    const meta = metadata.get(entityName, false, false);
    node.payload = Object.keys(payload).reduce((o, item) => {
      // prop is defined because "user" is a defined relationship
      // as well as the name of the database field that holds the foreign key
      const prop = meta && meta.properties[item];
      const childEntity = prop && prop.reference !== ReferenceType.SCALAR ? prop.type : entityName;
      o[item] = CriteriaNode.create(metadata, childEntity, payload[item], node, item);

      return o;
    }, {});

    return node;
  }

You need to provide this information either via custom naming strategy, or manually in each property:
@Entity()
class Profile {
  @OneToOne({ joinColumn: 'user' })
  user?: User;
}

Btw that part of CriteriaNode is correct, if there is a problem, it will be in the processing part, not when creating. But first, you need to fix your entity definitions - if your db do not follow current naming strategy, nothing can work correctly.
If you have the database schema consistent, I'd suggest to go for custom naming strategy. Btw there is EntityCaseNamingStrategy that follows this convention on join columns, and afaik it is working good this way, so I don't think there is actually a bug here, it will be most probably just misconfiguration. Here is an example project that uses this strategy: https://github.com/lookfirst/mikro-example

I think you should be fine with the EntityCaseNamingStrategy, just noted you also have the table named Profile, not profile, which is exactly how that strategy behaves.
What EntityCaseNamingStrategy does:

table name is same as entity name (Profile -> Profile)
column name is same as property name (email -> email)
join column is same as property name (user -> user)
for M:N pivot tables it will use entity names (Ent1/Ent2 -> Ent1_to_Ent2)

To use it, simply provide it when initializing the ORM:
const orm = await MikroORM.init({
  // ...
  namingStrategy: EntityCaseNamingStrategy,
});

I am using the EntityCaseNamingStrategy and my db schema is consistent with it.
I am preferring to use ManyToOne over OneToOne because I want to minimize the unwanted join.
Here is the Profile class:
@Entity()
class Profile {
  @ManyToOne({ entity: () => User, fieldName: 'user', cascade: [Cascade.MERGE], nullable: true, })
  user?: User;
}

Here is my query call:
em.find(Profile, { user: { $in: [userId] }, })

And here is the query output:
select `e0`.* from `Profile` as `e0` left join `User` as `e1` on `e0`.`user` = `e1`.`id` where `e0`.`user` in (...)

I do not believe the join to User is necessary, and I notice that the query builder changes to add the User table in the code snippet I posted. It appears that in the reduce function, it loops through the properties and it sees that the queried field, user, is not a SCALAR and so it pulls in the User table.

Yeah it is non needed indeed. Will try to look into this next week, the auto joining feature is pretty new and there might be similar issues here and there...

Thank you, I would really appreciate that. You are correct that it seems to be autoJoin related. Let me know if I can help. When I'm done with all of these questions, we should be able to add a few more examples to the docs.

Any contribution to the docs is more than welcome :)

Fixed in 3.0.0-rc.1.

Just wanted to thank you (for the other issues too), I have everything working the way I want now!


update for android 13

Please double check and merge if possible.

please check again and let me know if all ok.