Typo "@failover_mode="→"@failover_mode="

bold with escape characters
https://docs.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sp-setreplfailovermode-transact-sql?view=sql-server-2017

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the update.
@hyoshioka0128 Thanks for submitting the fix.


Add shared scheme to scm

This merge request aims to add the framework as a shared scheme to version control, for the purpose of fixing Carthage builds.
This fixes #143.

Could you make a new version tag so this gets prioritized by Carthage?

Could you make a new version tag so this gets prioritized by Carthage?

Okay, I’ll do that tomorrow.
Sent from my iPhone

On Jan 18, 2021, at 11:52 PM, Jacob Rhoda<EMAIL_ADDRESS>wrote:

Thanks! Could you make a new version tag so this gets prioritized by Carthage?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

Okay, I’ll do that tomorrow.
Sent from my iPhone

On Jan 18, 2021, at 11:52 PM, Jacob Rhoda<EMAIL_ADDRESS>wrote:

Thanks! Could you make a new version tag so this gets prioritized by Carthage?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

Done.
Please have a try.

Done.
Please have a try.


Raise exception in checkWithRedirector when error occurs.

Previously checkWithRedirector would catch an error but would never reraise or do anything with it, so getBootstrapScript continues as if no error has been raised. Fixed it so checkWithRedirector raises the error.

We never directly merge the branch to main, so please raise the PR for the dev branch instead


HDDS-4051. Remove whitelist/blacklist terminology from Ozone

What changes were proposed in this pull request?

Replace whitelist/blacklist with allowlist/blocklist

There are still some occurrences of these terminologies in javascript frameworks like bootstrap and angular. Hopefully, these frameworks will update these terminologies in their future releases.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4051
How was this patch tested?
No test is needed.

+1 on the ignore/ignorelist proposal

Thanks @vivekratnavel for updating the patch, LGTM.

Thanks for the reviews @adoroszlai @avijayanhwx @amaliujia !


[v2 data] Event is deleted but it is not in delimiting age when delete by age

🐞 Bug Report
Affected Services

 The issue is located in: core-data
Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
Description and Minimal Reproduction

Create 2 events with the following body.

{"event": {"apiVersion": "v2", "deviceName": "Device-Test-001", "profileName": "Profile-Test-001", "sourceName": "Command-Test-001", "id": "a4762b40-109c-4f1e-ae19-a68ad4016d84", "origin":<PHONE_NUMBER>839000064, "readings": [{"deviceName": "Device-Test-001", "resourceName": "Simple-Reading", "profileName": "Profile-Test-001", "origin":<PHONE_NUMBER>839000064, "valueType": "Int8", "value": "3"}]}, "apiVersion": "v2"}

{"event": {"apiVersion": "v2", "deviceName": "Device-Test-002", "profileName": "Profile-Test-001", "sourceName": "Command-Test-002", "id": "9a17df51-68d1-4224-866d-e1c47ed1580c", "origin":<PHONE_NUMBER>881999872, "readings": [{"deviceName": "Device-Test-002", "resourceName": "Simple-Reading", "profileName": "Profile-Test-001", "origin":<PHONE_NUMBER>881999872, "valueType": "Int8", "value": "3"}, {"deviceName": "Device-Test-002", "resourceName": "Simple-Float-Reading", "profileName": "Profile-Test-001", "origin":<PHONE_NUMBER>881999872, "valueType": "Float32", "value": "39.5"}]}, "apiVersion": "v2"}

Delete by age

Current time:<PHONE_NUMBER>924000000
Send Delet http://localhost:48080/api/v2/event/age/44999936

🔥 Exception or Error

Both events are deleted.
Expected: Only the first event is deleted.
🌍  Your Environment
Deployment Environment:
**EdgeX Version:**master
Anything else relevant?

Test script error is fixed by edgex-taf PR #353.


Clarinet: requirements documentation

Clarinet requirements are poorly documented and hard to find.
Our docs should have a dedicated section for requirements.
Including some of the docs that exist in the readme https://github.com/hirosystems/clarinet/pull/1184

@hugocaillard noted: this will be added to new docs revamp


What is the total size in dimentions?

Hello,
I wonder if you just can share the size in dimentions :-)
Before I like to build this, i would like to know if this fits on my wall. (or ceiling)
Thanks

Guessing ~1.2m wide by ~1m high for current checked in code and build instructions, with hex arrangement of 23cm lengths.  However, people can customize the graph node-edge definitions in mapping.h to other dimensions and layouts to fit their chromance desires.
Would be interesting to see alternative mappings and layouts people end up building.

That is the plan ... but i would like to start with yours ;-)


Inject route params in fetchAll

#248

Will not merge, see https://github.com/zfcampus/zf-apigility-doctrine/issues/248


Sequence Support for LoadData

Today is possible load csv into tables with sequence columns this way:
id_pais|cod_bacen|descricao
SEQNAME.NEXVAL()|1234|BRASIL
SEQNAME.NEXVAL()|1234|ARGENTINA

<loadData tableName="pais" file="liquibase/seeds/pais.csv" separator="|" >
    <column type="computed" name="id_pais"/>
    <column type="numeric" name="cod_bacen" />
    <column type="string" name="descricao" />
</loadData>

but it is not multidatabase compatible.. because this I propose this change:
id_pais|cod_bacen|descricao
0|1234|BRASIL
0|1234|ARGENTINA

<loadData tableName="pais" file="liquibase/seeds/pais.csv" separator="|" >
    <column type="sequence" name="id_pais" defaultValue="pais_seq"/>
    <column type="numeric" name="cod_bacen" />
    <column type="string" name="descricao" />
</loadData>

Related issue: https://liquibase.jira.com/browse/CORE-2260

Made some minor changes, see https://liquibase.jira.com/browse/CORE-2540

I tried to use Sequences in CSV files but didn't manage to get it to work. As long as I use "updateSQL" it generates correct SQL which looks like this:
INSERT INTO principal (id, name, enabled, comment) VALUES (nextval('hibernate_sequence'), 'admin', TRUE, 'Superuser');
but when I use "update" there will be NULL instead of the correct value from the sequence
Batch entry 0 INSERT INTO principal(id, name, enabled, comment) VALUES(NULL, 'admin', 'TRUE', 'Superuser') was aborted: ERROR: null value in column "id" violates not-null constraint


Blue/green Deployment Strategy

DevEx/OpEx
Implement a blue/green deployment strategy to ensure zero-downtime deployments and rapid rollbacks.
Pre-conditions

[ ] Database CICD and Connection Pooling #753 story is complete.

Completion Criteria

[ ] Blue/green deployments are working in the staging environment

Tasks

[ ] x

Be sure to include a roll back plan (per Eng Block on 01/17).

Verify if deployments currently cause any measurable downtime.


how to do filter on multiple fields?

is it as simple as ?filter{field1.icontains}=query&filter{field2.icontains}=query&filter{field3.icontains}=query?

Yep, that should do it. Note that filters are ANDed together.

Thank you. I love this library by the way.


Fix: make runtime forward-compatible with new parameters

Problem: the existing runtime is not compatible with the addition of new parameters in ConfigurationPayload. Adding a parameter will result in an invalid argument error with older runtimes.
Solution: add a factory method that takes a dictionary and filters out useless fields.

We prefer using another approach with more validation.


"Illegal instruction" error on HaikuPorts buildbot

According to the logs the build fails on the x86_64 bots while it works ok on x86 bots.
As you can see we are targeting Northwood for x86 and Prescott for x86_64 arch:
https://github.com/haikuports/haikuports/blob/master/sci-libs/openblas/openblas-0.3.4.recipe
Here are the builder logs:
x86 : https://build.haiku-os.org/buildmaster/master/x86_gcc2/logviewer.html?buildruns/2437/builds/31495.log
x86_64 : https://build.haiku-os.org/buildmaster/master/x86_64/logviewer.html?buildruns/2558/builds/12604.log
Note, we call x86 as x86_gcc2, as it got 2 compiler toolchain, the primary is gcc2 and the secondary is gcc7. The x86_64 platform have only gcc7. Blas was built on both arch with gcc7.
Sadly i have currently no information about the buildbots hw, but i can ask the admins.
I built openblas on x86_64 locally with the same recipe, works fine.

You make tests with different parameters than build
You should either patch Makefile.rule or copy over all parameters to all invocations of make.
https://github.com/haikuports/haikuports/blob/662967332f95a4c5e0f44f8fefcb3fcd7e652564/sci-libs/openblas/openblas-0.3.4.recipe#L87

Not sure I see where make is invoked with different parameters ? To me this looks like one contiguous build and not separate invocations, what am i missing ?

If it helps repeating TARGET and OPENMP option....

Oh, that could be the culprit, lemme fix it then.

Hmm. I still do not see the different parameters (or separate invocation of make test). Given that this is a DYNAMIC_ARCH build, runtime failure would still be a bug (unless the buildbot runs on some virtual setup that happens to lack x86 fpu features from ten years ago). Given that it happens with the provided tests, at least it seems unlikely that this could be another case of unaligned data access.

If it would be possible to somehow get backtrace text, and disassembly of the instruction generating the exception....

Any progress on this ? (One option could be to inspect config.h and Makefile.conf , or better yet dump /proc/cpuinfo somewhere early in the build process)

You have to set NO_AVX512=1 since haiku "sysinfo" does not have bits to decode AVX512 ISA-s, thus permitting me to deduce that XSAVE and stuff ignores respective register file too (though GCC7 would try to use it)

Not sure if that would really preclude running AVX512 code if that is what the buildbot hardware is. (Any word on that ?)

actually context switch uses fxsaveq , i.e NO_AVX=1 NO_AVX512=1 is needed, though running single AVX-512 thread may go unpunished on 1-cpu system.

No support whatsoever of AVX, unconditionally:
https://git.haiku-os.org/haiku/tree/src/system/kernel/arch/x86/64/interrupts.S#n226
In addition webkit gets SIGILL at times, apparently very faraway relative ffmpeg is better at CPUID stuff.

Weird. How would that mesh with e.g. existing Eigen port to Haiku ?

Eigen is with minimal patch, changing some file IO
I will run its self-tests, probably GSL ones too.

Was your successful local build test also under Haiku ?

Eigen 'make check' finished without any breakage (i applied the only distro patch in existence over 3.3.7)
I set a virtualbox VM for haiku exactly according to doc recipe , it has slow ide disk, gets significantly faster attaching VDI to SATA disk emulation.
I changed hardware type to linux to no success - Linux says Piledriver is Piledriver, Haiku says Piledriver is Barcelona
Forcing target to Piledriver or Sandybridge goes into SIGILL in R DGEMM benchmark. Quicker if running many parallel threads(it is processes under R foreach, more than cores)
I would speculate that Haiku does not move threads between CPUs until last chance, so this problem is somewhat less visible as in effect 2 AVX+ threads rarely meet on same CPU
I think as a temporary measure you set port to exclude AVX - this is not video decoder where one survives short glitch, we do care about numeric accuracy in the end. It needs two parameters added.
btw while building eigen I saw sigill-s in smpeg library and sdl
Would be unfair to record it here "upstream" that Haiku had limited AVX support before beginning 2019, and quite impractical to try to detect it for short moment before Haiku shines.
Merry Christmas everyone,

In sysinfo in the bug report:
family 6, model 85, stepping 4
It is SKYLAKE-X series "Scalable Xeon", not showing a problem as long as tests run in same (Linux,BSD) system as built, just that by pure (un)luck it hit context from previous AVX512 use during tests.
To make result universal an old target without -march flags like TARGET=CORE2 needs to be added to make.
They say it was deduced from sysinfo alone, actually looking further after seeing no display of any AVX-512 bits in outout - core assembly implementing context switcher that has no chance to get around AVX(1) was located.
(not)glad to see numpy joining the party.

As a side effect of #1947 this may get mitigated by #1949 assuming that HaikuOS does not advertise the availability unsupported AVX* features in the cpuid bits. Up to this point, it was simply assumed that AVX2 and/or AVX512 was available if the model number of the cpu matched.


RKE1 - Force Provision

We want to give users more control over the RKE1 cluster provisioning process:
Force Provision
We also need a way for users to force Rancher to restart the provisioning after Rancher hits the backoff limit and stops retrying. This allows users to reconcile the cluster after the issue is fixed.
We also need to update the dashboard UI to make it easy to trigger a force provision.

Separate issues will be made in the rancher/rke and rancher/dashboard to track relevant works.

Due to a shift in priorities, this is no longer in the v2.6.6 release plans, so moving it to Unscheduled for now.


mistake on pre release to download

8.0.0-B2
@AlexTr AlexTr released this on 7 May
Trident v.8.0.0-B2 (beta 2)
Almost production ready, upgradable
Downloads
(30.4 MBDolphin-v.7.2.0.B2.zip)
-v.7.2.0.B2.zip must be  Trident-v.8.0.0-B2.zip

Fixed.
Thank you for pointing this out.


findOrCreateElement function

I have a little problem with findOrCreateElement function. I have multiple vendor libraries that creates element with class .dropdown-menu inside the body element. Text-complete will get first element with .dropdown-menu and override it with its own data. This problem can be bypassed with appendTo option, but from various reasons I need to keep my dropdown-menus directly inside the body element.
Is there another approach to this ? If not I  am suggesting following:
1.) introduce new option called e.g. dropdownId
2.) use it inside the findOrCreateElement function
Original function:
   findOrCreateElement: function (option) {
      var $parent = option.appendTo;
      if (!($parent instanceof $)) { $parent = $($parent); }
      var $el = $parent.children('.dropdown-menu')
      if (!$el.length) {
        $el = $('

').css({
          display: 'none',
          left: 0,
          position: 'absolute',
          zIndex: option.zIndex
        }).appendTo($parent);
      }
      return $el;
    }

Modified function:
   findOrCreateElement: function (option) {
      var $parent = option.appendTo;
      var dropdownId = options.dropdownId || '';
      if (!($parent instanceof $)) { $parent = $($parent); }
      var $el = $parent.children(dropdownId + '.dropdown-menu')
      if (!$el.length) {
        $el = $('

').css({
          display: 'none',
          left: 0,
          position: 'absolute',
          zIndex: option.zIndex
        }).appendTo($parent);
      }
      return $el;
    }

I can send a pull request if this is a way to go. Thanks for any answer.

I guess it relates to https://github.com/yuku-t/jquery-textcomplete/issues/130. There can be multiple dropdown menus that can be opened at the same time and they should not collide with each other.

Or another approach can be to used className option in findOrCreateElement function.
Anybody have some ideas please ?

Hi @char0n, thanks for opening the issue.
I understand your situation. Now dropdown element has "textcomplete-dropdown" class.
FYI all textcomplete objects having same appendTo share dropdown elements.

Thanks for the textcomplete-dropdown class. This should resolve my problem.
Anyway: is there a reason why share the sme dropdown menu ? This will effectively eliminate possibility to have 2 opened text complete dropdown menus. What is the logic behind this ?
Thanks a lot.


Metrics Explorer: improve VirtualizedGrid.svelte

[stub]
I have a very basic virtualized renderer for long lists of leaderboards. I would like someone to spend time with it and make sure that it is very solid. Please see the dev route in /dev/explore and find the VirtualizedGrid.svelte component.
Without a virtualized renderer for leaderboards, we will have substantial UI jank.
The component is lightly adapted from this one.
Ideally, it would have the following:

rendering should be very smooth in leaderboards. No jank.
it should know to render the leaderboard when it is just out of view in the viewport.
you don't have to go with this component. But please consider implementing your own before pulling in someone else's library. We want to reduce the number of dependencies on 3rd party code if possible.

This could be a good ticket for anyone who wants to go deep on virtual lists. It's entirely a frontend task.

@AdityaHegde or @ericpgreen2 – could y'all clarify the progress on this, if there has been progress made? We need to clean this up for the metrics explorer. There is some jank to my implementation.

Aditya and I discussed that I'd first address #316, in which I'm implementing an infinite scroll to add more rows to the preview table. Then, after my work is done, he's going to work on the virtualization aspect.
I'm actually working on #316 right now and think I'm close – will finish either tonight or tomorrow.

I believe we shipped this in 0.8 https://docs.rilldata.com/notes/0.8 -- closing as complete, please feel free to reopen if this is still needed for some reason, or perhaps file a new issue with updated requirements


Bump version to 0.0.32

#979  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dongreenberg and the rest of your teammates on  Graphite


Fix deprecation on Symfony 4.2

See https://github.com/symfony/symfony/pull/27476

build failure unrelated

I'll try to fix the build in a separate PR and then check this one. Thanks for the contribution!


Use .into_iter() method on array to avoid dereference

Arguably it’s nicer to just use .into_iter() instead of iterating over references which are immediately dereferenced.
I also changed the use of String::from to .to_string() because the latter has around twenty times more usages in rust-analyzer.

Thanks!
bors r+


distributed training

refactored ps-lite, no proto files in include
link *.a of the ps-lite deps (protobuf-lite, zmq)
updated dist train examples and docs
tools/pack.sh to pack mxnet library

relation to #527 ?

no idea what happened. #527 fixed some issues of this pr

OK, move to #527 then


What's the difference between @base64ToString and @decodeBase64?

They both seem to decode the 64-base encoded string.  Are there any differences?

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

ID: 03aa9e88-7fa0-56ab-f794-58f0c37b6d7a
Version Independent ID: 25c5ef5c-6ba4-f5ff-f94b-8f2818ec2feb
Content: Reference guide for expression functions - Azure Logic Apps
Content Source: articles/logic-apps/workflow-definition-language-functions-reference.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Thank you for your feedback! We will review and update as appropriate.

Hello @DrReedo - Thanks again for your feedback. The decodeBase64() function is deprecated, as noted on this doc, and the guidance is to use the base64ToString() function instead.
Since we didn't determine any changes to this documentation upon reviewing your feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Remove zustand and use React.Context instead to unnecessary dependency

Is your feature request related to a problem? Please describe.
No problem. This is to remove the zustand dependency, so that users don't have to install it.
Describe the solution you'd like
Let's remove zustand and use React.Context instead!
Describe alternatives you've considered
N/a
Additional context
N/a

Closing this due to research leading to this not being able to be implemented. The main point of entry for the alert and modal components also uses state, and there is no way to wrap it with context.


azure-keyvault-administration needs linting updates for pylint version 2.15.8

ACTION NEEDED: This version of pylint will be merged on 2024-01-15. The build will begin to fail for this library if errors are not fixed.
Library name: azure-keyvault-administration
Pylint version: 2.15.8
Pylint errors: Link to build (2023-12-17)
How to fix: Run the next-pylint tox command at the library package-level and resolve the linting errors.

../azure-keyvault-administration>pip install "tox<5"
../azure-keyvault-administration>tox run -e next-pylint -c ../../../eng/tox/tox.ini --root .

See the Pylint Guide for more information.

Resolved by https://github.com/Azure/azure-sdk-for-python/pull/33448


Add missing base type to cdef declarations

I presume the base type was forgotten in these two cases?
(found whilst testing out a cython linter)

/test mini


typos in opentelemetry configuration

The protocol for opentelemetry is called OTLP.
In core.yaml and the FSC codebase, it's called both optl and otpl. To the point where you have to:
tracing:
    provider: otpl
    optl:
      ...

This is quite confusing 😅.
Propose to change them to opentelemetry in core.yaml, and otlp where we want a shorthand in the code.

Yes you are right about the change. There was a typo and then I started using the correct one, but didn't change the value written in the config.


InlineTextContent alignment not working a expected.

I expected PlaceholderVerticalAlign.Center to align the centre of the image with the center of the text but it aligns the center of the image with the bottom of the text.
For all values of PlaceholderVerticalAlign.*, the align of the image is always aligned with the bottom of the text.
import androidx.compose.desktop.Window
import androidx.compose.foundation.Image
import androidx.compose.foundation.text.InlineTextContent
import androidx.compose.foundation.text.appendInlineContent
import androidx.compose.material.Text
import androidx.compose.material.icons.Icons
import androidx.compose.material.icons.filled.Person
import androidx.compose.ui.text.Placeholder
import androidx.compose.ui.text.PlaceholderVerticalAlign
import androidx.compose.ui.text.buildAnnotatedString
import androidx.compose.ui.unit.IntSize
import androidx.compose.ui.unit.em

fun main() {
	Window(title = "Bad Text", size = IntSize(300, 300)) {
		val text = buildAnnotatedString {
			appendInlineContent("image")
			append(" is a person.")
		}
		val person = InlineTextContent(
			placeholder = Placeholder(1.em, 1.em, PlaceholderVerticalAlign.Center),
			children = {
				Image(Icons.Filled.Person)
			}
		)
		Text(text, inlineContent = mapOf("image" to person))
	}
}

Is the same problem present in Android Compose? Why do you believe that behavior we get is not correct?

The reason why I think the behaviour is different is because the docs say otherwise.
https://developer.android.com/reference/kotlin/androidx/compose/ui/text/PlaceholderVerticalAlign#center
It says,

Align the center of the placeholder with the center of the entire line

I don't know if this behaviour is present in Android compose.

https://kotlinlang.slack.com/archives/CJLTWPH7S/p1623185440192700?thread_ts=1623185440.192700&cid=CJLTWPH7S
Looks like it's not present in Android compose.

@mgroth0 Could you post a reproducer? Both the op's reproducer and the test below seem to work correctly:
fun main() = singleWindowApplication {
    val text = buildAnnotatedString {
        append("Hello ")
        appendInlineContent(inlineContentId, "[myBox]")
        append(" inline content")
    }

    Column {
        Text(
            text = text,
            inlineContent = redSquare(2.em),
            style = TextStyle(fontSize = 32.sp),
            modifier = Modifier
                .padding(8.dp)
                .border(Dp.Hairline, color = Color.Black)
        )
        Text(
            text = text,
            inlineContent = redSquare(0.5.em),
            style = TextStyle(fontSize = 32.sp),
            modifier = Modifier
                .padding(8.dp)
                .border(Dp.Hairline, color = Color.Black)
        )
    }
}

const val inlineContentId = "inlineContent"

private fun redSquare(size: TextUnit) =
    mapOf(
        Pair(
            inlineContentId,
            InlineTextContent(
                Placeholder(
                    width = size,
                    height = size,
                    placeholderVerticalAlign = PlaceholderVerticalAlign.Center
                )
            ) {
                Box(modifier = Modifier.fillMaxSize().background(color = Color.Red))
            }
        )
    )

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add extra parameters to moved and resized events

I would like to know if it is already possible to pass extra parameters in (for example) the @moved event?
Following the documentation, I've used the @moved event as follow:
movedEvent: function(i, newX, newY){
    const info = i + "|" + newX + "|" + newY;
    alert(info);
}

What I would like is to pass an extra parameter like so:
@moved="movedEvent(myExtraParam)"
And then something like:
movedEvent: function(i, newX, newY, myExtraParam){
    const info = i + "|" + newX + "|" + newY + "|" + myExtraParam;
    alert(info);
}

But this obviously doesn't work. The reason I would like this, is that since my vue-grid-layout is being populated from a backend API, I would like to have the opportunity to pass a id field form my API to the grid-item events (in this case the @moved event).

Would this work?
@moved="(i, newX, newY) => movedEvent(i, newX, newY, myExtraParam)"

Will give this a try! Thanks for the reply!


pull-request

Description
Concise description of proposed changes, We recommend using screenshots and videos for better description
Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Related Issues
Closes #{issue number}
Note: If your changes are small and straightforward, you may skip the creation of an issue beforehand and remove this section. However, for medium-to-large changes, it is recommended to have an open issue for discussion and approval prior to submitting a pull request.
Your ENS/address:

I'm guessing this PR is opened to the wrong target branch? If there's anything you'd like merged into the main project, please open an issue first with some context that explains the issue you're trying to resolve.


fix(rpc): add all code from cache.accounts to witness output

WIP for trying to add missing created codes to reth's debug_executionWitness

closing in favor of https://github.com/paradigmxyz/reth/pull/11673


Add DataFrame.hist() data to pandas/profile.py

Add Pandas's hist() data to each Column:
https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.hist.html
Follow-on to:
https://github.com/recap-cloud/recap/pull/157

I would like to work on this issue. Should I add this to the ColumnProfile?
https://github.com/recap-cloud/recap/blob/1a064557d9eec68f8902bb70caf631d7cf738ee0/recap/analyzers/pandas/profile.py#L17-L31

Yea, I think a histogram in the column profile sounds right. :) Thanks for picking this up!

So, I noticed .hist() actually generates a chart, which isn't what we want. We want a textual representation. Not sure best way to do that.

Do you think that the result of np.hist(), would be a better fit? It could return a list of values of the histogram and the bin edges of the histogram.

Ah, yea, that looks like the right method to use. Good call.


vee-validate.d.ts mistake

Versions

vee-validate: 2.2.4
vue: 2.6.10

Describe the bug
I create a custom rule according to this manual using typescript and my linter give me an error, because interface Rule (types/vee-validate.d.ts) doesn't have getMessage field.

This is a bug that was resolved recently, it will be fixed in the next release later today. This is mentioned here #2040

Looking forward to this. Cannot build my stuff :)

Thanks for fix. Unfortunately, I found another miskate. According to this manual validate function in custom rule can return object containing 'data' and 'valid' fields. But in vee-validate.d.ts only boolean and Promise. Sorry, that i didn't notice before.


Create a Dynamic Plugins Guide

What do you want to improve?
We need a guide to how dynamic plugins work
What does the current documentation state?
Nothing about dynamic plugins / installation.
What would you like to be added/updated in the documentation?
Draft from Tom's message in slack:

The "dynamic plugins" are completely separate from "declarative integration". They are complementary, yes, however the relation is a bit more complicated:
The upstream "declarative integration" system is not fully implemented. Its implementation deviated pretty heavily from the RFC and we can't be building something on an ever changing unstable and incomplete API. Therefore we implement our own similar logic to achieve similar results without all the under the hood changes in Backstage core, using the current Backstage core facilities.
The "dynamic plugins" allows you to sideload plugins into a containerized instance. That means - to add a plugin you do not have to recompile your codebase and rebuild container.
In contrast, "declarative integration" is totally oblivious to this and works with local runtime only e.g. the plugin already has to be part of the codebase (installed as a NPM) and bundled in the application in order to configure it's placement. You can't decide to add a plugin later, not without of the cost of producing a new image.
With "dynamic plugins" we do not require the plugin to be present in build time, we pull it in, sideload it into the runtime and that's it. In addition to that, we offer similar features as "declarative integration" by allowing users to define UI element placements etc. - because just loading something is useless unless you surface it to users, right? And again, since the "declarative integration" is not complete at this moment, we can't rely on it so we have built our own solution.
"Dynamic plugins" are a Janus/RHDH only solution at this point and our intention is to bring the same to upstream and introduce the "dynamic plugins" on top of "declarative intergation" for upstream Backstage.
One additional way to look at this is - "declarative integration" is a brand new frontend system they want to introduce upstream. It requires both instance and plugin code changes. These changes are quite substantial and breaking. On the other hand "dynamic plugins" do not require any code changes for 90% of the plugins and should work out of the box. From plugin developer point of view it only requires you to produce additional set of assets and have a compatible core instance. However as I said before, the current "dynamic plugins" implementation is for Janus/RHDH only - so only instanced based on our codebase are compatible with such plugins at the moment. Other instances will not work.

See also https://github.com/janus-idp/backstage-showcase/blob/main/showcase-docs/dynamic-plugins.md


Correct words

Correct words
Thank you for the PR! Contributors like you keep React awesome!
Please see the Contribution Guide for guidelines:
https://github.com/reactjs/reactjs.org/blob/master/CONTRIBUTING.md
If your PR references an existing issue, please add the issue number below:
#

It's not the next section though.
Maybe "in the next two sections"?

Is it the word "next" that's throwing you?
I don't think the way it was before is really problematic. 😄  But would something like "in the following sections" work better?

I think the wording is probably okay as-is. 😄


Update SQL client version

Make component governance happy

@sebastienros we might need to do a bit more. The underlying culprit is the Azure.Identity dependency


[Bug] Doris1.2 select round(2.5) result is 2

Search before asking

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

Version
<IP_ADDRESS> and 1.2.8
What's Wrong?
ref to doc https://doris.apache.org/docs/1.2/sql-manual/sql-functions/math-functions/round/
select round(2.5); should return 3 , but actually 2
What You Expected?
same as mysql behavior return 3
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I will fix

set round_precise_decimalv2_value=true;
MySQL [(none)]> select round(2.5);
+------------+
| round(2.5) |
+------------+
|          3 |
+------------+
You can set it this way to get the results you want.


doc: add design doc for the ceph-csi-operator

Adding intial design doc for the ceph-csi-operator

LGTM


Mockiot upgrade

Bumping mockito from 1.0.19 as it suffers from
 Error:  WARNING: An illegal reflective access operation has occurred
Error:  WARNING: Illegal reflective access by org.mockito.cglib.core.ReflectUtils$2 (file:/home/runner/.m2/repository/org/mockito/mockito-all/1.10.19/mockito-all-1.10.19.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
Error:  WARNING: Please consider reporting this to the maintainers of org.mockito.cglib.core.ReflectUtils$2
Error:  WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
Error:  WARNING: All illegal access operations will be denied in a future release

When running with JDK 11 and fails on JDK 17 (I haven't checked where in between it changes from warning to error).
Bumping the other test scope dependencies while I'm there.

Build failed. Please check:
Error:  Failed to execute goal org.basepom.maven:dependency-versions-check-maven-plugin:3.2.0:check (basepom.default) on project kas-broker-plugins: Version conflict in direct dependencies detected!

One more thing, typo: Mockiot -> Mockito :)


View a dxf file

Hi,
I'm wondering if and how this library can be used to render and display a DXF file in a GUI.
More specifically I'm interested in using this lib in a LabVIEW application, so I'd need a .NET control, which doesn't exist in the latest binary I downloaded. My question is if the library already has the ability to render a DXF file and thus a .NET  control can be created quite easily.
Thank you in advance

netDxf do not provide any kind of functionality to display the contents of the a DXF, you will need to code it yourself.


BaseMetricObserver.updateImpl shouldn't be public

If I understood well the purpose of BaseMetricObserver.updateImpl method, it should not be public, since it's not meant to be called directly but through BaseMetricObserver.update method call.
https://github.com/Netflix/servo/blob/master/servo-core/src/main/java/com/netflix/servo/publish/BaseMetricObserver.java#L62

Agreed in general, but it looks like there are some internal use-cases that rely on it being public at this time and we are not planning to change the public server APIs. In general for improvements will be focused on spectator.


Error: URLs requested via Http on the server must be absolute.

EDIT: nevermind, just use ngx-universal, since I'm using Universal (hope I solve the Cannot resolve 'fs' module thing).

Writing this
export function HttpLoaderFactory(http: HttpClient) {
	return new TranslateHttpLoader(http, './assets/i18n/', '.json');
}

rather than this
export function HttpLoaderFactory(http: HttpClient) {
	return new TranslateHttpLoader(http, 'http://localhost:8000/assets/i18n/', '.json');
}

leads to Error: Uncaught (in promise). It would be nice to have an automatic way to retrieve the origin. Or I'm missing something.

The actual error spit by Universal is:
Error: URLs requested via Http on the server must be absolute. URL: /assets/i18n/en.json

ngx-universal (UniversalTranslateLoader) won't work for Angular 5.  See https://github.com/fulls1z3/ngx-translate/issues/44.

As per the docs https://angular.io/guide/universal#using-absolute-urls-for-http-data-requests-on-the-server, it says-

If you are using one of the @nguniversal/*-engine packages (such as @nguniversal/express-engine), this is taken care for you automatically. You don't need to do anything to make relative URLs work on the server.

But this is not working (in app.module.ts)-
export function HttpLoaderFactory(http: HttpClient): TranslateHttpLoader {
    return new TranslateHttpLoader(http);
}

TranslateModule.forRoot({
    loader: {
        provide: TranslateLoader,
        useFactory: HttpLoaderFactory,
        deps: [HttpClient]
    },
}),

Here is the error-
ERROR NetworkError
    at XMLHttpRequest3.send (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2445465)
    at Observable2._subscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:336270)
    at Observable2._trySubscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2091056)
    at Observable2.subscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2090838)
    at scheduleTask (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:4334513)
    at Observable2._subscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:4335116)
    at Observable2._trySubscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2091056)
    at Observable2.subscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2090838)
    at innerSubscribe (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2111774)
    at MergeMapSubscriber._innerSub (/Users/shashank/Projects/bharat-post/web-app/dist/web-app/server/main.js:1:2141009)


SQLX decoding doesn't support u64

Bug Description

It seems sqlx doesnt support  decoding into a u64
Minimal Reproduction
// Note Id::new() takes in a u64

struct Lobby {
    guild_id: Id<GuildMarker>,
    channel_id: Id<ChannelMarker>,
    category_id: Id<ChannelMarker>,
}

impl FromRow<'_, SqliteRow> for Lobby {
    fn from_row(row: &SqliteRow) -> Result<Self, sqlx::Error> {
        Ok(Self{
            guild_id: Id::new(row.try_get("guild_id")?),
            channel_id: Id::new(row.try_get("channel_id")?),
            category_id: Id::new(row.try_get("category_id")?)
        })
    }
}

Info

SQLx version: 0.6.2
SQLx features enabled:  tokio_,sqlite
Database server and version: SQLite
Operating system: windows 11
rustc --version: rustc 1.66.0 (69f9c33d7 2022-12-12)

sqlite doesn't have a database field that is u64, please look here
https://github.com/launchbadge/sqlx/issues/2252#issuecomment-1364273908

i thought it does support uint 64 due to reading this doc
https://www.sqlite.org/c3ref/int64.html

That documentation only describes the C interface with might use an uint64 for some stuff. That doesn't mean it can store an uint64, you need to read this https://www.sqlite.org/datatype3.html
As you can see sqlite has basically only for database types:
TEXT
NUMERIC
INTEGER
REAL
BLOB
In this case we are interested in INTEGER with is as the docs states "The value is a signed integer, stored in 0, 1, 2, 3, 4, 6, or 8 bytes depending on the magnitude of the value"
And that is all you get. If you type anything like "UNSIGNED BIG INT" when creating the table it is ignored and sqlite just makes an INTEGER


Add functionality to Upvote & Downvote Questions

Right now we do have functionality to up-vote or down-vote Answers in Q&A section. Do you think it would be a good idea to allow users to vote on Questions as well?
I can take a lead on this feature request if we agree that its worthy to implement this feature.

Hi @sidharthshah I think it does. There is always value into giving wider reach to a good question. We certainly will appreciate a lot your help if you're interested.

@sebastian-code let me work on this I will submit a PR once I am ready

Thanks a lot @sidharthshah

Is this issue still open for collaboration?

Hi @arpit1997, yes, it is; if you have a PR for it, please feel free to submit it.

I will have a look at it, understand the code and send a PR as soon as I completes it. I hope it does not take much time.

Merged de6fc13


router.back empty page query object

Hey Vladimir, loving the product so far!
I'm running into an issue where I load several pages with:
view.router.load(
{
	content: pageContent,
	query: {foo: bar, ...}
}

This works fine, but I have an issue when using router.back.  Imagine I have the following setup:
----------       ----------       ----------
|        |       |        |       |        |
| Page 1 |  ==>  | Page 2 |  ==>  | Page 3 |
|        |       |        |       |        |
----------       ----------       ----------

When router.back is called (or the "back" toolbar link pressed) when Page 3 is active, the onPageInit event is called for Page 1, but page.query is an empty object when I would expect it to be the query with which the page was initially loaded.  Am I missing something or could this be a bug?  I need a way to get data to a page that is loaded as the result of a router.back call.
Thanks,
Peter

Oh man, kudos to you! I guess you spent some time by making this cool picture with pages :+1:
Yes, in this case page query will be lost, because F7 doesn't save/store page data in this case, because when you on Page 3 then Page 1 will be removed from Dom and memory.
If you don't have too deep navigation, then i can recommend you to enable domCache to keep previous pages in Dom, and its data/state will be safe

haha thanks.  I managed to work around it by reorganizing my class structure.

Hi Vladimir.
I have the same issue.
I enabled domChache when I was creating this view
app.addView('.my-view', {
    dynamicNavbar: true,
    domCache: true
});

And navigate between pages inside current view using this:
app.getCurrentView().router.load({
    content: pageContent,
    query: {foo: bar, ...}
});

But when I navigate back using element with "back" class, old pages do not keep their data (I mean query data).
$(document).on('pageBeforeInit', function (e) {
    console.log(e.detail.page.query)); // is empty when I navigate back
});

Can you suggest something?
Thank you.

@fernap3 or can suggest some workaround that you made?

Hey @sanks, I ended up persisting the data in an object outside of Framework7 so I didn't need to use they page query object.

@sanks it was fixed here https://github.com/nolimits4web/Framework7/issues/674

@nolimits4web I built dev version but it still not works.

@sanks then i need detailed steps on how to replicate it. Also you wrote that you use pageBeforeInit - this callback is useless with domCache for "back" pages, if so, then change it to pageReInit

@nolimits4web thank you, pageReinit helps.


Runner to use new rules engine for variants

What is the context of this PR?
This PR allows the new rules engine to be used with question and content variants
How to review
Using test_variants_question.json, test_variants_content.json, test_variants_first_item_in_list.json Make sure everything works as expected
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

There is a bug in PathFinder._remove_rule_answers when changing old answer, as it is expecting old rules structure.
Steps to reproduce:

Launch a new variant question schema
Who is this questionnaire about?: Enter some name
Are you <name>?: Yes
Age?: 20 then save and continue
Click browser back until Are you <name>?
Change answer to No, I am answering on their behalf
<name> is over 16?: No
You should now get a 500 error.

Good spot, must be a gap in our tests

There is a bug in PathFinder._remove_rule_answers when changing old answer, as it is expecting old rules structure.
Steps to reproduce:

Launch a new variant question schema
Who is this questionnaire about?: Enter some name
Are you <name>?: Yes
Age?: 20 then save and continue
Click browser back until Are you <name>?
Change answer to No, I am answering on their behalf
<name> is over 16?: No
You should now get a 500 error.

Good spot, although is this specific to variants?

As discussed, this isn't directly related to variants but has highlighted an issue that has been generate by the new routing engine. I new card will be raised


How to debug server side errors?

I have a particular page, that works OK when it rendered by the client. But when I trying to render it server-side, it only returns a blank page with Not Found caption. In the console or terminal there is no exceptions or errors.

This is a good point. I've tried few thing but no luck so far. It's difficult to debug it with all the babel stuff source mapping configured in the IDE (I use WebStorm). These days I'm very busy so that I had no time moving forward with this project, but I'll let you know if any progress happens. Or feel free to contribute. 👍

Had the same thing, adding an try catch to the renderToString part showed me the error

Thats a good and simple solution - I like it - but I was thinking of something like setting up a proper debugging tool that allows us to use breakpoints in the IDE.
I'll adopt @TriPSs idea for now because it's way better then nothing. 👍

I'm gonna close this issue for now.


Fix signed/unsigned mismatch in match.c

I'd like to keep this one on hold for a little while, since I am doing some research in the same code areas and this does not fix any real bugs.

I don't mind... sign mismatches only avoid overflow in rare cases when a lot of data is accessed. I just separated this from another patch I was working on.

NOTE: Commit a73defc28c29abcfefdebcf995100f20ba528f01 touches same file...


037A YPOGEGRAMMENI

should this one follow the AGD list like: iotasubscript (see commit https://github.com/LettError/glyphNameFormatter/commit/72cf2ae48e94a58b199f0272df5079d264e87557)
or be set as iotalenissubscript or just ypogegrammeni
this is creating an conflict within the same script with iotalenis 1F30 GREEK SMALL LETTER IOTA WITH PSILI

double see https://github.com/LettError/glyphNameFormatter/issues/6


Changes Unknown when pulling 2b5e4ee0c0f2a99f3f62f0c4798130b9f3c61be8 on ginoledesma:use-super into ** on falconry:master**.

Changes Unknown when pulling 2b5e4ee0c0f2a99f3f62f0c4798130b9f3c61be8 on ginoledesma:use-super into ** on falconry:master**.

Changes Unknown when pulling 2b5e4ee0c0f2a99f3f62f0c4798130b9f3c61be8 on ginoledesma:use-super into ** on falconry:master**.

Changes Unknown when pulling 2b5e4ee0c0f2a99f3f62f0c4798130b9f3c61be8 on ginoledesma:use-super into ** on falconry:master**.

Changes Unknown when pulling 2b5e4ee0c0f2a99f3f62f0c4798130b9f3c61be8 on ginoledesma:use-super into ** on falconry:master**.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Coverage remained the same at 100.0% when pulling 3a77ec34c40e0c3e4bae6f3c9a09ba3b5419853e on ginoledesma:use-super into 62f63a2ec384a804b3005397c4ba54311bd77e06 on falconry:master.

Nice work, thanks!
LGTM


Can the function be modified so that deleted nodes can be added to the cluster again after RemoveData is called?

Can the function be modified so that deleted nodes can be added to the cluster again after RemoveData is called?

No.
Once data belong to that replica is deleted after calling RemoveData, you should never try to add that node back to the shard. In fact, extra feature should be added to detect such operation that can corrupt your shard.


feat: Add One Click Trading Feature Flag and Introduction Component

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Add introduction component and include it in wallet select modal
Add feature flag for one-click trading
Hide 'What is a wallet' tutorial when user has more than one wallet

Testing and Verifying

[ ] Should display 'What is a wallet' tutorial when users have no wallets installed (test in incognito)
[ ] Should display 1CT introduction (including translations) now wallet select when users have wallets

@DavideSegullo @jonator Thank you for the feedback! Addressed this on my latest commit


Fix target mask at inference time

I'm still working on getting the model to sample correctly, but I noticed a couple of issues while debugging:

The passed in conditioning + target duration was ignored, so the model never generated outside of the conditioning sequence without the duration predictor in place.
The target mask needs to exclude the conditioning, like what's done at training time with the random span mask.

I also made a fast path when CFG is disabled so we don't compute and throw away the prediction with no conditioning.

@lucasnewman thanks for catching the duration bug! :bug:
the CFG change also makes sense, forgot about that :facepalm:
so the cond_mask refers to the prefix conditioning length, while the mask is the self attention mask that should encompass both the condition as well as the target sequence that follows (cond sequence length + target length == duration length)
the way you have it the transformer would not be able to attend to the condition

Ah, ok, my mistake! You can close this if you want!

@lucasnewman the other two changes look good! let me merge those in :pray:


Travis CI

Hi @remen ,
Can you please enable the build of this repo in https://travis-ci.org?
Or perhaps you could create a new open-source organization and that way we could both retain ownership of this project and control these things?
Thanks,

Done. Moved the repository to https://github.com/forkbreak/fork_break (underscores are not allowed in github orgs)

:+1:


plugins should have a description that can be used by interfaces

It'd be nice to have short descriptions that could be used as help text. For example, here:

Related to qiime2/q2cli#16

It would be useful to have short and long descriptions, similar to actions, so that the CLI and other interfaces can choose how much detail to display. How about adding two new keyword arguments (to preserve backwards-compatibility): short_description=None and description=None ? short_description would be a single short sentence like what's displayed in your screenshot, and description is longer text that is displayed when passing --help to a plugin.

I like it.

So just a brief update on progress, I've set up my dev environment and read around 40% of the code from the 5 repositories.  I've added short_description and description to plugin.py with =None in the params so (as you suggested) we keep backward compatibility.  I'll do some tests and make sure nothing's broken but I mean I really see no reason why this would break anything.
It seems like the logical next step is to incorporate description and short_description into the code in appropriate places.  I can start by throwing in an extra argument to print the short_description tab'd across from the plugin name in the output from --help per @gregcaporaso's suggestion.  Are there any other particular places you can think of where these two values would definitely be worth printing?  Should I just read the code some more and see if I find anything?  Or should I just add the short_description print statement to the --help output as suggested, test to make sure nothing's broken, PR and call it a day?

Thanks for working on this @maxvonhippel! I'd start by submitting a PR to this repo (the QIIME 2 framework) that includes the new short_description and description API and some unit tests. I can take the lead on reviewing and merging that PR.
After that, it'd be great to hook up to q2cli so that the short description is displayed when running qiime --help (like in @gregcaporaso's screenshot), and the long description is displayed when running qiime <plugin-name> --help. To test that it's working, I recommend submitting a PR to q2-feature-table to give the plugin a description, which should become visible from the command line. Let me know if you need some description text for that plugin.
Note: while working on the CLI, you'll need to run qiime dev refresh-cache to see your changes take effect. More details on that in the note towards the bottom of this section.
Does that all sound good to you?

@jairideout yeah absolutely, thanks!  I'll get on it.

This is my first time working with unit tests in Python, so it's certainly possible I may have messed something up.  That said, I ran all the unit tests in verbose on my machine in the virtual env with the plugins loaded per the instructions I got in the command line and it appears to be working:
....
. ok
test_description_text (qiime2.plugin.tests.test_plugin.TestPlugin) ... ok
test_methods (qiime2.plugin.tests.test_plugin.TestPlugin) ... ok
test_name (qiime2.plugin.tests.test_plugin.TestPlugin) ... ok
test_package (qiime2.plugin.tests.test_plugin.TestPlugin) ... ok
test_short_description_text (qiime2.plugin.tests.test_plugin.TestPlugin) ... ok

If I didn't mess anything up and this works, let me know and I'll get started on hooking up q2cli and any other cool integrations we can do.  I wonder if these values could also be used for automated documentation processes?  Could be cool to work that in, if you have something like that already in place.
I'll be off-and-on busy over the next week.  I have big exams Monday (math 401B) and Thursday (math 464).  But when I need a break from studying I can do some coding, so let me know what to fix or work on and I'll work on it when I can!

Thanks @maxvonhippel! I just reviewed your PR (#222), it looks great. See my review there for specific comments.
Your tests are working just fine -- at the bottom of the page on your PR, you can see the status of the tests that are run by Travis-CI. Those tests are automatically run every time you commit and push to the branch you opened the PR from. If the status is green and says "All checks have passed", you're good to go! If tests are failing, click "Show all checks" and then "Details" next to the Travis-CI line. That'll show you the output from the tests that were run by Travis.
To see what Travis is running, check out this section of the .travis.yml file that's in the root of the repo. You can run those commands locally to confirm that the tests are passing.

I wonder if these values could also be used for automated documentation processes? Could be cool to work that in, if you have something like that already in place.

Funny you mentioned that -- the QIIME 2 docs repo has a Sphinx extension that auto-generates plugin docs! You can see what the rendered docs look like here. It'd be great to hook up these descriptions there too -- your call whether you want to work on q2cli first or the docs. For the docs, I'm thinking we'd want the "long" description in the section listing version, citation text, etc. (e.g. check out this page). For short description, it'd fit nicely in the title heading of that page (e.g. "diversity: short_description", or just "diversity" if there isn't a short description).
Let me know if you have any questions about these integrations and thanks again for working on all of this! No worries if you're busy with school and can't get to this stuff right away.

@jairideout awesome!  I love ReStructuredText - don't know it well enough yet, but I've been very impressed with it in my time using it on BASE9 (another project).
I think I should probably work on q2cli first - that'll help me get a better handle on how all of the various parts of the code talk to each other and what the terminology ("module", "template", "visualizer", etc.) means.  Then I'll be in a more solid spot to give the docs a shot, since I'll actually understand what it is that I'm documenting.

All sounds good, thanks @maxvonhippel!


SecurityError: OffscreenCanvas.convertToBlob: Cannot get blob from write-only canvas.

When using the onDownloadCropClick code from the demo i am running into a SecurityError.
After using the ctx.drawImage() function, the OffscreenCanvas becomes write-only.
Anyone maybe having a similar problem?

found the answer. my image was from another domain and so the img tag needed crossOrigin='anonymous'


utilizing wq_fd together with IORING_SETUP_ATTACH_WQ returns EINVAL

We are trying to utilize multiple threads to submit and reap i/o events. As suggested we initialize the first ring as normal and it works. But when we set the above on the second uring it returns error EINVAL. Does this feature even work at the moment?
#define IO_URING_DEPTH 4
struct io_uring_params params;
memset(&params, 0, sizeof(params));
params.flags |= IORING_SETUP_SQPOLL;
params.sq_thread_idle = __SQ_THRD_IDLE;
if(__first_ring_fd_set) {
/* this allows multiple user io_urings to utilize the same async
* backend. Thus it should utilize the same SQLPOLL thread as well */
params.wq_fd = cntxt->ring.ring_fd;
params.flags = IORING_SETUP_ATTACH_WQ;
}
if((rc = io_uring_queue_init_params(IO_URING_QDEPTH, &cntxt->ring, &params))!=0) {
rc = -1;
goto cleanup;
}
if((rc = io_uring_register_files(&cntxt->ring, &fd, 1))!=0) {
rc = -1;
goto cleanup;
}

What kernel? And are the rings created from the same task?

Thanks for the response, This is being done in linux 5.4 in AWS linux

5.4 is too old to support that feature. Basically -EINVAL here means "some flag you set in the ring setup is not supported".

What is the oldest linux version that supports that feature. We really need to get this working as soon as we can.

5.6 would be the oldest, but honestly if you're swapping kernels, I'd recommend using the latest stable. Even 5.6 is pretty old in that regard, that's almost 2 years ago.

We can utilize linux kernel-5.10. Hopefully it will work well enough there. Do we have to download a specific version of liburing code for that kernel? Or can we simply download the most recent version of liburing and it will know that we are working on kernel-5.10 and support features that it can?
Basically we create the first ring single threaded and the remaining rings are created concurrently... hopefully there is no issue with concurrently created/destroying rings. If there is we can isolate those code points as they are quite infrequent.
regards

5.10 seems reasonable. The current release of liburing will work fine, there are really no library dependencies for this feature.

thanks so much for the quick response. We will download the most recent version of liburing from Github ...and i will close this issue.

I spent the whole of last day upgrading to Linux 5.10 in AWS and i also downloaded the latest version of liburing from Github and rebuilt everything from scratch and i still get EINVAL when utilizing wq_fd together with IORING_SETUP_ATTACH_WQ.
The code is as indicated in the first comment. And this is what i see with uname -r:
5.10.50-44.132.amzn2.x86_64
We are doing this in docker containers. However the host is also the same kernel.  Is it possible that this is not working because we are utilizing docker containers?
Any help is greatly appreciated.

Could very well be that it's not working since it's in a docker container, I really have very little idea about that. I'd recommend cloning the liburing repo and building it all, then run test/shared-wq from there. That's a basic functionality test. If that works, then you are likely doing something wrong. If it doesn't, then likely docker is doing something funky.

It sounds strange that it should be an issue. We are also utilizing this in combination with  IORING_SETUP_SQPOLL.
Is it known to work together with IORING_SETUP_ATTACH_WQ?
Allright i will rebuild and run. So far we were running in Docker and without ATTACH_WQ some of our tests pass.

Yes, it'll work with SQPOLL, in fact liburing also has a test case for that, test/sq-poll-share

When rebuilding liburing i do run into the following problem:
io_uring_register.c:509:10: warning: implicit declaration of function 'memfd_create'; did you mean  timer_create'? [-Wimplicit-function-declaration]
memfd = memfd_create("uring-shmem-test", 0);
^~~~~~~~~~~~
timer_create
/tmp/cc91MvfX.o: In function `test_shmem':
after a bit of looking around it seems that including <syscall.h> together with utilizing SYS_memfd_create seems to make it work and additionally all the tests appear to pass, i ran
source ./test/runtests.sh
So the tests seem to pass ...
So inside the Docker container, all the tests pass. But when we run this in the product it fails with EINVAL. Is there anyway we can debug this further to see what may be causing it to return EINVAL. Is there some system call it is attempting which returns EINVAL. If so we can poke around to see what may be causing it to fail.

should we fall back on strace to see why it failed ... let me try that?

Looking at your example:
if(__first_ring_fd_set) {
/* this allows multiple user io_urings to utilize the same async
* backend. Thus it should utilize the same SQLPOLL thread as well */
params.wq_fd = cntxt->ring.ring_fd;
params.flags = IORING_SETUP_ATTACH_WQ;
}

if((rc = io_uring_queue_init_params(IO_URING_QDEPTH, &cntxt->ring, &params))!=0) {
rc = -1;
goto cleanup;
}

it's not clear to me what is being set in params->wq_fd - just to be clear, that one should be set to the ring fd of the original ring, and I can't tell if that's what you're doing as cnxt appears to be the new ring being registered. But it could just be that you're not including all of the code, and cnxt is allocated after that setting and it's the original ring to begin with?
In any case, if the test cases work, then you're doing something wrong in your code. -EINVAL is a horrible error code, but there's really nothing else that covers it. It'll get set for any invalid field in the params or setup.

io_uring_queue_init_params() boils down to a io_uring_setup(), and then later ring mmaps if that succeeds. If you strace, you're most likely seeing io_uring_setup() return -EINVAL due to some invalid parameter.

You are right about that one. That is a copy paste error on my part. Sorry to bother you. Let me try to address that one.

Thanks so much for your help. Im really sorry for taking up your time for what is my bug. Hopefully this fix will work. regards.

It's Friday, wasting a bit of time on this is healthier than drinking beer ;-)

We are hoping that the SQPOLL will setup one thread which will be polling for the sqes. And our application on the other hand has potentially 100 threads each one of which has a tiny seq, cqe ring of length 4/8. All threads are actively submitting and reaping ... hopefully we should be able to leverage the random i/o aspects of SSDs with that. If you are interested i can keep you posted on any performance reads we see.
We are looking at doing random i/o on massive databases. In the ballpark of a few TBs.
You have done a great service to the community by providing this library. Incidentally we also intend to look closely at SPDK which is utilizing a user mode polling model. In some usage scenarios that might offer some further advantages.

Definitely, keep us updated. If completion polling ends up being interesting as well, you can use IOPOLL with SQPOLL, fwiw.

Sorry i read some articles which indicate that IOPOLL should not be combined with SQPOLL ... i don't know what the reasoning was provided ...  if you search for io_uring samsung, performance on google you will find the pdf document that talks about it ... it also compares it with SPDK.
Don't know why they suggest not combining IOPOLL with SQPOLL.

Below is the link, they suggest do not utilize IOP with SQPOL
https://www.usenix.org/sites/default/files/conference/protected-files/vault20_slides_lund.pdf

After the fix i get the error: ENXIO ... probably some incorrect fd sent it ... let me check ...

Here is what i am finding about ENXIO:
The test sq-poll-share.c  does not utilize io_uring_register_files() to register fd.  From what i can see we have to utilize this API in order to register files for SQPOLL. If we do not utilize this API we can't even make a most basic test work. So there is something amiss.

Incidentally we have a separate test that utilizes SQPOLL and it works very well and we follow nearly the same steps.
Its when we try to go multi-threaded that we run into issues. At the moment i am getting error ENXIO when invoking
io_uring_queue_init_params().

closing the issue for now. It looks like sq-poll-share.c test is not working on linux. We have to understand that, and hopefully that will resolve this issue.


[v2] add option to restore to original replica count after ScaledObject's deletion

Adding option to enable restoring to the original replica count after ScaledObject is deleted. The default behavior is the same as the current one, ie. the replicas count remain on the same number.
Signed-off-by: Zbynek Roubalik<EMAIL_ADDRESS>
Provide a description of what has been changed
Checklist

[x] A PR is opened to update the documentation on https://github.com/kedacore/keda-docs
[x] Commits are signed with Developer Certificate of Origin (DCO)
[x] Tests have been added

Fixes #947

Documentation: https://github.com/kedacore/keda-docs/pull/219


XCode 11.5 SwiftUI build error. 'optional' requirements are an Objective-C compatibility feature; add '@objc'

this line : optional func kenBurnsViewDidFinishTransition(kenBurnsView: APKenBurnsView)
this error : optional' requirements are an Objective-C compatibility feature; add '@objc'

any update on this issue?


Add tests for taskboard deletion

like the title says, add tests for deleting taskboards.

https://github.com/KhunakornP/A-plus-management/commit/8145832


Screen variables?

Describe your problem here.
So, in the documentation all I can find is screen width and screen height
I'm having trouble setting the values of these, I'm trying to do this:
function onEvent(name, value1, value2)
    if name == "ScreenCustomisation" then
        SetPropertyFromClass('FlxG', width, value1)
        SetPropertyFromClass('FlxG', height, value2)
    end
end

btw this is what I'm entering

what isn't working?
also are there variables for screen x , screen y and screen rotation ?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Everything I've done has been within the mods folder.

wouldn't width and height be surrounded by quotes?

setPropertyFromClass('flixel.FlxG', 'width', value1);
If it doesn't work, oh well.


Jumbotron heading color not changing with text-color classes

Jumbotron Heading h1 does not pick color of text-danger because its color is inherit.

Coffee ceramic cup

Lorem Ipsum is simply dummy text of the printing and typesetting industry.Lorem Ipsum is simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's.

but now

use !important
On Mon, Feb 9, 2015 at 10:59 AM, Moin Akbar<EMAIL_ADDRESS>wrote:

Jumbotron Heading h1 does not pick color of text-danger because its color
is inherit.

Coffee ceramic cup
Lorem Ipsum is simply dummy text of the printing and typesetting
industry.Lorem Ipsum is simply dummy text of the printing and typesetting
industry. Lorem Ipsum has been the industry's.
but now

—
Reply to this email directly or view it on GitHub
https://github.com/twbs/bootstrap/issues/15768.

@mdo Would you like me to submit a PR for text-* variants on the h1 in the Jumobotron?
Also JSBin to demonstrate the issue for everyone:
http://jsbin.com/yavuniqexa/1/edit?html,css,output

We've rejected some stuff like this previously, instead referring folks to the general workaround/solution explained in the "Dealing with specificity" callout on http://getbootstrap.com/css/#helper-classes-colors

Yeah, I'm still going to defer to @cvrebert's comment here. Nothing we can do except add more bloat to the project at this point by adding more specific code.


Fix orderer/common/multichannel flake when removing channel

Type of change

Test update

Description
Set the cluster config in the RemoveChannel tests to avoid an endless loop due to ReplicationMaxRetries = 0.
Related issues
FAB-18251

All good the first time. Going to force-push to retrigger things.


Add logic for chat rooms

Implemented basic logic for:

creating the room
joining the room
finding roomId by client sid

We should approve this pr in order to merge it with stage and work on the improvement on other branches imo


Autocomplete on dot

Just trying out this IDE and it's great so far, but one thing I noticed was that upon writing, for example re. the only autocompletion is a useless one from autocomplete plus:

As soon as I enter any more characters it works as expected:

It would be great to get the ide-python dot completions at that point like in the Java one:

Thanks!

Just published v0.2.1 with a fix for it.

Record broken for speedy fixing!

Autocomplete plus still puts the useless one first of course! https://github.com/atom/autocomplete-plus/issues/767. Your upgrade works perfectly though.

Autocomplete plus still puts the useless one first of course!

That's a issue with atom-languageclient. I submitted a PR to fix this: https://github.com/atom/atom-languageclient/pull/111
Technically we could fix it inside ide-python but a I think a upstream fix is better.

Agreed, but I also think it would be great to let the user configure their priority in autocomplete plus!


Update README.md

Checkmarx AST – Scan Summary & Details – 0d2d46a7-052d-4d0c-9da2-8ccc1c08c365
No New Or Fixed Issues Found


Registry tutorial for new users

Describe the issue
This is for regular users for now.
Rational/Why
Additional Context
"Figma files are here. Let's go with the first mock-up, the one with the blue background."
This epic was originally created in the developer experience repo (#3065) and the below tickets were completed as part of this epic:

https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3069
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3076
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3063
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3062
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3057
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3064
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3068
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3066
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3067
https://app.zenhub.com/workspaces/platform-experience-5bb7c5ab4b5806bc2beb9d15/issues/gh/bcdevops/developer-experience/3065

Dependencies
This Epic will come into play after the team will develop a new frontend for the registry for the Public Cloud.
Tests
Definition of done/Acceptance Criteria
create a FIGMA file
when a user has no projects in the registry, I want a message that indicates that instead of a blank page.

Moved to the Platform Experience board


Parentheses around variable assignment inside @if directive toggled on each formatter run

Given the following code, the formatter toggles parentheses around the variable assignment in the @if every time it runs:
@if ($foo = $user->events()->first())

Test

@endif

So it switches back and forth between these:
@if ($foo = $user->events()->first())

@if (($foo = $user->events()->first()))

It seems to be related to chaining something on after a method call, because this is fine:
@if ($foo = $user->events())

But this also triggers this issue:
@if ($foo = $user->events()->first)

The parentheses aren't necessary, so I would expect it to remove them if they're there, or at least not add them.

Thanks for the report. Are you just using the defaults, or have you also configured it to use Pint to format PHP inside directives? Thanks!

Not using Pint, pretty much the defaults I think. Here's my Prettier config in case it's helpful but it's mostly not related to Blade:
    {
        "htmlWhitespaceSensitivity": "ignore",
        "printWidth": 140,
        "singleQuote": true,
        "tabWidth": 4,
        "importOrder": [
            "^@(?!/)(.*)$",
            "^@/(.*)$",
            "^[./]"
        ],
        "importOrderSortSpecifiers": true,
        "plugins": [
            "@trivago/prettier-plugin-sort-imports",
            "prettier-plugin-blade",
            "prettier-plugin-tailwindcss"
        ],
        "pluginSearchDirs": false,
        "overrides": [
            {
                "files": "*.yml",
                "options": {
                    "tabWidth": 2
                }
            },
            {
                "files": [
                    "*.blade.php"
                ],
                "options": {
                    "htmlWhitespaceSensitivity": "css",
                    "parser": "blade"
                }
            }
        ]
    }

That is very helpful, thank you!

Got the same issue for the following statements:
@php(($count = WC()->cart->get_cart_contents_count())) <->

@RobertBlaauwendraad  Thanks for those examples. I was able to reproduce with that input; I'll work on a fix (hopefully ready sometime this upcoming weekend) 🙌

Fix available in v<IP_ADDRESS>


Add line/column number for test failure

Component
Forge
Describe the feature you would like
When a test fails, it would be great if, in addition to what kind of failure (revert, expectation failed, etc.), it would also spit out the line/column in the test file that it failed on. This would match the norms in other testing suites I've used. I can read the trace in -vvv to get this but it's not as intuitive/quick.
Additional context
No response

Requires us to do a PC-IC mapping (expensive) like we do for debugging, but also somehow backtrack from the internal code of e.g. assertEq in DSTest (and others like Forge Std?) to the call site, so definitely non trivial

Rolling up into https://github.com/foundry-rs/foundry/issues/2328


Adding api-driver to focus-core (with pre-fetch transformations utility)

@c3dr0x Should be good enough for testing now


[Windows][ft 1.0 and 0.12] cannot restore variables, raise OutOfRangeError (see above for traceback): Read fewer bytes than requested

step to reproduce:
attached file (https://www.dropbox.com/s/7yglwzjx4tyoguz/save_restore_model.zip?dl=0)
is my source code
I am testing saver.save and saver.restore function under windows.
working envirment : Windows R2 Server 2012 x64,
python 3.5 x64, TF 1.0 and master daily build, working with virtualenv
I also tested TF 0.12

download source code save into C:\work\tensorflow001 ,
create folder mnist
download files
(t10k-images-idx3-ubyte.gz,
t10k-labels-idx1-ubyte.gz,
train-images-idx3-ubyte.gz,
train-labels-idx1-ubyte.gz) from http://yann.lecun.com/exdb/mnist/
and save into mist folder(why not using input_data from examples? because that is another bug...I think.)
create folder "model" for saving values
python save_restore_model.py
output as fallow

(venv643) C:\work\tensorflow001>python save_restore_model.py
Extracting mnist\train-images-idx3-ubyte.gz
Extracting mnist\train-labels-idx1-ubyte.gz
Extracting mnist\t10k-images-idx3-ubyte.gz
Extracting mnist\t10k-labels-idx1-ubyte.gz
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "BestSplits" device_type: "CPU"') for unknown op: BestSplits
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "CountExtremelyRandomStats" device_type: "CPU"') for unknown op: CountExtrem
elyRandomStats
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "FinishedNodes" device_type: "CPU"') for unknown op: FinishedNodes
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "GrowTree" device_type: "CPU"') for unknown op: GrowTree
E c:\tf_jenkins\home\workspace\release-

win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "SampleInputs" device_type: "CPU"') for unknown op: SampleInputs
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "ScatterAddNdim" device_type: "CPU"') for unknown op: ScatterAddNdim
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "TopNInsert" device_type: "CPU"') for unknown op: TopNInsert
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "TopNRemove" device_type: "CPU"') for unknown op: TopNRemove
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "TreePredictions" device_type: "CPU"') for unknown op: TreePredictions
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "UpdateFertileSlots" device_type: "CPU"') for unknown op: UpdateFertileSlots

E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "ReinterpretStringToFloat" device_type: "CPU"') for unknown op: ReinterpretS
tringToFloat
WARNING:tensorflow:From save_restore_model.py:66: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use `tf.global_variables_initializer` instead.
Starting 1st session...
Epoch: 0001 cost=<PHONE_NUMBER>73
Epoch: 0002 cost= 43.585798355
Epoch: 0003 cost= 27.564399517
First Optimization Finished!
Accuracy: 0.9124
Model saved in file: C:\work\tensorflow001\model\result
Starting 2nd session...
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested

W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:993] Out of range: Read fewer bytes than requested
Traceback (most recent call last):  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 1
022, in _do_call
    return fn(*args)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 1
004, in _run_fn
    status, run_metadata)
  File "c:\python35\Lib\contextlib.py", line 66, in __exit__
    next(self.gen)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\framework\errors_impl.py",
 line 469, in raise_exception_on_not_ok_status
    pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.OutOfRangeError: Read fewer bytes than requested
         [[Node: save/RestoreV2_3 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/t
ask:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_3/tensor_names, save/RestoreV2_3/shape_and_slices)]]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "save_restore_model.py", line 113, in <module>
    saver.restore(sess, model_path)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 1
439, in restore
    {self.saver_def.filename_tensor_name: save_path})
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 7
67, in run
    run_metadata_ptr)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 9
65, in _run
    feed_dict_string, options, run_metadata)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 1
015, in _do_run
    target_list, options, run_metadata)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\client\session.py", line 1
035, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: Read fewer bytes than requested
         [[Node: save/RestoreV2_3 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/t
ask:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_3/tensor_names, save/RestoreV2_3/shape_and_slices)]
]

Caused by op 'save/RestoreV2_3', defined at:
  File "save_restore_model.py", line 69, in <module>
    saver = tf.train.Saver()
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 1
051, in __init__
    self.build()
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 1
081, in build
    restore_sequentially=self._restore_sequentially)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 6
75, in build
    restore_sequentially, reshape)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 4
02, in _AddRestoreOps
    tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\training\saver.py", line 2
42, in restore_op
    [spec.tensor.dtype])[0])
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\ops\gen_io_ops.py", line 6
68, in restore_v2
    dtypes=dtypes, name=name)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\framework\op_def_library.p
y", line 763, in apply_op
    op_def=op_def)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\framework\ops.py", line 23
92, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "C:\work\tensorflow001\venv643\lib\site-packages\tensorflow\python\framework\ops.py", line 12
64, in __init__
    self._traceback = _extract_stack()

OutOfRangeError (see above for traceback): Read fewer bytes than requested
         [[Node: save/RestoreV2_3 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/t
ask:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_3/tensor_names, save/RestoreV2_3/shape_and_slices)]

expect result
why fallow message shows up ?
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "BestSplits" device_type: "CPU"') for unknown op:
most import is when I using saver.restore, it  always show fallowing error message
OutOfRangeError (see above for traceback): Read fewer bytes than requested
         [[Node: save/RestoreV2_3 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/t
ask:0/cpu:0"](_recv_save/Const_0, save/RestoreV2_3/tensor_names, save/RestoreV2_3/shape_and_slices)]

why I cannot download example automatic by using
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("mnist", one_hot=True)

Thanks for reaching out and we're sorry to hear about the issue you're facing. Could you please provide a minimal inline code example demonstrating the bug and showing how to reproduce? Downloading a zip file and troubleshooting another app is not a level of support we're able to offer.

Thanks your response,
fallow is my code:
please download checkpoint that created by TensorFlow saver
https://www.dropbox.com/s/v5xl1dzssi58opf/model.zip?dl=0
copy it to "C:\work\tensorflow001\model"
in model will have 5 files, checkpoint, result, result.index.....
when TensorFlow restore checkpoint will crash.
from __future__ import print_function

# Import MNIST data
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("mnist", one_hot=True)

import tensorflow as tf

# Parameters
learning_rate = 0.001
batch_size = 100
display_step = 1
model_path = u"C:\\work\\tensorflow001\\model\\result"

# Initializing the variables
init = tf.initialize_all_variables()

# 'Saver' op to save and restore all the variables
saver = tf.train.Saver()

# Running a new session
print("Starting 2nd session...")

with tf.Session() as sess:
    # Initialize variables
    sess.run(init)

    # Restore model weights from previously saved model
    saver.restore(sess, model_path)

Hmm if you insert a line into your script to trigger the lazy loading of the contrib library, e.g. _ = tf.contrib.tensor_forest then does it work?

thanks your response,
I am insert that line of code, please check it out,
if that insert to right place, the result still crash.
from __future__ import print_function

# Import MNIST data
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("mnist", one_hot=True)

import tensorflow as tf

# Parameters
learning_rate = 0.001
batch_size = 100
display_step = 1
model_path = u"C:\\work\\tensorflow01\\model\\result"

# Initializing the variables
init = tf.initialize_all_variables()

# 'Saver' op to save and restore all the variables
saver = tf.train.Saver()

# Running a new session
print("Starting 2nd session...")
_ = tf.contrib.tensor_forest
with tf.Session() as sess:
    # Initialize variables
    sess.run(init)

    # Restore model weights from previously saved model
    saver.restore(sess, model_path)

Can you put that line right after the import tensorflow as tf line? I ask because your stack trace indicates the saver = ... line is what crashes, and the code above inserted it after that.
I'm pretty confident that my suggestion will fix your problem. This is the result of a known issue we're planning to fix eventually, where the build graph and op loading needs to be smarter about loading libraries. It's a hard problem to solve.
If your issue is not solved by my suggestion, let me know and I'll re-open.

Thanks your answer, but it not work, still crash
the fallow is my code:
can you help me to verify, may I miss understand your meaning, insert the code to wrong place.
from __future__ import print_function

# Import MNIST data
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("mnist", one_hot=True)

import tensorflow as tf

# Parameters
learning_rate = 0.001
batch_size = 100
display_step = 1
model_path = u"C:\\work\\tensorflow01\\model\\result"

# Initializing the variables
init = tf.initialize_all_variables()

# 'Saver' op to save and restore all the variables
saver = tf.train.Saver()

# Running a new session
print("Starting 2nd session...")
_ = tf.contrib.tensor_forest  #here is that code
with tf.Session() as sess:
    # Initialize variables
    sess.run(init)

    # Restore model weights from previously saved model
    saver.restore(sess, model_path)

Can you try this?
from __future__ import print_function

# Import MNIST data
from tensorflow.examples.tutorials.mnist import input_data
mnist = input_data.read_data_sets("mnist", one_hot=True)

import tensorflow as tf
_ = tf.contrib.tensor_forest  #here is that code

# Parameters
learning_rate = 0.001
batch_size = 100
display_step = 1
model_path = u"C:\\work\\tensorflow01\\model\\result"

# Initializing the variables
init = tf.initialize_all_variables()

# 'Saver' op to save and restore all the variables
saver = tf.train.Saver()

# Running a new session
print("Starting 2nd session...")
with tf.Session() as sess:
    # Initialize variables
    sess.run(init)

    # Restore model weights from previously saved model
    saver.restore(sess, model_path)

It still crash
do you have any idea ?
W c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel
.cc:975] Out of range: Read fewer bytes than requested
Traceback (most recent call last):
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 102
1, in _do_call
    return fn(*args)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 100
3, in _run_fn
    status, run_metadata)
  File "c:\program files\anaconda3\Lib\contextlib.py", line 66, in __exit__
    next(self.gen)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\framework\errors_impl.py", l
ine 469, in raise_exception_on_not_ok_status
    pywrap_tensorflow.TF_GetCode(status))
tensorflow.python.framework.errors_impl.OutOfRangeError: Read fewer bytes than requested
         [[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/tas
k:0/cpu:0"](_recv_save/Const_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "save_restore_model.py", line 114, in <module>
    saver.restore(sess, model_path)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 138
8, in restore
    {self.saver_def.filename_tensor_name: save_path})
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 766
, in run
    run_metadata_ptr)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 964
, in _run
    feed_dict_string, options, run_metadata)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 101
4, in _do_run
    target_list, options, run_metadata)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\client\session.py", line 103
4, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: Read fewer bytes than requested
         [[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/tas
k:0/cpu:0"](_recv_save/Const_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

Caused by op 'save/RestoreV2', defined at:
  File "save_restore_model.py", line 69, in <module>
    saver = tf.train.Saver()
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 100
0, in __init__
    self.build()
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 103
0, in build
    restore_sequentially=self._restore_sequentially)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 624
, in build
    restore_sequentially, reshape)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 361
, in _AddRestoreOps
    tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\training\saver.py", line 200
, in restore_op
    [spec.tensor.dtype])[0])
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\ops\gen_io_ops.py", line 441
, in restore_v2
    dtypes=dtypes, name=name)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\framework\op_def_library.py"
, line 759, in apply_op
    op_def=op_def)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\framework\ops.py", line 2240
, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "C:\work\tensorflow01\venv64\lib\site-packages\tensorflow\python\framework\ops.py", line 1128
, in __init__
    self._traceback = _extract_stack()

OutOfRangeError (see above for traceback): Read fewer bytes than requested
         [[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/tas
k:0/cpu:0"](_recv_save/Const_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

Oh now the problem seems obvious to me. Windows support is still brand new. We haven't had a chance yet to make contrib Windows compatible. It's only possible to use the core TensorFlow codebase right now. See: https://github.com/tensorflow/tensorflow/blob/d5062e8/tensorflow/python/BUILD#L81 This means it's not possible at the moment to load a model on Windows that uses ops from contrib.
Since I don't see an issue tracking this, I'm going to re-open and rename this one.

@jart Can you explain what this issue has to do with tf.contrib? I don't see any reference to tf.contrib in the original code, and—while you're right that the contrib libraries aren't fully supported on Windows—I can't see how that's related to @alarmz's original problem.

In the original post, the following error message is cited:

why fallow message shows up ?
E c:\tf_jenkins\home\workspace\release-win\device\cpu\os\windows\tensorflow\core\framework\op_kernel.cc:943] OpKernel ('op: "BestSplits" device_type: "CPU"') for unknown op:

That op is defined in tensorflow/contrib/tensor_forest/core/ops/best_splits_op.cc.

Ah I see. We should definitely fix this so that TensorForest works on Windows, but I think this log message (which should really be a warning or a fatal error) is a red herring. It looks like the real problem @alarmz is facing is in tf.train.Saver code and/or its related ops. Perhaps @concretevitamin has seen this before?

(PR #6908 should fix the error message and make much of tf.contrib.tensor_forest work on Windows.)

@alarmz can you confirm that tf.contrib.tensor_forest works after that PR?

Closing this issue, since tf.contrib.tensor_forest has (as far as I know) been working with tests passing on Windows for some time now. Feel free to open a new issue if there are other parts of tf.contrib for which you'd like to see us prioritize support!


Chaos test the cluster

explore some cluster resilience testing
A tool could be something like this https://github.com/chaostoolkit/chaostoolkit

Whatever decisions and learning come out of this are going to be applicable to the other beamline repositories

note: this tool is more suited to aws and isn't that well-developed
https://chaostoolkit.org/reference/tutorials/containerising/

potential alternatives

https://chaos-mesh.org/docs/production-installation-using-helm/

raised a chaos-mesh ticket with the cloud team about the namespace creation

moved to looking into a more restricted tool - https://github.com/asobti/kube-monkey

with a helm chart https://github.com/asobti/kube-monkey/blob/master/helm/kubemonkey/README.md

working on it

tracking the cloud aspect here https://jira.diamond.ac.uk/servicedesk/customer/portal/2/SCHD-6072