[BUG] Pods are stuck in NotReady after node becomes notReady for few seconds

What happened:
Some Pods are stuck in NotReady status after related node becomes not ready for few seconds and then is healthy again.
From kube-controller-manager logs, pods won't be evicted and are still running.
Give one Pod for example.
I0828 07:00:00.435121       1 controller_utils.go:140] Updating ready status of pod eeeeee1-79dd69d57c-g6spv to false
I0828 07:00:00.535153       1 event.go:291] "Event occurred" object="edgetest-edge/eeeeee1-79dd69d57c-g6spv" kind="Pod" apiVersion="v1" type="Warning" reason="NodeNotReady" message="Node is not ready"

I0828 07:00:31.843751       1 node_lifecycle_controller.go:893] Node vm-node-229-106 is healthy again, removing all taints
I0828 07:00:31.845457       1 event.go:291] "Event occurred" object="edgetest-edge/eeeeee1-79dd69d57c-g6spv" kind="Pod" apiVersion="" type="Normal" reason="TaintManagerEviction" message="Cancelling deletion of Pod edgetest-edge/eeeeee1-79dd69d57c-g6spv"

Today some pods are still not ready.
# kubectl get pod eeeeee1-79dd69d57c-g6spv -n edgetest-edge -o yaml
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T10:59:59Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2023-08-28T07:00:00Z"
    status: "False"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T11:00:19Z"
    status: "True"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T10:59:58Z"
    status: "True"
    type: PodScheduled

Also there are some other Pods recovered and became Ready.
Yurthub logs already were rotated and cannot find any other helpful information.
What you expected to happen:
Pods should recover and become Ready as soon as node is healthy again.
How to reproduce it (as minimally and precisely as possible):
Not sure if this can reproduce. In my env, the node suddenly cannot access apiserver and become not ready for few seconds. All its Pods become not ready too. Then it's healthy again and all Pods keep running. But some Pods cannot recover Ready status.
Anything else we need to know?:
To stop container can help force kubelet to sync status again.
Something is weird for those Ready Pods. Give Pod eeeeee1-79dd69d57c-8tb96 as example. KCM set it NotReady on 08/28.
I0828 06:59:57.481073       1 controller_utils.go:140] Updating ready status of pod eeeeee1-79dd69d57c-8tb96 to false
I0828 06:59:57.556615       1 event.go:291] "Event occurred" object="edgetest-edge/eeeeee1-79dd69d57c-8tb96" kind="Pod" apiVersion="v1" type="Warning" reason="NodeNotReady" message="Node is not ready"

But its Ready lastTransitionTime (08/25) never changes.
# kubectl get pod  -n edgetest-edge -o yaml eeeeee1-79dd69d57c-8tb96
status:
  conditions:
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T11:20:01Z"
    status: "True"
    type: Initialized
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T11:20:21Z"
    status: "True"
    type: Ready
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T11:20:21Z"
    status: "True"
    type: ContainersReady
  - lastProbeTime: null
    lastTransitionTime: "2023-08-25T11:20:01Z"
    status: "True"
    type: PodScheduled

Environment:

OpenYurt version<EMAIL_ADDRESS>Kubernetes version (use kubectl version): 1.22

others
/kind bug

Just created a k8s issue and hope to find more clues.
https://github.com/kubernetes/kubernetes/issues/120256

@AndyEWang Thanks for raising issue. would you like to tell me what OpenYurt components are installed in your cluster? and the version of these components?

@rambohe-ch Does yurthub has any chance to let kubelet miss the pod update event after re-connect the apiserver?

@rambohe-ch Does yurthub has any chance to let kubelet miss the pod update event after re-connect the apiserver?

@AndyEWang I think you can check the logs of yurthub whether pod update requests are forwarded or not.


Mitweb inconsistent option: web-iface

Problem Description
For mitmweb, the option is --web-iface; but for mitmproxy, the option is --listen-host.
These two settings are practically the same, since the interface value is actually an IP address.
Coming from Linux, I initially tried setting it to eth0, but that failed.
Searching for the option in the repository led me to the following command:
docker run --rm -it -p 8080:8080 -p <IP_ADDRESS>:8081:8081 mitmproxy/mitmproxy mitmweb --web-iface <IP_ADDRESS>

Source: https://github.com/mitmproxy/mitmproxy/blob/master/release/docker/README.md#usage
Instead, I think we should change it to --web-host to match the proxy.

Yes, that sounds reasonable. I'd be happy to merge a PR that unifies things! 😃

Please add this change to the documentation on dockerhub.
I was wondering why it wouldn't accept --web-iface <IP_ADDRESS> as suggested there.
Thanks!

Please add this change to the documentation on dockerhub.
I was wondering why it wouldn't accept --web-iface <IP_ADDRESS> as suggested there.
Thanks!

The source file was updated, (See: release/docker/README.md); perhaps someone will need to update it manually on Docker Hub.

Guys, please give a deprecation warning a few versions before you implement a breaking change in an interface. Keep the old one working during that time.
--web-iface was stable for 3 years.

@zeroum apologies, I only made the code change itself; I am not a maintainer.
However, it was put in the Change Log, (See: https://github.com/mitmproxy/mitmproxy/blob/master/CHANGELOG#L11); and also see below:

Mitmweb is currently in beta. We consider it stable for all features
currently exposed in the UI, but it still misses a lot of mitmproxy's
features.

Source: https://github.com/mitmproxy/mitmproxy/blob/master/docs/src/content/tools-mitmweb.md


Additional Resource

[Enter feedback here]
https://youtu.be/LfLFiAw7d28

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

ID: cb4428fc-215c-78e1-5ff0-fe73f7a08260
Version Independent ID: 6741b1de-38a4-fbb1-d79d-acc47d549a2d
Content: Additions to the csproj format for .NET Core - .NET Core CLI
Content Source: docs/core/tools/csproj.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Hello. Thank you for the suggestion. I don't believe we're currently promoting community videos through the documentation. Sorry about that. However, if users view the closed issues on this article here in github, they will find your link. I'm going to closet his. Cheers!


Reopen  #434: Too many columns can not view all columns

This reopen issue https://github.com/marmelab/admin-on-rest/issues/434
It is still reproducible in the latest code.
To reproduce, open following page:
https://marmelab.com/admin-on-rest-demo/#/commands?_k=wi0hso
Then change window size to make it small in width, you will see some part of right table can not be viewed and it can not horizontal scroll.

It has been stated in #434 that the fix landed on next branch, not master.

I see, It has been stated fixed for a while. when can be merged to master and released ?

We usually release once a month, around the 15th. Watch https://github.com/marmelab/admin-on-rest/releases for details.

Thank you so much.


Cleanup documentation

There have been a lot of changes recently, and some documentation is out of date.

3375b8bfd40d2403e747c362396f32b84ea136cc


MGMT-4501 - Added service dev dockerfile for fast image build

This new dockerfile FROMs the existing service image and simply replaces the existing service binary with the new locally built one. This allows for more quickly deploying modified images during development because it no longer requires rebuilding the entire image. Building the new service binary locally is done pretty quickly because it makes use of the go build cache

I like this approach much better. :)

Will open a more solid PR based on the same concept to the main AI service repo


When downloading the CSV file of CNCF Landscape, it is giving a file with a .css extension instead of a CSV.

Steps to Reproduce:

Go to the CNCF Landscape website (https://landscape.cncf.io/).
Click on the "Download CSV" button located at the top right corner of the page.
Select the location to save the file and click on "Save".
Check the downloaded file's extension.

Expected Result:

The downloaded file should have a .csv extension.

Actual Result:

The downloaded file has a .css extension instead of a .csv extension.

Impact:
This issue prevents users from being able to download the CSV file of CNCF Landscape correctly. As a result, users may not be able to access the data they need for their projects, research, or analysis.
Possible Causes:
There may be an issue with the website's code that is causing the file extension to be incorrect when downloaded.
Proposed Solution:
To fix this issue, the website's code needs to be reviewed and modified so that the CSV file downloads correctly with the .csv extension.
I think I should check the code related to the download functionality and ensure that the file is saved with the correct extension. But I am not able to find the source code in the master branch I searched for previous issues related to CSV but the file path src/components/ExportExcel.js is not in the master branch. Can Someone point out where should look?
Workaround:
As a temporary workaround, users can try manually changing the downloaded file's extension from .css to .csv. However, this workaround is not ideal as it may cause compatibility issues with some software that expects files to have the correct extension.
Priority:
This issue should be assigned a high priority as it impacts the usability and functionality of the CNCF Landscape website.

weird for me I get a .csv :)

interactive-landscape (test).csv

Yes you are right other browsers are giving CSV files but Firefox is only giving me CSS files I think something is wrong either with Firefox or with the website.
It's possible that the website sent the wrong file type to Firefox, or that Firefox interpreted the file type incorrectly?

it's really weird I tried to download it on my mobile phone on Chrome and Brave it gave me
interactive-landscape.csv.css and on mobile Firefox it gives interactive-landscape.csv just opposite of desktop.

Confirming I can re-create this issue with Firefox desktop, where clicking download csv results in interactive-landscape.css.
If I use Chromium desktop I get the correct interactive-landscape.csv.
The issue is also open here https://github.com/cncf/landscapeapp/issues/866
I've raised the pull request to fix: https://github.com/cncf/landscapeapp/pull/876

Hi!
We've just migrated the CNCF landscape to v2 and this issue should be resolved.
Please feel free to reopen if needed 🙂


Add JavaDoc for timedistance.js

Adds specific comments for automated JavaDoc documentation generation for the time/distance computation/injection feature. This depends on PR #3 being merged first.
Opened this PR a bit early just for the sake of allowing earlier review/template for format used in other modules.
TODO before merge:

[ ] Complete adding comments to source code (need to still add to second-half).

Ops I didn't see your comments first. I merged it. :)


Considering using an OpenAPI generated client internally

The new official vault C# client use a generated client based on the OpenAPI specs.
I think it might be a good option for us as well, both for ease of maintenance and correctness. The Rust generator is using reqwest as we already do.
We could only use it internally, and keep exposing the same public API so it won't break any user code.
With that and https://github.com/jmgilman/vaultrs/issues/80, I think we could get rid of most of our unmaintained dependencies

It's a great Idea but I wanted to do some tests regarding this and the generated openapi.json is missing a lot of responses objects which render the generated code unusable for some endpoints.
The issue has been there for a while now I don't see any fix soon:
https://github.com/hashicorp/vault/issues/11746
https://github.com/hashicorp/vault/issues/27269

Thanks a lot for digging this up! That's quite sad that they don't want to fix this. Maybe there is still some value to use the client generated request types with our handcrafted response types?
It would be interesting to see if the official openAPI generated clients, go and C#, have since implemented any workaround for the concerned endpoints.


Panel dragger workaround for Safari

Safari is not setting buttons or button to 1 on Drag events, which is causing our Panel resizing to early out.
See: https://github.com/xh/hoist-react/issues/3797
This workaround adds a fallback check to the deprecated MouseEvent.which property, which is being set:
https://developer.mozilla.org/en-US/docs/Web/API/UIEvent/which#value_for_mouseevent_non-standard
As a deprecated API we obviously should not rely on it, but I think its safe to use as fall back for this specific issue in lieu of a better solution.
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Thanks for the quick turnaround here Tom.  I think this is very pragmatic.


Snowpack attempts to use invalid file names on Windows

This is due to a bug which has been fixed in Rollup v2.40.0:
I believe if Snowpack updates to v2.40.0 this should solve the issue.

Thanks for filing! I'd love to let v2.40.0 settle a bit before adopting, since we've been bitten in the past by immediately updating to the latest Rollup releases. Once #2707 is out, then lets prioritize this.

Just checking in on this, and it still seems that the most recent version of Rollup (2.46.0) still has an issue with treeshaking as outlined here: https://github.com/snowpackjs/snowpack/pull/2572. Will hold off a bit longer in updating Rollup either until this is fixed in Rollup, or we have a better workaround.

I believe this is the issue Rollup is tracking, among other treeshaking bugs: https://github.com/rollup/rollup/issues/3878

#3853 might fix this.  🤞


Impact in Mumbai

Login cluster is down in mumbai region [start_datetime:1705577036] [end_datetime:1705578036]

comment 4


Create .npmignore

To cut down on the size of the module on npm

Coverage remained the same at 100.0% when pulling 8af83fc10d2ef9ffd48bb31641c4c6e9a7e7e284 on knownasilya:patch-1 into d606b2aecf8033a96cb512d509fa98ee33a3dc89 on GoogleCloudPlatform:master.

Thank you for this, but the package.json in the root of the repo is a private package: https://github.com/GoogleCloudPlatform/gcloud-node/blob/d606b2aecf8033a96cb512d509fa98ee33a3dc89/package.json#L3
The rest of our packages that are published to npm use the files property in their package.json's to limit what is published, e.g. google-cloud: https://github.com/GoogleCloudPlatform/gcloud-node/blob/d606b2aecf8033a96cb512d509fa98ee33a3dc89/packages/google-cloud/package.json#L37
Did you install one of the modules from npm that included tests and docs?

Coverage remained the same at 100.0% when pulling 8af83fc10d2ef9ffd48bb31641c4c6e9a7e7e284 on knownasilya:patch-1 into d606b2aecf8033a96cb512d509fa98ee33a3dc89 on GoogleCloudPlatform:master.

Ah, good!


Use input_datetime for completion time

Is your feature request related to a problem? Please describe.
The current implementation with select.ev_smart_charging_charge_completion_time  does not allow me to set i.e. "14:00 tomorrow" if the current time is still before "14:00 today".
Describe the solution you'd like
I'd very much like to set a date/time combination, so you could (for example) set it to complete by "monday 07:00" on Friday and let the integration figure out what to do (knowing that chances are it'll charge on Saturday already)
Describe alternatives you've considered
Not really sure if there is an alternative?
Additional context
n/a

Thanks for the suggestion. However, your example to set it to complete by "monday 07:00" already on Friday will be difficult to implement. On Friday, there are no price information available for Sunday and Monday.
Would it make sense if "monday 07:00" would mean:

Don't schedule charging until Sunday 14:00.
At Sunday 14:00, schedule charging to be ready by Monday 07:00
After Monday 07:00, go back to step 1).

Is this really what you want? Or would you prefer that step 3) is "don't schedule any more"?

That's actually a fair point. I'm not sure how I envisioned this 😉
This request (or example) makes little sense, as most lowest prices in our (NL) weekends happen Saturday mid-afternoon, in the middle of the night between Sat-Sun, and/or Sun mid-afternoon (the afternoons especially when solar kicks in). The current weekend is an (expensive) exception for some reason.
Maybe the example in the request isn't valid - the issue I have most is this

The current implementation with select.ev_smart_charging_charge_completion_time does not allow me to set i.e. "14:00 tomorrow" if the current time is still before "14:00 today".

So if I return at noon today and need the car by 14.00 tomorrow, I can't currently use 14:00 as a setting because it'll start charging right away.

I have a similar request. It would be nice to be able to set a one-off completion time. For example, on Friday i would prefer if the car didn't finish charging by 7 am, since Saturday's electricity prices are already known when I get home, and they're much cheaper during Saturday noon.
So being able to set a "finish charging by 16.00 tomorrow" would at least work pretty good for my needs. It would then ideally reset to my regular "7 am" finish time.


Update Kaarthik's details

Add Kaarthik's details
Closes #23

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Replace field "id" with unique labels

Removed field "id" and instead distinguishing based on the labels.

@andrejtonev I've put back the example field because that's definitely part of the spec. It's ok if we don't implement that for now.


add blog post re: new fqdns for experiments

This PR adds a new post about changes to FQDNs for M-Lab experiments. @nkinkade PTAL?

This change is

_posts/blog/2020-05-06-new-fqdns-experiments.md, line 11 at r1 (raw file):
Previously, nkinkade wrote…
Sorry I didn't catch this in the document. I would change "upgrade" to "change" in this sentence. I don't view the change we're making to FQDNs as an "upgrade"  per-se.

ack! will update this & the post date, & publish! thanks @nkinkade !


Fix automations listening to HOMEASSISTANT_START

Description:
To prevent false triggers, we only enable automations after all components are setup. This however introduced a breaking change: we no longer allow people to listen to HOMEASSISTANT_START.
This PR is just to start discussion, the included fix is actually a bad one.
I still want people to be able to run something on startup.

Make it backwards compatible by checking if listening for HOMEASSISTANT_START event and hass.state == CoreState.starting. This is a hack.
Add a new special automation platform that runs things on startup (and also checks against hass.state)

Do not merge, still needs tests
Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

I think the seconds variant is not realy bad. With this special automation platform we allow to trigger on startup and close. Other automation make it also in this way.
like:
action:
  platform: homeassistant
  state: startup
...
action:
  platform: homeassistant
  state: shutdown

I like that.

Alright. Will support the old method but deprecate it. Then will also add the new platform.


Update New Database screenshot

Minor observation but collation in screenshot isn't Latin1_General_CI_AS as required in documentation.

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

ID: 3874265a-c20c-ba96-5480-9c9652b8d3aa
Version Independent ID: 3a7aa125-aedb-836f-8cfe-ba2380d5f490
Content: Install Azure AD Connect using SQL delegated administrator permissions
Content Source: articles/active-directory/connect/active-directory-aadconnect-sql-delegation.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@PurpleFuton good catch.  I have updated the screenshot and it should be live later today.  Thanks again!
#please-close

@PurpleFuton We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Update or replace jasmine-core

We use the package jasmine-core in edx/edx-enterprise. It is a testing framework for Javascript and it hasn't yet added support for Django3.2. Please follow the guidance in https://openedx.atlassian.net/wiki/spaces/AC/pages/3036972032/Handling+Outdated+Dependencies to resolve the problem this poses for the Open edX Django 3.2 upgrade.

Hi, @natabene I would like to contribute to this problem, could you please assign it to me?

@Squirrel18 Absolutely, thank you for reaching out to us and picking this up. I will assign this to you. Please keep us updated on your progress here.

@Squirrel18 Just checking if you had a chance to look into this.

Hi @natabene, I don't know what happened but my feedback never got submitted, my fault. Yes, I did work on this and here is what I found:
Jasmine Core for Python is deprecated (https://jasmine.github.io/setup/python.html) so the options are:

Remove the Jasmine Core support and change it to the supported Jasmine Node package.
Remove the Jasmine Core support and run the JS test suites with Karma.

Either of those options is fine for running the JS test suites, let me know if you want to discuss or research about it. Again sorry for the late response.

@Squirrel18  Thank you for your research. @Jawayria @jmbowman Please advise on how to proceed.


Gesture error when applying pagination

Describe the bug
I copied the parallax horizontal code. It doesn't work but when I removed the pagination the carousel works. And I want to use the pagination
To Reproduce
Steps to reproduce the behavior:

Copy the code

import * as React from "react";
import { Dimensions, Text, View } from "react-native";
import { useSharedValue } from "react-native-reanimated";
import Carousel, {
  ICarouselInstance,
  Pagination,
} from "react-native-reanimated-carousel";

const data = [...new Array(6).keys()];
const width = Dimensions.get("window").width;

function App() {
  const ref = React.useRef<ICarouselInstance>(null);
  const progress = useSharedValue<number>(0);

  const onPressPagination = (index: number) => {
    ref.current?.scrollTo({
      count: index - progress.value,
      animated: true,
    });
  };

  return (
    <View style={{ flex: 1 }}>
      <Carousel
        ref={ref}
        width={width}
        height={width / 2}
        data={data}
        onProgressChange={progress}
        renderItem={({ index }) => (
          <View
            style={{
              flex: 1,
              borderWidth: 1,
              justifyContent: "center",
            }}
          >
            <Text style={{ textAlign: "center", fontSize: 30 }}>{index}</Text>
          </View>
        )}
      />
      <Pagination.Basic
        progress={progress}
        data={data}
        dotStyle={{ backgroundColor: "rgba(0,0,0,0.2)", borderRadius: 50 }}
        containerStyle={{ gap: 5, marginTop: 10 }}
        onPress={onPressPagination}
      />
    </View>
  );
}

export default App;

It will show this error

Error: NativeViewGestureHandler must be used as a descendant of GestureHandlerRootView. Otherwise the gestures will not be recognized. See https://docs.swmansion.com/react-native-gesture-handler/docs/installation for more details.

This error is located at:
    in NativeViewGestureHandler (created by RNGestureHandlerButton)
    in RNGestureHandlerButton (created by BaseButton)
    in BaseButton (created by GenericTouchable)
    in GenericTouchable
    in Unknown (created by Basic)
    in RCTView (created by View)
    in View (created by Basic)
    in Basic (created by App)
    in RCTView (created by View)
    in View (created by App)
    in App (created by AuthIndex)
    in RCTView (created by View)
    in View (created by ThemedView)
    in ThemedView (created by AuthIndex)
    in AuthIndex
    in Unknown (created by Route(index))
    in Suspense (created by Route(index))
    in Route (created by Route(index))
    in Route(index) (created by SceneView)
    in StaticContainer
    in EnsureSingleNavigator (created by SceneView)
    in SceneView (created by SceneView)
    in RCTView (created by View)
    in View (created by DebugContainer)
    in DebugContainer (created by MaybeNestedStack)
    in MaybeNestedStack (created by SceneView)
    in RCTView (created by View)
    in View (created by SceneView)
    in RNSScreen (created by Animated(Anonymous))
    in Animated(Anonymous) (created by InnerScreen)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by InnerScreen)
    in InnerScreen (created by Screen)
    in Screen (created by SceneView)
    in SceneView (created by NativeStackViewInner)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by ScreenStack)
    in RNSScreenStack (created by ScreenStack)
    in ScreenStack (created by NativeStackViewInner)
    in NativeStackViewInner (created by NativeStackView)
    in RCTView (created by View)
    in View (created by SafeAreaProviderCompat)
    in SafeAreaProviderCompat (created by NativeStackView)
    in NativeStackView (created by NativeStackNavigator)
    in PreventRemoveProvider (created by NavigationContent)
    in NavigationContent
    in Unknown (created by NativeStackNavigator)
    in NativeStackNavigator
    in Unknown (created by AuthLayout)
    in RNCSafeAreaView
    in Unknown (created by AuthLayout)
    in AuthLayout
    in Unknown (created by Route(auth))
    in Suspense (created by Route(auth))
    in Route (created by Route(auth))
    in Route(auth) (created by SceneView)
    in StaticContainer
    in EnsureSingleNavigator (created by SceneView)
    in SceneView (created by SceneView)
    in RCTView (created by View)
    in View (created by DebugContainer)
    in DebugContainer (created by MaybeNestedStack)
    in MaybeNestedStack (created by SceneView)
    in RCTView (created by View)
    in View (created by SceneView)
    in RNSScreen (created by Animated(Anonymous))
    in Animated(Anonymous) (created by InnerScreen)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by InnerScreen)
    in InnerScreen (created by Screen)
    in Screen (created by SceneView)
    in SceneView (created by NativeStackViewInner)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by ScreenStack)
    in RNSScreenStack (created by ScreenStack)
    in ScreenStack (created by NativeStackViewInner)
    in NativeStackViewInner (created by NativeStackView)
    in RCTView (created by View)
    in View (created by SafeAreaProviderCompat)
    in SafeAreaProviderCompat (created by NativeStackView)
    in NativeStackView (created by NativeStackNavigator)
    in PreventRemoveProvider (created by NavigationContent)
    in NavigationContent
    in Unknown (created by NativeStackNavigator)
    in NativeStackNavigator
    in Unknown (created by RootLayout)
    in ThemeProvider (created by RootLayout)
    in QueryClientProvider (created by RootLayout)
    in RootLayout
    in Unknown (created by Route())
    in Suspense (created by Route())
    in Route (created by Route())
    in Route() (created by ContextNavigator)
    in RNCSafeAreaProvider (created by SafeAreaProvider)
    in SafeAreaProvider (created by wrapper)
    in wrapper (created by ContextNavigator)
    in EnsureSingleNavigator
    in BaseNavigationContainer
    in ThemeProvider
    in NavigationContainerInner (created by ContextNavigator)
    in ContextNavigator (created by ExpoRoot)
    in ExpoRoot (created by App)
    in App (created by ErrorOverlay)
    in ErrorToastContainer (created by ErrorOverlay)
    in ErrorOverlay (created by withDevTools(ErrorOverlay))
    in withDevTools(ErrorOverlay)
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in main(RootComponent), js engine: hermes

Delete the <Pagination.Basic />
Carousel works but pagination gone

Expected behavior
Both Carousel and Pagination working
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):
"expo": "~51.0.28",
"react": "18.2.0",
"react-native": "0.74.5",
"react-native-gesture-handler": "~2.16.1",
"react-native-reanimated": "~3.10.1",
"react-native-reanimated-carousel": "4.0.0-alpha.12",
Smartphone (please complete the following information):

Device: Samsung A71 [e.g. iPhone6]
OS: Android 13 [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

This bug seems to be fixed for me in v4.0.0-canary.17.
I think it may have been fixed with v4.0.0-canary.15.
The error also went away for me in 4.0.0-alpha.12 if I wrapped both the carousel and pagination components in another GestureHandlerRootView.
The docs for react-native-gesture-handler don't seem to mention anything about nesting multiple GestureHandlerRootView components (the Carousel component includes it). But since they do recommend including it in open-source libraries this way, I assume it is probably safe to do so.


Download file: add query name and source

Add query name and source to each download feature.

Linked to SCM-373

Blocked by #540

This was added already, maybe a duplicate?

yup..closing

Tested anyway.  Was able to upload all three files geo.jsn, kml and csv.  Cleared map.  Restored all three instances and they all uploaded with source and query name.


refactor Project and add Project test

Two files update:

In Project.java, I replaced those get and set functions with @Getter and @Setter, lombok annotation. (Test all pass)
I added ProjectTest.java, which is used to test functions of Project.

Suggest not to use the main branch to initiate a pull request.
See this article for more details.

Also suggest rename this PR, because the word project is easy to misunderstand other meanings, like github ... etc.
Maybe you can change to refactor Project entity and add Project entity test ...

Before I started a review, there exist two same-name commit, makes me go into ponder, what are these commits doing? Why they have same message? Why the message is equal to this PR's name... XDDDD
So maybe the commit message refactor Project and add Project test can be change to a more clear expression~~

please notify us after this PR is ready for review and merge.


IntExtensions factors() func added

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[ ] I have added a changelog entry describing my changes.

          1 Warning

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

SwiftLint found issues
Warnings

File
Line
Reason

IntExtensionsTests.swift
120
Lines should not have trailing whitespace.

  Generated by :no_entry_sign: Danger

Looks like this has gone stale so I'm going to close it for now 👍


upgrade dependencies to avoid a high

npm reported a high vulnerability in our dependencies. Upgrading them.

@thpani It looks like we have to upload the contracts again?

Very likely, yes.
I automated deploying and populating for the TS monitor E2E tests. But I guess a one-off running the script is easier here.

Very likely, yes. I automated deploying and populating for the TS monitor E2E tests. But I guess a one-off running the script is easier here.

I am trying to repopulate the contract via ./scripts/setter-populate.sh. It fails with an error:
❌ error: transaction submission failed: Some(
    TransactionResult {
        fee_charged: 70341,
        result: TxFailed(
            VecM(
                [
                    OpInner(
                        InvokeHostFunction(
                            ResourceLimitExceeded,
                        ),
                    ),
                ],
            ),
        ),
        ext: V0,
    },
)

I guess, this is not expected?


fixes #2945 - oncomplete won't fire if instance is torndown

Description of the pull request:
This PR adds a check to only fire oncomplete if the instance is not already torndown. Some of the oncomplete lifecycle tests failed where we  create a Ractive instance followed immediately by a teardown. Because the oncomplete hook is fired asynchronously the teardown call is made before oncomplete is invoked.
As a workaround I call the teardown in the next tick by wrapping it in a setTimeout,
Fixes the following issues:
#2945
Is breaking:
Perhaps in tests, where creating a Ractive instance is immediately followed by a teardown call, and one expects the oncomplete hook to still fire.
Reviewers:
Yes

Looks good to me 👍 Might need to squash the commits tho.

Looks good to me too. I'm :+1: with a squash and forced push.

err, squash is something I need to do from my side? I've no idea what that is...

Basically reset to origin/dev (leaving all of your changed files sitting around), commit them again, and force push. I think this will work, though I've never tried resetting to a remote branch...
git remote add ractive git@github.com:ractivejs/ractive
git fetch ractive dev
git reset ractive/dev
git status
# do the files look right?
git add .
git commit -m "Add a check not to fire oncomplete if ractive instance is torndown - fixes #2945"
git push -f origin dev

Forgot to say what that should do. That should put all of your changes into one commit and then overwrite this PR's change set with that single commit. It's mostly to keep the main commit log a little cleaner, as we use it for changelogging :smile:

Thanks @evs-chris, managed to run those commands. Should be ok now.

:+1: Thanks!


Remove B325: tempnam

Describe the bug
In Python 3.0, the functions os.tmpnam(), os.tempnam() and os.tmpfile() were removed in favor of the tempfile module.  As a result, we no longer need to have a check for them in Blacklist.calls
https://docs.python.org/3/whatsnew/3.0.html?highlight=tempnam
To Reproduce
Steps to reproduce the behavior:

Read https://docs.python.org/3/whatsnew/3.0.html?highlight=tempnam

Expected behavior
We should remove unnecessary checks/plugins that don't apply any longer.
Bandit version

bandit 1.6.3
  python version = 3.9.0 (default, Dec  5 2020, 17:22:35) [Clang 12.0.0 (clang-1<IP_ADDRESS>)]

Additional context

Fixed with #859


Suggestion: Detect if website is already dark

What if we would detect if a website is already dark by checking whether the inverted colors are brighter than the original colors?
This would be useful in the following scenario, where one can't just blacklist the website in question, because the dark theme is dynamically applied on a per-user level:

Duplicate of https://github.com/alexanderby/darkreader/issues/81


Require that Python code conform to the Black formatter

Let's expand the scope of mandatory Black formatting in CI. Currently only two directories are tested, but I think this can safely be expanded to include all of calyx-py for instance.

All done! ✨ 🍰 ✨
22 files reformatted, 32 files left unchanged.

@polybeandip interested? Since it bit you once before and you figured this out in OCaml-land recently?

Happy to set up the github action for this

@anshumanmohan are you sure that your link to the action in format.yml actually checks that the python files in fud/ and frontends/systolic-lang have been formatted?
From the docs:

Please note that the --check flag is required so that the workflow fails if Black finds files that need to be formatted.

ATM, none of our current calls to psf/black@stable actually have the --check option. You can also inspect previous runs of CI to see that both these directories have files that black would have formatted, but CI doesn't fail.
Shall I setup CI so that it fails if black would format any files in fud/, frontends/systolic-lang/, frontend/queues/, or calyx-py? I imagine I should probably ask the people maintaining fud and systolic-lang before doing that.

Thanks for the catch! I just operated in faith and assumed that it was in fact working. But, as you've noted, there seem to be some slippages. To me it feels 100% worth doing, but let's get green lights from @calebmkim and @rachitnigam anyway!

Sounds good to me!


Display the cUSD balance to the right number of decimal places

Highlighted in Issue #9
The app is connected to my Valora
Earlier:
cUSD Balance:<PHONE_NUMBER>0057678186
New result:
cUSD Balance: 25.00

looks good, thanks!

Thank you @critesjosh


Package Requested

[ ] I would like someone else to build the manifest.
[X] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: Helm
Publisher Url: https://helm.sh
Package Name: Helm.Helm
Description: helm is a package manager for Kubernetes
Package Version: 3.7.0
Installer URL: https://get.helm.sh/helm-v3.7.0-windows-amd64.zip
Current Package Identifier (for new versions):

Not yet, that's being tracked in https://github.com/microsoft/winget-cli/issues/182

If you can get it merged into the helm repo so that it's automatically generated for every release (and signed?), we can add it. (Just want to make sure they're willing to support it so you don't have to maintain a fork for all eternity)

I think they are using circle ci build internally, so i could raise a PR for that.

Package is now available as Helm.Helm
<--
Close with reason: Resolved;
-->


handle standard pointer events

What does this fix?

Weird behavior on Windows 10 Phone version

And this fix will use pointers in compatible browsers (no MS prefixes)

Fine, I'll use the code but I'll put in another commit so that it adheres to the code style in dragula

Thanks


Cannot build tests

CMake Error at test/CMakeLists.txt:6 (find_pkg_config):
Unknown CMake command "find_pkg_config".

find_pkg_config was removed in 93111d9eb590a55181c86fb07ec4a27e8496b3d6.

My  bad, I didn't test since at that time I missed those files.
Thanks for the fix.


keystroke to open doc page in new browser doesn't work on macOS

I'm using version 0.0.5 on macOS Sierra. I cannot use the alt-o keystroke to open pages in the browser.

I'm not sure how I landed with v0.0.5, but 0.6.8 is great!


[Feature]

Description
Add bgcolor change on hover on buttons
Screenshots

Additional Information
No response

Hey @jayk-gupta Are you working on it ?

yes

Then make a pr

Sent PR


Upload 2016 PhenoTractor data to trait database

The earlier issue https://github.com/terraref/computing-pipeline/issues/211 was closed, apparently before the data was inserted into the trait database.
Data are in https://docs.google.com/spreadsheets/d/1NNKtmWFBdoh2iaaXZh9fsu_nNRLMpytJ8Av1bHxpgIg/edit#gid=698446872 Need to be uploaded
Method = 'Maricopa Phenotractor'
Citation title = 'Maricopa Field Station Data and Metadata'
Completion Criteria

[ ] reformat data as needed for upload
[ ] upload data from four dates in the spreadsheet.
[ ] upload raw datasets to Clowder (?)

I can help with the steps to reformat data and upload the data to betydb.  I'll talk with Pedro and we'll make a plan.  I've never uploaded to clowder but if that's of interest I can learn with some guidance from someone in Illinois.

Issue moved to terraref/reference-data #192 via ZenHub


:Clap files results in python import errors

I have latest vim-clap (v0.15) installed and built the python dynamic module successfully. However, when I tried running `:Clap files", I got the following error message:

vim-clap: Vim(python3):ImportError: cannot import name 'clap_fzy_rs' from 'clap.fzy' (/home/oulongwu/.vim/pack/plugins/start/vim-clap-0.15/pythonx/clap/fzy.py

Rebuild the Python dynamic module completely and if the error persists, you can disable the dynamic module using let g:clap_use_pure_python = 1. The python dynamic module is not neccessary. And there is no changes about Python dynamic module in since v0.14.

Rebuild the Python dynamic module completely
To be clear, this is a new install from scratch so the python dynamic module is built from scratch. I also tried master branch and the error message is the same.

Can't reproduce, could be an issue of your local environment.


White selection color obscures important details

Bug:
When a clip is selected, it is highlighted in white, which is the same color as some of the text on the clip, making the text unreadable. Furthermore, when adjusting a clip's color in the inspector, the clip color is not visible until you deselect the clip.
Suggested Fix:

The timecode text on each clip could flip from white to black depending on the clip's fill color (like the clip name text does already).
A better choice for the selection highlight color (perhaps a tint, brighten or darken) instead of pure white could work better.
Both of these problems would be avoided if the selection highlight was a border instead of coloring the entire clip.

FYI, this was partially addressed by #22 but could still use some improvement.

I'd love to take this issue on as part of ASWF Dev Days 2024!


fix(*): build process

Now out-of-date. Superseded bu subsequent dependency updates, e.g. https://github.com/thompsonsj/slate-serializers/pull/139.


Ensure storage markers are kept in std code

This is a follow-up to #3063 that:

Instead of turning on the address sanitizer to ensure storage markers are kept in MIR, it turns off the specific MIR pass that removes them (https://rust-lang.zulipchat.com/#narrow/stream/182449-t-compiler.2Fhelp/topic/.E2.9C.94.20Keeping.20MIR's.20storage.20markers)
Turns off that MIR pass while compiling std as well to ensure any bugs of the same nature in std are captured by Kani.

Resolves #3079
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

I moved the change that directly turns off the MIR pass that removes storage markers instead of using the address sanitizer into a separate PR (#3083). So this PR is now only concerned with keeping storage markers in std.


Update mkdirp for security

minimist had to be upgraded to 1.2.3
https://www.npmjs.com/advisories/1179

@malept can you please merge & release?

For reference, the changes from v0.5.1 to v0.5.3 of mkdirp are: https://github.com/isaacs/node-mkdirp/compare/d4eff0f06093aed4f387e88e9fc301cb76beedc7...v0.5.3 (the only practical change being the minimist dependency bump from 0.0.8 to ^1.2.5)

Just to connect to issue filed: #86

Anything else holding this back?

@brettz9 No, except that @maxogden, who I believe has sole authority to execute this merge, has not been active on this project since May 2018, despite being active in private repos.   It could be that he just needs a few more days for this to get to the top of his queue.    He seems very busy.
@brettz9 : Are you here because of the dependency from puppeteer or another package?
Looking at his github profile and https://maxogden.com/, I suspect he is located in Portland, Washington.  That state, along with California and New York, has been particularly hit hard by COVID-19, and it's only going to get worse.  Wishing everyone the best.

@SakiiCode's suggestion of using https://www.npmjs.com/package/npm-force-resolutions was tried successfully (along with npx and entries in package.json) as documented in https://github.com/vuejs/vue-cli/issues/5285#issuecomment-601280030

@SakiiCode There's a suggestion of using 1.0.3 (https://github.com/maxogden/extract-zip/issues/86#issuecomment-601699895) instead as it doesn't depend on minimist.

@cyclingzealot 1.0.3 (besides other changes) uses promises instead of callbacks so migration requires the rewrite of the main and the testing script as well. It may be possible in the future but it's a bit out of scope of this PR.
Thanks for pointing out though

@SakiiCode There's a suggestion of using 1.0.3 (#86 (comment)) instead as it doesn't depend on minimist. I'm not familiar enough with pull requests to see if I, as commentator, can edit them.

In node 10 (the currently oldest supported version), there's also a {recursive: true} option so mkdirp could potentially be skipped all together

@brettz9 No, except that @maxogden, who I believe has sole authority to execute this merge, has not been active on this project since May 2018, despite being active in private repos. It could be that he just needs a few more days for this to get to the top of his queue. He seems very busy.

Man, it is just a click! The merge is ready to go! How much time do you need to click a button? Or am I missing something here?

As suggested by @SakiiCode, I used https://www.npmjs.com/package/npm-force-resolutions as a temporary fix.  I documented on SO at https://stackoverflow.com/a/60795003/1611925 and https://stackoverflow.com/a/60794976/1611925 .    It fixed without difficulty but I have not tested execution yet.

@brettz9 : Are you here because of the dependency from puppeteer or another package?

svg2png -> phantomjs-prebuilt

@brettz9 No, except that @maxogden, who I believe has sole authority to execute this merge, has not been active on this project since May 2018, despite being active in private repos. It could be that he just needs a few more days for this to get to the top of his queue. He seems very busy.

Sure. Though per https://www.npmjs.com/package/extract-zip @malept is listed as an npm "collaborator" who should I think per this designation also be able to make an update.

Looking at his github profile and https://maxogden.com/, I suspect he is located in Portland, Washington. That state, along with California and New York, has been particularly hit hard by COVID-19, and it's only going to get worse. Wishing everyone the best.

Indeed.
@jfoclpf : Maintainers who put their name on a project typically want to take a good look, run tests, etc., so even a one-line change can need to be shelved until a person has the time to look everything over and be satisfied that the specific fix is the best way to go. Of course one hopes with vulnerabilities, there can be some added priority, but if it comes down to it, any of us can release a forked project.

If I understood correctly the only proposed difference is the version of a dependency, thus it is one line in package.json that changes. If npm test is well designed and configured in travis you don't even need to worry about tests cause you may only pull request if the test on the proposed changes passes successfully. When it comes to automatic pull requests from github regarding vulnerability I just blindly accept them. Look, I am also a developer of open source, and I perfectly understand we have no duties at all regarding our products that we set available for free. But here is just a minor version change in a dependency which is ready to go. Anyway this is a great package  ;)

@cyclingzealot Added a tutorial to the first post, please check if you did the same and got the same results.

Thanks for your patience in these uncertain times. I've released a version of extract-zip depending on mkdir 0.5.4 in version 1.6.8.

Thank you very much, @malept! Just may want to clarify that you released a version depending on "mkdirp" (as there is also a "mkdir" npm package as well). :-) Best wishes...

Thanks @malept :) All the best!


Why is size defined as an int64 for OCI content descriptor?

I would think size should be described as uint64 ?

On Thu, Sep 29, 2016 at 09:06:11AM -0700, asridharan wrote:

I would think size should be described as uint64 ?

More discussion on this in #153.

I still don't understand the reasoning behind this decision. The only thing I see from the conversation seems to be its okay to use int64 since we "don't" allow negative values, and somehow the motivation seems to be that "go" has done it this way. A spec should not be language specific, so seems very arbitrary at this point.
Also, the fact that size is expected to be positive and validation needs to be performed has not been stated in the spec. Just being "picky" here since this is the spec and should be treated as a gold standard. The reasoning has to be beyond question.

On Thu, Sep 29, 2016 at 12:04:34PM -0700, asridharan wrote:

I still don't understand the reasoning behind this decision.

I would also prefer to see an unsigned integer, but was unable to make
a convincing-enough case for it.

… and somehow the motivation seems to be that "go" has done it this
way.

The compat argument wasn't about Go, it was that docker/distribution
uses int64 1.  But docker/distribution uses different media types,
and all reasonable size values will fit into both an int64 and a
uint64, so I don't see a problem with switching the OCI type to use
uints that carry at least 64 bits (more discussion in #153 where I
argue for following C and JSON in not capping the maximum size).

Also, the fact that size is expected to be positive and validation
needs to be performed has not been stated in the spec.

The non-negative-ness is covered by the existing 2:
This REQUIRED property specifies the size, in bytes, of the raw
content.
because you can't have negatively-sized content.
Validation is covered by the existing 3:
If the length of the retrieved content does not match the specified
length, the content SHOULD NOT be trusted.
The only thing wrong about using a signed integer to store that
information is that you're wasting a bit and inviting these “why is
this signed?” discussions ;).

thanks @wking . Missed the argument of docker using int64. Yeah I agree using int64 rather than uint64 seems an unecessary confusion to percolate into the spec.
I wasn't aware the spec "needed" to be compatible with docker specification. I thought the idea was to move forward and look for something that is not tied to current specs?

For most of the reasons already discussed I'd prefer uint64, and I'm not convinced by the reasoning for staying with int64 - so, you would have my +1, but I am not going to drive this.

We've already discussed this in detail and made a decision. Let's not reopen this discussion.


Can't Add iKommunicate as Signalk Web Socket Connection

I can't successfully add iKommunicate gateway as a WebSocket "Connection" on a SignalK Server. It only results in 501 error. I can connect web socket using chrome web socket tool no issue.
But adding iKommunicate gateway as a WebSocket "Connection" on the SignalK Server results in 501 error.
/home/pi/.signalk/settings.json
...
{
"pipeElements": [
{
"type": "providers/simple",
"options": {
"logging": true,
"type": "SignalK",
"subOptions": {
"type": "ws",
"host": "<IP_ADDRESS>",
"port": "80",
"providerId": "53353100-4148-484e-3230-383039353035",
"useDiscovery": false
},
"providerId": "53353100-4148-484e-3230-383039353035"
}
}
],
"id": "53353100-4148-484e-3230-383039353035",
"enabled": true
}

Unexpected server response: 501

The error message is from a Varnish http server/reverse proxy. Are you sure you have the correct ip address?

For sure IP is right of the iKommunicate server...<IP_ADDRESS>

Oh right, I got served an error message from Github for the screenshot! I don’t have an iK available right now to be able to test.

You want to connect to my network?

This is not working because the iKommunicate does not conform to the SignalK subscription specs. It does not support subscription=none.
This was broke in the node server when I upgraded to use the new signalk-is-client library.
I will work to fix this,  it probably won’t be unit next week.
I’m the meantime you can use node server version 1.20.0.

Amazing.  I will try it now.  I have been messing with this stuff for a few years now and just finally to get serious.  The response from you guys is tremendous and inspiring.
So does that mean everyone who has an iKommunicate will be broken?  Is there anyway to adjust the subscription?  Thank you!

Yeah, will break anyone connecting to an iKommunicate.
I don't think there's an easy way to adjust the subscription, but I have not yet dug deep enough into the client library yet...

I can confirm 1.20.0 it is working.  Instrument panel now is not but I wasn't planning on using it anyway!  Thank you!

Too much work to try to make signalk-js-client work with this, so I made a small simple plugin to do it: https://github.com/sbender9/signalk-ikommunicate-connector
Should show up on the App Store shortly.
Remove your existing connection and install and configure this plugin.

Roger that, I am sure someone will find it useful besides me!  I can't believe more people do not have iK and alerting of the same issue.

Showed in the store but doesnt seem to work.

Never seen that before. On the boat and don’t have an iKomminicate to test with. Will be home tomorrow and will grab it.

When I tried it the first time, I had my previously attempted Connection still in the SignalK connections.  I removed it and now I am getting the below.  Possibly a "duplicate" conflict?
No data showing coming in but it says connected now.

Looks like iK server had crashed when I had taken above screen shot.  Now its back to this and its refreshing like crazy.  I think its crasing the iK gateway.

Now I am trying a work around.  Connect to iKommunicate Gateway via node-red / websocket and then send the data to the SignalK server.  Connecting the iKommunicate Gateway websocket was easy but now I am trying to figure out how to forward all the data to the SignalK server.
iKommunicate Gateway (<IP_ADDRESS>)
SignalK Server - Node-Red (<IP_ADDRESS>)

Thank you everyone for trying.  The work-around worked.  I used node-red to connect to the iK signalK gateway and then sent it to the SignalK server with a "signalk-send-pathvalue node".
Let me know if anyone needs help with it.  I hired a great guy that knew nothing about SignalK but knows Node-Red and he got it done!  Thanks to https://github.com/alexmed

I can't believe I am the only one connecting an iK to SignalK!


K8S002: allow configuration for 2 replicas

Use case
There are many applications that use leader election (e.g. controllers) to achieve redundency where there's not much practical benefit to running 3 pods over 2. This is of course different to qorum based HA where minimum 3 pods are appropriate
when running eksup, pods with 2 replicas are erroneously reported as violating K8S002 and therefore are not highly available
Solution/User Experience
some kind of configurable way to override or edit rules for specific workloads would be great, but I appreciate this means introducing a config file and that adds a lot of complication
An ideal solution would allow me to say "workload X should have minimum Y replicas" so that we don't accidentally green light a workload with a single replica
Alternative solutions
the override could also be a cli flag, but I can see this getting very large for large clusters

I think that makes sense - and would be applicable for other aspects such as https://github.com/clowdhaus/eksup/issues/5


JmxMetaMapBuilder.buildOperationMapFrom throw StringIndexOutOfBoundsException

JmxMetaMapBuilder.buildOperationMapFrom for method "get" or "set" throw StringIndexOutOfBoundsException
example:
class TestGet {
	String sb

	def get(String str) {
		str
	}

	static main(args) {
		def testGet = new TestGet()
		def property = testGet.metaClass.getMethods()
		def ms = testGet.getClass().getDeclaredMethods()*.name
		def jmx = new JmxBuilder()
		def beans = jmx.export {
			bean(
					target: new TestGet(),
					attributes: [],
					operations: "*"
			)
		}
	}
}

after run:
Exception in thread "main" java.lang.RuntimeException: Failed to create component for 'bean' reason: java.lang.StringIndexOutOfBoundsException: begin 2, end 4, length 3

It would be better to have a test in this PR

Merged with some minor tweaks.
Thanks for your contribution!

BTW, I find you are the 300th contributor. Congratulations!


fix: drop npm caches in CI

Drops the caches in CI because they are unused thanks to npm ci - which has a weird but expected behavior.
There is no significant difference in the time when installed from a cache or in "online" mode. We use workspaces and have largely frozen the packages to the root lock file. If anything, we should now be closer to production builds, avoiding outages like last week.

To be clear, the caches are used - but in a wasteful manner. Meanwhile, they have been hitting limits we do not need to hit.


Improve UI for "All streams" subscription status

The "All streams" view currently uses slider toggles to indicate stream subscription status and allow the user to change it. This UI feels off, because toggles are generally used for settings, and the stream itself is not a setting.
We should try changing the UI to something that feels more appropriate. A good candidate is using "+" for streams the user is not subscribed to, and a checkmark for streams the user is currently subscribed to. See screenshots below.
Related Zulip thread: https://chat.zulip.org/#narrow/stream/48-mobile/topic/.22All.20streams.22.20subscribe.20toggles
Current Zulip UI:

Alternative pattern (screenshot from Pocket Casts podcast app):

I am working on this.

Hey @alya, Looks like no works has been done so far. Can give it a shot to this. Thanks 😊

Hey @chrisbobbe!,

The idea is to build a Zulip button because when I was going through the code and I realised that, We are using ZulipSwitch component which uses Switch in itself.
So, I think we have to do the same thing, like we have to build the custom button for this issue such as ZulipSubscribe button which will behave the same way the only difference will be to change the icon on respective event.
Thanks 😊

I moved over to the react-native-vector-icons I found these two icons which we need

Both are from FontAwesome.
Thanks 😊

The reason we need separate component such as ZulipSubscribe because later in the future we might need switch button.

Hey @chrisbobbe!,
I've figured out some of the parts of related to this issue and UI is kind of done but when I'm pressing the button then event is not getting triggered.
So far what I've understood is we are using ZulipSwitch component as our switch button and it takes 3 props which is value, onValueChange, disabled. Now, I got the little idea from the comments that what these props are.
And, I've done the similar thing in my approach but, somehow when I'm pressing the button the event is not getting triggered.
It would be a great help if you could help me, where to look for. Thanks 😊

It would be a great help if you could help me, where to look for.

Sure. It's probably easiest for me to help if I could take a look at the code. It doesn't have to be polished into a reviewable PR for this purpose; you could make a PR marked as a draft, or just push your branch and tell me its name. 🙂

Thanks @chrisbobbe!, I've made a draft PR. Please have a look at that, #5333. Thanks 😊


Support for combined PDFs from multiple pages

Describe the feature
Is it possible to generate a single PDF file that combines content from multiple pages using astro-pdf? Specifically, I would like to:

Combine several pages (e.g., index, page1, page2, page3) into one PDF.
Ensure the pages are included in a specific order (e.g., starting with index, followed by page1, then page2, and finally page3).

If this is not currently supported, is it a feature that is planned for future updates? Alternatively, do you recommend any workarounds or configurations within astro-pdf to achieve this?
Thank you for your guidance!

hi, unfortunately this isn't something that can currently be done with astro-pdf itself, though I may consider adding this feature in the future.
however, you can configure astro-pdf to generate each of the pages separately, then use a library such as pdf-merger-js to combine the pdfs in the order you want. (and then delete the original pdfs if you wish)
in the latest version of astro-pdf (1.5.0) which I just released, you can specify a runAfter callback to run the pdf merger
here's an example of that: (the runBefore is optional you can ignore it)
https://github.com/lameuler/astro-pdf/blob/8d222980b79f614a0c49bebbca95535b960feb19/test/fixtures/merge-pdfs/astro.config.mjs#L1-L46

This is great! Thank you so much!


update list and dict to work with loadReduce

This commit updates findClass to handle builtins/list and dict, and adds Call() funcs to each to work with loadReduce().

Thank you!


[Fix](inverted index) fix memory leak in inverted index when encountering fault

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall


harry potter and the chamber of everything wrong with this

needs more cheese
more than you're thinking
still more than you're thinking
also could use a command that deletes sys32

it has a delete sys32 command
needs cheese tho


Crash with MutableObservableArray removeAll()

*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'attempt to insert row 0 into section 0, but there are only 0 rows in section 0 after the update'
Im working with MutableObservableArray
    var content       = MutableObservableArray<VM>()
for tableview
I use this methods for update when uiswitch changed, so I clean 'content' and create new one
func
updateVMS(mode: switchmode) {
        content.removeAll()
        content.insert(model1VM, at: content.endIndex)
        content.insert(model2VM, at: content.endIndex)
    }

also in subclasses I override it, and do:
override func updateVMS(mode: switchmode) {
        super.updateVMS(mode: mode)

        switch mode {

        case .switchmode1:
            let switchmode1VMS = users.array.map {
                return SomeItemVM($0)            }
            content.insert(contentsOf: switchmode1VMS, at: content.endIndex)
        }
    }

First time it calls updateVMS  twice, but when I change switch manually, it crashes just in code string:
       content.removeAll()
with issue at top, however I don't insert stuff, just trying to clear it.
Is there anything I'm doing wrong? Any ideas how to fix?
If I remove content.removeAll() code piece, it doesn't crashes
Bond 6.5.2

Can you verify that you are calling Bond's .removeAll() and not the one in Swift?

I'm puzzled why are you doing insert(model2VM, at: content.endIndex) instead of append(model2VM)?

^^ Also True. I believe I had a similar issue and I switched to using append instead. (after questioning why I was trying to insert at the last index anyway)

Well, you can't append(contentsOF: )
And Is there any diffence? I can't insert in .endIndex ?

If this is still a problem, I would suggest batching multiple changes:
content.batchUpdate { content in
       content.removeAll()
       content.insert(model1VM, at: content.endIndex)
       content.insert(model2VM, at: content.endIndex)
}


raft: raft learners should be returned after applyConfChange

fix https://github.com/pingcap/raft-rs/issues/12
add a new test test_raw_node_propose_add_learner_node() in test_raw_node.rs to cover the fix in apply_conf_change(..)

LGTM

@BusyJay test_remove_learner is updated now

Thanks!


Dutch translation

Hi,
I have added Dutch language files for Ticketit.
Kind Regards,
Rick

Thanks.


need to search terms with wildcard on both ends

When I type "test" it should search from

"hey test",
"test 123",
"notest"
"testyes"

2 and 4 works fine with following code, but 1 and 3  doesn't? Could anyone please help?
function searchPages(searchTerm) {
        var deferred = $q.defer();

        pouchdb.query(searchMap, {
            startkey     : searchTerm,
            endkey       : searchTerm + '\uFFFF',
            include_docs : true
        }).then(function (result) {

            console.log(result);

            var results = result.rows.map(function(r) {
                 return r.doc;
            });

            deferred.resolve(results);

        }).catch(function (err) {
            // handle errors
            deferred.reject(err);
        });

        function searchMap(doc) {
            if (doc.type === 'page') {
                emit(doc.content);
            }
        }

        return deferred.promise;
    }

P.S I've already went through comments at https://github.com/nolanlawson/pouchdb-quick-search/issues/8
I then decided to use map/reduce, so this issue might not exactly be related to this plugin, still wanted to give it a try :)

If you need to search within a string, then your map() function will have to emit() once for each word.

On Sep 23, 2016, at 5:07 AM, Prashant Patel<EMAIL_ADDRESS>wrote:
When I type "test" it should search from

"hey test",
"test 123",
"notest"
"testyes"

2 and 4 works fine with following code, but 1 and 3 doesn't? Could anyone please help?
function searchPages(searchTerm) {
var deferred = $q.defer();
    pouchdb.query(searchMap, {
        startkey     : searchTerm,
        endkey       : searchTerm + '\uFFFF',
        include_docs : true
    }).then(function (result) {

        console.log(result);

        var results = result.rows.map(function(r) {
             return r.doc;
        });

        deferred.resolve(results);

    }).catch(function (err) {
        // handle errors
        deferred.reject(err);
    });

    function searchMap(doc) {
        if (doc.type === 'page') {
            emit(doc.content);
        }
    }

    return deferred.promise;
}

P.S I've already went through comments at #8 https://github.com/nolanlawson/pouchdb-quick-search/issues/8
I then decided to use map/reduce, so this issue might not exactly be related to this plugin, still wanted to give it a try :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/nolanlawson/pouchdb-quick-search/issues/60, or mute the thread https://github.com/notifications/unsubscribe-auth/AARUwqCUk_uVReT_i0IpuR2zoB1uaBj3ks5qs8D0gaJpZM4KE6EM.


🚀 - Number-mask

Which package(s) are relevant/related to the feature request?
@maskito/kit
Description
We should create mask for InputNumber.
Overwrite-mode is equal to shift.
Proposed API
import {Maskito} from `@maskito/core`;
import {maskitoNumberOptionsGenerator} from `@maskito/kit`;

const inputNumber = new Maskito(
    element,
    maskitoNumberOptionsGenerator({
        min: -100,
        max: 100,
        precision: 2,
        decimal: 'always',
    })
)

Configurable options
min
Default value: Number.MIN_SAFE_INTEGER.
max
Default value: Number.MAX_SAFE_INTEGER.
precision
A number of digits after comma.
Default value: 0 (no digits after comma are allowed).
decimal
Possible values:

always - number of digits after comma ALWAYS is equal to the precision.
it is not possible for user to delete digits after characters!
If user press Backspace / Delete, mask should replace deleted characters with zeroes and place caret to the left/right.

not-zero - number of digits after comma can equal or less to to the precision.
User can leave decimals with tailing zeroes. For example (precision: 4), 3.4300.
User can delete tailing character. For example (precision: 4), 3.43.
The only restriction in this mode: user cannot insert more characters than precision.

Default value: not-zero.
TODO: should we replace decimal: 'always' | 'not-zero' with alwaysShowDecimals: boolean (or something similar) ?

I think yes, decimals should be boolean. And don't forget, that number format is locale dependent. We also want to make dot/comma autocorrect, if none of these symbols is used as thousands separator.

We probably want to make trailing zeroes configurable somehow. They make sense for money, but not for numbers in general.


Loading pretrained model and using predict.py on it

I tried to load the pretrained model you have provided and using it to predict on the dev set. I get the following error while loading the model by using keras load_model(model_path). Following is the error
Traceback (most recent call last):
File "predict.py", line 35, in
model = load_model(args.model, custom_objects())
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/models.py", line 240, in load_model
model = model_from_config(model_config, custom_objects=custom_objects)
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/models.py", line 314, in model_from_config
return layer_module.deserialize(config, custom_objects=custom_objects)
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/layers/init.py", line 55, in deserialize
printable_module_name='layer')
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/utils/generic_utils.py", line 140, in deserialize_keras_object
list(custom_objects.items())))
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/engine/topology.py", line 2500, in from_config
process_node(layer, node_data)
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/engine/topology.py", line 2459, in process_node
layer(input_tensors, **kwargs)
File "/Users/deepankar/anaconda3/envs/squad/lib/python2.7/site-packages/keras/layers/recurrent.py", line 514, in call
full_input_spec = self.input_spec + additional_specs
TypeError: unsupported operand type(s) for +: 'NoneType' and 'list'

Sorry for late response,
What Keras version do you use?

Which Keras version did you use @mahnerak  ??

Same problem
I am using keras 2.1.5 to use multi gpu model
btw, training is really slow
it takes about 5 hours for each epoch on one 1080ti with 150 batch_size


As an HR advisor I want to view an employee's details so that the employee's promotion request can be supported.

Haven't figured out the additional data but will hopefully.


PublishCodeCoverageResults should succeed with a warning when summary file does not exist

When the file PublishCodeCoverageResults task cannot find the summary file, the task fails with the following output:
Reading code coverage summary from 'C:\vsts-agent-win7-x64-2.111.1\_work\558\s/coverage/cobertura-coverage.xml'
##[error]Unable to process command '##vso[codecoverage.publish codecoveragetool=Cobertura;summaryfile=C:\vsts-agent-win7-x64-2.111.1\_work\558\s/coverage/cobertura-coverage.xml;reportdirectory=C:\vsts-agent-win7-x64-2.111.1\_work\558\s/coverage;]' successfully. Please reference documentation (http://go.microsoft.com/fwlink/?LinkId=817296)
2017-07-14T21:56:34.0259256Z ##[error]File 'C:\vsts-agent-win7-x64-2.111.1\_work\558\s/coverage/cobertura-coverage.xml' does not exist or is not accessible.

Contrast this with the output from the PublishTestResultsTask in a similar scenario:
##[warning]No test result files matching testresults\junitresults.xml were found.

Related code: https://github.com/Microsoft/vsts-tasks/blob/5100fd3719a26cb6021752f3e4326609df955df8/Tasks/PublishTestResults/publishtestresults.ts#L26
I propose the PublishCodeCodeCoverageTask should check for the existence of the summary file and issue a warning instead of an error in some cases. If the summary file does not exist and the "Fail When Code Coverage Results Are Missing" option is not checked, then output a warning and allow the task to succeed. If the summary file does not exist and the "Fail When...Missing" option is checked, then keep the current behavior: an error and a failed task.
If a missing file should fail the build, the Build Quality Checks task can handle that job.
This change will enable reuse of task groups across builds that do and do not include code coverage summary files.

OK, @Blackbaud-TommyVernieri, I'm convinced.  I've made a PR with your proposed behavioral change: https://github.com/Microsoft/vsts-tasks/pull/4820, which is currently under review.
Thanks for your contribution!


Reviewing all uses of an annotation

Currently, the Annotations tab shows the number of uses of any annotation.  It would be very helpful if we could get a view that gives us the subset of the submissions that used that annotation.  Even a list of group numbers would be great.
Use case: You are using deductive annotations, and TAs are in the middle of marking. You realize you need to change the way they've been grading, but don't have enough TA time to have them review every graded submission.  Almost always the use of a particular annotation indicates which ones need review.  This actually is relevant to rubric marking also.
Diane

Hi, I'm pretty sure the "Usage Details" table will show you the info you're looking for. It can be accessed by clicking the "Usage Details" button below each annotation definition.


[FLINK-14794] Fixing KeyedStream#transform to properly relay type information

What is the purpose of the change
Fixing KeyedStream#transform to properly relay type information if invoked with factories.
Brief change log

Overloading the correct function of DataStream in KeyedStream, such that the type information is passed to both operator and factory based transformation methods.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Hi @AHeise I've build your PR locally and validated that indeed I can use an *StreamOperatorFactory on a KeyedStream.

I believe many users (actually including me) are using the public interface, do we have to achieve this by breaking users' codes?


Breaks node4 compatability (4.2.6)

I do not know if this is something that you are after since this node version (4.2.6) is no longer officially supported but the 4.2.1 release breaks the package with this error:
TypeError: disallowedKeys.includes is not a function
    at Array.some (native)
    at pathSegments.some (/usr/src/app/node_modules/dot-prop/index.js:10:51)
    at Object.get (/usr/src/app/node_modules/dot-prop/index.js:40:19)
    at UpdateNotifier.check (/usr/src/app/node_modules/update-notifier/index.js:78:16)
    at __dirname.check.checkNpm.then.checkNpm.module.exports.options (/usr/src/app/node_modules/update-notifier/index.js:151:17)
    at Object.<anonymous> (/usr/src/app/node_modules/nodemon/bin/nodemon.js:15:29)

The issue comes from this https://github.com/sindresorhus/dot-prop/commit/3039c8c07f6fdaa8b595ec869ae0895686a7a0f2#diff-168726dbe96b3ce427e7fedce31bb0bcR10
It is kind of a bummer sincenodemon defines its dependency to this package as '^...' through update-notifier which will automatically pick the latest minor version which is 4.2.1
4.2.0 works fine.

That's a problem for you or nodemon to handle. This package correctly defined Node.js 8 when that change was made.


library-core.aar requiring READ_EXTERNAL_STORAGE_PERMISSION without reason

This is a slightly more unpleasant relative of  #68, but still not a big deal.
The  "unpleasant" part is that, while in #68 the permission can be safely ignored (it's just in the docs), the inclusion in library-core.aar's manifest trickles into the app and shows on the store (unless conscious effort is made to remove the line into the aar's manifest manually).
Apart from enforcing an unsophisticated practice, as I said: "no big deal".

The core library actually needs the READ_EXTERNAL_STORAGE permission to access the profile of the currently-paired Cardboard viewer.
See https://developers.google.com/cardboard/android/get-started#manifest_file

Nope :)
core.aar requires  WRITE_EXTERNAL_STORAGE already, therefore the READ_EXTERNAL_STORAGE permission is implied:
From the READ_EXTERNAL_STORAGE documentation:
Any app that declares the WRITE_EXTERNAL_STORAGE permission is implicitly granted this permission.

And btw, this doesn't prevent the read permission to show up in the google play store when installing, even if not really needed.
Again, not a big deal at all, just not the best possible style...

Oh I see, I didn't realize this bug was about READ_EXTERNAL_STORAGE being redundant. Thanks for clarifying.
Yes, we should get rid of that one too, thanks for reporting.

Fixed in v0.8.
WRITE_EXTERNAL_STORAGE is not required, so we just declare READ_EXTERNAL_STORAGE.


Just need to make on change

Hi
This script works perfectly for me except one cosmetic issue.
When I click on one of the mapped areas the box opens which says "area label clicked".
I have a popup which contains descriptive text and graphic which opens.
I would like to turn off the default click indication and have it go straight to my pre defined pop up so the user does not need to click okay to get to the info.
Here's the link to the page (using highslide for the pop ups) http://www.summitpacific.ca/about/campusmap.php
Rob McIntyre
Summit Pacific College

Hi
Sorry - I found the solution:
Just delete the alert string  - alert($(this).attr('alt') + ' clicked')
Rob McIntyre
Summit Pacific College


add missing ItemTableThink

Update to add missing ItemTableThink via AddThinkToEnt after the item is created through Customweapons.GiveItem
Also comments out ExtraItems so it can be used as intended (on outside scripts)

This should be taken care of elsewhere (popext_main) but I'll merge it for now anyway just so it works.


Submit blobfishes project entry to DO hackathon landing page

As e documented in the DigitalOcean Hackathon Official Rules :

Timing: The Hackathon will open at 9:00am PST on June 29, 2021. All submissions must be received by 10:00pm on July 20, 2021.

and

How to Enter: Eligible entrants can enter the Contest by submitting an entry at the Hackathon’s submission form found on the Hackathon’s landing page. As a participant, your entry must fully meet all Contest requirements, as specified in these Official Rules, in order to be eligible to receive the submission prizes specified below.

IMPORTANT: The deadline for submission is 10:00pm PST on July 20, 2021.
According to https://savvytime.com/converter/pst-to-cest/jul-20-2021/10pm this is 2021-07-21 07:00 CEST

Updated https://github.com/aquariophilie/blobfishes/blob/main/docs/deploy-blobfishes-to-digitalocean.md
Reopening issue until the blobfishes project entry has been submitted to DO.

Form submitted on 2021-07-21 05:27 CEST

DigitalOcean MongoDB Hackathon Submission
We got it! Great work and we'll contact you with next steps on shirt and stickers.
Edit your response
Submit another response


fix issue and  add multimodel prisma schema and tests

This PR closes #396
Problem
When the generator processes the Prisma namespace module, it iterates over all statements (type declarations) within the module. If an error occurs while processing a statement, the existing error handling logic uses a return statement inside the catch block. This causes the function to exit and stops further processing of the remaining statements. As a result, only the first model is processed correctly, and subsequent models are ignored.
Solution
To fix this issue, we need to modify the error handling in the handlePrismaModule function. Instead of using a return statement inside the catch block, we should log the error and use a continue statement to proceed to the next iteration of the loop. This ensures that all statements are processed, even if an error occurs with one of them.
A new multimodel test was added to ensure the generator is able to correctly type multiple models and it won't stop after the first one

I'm still trying to understand why this solves the problem since no errors should've been thrown in the first place.

I just pushed your test into a new branch without the changes you made and the tests passed, I don't think there's a problem with this plugin and I could not reproduce it locally in your branch as well.


Open Enclave memory fragmentation

An OE application is seeing considerably more memory usage than when using dlmalloc (the OE default).  The memory appears to grow quickly and causes crashes.
Here are some properties of the application

64MiB enclave size
16 worker threads + a few other threads
Heavy ECall usage so threads are rapidly entering and exiting the enclave.

The current conjecture is that the per thread caching of reservations and fragmentation in them is causing too much memory usage.  Although, it looks like it could be a memory leak that is unlikely.
Mitigations
To try to diagnose the issue, we have

Created detailed logging of per size class usage. (PR still required to integrate with main)
Used 4K chunk size to reduce the possible fragmentation
Disabled the per thread buddy allocators for large allocations
Reduced maximum size for use in thread local free lists to chunk size. (There were a small number per thread of 8KiB, 16KiB and 32KiB buffers that were causing long free lists to be constructed.

Chunks 4 KiB, no per thread buddy allocator.
With this design of snmalloc we have got the system to a steady state:

I left this running for about the same length of time ~4000 at the max did not move above the max on the graph.
At one of the higher points, we observe:

Requests for memory totalling: 25336960
Reservations of memory totalling: 31961088

This gives a 21% fragmentation due to chunks of memory not being fully used.
At one of the lower points, we observe

Requests for memory totalling: 15862784
Reservations of memory totalling: 22618112

This gives a 30% fragmentation.
Chunks 16 KiB, no per thread buddy allocator.
If we move to 16KiB chunks, then we see a higher memory usage due to increased fragmentation, but the overall pattern is almost identical:

Chunks 4 KiB, enable per thread buddy allocator.
This enables the per thread buddy allocator but with a maximum size of 256KiB instead of the 2MiB that is default in snmalloc.  It is also using the 4 KiB chunks:

Here you can see that the memory appears to still be growing even after 1000 seconds.  I believe this is due to the random nature in which the main worker threads, and the other threads are assigned allocators.  The thread local caches only return memory in larger chunks, so this leads to more fragmentation.  In larger memory scenarios this is not a problem, but for a 64MiB enclave with over 16 threads holding onto a couple of MiB per thread is a large percentage of the memory.  There is approximately 25% overhead introduced by this per thread reservations, this is in addition to the 20% fragmentation from underused slabs.
I ran the benchmark for a weekend and this is the plot of the maximum memory:

The system takes a very long time to stabilise the growth in memory usage, but after around 8 hours seems to have most reached almost the maximum, and then after another 8 hours seems to stop growing.  Though, leaving it another half day, and I observed an additional 256KiB growth in the max usage.
Short term solution

[x] Build new configuration that disables thread local buddy allocators when the heap is small (#617)
[x] Confirm with team that this addresses memory growth in production
[x] Build new release of snmalloc with these changes
[x] Integrate changes into OE
[ ] Run performance testing on OE partners work loads to see perf effects of changes.

Alternative solutions
Disabling the per thread buddy allocators is going to harm performance in highly contended scenarios.  An alternative solution is to build a concurrent buddy allocator that does not use a global lock to protect access.  This is substantial work and further investigations should be made to understand the performance.

In #617, we have an approach to disabling thread local buddy allocators if at startup we detect a small heap size.
I reran the experiment with #617 integrated into OE, and got.

This looks very promising for the next release for Open Enclave.

Just tracking the maximum on this run.
Duration   Maximum
<PHONE_NUMBER>0
<PHONE_NUMBER>2
    2088  36847616
    5192  37093376
    6946  37142528
    1574  37470208

The left column is how long in seconds the right hand column was the maximum for.
The final entry is the current maximum and has been for the last 1574 seconds.
As you can see there is a slight rise in the maximum, but this is expected the longer you run the system for more likely bad cases will occur.

This is now in the latest version of Open Enclave.


truffle deployment Error.

Deploying TokenUse...
  ... 0xddf7fb1c62f9b2ea8406955c6668ad331f20bd74f70d1df9895af3f7214c9109
Error encountered, bailing. Network state unknown. Review successful transactions manually.
Error: The contract code couldn't be stored, please check your gas amount.

Ah... it think TokenUse is also an abstract contract.
Truffle deploy didn't get the reason.


Backup / Restore

Background
There might be various reasons as to why people need to reinstall the wallet. Each time they reinstall the wallet, they also need to reacquire their credentials. This has observed to be a common occurrence. For example, people often experience issues with the wallet where reinstalling it is the only remedy or they have purchased a new phone (on average, every 2-3 years).
People are beginning to feel frustrated at the idea of reinstalling and reacquiring their credentials to resolve their issue.
"Is there a reason this happened? How do I prevent this from occurring again? My PIN was correct yet it was locking me out. There must be better remedies than counsel needing to delete the App and reloading their credentials every time. " - a BC Wallet user who had to reinstall and reacquire their credentials after experiencing an unknown error.
Situations where the issue is an error, there needs to be better ways to acquire more information on the issue to better investigate, identify and resolve the issue.
Acceptance Criteria
Wireframes/Design

@jeznorth considering the issues we're having with PIN errors, I would like to resurface this ticket.


[API] [Bug] get 500 on Dashborard when delete org

tested on current master: 29151b90c62d4f997a5503c3e0d42468f404a73c

Create Org
Delete Org via API

-> you get  routers/user/home.go:57:getDashboardContextUser() [E] GetOrganizations: user does not exist [uid: 3, name: , keyid: 0]

Found Reason: AdminDeleteUSER can delete user of Org and leave a Org without user behind :( !!


npm install @google-cloud/speech fails

When installing via NPM or Yarn I'm getting:
WARN notice [SECURITY] protobufjs has the following vulnerability: 1 moderate. Go here
for more details: https://nodesecurity.io/advisories?search=protobufjs&version=5.0.3 -
Run `npm i npm@latest -g` to upgrade your npm version, and then `npm audit` to get more info.
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/merge2-be1dfff2/package.json'
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/merge2-be1dfff2/index.js'
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/merge2-be1dfff2/index.mjs'
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/merge2-be1dfff2/LICENSE'
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/merge2-be1dfff2/README.md'
WARN tar ENOENT: no such file or directory, open '/usr/local/lib/node_modules/.staging/grpc-167d0fd9/deps/grpc/src/boringssl/err_data.c'
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/@google-cloud/speech/node_modules/ecc-jsbn):
npm WARN enoent SKIPPING OPTIONAL DEPENDENCY: ENOENT: Cannot cd into '/usr/local/lib/node_modules/.staging/ecc-jsbn-13c71c3d'

npm ERR! code E404
npm ERR! 404 Not Found: protobufjs@https://registry.npmjs.org/protobufjs/-/protobufjs-6.8.6.tgz`

I'm able to replicate this on multiple machines and environments.
Here's a sample replications of the issue - https://codesandbox.io/s/9lwjzolw3w

Meep.  I can reproduce this:
beckwith-macbookpro2:speeeeech beckwith$ npm install @google-cloud/speech
WARN tar ENOENT: no such file or directory, open '/Users/beckwith/Code/speeeeech/node_modules/.staging/yargs-c7f67d2c/lib/tokenize-arg-string.js'
WARN tar ENOENT: no such file or directory, open '/Users/beckwith/Code/speeeeech/node_modules/.staging/yargs-c7f67d2c/lib/usage.js'
WARN tar ENOENT: no such file or directory, open '/Users/beckwith/Code/speeeeech/node_modules/.staging/bytebuffer-a6d9511b/dist/bytebuffer.min.js'
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

npm ERR! code E404
npm ERR! 404 Not Found<EMAIL_ADDRESS>

@alexander-fenster I'm running out of people to assign P0 issues to 😆 Do you think you could take a look tomorrow?

I am unable to reproduce the problem, it installed fine on my machine.
$  node -v
v6.11.1

$ npm -v
3.10.10

My understanding is that it was caused by some temporary npm outage. We sometimes have some transient failures in our CI usually caused by npm but overall it's mostly green for speech. https://circleci.com/gh/googleapis/workflows/nodejs-speech/tree/master
Let me close this issue for now. @cconcise please feel free to reopen / comment if you still experience the install issue.

Just confirming that installing the package works fine now and it was in fact caused by an NPM outage


The general package

This looks like a really interesting package. I have a similar methodology for air emissions in a package carbonr. The use of the latitude and longitude is a great addition that you have included! I notice you haven't updated the emissions since the 2019 report. I was wondering if this is still being actively used? Or, if you were interested in any level of collaboration?
Would be great to hear from you.
Thanks.

Hi. I'll take a look at your package.
I have been meaning to update the GHG emissions and include an argument for year, but have not gotten the time to do so. So, for now, everything is based on 2019.
I'd be interested in collaboration. Contact me via email and we can discuss.


Attempt to fix release-master not including js/css

My assumption currently is that drone pipeline steps may not share files so files generated in the previous 'build' tasks may have been discarded for the previous 'static' task.

I think this can only really be tested once it has landed and the release-master pipeline runs with it.

No files are shared in same pipeline

Codecov Report

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

@@            Coverage Diff             @@
##           master    #9316      +/-   ##
==========================================
+ Coverage   41.18%   41.19%   +<.01%
==========================================
  Files         559      559
  Lines       73115    73115
==========================================
+ Hits        30114    30120       +6
+ Misses      39225    39218       -7
- Partials     3776     3777       +1

Impacted Files
Coverage Δ

routers/repo/view.go
39.47% <0%> (+0.87%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
: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 50da9f7...b08f3f8. Read the comment docs.

@lafriks Your're probably right. Drone documentation says files should persist between steps. I'm a bit out of ideas. release-master steps do run fine:
https://drone.gitea.io/go-gitea/gitea/17228/3/1
https://drone.gitea.io/go-gitea/gitea/17228/3/3
Still, the release binaries at https://dl.gitea.io/gitea/master/ seem to redirect to /install instead of serving js/css files. Maybe @techknowlogick has a better idea.

Probably not the right solution


Any way to assign hotkeys to switch between atom and the console within the IDE?

Not a priority I know, but would be a little UX tweak that could add up in terms of efficiency. :)

Hey @meelos! This exists with the current release, but only if you are on a mac (cmd + ; will do it).
In 2.5.0, which is currently in beta but will be released within the next week, this shortcut is extended to windows & linux (ctrl + ; for these platforms).
There are a couple other shortcuts we've added (toggling the terminal's visibility, and toggling the popout terminal) and you can see them all listed out on the package's settings page (on mac: Learn IDE > Preferences > Packages > Learn IDE, or on Windows & Linux File > Settings > Packages > Learn IDE).
Let us know if there is anything else we can add that would be helpful!


tests: create PSA compliant pod in renderContainerSpec()

This fixes the test "should successfully update a VMI's CPU set on migration" when run in a cluster with PSA enabled.
Signed-off-by: Antonio Cardace<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @fossedihelm @xpivarc

/retest-required

/retest-required

/retest-required

dropping --no-overwrite-dir seems to solve it (and should be possible to drop the privileged pod too)
but still not sure exactly what changed. Maybe it was my PSaification of exporter pod

@akalenyu I've tried you're suggestions but the storage lane still fails because of PSA, I suggest we merge this PR and then all the optimizations for storage can be addressed in a follow-up PR.
I think it's better to proceed with this as it unblocks testing on clusters with PSA enabled.

@akalenyu I've tried you're suggestions but the storage lane still fails because of PSA, I suggest we merge this PR and then all the optimizations for storage can be addressed in a follow-up PR.
I think it's better to proceed with this as it unblocks testing on clusters with PSA enabled.

sure np

/lgtm
/retest

/cherrypick release-0.58


No License

I noticed that despite being listed as available under an MIT license on its npm page (https://www.npmjs.com/package/react-tooltip-lite) there is no license file in the repository. I would like to use this library in a commercial project but I can only do so if everything is in order legally.
Would you please add a license file? Or if you prefer I can make a pull request to add it. I just need to know the name of the copyright holder so I can add it to the license file.
Thank you for your time.

Hey @Meeinotl I've added the MIT license it should show up no in v1.9.4

Thank you @bsidelinger912 for adding this so quickly.


How to listen to URL changes inside the iframe and prevent URL from redirecting!

what was the soln here?

I have a use case where I want to check if it redirects to a url on our own webpage (so it does not break the security rules) but I don't see a way to pass a ref in here

@maxwhoppa  I didn't use this one and  I use HTML iframe as I didn't find anything special to use it. besides that it's hard to listen to URL changes due to browser security especially when iframe content is external, so I couldn't listen to URL changes. prevented the URL redirect with some state to hide the iframe once the redirect happens. hope that clear


Isolated text align button active class names fix #589

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[x] Docs have been added/updated (for bug fixes/features)
[x] It does not introduce a breaking change or has description for the breaking change

Description

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

@seytar
Thank you for your second contribution.
The classname is just prefixed, so I'll merge it without further review.
Thank you again.


pyproject: Use hatchling as build system

Well this works, but I would like the confirmation from the maturin team
https://github.com/PyO3/maturin/discussions/1714

Already merged required changes: https://github.com/pypa/hatch/pull/1018
Waiting a new release from hatch.

@patrickelectric
https://github.com/pypa/hatch/releases/tag/hatchling-v1.19.0

@RaulTrombin thanks!


Update Composer dependencies (2022-03-02-00-09)

Visual regression test passed!
View the visual regression test report