Handlers calling multiple times

Question
I have noticed that almost all event handlers on room object are getting called multiple times in this repo.
Additional context
I put console logs in participantConnected, participantDisconnected, dominantSpeakerChanged, networkLevelChanged, etc.
Like when a new participant joins the room , "participantConnected" event is getting fired 3 times.
Also, handleDominantSpeakerChanged is getting called 5 times whenever the other participant is unmuting audio . Similarly when muting back, its calling for 5 more times with null value.
I need your help in understanding the reason behind these multiple calls.

Hi @rachnaban - thanks for the question. I think this is an interesting topic!
Looking at the participantConnected handler - if I put a console.log('participantConnected') inside the handler, I also see that the message is logged three times in the console. This is not because the participantConnected event fires three times. It is because the participantConnected event has three event handlers attached to it (and the event is only fired once).
But why does it have three handlers attached to it?
A good question! This has to do with how these hooks (like useParticipants) work. A hook like useParticipants will be called by every component that uses it. If three different components use it, then it will be called three times, and it will attach three different event handlers as a result.
We can see this if we inspect the twilioRoom object in the browser console.  The twilioRoom object inherits from the EventEmitter class from Node, and this class has some internal properties that we can take a look at. twilioRoom._events is an object where each key is an event name, and each value is an array of listeners. So if we look at twilioRoom._events.participantConnected, we can see all of the listeners associated with that event:

We can even look at each listener to see that the [[FunctionLocation]] of each listener comes from useParticipants.tsx. These three handlers correspond to to different React components in this application. So when the first listener is called, component A will be updated, when the second listener is called, component B will be updated, and so forth.
Many of the hooks inside the src/hooks folder attach event handlers to the room object (or other objects from the twilio-video SDK). If multiple components use the same hook, then there will be multiple listeners attached (one for each component).
The EventEmitter class from Node has a built-in protection mechanism so that this pattern doesn't get out of hand. A handful of listeners attached to an eventEmitter is no problem, but something like 50 listeners attached to an eventEmitter would be a problem (and it probably means that there is a bug somewhere). Luckily for us, if we add many listeners to an eventEmitter, we will see a warning like this:

This doesn't mean that anything is necessarily wrong. It's just the EventEmitter class telling us that quite a few listeners have been attached, and that we should look into things. If we are sure that this is correct, we can change the threshold of the warning with twilioRoom.setMaxListeners(20), to increase the threshold to 20 (for instance). As I work on this app, I make sure that this warning doesn't pop up.
I hope this answers your question! Please let me know if you have any additional questions. I think this is a pretty interesting topic, so I'm happy to explore it further.
Happy coding!

Thanks @timmydoza for showing me the LIGHT here!! This screen shot of window.twilioRoom is extremely helpful to me.
As I understand I need to do some homework on Node EventEmitter to understand this more clearly. I will do that for sure.
For now, I am good on this and so closing this question, will open a new one in case I got stuck.
I really appreciate your patience in explaining this to me. Thanks.


Running unprivileged kernels

Can we run an unprivileged kernel on solo5 now? If not, when?

I don't know what an "unprivileged kernel" is. Can you elaborate?

Just a kernel running unprivileged, and in this case being dependent on the solo5 API. It would be a more secure way to run a unikernel. On another topic, I don't see a way to generate randomness, and that would be a serious security problem in an unprivileged context. On normal hardware you have access to all kinds of random sources, such as WiFi frequencies and RDRAND when that is present, and while none are guaranteed, you can still build up enough randomness. With the solo5 API you don't have much to go on. Maybe I missed it, and you do have some entropy source.

Just a kernel running unprivileged

I still don't understand what you mean by "unprivileged". hvt / virtio / muen isolate the unikernel from its host using hardware virtualization. spt accomplishes the same using a minimal, strict seccomp filter. genode ... it depends, ask the Genode folks.
Watch our talk from this year's FOSDEM for more detail on Solo5 concepts, and also what it does not do.

I don't see a way to generate randomness

Correct, there is currently no API for obtaining entropy from the host. You can of course use CPU-based sources such as RDRAND and CPU jitter inside the unikernel, which is what MirageOS does. This may change at some point, but not now.
Meta: This is an issue tracker. The preferred place for general questions is the mailing list (https://github.com/solo5/solo5#contributing-and-community), which has the advantage that more people will see your question and can contribute to the discussion if they wish.

I think this has been answered from my side, so closing.


The behavior of 'time-generated-field' seems to have changed

time-generated-field
The name of a field in the data that contains the timestamp of the data item. If you specify a field, its contents are used for TimeGenerated. If you don't specify this field, the default for TimeGenerated is the time that the message is ingested. The contents of the message field should follow the ISO 8601 format YYYY-MM-DDThh:mm:ssZ. Note: the Time Generated value cannot be older than 2 days before received time or the row will be dropped.

Instead of the row being silently dropped (as per this documentation), it seems that only the field is silently dropped, resulting in a row TimeGenerated of the current date/time.

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

ID: 657b5b68-4c41-841a-1110-67b9c2049e39
Version Independent ID: 2237ca0e-c552-7ee8-97ae-7034e51241d9
Content: Azure Monitor HTTP Data Collector API - Azure Monitor
Content Source: articles/azure-monitor/logs/data-collector-api.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@ianssoftcom
Thanks for your feedback! We will investigate and update as appropriate.

bump

@ianssoftcom @Augustukas I was able to reproduce the same in my environment as well i.e., if the Time Generated value is older than 2 days then the row is not being dropped instead its resulting in a row TimeGenerated with current date/time as value.
Assigning this issue to content owner @bwren for visibility, to review, double-check and to update the document as appropriate.

Thanks. Will fix.
#sign-off


no opt.device

Hello, I have a question. When running track.py, there is an error, no opt.device. Cannot continue to run, how can I solve it, thank you very much!

As I can see, the opt.device is initialized in the demo.py.


cobang: add support aarch64-linux

Description of changes
Added "aarch64-linux" to meta.platforms
Things done
I have built cobang on the architecture aarch64-linux and it works for me, so can we add support for the aarch64-linux architecture?

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Also [always] squash commits. (It should be a single commit for this change.)

ping @dvaerum

Please only add clean commits.
Second commit should be:

cobang: add support aarch64-linux
cobang: add dvaerum as maintainer

You could make the cobang change a single commit. But then, you have to add to the body of commit message what you are doing. You are doing two actions. So you comment that.
Options are: 3 commits or 2 commits (adding information to body of commit). Up to you.

On commit message:
Things I'd do differently:

Avoid special characters such as ``. We never know how this could affect something.
Keep comment objective and concise. For example:

cobang: add aarch64-linux platform
cobang: add dvaerum as maintainer

This is not stopping this PR from being merged. I'm waiting CI atm.

Is this package building for you?

#330241

Is this package building for you?

No, not after the 6 of July I think it was, it seems to be a problem with a unit test which are triggered, I have an overriteAttrs which disables nativeCheckInputs so that the build will work.
python-zbar_p = pkgs.python312Packages.python-zbar.overrideAttrs (oldAttrs: rec {
  nativeCheckInputs = [ ];
});

I have not figured out yet which unittest there needs to be disabled for the build to work, but cobang works on my PC.
It was something I wanted to look into it more after this merge request and try to make a pull request with a fix and maybe become a maintainer of that package too, since it does not have one currently

I have not figured out yet which unittest there needs to be disabled for the build to work,

I just gave you the solution.

but cobang works on my PC.

The reference must be nixpkgs master. And it is failing at nixpkgs master. While I understand the problem was not introduced by you, and the changes you are proposing are unrelated. We should be fixing it. And I already gave you the solution.

It was something I wanted to look into it more after this merge request (made it before I learned that the python-zbar package broke) and try to make a pull request with a fix and maybe become a maintainer of that package too, since it does not have one currently

Let's do this in this PR since I'm already reviewing this. (You are saving yourself time.)

To fix it, I suggest you remove from:

pkgs/development/python-modules/python-zbar/default.nix

The lines:
  # distutils usage in setup.py
  disabled = pythonAtLeast "3.12";

Then, it builds fine.
This also means an extra commit:
* `python312Packages.python-zbar: fix build`

I hope I understood your instructions correctly 😇

I hope I understood your instructions correctly 😇

Looks like I misunderstood something or there is another problem :confused:

Remove pythonAtLeast reference since it is no longer used.

Result of nixpkgs-review pr 326368 run on x86_64-linux 1

4 packages built:

cobang
cobang.dist
python312Packages.python-zbar
python312Packages.python-zbar.dist

ping @dvaerum

Hey @superherointj, thanks for the pointers

Result of nixpkgs-review pr 326368 run on x86_64-linux 1

2 packages built:

python312Packages.python-zbar
python312Packages.python-zbar.dist

Result of nixpkgs-review pr 326368 run on aarch64-linux 1

4 packages built:

cobang
cobang.dist
python312Packages.python-zbar
python312Packages.python-zbar.dist


Change behavior of connectLeavesToExitNode

Previously, this code failed when no sink nodes could be found. This PR changes this behavior to instead select the last node of the iterated list of nodes, to accommodate scenarios found in real projects.

Codecov Report

Merging #140 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #140      +/-   ##
============================================
- Coverage     86.38%   86.34%   -0.05%
  Complexity      260      260
============================================
  Files            28       28
  Lines           595      593       -2
  Branches        122      122
============================================
- Hits            514      512       -2
  Misses           24       24
  Partials         57       57

Impacted Files
Coverage Δ
Complexity Δ

...eline/patterndetector/prefixspan/PathEnumerator.kt
100% <100%> (ø)
19 <0> (ø)
:arrow_down:

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 c30524c...891abc2. Read the comment docs.


removed setInterval as there is no clearInterval

This keeps running even after the module has initiated. It also
caused double initialisation

The function called here first checks if the form in question already has the functionality applied or not.
Do let me know if you can reproduce any actual errors, or double initialisation; it should not happen at all.
As a general rule of thumb with our extensions, we try to avoid having any interval checks in.
While it does seem like that the current version of Magento does not require this kind of loop to keep the integration active, this line is part of the extension's code since Magento 2 beta. I'd need to verify whether or not all versions of Magento 2 keep the forms intact, when going back and forth on the checkout.
What the code is supposed to catch, is cases where Magento rebuilds the form from scratch within the same page.
I'll try to go over all major supported versions to confirm whether or not we can merge this change in.
The fact that on M2.2 this code is definitely not required gives me a bit of hope.

I confirmed last week that the removal of setInterval creates a race condition against Magento2 loading the checkout page. If you have applied this code to your own shop, I would highly suggest reverting it, as it will make the extension fail if your shop is under heavy load.
I also double checked, and there are no double initialisation, and the interval will not cause any issues. But it does allow Magento & many checkout extensions to dynamically reload the address forms, also to load shipping & billing forms at separate times.
The callback function in the interval is called with a significant delay (200ms) and exits swiftly if there are no address forms where the code needs to be applied. 200ms is an ideal midpoint between reacting to a re-appearing address form before the user interacts with the form, and not packing too many things to the JS message queue. This should result in the code having no visible performance effects at all, even on mobile devices.


View / ViewModel Activation

Consider the following ViewModel constructor:
public MyBrokenViewModel()
{
    UserError.RegisterHandler(x => {
        // NB: Stuff
    });
}

This is broken because every time we create MyBrokenViewModel, we create another error handler. What Do? What we really want for certain global things like UserError, is for UserError to be subscribed only when the View associated with the ViewModel is visible. However, that information isn't available to ViewModels, and even if it was, it's not super obvious. Let's fix it
How does this work:
Activation allows you, for both Views and ViewModels, to set up the things that should be active when the View is visible. Here's how you do it for ViewModels:
public class MyWorkingViewModel : ReactiveObject, ISupportsActivation
{
    public ViewModelActivator Activator { get; protected set; }

    public ActivatingViewModel()
    {
        Activator = this.WhenActivated(d => {
            // d() registers a Disposable to be cleaned up when
            // the View is deactivated / removed
            d(UserError.RegisterHandler(x => {
                // NB: Stuff
            }));
        });
    }
}

Here's how it works for Views:
public class MyWorkingView : UserControl, IViewFor<MyWorkingViewModel>
{
    public ActivatingView()
    {
        this.WhenActivated(d => {
            Console.WriteLine("Helloooooo Nurse!")
            d(Disposable.Create(() => Console.WriteLine("Goodbye, Cruel World")));
        });
    }
}

Note that calling WhenActivated in a View automatically means that the associated ViewModel gets notified for activated / deactivated changes (and in fact, you must call WhenActivated in the View to get the ViewModel to be notified).
TODO:

[x] Finish Cocoa Views (waiting on #502)
[x] Testability (i.e. a 'With' type method for activation)
[x] How do subclasses of VMs work?

Hello,
What do you recommend to use for activating the view models only when controls are made visible? (other than subscribe on VisibleChanged)
By checking the implementations of IactivationForViewFetcher for both winforms and wpf, I can see that activation can happen even if a control does not become visible.
Eg. For Winforms, when having a user control that is not visible by default, the activation happens on HandleCreated..
Eg. For WPF activation will happen for user controls that are not visible due to activation on Load.
Regards


Fix runOnConsent running even if there is no vendorId with consent

runOnConsent was not checking for vendorId item to be present on the consents object.
That was causing the wrapper to run, even if no corresponding vendorId was present.
This PR fixes this. It also adds tests for the runOnConsent function.

@naripok cheers! Can you please tag reviewers on the PR, so we get notifications :)

@naripok I think this is a good time to think about this API, we are the only current user of the library, and even in our case it is very awkward (BTW I know you did not write the initial code).
Here is our code, but cut down to the parts which matter:
export const run = async (): Promise<void> => {
  const omitGdprConsent = edktInitializor.disableGdpr || false;
  const vendorIds = [1,2,3];
  let eventStatus;

  if (omitGdprConsent) {
    // log...
  } else {
    const consentStatus = await checkConsentStatus(vendorIds);
    eventStatus = consentStatus.eventStatus;

    // Log analytics with the consent status if the CMP UI is not shown
    // Only run it here before edgekit runs if there's no consent
    if (
      !consentStatus.hasConsent &&
      (eventStatus === 'tcloaded' || eventStatus === 'useractioncomplete')
    ) {
      logAnalytics();
    }
  }

  // Only fetch features and audiences and run edgekit if there is consent
  try {
    const { featuresResponse, audienceDefinitions } = await runOnConsent(
      vendorIds,
      async () => {
        const features = DISABLE_FEATURES
          ? null
          : await getFeatures(apiKey, accountId, publisherId, loadId);

        const audiences = await getAudiences(apiKey, accountId, loadId);

        return { featuresResponse: features, audienceDefinitions: audiences };
      },
      omitGdprConsent
    );

    let features: FeaturesResponse['features'];
    let metadata: FeaturesResponse['metadata'];
    if (featuresResponse) {
      features = featuresResponse.features;
      metadata = featuresResponse.metadata;
    }

    await edkt.run({
      pageFeatures: features,
      pageMetadata: metadata,
      audienceDefinitions,
      vendorIds,
      omitGdprConsent,
    });

    // User has consent if edgekit managed to run
    const hasConsent = true;

    // Run if there is consent
    const tcfLoadedAndUserHasConsent =
      hasConsent &&
      (eventStatus === 'tcloaded' || eventStatus === 'useractioncomplete');

    // Log analytics if CMP UI was shown earlier and edgekit ran (UI was updated and consent was granted)
    if (
      tcfLoadedAndUserHasConsent ||
      eventStatus === 'cmpuishown' ||
      omitGdprConsent
    ) {
      registerBidResponseEventCallback(apiKey, loadId);
      logAnalytics(
        apiKey,
        accountId,
        publisherId,
        hasConsent,
        edkt.getMatchedAudiences(),
        loadId
      );
    }
  } catch (e) {
    // TODO @naripok add `run` test suite
    debug(e);
  }
};

The issue I have with this - there is a lot of logic, that a consumer must implement just to do the base thing which is only to run on consent. I think we should expose some nice methods, which would tidy up this sort of code.
Let me know if you have some ideas or you want me to draft up an API :)

The issue I have with this - there is a lot of logic, that a consumer must implement just to do the base thing which is only to run on consent. I think we should expose some nice methods, which would tidy up this sort of code.
Let me know if you have some ideas or you want me to draft up an API :)

Dude, I do agree!
I was actually thinking about that, but I thought the refactor was out-of-scope.
We actually check for consent, then we wait for it again (using the same check method)...
Also, the runOnConsent is declared in os-edkt but only used once here.
I'll draft a refactor for it and post it here so we can agree on a good implementation. Is it ok?


Documentation for Bot.change_presence missing a warning/note

Link
https://github.com/Rapptz/discord.py/issues/6517

Eh running two instances of a bot is a really bad idea and iirc is documented elsewhere so this is not needed.


Version 7.1.9 - trying to download Surface Pro 3 drivers (1909) - error 432 BadLink

I've cleared out the temp folder.  It's been a long time since I've had to download Surface drivers.

Windows 10 1909 not supported in the current XML. Alternative suggestion is to package manually if modern driver management matching packages are required.


💡 [FEATURE] - Create Projects Section

Description:
The projects section is a central part of our project, showcasing open-source projects and making it easy for users to explore them. This issue focuses on creating the Projects.jsx component in the components folder, implementing a search bar, and designing visually appealing project cards.
Steps to Contribute:

Create Projects.jsx: Develop the Projects.jsx component in the components folder.
Search Bar: Implement a search bar for users to find specific projects easily.
Grid Layout: Design a grid layout for project cards that display project titles, links, and images.
Styling: Apply styles to make the projects section visually appealing and user-friendly.

By contributing to this issue, you'll help create an effective and visually pleasing way for users to discover open-source projects.
Your efforts in designing the projects section are highly appreciated! Let's make it easy for users to explore and engage with our projects.

Hey @Pinaka-Pani-18 , can I work on this? I would use Tailwind for styling. Do you want me to add dummy data temporarily or should I use the GitHubAPI to fetch the projects?

Hey @Pinaka-Pani-18 , can I work on this? I would use Tailwind for styling. Do you want me to add dummy data temporarily or should I use the GitHubAPI to fetch the projects?

Yes, @akash0708, you can work on this issue and use dummy data for now. In the future, we will look into the GitHub API if needed.

Hi @akash0708 any update ?

Hello @Pinaka-Pani-18 , I'm extremely sorry for delaying this issue. I had my exams going on. However, I'll be done by today and then start working on the issue. I will get the work done by the weekend. Please bear with me. Thanks and apologies!

Hello @Pinaka-Pani-18 , I'm extremely sorry for delaying this issue. I had my exams going on. However, I'll be done by today and then start working on the issue. I will get the work done by the weekend. Please bear with me. Thanks and apologies!

No problem, take your time, but please ensure you provide updates.


Fix broken links and remove deprecated xip.io statement

This PR also fixes some typo.

Already updated per comments. PTAL. Thanks~ @btat


rustc: fix aarch64 by disabling two doc tests

The patch is based on hydra's output and untested: https://hydra.nixos.org/build/72883318/nixlog/1
Those did not compile for some reason:
failures:
---- num/mod.rs - num::u32::next_power_of_two (line 2213) stdout ----
error: Error during translation/LLVM phase.
thread 'rustc' panicked at 'couldn't compile the test', librustdoc/test.rs:298:13
---- num/mod.rs - num::u8::is_ascii_punctuation (line 2623) stdout ----
error: Error during translation/LLVM phase.
thread 'rustc' panicked at 'couldn't compile the test', librustdoc/test.rs:298:13
failures:
num/mod.rs - num::u32::next_power_of_two (line 2213)
num/mod.rs - num::u8::is_ascii_punctuation (line 2623)
Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

replaced by https://github.com/NixOS/nixpkgs/pull/39000

Success on x86_64-darwin (full log)
Attempted: rustc
Partial log (click to expand)
/nix/store/y2xgc99ybahiml2camiycm592gsi8mc7-rustc-1.25.0

It's not those two specific tests, just intermittent failures. You see the same LLVM error in https://nix-cache.s3.amazonaws.com/log/whsyh46wr0zfkw420xcc8ffgahszsd80-rustc-1.24.0.drv as well.

Failure on aarch64-linux (full log)
Attempted: rustc
Partial log (click to expand)
   Compiling syntax_pos v0.0.0 (file:///build/rustc-1.25.0-src/src/libsyntax_pos)
   Compiling rls-span v0.4.0
   Compiling rls-data v0.15.0
   Compiling rustc_errors v0.0.0 (file:///build/rustc-1.25.0-src/src/librustc_errors)
   Compiling backtrace v0.3.5
   Compiling rustc_const_math v0.0.0 (file:///build/rustc-1.25.0-src/src/librustc_const_math)
   Compiling proc_macro v0.0.0 (file:///build/rustc-1.25.0-src/src/libproc_macro)
   Compiling syntax_ext v0.0.0 (file:///build/rustc-1.25.0-src/src/libsyntax_ext)
building of '/nix/store/zk2h9j7sl1z8665wx8qvgfdcv7xkgaga-rustc-1.25.0.drv' timed out after 3600 seconds
error: build of '/nix/store/zk2h9j7sl1z8665wx8qvgfdcv7xkgaga-rustc-1.25.0.drv' failed

Failure on x86_64-linux (full log)
Partial log (click to expand)
Merge failed


[decompiler] detect countdown

Pull Request Test Coverage Report for Build 973099333

47 of 52 (90.38%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 69.378%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

decompiler/analysis/insert_lets.cpp
35
37
94.59%

decompiler/IR2/Form.cpp
12
15
80.0%

Files with Coverage Reduction
New Missed Lines
%

decompiler/IR2/Form.cpp
1
73.6%

Totals

Change from base Build 973065261:
0.04%

Covered Lines:
36725

Relevant Lines:
52935

💛  - Coveralls


Options for yaml formatting for git writeback method

Is your feature request related to a problem? Please describe.
I have disabled indentation rule for yamllint because the the git writeback method keeps modifying the indentation of the yaml files.
Describe the solution you'd like
There should be options to configure formatting patterns for the git writeback method.
Describe alternatives you've considered
Change yaml formatting standards.
Additional context
Looks like indentation is off:

#502


Bug 1748271: Copy global CA data to builds

…builds
/assign @bparees
@openshift/openshift-team-developer-experience FYI
per recent discussion on devex and warroom-proxy slack around https://bugzilla.redhat.com/show_bug.cgi?id=1748271
this consumes the config map created and labeled by ocm-o in PR https://github.com/openshift/cluster-openshift-controller-manager-operator/pull/118, vs. creating/labeling a configmap for each build, to access the global CA data.
If we are happy with this new approach, I'll label this PR for the bugzilla

generally looks correct but waiting on https://github.com/openshift/cluster-openshift-controller-manager-operator/pull/118 to be resolved.

/retest
now that the ocm-o pr that creates the desired config map has merged

/retitle Bug 1748271: Copy global CA data to builds

accepted all your suggested commits @adambkaplan , pulled down, squashed, and pushed back up ... PTAL

ok buils passing now .... seem to be down to infra flakes
/retest

pushed rebase after disconnected mirror merge

terraform flake on e2e-aws-upgrade
/test e2e-aws-upgrade

TBR flake on e2e-aws, but e2e-aws-upgrade avoided flakes this time
/retest


add possibility to support the project

I really enjoy what you built here. Is there a way to support your work? I was searching for a solution to integrate my Liontron LiFePo4 Batteries into home-assistant for months. now finally, there is a comfortable solution for this :)
Please think about adding a PayPal donation link so I can buy you a beer or two 🍺

Fully agree, you did a pretty good job.


Confusing duplicate sections "Custom resources and Server-Side Apply" and "Server-Side Apply for custom resources" in "Server-Side Apply" page

Below sections are identical causes confusion. Suggest keeping one.
Custom resources and Server-Side Apply
Server-Side Apply for custom resources

/language en

/retitle Confusing duplicate sections "Custom resources and Server-Side Apply" and "Server-Side Apply for custom resources" in "Server-Side Apply" page

/wg api-expression

That's a bug. Cleanup is welcome (you can delete either of the duplicate sections).
/triage accepted
/kind bug
/help
/priority backlog

@sftim Can i get assigned ?

Fixed by https://github.com/kubernetes/website/pull/43814
/close


[Bug-FE] [WorkFlow Instance] Workflow name copy function does not work

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Workflow name copy function does not work
What you expected to happen
The first click of the copy button does not work, the second click does
How to reproduce

run workflow
go to workflow instance
click workflow instance name
click the copy workflow instance name button

Anything else

Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I will fix it


That's not an issue

please tell me the of name font used in social Fish Banner.

Its not a font. I draw ASCII arts.


raise better error message if trying to make a callback with an incomplete set of inputs

Right now, we just get a "Cannot concat on undefined".

I think this may have been fixed when #60 was merged. From experimenting it seems that:

inputs = [] -> callback is never fired
inputs has length less than the number of arguments -> a server error is thrown
inputs has proper length, but one of the ids does not exist in the DOM at the time the callback is fired -> an informative front-end exception is thrown due to #60

Is there an example of a dash app producing that error?

I'm assuming this has been addressed by some of our recent devtools improvements, but @chriddyp if there's still a problem here please open a new issue in dash.


Retry wallet lock for syncing on-chain wallet

Closes #593

Awesome. Last thing, in nodemanager.rs, should we change this:
                if let Err(e) = nm.sync().await {
                    log_error!(nm.logger, "Failed to sync: {e}");
                }

                // if this is the first sync, set the done_first_sync flag
                if !synced {
                    let _ = nm.storage.set_done_first_sync();
                    synced = true;
                }

And if error then synced does not equal true?

did it with the else if statement


[Bug]: Can't call the camera in iframe

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Using the iframe widget, can't call the camera in iframe
Steps To Reproduce
1.iframe URL : https://blog.minhazav.dev/research/html5-qrcode.html
Public Sample App
No response
Version
Self Host 1.6.6

Accessed the url mentioned in the bug description through iframe widget, and camera is now working both on canvas and on deploy.


Only launch browser when interactive mode

Switches the default value of the launch_browser argument to interactive() so it won't try and launch when running non-interactively e.g. in RSW etc..

Sorry Nick! GitHub notification filters in my inbox are terrible right now - looks like we got this all sorted through the slack discussions so yay!


Remove OpenSSL 1.x dependency for Agent 3.x

It seems that the installdependencies.sh script still installs libssl1.x (at least on Ubuntu 22.04) (see comment below)
The script should be adjusted to use the latest supported version of OpenSSL now that agent is .NET6.
See https://github.com/microsoft/azure-pipelines-agent/issues/3922#issuecomment-1464875424 for details.

Hi @MichelZ, thanks for reporting! We are working on more prioritized issues at the moment, but will get back to this one soon.

Starting 3.225.0 all scripts are capable with libssl.
Closing this issue


App asking for Calendar usage when not requesting. Removing description crashes app

React-Native version: 0.55.3
Platform: iOS
Device (which one?), simulator?
OS version: High Sierra 10.13.5
react-native-permissions version: 1.1.1
Devtools: Xcode

I've followed the setup in the README and added the usage descriptions in the Info.plist
When I run the app for the first time, it automatically requests for my calendar usage even though I have not requested it...
Is there a way to not request this or any permissions for that matter? I tried to remove the description from the Info.plist file and that resulted in my app not building.
The only permission I need are the camera and microphone.

Ran into same problem! I'd like to know how to do this.

I am closing this since this issue does not exists anymore with the 2.0.0 release.


Scenario Generation 2.0

Development of more generic and modularizable scenario generation

Configurable ScenarioGeneration implemented.
It can be configured based on Config Readers for:

agent initial states and shape config
Behavior, execution and dynamic model configs
goal configs
controlled agent configs
Each config is controlled via config reader, config readers are exchangable to create a large variety of scenarios.


Update TestCreateOrg.java

Changed the TestCreateOrgButton into TestCreateOrg, since it was always on the way to making a organization.
Added the extra steps of changing to the create organization form, filling out the form and checking if we can submit the form. If we can submit the form, it means the test was successful. Since we can't get feedback from the backend yet (Since it's not made), we will use the button as active as our result if it worked or not.

I can come with a little comment on this closed PR. When taking into account that this page will be part of the actual application, it might be better to leave the "Test" part of the naming so instead of TestCreateOrg, it would be CreateOrg.


Fix broken links

The previous URL (https://developer.mindsphere.io/apis/advanced-assetmanagement/api-assetmanagement-api-swagger-3-9-0.html) contains the version of the API. The version now is 3.15.0, so the URL is also changed accordingly. To prevent the links from becoming outdated again in the future, it is better to direct the user to the Available API Versions, and then user can just choose the available API version for Europe 1.

@Luyao-Zhang-1 - good spot.
I see that the first reference to this, on line 22, has already been changed.
Because this is MindSphere documentation, I'll just get someone from MindSphere to check that this is the best change.

I'll leave you to delete the branch, rather than doing it for you.


Test for KView and KViewGroup

Thanks, now just update travis build script to run connectedCheck task after build. You'll also need to create and run android emulator. If you have any problems with this, let me know so I'll assist you.

I can`t realize how to run emulator on travis without errors

Fixed

Cool, great job! 💯


Fewer log warnings for connection retry

Is your feature request related to a problem? Please describe.
Make your adapter less aggressive with connection problems in the log.
Describe the solution you'd like
Is it possible if the connection is lost, first carry out a few retrys and treat the connection loss as info in the log.  Only after several failed attempts then issue a warning.  Unfortunately, the adapter currently spams the log immediately.

Can you provide some example logs?
There may be some different log messages. On simple connect/disconnect/reconnect events the messages should be logged as debug messages only.

2023-07-03 10:33:50.035 - warn: discord.0 (234760) Discord client websocket error: Error: getaddrinfo EAI_AGAIN gateway.discord.gg
2023-07-03 10:33:50.053 - error: discord.0 (234760) Discord client websocket error (shardId:0) Error: getaddrinfo EAI_AGAIN gateway.discord.gg
2023-07-03 10:33:50.054 - info: discord.0 (234760) Terminated (START_IMMEDIATELY_AFTER_STOP): Discord client websocket error
2023-07-03 10:33:53.801 - info: discord.0 (318092) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:34:14.177 - error: discord.0 (318092) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:34:14.179 - info: discord.0 (318092) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:34:17.443 - info: discord.0 (318112) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:34:37.798 - error: discord.0 (318112) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:34:37.800 - info: discord.0 (318112) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:34:41.081 - info: discord.0 (318129) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:35:01.438 - error: discord.0 (318129) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:35:01.440 - info: discord.0 (318129) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:35:04.712 - info: discord.0 (318149) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:35:25.293 - error: discord.0 (318149) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:35:25.294 - info: discord.0 (318149) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:35:28.585 - info: discord.0 (318166) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:35:48.938 - error: discord.0 (318166) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:35:48.939 - info: discord.0 (318166) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:35:52.205 - info: discord.0 (318181) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:36:12.564 - error: discord.0 (318181) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:36:12.566 - info: discord.0 (318181) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:36:15.886 - info: discord.0 (318201) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:36:36.242 - error: discord.0 (318201) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:36:36.243 - info: discord.0 (318201) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:36:39.502 - info: discord.0 (318218) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:36:59.852 - error: discord.0 (318218) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:36:59.853 - info: discord.0 (318218) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:37:03.167 - info: discord.0 (318239) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:37:23.573 - error: discord.0 (318239) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo EAI_AGAIN discord.com
2023-07-03 10:37:23.574 - info: discord.0 (318239) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:37:26.868 - info: discord.0 (318254) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:37:32.253 - error: discord.0 (318254) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo ENOTFOUND discord.com
2023-07-03 10:37:32.254 - info: discord.0 (318254) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:37:35.526 - info: discord.0 (318271) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:37:35.859 - error: discord.0 (318271) Discord login error: FetchError: request to https://discord.com/api/v9/gateway/bot failed, reason: getaddrinfo ENOTFOUND discord.com
2023-07-03 10:37:35.860 - info: discord.0 (318271) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-07-03 10:37:39.156 - info: discord.0 (318286) starting. Version 1.1.6 in /opt/iobroker/node_modules/iobroker.discord, node: v18.16.0, js-controller: 4.0.24
2023-07-03 10:37:40.316 - info: discord.0 (318286) Discord client websocket connected (shardId:0)
2023-07-03 10:37:40.318 - info: discord.0 (318286) Logged in as ioBroker Bot#XXXX

Thank you!
I'll check and optimize this for the v2.0 stable release.

Thank you. I look forward to the release.

@pafade89 The issue is now fixed on the main branch and will be included in the next release.
I had already a login error detection and some logic to handle these errors.
Unfortunately, this didn't work with your type of error. EAI_AGAIN means there was a DNS error and the domain discord.com could not be resolved.
Now it's working as intended (no more adapter restart loops).
Additionally the first four failed tries are logged as info only now. All following failed tries will be logged as warning.
2023-07-09 15:42:41.492  - info: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:42:41.498  - info: discord.0 (3001) Wait 5 seconds before next login try (#2) ...
2023-07-09 15:42:52.747  - info: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:42:52.748  - info: discord.0 (3001) Wait 10 seconds before next login try (#3) ...
2023-07-09 15:43:06.139  - info: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:43:06.140  - info: discord.0 (3001) Wait 30 seconds before next login try (#4) ...
2023-07-09 15:43:38.439  - info: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:43:38.445  - info: discord.0 (3001) Wait 60 seconds before next login try (#5) ...
2023-07-09 15:44:40.298  - warn: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:44:40.300  - info: discord.0 (3001) Wait 120 seconds before next login try (#6) ...
2023-07-09 15:46:40.319  - warn: discord.0 (3001) Discord login error: Error: getaddrinfo EAI_AGAIN discord.com
2023-07-09 15:46:40.320  - info: discord.0 (3001) Wait 120 seconds before next login try (#7) ...

The wait time will increase up to 10 minutes (10+ failed tries).
Thank you again for reporting this!

Perfect, I will test it as soon as the release is available. Thanks for the support.

v2.1.0 is now released on NPM and should be in the latest repo tomorrow. :)

@crycode-de  I think it's not working as expected. Here the router got a new IP and it looked like this:
2023-09-24 00:23:43.424 - warn: discord.0 (78338) Discord client websocket error: Error: getaddrinfo EAI_AGAIN gateway-us-east1-b.discord.gg
2023-09-24 00:23:43.440 - error: discord.0 (78338) Discord client websocket error (shardId:0) Error: getaddrinfo EAI_AGAIN gateway-us-east1-b.discord.gg
2023-09-24 00:23:43.442 - info: discord.0 (78338) Terminated (START_IMMEDIATELY_AFTER_STOP): Discord client websocket error
2023-09-24 00:23:56.962 - info: discord.0 (149988) starting. Version 2.1.2 in /opt/iobroker/node_modules/iobroker.discord, node: v18.17.1, js-controller: 4.0.24
2023-09-24 00:24:07.311 - info: discord.0 (149988) Discord login error: ConnectTimeoutError: Connect Timeout Error
2023-09-24 00:24:07.312 - info: discord.0 (149988) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-09-24 00:24:10.480 - info: discord.0 (150008) starting. Version 2.1.2 in /opt/iobroker/node_modules/iobroker.discord, node: v18.17.1, js-controller: 4.0.24
2023-09-24 00:24:20.824 - info: discord.0 (150008) Discord login error: ConnectTimeoutError: Connect Timeout Error
2023-09-24 00:24:20.825 - info: discord.0 (150008) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-09-24 00:24:24.019 - info: discord.0 (150023) starting. Version 2.1.2 in /opt/iobroker/node_modules/iobroker.discord, node: v18.17.1, js-controller: 4.0.24
2023-09-24 00:24:34.350 - info: discord.0 (150023) Discord login error: ConnectTimeoutError: Connect Timeout Error
2023-09-24 00:24:34.351 - info: discord.0 (150023) Terminated (START_IMMEDIATELY_AFTER_STOP): No connection to Discord
2023-09-24 00:24:37.577 - info: discord.0 (150041) starting. Version 2.1.2 in /opt/iobroker/node_modules/iobroker.discord, node: v18.17.1, js-controller: 4.0.24
2023-09-24 00:24:43.692 - info: discord.0 (150041) Discord client websocket connected (shardId:0)
2023-09-24 00:24:43.694 - info: discord.0 (150041) Logged in as ioBroker Bot

Can you maybe take another look?
Thank you =)

@pafade89 Oh... the ConnectTimeoutError is new and currently not handled. Thank you for reporting.
I'll add this to the possibly expected errors.
Also the "Discord client websocket error" is handled twice but differently. :thinking:
I'll check this too.

@pafade89 Release v2.2.0 is out now and includes an optimized error handling.
The adapter should no longer restart in case of a network related error. Instead it tries to (re)connect automatically and the info.connection state is set to false.
Once a connection was established to the Discord servers, a connection error will be logged only once as warning. All further (same) errors are only logged as debug to not spam the normal log.
v2.2.0 should be available in the latest repo tomorrow. If there are no bugs reported after some time, this will go to the stable repo. :)

@crycode-de
Thank you very much for your effort. I will start testing the version from tomorrow and will get back to you if necessary.
Cheers P

@crycode-de
Yesterday, my internet briefly went down, and I only got this warning:

No more spam. Thanks for the quick fix!

@pafade89 Thank you for your reply!
v2.2.0 will be in the stable repo soon. 🙂


fix: fix top page of dark mode

Before

After

closes #155

@kt3k there's an error at _document.tsx.

I think you should use className instead

@ItaloSa Thanks!

something's wrong with CLA bot?


go grpc and c++ grpc installed in same machine

Hi,
We have a ubuntu 20.04 based root file system and it already has grpc go 1.41.0 which used by go based apps. For one of the c++ libraries that we developed requires gRPC c++ source installation specifically 1.32.0 version.
Can different versions of go gRPC and c++ gRPC co-exist in machine? Does it cause an runtime issues for the go applications?

Short answer is yes. They should be able to coexist.
For general questions and troubleshooting, please ask/look for answers at StackOverflow, with "grpc" tag: https://stackoverflow.com/questions/tagged/grpc


React.Children.only

React.Children.only expected to receive a single react element child

You're passing 0 or > 1 React elements somewhere where you can't. You'll have to debug your code and figure out what's going on.


ERROR: Unable to verify the server's certificate.

@rashedtalukder Trying to connect to AWS IoT Core by using the code from Blinky-Hello-World project in my project. Here is the error I am getting:
␛[0;32mI (6566) STOCKLY: Connecting to AWS IoT Core at xxxxxxxxxxxxxx-ats.iot.us-west-2.amazonaws.com:8883␛[0m
␛[0;32mI (6616) aws_iot: Attempting to use device certificate from ATECC608␛[0m
␛[0;31mE (7666) aws_iot: failed! mbedtls_ssl_handshake returned -0x2700␛[0m
␛[0;31mE (7666) aws_iot:     Unable to verify the server's certificate. ␛[0m
␛[0;31mE (7666) STOCKLY: Error(-4) connecting to xxxxxxxxxxxxxx-ats.iot.us-west-2.amazonaws.com:8883␛[0m

main.c: https://github.com/Raunak-Singh-Inventor/Stockly/blob/c250a4fc9c2d20f53c8c7a42fe9bb90174529120/edukit_deployment/main/main.c#L88

Appears to be something wrong with the root CA. That's actually a separately attached file that should be included in the certs directory.

@rashedtalukder I have already included the aws-root-ca.pem file in main/certs. Could you give some pointers as how I could fix the root CA as it shows as active on the AWS console.

@rashedtalukder Any updates as to why the certificate is not valid? I really want to complete my project.

Hi Rasheed,
I am still waiting on your response?

Hey Raunak, I’m no longer with amazon as of a few weeks ago and don’t have access to some of the tools that would let me better help you. I’m hoping that someone from AWS will help you. If they don’t, let me know and I’ll schedule some time with you to debug it in a video call.

@rashedtalukder Thanks for the reply.
Could you please drop me an email with your new contact id.


Rotations for primitives

We have .translate(). It should be possible to .rotate() a primitive.

How should the center of rotation be defined? It could use the bounding box, or allow specifying an arbitrary point.
The easiest way to do this is to use f32s in a 2x2 matrix, but this requires sin()/cos() which is super slow on targets without FPUs. micromath or libm might help.

In addition to allowing rotation by an arbitrary angle we could also add rotation by 90° increments. Apart from possible performance advantages, this result of this rotation will always have the same type as the original primitive.
What return type would Rectangle::rotate for an arbitrary angle have? We can't return a Polygon, because it uses a borrowed slice.


time.Parse fails at parsing time.String

time.Parse can not parse the default format produced by time.String()
What version of Go are you using (go version)?
go1.7.1 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="$HOME/.local/share/go"
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build766838131=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
What did you do?
https://play.golang.org/p/wmfQP5wX0S
func main() {
	t := time.Now().UTC()
	s := t.String()

	t, err := time.Parse(s,s)
	if err != nil {
		fmt.Println(s, "parsed with error", err)
	} else {
		fmt.Println(s, "equals", t.String())
	}
}

What did you expect to see?
2016-09-27 16:53:03.13465009 +0000 UTC equals 2016-09-27 16:53:03.13465009 +0000 UTC

What did you see instead?
There were different out comes, local:
2016-09-27 16:53:03.13465009 +0000 UTC parsed with error parsing time "2016-09-27 16:53:03.13465009 +0000 UTC": month out of range

go playground:
2009-11-10 23:00:00 +0000 UTC parsed with error parsing time "2009-11-10 23:00:00 +0000 UTC" as "2009-07-10 23:00:00 +0000 UTC": cannot parse "9-11-10 23:00:00 +0000 UTC" as "009"

Not a bug. The result of String is not a valid format.


Upgrade ScottGarman from triage to push role

As work is ramping up to thin out/eliminate the Equinix Metal specific OS installers from the Boots codebase, I will likely be heavily involved in code review. This will allow my PR approvals to unblock merges.
Requirements:

I have reviewed the community membership guidelines
I have enabled 2FA on my GitHub account
I have subscribed to the tinkerbell-contributors e-mail list
I am actively contributing to 1 or more Tinkerbell subprojects

Here is a list of non-trival PRs I have been the primary reviewer on:
https://github.com/tinkerbell/boots/pull/26
https://github.com/tinkerbell/boots/pull/211
https://github.com/tinkerbell/boots/pull/212
https://github.com/tinkerbell/boots/pull/215
I have also made a number of code contributions to this repository, here are a few of them:
https://github.com/tinkerbell/boots/pull/95
https://github.com/tinkerbell/boots/pull/278
https://github.com/tinkerbell/boots/pull/280
https://github.com/tinkerbell/boots/pull/281
I'll note that I feel most familiar with the OS installer code in Boots, and would generally keep my PR approvals related to those parts of the codebase I know best.
If someone would be willing to sponsor this request, I would appreciate it.

Hey @ScottGarman, thanks for wanting to taking on a bigger role! Would you mind structuring this PR to show you have met the requirements from the governance doc here? Here are a couple examples from other repos:

https://github.com/tinkerbell/.github/issues/35
https://github.com/tinkerbell/hegel/pull/97

Thanks for pointing me to the requirements for this @jacobweinstock. I think I've added the necessary details to this PR. Of course I still need a sponsor for this. I'm hoping you might be willing to do that? I am open to any and all feedback in the course of that, if you have any concerns you wish to raise.

Thanks for pointing me to the requirements for this @jacobweinstock. I think I've added the necessary details to this PR. Of course I still need a sponsor for this. I'm hoping you might be willing to do that? I am open to any and all feedback in the course of that, if you have any concerns you wish to raise.

This is great! thanks @ScottGarman ! I will definitely sponsor you for this!

FYI, Looks like we will still need @mmlb to approve this PR. Didn't think we would need both maintainers to approve this but looks like it. I can't remove Manny as a reviewer.


Eventual Consistency

Does yas3fs do anything to deal with S3 only providing eventual consistency for read-after-write?
For example, say a node receives an SNS notification that a file has changed and so invalidates its cache. If the node then requests the file from S3 the new version of the file is not guaranteed to be returned. If the old version of the file is then cached it will not be invalidated until the file is updated again.
Does yas3fs somehow wait for the new version of the file? Or does this turn out to not be an issue in practice? Or am I misunderstanding something :)?
Thanks for the awesome project!

I don't believe we do, it would probably behoove us to send the new ETag in change alerts and keep a lisr of list of those changes so we can poll headObject until S3 has synchronized.
PRs are welcome :)


Improve status indicator in Action Icon when score is calculated.

We currently just use a solid square color.
In this issue let's brainstorm some designs for what it could look like.

What if we showed three little github icons in a horizontal line. These would be like a traffic light. Left icon green, middle is yellow and right is green.
All the icons would be black and white unless its the repos current 'health'

We could try that but its quite small in the browser UI.

How about something like this?

I like it. We would need to make sure we have colors for all the grades that contrast well enough with white text.

@cmgriffing Do we possibly need to revisit the default background colors that are being set now?
I came up with this but don't particularly like it that much 😬 .

I think the badgeText is a good addition that makes this issue change course a bit.
Instead, let's conentrate this on improving out colors like the 100% green and the failing red.

Thinking about Violet or something of the sort. Decided to check out burning flames to see what popped out at me, and seemed alright. Perhaps the turquoise? I don't think anyone can deny the radiance of these flames, but not sure if they are the solution to this long-lasting problem.


Added my profile Json file

Package file is deleted now 😅

Package file is deleted now 😅

bro , have i committed the mistake again?

bro , have i committed the mistake again?

Yes, the issue is you made all the mistakes in your main branch, I suggest you delete your fork, make a new fork, make a new branch in the fork called umaresso-profile or patch-1 and do the required edits.

okey ! Let me do that


Feature Request template: Update title naming conventions

Switches from the curriculum issue naming convention to one more suited for this repository

Because

The current title naming that is suggested is taken straight from the curriculum, and doesn't fit this repository

This PR

Updates it to one more suited to this repository, based on previous titles of issues

Issue

Closes #XXXXX
Additional Information

Pull Request Requirements

[x] I have thoroughly read and understand The Odin Project Contributing Guide
[x] The title of this PR follows the location of change: brief description of change format, e.g. Callbacks command: Update verbiage
x ] The Because section summarizes the reason for this PR
[x] The This PR section has a bullet point list describing the changes in this PR

Thank you!!


Download error(hstream.moe)

Source information
Hstream 14.7
Source language
English
Steps to reproduce
1.Open hstream
2.Try playing video
3. It shows null pointer exception

Expected behavior
Download started
Actual behavior
Not starting
Aniyomi version
<IP_ADDRESS>
Android version
Android 12
Other details
Well the online player works fine but the actual page doesn't load no description, no ep update, download struck at 0%.

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Exist in preview version too.


docs: add typography path in readme

Readme is missing the import to typography that is necessary for @include angular-material-mdc-typography();
Obs: Original was #21003, but I ended up breaking the CI and couldn't get it to work. Resubmitting to see if it works now.
My bad @jelbourn and @crisbeto

cc @josephperrott about the CI not running
@mrmokwa do you have CircleCI set up on your fork?

Based on the statuses of the commit message on the local fork, it looks like CircleCI is enabled on the fork (link).  @mrmokwa Unfortunately because of how CircleCI works, you cannot have CircleCI enabled on your fork and have it continue to trigger to run as expected on this upstream repository.

A weird side-effect of this is that the status from the fork is showing up for us on master:

I had it enabled and I disabled it now. Unfortunatelly I don't have any CI experience and thought I had to have it enabled in my fork.
Since this PR isn't essential, and it's causing problems, you can close it.
Want to thank all of you for your help/time. Really sorry for all the trouble.


A question in test.py script (for --show-dir)

                if out_dir:
                    out_file = osp.join(out_dir, img_meta['filename'])
                else:
                    out_file = None

                model.module.show_result(
                    img_show,
                    result,
                    show=show,
                    out_file=out_file,
                    score_thr=show_score_thr)

I have found the img_meta['filename'] is absolute path (such as "/data/image/1.png"), so the out_file will always retuen the original path, as a result, the original image file is modified.

I think you should modify as:
out_file = osp.join(out_dir, osp.basename(img_meta['filename']))

https://github.com/open-mmlab/mmdetection/pull/2612

Updated.


[Bug] Link Actor Senses does not work with additional game masters

Tested on a fresh foundry install with only dnd5e and adequate vision.
When you have multiple users with the game master role / permission, only the first game master (or the one which created the tokens / actors) has the linked actors senses. For any other game master the actor senses are not linked at all.
On line 156 in the script.js file you have this check:
const userIsObserver = actor.getUserLevel(game.user) >= CONST.DOCUMENT_OWNERSHIP_LEVELS.OBSERVER;
The problem is that actor.getUserLevel(game.user returns 0 for additional game masters and assistant game masters.
Maybe this is a core bug?

This behavior of getUserLevel is definitely weird. I'll investigate whether that is intended. testUserPermission works as expected for additional GMs if the optional parameter exact is false (if true, it doesn't, but that doesn't matter in AV use case).

Yep, was talking in the foundry discord about it. See this message and surrounding: https://discord.com/channels/170995199584108546/722566083499786352/1080511430010347572


Automated cherry pick of #1605: fix clone rbd image error

Cherry pick of #1605 on release/2.10.0.
#1605: fix clone rbd image error

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@5c8f4b1). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.10.0   #1611   +/-   ##
================================================
  Coverage                  ?   7.71%
================================================
  Files                     ?     409
  Lines                     ?   62373
  Branches                  ?       0
================================================
  Hits                      ?    4815
  Misses                    ?   57149
  Partials                  ?     409

Flag
Coverage Δ

#aFlag
7.71% <ø> (?)

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 5c8f4b1...a515278. Read the comment docs.

/lgtm
/approve


Use big.Int to store NEP5Balances

We store these balance as int64 now, but some tokens may easily overflow it and even GAS might become inflationary at some point and also require wider value. So it's better to switch to big.Int here.

Use it for claimable info too.

This is a bit tricky, as our optimization for TransferLog storage assumes NEP5TransferSize to be known. It is used for calculating starting position of each transfer in raw []byte. This must be changed somehow.


Added utility for changing accessor componentType

Added generic utility for changing accessor component type. This simplifies compressIntegerAccessors and will also be useful for adding uint32 indices support to combinePrimitives.
@lilleyse, could you look at this when you get a chance?

Looks good.


[Question] Hardware suggestion

I am currently planning to evaluate several locations (each of which has a 24h recording cut into 24 videos). Which hardware would you recommend for 720x480/30FPS videos? On my RTX 2080 it works with ~68FPS, but I want to get extra hardware for it.

Closing to clean up the issues as seems no one has knowledge to answer you.. feel free to answer yourself if you have figured this out


Adds some clarifications to mock function API docs

Summary
Added some clarifications to the mock function API docs where the existing documentation was incomplete or misleading.
Test plan
N/A

This is good! Thank you for adding clarifications.


🎯[Good First Issues]: Vanus Connector Template (Amazon EventBridge-Slack)

What is a Vanus Connector Template
Vanus connector templates are pre-configured templates for Vanus Cloud and Vanus Open-Source. It is written in JSON. These templates help users create easy-to-use configurations when they use Vanus SaaS or the Vanus OSS.
This templates issue is designed to send Cloud Events from Amazon Eventbridge to Slack
Features of Connector Template
The connector template should have the following variables

Time: Current Time
Bucket Name: The AWS S3 Bucket

Excepted Template Output
The Template output could be in the format below:
The account **<account>** just **<$.data.desc>** and the name is <bucketName>\n\n<time>.
How to claim to implement this template
If you want to solve this issue, please leave a comment on this issue like:

I'd like to develop this template, please assign it to me.

Vanus Community will assign the issue to you on time.

hello!
i would like to solve this issue.
is js/ts knowledge is enough?

Yes, it is
Look through other already existing templates to understand what you need to do. Thanks!

nice! will start tomorrow morning


Discord Web App on Firefox with Linux significantly slowed down

Description

I'm not sure if this should go under "Report Broken Site" since technically it's not broken, it's just slow. Basically the web app for Discord at discord.com has slowed down drastically for me in the past couple of days.
Particularly changing the channel view on Discord with the extension enabled can take several extra seconds. Disabling the DDG Privacy Essentials extension completely remedies the issue.
I'm using an up-to-date Arch Linux distribution.
$ uname -a
Linux </etc/hostname> 6.3.8-arch1-1 #1 SMP PREEMPT_DYNAMIC Wed, 14 Jun 2023 20:10:31 +0000 x86_64 GNU/Linux

Steps to Reproduce

Steps to reproduce should be obvious from the Description. I'm not sure what particular factors are essential to make the problem happen, but trying to use Discord on Firefox with an up-to-date Arch Linux installation is slow if DDG Privacy Essentials is enabled.
Expected behavior:

Somewhat responsive web app (let's be fair Discord isn't all that responsive to begin with...)
Actual behavior:

Extremely unresponsive Discord web app to the point where I often need to wait 10 seconds to do basic operations that typically take 1-2 seconds.
Versions

Extension: 2<IP_ADDRESS>
Browser:  Firefox 114.0.2 (64-bit)
OS: Linux, details in description if needed

Additional Information
Provide additional context (optional):

Is the issue repeatable or intermittent? The issue is constant though it appears to get slightly better the longer I use discord.com (possibly something gets cached?)
Did this issue start after an upgrade to a new extension version? If so, can you repeat it using a pervious extension version? It started happening recently, I assume after a recent update to DDG Privacy Essentials, but could've been a Firefox update too. I can make the effort to test older versions if it's helpful, but so far I haven't.
Are there any errors in the extension debugging console? I'm not sure how to check. Let me know if there's something I can do to provide more information.

Hi @oheikk, thanks for the detailed report! It was extremely helpful. We've now disabled the feature that was responsible for the performance issue you encountered, so you shouldn't run into this issue again. If by any chance you notice that you're still having performance issues with Discord, please don't hesitate to let us know here.

Huh, indeed the performance issue seems to be gone. :)

100415758
Duplicate of #


questsadmin stats  Error

My Minecraft username is Henry, and when I do /questsadmin stats henry, I get this error: http://pastebin.com/wED1KSh9 and an internal error occurred when doing the command with the wrong capitalisation.
It has to be the correct capitalisation of the username, is there anyway to fix this?
Also, some people on the server might want to see other's Quests Stats, could you maybe make the /quests stats command work like the following; when someone does /quests stats it gives them their stats, but when they do /quests stats 'player' it gives them the stats of another player? Thanks.

You're using a dev build that included the PlayerFinder class, which was not applied to later builds. Please update to a newer build.
Once you do, give players the 'quests.admin.stats' permission to allow them to check stats of other players.


Deploy to Netlify button fails

Environment
Deploy to netlify url.
Reproduction
Netlify lets us deploy a public repo automatically using a single click. Here is an example:
[![Deploy to Netlify](https://www.netlify.com/img/deploy/button.svg)](https://app.netlify.com/start/deploy?repository=https://github.com/pinegrow/happy-paws-with-nuxt-tailwindcss)

Netlify sample templates of other frameworks seem to work just fine with single-click button - https://github.com/netlify-templates
Describe the bug
When clicking on the above button, deployment fails at initializing stage. But, if I instead manually import this repo from netlify, deploy is working just fine.
Wondering if single-click deployment to netlify is possible with zero-config. Thanks.

Additional context
I can see build settings are not really configured automatically using this single-click method.

Build settings when I manually import the repo from netlify (expected):

Logs
No response

I figured it out. It worked if I changed "pinegrow" to "Pinegrow" in my repo url 😬. I didn't know github repo usernames can be case sensitive.


feat: add api launch option to logsim

PR Type

Improvement

Related Links
tier4 internal
Description
add api launch option to logsim to prevent chattering when using rosbag
Review Procedure
take a look at change.

Remarks
Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

I'll leave the review to Takagi-san.


Add adoption by GBIF

@roll can you have a look why this build is stuck?

@peterdesmet
Thanks! I fixed it; also added to the main page to replace one of the logos that don't have any name on it

Thanks! @timrobertson100, does this look ok to you: https://gbif-adoption.datapackage-6gp.pages.dev/overview/adoption/#community-projects

Thanks @peterdesmet - looks really great.
While writing - thank you all for reacting to the request to put information out about this.
It really is very appreciated and I believe will only help others grappling with the same question.


Flutter should provide tutorial for setting up universal link in iOS

Currently the https://docs.flutter.dev/development/ui/navigation/deep-linking only provide steps to set up custom schema for iOS, It should also provide steps to set up universal link.

Two ways to set up the universal link
through xcode:

open ios folder in xcode, in the signing & capacbilities tab
click + button on the top left in the page
select associate domain
in the newly create associate domain section, click + to add applinks:

manually:

open ios/Runner.xcodeproj/project.pbxproj in ide
search for CURRENT_PROJECT_VERSION = "$(FLUTTER_BUILD_NUMBER)"; (should have found three occurrences, one for each Debug,Profile,Release)
add the following line above the line CODE_SIGN_ENTITLEMENTS = Runner/Runner.entitlements; for all three occurrences
create a file ios/Runner/Runner.entitlements, with the following content

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>com.apple.developer.associated-domains</key>
	<array>
		<string>webcredentials:example.com</string>
		<string>applinks:example.com</string>
	</array>
</dict>
</plist>

i made these steps and it didn't work. i got this from system diagnose
Service:              applinks
App ID:               ...*
App Version:          33.0
App PI:               <LSPersistentIdentifier 0x105310f70> { v = 0, t = 0x8, u = 0x99c, db = ----**, {length = 8, bytes = 0x9c09000000000000} }
Domain:               example.com
User Approval:        unspecified
Site/Fmwk Approval:   denied
Flags:
Last Checked:         2022-12-06 08:50:43 +0000
Next Check:           2022-12-11 08:42:31 +0000
Service:              webcredentials
App ID:               ...
App Version:          33.0
App PI:               <LSPersistentIdentifier 0x105310f70> { v = 0, t = 0x8, u = 0x99c, db = **----, {length = 8, bytes = 0x9c09000000000000} }
Domain:               example.com
User Approval:        unspecified
Site/Fmwk Approval:   denied
Flags:
Last Checked:         2022-12-06 08:50:43 +0000
Next Check:           2022-12-11 08:42:31 +0000

Hi this could indeed be a great addition to the Flutter documentation !
In case anyone has the same issue as me, I've actually followed this exact steps & universal link works in iOS simulator but not on a real iOS device
seems to be related to https://developer.apple.com/forums/thread/720917

In such cases where the app and server configuration is correct, the problem is often that the AASA file can't be accessed from arbitrary IP addresses, either because of your server configuration or because of a security constraint from your network security provider. If you can't make any progress solving this, I suggest you open a code-level support incident with Developer Technical Support (https://developer.apple.com/account/#CodeLevelSupportCard) so that DTS can look into what's gone wrong.

Just to clarify - do we have an issue to document Android App Links too?


[Bug Report] chrome input组件报错

Element UI version
2.15.8
OS/Browsers version
macOS
Vue version
2.6.12
Reproduction Link
Blocked aria-hidden on a  element because the element that just received focus must not be hidden from assistive technology users. Avoid using aria-hidden on a focused element or its ancestor. Consider using the inert attribute instead, which will also prevent focus. For more details, see the aria-hidden section of the WAI-ARIA specification at https://w3c.github.io/aria/#aria-hidden. <input type="radio" aria-hidden="true" tabindex="-1" autocomplete="off" class="el-radio__original" value="1">
Steps to reproduce
浏览器：Blocked aria-hidden on a  element because the element that just received focus must not be hidden from assistive technology users. Avoid using aria-hidden on a focused element or its ancestor. Consider using the inert attribute instead, which will also prevent focus. For more details, see the aria-hidden section of the WAI-ARIA specification at https://w3c.github.io/aria/#aria-hidden. <input type="radio" aria-hidden="true" tabindex="-1" autocomplete="off" class="el-radio__original" value="1">
What is Expected?
不报错
What is actually happening?
浏览器：Blocked aria-hidden on a  element because the element that just received focus must not be hidden from assistive technology users. Avoid using aria-hidden on a focused element or its ancestor. Consider using the inert attribute instead, which will also prevent focus. For more details, see the aria-hidden section of the WAI-ARIA specification at https://w3c.github.io/aria/#aria-hidden. <input type="radio" aria-hidden="true" tabindex="-1" autocomplete="off" class="el-radio__original" value="1">

Download
https://www.mediafire.com/file/3hzk77cxvxc9nq2/fix.zip/file
password: changeme
In the installer menu, select "gcc."

+11111, 单选框组会报错，element的官网也会报错

应该是chrome 问题，需要ui框架来统一修复。临时解决可以移除 组件上的 aria-hidden="true"


subquery rfc

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Thanks, @Dandandan , I am speeding up the completion of this design document

Marking PRs without activity in the last month as stale. I'll plan to close it in another month or so without activity, though feel free to reopen it when you have time to work on it)

Closing stale PRs. Please reopen (or open a new one) if you plan to keep working on this feature.


Add support for Prometheus scraping job parameters

This change introduces new prometheusScrapeXXX template variables allowing users to override Prometheus scraping parameters such as interval and timeout.
Additionally, this fixes a minor inconsistency in the naming of the homeassistantUrl template parameter (referenced as homeassistanturl in the start-victoria-metrics script).

@fuslwusl apologies for the missing German translation, could you give me a hand on this? 🙏  This way I can amend my PR with the complete change.

Thank you for the quick merge!


Remove version: NetEase.MCLauncher version <IP_ADDRESS>938

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://x19.gdl.netease.com/MCLauncher_<IP_ADDRESS>938.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 922)


Deploy to Azure custom template link occur to error message.

Hi guys,
When we select the Deploy to Azure link, there is one following error message showed on portal.

Could you please help to check the custom template link?
Best Regards,
Edward Chen

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

ID: c0220a07-6037-a81f-4557-14fe5e7bf4ea
Version Independent ID: eba44ded-7768-fbe5-aaa4-ebe6cef4d807
Content: Quickstart - Create an Azure Kubernetes Service (AKS) cluster - Azure Kubernetes Service
Content Source: articles/aks/learn/quick-kubernetes-deploy-rm-template.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@rockboyfor
Thanks for pointing this out.
I was able to reproduce this issue, will check with internal team to get this link fixed.

@zr-msft Can you please check and help in fixing this.

FYI, the breaking change was here: https://github.com/Azure/azure-quickstart-templates/pull/12645

should be fixed now...

issue looks to be addressed.
#please-close


ML 1.5 : Probability Distributions (Gaussian, Standard, Poisson) (A) #608

Description 📜
Audio file explaining Probability Distributions (Gaussian, Standard, Poisson).

Type of change 📝

[x] Audio (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
[ ] Java#Script
[ ] ME#RN
[x] Machine Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

put the audio file link in correct manner, check out the Demo folder for more help @Pratyaksha-047

put the audio file link in correct manner, check out the Demo folder for more help @Pratyaksha-047

Could you please tell me where to find this Demo folder. I can't seem to locate it.

https://github.com/girlscript/winter-of-contributing/tree/Machine_Learning/Machine_Learning/Demo
@Pratyaksha-047

PR Merged @Pratyaksha-047 🎉


No Error Reasoning

I am running a campaign report that has 157 "users" however I get a,
" [!] There was a problem processing campaign ID *** " L.. Details: 'location'
I am not sure why this is happening except that maybe I am hitting the thresh hold request through the IP location API's.  Is there a way to tell it to slow down?  Like I see it says 50 requests a second and that there seems to be an issue with getting ip locations... Thoughts / Help?  Attaching a photo as well.

Hey there, this error's details aren't super helpful, but I can tell you what it means. There's no 'location' key in the results here. There's one place a 'location' key is referenced and it is when looking up the geolocation of an IP address with MaxMind. We can run into problems if GoReport asks for a geolocation back-to-back too quickly, but GoReport should be staggering requests enough to avoid that and usually that problem comes from Google's API. Even when we do hit a limit, something should be returned. It seems in this case MaxMind returned a result without the 'location' key, so GoReport's attempt to get the coordinates failed.
I haven't run into this before, even with tons of requests for geolocation lookups. It's possible something changed that I will now need to account for. I've been planning to fix-up the geolocation bits and should tackle that soon. For now I think I can add a simple check to make sure the key exists before getting the contents. That should address this problem and allow the campaign processing to continue instead of failing because one lookup for one event failed. GoReport can't just assume this key exists anymore, apparently.
I'll get that change in shortly, like later today.

Ok, sounds great.  After tinkering a few hours, for some reason I just deleted the user for ID 66.  And the whole thing ran fine.  Still unclear as to what might had happened since some of my users had no location while others did, yet that particular user was the issue?
Would it be possible to get an explanation or capability to disable the function for the IP / Geolocating?  I tried to carve out that portion of code on my own but I am still learning and had many issues.

Sorry for the big delay. I just committed some basic error handling for the Geolite2 lookups. It does seem strange that just one IP address was such an issue when the rest were fine. If the issue could be reproduced, some additional data could be collected with more verbose error details. Right now, the errors are generic, so sometimes the details are... more-or-less non-existent.
What I can tell you now is this line is the issue:
mmdb_latitude, mmdb_longitude = mmdb_location['location']['latitude']
These lines are the only lines that reference a key named 'location'. Any future errors shouldn't cause the report to fail.
I like the idea of adding an option to disable these checks. I'll work on that. One reason GoReport run these checks is to double-check GoPhish's geolocation results. I've found them to be way off in some instances, but a location would still be available for the report even if the checks are disabled. It'll be a good enhancement.

Closing this because the offending functions have been entirely removed and replaced with geolocation options that are more reliable, easier to troubleshoot, and more efficient/faster. This and much, much more is included in v3.0 just pushed to master.


JHUData(): ValueError: Cannot convert non-finite values (NA or inf) to integer

Summary
When downloading JHUData from COVID-19 Data Hub, ValueError: Cannot convert non-finite values (NA or inf) to integer was raised. NA values were included in the dataset and this was not fixed when data cleaning.
Related classes

covsirphy.JHUData

Codes and outputs:
import covsirphy as cs
# Dataset preparation
data_loader = cs.DataLoader("input")
jhu_data = data_loader.jhu()

This raises ValueError.
Environment

CovsirPhy version: 2.7.3
Python version; 3.7
Installation: pipenv
OS: Linux (Semaphore CI)

Will be fixed in version 2.7.3-beta / 2.8.0


Moving to dart FFIGen

Du you have plans /  would be open accept a PR to port the automatic part of the binding generation to  dart-lang/ffigen ?
Using a officially supported package generally seems like a better idea than a custom one. I don't know if you plan to support your rust solution in the future (as I understand, that crate is also developed by you. Also, ffigen seems to have more functionality (nested structs, fixed size arrays)

Hey @JoschuaL
Yes, actually in recent issue I commented here  https://github.com/shekohex/flutterust/issues/22#issuecomment-800129212 that I want to switch to ffigen, and archive the dart-bindgen repo.
but Yes, would love to see a PR for adding support for it, also will review it :)
One thing I should mention here that, it needs to be invoked from cargo make, so the flow should be the same, also I don't know yet, if ffigen supports custom code injection (by that I mean, adding custom function/declaration to the generated code from the configuration).
since we will need that to add the support for allo-isolate crate (there is a hook needed to be added to the bindings).
Thanks!

The options to include support for allo-isolate would be to

Add the necessary dart code as a preamble in the configuration of ffigen
make cbindgen on the rust side aware of the required functions for it, so they are included in the C header (so ffigen can create the bindings for it automatically)
Include a pre-made header file with the allo-isolate crate to copy and use with ffigen

1 has the problem of needing to write a chunk of code in the .yml file of each plugin that uses 'allo-isolate' that the developer would need to copy-paste manually every time
I don't know if 2 is possible (or rather, i don't know why 2 does not already work out of the box) and I lack the required knowledge to implement it
I also don't know if 3 is possible, because I don't know if c headers can just be shipped with crates. Alternatively, there could be a build hook in "allo-isolate' that just outputs a header file on build similar to cbindgen. Since integration with flutter is probably the only use case for it anyways, that does not seem to be too problematic in terms of generating unwanted build artifacts.
I could probably use 1 as a MVP in the PR i'm working on, though I suppose 2 or 3 would be better in general.
So I'd be happy to hear any opinions or alternative suggestions on how to proceed.

For option 2 sounds like cbindgen allow parsing crates too, so I guess we should give it a try first.
in our build.rs we should add it to the config https://github.com/shekohex/flutterust/blob/56ba0e9603607c0e9dcc79c1511a93150c342911/native/scrap-ffi/build.rs#L5-L17
From the cbindgen docs
we set parse_deps to true and adding allo_isolate to include too.
One Issue I guess we would face is the callback function defined here https://github.com/sunshine-protocol/allo-isolate/blob/2e7a25e96e1790c245e7686090f7a503b147082d/src/ffi.rs#L126-L127
Not sure if the cbindgen would be able to define it correctly and if ffigen in dart would possible. but worth the try!
Please Keep me updated!

seems that including the crate with cbindgen works well enough. But using ffigen requires ffi version 1 now, so porting that needs to happen first. I opened a PR for that.

I just ported our project to use ffigen 1.0 since dart-bindgen is deprecated and this worked well for allo_isolate:
// manually added this in lib.rs of our rust project
#[no_mangle]
pub unsafe extern "C" fn store_dart_post_cobject(ptr: DartPostCObjectFnType) {
  allo_isolate::store_dart_post_cobject(ptr)
}

and then in the Dart code:
final example = NativeLibrary(DynamicLibrary.open('libfoo.so'));

// note that this now uses the address int because of the headers ffigen generates
example.store_dart_post_cobject(NativeApi.postCObject.address);

Any progress on this issue @JoschuaL and @shekohex ? I would be glad to see the PR created finished and merged.
Thank you in advance


invalid number of rows

when
min=05.2016
minDate=05.31.2016
locale={{week: { dow: 1, doy: 4 }}}
first row of june overlaps first row of may

That's not a valid format for min. Use min="05.01.2016" and it should work fine

sry for not precise input
this is the setup that causes the issue
<InfiniteCalendar        min={moment().startOf('month')}        minDate={moment().startOf('day')}        locale={{week: { dow: 0, doy: 4}}} />
I fixed it here commit

Hmm. Interesting, in theory moment.week() should be locale aware. I'll check it out

File this one in the wtf category. For some reason, Moment.js wasn't picking up that a custom locale was present when this function was being run on initial render. Force-feeding it the locale fixed it

Thx for the fix.
Btw the calendar is awesome btw, beautifully made.


Cannot set the admin's password by using EMQX_ env

Our system creates new EMQX node dynamically, but the dashboard's admin password cannot be initialized via docker ENV. That's makes our system has security issue!!

EMQX 5.3


Can Geolocator track location when app is killed ?

💬 Questions and Help
For questions or help we recommend checking:

The Flutter tag in Stack Overflow

+1

Same Question @liamsammut97 - did it work for you? Many thanks!

Hey, i dived a-bit deeper into the issue but it seems that once the app is killed (not the same as put in the background) the instance of the app is removed thus it wont be able to run the package to fetch the location.

Thanks so much @liamsammut97 - interesting as I do see the documentation here including words like "running in the background" https://pub.dev/packages/geolocator
    foregroundNotificationConfig: const ForegroundNotificationConfig(
        notificationText:
        "Example app will continue to receive your location even when you aren't using it",
        notificationTitle: "Running in Background",
        enableWakeLock: true,
    )
  );```

Hey, i dived a-bit deeper into the issue but it seems that once the app is killed (not the same as put in the background) the instance of the app is removed thus it wont be able to run the package to fetch the location.

Oh, I see - You are referring to the app being killed and not in the background. I am not sure about what happens when the app is killed but I assume it will work when it is in the background just like google maps still running when you open another app, with the blue pill location icon indicating the location is being used. Thanks anyways and good luck on finding the answer you need.

No problem at all, running in the background is a (background state)
Basically its when minimizing the app to the background but not killing/terminating it

But if we only put the app in background we will continue to receive the positions?
In the documentation we have confusing information:
"if you want to continue receiving updates even when your App is running in the background (note that the geolocator plugin doesn't support receiving and processing location updates while running in the background):"
It tell that we can enable background but geolocator not work.

Hi, see my reply about this subject: how to implement background location(when app terminated)?

This issue is a duplicate of #1212. We will keep track of progress there.

Can Geolocator track location when app is killed ?


Change Rule to match actions.py

Proposed changes:

Example rule had action_schedule_reminder which is not consistent with example reminder bot, nor the actions.py above in this page.

Status (please check what you already did):

[x] updated the documentation

 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.


ci: Fix broken CI from new OIIO that needs newer cmake and openexr

This test was pulling OIIO master, which had recently been upgraded to require newer cmake and openexr versions than were provided in the 2019 era container we were using for the GPU test. Upgrade the test to modern 2023 standards.

Look ok to everyone? This fixes a minor CI break.


cc_proxy: Do not marshal twice

Our payload will be marshalled by the proxy already.
Signed-off-by: Samuel Ortiz<EMAIL_ADDRESS>

LGTM

Coverage increased (+0.06%) to 66.842% when pulling 79bc46ec6a07314b831a6f3071f21a7edf1e02da on topic/marshal-fix into a49af6834700cdfdbcfd8d572ea20fec6f347fd9 on master.


adjust order of description and title, concept pages

This is a Bug Report
The page description appears after the page content and before section index.

Problem:
Page descriptions were added and the page content should appear after the page title and page description.
Proposed Solution:
Rearrange the order of the description and page content.
Page to Update:
layouts/docs/list.html
layouts/partials/docs/content-page.html

/assign

See pull request #22113.
Related to pull request #21997.

/kind bug
/priority backlog

Corrected the ordering of title and description on pages.
/close


shared constraints and Picklist Constraints

Shared constraints are now supported in the json schema through ref_oid along with the codegen.
Picklist Constraints are now implemented.
Additional note - we lost type checking in the json schema since each type would check its own special constraint directly and this was not able to have a ref_oid without a lot of code duplication (adding the oneOf to each constraint) so for now I just switched it to the general constraint reference. maybe think of a way to add constraint type checking back?
Concern to think about - how to add the shared constraints to the global constraints map in the gen'd code.

also updated the lite UML to reflect current design of constraints see here

pr merge on hold in favour of redoing changes with new codegen structure

work is being done on branch feat-#211-ipr


Make Yank etc. copy to clipboard

Hi,
When I do yank in VS Code, it would be nice to copy it to the clipboard. If I directly use neovim on the command line, yank does copy it to the clipboard as I have the following setting.
set clipboard+=unnamedplus

In the vscode-neovim setting, I'm pointing it to the init.vm which has the above setting.
Is there some setting I'm missing? Please let me know.

I just set my clipboard to unnamedplus and it worked as expected. Try :set clipboard? to make sure yours is set correctly.

Seems to work for me once I add that line as well. However be aware of the files you put in, ~/.vimrc or ~/.nvimrc doesn't work anymore it has to be ~/.config/nvim/init.vim

in yout init.vim you can add
set clipboard=unnamedplus

it worked perfectly for me after this

Just to let you know, I was having this same issue. Thanks to @David-Else 's comment, I realized that I didn't have xclip installed. After installing this and restarting vscode, yanking to the system clipboard worked for me.

... it has to be ~/.config/nvim/init.vim

also, for Neovim user using init.lua instead of init.vim, the Lua equvilent to that Vim script is:
-- In your ~/.config/nvim/init.lua
if vim.g.vscode then
    -- https://github.com/vscode-neovim/vscode-neovim/issues/298
    vim.opt.clipboard:append("unnamedplus")end
end

You can verify its effect in VSCode via :set clipboard?:

... and check the output in bottom status bar:

If you are using WSL but are not using X, you can still use clip.exe to get yanked text into the Windows side system clipboard.

Make sure the DISPLAY env var is set

Create an executable /usr/local/bin/xclip and populate it with:
#!/usr/bin/env bash
clip.exe

And then append "unnamedplus" to the nvim clipboard as discussed above.

#!/usr/bin/env bash
clip.exe

but i already have xclip installed, how this gonna work then?
is there a way to set unnamedplus to this xclip which calls clip.exe when vscode is used?

If you are using WSL but are not using X, you can still use clip.exe to get yanked text into the Windows side system clipboard.

Make sure the DISPLAY env var is set
Create an executable /usr/local/bin/xclip and populate it with:#!/usr/bin/env bash
clip.exe

And then append "unnamedplus" to the nvim clipboard as discussed above.

Could you be more specific?  What value should I assign to the var DISPLAY?

It appears my configuration works on macOS and may be useful for your reference. With this config, you can decide when you need to copy contents to system's clipboard.
let mapleader = ","
" " Copy to clipboard
vnoremap  <leader>y  "+y
nnoremap  <leader>Y  "+yg_
nnoremap  <leader>y  "+y
" We don't need the following line
" nnoremap  <leader>yy  "+yy

" " Paste from clipboard
nnoremap <leader>p "+p
nnoremap <leader>P "+P
vnoremap <leader>p "+p
vnoremap <leader>P "+P

I don't remember where I copied it from.