Can't clone one emulator blob to a new directory

Storage Explorer Version: 1.29.0-dev
Build Number: 20230319.3
Branch: main
Platform/OS: Windows 10
Architecture: ia32
Azurite Version: 3.21.0
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Install and run Azurite -> Launch Storage Explorer.
Expand 'Emulator & Attached' -> '(Emulator - Default Ports) (Key)' -> Blob Containers.
Create a blob container -> Upload one blob.
Right click the blob -> Click 'Clone...'.
Try to clone to blob to a new directory.
Check whether succeeds to clone the blob.

Expected Experience
Succeed to clone the blob.
Actual Experience
Can't clone the blob.

Verified this issue on the build main 20230321.12. Reproduce.
We are going to re-open this issue and verify it on a newer build.

Verified this issue on the build main 20230322.4, Fixed.


Initial de-duplication / record matching scripts

Description
As a first pass run over the trial records collected from registers, we want to run some basic matching using unique identifiers and trial titles. The goal of this should be a quick, cheap, simple run to see how far we can get with record matching using the most basic possible techniques.
Tasks

[ ]  TBD

DONE in https://github.com/opentrials/warehouse/pull/9


feat: rows and size limits for part size in ExecuteQueryRequest

https://github.com/ydb-platform/ydb/blob/main/ydb/public/api/protos/ydb_query.proto#L154

limit in bytes has been added
https://github.com/ydb-platform/ydb/pull/3609/files
https://github.com/ydb-platform/ydb/pull/3609
https://github.com/ydb-platform/ydb/pull/3471

rows limit will be implement here https://github.com/ydb-platform/ydb/issues/3630


Be able to upgrade permissions on iOS 14+

Starting with iOS 14, image permissions are managed through PHPhotoLibrary.authorizationStatus(for: .readWrite) (like correctly being used already, yey!). A good improvement in user experience could be to check the current authorization status and show a tiny "Select more photos …" bar above the picture selection if the current permission level is .limited. Instagram does that pretty well and encourages users to select more photos in a privacy-focussed context.
Two possible ways to add this could be the following (using SFSymbols, which work fine with iOS 13+):

I'm open for discussions, thank you!

Nice idea, but don't have time for it)

Any update on this?


Enable package installation from branch

This is an implementation of the discussion for the Enable package installation from branch proposal.
It uses your current branch name and prefixes it with 'pre-release', and then publishes the built package directory to this branch, which can then be installed via npm.
As an example I ran this script on this branch itself resulting in:

Result from running the script: https://github.com/alphagov/govuk-frontend/tree/pre-release-publish-build-to-branch
You can see this being consumed in: https://github.com/alphagov/govuk-design-system/pull/629

https://trello.com/c/QqsC621F/1556-work-out-how-to-preview-changes-from-frontend-in-another-app

I have found another way to do this that seems to be a lot quicker than doing git subtree split:
https://help.github.com/articles/splitting-a-subfolder-out-into-a-new-repository/

I have tested the following scenarios:

No branches exist, running for the first time: it succeeds
I have made a change locally and not committed it: it warns you and stops.
I have made a change and committed it with existing release pushed: it overwrites this release.
I have made no changes and run the release: it works

Tested this on Friday too. @nickcolley's work entirely 👏 .
Thanks for this! Hopefully it will ease the contribution process, especially for components.

@alex-ju we wouldnt have done it without you pushing to get this fixed, thanks Alex


Update events hero

Updates the Events Hero to match changes in #262
Additions

Combined .hero_card-bg and .hero_card-padding blocks to .hero_preview block
Added header and footer blocks to .summary block
Included event-meta_datetime to reuse available meta date pattern

Removals

Removed extra .hero_card-bg and .hero_card-padding blocks

Changes

Swapped divs for more semantic article where applicable
Moved em dash from event date to event address
Fixed whitespace

Testing
Fetch branch

navigate to /events page

Review

@anselmbradford
@sebworks

Preview

No visual differences

Notes

To reuse event-meta pattern the em-dash had to be moved to follow the .event-meta_address

LGTM :+1:

:+1: ×2


Unable to install SWA CLI on Alpine Linux

Describe the bug
I'm using a Docker container image, mcr.microsoft.com/azure-cli:2.33.1, which is based on Alpine Linux.
I installed nvm and the latest node v16.x successfully. But I wasn't able to install this SWA CLI. It threw an error:
~ # npm install -g @azure/static-web-apps-cli
npm notice
npm notice New minor version of npm available! 8.5.5 -> 8.10.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v8.10.0
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
npm ERR! code ENOENT
npm ERR! syscall spawn sh
npm ERR! path /root/.nvm/versions/node/v16.15.0/lib/node_modules/@azure/static-web-apps-cli/node_modules/keytar
npm ERR! errno -2
npm ERR! enoent spawn sh ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

To Reproduce
Steps to reproduce the behavior:

Open a container image like docker run -it mcr.microsoft.com/azure-cli:2.33.1
Run the following commands:

# Update and upgrade
apk update && apk upgrade \
    && apk add -U curl bash ca-certificates openssl ncurses coreutils python2 make gcc g++ libgcc linux-headers grep util-linux binutils findutils \
    && apk add -U bash icu-libs krb5-libs libgcc libintl libssl1.1 libstdc++ zlib \
    && apk add -U libgdiplus --repository https://dl-3.alpinelinux.org/alpine/edge/testing/ \
    && apk add -U libc6-compat gcompat

# Install nvm
cd ~
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh | bash
export NVM_DIR="$([ -z "${XDG_CONFIG_HOME-}" ] && printf %s "${HOME}/.nvm" || printf %s "${XDG_CONFIG_HOME}/nvm")"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh" # This loads nvm

# Install node.js
nvm install --lts
nvm use --lts

# Install SWA CLI
npm install -g @azure/static-web-apps-cli

See the error above

Expected behavior
A clear and concise description of what you expected to happen.
The SWA CLI should be installed successfully.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: A container image, mcr.microsoft.com/azure-cli:2.33.1, based on Alpine Linux

Additional context
Add any other context about the problem here.
If I ran the yarn command like:
corepack enable
yarn global add @azure/static-web-apps-cli

It was fine, but I still have the same issue after that, by running npm install or npm run build, etc...

Hey @justinyoo can you share the logs from npm install -g @azure/static-web-apps-cli --verbose please? It looks like an issue with Keytar, but I am not sure.

FYI, I opened an issue at https://github.com/atom/node-keytar/issues/461

Hey @justinyoo can you share the logs from npm install -g @azure/static-web-apps-cli --verbose please? It looks like an issue with Keytar, but I am not sure.

Sure. I also suspect the keytar side... Here's the log:
npm info run<EMAIL_ADDRESS>install node_modules/@azure/static-web-apps-cli/node_modules/keytar prebuild-install || npm run build
npm info run<EMAIL_ADDRESS>install { code: 'ENOENT', signal: undefined }
npm timing reify:rollback:createSparse Completed in 1502ms
npm timing reify:rollback:retireShallow Completed in 0ms
npm timing command:install Completed in 33815ms
npm notice
npm notice New minor version of npm available! 8.5.5 -> 8.11.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v8.11.0
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
npm verb stack Error: spawn sh ENOENT
npm verb stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:283:19)
npm verb stack     at onErrorNT (node:internal/child_process:478:16)
npm verb stack     at processTicksAndRejections (node:internal/process/task_queues:83:21)
npm verb pkgid<EMAIL_ADDRESS>npm verb cwd /root
npm verb Linux <IP_ADDRESS>-microsoft-standard-WSL2
npm verb argv "/root/.nvm/versions/node/v16.15.0/bin/node" "/root/.nvm/versions/node/v16.15.0/bin/npm" "install" "-g" "@azure/static-web-apps-cli" "--verbose"
npm verb node v16.15.0
npm verb npm  v8.5.5
npm ERR! code ENOENT
npm ERR! syscall spawn sh
npm ERR! path /root/.nvm/versions/node/v16.15.0/lib/node_modules/@azure/static-web-apps-cli/node_modules/keytar
npm ERR! errno -2
npm ERR! enoent spawn sh ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm verb exit -2
npm timing npm Completed in 34015ms
npm verb unfinished npm timer reify<PHONE_NUMBER>368
npm verb unfinished npm timer reify:build<PHONE_NUMBER>627
npm verb unfinished npm timer build<PHONE_NUMBER>628
npm verb unfinished npm timer build:deps<PHONE_NUMBER>628
npm verb unfinished npm timer build:run:install<PHONE_NUMBER>661
npm verb unfinished npm timer build:run:install:node_modules/@azure/static-web-apps-cli/node_modules/keytar<PHONE_NUMBER>661
npm verb code -2

@justinyoo To fix the npm install of the asw CLI, I had to add the following packages: apk add --no-cache python3 py3-pip libsecret-dev alpine-sdk, then the npm install passed.

@justinyoo To fix the npm install of the asw CLI, I had to add the following packages: apk add --no-cache python3 py3-pip libsecret-dev alpine-sdk, then the npm install passed.

@Nexyll I installed what I had installed above + what you suggested, but still have the same error.
This time, I use the container image of mcr.microsoft.com/azure-cli:2.37.0 (version updated from 2.33.1 to 2.37.0).

Hi, any updates on this issue or suggested workarounds?

hello team, I am also similar issue ..how to resolve it?

As of right now, I cannot reproduce,
/workspaces/alpine-tests # npm install -g @azure/static-web-apps-cli
npm warn deprecated<EMAIL_ADDRESS>This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported

added 320 packages in 18s

48 packages are looking for funding
  run `npm fund` for details
npm notice
npm notice New minor version of npm available! 10.7.0 -> 10.8.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.8.1
npm notice To update run: npm install -g<EMAIL_ADDRESS>npm notice

This is with image node:20-alpine3.19  - my Dev container looks like this:
{
    "name": "Alpine Dev",
    "image": "node:20-alpine3.19",
    "features": {
        "ghcr.io/devcontainers/features/common-utils:2": {}
    }
}

This is with @azure/static-web-apps-cli:1.1.10 (latest version at time of writing)
I did try to build with mcr.microsoft.com/azure-cli:2.37.0, but that does not contain all the requirements for running node.  (specifically, it does not contain the glibcxx libraries) and is based on an older (non-supported) version of Alpine Linux (3.15 - whereas the earliest Alpine Linux supported by Node is 3.19)
I'm not an expert on containerization, but my suggestion is to start with the requirements of your application (node, in this case) and go from there.


Remove duplicated words from Governance

What this PR does / why we need it

While reading the Governance of the project I noticed there was a duplicated series of words.
This PR simply removes them.

Wow you guys are fast! Thanks for the quick turnaround 👍


⚠ Bofrost.fr has degraded performance

In 21f4499, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1224 ms

Resolved: Bofrost.fr performance has improved in 95107d7 after 6 minutes.


Developer guide: use case did not specify which card to choose

The use case should include information of which card to choose from. Below image is an example of the issue.

Issue has been resolved in PR #220.


fix for dnsomatic

Great work; best I've found.
For DNS-0-Matic, there are undefined variables in ddnsup.py. Correcting the following lines resolves the issue and the record updates correctly.
DOM_USER = os.getenv('DOM_USER')
DOM_PASSWD = os.getenv('DOM_PASSWD')
DOM_WILDCARD = os.getenv('DOM_WILDCARD', 'NOCHG')
DOM_MX = os.getenv('DOM_MX', 'NOCHG')
DOM_BACKUPMX = os.getenv('DOM_BACKUPMX', 'NOCHG')`

Side note:
OpenDNS credentials are required. I signed up a long time ago and had DNS-O-Matic credentials. I feel it may help to mention that.

Not really a bug, though I'll certainly grant you the naming convention could certainly be a bit more consistent.
If you check around line 45 of ddnsup.py, you'll find this:
    case 'dnsomatic':
        # Required Vars for DNS-O-Matic
        DOM_USER = os.getenv('DOMUSER')
        DOM_PASSWD = os.getenv('DOMPASSWD')
        DOM_WILDCARD = os.getenv('WILDCARD', 'NOCHG')
        DOM_MX = os.getenv('MX', 'NOCHG')
        DOM_BACKUPMX = os.getenv('BACKUPMX', 'NOCHG')
        # (possibly) redefine RECORDS - If you're using DNS-O-Matic and want to
        # update all services, you don't need to set RECORDS, we'll default to
        # all.dnsomatic.com, which does all your services.
        RECORDS = os.getenv('RECORDS', 'all.dnsomatic.com')

The vars you'd need to set in your environment are DOMUSER, DOMPASSWD, and optionally WILDCARD, MX, BACKUPMX, and RECORDS. Definitely 2 typo's in the example compose file, which I'll definitely commit to fixing. I'll look at some slightly more consistent variable naming, but doing so in a way that's backward compatible, so as not to break existing installs.

Noticed. It looked like the DOM_ format was favored, so I went that route. To keep the source unmodified, I did update the yaml like you pointed out.
Saw an earlier commit mentioning plans to deprecate DNS-O-Matic. Hopefully it's decided to keep it. I tried 5 other clients, most broke, some spam updates, etc... Yours has everything going for it.

So, my immediate thought is just to simply introduce these new vars as the "defaults" but keep supporting the old ones. Sort of like this:
DOM_USER = os.getenv(DOM_USER) or os.getenv('DOMUSER')
Thoughts? This would prevent someone using DOMUSER from breaking, while enabling some better normalization of variables.

Hey - I just pushed a dev branch with all the variable normalization stuff. Take a look at the README in the dev branch, and take ghcr.io/jcostom/ddnsup:dev out for a spin. LMK if it's working the way you expect, ok?

I was out of town, just caught up. Great solution; switched to :dev from :latest, used the config from dev, and it worked flawless first try. Awesome!
Side note: Noticed on the dev branch, the configs for dns-made-easy and dns-o-matic are missing ":latest" from the URL. Only an observation in differences; tested, does the same thing either way.

Nice catch. Thanks. I'll update.


API Key Premissions

Just wondering what permissions I should set on the api keys? I assume most of them but if there are any that aren't necessary I wouldn't include them.

For the API permissions, you'll want to allow the follow:

placing long positions and margin short positions.
querying for balance.
querying for currently active orders.

You can disable withdrawals for security.


block /paper

holy SHIT

yes please fix this is very bad.

Looks like permissions were recently added for /paper subcommands. I'll go through them and see what needs to be disabled.

Merging until I have more time to investigate.

Looks like permissions were recently added for /paper subcommands. I'll go through them and see what needs to be disabled.

nothing really interesting in /paper, half of the commands are exploits


Reconnect backoff

To help minimize the impact on the servers during failures.

Only other comments is that with ReconnectWaitMax set by default, it might make more sense to rename ReconnectWait to ReconnectWaitMin


Cannot import data - error

Hi,
When I ran this command:
./map-matching.sh action=import datasource=/home/map-matching/map-data/leipzig_germany.osm.pbf
I got this error:
ipzig_germany.osm.pbf
ls: cannot access target/map-matching-*-dependencies.jar: No such file or directory
[INFO] Scanning for projects...
Downloading: https://oss.sonatype.org/content/groups/public/org/sonatype/oss/oss-parent/9/oss-parent-9.pom
Downloading: http://repo.maven.apache.org/maven2/org/sonatype/oss/oss-parent/9/oss-parent-9.pom
[ERROR] The build could not read 1 project -> [Help 1]
[ERROR]
[ERROR]   The project com.graphhopper:map-matching:0.5-SNAPSHOT (/home/isg/Desktop/map-matching/pom.xml) has 1 error
[ERROR]     Non-resolvable parent POM: Could not transfer artifact org.sonatype.oss:oss-parent:pom:9 from/to sonatype-oss-public (https://oss.sonatype.org/content/groups/public/): oss.sonatype.org: No address associated with hostname and 'parent.relativePath' points at no local POM @ line 17, column 13 -> [Help 2]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/ProjectBuildingException
[ERROR] [Help 2] http://cwiki.apache.org/confluence/display/MAVEN/UnresolvableModelException
Error: Unable to access jarfile action=import
Maven and Java are installed.
The output of echo $JAVA_HOME is the following: /usr/lib/jvm/java-7-openjdk-amd64/jre
Can you help me ?
Thanks,

Strange, maybe sonatype is offline? Would you try 0.4 instead of the 0.5 snapshot (just change the branch to 0.4 and try again)

No still not working.
I was thinking: If I used a proxy, could still work ?

No, idea. Never worked with proxies.
Can you access the sonatype and maven central repos?
https://oss.sonatype.org/content/groups/public/

Yes I'm able to go to the Maven repo of the website.

Okey A friend test the software without proxy and he didn't have the errors above except this one: Error: Unable to access jarfile action=import
So do you know for this error ? For the proxy, I'll try to see how I can deal with...I don't know if in the program I have to put some proxy parameters.
Thanks

if you cannot download the dependencies there is no other way to build it and therefor it cannot find the created jar file.

What you can try is to specify the proxy for the JVM: http://stackoverflow.com/questions/120797/how-do-i-set-the-proxy-to-be-used-by-the-jvm

We've fixed the script, can you try again with the latest version?


Feature/multi region support

Updated plugin to support docker image push to multiple regions. This will ensure images are available across multiple regions with the same digest value.
region property has been updated to regions. This could lead to a breaking change.
@72636c can you please have a look?

Hi @raksha1, thanks for your contribution 🙇.
I'd prefer to avoid a breaking change if possible. Can we retain support for the existing region option or even merge this into that option? For example, Buildkite steps support both command and commands, and either can take a single string or an array.

Thanks a lot for your feedback, @72636c! :)
Makes more sense to support both region and regions


Toolbar Icons from URL

Hey @iangilman hope all is well, is it possible to have the toolbar icons like 'zoom in' be sourced from a url as opposed to an images folder? I'm hoping to pull the icons from a url to an S3 bucket, for instance, as opposed to a local or server path to the images but not having any luck. I've seen a few people create custom toolbar divs when working in html but I'm not sure how that would work in React and if possible would love to just change the src on the images of the icons. Thanks in advance!

Nice :)


update vaadin 23.3.8

As said in the other PR:

https://github.com/vaadin/vaadin-flow-karaf-example/pull/144

To fix the two PR-s (similar problem) is to have the OSHI dependency with a proper version that is not too constraint anymore in the SLF4J, this version:

https://github.com/oshi/oshi/releases/tag/oshi-parent-6.4.1

I am up for introducing a version variable in the ./pom.xml with the name license.checker.version, not sure maybe the dependencyManagement tag can work as well. But for consistency in the poms and usage, I like better the pom version variable.

I think we can close this one, as a new release is needed to make this work (use the proper versions of license checker).


OCPBUGS-14829: Warn usernames without domain name

Fixes https://issues.redhat.com/browse/OCPBUGS-14829
We could emit a metric here as well, and create alerts but I am worried about customers then complaining about noisy alerts.

/retest-required
/lgtm

/cherry-pick release-4.14
/cherry-pick release-4.13
/cherry-pick release-4.12


Crashes on startup and then corrupts/loses accounts and settings

Expected behaviour
I get my inbox
Actual behaviour
I get a crash, and on trying again to open K-9 e-mail I get the welcome screen asking me whether I want to import my settings from somewhere.
Steps to reproduce
Sadly I don't know, but it's happened to me twice now. First time I don't have a backtrace of, but this time I caught it in time. There is some similarity with #1272 but that one doesn't happen at startup nor do I see a mention of settings getting lost.
This is the stacktrace, including the second attempt going straight to the welcome screen instead:
06-09 12:49:12.611  5440  6985 I ActivityManager: START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.fsck.k9/.activity.Accounts bnds=[13,242][224,495] (has extras)} from uid 10031 on display 0
06-09 12:49:12.736  5440  3544 I ActivityManager: Start proc 1674:com.fsck.k9/u0a121 for activity com.fsck.k9/.activity.MessageList
06-09 12:49:13.057  1674  1674 E AndroidRuntime: FATAL EXCEPTION: main
06-09 12:49:13.057  1674  1674 E AndroidRuntime: Process: com.fsck.k9, PID: 1674
06-09 12:49:13.057  1674  1674 E AndroidRuntime: java.lang.RuntimeException: Unable to start activity ComponentInfo{com.fsck.k9/com.fsck.k9.activity.MessageList}: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String com.fsck.k9.Account.getUuid()' on a null object reference
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2416)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2476)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread.-wrap11(ActivityThread.java)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1344)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.os.Handler.dispatchMessage(Handler.java:102)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.os.Looper.loop(Looper.java:148)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread.main(ActivityThread.java:5422)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at java.lang.reflect.Method.invoke(Native Method)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:726)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:616)
06-09 12:49:13.057  1674  1674 E AndroidRuntime: Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.String com.fsck.k9.Account.getUuid()' on a null object reference
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.fsck.k9.fragment.MessageListFragment.decodeArguments(MessageListFragment.java:957)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.fsck.k9.fragment.MessageListFragment.onCreate(MessageListFragment.java:810)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.Fragment.performCreate(Fragment.java:2198)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:942)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:1148)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.FragmentManagerImpl.moveToState(FragmentManager.java:1130)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.FragmentManagerImpl.dispatchCreate(FragmentManager.java:1948)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.FragmentController.dispatchCreate(FragmentController.java:141)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.Activity.onCreate(Activity.java:921)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.fsck.k9.activity.K9FragmentActivity.onCreate(K9FragmentActivity.java:19)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at com.fsck.k9.activity.MessageList.onCreate(MessageList.java:194)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.Activity.performCreate(Activity.java:6251)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1107)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2369)
06-09 12:49:13.057  1674  1674 E AndroidRuntime:        ... 9 more
06-09 12:49:13.060  5440  6985 W ActivityManager:   Force finishing activity com.fsck.k9/.activity.MessageList
06-09 12:49:13.258  5440  6858 D ConnectivityService: updateNetworkScore for NetworkAgentInfo [WIFI () - 561] to 60
06-09 12:49:13.592  5440  5507 W ActivityManager: Activity pause timeout for ActivityRecord{8b20213 u0 com.fsck.k9/.activity.MessageList t2810 f}
06-09 12:49:14.346  5440 10803 W InputMethodManagerService: Window already focused, ignoring focus gain of: com.android.internal.view.IInputMethodClient$Stub$Proxy@4ea8042 attribute=null, token = android.os.BinderProxy@10656c8
06-09 12:49:14.367  5440 10979 I ActivityManager: Process com.fsck.k9 (pid 1674) has died
06-09 12:49:14.545  5440  5553 W AppOps  : Finishing op nesting under-run: uid 1000 pkg android code 24 time=0 duration=0 nesting=0
06-09 12:49:14.876  5440  6987 I ActivityManager: START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.fsck.k9/.activity.Accounts bnds=[13,242][224,495] (has extras)} from uid 10031 on display 0
06-09 12:49:14.924  5440  6429 I ActivityManager: Start proc 1705:com.fsck.k9/u0a121 for activity com.fsck.k9/.activity.Accounts
06-09 12:49:15.037  5440  3544 I ActivityManager: START u0 {cmp=com.fsck.k9/.activity.setup.WelcomeMessage} from uid 10121 on display 0

Environment
K-9 Mail version: Version from Play Store which is currently 5.010.
Android version: 6.0.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

The crash is most likely caused by the data already being gone rather than being the cause of the data disappearing.
The Accounts activity seems to be able to do its job. That suggests accounts can still be accessed at that point.
Did this happen after an app update?

Oh yes, that's a much more plausible theory indeed.
Did you push out a new version to the Play Store recently? I don't know how I can tell whether an update has happened recently - there's a notification about 5 updates having been installed but all different ones.
I have a 1.2MB backlog of adb logs, sadly first mention of k9 there is the exact crash above.
I'm trying to remember whether this was the first time I tried to use K9 today but don't know. My mailserver doesn't seem to log logins so I can't find out there either. :-/

Not really. 5.010 was released in April. But failing database migrations are the only reason where we deliberately remove all data.

Strange, an upgrade seems unlikely this time then. I suppose there's not much else I can do to find useful info here then? I've restored my account now, hopefully this won't happen again too soon (last time was probably in April, likely around the upgrade then).
I've found Dovecot logs which showed one login from my phone around 11, an hour before this problem occured. Nothing logged about exact disconnect time, but this is probably not that interesting anyway.

I just "lost" all accounts on my personal phone. The account databases still seem to be there, but the main database is empty except for these key/value pairs:
sqlite> select * from preferences_storage;
68639468-2847-40c4-ab4e-02536e6457af.description.0|Initial identity
68639468-2847-40c4-ab4e-02536e6457af.description.1|identity 2
68639468-2847-40c4-ab4e-02536e6457af.signatureUse.0|false
68639468-2847-40c4-ab4e-02536e6457af.signatureUse.1|false
68639468-2847-40c4-ab4e-02536e6457af.latestOldMessageSeenTime|1503484153000
<EMAIL_ADDRESS><EMAIL_ADDRESS>68639468-2847-40c4-ab4e-02536e6457af.name.0|cketti
68639468-2847-40c4-ab4e-02536e6457af.signature.1|
68639468-2847-40c4-ab4e-02536e6457af.name.1|
68639468-2847-40c4-ab4e-02536e6457af.signature.0|--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
MailService.lastCheckEnd|1504226373375

It's unclear what caused this. I didn't see a crash dialog and couldn't find anything in the logs.

I think maybe we should never delete the DB, just error madly.

Duplicate of #750


Talking to Esker a second time near the end of the loop softlocks the game

Hey, first of all, I want to say that this mod is really good and I wish it was part of the main game!
Now uh, unfortunately, it seems there are some big oversights with Esker, because I've gotten myself softlocked when trying to talk with them multiple times - the logs complain about there being no conditions for their character tree, and as such, I get stuck in a screen functionally similar to dialogue screens, but in which Esker doesn't say anything and I never get a prompt to continue or exit dialogue with them:

I've had this happen twice:
• First time was with a mod that removed the supernova and I talked with them once (which they commented about the sun being red), and when talking with them again resulted in the softlock;
• Second time was in a regular playthrough with the supernova unchanged, I talked with them near the end of the loop (you can see the sun is already a Red Giant) about finding Feldspar, which resulted in them ending conversations with me. I interacted with them again, and I once more got softlocked.
As such, it seems that this generally happens when you try to talk to Esker a second time near the end of the loop, after they commented about the sun looking very red. After that, it seems an oversight left them with no valid conversation dialogues, causing this issue.

I was able to reproduce this issue.
The only two conditions that seem to be required are:

this is your first time talking to Esker in that loop
the sun is big and red

If those are both true, you can converse with Esker, but trying to talk to Esker again after ending your conversation in any way will result in a softlock. I'll stare more at the XML for the dialog, and see if I can figure out how and why this is happening.


No demo project added. Please add a demo project!

No demo project added. Please add a demo project!

Please add a demo project! @konstantinpavlikhin

Guys, I am really sorry for the long time it took me to implement this! 🙏🏻
I thought that the KSPAutomaticHeightCalculationTableCellView is so simple that it doesn't need a dedicated sample app, because the size of the latter would be an order of magnitude larger than the initial class.
Anyway, now we have a full-featured demo application. Give it a try.


Thank You!!!

Thank you for creating this repo.
Awesome Work!!!
Helped me complete almost all the problems in EPI in a structured manner.

@gardncl I believe this problem is solved yes?


[docs] Add optional database param and update /delete connector field

Added the doc fix of https://github.com/cloudera/hue/pull/2674 in this PR only.


Implement finite section based adaptive QL

Another PR related to https://github.com/JuliaApproximation/SemiclassicalOrthogonalPolynomials.jl/pull/70.
Here, I am working towards implementing infinite dimensional QL based on taking the QL decomposition of finite sections and checking for convergence in the desired block compared to a much larger finite section. The basic functionality works. Some stuff I am currently thinking about:
1) How to expose this functionality, if at all. Overloading ql() would be weird considering the method inherently wants a tolerance and the way the factors are updated does not really lend itself to the MatrixFactorizations type. Perhaps it should stay its own thing that a user only calls when they know what they are doing? I am very open to suggestions on this.
2) sort a  bunch of optimizations including getting the MemoryLayout of the adaptive cached L array to be LowerTriangular etc.

I believe tests are failing bc of an issue possibly (?) unrelated to my additions, see https://github.com/JuliaLinearAlgebra/InfiniteLinearAlgebra.jl/issues/126
I will have a closer look later.

There’s no reason to do QL on the full finite section. One can check convergence by checking just a piece of the bottom right
though the details might be tricky

There’s no reason to do QL on the full finite section.

Do you mean that as in when updating/expanding an existing finite section?

Right. You can look at Q,L = ql(A[m:n, m:n]). If L[1:b,1:b] is unchanged as n -> ∞ n is sufficiently large. But we can also let m grow with n.

That would make the computation a lot cheaper for large n, I will look into it.

Alright, I think functionality-wise this can do everything I need for the SemiclassicalOPs application we have in mind. @dlfivefifty Any further comments? Also looking for opinions on if and if yes how to expose this to users or to just have people explicitly import this.

This isn’t a good way to do things. One should follow the Adaptive QR example and have a single data structure to represent the underlying ql(A).factors (which is a banded matrix that contains both the Q and L). Then one can deduce the L via LowerTriangular(ql(A).factors) and the Q similarly.
It is odd in this version that you are using a single type for storing both Q and L, which shouldn’t be constructed at all.

I can match the QL structure but I will have to see how that works for Q without recomputing it

So @dlfivefifty just to confirm, you don't want me to mimic QLProduct as implemented in this package here: https://github.com/JuliaLinearAlgebra/InfiniteLinearAlgebra.jl/blob/8b122c89a429d325d4565c5536fb3f87479ccc59/src/banded/infqltoeplitz.jl#L148
You want it to follow this struct: https://github.com/JuliaLinearAlgebra/MatrixFactorizations.jl/blob/d72eb0626794736c7096f7b65c28494625020153/src/ql.jl#L37
Is that correct? It should be fine either way, just need to know what to work towards

Okay, now featuring .factors and .τ syntax - probably still some things to tidy up but it all works fine and we can now choose not to construct Q if it isn't needed in the usual QL syntax way. Still not overloading ql() at all for now as I am unsure on what the best way to do that would be.
Note that this relies on the current master branch of MatrixFactorizations, so we should version tag that before we merge this.

No idea what happened with CI here but this is doing everything I want now and all tests pass when julia actually builds (?).
@dlfivefifty any further comments or is this good to merge? I am trying to wrap up the semiclassical decomposition stuff and reduce the number of loose branches floating around

alright, the CI demons are gone and it all passes.

FYI we probably want adaptive ReverseCholesky as well. But this will involve making a ReverseCholesky first! That lives in MatrixFactorizations.jl

I assume reverse Cholesky can be done in the exact same way, i.e. finite reverse cholesky then double size until convergence.
Reverse cholesky for finite matrices reduces to cholesky via the "flip" matrix (I don't know what it's actually called) which exchanges top left and bottom right corners. We just need something that keeps everything banded while doing this (i.e. grab the bands, reverse the order, recreate a bandedmatrix) operation and it should be easy from there.

Also add some modifications like using rowsupport and colsupport so you automatically get fast banded reverse Cholesky

@dlfivefifty The big changes:

Now overloads ql(A::InfBandedMatrix{T}, tol = eps(float(T))), with optional argument for manual tolerance change, and uses QL struct. The remaining three structs are one for .factors one for .tau and one optional one if the user calls Q.
Renamed many variables. The important three main structs are now called AdaptiveQLFactors, AdaptiveQLTau,
and AdaptiveQLQ.

@dlfivefifty I think this is also ready to be merged now.

tests passing again with lazy Q instead of cached

@marcusdavidwebb this might be of interest

Note right now the algorithm returns the degenerate QL without errors:
julia> A = _BandedMatrix(Vcat(2*Ones(1,∞), ((1 ./(1:∞)).+1/4)', Ones(1,∞)./3), ℵ₀, 1, 1)
ℵ₀×ℵ₀ BandedMatrix{Float64} with bandwidths (1, 1) with data vcat(1×ℵ₀ Fill{Float64, 2, Tuple{Base.OneTo{Int64}, InfiniteArrays.OneToInf{Int64}}} with indices Base.OneTo(1)×OneToInf(), (((Int64) ./ (ℵ₀-element InfiniteArrays.InfUnitRange{Int64} with indices OneToInf()) with indices OneToInf()) .+ (Float64) with indices OneToInf())' with indices Base.OneTo(1)×OneToInf(), 1×ℵ₀ Fill{Float64, 2, Tuple{Base.OneTo{Int64}, InfiniteArrays.OneToInf{Int64}}} with indices Base.OneTo(1)×OneToInf()) with indices Base.OneTo(3)×OneToInf() with indices OneToInf()×OneToInf():
 1.25      2.0        ⋅         ⋅        …
 0.333333  0.75      2.0        ⋅
  ⋅        0.333333  0.583333  2.0
  ⋅         ⋅        0.333333  0.5
  ⋅         ⋅         ⋅        0.333333
  ⋅         ⋅         ⋅         ⋅        …
  ⋅         ⋅         ⋅         ⋅
 ⋮                                       ⋱

julia> ql(A).Q # Q is not orthogonal
ℵ₀×ℵ₀ MatrixFactorizations.QLPackedQ{Float64, InfiniteLinearAlgebra.AdaptiveQLFactors{Float64}, InfiniteLinearAlgebra.AdaptiveQLTau{Float64}} with indices OneToInf()×OneToInf():
 0.0  -1.0   0.0   0.0   0.0   0.0  …
 0.0   0.0  -1.0   0.0   0.0   0.0
 0.0   0.0   0.0  -1.0   0.0   0.0
 0.0   0.0   0.0   0.0  -1.0   0.0
 0.0   0.0   0.0   0.0   0.0  -1.0
 0.0   0.0   0.0   0.0   0.0   0.0  …
 0.0   0.0   0.0   0.0   0.0   0.0
 ⋮                             ⋮    ⋱

Was this intentional?

Also it only works for tridiagonal but doesn't actually error for larger bandwidths

The tridiagonal part is certainly not intentional. I think it should be an easy fix but I have to check where I assumed tridiagonal (probably in the adjoint lmul? I'll check and push a bugfix soon).
For the degenerate Q that was intentional but I guess it is up for debate whether it should have that behaviour? If we don't want it to do this then we'd have to think about how the algorithm is supposed to detect if it runs into that scenario.

I think the only possible case of a degenerate QL is the shift operator so it could just check if the first column of Q is zero (which probably has a simple condition on the τ but I'd have to think about it)

I believe this update fixed the tridiagonal assumption. Once you have MatrixFactorizations.jl updated we can remove some of the then-redundant overloads here.
Next I will have a look what we can do about detecting the degenerate case via tau. Just to clarify, you think the correct behavior should be to issue an error when this occurs?

Ideally it would error as it is no long orthogonal, just an isometry

The tests should "pass" once https://github.com/JuliaArrays/LazyArrays.jl/pull/250 is tagged but you'll notice I found another couple broken cases due to assuming tridiagonal

Yep, I saw the broken tests. Will investigate

I guess the problem with multiplication by Q as in the broken tests is that this will result in an infinite vector with all non-zero entries even if the vector we are multiplying with has only finitely many non-zeros.
Alternatives:

I can implement it to give a custom cached infinite output vector with it knowing how to adaptively resize. Guess this would require a new struct <:AbstractCachedVector.
alternatively we can just return ApplyArray(*,Q,B) which already works (but I assume this is slower).

Which do you prefer?

For tridiagonal A the only degenerate case for Q is the shift operator (since Q must be lower Hessenberg and there are limited forms this can take), but for higher bandwidths I'm not sure that there can't be other cases where Q is merely a transposed isometry instead of unitary.

The best thing to do the same thing as QR and apply the householder reflections until convergence:
https://github.com/JuliaLinearAlgebra/InfiniteLinearAlgebra.jl/blob/3ae935acfa3248ecbdb0f72179d1716749a2e0c5/src/infqr.jl#L282
But since Q * b is not really needed for solving systems just leaving it lazy for now is fine

but for higher bandwidths I'm not sure that there can't be other cases where Q is merely a transposed isometry instead of unitary.

Hmm maybe we just ignore this issue for the time being and return the isometry. We can return to the question how to detect if Q is merely an isometry later. (Probably there's a simple check of linear independence in the first u-1 columns where u is the upper bandwidth but that'll take some thought)

To clarify: The upper triangular shift isn't an isometry, but it's transpose is.
Also, now I think about it properly, there are other possible "transpose isometries" that are Hessenberg. Just times the shift operator by a Hessenberg Givens matrix.
Is it possible to find multiple QL factorisation for a given operator by using this finite section approach?

Consider:
Q =
[1 0 ..... ]
[0 0 1 ... ]
[0 0 0 1 ... ]
[0 0 0 0 1 ... ]
......

This operator satisfies QQ' == I but not Q'Q == I. The tail would always need to be a shift if it is orthogonal and Hessenberg.

Okay, I replaced multiplication by Q with lazy multiplication. We can always make this more sophisticated in the future if we want to multiply by Q in some application.

Is it possible to find multiple QL factorisation for a given operator by using this finite section approach?

Maybe.. I haven't thought about it. Perhaps one could use an infinite orthogonal matrix U as a left-preconditioner to change what Q we converge to, then apply U' to the resulting Q? Do we have a constructed example where we know two Qs exist and what they look like?

Do we have a constructed example where we know two Qs exist and what they look like?

Yes, any toeplitz operator where the winding number of the symbol is > 0 (or maybe it’s < 0)

The “branch” optional argument examples on the tests should provide examples

I will have a look and see if some modification can change what we converge to.

Deflation?

But this is a research question so not needed to merge this PR

The only open question for this PR is how we handle the degenerate case. If we leave it as is, then Julia will incorrectly overload ldiv for the shift operator special case with lmul by Q', causing a silent bug. But I guess we said we leave it like this for now?


Re-order rows

Hi! Great extension, really useful!
Would it be possible to implement row re-ordering? Or the ability to insert a new row in between a particular row number?

Great timing on that request - it was right at the top of my to-do list!
Latest commit has a (fairly primitive, experimental) version of this. An index column is automatically added (and recreated whenever you save or load). Change the values in that column, then sort by that column; when you save the display order will be the save order (and the index column re-created to reflect the order). Note that you aren't constrained to integer values (so use 1.5 to go between 1 and 2...).
Probably makes more sense hands-on than me describing it. Let me know if it works for you, and feel free to suggest changes,


Add a Transaction.get_weight() method, check it in fuzzing

Based on #68, cause I'm lazy.

Rebased.


Cow: Watch planmedicaltourism.com

Cow requests the watch of the watch_keyword planmedicaltourism\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

planmedicaltourism\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Report the content outlines

extract the headings outline
extract the HTML outline
extract the Nav Doc toc outline
add the outlines to the report

Done in v0.2.0


Recommended Heroku Deploy Not Working

Getting a near-default hubt up and running quickly with heroku yields the following:
/tmp/build_98cfa1dd4cdc4c9b4af155631ee1d3db/.heroku/node/lib/node_modules/npm/bin/npm-cli.js:79

      var notifier = require('update-notifier')({pkg})

                                                    ^

SyntaxError: Unexpected token }

    at Module._compile (module.js:439:25)

    at Object.Module._extensions..js (module.js:474:10)

    at Module.load (module.js:356:32)

    at Function.Module._load (module.js:312:12)

    at Function.Module.runMain (module.js:497:10)

    at startup (node.js:119:16)

    at node.js:945:3

-----> Build failed

Steps to reproduce:
Follow the hubot getting started instructions (https://hubot.github.com/docs/)

This turned out to be very outdated node engine limit in the default yo-generated package.json.


Add canonical url to frontmatter

This allows you to pass it in from the file, too

Folded into #11.


Add Tetris in Bootloader

Sorry im new to PRS i didnt knew it would update automatically when i do a  new commit


Fix failover Master might not release taskGroup

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (2119e41) 38.17% compared to head (e44787d) 38.17%.

:exclamation: Current head e44787d differs from pull request most recent head 2e23ea5. Consider uploading reports for the commit 2e23ea5 to get more accurate results

Files
Patch %
Lines

.../server/master/runner/WorkflowExecuteRunnable.java
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##                dev   #15287   +/-   ##
=========================================
  Coverage     38.17%   38.17%
- Complexity     4666     4667    +1
=========================================
  Files          1278     1278
  Lines         44479    44441   -38
  Branches       4783     4761   -22
=========================================
- Hits          16978    16967   -11
+ Misses        25639    25612   -27
  Partials       1862     1862

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


As a developer/designer, I want to understand the limitations of mapping tech and how this relates to designs, so that I can move quickly and focus on the right things

Description
We have come across a number of questions so far from design about proposed upcoming specs. We should research these questions so that we can come into discussions better informed and ready to weigh in on proposed next steps
Solution
We should enumerate a list of questions in "tasks" below, write out our research findings in this ticket, and come back to design to discuss before specs are finalized for next sprint
Describe alternatives you've considered
If either dev or design goes rogue we will have unimplementable or unusable solutions.
Links to user research or other resources
N/A
Tasks
We need to answer the below questions:

[ ] How can we make block groups more user-friendly as a concept? Can we associate a block group with a "census place" in some way? How difficult would it be do to do so? Is it sufficient to use county?
[ ] What is the best way to query non-geospatial information from our stored tiles and/or geoJSON given our new flat-file-first setup?
[ ] What is the best way to represent tabular data, taking into account paging,
[ ] What is the best way to combine choropleth layers that cover the same geographic space? we can only really have one enabled at a time, but need to talk through implications of this with design

Definition of "Done"
We have written out answers to the above questions in this ticket

@NatHillardUSDS I see this one snuck into our sprint :P What is the status here?

I have done research on these, will post results below

Awesome, thank you!

We may also want to have a dev design sync on the answers too. Feel free to go ahead and put that on the cal for a time that makes sense :)

I am going to schedule this convo for Monday!

Conversation scheduled for tomorrow after standup.

Wrote up the answers to these and shared results with core team in sync just now, so closing this one out


Fix fetcher throughput

The second stage of the fetcher couldn't proceed with flushing when the output method was not ready, which limited the fetch throughput. This PR fixes that.
This change makes the loop in the fibonacci test run at IPC=1 basically, instead of 0.66 (there were two unnecessary wait cycles which this PR removes).
Edit: Looks like this change somehow uncovered a correctness problem.

I found the bug. The problem was when the fetch unit was being stalled by an exception. In that case we want to set flushing_counter to the number of items in the pipeline, but m.d.sync += flushing_counter.eq(flushing_counter - 1) took precedence and set the counter to 0. This wasn't not a problem before the change, because stall_exception would block the second stage transaction and wouldn't let override the flushing counter.
The fix:
diff --git a/coreblocks/frontend/fetch/fetch.py b/coreblocks/frontend/fetch/fetch.py
index c3736066..bd7806e9 100644
--- a/coreblocks/frontend/fetch/fetch.py
+++ b/coreblocks/frontend/fetch/fetch.py
@@ -93,9 +93,6 @@ class FetchUnit(Elaboratable):
         def flush():
             m.d.comb += flush_now.eq(1)

-        with m.If(flush_now):
-            m.d.sync += flushing_counter.eq(req_counter.count_next)
-
         current_pc = Signal(self.gen_params.isa.xlen, reset=self.gen_params.start_pc)

         stalled_unsafe = Signal()
@@ -365,6 +362,9 @@ class FetchUnit(Elaboratable):
                 with branch(flushing_counter != 0):
                     m.d.sync += flushing_counter.eq(flushing_counter - 1)

+        with m.If(flush_now):
+            m.d.sync += flushing_counter.eq(req_counter.count_next)
+
         @def_method(m, self.resume, ready=(stalled & (flushing_counter == 0)))
         def _(pc: Value, resume_from_exception: Value):
             log.info(m, True, "Resuming new_pc=0x{:x} from exception={}", pc, resume_from_exception)

Looks like the benchmark numbers didn't change much. But if you look into the metrics, you can see that ROB utilization increased (see e.g. rob.size/bucket-32). The performance gain was probably eaten up by the miss penalty.


Too many circular dependencies in react-stately/vitualizer package

Please fix circular dependencies in react-stately/vitualizer package

Do you happen to know what they are? It'd help us fix them faster. What tool did you find it with?

Closing due to inactivity


ChatGPT Dracula Theme

ChatGPT Dracula Theme
ChatGPT Dracula Theme - GitHub Repository
As a Dracula Theme "lover", I’ve created a custom Dracula theme for ChatGPT, utilizing the signature color palette of the Dracula theme to personalize the interface. In this design, I’ve focused on improving aesthetics by incorporating gradients to enhance key elements of the UI.
A special emphasis has been placed on optimizing code syntax highlighting to improve readability and clarity, making it easier to distinguish between different parts of the generated code. My goal was to not only stay true to the Dracula theme, but also to enhance the overall user experience.
Since it is an Extension in the Chrome Web Store, anyone can download it for free and remove it at any time they want. I hope this extension satisfies many of the Dracula Theme fans.

Hey 👋🏻
While this is quite interesting, since it's an extension meant to be installed, the source code and installation instructions should be included in the same repository. Additionally, it's worth considering whether installing an extension to modify a page/platform's style is warranted. For example, why not use Stylus instead?

It could be a theme for the Stylus Extension.
So it would work in any browser that supports stylus like Firefox for example.

Ok, I'll think about that. By the way, what do you think about my theme? Do you think it is well-made enough that it's going to be appealing to the Dracula Theme users? Should I publish it?

It looks very good. I don't know why but I installed the extension and it didn't work for me.


Remove duplicate "internal" constants for service keys

In go-mod-core-contracts we have available the following constants (https://github.com/edgexfoundry/go-mod-core-contracts/blob/master/clients/constants.go):
const (
	ServiceKeyPrefix                = "edgex-"
	ConfigSeedServiceKey            = "edgex-config-seed"
	CoreCommandServiceKey           = "edgex-core-command"
	CoreDataServiceKey              = "edgex-core-data"
	CoreMetaDataServiceKey          = "edgex-core-metadata"
	ExportClientServiceKey          = "edgex-export-client"
	ExportDistroServiceKey          = "edgex-export-distro"
	SupportLoggingServiceKey        = "edgex-support-logging"
	SupportNotificationsServiceKey  = "edgex-support-notifications"
	SystemManagementAgentServiceKey = "edgex-sys-mgmt-agent"
	SupportSchedulerServiceKey      = "edgex-support-scheduler"
)

which happen to also be in located in https://github.com/edgexfoundry/edgex-go/blob/master/internal/constants.go. The exception is the WriteableKey which for now I have left in internal as it is not in go-mod-core-contracts, but potentially could be moved over as it seems relevant and could be used for other services as well.
With the intention of removing duplicate code, this pull request removes the duplicate "internal" constants for service keys and leverage go-mod-core-contracts throughout edgex-go. This pull request does not depend on any others and is purely a code clean up thing. Noticed it as I was adding the client info sections to the app-functions-sdk.
Signed-off-by: Mike Johanson<EMAIL_ADDRESS>

@rsdmike Thanks for the work. In the future please associate all PRs to an issue. I know this was an ad hoc thing but create the issue first.

Will do!

@rsdmike This looks good. Please make the updates to the go.mod we discussed yesterday. Update the ref to go-mod-core-contracts and also please update the following:
github.com/ugorji/go v1.1.4

Once that's done, let me know and we'll merge.


fatal error: 'ciso646' file not found

Description:

mac os high sierra,
macport 2.5.4,
llvm, clang 7.0,
boost 1.66.0_3 (devel)...
python 3.6.x

RDKit Version: 'latest master branch' of the RDKit
Platform: Mac OS High Sierra

I tried to compile by using clang 7.0 on the mac os high sierra
and it fails with an error message,

CMake configure
sh-3.2# cmake -D EIGEN3_INCLUDE_DIR=/opt/local/include/eigen3 \
  -D PYTHON_LIBRARY=/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/libpython3.6m.dylib \
  -D PYTHON_INCLUDE_DIR=/opt/local/Library/Frameworks/Python.framework/Versions/3.6/include/python3.6m \
  -D PYTHON_EXECUTABLE=/opt/local/bin/python3.6 -DBOOST_ROOT=/opt/local/include/boost \
  -DBOOST_INCLUDEDIR=/opt/local/include/boost \
  -DPostgreSQL_INCLUDE_DIR=/opt/local/include/postgresql96 \
  -DPostgreSQL_TYPE_INCLUDE_DIR=/opt/local/include/postgresql96 \
  -DPostgreSQL_CONFIG_DIR=/opt/local/lib/postgresql96/bin \
  -DCMAKE_MACOSX_RPATH=1 \
  -DRDK_BUILD_INCHI_SUPPORT=ON \
  -DRDK_BUILD_CAIRO_SUPPORT=ON \
  -DRDK_BUILD_PGSQL=ON \
  -DRDK_USE_BOOST_SERIALIZATION=OFF ..;
-- The C compiler identification is AppleClang <IP_ADDRESS>20039
-- The CXX compiler identification is AppleClang <IP_ADDRESS>20039
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check if the system is big endian
-- Searching 16 bit integer
-- Looking for sys/types.h
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of unsigned short
-- Check size of unsigned short - done
-- Using unsigned short
-- Check if the system is big endian - little endian
-- Found Catch2 source in /opt/local/RDKit/External/catch/catch
CATCH: /opt/local/RDKit/External/catch/catch/single_include
-- Could NOT find InChI in system locations (missing: INCHI_LIBRARY INCHI_INCLUDE_DIR)
CUSTOM_INCHI_PATH = /opt/local/RDKit/External/INCHI-API
-- Found InChI software locally
-- Found PythonInterp: /opt/local/bin/python3.6 (found version "3.6.8")
-- Found PythonLibs: /opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/libpython3.6m.dylib (found version "3.6.8")
PYTHON Py_ENABLE_SHARED: 0
PYTHON USING LINK LINE: -bundle -undefined dynamic_lookup -L/opt/local/lib -Wl,-headerpad_max_install_names
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - found
-- Found Threads: TRUE
-- Found PostgreSQL: /usr/lib/libpq.dylib (found version "9.6.11")
== Using strict rotor definition
-- maeparser include dir set as 'maeparser_INCLUDE_DIRS-NOTFOUND'
-- maeparser libraries set as 'maeparser_LIBRARIES-NOTFOUND'
-- Could NOT find maeparser (missing: maeparser_INCLUDE_DIRS maeparser_LIBRARIES)
-- Found MAEParser source in /opt/local/RDKit/External/CoordGen/maeparser
-- coordgen include dir set as coordgen_INCLUDE_DIRS-NOTFOUND
-- coordgen libraries set as 'coordgen_LIBRARIES-NOTFOUND'
-- coordgen templates file set as 'coordgen_TEMPLATE_FILE-NOTFOUND'
-- Could NOT find coordgen (missing: coordgen_INCLUDE_DIRS coordgen_LIBRARIES coordgen_TEMPLATE_FILE)
-- Found coordgenlibs source in /opt/local/RDKit/External/CoordGen/coordgen
== Making EnumerateLibrary without boost Serialization support
== Making FilterCatalog without boost Serialization support
== Updating Filters.cpp from pains file
== Done updating pains files
-- Found PkgConfig: /opt/local/bin/pkg-config (found version "0.29.2")
-- Checking for module 'cairo'
--   Found cairo, version 1.16.0
-- Found Cairo: /opt/local/include/cairo
-- Found RapidJSON source in /opt/local/RDKit/External
postgres: /opt/local/include/postgresql96;/opt/local/include/postgresql96

=====================================================================
After building, check
/opt/local/RDKit/build/Code/PgSQL/rdkit/pgsql_install.sh
for correctness of installation paths. If everything is OK, gain
administrator privileges, stop the PostgreSQL service, run
/opt/local/RDKit/build/Code/PgSQL/rdkit/pgsql_install.sh
to install the PostgreSQL RDKit cartridge, then start again
the PostgreSQL service
=====================================================================

-- Configuring done
-- Generating done
-- Build files have been written to: /opt/local/RDKit/build

# make -j4 -l4
[ 14%] Linking CXX executable testMatCalc
[ 14%] Built target testMatCalc
[ 14%] Building C object Code/ML/Cluster/Murtagh/CMakeFiles/hc_static.dir/hc.c.o
In file included from /opt/local/RDKit/Code/ML/Cluster/Murtagh/hc.c:6:
In file included from /opt/local/RDKit/Code/ML/Cluster/Murtagh/f2c.h:7:
In file included from /opt/local/RDKit/Code/RDGeneral/export.h:9:
In file included from /opt/local/include/boost/config.hpp:39:
/opt/local/include/boost/config/compiler/clang.hpp:243:10: fatal error: 'ciso646' file not found
#include <ciso646>
         ^~~~~~~~~
1 error generated.
make[2]: *** [Code/ML/Cluster/Murtagh/CMakeFiles/hc_static.dir/hc.c.o] Error 1
make[1]: *** [Code/ML/Cluster/Murtagh/CMakeFiles/hc_static.dir/all] Error 2
make: *** [all] Error 2

locate ciso646 will print the fullpath to ciso646 (maybe different from https://lists.llvm.org/pipermail/lldb-dev/2012-August/001019.html) if your machine has it.
What happens if appending the parent directory to CPLUS_INCLUDE_PATH environment variable?

I tried locate ciso646, but there is message repeated "the database can take some time to generate". I found ciso646 header file path with getting some hints by executing below command. I guessed that this header file seems to be somewhere around resource directory.
clang -print-resource-dir
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/clang/10.0.0
so, It's,..
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1
Ok, great! but It did not included as CPLUS_INCLUDE_PATH within cmake command line, but I tried to "export" as ENVVAR.
export CPLUS_INCLUDE_PATH="/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include/c++/v1"
and tried,
make -j5 -l8  (for my mba 2011 mid.)
Ah... It also shows 'ciso646' file not found.
then ... How about to include option info CFLAGS and CXXFLAGS?

I realized that there is no custom variable allowed to set CPLUS_DIRECTORY_PATH for RDKit cmake command.

Removing #include <ciso646> from /opt/local/include/boost/config/compiler/clang.hpp also seems to do the trick.  A conforming implementation of this header does nothing (https://en.cppreference.com/w/cpp/header/ciso646) and this line doesn't appear to actually be in the upstream boost sources, so I assume this is safe (also, it builds just fine).
Not an RDKit issue at all.


Update README.md

Just a start for User Story and how we stand apart!

Market Research Insights
To refine this platform specifically for bootcamp students collaborating during classroom times, consider these additional insights:
Collaborative Learning Trends: Educational institutions are increasingly adopting collaborative learning approaches, which are proven to enhance engagement and knowledge retention. Your platform can cater to this trend by providing tools that facilitate real-time collaboration and peer-to-peer learning.
Needs of Bootcamp Students: Bootcamp students often have varying levels of experience and need to learn both coding skills and teamwork. A platform that integrates coding, communication, and project management into one seamless experience will help them build these skills more effectively.


Adopt CMS_PATH and SITECONFIG_PATH to locate the site catalog

Fixes #11449
Status
not-tested
Description
In light of the migration from storage.xml to storage.json, CMSSW went through some modifications to support those catalogs, in short:

any releases made before 12_6_0_pre2 will support storage.xml (through CMS_PATH environment variable)
any releases starting in 12_6_0_pre3 (exception to 12_6_3) will support only storage.json (through SITECONF_PATH environment variable).

That means, our Scram module needs to keep explicitly defining those variables in the CMSSW environment. As confirmed by Matti below, it should not be a problem to have both variables defined.
Is it backward compatible (if not, which system it affects?)
YES (to the best of my knowledge)
Related PRs
Somehow related to https://github.com/dmwm/WMCore/issues/11472
External dependencies / deployment changes
None

@amaltaro I think WM should keep the propagation of CMS_PATH. CMSSW versions before 12_6_0_pre2 still rely on CMS_PATH for the location of site-local-config.xml. It's ok to have both defined (cmsset_default.sh anyway defines both if they are not already defined)

Noted and code updated in the Scram module. Thanks Matti.

The way I understand it is that SITECONFIG_PATH is actually defined in the CMS pilot (the same as it is for CMS_PATH). So I'd simply take it for granted as we used to do for CMS_PATH.
Regarding the local symlink, it should be just a special file name that is part of the absolute path. I fail to see any problems with that.

The way I understand it is that SITECONFIG_PATH is actually defined in the CMS pilot (the same as it is for CMS_PATH). So I'd simply take it for granted as we used to do for CMS_PATH.

This is correct.
There is one detail in the conceptual difference between SITECONFIG_PATH and CMS_PATH that may or may not be relevant here. By default SITECONFIG_PATH is defined as $CMS_PATH/SITECONF/local, but sites are allowed to override the SITECONFIG_PATH to be something else for sub-sites. This works fine for CMSSW releases >= 12_6_0_pre2 that use the SITECONFIG_PATH.
But I wonder if there is anything to be concerned about for CMSSW releases <= 12_6_0_pre1 that still use CMS_PATH? (mainly because I don't know what information exactly WM uses from the site-local-config.xml and for what) My understanding is that sites ought to keep storage.xml and storage.json consistent, and the subsites should function in some sensible way with CMSSW releases using CMS_PATH, but I'm not sure if there could be a room for some inconsistency when SITECONFIG_PATH points to different place than CMS_PATH.
Maybe @stlammel would have a better view?

The way I understand it is that SITECONFIG_PATH is actually defined in the CMS pilot (the same as it is for CMS_PATH). So I'd simply take it for granted as we used to do for CMS_PATH.
Regarding the local symlink, it should be just a special file name that is part of the absolute path. I fail to see any problems with that.

Just some extra content here:
From what I see, both variables get defined when sourcing /cvmfs/cms.cern.ch/cmsset_default.sh
(base) [khurtado@earth /etc/profile.d]$ unset CMS_PATH SITECONFIG_PATH
(base) [khurtado@earth /etc/profile.d]$ echo $CMS_PATH $SITECONFIG_PATH

(base) [khurtado@earth /etc/profile.d]$ source /cvmfs/cms.cern.ch/cmsset_default.sh
(base) [khurtado@earth /etc/profile.d]$ echo $CMS_PATH $SITECONFIG_PATH
/cvmfs/cms.cern.ch /cvmfs/cms.cern.ch/SITECONF/local

The environment variable SITECONFIG_PATH defaults to `$CMS_PATH/SITECONF/local" but it changes when they are already pre-defined for subsites:
https://github.com/cms-sw/cms-common/blob/9c8203bf637b153697b855dfa522d6741379bcc9/cmsset_default.sh#L40-L44
An example for a file structure of a subsite would be Wisconsin and its CHTC resource (subsite resource of Wisconsin)
(base) [khurtado@earth /cvmfs/cms.cern.ch/SITECONF]$ ls T2_US_Wisconsin/
GlideinConfig/ JobConfig/     PhEDEx/        storage.json   WISC-CHTC/
(base) [khurtado@earth /cvmfs/cms.cern.ch/SITECONF]$ ls T2_US_Wisconsin/WISC-CHTC/
JobConfig  storage.json

An example for a file structure of a subsite would be Wisconsin and its CHTC resource (subsite resource of Wisconsin)

@khurtado Kenyi, but in this case wouldn't the worker node/pilot use a different local symlink?
In other words, it's up to the site admin to define the correct symlink that each site and subsite has to use. While our jobs simply set the environment and consume the actual catalog files provided by SITECONFIG_PATH (meant to have different different content between subsites, but still accessed through the same path). Or am I misunderstanding something in here?

An example for a file structure of a subsite would be Wisconsin and its CHTC resource (subsite resource of Wisconsin)

@khurtado Kenyi, but in this case wouldn't the worker node/pilot use a different local symlink?
In other words, it's up to the site admin to define the correct symlink that each site and subsite has to use. While our jobs simply set the environment and consume the actual catalog files provided by SITECONFIG_PATH (meant to have different different content between subsites, but still accessed through the same path). Or am I misunderstanding something in here?

I believe the local symlink doesn't change, but rather SITECONFIG_PATH would point to the subsite directory in the local symlinks, e.g.:
SITECONFIG_PATH = /cvmfs/cms.cern.ch/SITECONF/local/WISC-CHTC
But yes, I was just putting some extra information there for the record, regarding what happens with this new variable. But for us, we ultimately don't care how this is set, we just take it for granted. The validation of these paths are done elsewhere (during the setup of a site or sub-site, during the pilot validation tests, etc).

So sites are keeping access via both CMS_PATH (for older CMSSW versions) and SITECONFIG_PATH (for newer CMSSW versions) working right now. How the local link in CVMFS is set is up to each site. Most sites set it to the sitename and site/subsite for subsites but in case of a local SITECONF copy this doesn't need to be the case. ${CMS_PATH}/SITECONF/local is guaranteed to point to a good SITECONF area and ${SITECONF_PATH}.
Hope this helps.

Stephan

Thanks for the review, @khurtado. Some unit tests were actually broken and I managed to get them fixed with this PR. For now, I kept unit tests defining both environment variables, as you suggested. Please have another look

@amaltaro Looking good! There is just one unit test that I think is still missing the CMS_PATH, besides that it all looks good to me.

@amaltaro I tested #11473 and this  on top of T0 agent 3.0.8 (WMCore <IP_ADDRESS>) and the replays were successful, both with CMSSW_12_6_2 (RucioCatalog) and CMSSW_12_6_3 (TrivialFileCatalog)

Thanks everybody for discovering and handling the dependency!   - Stephan

@amaltaro I tested #11473 and this on top of T0 agent 3.0.8 (WMCore <IP_ADDRESS>) and the replays were successful, both with CMSSW_12_6_2 (RucioCatalog) and CMSSW_12_6_3 (TrivialFileCatalog)

Thanks  @germanfgv! Does this mean we could safely switch 12_6_X back to RucioCatalog?

@makortel I still would like to test how it will work with T2_CH_CERN_P5 worker nodes. I'll launch test this tonight with a replay.

@germanfgv In the mean time I took the liberty of opening a PR to change the file catalog to Rucio catalog in 12_6_X in https://github.com/cms-sw/cmssw/pull/40755 . If your tests succeed and the PR could proceed, could mention it in that PR as well?


Update the go version to 1.9.1 in the .travis.yml for openebs/maya

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST

What happened:
Go Versions < 1.8.4, seem to have a couple of security vulnerabilities.
https://golang.org/issue/22125
https://golang.org/issue/22131
It has been recommended by the Go Core Community to upgrade the version of Go to 1.9.1.
What you expected to happen:
All new builds for maya will use the latest version of Go, thereby resolving the  security vulnerabilities.
How to reproduce it (as minimally and precisely as possible):
https://golangweekly.com/link/26068/47a3e5168d
Anything else we need to know?:
Environment:

kubectl get nodes
kubectl get pods --all-namespaces
kubectl get services
kubectl get sc
kubectl get pv
kubectl get pvc
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@yudaykiran in the .travis.yml contains only python2.7 not go

@prashant0598 please look into openebs/maya repo, it is there. The issue has been raised in openebs/openebs for our tracking purposes.

@yudaykiran ya found it 😄. Do i just need to change the go version to 1.9.1?

@prashant0598 , that should be enough :+1:


Add glossary file

Signed-off-by: Christian Berendt<EMAIL_ADDRESS>

As reminded by @itrich  in PB (where this PR came from): we should see about merging #12.

We can merge this one and then rebase the other PR and adjust it a bit in terms of format? I think the glossary should not be included in the design docs but should be available independently for everything.

We can merge this one and then rebase the other PR and adjust it a bit in terms of format? I think the glossary should not be included in the design docs but should be available independently for everything.

Yes, let's proceed as suggested.


fetch inputbox whenever l1finalized is updated

This PR is ready for review whenever you have time :)

reminder to review this :)


[SPARK-8214][SQL]Add function hex

cc @chenghao-intel  @adrian-wang

Merged build triggered.

Merged build started.

Test build #35653 has started for   PR 6976 at commit f534efc.

Merged build triggered.

Merged build started.

Test build #35664 has started for   PR 6976 at commit aaaf841.

Test build #35653 has finished for   PR 6976 at commit f534efc.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

Test build #35664 has finished for   PR 6976 at commit aaaf841.

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

Merged build finished. Test PASSed.

LGTM, except a minor issue. Can you also remove the WIP?

Merged build triggered.

Merged build started.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [spark] [SPARK-8214][SQL]Add function hex (#6976)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.

Test build #35818 has started for   PR 6976 at commit a647641.

Test build #35818 has finished for   PR 6976 at commit a647641.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

@rxin could you please take a look at this ?

Merged build triggered.

Merged build started.

Test build #35950 has started for   PR 6976 at commit 3b2fa13.

Merged build triggered.

Merged build started.

Test build #35951 has started for   PR 6976 at commit 967ec90.

Test build #35950 has finished for   PR 6976 at commit 3b2fa13.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

Test build #35951 has finished for   PR 6976 at commit 967ec90.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #35957 has started for   PR 6976 at commit cf9c936.

Test build #35957 has finished for   PR 6976 at commit cf9c936.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #35972 has started for   PR 6976 at commit de3f5ea.

Test build #35972 has finished for   PR 6976 at commit de3f5ea.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #35978 has started for   PR 6976 at commit e218d1b.

Test build #35978 has finished for   PR 6976 at commit e218d1b.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Hex(child: Expression)
case class Hypot(left: Expression, right: Expression)

Merged build finished. Test PASSed.

LGTM, merging this into master, thanks!


feat(babel): Added babel transpiler to build system

Hey, awesome job, @justindra 😄

Thanks @Gottwik!


Improve Nuxt test-apps and docs

Hello @pilcrowOnPaper ! The Nuxt app wasn't working, so I fixed a bug and made some improvements along the way.

Fix a bug where the callback route was using post instead of get.
Leverage useFetch and native errors everywhere.
Removed pnpm hoisting and added needed deps (this is not officially recommended yet but it works)
Added middlewares to the docs.

The natural place to do authentication logic in Nuxt is with a route middleware (they run both in SSR and CSR alongside vue-router), so I extended  the app and added documentation.

Any differences between useFetch() or $fetch() except for how errors are thrown? Is one preferred over another? (@danielroe)

Any differences between useFetch() or $fetch() except for how errors are thrown? Is one preferred over another? (@danielroe)

useFetch combines useAsyncData and $fetch, which means that useFetch will fetch 1 time (on SSR) and $fetch 2 times (on SSR and CSR).
A good rule of thumb is to use useFetch when you need to talk to your internal API and $fetch when you need to do something more advanced or talk to an external API.

The key difference here is that useFetch is a composable meant to be called directly in a setup function. It returns reactive composables and handles adding responses to the Nuxt payload so they can be passed from server to client without re-fetching the data on client side when the page hydrates.
When wanting to make a network request based on user interaction, $fetch is almost always the right handler to go for. It's used under-the-hood in useFetch and is a drop-in replacement with convenience layers (using the fetch API) for the Fetch API. It works just as well as useFetch for talking to your own internal API and will also make 'internal' calls if invoked on the server side. Both $fetch and useFetch allow you to customise the fetch options via the second argument.

Sorry, there are a lot of duplicate comments as the test + example apps are (virtually?) the same. But the most significant point is not to call useFetch outside of setup.

Thanks for the review Daniel, I just learned something new about useFetch. On a side note, we should add this point about useFetch and setup in the nuxt docs, and perhaps the recommendation of using $fetch for submitting data to an event handler too?

I'm closing this since most issues have been already addressed with the latest PR. Thanks for opening the PR, and also @danielroe for chipping in!


Not able to use a "jumpbox" to create SSH tunnels due hardcoded agent registry

I am trying to deploy a microbosh instance using bosh-init in a VPC without a public IP, but using instead a SSH jumpbox.
In theory this would be technically possible using the SSH tunnel feature, so that bosh-init creates SSH tunnels to the jumpbox and the agent of deployed VM connects to the exposed port in the internal IP of the bosh VM.
But in practice I was not able to do it, and I think the blocker I his was this commit which replaces the IP of the registry which the agent connects to to <IP_ADDRESS> when SSH tunnel is setup.
Is there any reason to force this config?
Am I right thinking that if we fix this code the agent will connect to the registry served as a ssh-tunnel in the jumpbox?
As a workaround, I guess the approach would be create the SSH tunnel manually outside of the bosh-init tool, so bosh-init won't override the registry values.

The ssh_tunnel configuration is used by bosh-init to reverse tunnel registry-access from the VM to come back to the local bosh-init process (docs). This is why the registry connection is hard-coded to <IP_ADDRESS>:6901 and ssh_tunnel.ip is supposed to be an accessible IP that your bosh-init can SSH to (either by LAN, VPN, or WAN).
When using a jumpbox it gets a bit more difficult since, as you suggest, you need to manually set up an SSH tunnel and forward the ports that bosh-init and the VM agent care about. For example, if MICROBOSH_IP is your instance without a public IP and JUMPBOX_IP is your publicly-accessible jumpbox IP...
Before running bosh-init, create local, forwarding ports for SSH and mbus so bosh-init can communicate with the VM it'll create...
ssh -L 33322:$MICROBOSH_IP:22 -L 36868:$MICROBOSH_IP:6868 $JUMPBOX_IP

Then you'll need to modify your bosh-init.yml so it talks directly to those local ports that you're forwarding...
cloud_provider:
  ssh_tunnel:
    host: <IP_ADDRESS>
    port: 33322
    ...
mbus: "https://mbus:mbus-password@<IP_ADDRESS>:36868"
...

Something like that should work for jumpbox-ing.

Yes, the idea would be additionally set the reverse proxy of the registry
ssh -R 6901:localhost:6901 -L 33322:$MICROBOSH_IP:22 -L 36868:$MICROBOSH_IP:6868 $JUMPBOX_IP

And the client would use the private ip of the jumpbox:
registry:
  host: (( jumpbox_private_ip ))

Indeed this can be implemented by creating the SSH tunnel before running bosh-init, as the jumpbox exists already.
But I wonder if:

Shall we allow bosh-init to create the SSH tunnel to the jumpbox for you? I think the only thing required is remove the code that overrides the registry to <IP_ADDRESS>.

If not: Shall we update the documentation to explain this?

Thank you.


Consolidate context directory for Kaniko and BuildAh

Changes for issue #98.

I am removing pathContext from the README and the samples and replace it by source.contextDir which was already read in the code
I am also updating the build strategy to read from build.source.contextDir for consistency and adopted the generate_taskrun accordingly

/assign @avni-sharma

/assign @zhangtbj

Assigning to @zhangtbj too - I will review shortly, but feel free to get to it before me :)

/ok-to-test

Thanks for the PR @SaschaSchwarze0 ,
The function looks good to me.
I think we need another PR to refine the git for the current e2e tests.

Change all tests to use more official or common git repo
Change another buildpacks(namespaced) to try another buildpack and git repo
Change one buildah and another kaniko to try another git or language like golang.

I opened another issue to track:
https://github.com/redhat-developer/build/issues/106

Validated e2e locally because travis e2e tests are yet to be up ( work-in-progress ).
TEST_IMAGE_REPO=quay.io/shbose/nodejs-ex:latest TEST_IMAGE_REPO_SECRET=regcred  operator-sdk test local ./test/e2e --up-local --namespace build-examples --go-test-flags -timeout=20m
]INFO[0000] Testing operator locally.
ok  	github.com/redhat-developer/build/test/e2e	556.219s

operator-sdk test local ./test/e2e --up-local --namespace build-examples --go-test-flags "-timeout=20m"

/lgtm
/approve


add OpenStack Swift under cloud-native storage

Despite OpenStack already being mentioned under "Bare Metal Provisioning", Swift should be added under "Cloud-Native Storage".
Swift does not provide bare metal provisioning, and it is closely associated with the other projects listed under cloud native storage. It is commonly deployed (and fully supported) independently of other OpenStack projects.
Swift's project logo is at https://www.openstack.org/themes/openstack/images/project-mascots/Swift/OpenStack_Project_Swift_vertical.png
Swift's GitHub mirror is at https://github.com/openstack/swift

Hi @notmyname, we will include OpenStack Swift in the upcoming version, releasing soon! If you can reply back today with an updated logo path (one provided is not working), we'll include that otherwise we'll place "Swift" text underneath the OpenStack logo. Thanks

I'm happy to hear it, and I'm sorry the other link isn't working for you. I've re-uploaded it to http://d.not.mn/OpenStack_Project_Swift_vertical.png. Does that link work for you?

Thanks for the quick response @notmyname I've updated the logo per your link.

Hi @notmyname, closing as this update was included in the most recent version https://github.com/cncf/landscape#current-version


ScipyOptimizer options are overriden by default values

Information

Qiskit Terra version:
Python version: 3.7.9
Operating system: macOS

What is the current behavior?
The ScipyOptimizers allow two ways to specify settings of an optimizer:

directly as named arguments (e.g. L_BFGS_B(maxiter=10))
via an options argument (e.g. L_BFGS_B(options={"maxiter": 10}))

However using the second method via options, does not actually set the options since they are overriden by the default value.
Steps to reproduce the problem
from qiskit.algorithms.optimizers import L_BFGS_B

opt = L_BFGS_B(options={"maxiter": 2})
print(opt._options)

prints
{'maxiter': 15000, 'maxfun': 1000, 'ftol': 2.220446049250313e-15, 'iprint': -1, 'eps': 1e-08}

What is the expected behavior & suggested solution
I think it would be cleaner to not have the options in the initializer and let the user just use named arguments. We could still keep _options under the hood in the SciPyOptimizer, just not expose it.
Alternatively we could make sure all named arguments are None by default and then only set the value in the options if they are not none. However that is a bit cumbersome and less nice to read in the docs, so I think just removing options from the initializer would be cleaner.

I would like to contribute to this

Great! How would you like to solve the issue? 🙂

I think the methods mentioned in the issue itself are good paths to solve the given problem:

Simply remove options this will take away the problem.
Correct the options parameter by initialling the parameters to None and whatever other thing is required. - I think this would be prefered as it provides more options to the user

Ok, sounds good @singhmeet11 let's go with option 2 👍🏻


Migarate to AndroidX

I am using sardine-android as submodule. I am facing "error: package android.support.annotation does not exist" with "android.support", so I migrate project to AndroidX.

I'm not in favor of migrating to Androidx as it would break the compatibility with those who are still using the Support Library. However, we could get rid of the Support Library in this project as it's only used for annotations I think. It would make the library thiner and solve the issue of AndroidX at the same time.

@rt1shnik Can you try this branch? https://github.com/thegrizzlylabs/sardine-android/tree/remove_support_library

@guillaume-tgl
It's fine.


Compile-AppInNavContainer with dependency generates an error

Dear Freddy,
I am trying to use Compile-AppInNavContainer in project where I have the dependency in the Application (App.json) but I get strange error then:
ConvertFrom-Json : Invalid JSON primitive: Â.
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\AppHandling\Compile-AppInNavContainer.ps1:132 char:60
+ ... ppJsonObject = Get-Content -Raw -Path $appJsonFile | ConvertFrom-Json

    + CategoryInfo          : NotSpecified: (:) [ConvertFrom-Json], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,Microsoft.PowerShell.Commands.ConvertFromJsonCommand

when deleting the dependency it works fine (mean this particular project I cannot compile because I have errors related to missing symbols but this is expected).

Sounds like a bug - will investigate

Just tried this:
$auth = "NavUserPassword"
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'P@ssword1' -AsPlainText -Force)
$containerName = "bc"
$imageName = "mcr.microsoft.com/businesscentral/onprem:1904"

New-BcContainer -accept_eula `
                -accept_outdated `
                -containerName $containerName `
                -imageName $imageName `
                -auth $auth `
                -Credential $credential `
                -updateHosts `
                -licenseFile $licenseFile `
                -includeTestToolkit `
                -includeTestLibrariesOnly

$app1ProjectFolder = "c:\programdata\navcontainerhelper\bc-app"
Compile-AppInBCContainer -containerName $containerName -credential $credential -appProjectFolder $app1ProjectFolder -appOutputFolder $app1ProjectFolder -appName "bc1.app" -UpdateSymbols
Publish-NavContainerApp -containerName $containerName -appFile (Join-Path $app1ProjectFolder "bc1.app") -skipVerification -sync -install

$app2ProjectFolder = "c:\programdata\navcontainerhelper\bc2-app"
Compile-AppInBCContainer -containerName $containerName -credential $credential -appProjectFolder $app2ProjectFolder -appOutputFolder $app2ProjectFolder -appName "bc2.app" -UpdateSymbols
Publish-NavContainerApp -containerName $containerName -appFile (Join-Path $app2ProjectFolder "bc2.app") -skipVerification -sync -install

That works.
Can you elaborate on what you are doing? Potentially repro steps?

Hi,
Thx for answer.
Do you have in App.json file dependency between app1 and app2?
 "dependencies": [
    {
      "appId": "My Id",
      "name": "My Name",
      "publisher": "My Publisher",
      "version": "<IP_ADDRESS>"
    }
  ],

I run it on image: mcr.microsoft.com/businesscentral/onprem:13.4.28874.29613-dk
I simply run:
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'P@ssword1' -AsPlainText -Force)

Compile-AppInBCContainer -containerName bcbuild -credential $credential -appProjectFolder "C:\Users\myuser\Documents\AL\App1" -appOutputFolder "C:\Users\myuser\Documents\AL" -UpdateSymbols

Best regards,
Krzysztof

Yes, my dependency is from bc2-app to bc-app (obviously), which is why I compile and publish bc-app first and then bc2-app can get symbols, compile and publish.
my bc2-app json file has:
"dependencies": [
{
"appId": "cb99d78b-f9db-4a1e-822a-0c9c444535df",
"name": "Hello World",
"publisher": "Microsoft",
"version": "<IP_ADDRESS>"
}
],
What is the exact content of your dependency?
Any special characters?

I am trying to reproduce the problem on the simpler (as your example). I have the special character in the Publisher Name for example: Hello A/S
But now I face issue:
Invoke-RestMethod : Could not find a part of the path 'C:\Users\user\Documents\AL\BC2\.alpackages\Hello A\S_hello 250 and hello 32_<IP_ADDRESS>.app'.
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\AppHandling\Compile-AppInNavContainer.ps1:265 char:13
+             Invoke-RestMethod -Method Get -Uri $url @AuthParam -OutFi ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], DirectoryNotFoundException
    + FullyQualifiedErrorId : System.IO.DirectoryNotFoundException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

I run the code:
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'P@ssword1' -AsPlainText -Force)
Compile-AppInBCContainer  -credential $credential -containerName bcbuild  -appProjectFolder "C:\Users\user\Documents\AL\BC2" -appOutputFolder "C:\Users\user\Documents\AL\BC2" -UpdateSymbols

and here is app.json:
{
  "id": "f7cc19ea-4df4-4298-8461-f4c23e003864",
  "name": "Hello 360 Second",
  "publisher": "XXX NewCompany Danmark A/S",
  "version": "<IP_ADDRESS>",
  "brief": "",
  "description": "description description.",
  "privacyStatement": "",
  "EULA": "",
  "help": "",
  "url": "https://www.dxc.technology",
  "logo": "Resources/DXCLogo.png",
  "dependencies": [
    {
      "appId": "c73da900-7dd8-4f93-a364-21f8fd381712",
      "name": "Hello 360 New",
      "publisher": "XXX NewCompany Danmark A/S",
      "version": "<IP_ADDRESS>"
    }
  ],
  "screenshots": [],
  "platform": "<IP_ADDRESS>",
  "application": "<IP_ADDRESS>",
  "idRange":  {"from": 50000, "to": 60000},
  "showMyCode": true,
  "runtime": "2.3",
  "target": "Internal",
  "features": ["TranslationFile"]
}

I am trying to reproduce the problem on the simpler (as your example). I have the special character in the Publisher Name for example: Hello A/S
But now I face issue:
Invoke-RestMethod : Could not find a part of the path 'C:\Users\user\Documents\AL\BC2\.alpackages\Hello A\S_hello 250 and hello 32_<IP_ADDRESS>.app'.
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\AppHandling\Compile-AppInNavContainer.ps1:265 char:13
+             Invoke-RestMethod -Method Get -Uri $url @AuthParam -OutFi ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], DirectoryNotFoundException
    + FullyQualifiedErrorId : System.IO.DirectoryNotFoundException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

I run the code:
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'P@ssword1' -AsPlainText -Force)
Compile-AppInBCContainer  -credential $credential -containerName bcbuild  -appProjectFolder "C:\Users\user\Documents\AL\BC2" -appOutputFolder "C:\Users\user\Documents\AL\BC2" -UpdateSymbols

and here is app.json:
{
  "id": "f7cc19ea-4df4-4298-8461-f4c23e003864",
  "name": "Hello 360 Second",
  "publisher": "XXX NewCompany Danmark A/S",
  "version": "<IP_ADDRESS>",
  "brief": "",
  "description": "description description.",
  "privacyStatement": "",
  "EULA": "",
  "help": "",
  "url": "https://www.xxx.xxx",
  "logo": "Resources/DXCLogo.png",
  "dependencies": [
    {
      "appId": "c73da900-7dd8-4f93-a364-21f8fd381712",
      "name": "Hello 360 New",
      "publisher": "XXX NewCompany Danmark A/S",
      "version": "<IP_ADDRESS>"
    }
  ],
  "screenshots": [],
  "platform": "<IP_ADDRESS>",
  "application": "<IP_ADDRESS>",
  "idRange":  {"from": 50000, "to": 60000},
  "showMyCode": true,
  "runtime": "2.3",
  "target": "Internal",
  "features": ["TranslationFile"]
}

I am trying to reproduce the problem on the simpler (as your example). I have the special character in the Publisher Name for example: XXX NewCompany Danmark A/S
But now I face issue:
Invoke-RestMethod : Could not find a part of the path 'C:\Users\user\Documents\AL\BC2\.alpackages\Hello A\S_hello 250 and hello 32_<IP_ADDRESS>.app'.
At C:\Program Files\WindowsPowerShell\Modules\navcontainerhelper\<IP_ADDRESS>\AppHandling\Compile-AppInNavContainer.ps1:265 char:13
+             Invoke-RestMethod -Method Get -Uri $url @AuthParam -OutFi ...
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Invoke-RestMethod], DirectoryNotFoundException
    + FullyQualifiedErrorId : System.IO.DirectoryNotFoundException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

I run the code:
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'P@ssword1' -AsPlainText -Force)
Compile-AppInBCContainer  -credential $credential -containerName bcbuild  -appProjectFolder "C:\Users\user\Documents\AL\BC2" -appOutputFolder "C:\Users\user\Documents\AL\BC2" -UpdateSymbols

and here is app.json:
{
  "id": "f7cc19ea-4df4-4298-8461-f4c23e003864",
  "name": "Hello 360 Second",
  "publisher": "XXX NewCompany Danmark A/S",
  "version": "<IP_ADDRESS>",
  "brief": "",
  "description": "description description.",
  "privacyStatement": "",
  "EULA": "",
  "help": "",
  "url": "https://www.xxx.xxx",
  "logo": "Resources/XXXLogo.png",
  "dependencies": [
    {
      "appId": "c73da900-7dd8-4f93-a364-21f8fd381712",
      "name": "Hello 360 New",
      "publisher": "XXX NewCompany Danmark A/S",
      "version": "<IP_ADDRESS>"
    }
  ],
  "screenshots": [],
  "platform": "<IP_ADDRESS>",
  "application": "<IP_ADDRESS>",
  "idRange":  {"from": 50000, "to": 60000},
  "showMyCode": true,
  "runtime": "2.3",
  "target": "Internal",
  "features": ["TranslationFile"]
}

Ok, that might be the missing piece.
No national characters?
Will look into this.

not at all. I even deleted the translation file

Have found 2 bugs on this thanks.
a fix will be included in a new containerhelper tomorrow, which should work.
I have also changed my tests to have a publisher with a / and a name with unicode characters.

Fixed in <IP_ADDRESS>

Tested and now working all fine. Thx Freddy!!!