Invalid conversion from radians to yards

Seems like conversion from radians to length if using Unit.yards calculates wrong because 1 meter = 1.09361 yards, but in factors earthRadius is divided on 1.0936 but there we should multiply.
/// Convert a distance measurement (assuming a spherical Earth) from radians to a more friendly unit.
/// Valid units: miles, nauticalmiles, inches, yards, meters, metres, kilometers, centimeters, feet
num radiansToLength(num radians, [Unit unit = Unit.kilometers]) {
  var factor = factors[unit];
  if (factor == null) {
    throw Exception("$unit units is invalid");
  }
  return radians * factor;
}

/// Earth Radius used with the Harvesine formula and approximates using a spherical (non-ellipsoid) Earth.
const earthRadius = 6371008.8;

/// Unit of measurement factors using a spherical (non-ellipsoid) earth radius.
/// Keys are the name of the unit, values are the number of that unit in a single radian
const factors = <Unit, num>{
  Unit.centimeters: earthRadius * 100,
  Unit.degrees: earthRadius / 111325,
  Unit.feet: earthRadius * 3.28084,
  Unit.inches: earthRadius * 39.370,
  Unit.kilometers: earthRadius / 1000,
  Unit.meters: earthRadius,
  Unit.miles: earthRadius / 1609.344,
  Unit.millimeters: earthRadius * 1000,
  Unit.nauticalmiles: earthRadius / 1852,
  Unit.radians: 1,
  Unit.yards: earthRadius / 1.0936,
};

Hi @dimonkomaron, thank you for bringing this issue to our attention. 👍 👍
Would you be open to contribute and create a pull request to fix this problem?
Otherwise it might take longer until this bug is fixed.

@lukas-h Yes, ok, will try :)
Also have a question is there reason why spherical radius is used as it can loss precision near the equator or ellipsoidal radius has it's own flaws?
/// Earth Radius used with the Harvesine formula and approximates using a spherical (non-ellipsoid) Earth. const earthRadius = 6371008.8;

@dimonkomaron It is merged. Until the next release on pub.dev you can just use the package like this:
dependencies:
  turf:
    git:
      url: https://github.com/dartclub/turf_dart.git
      ref: main

I will mark this issue as done.


Added appendTo to Halogen.Util

Users often might want to append to something else than body.
Of course the problem of "Get an element and append another element to it", should be solved by some other libraries (purescript-simple-dom, purescript-jquery), but it seems that at the moment we end up with purescript-dom which is a bit unwieldy for such a simple task.

May as well re-implement appendToBody as appendTo "body" if we're doing this? :)
Regarding your second point, I agree, that's why I only provided appendToBody originally - but querySelector is a perfectly valid thing to appeal to here, I didn't think of it for some reason, so :+1:

True

Thanks!


HDDS-7282. Add EstimatedKeyCount metrics for SCM DB

What changes were proposed in this pull request?
This ticket is to add the metric of EstimatedKeyCount for SCM DB.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7282
How was this patch tested?
unit test.

This change looks good now - thanks for taking on the changes. However there is one more thing I think we should change. We are calling getEstimatedRowCount on each RocksDB table twice, due to forming the concatenated string in getEstimatedKeyCountStr() and also the individual metrics. I guess the estimatedRowCount call is quite cheap in RocksDB, but it seems wasteful to call it twice.
Could you remove the getEstimatedKeyCountStr method, and build up the gson object as you iterate and create the individual metrics? That would move all the metrics logic into the metrics class, and also stop us calling into RocksDB twice per table.

I think the test failiures are related to this change. I tried running TestContainerStorageManagerHA from intellij and it fails with:
2022-10-05 15:20:37,004 [Listener at <IP_ADDRESS>/51308] INFO  impl.MetricsSystemImpl (MetricsSystemImpl.java:start(191)) - StorageContainerManager metrics system started
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x000000011f0238c8, pid=82478, tid=0x0000000000001603
#
# JRE version: OpenJDK Runtime Environment (Zulu <IP_ADDRESS>-CA-macos-aarch64) (8.0_322-b06) (build 1.8.0_322-b06)
# Java VM: OpenJDK 64-Bit Server VM (25.322-b06 mixed mode bsd-aarch64 compressed oops)
# Problematic frame:
# C  [librocksdbjni896780423911491940.jnilib+0x238c8]  Java_org_rocksdb_RocksDB_getLongProperty+0x6c
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# /Users/sodonnell/source/ozone2/hadoop-ozone/integration-test/hs_err_pid82478.log
#
# If you would like to submit a bug report, please visit:
#   http://www.azul.com/support/
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#

Process finished with exit code 134 (interrupted by signal 6: SIGABRT)

Same error two times. I tried with the master branch and the tests run fine, so there is something related to this test which is causing the problem.


Jodis connection error

Opening socket connection to server <IP_ADDRESS>/<IP_ADDRESS>:2181. Will not attempt to authenticate using SASL (unknown error)
Socket connection established to <IP_ADDRESS>/<IP_ADDRESS>:2181, initiating session
Unable to read additional data from server sessionid 0x0, likely server has closed socket, closing socket connection and attempting reconnect
Opening socket connection to server <IP_ADDRESS>/<IP_ADDRESS>:2181. Will not attempt to authenticate using SASL (unknown error)
Session 0x0 for server null, unexpected error, closing socket connection and attempting reconnect
java.net.SocketException: 断开的管道
at sun.nio.ch.SocketChannelImpl.checkConnect(Native Method)
at sun.nio.ch.SocketChannelImpl.finishConnect(SocketChannelImpl.java:739)

Have you started ZooKeeper in <IP_ADDRESS>:2181 before starting codis? If so, we should check out the log of ZooKeeper.

0x0 is not a valid session id. Same suggestion with @tobegit3hub , can you please paste the zookeeper log at the same time here? Thanks.

thanks I'm ok


Don't fail on re-search-forward.

Try to fix #357.

r? @brotzeit
(rust_highfive has picked a reviewer for you, use r? to override)

@jamii @Dushistov  can you please try this fix?

I get the same behaviour as before.

Thanks for the help. I'll ty again in the next few days; I tend to believe that this is a change of output format from rustfmt.

I can not reproduce #357 since I sync my rust-mode with master to get debug commit. Before it was reproduced once a day, now I can not reproduce it at all. But may be I need to write new code , and not edit old one as mentioned here. But I should say that, when I able to reproduce this bug, sometimes it shows "rustfmt" error just fine, and sometime it switch to stdin, and I couldn't find out regularity.


Redirect does not find subpath when noThrow

Hi,
My router component looks like this
class MainRouter extends PureComponent {
  render() {
    return (
      <Router>
        <Base path="/">
          <Redirect from="/" to="signIn" noThrow />
          <SignIn path="signIn" />
          <SignUp path="signUp" />
          <ConfirmEmail path="confirmEmail" />
        </Base>
      </Router>
    );
  }
}

I can navigate to /signIn and my SignIn component correctly renders as a child of my Base component.
If I navigate to the root /, I can see that the redirection changes my url from / to /signIn, my Base component renders but my SignIn component does not. I have to manually refresh the page to see the component appear.
If I remove the noThrow prop from the Redirect, everything works as expected (but I would prefer not to have an exception thrown).
My Base component always renders its children so I don't think it is related to that component.
Is there anything I am doing wrong ?

Thanks, closing as duplicate of #46


fix 表述不明确

fix “客户端GET和POST参数”表述不太清楚，改为“获得客户端GET和POST参数”使上下文表述统一且表述明确

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Session Creation and Edit on Backpack Leads to Unauthorized Error

Description

Creating or editing a session on the admin panel does not work.
Expected Behavior

The forms for creating and editing a session should redirect to the index table on the admin panel.
Actual Behavior

A 403 Unauthorized error is shown after submitting the form.
Steps to Reproduce

Log onto Backpack.
Open the Sessions tab.
Create or edit an existing session and hit the Save and Back button.

Context

Possible Fix

Other
Description

Additional Notes

Hey team! Please add your planning poker estimate with ZenHub @bcobo341 @JasonGadoury @m-triassi @RobertoNittolo


Fix include in manpage of X509_check_host

This changes the include to the file where the constants are defined.
Fixes #5255

Is a CLA really needed for a trivial fix like this?

No,  Edit your commit text to put "CLA: trivial" in the message and push-f it.
I agree this is trivial.

Merged to master (not going to backport); thanks!


Support connect-style middleware

Draft to support existing connect-style middleware.
To Do

[X] Support middleware that modifies response headers
[x] Support middleware that modifies response body
[x] Support middleware that ends response
[x] Documentation

👀


Modal won't animate on visible set to false when unmounting component

Environment
System:
OS: macOS 12.2.1
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 37.14 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
Yarn: Not Found
npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
Watchman: Not Found
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7935034
Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
Languages:
Java: 11.0.13 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.2 => 0.68.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Platforms
iOS & Android
Versions

Android: 11
iOS: 15.5
react-native-modal: 13.0.1
react-native: 0.68.2
react: 17.0.2

Description
The modal component won't animate fade out when it is used like this:
{isVisible && }
the original RN component completes the animation when declared in the manner above. Sometimes for performance purposes I prefer to not render the modal and its logic so it'd be nice to be able to use it as such.
Reproducible Demo
Use the modal in any component like this:
{isVisible && }

same here. any luck with that?

Any update here?


block_on in wasm

New to rust, so appreciate the help.
I'm trying to call block on an async function within rust and it's throwing errors, was wondering if this is possible.
Using
futures = {version="0.3.28", features=["executor"]}
use futures::executor::block_on;
pub fn test() -> Vec<f32> {
    block_on(test_diff())
}

pub async fn test_diff() -> Vec<f32> {
    let result = async_func.await;
    result
}

but getting a RuntimeError: unreachable error.
Any guidance would be greatly appreciated.

Generally, in WASM, you never want to actively "block". It's all single threaded, so "blocking" would make everything unresponsive. This is why mutexes and other thread blocking primitives don't work on WASM.


[BLD] add lint workflow

Currently, this will run in CI but never error. There's 80-90 files that need to be (automatically) updated, so I figured I'd do that in a separate PR to reduce noise.
See https://github.com/chroma-core/chroma/actions/runs/8805438845/job/24168001018?pr=2045 for an example workflow run.

thanks


Fix indentation of > for void elements.

This happens right now:
<form>
  <input
    type="number"
    name="quantity"
  >
    this text is indented but it shouldn't. And the </form> is not decreased enough as a result.
  </form>

The workaround is to use a self closing tag for the void element:
<form>
  <input
    type="number"
    name="quantity"
  />
  this works as indented but the void element does not really need to be closed.
</form>

It works fine for non-void elements though:
<form>
  <textarea
    type="number"
    name="quantity"
  >
    this works as indented.
  </textarea>
</form>

Some notes:
I tried only matching lines that start in \s*> if and only if a negative lookbehind for void element passed but I can't do that since lookbehinds must be of fixed length and the regex for a void element has [^>]+ in it.
This makes me think it's impossible to solve but I might be wrong.


AUR-Update without flags is broken on master

Hangs indefinitely when invoked with aur-update. Likely related to new handling for long options.

This has been fixed in the now-merged fix-update-all branch.


Remember original data when saving models with missing components

When loading a model with components from a missing library, the original XML data is stored in "fallback dom elements" for both the model object and the dangling connectors. These are then used when saving the model. This both elliminates the problem of accidentally saving a model with missing components and thereby losing data, and with the backup models being damaged during a failed library recompilation.
Resolves #1009.

Wow, this is an old issue, finally getting a fix. Good!
I will need to look at this a bit more thoroughly, hopefully tomorrow.

The only potential problem is if we change the HMF format in the future, and someone loads a model in the old format with a missing component. Then the old XML code will be stored and used for saving. I doubt this will cause too much trouble though, and it can be solved with some format conversion utility.


Incorrect values for fields price and price_incl in feed for products with prices below 1 EUR

Hi,
we recently got some notifications in Google Merchant Center for products that have prices differ in feed and on the page.
When we looked further into it we noticed that the common thing between these products is that they are all below 1 EUR. In this case the min_price and max_price correctly show the price without tax, however the fields "price" and "price_excl" already include the tax and as such the field "price_incl" will include the already taxed price including tax.

We traced the issue back to this line:
https://github.com/magmodules/magento2-channable/blob/master/Helper/Product.php#L1006
Since the price is below 1 and we're talking floats here the else-branch is executed. So instead of using
$price = $product->getPrice();

this code is used:
$price = $product->getPriceInfo()->getPrice('regular_price')->getAmount()->getValue();

Which returns the price already taxed, which is then put into processPrice and taxed again due to how the store configuration is set. As far as I can see there are two changes that work, however I don't know enough about Magento's price calculation to know if this is indeed a bug in Magento or in the Channable Extension, which is why I didn't want to do a pull request directly.
As far as I can see it, the AmountInterface always returns the price with all things included, unless you explicitly tell getValue() what to exclude:
$price = $product->getPriceInfo()->getPrice('regular_price')->getAmount()->getValue('tax');
or
$price = $product->getPriceInfo()->getPrice('regular_price')->getAmount()->getValue(['tax']);

The correct price is also returned when not using the Amount interface but instead get the Value directly:
$price = $product->getPriceInfo()->getPrice('regular_price')->getValue();

But again I am not qualified enough to know the difference or if that would break anything.
Could someone look into this please?

Hello @s-renz, thanks for bringing up this issue. We're currently looking into it, taking into account your detailed description and thorough review of the affected code lines and locations. I'll update you shortly, hopefully with a PR and/or a release where we've addressed this.

Any news on this yet?

Hello @s-renz, Yes, sorry for the delay but we have good news! We've addressed this issue with a fix that will be included in the upcoming release later this month. Thank you for your patience, and I've also ensured that the appropriate label has been added to this issue.

Hello @s-renz, Yes, sorry for the delay but we have good news! We've addressed this issue with a fix that will be included in the upcoming release later this month. Thank you for your patience, and I've also ensured that the appropriate label has been added to this issue.

Thanks! Looking forward to the new version

Hi There @s-renz ! Thank you again for opening this issue. I'm happy to share that we have covered this issue in our new release. I'm marking this issue as resolved for now. Please don't hesitate to reopen it if necessary. Thank you again for opening this issue!


Can't start autopilot under windows

On windows:
I run this
import AutoPilot from 'https://raw.githubusercontent.com/divy-work/autopilot-deno/master/mod.ts';

var pilot = new AutoPilot();

But I got an error:
PS C:\Users\IEUser\Downloads> deno run --unstable  -A .\autopilot.ts
Compile file:///C:/Users/IEUser/Downloads/autopilot.ts
[2020-06-12T10:47:35.601Z INFO] Downloading latest Autopilot release from Github
INFO loading autopilot plugin "autopilot_deno" from local "C:\Users\IEUser\Downloads\.autopilot_plugin\autopilot_deno_01d30cec9d205cfbaac871ba17c0ba9e.dll"
error: Uncaught Error: The specified module could not be found. (os error 126)
    at unwrapResponse ($deno$/ops/dispatch_json.ts:43:11)
    at Object.sendSync ($deno$/ops/dispatch_json.ts:72:10)
    at Object.openPlugin ($deno$/ops/plugins.ts:4:15)
    at prepare (https://raw.githubusercontent.com/divy-work/autopilot-plugin-prepare/master/mod.ts:66:15)
    at async https://raw.githubusercontent.com/divy-work/autopilot-deno/master/plugin/index.ts:26:20

My setup:
PS C:\Users\IEUser\Downloads> deno --version deno 1.0.5 v8 8.4.300 typescript 3.9.2

@darkship could you show the contents of .autopilot_plugin folder

PS C:\Users\IEUser\Downloads> ls .\autopilot.ts

    Directory: C:\Users\IEUser\Downloads

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----       12/06/2020     03:47            150 autopilot.ts

@darkship Not autopilot.ts. I meant .autopilot_plugin.
Run dir .autopilot_plugin and paste the output

sorry, I edited my post

I couldn't reproduction the issue. Could you try again in a new folder?

doesn't work...
on Linux I had to re-install deno because --unstable didn't exists

@darkship This is a big in the current version of Deno. Tommorow they plan to release a fix for it.
https://discord.com/channels/684898665143206084/684911491035430919/720987126468182157

@darkship Today's release should fix the bug. Could you try upgrading Deno deno upgrade and try again?

I tried on two different windows, one of them still has this issue

If you're getting the same error message even after upgrading to Deno 1.1.0, this is a deno bug itself.
You can try:

Create a fresh directory and try again.
Or
Reinstall deno 1.1.

If none of them work, it is worth an issue at the Deno repo.

The windows where it didn't work is a fresh windows install.
The windows where it worked is a development computer so it already had some development component like msbuild or c++ compiler.
=>I installed it on the fresh windows and now it works too.


Minor fix with checkHighResPoster function

this.height and this.width did not seem to return the image dimension when checking. Replaced this with image and it works fine.

Awesome. Can you also fix the this for naturalHeight?

Thanks for the PR!


I18n is not present on server side causing NoClassDefFoundError

https://github.com/T145/metaltransport/blob/master/src/main/java/t145/metaltransport/entities/EntityMetalCart.java#L326
Server crashes when this line is executed, since I18n is a client-only class (net.minecraft.client.resources.I18n).

Appears to be the same issue here:
https://pastebin.com/q2eq2km9


Use PCRE2 instead of PCRE

PCRE is now at end of life and is no longer actively maintained.
This PR lifts the required version to the maintained PCRE2, which makes sure that security vulnerability fixes are available.
There are some differences in behaviour that is expressed in this list, but in general PCRE2 seems to have a more explicit pattern interpreter, so invalid patterns are checked more aggressively.
There are no need to change existing testcases which is a good sign.
This PR also removes the PCRE study option since:
The new API ... was simplified by abolishing the separate "study" optimizing function; in PCRE2, patterns are automatically optimized where possible. (link)
Fixes #112

As a reference here are some benchmark tests of r3 with and without this PR, and using different version of PCRE2.
The latest PCRE2 version is required to get a bit better performance.
Results from benchmarking of "pcre_dispatch"

PCRE version
Min
Median
Max

pcre   8.39
1.418703 seconds10573038.67 i/sec
1.511755 seconds9922242.76 i/sec
1.572873 seconds 9536687.89 i/sec

pcre2 10.42
1.399972 seconds10714500.30 i/sec
1.488088 seconds10080049.76 i/sec
1.551644 seconds 9667165.38 i/sec

pcre2 10.41
1.442291 seconds10400120.21 i/sec
1.500147 seconds9999020.99 i/sec
1.608765 seconds 9323923.04 i/sec

pcre2 10.39
1.467327 seconds10222671.04 i/sec
1.535990 seconds9765688.58 i/sec
1.667526 seconds 8995361.96 i/sec

pcre2 10.37
1.504090 seconds9972807.01 i/sec
1.540420 seconds9737603.68 i/sec
1.691264 seconds 8869106.63 i/sec

Results taken from:
./run-benchmark (10 iterations of running 'bench', 3 runs of pcre_dispatch, 5000000 iterations each run)
gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
JIT enabled on PCRE/PCRE2

wow! this is just too awesome!


Improve types for button.presets.ts

Please verify the following:

[x] yarn ci:test jest tests pass with new tests, if relevant
[x] README.md has been updated with your changes, if relevant

Describe your PR
Improving the typings on the buttons preset by using Record<string, VIewStyle | TextStyle>.
I ran into a cryptic error that took a very long time to debug until I realized one of the custom presets I added in the textPresets was not inline-typed with an as TextStyle. Putting these top-level types on the object made the blunder easier to spot.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


✨ rename _op to _pull

What are you trying to do?
Given an action like:
		_alpine: alpine.#Build & {
			packages: bash: _
		}

The output is
6:59PM INF actions._alpine._dag."0"._op | computing
6:59PM INF client.env | computing
6:59PM INF client.env | completed duration=0s
6:59PM INF actions._pull | computing
6:59PM INF actions._pull | completed duration=0s
6:59PM INF actions._alpine._dag."0"._op | completed duration=3.7s
6:59PM INF actions._alpine._dag."1"._exec | computing
6:59PM INF actions._alpine._dag."1"._exec | completed duration=0s

It's not self-evident that the line
6:59PM INF actions._alpine._dag."0"._op | completed duration=3.7s

refers to an image pull.
Why is this important to you?
Improves the logging UX for users new to dagger.
How are you currently working around this?
Diving into pull.cue to figure out what _op is 🙂

Nice catch, I think it's a good idea to rename it.
Do you want to take care of it?


Incorrect formatting in API docs

Refer the below formatting issue in [1]

[1] https://ballerina.io/learn/api-docs/ballerina/#/ballerina/auth/1.1.0-alpha4/auth/classes/ListenerLdapUserStoreBasicAuthProvider

This issue is caused by: https://github.com/ballerina-platform/ballerina-lang/issues/28916

Closing as the above issue is fixed.


UI Optimization for  SSP

Issue/Feature Description:
Sure. Will look into it at a later date. This will require updating the menu creation and population which is out of scope of the current PR. Could you raise an issue for this. we could track it there.
https://github.com/sodafoundation/dashboard/pull/600/files/0235ad537a543acc210f2678efc3483be5d4fa71
Why this issue to fixed / feature is needed(give scenarios or use cases):
How to reproduce, in case of a bug:
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

This is not an optimization for SSP feature. It is an optimization of the menu system of the dashboard.


Add check if radio is on before starting the advertising and listening

This change is

PR is added to the queue for testing as 1. task. (b3b02b3)

Test 103941844 (b3b02b3) build started.

Test (Success) 103941844 build is completed (b3b02b3)
See https://github.com/ThaliTester/TestResults/tree/103941844b3b02b3_Add_check_if_radio_is_on_before_starting_the_advertising_and_listening_evabishchevich/ for the logs

Test 103941844b3b02b3(b3b02b3) has failed
See https://github.com/ThaliTester/TestResults/tree/103941844b3b02b3_Add_check_if_radio_is_on_before_starting_the_advertising_and_listening_evabishchevich/ for the fail logs

Connected with #1743

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


[stable/mongodb]: configure update strategy for deployments

Is this a new chart
No
What this PR does / why we need it:
The PR makes it possible to configure the updateStrategy for a standalone deployment.
This is required so that the Recreate updateStrategy can be specified while upgrading a standalone deployment
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/retest

/lgtm


Unable to use

i'm getting below error
Target of URI doesn't exist: 'package:filesystem_picker/filesystem_picker.dart'.
Try creating the file referenced by the URI, or Try using a URI for a file that does exist
used
filesystem_picker: 2.0.0-nullsafety.0
in pubspec.yaml

@selvam920 Please provide the output of the flutter doctor -v.


SignerListSet and SetRegularKey

Title
This PR aims to add the Validate and Flatten function to SetRegularKey and SignerListSet.
The tests have 100% coverage.
The PR also adds comments for the SignerList entry and the relevant Flatten functions.

@GuillemGarciaDev , gentle reminder about this PR as well


Fail when invoking "az aks get-credentials"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks get-credentials
Errors:
'clusters'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 711, in _run_job
    six.reraise(*sys.exc_info())
...
cli/command_modules/acs/custom.py, ln 1184, in merge_kubernetes_configurations
    _handle_merge(existing, addition, 'clusters', replace)
cli/command_modules/acs/custom.py, ln 1120, in _handle_merge
    if existing[key] is None:
KeyError: 'clusters'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks get-credentials --name {} --resource-group {}

Expected Behavior
Environment Summary
Linux-5.3.0-53-generic-x86_64-with-debian-buster-sid
Python 3.6.5
Installer: DEB

azure-cli 2.6.0

Additional Context

routing to appropriate team

I couldn't repro in 2.6.0
@wenwu449 could you take a look

@sshiba Apologies for the late reply. You can check under the user profile for file ~/.kube/config. There are 3 required keys which must be present: clusters, contexts and users. May be due to an empty section under clusters you might be facing this issue. You can also rename or delete the file and az aks get-credentials should start working again.
Alternatively you can pass the --file or --overwrite-existing parameter and run the command again. More Info here.
This issue is open for quite sometime now. Please let us know if you still need assistance on this. Awaiting your reply.

No, it can be closed. Thanks!

@sshiba  Thanks for getting back. We will now proceed with closure of this github issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


gpodder: 3.10.1 -> 3.10.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/gpodder/versions.
These checks were done:

built on NixOS
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/gpo had a zero exit code or showed the expected version
/nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/gpodder passed the binary check.
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/gpodder-migrate2tres had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/..gpo-wrapped-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/.gpo-wrapped had a zero exit code or showed the expected version
/nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/..gpodder-wrapped-wrapped passed the binary check.
/nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/.gpodder-wrapped passed the binary check.
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/..gpodder-migrate2tres-wrapped-wrapped had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2/bin/.gpodder-migrate2tres-wrapped had a zero exit code or showed the expected version
3 of 9 passed binary check by having a zero exit code.
0 of 9 passed binary check by having the new version present in output.
found 3.10.2 with grep in /nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2
directory tree listing: https://gist.github.com/cca3f358e73f8f49b3d0dcd3e5549728
du listing: https://gist.github.com/54e8debaadfab87d705cbf72656b4f35

meta.description for gpodder is: '"A podcatcher written in python"'.
cc @svenkeidel @mic92 for testing.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 520682ccf6ab29d5849b6ece9a3474f6312d531c && nix-shell --pure -I nixpkgs=. -p gpodder

Success on x86_64-linux (full log)
Attempted: gpodder
Partial log (click to expand)

OK
Name                    Stmts   Miss  Cover
-------------------------------------------
gpodder/jsonconfig.py      93     48    48%
gpodder/model.py          689    640     7%
gpodder/util.py           832    652    22%
-------------------------------------------
TOTAL                    1614   1340    17%
/nix/store/d43pzkl3230xay7l9wh65whsmsf9lwxa-gpodder-3.10.2

Success on aarch64-linux (full log)
Attempted: gpodder
Partial log (click to expand)

OK
Name                    Stmts   Miss  Cover
-------------------------------------------
gpodder/jsonconfig.py      93     48    48%
gpodder/model.py          689    640     7%
gpodder/util.py           832    652    22%
-------------------------------------------
TOTAL                    1614   1340    17%
/nix/store/hdjvssq2288hrrvbghlsw42bml71kv3v-gpodder-3.10.2

Success on x86_64-darwin (full log)
Attempted: gpodder
Partial log (click to expand)

OK
Name                    Stmts   Miss  Cover
-------------------------------------------
gpodder/jsonconfig.py      93     48    48%
gpodder/model.py          689    640     7%
gpodder/util.py           832    652    22%
-------------------------------------------
TOTAL                    1614   1340    17%
/nix/store/741phx75gx2mq7pw7ifrwh181r0bgdgl-gpodder-3.10.2


Add the esutil.BulkIndexer component

This pull request adds a parallel, asynchronous and efficient bulk helper.
The API is defined by the esutil.BulkIndexer interface:
https://github.com/elastic/go-elasticsearch/blob/d065e908b3c79d6fecb8387865c9b9a7c4f30e0b/esutil/bulk_indexer.go#L26-L41
The caller uses the Add() method to add a esutil.BulkIndexerItem, which is passed on to the workers, which encode the operation into the Bulk API format, and send it to Elasticsearch, when the size of the buffer passes the configured threshold. The indexer also performs a periodic flush, based on the configured interval.
The indexer accepts a configuration in the esutil.BulkIndexerConfig struct.
A simple, but realistic, example of using the indexer:
es, _ := elasticsearch.NewClient(elasticsearch.Config{
	RetryOnStatus: []int{502, 503, 504, 429},
	RetryBackoff: func(i int) time.Duration { return time.Duration(i) * 100 * time.Millisecond },
	MaxRetries: 5,
})

indexer, _ := esutil.NewBulkIndexer(esutil.BulkIndexerConfig{
	Client:        es,              // The Elasticsearch client
	Index:         "test",          // The default index name
	NumWorkers:    4,               // The number of worker goroutines (default: number of CPUs)
	FlushBytes:    2e+6,            // The flush threshold in bytes (default: 5MB)
	FlushInterval: 30 * time.Second // The flush interval (default: 10sec)
})

indexer.Add(
	context.Background(),
	esutil.BulkIndexerItem{
		Action: "index",
		Body: strings.NewReader(`{"title":"Test"}`),
	})

indexer.Close(context.Background())

The indexer provides OnSuccess() and OnFailure() callbacks for each item, to propagate the individual operation results to the caller. It also provides the Stats() method, returning the esutil.BulkIndexerStats struct for debugging and monitoring.
A commented, executable example is available in _examples/bulk/indexer.go:
$ go run indexer.go -count=100000 -flush=1000000

# BulkIndexer: documents [100,000] workers [8] flush [1.0 MB]
# ▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁▁
# → Generated 100,000 articles
# ▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔
# Sucessfuly indexed [100,000] documents in 1.909s (52,383 docs/sec)

A great deal of care has been taken to make the indexer as efficient as possible. A benchmark test is available in esutil/bulk_indexer_benchmark_test.go:
$ go test -v -run=none -bench=BenchmarkBulkIndexer -race -benchmem -benchtime=10s .
BenchmarkBulkIndexer/Basic-4  1376808  8735 ns/op  178 B/op  3 allocs/op

The performance has been verified by end-to-end benchmarks against a real cluster, with both client and server running on separate Google Cloud Platform instances.
For a small document, the throughput is around 250,000 documents per second:
ELASTICSEARCH_URL=http://server:9200 go run benchmarks.go --dataset=small --count=1_000_000 --flush=2MB --shards=5 --replicas=0 --fasthttp=true --easyjson=true
small: run [10x] warmup [3x] shards [5] replicas [0] workers [8] flush [2.0 MB] wait [1s] fasthttp easyjson
▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔▔
docs/sec: min [279,173] max [289,351] mean [286,987]

For a document representing the Nginx log event captured by Filebeat, the throughput is around 50,000 documents per second:
ELASTICSEARCH_URL=http://server:9200 go run benchmarks.go --dataset=httplog --count=1_000_000 --flush=3MB --shards=5 --replicas=0 --fasthttp=true --easyjson=true
httplog: run [10x] warmup [3x] shards [5] replicas [0] workers [8] flush [3.0 MB] wait [1s] fasthttp easyjson
docs/sec: min [50,165] max [53,072] mean [52,011]

The "theoretical" performance of the indexer has been verified by running against an Nginx server, with a throughput surpassing 1,000,000 documents per second:
ELASTICSEARCH_URL=http://server:8000 go run benchmarks.go --dataset=small --count=1_000_000 --flush=2MB --warmup=0 --mockserver
small: run [10x] warmup [0x] shards [3] replicas [0] workers [8] flush [2.0 MB] wait [1s]
docs/sec: min [1,187,648] max [1,253,132] mean [1,233,818]

Please see _examples/bulk/benchmarks for more detailed information about the performance, the testing environment used, and run the script to see percentile statistics, omitted from the output here.
Note that the benchmarks use the valyala/fasthttp package for improving the performance of HTTP transport, and a custom JSON decoder provided by the mailru/easyjson package.

/cc @elastic/es-clients @axw @urso @simitt @tonymeehan

Commit history cleaned up and merged as 56b3816 and aae4e81.


Enabling query contains to ignore case

Just adding an option to ignore case when you are going Query.Contains =)

Hi @jkvargas, thanks! I will merge soon.


Operator projects using the removed APIs in k8s 1.22 requires changes.

Problem Description
Kubernetes has been deprecating API(s), which will be removed and are no longer available in 1.22. Operators projects using these APIs versions will not work on Kubernetes 1.22 or any cluster vendor using this Kubernetes version(1.22), such as OpenShift 4.9+. Following the APIs that are most likely your projects to be affected by:

apiextensions.k8s.io/v1beta1: (Used for CRDs and available since v1.16)
rbac.authorization.k8s.io/v1beta1: (Used for RBAC/rules and available since v1.8)
admissionregistration.k8s.io/v1beta1 (Used for Webhooks and available since v1.16)

Therefore, looks like this project distributes solutions in the repository and does not contain any version compatible with k8s 1.22/OCP 4.9. (More info). Following some findings by checking the distributions published:

group-sync-operator.v0.0.5: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.7: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"]),ClusterRole: (["group-sync-operator-metrics-reader"]),
group-sync-operator.v0.0.8: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"]),ClusterRole: (["group-sync-operator-metrics-reader"]),
group-sync-operator.v0.0.9: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.2: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.10: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.11: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"]),ClusterRole: (["group-sync-operator-metrics-reader"]),
group-sync-operator.v0.0.12: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.3: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.6: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])
group-sync-operator.v0.0.1: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["groupsyncs.redhatcop.redhat.io"])

NOTE: The above findings are only about the manifests shipped inside of the distribution. It is not checking the codebase.
How to solve
It would be very nice to see new distributions of this project that are no longer using these APIs and so they can work on Kubernetes 1.22 and newer and published in the community-operators collection. OpenShift 4.9, for example, will not ship operators anymore that do still use v1beta1 extension APIs.
Due to the number of options available to build Operators, it is hard to provide direct guidance on updating your operator to support Kubernetes 1.22. Recent versions of the OperatorSDK greater than 1.0.0 and Kubebuilder greater than 3.0.0 scaffold your project with the latest versions of these APIs (all that is generated by tools only). See the guides to upgrade your projects with OperatorSDK Golang, Ansible, Helm or the Kubebuilder one. For APIs other than the ones mentioned above, you will have to check your code for usage of removed API versions and upgrade to newer APIs. The details of this depend on your codebase.
If this projects only need to migrate the API for CRDs and it was built with OperatorSDK versions lower than 1.0.0 then, you maybe able to solve it with an OperatorSDK version >= v0.18.x < 1.0.0:

$ operator-sdk generate crds --crd-version=v1
INFO[0000] Running CRD generator.
INFO[0000] CRD generation complete.

Alternatively, you can try to upgrade your manifests with controller-gen (version >= v0.4.1) :
If this project does not use Webhooks:

$ controller-gen crd:trivialVersions=true,preserveUnknownFields=false rbac:roleName=manager-role  paths="./..."

If this project is using Webhooks:

Add the markers sideEffects and admissionReviewVersions to your webhook (Example with sideEffects=None and admissionReviewVersions={v1,v1beta1}: memcached-operator/api/v1alpha1/memcached_webhook.go):

Run the command:

$ controller-gen crd:trivialVersions=true,preserveUnknownFields=false rbac:roleName=manager-role webhook paths="./..."

For further information and tips see the comment.

@camilamacedo86 v0.0.13 has been released and no longer uses the deprecated API versions

Hi @sabre1041,
Did you release it here (https://github.com/redhat-openshift-ecosystem/community-operators-prod/blob/main/operators/group-sync-operator)? I cannot check the v0.0.13 version there.


projects:ad469x_iio: Update no-OS Drivers

Firmware updates to sync up with no-OS drivers

@jansunil Can we add relevant high-level changes (sw-double buffer implementations, unipolar mode as default, etc) in the commit message as the changes are not just the no-os updates?

@jansunil, Can we fix the no-os lib checkout build fail?


Max cut/QAOA applied to Unsupervised learning

Summary
The Max-cut_vs_k_means.ipynb notebook implements k-means and the max-cut problem solved by QAOA to cluster data. The max-cut problem solved by QAOA is effectively a binary classifier. The Max-cut_2+_divisive_clustering.ipynb notebook is a sequel tutorial where 2+ clustering groups are formed by applying the same max-cut problem solved by QAOA approach but in a divisive hierarchical scheme.
Details and comments
These are the first tutorials I have contributed, and would appreciate feedback so they become as useful as possible. I did see the max-cut tutorial under optimization. However, this has an AI focus--particularly in the second notebook

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

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


Proof read my wiki page?

Could someone cast their eyes over the wiki page I have created? I don't want to give the wrong instructions!
https://github.com/github/hubot/wiki/Hubot-on-Heroku

It's good, added an example of extracting the Hubot tar.gz and, committing the files into git after adding them. Updated single quotes into back ticks for the command line examples.
Same process would also work on Linux/BSD systems, so probably worth giving that a mention.

Ahh thanks!
Would the steps need to alter at all for Linux/BSD systems? Heroku gem works the same etc.?


linux version number in controlfile must be part of automation

a dev or rel tag must set the linux version provided in the control file properly.
preferred would be hat the deb build get's the version number a argument.
Possible, but not nice would be to manipulate the control file before build.

Hi Thomas,
As I known that we've already updated the linux version number in controlfile in ticket https://github.com/test-fullautomation/RobotFramework_AIO/issues/118.
However, it only updated on local machine and it can't update into the source code lead to the Gitlab repos can't receive the exact version for installing tool.
Solution

To resolve this issue, I will create the necessary steps for automating commit the changes on local machine and generating a merge request automatically, allowing you to merge it

How do you think about it?
Thank you,
Thong Hua

Hi @huavanthong ,
actually I'm confused now.
I checked both, latest dev/<IP_ADDRESS> build with Gitlab and Githab. Both have correct version information for Linux and Windows.
@HolQue : You had a finding that the built files don't have correct version information. Where did you find this. Already merged #146 should solve this problem. für dev/<IP_ADDRESS> I could not find consistencies.
Thank you,
Thomas

Hi @test-fullautomation,
Installer RobotFramework_AIO_setup_merged_develop.exe taken from:
https://gitlab-apertispro.boschdevcloud.com/robotframework-aio/main/robotframework_aio/-/jobs/2808578/artifacts/download

Hi @HolQue ,
O.K. then I understand the problem. Here I didn't provide a release tag.
Solution could be that I use from now onwards always the dev  tag, when we start testing release candidates.
@huavanthong : please check also automating commiting an ujpdated control file. This will not solve Holgers problem, but it creates more consistency in our repositories.
Thank you,
Thomas

Hi all,
The Holger's observation is correct!
Currently, the version information is only taken from dev and rev in case the pipeline is triggered by dev or rel.
Otherwise, there is no information about the version (except control file, it is also not up-to-date), so I used the trigger event (merge_develop or dev branch) as version information for Windows installer.
So that, for the daily build and merged PR build will not have the proper version info that we expected (such as 0.9.0)
I wonder that do we need to maintain version information (in files such as control file) so that all the stuff with related to version will refer to that file?
Thank you,
Ngoan

@ngoan1608 : from my point nothing to do. I will take care that I create releases based on dev tags when I request testing.
Hm... but there is one disadvantage: the daily builds will be inconsistent...

Hi Thomas,
I think the job to commit an updated version in control file (when building with tdev/rel tag) will help to maintain the version information.
But, I concern that it is not a good idea to commit and push change directly yo develop branch for updated control file.
Besides, the commit for update version of control will be ahead of dev/rel tag if we do it in build pipeline.
Thank you,
Ngoan

Hi Thomas,
Cc Ngoan
I would like to make clear this issue on GitLab.

Currently, we can build correctly Tool version on Gitlab by define tag = <IP_ADDRESS>, however, because the control file specific the version for Tool is 0.8.x.x.
==> On OSD platform will need downgrade from version <IP_ADDRESS> to 0.8.x.x
On Gitlab source code, it will get source code from RobotFramework Github, and because we don't update control file manually lead to it can't know the version info correctly.

As your suggestion, we need to automate this behavior, so my solution will create a commit on the changes of version in control file, and create a merge request automatically => Then you only merge it for correcting the version info.
For an example of workflow:

Trigger tag job -> Build AIO job -> Test AIO job ->  Automate the control file update job.

Operation of Automate the control file update job:

Create a new branch based on tag info -> Update control file -> Commit -> Push on the new branch -> Create a pull request.

Conclusion
With this solution, it will allow you merge it to correct info on control file after trigger tag job.
If you have any concerns,  please let me know.
Thank you,
Thong Hua

Hi Thong,
"Create a new branch based on tag info -> Update control file -> Commit -> Push on the new branch -> Create a pull request."
is a good idea.
We try it this way. Please provide a pull-request for this.
Thank you,
Thomas

Hi Thomas,
This issue was resolved. Please help me close this ticket.
Thank you,
Thong Hua

solution is part of version 0.10.0


please solve this issue

[Enter feedback here]

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

ID: c655c87d-4372-6fb7-3bd7-4ba2531415b6
Version Independent ID: c655c87d-4372-6fb7-3bd7-4ba2531415b6
Content: MSB3021: Unable to copy file 'source' to 'destination'. - MSBuild
Content Source: docs/msbuild/errors/msb3021.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@Ramanand-maurya -- thank you for your request.  This repo is solely for documentation issues, and this is product-related.  Please add it here:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Use the new Suggest a feature for Visual Studio IDE menu item.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


does not work with latex

equations in latex are almost unreadable since the bg color is changed in a weird way.
e.g.
\documentclass[a4paper]{article}
\usepackage{amsmath}
\begin{document}
\begin{align*}
  S_1 > S > S_2 \\
  \Rightarrow S_1 & = \mathrm{e}^{}{x}  \\
  \Rightarrow S_1 & = \mathrm{e}^{}{y} \\
  \Rightarrow S_1 & = \mathrm{e^{}{x} \\
             & = \boxed{\mathrm{e}^{}{z}}
\end{align*}
\end{document}

Took a while to reproduce, you probably have conceallevel set to 2 or something, which folds and converts LaTeX symbols on the fly.
Changed the VIM highlighting for Concel globally, should work for you now.
Let me know if you find other shortcomings for LaTeX, it's been a while since I used it extensively.

thanks for looking into this. in fact my conceallevel is 0. i'm on macvim, with vim 8.0 . i'm using the lervag/vimtex package for editing latex, but i did not think that should matter.
i updated the plugin and now the weird bg did disappear. i still see some unexpected bolding of parentheses and letters. e.g. this example:
\documentclass[a4paper]{article}
\usepackage{amsmath}
\begin{document}

\begin{align*}
  S_1 > S > S_2 \\
  \Rightarrow S_1 & = \mathrm{e}^{}{x}  \\
  \Rightarrow S_1 & = \mathrm{e}^{}{y} \\
	\Rightarrow S_1 & = \mathrm{e}^{}{x} \\
		(1,2) \left[ sdfa \right]
             & = \boxed{\mathrm{e}^{}{z}}
		\rec_i
	\xrightleftharpoons[k_u]{\,k_b\,}
	\rec_i + \lig & \xrightleftharpoons[k_u]{\,k_{b,i}\,}
\end{align*}

the equilibrium fraction $f=C$ is

\begin{equation}
	\label{eq:KD3d}
	K_{D,1}^\text{3d} \gg K_{D,2}^\text{3d}.
\end{equation}
Therefore, in the corresponding reaction scheme
\begin{align}
	\rec_i + \lig & \xrightleftharpoons[k_u]{\,k_{b,i}\,}
	\rec_i\lig \quad
	\text{for} \ i \in \{1,2\}  \label{eq:ligandbinding}\\
	\rec_i + \rec_j\lig &\xrightleftharpoons[k_{d,i}]{\,k_{a,i}\,} \comp \quad
	\text{for}\ (i,j) \in \{(1,2), (2,1)\},
	\label{eq:recruit}
\end{align}

\end{document}

(it won't compile some commands are not defined).
i get inconsistent bolding of [] after the xrightleftharpoons for instance...

actually i just noticed, the inconsistent bolding goes away when i change the two math environments to be the same. (align* to equation in the first). is this something that the vimtex plugin does wrong? or the regular vim latex filetype?

I viewed your code with VIM 8.0 without the vimtex package and only when I set conceallevel to 2 it collapses the math output and shows the symbols instead, but I guess we have the same output by different means then.
I matched the highlighting for the two different math environments and pushed the changes, is that what you meant?

hi, i never got any concealing in my screen output before. the only thing i got was a weird background color at various places inside the math environment. the subscript _ markers but also some of the whitespace if i recall correctly.
anyway with your earlier update that went away. what remained was inconsistent bold-symbols. in your latest output i don't see those anymore, so this seems to be working now.
it seems weird to me that a colorscheme would have to know about the names of latex environments. should that not be handled within the filetype plugin?

Okay, so I misunderstood that, but as the weird background color is gone for you, that should be fine i guess.
Usually the ft-plugin has sensible defaults, so little to no adaptions are necessary to make it look good with a certain colorscheme. In reality some need a little bit more care. See for example the quite popular base16 colorscheme (Github variant), which has around 15 specific highlights for Git, but only four for HTML.

yes, thanks, the weird bg color is fixed! interesting that the colorschemes have to look into the individual filetypes. i think it's perfectly understandable but definitely not ideal...

Yeah certainly not ideal but also it often comes down to personal taste of the colorscheme author.
The author may be of the opinion that a certain orange color should be the primary color of her colorscheme and when adapting say for Ruby, she thinks that Ruby symbols should be painted in that certain orange color because they are an important language feature and so she changes that, to make it feel "right" for her.
After all its some kind of styling and design, which not always has to be perfectly reasonable …

gotcha. 'de gustibus...'


Delete BAM upload SOP

Description of the task:
Delete the BAM upload SOP as it got deprecated.

Acceptance criteria for the task:

[ ] Review SOP to ensure no useful information will be lost
[ ] Delete SOP

Reviewed doc, no important info will be lost. I am going to delete, push to master and move this ticket to Done


CMake: Use try_compile in place of execute_process

The previous approach was leaving compilation artefacts in the source directory.

Better!


Updated name of AccessToken variable

Azure DevOps uses SYSTEM_ACCESSTOKEN for the Access Token and not the value that was in the documentation. Fixing documentation to avoid having to find that out the hard way.

Thanks @krmeda !


Added pull request to merge branches

Changed readme.

Audit4j » audit4j-db #53 FAILURE
Looks like there's a problem with this pull request
(what's this?)


CatchException middleware

@cleemullins @tomlm Has the CatchException middleware component been removed from the SDK?  If so, do you have any objection to us adopting it into the Community Project?
Also planning to do the same for the AzureTableStorage as well.

It was removed, but there is now an OnTurnError property that you can specify in the BotFrameworkOptions object, and it gets set on the adapter.

@garypretty the advantage of the OnTurnError property is that it catches the exception wherever it occurs in the turn (whether in the application or the middleware).
The problem we found with using a middleware for this is that where you place it in the stack can alter whether you catch the exception and that can get confusing.
This doesn't preclude you from catching or throwing exceptions in middleware - if that's how you want your code to work - the OnTurnError is the last change catch all and comes after everything else.

Thanks for the explanation of the thinking for this change. It makes sense. We might adopt the middleware approach in the community repo to show / allow people to catch particular exceptions etc. but I'll ensure we explicitly call out the OnTurnError property as part of the picture. Thanks.


reconnect with an exponential backoff

closes #36

@Ciremun looks goode to me! :+1: Thank you for your contribution!


Something went wrong installing the "sharp" module

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?

System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz
    Memory: 11.51 GB / 31.93 GB
  Binaries:
    Node: 16.13.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.17.3 - C:\Users\kotori\AppData\Roaming\npm\yarn.CMD
    npm: 8.1.0 - C:\Program Files\nodejs\npm.CMD

What are the steps to reproduce?

I build an electron app,
When I run it on a 64-bit win7 system, it prompts the following

But it works fine on another win7 64 bit computer.
They use the same installation package
The normal operation is win7 Enterprise，error is Win7 Professional
What is the expected behaviour?

normal operation
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem

Please provide sample image(s) that help explain this problem

There should be a problem with the dll on the problem computer


Add unit tests for Exchange (or Protocol) header encode/decode

Problem
No unit tests exist within WML layer baseline
Proposed Solution
add said unit tests

Not valid since wml solution is not selected


v1.15 Backports 2024-07-15

[ ] #33283 (@bimmlerd) :warning: resolved conflicts

:warning: Fixed minor conflicts in allocateHealthIPs, due to different indentation, and extended the same changes to the IPv6 branch (coalescenceCIDR got removed there by https://github.com/cilium/cilium/commit/023818c9dade7a1e99852b617efc3faee0f7e159 as IPv6 is not supported in ENI mode).

[ ] #33373 (@aditighag) :warning: resolved conflicts

:information_source: Skipped 69e5edee4aa8, as codeowners changes are not propagated to stable branches.

[ ] #33449 (@jrajahalme) :warning: resolved conflicts

:warning: Hit minor conflict in pkg/policy/mapstate.go due to different surrounding context, resolved accepting combination. Additionally hit multiple conflicts concerning the removal of the testing.T parameter from the Diff method in test files. Resolved accepting the current changes, and manually removing the parameter.

[x] #33621 (@brb)
[x] #33626 (@giorio94)
[x] #33680 (@joestringer)
[ ] #33655 (@aditighag) :warning: resolved conflicts

:information_source: Hit minor conflict due to different surrounding context; resolved accepting combination.

[ ] #33634 (@chaunceyjiang)
[x] #33683 (@sayboras) :warning: resolved conflicts

:information_source: Hit minor conflicts, resolved adapting the upstream changes to the local structure.

[x] #33592 (@gandro) :warning: resolved conflicts

:information_source: Hit minor conflict in pkg/defaults/defaults.go due to different surrounding context; resolved accepting combination. Additionally dropped the values.schema.json hunk, as not applicable in v1.15.

[x] #33708 (@Mais316)
[x] #33679 (@giorio94) :warning: resolved conflicts

:information_source: Hit conflict as v1.15 still used an old etcd image for integration tests; accepted the incoming changes, as we want to keep the etcd version up-to-date.

[x] #33725 (@brb)
[ ] #33616 (@mrproliu)
[x] #33769 (@pchaigno)
[x] #33735 (@giorio94) :warning: resolved conflicts

:information_source: Hit minor conflict due to different surrounding context; resolved accepting combination.

[x] #33306 (@skmatti) :warning: resolved conflicts

:warning: Adapted the TestUpdateHostEndpointLabels test due to the different structure in v1.15.

Once this PR is merged, a GitHub action will update the labels of these PRs:
 33283 33373 33449 33621 33626 33680 33655 33634 33683 33592 33708 33679 33725 33616 33769 33735 33306

/test-backport-1.15

@cilium/tophat here, I'll hold off on merging until we get more acknowledgements for the conflicting PRs. (I'm a bit surprised how many conflicts there are :thinking: )

@cilium/tophat here, I'll hold off on merging until we get more acknowledgements for the conflicting PRs. (I'm a bit surprised how many conflicts there are 🤔 )

Indeed, the number of conflicts seemed higher than usual to me as well. OTOH, most of them were pretty trivial to solve (e.g., caused by slight differences in the surrounding context).

@aditighag @jrajahalme Gentle ping :pray:


Enable/Disable vsync as a flag to init

Let the client determine whether or not they want vsync, via a flag. See rltk.cpp line 34.

I've added a function to toggle framerate to rltk.cpp, maybe also useful for other people?
http://www.sfml-dev.org/documentation/2.0/classsf_1_1Window.php#af4322d315baf93405bf0d5087ad5e784


Make the default stack.yaml be for GHC 8.2.2

BTW, technically this PR is intended to just be a renaming of files, to bring a different default into scope for stack.yaml.
So perhaps we should keep it as that, and then clean it up.

So perhaps we should keep it as that, and then clean it up.

I agree


 Retrying in 180 seconds.
2021-04-08 17:23:02 WARNING (MainThread) [pymyq.api] Unknown device family camera
2021-04-08 17:23:04 DEBUG (MainThread) [custom_components.mail_and_packages.camera] Camera Update: usps_camera
2021-04-08 17:23:04 WARNING (MainThread) [custom_components.mail_and_packages.camera] Could not read camera Mail USPS Camera image from file: /config/custom_components/mail_and_packages/images/cb6a88ca-180c-4e90-a5f6-a384ce195671.gif
2021-04-08 17:23:04 DEBUG (MainThread) [custom_components.mail_and_packages.camera] Camera Update: amazon_camera

Additional context
As far as I can remember, I did not change anything. However there is a chance I enabled and disabled DHL around the time this started happening.

Remove the<EMAIL_ADDRESS>from your "Amazon Forwards" part of your configuration.

Unfortunately that is already set:

Should be fixed in 0.3.2-b10 once merged.

Worked a treat, confirmed resolved.
Thank you!


Windows App (Microsoft Remote Desktop)

Before Submitting Your Bug Report

[X] I have verified that there isn't already an issue reporting the same bug to prevent duplication.
[X] I have seen the FAQ.

Maccy Version (see 'About' window)
2.2.1
macOS Version
15.1.1
Maccy Settings
defaults read org.p0deje.Maccy
{
    "KeyboardShortcuts_delete" = "{\\"carbonKeyCode\\":51,\\"carbonModifiers\\":2048}";
    "KeyboardShortcuts_pin" = "{\\"carbonKeyCode\\":35,\\"carbonModifiers\\":2048}";
    "KeyboardShortcuts_popup" = "{\\"carbonKeyCode\\":46,\\"carbonModifiers\\":6400}";
    "LaunchAtLogin__hasMigrated" = 1;
    "NSStatusItem Preferred Position Item-0" = 475;
    "NSStatusItem Visible Item-1" = 0;
    "NSWindow Frame SUUpdateAlert" = "650 505 620 398 0 0 1920 1055 ";
    "NSWindow Frame com.sindresorhus.Settings.FrameAutosaveName" = "705 401 510 473 0 0 1920 1055 ";
    SUAutomaticallyUpdate = 0;
    SUEnableAutomaticChecks = 0;
    SUHasLaunchedBefore = 1;
    SULastCheckTime = "2024-11-27 12:33:36 +0000";
    SUSendProfileInfo = 0;
    avoidTakingFocus = 1;
    clearOnQuit = 0;
    enabledPasteboardTypes =     (
        "public.rtf",
        "public.utf8-plain-text",
        "public.html"
    );
    highlightMatch = color;
    historySize = 50;
    ignoreEvents = 0;
    ignoredPasteboardTypes =     (
    );
    maxMenuItemLength = 50;
    maxMenuItems = 9;
    migrations =     {
        "2024-07-01-version-2" = 1;
    };
    pasteByDefault = 1;
    pinTo = bottom;
    popupPosition = statusItem;
    removeFormattingByDefault = 1;
    searchMode = exact;
    showFooter = 0;
    showSearch = 0;
    showTitle = 0;
    sortBy = lastCopiedAt;
    windowPosition = "[0.8348958333333333,1]";
    windowSize = "[386,254]";
}
timoteistefanescu@Timoteis-MacBook-Air ~ %

Description
A lot of the times, when I try to paste something from Maccy into the Windows App (Microsoft Remote Desktop), Maccy stops working, showing not responding and I have to force close it and restart it to make it work again.
Also, almost every time I have to click a second time to paste the correct item, it initially inserts the last copied item, not the one I selected from the list.
Steps to Reproduce
No response

@Timotei-S Can you please record a small video showing the issue? I am using RDP myself and don't have such issues.

Attachment available until 27 Dec 2024
I have updated the app today and can't seem to replicate the freeze bug right now, but here you can see the other issue, with pasteing the wrong item. And not saving into memory the copied item.

On 27 Nov 2024, at 16:07, Alex Rodionov @.***> wrote:
@Timotei-S https://github.com/Timotei-S Can you please record a small video showing the issue? I am using RDP myself and don't have such issues.
—
Reply to this email directly, view it on GitHub https://github.com/p0deje/Maccy/issues/970#issuecomment-2503969457, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWNJ3RJWEAZPHWWNG7ABBW32CXGYVAVCNFSM6AAAAABSSX7IGGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBTHE3DSNBVG4.
You are receiving this because you were mentioned.

Click to Download
https://www.icloud.com/attachment/?u=https%3A%2F%2Fcvws.icloud-content.com%2FB%2FAaxX4V8atdM-lZIz8QQUwvvZRrf4AXibTzKHdiadATkZhS2Y3J5aLNhI%2F%24{f}%3Fo%3DAkySLRUZ5ix1DAouspE7L-RXgnavxHjD1DYb32n2YKBw%26v%3D1%26x%3D3%26a%3DCAoggMa2EQgtZcIOaVFXk6p-PHu2QNmT1jtMyatC4zOLucwSehCCtY7xtjIYgsWJxcAyIgEAKgkC6AMA_2KecC1SBNlGt_haBFos2EhqJy7hO-2l-nsE4SvVGk01g0ingv5f_jggKOorm8GGQqhcKMSrn-dDq3InySiCyUtZfblGw4z2EUgPH-n2dy-yE8S2qAqv2UJfkAnJ022O3lED%26e%3D1735311647%26fl%3D%26r%3D5C849291-EF14-44E1-93A1-3BE88AC6CB58-1%26k%3D%24{uk}%26ckc%3Dcom.apple.largeattachment%26ckz%3D33817D1B-3E40-4B9C-933B-F2248B995A27%26p%3D49%26s%3DgxwcguH6uTURvRsivJanLq5S2H0&uk=MWAhl0rng_CxOnvxyH1s1Q&f=Screen Recording 2024-11-27 at 16.49.45.mov&sz=60509071Screen Recording 2024-11-27 at 16.49.45.mov
60,5 MB

Did you enable bidirectional clipboard on your RDP connection?

I have issues with pasting (though they are fairly known and backtrack to old versions of RDP), but I cannot reproduce the copy issue:
https://github.com/user-attachments/assets/5f3eb296-c6a8-4c72-8c6d-ed58bf460a89

How do I enable "Scancode" keyboard mode?
*Also, I use Karabiner to switch cmd and ctrl. Could that add additional problems?

https://github.com/user-attachments/assets/aedddaad-f078-49e9-8c56-97632e2dacfb
Here you can also see how if I open the app it saved the copied item into memory, but if I don't, it does not.

https://github.com/user-attachments/assets/600c6707-d8d1-456f-ba2a-f0c0eda2c55c
Here is the app freeze in action. I just tried to paste something into the Windows App.

*Also, I use Karabiner to switch cmd and ctrl. Could that add additional problems?

It could be related. Can you disable Karabiner and re-test?

Sorry, can load any of your videos (1 doesn't load at all while 2 has just 4 seconds). Can you re-upload please?

I think it has something to do with the Windows App. When I restart the app, some of the problems seem to disappear.
I tested working without Karabiner opened and it does not seem to make any difference.

https://drive.google.com/file/d/1QnnOin38Ylqwa07JV4VbOXumDu50h8Ub/view?usp=share_link

Ok, that hanging looks weird, but I cannot reproduce it with the Windows App myself.

Does it happen consistently when you copy something from the Windows guest?
Can you try disabling Maccy with Option+Click on the menubar icon and see if it helps?
When Maccy hangs, can you please start the Pasteboard Viewer app and share its screenshot? I wonder if there is anything in the copy itself that is causing this behavior.

No, not consistently, but I'll try o see if a pattern starts to form.
I'll definitely try!
You mean this? > https://apps.apple.com/ro/app/pasteboard-viewer/id1499215709

Yes, exactly

Hi,
I'm experiencing the same issue as Timotei-S with the text copying as shown on the video below.
https://github.com/user-attachments/assets/0e92cde5-61c0-4d2c-bcb3-c6cc0c31f9d6

@br1nqq I think it might be because the Windows App syncs the clipboard when the window focus is lost or periodically. With that said, there is nothing Maccy can do about it.
@Timotei-S Please share the pasteboard contents.


Handle mid-token issue

Fix the so called "mid-token" issue by stripping the last token from the prompt and then sampling the first processed token after the prompt among the ones that start with the last token's string.
We also got a new feature for free. We can now explicitly set last word boundary in the prompt, i.e. signal if the last word of the prompt should be considered a full word or just a partial word.
For instance, in this case:
./run model.bin 0 5 "On"
<s>
One day, a little

Last word of the prompt is expanded On -> One by the model. Instead if we want the sentence to specifically start with the word "On" we can now simply submit:
./run model.bin 0 5 "On "
<s>
On a hot summer day

Tempting to merge this though it does bloat the complexity a bit with a fairly subtle issue sad. I think I will merge this though...
Btw here is a related thing on the topic https://github.com/microsoft/guidance/blob/main/notebooks/token_healing.ipynb
This is called "token healing"

@karpathy thanks for the link
Now that Top-p sampling has been merged a straightforward approach (with minimal number of LOC added) would be
Top-p -> add Top-K option too (I did it in my repo, a very minimal change) -> add token healing (naturally fits in Top-K logic after setting all the other logits to a minimum)

@xefoci7612 do you think we should be able to just clamp the logits at the excluded locations to -inf? Instead of taking it into the sampling code.

@karpathy
This is another attempt according to your suggestion. It is compatible with any kind of sampling, argmax, top-p, etc because tokens clamping is done upstream at logits level.
#248


Start markdown

Comparing the main from the start_markdown

New Tasks

[x] Learn deeply about Markdown syntax.
[ ] Go deep into collaboration using GitHub.
[ ] Adapt to best practices and techniques e.g. pull requests and useful commits.


title : 16-puzzle game

Game logic and basic description
n this puzzle game, you have a grid with numbered tiles and one empty space. The objective is to slide the tiles around the grid to arrange them in a specific order, usually from lowest to highest number, with the empty space in a designated position (often at the bottom-right corner).
8-Puzzle: A 3x3 grid with 8 numbered tiles (1-8) and one empty space.
16-Puzzle: A 4x4 grid with 15 numbered tiles (1-15) and one empty space.
Game points

Starting Points: 1000
Moves: 45 (45 points deducted)
Time: 7 minutes (35 points deducted)
Bonus: Solved under 50 moves (awarded 50 points)

Screeenshot

why should be added

fun to play
everyone love number game
already know the code in DSA(so fun to develop)
interesting project

@swapnilsparsh please assign me this project
i think this project is very interesting add

@Yashgabani845 Thank you for providing your insight but it is not required as of now

@swapnilsparsh hi , sir can you please tell me which kind of issues i should raise that are required in this repo to get assigned
please answer it will follow you feedback

@Yashgabani845 Try to find bugs in the existing projects which are there so you can resolve them or come with some new and unique ideas where some challenging js concepts are used which can be useful for others.


about azkaban one project create many flows

I have one question about azkaban can support one project create many flows ，how to do it>?

Yes. you can have multiple flows in a given project. Let me know if this documentation helps,
https://azkaban.readthedocs.io/en/latest/createFlows.html


fix(debugger): improve output when assertion fails in Brillig

Description
When an assertion fails on Brillig mode, the debugger fails to indicate the user where the program failed

while when running on ACIR mode it provides enough information

Problem*
Resolves
Summary*

assert constraints are translated as "jump to trap" if the condition is not met when running in Brillig mode
Solution: treat FailureReason::Trap specifically when handling brillig vm status. Treat these failures as an OpcodeResolutionError::UnsatisfiedConstrain instead of an OpcodeResolutionError::BrilligFunctionFailed

By doing these, the error will be reported the same way as when running in ACIR mode

Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Closing this PR since these changes make nargo tests to fail. I suspect it has to do with the noir assert_eq function using constraints and I modified its behavior. I'll try to figure out what's going on


Replay in pr-builder pipeline doesn't update GitHub PR status OR restarting a pipeline stage will not update pull request status

Re-opening because the issue https://github.com/jenkinsci/ghprb-plugin/issues/674 still occurs. and the solution did not work. Can someone please provide detailed steps if this is fixed? Thank you!
A Little more background:
We are using Github pull request builder to poll a private GitHub repository and using Jenkins pipeline script to execute a multiple-stage pipeline build. When triggering a build via GitHub, the checks are updated on the pull request correctly but when restarting the pipeline the checks on the PR are not updated.
We need this to work because sometimes tests get re-tried and disabled a second time. and we do not want to re-run the entire pipeline
The ghprb config is not using web hooks but instead polling the repository

After working on this issue for a while and much Google searching I found an alternative which is to manually send status updates to GitHub in the pipeline - using this as a guide https://github.com/KostyaSha/github-integration-plugin/issues/102#issuecomment-299641073 . I have attached a full pipeline example below to help any other poor developers who encounter this issue. I took me all day to figure it out, pipelines are very difficult
// Manages communication with github for reporting important status
// events. Relies on Github integration plugin https://github.com/jenkinsci/github-plugin
class GithubStatus {

    // Repository to send status e.g: '/username/repo'
    String ghprbGhRepository

    // Commit hash to send status e.g: `909b76f97e`
    String ghprbActualCommit

    // Build url send status e.g: `https://jenkins/build/123`
    String BUILD_URL

    // Set pending status for this pull request
    def setPending(script) {
        this.setBuildStatusStep(script, "In Progress", "PENDING")
    }

    // Set failed status for this pull request
    def setFailed(script) {
        this.setBuildStatusStep(script, "Complete", "FAILURE")
    }

    // Set success status for this pull request
    def setSuccess(script) {
        this.setBuildStatusStep(script, "Complete", "SUCCESS")
    }

    // Manually set PR status via GitHub integration plugin. Manual status updates are needed
    // Due to pipeline stage retries not propagating updates back to Github automatically
    // https://github.com/KostyaSha/github-integration-plugin/issues/102#issuecomment-299641073
    def setBuildStatusStep(script, String message, String state) {
        def commitContextName = "Jenkins Build and Test"
        script.step([
            $class: "GitHubCommitStatusSetter",
            reposSource: [$class: "ManuallyEnteredRepositorySource", url: "https://github.com/${this.ghprbGhRepository}"],
            contextSource: [$class: "ManuallyEnteredCommitContextSource", context: commitContextName],
            errorHandlers: [[$class: "ChangingBuildStatusErrorHandler", result: "UNSTABLE"]],
            commitShaSource: [$class: "ManuallyEnteredShaSource", sha: this.ghprbActualCommit],
            statusBackrefSource: [$class: "ManuallyEnteredBackrefSource", backref: "${this.BUILD_URL}flowGraphTable/"],
            statusResultSource: [$class: "ConditionalStatusResultSource", results: [[$class: "AnyBuildResult", message: message, state: state]] ]
        ])
    }
}
def githubStatus = new GithubStatus(
                        ghprbGhRepository: env.ghprbGhRepository,
                        ghprbActualCommit: env.ghprbActualCommit,
                        BUILD_URL: env.BUILD_URL
                    )

pipeline {
    agent any
    stages {
        stage ('A') {
            steps {
                node('node') {
                    script {
                        githubStatus.setPending(this)
                    }
                }
            }
        }

        stage ('Test distribute') {
            steps {
                script {
                    githubStatus.setPending(this)
                }
            }
        }
    }
    post {
        success {
            node('master') {
                script {
                    githubStatus.setSuccess(this)
                }
            }
        }
        unsuccessful {
            node('master') {
                script {
                    githubStatus.setFailed(this)
                }
            }
        }
    }
}

You can trigger the change to the pr manually by clicking release change in the upper right hand corner of the pipeline execution

I wish it would be somewhere documented what @anton-mesnyankin mentioned above 🙏🏽 It saved me hours


refactoring the job code

This PR will create create a layer of jobspec between job and podtemplatespec
Signed-off-by: sanjay1611<EMAIL_ADDRESS>

Add the comment for the exported function


ignore Funnel basic credential if task submitted to TESK

Describe the bug
I am reproducing Alex's "hashsplitter" CWLworkflow but this time using snakemake.
For this Snakemake will be extended to forward FUNNEL_SERVER_USER and  FUNNEL_SERVER_PASSWORD to py-tes via envvars.
In my use case the "hashsplitter"  tasks are submitted to proTES:
snakemake --tes "https://protes.rahtiapp.fi/ga4gh/tes/" --snakefile Snakefile_s3 --jobs 1 --cores 1 --envvars HOME FUNNEL_SERVER_USER FUNNEL_SERVER_PASSWORD
Depending on where proTES sends the task, the task is either executed or not.
For Funnel the execution works fine while TESK shows the following error:
[2023-03-20 14:33:16,825: ERROR   ] Traceback (most recent call last):\n  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1516, in full_dispatch_request\n    rv = self.dispatch_request()\n  File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 1502, in dispatch_request\n    return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)\n  File "/usr/local/lib/python3.10/site-packages/connexion/decorators/decorator.py", line 68, in wrapper\n    response = function(request)\n  File "/usr/local/lib/python3.10/site-packages/connexion/decorators/uri_parsing.py", line 149, in wrapper\n    response = function(request)\n  File "/usr/local/lib/python3.10/site-packages/connexion/decorators/validation.py", line 193, in wrapper\n    response = function(request)\n  File "/usr/local/lib/python3.10/site-packages/connexion/decorators/parameter.py", line 116, in wrapper\n    return function(**kwargs)\n  File "/usr/local/lib/python3.10/site-packages/foca/utils/logging.py", line 61, in _wrapper\n    response = fn(*args, **kwargs)\n  File "/app/pro_tes/ga4gh/tes/server.py", line 45, in CreateTask\n    response = task_runs.create_task(request=request, **kwargs)\n  File "/app/pro_tes/ga4gh/tes/task_runs.py", line 152, in create_task\n    input.url = remove_auth_from_url(input.url)\n  File "/app/pro_tes/utils/misc.py", line 19, in remove_auth_from_url\n    elements[1] = elements[1][elements[1].rfind("@") + 1 :]  # noqa: E203\nTypeError: argument should be integer or bytes-like object, not 'str' [foca.errors.exceptions]
According to the traceback, proTES tries to remove basic authentication information which is not part of the URI.
To Reproduce
Steps to reproduce the behavior:

clone https://github.com/DMBS-BIH/hybrid-cloud-snakemake-demo and complete/adjust the Snakemake_s3 workflow
clone the latest Snakemake repo install the "tes_auth" fork. Not yet merged into the main branch.
export HOME=/tmp
source .env as described in https://github.com/elixir-cloud-aai/elixir-cloud-demos/blob/main/demos/2023-ecp-f2f/demo.ipynb
ecexute:  snakemake --tes "https://protes.rahtiapp.fi/ga4gh/tes/" --snakefile Snakefile_s3 --jobs 1 --cores 1 --envvars HOME FUNNEL_SERVER_USER FUNNEL_SERVER_PASSWORD

Expected behavior
Task successfully submitted to the random TES instances.
Software

OS Windows 11 -> WSL -> Ubunut 20.04

Additional info
Submission to individual TESK instances works fine.

Thanks @vschnei. Did you try what we discussed on Slack?

Just did a deployment of proTES locally and did some logs. The Snakemake request/payload does not contain any URL.
So in ../pro_tes/ga4gh/tes/task_runs.py:147 onward in the method create_task the empty ulr is passed to remove_auth_from_url  wich triggers the observed error.
Made a small fix by double checking if URL is present at all.
check if url is present


Also index tags from contacts-admin

contacts-admin has been migrated since https://github.com/alphagov/contacts-admin/pull/231.
Trello: https://trello.com/c/afa0Mfa4

:+1:


Delete theta output in MBAR.getFreeEnergyDifferences()?

def getFreeEnergyDifferences(self, compute_uncertainty=True, uncertainty_method=None, warning_cutoff=1.0e-10, return_theta=False):
"""Get the dimensionless free energy differences and uncertainties among all thermodynamic states.
Parameters
----------
compute_uncertainty : bool, optional
If False, the uncertainties will not be computed (default: True)
uncertainty_method : string, optional
Choice of method used to compute asymptotic covariance method,
or None to use default.  See help for computeAsymptoticCovarianceMatrix()
for more information on various methods. (default: svd)
warning_cutoff : float, optional
Warn if squared-uncertainty is negative and larger in magnitude
than this number (default: 1.0e-10)
return_theta : bool, optional
Whether or not to return the theta matrix.  Can be useful for complicated differences.
Returns
-------
Deltaf_ij : np.ndarray, float, shape=(K, K)
Deltaf_ij[i,j] is the estimated free energy difference
dDeltaf_ij : np.ndarray, float, shape=(K, K)
If compute_uncertainty==True,
dDeltaf_ij[i,j] is the estimated statistical uncertainty
(one standard deviation) in Deltaf_ij[i,j].  Otherwise None.
Theta_ij : np.ndarray, float, shape=(K, K)
The theta_matrix if return_theta==True, otherwise None.
    Notes
    -----
    Computation of the covariance matrix may take some time for large K.
    The reported statistical uncertainty should, in the asymptotic limit, reflect one standard deviation for the normal distribution of the estimate.
    The true free energy difference should fall within the interval [-df, +df] centered on the estimate 68% of the time, and within
    the interval [-2 df, +2 df] centered on the estimate 95% of the time.
    This will break down in cases where the number of samples is not large enough to reach the asymptotic normal limit.
    See Section III of Reference [1].
    Examples
    --------
    >>> from pymbar import testsystems
    >>> (x_n, u_kn, N_k, s_n) = testsystems.HarmonicOscillatorsTestCase().sample(mode='u_kn')
    >>> mbar = MBAR(u_kn, N_k)
    >>> Deltaf_ij, dDeltaf_ij, Theta_ij = mbar.getFreeEnergyDifferences()
    """
    Deltaf_ij, dDeltaf_ij, Theta_ij = None, None, None  # By default, returns None for dDelta and Theta

    # Compute free energy differences.
    f_i = np.matrix(self.f_k)
    Deltaf_ij = f_i - f_i.transpose()

    # zero out numerical error for thermodynamically identical states
    self._zerosamestates(Deltaf_ij)

    Deltaf_ij = np.array(Deltaf_ij)  # Convert from np.matrix to np.array

    if compute_uncertainty or return_theta:
        # Compute asymptotic covariance matrix.
        Theta_ij = self._computeAsymptoticCovarianceMatrix(
            np.exp(self.Log_W_nk), self.N_k, method=uncertainty_method)

    if compute_uncertainty:
        dDeltaf_ij = self._ErrorOfDifferences(Theta_ij, warning_cutoff=warning_cutoff)
        # zero out numerical error for thermodynamically identical states
        self._zerosamestates(dDeltaf_ij)
        # Return matrix of free energy differences and uncertainties.
        dDeltaf_ij = np.array(dDeltaf_ij)

    if return_theta == True:
         return Deltaf_ij, dDeltaf_ij, Theta_ij
    else:
         return Deltaf_ij, dDeltaf_ij

?

You aren't looking at the most recent code.
https://github.com/choderalab/pymbar/blob/master/pymbar/mbar.py#L479

Reviving this to avoid having multiple issues. Probably the best thing to do is to SAVE  the theta matrix when calculating the uncertainties, and then write a method to retrieve it separately.  It can be a big matrix, so one might not want to have it around in some cases, but we can deal with that a bit later (perhaps have saving be optional, but default on, then it can be listed as one of the things that can be done to save memory)

Reviving this to avoid having multiple issues. Probably the best thing to do is to SAVE the theta matrix when calculating the uncertainties, and then write a method to retrieve it separately.

Sounds great!


Faulty position indication after update to 0.3.2

Since the update two of my blinds show wrong value when opened.
When opening and fully openened one shows 89% open and tthe other 91% but they are fully opened.
Adding a second to the travelling_time_up didn't improve, it actually got worse because then both show 86% when fully opened.
I have to press stop followed by up to get the value on 100%.
`      kitchen:
friendly_name: "Kitchen Cover"
# Becker Centronic USB stick provides up to five units (1-5) with up to seven (1-7) channels
# Unit 1 - Channel 1
channel: "1"
tilt_blind: off
tilt_intermediate: off
# The travel time for direction up is sufficient if travel time for up and down are equal
travelling_time_up: 27
# Optional travel time for direction down
travelling_time_down: 24
# Optional intermediate positions for up and down
# intermediate_position_up: 75
# intermediate_position_down: 25
# Disable intermediate position when not used
intermediate_position: off
bedroom_1:
friendly_name: "Master Bedroom Cover"
# Unit 1 - Channel 2
channel: "2"
tilt_blind: off
tilt_intermediate: off
travelling_time_up: 16
travelling_time_down: 14.5
intermediate_position: off
More info:
The kitchen cover also becomes some kind of unresponsive. After closing it doesn't accept up although the up button is available. Pressing stop and down a few times makes it respond again to open.
But then the counter gets stuck on 89% and only after pressing stop it becomes 100%.
Already deleted the db file and restarted HA but no improvements yet.
Only strange behaviour on two of the five covers.
Also did update to  2023.5.2  this morning way before covers were opened.

Back to v0.3.0 and everything seems to be back to normal, for now.

v0.3.3 fixes your issue


SQL Unique constraint generation for h2 should not contain "USING INDEX" clause

Search first

[X] I searched and no similar issues were found

Description
When generating a diff changelog with liquibase, the output generation of unique constraints contains "USING INDEX" for a h2 database output sql file when the reference database was a oracle database. This could be improved by adding a condition !h2Database to the generation code.
The generated code is:
CREATE UNIQUE INDEX FALL_FKRQ_IX ON FALL(FALL_KEY, QUARTAL);
ALTER TABLE FALL ADD CONSTRAINT UC_FALL_KEY_QUARTAL UNIQUE (FALL_KEY, QUARTAL) USING INDEX FALL_FKRQ_IX;

But it should be
ALTER TABLE FALL ADD CONSTRAINT UC_FALL_KEY_QUARTAL UNIQUE (FALL_KEY, QUARTAL);

It works fine, when using a h2 database as a reference database.
The usecase we have is to automatically generate incremental diff changeset for various databases (using the liquibase gradle plugin) based on our Spring / JPA Entities compared to a reference Database (oracle).
So we compare the Spring JPA Entites with the Oracle Database and write changelog files for different database types.
Steps To Reproduce

Create a JPA entity with a @UniqueConstraint annotation, for example

@Table(
        name = "FALL",
        uniqueConstraints = {
                @UniqueConstraint(
                        name = "UC_FALL_KEY_QUARTAL",
                        columnNames = {"FALL_KEY", "QUARTAL"}
                )
        }
)

Use a spring url as a reference url (hibernate:spring:...)
Use a oracle jdbc url as a url
Set the changelog filename to my-filename.h2.sql
Generate diff changelog

Expected/Desired Behavior
Liquibase omits the "USING INDEX" clause when the database detected based on the filename is h2.
Liquibase Version
4.24.0
Database Vendor & Version
Oracle, H2
Liquibase Integration
gradle
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
Windows 10
Additional Context
The fix would be to change
if (statement.getForIndexName() != null) {
            sql += " USING INDEX ";
            sql += database.escapeObjectName(statement.getForIndexCatalogName(), statement.getForIndexSchemaName(),
                statement.getForIndexName(), Index.class);
            isInUsingIndexClause = true;
        }

to
if (statement.getForIndexName() != null && !(database instanceof H2Database)) {
            sql += " USING INDEX ";
            sql += database.escapeObjectName(statement.getForIndexCatalogName(), statement.getForIndexSchemaName(),
                statement.getForIndexName(), Index.class);
            isInUsingIndexClause = true;
        }

in liquibase.sqlgenerator.core.AddUniqueConstraintGenerator#generateSql
Are you willing to submit a PR?

[X] I'm willing to submit a PR (Thank you!)

Might be related: https://github.com/liquibase/liquibase/issues/1384

Hi @Guschtel! Thank you for reporting this issue.
If you could go ahead and create a PR, our devs will review it and guide you if necessary. If you add a fix to the related issue you've mentioned, that would be great, but otherwise mention the issue in your PR so we can work on it.
Thanks again,
Tatiana

Hi @Guschtel,
Your proposed code change looks good to me. Please go ahead and apply these changes when you wish. I think it would be nice to also add some H2  integration test/s for this so we have some test coverage for this case. I can help you with it if you have any questions.
You can also add go ahead and apply the required changes to fix #1384 as well in the same PR if you are happy to address that issue as well.
Thanks,
Daniel.

I haven't forgotten this issue :) I'll definitely look into it as soon as time allows :+1:


Incorrect route gets rendered

Hi, I'm new to Leptos, so it's possible that I'm not defining the routes properly. I've also tried changing the <Home /> route to path="" and the <Post /> route to path="/:id" but they yield the same result.
Describe the bug
Given the following routes:
<Router trailing_slash={TrailingSlash::Drop} fallback={|| view! { <NotFound /> }}>

            "Home"

            <Routes>
                  <Route
                        path="/"
                        view={|| view! { <Home /> }}
                  />
                  <Route
                        path=":id"
                        view={|| view! { <Post /> }}
                  />
            </Routes>

</Router>

Expected behavior:

https://example.com should render <Home />
https://example.com/123 should render <Post />

Actual behavior:

https://example.com renders <Post />
https://example.com/123 renders <Post />

Leptos Dependencies
leptos = { version = "0.5", features = ["csr"] }
leptos_router = { version = "0.5", features = ["csr"] }

Your understanding of the expected behavior is correct. Unfortunately it looks like #2172 broke matching on / if there is a possible param at the same level. Thanks for raising this — I will need to fix this and likely yank the last two releases.


@ added to forecast formulae

Description
When setting a 'FORECAST' formula, '@' is inserted in between the '=' and the rest of the formula.
Steps to reproduce the issue:
file := excelize.NewFile()
file.NewSheet("Sheet 1")
file.SetCellFormula("Sheet 1", "A1", "FORECAST.LINEAR(2021,B3:D3,B1:D1)")
file.SaveAs("file.xlsx")

Describe the results you received:
When opening the spreadsheet, the cell formula is set as "=@FORECAST.LINEAR(2021,B3:D3,B1:D1)"
Output of go version:
go version go1.14.2 darwin/amd64

Excelize version or commit ID:
Excelize v2.1.0

Thanks for your feedback. Which version of the spreadsheet application are you using? I have tested on Excel for mac 16.35, Excel 16.0.12624.20422 on Windows and it works well.

Thanks for the quick response! I am using Excel for mac 16.36.

The same issue with XLOOKUP. It’s new Dynamic Arrays feauture. And excelize has no ways to deal with it
Here some info


[No ticket] Fix draft reg reroute logic for read-only users

Ticket: []
Feature flag: n/a

Purpose

take read-only users to registration overview page if the draft registration is already submitted

Summary of Changes

prevent reroute to review page from happening if the draft is already submitted

Screenshot(s)

NA

Side Effects

NA

QA Notes

This should only impact read-only users on a submitted registration. Read-only users will still be routed to the review page if the draft is still in progress

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.819%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4993

Relevant Lines:
6608

💛  - Coveralls


chore: BED-4141 - added parameters and supporting schemas to openapi src

Description
This PR adds common parameters and supporting schemas to the openapi document source.
Motivation and Context
Part of an ongoing effort to improve API documentation
How Has This Been Tested?
Manual testing of openapi generation. Documentation only changes
Screenshots (if appropriate):
Types of changes

[x] Chore (a change that does not modify the application functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] Documentation updates are needed, and have been made accordingly.
[ ] I have added and/or updated tests to cover my changes.
[x] All new and existing tests passed.
[ ] My changes include a database migration.

I appreciate that the files are small and easy to read through. It looks like it'll be good for composability too. Other than the prefer header default timeout, the content looks to be accurate to me.
Will there be a paths directory under src eventually? Is there a reference or documentation that you would suggest reading through to understand the intended structure and organization of this project?

@urangel yes, there will be a paths folder. The goal is to produce a single JSON file following this spec: https://spec.openapis.org/oas/v3.0.3. But as for how the source will be broken up into multiple files, there's not really a one-size-fits-all guideline for it. I'm putting together a structure that should be plugin friendly for both JetBrains and VSCode, as well as compatible with tools like Redocly, which is what will be used to compile the yaml source files into a single JSON doc