Known samples for AndroidEnableMarshalMethods=true

Android application type
.NET Android (net7.0-android, etc.)
Affected platform version
.NET 8 Preview 6
Description
We have turned off AndroidEnableMarshalMethods for .NET 8 due to too many issues surfaced during public testing.  We plan on resolving those issues so it can be turned on for .NET 9.
This is just an issue of known examples that fail at runtime when AndroidEnableMarshalMethods=true:

https://github.com/dotnet/android/issues/7876
https://github.com/dotnet/android/issues/7893
https://github.com/dotnet/android/issues/8147
https://github.com/dotnet/android/issues/9064
https://github.com/dotnet/android/issues/9115
https://github.com/dotnet/android/issues/9209
https://github.com/dotnet/maui/issues/16514
https://github.com/dotnet/maui/issues/15947
(maybe) https://github.com/dotnet/maui/issues/15829
(maybe) https://github.com/dotnet/maui/issues/16418

Just linking them here for us to test later in future releases.
Steps to Reproduce
See links above.
Did you find any workaround?
No response
Relevant log output
No response

@grendello
Why did you enable marshal methods by default in net9-pre6? #8925
Did you test the known samples before releasing this feature again?
My sample #8147 is broken in net9-pre6, and once again, I have to manually disable marshal methods.

@tranb3r This is a very useful feature which gives a noticeable performance increase for huge majority of apps.  The only known bad scenario we had was with Blazor apps, and those disable marshal methods by default.  We enabled them in preview precisely for the reason of finding what else breaks and add it to the "disable" list if we can identify the issue.  Note that marshal methods themselves are fine, the bug doesn't lie in their implementation as far as we can tell (we've had them around for nearly two years, they essentially implement in native code what is implemented in managed code when they are disabled) - the sole fact that they are that much faster triggers a race condition (at least in Blazor) of some sort.  We haven't been able to determine what exactly trips up Blazor.  If your sample has a pattern we can identify, we'd add it to the "disabled" list.
For the gain in 90% (or more) of apps out there, it is worth enabling the feature by default and disable it explicitly (or automatically, if the pattern is identified as explained above) for the remaining 10% (or less) apps.

@tranb3r as far as testing is concerned, we have hundreds of thousands of tests that run on CI and none of them fail with marshal methods.  Without enabling the feature by default, we won't ever be able to gather information about what doesn't work "out there".

@tranb3r have you seen the problem with anything other than Plugin.Fingerprint?

For the gain in 90% (or more) of apps out there, it is worth enabling the feature by default and disable it explicitly (or automatically, if the pattern is identified as explained above) for the remaining 10% (or less) apps.

Sure.
However, considering I'm facing at least 2 different issues in my app, caused by this feature, I'm a bit concerned.
Race consider usually occur with complex scenarios, that are difficult to reproduce and integrate in CI.

As far as testing is concerned, we have hundreds of thousands of tests that run on CI and none of them fail with marshal methods.

This is why I'm asking if the samples in this issue have been tested.
My sample is failing, so I suspect this one at least hasn't been investigated/tested.

Have you seen the problem with anything other than Plugin.Fingerprint?

Yes. ANR when doing http requests. However I haven't been able to create a simple repro for it.

This option should only be used in .net9 previews right? It is supposed to be somewhat broken on .net8? Because I have tried in the previous maui - sr release  and also had some issues, and disabled it.

@bcaceiro it was present in net8 but was off by default, correct.

For the gain in 90% (or more) of apps out there, it is worth enabling the feature by default and disable it explicitly (or automatically, if the pattern is identified as explained above) for the remaining 10% (or less) apps.

Sure. However, considering I'm facing at least 2 different issues in my app, caused by this feature, I'm a bit concerned. Race condition usually occur with complex scenarios, that are difficult to reproduce and integrate in CI.

Superficial simplicity is unfortunately par for the course in a lot of .NET.  One example of that is Blazor apps, another is async/await or LINQ.  There is an awful lot of asynchronous or simply complicated code in those features that can fail under some circumstances without any obvious signals as to what's going on.  Your sample uses an external service which definitely can introduce threading and/or race condition scenarios.  That said, it's much, much simpler than the Blazor "hello world" app that we were investigating.  I need to bring some tools up to date and hopefully your sample will give a clue as to what's failing and why.

As far as testing is concerned, we have hundreds of thousands of tests that run on CI and none of them fail with marshal methods.

This is why I'm asking if the samples in this issue have been tested. My sample is failing, so I suspect this one at least hasn't been investigated/tested.

The sad truth is that we don't have much time for manual testing.  However, product previews are exactly where we can hope people like you, early testers, will discover issues and let us know about them.  Alas, there are very few people following your footsteps and sometimes we learn about issues very late in the game.  If we discover that marshal methods cause more trouble than we thought, we're going to disable them by default before the release and hopefully re-enable for .NET 10 - the feature is too valuable to just drop it (and we have more optimizations in mind that build on top of it).

Have you seen the problem with anything other than Plugin.Fingerprint?

Yes. ANR when doing http requests. However I haven't been able to create a simple repro for it.
That's another scenario involving threads and synchronization, open for race conditions.  Hopefully all of them are caused by the same issue and maybe your sample app will help us pin point the problem and swat it for good.

Here's a significantly reduced repro for the deadlock: https://github.com/filipnavara/mm-deadlock-repro. There's no MAUI or Blazor, just 50ish lines of .NET Android code in an empty template.
Essentially it boils down to the .NET GC being dead-locked / live-locked in the stop-the-world stage. I have seen several pattern of this.
Just run it in Release mode and it will immediately freeze before getting to display the "Hello" page.
Once the app is in frozen state, the main thread is waiting in Android code in MediaPlayer.Prepare(). Meanwhile, the media player loads the data in separate thread which calls back StreamMediaDataSource.ReadAt. This intentionally allocates empty arrays to trigger GCs to make the repro deterministic. In real world this allocation will happen at the marshalling level for byte[] arrays and trigger the same code path. The GC will deterministically freeze in the stop-the-world stage. It seems to wait for one of the threads, either the one executing MediaPlayer.Prepare() (less likely) or a random Java thread pool thread not running any .NET code at the moment (more likely).
In an unreduced repro, when running under debugger, there was a native exception in art_quick_generic_jni_trampoline prior to getting into the broken state. This may have caused some corruption that ultimately puts the Mono thread list in inconsistent state. I haven't been able to verify if this also happens in the reduced repro due to some struggle with LLDB.

I have narrowed down the thread state transition issue to this:
09-29 09:38:40.655  7970  7985 E THREAD  : attaching 0xb4000073ec62c9a0
09-29 09:38:40.655  7970  7985 E THREAD  : registering info 0xb4000073ec62c9a0 tid 0x726597f730 small id 43
09-29 09:38:40.655  7970  7985 E THREAD  : [ATTACH][0x726597f730] STARTING . -> RUNNING . (0 -> 0)
09-29 09:38:40.655  7970  7985 E THREAD  : [DO_BLOCKING][0x726597f730] RUNNING . -> STATE_BLOCKING . (0 -> 0) mono_thread_info_suspend_lock_with_info
09-29 09:38:40.655  7970  7985 E THREAD  : [DONE_BLOCKING][0x726597f730] STATE_BLOCKING . -> RUNNING . (0 -> 0) mono_thread_info_suspend_lock_with_info
09-29 09:38:40.655  7970  7985 E THREAD  : [ABORT_BLOCKING][0x726597f730] RUNNING . -> RUNNING . (0 -> 0) mono_class_get
09-29 09:38:40.656  7970  7985 E THREAD  : [ABORT_BLOCKING][0x726597f730] RUNNING . -> RUNNING . (0 -> 0) mono_method_get_unmanaged_callers_only_ftnptr
09-29 09:38:40.656  7970  7985 E THREAD  : [ABORT_BLOCKING][0x726597f730] RUNNING . -> RUNNING . (0 -> 0) mono_threads_attach_coop
09-29 09:38:40.656  7970  7985 E THREAD  : XX: mono_threads_attach_coop_internal external 0 cookie 0x0

At that point the thread pool thread is attached to the runtime. Yet the very next call to mono_threads_attach_coop_internal, ie. entering the UnmanagedCallersOnly method, fails to identify the thread as external. This causes the transition cookie to be NULL, and exiting the call with mono_threads_detach_coop becomes no-op which leaves the thread in a wrong state.

The sequence above is result of this:
https://github.com/dotnet/android/blob/3fbc4510b0ef4c36589cbccac0e118e9c0a20eaa/src/native/monodroid/xamarin-android-app-context.cc#L60-L71
get_function_pointer knowingly calls mono_thread_attach which attaches the thread pool thread to Mono runtime. It never detaches it, which leads to an incorrect state of the thread pool threads.

So, the simple fix is to replace mono_thread_attach with the deprecated mono_jit_thread_attach API. Unfortunately, mono_threads_attach_coop/mono_threads_detach_coop was never added to the public API headers and neither was mono_threads_enter_gc_unsafe_region/mono_threads_exit_gc_unsafe_region. These would be the proper way to fix it, I think.
Notably, mono_jit_thread_attach is still the way used by xamarin-macios.
cc @lambdageek


Add Uninstall CTT PowerShell Profile feature

Type of Change

[x] New feature
[ ] Bug fix
[ ] Documentation update
[ ] Refactoring
[ ] Hotfix
[ ] Security patch
[x] UI/UX improvement

Description
Implements the ability to uninstall the CTT PowerShell Profile and all its related dependencies.
Testing
I have tested the profile uninstallation functionality by running it in the following test scenarios:

Baseline: Installing the CTT PowerShell Profile
Test 1: Uninstallation of OhMyPosh dependency: [✅] Successful
Test 2: Uninstallation of Nerd Fonts dependency: [❓] Conditional

Additional Context: See Notes for Test 2 in Additional Information.

Test 3: Uninstallation of Terminal-Icons dependency: [✅] Successful
Test 4: Uninstallation of Zoxide dependency: [✅] Successful
Test 5: Skipping uninstallation of in-use dependencies: [✅] Successful
Test 6: Uninstallation of CTT PowerShell Profile: [✅] Successful

Impact

Covered by ## Description, ## Testing, and ## Additional Information.

Issue related to PR

Resolves #2836

Additional Information
New Files Created

$PSProfile.hash: A file that stores the hash of the CTT PowerShell Profile; this is used to check if the profile is installed on the system.

Additional Context:

The path is derived from the $PSProfile variable with .hash being appended to the end of the file path.
Example Hash File Path: C:\Users\<user>\Documents\PowerShell\Microsoft.PowerShell_profile.ps1.hash.
This file will be cleaned up and removed when the CTT PowerShell Profile is uninstalled from the system.

$PSProfile.bak: A file that is used as a backup of the PowerShell profile prior to the CTT PowerShell Profile being installed on the system.

Additional Context:

The path is derived from the $PSProfile variable with .bak being appended to the end of the file path.
Example Backup File Path: C:\Users\<user>\Documents\PowerShell\Microsoft.PowerShell_profile.ps1.bak.
This file will be moved back to the original PowerShell profile's location when the CTT PowerShell Profile is uninstalled from the system.

Notes for Test 2

Note 1: Nerd Fonts won't always be installed due to an issue with the setup script for the PowerShell profile.
Note 2: If the font is not installed, the user will see the message: ===> Already Uninstalled: Nerd Fonts. <===.

Checklist

[x] My code adheres to the coding and style guidelines of the project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no errors/warnings/merge conflicts.

If the profile was installed using a previous version of winutil, it will say it's already installed. This is fixed by clicking 'Install CTT PowerShell Profile' first, then clicking 'Uninstall CTT PowerShell Profile'.

If the profile was installed using a previous version of winutil, it will say it's already installed. This is fixed by clicking 'Install CTT PowerShell Profile' first, then clicking 'Uninstall CTT PowerShell Profile'.

@tmcnulty387
I understand your concern but this isn't a bug, it's how the uninstallation functionality was designed, if the profile doesn't exist it defaults to saying it is already uninstalled, here is why:
When I run (Get-FileHash $PROFILE).Hash on my current PowerShell profile I get ED277A035A46B5CAF14568477B9F6107742FA99873B01CA2B627DEDB8DA3E2EB which represents the hash of the profile at that specific point in time, the file hash is directly based off of the content of the file. when the file content is changed, so is the file hash.
However, if I install Chris Titus Tech's PowerShell profile and run (Get-FileHash $PROFILE).Hash again I will get a completely different value: CE13DF5DD10B48713EE2EF6A48D98A8DA8F9E14890412F833D8FF4E430735855, as your profile has been replaced with the new one, again changing the file content changes the file hash.
If I run (Get-FileHash "$PROFILE.bak").Hash, you will see it is the same hash as my original PowerShell profile since it is copied to "$PROFILE.bak" and restored when the profile is uninstalled: ED277A035A46B5CAF14568477B9F6107742FA99873B01CA2B627DEDB8DA3E2EB.
If the profile is already uninstalled the hash will not match, hence why ===> Already Uninstalled CTT PowerShell Profile. <=== is shown, this same thing will happen if you don't have a PowerShell profile as the file hash it is comparing against will be empty.
As far as I know, there isn't a way to differentiate between the profile not being installed to begin with and being uninstalled, unless another contributor steps in and knows another way to check for this, this is how it has to be done.

Additionally, I would like to mention the changes I added to Invoke-WinUtilInstallPSProfile.ps1 only do two things:

It saves the hash of Chris Titus Tech's PowerShell profile to "$PSProfile.hash" to allow for checking if it's installed or not.
It creates a backup copy of your original profile before Chris Titus Tech's PowerShell profile is installed on the system.
At no point does Invoke-WinUtilInstallPSProfile.ps1 say the profile is already uninstalled, that takes place in Invoke-WinUtilUninstallPSProfile.ps1 as shown in the following code snippet:

# Check if profile is installed
$PSProfileHash = Get-Content "$PSProfile.hash"
if ((Get-FileHash $PSProfile).Hash -eq $PSProfileHash) {
    # Uninstall Chris Titus Tech's PowerShell profile and all its dependencies, then restore your original PowerShell profile.
} else {
    Write-Host "===> Already Uninstalled CTT PowerShell Profile. <===" -ForegroundColor Magenta
}

If the profile was installed using a previous version of winutil, it will say it's already installed. This is fixed by clicking 'Install CTT PowerShell Profile' first, then clicking 'Uninstall CTT PowerShell Profile'.

Hello, @tmcnulty387! I'm looking into this as we speak but while I do that: Can I have some more details about the issue?
Screenshots would be a big help if I could see what buttons you are clicking, as similar behavior is happening on my end.

Steps to reproduce:

Install the CTT PowerShell Profile using stable branch of winutil (irm "https://christitus.com/win" | iex)
Attempt to uninstall CTT PowerShell Profile using this PR

This can be fixed by first clicking on install CTT PowerShell Profile, then clicking on uninstall CTT PowerShell Profile, both using the PR Branch.

Once restarted, PowerShell should then have default settings properly restored.

Thanks for fixing the issue, @Cryostrixx! I tested it again by installing the CTT PowerShell Profile using the stable branch of WinUtil (v24.10.07). I was successfully able to uninstall it by using your PR. I'm glad my report was helpful.

Check out the Contributor Note section for details on why this pull request was closed. In short, I prefer not to keep a PR open that is unlikely to be accepted, as it clutters the pull requests section.
As a side note, I wish there was more communication regarding PR statuses. I understand that there are multiple PRs and that it can be challenging to address them all, but it occasionally feels like contributions from more well-known contributors are prioritized over contributions from smaller contributors. Even a simple acknowledgment of a PR could greatly enhance motivation and engagement, rather than leaving it with no activity other than the fixes that get pushed to the feature branch.

@Cryostrixx Can you reopen this? I'd like to commit to the project. Sorry it has taken a while to do the merge.

@ChrisTitusTech Sure, but it'll take me a while as I don't have a backup, I'll request your review when it's ready.

@ChrisTitusTech Alright, I remade the PR, I'm on Manjaro so I can't test it myself but it did compile successfully with PowerShell Core.
Please check out #3012.


Device Setup

Is it possible to setup a new device with this library?
So if I have a brand new Yeelight I would like to be able to tell it which SSID to connect to and to enable local access without requiring a Xiaomi Mi account or having to install the companion app to complete the initial setup process.
Is this supported, and if not, are you aware if this is possible to implement?

Any answer for that?
I know it can be done for setting up a xiaomi.repeater.v2 like so:
miio configure <IP> --ssid <SSID> --passwd <PWD>

which outputs:
INFO  Attempting to configure <IP_ADDRESS>
Updated wireless configuration

But haven't figured out to do it for philips.light.bulb to provide my router SSID and PWD settings


Unable to receive webhook notifications.

What happened.
https://pipecd.dev/docs-v0.21.x/operator-manual/piped/configuring-notifications/#sending-notifications-to-external-services-via-webhook
↑I set up webhook as per this page, but no notifications are flying.
How to reproduce.
Add the following to Pipecd.yaml
apiVersion: pipecd.dev/v1beta1
kind: Piped
spec:
notifications:
routes:
# Sending all events an external service.
- name: all-events-to-a-external-service
receiver: a-webhook-service
receivers:
- name: a-webhook-service
webhook:
url: https://test.com/hooks/abcdefghijklmn
Environment:

piped` Version.
v0.22.0
Control plane` version.
v0.22.0

Translated with www.DeepL.com/Translator (free version)

/assign @knanao

Hi @jo-jo-1217 , thank you for your repot.
Can you check the log of piped? and are there any error in it?

Hi @knanao
There is no error log in piped...

@jo-jo-1217
Oh, I see. Let me check this issue.
Just in cases, Could you please recheck outbound request?

When I hit the curl command, the text jumped without any problem.
The URL seems to be correct...

@jo-jo-1217 Thanks for your issue.
We have just checked this case but it worked normally as expected.
This is our Piped's configuration
  notifications:
    routes:
      - name: all-notifications
        receiver: webhook-service
    receivers:
      - name: webhook-service
        webhook:
          url: https://webhook.site/120d3814-c1e9-4b18-b197-4fc790afd6c4

And as you can see the event package has been delivered successfully.
https://webhook.site/#!/120d3814-c1e9-4b18-b197-4fc790afd6c4/524adebe-420b-486f-b936-b63b5c7d314a/1
So please recheck your configuration or try to update to use a newer version of PipeCD.


Output - Dynamic Imgui widgets

Current track: generate GIF format when at least one dynamic option is set to true.

Removing CLI option in favor of dedicated Imgui widgets


fix: proper TypeScript support for graphql-sse client

Screenshot

Expected Behaviour
Should allow me to set to true
Actual Behaviour
TypeScript forces me to keep it false
Debug Information
Editing the type definition to the one bellow fixes the issue:
export interface ClientOptions<SingleConnection = boolean extends boolean = false>

typed client without the use of any. somebody explains please why one would prefer any.
import {
  createClient,
  Client,
  ClientOptions,
  Sink,
  ExecutionResult,
} from "graphql-sse";
import { TypedDocumentString } from "./codegen/graphql";

export interface TypedClient
  extends Omit<
    Client,
    "iterate" | "subscribe"
  > {
  iterate<TResult, TVariables extends Record<string, unknown> | undefined>(
    document: TypedDocumentString<TResult, TVariables>,
    variables?: TVariables
  ): AsyncIterableIterator<ExecutionResult<TResult>>;
  subscribe<TResult, TVariables extends Record<string, unknown> | undefined>(
    document: TypedDocumentString<TResult, TVariables>,
    variables: TVariables,
    sink: Sink<ExecutionResult<TResult>>
  ): () => void;
}

export function createTypedClient(options: ClientOptions): TypedClient {
  const client = createClient(options);
  return {
    ...client,
    iterate: (document, variables) =>
      client.iterate({
        query: document.toString(),
        variables,
      }),
    subscribe: (document, variables, sink) =>
      client.subscribe(
        {
          query: document.toString(),
          variables,
        },
        sink
      ),
  };
}

I think there should be an options to have the actual event abject at hand, so you can check for the connection status and stuff like that.

Following the simplest example, I created a TypeScript version. Here an event object and a set callback function are returned.

export function subscribeOperation<TResult, TVariables>(p: {
  query: TypedDocumentString<TResult, TVariables>;
  variables?: TVariables extends Record<string, never> ? [] : [TVariables];
}) {
  let nextCallback: (result: ExecutionResult<TResult>) => unknown = () => {};

  const next = (callback: typeof nextCallback) => {
    nextCallback = callback;
  };

  const url = new URL("/graphql", window.location.origin);
  url.searchParams.append("query", p.query.toString());
  if (p.variables) {
    url.searchParams.append("variables", JSON.stringify(p.variables));
  }
  const source = new EventSource(url.toString());
  source.addEventListener("next", (event) => {
    const executionResult = JSON.parse(event.data);
    return nextCallback(executionResult);
  });
  return { source, next };
}

Example usage:
const { next, source } = subscribeOperation({
  query: HealthSubscription,
});
next(({ data }) => {
  console.log(source.OPEN);
  console.log(data?.health);
});

I don't plan to use SSE


feat: Carousel with OFF logo

Scan card with new logo and a few UI fixes

Codecov Report

Merging #2158 (5958ce5) into develop (2ea0da3) will decrease coverage by 0.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #2158      +/-   ##
==========================================
- Coverage     8.86%   7.90%   -0.96%
==========================================
  Files          161     177      +16
  Lines         6623    8640    +2017
==========================================
+ Hits           587     683      +96
- Misses        6036    7957    +1921

Impacted Files
Coverage Δ

...kages/smooth_app/lib/widgets/attribute_button.dart
0.00% <0.00%> (-92.00%)
:arrow_down:

packages/smooth_app/lib/themes/smooth_theme.dart
53.48% <0.00%> (-29.50%)
:arrow_down:

...s/smooth_app/lib/data_models/user_preferences.dart
11.11% <0.00%> (-21.20%)
:arrow_down:

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
19.23% <0.00%> (-14.98%)
:arrow_down:

...mooth_app/lib/data_models/product_preferences.dart
26.47% <0.00%> (-4.96%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00% <0.00%> (-3.13%)
:arrow_down:

.../smooth_app/lib/pages/onboarding/scan_example.dart
0.00% <0.00%> (-2.28%)
:arrow_down:

packages/smooth_app/lib/main.dart
15.74% <0.00%> (-2.16%)
:arrow_down:

...s/smooth_app/lib/pages/onboarding/next_button.dart
2.22% <0.00%> (-1.78%)
:arrow_down:

...p/lib/pages/onboarding/consent_analytics_page.dart
0.00% <0.00%> (-1.57%)
:arrow_down:

... and 161 more

Continue to review full report at Codecov.

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

Ooops, forgot this case.

Note the splash screen uses similar assets

Not really.
Splash screens have their own Android/iOS assets. Here, we use SVG files without paddings

Yes… I know, but that's what is on Figma 🤷♂

Note-Figma does not have to be interpreted literally if it causes issues.
Let's merge and refine ?


npm release

https://github.com/SAP/chevrotain/commit/221eca55170ad9ffd0746fcec57ed7bc26640b52

https://www.npmjs.com/package/chevrotain


New version: Micron Storage Executive version 9.09.092023.03

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @drshtmstry,
I think it slipped past my automation, but this PR appears to remove Apps and Features entries that are present in previous versions. Should these entries be readded?


Return action taken and candle data from advice() function

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Allows return data including candle (id, open close etc....) as well as action taken returned from advice() function like so.... var tradeResult = advice(tradeDecision) where trade result will be an object accessible like so: tradeResult.action or tradeResult.candle['close']

What is the current behavior? (You can also link to an open issue here)

strats have no way of knowing what action was taken or exactly what candle it happened on when using advice();

What is the new behavior (if this is a feature change)?

A more straightforward and accurate (less prone to error) way of knowing exactly when and where a trade actually happens vs trying to sync your strat to what is really going on when advice() is processed

Other information:

I don't fully understand the need for this: The advice will always be emitted if it was "short" or "long" (and if it wasn't the same as the previous advice). So the only case in which it gets ignored is when you advice SHORT - SHORT or LONG - LONG, if you really want to know when this happens, why don't you track it yourself in your strat?

thats exactly my point. we can eliminate the need to track it ourselves in our strats and allow us to keep our code simpler and cleaner. also saying that we could track it ourself in our strat is the same as saying we can have our strat decide if advice long should mean to advice long whether or not the last advice was long or short

You can already access this._prevAdvice in your strategy which is set to whatever the previous advice was.
Though I really don't think this is the way to go: a strategy can always advice, but it's up to Gekko whether it does something what that advice (ignore if duplicate, not enough money to make a trade, etc). And your PR only feedbacks whether Gekko decided to emit the advice or not, nothing more (like response from the trader plugin, etc). So the action prop doesn't actually tell you what action Gekko took (if any at all).

And that's another thing I wanted to add to it was return values for amount of asset or currency along with candle ID and action taken. Honestly I took a look and was like well for now it looks like we're only gonna be able to retrieve the result of gekkos decision after blocking a double buy or sell and the candle that it happened at with ID price etc. I realize that I can do all this in strat. But I think it's a start and should actually be expanded on.
What actually led to this for me is that I need to process data in strat and have the results in csv.  And this would actually eliminate the problem altogether for me personally. If I could have processed data emitted as api output. Then I wouldn't be running the command line backtest I would just post to the api and get the result and it would have everything the buy sell candle id's prices indicator data... And the output I need from data processed in strat.
If you don't want to accept the PR then ok I am fine with it. I agree it can be done in strat. But if you can point me in the right direction with how can I go about exporting my data for each candle through the api that would actually be even better and I would care less about returning data from advice function

This is a good change to make.

@generalectric I am pretty sure we want to end up with the same goal:
The strategy wants to know what happened to the advice: if a trade was executed and if so at what time/price. The good thing is that most of this is already present: after a trade an event is emitted, we only need to expose this to the strat and you should have what you want.
Because do you really care WHY no action was taken? because either duplicate action or not enough funds on account? I think you just want the result.
The thing is that the real trader needs a while to figure this out: on new signal it needs to check exchange balances, try to add a limit order, wait until it gets taken, if not re issue, etc. This takes time and thus means we cannot return the action from this.advice (because by the time we propogate the advice through Gekko all of the above things are not yet known).
Let me know if I correctly understand what you are trying to do, because if that's the case this PR is not the way to achieve it.

Well explained @askmike, makes perfect sense. And yes you understand correctly, and now I understand as well. I was only thinking about backtesting, I wasnt at all thinking about live trading. So I agree and like your idea better. However.... can we find a way to emit custom data from a strat to the API? Im still not familiar enough to figure out a way to do this.... but cant we just reserve an object designated for custom data... like customdata:{} that can be retrieved through the api?

It is an important goal for Gekko to behave the same when backtesting as in
the live market (else what is the point of the backtesting).
It will take some time to hook it up, but the best way would be for
strategies to register a new method called 'processTrade' where they can
handle a trade event coming from the trader. All the logic is actually
already there in most of gekko (the code that calculates statistics already
implements this processTrade).
On 28 Oct 2017 12:52, "generalectric"<EMAIL_ADDRESS>wrote:

Well explained @askmike https://github.com/askmike, makes perfect
sense. And yes you understand correctly, and now I understand as well. I
was only thinking about backtesting, I wasnt at all thinking about live
trading. So I agree and like your idea better. However.... can we find a
way to emit custom data from a strat to the API? Im still not familiar
enough to figure out a way to do this.... but cant we just reserve an
object designated for custom data... like customdata:{} that can be
retrieved through the api?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/pull/1239#issuecomment-340141586, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD2osJEsmEG-_4i99rh9GH_LlL5Mbks5swsEsgaJpZM4QCBBS
.


Question about similar application

Hey there,
Cointop is great, I use it to output my data to file then grab it with Conky to display in my system stats panel. I was wondering if there happens to be something similar that anyone knows about for stocks? I know there are a few terminal based stock tickers and such, but I can't seem to locate any that allow you to have listed holdings and then output straight to a file without having to keep that app up and running at all times. If anyone knows of any, I would definitely appreciate it.
At the same time, I know this post is not on topic, so if you want/need to close/remove it, I certainly understand.
Thanks,
-MH

Seems like there should be more straight-forward ways of grabbing data from a source, and writing it to a file (would only be a few lines of code).   Each exchange typically has an API, which requires (relatively simple) code to fetch and save.  If you know of a public API for stocks, it shouldn't be too hard to find someone to write that for you.
Cointop mostly uses https://www.coingecko.com/ for data, which only supports crypto.

Heya, I appreciate the reply. Yeah, for sure, I would imagine there are a good number of api's to hit, I was just wondering if something happened to exist already, it definitely seems like there would be, but perhaps not.
Honestly, it's not a big enough concern to go create a whole new thing to do it, but, if there had happened to be something that already existed that would allow me to borrow from what I had already done with cointop, I would be all about it.

I appreciate the ideas, this ended up working out perfectly. https://github.com/pstadler/ticker.sh

Cool.  Looks pretty minimal!  Close this issue if you're satisfied.


Remove online tarball build from CI in favor of enforced offline build.

Skip CI please.

@dotnet-bot test CI please.

@dotnet-bot test CI please.

@dotnet-bot test CI please.


bad rough state

caught TypeError: Cannot read properties of undefined (reading 'type')
    at p (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/roughjs@4.5.2/node_modules/roughjs/bundled/rough.esm.js:5:4728)
    at f (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/roughjs@4.5.2/node_modules/roughjs/bundled/rough.esm.js:5:5217)
    at eval (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/roughjs@4.5.2/node_modules/roughjs/bundled/rough.esm.js:5:20184)
    at N.path (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/roughjs@4.5.2/node_modules/roughjs/bundled/rough.esm.js:5:20690)
    at B.path (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/roughjs@4.5.2/node_modules/roughjs/bundled/rough.esm.js:5:24153)
    at CanvasRenderingContext2D.fill (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chartjs-plugin-roughness@1.0.4_chart.js@4.3.0/node_modules/chartjs-plugin-roughness/context2d.patch.js:62:24)
    at chart_js__WEBPACK_IMPORTED_MODULE_1__.BarElement.draw (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chartjs-plugin-roughness@1.0.4_chart.js@4.3.0/node_modules/chartjs-plugin-roughness/bar.element.patch.js:26:7)
    at BarController.draw (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:1678:26)
    at Chart._drawDataset (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:6145:25)
    at Chart._drawDatasets (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:6120:18)
    at Chart.draw (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:6091:14)
    at eval (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:115:23)
    at Map.forEach (<anonymous>)
    at Animator._update (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:93:22)
    at eval (webpack-internal:///(:3000/app-client)/./node_modules/.pnpm/chart.js@4.3.0/node_modules/chart.js/dist/chart.js:84:18)

catched and logged by #115


Rerendered the feedstock with version 1.0.

An attempt to fix #12.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

😢
The following NEW packages will be INSTALLED:
    anaconda-client: 1.4.0-py27_0   defaults
    clyent:          1.2.1-py27_0   conda-forge
    conda:           4.1.6-py27_0   defaults
    conda-build:     1.21.5-py27_0  defaults
    conda-build-all: 0.13.1-py27_0  local
    conda-env:       2.5.1-py27_0   defaults
    funcsigs:        1.0.2-py27_0   conda-forge
    gitdb:           0.6.4-py27_0   conda-forge
    gitpython:       1.0.1-py27_0   conda-forge
    jinja2:          2.8-py27_0     conda-forge
    markupsafe:      0.23-py27_0    conda-forge
    menuinst:        1.4.1-py27_0   defaults
    mock:            2.0.0-py27_0   conda-forge
    patch:           2.5.9-1        defaults
    pbr:             1.10.0-py27_0  defaults
    pip:             8.1.2-py27_0   defaults
    psutil:          4.3.0-py27_0   defaults
    pycosat:         0.6.1-py27_0   conda-forge
    python:          2.7.12-0       defaults
    python-dateutil: 2.5.2-py27_0   conda-forge
    pytz:            2016.3-py27_0  conda-forge
    pyyaml:          3.11-py27_0    conda-forge
    requests:        2.10.0-py27_0  conda-forge
    ruamel_yaml:     0.11.7-py27_0  defaults
    setuptools:      23.0.0-py27_0  conda-forge
    six:             1.10.0-py27_0  conda-forge
    smmap:           0.9.0-py27_0   conda-forge
    vs2008_runtime:  9.00.30729.1-2 defaults
    wheel:           0.29.0-py27_0  defaults
    yaml:            0.1.6-vc9_0    conda-forge [vc9]
[conda] C:\conda\conda-bld\test-tmp_dir>call activate _test
******  B A T C H   R E C U R S I O N  exceeds STACK limits ******
Recursion Count=699, Stack Usage=90 percent
******       B A T C H   PROCESSING IS   A

One question, otherwise LGTM.


Better Visualizer

Improved Smoothing and New Smoothing Customization.
Hi @MarcoPixel,
It's been a while. As you promised, I've twerk the spectrum. I also made a small change in visualizer and some changes in my MeasureBandsSmoothed1.inc. I'd made some coding mistakes before in my MeasureBandsSmoothed1.inc, but it doesn't affect the spectrum.
Next, this idea was made for those who wanted to customize the smoothing. I've made a new file - MeasureBandsSmoothed2.inc. This file may be slightly same as the default one, but still using my multipliers and divider for customization.
What do you think? I hope you like the changes and my idea. That's all. Thanks.

Unless I am missing something I don't seem to see any change with the version I just pulled from your repo. I'll try playing around with it to see what I am doing wrong.
Any chance you could make a gifv comparison between this one, the old one, and the one in the videos?

Unfortunately, I can't make gifv. But, I got some screenshots.
The mistakes shown in this picture (supposedly 2 split pictures but I merged it):

I could do this kinda comparison, but doing all at once, my laptop is done:

Hey @Matter-Shifter !
I've cloned your branch, and it looks a bit better... Still some stuff to change !
I've made a comparaison, will upload and edit.
Also, I can't make the dynamic genre colors working... It worked well with the unmodified visualizer,  but now it is always showing the base color. I don't see where is the problem there.

Hey, it's been a while.
I did some changes in my branch(more twerking), and I did a video to compare both better and stock smoothing. Here. I hope you satisfied with the result.
The changes that I did may consume more memory and use more cpu.

The changes that I did consume more memory and use more cpu
The changes are great but I think we've reached a point where even mid-budget CPUs will be struggling. My i7 is working but fps not stable. I've found that using a value between 6000-7500 is a good balance. I'm using 7100 for now.

Not to sound rude but there is no reason to take a CPU hit after all the past and ongoing attempts to reduce it, especially since it is only marginally closer to the effect in the videos.
This is just not something we want compromise on to add to the project. Even if we did take the CPU and latency hit to try to replicate it will be almost impossible to replicate the visualizer from the videos since we have to do it in real time.

Maybe. But you should document FFTSize in the settings and explain to the user the effects of it. Also I'll stay with the old version from Matter-Shifter which was better while not consuming as much CPU.


Feature Request: Let's turn down the frequency on the notifications when an update to the app is available

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Install Expensify app
Open and keep in background
Allow notifications & wait for new build

Expected Result:
User should see new build notification with daily or once per 8hrs
Actual Result:
Notifications are sent every hour
Workaround:
unknown
Platform:

Where is this issue occurring?

Desktop App

Version Number: 1.1.37-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @deetergp
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1643828529185399
View all open jobs on GitHub

Triggered auto assignment to @nickmurray47 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Seems like a reasonable request! Going to put back into pool while I work on other items


PreConditions for Commands

What we have already agreed on:

To provide a Levels system, and a replacement for IPermissionCheckers

What I would recommend:

Having the checks be done before CommandService.Execute, not as a part of, so Users have flexibility in that regards. Maybe as a part of CommandService.Search?
Making use of our DI, again, for the IPermissionChecker's, providing IMessage and IDependencyMap

Question: Will there be a built-in help command again, and if so, a way to hide certain commands from it?
Also, it seems that "precondition" is a single word (no hyphen or anything), so I would suggest to have a lower-case 'c' as is convention.

I say that we introduce another attribute, similar to MVC; AuthorizeAttribute. This attribute would have a get/set property for level required which the commands system will detect and honor.
I'm thinking it would look something like this:
class AuthorizeAttribute : Attribute
{
    public int MinLevel { get; set; }
}

Another option would be to have an attribute ('FilterAttribute'? Any other names?) which developers create custom attributes from to add custom 'preconditions'. The attribute could look something like this:
class FilterAttribute : Attribute
{
    public void OnCommandExecuting(CommandExecutionContext context)
    {
        // something here
        // set context.Handled = true to ignore command
    }
}

As for the help command, I believe that should belong in its own issue as it is not entirely related to preconditions.

While an Attribute would be consistent with the design of 1.0 Commands, aren't those entirely limited to static content, because they get compiled as metadata? That would preclude any kind of runtime checks.

As for the help command, I believe that should belong in its own issue as it is not entirely related to preconditions.

Maybe so, but the built-in help command in 0.x was pretty smart in that, if a command was hidden or a check failed, it wouldn't be displayed in the list to the user. I think that's a highly desirable feature, and there should at least be some consideration to make the same behavior possible, if not for a built-in command, then for bot authors who may want to roll their own help.

While an Attribute would be consistent with the design of 1.0 Commands, aren't those entirely limited to static content, because they get compiled as metadata? That would preclude any kind of runtime checks.

ASP.NET uses attributes for stuff like this; so it is definitely possible. The code would look something like this:
var ctx = new CommandExecutionContext();
var attrs = method.GetCustomAttributes<FilterAttribute>();
foreach (var attr in attrs)
{
    attr.OnCommandExecuting(ctx);
}

if (ctx.Handled)
    return;

As for your help command, we could provide an addon to provide a help command which respects these attributes. An extension method for the CommandService ('UserMeetsPermissionsRequirements') could be added to make this easier.


pango: Trust upstream nullable annotations for return values

Part of #789.

Updated!

For moving forward here, I would suggest you add overrides for the wrong nullable cases to Gir.toml. Then the API here is fixed at least and we can get rid of the overrides once we have the new .gir file

regen check is unhappy


Fix typo in README.md

Summary
I fixed a typo in README.md (lone parenthesis after "Qiskit Ignis").
Details and comments

 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.


Added pause screen

Added a pause screen with text that displays "Paused".

Pressing Down (crouch) + A (jump) pauses. Currently crouching and jumping does nothing so it was a good candidate for avoiding misclicks
Pressing A or B unpauses

Good initiative! Actually it made my think that current behaviour of down + A is wrong. It should make the player jump even if he is crouching. But well.. it has never been a problem so let it be the pause option.

Yeah. I have been missing my crouch jump. I wonder if it would make that first boss easier. Depends if the crouch lifts the hitbox in the air or lowers it.
If I or you bother to add it, we can probably find a different mapping (Hold A + Press Up is my fallback)


Upgrade MAY break postgres container

Description:
This project was initially set up with postgres:latest in the Docker configuration. Recently, PostgreSQL updated from version 16 to 17, and this upgrade is not seamless. The data directory initialized by PostgreSQL 16 is not compatible with PostgreSQL 17, causing compatibility issues.
Temporary Solution:
As a temporary fix, I have pinned the PostgreSQL version to postgres:16 in the Docker setup. This will prevent unexpected upgrades from breaking the container.
For Users Already on PostgreSQL 17:
If your setup is already using PostgreSQL 17, you’ll need to edit the docker-compose.yml file to ensure the PostgreSQL version matches the data directory you’re using.
Feel free to reach out if you encounter any further issues related to this upgrade.

This is an old issue from 2014 https://github.com/docker-library/postgres/issues/37
Other related issues:
https://github.com/docker-library/postgres/issues/157
https://github.com/docker-library/postgres/issues/1125

It is old, but still very relevant. I was just wondering why my app stopped working.
It would be very convenient that the data migration would be done automatically. An approach similar to https://github.com/pgautoupgrade/docker-pgautoupgrade would be desirable so that we users don't have to worry about any updates :)

It is old, but still very relevant. I was just wondering why my app stopped working. It would be very convenient that the data migration would be done automatically. An approach similar to https://github.com/pgautoupgrade/docker-pgautoupgrade would be desirable so that we users don't have to worry about any updates :)

I believe that this docker-pgautoupgrade is mentioned as a possible solution for this issue on docker-postgres. This should be fixed on postgres project.
This is out of the scope of routerfleet project. I don't feel comfortable orienting people to manually migrate their databases. There is a huge potential for data loss. For this moment, I will just lock the project to postgres 16 instead of latest.


R Markdown Challenges in RTVS

Hello ,
Trying to find some guidance on R Markdown in RTVS . I am unable to find any  resources with good examples . I had a pre-built R Markdown which I am unable to run / launch through RTVS . Can someone kindly provide documentation for using R Markdown in RTVS .
Best
Uday

RTVS uses same rmarkdown package and pandox/mklatex components. What exactly are you trying to do? Publish it as HTML? PDF?

@MikhailArkhipov  : I was trying to 2 types of reports

HTML
PDF

What I want to do is call a function that renders the rmarkdown  ?  Can you provide any examples that guide me in this direction
best
Uday

Right click -> Preview-> HTML
https://docs.microsoft.com/en-us/visualstudio/rtvs/rmarkdown


feat: wallet screen detail screen

Adds stuff for the wallet screen and detail screen. Activity is stil todo.
@janrtvld some things maybe you could help with:

showing toast in a modal screen (doesn't work atm -- needed for biometric failed toast)
allowing headers to be transparent / not sohwn / hidden on scroll

i think on wallet screen we don't want a header?
on the pid detail screen we want to hide it on scroll maybe?
how to style the credential attributes rows background color differently in paradym vs ausweis wallet?

https://github.com/user-attachments/assets/c20f351c-2de9-4473-8162-ec650e3edd7a

I'll take a look at it now.

showing toast in a modal screen (doesn't work atm -- needed for biometric failed toast)

Will look into this.

i think on wallet screen we don't want a header?

Ideally we do it when you scroll, the header will scroll under the top navigation and a smaller version appears in the top navigation. Like this: https://imgur.com/a/84iq0BP

on the pid detail screen we want to hide it on scroll maybe?

Hmm, I wouldn't bother for now. The current way has the back button always visible which I think is better UX.

how to style the credential attributes rows background color differently in paradym vs ausweis wallet?

Probably a 'theme' or 'variant' property on  which you use to style the TableContainer or TableRow with the correct colour.

You can render toast inside a modal screen by rendering <ToastViewport /> inside the screen component.

For after funke, we should probably check how we would like to display booleans. Because the age over thingy is quite weird unless you know the underlying data model.

For after funke, we should probably check how we would like to display booleans. Because the age over thingy is quite weird unless you know the underlying data model.

Worked, thanks!


Invalid invitation token until third view of invitation link

I have a rails 4 multitenant application using apartment, devise, devise_invitable, and postgres schemas. Versions are as follows...
Rails 4.1.5
devise 3.3.0
devise_invitable 1.3.6
Postgresql 9.3.5
I also have a publicly viewable git repo here: https://github.com/bbrstw/warehouse
I am able to create an invitation. Upon opening the invitation URL, the user is redirected to the sign in page with the flash error: The invitation token provided is not valid!
However, if I follow the link three times (i.e. follow the link once, paste the url into the browser and press enter to follow it again, and then repeat once more), on the third time I am directed to the complete signup page (devise/invitations/edit.html.erb), and I am able to accept the invitation and register. This happens every time, exactly on the third time each time. If I wait for a short while then follow the link again, I will again need to follow the link three times before finally being directed to the complete sign up page, as if a session times out and the click count resets or something.
I have tested this in Chrome, Firefox, Safari and Internet Explorer 10 all to the same effect. I have reproduced this on Mac OSX 10.9 and Windows 7 with the rails project at the git repo provided.
Here are some of the rails dev log read outs...
This block will appear twice with the first two times I follow the link (http://myaccount.lvh.me:3000/users/invitation/accept?invitation_token=sJ5d-ECNLqp38mi_sTmJ):
Started GET "/users/invitation/accept?invitation_token=sJ5d-ECNLqp38mi_sTmJ" for <IP_ADDRESS> at     2014-09-07 12:23:16 +0800
  ActiveRecord::SchemaMigration Load (2.2ms)  SELECT "schema_migrations".* FROM "schema_migrations"
Processing by InvitationsController#edit as HTML
  Parameters: {"invitation_token"=>"sJ5d-ECNLqp38mi_sTmJ"}
  User Load (4.4ms)  SELECT  "users".* FROM "users"  WHERE "users"."invitation_token" =  'b58b6e22ea9f3fe3d0b608fcb9910ee34b64dc3a809a13fe18d4d399631d7daf'  ORDER BY "users"."id"   ASC LIMIT 1
Redirected to http://myaccount.lvh.me:3000/users/sign_in
Filter chain halted as :resource_from_invitation_token rendered or redirected
Completed 302 Found in 837ms (ActiveRecord: 7.1ms)

Started GET "/users/sign_in" for <IP_ADDRESS> at 2014-09-07 12:23:17 +0800
Processing by Devise::SessionsController#new as HTML
  Account Load (1.4ms)  SELECT  "public"."accounts".* FROM "public"."accounts"  WHERE "public"."accounts"."subdomain" = 'myaccount' LIMIT 1
  Rendered devise/shared/_links.erb (1.7ms)
  Rendered devise/sessions/new.html.erb within layouts/application (54.6ms)
Completed 200 OK in 232ms (Views: 204.8ms | ActiveRecord: 4.9ms)

And then finally this block is logged upon successfully accepting token and serving the complete registration page...
Started GET "/users/invitation/accept?invitation_token=sJ5d-ECNLqp38mi_sTmJ" for <IP_ADDRESS> at 2014-09-07 12:23:26 +0800
Processing by InvitationsController#edit as HTML
  Parameters: {"invitation_token"=>"sJ5d-ECNLqp38mi_sTmJ"}
  User Load (755.8ms)  SELECT  "users".* FROM "users"  WHERE "users"."invitation_token" = 'b58b6e22ea9f3fe3d0b608fcb9910ee34b64dc3a809a13fe18d4d399631d7daf'  ORDER BY "users"."id" ASC LIMIT 1
  Account Load (0.3ms)  SELECT  "public"."accounts".* FROM "public"."accounts"  WHERE "public"."accounts"."subdomain" = 'myaccount' LIMIT 1
  Rendered devise/invitations/edit.html.erb within layouts/application (4.8ms)
Completed 200 OK in 784ms (Views: 23.0ms | ActiveRecord: 756.9ms)

Any ideas as to whether or not this is devise_invitable issue?

@scambra Facing the exact same issue as @L33tH4x0r . Any luck in resolving this?

I think only multi tenant apps have this issue. If anyone can reproduce with non multi tenant, it would help to fix it, if there is a bug in devise invitable not catched with tests, because all tests are passing.
With multi tenant, you may add some log to ensure right database is connected, e.g. define self.accept_invitation! in your model:
def self.accept_invitation!(update_resource_params)
  log "#{connection.current_database}"
  super.tap { |resource| log resource.inspect }
end

@scambra I apologize. I forgot to change the @resource.token to @token when I upgraded the versions and that's why I was facing the issue. Once changed it worked perfectly.


Refactored XenonDevice.generate_payload()

The generate_payload() method is now more generic and is also used by
OutletDevice.set_timer(). This removes a lot of duplicate code.

Apologies for not responding sooner. I've not had chance to run this yet. From quick scan, code looks good.
Things that stick out:

I like the use of dicts
commit states "and is also used by OutletDevice.set_timer()" - pre this change this was also the case. This should be removed from commit message
removes OFF and ON in dict, OFF no longer used in preference for SET (great idea 👍 ). ON still used in one place. Please continue removal of ON and OFF constant usage along with definitions and mention in commit.

Thanks for the review. I will clean up the commit. I obviously forgot to change the commit message after syncing back with your repo.
Do you think amending and rebasing would break the pull request?

I've done a rebase and force push on an open PR and it did work (was a while ago). Worst scenario, it doesn't work and open a new PR.

Okay, I already rewrote the message and removed the ON and OFF constants but didn't push my commits. I will force push my changes next week.

You have been right. Github seems to be able to resolve rebased branches in pull requests.

Thank you! Sorry for the delay in getting to this.

No problem, thanks for merging.


Does pact broker client support base_authentication of pact broker server

Hi,
We've set up the base authentication in our pact broker which runs on the nginx server.
Initially we thought we can pass the pact url like http://username:password@pact_broker_url, however, after reading the code, we found the pact:verify uses openuri and publish task to httpparty, both of them needs to pass additional options for the base_authentication.  Is there any pact configuration that we can use to pass the options through?
It will be great that the pact client and support can provide this feature, as the pact broker server can be shared by our multiple api users with basic authenticated access.  Thanks.

That feature currently does not exist, but it would be a great feature to add. Are you comfortable with Ruby code, and willing to work on a PR?
In the mean time, as a work around:
You could work around the publish issue by just doing a curl request: https://github.com/bethesque/pact_broker/wiki/Publishing-and-retrieving-pacts
And you could fetch the pacts in the same way (using curl) and put them in a temp dir, and run verify against the pacts in the temp dir.

Thanks very much.  I am happy to work on a PR for this feature.

That would be great :)
On Tue, Feb 10, 2015 at 10:02 PM, Lifei Zhou<EMAIL_ADDRESS>wrote:

Thanks very much. I am happy to work on a PR for this feature.
—
Reply to this email directly or view it on GitHub
https://github.com/bethesque/pact_broker/issues/20#issuecomment-73681662
.

Using basic authentication is now supported by the pact retrieving and publishing code.


Cartographing 2D with VLP16 Hires without IMU

Dear community,
I am working with Ubuntu 16.04.03 and ROS kinetic.
I installed googlecartographer and the DEMOs work fine.
Furthermore I installed the velodyne drivers and I can see
the pointcloud etc.
I now want to combine these two things, in order to cartograph
with my VLP16 Hires and Google Cartographer 2D.
http://google-cartographer-ros.readthedocs.io/en/latest/index.html
Following the link above, one can use the cartographer 2D without
an IMU. All examples provided do use bag files with IMU data.
Can you show me an example of how to use this great cartograph software without IMU data?
How to do that? I tried to change the launch files and lua files, to deactivate
SLAM and to activate RealTimeCorrelativeScanMatcher, but I dont know how to!
The tries I managed so far end up in not working due to missing tfs. No node is publishing tfs.
Thank you very much,
Sincerely yours, Kevin Riehl

TRAJECTORY_BUILDER_2D.use_imu_data = false

Okay. This does not work. There is a problem with tf I guess.
This is my Console output:
console.txt
This is my lua file:
backpack_2d.lua.txt
This is a screenshot:

This are my launch files:
backpack_2d.launch.txt
demo_backpack_2d.launch.txt
I am calling "roslaunch cartographer_ros demo_backpack_2d.launch".
I compiled my catkin_ws multipletimes after changes.
There seem to be problems with tf.
The topic "/scan" is posted on a frame called "/map". What am I doing wrong?
Thanks, Kevin

The topic "/scan" from my velodyne is posted on a frame called "/map". What am I doing wrong?

Remap from scan to map in the launch file?

No you misunderstood me. The topic "/scan" is published with the name "/scan".
Echoing with rostopic echo /scan  delivers that it is in frame "/map".
header:    seq: 13   stamp:      secs:<PHONE_NUMBER>     nsecs: 983551000   frame_id: "map" angle_min: -3.14159274101 angle_max: 3.14159274101 angle_increment: 0.00700000021607 time_increment: 0.0 scan_time: 0.0 range_min: 0.0 range_max: 200.0 ranges: [3.1577866077423096, 3.14778733253479, 3.139787435531616, 3.1497867107391357, 3.1377878189086914, 3.1537864208221436, 3.1377878189086914, 3.143787145614624, 3.1377878189086914, 3.1317882537841797, 3.1497867107391357, 3.139787435531616, 3.141787528991699, 3.147787094116211, 3.1357877254486084, 3.1517865657806396, 3.145787239074707, 3.1537864208221436, 3.1497867107391357, 3.145787239074707, 3.165785789489746, 3.1597859859466553, 3.1557865142822266, 3.16778564453125, 3.1617860794067383, 3.1617860794067383, 3.171785354614258, 3.1757850646972656, 3.1857845783233643, 3.1757850646972656, 3.1957833766937256, 2.943801164627075, 2.945800304412842, 2.901803731918335, 2.8978042602539062, 2.9498002529144287, 3.189784049987793, 3.213782548904419, 3.2177822589874268, 3.209782600402832, 3.233781337738037, 3.2217822074890137, 3.2257814407348633, 3.233781337738037, 3.235781192779
Even though I added the line:
	<remap from="scan" to="map"/>
to "demo_backpack_2d.launch"
Nothing changes...problem stays the same...
Using "rosrun tf tf_monitor" we can see...that no tf at all is there.

Frame "map" is not ideal because cartographer_ros usually publishes under this frame.
You can edit the frame names in the lua configuration.
It also needs to match with the urdf model.

Thank you very much, I could now solve the problem.
The frame that my velodyne driver should post to is called
"horizontal_laser_link".
For all those, who are interested in the solution, the following
files contain all configurations to use the VLP 16 without IMU:
Launch Files in package "cartographer_ros":
VELbackpack_2d.launch.txt
VELdemo_backpack_2d.launch.txt
LUA Files in package "cartographer" and "cartographer_ros":
velobackpack_2d2.lua.txt
velomap_builder.lua.txt
velotrajectory_builder.lua.txt
velotrajectory_builder_2d.lua.txt
Launch Files in package "velodyne_points":
VLP16_HiRes_points.launch.txt
Kind regards,
Kevin Riehl

@KevinRiehl
Hi,  I met the same problem as you, and I am glad to see that you have solved it. I looked at your answer carefully, and changed 4 Lua and 2 launch files according to your prompt.The results after the compilation run are as follows：
[FATAL] [1528900046.641877316]: F0613 22:27:26.000000 29707 lua_parameter_dictionary.cc:399] Check failed: HasKey(key) Key 'grid_options_2d' not in dictionary:
{
num_range_data = 35.000000,
range_data_inserter = {
hit_probability = 0.550000,
insert_free_space = true,
miss_probability = 0.490000,
},
resolution = 0.050000,
}
So I changed the file as follows：
submaps = {
resolution = 0.05,
num_range_data = 90,
grid_options_2d=false,
range_data_inserter = {
insert_free_space = true,
hit_probability = 0.55,
miss_probability = 0.49,
[ INFO] [1528899340.824396059]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velobackpack_2d2.lua' for 'velobackpack_2d2.lua'.
[ INFO] [1528899340.824629976]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velomap_builder.lua' for 'velomap_builder.lua'.
[ INFO] [1528899340.824667907]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velomap_builder.lua' for 'velomap_builder.lua'.
[ INFO] [1528899340.824724828]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer/configuration_files/pose_graph.lua' for 'pose_graph.lua'.
[ INFO] [1528899340.824756388]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer/configuration_files/pose_graph.lua' for 'pose_graph.lua'.
[ INFO] [1528899340.824859762]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velotrajectory_builder.lua' for 'velotrajectory_builder.lua'.
[ INFO] [1528899340.824889941]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velotrajectory_builder.lua' for 'velotrajectory_builder.lua'.
[ INFO] [1528899340.824938652]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velotrajectory_builder_2d.lua' for 'velotrajectory_builder_2d.lua'.
[ INFO] [1528899340.824967197]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files/velotrajectory_builder_2d.lua' for 'velotrajectory_builder_2d.lua'.
[ INFO] [1528899340.825050086]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer/configuration_files/trajectory_builder_3d.lua' for 'trajectory_builder_3d.lua'.
[ INFO] [1528899340.825081111]: I0613 22:15:40.000000 28690 configuration_file_resolver.cc:41] Found '/home/bryant/catkin_ws/install_isolated/share/cartographer/configuration_files/trajectory_builder_3d.lua' for 'trajectory_builder_3d.lua'.
F0613 22:15:40.825194 28690 lua_parameter_dictionary.cc:116] Check failed: lua_istable(L, -1) Topmost item on Lua stack is not a table!
[FATAL] [1528899340.825342131]: F0613 22:15:40.000000 28690 lua_parameter_dictionary.cc:116] Check failed: lua_istable(L, -1) Topmost item on Lua stack is not a table!
*** Check failure stack trace: ***
@     0x7fca67c23daa  (unknown)
@     0x7fca67c23ce4  (unknown)
@     0x7fca67c236e6  (unknown)
@     0x7fca67c26687  (unknown)
@           0x5f647f  cartographer::common::(anonymous namespace)::CheckTableIsAtTopOfStack()
@           0x5f7cdb  cartographer::common::LuaParameterDictionary::PopDictionary()
@           0x5f9028  cartographer::common::LuaParameterDictionary::GetDictionary()
@           0x5df831  cartographer::mapping::CreateSubmapsOptions2D()
@           0x5d6abf  cartographer::mapping::CreateLocalTrajectoryBuilderOptions2D()
@           0x5d5cb3  cartographer::mapping::CreateTrajectoryBuilderOptions()
@           0x5d3a12  cartographer_ros::CreateTrajectoryOptions()
@           0x599230  cartographer_ros::LoadOptions()
@           0x596e83  cartographer_ros::(anonymous namespace)::Run()
@           0x5940d6  main
@     0x7fca640a2f45  (unknown)
@           0x596cd7  (unknown)
@              (nil)  (unknown)
[cartographer_node-2] process has died [pid 28690, exit code -6, cmd /home/bryant/catkin_ws/install_isolated/lib/cartographer_ros/cartographer_node -configuration_directory /home/bryant/catkin_ws/install_isolated/share/cartographer_ros/configuration_files -configuration_basename velobackpack_2d2.lua __name:=cartographer_node __log:=/home/bryant/.ros/log/e8ae4e78-6f11-11e8-abe3-3497f693bf66/cartographer_node-2.log].
log file: /home/bryant/.ros/log/e8ae4e78-6f11-11e8-abe3-3497f693bf66/cartographer_node-2*.log
Now there is an error, and I checked a lot of information and did not know how to solve it. I am a novice programmer. I would be very grateful if you could give me some suggestions.

Hi,
I use the code contributed by @KevinRiehl . And I also get the same error like @xukuanHIT .  Could someone please give me some advice?
Detail are as follows:
`F0917 23:14:02.553182 15188 lua_parameter_dictionary.cc:399] Check failed: HasKey(key) Key 'grid_options_2d' not in dictionary:
{
num_range_data = 35.000000,
range_data_inserter = {
hit_probability = 0.550000,
insert_free_space = true,
miss_probability = 0.490000,
},
resolution = 0.050000,
}
[FATAL] [1537197242.553567581]: F0917 23:14:02.000000 15188 lua_parameter_dictionary.cc:399] Check failed: HasKey(key) Key 'grid_options_2d' not in dictionary:
{
num_range_data = 35.000000,
range_data_inserter = {
hit_probability = 0.550000,
insert_free_space = true,
miss_probability = 0.490000,
},
resolution = 0.050000,
}
*** Check failure stack trace: ***
@     0x7f14d7dc1daa  (unknown)
@     0x7f14d7dc1ce4  (unknown)
@     0x7f14d7dc16e6  (unknown)
@     0x7f14d7dc4687  (unknown)
@           0x5e259f  cartographer::common::LuaParameterDictionary::CheckHasKey()
@           0x5e25ed  cartographer::common::LuaParameterDictionary::CheckHasKeyAndReference()
@           0x5e288e  cartographer::common::LuaParameterDictionary::GetDictionary()
@           0x6dad21  cartographer::mapping::CreateSubmapsOptions2D()
@           0x682acf  cartographer::mapping::CreateLocalTrajectoryBuilderOptions2D()
@           0x5e7c33  cartographer::mapping::CreateTrajectoryBuilderOptions()
@           0x5d4492  cartographer_ros::CreateTrajectoryOptions()
@           0x5d3d60  cartographer_ros::LoadOptions()
@           0x596e83  cartographer_ros::(anonymous namespace)::Run()
@           0x5940d6  main
@     0x7f14d423cf45  (unknown)
@           0x596cd7  (unknown)
@              (nil)  (unknown)
[cartographer_node-3] process has died [pid 15188, exit code -6, cmd /home/slam/catkin_ws/install_isolated/lib/cartographer_ros/cartographer_node -configuration_directory /home/slam/catkin_ws/install_isolated/share/cartographer_ros/configuration_files -configuration_basename velobackpack_2d2.lua __name:=cartographer_node __log:=/home/slam/.ros/log/4f024ab0-ba8c-11e8-aaef-1c6f659f55c1/cartographer_node-3.log].
log file: /home/slam/.ros/log/4f024ab0-ba8c-11e8-aaef-1c6f659f55c1/cartographer_node-3*.log
`

The config syntax was changed so we can support the truncates signed distance field (TSDF) feature.
This change was made in https://github.com/googlecartographer/cartographer/pull/1108/files#diff-b612d6fd6de960e1b7974146d059f2fb
Please adapt your config accordingly.


Integrate kubernetes events to operator

What this PR does / why we need it:

Integrate events into operator for better troubleshooting and debugability

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
How Has This Been Tested?:

Credential error

Provisioning request error

Sucessfully provisioned

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration and test output
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
Release note:

General comment: We need to revisit what all places we need to emit the event for. Events is not a replacement for looking up the controller logs so we need to emit events for only the changes that need to be visible to end users.


ML 8.15 : Shear, Scale and Reflect Images (D) #7514

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #issue_number_here

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[#] Doc#umentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[#] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[#] I follow Contributing Guidelines & Code of conduct of this project.
[#] I have performed a self-review of my own code or work.
[#] I have commented my code, particularly in hard-to-understand areas.
[#] My changes generates no new warnings.
[#] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

changes done...kindly review the PR again
On Tue, Nov 9, 2021 at 9:26 AM Akshay Raina @.***>
wrote:

@.**** requested changes on this pull request.
@Ruchika-20 https://github.com/Ruchika-20 you should shear, scale and
reflect images using Python and not manually upload some image/s. You can
look into OpenCV specifically for this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/7586#pullrequestreview-800822960,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEIB53XQIGXS4SJAV2XVALULCLWJANCNFSM5HTILVVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Created default splits for Fluo table that are now set during init

Also added kryo to app classpath in 5ff169f2aafa9

+1


Standalone Client: Raise an error if more than one primary node is found

The standalone client receives a list of node addresses and tries to find the primary by getting the node's role from 'info replication'. In some cluster issues there might be more than a single primary so the client won't be able to identify which one is the legit primary, and therefore we should fail the client creation.

@eifrah-aws round

How to reproduce and test it manually? Won't GLIDE be confused if a cluster lost a replica or few nodes? It may come into the same situation: 2 primary + 1 replica.

You can create two primaries and create with their addresses a standalone client
I don't understand the scenario..? how loosing replicas will end up with multiple primaries?


Importer imports only a single folder

Importer will detect multiple folders but exits after first show successfully imports

Fixed in 989a33a4a85cdafdee1b1c881227b1996b6a4232


TimeSelect not working

I user react-datepicker in my project.
I tried to show time picker but I can't.
It's my code.
package.json
"react-datepicker": "^0.54.0"

js
<DatePicker
  selected={this.state.disp_date}
  onChange={this.handleChange}
  showTimeSelect
  dateFormat="LLL"
/>

Please help me...

Are you using React 16 ? #1060

It's a feature of 0.55.0 : https://github.com/Hacker0x01/react-datepicker/releases

@GautierT Thanks! When I upgrade react-date-picker for 0.55.0, that works fine!


OffStreetParking doc examples include not documented properties

In https://github.com/Fiware/dataModels/blob/master/Parking/OffStreetParking/doc/spec.md there are some examples in the lower part including properties such as maximumParkingDuration which are not documented, including its purpose and other details such as the type.

oh it is documented but there is a typo
maximumAParkingDuration vs maximumParkingDuration


Newer VSF crashes on Ipad Pro

When visiting a newer version of a vsf store on the actual device (ipad pro) not emulated some javascript crash happens and the page does not load correctly.
I have tested in several versions of VSF but cant conclude from where it the bug starts, but testing on demo.vuestorefront.io also allows for the issue to appear.

@Fifciu @gibkigonzo can you give it a try on Safari?

Just checked it on iPad Pro. demo.vuestorefront.io works well. Try to purge the browser's cache and use a private cart.
A few weeks ago we had a similar situation on VSF Slack. One developer had a problem opening VSF on MacOS. I checked it both on Ubuntu and macOS. In the browser's console, I've found errors about not existing .js files. It looked like server would deploying a new version right now or something went wrong during the previous deploy.
I believe the author of the issue has cached bugged deploy and purging cache + incognito is a solution.


SecurityTokenInvalidIssuerException with OpenID Connect and the Azure AD "common" endpoint

When you use OpenID Connect against Azure AD's "common" endpoint, configuring https://login.microsoftonline.com/common/v2.0 as the OpenIdConnectOptions.Authority value, the metadata document will provide an issuer value of https://login.microsoftonline.com/{tenantid}/v2.0. Note the "{tenantid}" placeholder.
Within an actual ID token, the issuer value no longer contains that placeholder, but the actual tenant ID of the common endpoint (https://login.microsoftonline.com/9188040d-6c67-4c5b-b112-36a304b66dad/v2.0). This causes the issuer validation in Microsoft.IdentityModel.Tokens to fail.
I guess Azure AD decided to return an issuer with a non-standard (?) placeholder here, but shouldn't the code in
azure-activedirectory-identitymodel-extensions-for-dotnet be able to deal with such Azure AD specifics? If so, I'd consider this a bug.
Also, the exception message is "IDX10205: Issuer validation failed. Issuer: 'System.String'. Did not match: validationParameters.ValidIssuer: 'System.String' or validationParameters.ValidIssuers: 'System.String'." Which is not at all helpful, but I'll open a separate issue for this.

For anyone reading this and needing a workaround: You can use the Guid-based version of the common endpoint instead. As the OpenIdConnectOptions.Authority value, use https://login.microsoftonline.com/9188040d-6c67-4c5b-b112-36a304b66dad/v2.0.

@jennyf19 Thanks for the pointer (and sorry for the delay). My point was that using the validator will not be possible using appsettings.json. So, if we want to support Azure AD multitenancy, we'll have to adapt our application code.
Thanks for the quick reaction!


add xhr, fetch

I had reviewed this PR after it had been merged already. It looks there is no issue on the documentation. Great job 👍


add questions to web development quizes and show results

Is your feature request related to a problem? Please describe.
add questions to web development quizes and show results
Describe the solution you'd like
add questions to web development quizes and show results
Describe alternatives you've considered
add questions to web development quizes and show results
Additional context
Add any other context or screenshots about the feature request here.

@KapuluruBhuvaneswariVspdbct Just a reminder or warning: when you get assigned an issue, kindly complete it to the fullest extent. If you’re only raising PRs for points, please don’t. If I see this kind of behavior again, I will take the necessary actions.
When you were already assigned to your issue, why didn't you complete it already? Now, you're raising another issue.

the changes are not at all displayed in website even after merging that is why i raised the pr

the changes are not at all displayed in website even after merging that is why i raised the pr

@KapuluruBhuvaneswariVspdbct its because its in different branch, it will be merged to the deployment branch once it is approved by all the team members, I merged you PR so that atleast you can get points for your work.

not about the points sir,i just want you to look at the sample quiz conducted  based on your review ,if you want me to proceed i would have added all other quizes in a single pr sir

thanlk you sir

@KapuluruBhuvaneswariVspdbct Could you please work on a different issue? I don’t want to continue with the quiz feature for now.

Actually I have added the quizes the incomplete work is adding questions to
remaining issues
On Sat, Oct 26, 2024, 5:56 PM Yajush Vyas @.***> wrote:

Closed #163 https://github.com/notsoocool/codecache/issues/163 as not
planned.
—
Reply to this email directly, view it on GitHub
https://github.com/notsoocool/codecache/issues/163#event-14901952912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHOWNHB765AAVZQSIBGNOQTZ5ODA3AVCNFSM6AAAAABQSTWC32VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUHEYDCOJVGI4TCMQ
.

Because of this I am not going to entertain this issue any further

I have added the quiz cards and added the different topic cards and a
sample quiz if it is reviewed by you then I can add all the quizes ,if is
not about creating quizes ,I wanted to add the questions to all quizes
based on your review
On Sat, Oct 26, 2024, 6:03 PM Yajush Vyas @.***> wrote:

That’s the whole point I was referring to earlier: when an issue is
assigned to you, it’s important to complete it to your fullest extent and
not just raise a partially complete PR.
—
Reply to this email directly, view it on GitHub
https://github.com/notsoocool/codecache/issues/163#issuecomment-2439567007,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHOWNHALDOCG45QAOCXBXM3Z5OD2BAVCNFSM6AAAAABQSTWC32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGU3DOMBQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok sir I will respect your decision,it is up to you to assign or close the
issue sir
On Sat, Oct 26, 2024, 6:06 PM Bhuvaneswari Kapuluru <
@.***> wrote:

I have added the quiz cards and added the different topic cards and a
sample quiz if it is reviewed by you then I can add all the quizes ,if is
not about creating quizes ,I wanted to add the questions to all quizes
based on your review
On Sat, Oct 26, 2024, 6:03 PM Yajush Vyas @.***>
wrote:

That’s the whole point I was referring to earlier: when an issue is
assigned to you, it’s important to complete it to your fullest extent and
not just raise a partially complete PR.
—
Reply to this email directly, view it on GitHub
https://github.com/notsoocool/codecache/issues/163#issuecomment-2439567007,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHOWNHALDOCG45QAOCXBXM3Z5OD2BAVCNFSM6AAAAABQSTWC32VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGU3DOMBQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Firmware update fail

Hi,
I bought a new conbee II for my home assistant (been using the first conbee for a long time). After I plugged it in, I wanted to do a firmware update, however this failed (see error below). After the error occurred, the usb isn't recognised anymore. Do you have any idea how I can recover my conbee?
Thanks in advance.
Device Path : /dev/ttyACM0

-------------------------------------------------------------------

Firmware available for flashing:

deCONZ_ConBeeII_0x264a0700.bin.GCF
deCONZ_ConBeeII_0x26580700.bin.GCF
deCONZ_Rpi_0x26330500.bin.GCF
deCONZ_Rpi_0x26350500.bin.GCF

Enter the firmware file name from above, including extension,
or press Enter now to exit.

File Name : deCONZ_ConBeeII_0x26580700.bin.GCF

-------------------------------------------------------------------

Device: /dev/ttyACM0

Firmware File: deCONZ_ConBeeII_0x26580700.bin.GCF

Are the above device and firmware values correct?
Enter Y to proceed, any other entry to exit: Y

Flashing...

GCFFlasher V3_13 (c) dresden elektronik ingenieurtechnik gmbh
Reboot device /dev/ttyACM0 (ConBee II)
deCONZ firmware version 264A0700
R21B18 Bootloader
Vers: 2.07
build: Jun 17 2019
flashing 160930 bytes: |====error: timeout flashing firmware after 3006 ms

Flashing Error! Please re-run this script...

Running on ubuntu 18.04 with docker.

The same thing happened to me.  Docker on Ubuntu 18.04.  Everything works well, I try to update the firmware by following the instructions, stop the container, run the command with the script, it detect the conbee 2, I enter the path, I enter the file name, it gives me an error and now the conbee 2 is no longer recognized and  I can't update the firmware or start the container. Conbee 2 is bricked???

I have tried to recover the bricked USB device on a rasp-pi following the manual upgrade steps. However, it seems the firmware is been corrupted completely as the device is not able to connect correctly:
[2227002.233815] cdc_acm 1-2:1.0: ttyACM0: USB ACM device
[2227005.687735] usb 1-2: USB disconnect, device number 46
[2227014.075799] usb 1-2: new full-speed USB device number 47 using xhci_hcd
[2227014.225990] usb 1-2: New USB device found, idVendor=1cf1, idProduct=0030
[2227014.225996] usb 1-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[2227014.226000] usb 1-2: Product: ConBee II
[2227014.226003] usb 1-2: Manufacturer: dresden elektronik ingenieurtechnik GmbH
[2227014.226006] usb 1-2: SerialNumber: DE2215531
[2227014.226726] cdc_acm 1-2:1.0: ttyACM0: USB ACM device

Then I tried to connect to another machine that runs ubuntu 18.04, but I got the same results.
[67811.446199] cdc_acm 3-2:1.0: failed to set dtr/rts
[67820.010355] usb 3-2: new full-speed USB device number 103 using ohci-pci
[67820.217126] usb 3-2: New USB device found, idVendor=1cf1, idProduct=0030, bcdDevice= 1.00
[67820.217133] usb 3-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[67820.217136] usb 3-2: Product: ConBee II
[67820.217139] usb 3-2: Manufacturer: dresden elektronik ingenieurtechnik GmbH
[67820.217142] usb 3-2: SerialNumber: DE2215531
[67820.219366] cdc_acm 3-2:1.0: ttyACM0: USB ACM device
[67823.765272] usb 3-2: USB disconnect, device number 103
[67823.766192] cdc_acm 3-2:1.0: failed to set dtr/rts
[67832.334197] usb 3-2: new full-speed USB device number 104 using ohci-pci
[67832.541141] usb 3-2: New USB device found, idVendor=1cf1, idProduct=0030, bcdDevice= 1.00
[67832.541147] usb 3-2: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[67832.541150] usb 3-2: Product: ConBee II
[67832.541153] usb 3-2: Manufacturer: dresden elektronik ingenieurtechnik GmbH
[67832.541156] usb 3-2: SerialNumber: DE2215531
[67832.543337] cdc_acm 3-2:1.0: ttyACM0: USB ACM device
[67836.071991] usb 3-2: USB disconnect, device number 104

Is there a way to recover the stick?

I was able to solve the issue using windows. See https://github.com/dresden-elektronik/deconz-rest-plugin/issues/3015
I think the problem is with the update script, it is missing the -t 60 option for conbee 2. If that is added, it should also be possible for the docker to upgrade the firmware.

I was able to recover AND upgrade the firmware after having bricked it myself -> https://github.com/dresden-elektronik/deconz-rest-plugin/issues/4342


color$1.Color is not a constructor

Expected behavior
Hovering and toggling series should not throw exception.
Current behavior
Hovering and toggling series throws exception
Reproducible sample
https://codesandbox.io/p/github/seikosantana/chart-test-remote/master?file=%2Fpackage.json&workspace=%257B%2522activeFileId%2522%253A%2522clfdl46qp000yg3d0bapxfjqr%2522%252C%2522openFiles%2522%253A%255B%2522%252FREADME.md%2522%252C%2522%252Fwebpack-demo.config.js%2522%255D%252C%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522gitSidebarPanel%2522%253A%2522COMMIT%2522%252C%2522spaces%2522%253A%257B%2522clfdl48wb000x3v6mn9qbcihp%2522%253A%257B%2522key%2522%253A%2522clfdl48wb000x3v6mn9qbcihp%2522%252C%2522name%2522%253A%2522start%2522%252C%2522devtools%2522%253A%255B%257B%2522type%2522%253A%2522PREVIEW%2522%252C%2522taskId%2522%253A%2522start%2522%252C%2522port%2522%253A9090%252C%2522key%2522%253A%2522clfdlby3o01ev3v6m1j005uyo%2522%252C%2522isMinimized%2522%253Afalse%257D%252C%257B%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522start%2522%252C%2522key%2522%253A%2522clfdlbv1v01at3v6mvnrk3zcz%2522%252C%2522isMinimized%2522%253Afalse%257D%255D%257D%257D%252C%2522currentSpace%2522%253A%2522clfdl48wb000x3v6mn9qbcihp%2522%252C%2522spacesOrder%2522%253A%255B%2522clfdl48wb000x3v6mn9qbcihp%2522%255D%252C%2522hideCodeEditor%2522%253Afalse%257D
Optional extra steps/info to reproduce
Hover on the chart to see the error being thrown.
Possible solution
No response
Context
I am using chart.js in a (remote-component)[https://www.npmjs.com/package/@paciolan/remote-component], and the repro sample is a fork of (remote-component-starter)[https://github.com/Paciolan/remote-component-starter] with updated dependencies and typescript added.
I tried on a freshly created project with CRA and the issue does not occur. I suspect it is related to modules thing or webpack configurations, but I'm not yet familiar enough to figure out what to do about it.
chart.js version
4.2.1
Browser name and version
No response
Link to your project
No response

Your codepen does not load

I'm sorry, i fixed it, would you mind to take a look again? @LeeLenaleee

My current workaround is to modify node_modules\chart.js\dist\chunks\helpers.segment.cjs
replacing
function color(value) {
    return isPatternOrGradient(value) ? value : new color$1.Color(value);
}
function getHoverColor(value) {
    return isPatternOrGradient(value) ? value : new color$1.Color(value).saturate(0.5).darken(0.1).hexString();
}

with
function color(value) {
    return isPatternOrGradient(value) ? value : new color$1(value);
}
function getHoverColor(value) {
    return isPatternOrGradient(value) ? value : new color$1(value).saturate(0.5).darken(0.1).hexString();
}


Obfuscate chunk filename on production build

Fix https://github.com/Shopify/hydrogen/issues/1084
Description
Obfuscate chunk filename on production build - Some ad blockers blocks certain files simply due to how it is named.
Additional context

Before submitting the PR, please make sure you do the following:

[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

Thoughts - should I substring the encoded filename so that it is no more than certain amount of characters?
We don't ever need to convert this encoding back to the original string but then I don't know if we would get into a name collision if I substring it

This error originates in entry-client.tsx when client components are being loaded dynamically. Because React expects there to be a module there, an error is thrown, and the nearest Error Boundary is rendered.
I think this is a larger problem, and obfuscating the filenames is addressing the symptom, and not the real problem, which is: React needs to have a better fallback in place for client components which fail to load. We're bound to run into many other adblocker-related issues, not just with Analytics.
How can we address this in a more sustainable way? Maybe with a catch-all error boundary which console.errors details about the component which failed to load? Then allow the rest of the layout to fail if other components depended on that component to properly hydrate?

I didn't realize that I was duplicating your work here. I solved it slightly differently. Maybe we'd like to do both? #1099

Both seems good; it never hurts to add an error boundary, and hope that it isn't ever used. 😄

I would prefer we do both @blittle
@jTotally agree that we still do need ErrorBoundary but then that goes for any client components


Fix alertmanager flag for newer versions

Version 0.17 of Prometheus changed the flag alertmanager.http-deadline to alertmanager.timeout. Shouldn't change current behavior.

I've merged this change in manually due to conflicts with other patches.


avoid iterating possible None returned by 'get_app_port_mappings'

This PR fixes an error seen when utils.py attempts to iterate None:
2017-08-18 16:43:34,454 marathon_lb: Unexpected error!
Traceback (most recent call last):
  File "/marathon-lb/marathon_lb.py", line 1517, in do_reset
    self.__apps = get_apps(self.__marathon)
  File "/marathon-lb/marathon_lb.py", line 1414, in get_apps
    task_ip, task_ports = get_task_ip_and_ports(app, task)
  File "/marathon-lb/utils.py", line 407, in get_task_ip_and_ports
    task_ports = get_app_task_ports(app, task, mode)
  File "/marathon-lb/utils.py", line 385, in get_app_task_ports
    return get_port_mapping_ports(app)
  File "/marathon-lb/utils.py", line 364, in get_port_mapping_ports
    for p in port_mappings
TypeError: 'NoneType' object is not iterable

Can one of the admins verify this patch?

Fixes #485.

I believe this is fixed by #494.  @matt-deboer can you let me know if you still need this?  Otherwise will close in a week or two.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Sorry for the delay in getting to this pr. This was fixed in https://github.com/mesosphere/marathon-lb/commit/0a5906fa6401fa1c7cecce29b6775a59446a3019#diff-04d2c9518498da64bfa9db44c6211645R343. Closing this and the linked issue.


Corrected a link in old Demo

Perhaps this was the issue.

not sure whats going on here, seems to be unrelated to this PR though
Traceback (most recent call last):
File "/home/runner/work/qml/qml/venv/lib/python3.10/site-packages/sphinx/config.py", line 350, in eval_config_file
exec(code, namespace)
File "/home/runner/work/qml/qml/conf.py", line 21, in
from pennylane import PennyLaneDeprecationWarning
ModuleNotFoundError: No module named 'pennylane'

not sure whats going on here, seems to be unrelated to this PR though
Traceback (most recent call last): File "/home/runner/work/qml/qml/venv/lib/python3.10/site-packages/sphinx/config.py", line 350, in eval_config_file exec(code, namespace) File "/home/runner/work/qml/qml/conf.py", line 21, in from pennylane import PennyLaneDeprecationWarning ModuleNotFoundError: No module named 'pennylane'

Yikes. Yeah, it seems to be unrelated. I'll follow up. Thanks!

When I tested locally, keeping the two slashes allows the link to point to the demo, but one slash just doesn't show up as a link.

For cross-referencing sections, I don't think this is allowed syntax with Sphinx 🤔 (but could be wrong). Instead, I typically use :ref:

Don't see any examples of :ref:`...#` to a section :thinking:
and found some broken :ref: links.

https://pennylane.ai/qml/demos/tutorial_quantum_chemistry/ :ref:`hamiltonian` -
https://pennylane.ai/qml/demos/tutorial_quantum_natural_gradient/ :ref:`Concepts section <glossary_variational_circuit>`

I just made the links external, it seems like that works reliably at the moment.


GETting null property /{repo}/{item}/{prop} throws NullPointerException

The problem is that BasicItemDelegate.getProperty() determines the property type by looking at the property value instead of using the metamodel, and this fails when the property value is null.

Fixed by cbca2ee7a6b6f989a17e8c628901cbddf4c9f389.