Node.js 16 deprecation warning message

Hi,
I ran an action using setup-fpm@v5 and got the following warning:

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: fortran-lang/setup-fpm@v5. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Have you seen this before? should I use a different version of the setup-fpm?
Thanks,

Still a problem today:

The following actions use a deprecated Node.js version and will be forced to run on node20: fortran-lang/setup-fpm@v5. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/

Looks to me like it has been update here and all we need is a new release? 🤔 @gnikit

Will get to it today, thanks for the ping

Hi @gnikit - New v6 appears to be broken?
From https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431 :
Run fortran-lang/setup-fpm@v6
  with:
    github-token: ***
    fpm-version: latest
    fpm-repository: https://github.com/fortran-lang/fpm
  env:
    FC: gfortran
    GCC_V: 12
node:internal/modules/cjs/loader:1148
  throw err;
  ^

Error: Cannot find module '@actions/core'
Require stack:
- /home/runner/work/_actions/fortran-lang/setup-fpm/v6/index.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1145:15)
    at Module._load (node:internal/modules/cjs/loader:986:27)
    at Module.require (node:internal/modules/cjs/loader:12[3](https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431#step:3:3)3:19)
    at require (node:internal/modules/helpers:179:18)
    at Object.<anonymous> (/home/runner/work/_actions/fortran-lang/setup-fpm/v6/index.js:1:1[4](https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431#step:3:4))
    at Module._compile (node:internal/modules/cjs/loader:13[5](https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431#step:3:5)8:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:141[6](https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431#step:3:6):10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:1[7](https://github.com/bonachea/assert/actions/runs/10908956755/job/30276101431#step:3:7)4:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/home/runner/work/_actions/fortran-lang/setup-fpm/v6/index.js' ]
}

Will have a look thanks

I am pushing a fix now. It turns out that node_modules need to be committed and do not get added during the release process. We should probably find a smarter way of doing this since I don't like shipping all these things manually.

@bonachea I just release v6.0.1 hopefully fixing this. Replacing the setup-fpm for my fpm fork seems to have worked. I am closing this Issue. Let's open a new one unrelated to node 16 if the problem persists.


Added Mobile Web/App Tests to SC Quickstart

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[ ] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hi @lauramthomas , I've documented how to utilize Sauce Connect Proxy with Live Testing for Mobile and Web Apps. Can you please review? Per Meredith, we should also cross-reference it from Live Testing documentation.


leukemia is a "musculoskeletal system disorder"

When adding non-human diseases, we realized that leukemia is classified as a "musculoskeletal system disorder", which is weird (and Katie and I think it shouldn't be).
This classification comes from the fact that leukemia has the SC:
'disease has location' some 'bone marrow'
and

bone marrow : 'has part' some 'bone element'
'bone element' is part of 'skeletal system'

@nicolevasilevsky can we please discuss how we should resolve this? I think I should just remove the SC, but I don't think that there is a "excluded subclass of" equivalent for axiom.

I agree that the best approach is to remove the subclassOf axiom.


Add support to get channel lifecycle status

Add support to get channel lifecycle status
Channel lifecycle status is described at Channel status / REST / [Docs]
It is currently possible to retrieve channel lifecycle status in ably-go by using REST.Request and marshalling the result into a slice of empty interface.
Example:
result, _ := restClient.Request("get", "/channels/test").Pages(context.Background())

result.Next(ctx)

var items []interface{}
if err := result.Items(&items); err != nil {
	fmt.Println(err)
}

fmt.Printf("%+v\n", items)
// [map[channelId:test name:test status:map[isActive:true occupancy:map[metrics:map[connections:1 presenceConnections:1 presenceMembers:1 presenceSubscribers:1 publishers:1 subscribers:1]]]]]

Out of Scope
Realtime subscription to Channel Lifecycle Events. This ticket is only for supporting REST client ‘Get’.
┆Issue is synchronized with this Jira Uncategorised by Unito

Blocked by https://github.com/ably/ably-cocoa/issues/1415 and https://github.com/ably/ably-java/issues/769


Resolves an issue where querying 100+ fields on a relationship causes a pg error

What kind of change does this PR introduce?
Resolves #551. Applies the same fix as #390, but for relational queries.
What is the current behavior?
See #551. Current behavior is querying a related entity with 100 fields breaks.
What is the new behavior?
Transpilation works and results are returned.
Additional context
I'm not yet familiar with Rust, so please use additional scrutiny.
While this would certainly unblock my team in the short-term, it seems like this issue indicates an area where a concerted refactor like recursively handling the to_sql operation may be more appropriate in the long run to avoid similar issues in the future.

Sorry for the delay, I'll get this reviewed before EOW. Looks great at a glance!

No worries, thanks for keeping up on it!


2048 - Keystokes don't change board

I'm trying out the 2048 example. I was able to build it successfully using npm install without any errors, but I can't get it to do anything besides show a static board. None of the keystrokes work. Also I get a warning for RCTImageView Does Not Exist.
Running the iOS version on Mac OS, Node v5.7, React v0.20

the game isn't set up for keystrokes; it's a mobile app :D try tapping on the board.

Ah thanks @jaredly ! After giving it another try, I found out the problem was that I wasn't dragging with the simulator and instead just tapping.


Switch to GitHub Action CI for Linux and Windows builds

Notes:

I didn't limit the action on master only so that it works on other branches too
After this gets merged, you can probably remove any AppVeyor hooks/integrations, assuming you don't plan on using it again

Thanks a lot for this :100:


Crash on opening files

Currently, when you open a file with syntax highlighting (or it may be the language server?) we crash and don't recover.
I've tested JSON files and Reason files and in both you get no syntax highlights at all, and then crash moments later. This is on the latest master, macOS Catalina, node 13 and esy 0.6.0.
I'm nipping out for a bit, but I can do some bisection to work out what commit caused it since it happens on the released builds and the dev builds so its not a release artifact issue.

Strange! I just tested on Catalina (the latest release, and the downloaded from master), and am not reproducing it.
It's interesting that it reproduces for both the released builds and dev builds. I wonder if there is a bug with node 13 and we happen to be jumping to the system one instead of the vendored version.
Some things that might help track it down:

The log file (preferably with ONI2_DEBUG=1)
The results of --checkhealth
Any custom configuration

I'll also add some CLI commands to:

Disable syntax highlighting (--no-syntax-highlighting)
Disable extensions (--no-extensions)
Disable configuration file (--no-configuration)

so that we can at least narrow down whether it is syntax highlighting vs extension host, or possibly configuration.

I actually only get this behaviour in my dev build, though someone on discord had mentioned syntax was dead on master in the release build. Weirdly for me, the dev build doesn't work and the release build does.
Check health is as follows:
┌─[16:49:40 MacBook (master $) ~/git/oni2]
└─╼ esy run run -f --checkhealth

[DEBUG]    +2ms Global : Setup: Looking for setupJson at: /Users/ryanc/git/oni2/_esy/default/store/i/oni2-5ca89004/bin/setup.json
[INFO]     +0ms Global : RUNNING CHECK: Verify camomile:datadir
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify camomile:localedir
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify camomile:charmapdir
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify camomile:unimapdir
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify node executable
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify node executable can execute simple script
[INFO]   +111ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify node dependencies
[INFO]   +131ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify ripgrep (rg) executable
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify bundled extensions path exists
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify bundled font exists
[INFO]     +0ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify bundled reason-language-server executable
[INFO]    +24ms Global :  -- RESULT: PASS
[INFO]     +0ms Global : RUNNING CHECK: Verify bundled syntax server
[INFO]     +0ms Oni2.SyntaxClient : Starting executable: /Users/ryanc/git/oni2/_esy/default/store/i/oni2-5ca89004/bin/Oni2_editor
[INFO]     +1ms Oni2.SyntaxClient : started syntax client
[INFO]    +61ms Oni2.SyntaxServer : Starting up server. Parent PID is: 32256
[INFO]     +0ms Oni2.SyntaxServer : Waiting for incoming message...
[INFO]     +0ms Oni2.SyntaxServer : Initialized!
[INFO]     +0ms Oni2.SyntaxServer : No pending work.
[INFO]     +0ms Oni2.SyntaxClient : Tokens applied
[INFO]     +0ms Oni2.SyntaxServer : Token updates sent.
[INFO]     +0ms Oni2.SyntaxServer : Waiting for incoming message...
[INFO]   +185ms Oni2.SyntaxServer : Closing
[ERROR]    +2ms Oni2.SyntaxClient : Syntax process closed with exit code: 32261
[INFO]   +271ms Global :  -- RESULT: PASS
[INFO]     +0ms Global :
[INFO]     +0ms Global : ** PASSED **
[INFO]     +0ms Global :
[INFO]     +0ms Global : All systems go.

(Not sure why the error and then the pass?)
Log File: https://gist.github.com/CrossR/1ff0fc350a19119795d870905ddfd488
This is for esy run run and then just Cmd-P and opening Oni2.re. I scroll down a few lines and then it crashes.
Both of these were with a rm -rf _esy and node install-node-deps.js beforehand.

Thanks for the details, @CrossR !

I actually only get this behaviour in my dev build

Do you happen to have any local changes - like any resolutions or other changes?
For the crash:

Does Onivim 2 instantly 'disappear'? Or does a 'beach ball' come up for a second? If it's the former - that suggests a crash on the OCaml side. If it's the latter - that suggests a crash in the native code.

I opened #1100 to add some CLI options to disable syntax highlighting / extensions as a way to rule out some particular issues.

Do you happen to have any local changes - like any resolutions or other changes?

Nope this is with a clean build of Oni2 with no changes. I could try and rm -rf _esy ~/.esy && esy as I've currently only removed the local cache dir. I could also try a full re-clone.
I'm getting a beach-ball and then a hang, where it seems to wait indefinately. I'm assuming its the syntax highlighting that isn't happy, since I get no highlights in either Reason or JSON files, which should both have it.

Interesting... I do see this show up in your log file:
[ERROR] Oni2.SyntaxClient Syntax process closed with exit code: 33240

Something is indeed causing the syntax server process to crash / close

Based on the log, it seems to be crashing in between these log messages:
            log("Running unit of work...");
            map(State.doPendingWork);
            log("Unit of work completed.");

since we see "Running unit of work..." but not "Unit of work completed."

One thing that might help - which arguments are you specifying when you run the build, @CrossR ? Which arguments / environment variables do you specify?
Might help me be able to repro it (we should also log these out... should be able to get these from the log)

One thing that might help - which arguments are you specifying when you run the build, @CrossR ? Which arguments / environment variables do you specify?

I build with esy/esy bootstrap and run with esy run run -f and I've got the ONI2_DEBUG and ONI2_LOG_FILE vars set.

Thanks @CrossR ! Does it reproduce without the ONI2_DEBUG and ONI2_LOG_FILE vars set?
I'm thinking that it might be an issue where the syntax server process is logging out to the same file as the editor process.

I'm still getting this crash on master unfortunately.
The crash consistently happens on line 16/17 (of multiple files), and seems to come after a Protocol.ClientToServer.VisibleRangesChanged is sent. I then just hang there. I don't necessarily think that notification is the cause, as its sent many times before reaching line 16/17.
Adding additional logs around that (i.e. to confirm it send and to confirm it is picked up) has it fail here:
https://github.com/onivim/oni2/blob/447df264f4d52adab7aaf498fb7dd33c2e5f2869/src/Syntax_Client/Oni_Syntax_Client.re#L27
A "Starting flush" log is printed, but no "Finished flush" after. Again though, this passes many times before it, so not sure what happens after 17 updates. That is, it happens consistently after 16/17 updates so 17 j or a mix of j/ks etc.
I tested again with the new flags and it works with either disabling the syntax highlights or the extension host. Could this be due to a conflict between the two running processes? Its also somewhat odd that when I Ctrl-c to stop the editor when running with -f the final bits of the log are as follows:
[DEBUG]    +0ms Oni2.StoreThread : Running effect: Batch:
 syntax.visibilityChange

[INFO]     +0ms Oni2.SyntaxClient : Sending visibleRangesChanged notification...
[INFO]     +0ms Oni2.SyntaxClient : Starting write...
[INFO]     +0ms Oni2.SyntaxClient : Making channel...
[INFO]     +0ms Oni2.SyntaxClient : Starting flush...
[DEBUG]    +0ms Global : [69561|bootstrap-fork.js]: %cTRACE color: #888 [nullExtensionDescription] INVOKE provider 'DefinitionAdapter'
[DEBUG]    +1ms Global : [69561|bootstrap-fork.js]: %cTRACE color: #888 [nullExtensionDescription] INVOKE provider 'DocumentHighlightAdapter'
[DEBUG] +1415ms Global : [69561|bootstrap-fork.js]: %cTRACE color: #888 [nullExtensionDescription] provider DONE after 2157ms
^C
[INFO]  +93.65s Oni2.ExtensionClientStoreConnector : ext host closed

Which implies we are in the extension client (I think, I'd normally get the revery can quit log out on a ctrl-c).
https://gist.github.com/CrossR/285feb1093490540a00543b2b52ee661 - Full new log from clean master build and just run with esy run run -f. Are the bootstrap-fork.js errors anything to worry about either?

A "Starting flush" log is printed, but no "Finished flush" after.

Good observation! The reason this is hanging, though, is because earlier - the syntax highlighting service crashed earlier:
1703 [ERROR] Oni2.SyntaxClient Syntax process stopped with signal: -10

The last log we see from the syntax server is:
[INFO] Oni2.SyntaxServer Grammar loaded successfully

...so something is crashing it after.
Unfortunately, I don't think we have any more clues on why this process is crashing. I'll open a PR for you to run where maybe we can get some more fine-grained logging to get clues about this in particular.

Again though, this passes many times before it, so not sure what happens after 17 updates. That is, it happens consistently after 16/17 updates so 17 j or a mix of j/ks etc.

This is really interesting... it sounds like maybe some buffer is getting filled up.

I tested again with the new flags and it works with either disabling the syntax highlights or the extension host. Could this be due to a conflict between the two running processes

Also interesting... it could be possible, but I'm not sure what conflict would cause this. Some sort of deadlock?
The way the process exited was a Unix signal - I'm not sure what -10 corresponds to, though!  There's a list of signals here: https://caml.inria.fr/pub/docs/manual-ocaml/libref/Sys.html (but it doesn't have the numeric values...). It'd be useful to know which signal is aborting the process. There are also OCaml utilities for handling signals like this
A few more points of clarification:

Does this reproduce for you in the packaged builds?
Does this reproduce for you in a build you package yourself (ie, esy create-release)?
Which terminal do you use on OSX?
Was there any progress on git bisect'ing for us to know the 'last known good state'?

One other idea - it would be helpful to see if this reproduces via an integration test.
We could create a new test like 'RegressionTest1097', based on: https://github.com/onivim/oni2/blob/master/integration_test/SyntaxHighlightTextMateTest.re - but  do a bunch of key updates prior to the verification of syntax highlights:
input("j");
input("k");

in a loop. In theory... that should reproduce it (and it would be interesting to see if it manifests on CI, or in my machines).

Does this reproduce for you in the packaged builds?

Nope! The packaged builds do work for me with syntax highlights.

Does this reproduce for you in a build you package yourself (ie, esy create-release)?

It does repro in builds I make myself, implying my build is going wrong somewhere.

Which terminal do you use on OSX?

I use kitty normally, though it reproduces with Terminal as well. Both use bash from brew.

Was there any progress on git bisect'ing for us to know the 'last known good state'?

I did test some bisection all the way back to the async additions, and it repros before that. I'm testing a pretty far back commit now, just to check that its not something very wrong with my machine.

Interesting!
I suspect now that there is something broken / corrupted in reason-textmate or oniguruma libraries. I'm going to add health-checks for those libs, and some extra logging in the textmate tokenizer...

I suspect now that there is something broken / corrupted in reason-textmate or oniguruma libraries.

That seems reasonable! A quick test of esy @test run over in reason-textmate does fail for me:
┌─[17:39:41 MacBook (master) ~/git/reason-textmate]
└─╼ esy @test run
Running 9 test suites
 PASS  Trie
 PASS  TokenTheme
 PASS  ThemeScopes
 RUNS  OneDark============ BEGIN ===============
============ END ===============
 PASS  OneDark
 PASS  RegExpFactory
 PASS  Pattern
 RUNS  Grammar┌─[17:39:45 MacBook (master) ~/git/reason-textmate]
└─╼

Seems to crash over in Grammar.

Interesting - how about this lib?
https://github.com/onivim/reason-oniguruma

From Discord - @CrossR narrowed it down to an issue with oniguruma - likely related to this one:
https://github.com/kkos/oniguruma/issues/159
https://discordapp.com/channels/417774914645262338/618478669282934806/660594843059355658

Was fixed with bringing in the new version of oniguruma.

Thanks for the great investigation + fix, @CrossR !


Allow spaces in "--prefix" path

Fixes #4668 (on unix-like platforms)
Update the unix Makefile template and some c sources to handle paths with spaces.
Testing:
rm -rf "$HOME/tmp/beforespace afterspace"
./Configure -Werror --strict-warnings --prefix="$HOME/tmp/beforespace afterspace"
make -j6 update
make -j6
make install
make test

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

CI is relevant.

I'm not sure I understand what this actually achieves...  but, it has an issue too: it assumes that GNU make is used, something we haven't seen as acceptable before, because some places will simply not allow frivolous installation of extra stuff (some places consider GNU make installation just to build another project to be frivolous).

:= is a GNU make construct.

:= is a GNU make operator.

Thanks for pointing that out.  I did not realize that GNU make was not a prerequisite on unix-like platforms.
In INSTALL.md it says only that a "make" implementation is needed (and there is nothing more specific mentioned in NOTES-UNIX.md).

Also, if you want to allow installing paths with spaces in them, it seems more straightforward to fiddle with the installing commands

Thanks, I will give that a try.
It is has been more than a month since I opened this PR and I cannot remember if I had investigated that originally.

the *BSDs have their own makes that are not GNU make.

I believe BSD make supports :=.
https://man.freebsd.org/cgi/man.cgi?make(1)

When looking at make features, I've looked back as far as the Unix v7 manual, and kept to what that allows.  That's the common ancestor.  And we really cannot know what someone will actually run with their make command.
Just so we know our different baselines.

the "cla-check" failure is due to the use of my new email address.

Just a nit about the commit message - the subject (the first line) should be Update the unix Makefile template to handle paths with spaces. and the Fixes ... line should be in the body of the message. We can fix that when merging or you can amend the message right now.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Is there a reason why we shouldn't cherry-pick this back to 3.1 and 3.0?

Is there a reason why we shouldn't cherry-pick this back to 3.1 and 3.0?

I am OK with it going to 3.x branches.

IMO this is rather a bug fix.

The merge to openssl-3.1 and openssl-3.0 are non-trivial. @jamuir could you create a PR against openssl-3.1?
Thanks!

@tmshort :  please see #22386


Broken installation with v1.16.0

Describe the bug:

I tried updating cert-manager to version 1.16.0 and the Helm Chart Version 1.16.0 and since then, I am no longer able to deploy the application as I get the following errors:
Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook: Post "https://cert-manager-webhook.cert-manager.svc:443/validate?timeout=30s": context deadline exceeded

I already set the global.leaderElection.namespace to cert-manager as I thought this might fix the issue, but the issue still remains.
Expected behaviour:

I expect the application to deploy just fine
Steps to reproduce the bug:

Anything else we need to know?:
Environment details::

Kubernetes version: 1.31
Cloud-provider/provisioner: Openstack
cert-manager version: 1.16.0
Install method: Helm

/kind bug

Not sure if this has something to do with Kubernetes Version 1.31. No matter what version I try to install, I always get the same error. I already tried v1.15.3 and 1.14.7.

Okay, I fixed it by setting hostNetwork: true for the webhook but thats not really ideal right?
I also noticed when running a temporary pod via kubectl run dns-test --rm -ti --image=busybox --restart=Never -- sh to check the networking via nslookup cert-manager-webhook.cert-manager.svc I couldn't resolve the hostname but using nslookup cert-manager-webhook.cert-manager.svc.cluster.local it worked. However, looking at the error above, the pod uses https://cert-manager-webhook.cert-manager.svc:443 and this is the reason why the hostname cannot be resolved.

this is supported --dns01-recursive-nameservers-only and --dns01-recursive-nameservers=<IP_ADDRESS>:53\,<IP_ADDRESS>:53 in v16.0 certmanager version ? its mandatory or not ?

@jayantverma11 sorry I am not sure about what you mean?

@amrap030 Setting hostNetwork: true is our suggested work around for various on-prem clusters and certain CNI configurations:

https://cert-manager.io/docs/troubleshooting/webhook/#error-io-timeout-connectivity-issue

You say that this problem occurred only after you upgraded cert-manager,
but judging by your subsequent comments, the problem continued even when you tried downgrading to cert-manager 1.15 and 1.14.
So perhaps the act of upgrading cert-manager has caused the webhook Pod to be scheduled to a node which the Kubernetes API server is unable to reach. Perhaps due to some quirk of your CNI.
We run regular E2E tests with cert-manager on Kubernetes 1.31 and those are all passing.

https://testgrid.k8s.io/cert-manager-periodics-release-1.16#ci-cert-manager-release-1.16-e2e-v1-31
https://testgrid.k8s.io/cert-manager-periodics-release-1.16#ci-cert-manager-release-1.16-e2e-v1-31-bestpractice-install
https://testgrid.k8s.io/cert-manager-periodics-release-1.16#ci-cert-manager-release-1.16-e2e-v1-31-upgrade

I can only assume that the "context deadline exceeded" error was caused by some peculiarity of your Kubernetes + OpenStack infrastructure. It's a very opaque error which users often report. Read more here:

https://cert-manager.io/docs/troubleshooting/webhook/#context-deadline-exceeded

If you feel have further evidence to suggest that this is a bug in cert-manager, please re-open.

It is really frustrating, I have no idea why my cert-manager won't get deployed correctly. I thought I fixed cert-manager by setting hostNetwork: true but I am now always receiving the following error:
Internal error occurred: failed calling webhook "webhook.cert-manager.io": failed to call webhook: Post "https://cert-manager-webhook.cert-manager.svc:443/validate?timeout=30s": Forbidden

I tried again updating to the latest helm version but without success. Please help me to fix this.


Retry strategies

Hi
Do you plan to implement retries? Do you have a roadmap that lists all future features?

KubeMQ is a message broker, this kind of functionality should be done by the service that uses the message broker.
Have a look on KubeMQ Targets / Sources / Bridges retries middleware for example


[MOB-1326] add trackEvent

Sorry that the description of this jira issue needs to be changed.
Since, unlike Obj C and Swift, Javascript goes only by the method name and not the param names, we should name the methods as trackEvent and not track.
I think in the future we should also look at renaming and deprecating our SDK. But that is for later.


Can not get mjml-react to work with Storybook

I have been trying to get mcml-react (v3) to work nicely with storybook.
An example project, with a fairly clear readme, can be found here: https://github.com/Bertg/mjml-react-storybook-example
The basic issue is that the mjml-react and its dependancies seem to try and use the fs package, which is not available in the storybook runtime.
I create this ticket as a followup of the conversation with @IanEdington from last November.

Hi @Bertg ! If the issue is that storybook is looking for the fs package you should be able to just add a fallback for webpack >5 using the webpackFinal option in the storybook config https://storybook.js.org/docs/react/configure/overview.
  webpackFinal: async (config, { configType }) => {
    // Make whatever fine-grained changes you need
    // Return the altered config
    return {...config, resolve: {...config.resolve, fallback: {fs: false}}};
  },

The other thing to consider is the alias of mjml to mjml-browser.
  webpackFinal: async (config, { configType }) => {
    // Make whatever fine-grained changes you need
    // Return the altered config
    return {
      ...config,
      resolve: {
        ...config.resolve,
        alias: {mjml: "mjml-browser"},
        fallback: { fs: false },
      };
    };
  },

Ok, ok I'm getting somewhere.
I still can use @faire/mjml-react/utils/render. It loads otters libraries like ehtml-minifier, uglify-js , clean-css etc... which seem to be beaking the setup.
However, I did get it to work using @faire/mjml-react/utils/renderToMjml and mjml2html from mjml-browser. (I have not pushed this).
It's not super clean, but could be neatly wrapped up into a Readme, or even a dedicated package.

Ah yes, we do also alias uglify-js like so:
const extraAlias = {
  path: path.resolve(__dirname, "browser-mocks/path.js"),
  mjml: "mjml-browser",
  "uglify-js": path.resolve(__dirname, "browser-mocks/uglify-js.js"),
};

where the uglify-js.js file is just
module.exports = {};

I think the remaining errors come from htmlMinify using other libraries that break the storybook setup. Perhaps the render function from @faire/mjml-react/utils/render should allow for customizing the htmlMinify configuration as well. We didn't initially look into this much when taking ownership of maintaining mjml-react. We recommend writing your own render function that suits your needs but wanted to keep this one in as an example.

👋 Hi @Bertg, Thanks for following up! We would welcome a "getting started with storybook" contribution to the yet to be created docs directory.
@emmclaughlin I wonder if we should just remove html minifier from v4. I know mjml has indicated it wants to move away from minification and it seems like overkill for this library.

A bit of a crazy suggestion, but could this project depend on mjml-browser instead of mjml?
Are there huge benefits or disadvantages in this choice?
The mjml-browser plugin mentions 2:

mj-include tags are unavailable and will be ignored.
features involving the .mjmlconfig file are unavailable, which means no custom components.

I think both points are mostly covered by the fact that this library uses react. We can make out own react based components, same goes for mj-include.

We currently only depend on mjml in the render function:
https://github.com/Faire/mjml-react/blob/e8befa5575d62fb3914a1dfc92137dedbf733afb/src/utils/render.ts
I think we could remove mjml as a dependency, requiring people to install mjml seperately to use the render function but we can't change the render function to depend on mjml since that will silently break any emails people have written that have the include tag.
It seems like your problem is with the render function.
Could you try using this file as a substitute to your render function? I think something like this should work even though you have "mjml" installed as a dependency since it won't get imported anywhere in the browser context.
browser-renderer.ts
import mjml2html from "mjml-browser";
import { renderToMjml } from "@faire/mjml-react/utils/renderToMjml"; // might have to tweak this
import { MJMLParsingOptions, MJMLJsonObject, MJMLParseError } from "mjml-core";
import React from "react";

interface ConvertedHtml {
  html: string;
  json?: MJMLJsonObject;
  errors?: MJMLParseError[];
}

export function render(
  email: React.ReactElement,
  options: MJMLParsingOptions = {}
): ConvertedHtml {
  const defaults: MJMLParsingOptions = {
    keepComments: false,
    beautify: false,
    validationLevel: "strict",
  };

  const parseResults: ConvertedHtml = mjml2html(renderToMjml(email), {
    ...defaults,
    ...options,
  });

  return parseResults;
}

@IanEdington I was suggesting that as a v4 suggestion.

Agree with the above discussion around removing mjml dependency in v4. That is, the main goal of this project is to be able to get the benefits mjml provides for writing emails while still developing in a react environment. With that in mind, our focus should stick to going from react -> mjml. Going from mjml -> html can be left up to the developer to implement in a way that suits their needs.

@Bertg were you able to get storybook working? Would you be willing to share what you had to do to get it working?

@IanEdington Yeah, absolutely. I'm just kinda swamped with work the last 2 weeks, so progress is slow to "finish" it.
@emmclaughlin I understand your argument, but I'm looking at this from an other perspective. As technical team lead I'm looking at the building blocks that I need to send out emails, and make those choices ideal for my team.
So in case of sending emails, I choose nodemailer to send the emails, that's pretty clear. And I'm looking for a tool so that my team with React experience can design and implement the emails. I want as little hassle as possible to make this work.
If this project would make me choose "how" the react mjml is converted to the actual HTML, I would be far less inclined to use it. To me this would be similar to react saying the entire react-dom responsibility lays with the developer.
Sure the developer should be able to "change" the mjml -> html implementation, but this project needs a good default react -> html option built in.
To be even a bit more extreme, I would say that this project should do react -> [html, title] to be exact. Mjml allows you to define a mjml-title attribute, and right now, it's a bit of a hassle to extract that from the HTML or the JSON structure.

@IanEdington Ok, I pushed my working version here: https://github.com/Bertg/mjml-react-storybook-example
Any feedback on the code or the readme would be greatly appreciated :)

cool thanks


Plugin shows Please sign in to Choreo to view performance predictions message for each and every keystroke

Description:
If we have a service, for each and every keystroke, the following info is shown, which is disrupting the user's flow.
Consider the following source capture as well,
https://user-images.githubusercontent.com/1329674/139789864-f73bf86e-a12a-46fe-b4a9-23bbd1f58560.mp4

We can add an option such as 'Don't show this message' to the popup notification.

We can add an option such as 'Don't show this message' to the popup notification so that the user can turn it off.

I think the problem here is that we are prompting this via an info message and even if we have/provide an option for the user to turn it off there is a chance of ignoring it. IMHO,

In the first place this should not appear via show message. Even if we choose it, it should be a one time thing.
This kind of information is usually shown to the user through the status bar, as a permanent message/ error/ warning until user attend and resolve the issue. (Consider how windows office shows the activation expiration status)

Suggestions,

Utilise the status bar to show the error with a notifiable colour.
Use the message in combination with the status Bar item. We show the message one time, and then show the status through the status bar item.


Personabar users/pages error retrieving

Description of bug
In the peronabar if selecting users or pages the error shows 'There was an error retrieving your content'
Steps to reproduce
List the steps to reproduce the behavior:

Go to 'personabar' logged in as superuser or admin
Click on 'pages'
Error shows

Current result
Tested a bit and seems that on new site this is not shown because there is only one or none user present. Error shows only with multiple items. This shows also in other lists of the personabar functions as eq. site settings - 'Aliases' tab and the 'Recycle bin' tabs if multiple items are present.
Screenshots

Error log
In Chrome inspect - Failed to load resource: the server responded with a status of 500 (Internal Server Error) - /API/PersonaBar/Pages/GetPageList?searchKey=
No error log because java?
Affected version

[x] 9.2.0 but was also present in earlier versions (upgraded but did not improve, that's why I report it now)

Affected browser
Tested with Chrome and Internet Explorer

I can't reproduce this on 9.2.2 and haven't seen it on earlier version. Have you checked the Event Log for more information on the error? Since it's a 500, the error occurs on the server and should have a log, either in the logfile or in the event log.

The only errors I found where in the serverlog (in the _default/logs folder) and I can't see if it is related.
2018-10-24 15:06:01,904 [xxx][Thread:25][FATAL] DotNetNuke.Framework.PageBase - An error has occurred while loading page.
System.MissingMethodException: Method not found: 'System.String DotNetNuke.Entities.Modules.PortalModuleBase.get_ModulePath()'.
at DotNetNuke.Modules.Admin.Users.Register2.OnLoad(EventArgs e)
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)
2018-10-24 15:06:01,904 [xxx][Thread:25][ERROR] DotNetNuke.Services.Exceptions.Exceptions - ~/Default.aspx?tabid=2232&error=An unexpected error has occurred
System.MissingMethodException: Method not found: 'System.String DotNetNuke.Entities.Modules.PortalModuleBase.get_ModulePath()'.
at DotNetNuke.Modules.Admin.Users.Register2.OnLoad(EventArgs e)
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)

I also upgraded DNN Platform site from 8.0.4 to 9.2.1 version and getting same issues. Is there any one find solutions for this ? Getting same issue while opening Users page.
Added same issue in detail to DNN forum as well
https://www.dnnsoftware.com/answers/after-upgraded-site-to-9-21-pages-users-recyclebin-page-are-not-working

Ok, can you go into the DotNetNuke.log4net.config file and change the level from Error to All. Then go to your _portals\default\logs delete the file from today, do your thing and check if you get more interesting logs. After that, restore the config to Error or you will have huge log files.
I have not had this issue in 9.1.1, 9.2.2 and 9.3.0 RCs.

I am closing this issue due to lack of response since feb 28 and that we cannot reproduce. I would suggest doing a folder compare between your site and a clean site, maybe there is some extra dlls or something local that is causing this issue for this site.


Fix/instance edit

MANAGER-3020

@jleveugle Please pay attention! This pull request shouldn't have been "Squash and merge".
We were lucky because some fix's commits are already in the develop branch which relies on the @ovh-ux/manager-pci module.
Without theses commits we weren't be able to provide a new patch for the expected module.


[QUESTION]: v2.0 fully unsupported due to lack of remote server?

Consent

[X] I have searched the existing issues and verified that there is no open issue for the same subject.
[X] I understand that the YTMDesktop Elgato Plugin has NO affiliation with Google, YouTube or the YTMDesktop application.

Question
Hi there! I am asking this here because I assume you are very much aware of this -- the latest version of YTMDesktop doesn't seem to have a functional remote server, and with the older versions being deprecated that means this plugin is unable to be used, right? Or is there a workaround somehow? I would love to make use of this!
Thanks!

Hey! v2 will be supported in the future.
v1.13 is broken, correct.
But you can still use the latest nightly version, that should work just fine.

I'm gonna /close this issue due to lack of interactivity and because it's kinda a duplicate of #85
Further progress will be shown there.


Disable the default site of apache and nginx

Hi, thanks for sharing this excellent role.
I tell you that I mounted an instance on Apache with forced HTTPS (nextcloud_tls_enforce: true) in which everything went fine except for one detail. If I access my instance directly with the URL (my-nextcloud.org), I match to the default Apache site instead of my Nextcloud. Therefore I must explicitly specify the HTTPS prefix to access the site (https://my-nextcloud.org)
The configuration file of the Nextcloud site is generated correctly (making a permanent redirection of HTTP traffic to HTTPS), but the default Apache site (000-default.conf) is still enabled and traps me first.
In my opinion the solution is simply disable the default site, which is what I propose in this PR (also for Nginx).
I can't imagine a scenario in which someone wanted to keep this site enabled in production, so I did not consider it necessary add a variable to make a conditional about this task.

Hi @santiagomr
thank you for your feed back !
I noted this bad behavior  today while working on improving the role on ubuntu 18.04 and debian 9.
Removing the default site for apache or nginx could be a easy workaround .... but .... :thinking: this could break the user's web site :thinking:
It's true that the best use case is to install nexcloud from scratch and not using the server for something else
but what in the case the operator added custom configuration in the default ?
For instance, the apache's default site contain logging directive while the nextcloud server file  does not.
disabling the default may disable logging
in the case of nginx this is more straightforward, but the risk to unintentionally breaking the users web's pages configuration is noticeable.
this may need more tweaks on the template generating the configuration directives for each "hostname"  used to access the nextcloud instance.
May be it's bad practice, i'll look into it
Conclusion:

need more impact study and search of a risk free fix

Regards,
Aal

Hello again, you are right that disabling the default site can cause many problems in certain environments and scenarios.
Maybe the best solution is include this characteristic in conditional to a variable (default set to false), as proposed in this commit.
P.S. I already made a rebase on top of master to update these months of progress.


Add Origin network to truffle config

First pull request? Read our guide to contributing
Checklist:

[x] Code contains relevant tests for the problem you are solving
[x] Ensure all new and existing tests pass
[x] Update any relevant READMEs and docs

Description:

Adds our own Ethereum network to the truffle configuration so we can run migrations on it

Thanks @franckc, all set!


Change: Openraft does not require Clone for WatchSender

Changelog
Change: Openraft does not require Clone for WatchSender

WatchSender is added in: 1b4ec53df2c548757607261e88287a2fd6cb5131

This change is

@ariesdevil


Line breaks within :param: text not recognized, messes up formatting of other sections

Jazzy correctly recognizes and interprets line breaks within some blocks (such as :notes: and :returns:) but not in the :param: block.
In the following screenshot, the text from the :param: block for the scheme parameter appears in two places: the portion before the line break appears correctly in the parameter table for scheme, but the portion after the line break is incorrectly interpreted as being part of the function's description:

Only the text "Attempts to construct a URL to represent the specified destination." should appear in the description; everything coming after should be part of the scheme parameter description.
The screenshot above is from the latest release of Jazzy (0.1.1 at the time of this writing), and was rendered from the following source:

Closing — erroneously posted to wrong repo. Apologies!

Just saw this now, completely missed it.
No worries! Happened to me before too! :)


Update the screenshot

[2. Drag the Year field from the Calendar table onto the new blank table visualization.
Because Year is a numeric field, Power BI Desktop sums up its values. This summation doesn’t work well as an aggregation; we'll address that in the next step.]
The screenshot should be as follows

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

ID: 5439d8e1-71bb-cdf7-53bc-952f08a359e0
Version Independent ID: 11f475af-c858-0a7e-d07d-7083e4fc8e39
Content: Tutorial: Create your own measures in Power BI Desktop - Power BI
Content Source: powerbi-docs/transform-model/desktop-tutorial-create-measures.md
Service: powerbi
Sub-service: pbi-transform-model
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @JamesVanWaza. This issue has been assigned to the doc author for review.

This article is in the process of being refreshed. Thanks for your comment!


ModSecurity default config on OpenBSD + Nginx - Error:Abort Trap (Core Dumped)

I've installed Nginx + ModSecurity from source on OpenBSD, without problems.
But when I try to configure modsecurity with the default config, upon starting nginx I get:
"apr_table_copy: t's pool is not an ancestor of p
Abort Trap (Core Dumped)"
AFAICT the problem lies in the modsecurity.conf rules that start with "SecRule", if I comment this lines out nginx starts without problems.
I've also installed the OWASP CRS (only base_rules), and didn't have any problems with the SecRule rules.
Any ideas on what I may be doing wrong?
Thanks in advance

Are you using the nginx_refactoring branch?
https://github.com/SpiderLabs/ModSecurity/tree/nginx_refactoring
If not, please give it a try and let as know the result.

having same issue on OpenBSD 5.9 with 1.9.10  and latest nginx_refactoring branch
also with normal 2.9 from https://www.modsecurity.org/tarball/2.9.1/modsecurity-2.9.1.tar.gz

Hi @gwurr3,
Please try the ModSecurity-nginx connector: https://github.com/SpiderLabs/ModSecurity-nginx

This would happen if APR is compiled with APR_POOL_DEBUG (configure option --enable-pool-debug=yes).
In this case apr_table_copy will make an assetion that memory pool for the new table is a descendant of a pool of original table, which is violated at least here https://github.com/SpiderLabs/ModSecurity/blob/9675653/apache2/re.c#L1250


Characteristics incorrectly terminate in SW section 1.4.5

This refers to the example with the entropy-violating rarefaction imposed in the dam-break problem.
The text says that that the 2-characteristics cross within the 2-rarefaction, but something has been changed with the characteristic plotting so that the characteristics coming from the left actually terminate at the left edge of the rarefaction.  I don't know when this was changed.

Okay, I tracked this behavior back to this commit, where we made reval return nans inside the unphysical rarefaction:
https://github.com/clawpack/riemann_book/commit/d39b7e2f19aec4a3202ef4848aa446a6e99bd76c
I decided the simplest approach is just to modify the text, which now says (incoming PR):

Plot the 2-characteristics and notice that they impinge on the 2-rarefaction; if plotted within the rarefaction, they would cross.


⚠ ClamAV has degraded performance

In 0ae08a7, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 450 ms

Resolved: ClamAV performance has improved in 622a498.


Added formatting keymaps in README

Added the keymaps in README which were missing.

Tysm for merging, this was my first pull request.


Make the retrieval of breakpoint line numbers more robust

Description
The JSON-serialised representation of the breakpoints array in the localstorage is unable to distinguish between empty slots, and slots where the value is null. As a result, after serialising into JSON and deserialising back to JavaScript, the empty slots of an array are now filled with null. This breaks the invariant that only line numbers with a breakpoint are non-empty slots that was previously relied on in getBreakpointLineNumbers.
To resolve this issue, we now check that the value corresponding to the line number is the string 'ace_breakpoint' to determine whether the line number has a breakpoint set or not.
Fixes #2362.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] Code quality improvements

How to test
Follow the steps to replicate in the linked issue.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 7 (14.29%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 34.43%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/editor/Editor.tsx
1
7
14.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
4597

Relevant Lines:
12421

💛  - Coveralls


Add devday test data for 2020 (#197)

the assert in test_utils_devdata.py is magic for me ..

Pull Request Test Coverage Report for Build 604

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0e**-05%**) to 99.954%

Totals

Change from base Build 599:
-1.0e-05%

Covered Lines:
3534

Relevant Lines:
3535

💛  - Coveralls


Timestamp fix when downloading option chains

If using this code in a timezone other than UTC, timestamp strings created when querying option chains in ticker.py may be off by a day.
Fixed by using datetime.datetime.utcfromtimestamp instead of datetime.datetime.fromtimestamp.

Thanks for fixing this! I ran into the exact same thing locally on mine, and came here to submit the same MR. I don't know anything about the checks that were unsuccessful but I'd like to bump this to be merged - its essential for pulling options data.


FR: Wrap Field with Markdown before render

Everyone has different field data.   If the user is able to wrap their field in markdown before rendering the cell it will enable very flexible column types.
For example:
From: https://github.com/RafaelGB/obsidian-db-folder/issues/17#issuecomment-1129118138
If I could wrap a link in transluded image markup, the link can now get rendered as an image:
cover:: https://images-na.ssl-images-amazon.com/images/I/41wd4dlgGBL._SX322_BO1,204,203,200_.jpg
Could get setup as "![anyTexthere|100](" + cover + ")".
The above is a javascript syntax that is essentially putting a ![anyTexthere|100]( before cover field and ) after the field.    The rendered markdown would look like:
![anyTexthere|100](https://images-na.ssl-images-amazon.com/images/I/41wd4dlgGBL._SX322_BO1,204,203,200_.jpg)
...Resulting in an image that is 100px wide.
This could be used for **bold**, *italic*, ==highlights==...
Or if we can get really fancy and do multiline... things like callouts:
> [!TIP] Callouts can have custom titles
> **fields go here**!

I'm not sure JS is the way to do this.  Maybe some templating engine like ![anyTexthere|100]( {{cover}} ) would be simpler.
My assumption: This could probably hinder your ability to sort/filter on that column.  To some that might be worth the trade off.

Starting with this soon. Will be the goal for 1.2.0

Nice... getting so close.

After testing it, it works great!
I think that the

should probably be on a per column basis.    B/c if a different column is just a normal link, it will try to embed it.... like so:

the last visual bug I could not reproduce it.
About move the configuration to column adjustment is one of the dots to 1.3.0
Right now, the header menu is not scalable at all. The refactor idea is to open another Obsidian Modal just as local database config, then add embedded media and the current inline option. So much better I think

the "embeded link" bug was if I used a regular link (like to this github page) and not a link to an asset.

I mean the other with blank settings section.
About regular link, if the header settings are refactored, could add an Iframe option :)

About regular link, if the header settings are refactored, could add an Iframe option :)

Sorry, that's not what I meant.   Here is an showing what I mean:

because the "enable media links" is global to the DB rather than the column... links try to embed... even if they aren't assets.
Here is the page content:
# AntiFragile

cover:: https://images-na.ssl-images-amazon.com/images/I/41wd4dlgGBL._SX322_BO1,204,203,200_.jpg
buyLink::  https://www.amazon.com/Antifragile-Things-That-Disorder-Incerto/dp/0812979680
creator:: Nassim Taleb
priority:: Medium
purpose:: Execution, Strategy
referrer:: CEO Summit (june 2016)
status:: To Read
subtitle:: Things That Gain from Disorder
tags:: #source/books


dockerTools buildImage's contents (and config) nukes the prior layer's files for linked directories

Describe the bug
If you use buildImage with a base image (fromImage) and define a non-empty contents such as contents = [ pkgs.gnumake ], then the base image's /bin or other linked paths will not be available in the final image.
Steps To Reproduce
Thread: https://discourse.nixos.org/t/building-on-dockerfile-based-images/29583/10
Consider this file:
{ pkgs ? import (fetchTarball "https://github.com/NixOS/nixpkgs/archive/a71e45961e88c8a6dde6287fa1e061f30f8c2fb7.tar.gz") { }
}:

pkgs.dockerTools.buildLayeredImage {
    name = "owner";
    tag = "latest";
    fromImage = pkgs.dockerTools.pullImage {
        imageName = "ubuntu";
        imageDigest = "sha256:83f0c2a8d6f266d687d55b5cb1cb2201148eb7ac449e4202d9646b9083f1cee0";
        sha256 = "sha256-5y6ToMw1UGaLafjaN69YabkjyCX61FT3QxU4mtmXMP0=";
        finalImageName = "ubuntu";
        finalImageTag = "latest";
        os = "linux";
        arch = "x86_64";
    };
    contents = with pkgs ; [ gnumake ];
}

Build it an look at the /bin directory, which should have > 100 files if it had all the base image (Ubuntu) files:
docker load < $(nix-build Dockerfile.nix)  0.89s user 0.21s system 17% cpu 6.164 total
tommd@wr /tmp% docker run --rm -it owner:latest bash
root@f66c66371ee9:/# ls /bin
make

Just the one make file provided by the pkgs.gnumake contents.  The story is the same if we use copyToRoot (see above-linked thread).
Expected behavior
The semantics are surprising and quite likely a bug.  Most people seem to expect the extra packages to be linked or copied into the target directories (i.e. /bin) and the prior files provided by the base image (ubuntu) to remain available.
nix-env --version output
% nix-env --version
nix-env (Nix) 2.8.1

When /bin is a directory, the container runtime would merge the layer correctly, so I would guess that the "customization layer" of buildLayeredImage makes /bin a symlink which then shadows the lower layers.
You might work around this problem by installing packages manually using fakeRootCommands. We should have some standard code that does that for us, and I think that should be based on NixOS modules.
I've previously made an effort in this direction

https://github.com/NixOS/nixpkgs/pull/148456

Sometimes I clean up and forward-port a commit or idea from there, but this project really needs more than just me to make some real progress.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/building-on-dockerfile-based-images/29583/18

At least in case of Debian starting image I was able to fix this with minimal changes by using the following copyToRoot for my simple use case (a static binary, which requires CA certs):
copyToRoot = pkgs.buildEnv {
  name = "wasmcloud";
  extraPrefix = "/usr"; # /bin is a symlink to /usr/bin on Debian, add a prefix to avoid replacing original `/bin`
  paths = [
    bin # the binary

    pkgs.dockerTools.caCertificates
  ];
  postBuild = ''
    mv $out/usr/etc $out/etc
  '';

This merged the binary in /usr/bin with all existing Debian binaries as expected and I was also able to access the CA cert bundle in /etc


ListView rebuilds in wrong order when scrolling up

Weird behavior listview with divider based on value from listview items (tested on Android)

Scroll down
Scroll up: the divider is now in the wrong place in the list

Below the getInitFeedData() calls the bloc, which updates the Streambuilder and returns the snapshot.data, theEntireFeed equals snapshot.data returned. The problem seems to be with the DateTime lastTimeOfEvent inside of the ListView.builder.
Waypoint model for this purpose:

code sample

import 'dart:async';
import 'package:flutter/material.dart';
import 'package:intl/intl.dart';

class AllWayPointsBloc {
  final feed = StreamController<List>();

  Stream<List<WayPoint>> get changeFeed =>
      feed.stream.transform(changeToListOfWaypoints);

  void dispose() {
    feed.close();
  }

  final changeToListOfWaypoints = StreamTransformer<List, List<WayPoint>>.fromHandlers(
      handleData: (allWaypoints, sink) {
        sink.add(allWaypoints);
  });

  createFeed(wps){
    feed.sink.add(wps);
  }
}

class WayPoint {
  DateTime timeOfEvent;

  WayPoint(
      {
      this.timeOfEvent,
      });
}

class Waypoints extends StatefulWidget {
  @override
  WaypointsState createState() => WaypointsState();
}

class WaypointsState extends State<Waypoints> {
  DateTime lastTimeOfEvent;
  AllWayPointsBloc wayPointsBloc;
  Stream _stream;
  List<WayPoint> wpFeed = [];
  int currentCount;
  final _debouncer = Debouncer(milliseconds: 500);

  Future<void> getInitFeedData() async {
    lastTimeOfEvent = DateTime(2500); //now();
    wpFeed = [WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)),WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)) ,WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)),WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000))];
    for(int i = 0; i < 2; i++){
      for(int j = 0; j < 9; j++){
        currentCount++;
        wpFeed.add(WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(i *<PHONE_NUMBER>000)));
      }
    }
    wayPointsBloc.createFeed(wpFeed);
  }

  Future<void> moreData() async {
    lastTimeOfEvent = DateTime(2500); //now();
    wpFeed = [WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)), WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)),WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)) ,WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000)),WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(1500000000000000))];
    for(int i = currentCount; i < (currentCount + 18); i++){
      for(int j = 0; j < 9; j++){
        wpFeed.add(WayPoint(timeOfEvent: DateTime.fromMillisecondsSinceEpoch(i *<PHONE_NUMBER>000)));
      }
    }
    wayPointsBloc.createFeed(wpFeed);
  }

  @override
  void initState() {
    lastTimeOfEvent = DateTime(2500);
    wayPointsBloc = AllWayPointsBloc();
    _stream = wayPointsBloc.changeFeed;
    currentCount = 0;
    getInitFeedData();
        super.initState();
      }

      @override
      Widget build(BuildContext context) {
        return Scaffold(
            body: SafeArea(
                child: StreamBuilder<List<WayPoint>>(
                    stream: _stream,
                    builder: (context, snapshot) {
                      if (snapshot.hasData) {
                        if (snapshot.data.length > 0) {
                          print(snapshot.data);
                          List<WayPoint> theEntireFeed = snapshot.data;

                          return NotificationListener<OverscrollNotification>(child: ListView.builder(
                              key: PageStorageKey<String>('thelistofwaypoints'),
                              physics: const AlwaysScrollableScrollPhysics(),
                              itemCount: theEntireFeed.length,
                              itemBuilder: (BuildContext ctxt, int i) {
                                if (DateFormat('dd-MM-yyyy')
                                        .format(theEntireFeed[i].timeOfEvent) !=
                                    DateFormat('dd-MM-yyyy')
                                        .format(lastTimeOfEvent)) {
                                  lastTimeOfEvent = theEntireFeed[i].timeOfEvent;
                                  return Column(
                                    key: UniqueKey(),
                                    children: <Widget>[
                                      Container(
                                        width: double.infinity,
                                        child: Container(
                                          child: Row(
                                            children: <Widget>[
                                              Expanded(
                                                  child: Text(
                                                DateFormat.yMMMEd().format(
                                                    theEntireFeed[i].timeOfEvent),
                                              )),
                                            ],
                                          ),
                                        ),
                                      ),
                                      Container(
                                        height: 50,
                                        width: 50,
                                        color: Colors.red,
                                      )
                                    ],
                                  );
                                } else {
                                  return Container(
                                    height: 50,
                                    width: 50,
                                    color: Colors.blue,
                                    child: Image.network("https://upload.wikimedia.org/wikipedia/commons/3/39/Lichtenstein_img_processing_test.png"),
                                  );
                                }
                              }),
                              onNotification: (sn) {
                          if(sn.metrics.axisDirection == AxisDirection.down){
                              if(sn is OverscrollNotification){
                              if(sn.overscroll < 0){

                              } else{
                                _debouncer.run(() {
                                  if(mounted){
                                    moreData();
                                  }
                                });
                              }
                            }
                            }
                        }
                              );
                        }
                        return Container();
                      } else{
                        return Container();
                      }
                    })));
      }
}

class Debouncer {
  final int milliseconds;
  VoidCallback action;
  Timer _timer;

  Debouncer({this.milliseconds});

  run(VoidCallback action) {
    if (_timer != null) {
      _timer.cancel();
    }

    _timer = Timer(Duration(milliseconds: milliseconds), action);
  }
}

Flutter doctor:
[√] Flutter (Channel stable, v1.12.13+hotfix.5, on Microsoft Windows [Version 10.0.17763.914], locale en-US)
• Flutter version 1.12.13+hotfix.5 at C:\flutter
• Framework revision 27321ebbad (3 weeks ago), 2019-12-10 18:15:01 -0800
• Engine revision 2994f7e1e6
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at C:\Users\Msi\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[!] Android Studio (version 3.5)
• Android Studio at C:\Program Files\Android\Android Studio
X Flutter plugin not installed; this adds Flutter specific functionality.
X Dart plugin not installed; this adds Dart specific functionality.
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
[√] VS Code (version 1.41.1)
• VS Code at C:\Users\Msi\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.7.1
[√] Connected device (1 available)
• Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)
! Doctor found issues in 2 categories.

@chunhtai might know.

@Martibis - can you include the output of flutter doctor -v, and can you include a complete and minimal example to reproduce this?  The code you pasted is missing some information, and probably a lot more complicated than it needs to be to reproduce this bug :)

@Martibis - can you include the output of flutter doctor -v, and can you include a complete and minimal example to reproduce this? The code you pasted is missing some information, and probably a lot more complicated than it needs to be to reproduce this bug :)

Updated the initial issue, hope this is better.

Thank you for providing more details, however, we would need a complete runnable example in order to reproduce this bug. A complete runnable example is a piece a code that we can directly paste into a main.dart and run it with 'flutter run'

Thank you for providing more details, however, we would need a complete runnable example in order to reproduce this bug. A complete runnable example is a piece a code that we can directly paste into a main.dart and run it with 'flutter run'

I updated the OP and also tracked down that the issue seems to be with the overscrollnotification

Reproduces with current dev 1.19.0-1.0.pre. Used Android 9 (API 28)
(with iOS and web the list doesn't build on scroll down).

Reproducible on latest master and stable channels.

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.22.5 at /Users/pedromassango/Code/flutter_stable
    • Framework revision<PHONE_NUMBER> (5 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin installed
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64 • Android 11 (API 30) (emulator)

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.26.0-2.0.pre.313, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.313 at /Users/pedromassango/Code/flutter_master
    • Framework revision 1cb0a24a46 (7 hours ago), 2021-01-12 21:39:27 -0800
    • Engine revision 10cee6abcb
    • Dart version 2.12.0 (build 2.12.0-214.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.1.5
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 11 (API 30) (emulator)
    • macOS (desktop)            • macos         • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 87.0.4280.141

! Doctor found issues in 1 category.

Reproducible on latest master and stable channels.

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.22.5 at /Users/pedromassango/Code/flutter_stable
    • Framework revision<PHONE_NUMBER> (5 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin installed
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (1 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64 • Android 11 (API 30) (emulator)

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.26.0-2.0.pre.313, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.313 at /Users/pedromassango/Code/flutter_master
    • Framework revision 1cb0a24a46 (7 hours ago), 2021-01-12 21:39:27 -0800
    • Engine revision 10cee6abcb
    • Dart version 2.12.0 (build 2.12.0-214.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.1.5
    • Dart plugin version 203.6912

[✓] VS Code (version 1.52.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.18.1

[✓] Connected device (3 available)
    • sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 11 (API 30) (emulator)
    • macOS (desktop)            • macos         • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)               • chrome        • web-javascript • Google Chrome 87.0.4280.141

! Doctor found issues in 1 category.

@Martibis Hello,
Revisiting this issue to check if it still persists or not using latest stable version, I tried to use the code sample provided in original description but it seems lot of implementation need to be updated based on current version, so I am unable to correctly verify the behavior. Is it possible for you to update the code sample based on latest version and provide only minimal runnable code that still shows the issue or if it is still valid one ?


Disable flaky #827 test

This test (which is btw testing that we are still broken) is unfortunately very flaky, to the point it is a PITA.
Disabling it for now - at least until we fix the actual issue this is meant to test.
@AkihiroSuda @djdongjin

@djdongjin done


How to delete a DistributedVirtualPortgroup

Hello,
I can't seem to make this work. I've tried following the example here, but it raises vim.fault.NotFound.
I think it's because I'm trying to delete a DVPortgroup; does that make sense?
I've also tried finding the object by name via vim.Network, then calling DestroyNetwork on that object, but the DVPortgroup still exists under the networking tab in vCenter. Any pointers would be greatly appreciated, thanks!
Python: 3.5
pyVmomi: 6.7.0
vCenter: 6.5.0 build-5973321

Just commented on it here: https://github.com/vmware/pyvmomi/issues/704
In short you need to get the dvs-portgroup object (https://github.com/vmware/pyvmomi/blob/master/docs/vim/dvs/DistributedVirtualPortgroup.rst) and call the DeleteTask(). Simple example:
from pyVim.connect import SmartConnectNoSSL
from pyVmomi import vim

dvs_name = "dvs1"
portgroup_name = "portgroup1"
portgroup = None

vc = SmartConnectNoSSL(host=host, user=username, pwd=password, port=443)
content = vc.RetrieveContent()
container = content.viewManager.CreateContainerView(content.rootFolder, [vim.DistributedVirtualSwitch], True)
for dvs in container.view:
    if dvs.name == dvs_name:
        dv_switch = dvs

portgroups = dv_switch.portgroup
    for pg in portgroups:
        if pg.name == portgroup_name:
            portgroup = pg

task = portgroup.Destroy_Task()
# Task handling not included in example


Add image update banner and status components

padding-left difference between code blocks
padding-top and padding left are too different
black code block is too black, should be more like a dark-purple
padding between text and code-block
and copy button should be at top left of the code-block, with icon and tooltip of copy


setting

Hello author, I would like to know the settings for the experimental test set and training set, as well as the specific settings for the experimental parameters. Because I set the effect of (val_seq) according to the configurations of 1,5,16, which is 3.6 points lower than the effect in the paper. But if set to 1,5,10,15, it is only one point lower than the effect in the paper。

It is weird, I got the following results without any parameters modified.
        # data location
        train_seq = [2, 3, 4, 6, 7, 9, 10, 11, 12, 14, 15]
        val_seq = [1, 5, 16]

Total files: 1560 | Total question: 9014
Total files: 447 | Total question: 2769
Total files: 97 | Total question: 472
{'emb_dim': 300, 'n_heads': 8, 'dropout': 0.1, 'encoder_layers': 6}
model params:  826456574
Test: epoch: 79 loss: 93.453040 | Acc: 0.604912 | Precision: 0.772923 | Recall: 0.507540 | FScore: 0.408044| mIoU: 0.739760
Test: epoch: 79 loss: 23.430898 | Acc: 0.302966 | Precision: 0.604603 | Recall: 0.382693 | FScore: 0.182442| mIoU: 0.738889

Hey, did you manage to solve that issue? Because I'm facing the same problem.


error with region turkey TR

Hi, i forgot to change the region in infogg file and my accounts were turkish region but the script started generating info for those accounts and the info was wrong. The moment i realised that the info is wrong, i changed region from na to tr but its giving an error now with tr region:
Traceback (most recent call last):
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connection.py", line 174, in _new_conn
    conn = connection.create_connection(
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\util\connection.py", line 73, in create_connection
    for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\socket.py", line 954, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno 11001] getaddrinfo failed

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connectionpool.py", line 382, in _make_request
    self._validate_conn(conn)
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connectionpool.py", line 1010, in _validate_conn
    conn.connect()
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connection.py", line 358, in connect
    conn = self._new_conn()
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connection.py", line 186, in _new_conn
    raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x000001BB892C5640>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\adapters.py", line 439, in send
    resp = conn.urlopen(
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\urllib3\util\retry.py", line 574, in increment
    raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='pd.tr.a.pvp.net', port=443): Max retries exceeded with url: /store/v2/storefront/4facd75b-7814-5821-b035-8402bee2aef9 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x000001BB892C5640>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "E:\scripts\checker\NightMarket Checker\nmchecker.py", line 121, in <module>
    main()
  File "E:\scripts\checker\NightMarket Checker\nmchecker.py", line 114, in main
    price = getNight(puuid[0], puuid[1])
  File "E:\scripts\checker\NightMarket Checker\nmchecker.py", line 78, in getNight
    response = sess.get("https://pd.{region}.a.pvp.net/store/v2/storefront/{puuid}".format(
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "C:\Users\saife\AppData\Local\Programs\Python\Python39\lib\site-packages\requests\adapters.py", line 516, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='pd.tr.a.pvp.net', port=443): Max retries exceeded with url: /store/v2/storefront/4facd75b-7814-5821-b035-8402bee2aef9 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x000001BB892C5640>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed'))

These are the list of Regions
NA      - North America
EU      - Europe
BR      - Brazil
AP      - Asia Pacific
KR      - Korea
LATAM   - Latin America

These are the list of Regions
NA - North America EU - Europe BR - Brazil AP - Asia Pacific KR - Korea LATAM - Latin America

OMG my bad.. turkey region falls under EU. i am soo noob. pls close the issue when you see it dev. Apologies.


Can't dock DevTools when in an "Edge App"

If I try to use DevTools within an app created from "install this site as an app" I don't see any way to dock the DevTools panel. Floating only.

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

ID: a0dff9d4-34da-5563-4b34-ca5ef03ade06
Version Independent ID: 55519333-e802-ec65-1b49-8fe64fe04825
Content: Change Microsoft Edge DevTools placement (Undock, Dock To Bottom, Dock To Left) - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide-chromium/customize/placement.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Hello @dzg, thanks for the feedback! We will add this to our backlog. If possible, can you tell me a bit more about what you'd like your DevTools experience to look like within the apps (Progressive Web Apps) that you use?


Ensure user input dialog timeout before request

Introduce a timeout attribute to interaction parameters. Use this
timeout value in password plugin to ensure that dialog is dismissed
before the request itself timeout. This avoid the request to return
an empty user input with a successful result while the dialog itself
is still being displayed.
Solve issue #155
@chriadam there is still a possible issue with inconsistent hard coded timeout values: in secretdaemonconnection.cpp:createInterface() (/resp./ in crypto) the timeout is fixed to 3 minutes, which is fine for the default timeout of 2 minutes, but can create issues in my opinion if user sets up a longer timeout. The problem is that this timeout is set for the manager itself, so for all requests. I don't think it can be a good idea to change the timeout after creation when we want to manage a user input with a longer timeout.
A possible solution would be to fix timeouts as a constant somewhere in secret API and use this constant in password plugin. What do you think ?

Not commenting on whether this is a feasible change or not, but do note that added properties are not automatically passed over D-Bus, they need to be changed in serialization.cpp; and such changes should be reflected in D-Bus interface introspection data, and any changes there should also be adapted onto the C API side.
If the timeout should only be passed between daemon and plugins, it is not necessary to touch the D-Bus interfaces.

I think specifying the timeout which the manager uses as some MaxTimeout value and exposing that in the API, makes sense.  Then adding the timeout()/setTimeout() for the user makes sense, so long as we document that values over MaxTimeout will be ignored (e.g. have setTimeout(int seconds) { m_timeout = qMin(MaxTimeout, seconds); }
Also as inzanity mentioned, will need to update the C API etc.

Thank you @inzanity for your reminder. I've updated the first commit accordingly. Tell me if that's enough, particularly from the C API. It seems to me that InteractionParameters class is not wraped directly, but is just present in the xml file describing the API for DBus. I've added a i there.
@chriadam I'm proposing a second commit on top of the initial one, modifying the timeout setting per request in the Secret manager. I'm not convinced that it's a good choice. For requests without user interaction, I've used the default DBus value, for request with explicit interaction parameters, I've used the timeout chosen by the user. For the other requests that may use a built-in interaction parameter (read not set up by the user, but hard coded in the daemon), I'm using the default timeout value.
This is not translated into Crypto part. Indeed, trying to transfer it, I encounter the fact that InteractionParameters is duplicated (I know why), and didn't want necessary to transpose all Secret changes without discussion. And particularly to the fact that, even without actual user interaction, action may take time, more than the DBus default of 25 seconds, e.g. generation of random data. And some actions that don't take InteractionParameters as arguments may indeed create one from implementation. I'm thinking to the sign operation. In normal operation, it should not require a user interaction to unlock the key, because this is done in the fetch of the key. But for GnuPG plugin, the fetch is not unlocking the key, so the sign operation will take input from user to unlock the key… So when reading the implementation in cryptomanager.cpp, I didn't see any way to know which operation will actually require to wait for user interaction. So letting timeout at a hardcoded value like defaultTimeout() may work…
…or not: this is not taking into account possible operation time. Worst case scenario:

the user choose a timeout of 3min ;
manager set the timeout to 3min and launch the DBus call ;
the operation starts by generating random data and show the user interface at T0+2min ;
the user is distracted and takes more than one minute to enter is passphrase.
The issue #155 is then happening. The request will timeout but the password dialog will stay there for an additional two minutes before timeouting itself…

I don't know if you're still following me, but I'm afraid, correcting #155 is more complicated than just synchronizing timeouts as done here. I currently don't see how the client can signal the daemon, that OK, I give up, trash the request and clean pending operations.

The xml change is correct, but you'll need to add similar change in the second occurrence too, and the embedded xml in daemon/SecretsImpl/secrets_p.h and lib/Secrets/interactionservice_p.h. Also the C implementation needs to be changed, adding the i to the signatures where appropriate, and a value for it. Personally I'd suggest adding a sf_secrets_manager_set_user_interaction_timeout() and use the value wherever needed.

As discussed with @chriadam on IRC today, we've decided to postpone the addition of client-side timeout. @inzanity thank you very much for detailed explanations, I'll use it nonetheless on a later MR.
This MR is now, just a quick and dirty fix for #155 ensuring that the password dialog timeout is lower than the request timeout.
I'm investigating with @chriadam advices how the daemon can notify the password plugin that the client has disconnected already and withdraw the dialog accordingly. Some news to come later today or this week in a new MR.
Then, I can reintroduce this timeout attribute from client-side in a third MR.

LGTM.  Merging.


Explicit setting of JTextPane background color

fixes #594
OSX:

Pending test in Linux.

note to self: turn on optimize imports automatically.

:ship:

confirmed working in linux


As a dev, I want to delete a user from the database

Context:
Currently when we run tests on the backend for database operations tied to a user, a user either has to be pre-created or created before each test. If the user is created before each test, the rollback can delete the user but it will not delete the container associated with him.
We need a way to delete the user along with his azure container.
Acceptance Criteria:

The azure container must be destroyed.
The user must no longer exist in the database.

I understand the need behind creating a f() to remove a user. However, I don't think the current reason behind this issue is a good enough motivation to use such functionality.

Your issue.
When doing a test, you can create the user in the setUp(self) then rollback the cursor and delete the container
class some_test
def setUp(self):
        self.con = db.connect_db(DB_CONNECTION_STRING, DB_SCHEMA)
        self.cursor = self.con.cursor()
        db.create_search_path(self.con, self.cursor, DB_SCHEMA)
        self.connection_str = BLOB_CONNECTION_STRING
        self.user_email = "test@email"
        self.tier="test-user"
        self.user_obj = asyncio.run(
            datastore.new_user(
                self.cursor, self.user_email, self.connection_str, self.tier
            )
        )
self.container_name = "test-container"
        self.user_id = datastore.User.get_id(self.user_obj)
        self.container_client = asyncio.run(
            datastore.get_user_container_client(
                self.user_id,
                BLOB_CONNECTION_STRING,
                BLOB_ACCOUNT,
                BLOB_KEY,
                "test-user",
            )

    def tearDown(self):
        self.con.rollback()
        self.container_client.delete_container()
        db.end_query(self.con, self.cursor)

Those should be sufficient to disable a user from being created in the db (self.con.rollback()) and will remove all traces of the user being created in the Blob Storage (self.container_client.delete_container()) .

What is asked in this issue
We can create a way to erase Fertiscan users, however we need to decide of our strategy on how to handle edge cases like what if the user has validated and important data in its storage.
The need to erase users is real, yet it has not seemed important enough to be implemented into the workflow of Fertiscan yet. I will discuss with @k-allagbe to determined the best solution for all edge case and better plan the work behind what this issue require.
I personally don't think just erasing the container of a user is a good practice if we don't care for the content or transaction done on the tool by said usere.

We actually already found a solution for our problem in the backend. We'll update the backend if anything changes on this aspect in the datastore. Closing this.


App fails when updating to latest npm packages on Ubuntu 24.04

Hi there, so on Ubuntu 24.04 updated:
I cloned the repo and, installed npm packages and can succcesfully run the app.
Then I update to latest npm packages with ncu -u ; npm install.
Now the app fails to start with the following error:
"qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found. This application failed to start because no Qt platform plugin could be initialized.  Reinstalling the application may fix this problem.
I investigating further by setting QT_DEBUG_PLUGINS=1, pinpoints the culprit:
<EMAIL_ADDRESS>(libxcb-cursor.so.0: cannot open shared object file: No such file or directory)"
The file libqxcb.so does exist at that location.
Any suggestions on how to fix?
I can see it's not an issue with this example project itself.
(Staying at lower versions is not an options for my project)

Ah, found the fix here:
https://stackoverflow.com/questions/76802977/ubuntu-22-04-qt-6-5-qt-qpa-plugin-could-not-load-the-qt-platform-plugin-xcb
You need to install this library: sudo apt-get install libxcb-cursor0


add config for auto complete ref with ' over "

Describe the feature
when adding refs we use ' around the model name instead of " making the auto complete not 100% useful as we have to change the quotes every time
Describe alternatives you've considered
No response
Who will benefit?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

@JackWolverson: ideally the code just picks up the single quotes or double quotes you typed and apply it automatically. No need for config. I think this has always been the intended behaviour.
Can you check and implement this behaviour?

Chiming in because this is still desired functionality. As of 11/8 it's not picking up the use of single or double quotes and it's simply replacing all single quotes with doubles.

I have a follow up question. Do you have sqlfmt installed? @JackWolverson @edwardofclt . you can try it out by typing sqlfmt in the terminal. if its installed its help command will display.


Add Metal support for I2_S

This PR supports both prefilling and decoding function for I2_S

Thanks for your commit! Merged in https://github.com/microsoft/BitNet/pull/141


Tree shaking not working on example repo

Hi,
It seems to me that if I want to use 'vuetify-loader' it does not work for the components that are outside of .vue components. I have tried with your example repo here https://github.com/Nothing-Works/mix and if I do something like this:
// resources/views/welcome.blade.php
<div id="app">
        <v-app>
            <v-btn>
                Test
            </v-btn>
        </v-app>
        <example-component></example-component>
    </div>

It will give me an error:
[Vue warn]: Unknown custom element: <v-app> - did you register the component correctly? For recursive components, make sure to provide the "name" option.
[Vue warn]: Unknown custom element: <v-btn> - did you register the component correctly? For recursive components, make sure to provide the "name" option.
Is this expected?
Second, in this same repo, it does not seem to load all the styles properly. This is how it looks:

I installed everything as it is and no changes except the one I reference here. Am I doing something wrong, or this is a bug?
Thanks!

@schrink I think the repo has not been updated for a long time, I will update it after fixing some bugs.

@Nothing-Works Yes, but the problems I explained are not only example repo related. I can’t make Vuetify work for components that are not within .vue files if I enable tree shaking. Otherwise it works normally.
I used your repo just to isolate other possible reasons related to my project. This only confirms that this is most likely a bug.
Btw, thank you for your awesome work!


Replace Dockerfile with Image each run

Fix problem in #193 and work around GHA's lack of build-arg support for users behind proxy

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


Add ssl support

Set cert_cert and cert_key to support client certificates. Set ssl=True or ca_cert to support etcd over ssl

Hey @pengyao, many thanks for your MR. I'll take a look after #19.


Add support for verification page property

The verification page property was added to the Page properties

Example verification page property values as returned in a GET page request
Unverified
{
  Verification: {
    id: "fpVq",
    type: "verification",
    verification: { state: "unverified", verified_by: null, date: null },
  },
}

Verified with no expiration date set
{
  Verification: {
    id: "fpVq",
    type: "verification",
    verification: {
      state: "verified",
      verified_by: {
        object: "user",
        id: "01e46064-d5fb-4444-8ecc-ad47d076f804",
        name: "User Name",
        avatar_url: null,
        type: "person",
        person: {},
      },
      date: { start: "2023-08-01T04:00:00.000Z", end: null, time_zone: null },
    },
  },
}

Verified with 90-day expiration date
{
  Verification: {
    id: "fpVq",
    type: "verification",
    verification: {...},
      date: {
        start: "2023-08-01T04:00:00.000Z",
        end: "2023-10-30T04:00:00.000Z",
        time_zone: null,
      },
    },
  },
}

Notion Change log: https://developers.notion.com/page/changelog#july-25---august-7-2023

#368