Current draw of leg motors

Some of the motors tripped a reply in my power supply that indicated to me that they were drawing more current and/or voltage than other motors. I think that we should figure out how much current is going to be drawn by the combined leg motors.

I remember measuring the current drawn by the servos, and noting that the
current drawn depends on the position of the servo. I'll see if I can find
those notes, and/or a power supply that makes it easy to measure current.

Doug Elliott / VA3DAE / @.***

On Sun, Sep 12, 2021 at 9:33 PM va3wam @.***> wrote:

Some of the motors tripped a reply in my power supply that indicated to me
that they were drawing more current and/or voltage than other motors. I
think that we should figure out how much current is going to be drawn by
the combined leg motors.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/va3wam/hexBot/issues/15, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOZJYXJ3YB2KYZ2HPV2NGDUBVIFBANCNFSM5D4XFWBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

assigning to myself. Once I get the servos on my hexbot all working and calibrated, I'll use the SSD command to measure current drawn for various servo positions. (Looks like I need to replace a bunch of servos first.)

should increase the size of the capacitor on the PWM board before doing current measurements.

The capacitors have been upgraded from 470uF to 1000uF and the motors have become more reliable. Closing this issue.


poc: proxy webkit message handlers

@jonathanKingston the idea here would be:

allow a list of known handlers to us ~(we could pass it via config later)~ using remote config

allow certain handlers to return 'undefined'

allow other handlers to return a polyfilled .postMessage

This will fix a known breakage, but is less-than-ideal in the longer term.
Privacy config PR: https://github.com/duckduckgo/privacy-configuration/pull/1155
Known problems:

what if a site expects a particular result from postMessage?
the presence of window.webkit itself is the larger issue

Current dependencies on/for this PR:

main

PR #655   👈

This comment was auto-generated by Graphite.

@jonathanKingston I removed the validation stuff, should be closer now

Looks good, is it still poc or can we merge?

@jonathanKingston given it's 100% guarded behind config, I think it's good to merge ASAP, if you can R+


Rename Upgrade::new into Upgrade::executor

Upgrade::new is very vague. Either rename it into Upgrade::executor or UpgradeExecutor::new.
The assumption here is that there will be other things to upgrade, not only executor. Is this correct?

Hi is this issue still open to contributions?


iOS only: Setting height of WebView to zero doesn't resize the WebView

Steps to Reproduce
https://gist.github.com/cvolzke4/349cbe658003dbfd5673b47d75656500

Create a new widget with a WebView, with a button that changes the WebView height.
Set the WebView height.
Click the button to set the WebView height to zero.

Expected results:

WebView should be hidden from view, as it does on Android.

Actual results:

WebView retains the original height.
When in this 'height=0' state, dialogs are also rendered behind the WebView widget.

Flutter doctor output
[✓] Flutter (Channel beta, v0.11.9, on Mac OS X 10.14.4 18E227, locale en-US)
    • Flutter version 0.11.9 at /Users/cvolzke/development/flutter
    • Framework revision d48e6e433c (6 months ago), 2018-11-20 22:05:23 -0500
    • Engine revision 5c8147450d
    • Dart version 2.1.0 (build 2.1.0-dev.9.4 f9ebf21297)

[!] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/cvolzke/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-stable, build-tools 28.0.3
    • Java binary at: /Library/Java/JavaVirtualMachines/jdk-10-latest/Contents/Home/bin/java
    • Java version Java(TM) SE Runtime Environment 18.3 (build 10.0.1+10)
    ✗ Android license status unknown.

[!] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[!] IntelliJ IDEA Ultimate Edition (version 2018.1.4)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[!] Connected device
    ! No devices available

I tried your example with webview_flutter: ^0.3.6 and it works. Can you please report the webview_flutter version you use.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


[ID] Change Title Pages in Plugins

cc : @CikiMomogi
What is your opinion about this?
and
Plugin name and package name both are also links to their pages. What should we use?

Hey @CikiMomogi, would you agree this? that word "Tanpa" changed to "Nir". I mean like "Nirlaba" or something from "Endless" which meaning "Nirakhir" instead of "Tanpa Akhir".
But, I think it's also weird.

Hey @CikiMomogi, would you agree this? that word "Tanpa" changed to "Nir". I mean like "Nirlaba" or something from "Endless" which meaning "Nirakhir" instead of "Tanpa Akhir".
But, I think it's also weird.

You know, I was pretty stuck on finding the right translation of that "stateless question" phrase. But you nailed it. Adding "nir" is a brilliant idea. The meaning is more or less similar to "non-status question", but using the word "nir" sounds more suitable.

Can you remove the changes that are not related to this PQ? Like changing a plugin list that belongs to another PQ.

Sure.


Tool not refreshed upon tag creation

Bug Report
Expected behaviour
Upon tag creation, tool should be refreshed so that the new tag will be displayed and the file contents will be retrieved from source code repository and then visible in the files tab.
Actual behaviour
Upon tag creation, tool is not refreshed.  There is no indication that the tag was created.
Steps to reproduce the behaviour

Manually add a tool with a GitHub repo and GitLab registry (it will be in manual mode)
Add tag with valid descriptor, dockerfile, test parameter file
The new tag does not immediately appear and the tool is not refreshed.

Environment (Browser or OS and Dockstore version)
Firefox
UI and API 1.2.1

This looks similar to what I encountered in #584.  The difference is that I linked in a release branch for testing which it tagged as manual and subsequently all new tags aren't picked up unless I de/re-register the tool.  I end up doing them all manually due to this.

@garyluu
Let's take a look at this. I looks like between this and the previous bug, there are a few problems in play:

New tags aren't being picked up in some circumstance even with automated builds
Tools in manual mode are having issues when adding new tags (as described in the bug report above)
Tools in automated mode are being flipped to manual mode when they should not be

You may be able to experiment with https://dockstore.org/containers/quay.io/wtsicgp/dockstore-cgpwgs and https://dockstore.org/containers/quay.io/wtsicgp/dockstore-cgpwxs on staging. The former seems to be working properly, the lattter is not

@keiranmraine  For the time being, I flipped quay.io/wtsicgp/dockstore-cgpwxs into automated mode. After refreshing, it picked up two new versions 2.0.5 and 2.0.6 although I left the default version at 2.0.4.
@garyluu  When you look into this, the old database state will still be present on staging for your experiments

For the specific issue I posted, it somehow got fixed between API version 1.2.1 and develop.  May possibly have just been a GitLab issue.

@garyluu The tag does automatically get added now and the tool is refreshed. However, while it's loading the form puts the first two fields in red and says they cannot be empty. Not sure if new bug with this fix, or just a separate issue when you add a tag to any tool.

Tossed the fix in https://github.com/dockstore/dockstore-ui2/pull/591/commits/c64f36db550dd88e0c6738150e0f367bb366cd22


Wires not receiving power through solid blocks

I'd understand if this was unintended, but I didn't know if it was or not ;)

Not intended.

https://github.com/asiekierka/CharsetMC/commit/7c119924a23452c604349fe81bb01ee8bed7c17b


New repo for showing mutliplayer

Hi!
I am really interested in this engine because of the capability of multiplayer could you please make a new repo for showing how to make a simple multiplayer.
I doesnt need to be anything fancy just a cube that you can control with a random color and other players can join via steam (because i saw it uses steam sockets)
Thank you in advance!

nvm i found you already made almost what you requested in the examples my bad.


fun redefinition with different signature

This error happens when trying to redefine an existing fun
fun redefinition with different signature

Running this code shows the issue:
@[Link(ldflags: "`command -v pkg-config > /dev/null && pkg-config --libs --silence-errors libcrypto || printf %s '-lcrypto'`")]
lib LibCrypto2
  fun ec_key_new_by_curve_name = EC_KEY_new_by_curve_name(i : LibC::Int) : LibC::Int*
end

require "openssl"

The fun is already defined in the std lib but with a different signature.
This means that libraries can be easily broken if adding more funs to the std lib or any other lib that is imported into a project.

In this case changing the return type to match the std lib fixes the issue:
fun ec_key_new_by_curve_name = EC_KEY_new_by_curve_name(i : LibC::Int) : Void*

There's only one definition by name of a given function in C. If you want to redefine it the arguments must match. This actually allows two libraries that import the same C function to coexist, and their signatures must trivially match because there's only one C signature (unless you made a mistake).


feat: Enhance the 'hasMany' and 'belongsTo' relationships

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[ENH] Added 'order' argument

Added the 'order' argument (can be either 'LR' or 'RL'), and function will split the data accordingly

Codecov Report

Merging #54 into master will decrease coverage by 0.41%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##           master      #54      +/-   ##
==========================================
- Coverage   63.03%   62.62%   -0.42%
==========================================
  Files          21       21
  Lines        1204     1212       +8
==========================================
  Hits          759      759
- Misses        445      453       +8

Impacted Files
Coverage Δ

netneurotools/plotting.py
7.53% <8.33%> (-0.32%)
: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 6f10f85...8890d63. Read the comment docs.


Feature request for a resource configuration file

Something like a .driverc file? I use --hidden=true often and would like something other than aliases.

Hello @nkmathew thank you for reporting this and welcome to drive!
+1, great suggestion. So would  local .driverc in your .gd dir as well as global ~/.driverc suffice?

In the home folder preferably. Placing one in the .gd folder seems unnatural

I see, although drive supports an infinite number of accounts mounted, so maybe multiple different configurations. This is analogous to have a special .driveignore file for each account.

Wasn't aware of that. Sounds fair

I'll work on this later on when I get home, this is going to be a big win for lots of users because then defaults can be set once and for all. Thanks again for reporting this.

:+1: for this

+1
I'd like to be able to set a default of --exclude-ops "delete" so all push/pull commands don't delete anything, ever.   My use case is backing up the irreplaceable data (personal photos) from my NAS (60GB out of 7TB data). I don't need to backup movies or tv, so I want to push Photos folder only. If when I pull, I don't want to accidentally delete everything local that isn't in Drive.

Sorry for the really late reply folks. I had actually implemented most of it in mid May as you'll see from commits in the PR https://github.com/odeke-em/drive/pull/259. However, I am not yet decided on how to get the unused flags per command, but I'll be working on that sometime this week. Please watch this space and thank you for your patience.

Did you manage to get it working?

Hello @nkmathew, unfortunately I haven't glued everything together yet. The trouble is piecing together the defaults and then the defined args: separately they are being parsed alright. What is left is figuring out if a flag was defined -- for that I worked on https://github.com/odeke-em/command/commit/3e1c027705d997ee25b982cc1d96fc5cfa64066e. I'll try one more thing that I just got an idea for, to correlate defined flags, rc-read/default flags.

Addressed by PR https://github.com/odeke-em/drive/pull/259. Thank you for requesting this feature  and for the patience. Please get the latest from master by running
$ go get -u -v github.com/odeke-em/drive/drive-gen && drive-gen


ADD caching support using WP transient API as exposed from WP-CLI

I guess this is the simplest solution even though adding a caching layer to guzzle client maybe would be a more sound solution as the WP-CLI caching is "site" based.

I like the idea of caching. A developer of wpvulndb.com told me that a limit of 30 calls every 30 seconds is acceptable.
However, to keep things simple I would suggest to disable caching by default, this way we have more control (e.g. during development this is cumbersome when one needs to clear the cache to check for new values when updates are finished). So adding an option like:
--caching  |  protects from lockouts on wpvulndb.com (max 30 calls/30 seconds) by caching results
would be better I think, unless I'm missing something here. Are there any ideas to suggest otherwise?

Well, no problem in adding a switch. What is indeed cached is the call to wpvulndb.com (for eight hours and I don't know how much often is updated) not the version checking.

BTW in the pipeline (zipangotes/wp-sec@2b9d0903c019438aea43703278594bbe767ac278) there is the fix for the behat JSON testing.

Would this caching be able to be shared between users.  We already do this with the download cache in wp-cli by setting the WP_CLI_CACHE_DIR to a shared folder.  It would be great if we could do the same with this and result in a lot less calls to wpvulndb.  Thank you for this and all your hard work with it.

Uhm, it would be possible by implementing the caching in the guzzle stack.
I don't know exactly how the wp-cli download cache is working. I asked in slack (https://wordpress.slack.com/archives/cli/p1480249573000275) but so far no answers.

In the wp-cli API there is WP_CLI\Utils\http_request but I don't know if it implements a caching.
But indeed I found this on CoreUpgrader.php an implementation of the cache system related to the WP_CLI_CACHE_DIR setting. So it could be something like:
$cache = WP_CLI::get_cache();
$cache_key = "wp-sec/core";
$cache_file = $cache->has( $cache_key, $ttl );
if ($cache_file) {
  $json = $cache->read($cache_key);
}
else {
  ...
  $json = json_decode($res->getBody(), true);
  $cache->write($cache_key, $json);
}

BTW using WP_CLI\Utils\http_request would remove the dependency on Guzzle. What you think @markri?

Agreed to loose the Guzzle dependency and to use the http_request() by WP_CLI. Advantages I see:

Less overhead by Guzzle; which is a bit of too much weight for this plugin. I just chose it because of habitual patterns. As it seems; WP_CLI uses rmccue/Requests for the actual request, which is a better match for the purpose it is used for.
More simplified install profile, no dependencies :-)
Better BC compatibility with PHP 5.3

This should work: zipangotes/wp-sec@c2c4989037364e46260b8658c10ff3e135213b95


VSCode Omnisharp Server Doesn't Recognize RTLTMPro

Hi,
I'm using Unity 2019.4.13f1 and vs code as my code editor.
I've downloaded RTLTMPro and I can use it fine in action. However, vs code can't recognize the whole "RTLTMPro" namespace! It says:
The type or namespace name 'RTLTextMeshPro' could not be found (are you missing a using directive or an assembly reference?) [Assembly-CSharp]csharp(CS0246)

For example, when I use using RTLTMPro;, unity can run fine with no problem, even I can access RTLTextMeshPro component in my code and change the text, but vs code's omnisharp server shows the error above in its editor, which renders the whole auto-completion tool useless.
I hope someone can provide a workaround for this.
Thanks

it looks like that changing C# Extension to v1.23.2 and removing omnisharp.path from settings.json fixes the issue. However it didn't work instantly for me. It took a couple of days for changes to take effect (with no reason!)
closing issue.

it looks like that changing C# Extension to v1.23.2 and removing omnisharp.path from settings.json fixes the issue. However it didn't work instantly for me. It took a couple of days for changes to take effect (with no reason!)
closing issue.


Go2rtc stream stuck issue randomly

I'm using 1.9.4 latest version
Issue 1 : in which I'm getting stream stream issue and restarting the stream it is rediscovering.so can you please provide the solution for this
error log : WRN go2rtc-1.9.4/internal/streams/producer.go:170 > error="read tcp <IP_ADDRESS>:49370-><IP_ADDRESS>:554: i/o timeout" url=rtsps://admin:admin@123@<IP_ADDRESS>:554/main_stream
Issue2 . also I'm getting randomly stream stuck which is not recovering by doing anything it will recover after restarting the full process
error log for the same : WRN go2rtc/go2rtc-1.9.4/internal/streams/producer.go:170 > error="read tcp <IP_ADDRESS>:49370-><IP_ADDRESS>:554: i/o timeout" url=rtsps://admin:admin@123@<IP_ADDRESS>:554/main_stream
what can be the reason for this

Those are the same error logs. Issue 1 doesn't make sense as to what you're trying to say.
If you use an audio stream (ffmpeg:CAMERA#audio=opus) then look at #1254

"i/o timeout" - problems with network connection or camera software.


Don't trim path from builder URI

Summary
This PR modifies the identity extraction function to preserve the full path of the attestation identity recorded in the attestation identity.
Trimming the path opens up a vulnerability as an attestation may be validated by a signature triggered from a workflow with the same filename as the expected one.
The identity extraction function is still timing the branch and tag from the identity which we should probably also not do but we handle it in the rule definition so probably we are ok. Further discussion is needed.
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Added a unit test to test the treatment of the signer identity
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 50.614% (+0.1%) from 50.514%
when pulling 84976e7b3eb87652731bd993e1a3f0cf3c3c74da on puerco:check-identities
into f8944766bfbfe972a4572896c4e50678e09cbe83 on stacklok:main.


Reverting some pins

Reverting two PRS:

https://github.com/chef/chef-dk/pull/768 because https://github.com/mizzy/specinfra/pull/538 was merged and now the entire ChefDK omnibus should be using net-ssh 3.1.1
https://github.com/chef/chef-dk/pull/772 because https://github.com/jmespath/jmespath.rb/issues/20 was fixed and should prevent load order issues with json vs json_pure

NOTE: the first commit shouldn't actually be merged, we should merge https://github.com/chef/omnibus-software/pull/643 and switch the code here back to using master

For anyone coming across this later:
The reason we now include gem 'json' in the Gemfile is to ensure that someone running bundle exec does not use the json_pure gem.  When using the json_pure gem there is an error we encounter trying to serialize a hash into JSON using .to_json.  Ultimately we should fix that bug but I couldn't repro it locally and this was the fastest fix to get our build matrix green again.
json_pure is included by jmespath (which is included by aws-sdk-ruby).  json_pure is included so that jmespath is compatible with older versions of Ruby which do not include the pre-built JSON extensions.  Jmespath first tries to load the included json gem and then falls back to json/pure if the included gem is not available.
Because we build our own omnibus and ensure that we are using recent version of Ruby, we should always have the included json gem available.
But an issue arrises when trying to run bundle exec when the JSON gem is not included in the Gemfile.  The gem 'json' call fails with a LoadError because the JSON gem is not included in the Gemfile, so json_pure is loaded.  That is why this PR adds the JSON gem to the bundle - so that gem 'json' will succeed and json_pure will not be loaded.

:+1:

:+1:


Nimble-Button has a small white line between the 2px outer border

🐛 Bug Report
When the outer 2px border is visible for the Nimble-Button (ex. on hover / focus-visible), there is a small white line between the two pixels.

💻 Repro or Code Sample

See patterns/button/styles.ts.
The main cause of this issue is that the CSS attributes border and box-shadow are being used next to each other to make up the 2px outer border of the button. When the box-shadow and border attributes are put next to each other and are the same color, the white line appears. A 2px border or box-shadow cannot be used to fix this because the border has to always be 1px (It cannot shrink or grow, or else it will change the size of the entire button), so a different CSS attribute has to be used for the other 1px part of the outer border that is seen on actions like hover and focus-visible.
🤔 Expected Behavior
The 2px border around the Nimble-Button should not have a white line between the pixels, as shown in the Figma Design.

😯 Current Behavior
Currently, the white line varies in visibility depending on how zoomed in the button is in storybook and what browser is being used.
On Chrome:

On Firefox (Different zoom level than Chrome):

💁 Possible Solution
Attempts were made to fix this problem in (#1300), but each required making a change that affected another part of the button. Using a 2px inset box-shadow for the outer border could be a possible solution, but this would shrink the entire width of the button by 2px because the border attribute would no longer be used.

🔦 Context
This issue does not affect the usability of the button and is only visible when the button is extremely zoomed in- when the button is a normal size (ex. the size of a small icon button or submit button), it would be hard to notice the line. However, this issue may cause problems when users zoom in or out on a web browser due to the outer borders not being connected.

🌍 Your Environment

OS & Device: Windows on PC
Browser: Edge, Chrome, Firefox, all browsers
Version: 114.0.1823.58, 114.0.5735.199, 115.0.2, respectively.

I also looked at this in SLE and didn't see this issue, even when zooming in to extreme levels. I agree that this issue should be closed- It seems like the white line is something that only occurs in Storybook.


Add role based authorization

I know this is not the purpose of this gem, but if you had to implement it to work with ng-token-auth, how would you do that ?
I'm working on a project based on this Gem, and I will need authorization for the iOs / Android app and later, the website.

My current approach has been as follows:

Return a list of roles / permissions with the user object on validation. This will look something like this in the $auth.user object:
{
  id: 1,
  // ...
  permissions: {
    // ...
    applications: {
      view: true,
      edit: false,
      destroy: false
    },
    // ...
}

Define a permissions service on the client can check the user's permission object to determine their access to views / actions. This will look something like this when used in a template:

    Applications

This is important - access to resources must be limited on the API as well as the client. We're using pundit for this.

This has been working for me in my apps, but I'm not sure if it can be generalized for every use case.

Hi @lynndylanhurley, I just stumbled across the same requirement / issue.  I understand your suggestion, however from an implementation standpoint I am struggling with (1).  How does one modify the user object which is being returned by devise for both sign_in as well as validate_token?  I am sure it has been asked before, but I could not find it.  Basically in my case I have the "User" class and a "UserRole" class which roughly represents your "permissions" in the example above.  So I would expect a return JSON object along these lines:
{
id: 1,
email: ...
//....
roles: [ phone_operator, printer_administrator ]
}
Any suggestions on how I tie in the relation?

@pommerien - just override the token_validation_response in your User model. (You may need to update to the latest beta version for this to work. I'll push a new release ASAP).
The method to override is here.

@lynndylanhurley overriding token_validation_response doesn't work, I had to override whole SessionController and TokenValidationsController for desired response.


Accessing the SharedVariable in the ParentContext

This sample includes the following code:
        if (context.ParentContext != null
            && context.ParentContext.ParentContext != null
            && context.ParentContext.ParentContext != null
            && context.ParentContext.ParentContext.ParentContext.SharedVariables.ContainsKey("CustomPrice")
            && (bool)context.ParentContext.ParentContext.ParentContext.SharedVariables["CustomPrice"])
            return;

Which I think is supposed to actually be this:
        if (context.ParentContext != null
            && context.ParentContext.ParentContext != null
            && context.ParentContext.ParentContext.ParentContext != null
            && context.ParentContext.ParentContext.ParentContext.SharedVariables.ContainsKey("CustomPrice")
            && (bool)context.ParentContext.ParentContext.ParentContext.SharedVariables["CustomPrice"])
            return;

This doesn't represent a good practice for accessing SharedVariables in the ParentContext because it is looking exactly 3 levels up. The better practice to make this code resilient to changes would be to loop up through the ParentContext until the SharedVariable is found, regardless of the level. This way, the code won't break if the underlying implementation changes.

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

ID: 92a3bbda-0bc0-b89a-ad5d-8ff3067dd2d5
Version Independent ID: fe2cca04-3084-978d-b71c-38b1ac3ad108
Content: Sample: Calculate Price plug-in (Developer Guide for Dynamics 365 for Customer Engagement)
Content Source: ce/developer/sample-calculate-price-plugin.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

The code to perform similar logic using a loop would be
       // Obtain the execution context from the service provider.
        IPluginExecutionContext ctx, context = (IPluginExecutionContext)
            serviceProvider.GetService(typeof(IPluginExecutionContext));

        // Search up the parent context hierarchy for a shared variable
        // containing a custom price key and return if found.
        ctx = context.ParentContext;
        while (ctx != null) {
            if (ctx.SharedVariables.ContainsKey("CustomPrice") &&
                (bool)ctx.SharedVariables["CustomPrice"]) return;
            ctx = ctx.ParentContext;
        }

@JimDaly Thanks for your feedback. We are closing this issue because we are either unable to find relevant information or reproduce this. If you're still experiencing the issue described here, please reopen or create a new issue.


Parse "NIST Research Library (2022)"

We need to be able parsing a specific ID "NIST Research Library (2022)".
This issue blocks https://github.com/relaton/relaton-nist/issues/68#issuecomment-1685721201

@ronaldtse @andrew2net do we have other identifiers like this? I'll probably need to create separate parser and renderer for this type of identifiers.

@ronaldtse this identifier doesn't comply with PubID 1.0, what is this identifier?

@ronaldtse @andrew2net do we have other identifiers like this? I'll probably need to create separate parser and renderer for this type of identifiers.

@mico At this moment we have NIST Research Library (2022) and NIST Research Library (2023). It seems we will have documents like this in this and next years. @ronaldtse Am I correct?

Also, the "NIST Research Library" is now in 2023, we should support that.

@ronaldtse @andrew2net do we have other identifiers like this? I'll probably need to create separate parser and renderer for this type of identifiers.

@mico At this moment we have NIST Research Library (2022) and NIST Research Library (2023). It seems we will have documents like this in this and next years. @ronaldtse Am I correct?

Yes. We need to support it.

We need to support it as this:

https://github.com/usnistgov/NIST-Tech-Pubs/blob/8882a6e24d809d934c4fdacc42b1a3602313c222/xml/readme.txt#L48-L50

Recommended citation for the data:
NIST Research Library (2023) NIST Technical Series Publication Metadata. (National Institute of Standards and Technology, Gaithersburg, MD) https://doi.org/10.18434/M32192


std.experimental.numeric.sum

Previous PRs:

https://github.com/dlang/phobos/pull/2513
https://github.com/dlang/phobos/pull/2991

Phobos Issues:

https://github.com/dlang/phobos/blob/master/std/internal/math/biguintcore.d assers at line 100.

Remaining DMD Issues (not blocking):

ICE for SIMD vectors
Slow compilation time with flags (-O -inline -release -main -unittest): 15 secs with LDC, but 90 secs with DMD

@don-clugston-sociomantic @donc Could you please looks at biguintcore.d's invariant at line 100. It asserts for the unittest
unittest
{
    import std.bigint;
    import std.range;

    immutable BigInt[] a = BigInt("1_000_000_000_000_000_000").repeat(10).array();
    immutable ulong[]  b = (ulong.max/2).repeat(10).array();
    auto sa = a.sum();
    auto sb = b.sum(BigInt(0)); //reduce ulongs into bigint
    assert(sa == BigInt("10_000_000_000_000_000_000"));
    assert(sb == (BigInt(ulong.max/2) * 10));
}


missing relation GO:0061821 telomeric D-loop binding child of GO:0061820 telomeric D-loop disassembly?

or better suggestions

Is this always true? Do things bind that are not involved in disassemby? What about just things that localize there or are involved in assembly?

@NancyCampbell
A quick search of the literature suggests that there might be binding proteins that stabilize the loop, do you agree?

yes you are right.

OK. I will close this one then.


GQL causes TypeError: "Cannot read properties of undefined (reading 'name')"

This query was in some logs:
mutation {
  __typename
}

which caused:
TypeError: Cannot read properties of undefined (reading 'name')\n    at QueryComplexity.nodeComplexity (/app/node_modules/graphql-query-complexity/dist/cjs/QueryComplexity.js:105:46

Are mutations configured in the schema of the GraphQL server? Might be the same issue as this one: https://github.com/slicknode/graphql-query-complexity/pull/84

We are having the same (similar) issue. Even if the operation is defined in GraphQL schema but you don't give your actual query a name then it will issue the type error.
For example:
mutation {
	doSomething(input: { amount: 1 }) {
		result
	}
}

will error but
mutation myOperation {
	doSomething(input: { amount: 1 }) {
		result
	}
}

will not.
We are on Apollo 3 and GraphQL (graphql package) 15


fixed time serialization to include nano seconds

How do we deal with reading old data from disk ?

I just thought it might not be likely someone did it already so we get away with it 😉


[WFLY-8053] add warning on conf change to worker in remoting/http-listener(undertow)

WFLY: https://issues.jboss.org/browse/WFLY-8053
EAP 7.1: https://issues.jboss.org/browse/JBEAP-7284
EAP 7.0.z: na
WFLY:  https://github.com/wildfly/wildfly/pull/9613
EAP 7.1: https://github.com/jbossas/jboss-eap7/pull/1289
EAP 7.0.z: na
Requires core upgrade with: Requires core upgrade with: https://issues.jboss.org/browse/WFCORE-1987

retest this please

Updated PR, something is wrong, it sucked some commit from the middle of commit list( no idea why). Closing in favor of https://github.com/wildfly/wildfly/pull/9976


Newby Question: Could I use your project to build a C# interface for https://github.com/infiniflow

Sorry guys dumb question but it looks like I could use your framework to build a C# interface for https://github.com/infiniflow.
Your project is a work in progress? And infiniflow is as well. But this looks like a good match?
Best Regards,
Alistair

Your project is a work in progress?

The project itself is pretty stable and used by several production quality dependencies. It gets updates when Clang itself gets updates (about every 6mo) and when there is an ask for a new ffeature.

but it looks like I could use your framework to build a C# interface for https://github.com/infiniflow.

I'm not familiar with that project in particular, but ClangSharp supports generating 1-to-1 blittable P/Invoke bindings to many types of C (and a smaller subset of C++) headers. It's known to work well with libraries like Windows SDK ( D3D9, D3D10, D3D11, D3D12, D2D1, DWrite, Win32, Gdiplus, WIC, XAudio2, etc), Vulkan, Xlib, Xcb, Wayland, PulseAudio, and much more.
So provided your project has a generally stable C ABI, it should be possible to generate bindings for it.


Fix package publish due to missing repository declaration

Publishing the package is triggered when a release is created.
The workflow was configured but it needs the repository declaration to be able to push there.
Given error:
Deployment failed: repository element was not specified in the POM inside distributionManagement element or in -DaltDeploymentRepository=id::layout::url parameter

Fixed at ba279fd


feat: Add jobs.delete and update to latest discovery types

[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #1010 🦕

@JustinBeckwith I realized I needed to refactor, so the type issue are no longer a thing.


Add Array Partition leetcode

I have solved Array Partition problem of leetcode in Python. Please review it.

Thanks.


Change

Pull Request Template
Description:

Issue no. - #992
Added documentation to clickhouse module

Checklist:

[X] I have formatted my code using  goimport  and  golangci-lint.
[X] All new code is covered by unit tests.
[X] This PR does not decrease the overall code coverage.
[X] I have reviewed the code comments and documentation for clarity.

Thank you for your contribution!

Can you please update the name of the PR

@nidhi752 Please update the PR name and resolve the review comments.
Thankyou


Unrendered emoji codes - update

Same as #2 - Observed unrendered emoji codes:

:technologist:  :technologist:
:necktie:  :necktie:
:bricks:  :bricks:

Perhaps more : see Gitmoji for more info.

We've responded! Please use 1.0.7.


Update Package.swift

an issue: 'TelegramVaporBot' was identified as an executable target given the presence of a 'main.swift' file. Starting with tools version 5.4.0 executable targets should be declared as 'executableTarget()'

tnx 🤝


Made many updates to ASE creation document

Simplified,
reflects new UI,
removed redundant information.

Hi @ChrisCompy, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ccompy). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

#sign-off

@ChrisCompy : Thanks for your contribution to the Azure documentation! The author, @ccompy, has been notified to review your proposed change.

Only the author of the article can use the sign off comment to add the ready-to-merge label.

#sign-off

Hi Christina,
Please pull these changes over to the private repository and submit them there so the updates are validated, staged, and run through Acrolinx.
Also, please make sure you are working in the repo using the GitHub account you list in your articles. The automation is relying on the accounts to match exactly.
Thanks,
Tyson

Hi Tyson. Christina said that in the past both @ChrisCompy and @ccompy had access to azure-content-pr but now @ChrisCompy doesn't anymore (@ccompy still does). Is it possible to still get that access for both accounts?


add PostreSQL and Redis

I'm working in a PR to add PostgreSQL and Redis. Any mistake feel free to correct me.

Hey @gabriel-q7
Please kindly specify the need for the PostgreSQL and Redis, simply edit the issue detailing what you want to do, and what it solves in the website.
Thanks


Make it possible to pan the map in public item show views

In the public item show view, the map control defaults to a box selection tool that lets you drag out an area on the map. When I hit the page, thought, I wanted to be able to pan the map, not select features.
It could be useful to add a simple control that would let you toggle back and forth. I suspect, though, that in most cases people will want / expect to be able to pan, since that's the default pattern in Google Maps, etc.

This is a duplicate of #35.


Implement pawn promotion

The promotion event happens when a pawn reaches the last row (white player) or the first row (black player).

The player should see a modal asking for a piece to replace his pawn
A pawn can only be replaced by a queen, a rook, a bishop, or a knight (with the player's color)
The player has to choose one piece to replace

Can I work on this one ? :smile:

Before adding a modal I think the code might need some rearranging, because when using a modal, you need to make the game "wait" for the player to make his choice, and currently the code is not designed to wait on something. I think it would be useful to create an other issue to adapt the code first :smile:

sorry to answer just now, i've been a bit busy these days

No problem, I've been busy these days too. Could you describe better which adaption it would be? I created this repo when I get started with TypeScript, that is why my code doesn't look so good though.
But I think the modal isn't dependent on the game progress, might there is a way to create a function which observes if a pawn can be promoted, then open a modal and handle the promotion event. In that approach, the logic wouldn't be on the piece's logic.
But I may be wrong, thanks for your suggestion! :smile:


"Browse stations" does not work with dark theme

OS:  Windows
Issue Descriptions:  The "Browse stations option is completely unusable when using the dark theme.  Perfectly fine with light theme.
Steps to Reproduce:  Desktop settings > Custom theme > Dark, then click on "Browse stations."

Fixed in 4.4.0 release should be going out ASAP

Any update on 4.4.0 for the MAC release?


Add plugin for Treasure Data Toolbelt

Summary
Add support for a new plugin for the TreasureData Toolbelt
Testing
Setup

Create a TreasureData account through the site here.
Get the API key for CLI tools according to the guide.
Following the guide, setup the Treasure Data Toolbelt in your environment
Build and test the TreasureData plugin locally according to these instructions.

$ make treasuredata/validate
$ make treasuredata/build

Importer
After installing and setting up Treasure Data Toolbelt, a config file should have been created at ~/.td/td.conf.
Initialize the Treasure Data plugin and test the importer using this config file. The credentials should be saved in 1Password successfully by selecting this config file. The entry should have user and api key fields.
Provisioner
After initializing the Treasure Data plugin and having your Treasure Data credentials saved in 1Password, run a td command. The provisioner should inject TD_API_KEY environment so that the tool can refer to the necessary credential at runtime.
Try running the following command.
$ td db:list
+---------------------+----------+
| Name                | Count    |
+---------------------+----------+
| sample_datasets     | 0        |
+---------------------+----------+

Note: While testing the provisioner, ensure that your credentials are not currently stored in ~/.td/td.conf nor environment variables.

Thanks for the contribution @Lewuathe !
Code lgtm but static check is failing due to unformatted files.

@AndyTitu Thank you for letting me know. I fixed the format. Could you please take a look at this one again?

Terribly sorry for the long turnaround! Only one other comment on an older thread, otherwise, this looks good to go! Thank you for your contribution!!

Hi @Lewuathe, since Friday lint jobs started to fail across this repo, and even in other repos on GitHub Actions as well. So it appears to be independent of the changes introduced in this pull request. We're looking into the issue https://github.com/1Password/shell-plugins/issues/195

@SimonBarendse Thanks!


Multiple xml editors in one page

I see the author of xonomy emphasize do not use multiple xml editors in one page, but I do have a requirment that need show multiple, any advice for quick fix?

One could probably use iframes.


Connecting to Docker via TLS fails when configuring docker hosts but not when manually connectiong with openssl

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
I have configured a remote docker server on my network to accept TLS connections so I can monitor the containers directly.  I have pored over the other issues on this project and have implemented all of the suggested fixes.  I have raised this as a new issue as the behavior does not correspond with the previous similar issues and is not resolved by their suggested fixes.  What appears to be happening is that uptime-kuma is not presenting the certificate properly when connecting.  If I drop into the container and connect via openssl manually, specifiyng the cert, CA, and key files in their in-container locations, it accepts the connection.  When I try to use the configured host in the GUI, it fails and throws a bad cert error (ssl alert 42).
docker compose:
version: '3.8'

services:
  uptime-kuma:
    image: louislam/uptime-kuma:1
    container_name: uptime-kuma
    environment:
     NODE_EXTRA_CA_CERTS: '/etc/ca.pem'
     UPTIME_KUMA_SSL_CERT: '/etc/cert.pem'
     UPTIME_KUMA_SSL_KEY: '/etc/key.pem'
    volumes:
      - uptime-kuma:/app/data
      - /ssl:/etc
    ports:
      - "3001:3001"  # <Host Port>:<Container Port>
    restart: always

volumes:
  uptime-kuma:

Running openssl s_client -connect https://<serverip>:2376 -cert /etc/cert.pem -key /etc/key.pem -CAfile /etc/ca.pem from within the container successfully opens a connection and the certificates all jive.
👟 Reproduction steps

configure a directory to mount a volume against, in this case i am using /ssl to mount the container's /etc against
put the certificates in this directory
configure the appropriate environment variables:

 NODE_EXTRA_CA_CERTS: '/etc/ca.pem'
     UPTIME_KUMA_SSL_CERT: '/etc/cert.pem'
     UPTIME_KUMA_SSL_KEY: '/etc/key.pem'
    volumes:
      - uptime-kuma:/app/data
      - /ssl:/etc

restart/recreate the container for shiggles and/or good measure
configure a docker host using the TCP/HTTPS method, pointing at http://<serverIP>:2376 and click test
observe the failure

👀 Expected behavior
I expect the behavior of node.js utilizing the specified certificates to mirror the behavior of a manual openssl cli connection.
😓 Actual Behavior
Actual behavior is uptime-kuma seems unable to properly grab or utilize the certificates, or is using the wrong one.  log output:
 | Interval: 20 seconds | Type: docker | Down Count: 0 | Resend Interval: 0
 | Max retries: 1 | Retry: 1 | Retry Interval: 20 seconds | Type: docker
2023-11-26T01:11:32-05:00 [MONITOR] WARN: Monitor #12 '<editedContainerName>': Pending: C027E047787F0000:error:0A000412:SSL routines:ssl3_read_bytes:sslv3 alert bad certificate:../deps/openssl/openssl/ssl/record/rec_layer_s3.c:1586:SSL alert number 42
2023-11-26T01:11:52-05:00 [MONITOR] WARN: Monitor #12 '<editedContainerName>': Failing: C027E047787F0000:error:0A000412:SSL routines:ssl3_read_bytes:sslv3 alert bad certificate:../deps/openssl/openssl/ssl/record/rec_layer_s3.c:1586:SSL alert number 42

🐻 Uptime-Kuma Version
1.23.7
💻 Operating System and Arch
ubuntu <IP_ADDRESS> LTS
🌐 Browser
Firefox 120
🐋 Docker Version
Remote server - 20.10.23, Local Server - 24.0.5
🟩 NodeJS Version
No response
📝 Relevant log output
No response

Please see PR #2852:
you need to put the certificate files in <your data directory>/docker-tls/<FQDN to your docker host>.

From the openssl command looks like you are using TLS client authentication. Is that correct?
If so, you need to follow the steps in PR #2852 and place the certificate files in the correct location in the data folder.

I must've glossed over this particular PR when searching for answers last night.  Many thanks, it's connecting now!  This definitely needs to be in the wiki/instructions.  I will endeavor to add this information to help the next person that comes along.


Update Publishing Workflows

See https://github.com/jupyter-server/jupyter_releaser/pull/557 for background.
If y'all set up Trusted Publishing (https://pypi.org/manage/project/jupyter-scheduler/settings/publishing/#errors) for this repo, I can make a PR to update the workflows to use the Jupyter Server GitHub App for publishing.

I have replaced the branch rules with  a branch "Ruleset"
I added a tag Ruleset
I configured the Release environment with appropriate credentials.

@blink1073 Thank you for upgrading workflows and creating this issue.
JupyterLab PR for more context: https://github.com/jupyterlab/jupyterlab/pull/15973

@Zsailer can you please add me (blink1073) to the project on pypi?  https://pypi.org/manage/project/jupyter-scheduler/collaboration/

cc @3coins @dlqqq this is the last repo using a bot account.

I added you @blink1073! 😎 🚀


Added samba share write support

bytestream2smb.py - new module for writing to samba share
added new ENV vars for samba share access
updated requirements
changed port to avoid potential conflicts
updated docs
moved TODO-stuff to TODO

@ashkanvahidishams / @BaardBouvet , kan en av dere merge inn denne ved anledning?


Design Panel: Bugfix - reset stored height of saved colors panel

Context

The issue is, that before the 1.5 release (so in 1.4), the saved color panel height was only a single line of color circles, so it was much smaller. And yes, that height was saved in local settings. When a user upgrades, the saved height persists, ignoring that the panel now should never be this tiny.

https://xwp.slack.com/archives/CCCAUAH6F/p1616176802029600?thread_ts=1616176594.029300&cid=CCCAUAH6F
Summary
Changes the presetType for the colors panel. This changes the local storage name and resets the saved sizing values.
Relevant Technical Choices
n/a
To-do
none
User-facing changes
Opening the colors panel will ignore the current saved sizing data. This can then be overwritten.

In this gif I set the height in this panel. This sets the value in local storage
I reload the page, but the presetType has been changed in my local code
You see that the height does not persist
Changing the height and reloading again persists the new height (since the data has a different key in local storage)

Testing Instructions

Open the editor
Click on the background or an element with color
Open the dev tools. There should now be two keys in local storage for colors:

web_stories_ui_panel_settings:stylepreset-colors (new)
web_stories_ui_panel_settings:stylepreset-color (old)

Expanding the panel should update the new key's value

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Open the editor
Click on the background or an element with color
Open the dev tools. There should now be two keys in local storage for colors:

web_stories_ui_panel_settings:stylepreset-colors (new)
web_stories_ui_panel_settings:stylepreset-color (old)

Expanding the panel should update the new key's value

UAT

[x] UAT should use the same steps as above.

Reviews
Does this PR have a security-related impact?
no
Does this PR change what data or activity we track or use?
This pr changes the way we store the sizing data for the saved colors panel.
Does this PR have a legal-related impact?
no
Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #6740

@samwhale I'm looking at the gif in the description and wondering how come the Saved Styles and Saved Colors have exactly the same content -- is this a bug happening for you? (Note: I haven't checked the PR yet!)

@samwhale I'm looking at the gif in the description and wondering how come the Saved Styles and Saved Colors have exactly the same content -- is this a bug happening for you? (Note: I haven't checked the PR yet!)

Oh haha, I added both of the same colors to each panel while I was playing with the panels 😅. Sorry, that's a little confusing to watch

Oh haha, I think I added both of the same colors to each panel while I was playing with the panels 😅. Sorry, that's a little confusing to watch

The Saved Styles looks very different from the Save Colors content (it has square preview with text inside) -- so even if you'd add similar colours, it would still look very different 😛

Oh haha, I think I added both of the same colors to each panel while I was playing with the panels 😅. Sorry, that's a little confusing to watch

The Saved Styles looks very different from the Save Colors content (it has square preview with text inside) -- so even if you'd add similar colours, it would still look very different 😛

OH YOU'RE RIGHT I remember what happened 🤦. I had the presetType for both as color when I took this gif
I realized that they weren't different right as I pushed this up


[ENH]  add troubleshooting to docs

Add troubleshooting steps to docs for various issues we have seen ..

Can you quickly list before we forget?


Implement Hive uninstallation via deleting HiveConfig 'hive'.

If the HiveConfig named 'hive' (the only name we recognize) is deleted,
Hive components will be torn down.
This is accomplished by automatically adding owner references to most
resources the hive-operator creates, to the HiveConfig hive.

This is a cherry-pick of https://github.com/openshift/hive/pull/952 for OCM, mostly just to make sure e2e tests pass after backport.

/test e2e

Added a fix for the OpenShift specific error.

/lgtm


clean-conversation-sidebar: Cannot read property 'remove' of undefined

CHROME Version 90.0.4430.72 (Official Build) (64-bit)
Installed the Chrome Extension and it is not working. I am getting the below console error. (It is a private repo).
Refined GitHub → clean-conversation-sidebar → TypeError: Cannot read property 'remove' of undefined
at clean (refined-github.js:6568)
at runFeature (refined-github.js:1942)
at setupPageLoad (refined-github.js:1955)
at refined-github.js:1988
Search issue: https://github.com/sindresorhus/refined-github/issues?q=is%3Aissue+Cannot read property 'remove' of undefined
Open an issue: https://github.com/sindresorhus/refined-github/issues/new?labels=bug&template=bug_report.md&title=`clean-conversation-sidebar`%3A Cannot read property 'remove' of undefined
logError @ refined-github.js:1923

Yup, Looks like they are removing all the class names and making the generic

We’ll have to use select().closest() to find them. Each block likely has a form element with a URL that ends in /assignees or something

Actually that form and .js-* classes might appear only when you’re a collaborator/triager. If there are no specific classes we might have to find each section via textContent 😟

I see aria-label="Select milestones" on for example https://github.com/sindresorhus/got/issues/1306

Indeed each section has either the form itself, even when not editable, or some identificative class 🎉


Update User Guide for Delete and Clear commands

Fixes #39

Codecov Report

Merging #47 (de4a09b) into master (19ed953) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #47   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

changed branch name


Changed default value of "required" in config.xsd

1. Why is this change necessary?
The config.xsd and XmlConfigDefinitionReader.php have different default values for the required-property of configuration elements. This leads to unexpected behaviour if you omit the property from your plugin's config.xml, and to irritating notifications from IDEs if you manually set it to true
https://github.com/shopware/shopware/blob/a2dfabd84566195c9413aa6ef41977f89c2cf17c/engine/Shopware/Components/Plugin/XmlConfigDefinitionReader.php#L115
https://github.com/shopware/shopware/blob/a2dfabd84566195c9413aa6ef41977f89c2cf17c/engine/Shopware/Components/Plugin/schema/config.xsd#L36
2. What does this change do, exactly?
In the config.xsd, the default value of the property is now defined as false. I would much rather have set the default value in XmlConfigDefinitionReader.php to true, since I believe optional configurations are the exception, however, that might break existing plugins.
3. Describe each step to reproduce the issue or behaviour.
A picture is worth a thousand words:

4. Please link to the relevant issues (if any).
None that I know of
5. Which documentation changes (if any) need to be made because of this PR?
None
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-21228
Please use this issue to track the state of your pull request.

Hi @oktupol, thank you very much for your contribution! 💙
Sorry it kept us so long 😭


Documentation: add a high-level diagram of the LSDB pipeline

It would be useful to show users what LSDB does on different stages of the pipeline: where we read metadata, where we read data and do computations, where we operate on hipscat-level w/o reading the data.

Closed by https://github.com/astronomy-commons/lsdb/pull/363.


Tolerate i64 call indirect

This change lets the validator accept i64 indices.
Notice that this modification does not conform with the current WASM spec, so we cannot upstream it.
Eventually a 64-bit table version will land, coming from upstream, at which point we can remove our local hack.

This should be ready to go now (after the sandbox-feature PR).

I tried to rebase but it looks like all the changes from the other PR are still there.  (I guess something like diffbase is needed, but I don't know how that is done.)

This is good to go now.


UI: Topics and keywords into left bar in detailed study view

Original report on BitBucket by Taina Jääskeläinen.

Move topics and keywords to the left bar in the detailed study view, with a cut-off point.
Cut-off for topics: 6. Cut off for keywords: 15.
Similar results are moved to be below Topics and Keywords.

Keywords are currently cut off at 12, having more will give a "read more" button, see: https://datacatalogue.cessda.eu/detail?lang=en&q=b5b5cef69fa436922d5328d538d72ffc924a33229aac9f89fed42709e5327035
Topics appears to not have a cut off based on numbers in this and #353 , see: https://datacatalogue.cessda.eu/detail?lang=en&q=3b4951c1746400c006863d77a2c810e335d241101baf5db774bcdc5950688746
Closing this issue, so that the main issue for this is #353


Number of projects

The number of projects would be interesting and could look like this:

better?

yeah

done with https://github.com/dotabap/dotabap.github.io/pull/23


Last column float direction setting

Consider adding last-column float direction (which defaults to right) variable back into Foundation settings file please. Currently removed

Many thanks for pushing this request back in. Great timing as I'm starting on a new build today!


Flaky CLI Test

This failure popped up during a run of this PR.
--- PASS: TestRunCLI_ExecuteInsert_WithSignals (0.02s)
=== CONT  TestRunCLI_ExecuteInsert
Using database db
Using database db db
Unknown precision "". Please use rfc3339, h, m, s, ms, u or ns.
--- PASS: TestParseCommand_Use (0.03s)
=== CONT  TestRunCLI
--- PASS: TestRunCLI_ExecuteInsert (0.01s)
=== CONT  TestNewCLI
--- PASS: TestNewCLI (0.00s)
=== CONT  TestParseCommand_Consistency
Unknown consistency level "one;". Please use any, one, quorum, or all.
Unknown consistency level "one;". Please use any, one, quorum, or all.
--- PASS: TestParseCommand_Consistency (0.00s)
--- PASS: TestParseCommand_Insert (0.04s)
Unknown precision "". Please use rfc3339, h, m, s, ms, u or ns.
Connected to http://<IP_ADDRESS>:46472 version x.x
InfluxDB shell version: y.y
Enter an InfluxQL query
> panic: close of closed channel

goroutine 124 [running]:
github.com/influxdata/influxdb/cmd/influx/cli_test.TestRunCLI.func1(0xc00033fc00)
	/root/go/src/github.com/influxdata/influxdb/cmd/influx/cli/cli_test.go:53 +0x57
created by github.com/influxdata/influxdb/cmd/influx/cli_test.TestRunCLI
	/root/go/src/github.com/influxdata/influxdb/cmd/influx/cli/cli_test.go:52 +0x36e
FAIL	github.com/influxdata/influxdb/cmd/influx/cli	1.413s
?   	github.com/influxdata/influxdb/cmd/influx_inspect	[no test files]
?   	github.com/influxdata/influxdb/cmd/influx_inspect/buildtsi	[no test files]
?   	github.com/influxdata/influxdb/cmd/influx_inspect/deletetsm	[no test files]
?   	github.com/influxdata/influxdb/cmd/influx_inspect/dumptsi	[no test files]
testing: warning: no tests to run
PASS
ok  	github.com/influxdata/influxdb/cmd/influx_inspect/dumptsm	1.019s [no tests to run]
?   	github.com/influxdata/influxdb/cmd/influx_inspect/dumptsmwal	[no test files]
=== RUN   Test_exportWALFile
--- PASS: Test_exportWALFile (0.01s)
=== RUN   Test_exportTSMFile
--- PASS: Test_exportTSMFile (0.01s)
PASS
ok  	github.com/influxdata/influxdb/cmd/influx

This is racy, as the c.Quit channel is closed in the test here:
https://github.com/influxdata/influxdb/blob/9ac37ca459bf15572f4d6c3adfc5ad99452b3a99/cmd/influx/cli/cli_test.go#L52-L54
In addition, it will be closed if the following line results in io.EOF
https://github.com/influxdata/influxdb/blob/c5ec3a32441c56be6477c0a897ec56e1c1714d7c/cmd/influx/cli/cli.go#L262
which leads to l being set to "exit", calling ParseCommand("exit") and finally executing the "exit" case in the switch statement:
https://github.com/influxdata/influxdb/blob/c5ec3a32441c56be6477c0a897ec56e1c1714d7c/cmd/influx/cli/cli.go#L284-L285

The two tests which close the channel in the test function don't actually test any useful behavior, so I'm deleting them.


Simplify renameSeq logic in InlineInstances

Apply @jackkoenig, @albert-magyar code review suggestion on https://github.com/freechipsproject/firrtl/pull/1868.

So, I fat fingered this with foldLeftOption instead of reduceLeftOption. :facepalm:


fix: add missing TouchEvent and KeyboardEvent to pc namespace

From https://forum.playcanvas.com/t/is-there-any-way-to-prevent-mouse-event-propagation/6077, I notice that we can not access pc.TouchEvent and pc.KeyboardEvent because of TouchEvent and KeyboardEvent are not exposed to pc namespace.

Thanks! 👍


fix(agg): do not gen unexpected agg expression

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
https://github.com/risingwavelabs/risingwave/pull/10252#discussion_r1234797162

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

It fix the bug which is introduced in this release so remove the "need-cherry-pick-v0.19" tag


[IOTDB-221] Add C++ client

Hi, I will close this PR because a new c++ module is added to the master branch in #1616


Bottom navigation style

Steps to reproduce
I can not change the bottom navigation style anymore.
Expected behavior
There use to be an option to change that.
Actual behavior
I have two manga sections instead of an history section once I click more
Crash logs
No response
Aniyomi version
preview r7447
Android version
Android 13
Device
Realme 8
Other details
No response
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] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

https://github.com/aniyomiorg/aniyomi/issues/1247


Add of grid context menu choice for "master reset" of all grid row coloring

DBeaver - Version 4.2.2 CE
Operating System: Windows 7 / Windows 8.1
It would be benificial if the grid context menu contained a choice for:
View Format -> Reset color by any (or equivalent)
This would reset all colors on all column values.
If I have set row coloring on different columns and in them different values, I have to choose each column and choose Reset color by .
Would be much easier to have a context menu choice that reset all row coloring independet on any columns and any values.
PS.  Yes/No" dialog that popped up. Just so that I don't reset coloring by mistake.

Agreed, this will be very helpful.


Can support kafka <IP_ADDRESS> version?

Hi ,  i want to ask  if  kafka-minion  support kafka <IP_ADDRESS>?

Hi, no Kafka 0.10 is not supported, see Feature list:

Supports Kafka <IP_ADDRESS> - 2.2.x (last updated 16th Jun 2019)

This is neither planned to add, I'm sorry.


Startup with Offline Wallbox(es) in EVCC

Is your feature request related to a problem? Please describe.
evcc fails to start if any of the connected charge points (Wallboxes) are offline. Currently, evcc waits for all Wallboxes to respond during the startup process. If one Wallbox is offline or unresponsive, evcc does not start and logs a fatal error message, then attempts to restart after a set period, but remains down.
Describe the solution you'd like
I would like evccto have the capability to start even if some of the Wallboxes are offline. Specifically, evcc should start without the offline Wallbox(es) and continue operating with the available Wallboxes. This would ensure that the charging system remains functional even if one or more Wallboxes are temporarily offline. A more resilient startup behavior would improve the reliability and user experience of  evcc.
Describe alternatives you've considered
An alternative solution might be adding a feature where evccretries to connect to the offline Wallbox(es) in the background while remaining operational with the available Wallboxes.
Additional context
In the current scenario, the evcc logs the following fatal error when a Wallbox is offline:
[main  ] FATAL 2024/06/19 11:30:35 cannot create charger 'EV_Box_Garage': cannot create charger type 'template': cannot create charger type 'ocpp': timeout [main  ] FATAL 2024/06/19 11:30:35 will attempt restart in: 5m0s

On the otherhand side doing so would mean that the fauly box stays offline indefinitely. Sofar we have always opted for the seemingly safer path and try to restart instead. We're not planning to change this anytime soon.

Wouldn't it be an option for evcc to start as soon as at least one wallbox is on? And for the other wallboxes, it could be monitored in the background to see if they respond.
This way, evcc would be operational, and if an inactive wallbox goes online, it would also change to active status in evcc.

Imagine 2 boxes. The car is connected to the flaky one. Restarting often can fix the problem. Now you don't restart and instead are happy with the box without vehicle. The car will not get charged. Does that sound good?
Imho the real question here is: why would the box not be available or become available when retrying? What is the actual use case? My 2 boxes have been running for 5 years without any single fault...

I've created a more general issue describing the desired behavior. I'll close this one as duplicate the the new https://github.com/evcc-io/evcc/issues/14496


Should i test action creators created by createAction?

Howdy.
I don't know where else to ask, so i'll ask here:

Should i test action creators created by createAction()?

For example, i want to have action creator, that adds apple to the basket. So i write this code:
import {createAction} from 'redux-actions';
import {ADD_APPLE} from 'AppConstants.js';

export const addApple = createAction(
  ADD_APPLE,
  (apple) => ({apple}),
);

Should i have tests for it? Something like that:
import {addApple} from 'Actions.js';
import {ADD_APPLE} from 'AppConstants.js';

describe('Actions', () => {
  describe('addApple', () => {
    it('should create action with passed data', () => {
      const apple = {kind: 'red', isRipe: true};
      expect(addApple(apple)).toEqual({type: ADD_APPLE, payload: {apple}})
    });
  });
});

It seems to me, that this is monkey business: copying and pasting type constants, writing same tests for every action creator (dozens of them in my current projects). But on other side, it seems like right thing to do, if you want 100% test coverage for your code.
So, should i test them? Or testing reducer function is enough?

PS: I'm a novice in redux and novice in testing, just wanna know what big boys doing.

Imo you should test action creators and your given example is the way to go. You are actually testing if your passed arguments, like type, payload function and/or meta function are working correctly.


ensure_index syntax

Does anyone know about ensure_syntax?
I tried
mongo:ensure_index(DBConn, conversations, {key, {page_id, 1}, unique, true, dropDups, true})
but not work at all

mongo:ensure_index(Worker, Collection, {key, {tag, 1}, name, <<"tag_index">>})
this works, name parameter is optional.
The dropDups option is no longer available since mongo 3. What version of mongo do you try?

i'm using mongo 2.6.5
It turns out when I put my code into console, it works and return ok, but if I put it in my application, even at the end, the app crashs

Could you post a crash log here?

This is crash log. Thank you
=ERROR REPORT==== 16-Apr-2015::14:50:25 ===
** Generic server <0.104.0> terminating
** Last message in was {ensure_index,conversations,{key,{page_id,1}}}
** When Server state == {state,#Port<0.3347>,49181,<<>>,
                               {conn_state,unsafe,master,pancake_development}}
** Reason for termination ==
** {function_clause,
       [{mongo_protocol,'-put_message/3-lbc$^0/2-0-',
            [{name,<<"i_page_id_1">>,unique,false,dropDups,false,key,
                 {page_id,1},
                 ns,<<"pancake_development.conversations">>},
             <<>>],
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mongo_protocol,put_message,3,
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mc_worker_logic,'-encode_requests/2-fun-0-',3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,22}]},
        {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
        {mc_worker_logic,make_request,3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,71}]},
        {mc_worker,handle_call,3,
            [{file,"src/connection/mc_worker.erl"},{line,44}]},
        {gen_server,try_handle_call,4,[{file,"gen_server.erl"},{line,607}]},
        {gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,639}]}]}

=ERROR REPORT==== 16-Apr-2015::14:50:25 ===
** Generic server <0.98.0> terminating
** Last message in was {'EXIT',<0.48.0>,
                        {function_clause,
                         [{mongo_protocol,'-put_message/3-lbc$^0/2-0-',
                           [{name,<<"i_page_id_1">>,unique,false,dropDups,
                             false,key,
                             {page_id,1},
                             ns,<<"pancake_development.conversations">>},
                            <<>>],
                           [{file,"src/core/mongo_protocol.erl"},{line,49}]},
                          {mongo_protocol,put_message,3,
                           [{file,"src/core/mongo_protocol.erl"},{line,49}]},
                          {mc_worker_logic,'-encode_requests/2-fun-0-',3,
                           [{file,"src/connection/mc_worker_logic.erl"},
                            {line,22}]},
                          {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
                          {mc_worker_logic,make_request,3,
                           [{file,"src/connection/mc_worker_logic.erl"},
                            {line,71}]},
                          {mc_worker,handle_call,3,
                           [{file,"src/connection/mc_worker.erl"},{line,44}]},
                          {gen_server,try_handle_call,4,
                           [{file,"gen_server.erl"},{line,607}]},
                          {gen_server,handle_msg,5,
                           [{file,"gen_server.erl"},{line,639}]}]}}
** When Server state == {state,#Port<0.2447>,45084,<<>>,
                               {conn_state,unsafe,master,pancake_development}}
** Reason for termination ==
** {function_clause,
       [{mongo_protocol,'-put_message/3-lbc$^0/2-0-',
            [{name,<<"i_page_id_1">>,unique,false,dropDups,false,key,
                 {page_id,1},
                 ns,<<"pancake_development.conversations">>},
             <<>>],
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mongo_protocol,put_message,3,
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mc_worker_logic,'-encode_requests/2-fun-0-',3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,22}]},
        {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
        {mc_worker_logic,make_request,3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,71}]},
        {mc_worker,handle_call,3,
            [{file,"src/connection/mc_worker.erl"},{line,44}]},
        {gen_server,try_handle_call,4,[{file,"gen_server.erl"},{line,607}]},
        {gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,639}]}]}
[*] Waiting for messages. To exit, press Ctrl+C

I investigate this issue.
When using mongodb-erlang name parameter is a must. That is why mongo:ensure_index(DBConn, <<"test">>, {key, {key, 1}}) won't work.
In shell, when you add index with db.test.createIndex({"test" : 1}, []) - index name will be assigned from key parameter (name = test).
So, for your situation - just add name to the document:
mongo:ensure_index(DBConn, conversations, {key, {page_id, 1}, unique, true, dropDups, true, name,  <<"page_id">>}).

Please post me your call to mongo, which leads to the crash.

This is my call to mongo:
mongo:ensure_index(DBConn, conversations, {key, {page_id, 1}, unique, true, dropDups, true})
I called it immediately below mongo connect.
By the way, please apologize for the delay in response.

I tried your method, didn't work.
Do you have any idea about this issue?

Please - post you call to mongo:ensure_index

this is my mongo:ensure_index call:
mongo:ensure_index(DBConn, conversations, {key, {page_id, 1}, unique, true, dropDups, true, name, <<"page_id_1">>})

and crash log
=ERROR REPORT==== 9-Jun-2015::18:29:09 ===
** Generic server <0.94.0> terminating
** Last message in was {ensure_index,conversations,
                           {key,
                               {page_id,1},
                               unique,true,dropDups,true,name,
                               <<"page_id_1">>}}
** When Server state == {state,#Port<0.2735>,45084,<<>>,
                               {conn_state,unsafe,master,cake_development}}
** Reason for termination ==
** {function_clause,
       [{mongo_protocol,'-put_message/3-lbc$^0/2-0-',
            [{name,<<"page_id_1">>,unique,true,dropDups,true,key,
                 {page_id,1},
                 ns,<<"cake_development.conversations">>},
             <<>>],
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mongo_protocol,put_message,3,
            [{file,"src/core/mongo_protocol.erl"},{line,49}]},
        {mc_worker_logic,'-encode_requests/2-fun-0-',3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,22}]},
        {lists,foldl,3,[{file,"lists.erl"},{line,1261}]},
        {mc_worker_logic,make_request,3,
            [{file,"src/connection/mc_worker_logic.erl"},{line,71}]},
        {mc_worker,handle_call,3,
            [{file,"src/connection/mc_worker.erl"},{line,44}]},
        {gen_server,try_handle_call,4,[{file,"gen_server.erl"},{line,607}]},
        {gen_server,handle_msg,5,[{file,"gen_server.erl"},{line,639}]}]}

May be you should try to set name as key? <<"page_id">> instead <<"page_id_1">>

I tried both cases and they both crashed. Does this issue relate to my mongo version?
Thanks

I do not test ensure index on 2.6.5, so I can't answer. It will be good - if you test your code on newer version of mongo.

I will upgrade mongo and report back to you asap!

On Jun 9, 2015, at 6:58 PM, Valery<EMAIL_ADDRESS>wrote:
I do not test ensure index on 2.6.5, so I can't answer. It will be good - if you test your code on newer version of mongo.
—
Reply to this email directly or view it on GitHub https://github.com/comtihon/mongodb-erlang/issues/64#issuecomment-110331136.

Thanks!

Hello,
I just tried mongo 2.6.9, 2.6.10, 3.0.3 and they did not work at all :(

Broken Tech.  Sorry. NUB database is at SourceForge.
On Fri, Jun 19, 2015 at 3:47 PM, Phong Mai<EMAIL_ADDRESS>wrote:

Hello,
I just tried mongo 2.6.9, 2.6.10, 3.0.3 and they did not work at all :(
—
Reply to this email directly or view it on GitHub
https://github.com/comtihon/mongodb-erlang/issues/64#issuecomment-113622735
.


function-name: add private static method regex

If you have a private static method, currently I believe the linter reads it as private first and then uses the private regex. But I believe it should still use the static regex, since its a static method and its a great way to show that it is.
so private static methods currently are linted to be: getPotatoes() instead of GET_POTATOES().
Was wondering if y'all agree.

Great catch, thanks for posting this! This feels like something that should be configurable. Relevant options:

Validate private statics as privates
Validate private statics as statics
Validate private statics as both both privates and statics

Thoughts?


Stack Traces for mapping handlers

Often when running a subgraph, a failure will happen, and there is no indication of what line caused the error. This can be frustrating for debugging. Often this leads to commenting out blocks of code until the line can be identified. Not a bad solution, but if the failure happens after 30 mins or more of subgraph syncing, it can slow down the process of iterative development a lot.
Showing stack trace errors, up to the line that failed would help, along with an error message.
This could have some overlap with this issue in graph-ts -  https://github.com/graphprotocol/graph-ts/issues/40

Our stack traces have since improved.


Add zone inputs, update tests, add usage

what

Add zone inputs, update tests, add usage

why

Verify zone name exists
Dynamically retrieve zone name based on data source inputs

references

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/route53_zone
Closes https://github.com/cloudposse/terraform-aws-route53-cluster-hostname/issues/41

/test all


View Submissions page - sorting by date

The View Submissions page in the Curation Interface does not sort correctly by 'date created'.
https://www.ebi.ac.uk/gwas/curation/imported_submissions
Sorting correctly would be particularly useful for the 'imported' sub-tab.

Investigating this issue I discovered that the date format is not consistent with the rest of the app, namely Studies page, so I changed the date format to "yyyy-mm-dd" for Submissions page which in turn solved the sorting issue.

@earlEBI please verify this is working now

Yes, looks good!


added Niekon01 as a contributor

@dhakal0kushal can you undo the changes to package-lock.json?

@angle943 done :)


Aerogear 3132

Motivation
Use fixed versions of platforms for showcase application to avoid constant refreshes that can affect stability.

Trying now.

@aliok I is still not finished. Js-sdk changes are required.

Ok, was stuck at some other thing anyway. @wtrocki let me know when it's ready for verification.

It should be ok now!


After actions don't trigger when rehydrating state using State.from

Bug or feature request?
Bug
Description:
I am trying to resume an interpreter using a state as the argument when starting. However the after actions on that state do not trigger leaving my interpreter stuck in that state.
(Bug) Expected result:
State transition after the specified time.
(Bug) Actual result:
Interpreter waiting for further input.
(Bug) Potential fix:
I haven't had a chance to look at the source code for Xstate, but I did find a very dirty workaround. I added an extra loading state to my machine that I will load up first and then transition into the actual state I want to restore to. This way everything works as expected.
Link to reproduction or proof-of-concept:
Here's a code sandbox that illustrates the problem:

Here's the actual code of my minimal example for easy reference:
import { Machine, State } from "xstate";
import { interpret } from "xstate/lib/interpreter";

const stateChart = {
  context: {},
  states: {
    ping: {
      after: {
        1000: "pong"
      }
    },
    pong: {
      after: {
        1000: "ping"
      }
    }
  }
};

const machine = Machine(stateChart);
const service = interpret(machine);
service.onTransition(current => console.log(current.value));

const initState = State.from("ping", {});
service.start(initState);

When a delayed transition occurs, a special action is internally sent to send a timer event after the specified delay.
However, when you specify the initState, you aren't specifying any actions. To fix this:

Define an initial state (which you should do anyway)
Use the machine.initialState as the initial state (this is the default when calling service.start()).

The modified code below works:
import { Machine, State } from "xstate";
import { interpret } from "xstate/lib/interpreter";

const stateChart = {
  context: {},
  initial: "ping",
  states: {
    ping: {
      after: {
        1000: "pong"
      }
    },
    pong: {
      after: {
        1000: "ping"
      }
    }
  }
};

const machine = Machine(stateChart);
const service = interpret(machine);
service.onTransition(current => console.log(current.value));
service.start();

@davidkpiano Sure it works if you don't pass an initial state to the interpreter and set the initial up front. However that wasn't the issue here. I want to be able to enter the state machine with a specific context at any point of the machine when restoring the system. Therefore I'm not using the initial state but State.from.
What if I wanted to start the above machine on the pong state state with an arbitrary context? Now the delayed transition on that wouldn't trigger.

What if I wanted to start the above machine on the pong state state with an arbitrary context? Now the delayed transition on that wouldn't trigger.

There's two approaches to this:

Previously existing state - https://xstate.js.org/docs/guides/states.html#persisting-state
"Dynamic" initial state - https://xstate.js.org/docs/guides/statenodes.html#transient-state-nodes


QAOA: remember intermediate results and choose the best of all known eigenstates

What is the expected enhancement?
When using result = MinimumEigenOptimizer(QAOA(...)).solve(qubo), the intermediate results of QAOA are not considered in the final outcome. Only the eigenstates from the final parameter values are sampled in
https://github.com/Qiskit/qiskit-terra/blob/23acf363e52b6436998afc3150e117625cc6b7d6/qiskit/algorithms/minimum_eigen_solvers/vqe.py#L576
and are returned to  MinimumEigenOptimizer._solve_internal in qiskit-optimization to choose the best from. This is highly counter-intuitive since valuable results might get lost during the optimization. Or, in other words, the final result might be worse than an intermediate result.
To fix this issue without much effort, the optimizer function
https://github.com/Qiskit/qiskit-terra/blob/23acf363e52b6436998afc3150e117625cc6b7d6/qiskit/algorithms/minimum_eigen_solvers/vqe.py#L598
can be extended to store state_fn from self._circuit_sampler.convert (as in _get_eigenstate) over the course of the optimization.  With this improvement, all eigenstates of the optimization history can be returned instead of just one from the best parameter set.
(Naturally, this enhancement request also applies to VQE in general.)

I will just note that when an optimizer is computing a finite diff gradient that the objective function here will get called for that gradient computation as well which would leads to a much larger set of values than just the points traversed.
Also the minimum_eigenvalue/state  is a result of the minimum_eigen_solver and may not be solved variationally where intermediate states are available. Hence the availability of such intermediate results to a higher level algorithm would have to be treated conditionally upon how things are solved.
When using QAOA (or VQE) directly one can use the callback to capture all the intermediate parameters and given the anstaz get the states afterwards so you can do this already at this level of the algorithms with whats there. This request seems perhaps to be asking to make this more readily available for consumption, particularly for the Qiskit Optimization level. As such perhaps a corresponding Issue should be opened there referencing this one in order to have that discussion at that level too.

Thanks @woodsp-ibm, I have a follow-up question regarding your hint for the callback: If I understand correctly, the callback only takes the parameter values as an argument and therefore, the measurement results (i.e., counts) can not be retrieved in this way. Consequently, another run has to be performed to get (new) counts, which is an unsuited approach for real hardware in my opinion and also not ideal for simulators.

VQE and QAOA changed when they were re-written to primitives. As this is about the former algorithms, which are now deprecated and soon to be removed, I am going to close this off.


PHP: fix php pecl installation error on windos

WIP
Use this PR to checkout the changes on Windows and make test.
Update:
Seems it works:
$c = new Grpc\Server();
var_dump($c);
print("INVALID_ARGUMENT code = ".Grpc\STATUS_INVALID_ARGUMENT);

It has output
object(Grpc\Server)#1 (0) {
}
INVALID_ARGUMENT code = 3

It is better if you can double check. I was using the php.exe under .\Release\php.exe.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[microbenchmarks] No significant performance differences


[PE-D][Tester B] Edit Command shows incorrect error messages for index numbers longer than Long (>MAX)

In the first case the index provided is 20 and the error message shown is correct.

In the second case, the index provided is a long number and an overflow error occurs and the error message shown is incorrect

Labels: severity.Low type.FunctionalityBug
original: Nihirraa/ped#3

Per guidelines, integer overflow is not considered a bug


feat: support snapshoting

Fix #20

Update react-snap documentation https://github.com/stereobooster/react-snap/pull/58 ❤


Add top border to tables that do not have a  element

Consider a table without a thead, such as:

      The Shawshank Redemption
      Crime, Drama
      14 October 1994

      The Godfather
      Crime, Drama
      24 March 1972

      Schindler's List
      Biography, Drama, History
      4 February 1994

      Se7en
      Crime, Drama, Mystery
      22 September 1995

In this case, Spectre will not add a border to the top row, as the
corresponding CSS only defines bottom borders:

This fix adds the top border if there is no thead element (i.e.
tbody is the first child of the table).
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Hey @picturepan2, did you have time to look at this PR?

Closing stale PRs...


Update chance.js

Make address numbers in US more accurate, 6 digit address numbers are very rare and 5 digit numbers are rare. Sources:
https://seanmarshall.ca/2019/06/10/punkeydoodles-corners-and-the-worlds-highest-numbered-address/ https://github.com/faker-js/faker/blob/next/src/locales/en/location/building_number.ts https://www.city-data.com/forum/rural-small-town-living/275154-whats-highest-address-number-youve-seen.html

I would include the unit_number and civic_fraction, with unit_number 10% chance and civic_fraction 1% chance, and test/test_address.js should be updated.


No ViewManager defined for class RNSVGPath

I used you example in 'react-native-svg-animations/examples/IngenuityPreloaderSVG'
import Example from 'react-native-svg-animations/examples/IngenuityPreloaderSVG'

It seems to be a native code linking problem (of one of the dependencies).
I've had the same issue, solved it by running
react-native link react-native-svg
cd android && ./gradlew clean

Updated dependencies already, also on RN 0.60+ auto linking is enabled and I think there is no dependency that needs to be linked.


Develop 60 add dose rate docs

closes #60
The updated documentation can be found here

Codecov Report

Merging #133 (0bf0bdd) into main (0eafedd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   81.35%   81.35%
=======================================
  Files         100      100
  Lines        5299     5299
=======================================
  Hits         4311     4311
  Misses        988      988

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Updating Existing Connectors

This is more a question than feedback, but is it currently possible to update existing Connectors with additional Target URLs or do we have to request a whole new provider?

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

ID: b2eaf591-7ab8-e405-3366-94b46146be71
Version Independent ID: 2783a865-1195-02ed-e9c7-f5e40c6f9d56
Content: Actionable messages via email developer dashboard guide - Outlook Developer
Content Source: docs/actionable-messages/email-dev-dashboard.md
Product: outlook
Technology: o365-connectors
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

I have been looking for REST or PowerShell API for updating but haven't found any, although, this doesn't mean it doesn't exists.
Being able to update a provider (at least with the Organization scope) is very much needed, if not, all applications using the provider also needs to be updated.
MS: Please update your user stories for this

There are no APIs for this as @erapade has found. You can update an existing provider in the portal at https://aka.ms/publishoam to add additional target URLs.


Use unknown type for Typescript files

Is your feature request related to a problem? Please describe.
Eslint complains about using {} as a type instead of unknown.
Describe the solution you'd like
User unknown type in place of {} for typescript gen files.
Describe alternatives you've considered
No response
Is your feature request related to a problem? Please describe.
No response

Please disable eslint for the emitted artifacts by GraphQL Code Generator.
You can either add those files to your eslint ignore list or instead add the eslint ignore comment via the add plugin. https://the-guild.dev/graphql/codegen/plugins/other/add


russian locale files

thanks for your contribution :) I will release a new version containing the russian language files the next days.


CC0009 should ignore IDisposable types

CC0008 should ignore IDisposable types as it will pop a CA2000 code analysis warning considering an initializer declares a temporary variable (see : http://haacked.com/archive/2013/01/11/hidden-pitfalls-with-object-initializers.aspx/ for more details)
Example:
public sealed class Owner : IDisposable
{
    Dependency dependency;

    public Owner()
    {
        dependency = new Dependency();
        dependency.SomeProperty = "Blah";
    }

    public void Dispose()
    {
        dependency.Dispose();
    }
}

public sealed class Dependency : IDisposable
{
    public string SomeProperty { get; set; }

    public void Dispose()
    {
    }
}

Current output after fix applied (if it is a code fix bug):
public sealed class Owner : IDisposable
{
    Dependency dependency;

    public Owner()
    {
        // This is going to cause some issues.
        this.dependency = new Dependency { SomeProperty = "Blah" };
    }

    public void Dispose()
    {
        this.dependency.Dispose();
    }
}

public sealed class Dependency : IDisposable
{
    public string SomeProperty { get; set; }

    public void Dispose()
    {
    }
}

Expected output after fix applied (if it is a code fix bug):
public sealed class Owner : IDisposable
{
    Dependency dependency;

    public Owner()
    {
        this.dependency = new Dependency();
        this.dependency.SomeProperty = "Blah";
    }

    public void Dispose()
    {
        this.dependency.Dispose();
    }
}

public sealed class Dependency : IDisposable
{
    public string SomeProperty { get; set; }

    public void Dispose()
    {
    }
}

Have you tried with a DEBUG build? I wonder if the debug and production compiled output differs. In any case, I may just link to this issue in my post. :smile:

@Haacked Yes, I did, no change there. That is good news, right? :)

👍 Nice!


Add .gitignore file + remove *.class files

Only java source code files should be pushed to this repository.
Maybe you should add this guideline to the contribution guide.

I added a significant amount of files to .gitignore in pull request 92 (and ignored/removed .class files). I think it'll cover this PR.

I checked your PR;
It consist of a lot of changes and an over defined .gitignore file that cover a lot files extensions that are out of java project context (.iso, .tar, ....).
NB :
i think that PR should handle one atomic change and not many changes (New files, Code formatting, ....). Because a big PR may contains some stuff that should be merged and another one that shouldn't.
Thanks

Even if I completely delete the .gitignore, this project needs a build system.

I agree with you,
Because with the actual structure it is hard to work on it on IDE,

I also understand the need to keep things simple per PR. That's the whole point. However, a single commit for each of the following is dumb:
VisualStudioCode, SublimeText, Eclipse, Windows, archives, Emacs, Microsoft Office temporary files, macOS, Java, JetBrains IDEs, Xcode, VisualStudio IDE, Maven, images
While it's easy to see this as overkill, I know that "beginners" (those for who this project is intended) will use anything they like and will most likely not know how to keep the repo clean. This is a precautionary measure. If you think that any of the above should be excluded, I'm all ears.

By the way, @Kyle-Pu, was this ever addressed, or is this repository dead?

By the way, @Kyle-Pu, was this ever addressed, or is this repository dead?

Hey @jwheeler31 ! Thanks for bringing this back up again. This repository is back under active moderation again. I've talked to Kyle, and we'll merge the pull request.

Hey @jwheeler31 ! Thanks for bringing this back up again. This repository is back under active moderation again. I've talked to Kyle, and we'll merge the pull request.

Awesome!
A follow up - is there a desire to add a build system to this project? I do not recommend Ant (does the job, but isn’t mainstream anymore). I haven’t used Gradle, but I’m decent with Maven.
The upsides of a build system are: simplified builds (new programers, see: compilation), simple IDE setup, use of other open source libraries, and ease of quality control (unit tests/code coverage/etc) amongst other things (this assumes that it sticks to one compiled language - I’d recommend that differing languages be separated into their own repos). The downside is that it requires a large PR as it requires reorganization of the project, which results in package name changes and imports.

Awesome!
A follow up - is there a desire to add a build system to this project? I do not recommend Ant (does the job, but isn’t mainstream anymore). I haven’t used Gradle, but I’m decent with Maven.
The upsides of a build system are: simplified builds (new programers, see: compilation), simple IDE setup, use of other open source libraries, and ease of quality control (unit tests/code coverage/etc) amongst other things (this assumes that it sticks to one compiled language - I’d recommend that differing languages be separated into their own repos). The downside is that it requires a large PR as it requires reorganization of the project, which results in package name changes and imports.