Find ways to avoid ceph CLI keyring error strings in logs

As we do not have ceph keyring files on the CSI containers, CLIs that fail and are reported in the logs have additional errors calling out the same. This is a little noisy when debugging issues and is at times misleading. Need to find ways to determine how to let Ceph CLI commands ignore the absence of a keyring.
Sample output:
 -1 auth: unable to find a keyring on /etc/ceph/ceph.client.admin
.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,: (2) No such file or directory

Does that mean you're executing a ceph command without a key? Can you describe in under which circumstances you are getting this error?

For example the following command from within the rbdplugin provisioner container would throw such warnings(?) but proceed. These get captured by the plugin when executing these commands from the code and pollute the logs.
$ kubectl exec -it csi-rbdplugin-provisioner-0 -c csi-rbdplugin bash
[root@csi-rbdplugin-provisioner-0 /]# rbd snap rm --pool replicapool --snap id1 image1 --id admin -m <IP_ADDRESS>:6790,<IP_ADDRESS>:6790,<IP_ADDRESS>:6790 --key=AQC50NBcusHYGBAAJmSLm9Aw98mBRT5FyeS1/w==
2019-05-13 11:12:31.544093 7f4532f68d40 -1 auth: unable to find a keyring on /etc/ceph/ceph.client.admin.keyring,/etc/ceph/ceph.keyring,/etc/ceph/keyring,/etc/ceph/keyring.bin,: (2) No such file or directory
rbd: error opening image image1: (2) No such file or directory

We do have a minimal ceph.conf in the code, but providing the same does not make the logs go away, is that a way forward? Does it need any further options in it?

@ShyamsundarR Until the CSI gets to the point where it no longer passes the key on the command-line, you can just create an empty "/etc/ceph/ceph.conf" to remove the log message.


Cannot change Switch state in test cases, fireEvent has no effect

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
onChange is not triggered
Expected Behavior 🤔
onChange should be triggered
Steps to Reproduce 🕹

it("should trigger onChange", () => {
    const onChange = jest.fn();

    const wrapper = render(

        Switch
        <Switch id="mySwitch" name="mySwitch" onChange={onChange} />

);

    const switchElement = wrapper.getByLabelText("Switch");

    fireEvent.change(switchElement, {target: {checked: true}});
    // Test fails
    expect(onChange).toHaveBeenCalledTimes(1);
});

Context 🔦
I have a list of labels with switches. When a label is clicked the related switches state should be toggled.
All labels are correctly linked to their switches via htmlFor and all inputs have the correct ids.
Your Environment 🌎

`npx @material-ui/envinfo`
 System:
    OS: Windows 10 10.0.19042
  Binaries:
    Node: 14.17.0 - C:\Program Files (x86)\nodejs\node.EXE
    Yarn: 1.22.5 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.14.5 - C:\Program Files (x86)\nodejs\npm.CMD
  Browsers:
    Chrome: 91.0.4472.77
    Edge: Spartan (44.19041.964.0), Chromium (91.0.864.41)
  npmPackages:
    @emotion/styled:  10.0.27
    @material-ui/core: ^4.11.4 => 4.11.4
    @material-ui/styles:  4.11.4
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react: ^17.0.0 => 17.0.5
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    styled-components: ^5.3.0 => 5.3.0
    typescript: ^4.1.2 => 4.2.4

Ive found issue #17697. I thought it wasn applicable for my case in the first case but the key is not the getByRole but the fact that you have to trigger click and change.
The following test works:
it("should trigger onChange", () => {
    const onChange = jest.fn();

    const wrapper = render(

        Switch
        <Switch id="mySwitch" name="mySwitch" onChange={onChange} />

);

    const switchElement = wrapper.getByLabelText("Switch");
    fireEvent.click(switchElement);
    fireEvent.change(switchElement, {target: {checked: true}});
    // Test fails
    expect(onChange).toHaveBeenCalledTimes(1);
});


[Das_Schwarze_Auge_4-1] Hotfix: Talent checks broken

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Fix missing braces and equal sign (d459376b4651d30f4f1a381775f1fbe344f9f271)
Bump version number, add info on new version, regenerate sheet files (d459376b4651d30f4f1a381775f1fbe344f9f271)

Character Sheet Info Roll20 Internal Use only.


Automatically redirect logged in user

LiveURL: https://kiwicomsmart-faq-jpsndiefpv.now.sh

fixes #137

Please add the issue that this is fixing, I believe it's #137 in this case?
Also, IMHO, while this will work well I believe it's worth to think about making a HOC out of this logc, something like requireLoggedOut. Then export a requireLoggedOut-decorated IntroPage component, will will redirect upon mounting if the user is logged-in.
This will allow us to apply the same logic to other components/screens that only should be rendered when there is no session present.

@kantord Agree requireLoggedOut is awkward :sweat_smile: The name was just a suggestion, also loggedOutPage or sth alike would convey the point.
Not sure I understand your login page suggestion, what would it do exactly? Check that it's logged-in or logged-out? Redirect where? Or parameterized redirection?

Well, normally you'd see something like requireLoggedIn which would redirect to a login page if the user is not logged in, then the login page would redirect back to the original page after login.
In our case we have something similar, but it's more a switch, because we don't require the user to log in.
Eh. That also sounds awkward and confusing... Actually I'm content with the current solution :-D

Yes but the requirement for this page is for the user to be logged out, not logged-in.
I think this same logic(ie. requiring logged-out to be rendered) applies to other screens as well, so making an HOC out of it feels better.

hmm, login is required for the upcoming booking part only, though, isn't it?

Please ignore what I just said, ofc we are talking about requireLoggedIn not  loggedOut.
I was thinking about a completely different topic :smile_cat:

Ok now?


STOR-1767: Delete resources if spec.ManagementState is equal to Removed for removable CSI drivers

The first commit bumps library-go to a newer version with https://github.com/openshift/library-go/pull/1741. The second commit updates operator driver_starter.go to delete static assets if ManagementState equal to Removed.

/retest-required

We don't have go 1.22.3 in CI yet :-(
https://issues.redhat.com/browse/ART-9927

/retest-required

/retest-required

/retest-required

/hold
This PR will wait for a small change in CSO's csidriveroperators/vsphere/08_deployment.yaml asset:
+        - name: OPERATOR_NAME
+          value: vmware-vsphere-csi-driver-operator

/test e2e-vsphere-csi

/test e2e-vsphere

/lgtm

/unhold
/label docs-approved
/label px-approved

/hold
Waiting for https://github.com/openshift/library-go/pull/1760 to merge. This PR will need update (bumping library-go) after it merges.

New changes are detected. LGTM label has been removed.

Two updates from the latest review:

go get github.com/openshift/library-go to pickup latest change in library-go
added import "k8s.io/klog/v2" to driver_starter.go because another PR recently removed it

/retest-required

/unhold
(https://github.com/openshift/library-go/pull/1760 merged)

/lgtm

pre-merge verified with open PR:

Cluster admin sets vSphere’s ClusterCSIDriver spec.managementState: Removed

vSphere CSI driver operator un-installs the CSI driver and deletes the related resources
Storage CO should be healthy
All existing PVs, PVCs, StorageClasses and vSphere StoragePolicies stay as they are

vSphere PVs cannot be used in new pods
vSphere PVs stay mounted + attached forever to existing nodes for existing Pods.
These pods stay Terminating forever after deletion.
StorageClasses could be removed

Cluster admin sets vSphere’s ClusterCSIDriver spec.managementState: from Removed back to Managed

vSphere CSI driver operator installs the CSI driver back.
The CSI driver works well

Performed one upgrade from one cluster with managementState:Removed to another version with the PR, the managementState stays Removed and no CSI Driver is installed

/test e2e-vsphere-zones

/label qe-approved

/retest-required

/retest-required

/retest-required

/retest-required

OK, now it's clear why e2e-vsphere and e2e-vsphere-csi failed:
E0724 19:50:08.797495       1 controller.go:974] error syncing claim "4719b799-f7cb-4a72-bc85-c56f252adf28": failed to provision volume with StorageClass "thin-csi": error generating accessibility requirements: no topology key found on CSINode ci-op-ztk028vs-5748b-pvbrp-worker-0-9c8h5

It's useless to /retest-required until that revert merged

/retest

/retest


Refactor JIT code for readability

There are a lot of long and confusing functions that can be factored into smaller bits.

Good as it gets, for now.


Add shape aggregator based on fuse

For data shape exploration, it is useful to be able to view the union of type descriptors for a set of records.  For example, when coming up with the json type schema for suricata logs, I wrote hacky one-off scripts to do the equivalent of
what fuse (typeof(.)) by alert.signature, alert.category. Maybe the aggregator will be called shape rather than fuse.
@mccanne prototype this on a branch, but it had bug https://github.com/brimsec/zq/issues/1959 and stayed unmerged. Once we fix #1959, the aggregator should follow easily.

Verified in zq commit 7ca6e7ec.
@henridf tipped me off to a good real-world use case for this aggregator. At the moment, Brim applies a single unifying schema to all Suricata alerts that are generated from imported pcaps. However, we know from observation that certain categories of alert have unique sets of fields. If you wanted to see the unique schemas by category (such as if you wanted to create a shaper config that stored each with its minimal schema), you could do this with an EVE JSON:
$ zq -version
Version: v0.29.0-41-g7ca6e7ec

$ zq -f zson 'fuse(alert) by alert.category' eve.json
{
    alert: {
        category: "Not Suspicious Traffic"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64,metadata:{updated_at:[string],former_category:[string],created_at:[string]}})
}
{
    alert: {
        category: "Generic Protocol Command Decode"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64})
}
{
    alert: {
        category: "Attempted Administrator Privilege Gain"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64,metadata:{updated_at:[string],signature_severity:[string],performance_impact:[string],former_category:[string],deployment:[string],created_at:[string],attack_target:[string],affected_product:[string]}})
}
{
    alert: {
        category: "Unknown Traffic"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64,metadata:{updated_at:[string],former_category:[string],created_at:[string]}})
}
{
    alert: {
        category: "Potentially Bad Traffic"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64,metadata:{updated_at:[string],signature_severity:[string],former_category:[string],deployment:[string],created_at:[string],attack_target:[string],affected_product:[string]}})
}
{
    alert: {
        category: "Misc activity"
    },
    fuse: ({action:string,gid:int64,signature_id:int64,rev:int64,signature:string,category:string,severity:int64,metadata:{updated_at:[string],former_category:[string],created_at:[string]}})
}

Thanks @henridf!


x/vuln/vulncheck: TestCommand, TestBinary, TestExtractPackagesAndSymbols, TestLineFromAline failing after compiler generated symbol change

https://build.golang.org/?repo=golang.org%2Fx%2Fvuln shows test failures on Go tip as of CL 317917 ("all: use ":" for compiler generated symbols").
CC @golang/vulndb, @cuonglm.

@jba @julieqiu


Steps to Test the Whole Project?

Hello, is it possible to provide proper documentation or guideline to test the whole project?
Suppose, what are the pre-requisites and steps needed to follow?
Thanks.

Dear @cserezwan ,
Honestly, we are not planning to prepare any additional materials. We believe that the paper itself and all READMEs are enough to reproduce our experiments. Since this is not a production code but a research code, we don't make it workable in each possible environment. However, if you meet any issues, feel free to ask for our help!
Note that the README in the root folder isn't the onliest. Other READMEs are in subdirectories.


Current components scrolling bug

In the level editor, inside components, when I scroll down (by hovering the mouse over the names), the scrolling bar doesn't update.
On the other hand, when I scroll down (by hovering the mouse over the scrolling bar), both the scrolling bar and the names scroll at the same time.

I think this is the same as #2583.

What I meant was that, in the level editor (components tab), this happens:
Initial state:

If I scroll down (by hovering the mouse over the names), the scrolling bar doesn't update:

(here I have reached the bottom of the list, but the bar is still up there)
IfI scroll down (by hovering the mouse over the scrolling bar), both the scrolling bar and the names scroll at the same time:

(here I have reached the bottom of the list, and the bar came with the names)


X To Remove Tracking with automatic tracking enabled

Describe your suggested feature
Have the option to remove tracking with automatic tracking enabled, so if you remove something from your library that you never even read, you can remove it from the tracker as well.
Other details

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to the newest version Latest.
[X] I have checked through the app settings for my feature.
[X] I will fill out all of the requested information in this form.

As of right now, I dont plan to add this.  I don't want to have to track when a user manually removes the auto tracker and to not readd it (it gets added everytime you enter the manga details page unless it doesnt fit the content rating exclusion)

Could you make it be removable if you remove the manga from your library first?

that would be doable

2.10.2


Plot Twist: Introducing Training Metrics Visualization with Plotting_Utils in model.py for Deep Learning Insights

This pull request introduces a new utility class for visualizing training metrics (loss and accuracy) in the Multi-Layer Artificial Neural Network (ANN) model. The enhancements aim to provide users with visual feedback on model training and performance, facilitating better tracking and debugging.
Key Changes:
1.Added Plotting_Utils Class:

Introduced a utility class for plotting training and validation metrics.
Implemented a method plot_training_history() that:
Accepts a history dictionary containing training and validation loss/accuracy.
Utilizes Matplotlib for visualizations.
Allows users to specify which metrics to visualize (loss and/or accuracy).
Saves generated plots as images.

2.Enhanced Multi_Layer_ANN Class:

Modified the training loop to store training and validation metrics in a history dictionary for later visualization.
Ensured` compatibility with the new plotting utility.

3.Updated runner.py:

Integrated` the Plotting_Utils class to plot the training history after model training.
Provided examples of how to use the plotting utility with the training history data.

4.Requirements Update:

Added Matplotlib to the requirements.txt file to ensure that the necessary libraries for plotting are included in the project dependencies.

Benefits:
Users can visually track model performance over epochs, making it easier to identify trends and diagnose potential issues during training.
Improved overall user experience by incorporating visual feedback into the training process.

Testing:
The new plotting functionality has been tested, ensuring that the plots accurately reflect the training metrics and are saved as specified.

Hello @ravin-d-27 I have resolved the issue https://github.com/ravin-d-27/PyDeepFlow/issues/13 If there are any changes or further adjustments needed, please let me know. I would appreciate it if you could accept my pull request.

Thank you @SujitSakore  for your contribution !

Please consider starring the repository !


Add cuda 11.2 driver and runtime inc files

CC @sanjoy

The test failure looks unrelated to the changed made in the PR.

The test failure looks unrelated to the changed made in the PR.

TF/xla with CUDA 11.2 gives the following warning:
warning: Linking two modules of different target triples: '/usr/local/cuda-11.2/nvvm/libdevice/libdevice.10.bc' is 'nvptx64-nvidia-gpulibs' whereas '' is 'nvptx64-nvidia-cuda'
cc @jpienaar @timshen91

TF/xla with CUDA 11.2 gives the following warning:
warning: Linking two modules of different target triples: '/usr/local/cuda-11.2/nvvm/libdevice/libdevice.10.bc' is 'nvptx64-nvidia-gpulibs' whereas '' is 'nvptx64-nvidia-cuda'
cc @jpienaar @timshen91

@byronyi We have not yet fully tested TF with 11.2 so some failures are expected.  Having said that, that particular issue is probably a false positive, we should not print this warning.  We can either remove the diagnostic printer while linking (it is created and added here), or change the printer to filter out just the linker diagnostics.

@byronyi We have not yet fully tested TF with 11.2 so some failures are expected.  Having said that, that particular issue is probably a false positive, we should not print this warning.  We can either remove the diagnostic printer while linking (it is created and added here), or change the printer to filter out just the linker diagnostics.

@sanjoy Except this particular warning, the CUDA 11.2 build seems pretty fine for the model garden RN50.

@sanjoy Except this particular warning, the CUDA 11.2 build seems pretty fine for the model garden RN50.


Cycle in the constraint graph, involving Boost.Regex and Zlib

Getting a Cmake warning:
VCPKG/scripts/buildsystems/vcpkg.cmake:639 (_add_library):
  Cannot generate a safe runtime search path for target TargetName
  because there is a cycle in the constraint graph:

dir 0 is [/Users/Development/Build/vcpkg_installed/arm64-osx-dynamic/debug/lib]
      dir 1 must precede it due to runtime library [libz.dylib]
    dir 1 is [/Users/Development/Build/vcpkg_installed/arm64-osx-dynamic/lib]
      dir 0 must precede it due to runtime library [libboost_regex.dylib]

Environment

OS: [macOS]
Compiler: Apple Clang
arm64-osx-dynamic

Steps to reproduce the behavior:

Install OpenImageIO package, that in turn pulls in Zlib.
Install Boost package.
call find_package(OpenImageIO REQUIRED) and find_package(Boost REQUIRED COMPONENTS regex)
Link to OpenImageIO::OpenImageIO and Boost::regex targets.

Expected behavior
No warning should be produced

dir 0 is [/Users/Development/Build/vcpkg_installed/arm64-osx-dynamic/debug/lib]
dir 1 must precede it due to runtime library [libz.dylib]
dir 1 is [/Users/Development/Build/vcpkg_installed/arm64-osx-dynamic/lib]
dir 0 must precede it due to runtime library [libboost_regex.dylib]

There seems to be an unexpected cross-linking between dylibs from debug into release or from release into debug.
Maybe with additional context the offending lib/port would be obvious.

Where should O point my attention? To Zlib/Regex portfile.cmake? Somewhere else? Not sure where to start here

On linux, I would look at the relevant dylibs with ldd. Somewhere in this chain is the bad one, and the providing port must be fixed.

Sorry for slow responses. Lots have piled up. Here is the analysis of the linked libs.
The two libraries in question: Zlib and Boost.Regex:
Regex DEBUG:
Library: /Users/Development/Project/Build/macos-openfx-debug/vcpkg_installed/arm64-osx-dynamic/debug/lib/libboost_regex.dylib Linked Libraries:   @rpath/libboost_regex.dylib   /usr/lib/libc++.1.dylib   /usr/lib/libSystem.B.dylib @rpath Values:
Regex RELEASE:
Library: /Users/Development/Project/Build/macos-openfx-debug/vcpkg_installed/arm64-osx-dynamic/lib/libboost_regex.dylib Linked Libraries:   @rpath/libboost_regex.dylib   /usr/lib/libc++.1.dylib   /usr/lib/libSystem.B.dylib @rpath Values:
Zlib DEBUG:
Library: /Users/Development/Project/Build/macos-openfx-debug/vcpkg_installed/arm64-osx-dynamic/debug/lib/libz.dylib Linked Libraries:   @rpath/libz.1.dylib   /usr/lib/libSystem.B.dylib @rpath Values:
Zlib RELEASE:
Library: /Users/Development/Project/Build/macos-openfx-debug/vcpkg_installed/arm64-osx-dynamic/lib/libz.dylib Linked Libraries:   @rpath/libz.1.dylib   /usr/lib/libSystem.B.dylib @rpath Values:
These even do not have rpaths set in them, so they cannot "go out" of their respective directories to cross-point to a dylib in another directory (debug->release, release->debug).
There were no dylibs found that would contain rpaths with ../.., that would allow it to go outside the debug/lib directory into lib and do ../debug that would allow release libs to point to debug libs.
TL;DR I do not see cross links


Send detailed volume data on instance.activate

So that we can create volumes on-demand. Also, fix back-population to
account for volumes that don't yet belong to a storage pool.
Also, set externalId on user created volumes to the value of the name
field.
Needs a python-agent update before it can be merged.

LGTM

LGTM


Question: order rate

Hello. Is it possible to manually set rate for a poloniex order? Thanks

Hi, you can set the price for limit orders (tradeService.placeLimitOrder(limitOrder), see also LimitOrder class).


Slate doesn't provide an initial base theme to upload.

Problem
I created a new Slate V1 theme to connect to a new dev store, as you know, new stores come with Debut pre-installed. So when creating the new theme, there's no base Slate theme to upload as there was with V0. There was also a message saying Slate will overwrite whatever theme I've configured Slate to deploy to, I assumed this meant it would erase the contents of Debut and push it's own files. Instead, it's simply added the new files to the Debut theme, so I'm using some pre-existing sections that came with Debut but with no styling for that section.
I'd propose that on the initial yarn create whatever-theme, the script creates both the Slate-compatible theme plus an initial .zip theme build to upload to the store to connect to.

Upvote for this matter as this is a blocking situation for theme developers.
The fact that we can't use slate zip with V.1 is frustrating :/

I used to get this issue in the very first time using Slate. Then I think
step that i missed before connecting the store with theme ID is uploading
Slate starter-theme from my computer to Shopify. So instead of using Debut
we can use Slate starter-theme from our computer by using theme ID that we
uploaded to Shopify.
On Tue, Jul 31, 2018 at 1:06 AM nemoeric<EMAIL_ADDRESS>wrote:

Upvote for this matter as this is a blocking situation for theme
developers.
The fact that we can't use slate zip with V.1 is frustrating :/
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Shopify/slate/issues/603#issuecomment-408956921, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAvbjC5lCg0qJP7K70CXRaV-rPsPbA4Tks5uL0slgaJpZM4UQbNL
.

--
Sent from Gmail for iOS

I usually fill out SLATE_STORE and SLATE_PASSWORD in the .env file and then run yarn zip to generate the blank theme, upload that to the store, grab the theme id for SLATE_THEME_ID then run yarn start. It's an extra minute at the start of a project, but it works!

Trying to get started with slate and this was a massive barrier to entry - there is no suggestion in the docs to zip and upload the starter theme to your store before connecting slate to your store. Following the steps in the docs resulted in connecting and deploying to the Debut theme and a very mashed up confusing preview!
Suggest the following docs update on https://shopify.github.io/slate/docs/connect-to-your-store:
"Now that you have a new Slate project, you need to compile the contents of the dist directory and create a ZIP file which you can upload as a theme to your remote Shopify store. You can then link your project to this theme on your remote Shopify store by updating the .env file with your API credentials.
To compile the contents of the dist directory and create a ZIP file of your theme in the root of the project
yarn slate-tools zip

Add this theme ZIP file to your remote Shopify store using the 'Upload theme' button under 'More themes' in the Themes section of the store admin."

Wasted half an hour trying to wrap my head around the docs. Thanks for the suggestions!


How to dynamically hide and show material tab?

What is the current behavior?
enable disable tab functionality is available but i couldnt find hide/show features

Does putting an ngIf on the tab do what you want?

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate

This is just a feature request, so that we are able to show/hide mat-tab elements. Very useful and needed feature for sure.
But, what kind of investigation is needed to understand this simple request? I think closing this is just a comfortable way to not solve the problem. :(


Thumbnail images overflow becuase of border width

How can we reproduce this bug?

What did you expect to happen?
Image wouldn't overflow it's containing row.
What happened instead?
It did.
What email clients does this happen in?
Tested on chrome, so likely all email clients

This will be a feature - to bring in Thumbnail styles that work on every device.

Closing in favor of https://github.com/zurb/foundation-emails/issues/220


Bug: $getAuditLogs doesn't return target ID

Bug Type
Function
Bugged function
$getAuditLogs {target.id} doesn't return a user ID

Version of aoi.js
v5.5.5 (Default)
Version of Node.js
v18.6.0+ (Latest)
Excepted Output
It returns some random numbers but it isn't a user ID
Relevant log output
No response
Code of Conduct

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

fixed it in : b873d83e3ea5c300a26cfcf271a3bebf1422936f


App Wall: Filter multi select

WIP
Depends on #1841

@richard-cox can we close this and tackle it again another time?

Closing. Will take another look once the cf/org/space service is refactored which will make a lot of this obsolete.


Correction for raw affiliation Brain and Mind Institute, Western University

Correction needed for raw affiliation Brain and Mind Institute, Western University
raw_affiliation_name: Brain and Mind Institute, Western University
new_rors: 02grkyz14
previous_rors: 00we1gw23;05cgtjz78
works_examples: W4366083753;W4389078325;W4372286277;W4386249051
contact: 29b96ec825b8f05a4117adbf600afe8d:b8f7b60c33b93f5e @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


GPU Supported SDL Backend

Hi,
It seems this PR was forgotten.
I resolved the conflicts and marked this PR as ready. Is it ready to merge?

The merge looks good to me!

I've just tested it it still crashes with this initialization:
  sdl_disp_drv_init(&disp_drv, SDL_HOR_RES, SDL_VER_RES); /*Basic initialization*/
  lv_disp_t * disp = lv_disp_drv_register(&disp_drv);

WARN:

Assertion failure at lv_draw_sdl_rect_bg_frag_obtain (../lvgl/src/draw/sdl/lv_draw_sdl_rect.c:156), triggered 1 time:
  'texture'

Do you what can be still missing?

@kisvegabor I think this is because lvgl/lvgl#3167. I have created another PR https://github.com/lvgl/lvgl/pull/3280 to revert the change, after the change, sample app will run.

Thank you, it's working now.
But there are some minor issues :slightly_frowning_face:
The widgets demo looks like this:

Instead of:

So

All texts are black (some should be white)
No images

Is this on Linux? On Windows (with MinGW) it works fine. My code is based on lv_sim_vscode_sdl.

Yes, I'm on Linux. Do you use the latest LVGL?

I think they are quite new.
 2d7f3f619b447c97f2af825b82a75611124f5e04 lv_drivers (v8.1.0-16-g2d7f3f6)
 6a179e2c8e67afbcd30fbd7a1f6134799e602828 lv_examples (v7.11.0-120-g6a179e2)
 9997fb00aa60b4478c76fa8387a74ca5b3c595b2 lvgl (v8.2.0-173-g9997fb00a)


Selection of images in Table 1

I've sorted the images in Table 1 according to waveband, it now looks like this:
https://github.com/gammapy/gammaskynet-gamma2016/blob/master/proceeding/tables/image_table.tex
The MWL data that's most relevant to compare with gamma-ray data is other non-thermal emission.
That means mostly radio and X-ray.
Currently there's no X-ray image.
Suggest you add one for ROSAT and XMM here (we can add it to the webpage later).

I think this is the best ROSAT image they have:
http://cade.irap.omp.eu/documents/Ancillary/4Aladin/RASS/

For XMM Newton it's this one:
http://skies.esac.esa.int/XMM-Newton/EPIC-RGB/
Yes, the X-ray images don't look so nice.
But they are the most relevant for gamma-ray astronomy, because hard X-ray emission is nonthermal emission, i.e. also shows the cosmic particle accelerators.
There is no great X-ray survey yet.
We could mention that e.g. http://www.mpe.mpg.de/eROSITA will be available in the coming years and then also some day on gamma-sky.net.

If you want to mention more future data and surveys, you could also mention that CTA is coming and will do nice surveys of the TeV sky:
http://adsabs.harvard.edu/abs/2013APh....43..317D

ROSAT image added to Image Table.

XMM-Newton image added to Image Table.

Referenced CTA in the introduction.

But I did not mention that http://www.mpe.mpg.de/eROSITA (yet).
Closing this issue...


Question: Register multiple messages in protobuf file

It seems to me that if there are multiple messages defined (cases of both dependent and independent messages) in one single protobuf file, wonly the first one will/can be registered. Is this correct? If so, is there any one in this tool that can help register all schemas? @domnikl

A related question, how can one register a schema with imports from external packages, e.g.
import "google/protobuf/wrappers.proto";

message ExampleFeatureGroup {
  int64 foo_count = 1;
  google.protobuf.StringValue string_value = 2;
}

getting error
ERROR io.confluent.kafka.schemaregistry.client.rest.exceptions.RestClientException: Either the input schema or one its references is invalid; error code: 42201

Hi @pjthepooh sorry took some time to get back to you ... life happened 😄  I took a simple test and added another message to the example foo.proto, ran apply and got the exact same schema containing both messages  back with dump.
I also ran your second example and it just works for me. Which version of the schema registry are you using and what does your YAML file look like?

Closing because of inactivity.


v0.1.3

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/55695
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" fa9cc39562ceeda4423a248aa6237312aad8f62d
git push origin v0.1.3


test-cli docs update

Updated the test-cli docs

Codecov Report

Merging #152 (49348de) into main (d693f5c) will increase coverage by 0.52%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   70.00%   70.52%   +0.52%
==========================================
  Files           6        6
  Lines         570      570
==========================================
+ Hits          399      402       +3
+ Misses        139      137       -2
+ Partials       32       31       -1

Flag
Coverage Δ

unittests
70.52% <ø> (+0.52%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

server/service.go
72.43% <0.00%> (+1.06%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d693f5c...49348de. Read the comment docs.


Replace Report globally

In Directions there was talk About replacing Reports globally in a System via extensions. I can't find any Mention in the documentation, how would we Need to go About that?

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

ID: 185e15dd-e5ae-2272-7d29-3207bdd08089
Version Independent ID: dc0cdfd9-7480-6cc4-8c9e-db6ac05f2f6c
Content: Report Design Overview - Business Central
Content Source: dev-itpro/developer/devenv-report-design-overview.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hi! Thank you for your feedback! You can find more information about  "Report substitution" in this blog post but also in the "NAV TechDays 2018 - What’s new in Developing for Business Central" video (from minute 24).
Thanks!

Could you Maybe provide this Information in the documentation? If we have to search the documentation, several Blogs and even look for it in external 90 Minute Videos it's a bit complicated to find what we Need.

Fully agree with Julian. A docs topic is needed on this.

I do also agree, please provide information directly, rather than pointing out links, which might be broken at any time.


define the "inner_mask" for locstream case

Define the inner_mask in subset_shape() for the locstream case.
subset_shape() would return all lat/lon inside inner_mask which is equivalent to the bounding box of the polygon, not the area inside the polygon.

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes a bug in issue #288 ; subset_shape(...) would return all points in the bounding box without filtering the points inside the polygon.

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:
The inner_mask in subset_shape() is defined for the locstream case.

Does this PR introduce a breaking change?:
No.

Other information:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 78.485%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1109

Relevant Lines:
1413

💛  - Coveralls

@vindelico Do you have any free time to work on/finalize this PR? I have use cases that could benefit from your changes!

@vindelico Do you have any free time to work on/finalize this PR? I have use cases that could benefit from your changes!

Can you check HISTORY.rst and AUTHORS.rst, plz? - otherwise I don't have additional changes.


Notifier not fetching all proposals due to RPC rate limits

The governance notifier is currently using the public RPC endpoint and is being rate limited on the getProgramAccounts instruction.
There is no retry logic when this happens the data it uses is incomplete. I think it either needs to implement retry or throttling behaviour, or use a different endpoint.
public rpc - api.mainnet-beta.solana.com
Unexpected response {"jsonrpc":"2.0","error":{"code":429,"message":"Too requests for a specific RPC call, contact your app developer or support@rpcpool.com."},"id":1}

-- countJustOpenedForVoting: 0, countVotingNotStartedYet: 19, countClosed: 43

genesysgo rpc
-- countJustOpenedForVoting: 0, countVotingNotStartedYet: 20, countClosed: 45
theindex.io
-- countJustOpenedForVoting: 0, countVotingNotStartedYet: 20, countClosed: 45

NB: there is no noticeable speed difference between the requests made to theindex.io and other RPC services in this instance

Ah, I just found something in the code that might fix it. BRB!

yeah, we never bother in working around rate limits and always just set a custom RPC via env. feel free to add another default that works better.

yeah, we never bother in working around rate limits and always just set a custom RPC via env. feel free to add another default that works better.

before seeing this comment I posted PR that fixes rate limits here #104
or an faster alternative WIP PR if you're planning to use private RPC infra here #105


add missing implementation of the hypercall UHYVE_PORT_EXIT

The hypercall was missing for MacOS on aarch64.

bors r+

bors r+

bors r+


Some check_config enhancement

Signed-off-by: Qiang Huang<EMAIL_ADDRESS>

ping @tianon

ping @tianon could you have a look at this?

LGTM assuming @LK4D4 is OK with the new requirement additions :+1:

LGTM

Thanks @hqhq !


tweak: Remove old Zip in case, to prevent any issues

Seems like Zip might've not updated/wrote in position of the file properly, so this removes the Zip just in case.
(I created this branch as the pull was accepted, and it wrote over Voxellius/Voxel/pull/5.)

Alright, sounds good! Will merge now :+1:


Add ability to add inputs to records table

I would like to have an ability to edit records directly in records list table.
Now, if you add the input to the table, on click event on the input , page is forwarding to record show action.
I see, that you have RecordInList handleClick handler with conditions:
    if (actionName
        && targetTagName !== 'a'
        && targetTagName !== 'button'
        && targetTagName !== 'svg') {
   }

May be I missed smth, and there is the right way to have the input in the table.

I was able to resolve click issue with onClick event  and event.stopPropagation();
However, prop onChange is undefined, as, of course, it was never designed to change data from record table list. I will use your ApiClient() to handle this.
Still think, this is might be good feature to have out of the box.


[Snyk] Fix for 2 vulnerabilities

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

gravitee-am-ui/package.json
gravitee-am-ui/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-ANSIREGEX-1583908
  159

Prototype Pollution SNYK-JS-MINIMIST-2429795
  59

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)
🦉 Prototype Pollution

3.21 is EOL


Support font name in a text element

Currently font name is not supported in a text element although it is supported in a markdown element. There are 2 possible patterns to support the property.
A. Add fontName property under text property

text (String, [langId:String] or ["ref":StringId]): Text to display

fontSize (Float or Percent): Font size
fontName (String or [String]): Font name or names (the first name existing in the system is used)

Advantage: This flat pattern matches the other property specifications like strokeColor and fillColor for path where they are not stroke and fill properties under color
B. Add name property under font property reorganized with size property

text (String, [langId:String] or ["ref":StringId]): Text to display

font:

size (Float or Percent): Font size
name (String or [String]): Font name or names

Advantage: This is the same pattern to specify a font in a markdown element, which uses SwipeParser.parseFont function to parse it.

I think the pattern A above is better as a property in an element because the specification of Swipe avoids deep nesting.
@Pstoppani: Which pattern to support font name looks better for you?

blockquote, div.yahoo_quoted { margin-left: 0 !important; border-left:1px #715FFA solid !important; padding-left:1ex !important; background-color:white !important; }  A - I like the cosistsency with othe property specs.W
Sent from Yahoo Mail for iPhone
On Monday, June 13, 2016, 06:00, Yoichi Tagaya<EMAIL_ADDRESS>wrote:
Currently font name is not supported in a text element although it is supported in a markdown element. There are 2 possible patterns to support the property.
A. Add fontName property under text property

text (String, [langId:String] or ["ref":StringId]): Text to display

fontSize (Float or Percent): Font size
fontName (String or [String]): Font name or names (the first name existing in the system is used)

Advantage: This flat pattern matches the other property specifications like strokeColor and fillColor for path where they are not stroke and fill properties under color
B. Add name property under font property reorganized with size property

text (String, [langId:String] or ["ref":StringId]): Text to display

font:

size (Float or Percent): Font size
name (String or [String]): Font name or names

Advantage: This is the same pattern to specify a font in a markdown element, which uses SwipeParser.parseFont function to parse it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @wojciechb2000!

Pattern A works fine for me

Implemented as #30.


First 2.0.0 spin with Direct Message to Module on Edge - not working

Took the brand new 2.0.0 for a spin (I was really looking forward to that) and it looks promising, but still no success:
MQTTTransportStage: _on_mqtt_disconnect called: Paho returned rc==1 caused by None
MQTTTransportStage: disconnection was unexpected
Exception caught in background thread.  Unable to handle.
['azure.iot.device.common.transport_exceptions.ConnectionDroppedError: None caused by Paho returned rc==1 caused by None\n']
MethodInvokeOperation: completing with error HTTP operation returned: 504 GatewayTimeoutError(Error: Gateway Timeout) caused by None
Callback completed with error HTTP operation returned: 504 GatewayTimeoutError(Error: Gateway Timeout) caused by None
azure.iot.device.exceptions.ServiceError: HTTP operation returned: 504 GatewayTimeoutError(Error: Gateway Timeout) caused by None
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/azure/iot/device/iothub/aio/async_clients.py", line 29, in handle_result
    return await callback.completion()
  File "/usr/local/lib/python3.5/dist-packages/azure/iot/device/common/async_adapter.py", line 93, in completion
    return await self.future
  File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
azure.iot.device.exceptions.ServiceError: HTTP operation returned: 504 GatewayTimeoutError(Error: Gateway Timeout) caused by None

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "dm_2.py", line 42, in <module>
    loop.run_until_complete(main())
  File "/usr/lib/python3.5/asyncio/base_events.py", line 466, in run_until_complete
    return future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/asyncio/tasks.py", line 241, in _step
    result = coro.throw(exc)
  File "dm_2.py", line 30, in main
    device_id="Haven2", module_id="motion-detection", method_params=fake_method_params
  File "/usr/local/lib/python3.5/dist-packages/azure/iot/device/iothub/aio/async_clients.py", line 466, in invoke_method
    method_response = await handle_result(callback)
  File "/usr/local/lib/python3.5/dist-packages/azure/iot/device/iothub/aio/async_clients.py", line 39, in handle_result
    raise exceptions.ClientError(message="Unexpected failure", cause=e)
azure.iot.device.exceptions.ClientError: Unexpected failure caused by HTTP operation returned: 504 GatewayTimeoutError(Error: Gateway Timeout) caused by None

Is the output, when running https://github.com/Azure/azure-iot-sdk-python/blob/master/azure-iot-device/samples/async-edge-scenarios/invoke_method_on_module.py inside a module on a device. Python 3.5.

Hey @njbuch let’s figure out what’s wrong! When you are invoking the method on the target device or module, is the client set up to respond to your specific method invoke parameters? A 504 timeout to me might hint at that. If you could post some anonymized debug logs of the run here too that would be helpful in addition to the stack trace already provided.

Hey @njbuch let’s figure out what’s wrong!
Thanks!

When you are invoking the method on the target device or module, is the client set up to respond to your specific method invoke parameters? A 504 timeout to me might hint at that.

I have other modules running on the device, and they are in C# and are receiving direct method calls nicely. There is also some C# code running in the same module that can receive direct method calls.
I am running the code linked in the OP out of the box to check that it works, except I have correct the device-name and module-name. So I am calling the "doSomethingInteresting" on the device and module I have specified. Is that what you mean by setup?

If you could post some anonymized debug logs of the run here too that would be helpful in addition to the stack trace already provided.

Not sure what you mean, I run "python3 sample_code.py" and the output is above.

For the logging, add the following lines to your sample code at the very top:
import logging

logger = logging.getLogger('njbuch_app')
logger.setLevel(logging.DEBUG)

And, just for due diligence, when you're adjusting the code, you're adding in your respective device_id and module_id, and editing the fake_method_params object so that the methodName is a method that you have enabled on the C# module that is receiving method calls presently? I assume the answer is yes, but want to make sure.

Argh. I misunderstood the purpose. I need code that listen to a method Call inside a module. Sorry. My mistake.

@YoDaMa, @njbuch, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


OGNL-102 Improves performance when null was returned

This PR introduces options to short-circuit chain operations on null result in case if accessing property of the object, eg. obj1.property where obj1 is null or obj1.obj2.property where obj2 is null.
This functionality is disabled by default, use -Dognl.chain.short-circuit=true to test it.
Refs https://github.com/orphan-oss/ognl/pull/140
Refs OGNL-102

@davoustp @harawata @quaff could you take a look? This is another approach to improve performance, it's in relation to #138 and the JIRA ticket. @harawata I have added your test cases from #140 and they passed.


nitro apply exit status 2 on windows

Using develop, have a machine successfully running with:
php: "7.4"
mounts:
- source: D:\dev\nitro
  dest: /home/ubuntu/sites/site.test
sites:
- hostname: site.test
  webroot: /home/ubuntu/sites/site.test/www

Change to:
php: "7.4"
mounts:
- source: D:\dev\nitro
  dest: /home/ubuntu/sites/site2.test
sites:
- hostname: site2.test
  webroot: /home/ubuntu/sites/site.test/www

Run nitro apply and get Error: exit status 2

Resolved this in beta 7.


{{#block}}{{#if}}... that returns None throws error

The bug
{{#if}} blocks can return None if their condition evaluates to False. If this occurs inside a {{#block}}, the block will throw inside strip_markers, because you can't run a regex against None.
To Reproduce
def test_empty_block():
    """ Test the behavior of a completely empty `block`.
    """

    prompt = guidance(
        "{{#block}}{{#if nonempty}}{{nonempty}}{{/if}}{{/block}}",
    )
    out = prompt(nonempty=False)
    assert out.text == ''

  File "/usr/local/lib/python3.8/dist-packages/guidance/library/_block.py", line 25, in block
    parser.set_variable(name, strip_markers(out))
  File "/usr/local/lib/python3.8/dist-packages/guidance/_utils.py", line 91, in strip_markers
    return re.sub(r"{{!--G.*?--}}", r"", s, flags=re.MULTILINE | re.DOTALL)
  File "/usr/lib/python3.8/re.py", line 210, in sub
    return _compile(pattern, flags).sub(repl, string, count)
TypeError: expected string or bytes-like object

Error in program:  expected string or bytes-like object

System info (please complete the following information):

OS (e.g. Ubuntu, Windows 11, Mac OS, etc.): Ubuntu/WSL2
Guidance Version (guidance.__version__): 0.0.57

I didn't attempt a more general fix for if because I had assumed that you might want an if block to return None. Frankly I'm not clear on the semantics of when blocks should return values vs. when they should add to the executor prefix.


Don't export functions from special types like Allocator, Promise, and AbortSignal

Decls of Allocator are needed actually.


Current import functionality does not match description

The bullet point below from the important points section under "Bulk add or edit bookmarks" does not seem to be correct. When I try it in my tenant, bookmarks that match existing titles and/or URLs are listed as errors that prevent import.
"If there is an existing bookmark with the same title or URL, the bookmark will be updated with information in the import file."

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

ID: 86601b66-7664-6062-d56b-70e53dfbdc36
Version Independent ID: 4b17efd0-658b-9a68-04d3-ea1aaa0a619c
Content: Manage bookmarks
Content Source: MicrosoftSearch/manage-bookmarks.md
Service: mssearch
GitHub Login: @adefowler
Microsoft Alias: anfowler

Thanks! @ekdegerman - We'll get this updated! I'll let you know when it's done (probably November 4th).

Thanks for the feedback @ekdegerman. We have updated the content on the page to better reflect the import behavior and how to prevent errors and duplication.


Clicking the linked version in the footer results in a 404 on GitHub

If I click the linked text in the footer reading MapRoulette v3.7.8-rev1, I'm brought to the following page, but this is an invalid URL, and GitHub returns a 404.

The true version is v3.7.8rev1.  A - is being added to the URL mistakenly.  This will be resolved in our next patch, but for now please remove the hyphen and you should be directed to the right webpage.


osv-scanner: Suggest fixes

Automatically suggest the minimal package upgrades required to remediate all vulns.

This is done for npm: https://google.github.io/osv-scanner/experimental/guided-remediation/ and Maven is in progress: https://github.com/google/osv-scanner/issues/1141


Allow enumerations to be of any base numeric type

This allows enumerations to be of any base numeric type.
The downside is that there is no general value that can be used for "invalid enumeration", so this has to change the interface of EnumOf to be (T, bool) instead of returning an invalid value. This is arguably more Go-style anyhow. Plus it is also just generally true that you can't pick an enumeration value to be a generic "you can't use this" because you don't know what users may want.
Not necessarily suggesting you take this, this was just easier than trying to describe what I meant in /r/golang.
The "mtmap.go" is stripped of docs since it is not exported, but it's an adaptation of mtmap. As per the README itself, it's not something you use as a library exactly, but best copied & pasted & modified as needed. In this case, the key type changed quite substantially for your use case. This allows for an enums map that stores everything directly, and strongly-typed. However, the API described in the docs is still effectively the same.
Some more propagation of "there isn't really an invalid value for enum" may be useful.

Allow me to change and merge the PR. Thanks very much!


Update scalafmt-core to 3.2.0

Updates org.scalameta:scalafmt-core from 3.1.2 to 3.2.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.1.2).
You might want to review and update them manually.
website/yarn.lock

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains

Superseded by #94.


Replaced usage of CMAKE_SOURCE_DIR with AWS_NATIVE_SDK_ROOT

This will allow the SDK to be pulled into another project via
add_subdirectory. Note that there are still some occurrences of
CMAKE_SOURCE_DIR left in the android build.
This is a partial fix for issue #509.

Merged


Making the script more user friendly

Now accepts input when run for each environment variable to be set and gives the user an option to create the variables at either machine or user level.

Looks, good!  Thanks, Nick - you're now a GitHub contributor! Yay!

Lol, not my first time, but I'm trying to do it more often :)
On Fri, Aug 28, 2015 at 9:19 AM, Doug Seelinger<EMAIL_ADDRESS>wrote:

Looks, good! Thanks, Nick - you're now a GitHub contributor! Yay!
—
Reply to this email directly or view it on GitHub
https://github.com/dseelinger/ms-idm-net/pull/1#issuecomment-135773535.


fix: toc issue with remark-autolink-headings

TOC won't work if remark-autolink-headings option behavior is set to append

Thanks, makes sense to me. You are running a version with remark-autolink-headings set to append?

You're welcome. Yes, and I couldn't get the TOC to work until I made those changes.


make missing fraction an optional warning

currently implemented as print statement (which can become annoying) --> calc_vector function in vector_operations.py

Fixed by adding logging in #73


Primary key fix

Fixes crash when mapping primary keys.

Confused why this is failing. Passes locally :(

Seems like libc++.dylib might not be in the project settings or the schema is not shared maybe?

Scheme is definitely shared. Never had to include libc++.dylib directly before...

I just switched from using -sdk to -destination (like described in Apple's documentation) and :boom: it worked. ¯_(ツ)_/¯

Seems legit. :laughing:

@minuscorp have to leave right now, will update the pod later tonight :+1:

Nice work right here. Awesome!

Pod's been updated in cocoapods :+1:


Duplicate


Remove non addons

Removed some mods (and even a modpack) that aren't addons

Thank You very much for your contibution :)
From my look everything is ok :+1:


Feature th http client for RTS-258

Re-worked to use Webmachine better.
Open issues:

The query API only accepts POST and uses a ?query="..." query string in the URL to pass in the query. This is to make it consistent across HTTP methods if we want to match the side effects of the query with the HTTP method.
The output from list_keys is some hrefs inside a <html> block. Perhaps we should change the design and provide something more useful. It is not of much use when doing a curl or http (httpie.org) request on the command line.
Dialzyer is not fully happy about the listkeys module, but it smells like a Dialyzer bug - see comment in that file.
The commits have not been squashed - open to doing that before doing the final PR.

General notes:

The coverage request is missing.
An r_t module is needed to see it in operation.

Raw notes from our discussion...
riak_kv_wm_timeseries_query
     - why bother with prefix, it is unused.
     - why bother with riak, it is unused.
   - may as well just do C = riak_client:new(node(), undefined).

service_available
     - KV endpoints always use the the local riak_client.  That can coordinate with the cluster even if the local KV/TS service is down until shutdown.
     - if you wanted to be pedantic you could use init:get_status() and check in started state.

malformed_request
    - if a new statement type is added, it will throw a non-match in the case statement.

content_types_accepted
   - what happens if you supply a content-type in the query - is it ignored?

resource_exists
  - on create, no differentiation between exists/does not exist

process_post
  - would have expected to look in the request body.

checkpoint
     - what's the value of this function versus a -define(checkpoint, lager:info), with the current system the module and location information is not picked up by the lager parse transform.  You would get more data by directly calling lager:info() or using the define above if you wanted to segregate difference message types.
   - is the intent to make this at debug level eventually? At the moment this seems way to chatty for an informational message in the main logs - service_available and malformed_request will be called for each HTTP request.

Finally got this to work.
curl --trace-ascii - -X POST 'http://localhost:8098/ts/1/query?query=CREATE%20TABLE%20t%20%28f%20VARCHAR%20NOT%20NULL%2C%20s%20VARCHAR%20NOT%20NULL%2C%20ts%20TIMESTAMP%20NOT%20NULL%2C%20PRIMARY%20KEY%20%28%28f%2Cs%2Cquantum%28ts%2C1%2Cd%29%29%2Cf%2Cs%2Cts%29%29'

which you can do as
 curl -G -X POST http://localhost:8098/ts/1/query --trace-ascii - --data-urlencode "query=SELECT * FROM t WHERE f='f' AND s='s' AND ts > 0 AND ts < 10"

@jonmeredith

prefix and riak removed from #ctx{}
init_get:get_status/0 used in service_available
503 when a new SQL type is encountered
resource_exists now issues a 409 when trying to create an existing table
process_post does not have to look in the body as we have to look at the body in malformed_quest where we store things to be used later
checkpoint removed
will turn all responses into JSON later today

@hmmr

coverage is not part of the HTTP API, so we are safe there.
a RT test has been created and will be updated when I change the return formats to be JSON.

I think the list_keys endpoint may need something of a rethink.  Here is what you get when you query it.
 curl -v -v -v  http://localhost:8098/ts/v1/tables/t/list_keys
*   Trying <IP_ADDRESS>...
* Connected to localhost (<IP_ADDRESS>) port 8098 (#0)
> GET /ts/v1/tables/t/list_keys HTTP/1.1
> Host: localhost:8098
> User-Agent: curl/7.43.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Vary: Accept-Encoding
< Transfer-Encoding: chunked
< Server: MochiWeb/1.1 WebMachine/1.10.8 (that head fake, tho)
< Date: Thu, 24 Mar 2016 22:39:12 GMT
< Content-Type: application/json
<
* Connection #0 to host localhost left intact
<html>{"url":"http://<IP_ADDRESS>:8098/ts/v1/tables/t/keys/f/f/s/s/ts/789/"}{"url":"http://<IP_ADDRESS>:8098/ts/v1/tables/t/keys/f/f/s/s/ts/456/"}{"url":"http://<IP_ADDRESS>:8098/ts/v1/tables/t/keys/f/f/s/s/ts/123/"}</html>

Content type is given as JSON, but the output is not JSON.  It's malformed HTML without a head/body element containing some fragments of JSON.  I think most languages would be sad if they had to parse this.
The problem is that the output could be large so we would want to chunk the data.  Ideally we would have a paginated interface for queries so we could just return a single json object.
Options:

Generate a real HTML document with links to the keys.
Just write out a list of CR/LF delimited URLs to the keys
Send chunks made up of JSON arrays, delimited by \r\n or something like that.

@lehoff have a look how keys are sent in standard riak: https://github.com/basho/riak_kv/blob/riak_ts-develop/src/riak_kv_wm_keylist.erl#L230, and how they are received: https://github.com/basho/riak-erlang-http-client/blob/riak_ts-develop/src/rhc_listkeys.erl#L86.
There is also my branch in riak-erlang-http-client, https://github.com/basho/riak-erlang-http-client/tree/feature/az/http-client, where I have implemented all the TS-specific API calls. I then used riakhttpc as the alternative client in ts_cluster_comprehensive (https://github.com/basho/riak_test/pull/1016) to run the same scenario as it normally does with the protobuffer client (riakc). The wrappers in riak-erlang-http-client will probably need to be modified per your changes in riak_kv.

It should be stream - fix for that coming up.
For the list_keys the idea was to return a stream of {"url": "http://..."} pairs.
The html tag is a leftover that should have been removed.
All the other timeseries HTTP endpoints return JSON, so it might be best to continue that.
So chunks in JSON arrays seems best. But should it just be an array of `["http://...", "http://..."] or should we tag them with "url" in pairs I originally envisioned?

The query callback still needs some tweaks to accept the recently implemented WITH-clause for the CREATE query. The changes are in https://github.com/lehoff/riak_kv/pull/2, for you to consider and pull.
With those changes, I can confirm the test in https://github.com/basho/riak_test/pull/1016 passes.

+1 383e049
The tests (https://github.com/basho/riak_test/pull/1015) all pass.

... and the dialyzer says no, with a strange unrelated error to eleveldb.
eleveldb.erl:324: Call to missing or unexported function lager:error/2

Okaay, it appears @lehoff forgot to accept https://github.com/lehoff/riak_kv/pull/3, something I had presumed he did..
I have opened a yet another PR, with identical contents as this one plus the last commit to fix the dialyzer issue: https://github.com/basho/riak_kv/pull/1382.

Should be covered by #1398


Asciimath with variable name of more than one letter not rendered properly

let us say i have two variable:
unit1 and unit2
and my equation is: unit1/unit2 then =>
it is rendered like
unit (1/u) nit2
How to use variable name containing more than one letter?

Was this fixed? Maybe, a solution would be  requiring spaces around operators?

@zanedp, this is an issue with the AsciiMath language, not with MathJax itself.  MathJax incorporates the AsciiMath processed verbatim, so we don't control how that works.  You might want to raise the issue on their issue tracker.

Using "length" makes it be in roman text, appearing to be a unit of measure rather than a variable.

Multi-letter variable names normally are typeset in upright rather than italic text.  The math italics font is not designed for mutli-letter identifiers (see the poor rendering of office for example).  So "length" actually produces the result preferred by math typesetting conventions.


Support bindProxyAndYDoc

bindProxyAndYDoc can support Yjs's subdoc.

https://github.com/yjs/yjs/blob/97ef4ae1e0c7cefdaf1673a41b6cc83d68948457/src/utils/Doc.js#L150-L182

A doc can be easily convert to json with subdoc. But It is hard to convert json to a doc has a subdoc.

export const bindProxyAndYDoc = (
  p: Record<string, unknown>,
  y: Y.Doc,
  opts: Options = {},
) => {
  y.share.forEach((subY, k) => {
    if (subY instanceof Y.Doc) {
      const pv = proxy({});
      bindProxyAndYDoc(pv, subY, opts);
      p[k] = pv;
    }
    if (subY instanceof Y.Array) {
      const pv = proxy([]);
      bindProxyAndYArray(pv, subY, opts);
      p[k] = pv;
    }
    if (subY instanceof Y.Map) {
      const pv = proxy({});
      bindProxyAndYMap(pv, subY, opts);
      p[k] = pv;
    }
  });

  y.on(
    'subdocs',
    ({
      added,
      removed,
      loaded,
    }: {
      added: Set<Y.Doc>;
      removed: Set<Y.Doc>;
      loaded: Set<Y.Doc>;
    }) => {
      console.log(added, removed, loaded);
    },
  );
};

(oops, I had my commend as a draft for a few days.)
I was thinking like #3, and Y.Doc can be supported. But,

A doc can be easily convert to json with subdoc. But It is hard to convert json to a doc has a subdoc.

subdocs seem too much trouble for us.


Update Promise.fs

Make Error type in bindResult generic.

I was thinking about this at the time, the main advantage of having Exception explicitly is you don't need to qualify it in the passing function but I guess this is preventing some scenarios so I'm OK with the change :smile: Thanks for the PR!


Cleanup async code

fully switches to the recommended high-level API by not interfering with the loop in any way
removes the need to manually cleanup tasks by having to wait for a signal
fixes the problem with the update feeds task where exceptions where only shown on shutdown

This is the first part of the fix for #10

I tested this manually by verifying

that the task is not pending after CTRL-C or sending SIGINT
that the bot still responds while updating the feeds
exceptions while updating the feeds are now actually raised and stop excecution (e.g. manually raise an exception inside the task or configure an invalid feed like https://herndl.org/ that will raise an error while parsing)

Hrm, I guess I wanted to be compatible with Python <3.7. But I don't remember why.
Will test this in the coming days, hopefully.

If compatibility is important after all, then an alternative would be to use something like https://docs.python.org/3/library/asyncio-task.html#asyncio.Task.add_done_callback as they do in https://quantlane.com/blog/ensure-asyncio-task-exceptions-get-logged/
If not the high-level API is cleaner / less code I guess :)

Btw. I'll run it like that for now with the DEBUG logging level to find out what the real issue is behind the failing updates. I guess something very trivial like network hickups, unexpected HTTP status codes or unexpected responses or so. That should be easy to improve then I guess.


An issue on deploying to Tensorflow savedmodel

Hello
How are you?
Thanks for contributing this project.
I tried to deploy this pre-trained model to Tensorflow savedmodel but failed.
Could u provide tensorflow savedmodel too?
Thanks

Hi @rose-jinyang , I haven't tried implementing the model in tensorflow, but I will let you know if I do.
Maybe contacting the original authors of the U^2-Net model may help.

I need a Tensorflow savedmodel.
Thanks

I will try

Please have a look at this repository Implementation of the paper - U^2-Net: Going Deeper with Nested U-Structure for Salient Object Detection in TensorFlow.


Update intellij-idea to 2018.1.2

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you, but done in https://github.com/caskroom/homebrew-cask/pull/46290.

oh, sorry. maybe next time 😆


free-programming-book-Spanish

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

This one got lost in the shuffle of reviewing over 1,300 PRs.
Thanks, and welcome again to free-programming-books!


Fix patching

Fixing patching introduced in https://github.com/conda-forge/openblas-feedstock/pull/35
cc @ocefpaf

Thanks @isuruf!

Could we move to openblas 0.2.20 ? see https://github.com/conda-forge/conda-forge.github.io/pull/418

Could we move to openblas 0.2.20 ? see conda-forge/conda-forge.github.io#418

If this works that is the plan :wink:

@jschueller, once this PR is in, we can. Let's see whether scipy test suite passes with this and if it does let's move to 0.2.20.

Travis on my fork passes, https://travis-ci.org/isuruf/openblas-feedstock/jobs/292625984

Travis on my fork passes, https://travis-ci.org/isuruf/openblas-feedstock/jobs/292625984

That's a merge :wink:


Raise minimal Java version to 17

Is your feature request related to a problem? Please describe.
Current minimal version is 11. This should be raised. See the discussion in the forum.
Describe the solution you'd like

Configure enforcer plugin to require Java 17
Change compiler properties in root pom.xml
Ensure that the CI build uses at least Java 17
Document prerequisite in README

Describe alternatives you've considered
Leaving lower bound 11, or upgrade to current LTS release 21.

The build workflow already uses Java 17.
Configuration of Java 17 in the enforcer configuration in the root pom.xml makes several configuration in submodules obsolete.


fix default tag kube-scheduler (cluster version)

Context
On SNCF organization, clusters doesn't have internet. We worked with private registry an authentication.
In this case, we need for each upgrade tag version (Kubernetes version) kube-scheduler.
This fix sets automatically the Kubernetes version for kube-scheduler.

Ready to review


move build script after cypress

cypress:ci would start the server with yarn start script which would remove dist folder causing Post-build stage to fail.

Thanks!


Watcher's BlockGetter doesn't cover for unexpected structure of the JSON from block.get

It seems there is an unexpected resonse from the said child chain's endpoint that can confuse the watcher. It would be when the JSON response from that RPC is decoded correctly, but doesn't have the required fields, e.g. is missing the hash field.
In such case, BlockGetter will go into restarts, because:
2019-08-06 12:38:53.537 [error] ⋅Task #PID<0.2395.0> started from OMG.Watcher.BlockGetter terminating
** (FunctionClauseError) no function clause matching in OMG.Watcher.BlockGetter.Core.validate_download_response/5
    (omg_watcher) lib/omg_watcher/block_getter/core.ex:506: OMG.Watcher.BlockGetter.Core.validate_download_response({:ok, %{blknum: 1000, transactions: ["0xf901d2f9010cb8410e4b11e3fde0824d81c7f07daa969b3d1ec1c7a62f91d0c389fb2eb82287b6dd2ef879f786d0065ecea2f22dc6f51c8206d85bd05fda0203f306af7f1c0358631cb8410000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000b8410000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000b8410000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000d0c3018080c3808080c3808080c3808080f8b0eb947d6ff7053301d9f2f445560a99ad55664ecc482e94000000000000000000000000000000000000000007eb94dec33023dbd2919f837a51cca1117a3a8329b7c494000000000000000000000000000000000000000003eb94000000000000000000000000000000000000000094000000000000000000000000000000000000000080eb94000000000000000000000000000000000000000094000000000000000000000000000000000000000080"]}}, <<152, 177, 209, 234, 48, 113, 182, 131, 161, 47, 50, 216, 48, 101, 206, 74, 135, 249, 0, 189, 122, 84, 234, 249, 70, 90, 120, 74, 247, 152, 215, 134>>, 1000,<PHONE_NUMBER>,<PHONE_NUMBER>536)
    (omg_watcher) lib/omg_watcher/block_getter.ex:272: anonymous fn/1 in OMG.Watcher.BlockGetter.run_block_download_task/1
    (elixir) lib/task/supervised.ex:90: Task.Supervised.invoke_mfa/2
    (elixir) lib/task/supervised.ex:35: Task.Supervised.reply/5
    (stdlib) proc_lib.erl:249: :proc_lib.init_p_do_apply/3
Function: #Function<3.38199213/0 in OMG.Watcher.BlockGetter.run_block_download_task/1>
    Args: []⋅

Instead it should be treated as either potential block_withholding or invalid_block (rethink the choice here, considering that garbage JSON leads to block_withholding while, things like incorrect (different) hash in a correct JSON structure lead to invalid_block).
How to reproduce
add
    |> Map.delete(:hash)

in apps/omg_child_chain_rpc/lib/omg_child_chain_rpc/web/views/block.ex:23
and run
mix test --only integration test/omg_watcher/integration/block_getter_test.exs:50

note, the client is also unpermissive towards fields that are expected to be hex bineries and but aren't valid hex binaries.
To reproduce, pretend the childchain returns hash: "hash" in a block response.


Audio output selection

Hello there,
I'm looking for a Python library that can support external output selection like https://github.com/merdok/homebridge-webos-tv/blob/master/index.js#L344 - I was wondering if this planned for (I can test/possibly contribute depending on time).

If you mean audio output selection, then I don't think this library supports it as of now. However, it would be extremely straightforward to implement it.
If you're willing to create a PR, I'd be happy to merge it in. :) Please look at a similar class: https://github.com/supersaiyanmode/PyWebOSTV/blob/e73438c4de9817bb98d8aa300a59cdaa218771c7/pywebostv/controls.py#L398

Hmmm. Nice. I hadn’t noticed that part of the code.
Besides the usual time constraints testing this is going to be tricky (can’t mess around with the TV most of the time), but I appreciate the pointer and will have a look at it.

This was recently contributed by user Felix-Pi here: https://github.com/supersaiyanmode/PyWebOSTV/pull/43


add time series checker function

Add a function that allows users to check the time series in a system for:

valid scaling_factor_multipliers
values in range 0 - 1.0
constant resolution and length
consistency across device types (all active RE gens should have a time series...)
anything else?

Implemented post- parsing not as part of the parsing

add helper function to the time series. Similar to show_components but for time series.

closing since this is stale and most of this list is implemented by default.


Visualreport handling of missing files

Describe the bug
When a dataset has recording for which part 3 did not generate output, then the visual report generations seems to be confused.
To Reproduce
Someone sent me files which I can run locally, but I think this can be reproduced by deleting the first part 3 milestone files in ms3.out and then trying to generate the csv reports and the visual report.
Expected behavior
We expect the visual report to always match the files display in the filename and to correctly skip files that have no data.

While working on this we noticed that there was also a problem in part 1, 2, 4 and 5 with sorting of filenames when user works with files stratified across subfolders.
group1/subject2
group2/subject1
would result in output where the incorrect filename is reported for each ID. The ID however is consistent with the results in the rest of the report.


golint fixes

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   93.54%   93.54%
=======================================
  Files           7        7
  Lines         124      124
=======================================
  Hits          116      116
  Misses          4        4
  Partials        4        4

Impacted Files
Coverage Δ

config/config.go
100% <ø> (ø)
:arrow_up:

api/api.go
100% <ø> (ø)
:arrow_up:

config/flag/flag.go
100% <100%> (ø)
:arrow_up:

handler/handler.go
100% <100%> (ø)
:arrow_up:

api/cepaberto/cepaberto.go
88.23% <100%> (ø)
:arrow_up:

config/env/env.go
100% <100%> (ø)
:arrow_up:

api/viacep/viacep.go
88.57% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 289c580...13bfd10. Read the comment docs.


chore(tools): update curriculum-helpers

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Update @freecodecamp/curriculum-helpers to 1.2.0
Remove @freecodecamp/curriculum-helpers from deps in client/

After spending >1 hour trying to figure out why updating this did not do anything 🤦♂

Note: Unexpectedly, in our editor/environment, we treat ENTER as \r\n which means the helper currently does not work without first remove all \r. I am not sure if this is something we want to fix in the helper, or it shows something weird about the way we store/read the codestring?

 SuperBlock: 2022/responsive-web-design - Block: Learn Intermediate CSS by Building a Cat Painting Challenge: Step 3(646c48df8674cf2b91020ecc) seems to fail in the mobile app. But that is it.

 SuperBlock: 2022/responsive-web-design - Block: Learn Intermediate CSS by Building a Cat Painting Challenge: Step 3(646c48df8674cf2b91020ecc) seems to fail in the mobile app. But that is it.
edit: that one seemed to fail before updating too v1.2.0

I would be impressed if this change broke that lesson.


Add ability to customize test command for presubmit

For angular, we need to have custom commands for running tests (as opposed to just pub run test). Right now, there is no way to specify this in .travis.yml files.
One alternative is to do what we're doing now, which is to have custom tool/travis.sh for sub-packages.

Thinking about this a little, I can see a few different ways we might want to do this:

Check for presence of tool/travis.sh, and run that as a stand-alone test task.
Define a custom test_script tag in the .travis.yml files. This would mean that nested travis configs wouldn't necessarily be valid on their own.
Travis has a before_script step. We could use this as a signal to run a command before the dart task(test in this case), but after calling pub upgrade.

I'm actually liking the idea of 3., as that would probably fit with a long-term solution around running the build step before running tests. Unless there are any concerns, I'm going to start experimenting with that solution and see if it will work for the Angular use case.


[AIRFLOW-3522] Add support for sending Slack attachments

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3522
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

Adding support for sending Slack attachments with messages via the SlackWebhookOperator. This will allow customized messages to be sent with interactive content.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Modified test_slack_webhook_hook.py
Modified test_slack_webhook_operator.py

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.
All the public functions and the classes in the PR contain docstrings that explain what it does

Code Quality

[x] Passes flake8

Codecov Report

Merging #4332 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4332      +/-   ##
=========================================
+ Coverage   78.09%   78.1%   +<.01%
=========================================
  Files         201     201
  Lines       16471   16471
=========================================
+ Hits        12863   12864       +1
+ Misses       3608    3607       -1

Impacted Files
Coverage Δ

airflow/models.py
92.35% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 067b967...358f84f. Read the comment docs.

Codecov Report

Merging #4332 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #4332      +/-   ##
=========================================
+ Coverage   78.09%   78.1%   +<.01%
=========================================
  Files         201     201
  Lines       16471   16471
=========================================
+ Hits        12863   12864       +1
+ Misses       3608    3607       -1

Impacted Files
Coverage Δ

airflow/models.py
92.35% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 067b967...358f84f. Read the comment docs.


Test Build to check GH

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

This PR was just to check something out, so removing it


Update platform_file_store_file.md

Describe your changes
fix platform_file_store_file import command on docs
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck
gitleaks: trigger gitleaks

trigger build

trigger gitleaks

trigger subcategoryfieldcheck

trigger gitleaks

@krishi0408 can you help me? for some reason, gitleaks failed... but I cannot see the logs

This PR has been automatically closed due to inactivity.


Error - UnboundLocalError: local variable 'authpair' referenced before assignment

Hi @karask ,
Thanks a lot for putting together the project.
I tried creating certificates using the project and got the  error "UnboundLocalError: local variable 'authpair' referenced before assignment"
I tried printing out service_url in /new_py3_env/lib/python3.5/site-packages/bitcoin/rpc.py and see that the value is set. authpair variable seems to be have  local scope in the if block. Appretiate your help in fixing the issue.
(new_py3_env) bash-4.3# create-certificates -c /etc/data/config.ini

Configured values are:

working_directory:	/etc/data
pdf_cert_template_file:	/etc/data/MOOC7_Cert_Template.pdf
graduates_csv_file:	/etc/data/graduates.csv
certificates_directory:	/etc/data/certificates
cert_names_csv_column:	student_name
issuer:			University of Nicosia
issuer_address:		mr5TnhzymxLLpftDXNHUK7kNNmUnaZY6of
cert_metadata_columns:	student_name,degree_title,percentage_grade,date_of_issuance
Do you want to continue? [y/N]: y
..........
Configured values are:

working_directory:	/etc/data
issuing_address:	mr5TnhzymxLLpftDXNHUK7kNNmUnaZY6of
full_node_url:		<IP_ADDRESS>:18332
full_node_rpc_user:	foo
testnet:		True
tx_fee_per_byte:	100
hash_prefix:		None
Do you want to continue? [y/N]: y

Please enter the password for the node's RPC user:
service_url http://foo:bar@<IP_ADDRESS>:18332
Traceback (most recent call last):
  File "/new_py3_env/bin/create-certificates", line 11, in <module>
    load_entry_point('blockchain-certificates==0.10.3', 'console_scripts', 'create-certificates')()
  File "/new_py3_env/lib/python3.5/site-packages/blockchain_certificates/create_certificates.py", line 84, in main
    txid = publish_hash.issue_hash(conf, True, cp.get_merkle_root())
  File "/new_py3_env/lib/python3.5/site-packages/blockchain_certificates/publish_hash.py", line 68, in issue_hash
    conf.full_node_url))
  File "/new_py3_env/lib/python3.5/site-packages/bitcoin/rpc.py", line 312, in __init__
    **kwargs)
  File "/new_py3_env/lib/python3.5/site-packages/bitcoin/rpc.py", line 191, in __init__
    authpair = authpair.encode('utf8')
UnboundLocalError: local variable 'authpair' referenced before assignment

Thanks
Vishnu

Hi @vishnu6266
I committed a fix (a new version of bitcoinlib was responsible for this, I fixed the version). Just uninstalling and installing version 0.7.0 would also work.
I also got an opportunity to use more generic templates and examples (sample directories).
Let me know if that solved your issue.
Cheers,
Kostas
PS. If you use it in production (mainnet) you are welcome to let me know so I can check that everything is alright before you disseminate any certificates.

Hi @karask
Thanks a lot for the help. The issue is fixed with the commit. The samples are really helpful. I will continue testing and let you know if any issues. We are still in the initial phase of the projects.
One suggestion is to have a capability to test the project in regtest mode also.
Regards
Vishnu


SSL_connect returned =1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (Ope nSSL::SSL::SSLError) ---> Windows

C:\Users\Devang\Desktop\chef-rails-dev-box\chef>librarian-chef install
DL is deprecated, please use Fiddle
Installing chef_handler (1.1.6)
C:/Ruby200-x64/lib/ruby/2.0.0/net/http.rb:921:in connect': SSL_connect returned =1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (Ope nSSL::SSL::SSLError)         from C:/Ruby200-x64/lib/ruby/2.0.0/net/http.rb:921:inblock in connect'
    from C:/Ruby200-x64/lib/ruby/2.0.0/timeout.rb:52:in `timeout'
    from C:/Ruby200-x64/lib/ruby/2.0.0/net/http.rb:921:in `connect'
    from C:/Ruby200-x64/lib/ruby/2.0.0/net/http.rb:862:in `do_start'
    from C:/Ruby200-x64/lib/ruby/2.0.0/net/http.rb:851:in `start'
    from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li

brarian/chef/source/site.rb:352:in block in http_get'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li brarian/chef/source/site.rb:348:inloop'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li
brarian/chef/source/site.rb:348:in http_get'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li brarian/chef/source/site.rb:238:incache_remote_object!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li
brarian/chef/source/site.rb:207:in cache_version_uri_package!'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li brarian/chef/source/site.rb:216:incache_version_uri_unpacked!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li
brarian/chef/source/site.rb:212:in cache_version_unpacked!'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li brarian/chef/source/site.rb:30:ininstall_version!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li
brarian/chef/source/site.rb:436:in install!'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/manifest.rb:101:ininstall!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari
an/action/install.rb:49:in block in install_manifests'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/action/install.rb:48:ineach'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari
an/action/install.rb:48:in install_manifests'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/action/install.rb:39:inperform_installation'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari
an/action/install.rb:12:in run'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/cli.rb:165:ininstall!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li
brarian/chef/cli.rb:42:in install'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/comman d.rb:27:inrun'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/invoca
tion.rb:126:in invoke_command'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359 :indispatch'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/thor-0.19.1/lib/thor/base.r
b:440:in start'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/cli.rb:26:inblock (2 levels) in bin!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari
an/cli.rb:31:in returning_status'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/cli.rb:26:inblock in bin!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari
an/cli.rb:47:in with_environment'         from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-0.1.2/lib/librari an/cli.rb:26:inbin!'
from C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/bin/li
brarian-chef:7:in <top (required)>'         from C:/Ruby200-x64/bin/librarian-chef:23:inload'
from C:/Ruby200-x64/bin/librarian-chef:23:in `'

C:/Ruby200-x64/lib/ruby/gems/2.0.0/gems/librarian-chef-0.0.3/lib/li brarian/chef/source/site.rb
def http(uri)
  net_http = environment.net_http_class(uri.host).new(uri.host, uri.port)
  net_http.use_ssl = true if uri.scheme == 'https'
  net_http
end

replace
def http(uri)
  net_http = environment.net_http_class(uri.host).new(uri.host, uri.port)
  net_http.use_ssl = true if uri.scheme == 'https'
  net_http.verify_mode = OpenSSL::SSL::VERIFY_NONE
  net_http
end

:)

Thank you @POStroi
That resolved the issue for me.

@whaleyHF
The long-standing problem :(
Pull request https://github.com/applicationsonline/librarian-chef/pull/49
Automatic windows platform chek  and disable SSL verify.

Still running into this problem. Thanks for the solution @SysAlex

excellent, how do you know these things?


Load data for IE & Edge

First challenge: need to identify the list MS is using.

also on support.microsoft.com (https://support.microsoft.com/en-us/kb/3071338) they say the same, including ie11.

Like other browsers that implemented this feature, Microsoft Edge and Internet Explorer 11 base their preload list on the Chromium HSTS preload list. Microsoft plans to update the preload list on a quarterly basis and deliver it in the corresponding Internet Explorer cumulative update


Suppress Flatten Tests in Dataflow

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @dhalperi

Thanks, LGTM, merged.


aggressive-loop-optimizations error

Fair warning, this looks the same as #1491, but I can repro on g++ 8.2.
Expected Behavior
It would be great if dlib compiled without any compiler warnings (-Werror).
Current Behavior
With g++ 8.2, I get
dlib/dlib/filtering/../matrix/matrix_la.h:225:32: error: iteration 1 invokes undefined behavior [-Werror=aggressive-loop-optimizations]
             y = abs(q(i)) + abs(e(i));
                             ~~~^~~~~~
dlib/dlib/filtering/../matrix/matrix_la.h:163:20: note: within this loop
         for (i=0; i<n; i++)
                   ~^~

Steps to Reproduce
Temporarily adding set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Werror=aggressive-loop-optimizations") to cmake will trigger this on a x86 build.

I would quite like to fix this myself, but on the first glance that part of the code is pretty tough to follow. It seems to be the e() that causes the error. A tiny bit of guidance and I can probably do this myself.

I have no idea.  This really seems like a compiler error since e is just an
array of n numbers, and that loop obviously doesn't go outside the bounds
of e.


Has no effect on only one key "control"

I want to listen the control key on keyboad with mouse left click to select multiply widget on Windows or Linux desktop platform.
But it is has no effect when i listen control key.
// register control hostkey
hotKeyManager.register(HotKey(KeyCode.control), keyUpHandler: _hotkeyUpHandler);

How do you solve this problem?


Restore docker-compose usage + upgrade HCD images

without committing to python 3.9, as a modern testcontainers would require.

An alternative would be to drop support for Python 3.8.
It will be EOL in October anyway: https://devguide.python.org/versions/


Wallet signer functionality

Create a wallet signer that presents the basic signer functionality.
Sample app should show case the basic session request handling, parsing the string object to expected object.
Sample app should present what tools, libraries are recommended for signer functionality.
Code samples of parsing the string to expected json rpc object should be added to ReadMe

@jakobuid any update on this ? . I'm kinda struggling on this "signing functionality" with web3j.
For personal sign I'm sending the Numeric.toHexString(Sign.signPrefixedMessage(Numeric.fromHexString(the given string jon doe etcc), ecKeyPair))
The react-app.walletconnect.com is kind of saying valid: false for personal sign and sign typed data

personal_sig was added to wallet sample


Documentation on setting up bash alias

You covered setting up bash alias to enable shortening URL by running a command in terminal from anywhere but it's no where to be found here in the documentation yet.
I think adding this to README will vastly increase the value of this module.

Thanks Joseph! This should be fixed now :)


Handle request for GetSyncCommitteeRewards

#6701
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

overall pretty good, lets add an integrationTest based on AbstractDataBackedIntegrationTest (from memory) that has an actual altair chain, and we query it for results...
The test cases:

empty list ([]) -> all -> just a count would be fine to check
empty body -> 400 error
["asdf"] -> 400 error
["99999999999999"] (out of validator range) -> empty list returned
["X","Y"] and X in sync committee, Y not -> x returned in list

something like this anyway... just some negative tests
The point here is we're not mocking, we're fetching data from a chain and seeing it end-2-end being fetched etc.

 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.


[Feature] How do I create a double click sign catcher with lable

Is your feature request related to a problem? Please describe.
How do I create a double click sign catcher with lable?
Describe the solution you'd like
I search in google, and I find a solution like this:
gtk_widget_add_events(window, GDK_BUTTON_PRESS_MASK); g_signal_connect(window, "button-press-event", G_CALLBACK(clicked), NULL);
but I can not run the same code in gotk3, code:
win.AddEvents(gdk.EVENT_DOUBLE_BUTTON_PRESS)
it notice that: Cannot use 'gdk.EVENT_DOUBLE_BUTTON_PRESS' (type EventType) as type int
Describe alternatives you've considered
None
Additional context
None

Suddently, I find a way to catch double click event in gotk3, hope itwill help people who need it, the code like this:
win.Add(box)  eventBox, _ := gtk.EventBoxNew()  box.Add(eventBox)  lbl, _ := gtk.LabelNew("double click")  eventBox.Add(lbl)  eventBox.Connect("button-press-event" , func(tree *gtk.EventBox, ev *gdk.Event) bool {  if gtkutil.EventIsLeftDoubleClick(ev) {  log.Printf("double click")       return true  }  return false  })
func EventIsLeftDoubleClick(ev *gdk.Event) bool { 	btn := gdk.EventButtonNewFromEvent(ev) 	return btn.Type() == gdk.EVENT_DOUBLE_BUTTON_PRESS }