Client and Server certificates on Ubuntu

Trying to generate client and server certificates, it fails with an error message:
Failed to load config file: {"code":5200,"message":"failed to unmarshal configuration: invalid character 'k' after array element"}Failed to parse input: unexpected end of JSON input
Could you please assist?

Sorry for the delay in responding to your assistance.
Am following the document, the area where i got stuck was: Generate the
admin client certificate and private key:
cfssl gencert
-ca=ca.pem
-ca-key=ca-key.pem
-config=ca-config.json
-profile=kubernetes
admin-csr.json | cfssljson -bare admin
And the operating system details:
Distributor ID:    Ubuntu
Description:    Ubuntu 16.04.3 LTS
Release:    16.04
Codename:    xenial
Let me know if you need any additional information, will be happy to
provide and continue with the tutorial. Thank you for a great tutorial.
Regards,
Sri
On Wed, Jan 31, 2018 at 1:27 PM, Rahul Mishra<EMAIL_ADDRESS>wrote:

Could you please post the command you ran, the CSR, config files you used?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/kubernetes-the-hard-way/issues/306#issuecomment-362025423,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiUOVwOuu0psgIxcqt9uCxPkEkbGGE-xks5tQLCpgaJpZM4RzVr_
.


[Issue] "apply to metrics" field is "blank" after saving with "undefined" after performing the transformation such as "Merge" and "Group By"

I performed transformations such as "Merge" and "Group By".
But I can only select "undefined" on "apply to metrics" menu of flowcharting.
It seems to work well when I select "undefined" and procced.
But, When reloading the page after saving, it encounters "Blank" on "apply to metrics" field and does not work.
If you know a solution to this issue, please share it.
■ Json File Information when saving
.
.
.
"rulesData": {
"rulesData": [
{
"order": 1,
"pattern": "/.*/",
"unit": "short",
"type": "number",
"metricType": "table",
"alias": "AAA",
"column": "spent-time(reassigned)",
"hidden": false,
"aggregation": "avg",
"decimals": 2,
"colors": [
"rgba(245, 54, 54, 0.9)",
"rgba(237, 129, 40, 0.89)",
"rgba(50, 172, 45, 0.97)"
],
"reduce": false,
"dateFormat": "YYYY-MM-DD HH:mm:ss",
"thresholds": [
50,
80
],
.
.
.
■ Json File Information when reloading
--> Unexpected "refId information" was added ("refId": "A") after reolading and "reduce" value changed from false to true
.
.
.
"rulesData": {
"rulesData": [
{
"order": 1,
"pattern": "/.*/",
"unit": "short",
"type": "number",
"metricType": "table",
"alias": "AAA",
"refId": "A",
"column": "spent-time(reassigned)",
"hidden": false,
"aggregation": "avg",
"decimals": 2,
"colors": [
"rgba(245, 54, 54, 0.9)",
"rgba(237, 129, 40, 0.89)",
"rgba(50, 172, 45, 0.97)"
],
"reduce": true,
"dateFormat": "YYYY-MM-DD HH:mm:ss",
"thresholds": [
50,
80
],
.
.
.

My Grafana v7.3.7 (1e261642f4)
Flowcharting version - 0.9.0
Datasource - Json API

Grafana 8.1.2
Plugin 0.9.1
Firefox 78-esr
Same here. I'm getting a lot of this in FF's console :
GF ERROR : Metric => getName : Name is null
the datasource output is a table, not a timeseries.

probably a duplicate of https://github.com/algenty/grafana-flowcharting/issues/283


Simultaneous play record

Hello,
I would like to ask if it is possible to play the video while recording it. I would like to capture an event and while doing so, go back to one specific momento to review it.
Thank you in advance!

You have to do much by your own code.
DLABridging.framework does not provide any time shift feature. It simply passes CMSampleBuffer one each via callback to user application only. Audio and Video callback could be triggered in different time resolution.
If you want to do time shift while capture is running, you have to 1) deep copy sample buffer every time, and create two stream which 2)produces buffer stream for “time shift cache - delayed playback”, and 3)write sample buffer into captured movie file.
Yes it will be possible though, quite hard to handle delayed play on demand.


Zxing Net Mobile for Xamarin Forms Flash (torch) button can't be hidden?

I am using a custom button for torch on my app scanner page, but weird,
even i set "ShowFlashButton = false " in overlay but it still appear?
many thanks

Could you us some code how you create to View/Page and send in the options? What version of the nuget package are you using?

I have the same problem on both iOS and Android. The flash button is always displayed.
I'm using ZXing.Net.Mobile nuget 2.1.45 on XF<IP_ADDRESS>.
It does not matter wether I'm doing it in code or in XAML like:
<!-- Scanning top view -->
<zxing:ZXingScannerView x:Name = "scannerView" Grid.Row = "0" />
<zxing:ZXingDefaultOverlay ShowFlashButton = "false" TopText = "Hold your phone up to the barcode"  BottomText = "Scanning will happen automatically" Grid.Row = "0"/>

<!-- Delimiter -->
<BoxView Style="{StaticResource horizontalBoldDivider}" Grid.Row = "1" />

<!-- List of barcodes scanned -->
<local:StyledListView HasUnevenRows = "true" ItemsSource = "{Binding Parcels}" ItemTemplate = "{StaticResource parcelTemplate}" Grid.Row = "2"/>

I am having the same problem here. "ShowFlashButton = false" but it still appears on the overlay, any solutions/workarounds yet?

I have same problem in both android and IOS. Any suggestion to solve this issue?

Possible duplicate of #427

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Recipe Website

Issue Type:

[ ] Bug Report
[x] Feature Request
[ ] Documentaion

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

@DragonUncaged add more description of the issue.


updates to simple-untyped.k

@cos, @daejunpark please take a quick look.  Just trying to update it and minimize the diff to 3.6.

added a few more notes.  @daejunpark , probably the issues in this NOTES.md under simple untyped will apply to KOOL as well.

lgtm

lgtm


central-logging README installation needs version bump

installation says to use 0.1.0 but version is at 0.2.0

https://github.com/samsung-cnct/k2-charts/pull/62


cameFromInternalNavigation logic and search

Revisit the logic around this and search and remember that state seems to be maintained, not refreshed from scratch for each navigation.

Search a polling place, but don't choose it
Dismiss the search drawer
Tap on the search bar
Hit Back and observe you've been sent back to the map, not to a reset version of the search interface

I think all of our entry points into search need to send cameFromInternalNavigation as true or false for our logic to work.

Updated the navigation helpers to always send cameFromInternalNavigation, but we'll still run across this issue in the very specific use case where someone duplicates a tab and then tries to hit one of the 'Back' buttons. That's not typical behaviour, so we're safe to ignore it.

At the nub of this issue is that the state in react-router-dom gets persisted when you duplicate a tab, but the duplicated tab doesn't seem to get the history in a way that navigate(-1) will trigger the usual "just go back one page" behaviour that we trigger when cameFromInternalNavigation === true.


Import not working in TypeScript/CommonJs

I'm using typescript and commonjs module loading to try to bring iziToast (1.0.2-c) into my module/class. The import does not appear to be working and provides undefined at runtime. Weirdly the debugger tooltip of the iziToast variable in the chrome debugger tools suggest that the type has loaded correctly.
There is an iziToast_1 variable available that does provide the correct instance but I have to get to it via hacky and non-CSP compliant eval call.
import Failure from "./failure";
import iziToast from "iziToast";

export class Notify {
    private notifier: any;

    public constructor() {
        this.notifier = iziToast;

        if (!this.notifier) {
            // tslint:disable-next-line:no-eval
            this.notifier = eval("iziToast_1");
        }
    }
}

The only other thing I've been able to find is that iziToast_1 gets registered in the map file but does not exist anywhere else in compiled source (via Webpack) or anywhere in the node_modules directory.
I don't think this is to do with minification however because I find the same results on code that isn't minified. I also doubt that this is an issue with webpack as all other modules load fine.
Any idea what is going wrong here?

If I try to trick this by declaring iziToast_1 before the import in order to avoid the eval, iziToast_1 remains undefined and iziToast_2 is available with the correct instance.

The answer from @TheLarkInn is to import using import * as iziToast from "iziToast"


[FEATURE] PlantUML Processing - Parameter Support

Description
Instead of only providing a [Refresh]-Button it would be nice to influence the processing similar to some PlantUML Command Line features.

Run Simple Instructions directly without entering this code.

@startuml
instruction
@enduml

That can be achieved in the command line by calling:
java -jar plantuml.jar -instruction

So open a Pull Down Area that shows further buttons in the Preview Pane:
[Author], [Version], [Stdlib]

Run more complex instructions without needing to go to the command line.
Add addition Buttons, that use the PlantUML command line to provide
[Help],  [Language],  [Printfonts]

Process the the source code similar to using the commandline

Provide a Button [Process] besides [Refresh]
In the drop down area described before, provide a list of PlantUML Command Line parameters to select and may be enable to choose or enter a value (setting the standard parameters for Processing)
Show the resulting Command line to copy paste it to Git Bash or any other shell.

Stepwise you can support -nometadata, -charset, ... all that make sense for a single source file.
With different preview styles you can do this for
-teps, -teps:text, -thtml, -tlatex, -tlatex:nopreamble, -tpdf, -tpng, -tscxml, -tsvg, -ttxt, -tutxt, -tvdx, -txmi, ...
Solution
On Processing - different to Refresh - the PlantUML Processing ist used to deliver a result into the output area.
[View Code] might be an option to provide a button to show the source code of some of those (svg, txt, latex, ...) in notepad, if that makes sense.

Hi Philipp, well done so far.
There are two major reasons in case you only have N++ to edit the code, then

it would be nice to preview directly the SVG to be readable in any scaling
it would be nice to directly produce the same result as if the command line is used, just to avoid switching between n++ edit, shell process and browser preview (or what ever is needed).
Does that make sense to you?
Best regards
SieDa


Bad default location for REPORTS_ROOT

Current Behavior
Due to the way your current implementation references configuration files located in /etc/netbox/ instead of /opt/netbox/netbox/netbox/, when you browse to http://localhost/extras/reports/, the following is seen:

Yes, I know this can be changed in the netbox.env by setting a value for REPORTS_ROOT, however, in my opinion, this "default" is not a good idea.  I don't believe anyone is going to use the /etc/reports location to store custom python scripts specific to NetBox.
I'd like to propose using /etc/netbox/reports/ as an alternative, but I didn't want to go through the hassle of creating a PR if it wasn't going to be merged.
What are your thoughts one way or the other?
Expected Behavior
N/A
Debug Information
N/A

You're right, /etc/reports is a bad location for the reports. IMO this happened when we moved the 'main' configuration file to /etc/netbox and BASE_DIR on this line now expands to /etc rather than /opt/netbox/netbox.
I think /etc/netbox/reports would be a good location. But I want to make sure this works out when using a configuration managed by Kubernetes/OpenShift. And I'm not entirely sure it does in a nice way.
IMO the nice way would be to have two separate ConfigMaps, one for reports and one for configuration. And when the one for configuration is mapped to /etc/netbox, I'm not sure if we can map the one for reports to /etc/netbox/reports.
So the main configuration might have to be moved to somewhere else, e.g. /etc/netbox/config, so that config and reports can co-exist.
Or am I over-complicating matters and it's fine to have one ConfigMap for config and reports?
I'd love to hear your opinion on this, and @tobiasge's as well, as he initially suggested to move the configuration to /etc/netbox.

Hello,
at the moment we're not using reports but I also think /etc/netbox/reports would be a good location for that. But you're right. You can't map to /etc/netbox/reports when the configuration is mapped to /etc/netbox in Openshift.
So the proposal with mapping to /etc/netbox/config and /etc/netbox/reports is a good solution.
I won't be able to implement this now because I'm on holiday for the next weeks. If this can wait I will do the implementation.
@flipkill1985 Could you test this in our deployment?

@tobiasge Have you had the chance to look into this? Would one ConfigMap for config and reports be fine or do you think they should better be separated?

Hello,
I'm back now. I think it would be better to have them separated. This makes it possible for the user to also define a persistent volume in Openshift.


First version of Azure NetApp Files CLI for GA

Previously the Azure NetApp Files CLI was an extension only.
R4 standard of RP for GA
Replaces https://github.com/Azure/azure-cli/pull/9365 which was incorrectly applied to master.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

@tjprescott  Comments from command review incorporated in _help.py and code. The account set command was able to be removed by incorporating the subgroup commands as suggested.

@tjprescott  As I have seen every time, all tests involving volume creation, specifically vnet, fail. These were newly run locally in live mode and verified with a playback with all passing (there is also a temporary test,  a placeholder for a new test which I will clean up but I was keen to push the new changes for command and code review).

@tjprescott I have tidied the subgroup test so there is now a test for each of the two new subgroups. From my side then I believe I have addressed all comments from the review to this point. Just for info, I will be on vacation from Wednesday next week for two weeks. It would be great to get an agreed code base before then but if not I will be passing this on to one of my team.

@tjprescott made all the changes locally and will push once I get feedback from @limingu on the remaining item. Thanks.

All changes made I think.

I see failures here https://travis-ci.org/Azure/azure-cli/jobs/540766170. I'm not sure about the cause "FAIL: no_ids_for_list_commands".
There is guidance for fixing "no --ids in list commands" here:
https://github.com/Azure/azure-cli/blob/dev/doc/authoring_command_modules/authoring_commands.md#supporting-the-ids-parameter
It is at the very end of the section.

@leonardbf looks like you either need to re-record your tests or that you have randomness in your tests such that they won't play back.
There is guidance for fixing "no --ids in list commands" here:
https://github.com/Azure/azure-cli/blob/dev/doc/authoring_command_modules/authoring_commands.md#supporting-the-ids-parameter
It is at the very end of the section.

Yeah I have fixed the ids issue. I hadn't added the id_part=None. As for the randomness, I am aware that some of the tests use the random name generation offered but I have tested this before to see if this was the problem. The mount_target test for instance does not have any but still fails. Also they do not fail in record or playback locally, only in the azure pipeline so I'm really not sure how to fix this. It is always the vnet command that fails it seems and has done so in every version of the CLI (extension) so far.

As well as the possible issue with vnet API version (as emailed) I continue to get the naming failure in the help https://travis-ci.org/Azure/azure-cli/jobs/541009105. All have been removed except these 7 for account despite them working locally. Note that I changed the ordering for pool and volume but that did not work for account so there seems to be an additional consideration. I'm not certain what all the rules are that govern the correct order for this list.

Why is sql module affected by these changes?

@jaredmoo sql should not be affected. Perhaps there are some changes due to my merge from azure dev into my fork to pull latest due to failing tests (network version).

@tjprescott I've pulled from azure dev and my tests now pass. However there are other failures which seem to be outside of my changes.

Hi, Lenny is on a vacation and I am taking over this. Is anything that needs to be done on our end?

@tomasm9 you need to resolve the conflicts and get the CI to pass for us to be able to merge this.


Missing instructions on how to use v3 with Jest and ts-node

Reproduction
Well, just update the lib for a project that used Jest and ts-node.
Somewhat weird error messages start to appear in the output when running the tests:     SyntaxError: Cannot use import statement outside a module
Expected behavior
The library would work like before. Another option is to add instructions about how to configure the library with a standard NodeJs environment that uses Jest and TypeScript.
Screenshots

Your Environment

software
version

node-fetch
3

node
14

npm
6

Operating System
macOS

Additional context

Is this a duplicate of #1265?

Gettings the same problem with 3.0.0, since the project is a TypeScript project that is using ts-node to run - it transpiles everything either into CommonJS or ES2020 both of which cannot handle any kind of import of node-fetch v3.0.0, not a dynamic import, and not a regular import. Making node-fetch completely unusable.

@jimmywarting Not sure since #1265 is not specifically about ts-node which is the problem we are facing

ts-node has experimental support for ESM, you can use it as such:
node --loader ts-node/esm ./my-script.ts

Full instructions is available here: https://github.com/TypeStrong/ts-node/issues/1007

@LinusU If I read that right it seems to be an experimental feature that is likely going to have breaking changes. That does not sound like something that we should include in a production code base.

I will probably stay on node-fetch@2 in my Typescript project for now.

This is my configuration for jest and ts-jest.
moduleNameMapper is necessary because imported files have to include the extension in ESM and without this workaround jest fails to find the imported file.
  {
    "preset": "ts-jest/presets/js-with-ts-esm",
    "testEnvironment": "jest-environment-node",
    "extensionsToTreatAsEsm": [
      ".ts"
    ],
    "moduleNameMapper": {
      "^(\\.{1,2}/.*)\\.js$": "$1"
    },
    "setupFilesAfterEnv": [
      "jest-extended"
    ],
    "globals": {
      "ts-jest": {
        "useESM": true,
        "tsconfig": "./tsconfig.test.json"
      }
    },
    "testMatch": [
      "<rootDir>/test/**/*.test.ts"
    ]
  }


allowing mean pixel value inference from int arrays

Now only passes None to self.pixel_values i ncsr.py if subdtype is np.bool_ rather than only collecting pixel_values when subdtype is np.float_ allowing for pixel_values to be obtained from int and uint images.
add skeletonlabel to _testdata.py to have an example int array
add test_skeletonlabel to test_csr.py to confirm int array behaves as expected with csr.summarize

Good news, I figured out how to run pytest independently of whatever cookie cutter has helped me in the past, and I realize my changes don't pass. So I'll work on this some more to sort it out!

Should be good to go now. Old tests pass, as does the new test.

Brilliant, thank you! I had to push some dependency fixes because of pydantic v2 so that the tests would run on CI, but this is ready as far as I'm concerned, will merge! Thanks! 🚀


Update readme with links to new repos

@lesv @ludoch PTAL

LGTM although do we want to say something about the future of compat?

No let's just keep all comms in the new repo + our docs. Thanks. LGTM.

LGTM - do you want to move this repo to googlearchive?

Sure, if we can continue to make changes to the project from there? Would the current URL (https://github.com/GoogleCloudPlatform/appengine-java-vm-runtime) redirect there?

Yes - it would be github.com/googlearchive/appengine-java-vm-runtime with
an auto-redirect.
On Mon, Dec 5, 2016 at 12:36 PM, meltsufin<EMAIL_ADDRESS>wrote:

Sure, if we can continue to make changes to the project from there? Would
the current URL (https://github.com/GoogleCloudPlatform/appengine-
java-vm-runtime) redirect there?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-java-vm-runtime/pull/339#issuecomment-264920728,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFVjn7H_Tqq7ogInEyy4_2frMA6ohRAks5rFEuPgaJpZM4LC8Y5
.

--

•  *Les Vogel
•  *Cloud Developer Relations
•<EMAIL_ADDRESS>•  *+1-4 <%2B1-650-338-7103>08-676-7023

I would like to get an OK from @arouzrokh and @ludoch before doing this.


Add support to upload secrets to Worker Vault

Hi,
Thankyou for this plugin. I found today that workers support secret vault, would it be possible to support this in this plugin?
Cloudflare Worker Secrets Vault
Securely store and manage secret keys, tokens, and certificates for accessing protected resources inside of a Cloudflare Worker.
Docs: https://developers.cloudflare.com/workers/api/resource-bindings/secrets-vault/

Nice idea 👍 I've added support for a metadata json file path to the configuration options in v1.3.0


AudioElement.play()'s promise is not properly handled when erroring

Description

We do not fully handle the promise returned by the AudioElement's play function: https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/play#return_value
We need to resolve errors in the promise by catching and handling them. We should not assume that the audio will play without issue.
https://sentry.io/share/issue/ba82b7f580074af3901151497c1ef3e2/
Reproduction

Go to https://search.openverse.engineering/search/audio?q=birds and open your browser's developer console
Quickly play and then pause an audio file before it has time to load. Note the error message in your console.

Resolution

[ ] 🙋 I would be interested in resolving this bug.

🙋 I would be interested in resolving this bug.

Thanks @ramadanomar :tada: I've assigned the issue to you.

Noting that this also impacts audio files that are invalid, not just the condition identified in reproduction step 2.


fix: correctly poll BasicBlockDownloader

Right now if we schedule block download, on next poll of BasicBlockDownloader we'll return Poll:Ready with NewDownloadStarted event and never register waker with actual downloading future.
Found this to be an issue when writing e2e tests, not sure if can occur during normal node operation

we'll return Poll:Ready with NewDownloadStarted event and never register waker with actual downloading future.

hmm, this is expected because a ready indicates that there can be more work and the caller is supposed to call it again
which I think we do:
https://github.com/paradigmxyz/reth/blob/5532d34ec972ed734267616ef629e57d0ddce65b/crates/engine/tree/src/engine.rs#L115-L120

Found this to be an issue when writing e2e tests, not sure if can occur during normal node operation
which issue did you encounter?

ah, then I think the issue is in this line:
https://github.com/paradigmxyz/reth/blob/5532d34ec972ed734267616ef629e57d0ddce65b/crates/engine/tree/src/engine.rs#L116
if we get Poll::Ready with any other DownloadOutcome variant we never poll it again

the issue I've seen is that block fetching future was not being polled

Updated logic to instead make sure that we drain downloader events in EngineHandler


Pdf2text cli > Process documents in s3 bucket

The cli developed in #53 will only process files in a local directory.
It should be possible to specify an inbound s3 bucket and an s3 bucket to place the extracted data.
The documents should be processed in the inbound s3 bucket.
The extracted json and text documents should be placed in the second bucket.

Existing code has already been implemented which provides an S3Client class:
https://github.com/climatepolicyradar/navigator/blob/c1c5e9431ce85eac42f7f683f18103f61932a31f/backend/app/core/aws.py#L38-L46
This provides an upload_file method which can be used to upload the output of pdf2text cli, but I will need to implement a similar download_file method to pull down files in an input bucket to be processed by the cli.
This code sits in /backend/app which is not a package and sits at the same level in the project as pipeline. To share this code between the backend and the pipeline some code restructuring is going to be necessary. As I see it, these are the options:

move pipeline into \backend at the same level as \backend\app. This would involve making app a package and the Dockerfile in \backend would need to be moved

As we're using a monorepo, introducing a common or core would make sense.

@opyate @kdutia I've updated the comment above with more specific details about the changes I'm making today to create a common python package outside of backend and pipeline. I'm proposing to call it navigator as it would allow us to deploy it to pypi in future. Are you both happy with that?

@chrisaballard there's a Navigator package on pypi already. How about cpr-navigator?

The pdf2text cli has been updated to support the processing of pdf documents stored on s3. This can now be used to batch extract text from documents in an s3 bucket, and then upload the extracted json and text files to a second bucket.
There have been some changes to the cli arguments. You no longer need to specify --text or --json`` arguments as this is implicitly assumed. There is a new argument s3` which is used to inform the cli that you wish to process files on s3 rather than locally, and it will interpret the paths as s3 bucket/folder locations.
In this regard, the bucket folder locations are specified in the following format:
[bucket]/[folder]/[folder]/...
It will parse the string to obtain the bucket name and folder path and will read/write files into the corresponding bucket and nested folders.
In addition to this, the codebase has been refactored to introduce a common navigator package in common/navigator. This is now referenced in backend and pipeline and is included in the respective Dockerfiles. This means that any common code can be placed in this package and imported like so:
from navigator.core.aws import S3Client
I have not updated the unit tests for pipeline as all changes have been made to the pdf2text cli script rather than package/module code. I think later we may well want to put this code into separate modules, at which point unit tests could be created. But I think its worthwhile incorporating the Adobe API changes first to see what impact this has.


feat: add @web3-onboard/okx

Description

okx web3 wallet extension & mobile app support
PLEASE NOTE- Checklist must be complete prior to review.
Checklist

[x] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
[x] Check the box that allows repo maintainers to update this PR
[x] Test locally to make sure this feature/fix works
[x] Run yarn check-all to confirm there are not any associated errors
[x] Confirm this PR passes Circle CI checks
[x] Add or update relevant information in the documentation

Docs Checklist

[x] Include a screenshot of any changes (see docs README on running locally)
[x] Add/update the appropriate package README (if applicable)
[x] Add/update the related module in the docs demo (if applicable)
[x] Add/update the related package in the docs/package.json file (if applicable)

If this PR includes changes to add an injected wallet or SDK wallet module:
Please complete the following using the internal demo package.
To run this demo use the command yarn && yarn dev to get the project running at http://localhost:8080/
Tests with demo app (injected)

[ ] send transaction
[ ] switch chains
[ ] sign message
[ ] sign typed message
[ ] disconnect

Tests with demo app (SDK)

[x] send transaction
[x] switch chains
[x] sign message
[x] sign typed message
[x] disconnect

@vanishcode I am seeing the following build errors : https://github.com/blocknative/web3-onboard/actions/runs/9387915377/job/25851950284
Can oyu take a look when you have a moment?

@vanishcode I am seeing the following build errors : https://github.com/blocknative/web3-onboard/actions/runs/9387915377/job/25851950284 Can you take a look when you have a moment?

Of course 👌


reply-to email field on autogenerated emails

The auto-generated emails notifying Subject Matter Experts (SMEs) of downtimes are sent from a bogus app email<EMAIL_ADDRESS>and users who attempt to reply to it have the reply bounced back at them.   It may be possible to configure a reply-to field in the emails such that responses go somewhere instead of being bounced.

Alternatively, we could configure the sender to be the person who wants these replies.  Though the emails would appear to be coming from them then...

Turns out we have an email aliases capability.  I've set "dtm" to be an alias for Daniel.   Takes 24 hours to take effect apparently.   See internal doc: https://accwiki.acc.jlab.org/do/view/SysAdmin/HowToEditEmailAlias

Scratch that.  Takes 2 hours apparently.


Streaming

https://github.com/anysphere/anysphere/pull/84

small notes:

documents all the scripts pls.
This is a remarkably long PR oops and I cannot verify most parts of it but it seems correct. More confident about the TS (easy) part.

Otherwise, lgtm. Lets merge.

Also, lets use one consistent formatted using the .clang-format file and running trunk/clang-format before pushing. There are a ton of just formatting changes :(


Feature Request: GoogleCloudSpeech - Add method parameters to set use_enhanced  and model options

Feature Requested
Add parameters use_enhanced and model to the recognize_google_cloud method to set their respective values in the RecognitionConfig
Expected behaviour
Give more customizable options for the user and better results in specific cases

Would you like me to give it a try or would you like to think about this Feature Request a little?
Thank you for all your work,
Vitor Hideyoshi

I created a the merge request #735 proposing a implementation for this feature request.
Please take a look and if this feature is of interest please give your opinion on my implementation. :)


Supplemental development process for Docker

Updated the approach from #1312

Docker environment no longer used in CircleCI
Added support for using npm run dev by proxying commands to make docker-run command=<npm-command>
Documentation updated
Found an approach to install Python package in editable mode without writing to the external environment (that's the intention!)

So, similar to the site-community change to the workflow, isn't this what you'd use docker compose for? Seems this is reproducing much of what that gives us already -- defined volumes, ports, image building.

So, similar to the site-community change to the workflow, isn't this what you'd use docker compose for? Seems this is reproducing much of what that gives us already -- defined volumes, ports, image building.

Yes, I have noted the choice several places: Either you have docker-compose as a dependency or you write a rather long docker run command. I was happy about this one in particular because I didn't have to add docker-compose in CI (when I was doing that). But it also saves at least a line of text in the instructions for setting this up :)

Aye. We're already using docker compose in other repos, so seems that should be safe to include as a dependency here too. A homebrew approach is almost certainly bound to cause more problems than introducing docker compose would cause, and docker compose is way more approachable.
I still don't think we want to go as far as to worry about docker in CI, as we aren't using that pattern in any of our repos.

Aye. We're already using docker compose in other repos

I disagree. This repo is much more public-facing.

Anyways, this works, and since you're not using Docker in your development environment and this is just a supplemental change, I would like to conclude it without adding docker-compose. I'm totally open to adding it later :+1:  But perhaps we can wait and see if there's a CI integration coming up as well :crystal_ball:

I disagree. This repo is much more public-facing.

Regardless of who is using it, I don't think asking for docker-compose is that high of a barrier. I would say it's a much lower barrier, as I can work with docker compose, but would not feel comfortable mucking around with a homebrew replacement for docker compose.

This additions is high priority for working through the remaining PRs for 1.1 to be fast-tracked. Since it's an optional and supplemental part of the development/build process, I'll merge it.
I'm quite sure that I'll be back with some adjustments, since this isn't really perfect.
@humitos as you can see from the current setup, there's a bit of simplification work left in the separation logic between docker image, docker runtime and docker host. I'll happily introduce this on a call :)
https://github.com/readthedocs/sphinx_rtd_theme/blob/ba059c7970ad71fe5bcbf1e358b74b52474b29e0/Makefile#L7-L13


typo for extraneous

Typo

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks!


Add IntDictionaryCompilerSmallData

adds IntDictionaryCompilerSmallData
changes IntValueStoreBase::value_t to uint64_t

Coverage increased (+0.4%) to 96.031% when pulling eb80c2c88fd49c566c36f2038d19c739098a9e7b on narekgharibyan:IntDictionaryCompilerSmallData into e6b673704a85e79446d9a4ff2532bd768619c662 on KeyviDev:master.


crash because of an out of memory error

Hi,
i got a crash caused by an out of memory error today. It seems to be related to a type alias for a record which got not used for the generic parameter of List.
I have created a short as possible example where it happens on my computer everytime i run "elm-analyse" in the command line. You can find it at the end of this post. I used the version 0.14.2 on windows 10.
There are several ways to get rid of the out of memory error, these are:

using the AnimatablePath type alias for the staticGeometryList function.

staticGeometryList : List { geometry : Int, attributes : List (Html Msg) }

becomes
staticGeometryList : List AnimatablePath

removing one of the createSvg, init, update, subscriptions or view functions. The Msg type alias can not be removed, because i need to export something.
moving at least one of createSvg, init, update, subscription and view functions above the staticGeometryList function. Are all 5 below staticGeometryList it gives the out of memory error.
changing the return type of the staticGeometryList function to a non generic one
removing the function declaration of the staticGeometryList and let it throw a missing declaration warning instead

Which does not help:

moving the AnimatablePath type alias below the staticGeometryList function
moving type aliases in general does not affect anything
changing the declaration of staticGeometryList from List to Maybe

I came to the problem, because i had left out the declaration of the staticGeometryList function and used the code fix of vscode to let it generate it for me. After i saved the document i noticed that the elm-analyse server has crashed.
elm-package.json
{
    "version": "1.0.0",
    "license": "MIT",
    "summary": "Test project where elm-analyse 0.14.2 crashes with oom",
    "source-directories": [
        "."
    ],
    "exposed-modules": [],
    "dependencies": {
        "elm-lang/core": "5.1.1 <= v < 6.0.0",
        "elm-lang/html": "2.0.0 <= v < 3.0.0"
    },
    "elm-version": "0.18.0 <= v < 0.19.0"
}

Index.elm
module Index exposing (Msg)

import Html exposing (Html, div)

type alias AnimatablePath =
    { geometry : Int, attributes : List (Html Msg) }

staticGeometryList : List { geometry : Int, attributes : List (Html Msg) }
staticGeometryList =
    []

createSvg : AnimatablePath -> Html Msg
createSvg _ =
    div [] []

type alias Msg =
    ()

init : ( Int, Cmd Msg )
init =
    ( (), Cmd.none )

update : Msg -> Int -> ( Int, Cmd Msg )
update _ model =
    ( model, Cmd.none )

view : Int -> Html Msg
view _ =
    div [] []

subscriptions : Int -> Sub Msg
subscriptions _ =
    Sub.none

Output to the command line
> elm-analyse
Cached package information invalidated. Fetching new data from package.elm-lang.org
Fetched dependencies
SyntaxError: Unexpected token T in JSON at position 1
    at JSON.parse (<anonymous>)
    at setupCache (%appdata%\npm\node_modules\elm-analyse\js\util\format-service.js:45:43)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\util\format-service.js:58:5)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\fileReader.js:8:59)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\util\file-loading-ports.js:8:48)
    at %appdata%\npm\node_modules\elm-analyse\js\analyser.js:22:9
    at %appdata%\npm\node_modules\elm-analyse\js\util\dependencies.js:34:9
    at Request._callback (%appdata%\npm\node_modules\elm-analyse\js\util\dependencies.js:19:9)
    at Request.self.callback (%appdata%\npm\node_modules\elm-analyse\node_modules\request\request.js:186:22)
    at emitTwo (events.js:125:13)
INFO: No configuration provided. Using default configuration.
INFO: Load dependency elm-lang/core 5.1.1
INFO: Load dependency elm-lang/html 2.0.0
INFO: Load file Index.elm...

#
# Fatal error in , line 0
# API fatal error handler returned after process out of memory
#

==== C stack trace ===============================

        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF6DF95D8E1+47217]
        v8::internal::Malloced::operator delete [0x00007FF6DF25D201+18497]
        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF6DF95FD27+56503]
        v8::internal::FatalProcessOutOfMemory [0x00007FF6DF518CC2+162]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF5743B3+24131]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF5741CC+23644]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF5850F0+93056]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF57EA10+66720]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF57E597+65575]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF6DF5888EA+107386]
        v8::internal::Malloced::operator delete [0x00007FF6DF263A8F+45263]
        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF6DF95F68E+54814]
        uncompress [0x00007FF6DFDC90A5+134293]
        BaseThreadInitThunk [0x00007FFFB4CC1FE4+20]
        RtlUserThreadStart [0x00007FFFB4EAF061+33]

What version of elm-format are you using? The SyntaxError: Unexpected token T in JSON at position 1 worries me.
Could you remove the elm-stuff directory and, try again? Does this resolve the issue?

Currently elm-format-0.18 0.6.1-alpha is installed. I tried to update it with npm install-elm format -g but it resulted in the same version. I installed the elm-format-0.18 0.7.0-exp version. The T-SyntaxError is gone after i delete the elm-stuff folder - with both versions of elm-format. On second run it appears again.
This is the output with elm-format-0.18 0.7.0-exp after i removed the elm-stuff folder:
> elm-analyse --version
0.14.2
> elm-format --help
elm-format-0.18 0.7.0-exp
...
> elm-package install
Some new packages are needed. Here is the upgrade plan.

  Install:
    elm-lang/core 5.1.1
    elm-lang/html 2.0.0
    elm-lang/virtual-dom 2.0.4

Do you approve of this plan? [Y/n] Y
Starting downloads...

  ÔùÅ elm-lang/html 2.0.0
  ÔùÅ elm-lang/virtual-dom 2.0.4

ÔùÅ elm-lang/core 5.1.1
Packages configured successfully!
> elm-analyse
INFO: No configuration provided. Using default configuration.
INFO: Load dependency elm-lang/core 5.1.1
INFO: Load dependency elm-lang/html 2.0.0
INFO: Load file Index.elm...

<--- Last few GCs --->

[9928:0000026FE1579700]     8264 ms: Mark-sweep 1399.9 (1442.8) -> 1399.9 (1447.3) MB, 785.5 / 0.0 ms  allocation failure scavenge might not succeed
[9928:0000026FE1579700]     9063 ms: Mark-sweep 1404.0 (1447.3) -> 1404.0 (1458.8) MB, 797.5 / 0.0 ms  allocation failure scavenge might not succeed
[9928:0000026FE1579700]    10024 ms: Mark-sweep 1415.7 (1458.8) -> 1415.8 (1458.8) MB, 956.0 / 0.0 ms  allocation failure scavenge might not succeed

<--- JS stacktrace --->
Cannot get stack trace in GC.
FATAL ERROR: MarkCompactCollector: semi-space copy, fallback in old gen Allocation failed - JavaScript heap out of memory

On the second run without deleting elm-stuff the T-SyntaxError appeared again:
> elm-analyse
SyntaxError: Unexpected token T in JSON at position 1
    at JSON.parse (<anonymous>)
    at setupCache (%appdata%\npm\node_modules\elm-analyse\js\util\format-service.js:45:43)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\util\format-service.js:58:5)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\fileReader.js:8:59)
    at module.exports (%appdata%\npm\node_modules\elm-analyse\js\util\file-loading-ports.js:8:48)
    at %appdata%\npm\node_modules\elm-analyse\js\analyser.js:22:9
    at %appdata%\npm\node_modules\elm-analyse\js\util\dependencies.js:57:17
    at %appdata%\npm\node_modules\elm-analyse\js\util\cache.js:33:13
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:504:3)
INFO: No configuration provided. Using default configuration.
INFO: Load dependency elm-lang/core 5.1.1
INFO: Load dependency elm-lang/html 2.0.0
INFO: Load file Index.elm...

#
# Fatal error in , line 0
# API fatal error handler returned after process out of memory
#

#
# Fatal error in , line 0
# API fatal error handler returned after process out of memory
#

==== C stack trace ===============================

==== C stack trace ===============================

        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF798A7D8E1+   v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF798A7D8E1+47217]
47217]
                v8::internal::Malloced::operator delete [0x00007FF79837D201+18497]
v8::internal::Malloced::operator delete [0x00007FF79837D201+18497]
        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF798A7FD27+56503]
        v8_inspector::protocol::Runtime::API::StackTrace::fromJSONString [0x00007FF798A7FD27+56503]
        v8::internal::FatalProcessOutOfMemory [0x00007FF798638CC2+162]
        v8::internal::FatalProcessOutOfMemory [0x00007FF798638CC2+162]
        v8::internal::IdentityMapBase::NextIndex [0x00007FF7986943B3+24131]
        v8::internal:

The out of memory error is still there. It looks a little bit different everytime, but every output says something about out of memory.

I've created an version of elm-analyse (0.15.0-alpha.2) in which this is supposed to be fixed. You can install it using:
npm install<EMAIL_ADDRESS>
Can you verify if this is resolved? Thanks

Not the author of this issue, but it fixed it for me. Thanks!

Can also confirm it working. But I also can't start the server in this version:
$ elm-analyse -s
Elm Analyser server starting with config:
{ port: 3000,
  elmFormatPath: 'elm-format',
  format: 'human',
  open: false }
module.js:549
    throw err;
    ^

Error: Cannot find module 'open'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/Users/bardt/.nvm/versions/node/v8.11.1/lib/node_modules/elm-analyse/js/server/worker.js:1:76)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)

@bardt Thanks for testing it. Just fixed the missing module in this commit (new alpha available though). Issue was introduced in #162 due to a dependency listed under devDependencies.

I can also attest that it fixes the problem I reported above 👍

I can also confirm that it is fixed with version 0.15.0-alpha.3. Tested it on my simple example and the real project where i encountered it first.

Thanks everyone for testing this out. I'll be releasing this soon in a GA release.

ping..

I'm working a few things out before the 0.15 release. It will be released as soon as I figure that out.
@Birowsky This does not work. Constructive messages are much more valuable. Like "at this point I cant use elm-analyse at all, I need this new version to resolve it". For example, I can response with "at this point you can use the 0.15.0-alpha safely, what is in there will be released in the stable version soon", or "maybe you can downgrade to a version prior to 0.14.2, because from the reports this version seems to introduce the bug". I really do not want to sound harsh, I just dont think that messages such as "like", "ping", "this must be fixed or the world exploded" help in the proces of bringing this tool forward.

Dammit. Sorry. Never meant to be hurtful. This is what I thought when I wrote 'ping': This seems like such a cool tool! I can barely wait to use it.
Sorry again. I have the outmost respect for folks enriching the Elm ecosystem!


[REVIEW] Check for null_count > 0 instead of has_null_mask

LGTM, thanks @kkraus14!

This is in preparation for allowing nullmasks to be allocated when there is 0 null values.

Looks good, merging.


Linux Chinese Charactor Unable to 'WRITE'

(Manjaro Linux, Python3.7.3, Texlive)
I've read this issue:
https://github.com/3b1b/manim/issues/209
But by doing as what it told, I still cannot 'WRITE' Chinese charactors.
Here is what the .log file says:

! Package inputenc Error: Unicode character 者 (U+8005)
(inputenc)                not set up for use with LaTeX.

It seem that there are some problems with LaTeX.
I've not got too much practice on Python, so now I'm getting into a annoying position.
Somebody help me?

You have to modify the file manimlib/tex_template.tex, like this.

You have to modify the file manimlib/tex_template.tex, like this.

Great thanks.


We probably shouldn't test on specific implementation details

Like: https://github.com/zjf2012/daos/commit/eb56c5354e2b31ca3becbe305881f989b59e4f51#r36519003
UTs should test on the outcome of a function, but not how the function is implemented.
If we test using the above ways, code refactor may make UTs not passed, which is wrong.

If DaosFile/DaosFsClient are interfaces, we can create our test implementations of them. Then we examine their internal state outside of functions being tested. There will be no hijiack and mock. I totally agree with your idea.
Unfortunately, they are not interfaces, but concrete classes. There is no better way than mock and verification to do the similar thing as what we do for interfaces. If you look at this mock and verification from angle of interface implementation, they are same.
You can also check test cases shipped by Hadoop test which examines statistics of reading and writing times. It's same intention as mine.


Release Google.Cloud.Metastore.V1Alpha version 1.0.0-alpha03

Changes in this release:

Commit ac367e2: feat: Regenerate all APIs to support self-signed JWTs

Created release for Google.Cloud.Metastore.V1Alpha-1.0.0-alpha03


Running tmux from bash-it

Is it a good idea to automatically run tmux when i launch my terminal and if so, how do i configure that in bash-it?

Probably not something for Bash-it itself. You might want to do that as the last step in your ~/.bashrc file...

I added it to my Terminal launch command. All good. Love tmux!

Thanks for confirming!


页码移动端跑版问题

https://github.com/hugo-fixit/FixIt/blob/ea1e62419c3b141220146ec9d3b646c34adda39b/layouts/partials/paginator.html#L1

resolved


Wave effect is not working well on submit button

Wave effect is nice and work well in every button except from form submit buttons. Could you please check it.

It isn't very clear what you mean. But it might be because it's a submit button. The reason being is that the wave effect isn't 100% done when the page is submited. Could this be the case?

When I use wave effect on submit button it shows like the above image.

<input type="submit" value="Create Department" class="waves-effect waves-light btn">

That looks oke. Do you have a other stylesheet in your website that overwrites the material css. What browser are you using?

I use chrome. I will recheck it. But I have no other css file except from materialize.min.css.

Fixed in 02cf1dc


Update lightcontrol to 0.5.0

Please update my adapter ioBroker.lightcontrol to version 0.5.0.
This pull request was created by https://www.iobroker.dev c0726ff.

Adapter tests at node  16 have been dropped. So engines clause must be adapter.
Please merge https://github.com/Schmakus/ioBroker.lightcontrol/pull/201
A new release is not required immidaitly.

PR merged - thanks


Shares server contexts, adds isEigen

This PR, for PURCHASE-464, does two things:

Adds isEigen to the context passed into the OrderApp in Reaction (PR coming soon to consume this).
Consolidates all SSR context-creation into its own file, adds tests.

This is nice because we only have one place to change things as we add new contexts, and any Reaction component apps mounted within Force can expect the same contexts (indeed, these were already inconsistent).
We could go further and consolidate the server-creation into its own file, since this looks mostly the same, but I didn't want to go too wild, especially with upcoming force modernization conversations.

All righty. Welp, this LGTM! 🎉


Ores have been rebalanced.

After researching intended ore amounts by reading the original mods' source code on github, The vein sizes, chunk chances, and y levels have been recorded, and converted into COG ore configuration values.  The final tally was reduced by 1/12 to account for the vanilla overabundance due to ores being spread out.
End results:

Ores are as rare in vein and cloud form as they were in vanilla form... at least when compared to each other (the comparative rarity of iron, redstone, and certus quartz, for example, are consistent between vanilla, vein, and cloud).
Heights have all be restored to their mod-specified levels.
Dense Ores and Applied Energistics makes use of "weighted and mixed ores," in which a single vein can contain more than one ore type.
In Railcraft, sulfur, saltpeter, and firestone have been disabled, due to the distributions breaking their original design in the Railcraft mod (they are designed to regenerate, and so need to be where RC expects them).

Are you sure that firestone and sulfur regenerate? I know saltpeter does, but I thought the others were static.

I'll be honest, I'm not sure.  It took me almost a couple days to figure out WorldGenMinable, and in quite a few cases, track down the variables to determine the vein (cluster) sizes, spawn chances per chunk, and Y levels, and even so, Railcraft is quite difficult to follow.  Considering the end result at my attempts with the poor ores, I'm somewhat convinced I'm reading this wrong, and that the only thing I'm sure I'm getting right are the geodes (at sea floor level, with 5% diamonds, 5% emeralds, and 15% lapis sprinkled around inside).

I'm pretty sure firestone and sulfur don't regenerate. It doesn't make any sense. COG generation is so different the only way we can really test balance is through play-testing, so I wouldn't get too worried about it.

Either way, I'm still trying to unravel the source enough to figure out the vein sizes, spawn chances, and Y ranges of sulfur, saltpeter, and firestone, so I can make the oregen consistent with the mod's expectations.  I'm probably better off resuming the search in wikis and forum posts, tho.

Okay, firestone, sulfur, and saltpeter have been returned to the configuration.  Additionally, all configurations now have a user option to bypass a specific mod's configuration (to allow a mod's vanilla gen to work while keeping COG working for everything else), and a "BlockExists" check will be applied to every single block, preventing future issues with missing blocks due to disabled mod features.
I think I've figured out the problem with poor ores, and it's a bit of sniffed derp-entine on my part; sparse ores require larger values, but while I set the larger values, I forgot to set the poor ores distribution to sparse (they were set to layered veins, which were way overkill).


feat(deploy): Enable local scanner via deploy scripts

Description
Add support to ./deploy/deploy.sh for installing scanner slim in secured cluster and/or configuring Sensor to use Central Services Scanner.
Disabled by default, to enable set env var SENSOR_SCANNER_SUPPORT=true
Checklist

[ ] Investigated and inspected CI test results
[ ] ~Unit test and regression tests added~
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Ran the following against an OCP and GKE infra cluster:
export LOAD_BALANCER=lb
export MAIN_IMAGE_TAG=4.1.0-rc.6
export SENSOR_SCANNER_SUPPORT=true

cd <stackrox repo>
./deploy/deploy.sh

Confirmed Sensor was configured to use scanner by verifying the following env vars were set (observed by running kubectl get deploy/sensor -o yaml | grep -C 5 -i SCAN)
ROX_LOCAL_IMAGE_SCANNING_ENABLED=true
ROX_SCANNER_GRPC_ENDPOINT=scanner.stackrox.svc:8443

And clusters UI shows a healthy scanner:

/retest


[Bug] Controller doesn't prioritize drawers with items when inserting.

If you have a large array of drawers connected to a controller, some empty and some with items, when you import items that already have a drawer the controller will not necessarily put the items there. I think the empty drawers are being found by whatever algorithm is trying to find a spot and the controller just accepts that and moves on.
I'm finding this from a Mekanism Digital Miner pushing ores into a controller. Sometimes an ore that already has a drawer will be put into an empty one despite having plenty of room in the other drawer. I can't lock all the drawers because the miner is pulling anything with the ore tag, so there's still things I don't have yet.

What version are you using?

1.18.2-0.0.15

I recommend you to lock the drawers as I can't internally safely modify the inserting behaviour of external mods.


Fix with clause in wrong order.

Thank you! I have already received a fix for this but I will credit you in the changelog.


Self-created users cannot accept ToS [v5.5]

From BMEC:

When first-time users register under their own netIDs, they used to have to check the box for “Accepting the Terms of Service”. Now, they can just register without accepting the ToS, and it slows down the checkout process when the Checkout Person has to certify that the patron does accept. It’s no problem when we have the time to add new users ourselves, but it’s easier for patrons to read through the ToS on their own time (or just blindly accept them) before picking up their reservations.

This was changed in #774 / 818959c266750aaba1aac1dfca92f5a56fa1b694, let's get it fixed in v5.5.6.

Not sure if this would be better here or in another issue, but the disabled ToS box(#1320) appears again if you can't manage reservations if you try to submit an incomplete user registration form or an incorrectly filled out one. It probably will be addressed through this issue though.

Good catch - when UsersController#create fails some of our display logic apparently no longer applies. We'll consolidate that bug into this issue, thanks!


Generated GLSL samplers lose their binding points

You can repro this with reference/shaders-hlsl/frag/basic.frag shader:
Texture2D<float4> uTex : register(t0);
SamplerState _uTex_sampler : register(s0);

static float4 FragColor;
static float4 vColor;
static float2 vTex;

struct SPIRV_Cross_Input
{
    float4 vColor : TEXCOORD0;
    float2 vTex : TEXCOORD1;
};

struct SPIRV_Cross_Output
{
    float4 FragColor : SV_Target0;
};

void frag_main()
{
    FragColor = vColor * uTex.Sample(_uTex_sampler, vTex);
}

SPIRV_Cross_Output main(SPIRV_Cross_Input stage_input)
{
    vColor = stage_input.vColor;
    vTex = stage_input.vTex;
    frag_main();
    SPIRV_Cross_Output stage_output;
    stage_output.FragColor = FragColor;
    return stage_output;
}

Compile command (glslc from ndk r17, latest spirv-cross):
./glslc -x hlsl -fshader-stage=frag basic.frag -o /tmp/a.spv && ./spirv-cross /tmp/a.spv

You get this output:
#version 450

struct SPIRV_Cross_Input
{
    vec4 vColor;
    vec2 vTex;
};

struct SPIRV_Cross_Output
{
    vec4 FragColor;
};

uniform sampler2D SPIRV_Cross_CombineduTex_uTex_sampler;

layout(location = 0) in vec4 stage_input_vColor;
layout(location = 1) in vec2 stage_input_vTex;
layout(location = 0) out vec4 _entryPointOutput_FragColor;
vec4 FragColor;
vec4 vColor;
vec2 vTex;

void main()
{
    vColor = stage_input_vColor;
    vTex = stage_input_vTex;
    FragColor = vColor * texture(SPIRV_Cross_CombineduTex_uTex_sampler, vTex);
    _entryPointOutput_FragColor = FragColor;
}

Notice
uniform sampler2D SPIRV_Cross_CombineduTex_uTex_sampler;

loosed its binding point, resulting in incorrect binding point assumptions by a graphics driver. In this case it may be correct since SamplerState _uTex_sampler : register(s0); starts from register 0, for any other register this will result in incorrect sampler picking.
x86_64 Linux binaries used for this repro, just in case:
glslc.zip
spirv-cross.zip

How that line should look for correct behaviour:
layout(binding = 0) uniform sampler2DArray SPIRV_Cross_Combinedt_colors_color;

Maybe have an option to map them directly? Because for my shaders if I map sampler at register s2, I use it only for texture at register t2...

Without this mapping, some of my shaders mention only some sampler registers like s2 and s3 for example, and in OpenGL I bind samplers at binding points 2 and 3 respectively, but currently without GLSL sampler binding points the driver interprets them as if they are bound to s0 and s1 given no explicit binding points was provided.

Manually setting samplers to correct binding points helps, I just can't see why spirv-cross loses register(s0), register(s1), register(s2) information I provide in shaders so it could use them to generate direct layout(binding = 0), layout(binding = 1), layout(binding = 2) GLSL variants?

That assumes you want a 1:1 mapping, but not everyone would want that (it's not always that easy).

Can we have an option for that? Or, where should I look in spirv-cross source code to at least make the change my shaders need for my local version?

By the way, I should clarify that I don't set texture's implicit samplers, I always create and set separate sampler objects: https://www.khronos.org/opengl/wiki/Sampler_Object

When a sampler object is bound to a texture image unit, the internal sampling parameters for a texture bound to the same image unit are all ignored. Instead, the sampling parameters are taken from this sampler object.

I'm definitely not the only one who does this if even the official documentation specifically mentions it...

So, from an API point of view, what you need to do for this kind of mapping is:
compiler->build_combined_image_samplers();
const vector<CombinedImageSampler> &samplers = compiler->get_combined_image_samplers();
for (auto &s : samplers)
{
    uint32_t binding = compiler->get_decoration(compiler->image_id, spv::DecorationBinding); // or sampler_id.
    compiler->set_decoration(compiler->combined_id, spv::DecorationBinding, binding);
}
compiler->compile(); // Get GLSL.

If you think this is good enough for this use case, I will consider adding it as a CLI option.
If I'm not asking too much, could you please add it to CLI as an option and paste source difference patches here so I could apply them and test on my end? I don't use API and dunno how...

If I make a pull request with this feature, would you be able to pull that and see if it fixes the issue?

Sure!

Try this: https://github.com/KhronosGroup/SPIRV-Cross/pull/586
Add --combined-samplers-inherit-bindings to the command line.


Kotlin syntax highlighting breaks with contractions in method names

When using backticks for method names, if one of the words has a contraction, everything after the apostrophe (including the following lines) gets highlighted as a String literal.
I expect the following to be highlighted correctly:
[...]
    fun `method with can't in the name should be displayed correctly`() {
        myDependency.callMethod()
    }
}

The Kotlin mode isn't actively maintained at the moment, I believe, but you could try submitting a pull request.


Update vSphere distro version and website link

Update vSphere Kubeflow distribution version and the website link

/lgtm
@zijianjoy only you or @james-jwu can approve

@harperjuanl Also, are you sure you don't also want to update the name from "Kubeflow on vSphere" to "vSphere Machine Learning Extension" which is what you list on your website: https://vmware.github.io/vSphere-machine-learning-extension/index.html

@harperjuanl Also, are you sure you don't also want to update the name from "Kubeflow on vSphere" to "vSphere Machine Learning Extension" which is what you list on your website: https://vmware.github.io/vSphere-machine-learning-extension/index.html

@thesuperzapper Thanks a lot, we don't need to update the name.

@zijianjoy can you please approve this small fix of the link?

/lgtm
/approve


tap on notification opens app but dont show alert of notification

Expected Behaviour
on tap on notification app should open and notification should display.
Actual Behaviour
i get my push notification on android notifications window but when i tap on it it opens my app but no alert with notification is displayed
When
Reproduce Scenario (including but not limited to)
when my app is not running or in bg and my msg is sent by firebase
Steps to Reproduce
JS:

var app = {
    // Application Constructor
    initialize: function() {
        this.bindEvents();
    },
    // Bind Event Listeners
    //
    // Bind any events that are required on startup. Common events are:
    // 'load', 'deviceready', 'offline', and 'online'.
    bindEvents: function() {
        document.addEventListener('deviceready', this.onDeviceReady, false);
    },
    // deviceready Event Handler
    //
    // The scope of 'this' is the event. In order to call the 'receivedEvent'
    // function, we must explicitly call 'app.receivedEvent(...);'
    onDeviceReady: function() {
        console.log('Received Device Ready Event');
        console.log('calling setup push');
        app.setupPush();
    },
    setupPush: function() {
        console.log('calling push init');
        var push = PushNotification.init({
            "android": {
                "senderID": "xxxxxxxxx"
            },
            "browser": {},
            "ios": {
                "sound": true,
                "vibration": true,
                "badge": true
            },
            "windows": {}
        });
        console.log('after init');

        push.on('registration', function(data) {
            console.log('registration event: ' + data.registrationId);

            var oldRegId = localStorage.getItem('registrationId');
            if (oldRegId !== data.registrationId) {
                // Save new registration ID
                localStorage.setItem('registrationId', data.registrationId);
                // Post registrationId to your app server as the value has changed
            }

            var parentElement = document.getElementById('registration');
            var listeningElement = parentElement.querySelector('.waiting');
            var receivedElement = parentElement.querySelector('.received');

            listeningElement.setAttribute('style', 'display:none;');
            receivedElement.setAttribute('style', 'display:block;');
        });

        push.on('error', function(e) {
            console.log("push error = " + e.message);
        });

        push.on('notification', function(data) {
            console.log('notification event');
            navigator.notification.alert(
                data.message,         // message
                null,                 // callback
                data.title,           // title
                'Ok'                  // buttonName
            );
       });
    }
};

android manifest
 <preference name="StatusBarOverlaysWebView" value="false" />
    <preference name="StatusBarBackgroundColor" value="#ee6e73" />
    <preference name="StatusBarStyle" value="blacktranslucent" />
    <plugin name="cordova-plugin-statusbar" source="npm" spec="~2.1.3" />
    <plugin name="phonegap-plugin-push" source="npm" spec="~1.8.0">
        <variable name="SENDER_ID" value="655844105827" />
    </plugin>

Platform and Version (eg. Android 5.0 or iOS 9.2.1)
android 6.0 phonegap build plugin  spec="~1.8.0"
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Samsung s6
Cordova CLI version and cordova platform version
cordova --version                                    # e.g. 6.0.0
cordova platform version android                     # e.g. 4.1.1

Plugin version

Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem

@brunoalex version 1.8.0 of the plugin does not support FCM. We are working on support in the v2.0.x branch. Give that branch a try.

tks! how can i use that branch .. im usimg phonegap

phonegap build or phonegap CLI?
If you are using phonegap build I don't think v2.0.x branch is supported there yet
Also, if you are sending the push from Firebase console, on notification is not called when the app is in background or closed, we are looking into it

well in my end using phonegap   build and firebase console its all working (even if its closed ) but the "Tap on notification open app and alert"  dont fire on s6 android 6.1...strange thing is that it works on an older android os :/


[SPARK-12841][SQL] fix cast in filter

In SPARK-10743 we wrap cast with UnresolvedAlias to give Cast a better alias if possible. However, for cases like filter, the UnresolvedAlias can't be resolved and actually we don't need a better alias for this case.  This PR move the cast wrapping logic to Column.named so that we will only do it when we need a alias name.

cc @yhuai @marmbrus

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49503/
Test FAILed.

retest this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49505/
Test FAILed.

Test build #49506 has started for PR 10781 at commit aa3b4e8.

Test build #49506 has finished for PR 10781 at commit aa3b4e8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49506/
Test PASSed.

Test build #49604 has started for PR 10781 at commit af96c5d.

Test build #49604 has finished for PR 10781 at commit af96c5d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49604/
Test FAILed.

Test build #49612 has started for PR 10781 at commit d8681b4.

Test build #49612 has finished for PR 10781 at commit d8681b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49612/
Test PASSed.

Merged build finished. Test PASSed.

ping @yhuai

LGTM. Merging to master and branch 1.6.

there are a few conflicts with branch 1.6. Can you create a pr for the backport? Thanks!

yea, sure


AWS SDK integrations does not trigger partial imports

We require user to import the whole aws sdk to trigger instrumentation like below,
const AWS = require('aws-sdk');
const ddb = new AWS.DynamoDB({ apiVersion: '2012-10-08', dynamoDbCrc32: false });

but user can import a specific client with partial import, the below code does not start aws integration in the agent,
const DynamoDB = require('aws-sdk/clients/dynamodb');
const ddb = new DynamoDB({ apiVersion: '2012-10-08', dynamoDbCrc32: false });

Fixed via https://github.com/thundra-io/thundra-lambda-agent-nodejs/pull/69


Bypassing Zinc for non-incremental Scala compilation

Zinc is helpful in speeding up incremental builds of Scala code, but it comes with a startup cost ( https://github.com/gradle/gradle/issues/3725 ) and a complexity/maintenance cost ( https://github.com/gradle/gradle/issues/2605 ).
When you run Gradle mostly during full builds in Jenkins or CircleCI, and when you let your IDE handle incremental builds, you don't really reap any benefits from Zinc-based Scala builds in Gradle.
Expected Behavior

For use cases that only need full (non-incremental) builds, it would be nice to bypass zinc and run scalac directly -- similarly to how gradle runs javac.
Is this doable through configuration in the gradle scala plugin today?  If not, what would it take to make that possible?
Current Behavior

All scala builds seem to go through zinc before scalac.
Context

Longer build times and concurrency bugs when using zinc.
Steps to Reproduce (for bugs)

Your Environment

Build scan URL:

How about something like this?  A gradle buildscript to override compileScala and compileTestScala tasks with replacements that call the Scala compiler directly (without zinc and without a new jvm process).
It doesn't handle compiler flags, other xxScala tasks, and probably a lot of other functionality from the tasks provided by the scala plugin.
apply plugin: 'scala'

dependencies {
  compile "org.scala-lang:scala-library:2.11.12"
}

buildscript {
  repositories {
    jcenter()
  }
  dependencies {
    classpath "org.scala-lang:scala-library:2.11.12"
    classpath "org.scala-lang:scala-compiler:2.11.12"
    classpath "org.scala-lang:scala-reflect:2.11.12"
  }
}

import scala.tools.nsc.*
import scala.collection.JavaConversions

def doCompile(sourceSet, depSourceSets, configuration) {
  Settings settings = new Settings()

  // https://stackoverflow.com/questions/27934282/object-scala-in-compiler-mirror-not-found-running-scala-compiler-programatical
  // http://www.scala-lang.org/old/node/7949
  // https://github.com/scala/scala/blob/2.11.x/src/compiler/scala/tools/nsc/settings/StandardScalaSettings.scala
  def deps = depSourceSets.collect { it.outputDir.toString() }.join(':')
  settings.classpath().value_$eq(configuration.asPath + ":$deps")

  sourceSet.scala.outputDir.mkdirs()
  settings.d().value_$eq(sourceSet.scala.outputDir.toString())

  for (srcDir in sourceSet.scala.srcDirs) {
    settings.sourcepath().append(srcDir.toString())
  }

  Global global = new Global(settings)

  // https://stackoverflow.com/questions/27980332/how-do-i-construct-a-non-static-java-inner-class-from-groovy
  Global.Run run = new Global.Run(global)

  def fileNames = sourceSet.allScala.collect { it.toString() }
  run.compile(JavaConversions.asScalaBuffer(fileNames).toList())

  if (global.reporter().hasErrors()) {
    throw new GradleException("Failed to compile.")
  }
}

task compileScalaDirect {
  dependsOn compileJava
  inputs.files(compileScala.inputs.files)
  outputs.dir(project.sourceSets.main.scala.outputDir)
  doFirst {
    doCompile(project.sourceSets.main, [project.sourceSets.main.java], configurations.compile)
  }
}

task compileTestScalaDirect {
  dependsOn compileJava
  dependsOn compileScala
  dependsOn compileTestJava
  inputs.files(compileTestScala.inputs.files)
  outputs.dir(project.sourceSets.test.scala.outputDir)
  doFirst {
    doCompile(
      project.sourceSets.test,
      [project.sourceSets.main.java, project.sourceSets.main.scala, project.sourceSets.test.java],
      configurations.testCompile)
  }
}

compileScala {
  dependsOn compileScalaDirect
  onlyIf { false }
}

compileTestScala {
  dependsOn compileTestScalaDirect
  onlyIf { false }
}


recent activities limit and balance modification on overview page

Resolves #300 and Resolves #301 by showing a maximum of 5 transactions under recent activities. The details link on the page is taken out and the total balance is shown directly to the user

Two outstanding issue he found.

font size of the DCR suffix should be the smaller variation.
case of 0.00000000 should just be 0. Remove trailing zeros for any amount.

Updated


Error: failed to generate new wallet: Wrong Device Status : SW=6930 (EXPECT=9000, MASK=ffff)

Currently using CLI <IP_ADDRESS> on Mac OSX - 10.14.5 (Mojave)
Ledger has Monero 1.3.1 installed on it.
Daemon is running and fully synced.
Ledger is plugged in and unlocked (confirmed working with Ledger Live).
Monero app is open on Ledger.
When I try to either open a monero wallet, or generate a new one, I get the error:
Error: failed to load wallet: Wrong Device Status : SW=6930 (EXPECT=9000, MASK=ffff).
Example of command I'm running:
./monero-wallet-cli --generate-from-device exampleWallet --subaddress-lookahead 2:5
Any suggestions as to what I might be doing wrong?

duplicate issue #43

Thanks for the reference @cslashm
Is there a way that we can turn the error message:
Error: failed to load wallet: Wrong Device Status : SW=6930 (EXPECT=9000, MASK=ffff)
Into something like "This version of Monero is not supported, please use version xxx"?
Or, if that's not possible, a similar solution?
This would help avoid unnecessary confusion/frustration/anxiety. Currently the existing error message is unintelligible to myself, and no doubt others.

Yes I will add error message in 1.4.0 on the device.
For this one I will make quick fixe in order to release this week

Thanks @cslashm - error messages with more info will help a lot.
Thanks for the reply & all your hard work.

Hi I'm running on monero-gui-v<IP_ADDRESS>
Update the lastest ledger app and Monero App on ledger nano s
Also enable developer mode
Still receiving this error when attempting to connect my wallet:
Error: failed to generate new wallet: Wrong Device Status : SW=6930 (EXPECT=9000, MASK=ffff)


background color alignment fix for issue #504

Thanks @jtthrillson! I gave this a try in the example Espressos project, adding a background color to the active link state of the example label.
Here's the label's highlighted state on current master:

And on this pull request's branch:

It does look like master is handling this properly. It seems like there may be something else going on here, or perhaps I'm not following exactly what you're trying to fix here. Do you have any more details on the behavior you're seeing and how this patch resolved the problem in your use of the label?


Adding a unit test to reproduce the NotFoundException in change feed after partition split

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

Annie has picked-up this test in the actual hotfix PR


Problems with amd64 build?

Is there a particular reason why you only support i386 for now?
I was able to build libagsjoy.so successfully on amd64 by just installing libsdl2-dev:
$ file libagsjoy.so
libagsjoy.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, BuildID[sha1]=a8ad0fb065e6572a7c72c71588639c1c0cb26edc, not stripped

This was on Debian buster (testing), however, not Ubuntu 16.04.

Hey @onitake I
I have the same output for file:
$ file libagsjoy.so
libagsjoy.so: ELF 64-bit LSB shared object, x86-64, version 1 (SYSV), dynamically linked, BuildID[sha1]=99cf8dea57a9854d1ecf52700ea4507868a7a229, not stripped

I understand this means this is a 64-bit binary. I don't really know this and believed the voted answer here.
What I couldn't find out, is how to build for a 32 bit system (i386) while I am using a 64 bit system. Unfortunately, I don't have a 32 bit computer anymore, so I was looking in a way to do this on my 64 bit machine (but all dependencies I tried to install, depended on more stuff until I got the "you have held broken packages" message).
This is why for now, I think it should be easier to create a virtual machine running some 32-bit Ubuntu to do it - I didn't had the time yet to do it though.

Ah, I see now. I misinterpreted the README and thought only i386 was supported.
And I agree that cross-compiling on Debian/Ubuntu is a bit complicated.
The correct way to this would be to add the additional architecture to the system, then install dependencies using the appropriate arch tag. The compiler can then be launched with -m32 (or -m64 for 64bit cross-compilation).
Something like:
dpkg --add-architecture i386
apt update
apt install libsdl2-dev:i386

Note that libsdl2-dev does not support Multi-Arch properly, so you can only have the -dev files installed for either i386 or amd64. Other -dev packages handle this better.


Prevent button trigger during start / shutdown?

I would like to ask how I can disable the button trigger once it was fired for an certain amount of time (e.g. 10 seconds).
Use case: Having the on/off shim implemented in a music for kids. sometimes they double tap on the button and in some cases the whole system crash. So I want to prevent that the button triggers any action once it was fired.
Thank you for any hints an tips.
farbschein

Good question- the shutdown script is written in bash, so it's not the friendliest thing to reason about and change. I may not have an answer anytime soon.


help on framework

hi, we are unable to build the framework with the code example given, kindly help if it is possible to extend more that 1000 calls per day from API or how to build readability analysis from framework, please provide some reference

As I explained on the other ticket that you opened (Code Examples issue #4, the unit-tests and the code examples provide reasonably enough info to reproduce all the API functions of Datumbox. If you are having problems, I would recommend you to either ask questions on Java forums or seek for help from a seasoned Java developer. At any case please note that the Github issues page is not the right place to post requests for extending the API limits of your account.
To answer your question, lifting API limitations is not feasible at the moment. Datumbox is a 100% free project and unfortunately no one is sponsoring the hosting expenses for keeping the API running. This is the reason I recently published all the API models on the Datumbox Zoo so that people can build use them directly without having to deal with API limits. I really don't know for how long I'll be able to keep the API running without getting a sponsor, so it is safer no to use Datumbox Zoo directly. If you are interesting in sponsoring the servers and keep the API running, feel free to (contact me)[http://www.datumbox.com/contact/].


Adds Pascal Triangle in python

PR for issue #206
Wrote code in python to print pascal triangle.

The folder name is Pascal not Pascal-Triangle. change it.

The folder name is Pascal not Pascal-Triangle. change it.

Yeah Sure.

The folder name is Pascal not Pascal-Triangle. change it.

Did the changes. Please review.

@all-contributors please add @Cybertron-Avneesh  for code


rageshake should prompt you to enter an explicit problem report before trying to send a mail

and so prepopulate up the email form, to discourage people from sending blank reports

We should add the same prompt as android
Note: "Please tell us what went wrong" rather than "Enter a bug description"
and focus that field by default


Added Request.sign() option to omit oauth_body_hash

I'm using oauth2 to consume Twitter's streaming API.  To make the library compatible with their API, the oauth_body_hash header must be omitted.  To allow that, without breaking any existing functionality, I added a parameter defaulted to True called omit_body_hash.  With this small change, my code to consume twitter's streaming API works.

The folks at twitter would like me to point out that oauth_body_hash is a nonstandard addition to the protocol, and that when you're connecting to twitter it shouldn't ever be included.  While the oauth2 library works with their REST API currently, it's possible they will be more strict in the future and it will cease to work.  To maintain compatibility with twitter in the long run, it would be best to make omit_body_hash true by default.

I'd love to see this merged as soon as possible. Not having a way of controlling the body_hash results in new versions of oauth2 immediately breaking when used against Oauth providers that do not support this extension. I

Sadly, I've heard simplegeo is defunct, so I'm not sure it will ever get merged.  I guess somebody else needs to take up the torch and maintain this.

I just ran into the same issue. I've hacked it to work by setting is_form_encoded=True when I create my Request. Clearly a massive hack. I think this OAuth library needs a bit of an overhaul. There are a lot of open issues, which is a pity since it appears to be the best Python OAuth lib out there.

I'd love to see somebody take it and run with it, but I don't know who would do it.  I don't think I have time to maintain it full time, but I'll keep patching it as I see bugs that I know how to fix.

@rickhanlonii this seems like something that's relevant to your work in #138 as well?

@bensonk thanks for the PR! @jaitaiwan and @rickhanlonii are probably going to chime in on this as well. We need tests on this before we merge, but feel free to wait on those for now.

It looks like section 4.1.1 specifies when to include the oauth_body_hash:

4.1.1.  When to Include the Body Hash
Not all requests should contain the oauth_body_hash parameter.

OAuth Consumers SHOULD NOT include an oauth_body_hash parameter when making Request Token or Access Token OAuth requests.
OAuth Consumers MUST NOT include an oauth_body_hash parameter on requests with form-encoded request bodies.
OAuth Consumers SHOULD include the oauth_body_hash parameter on all other requests.

So I think this change in general is necessary according to 4.1.1.
The only question I have here is whether or not the default value should be True. It seems to me that it would be straightforward to pass omit_body_hash=True for only calls to Request Token or Access Token OAuth and default to False for all other requests.
In other words: I think the default value should match the requirements of "all other requests" rather than the two special defined cases. I understand from the above discussion that the requirements of "all other requests" to the twitter API would require omit_body_hash=True, but supporting a particular third party API should not be a motivating factor in the design at hand.

Also, I think it is more clear for the parameter to be defined positively as in include_body_hash=True, rather than negatively as in omit_body_hash=False since the latter requires parsing a double negative to understand that the body hash will be included.

@rickhanlonii I concur all around. @bensonk if you could add tests I'm :+1: after @jaitaiwan has reviewed.

@bensonk @joestump adding this to my long list of things to review.

LGTM after tests are added. @bensonk are you able to provide the tests for this?


Fix downloading KOTS bin for versions with build suffix

What this PR does / why we need it:

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?

Does this PR require documentation?

Merging as this does not affect the current version of KOTS (v1.117.0)


Travis failures - bad config

Looks like Travis has been partly failing for a while. This isn't ideal as you can quickly spiral into ignoring more and more errors until the whole test suite is broken and you're merging blind.
One issue seems to be missing jRuby binary.
The other issue is missing PhantomJS download, for an ancient version.
As per #201 we should stop supporting ancient versions of Ruby / jRuby / PhantomJS,  aka just delete those Travis configs. Currently Travis only tests up to Ruby 2.1 so it would be good to add 2.4 to the matrix.

most of this is done in sachsenring, yet travis config needs to be updated to use -pre gems

Travis is absolutely useless for this project. Tests are completely unreliable and subtle versions changes result in failures ONLY ON TRAVIS. I guess that is, because they only provide ancient dinosaur software, where various bugs, long ago fixed in modern of versions of that basic software, are present all over the place.


implement starknet-rs for contract class endpoints

Implemented class related starknet methods using the starknet-rs package.
Closes #15

Following up on this PR, added support for trace endpoints.
Closes #17
Also added initial support for crafting an array of transactions.
Fixed some bugs in the params builder (use structuredClone to deep clone objects instead of performing a shallow copy).

Since most of the work for #16 was already done on this PR I went along and implemented support for write endpoints.
Closes #16

Following up on the previous work to support starknet-rs, I've implemented call endpoints.
Closes #18

Can you please create separate PRs for the issues? It'll make it easier to review
@saimeunt

@jelilat I've created separate PRs for each commit, they will need to be reviewed in order because they depend on each other.
Closing in favor of #30 #31 #32 #33 #34 #35


add example ram + fix log example + fix secureCon.md

The workflow issue should be fixed tomorrow with the update @iobroker/testing.


Logging Improvement: EntityNotExistError on Activity completion should not be error level

Currently we log EntityNotExistError on activity completion as log level 'Error'.  It should instead be log level 'Info' as it is expected when workflow execution completed.
cadence_1    | time="2018-01-29T21:24:07Z" level=error msg="RespondActivityTaskCompleted. Error: EntityNotExistsError({Message:Activity task not found.})" RunID=084cbc76-1db4-4eb5-a05b-de6731f5e9b6 ScheduleID=11 Service=cadence-frontend WorkflowID=f3d2eb19-e739-4549-91bd-f779b85037c0

Move to server side issue.

move back to client since this should be client specific


Qingwei/asset picker multi col

Coverage increased (+0.2%) to 31.422% when pulling 755b73e5701ae50e98e5c42470133e765d361413 on qingweibinary:qingwei/asset_picker_multi_col into f906d12ba6f8d4554d448a73c6ddd008a1994d67 on binary-com:master.

OK to merge

Coverage increased (+0.2%) to 31.422% when pulling aae63353c98914e7b7697d849b095d4cd4816636 on qingweibinary:qingwei/asset_picker_multi_col into f906d12ba6f8d4554d448a73c6ddd008a1994d67 on binary-com:master.


header.column.columnDef.header return undefined

Describe the bug
flexRender function is not accept the undefined type
this error is currently on the codesandbox  React Example: Filters
Your minimal, reproducible example
CodeSandbox
Steps to reproduce

Go to React Example Filters
Open src folder in codesandbox
Scroll to thead jsx element
look to thead > tr > th
In the th element header text flexRender function

Expected behavior
I expect to not see any errors
How often does this bug happen?
Every time
Screenshots or Videos

Platform
Windows
react-table version
@tanstack/react-table 8.1.1
TypeScript version
4.8.0
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Good catch. Let’s update that.

I currently testing this library I using with my project.


[2206] Remove pen and rollover configuration

Important
Removed from workflow and Makefile in https://github.com/DFE-Digital/register-trainee-teachers/pull/1122
The above PR should be merged before this one
Context
Delete unused envs
Changes proposed in this pull request
Delete config
Guidance to review

 Delete rollover and pen environments


add comma thousand separator for numbers?

on the leaderboard and

on profile pages

This was fixed by adding toLocaleString() to numbers.


Want to improve feedback form

I would like to work upon improving the feedback form. Please assign.

how? any screenshot or reference?


Enhancements

Git integration?
Branch deploy or sync.
Mount local sync? Remote mount changes are synced to a local git repo.
Git bare set working tree to remote mount.
Rsync?

Overlay fs?

I notify on a network mount to copy locally

(moved to project board)


Cadence Cookbook Updates

Cadence Cookbook Updates
Grant category
Please select one:

Open source maintenance
Developer tools / services
Educational material

Description
This proposal seeks to formalize requirements around potential improvements to the Cadence Cookbook, including:

Improvements to the site's information architecture
Improvements to the site's UI and branding in alignment with Flow
An MVP to demonstrate how Cadence test cases can be added to existing smart contract examples in the Cookbook

Additionally, this proposal includes updating several existing blog posts<EMAIL_ADDRESS>The aim of revamping the articles' content includes: highlighting differences between Cadence and EVM-based programming paradigms, showcasing use-cases/features unique to Cadence, refreshing content to reflect latest Cadence standards, general improvements.
Problem statement
Target audience:

Developers who are new to Flow and are looking for a hands-on approach to learning Cadence via smart contract examples that follow a logical, journey-based structure.
Blockchain developers already familiar with basic Cadence but are looking for examples of functionality and features unique to Flow.

Evidence for the need:

The current state of the Cadence Cookbook is in need of a refresh; this includes the UI, information architecture, and content presented on the site

Proposed solution
This proposal is an initial starting point for updates to the Cadence Cookbook, primarily focused around ideation, wire framing, and an MVP for adding smart contract test cases to the site.
Impact

In what ways does this benefit the broader Flow developer ecosystem?

By adding relevant examples and pertinent smart contract use-cases to the site, the Cadence Cookbook can serve as an authority/knowledge base for Cadence developers and those new to Flow. Many blockchain applications require similar smart contract paradigms; these can be templatized as code examples within the Cookbook and serve as a useful point of reference for developers, featuring high quality code examples and robust testing of the smart contract. Additionally, examples featured in the Cookbook can highlight the unique capabilities of Cadence/Flow and encourage new developers to join the ecosystem.
Milestones and funding

Note: Please consider adoption and/or maintenance milestones at the end of your project.

Milestone
Deliverables
Timeline
Risks
USD proposal

1 - Information Architecture
Develop an updated information architecture for the site. This includes: defining new journey-based information flows that follow a sequential ordering, defining Flow-centric themes that highlight the strengths/unique aspects of Cadence, regrouping existing content on the site in a logical way.
1 week
-
400

2 - UI
Prepare wireframes for updated site UI. This includes: a) refreshing existing screens/modals to have a more modern appearance in line with Flow branding, b) designing UI mock-ups of all new screens and components needed for updated site architecture, c) UI for test cases MVP. This milestone will be delivered in Figma (or other compatible UI) software. At this stage we can iterate on the UI/IA further with the Flow team based on initial feedback.
1 week
-
500

3 - MVP - Smart Contract Test Cases
Develop an MVP for how Cadence test-cases could be displayed alongside each smart contract example, ideally in an interactive format with explanation and references to the Flow documentation. This includes facilitating the data entry process for test cases when contributing to the Cookbook. Additionally, as a prototype, this milestone includes developing sample test cases for one of the existing smart contract examples in the Cookbook.
1 week
-
800

4 - Ideating new content for the Cookbook
Ideating new use-cases/smart contract examples to add to the Cadence Cookbook, as well as more generally brainstorming new directions for the project. The deliverable for this milestone will be in the form of a written document/mindmap.
1 week
-
350

5 - Article updates
Revamp of the 5 Flow articles currently published on Medium: link. This includes: highlighting differences between Cadence and EVM-based programming paradigms, showcasing use-cases/features unique to Cadence, general improvements.
1 week
-
400

*Milestones 1,2,4 can be worked on concurrently
Total funding proposed: $2450
Team

Name
Role
Bio
Contact

Lea Lobanov
Full-Stack/Blockchain Developer
Founder at Purple Dash.
<EMAIL_ADDRESS>

Thanks, have accepted this and looking forward to getting this done!

This grant has now been completed, will create subsequent follow-up grant for next stages of the project shortly. Thank you!


"nsxtea --version" should show currently installed version

@nrekretep should be implemented in v0.2.1


I want to scroll through a ranked list of students

As a user, I want to scroll through a ranked list of students that are the closest match to me so I know the people I choose will better fit to my schedule

[ ] #44
[ ] #51
[ ] #58
[ ] #62


Ctrl-Shift-L On Linux

On Linux I can not use the keyboard shortcut. If I remember right, long ago. The original allowed for this but has not for some time. Its also bork in other ways, but you know that ;)  So far your fork has fixed all the issues I have had, save the keyboard shortcut.
Anything I can do to help Ill jump right in!

I am on Windows (at least for now) so I didn't test other OS.
I have made a change right now, please have a look if that fixes it for you.
If not, please check the Developer Tools Console for any KeyBinding errors.

After some testing I assume the fix works, but is conflicting with "split section into lines" in the edit menu. This is a default keybinding. The stable version does not throw the conflicting error.
Thanks for your quick response.

On Windows that is Ctrl+Alt+L, so there is no conflict for me.
Should I use a different key binding for Linux then?

And the winning combo is
    var keyBindings = [
        {
            key: 'Ctrl-Shift-L',
            platform: 'win'
        }, {
            key: 'Cmd-Shift-L',
            platform: 'mac'
        }, {
            key: 'Ctrl-Alt-L'
        }
    ];

Thank you!

I just hope that wont lock peoples desktop sessions, some might. "Ctrl-Alt-B" Might be a good alternative

Yes, that seems to be the case.
I think I will go with
var keyBindings = [
        {
            key: 'Ctrl-Shift-L',
            platform: 'win'
        }, {
            key: 'Ctrl-Alt-B',
            platform: 'win'
        }, {
            key: 'Cmd-Shift-L',
            platform: 'mac'
        }, {
            key: 'Ctrl-Alt-B'
        }
    ];

for now, therefore giving Windows user the choice.
And finally, you could still overwrite that in your user-key-mapping.

Awesome thank you so much!

I have released a new version, could you make sure it works for you?

All good here


Shoppe authentication should be able to be linked with the base application

See #167. You can create Customers with Addresses


util: bump kvproto version

What is changed and how it works?

Issue Number: Close https://github.com/tikv/tikv/issues/17513

What's Changed:

Check List
Tests

[x] Unit test
[x] Integration test

Release note

None

/check-issue-triage-complete

/test pull-unit-test

/test pull-unit-test

/retest

/approve


Adding meta tags for increasing Seo and changing category view in the publication

… publication
The documentation updated for ...
Link checker
The links will be checked by the build agent automatically once you create or update your PR.
You can use the following command to check the broken links locally.
docker run --rm -it --network=host --name=linkchecker ghcr.io/linkchecker/linkchecker --check-extern --no-warnings http://<IP_ADDRESS>:4000/

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