Calico 3.24.6 and 3.25.1 POD Networking Issue

I am using kubernetes 1.25.9 with calico 3.24.6 on fedora coreOS 38. I have 1x master and 2x worker nodes. i have deployed 2x busybox containers on node0 and node1. The busybox container running on node0 unable to ping node1 busybox container. The ping starts working when I initiate a ping from node1 busybox container.
I have checked the node status via calicoctl. All BGP is up.
I have used manifest https://raw.githubusercontent.com/projectcalico/calico/v3.24.6/manifests/calico.yaml
Expected Behavior
The ping should be working all the time from both direction all the time.
Current Behavior
The busybox container running on node0 unable to ping node1 busybox container. The ping starts working when I initiate a ping from node1 busybox container. All BGP is up, checked via calicoctl
Possible Solution
not sure.
Steps to Reproduce (for bugs)

deploy k8s 1.25.9 with calico 3.24.6 on fcos 38 with 1 master and 2 workers
deploy 2x busybox containers one on node0 and one on node1
ping from busybox1 to busybox2 will not work
Initiate ping from busybox2 to busybox1, ping will work
then initiate ping from busybox1 to busybox2 it will start working to.
the traffic will stop working again when there is no activity between busybox containers

Context
Planning an upgrade of kubernetes and calico to new release.
Your Environment
FCOS 38
kubernetes 1.25.9
calico 3.24.6

Hi @syedammad83, can you please share more information about your setup? Are you running on bare metal or on managed instance. Also is there a firewall that is running on those nodes, or did you set up any policies?

Closing due to inactivity. Feel free to re-open if you still need support.

Any update on resolution of above issue. I am facing the same issue and this is really frustrating now. Unable to resolve since 3 days.
OS: RHEL 9.3
Calico v3.25.2
Kubernetes v1.28.13
On describing calico it says something like "dial unix: /var/run/calico/bird.ctl connect: connection refused". This is the only error I can see other than that everything is established including BGP peering.

@TahaYousufAli52 please open another GH issue and explain your setup in more details. Or you can join the slack channel and ask your questions there: https://slack.projectcalico.org/ .


Missing published @jest/test-utils npm package

🐛 Bug Report
Trying to upgrade my jest plugin to support jest v27, then find out @jest/test-utils v27.0.2 is not available on npm registry
To Reproduce
Just try npm i<EMAIL_ADDRESS>> npm i<EMAIL_ADDRESS>npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.

npm ERR!      /Users/huocp/.npm/_logs/2021-05-31T06_23_48_633Z-debug.log

The npmjs.com shows the only version published is 0.0.0??
https://www.npmjs.com/package/@jest/test-utils

Expected behavior
Expect @jest/test-utils npm package available.
Link to repl or repo (highly encouraged)
envinfo

this package should not be published at all, not sure why it is... But anyways, it's an internal util that's not published and should not be used outside of this repo

I was trying to mimic some unit test code here. Do I suppose to copy those code from test-utils to my code base?

Yep, this module does not have any public APIs we want to maintain


runtime: Move String implementation stubs that want need the auto-rel…

…eased return value optimization to an ARC compiled file
String's hashValue function is implemented in terms of Foundation's hash
function in a runtime function on darwin platforms. For non-ASCII strings we
will call str.decomposedStringWithCanonicalMapping inside this runtime function
which will allocate a new NSString and return the result in the current
autorelease pool. We implemented this function in a file compiled without ARC.
This meant that we would leak said NSString into the current active autorelease
pool.
This patch moves the implementation to a file compiled with ARC. ARC will insert
objc_retainAutoreleasedReturnValue call and on platforms that require it an
marker for the hand-off of the autoreleased return value optimization.
SR-4889
rdar://32199117

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - ec5f40f12f2304a87b8d6f50040ed7485bf552fb

Build failed
Swift Test OS X Platform
Git Sha - ec5f40f12f2304a87b8d6f50040ed7485bf552fb

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - ec5f40f12f2304a87b8d6f50040ed7485bf552fb

Build failed
Swift Test OS X Platform
Git Sha - ec5f40f12f2304a87b8d6f50040ed7485bf552fb

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 75a82d7a7ceefba2ccd6ea8aa2ffda3a0de2cc09

Build failed
Swift Test OS X Platform
Git Sha - 75a82d7a7ceefba2ccd6ea8aa2ffda3a0de2cc09

@aschwaighofer Please be careful about using the word leak and autorelease pool together. Hitting the autorelease pool is not a "leak". I would say "unnecessary memory growth" or something like that. Leak is misleading.
Or am I understanding the pull request, wrong?

@swift-ci Please clean test linux

@swift-ci Please clean test

Build failed
Swift Test Linux Platform
Git Sha - f691657f3e11baeb491624c119e39830144cdac7

Build failed
Swift Test OS X Platform
Git Sha - f691657f3e11baeb491624c119e39830144cdac7


Add support for a JsonAnyPropertyAttribute

This would do the same thing as the XmlAnyElementAttribute for XML serialization, i.e. act as a "catch all" for JSON properties that have no corresponding property.
Example:
{
    "Id": 42,
    "Name": "test",
    "Description": "foo",
    "Category": "bar"
}

public class MyClass
{
    public int Id { get; set; }
    public string Name { get; set; }

    [JsonAnyProperty]
    public IDictionary<string, object> OtherProperties { get; set; }
}

OtherProperties will contain the values of Description and Category, because the have no corresponding property in the class.
This would allow us to map only properties we care about, but without losing the other ones when we serialize back to JSON.

That already exists: JsonExtensionDataAttribute

Oh, right. I thought I remembered seeing this somewhere, but when I looked for it I couldn't find it. I think it should be mentioned on the Serialization attributes page in the documentation, it would make it easier to find.

Good point. I added document with a sample for it, but didn't add it to that page.

Documentation++

Perfect :+1:


wire-desktop: 3.3.2872 -> 3.5.2881

Motivation for this change
Version bump.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@jollheef Thanks :+1: I totally forgot to update this recently


findbar (ctrl f) not properly styled.

If the default dark theme is disabled in favour of an image theme (installed from addons), findbar is not styled.

Hmm, interesting.  This repo has been designed with the intent to supplement the dark theme as opposed to entirely re-create one.  I imagine if you disable the dark theme, there will be a few other quirks you encounter. If you want to make it dark, you could add something like the following to youruserChrome_customization.css file:
findbar {
  background-color: var(--in-content-category-header-background) !important;
}

.browserContainer > findbar .findbar-find-status,
.browserContainer > findbar .found-matches,
.browserContainer > findbar .findbar-button {
    color: var(--in-content-page-color) !important;
}

.findbar-closebutton.close-icon {
  fill: var(--in-content-page-color) !important;
}

If you want to keep it light, use something to the effect of:
.findbar-textbox:not([status=notfound]) {
  background-color: -moz-Field !important;
  color: -moz-FieldText !important;
  border-color: rgba(0,0,0,.35) !important;
}

.findbar-find-next,
.findbar-find-previous {
  fill: black !important
}

Ah, I see. I tried both and re-ran the windows updater script however nothing seemed to change. Unfortunately, I am not a developer and currently my knowlege of css is very limited so I don't know where to begin in terms of what to change, however thanks for being so helpful as to reply.

After re-running the updater script, did you restart Firefox?  Unfortunately, that has to be done in order for changes to go into effect.
This issue does bring up an interesting point though, and one I'll have to give a little more thought to.  With the default dark theme expanding in FF Nightly, I was planning on removing the corresponding code in this repo (default styling by FF will be more stable, less work for me, etc.)...but I hadn't given consideration to this use case.

I would assume my use case is fairly rare, I was actually reccomended shadowfox when I enquired on the mozilla help forums:
https://support.mozilla.org/en-US/questions/1213404
Honestly, I don't think it should be your problem to solve and rather one that could be adressed in a future browser update. There is a hint that this might be the case on the FAQ page
https://support.mozilla.org/en-US/kb/frequently-asked-questions-firefox-addon#w_what-happened-to-my-complete-theme:

What happened to my complete theme?
Complete themes do not work with Firefox 57 and are disabled. If you use a dark complete theme, the Dark Theme option in the Add-on Manager might be a good option for you. Go to the Themes tab in about:addons and give it a try. We are making improvements to themes in the coming months, so look for themes with additional functionality beyond an image overlay in future releases.

In the meanwhile, if you do plan on refactor your code for the newer browser versions, so long as I keep a copy of the current userContent.css and userChrome.css I should be fine until I find a better solution!

Well I'm glad you stumbled across the project!

so long as I keep a copy of the current userContent.css and userChrome.css I should be fine until I find a better solution

I have no definitive plans as to what I'm going to do yet (I'm waiting to see what a more stable version of the default dark theme is going to look like - there's still quite a few changes to, for example, color shades in Nightly for me to make any big decisions), but I will certainly keep the current versions available in the repository through some means.
FF Nightly is on v61, and the next FF ESR is v60.  As such, the improvements to the dark theme won't land in the next ESR cycle (unless they uplift the changes).  I would like to accommodate any ESR/TOR/FF forks, so I won't be removing it all together :)


How to filter some atom changes?

How to filter some atom changes?
I think, withEquality is deprecated: not used in next version of derivablejs.
My bicycle. But this is mokey-patching of atom.set method:

import shallowEqual from 'shallowequal'

function wrapAtomSet<V>(value: Atom<V>, isEqual: (old: V, new: V) => boolean) {
    const oldSet: (value: V) => void = value.set
    return function set(v: V): void {
        if (!isEqual(value.get(), v)) {
            oldSet.call(value, v)
        }
    }
}

const value = atom({a: 123})
value.set = wrapAtomSet(value, shallowEqual)
value.set({a: 123}) // does not really call set method

AFAIK withEquality is deprecated only at module level. But on any single atom/derivable it can be used safely.

withEquality is deprecated only at module level. But on any single atom/derivable it can be used safely.

True. You can still do, e.g.
const value = atom({a: 123}).withEquality(shallowEqual);


Renamed custom action method

This prevents the custom action being present in the input text view menu. Another case of "Singletons shouldn't be the first choice" - this application wide shared menu controller is just horrible :-(
@nikkoss the issue you mentioned in #811 arose because the customAction: selector is registered at the UIMenuController and DemoMessageViewController did implement it. When the context menu of the text input is shown, the menu controller checks if the underlying object(s? - I guess the views, but for sure the view controller) responds to the selector. In case of customAction: it did, so the action was added.
I see two ways around this: 1. proxy to a different selector (see this PR) with the downside of Undeclared selector warnings. 2. Let another object than the view controller be the collection view delegate. This would require some major code restructuring.

thanks @sebastianludwig !

Hey @sebastianludwig - After thinking some more, I think (2) is actually the better solution, which will eventually be resolved by #453. Having the undeclared selector warnings in the demo doesn't feel right.
Going to close this.


Style helpers improvements

Plan
By helper and geometry type review:

Size
Stroke width
Stroke color
Default palette

After that, a new PR will come checking every helper and case:

create examples (there are some of them already) and validate them
add more parameters validations

Acceptance checklist

[ ] basic

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] colorBins

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] colorCategories

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] colorContinuous

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] sizeBins

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] sizeCategories

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

[ ] sizeContinuous

[ ] size
[ ] strokeWidth
[ ] strokeColor
[ ] default palette

Nice work on this. Debug examples you have prepared are being really useful for validation.
I made a first review of all the debug examples, and everything looks fine except the size helpers. I still see 'jumps' and wrong pixel sizes (sizeBins, sizeCategories and sizeContinuous). Logic for adjusting based on viewport is not yet working fine
The rest of tests I made look fine: default palettes, override style options...

I still see 'jumps'

In the basic helper, as all the points have the same size, I changed it to use pointRadiusMinPixels  and pointRadiusMaxPixels with the same value: the expected point size. That way, size does not change over zoom (it can not change)
I tried to follow a smilar idea with this cases, but it is not possible. All the stuff we can to to force size, it does not depends on features, but are general to the layer. The only way I can see is to change deck.gl to support pixels width in points.

wrong pixel sizes
I am checking it.

As we said, let's focus on wrong side (a-must), and forget a bit about 'jumps' (nice-to-have).

After playing with the @jesusbotella deck example, I see the pointRadiusScale does not affect to pointRadiusMinPixels and pointRadiusMinPixels. So:

we should not use together or
we should apply the pointRadiusScale to pointRadiusMinPixels and pointRadiusMinPixels

I a going to follow the second one if possible.

Needs to fix conflicts after merge interactivity


Fix typescript type with ESM package

I fix the typescript type when using the ESM project.
Change:

Add the type file to the exports field in the package.json
Add file extension in *.d.ts when relative import.
Remove PropType in the Vue component prop with .d.ts.

When exist PropType, will get Type  'string' is not assignable to type 'BooleanConstructor | StringConstructor | undefined' error.

wrong

correct

Fixed #4055
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
Always follow the contribution guidelines when submitting a pull request.

Add file extension in *.d.ts when relative import.

Why is this required? As far as i am aware it is not allowed to import from .js in .d.ts

Add file extension in *.d.ts when relative import.

Why is this required? As far as i am aware it is not allowed to import from .js in .d.ts

Ref: https://www.typescriptlang.org/docs/handbook/esm-node.html
This code works in CommonJS modules, but will fail in ES modules because relative import paths need to use extensions. As a result, it will have to be rewritten to use the extension of the output of foo.ts - so bar.ts will instead have to import from ./foo.js.
// ./bar.ts
import { helper } from "./foo.js"; // works in ESM & CJS
helper();
This might feel a bit cumbersome at first, but TypeScript tooling like auto-imports and path completion will typically just do this for you.
One other thing to mention is the fact that this applies to .d.ts files too. When TypeScript finds a .d.ts file in package, whether it is treated as an ESM or CommonJS file is based on the containing package.

I also test tsc and it will generate .js in .d.ts.
 yoyo930021@yoyoIUdeMacBook-Pro-2: cat package.json
{
  "name": "my-app",
  "private": true,
  "version": "1.0.0",
  "description": "My App",
  "repository": "",
  "license": "UNLICENSED",
  "type": "module",
  "scripts": {
    "start": "npm run dev",
    "dev": "cross-env NODE_ENV=development vite",
    "build": "cross-env NODE_ENV=production vite build"
  },
  "browserslist": [
    "IOS >= 13",
    "Safari >= 13",
    "last 5 Chrome versions",
    "last 5 Firefox versions",
    "Samsung >= 12"
  ],
  "dependencies": {
    "dom7": "^4.0.4",
    "framework7": "^7.0.5",
    "framework7-vue": "^7.0.5",
    "skeleton-elements": "^4.0.0",
    "swiper": "^8.2.6",
    "vue": "^3.2.37"
  },
  "devDependencies": {
    "@vitejs/plugin-vue": "^2.3.3",
    "@vue/compiler-sfc": "^3.2.37",
    "cross-env": "^7.0.3",
    "postcss-preset-env": "^7.7.2",
    "sass": "^1.53.0",
    "typescript": "^4.7.4",
    "vite": "^2.9.13"
  }
}
yoyo930021@yoyoIUdeMacBook-Pro-2:  cat tsconfig.json
{
  "compilerOptions": {
    "target": "esnext",
    "module": "esnext",
    "moduleResolution": "node16",
    "strict": true,
    "jsx": "preserve",
    "sourceMap": true,
    "resolveJsonModule": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "lib": ["esnext", "dom", "WebWorker"],
    "skipLibCheck": true,
    "declaration": true,
    "outDir": "./dist",
    "baseUrl": ".",
    "paths": {
      "@/*": ["src/*"],
      // "framework7/lite": ["node_modules/framework7/"],
      // "framework7-vue/bundle": ["node_modules/framework7-vue/"],
      // "framework7/components/login-screen": ["node_modules/framework7/components/login-screen/login-screen"]
    },
    "useDefineForClassFields": false
  },
  "include": ["src/**/*.ts", "src/**/*.d.ts", "src/**/*.tsx", "src/**/*.vue"]
}                                                                                /0.0s
 yoyo930021@yoyoIUdeMacBook-Pro-2: cat src/lib.ts
export const a = 0                                                                                /0.0s
 yoyo930021@yoyoIUdeMacBook-Pro-2: cat src/test.ts
import { a } from './lib.js'

console.log(a)

export default a                                                                                /0.0s
 yoyo930021@yoyoIUdeMacBook-Pro-2: tsc -p tsconfig.json
zsh: command not found: tsc                                                                                /0.0s
 ✘ yoyo930021@yoyoIUdeMacBook-Pro-2: yarn tsc -p tsconfig.json
yarn run v1.22.18
$ /Users/yoyo930021/universetech/test-f7/node_modules/.bin/tsc -p tsconfig.json
src/main.ts:25:17 - error TS2307: Cannot find module './app.vue' or its corresponding type declarations.

25 import App from './app.vue';
                   ~~~~~~~~~~~

src/routes.ts:1:22 - error TS2307: Cannot find module './views/home.vue' or its corresponding type declarations.

1 import HomePage from './views/home.vue';
                       ~~~~~~~~~~~~~~~~~~

src/routes.ts:2:26 - error TS2307: Cannot find module './views/404.vue' or its corresponding type declarations.

2 import NotFoundPage from './views/404.vue'
                           ~~~~~~~~~~~~~~~~~

src/routes.ts:3:23 - error TS2307: Cannot find module './views/LoginView.vue' or its corresponding type declarations.

3 import LoginView from './views/LoginView.vue'
                        ~~~~~~~~~~~~~~~~~~~~~~~

Found 4 errors in 2 files.

Errors  Files
     1  src/main.ts:25
     3  src/routes.ts:1
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.                                                                                /1.3s
 ✘ yoyo930021@yoyoIUdeMacBook-Pro-2: cat dist/test.d.ts
import { a } from './lib.js';
export default a;                                                                                /0.0s

<EMAIL_ADDRESS>


runtime:compiler:pandoc: escape quotes in &errorformat

as discussed at https://github.com/vim/vim/pull/15529

thanks


onboarding: add FS events to connect dataset modal

re: https://github.com/CartoDB/cartodb/issues/12912

Event in the lock icon right next to the "Connect dataset" button

$('.fs-PrivacyToggler-connectDataset')

Event in the "Connect" button of Google Drive

$('.fs-connect-gdrive')

Event in the "Connect" button of Dropbox

$('.fs-connect-dropbox')

Event in the "Connect" button of Box

$('.fs-connect-box')
take into account privacy lock selector only appears if not disable
this can be tested in ded11

Tested in staging, all classes are where they should, moving to deploy!

Thanks guys!


integration tests for pub/sub sample

Something is wrong with your changes:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:testCompile (default-testCompile) on project spring-cloud-gcp-integration-pubsub-sample-test: Compilation failure: Compilation failure:
[ERROR] /home/travis/build/spring-cloud/spring-cloud-gcp/spring-cloud-gcp-samples/spring-cloud-gcp-integration-pubsub-sample/spring-cloud-gcp-integration-pubsub-sample-test/src/test/java/com/example/SampleAppIntegrationTest.java:[76,80] cannot find symbol
[ERROR] symbol:   class SenderApplication
[ERROR] location: class com.example.SampleAppIntegrationTest
[ERROR] /home/travis/build/spring-cloud/spring-cloud-gcp/spring-cloud-gcp-samples/spring-cloud-gcp-integration-pubsub-sample/spring-cloud-gcp-integration-pubsub-sample-test/src/test/java/com/example/SampleAppIntegrationTest.java:[80,82] cannot find symbol
[ERROR] symbol:   class ReceiverApplication
[ERROR] location: class com.example.SampleAppIntegrationTest

See Travis report.


Add tvOS Remote Tap Gestures

Motivation
On tvOS there is a gesture for tapping a region of the touch pad on the remote. For example, the common use for the left/right tap gesture is to skip -/+ 10s during playback. Currently this gesture is recognized but reported as a generic "left"/"right"/"up"/"down" event, which is the same event type used for swipes. This PR is to add a "tap" event that is separate from the "swipe" event.
Test Plan
Using RNTester's TVEventHandlerExample, the tap<Left/Right/Top/Bottom> event type should be displayed when the edge of the remote trackpad are tapped, and swipe<Left/Right/Up/Down> should be displayed after a swipe.
Related PRs
None
Release Notes
[IOS][ENHANCEMENT][RCTTVRemoteHandler] - Change tvOS Gesture Recognizers to Accept Taps

So there's two minor things:

Currently this is a minor breaking change if you are using the left/right/up/down` events, because those events are broken into tap and swipe events. If keeping the old behavior is desired, the old events could be additionally emitted, but that seems suspect given that the old behavior didn't really make sense in the first place.

I've gone with "top" and "bottom" for the tap location names, which differs from Apple's usage of "up" and "down". I think this makes more sense spatially, but it might be confusing if you're aware of Apple's naming.

cc @dlowder-salesforce

@ultramiraculous this looks fine, but should be marked as BREAKING in the release notes.

Sounds good to me @dlowder-salesforce. I looks like I add an entry to the wiki after the change lands?

@ultramiraculous no, you edit the "Release Notes" section of your first comment above, and add "[BREAKING]".


Update TestRunCompleteExample in other_test.go

Description
Adds the capability to create admin signature keys in complete test (TestRunCompleteExample).
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

This PR updates test in other_test.go, that will run in continuous test pipeline.
Ran the test locally, and it passed. Details are attached here.
make run-tests-local RUN=TestRunCompleteExample
cd tests && go test -run TestRunCompleteExample -count=1 -v -timeout 300m
=== RUN   TestRunCompleteExample
=== PAUSE TestRunCompleteExample
=== CONT  TestRunCompleteExample
...
...
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66: Plan: 2 to add, 0 to change, 0 to destroy.
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66: Changes to Outputs:
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:   + hpcs_instance_details = {
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:       + crn       = (known after apply)
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:       + endpoints = (known after apply)
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:       + guid      = (known after apply)
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:       + hpcs_name = "example-hpcs-2eilg3-instance"
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:       + id        = (known after apply)
TestRunCompleteExample 2024-03-28T10:23:14+05:30 logger.go:66:     }

...
...
TestRunCompleteExample 2024-03-28T10:28:21+05:30 logger.go:66: module.ibm_hpcs.ibm_hpcs.hpcs_instance[0]: Still creating... [5m0s elapsed]
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66: module.ibm_hpcs.ibm_hpcs.hpcs_instance[0]: Creation complete after 5m6s [id=crn:v1:bluemix:public:hs-crypto:us-south:a/abac0df06b644a9cabc6e44f55b3880e:f6aaf174-e1da-444b-b075-e93e1c1e9fa6::]
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66: Apply complete! Resources: 2 added, 0 changed, 0 destroyed.
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66: Outputs:
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66: hpcs_instance_details = {
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   "crn" = "crn:v1:bluemix:public:hs-crypto:us-south:a/abac0df06b644a9cabc6e44f55b3880e:f6aaf174-e1da-444b-b075-e93e1c1e9fa6::"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   "endpoints" = tomap({
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:     "allowed_network" = "public-and-private"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:     "endpoints.private" = "https://api.private.us-south.hs-crypto.cloud.ibm.com:8725"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:     "endpoints.privateGrep11" = "https://ep11.private.us-south.hs-crypto.cloud.ibm.com:8725"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:     "endpoints.public" = "https://api.us-south.hs-crypto.cloud.ibm.com:8725"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:     "endpoints.publicGrep11" = "https://ep11.us-south.hs-crypto.cloud.ibm.com:8725"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   })
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   "guid" = "f6aaf174-e1da-444b-b075-e93e1c1e9fa6"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   "hpcs_name" = "example-hpcs-2eilg3-instance"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66:   "id" = "crn:v1:bluemix:public:hs-crypto:us-south:a/abac0df06b644a9cabc6e44f55b3880e:f6aaf174-e1da-444b-b075-e93e1c1e9fa6::"
TestRunCompleteExample 2024-03-28T10:28:28+05:30 logger.go:66: }
TestRunCompleteExample 2024-03-28T10:28:28+05:30 tests.go:828: FINISHED: Init / Apply

..
...
module.ibm_hpcs.ibm_hpcs.hpcs_instance[0]: Destruction complete after 2m52s
module.resource_group.ibm_resource_group.resource_group[0]: Destroying... [id=a90393724cca4457a01b0ec87bbf0508]
module.resource_group.ibm_resource_group.resource_group[0]: Destruction complete after 2s

Destroy complete! Resources: 2 destroyed.

TestRunCompleteExample 2024-03-28T10:34:21+05:30 tests.go:434: END: Destroy
--- PASS: TestRunCompleteExample (696.22s)
PASS

:tada: This PR is included in version 1.5.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Map build files to language configurations via DBB File property syntax

This is enhancing the existing language configuration approach in zAppBuild by allowing the user to map build files to the language configuration via the DBB file property syntax :
# Define build configuration for submodules
languageConfiguration = languageConfigProps01 :: **/cobol/epsnbrvl.cbl, **/cobol/epsmpmt.cbl
# Define build configuration for main modules
languageConfiguration = languageConfigProps02 ::  **/cobol/epscmort.cbl

This is implementing the user feedback #497

Updated the output in verbose logging:
** Loading file level properties overrides.
*** Loading build property overrides for MortgageApplication/cobol/epsnbrvl.cbl
*** Loading build properties defined in language configuration languageConfigProps01.properties for MortgageApplication/cobol/epsnbrvl.cbl
    cobol_linkEditParms = ${cobol_linkEditParms},MAP
*** Validating file property overrides
    Setting file property override cobol_linkEditParms = ${cobol_linkEditParms},MAP for MortgageApplication/cobol/epsnbrvl.cbl
*** Loading build property overrides for MortgageApplication/cobol/epsmlist.cbl
*** Loading build properties from individual artifact properties file properties/epsmlist.cbl.properties for MortgageApplication/cobol/epsmlist.cbl
    cobol_compileParms = ${cobol_compileParms},SOURCE
*** Validating file property overrides
    Setting file property override cobol_compileParms = ${cobol_compileParms},SOURCE for MortgageApplication/cobol/epsmlist.cbl
*** Loading build property overrides for MortgageApplication/cobol/epsmpmt.cbl
*** Loading build properties defined in language configuration languageConfigProps01.properties for MortgageApplication/cobol/epsmpmt.cbl
    cobol_linkEditParms = ${cobol_linkEditParms},MAP
*** Validating file property overrides
    Setting file property override cobol_linkEditParms = ${cobol_linkEditParms},MAP for MortgageApplication/cobol/epsmpmt.cbl

Test framework passed. @M-DLB ready for review.
497-dbb-file-property-to-define-languageConfiguration.log


LogManager + groovy problem

Hi all,
Already tried:
...
 static main(args) {
    System.properties.put('java.util.logging.manager','org.jboss.logmanager.LogManager')
...

export JAVA_OPTS="-Djava.util.logging.manager=org.jboss.logmanager.LogManager"

but still not working.
I guess we need to point to log manager jar with bootclasspath to get this working but this is hard path coded.
1) Have no idea if this is an issue related to wildfly-swarm.
2) Have no idea if it was an issue from groovy and is correct to open this issue here.
example and stack error:
https://github.com/eprogramming/wildfly-swarm-gradle-groovy

Does groovy use JUL?
If so, does it provide any hooks to initialize before its first use of JUL?
If so, check Container.java for how we bootstrap JUL for JBoss logging.
--
Sent from my iPhone

On Jun 28, 2015, at 4:50 PM, Helio Frota<EMAIL_ADDRESS>wrote:
Hi all,
Already tried:
...
static main(args) {
System.properties.put('java.util.logging.manager','org.jboss.logmanager.LogManager')
...
export JAVA_OPTS="-Djava.util.logging.manager=org.jboss.logmanager.LogManager"
but still not working.
I guess we need to point to log manager jar with bootclasspath to get this working but this is hard path coded.

Have no idea if this is an issue related to wildfly-swarm.

Have no idea if it was an issue from groovy and is correct to open this issue here.

example and stack error:
https://github.com/eprogramming/wildfly-swarm-gradle-groovy
—
Reply to this email directly or view it on GitHub.

@bobmcwhirter thanks for the tip !
this helped me to search for other solution:
@CompileStatic

working from IDE and  './executable'
https://github.com/eprogramming/wildfly-swarm-gradle-groovy


Ensure connection for acknowledgement call (#895)

I've noticed an occasional NullPointerException crash (https://github.com/dooboolab/react-native-iap/issues/895) on some of the acknowledgement calls.
I'm still using the older 3.0.3 version but I've noticed that this can still be a problem in later versions as well.
I wonder why the code inside acknowledgePurchase isn't wrapped with ensureConnection like the other functions as this would solve the case of billingClient being null? (it's also missing in consumeProduct btw).
I haven't tested this in production but I'm pretty sure it will fix this particular issue.

Fixes #895

Nice, thanks for checking this so quickly. Shall I create another PR to wrap the code inside consumeProduct with ensureConnection as well? I think that function might have the same problem.
By the way, Is it easy for you to patch 3.0.3 with this fix as well? That would be really helpful :)

Sure!!! Also, I don't want to do the old fork update. Why mot upgrading instead? v3 and v4 aren't very different!

Was not able to upgrade yet because I'm using react-native v0.59 which doesn't support AndroidX yet which I think is necessary for v4 right?


Fix parallel stage in jenkins pipeline

Why is this PR needed?
The parallel block needs to be within a stage

https://ci.suse.de/view/CaaSP/job/caasp-jobs/job/caasp-v4-upgrade-nightly/1/console


Yang autofill not working consistently

One of the greatest feature of yangster is, auto fill syntax when typing on VS Code for Yang files. But it works when first time extension installed, but if VS code is reloaded or reopen it stops working.
Only solution to uninstall the extension and install again.
Any consistent solution for this?

@jbl241987
I can't reproduce this issue.
Normally Yang language server need a second to restart, than everything works for me, also the content assist.
Could you may be try with the lates extension version?


[BACKLOG-32443] Fix for new folder with pvfs url

:white_check_mark: Build finished in 1m 28s
Build command:
mvn clean verify -B -e -Daudit -amd -pl \
plugins/file-open-save-new/core

:ok_hand: All tests passed!
Tests run: 11, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Add HashiCorp to remote-friendly jobs

HashiCorp is distributed across the United States (not yet international): https://www.hashicorp.com/jobs.html

Thank you! :D

No, thank you :smile:


Web Crash on Release/Product

Hi, I have developed an app and published it please check it and see the error on console;
Pinlogy
As you can see on console there is an error;
RangeError (end): Invalid value: Not in inclusive range 5..22: -1
Although there is NO error when i debug it on VS Code.
 sdk: '>=3.2.3 <4.0.0'
Packages used on project;
caroby: ^0.1.2   go_router: ^13.0.1   easy_localization: ^3.0.3   encrypt: ^5.0.3   hive_flutter: ^1.1.0   json_annotation: ^4.8.1   firebase_core: ^2.24.2   firebase_auth: ^4.16.0   cloud_firestore: ^4.14.0   url_strategy: ^0.2.0   flutter_svg: ^2.0.9   crypto: ^3.0.3   url_launcher: ^6.2.3
[√] Flutter (Channel stable, 3.16.8, on Microsoft Windows [Version 10.0.19045.3930], locale tr-TR) [√] Windows Version (Installed version of Windows is version 10 or higher) [√] Android toolchain - develop for Android devices (Android SDK version 33.0.0) [√] Chrome - develop for the web [√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.5.1) [√] Android Studio (version 2021.2) [√] VS Code (version 1.85.1) [√] Connected device (3 available) [√] Network resources

without code a reproducible code sample it's impossible to know if the error is caused by your implementation
(or by your own package "caroby")

I have tested my own package and it didnt cause any error. My own package has only extensions and some core UIs so i dont think it is the one causing the problem. If it did then why cant i see that on debug? I really dont get this. There litereally zero error or warning during debug but only after "flutter build web".
This is a whole project so i dont know what part of code i can show you because console doesnt even tell the current location of problem :/// I'm open to any solution

Hi @AliKales, do you only face this issue when you deploy your app to a hosting site? If not, can you try running your app in profile mode to see if you get the same issue? (If you do experience the issue in profile mode as well, the stacktrace will not be minified).
What do you do on startup in your application?
You can also try integrating an error reporting service like sentry for your webapp to see if it captures a clearer stacktrace which would help with investigating the issue.
As pointed out above, without a minimal sample, it will be difficult for us to investigate this issue.
Thank you

Hi @danagbemava-nc, i kept debugging whole time with flutter run --profile and i kept getting same error. I tried many things but in the end i removed go_router and wrote my own router with core Dart codes. Right now it works as it should. I like to use go_router but for this project i will continue with core routing. Anyone who faces this issue i suggest you to remove go_router! Thank you both for commenting!

Hi again! I noticed my mistake here so i wanted to reopen this issue to let people know actually what was wrong with my code.
Anyone who is facing this issue it is most probably because of go_router but you don't need to remove go_router from your project!
As you know we set GoRoute s in GoRouter's routes list. I had written a code that does this automatically but apparently that code crushed and did not let app load in first place.
So long story short; Check your go router and see if everything is OK!
Thanks!


[grep] improve how tests run

What is the root cause of the problem?
It seems like this is just a band-aid, moving the entire test framework into grep tests.

Believed superceded by #233


[Feature]: convert F_parm to log(F_parm)

Describe the solution you would like.
F_parm logically cannot be < 0.0, but if catch for a fleet * time is very small, it can get close to 0.0 and impede convergence.
F_parm is used only in Procedure_Section to populate Hrate which is the F value used in calculations:
Hrate(f, t) = F_rate(g);
so easy to change this to:
Hrate(f, t) = exp(F_rate(g));
requires associated changes to the bounds on the F_rate parameter.
should do the same for the init_F parameters.
Describe alternatives you have considered
none considered
Statistical validity, if applicable
implementing parameters in log space is generally recommended by my statistician colleagues
Describe if this is needed for a management application
no
Additional context
No response

@Cole-Monnahan-NOAA
I'm interested in your opinion on this change.  It is nearly completed.

@Rick-Methot-NOAA does this actually change the model behavior when you have a really small catch? I'm generally skeptical of this for two reasons.
First, ADMB's bounding functions will, I believe, closely mimic a exp transformation if you use bounds (0,Z) where Z is some large number. So I wouldn't expect it to actually change behavior close to 0, and thus it's not clear why it's needed. One advantage of the default ADMB transformation is that it behaves better when the parameter estimate is very close to the bound. By that I mean that the Hessian remains invertible whereas a logistic transformation will fail. Some see this as a good thing, because it lets the user leave parameter estimation on when the mode is at a bound. If you had a logistic transformation (e.g., -hbf) then the user would have to turn off estimation to get uncertainty. So if anything I'd expect worse performance a priori, meaning you'd have more models failing to do the SD calcs. It may be worth testing the models w/ the hbf flag too.
Second, the new approach is now a double transformation which makes it harder to interpret. I don't think ADMB will choke on it or anything, but it's not "clean" in some sense. I think ADMB should produce a .std file for unbounded parameters, see e.g., https://github.com/admb-project/admb/issues/275, and such an approach would not work here.
I'm a pragmatist though, so if this improves model convergence then I'd find that compelling evidence, if a bit unexpected.

That was always my logic on this also.  Other community members have advocated for parameters being in log-space.  Here it is common for many F values to be near the natural lower of zero.  Your improvements of ADMB now flag that condition.  I will do a run in which I compare parameter variances with and without the log(F) approach.

I see, yeah that may be an unintended consequence of the new output. I've seen it as well and just ignored it.
I did a quick test this AM. Here's the R code
### Quick exploration of the bounding options in ADMB
a <- 0 # min bound
b <- 2 # max bound
## this is the ADMB default, note input range is (-1,1)
f1 <- function(x) a+(b-a)*(.5*sin(x*pi/2)+.5)
## invoked with '-hbf' flag, standard logistic, input (-Inf,Inf)
f2 <- function(x) a+(b-a)/(1+exp(-x))
## log transform, input (-Inf,Inf)
f3 <- function(x) exp(x)
x <- seq(-15,-1, len=1000)
## go from (-1,1) space to logit space so comparable
x2 <- -1+2/(1+exp(-x))
y1 <- f1(x2)
y2 <- f2(x)
y3 <- f3(x)
plot(x, y1, type='l', log='y', ylab='Bounded value',
     xlab='Unbounded value')
lines(x, y2, col=2)
lines(x, y3, col=3)
legend('bottomright', legend=c('sine', 'logistic', 'log'),
       col=1:3, lty=1)

Which results in this plot

Note the y-axis is in log space. So you can see that log and logistic approach the lower bound at the same rate just with an offset that depends on the max bound. The sine transform (default in ADMB) is very different.

I ran a test using the petrale sole west coast assessment.  It has 2 fleets and a long time series, so the model needed to calculate very small F's going back to 1910.
I ran three model configurations:

F as "hybrid F" which does not use F parameters and instead scales each F value to match the catch
F as a parameter with the parameter being the F value
F as exp(Fparm) in a trial branch.

The results are practically indistinguishable to many sig. digits between the runs.  I asked for F at age 10 as a derived quantity.  The se of this was the same no matter what method was used to produce the F values.  Similarly for derived quantities affected by F, like SSB.
I conclude that there is no value in converting to log(F) as the parameter, nor in providing users the option.
QED.

Agreed, and that makes sense to me. A good experiment to do to confirm.

Thanks for going through all this trouble @Rick-Methot-NOAA and @Cole-Monnahan-NOAA.
This type of exploration seems like the ideal process to figure out lots of potential model features for SS3 and any future model, so you've set a great example here.

Thanks Ian.  I think it good to summarize this in a paragraph for the next release notes.


Add withUploads() functionality to CRUD fields and columns

WHY
BEFORE - What was wrong? What was happening before this PR?
Managing uploads with backpack fields could be a challenge sometimes. Especially if we are talking about repeatable fields etc.
This PR aims to provide a tool to help developers setup their file uploads quicker ... well .. way way quicker 😄
AFTER - What is happening after this PR?
You should be able to get a fully working upload field in your crud with just:
CRUD::field('document')->type('upload')->withUploads();
CRUD::field('documentsss')->type('upload_multiple')->withUploads();
CRUD::field('passport_image')->type('image')->withUploads();

Is it a breaking change?
No it's not! Fantastic!
How can we test the before & after?
Will write some docs later.

A new inspection was created.

The inspection completed: 37 new issues, 71 updated code elements

A new inspection was created.

A new inspection was created.

A new inspection was created.

The inspection completed: 39 new issues, 75 updated code elements

The inspection completed: 40 new issues, 75 updated code elements

A new inspection was created.

A new inspection was created.

The inspection completed: 42 new issues, 75 updated code elements

A new inspection was created.

The inspection completed: 35 new issues, 68 updated code elements

A new inspection was created.

A new inspection was created.

The inspection completed: 36 new issues, 68 updated code elements

A new inspection was created.

A new inspection was created.

A new inspection was created.

The inspection completed: 42 new issues, 75 updated code elements

The inspection completed: 40 new issues, 75 updated code elements

A new inspection was created.

The inspection completed: 43 new issues, 75 updated code elements

A new inspection was created.

The inspection completed: 56 new issues, 75 updated code elements

WIP DOCS
Creating your own uploader.
Every uploader extends the base Uploader class. So let's start by doing it:
class MyUploader extends Uploader
{
}

You will notice that you can't override the constructor. Don't worry, we have left you a way for you to customize the initialization, using the static method for().
class MyUploader extends Uploader
{
    public static function for($crudObjectArray, $uploaderConfiguration) {
        // add something to $crudObjectArray or $uploaderConfiguration
        // eg: force a disk in this uploader specifically
        $uploaderConfiguration['disk'] = 'custom';
        return parent::for($crudObjectArray, $uploaderConfiguration)->multiple();
    }

    public function uploadFile($entry)
    {
         // your upload code.
    }

}

There is one "entry function" for each action performed by the uploaders:

Model Event
Uploader function

saving
processFileUpload()

retrieved
retrieveUploadedFile()

deleting
deleteUploadedFile()

processFileUpload()
To ease things out we separated the processFileUpload() into two separate methods: uploadFile() and uploadRepeatableFile().
That way you only need to overwrite the processFileUpload() method if you want to change the "core behavior" of saving the file paths in a database field linked with the CrudField, otherwise, just implement the uploadFile() method.
In case you need to support the uploader inside a repeatable container your should also implement the uploadRepeatableFile() method.
The uploadFile() function receive the $entry (a model instance that's being saved), so you can get your upload values from request, process them and return whatever is going to be stored on the entry, usually the file path string ('avatars/avatar.png').
retrieveUploadedFile
This function runs when the $entry is hydrated from the database. You can use this function to configure the entry before returning it for CRUD to display. (ex: get uploads from media table, and add them to the entry as attributes to work with the crud fields).
deleteUploadedFile
This function runs when the entry is deleted. If configured it will delete the associated entry files.

A new inspection was created.

A new inspection was created.

A new inspection was created.

The inspection completed: 36 new issues, 77 updated code elements

Uploaders validation:
image
Requires manual validation (no laravel rules supplied to handle base64). One can use a package like https://github.com/crazybooot/base64-validation

When nothing changes, the field send the db path in the request.
When file is deleted and a new one is NOT added, it will send an empty value (null) in request.

Even using the package mentioned it cannot be used with regular laravel validation (eg. form requests), requiring custom validation to properly work.
Possible solution: don't send nothing when nothing changes. allowing devs to supply two validators (create/update with required/sometimes respectively), see upload example bellow:
upload
Need two separate validators, one for create and another for update.
// create validation:
'field' => [
                'required',
                File::types(['pdf'])
                    ->min(1024)
                    ->max(10 * 1024),
            ],
// update validation:
'field' => [
                'sometimes',
                File::types(['pdf'])
                    ->min(1024)
                    ->max(10 * 1024),
            ],

When nothing changes the field is not posted in the request.
When file is deleted and a new one is NOT added, it will send an empty value (null) in request.

upload_multiple
Requires custom validator to properly validate. No other way around, needs breaking change to fix.
There is one hidden input that when nothing changes on field breaks the sending values and sends [ [0=>null] ].
Dropzone is out of scope here, since it's beeing built now we can do whatever we want.

 It works like freakin' magic! Well done Pedro 👏👏👏 Well freakin' done!


server command generates two terminal windows

Make sure to only create the terminal when the server is not already running.
Also, if the terminal app is already running, create a new window and run the server there.

Fixed in https://github.com/ksylvan/Zoom_OPM/commit/c1319a16af484500837342b22606f068372a32b8


Weird behavior for lines and points with high size

There is a weird behavior with points with high radius:

And the same with lines with high line width:

They seem to be affected by clipping extension. It is required for polygons in order to avoid artifacts and it was added by uber people for MVTLayers: https://github.com/visgl/deck.gl/pull/4336
An suggestion would be removing it for lines and points layers.

@manmorjim I would suggest creating an issue at deck.gl repo upstream, related to the pure MVTLayer to see if we can get some feedback from clip extension developers.
Then we can decide if work has to be done in deck.gl itself or our side on toolkit

Issue created: https://github.com/visgl/deck.gl/issues/4549

Fixed in https://github.com/visgl/deck.gl/pull/4557


Release Notes parser should warn a release engineer when portions of release notes are dropped

We've missed including some issues in our release notes when the parser dropped them due to lack of a preceding - in markdown to put them in a list. It would be awesome if the parser could warn us when some text below a header hasn't been included.

Lacking a - doesn't affect whether a line is included, but it can be hard to see when other items do have the bullet marks. Aside from that, notifying others automatically would be possible but would need lots of definition. Currently I generate release notes from my local machine. How would it know who to notify? What conditions would trigger a notification? There's often text that's not included, I think it would get to notification overload pretty quickly.

I was just thinking it would notify you while you're generating them that something under a header wasn't included in the notes. I'm not sure exactly why portions of the release notes were omitted, I thought it was the missing - but I could be wrong. I'll ask in standup tomorrow what the issue was.

One thing to keep in mind is that release notes are only generated when the release is created. We can manually regenerate the notes if needed. That means that if PR body changes later, those changes are not reflected in the generated release notes until we manually regenerate.

That could totally have been the issue.
On Thu, Nov 16, 2017 at 8:43 AM, Chris Polcyn<EMAIL_ADDRESS>wrote:

One thing to keep in mind is that release notes are only generated when
the release is created. We can manually regenerate the notes if needed.
That means that if PR body changes later, those changes are not reflected
in the generated release notes until we manually regenerate.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SalesforceFoundation/CumulusCI/issues/509#issuecomment-344982920,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-JVv8vPu9Ex1qwGYwGQ_z2d3dXQMe0ks5s3GYbgaJpZM4QfXyR
.

Closing, invalid


Merge "show all detailed text" and "hide all detailed text" buttons

In order to match the symmetry with "Expand all panels" and "Hide met or N/A criteria".
The detailed text starts out hidden, so the button can start out as "show all detailed text". When all detailed text is shown, the button should change to say "hide all detailed text".

I agree, it is an easy fix and like you said it matches the styles of the other buttons.


AOS Animating each time a user visit to section

This is:

Question

Expected Behavior
Actually i want to AOS to only animate once a user go through the page and redo the same behaviour only as of when the page reload
Actual Behavior
At the moment it's doing it everytime we faff through the sections
I hope it's been addressed before, any help on this would be a pleasure!
Cheers,

Hi,
set atribute 'once' to true in global config  or add it per animation. It makes animation run only once which seems like your desired behaviour. Check out documentation Everything is described in README: https://github.com/michalsnik/aos

Hey @ZuzannaFilipkowska ,
Really appreciate your take on it, Yes i rattled over the documentation, all is clear there!
Fondest Regards,
H


drivers: gpio: add new driver for STMPE1600 gpio expander

Add support for the STMPE1600 GPIO expander (see https://www.st.com/en/interfaces-and-transceivers/stmpe1600.html).
This first version of the driver does not support interrupts because I do not have any design at hand with the interrupt pin wired to test the feature.

Thank you very much for your reviews @mnkp. I just pushed the requested changes, but I still need to find some time to run the tests you suggested.
Would you need me to rebase (and possibly squash) my commits onto master ?

Thank you again for the reviews and help @mnkp.
I squashed and rebased the commits on top of main, including your latest remarks.
I also noticed, while running tests/drivers/gpio/gpio_api_1pin/ that I needed to add an implementation for the API func port_toggle_bits. With these changes, the entire test suite is passing.

Thanks again for your review !


memory leak when using this with gulp mocha, mockery, chai, and sinon

We're trying to use this where I work, and are finding that this runs into a memory leak:
(node:3597) Warning: Possible EventEmitter memory leak detected. 11 exit listeners added. Use emitter.setMaxListeners() to increase limit
When looking at your code, you continually add eventlisteners, but never remove them. If you need an example please let me know.
I will note that everything seems to run fine within wallaby js. It's just that we need gulp mocha because our build process depends on verifying various tests.
Best,
Kelly

Thanks for the bug report, a minimal code example would be great! If you have a pull request that would be even better! If not, I can probably take a look and make changes some time this week.
What version of node are you running?


Typescript Error

After npm installing the module in my project, and adding it to my app.module file I get three  errors stating:  Parameter 'index' implicitly has an 'any' type.
Is this an error because i haven't set it up correctly with system.js?

Hi, I am using angular-cli and ECMA2016 for compilation. What version of ecma are you using for compiling the code ?

cannot reproduce. hence, closed.


Failed builds when Coveralls is down for maintenance

Expected Behavior
The builds ignore the Coveralls and other utilities' failures.
Actual Behavior
The builds fail if Coveralls is down for maintenance:
https://travis-ci.org/zalando-incubator/kopf/jobs/571226118

On the other hand, it is better to allow the tests to be broken for a day or two (while being able to merge the urgent fixes forcely), than to lose the coverage information from the builds silently over the long period of time until accidentally noticed (coverage is not checked regularly) — in case the CLI tool is broken in the builds.


Making all the brackets editable

Hello,
Just wanna ask if it's possible to make all the brackets editable?
Currently, how it works is you can only edit the brackets on the first round. Once, you fill these brackets on the first round, now you can add or edit the brackets on the next round. What I want to achieve is I want to make all the brackets editable without filling in the first round. Is the plugin capable of doing that?
I based the editing of brackets from the Custom handlers on this demo.
Thank you!

same. also looking for this!
@boang3000 did you find anything?


Dead link on page

There is a link on this page that is dead.
Text "Change the look of your SharePoint site"
Link:  https://support.microsoft.com/en-us/office/change-the-look-of-your-sharepoint-site-06bbadc3-6b04-4a60-9d14-894f6a170818?ui=en-us&rs=en-us&ad=us

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

ID: 4c5279de-43c1-3a83-83ce-c1b34d25287d
Version Independent ID: 7cda1751-170a-7bbd-6d71-33daf5168f68
Content: SharePoint site theming
Content Source: docs/declarative-customization/site-theming/sharepoint-site-theming-overview.md
Product: sharepoint
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

The link seems to work well for me actually

So this issue and #7481 can be safely closed i think @VesaJuvonen

Yes.  Can be closed.
From: Charles Emmanuel S. Ndiaye @.>
Sent: Tuesday, November 16, 2021 8:24 AM
To: SharePoint/sp-dev-docs @.>
Cc: Scot W. Bickell @.>; Author @.>
Subject: Re: [SharePoint/sp-dev-docs] Dead link on page (Issue #7480)
So this issue and #7481https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-dev-docs%2Fissues%2F7481&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297922461|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=z8OnTndYVtP5y6C8NEb2aEMISOrcmf2Cj8uXQsFOG4I%3D&reserved=0 can be safely closed i think @VesaJuvonenhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FVesaJuvonen&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297932454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=DzCSa4NOBjYfg71dMAtk3td1%2BGrL93fc9Odbiwq4vSQ%3D&reserved=0

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-dev-docs%2Fissues%2F7480%23issuecomment-970381306&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297942454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=4se2AQbeUotyiEgLMGFWG7UeAPdqEFoSBWS3GICBVXc%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOLGNMPWOUG3H4M4Z3NAAILUMJZQDANCNFSM5HMAD6FQ&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297942454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=ZVxdwaxrB4dZl1U4etyz%2F7xeQRmMWrOerXCNzrRvZLY%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297952448|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=T8Fk1ZC9CvaxE50Cinw52MXTxYA9%2BCEVf7MV2czog70%3D&reserved=0 or Androidhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|scot.bickell%40asi.com|c3e73fac909049bab2fd08d9a9151538|6ed24dd2d730413b8f4eed3051f9d4ec|0|0|637726730297962443|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=II4sibolh96vdwVxYM8R2x1SFuFjPZgsiDCurkCCMFw%3D&reserved=0.
Our offices are operating at limited capacity to help do our part to flatten the COVID-19 curve and to minimize the chance of exposure between employees or guests. Many employees are working from home. During this time, our offices are only open to those employees and visitors who have been invited by management and deemed essential to be in our facilities.
This email may contain confidential or privileged information that is not to be used or communicated in any manner to unintended parties. Please inform sender immediately by e-mail if you believe you may not be an intended recipient. Thank you. TP: 06c8409d4f374ae22092c3fc3af8980ff003a8df

Closing based on the comments.


handling_invocation-number_to_xml_DP+F

Fix adds Dataprovider + Factory parameters to testng-failed.xml with 'invocation-numbers'

Hi, thanks for the change.
Could you fix the conflicting files, update CHANGES.txt file and add a non-regression test?

In order to add a test, you can have a look at https://github.com/cbeust/testng/blob/master/core/src/test/java/test/failures/FailuresTest.java

Hi, thanks for the change.
Could you fix the conflicting files, update CHANGES.txt file and add a non-regression test?

I have done. Could you please review it?

In order to add a test, you can have a look at https://github.com/cbeust/testng/blob/master/core/src/test/java/test/failures/FailuresTest.java

Thank you @juherr, I have based on core/src/test/java/test/failedreporter/FailedReporterTest.java.

@PavelSakharchuk - Thank you so much for taking the time to contribute to TestNG Appreciate it. Looking forward to more such contributions.

@PavelSakharchuk - Thank you so much for taking the time to contribute to TestNG Appreciate it. Looking forward to more such contributions.

Thank you, guys, for your review.
It is my first contributed PR :)
I'm really happy.
I'm going to continue fixing for 'DataProvider + Factory':

grouping  invocationNumbers to one 'invocation-numbers' for the same method;
running tests according to 'invocation-numbers'.


Requesting permissions for urbancode-velocity

Description
https://github.com/jenkinsci/urbancode-velocity-plugin
https://issues.jenkins-ci.org/projects/HOSTING/issues/HOSTING-592?filter=allissues

Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Please follow the instructions of the checklist, checking off items that you have completed.

Updated.  thanks


Don't autocomplete #' to #'' for r files

In R, #' has a special meaning. Can you please change the defaults to add this rule, (see title)? I tried to fix it myself but I was unable to do so and had to resort to the following
local npairs = require("nvim-autopairs")
local cond = require('nvim-autopairs.conds')
local cmp_autopairs = require('nvim-autopairs.completion.cmp')
local cmp = require('cmp')

npairs.setup({
  break_line_filetype = nil,
  check_ts = true,
  disable_filetype = {"TelescopePrompt"}
})

npairs.remove_rule("'")
npairs.add_rules({
  Rule("'", "'", "^#$")
    :with_pair(cond.not_before_regex("^#$"))
})

This is of course undesirable but I was unable to change the rule for #' for R files while keeping the existing rules for '.

Rule("'", "'", "^#$")
you set wrong file type the third argument should be a file type of R language   lua print(vim.bo.filetype)
npairs.add_rules({
  Rule("'", "'", "R")
    :with_pair(cond.not_before_regex("^#$"))
})

Or you can use
--- get the first rule of '
require('nvim-autopairs').get_rule("'")[1]
:with_pair(cond.not_before_regex("^#$"))

Ups, that was a typo (I used "r" instead of "^#$" before).
The second solution is the one I was looking for, thanks!


fix: check if alert is supported before decoding cluster_id

What?

Moves logic to decode cluster_id after checking if the alert is supported by CAD.

Why?
Some unsupported alerts don't contain the cluster_id, e.g. the AVO alerts on the app-sre-alertmanager service. This causes CAD pipelines to exit ungracefully.
Therefore, we need to move the logic to decode the cluster_id after the check if an alert is supported.

Nice catch!
/lgtm


Update interop_helpers.h with null check, conversions for String, Guid

interop_helpers.h was borrowed from CxUtils.h, which has since grown to include support for string and guid conversion.  Also the generic from_cx doesn't include a null check.
    template <typename T>
    T to_cx(winrt::Windows::Foundation::IUnknown const& from)
    {
        return safe_cast<T>(reinterpret_cast<Platform::Object^>(winrt::get_abi(from)));
    }

    template <typename T>
    T from_cx(Platform::Object^ from)
    {
        T to{ nullptr };

        if (from != nullptr)
        {
            winrt::check_hresult(reinterpret_cast<::IUnknown*>(from)->QueryInterface(winrt::guid_of<T>(), winrt::put_abi(to)));
        }

        return to;
    }

    inline Platform::String^ to_cx(winrt::hstring const& from)
    {
        return reinterpret_cast<Platform::String^&>(const_cast<winrt::hstring&>(from));
    }

    inline winrt::hstring from_cx(Platform::String^ const& from)
    {
        return reinterpret_cast<winrt::hstring&>(const_cast<Platform::String^&>(from));
    }

    inline Platform::Guid to_cx(winrt::guid const& from)
    {
        return reinterpret_cast<Platform::Guid&>(const_cast<winrt::guid&>(from));
    }

    inline winrt::guid from_cx(Platform::Guid const& from)
    {
        return reinterpret_cast<winrt::guid&>(const_cast<Platform::Guid&>(from));
    }

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

ID: 3e79918f-ced1-2809-d280-501e85ade810
Version Independent ID: 950b6e6a-f809-1c7a-5bcb-b8919c978110
Content: Interop between C++/WinRT and C++/CX - UWP applications
Content Source: windows-apps-src/cpp-and-winrt-apis/interop-winrt-cx.md
Product: uwp
Technology: platform
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Sorry for the delay: I just published the updates. One thing, the existing content had T^ to_cx; and the code above has T to_cx (without the hat). I had to put the hat back in to get the example project walkthrough to work; hope that's ok.
-Steve


CI speed / bitwise lookup table / faster MockProver

currently it costs more than 30 minutes to finish CI in appliedzkp/zkevm-circuits. I test it locally and find tests of bitwise op cost much time. (Other op test cost 10-30s, while bitwise op test needs 150-180s).  Then I find the reason behind this is that bitwise op test needs full lookup table ( with bitwise lookup table, eg bitwise xor table has 256*256 entries, same for bitwise and/or) and other ops need stripped lookup table (without bitwise table).    Using full lookup table needs more rows, namely more degree for the circuit.
In MockProver::verify, every gate and every lookup are evaluated on every row （ including the empty/padding rows).    So if we can verify on less rows, we can make verify much faster.   We can add a method call verify_at_rows(rows_ids: Box<Interator<usize>>), we know which rows can be skipped(padding rows, and non q_step rows), so from the application circuit side we can provide much less row ids to verifier to speedup verification.
We can do this after kilic rebase appliedzkp/halo2 on zcash/halo2.

seems we can close this via https://github.com/appliedzkp/zkevm-circuits/pull/354?

yes


GTA 4 Doesnt work

Software information
GTA 4
System information

GPU: Nvidia GT720M
Driver: 391.35
DXVK version: 1.5.1

Apitrace file(s)
(Not Traceable because it doesnt even start well it does start but in Details Tab on Task Manager it shows its memory consumption going up to like 17343k then it suddenly drops to 1393k and will stay like that forever)
Log files
info:  Game: GTAIV.exe
info:  DXVK: v1.5.1
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
warn:  OpenVR: Failed to locate module
info:  Enabled instance extensions:
info:    VK_KHR_get_physical_device_properties2
info:    VK_KHR_get_surface_capabilities2
info:    VK_KHR_surface
info:    VK_KHR_win32_surface
warn:  Failed to create Vulkan 1.1 instance, falling back to 1.0
info:  Game: GTAIV.exe
info:  DXVK: v1.5.1
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
warn:  OpenVR: Failed to locate module
info:  Enabled instance extensions:
info:    VK_KHR_get_physical_device_properties2
info:    VK_KHR_get_surface_capabilities2
info:    VK_KHR_surface
info:    VK_KHR_win32_surface
warn:  Failed to create Vulkan 1.1 instance, falling back to 1.0

I have used Vulkan 1.5.1 because in 1.5.2 the Vulkan 1.1 is necessary and this is the max my drivers can go, So uh, I wanted to run this GTA 4 with DXVK, any way it is possible please?

NVIDIA Fermi GPUs are at this point unsupported. You need a Vulkan 1.3 capable card.

NVIDIA Fermi GPUs are at this point unsupported. You need a Vulkan 1.3 capable card.

What about older versions of DXVK do they also not support?
Also just tried 1.5 same result:
info:  Game: GTAIV.exe
info:  DXVK: v1.5
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
warn:  OpenVR: Failed to locate module
info:  Enabled instance extensions:
info:    VK_KHR_get_physical_device_properties2
info:    VK_KHR_get_surface_capabilities2
info:    VK_KHR_surface
info:    VK_KHR_win32_surface
warn:  Failed to create Vulkan 1.1 instance, falling back to 1.0
info:  Game: GTAIV.exe
info:  DXVK: v1.5
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
warn:  OpenVR: Failed to locate module
info:  Enabled instance extensions:
info:    VK_KHR_get_physical_device_properties2
info:    VK_KHR_get_surface_capabilities2
info:    VK_KHR_surface
info:    VK_KHR_win32_surface
warn:  Failed to create Vulkan 1.1 instance, falling back to 1.0

What about older versions of DXVK do they also not support?

Your guess is as good as mine. You're using no longer supported drivers, with a no longer supported card, with no longer supported versions of dxvk.

What about older versions of DXVK do they also not support?

Your guess is as good as mine. You're using no longer supported drivers, with a no longer supported card, with no longer supported versions of dxvk.

Hmm... So its my GPU fault and older versions of DXVK also wouldn't work, Am I correct?

So its my GPU fault and older versions of DXVK also wouldn't work, Am I correct?

They might work in some situations, but it's hardly guaranteed at this point. If you run into issues I'm afraid there's nothing we can do at this point, other than advise you to use a supported card with the latest version of dxvk.

So its my GPU fault and older versions of DXVK also wouldn't work, Am I correct?

They might work in some situations, but it's hardly guaranteed. If you run into issues I'm afraid there's nothing we can do at this point, other than advise you to use a supported card with the latest version of dxvk.

Okay...     :/

I'm sorry but your GPU doesn't fulfil the minimum requirements of DXVK.


New feature: GitHub actions for Office 365 cli

Can we think of introducing GitHub actions for office365-cli? I had a look at the actions available for azure cli so we can do something on similar lines. If ok, then we can think of the following maybe?

an action for login
an action for running an office 365 cli command
(in future) an action to deploy an spfx package to the app catalog

Do you have an idea for a specific scenario/problem they would solve?

I was thinking mainly for adding SPFx package to app catalog, like we do in Azure pipelines using office 365 cli currently. E.g. In a DEV tenant, when the code is pushed to a GitHub repo, create the package and add it the app catalog. Something along these lines?

It sounds interesting indeed. Looking at the Actions' docs shows:

GitHub Actions is currently in limited public beta and is subject to change. We strongly recommend that you do not use this feature for high-value workflows and content during the beta pe

Do you think it would be worth investing in this already or would it be better for actions to stabilize and revisit it then?

Yes @waldekmastykarz let's revisit this later.

So @waldekmastykarz what I was thinking was we create a GitHub action named something on the lines of "Add and deploy SPFx package using Office 365 CLI". This action would do the following:

Login to the tenant
Add the app to the app catalog site
Deploy the app

To create the action, after reading this article, I was thinking if we start with the aptly named "Hello word docker action".
The idea would be to use a docker container which has Office 365 CLI installed in it and we run the above 3 commands. You have written posts on docker with SPFx before, so if you like this approach can you please let us know if there is a docker container with Office 365 CLI that we can make use of?

There is no docker image with the CLI yet, but the article you mentioned, talks about creating an image specifically for the action, so for now we could take the regular Node.js image as a baseline and include installing the CLI as one of the steps in the dockerfile. Once we get a docker image with the CLI in it, we could update the dockerfile to point to the CLI image instead. Does that make sense?

Just to chip If we are looking at potentially using a Docker Container for this purpose, there is a potential to re-use the same image to allow us to use Remote Development in VS Code for the CLI, this would help provide a consistent cross platform development environment for project contributors to use. Something we may want to consider...

I've been thinking about publishing a docker image for the CLI in the past but was stuck in how to tie it in into our release process. Ideally, I'd have the image use the latest version of the CLI but I'm not sure if publishing the CLI to npm and trying to use the just released version of the CLI from an image would work in the same release.

For now I will submit a PR with a docker container from Node.js and install Office 365 CLI. We can change it to the required one easily in the future.

As we now have multiple Actions published to GitHub marketplace and issues/PRs will be handled within their respective repositories, I will close this issue as the work is now complete.
Thank you for your initial suggestion and your efforts @anoopt 👏🏻


[Chore] Replacing /layer5io/meshery with /meshery/meshery

Description
This PR includes changes reflecting replaces for  /layer5io/meshery with /meshery/meshery
This PR fixes Nothing
Notes for Reviewers
Signed commits

[X] Yes, I signed my commits.

Codecov Report

Merging #3343 (d02d294) into master (05cda78) will increase coverage by 0.55%.
The diff coverage is 2.20%.

@@            Coverage Diff            @@
##           master   #3343      +/-   ##
=========================================
+ Coverage    7.02%   7.58%   +0.55%
=========================================
  Files          23      35      +12
  Lines        1409    2743    +1334
=========================================
+ Hits           99     208     +109
- Misses       1303    2488    +1185
- Partials        7      47      +40

Flag
Coverage Δ

gointegrationtests
2.91% <2.20%> (?)

integrationtests
∅ <ø> (∅)

unittests
6.27% <0.31%> (-0.76%)
:arrow_down:

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/config/config.go
1.11% <0.00%> (-1.12%)
:arrow_down:

mesheryctl/internal/cli/root/perf/perf.go
50.00% <ø> (ø)

mesheryctl/internal/cli/root/system/start.go
1.53% <0.00%> (-0.01%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
0.00% <0.00%> (ø)

mesheryctl/internal/cli/root/system/stop.go
1.08% <0.00%> (-0.07%)
:arrow_down:

mesheryctl/pkg/utils/auth.go
0.00% <0.00%> (ø)

mesheryctl/pkg/utils/error.go
0.00% <0.00%> (ø)

mesheryctl/pkg/utils/formatter.go
0.00% <0.00%> (ø)

mesheryctl/pkg/utils/healthcheck.go
18.75% <0.00%> (ø)

mesheryctl/pkg/utils/helpers.go
0.00% <0.00%> (ø)

... and 17 more

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 d8bfecb...d02d294. Read the comment docs.


Fix iron-dropdown padding covering the clear button

Fixes #176
Fixed by applying translateY instead of padding. Using margin does not work as expected as it probably affects with some calculations of iron-dropdown.

This change is

Tests are failing until https://github.com/vaadin/vaadin-date-picker/pull/184 is merged and this branch is rebased.

The dropdown position seems to be miscalculated once you scroll the page with the dropdown open.
Previously, tehapo (Teemu Pöntelin) wrote…
> Tests are failing until https://github.com/vaadin/vaadin-date-picker/pull/184 is merged and this branch is rebased.

Comments from Reviewable

So it seems. I'll fix this tomorrow.
Previously, tomivirkki (Tomi Virkki) wrote…
> The dropdown position seems to be miscalculated once you scroll the page with the dropdown open.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks broke.

Comments from Reviewable

Applied a better fix by replacing the deprecated verticalOffset property with marginTop and marginBottom calculations.
Previously, tehapo (Teemu Pöntelin) wrote…
> So it seems. I'll fix this tomorrow.

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

Comments from Reviewable

vaadin-date-picker.html, line 423 [r2] (raw file):

        var verticalOffset = this._fullscreen ? 0 : this.clientHeight + (bottomAlign ? 0 : 2);
        this.$.dropdown.style.marginTop = bottomAlign ? 0 : verticalOffset + 'px';
        this.$.dropdown.style.marginBottom = bottomAlign ? verticalOffset + 'px' : 0;

Making the other margin 0 leaves no gap between the dropdown and browser edge (should have a 10px gap on non-fullscreen)

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion.

vaadin-date-picker.html, line 423 [r2] (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
> Making the other margin 0 leaves no gap between the dropdown and browser edge (should have a 10px gap on non-fullscreen)

Done.

Comments from Reviewable

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

Comments from Reviewable

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

Comments from Reviewable


TrySaveStreamAsync doesn't create folder if not exist

I added an UploadMediaFile in IMediaService and used this method to save a stream.
_mediaFileStore.TrySaveStreamAsync(filePath, inputStream);
But if the directory doesn't exist, there is nothing that creates the folder hierarchy.

Why did you have to add something to IMediaService.


[WIP] Attempt to fix undefined function problem #425

Closing for now until more info is available

Closing for now until more info is available


Massive memory in Processes

When I import the entire library into Scenebuilder 2.0, it takes what I have right now, a bunch of imageviews and shapes and basic UI elements and says its taking up 855k memory under Windows Task Manager -> Processes.
Only when I change one UI element... be it by changing the location or the opacity, that the memory goes up by 40k, and if I save and restart Scenebuilder, I go back to it now being 860k memory!
Just letting it idle, it went from 860k to launch, to now 925k after three minutes. I literally cant do anything other than changing one thing about this layout, saving, closing, and relaunching in cycles. Its getting annoying. Whats up?

can you test with the latest version of scene builder 8.1.1?

I have tried this with Scene Builder 8.1.1. I attached the entire library, and it started off using 550k memory, and toggling any variable a few times rose it to 1.2m memory usage.
However, using nothing I get 150k with nothing on a new project view.

I will try this on a different computer, but I'd much rather work with it at my workstation.

this issue might be related to scene builder, you can open an issue using the following link:
https://bitbucket.org/gluon-oss/scenebuilder/issues/new


Plugin encountered an error

Terraform CLI and Provider Versions
I'm using TF Cloud version 1.6.3
Terraform Configuration
resource "snowflake_grant_privileges_to_role" "db_grants_to_fivetran_role" {
  provider = snowflake.securityadmin
  privileges = ["USAGE","MONITOR","CREATE SCHEMA"]
  role_name = snowflake_role.fivetran_role.name
  on_account_object {
    object_name = snowflake_database.fivetran_database.name
    object_type = "DATABASE"
  }
  with_grant_option = false
}

Expected Behavior
I expected TF/ the snowflake provider to create a db_grants_to_fivetran_role resource and apply the grants in the SF instance.
Actual Behavior
Error: Plugin did not respond
The plugin encountered an error, and failed to respond to the plugin6.(*GRPCProvider).ApplyResourceChange call. The plugin logs may contain more details.
Steps to Reproduce

terraform apply

How much impact is this issue causing?
High
Logs
No response
Additional Information
Error Log from provider:
Stack trace from the terraform-provider-snowflake_v0.79.0 plugin:
panic: runtime error: index out of range [1] with length 1
goroutine 101 [running]:
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk.NewDatabaseObjectIdentifierFromFullyQualifiedName({0xc0010d6090?, 0x1df9b46?})
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/sdk/identifier_helpers.go:159 +0xcc
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources.configureAccountRoleGrantPrivilegeOptions(0xc001296de0?, {0xc0012d3c10, 0x1, 0x1}, 0x0?, 0xc000afacf0)
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources/grant_privileges_to_role.go:748 +0x8f1
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources.CreateGrantPrivilegesToRole(0x0?, {0x1d3ff80?, 0xc0005b36c0?})
github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/resources/grant_privileges_to_role.go:460 +0x226
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).create(0x20b4e60?, {0x20b4e60?, 0xc0012c4810?}, 0xd?, {0x1d3ff80?, 0xc0005b36c0?})
github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/resource.go:766 +0x178
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*Resource).Apply(0xc00002a540, {0x20b4e60, 0xc0012c4810}, 0xc0012ccc30, 0xc0012bc300, {0x1d3ff80, 0xc0005b36c0})
github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/resource.go:909 +0xa7e
github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema.(*GRPCProviderServer).ApplyResourceChange(0xc00091cc18, {0x20b4e60?, 0xc0012c44b0?}, 0xc0012d6050)
github.com/hashicorp/terraform-plugin-sdk/v2@v2.29.0/helper/schema/grpc_provider.go:1060 +0xe8d
github.com/hashicorp/terraform-plugin-mux/tf5to6server.v5tov6Server.ApplyResourceChange({{0x20d4480?, 0xc00091cc18?}}, {0x20b4e60, 0xc0012c44b0}, 0x0?)
github.com/hashicorp/terraform-plugin-mux@v0.12.0/tf5to6server/tf5to6server.go:37 +0x5a
github.com/hashicorp/terraform-plugin-mux/tf6muxserver.(*muxServer).ApplyResourceChange(0x20b4db8?, {0x20b4e60?, 0xc00095f050?}, 0xc0013045f0)
github.com/hashicorp/terraform-plugin-mux@v0.12.0/tf6muxserver/mux_server_ApplyResourceChange.go:36 +0x1b5
github.com/hashicorp/terraform-plugin-go/tfprotov6/tf6server.(*server).ApplyResourceChange(0xc000927e00, {0x20b4e60?, 0xc00095e840?}, 0xc0009621c0)
github.com/hashicorp/terraform-plugin-go@v0.19.0/tfprotov6/tf6server/server.go:857 +0x574
github.com/hashicorp/terraform-plugin-go/tfprotov6/internal/tfplugin6._Provider_ApplyResourceChange_Handler({0x1ce79a0?, 0xc000927e00}, {0x20b4e60, 0xc00095e840}, 0xc000962150, 0x0)
github.com/hashicorp/terraform-plugin-go@v0.19.0/tfprotov6/internal/tfplugin6/tfplugin6_grpc.pb.go:467 +0x170
google.golang.org/grpc.(*Server).processUnaryRPC(0xc0002634a0, {0x20c4d00, 0xc000590680}, 0xc0011b4b40, 0xc000ac63f0, 0x2e01958, 0x0)
google.golang.org/grpc@v1.58.2/server.go:1376 +0xdd2
google.golang.org/grpc.(*Server).handleStream(0xc0002634a0, {0x20c4d00, 0xc000590680}, 0xc0011b4b40, 0x0)
google.golang.org/grpc@v1.58.2/server.go:1753 +0xa36
google.golang.org/grpc.(*Server).serveStreams.func1.1()
google.golang.org/grpc@v1.58.2/server.go:998 +0x98
created by google.golang.org/grpc.(*Server).serveStreams.func1
google.golang.org/grpc@v1.58.2/server.go:996 +0x18c
Error: The terraform-provider-snowflake_v0.79.0 plugin crashed!
This is always indicative of a bug within the plugin. It would be immensely
helpful if you could report the crash with the plugin's maintainers so that it
can be fixed. The output above should help diagnose the issue.

@adrian-montero-hkkd
Hey, from the error message, it seems that the error occurred in the resource that wanted to grant privileges on_schema, not on_object_account, and to resolve the error you have to pass the fully qualified schema name (probably somewhere in your config on_schema.schema_name = "<schema_name>", but should be on_schema.schema_name = "\"<database_name>\".\"<schema_name>\"").

I'm actually trying to do the grant at the db level. So that wouldn't work. Right now I'm using the previous snowflake_database_grant but I get the deprecated warning (but it does work, the permissions are granted successfully):
resource "snowflake_database_grant" "db_grant_usage_to_fivetran_role" {   provider = snowflake.securityadmin   database_name = snowflake_database.fivetran_database.name   privilege = "USAGE"   roles = [ snowflake_role.fivetran_role.name ]   enable_multiple_grants = true }
I was expecting it to work since it is an example just like the one in the provider docs:
`##################################
account object privileges
##################################
list of privileges
resource "snowflake_grant_privileges_to_role" "g3" {
privileges = ["CREATE", "MONITOR"]
role_name  = snowflake_role.r.name
on_account_object {
object_type = "DATABASE"
object_name = snowflake_database.d.name
}
}`
Ref: https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/grant_privileges_to_role
I would expect to have to provide a fully qualifier if I were granting the create table permission, not the create schema, which wouldn't make much sense to grant at a schema level, since well, I want the user to be able to create new schemas.
Thanks for the help.

@adrian-montero-hkkd
Hey, so maybe what I wrote is unclear, but the example and the code you provided look correct and they should work. What I was saying is that it looks like the error is pointing to another resource, not to the one you provided. The error looks to reference a "snowflake_grant_privileges_to_role" resource that uses on_schema instead of on_account_object like in your example.

Thanks. You are right, that was not the problematic block. Changed it for the fully qualified schema name and it worked.


Add tests for reverse_lazy for #858

Codecov Report

Merging #904 (49bf7ef) into master (b31af67) will increase coverage by 0.49%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##           master     #904      +/-   ##
==========================================
+ Coverage   99.39%   99.88%   +0.49%
==========================================
  Files          21       21
  Lines        1816     1806      -10
==========================================
- Hits         1805     1804       -1
+ Misses         11        2       -9

see 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@pedro-psb It _definitely doesn't seem composable. I made the following change in `settings.yaml
  ADMIN_NAMESPACE: admin
  LOGIN_URL: "@reverse_lazy @format {this.ADMIN_NAMESPACE}:login"

And it blew up
TypeError: descriptor 'format' for 'str' objects doesn't apply to a 'Lazy' object

If I try to change the order and use @format @reverse_lazy, then the @reverse_lazy value just gets treated as raw text.
I'm open to any suggestions, otherwise we just call this a "noncomposable" parser? ¯\_(ツ)_/¯

@wgordon17 I'll try something out tomorrow, I don't know much about the converters' mechanism yet.
But yes, although it would be better to provide a composable example, it is already very useful to have a non-composable one.

Sounds good @pedro-psb, let me know if I can help. Ideally, I wouldn't have to use Lazy at all, but I can't seem to figure out a way past early-evalution. Since reversing Django routes relies on a fully configured project, but you can't configure the project without settings being fully setup, and the settings can't get setup because the route is trying to be reversed...and around and around we go 😉
For me, https://github.com/dynaconf/dynaconf/blob/master/dynaconf/base.py#L894, was the offending line that was causing early value evaluation breaking the "laziness" of reverse_lazy. But wrapping it in a Lazy object seemed to fix the early evaluation issue. Perhaps there's a different way to wrap the parser/casting to still "act" like a str while still delaying evaluation ¯\_(ツ)_/¯

@wgordon17 I've found out that the caller of the converters (get_converters) optionally passes the box_settings to the converter functions.
Then, you can use it to evaluate the value if it got there as a Lazy object (because @format needs to be Lazy too).
def reverse_lazy_fun(value, box_settings=None):
    if isinstance(value, Lazy):
        # value got here as a Lazy object, needs evaluation
        return Lazy(value(box_settings), casting=reverse_lazy)
    else:
        # value is good to go
        return Lazy(value, casting=reverse_lazy)

parse_conf.converters["@reverse_lazy"] = reverse_lazy_fun

@pedro-psb I think we must add a builtin helper to register new converters then we abstract this logic
def reverse_lazy_fun(value, box_settings=None):
    if isinstance(value, Lazy):
        # value got here as a Lazy object, needs evaluation
        return Lazy(value(box_settings), casting=reverse_lazy)
    else:
        # value is good to go
        return Lazy(value, casting=reverse_lazy)

In something better to use like
settings.dynaconf.add_converter("reverse_lazy", callable, combo=True)

@rochacbruno Yes, I think this would be good

This is what I was playing with @pedro-psb
I will add to a PR later today, need to test with more scenarios.
Also I am not sure if keep it as a standalone function or bind it to the settings as a method

@rochacbruno I was definitely not being elegant, but I just threw this together as well https://github.com/dynaconf/dynaconf/pull/904/files#diff-81216dc760c0d7472d00bab2b35a37d32985b9b0e07ffa9877ed2bbb664373c1

@wgordon17 do we really need to pass box_settings ? maybe your implementation is better

@rochacbruno Heh, I really don't know 🙈 I only stole it from @pedro-psb's suggestion. I think you're the SME on the backend internals

I think add_converter should be a module function, as it is modifying a module variable.
It could live in this same file (parse_conf.py), but as it is a internals module, it would be better to have a public utils or similar namespace just for users, so their autocompletion don't get crowded with plumbing stuff.

then on dynaconf/__init__.py we can do
from dynaconf.utils.parse_conf import add_converter

__all__ = [
    ...
   "add_converter"
]

then the usage would be
from dynaconf import Dynaconf, add_converter

settings = Dynaconf()
add_converter("key", function)

Rebased and refactored with suggestions, thanks so much @pedro-psb @rochacbruno! Also included a quick change to the docs. Once you're good with this, I can squash all the commits down (or GitHub can do it ¯\_(ツ)_/¯)


Guide how to set up Heroku + Hasura Cloud deployment

Compose a page from this thread: https://discord.com/channels/846362414039695391/846418873233571840/1008797918104408084

Recent changes: https://hasura.io/docs/latest/projects/heroku-url-sync/

Heroku free instances are gone


Change the command "Cosmos DB: Detach" to "Azure Databases: Detach"

OS: All
Build Version: 0.22.0
Regression: Not a regression
Repro Steps:
F1->Check the command "Detach"
Expect:
This command shows "Azure Databases: Detach".
Actual:
This command shows "Cosmos DB: Detach"

More Info:
This issue also reproduces for command "Cosmos DB: Attach Database Account".

This bug also reproduces on the latest main branch build 20241126.1, reopen it.
The commands are "Cosmos DB: Attach Database Account..." and "Cosmos DB: Detach Database Account..." in Command Palette now.
We have the below suggestions because that we can attach and detach both Cosmos DB accounts and PostgreSQL servers with the above 2 commands, not just include Cosmos DB accounts.
Suggestions:

Change the attach command to "Azure Databases: Attach Database Account...".
Change the detach command to "Azure Databases: Detach Database Account". (Remove "..." from the current detach command, because the connection can be detached directly without any confirmation dialog.)