Tekton removal

Fix #1113
Note https://github.com/epinio/helm-charts/pull/73 - associated helm chart changes
Note https://github.com/epinio/installer/pull/7 - associated installer changes
PoC. Issues (at least):

[x] Staging logs do not stream on push
[x] Environment variables are not passed through to buildpack pod
[x] I didn't update any tests based on new "pipeline"
[x] Helm chart can drop installation of Tekton as requirement
[ ] Jobs are random string, should have some identifying information (?)
[x] Apparently no proper deletion of PVC/PV used by staging
[x] linkerd in the staging job and pod - Wanted ? Needed ?

Gaps that I'm aware of:

[ ] Staging logs do not stream on push
[ ] Environment variables are not passed through to buildpack pod
[ ] I didn't update any tests based on new "pipeline"
[ ] Helm chart can drop installation of Tekton as requirement
[ ] Jobs are random string, should have some identifying information (?)

Question: Did you manually patch helm-charts/chart/epinio/templates/server.yaml to give the epinio API server the necessary permissions for manipulating jobs ?

Yes, I did patch the service account. Sorry, I forgot that I did that.

Yes, I saw that the jobs are cleaned up. The pods are not AFAICT. That is ok actually.
The pod is like the pipelinerun from tekton staging staying around until cleaned up by the next successful staging.
The pod is where I can look for the builder image now, in the API tests.

Pushed the fixes for the -api section. New branch tekton-removal-fixes-no-ci to keep it from triggering the CI.
Have to determine how -apps and -cli sections got stuck first, and fix that too.

Better id in job/pod names - One API (/staged) gets only namespace and stage id. currently uses stage id for direct access to the job ... Needs something selector based to search for the stage id instead of by name ... Checking back ... tekton based used selector-based access to the pipelineruns.

Only 5 tests apparently failed (cli 2 api 2 apps 1), and a few flakes.
A common output is
[3]     2022/01/26 21:39:41 EpinioApiClient "msg"="response is not StatusOK" "error"="Internal Server Error: timed out waiting for the condition" "status"=500
[3]     error pushing app to server: waiting for staging failed: All attempts fail:
[3]     #1: Internal Server Error: timed out waiting for the condition

This seems to occur for flakes also.
One of the things I did in the last commits yesterday was to change the backoff limit for the stage job from 0 to 6. I.e. to retry the staging pod with increasingly larger backoff times, instead of simply stopping on the first failure.
In an earlier CI run the tests got stuck, and in a local repro it was seen that the job/pod had failed, and the API server did nothing. With the above change I suspect that a timeout in the API server or client got triggered while the retries were attempted.
In the local repro of the pod failure it was access to the internal registry which had failed at network level.
I.e. staging was complete, and the attempt to upload the resulting image failed.
Note that if that is true for CI also then having the ginkgo logs will not help, it would appear in the stage pod's logs.

Local testing of the 5 failures

cli: Custom builder to stage -- :heavy_check_mark: -- Assumed to be CI flake.
cli: app exec -- :boom: -- Checked, fails in app exec, not push (I added By statements to see that).
IOW new staging looks to be ok. :arrow_left: @jimmykarily Handing this one to you.
api: S3 object deletion -- :heavy_check_mark: -- Assumed to be CI flake.
api: Run a command and get output -- See above, app exec, underlying API, same thing (Note 1).
apps: Deploy rails -- :boom: Demo of getting stuck (Per log: Missing encryption key to decrypt file with ...

Notes:

Websocket bad handshake ... Looks to be server side permission issue. Guessing that main has an outdated server.yaml. Definitely for you @jimmykarily, sorry and not in scope for this issue. Checked and confirmed that my helm-charts submodule is up to date.

{"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"pods "apps-102481308-b6d998494-6brm5" is forbidden: User "system:serviceaccount:epinio:epinio-server" cannot get resource "pods/exec" in API group "" in the namespace "namespace-846522276"","reason":"Forbidden","details":{"name":"apps-102481308-b6d998494-6brm5","kind":"pods"},"code":403}

So, looks to be two flakes, and two failures for the an issue out of scope here.
One actual issue (rails) I can use to work on proper handling of a staging error.

I've added the missing permissions but forgot to create a PR against the helm-charts repo: https://github.com/epinio/epinio/pull/1148/files#diff-a3fac176621ab40a843e67ebc92316e2fdaf92328882e8b1e9d221aa5b8eedcb
so it was never merged :facepalm: . I will create a PR now.

@andreas-kupries here you are: https://github.com/epinio/helm-charts/pull/76/files . Sorry...

newly pushed checks pod state for completion and failure ... reports correctly to client ... client is not exiting ...
investigate and fix tomorrow
Also tomorrow, getting in the fix for the app exec permissions, with thanks to @jimmykarily .

A persistent issue is the failure to deploy rails in the apps section. Ok, log ends in

🕞  [f6ab2a9e4e80a5ae-j76bv] buildpack Paketo Rails Assets Buildpack 0.4.1
🕞  [f6ab2a9e4e80a5ae-j76bv] buildpack   Executing build process
🕞  [f6ab2a9e4e80a5ae-j76bv] buildpack     Running 'bundle exec rails assets:precompile assets:clean'
🕞  [f6ab2a9e4e80a5ae-j76bv] buildpack failed to execute bundle exec output:
🕞  [f6ab2a9e4e80a5ae-j76bv] buildpack Missing encryption key to decrypt file with. Ask your team for your master key and write it to /workspace/source/app/config/master.key or put it in the ENV['RAILS_MASTER_KEY'].

I know that I created the app with -e RAILS_MASTER_KEY=.... ... And I can confirm that the staging pod has this in its environment as well. I.e. the passing of the app env into the staging process is done.

env:
- name: CNB_PLATFORM_API
value: "0.4"
- name: RAILS_MASTER_KEY
value: ******

No idea why the build pack would not see it.

Of the other 4 failures only the uses the custom builder to stage case failed locally as well.
And on the second attempt it worked also. Rest were ok on first local try.
So likely flakes in the CI, despite their double failures. I may crank FLAKE_ATTEMPTS to 4.
That leaves the rails test as the only issue IMHO.

IIRC, Buildpacks do some copying of environment variables over to a directory. I'm not sure of the effect of this but if you want, we can pair on Friday when I get in. I can use my metabuildpack to generate some additional ways to see how the data is getting passed around. It might be that the folder that's being written to and read from doesn't exist, so it fails silently?

IIRC, Buildpacks do some copying of environment variables over to a directory. I'm not sure of the effect of this but if you want, we can pair on Friday when I get in. I can use my metabuildpack to generate some additional ways to see how the data is getting passed around. It might be that the folder that's being written to and read from doesn't exist, so it fails silently?

I am open to pairing (currently in the retro, will ping you after).
Note, a local test placing the key into the referenced file (config/master.key) makes it go further ... ArgumentError: invalid base64. I placed the key into the file as is. Maybe it expected it base64 encoded ? Regardless, it is a datapoint towards issues in the ENV setup.
Further, my script to set the rails test up locally is
#wget https://github.com/epinio/example-rails/tarball/main -O rails.tar
#mkdir rails
#tar xvf rails.tar -C rails --strip-components 1
#echo > rails/config/credentials.yml.enc 'uVPZWDUhuOVhjFhPhom5qL9dGAJqVOctoK8PZQpGp4i5rBnrcT7GiHFFAmPb3ZPSdAnW8sj00VlEECRem01LzI1pzfhg9TUGti6b2jyxiTxALVsDlmCg4V458jprpFfNJaAlK7RGRKp9oSNEI1DBliGX8aKTf6ye9wJV2AF+w4mdezj2xtsgN5lKhMN6YMFn8V/XNUC3cvmyEH6ot0Aj3N+BaiKXfTDJdaLqcr+awhMSNh0Es+vBLdYRvOgaMCGicKor/Oe0h8VkuVSIT0Ye08evYqoHkijKMH034T2M2rE5EhkKUzbK1YRhYPiPfHwoKYXviuarIuCZuR/q5WhVghc5YTRVUjFILWe5aLzrm9pCu0WweIDIDf4K7OGsQN07nY2a3974OR73qKEi1RCJGk+2dpn1c696f9ar--0GJc3grQhOubjNmy--+9a7S7qwSUi/ennPYg8XFg=='
#rm rails.tar
#exit

ep namespace create rails
ep target rails
ep app create rails
ep app env set rails RAILS_MASTER_KEY 75a74503267d5869281389d73cf8b90b

cd rails
export TRACE_LEVEL=255
ep app push -n rails

@agracey Attached the log of the first "successful" run. In the sense that I got the image downloaded, and actually running (I had mistyped the trigger file (runtests.log is needed, not runtest.log, note the s).
rails-test.log
Things of note:

I missed you typing ls -lr ... Should be ls -lR (uppercase R) for recursive.
The env|sort shows both env variables specified for the pod (CNB_PLATFORM_API, RAILS_MASTER_KEY) missing from what is seen by env etc.
The sleep command is simply not run ... Maybe the find exists with error, aborting the run ?
The TEST=TESTVAL assignment is present in the environment

The registry.hub.docker.com/a99k/testbuilder:1d is a new image I created by the patching the /cnb/buildpacks/atgracey_test/0.0.1/spec.json of your image with a new set of commands to run. (1* before 1c did not work out)
Attached is the spec.json used in the patched image. more echos to separate the commands. -R for ls, block issues via || true.
spec.json.txt
This gave me better output, and the sleep was active.
Basically we have to add || true to all the commands likely to error out, to prevent such errors from ending the script sequence.
As seen, the ENV is very small. The pod spec variables are missing.
Entering the pod however and executing env | sort directly I get
CNB_GROUP_ID=1001
CNB_PLATFORM_API=0.4
CNB_STACK_ID=opensuse.stack.leap15.3
CNB_USER_ID=1000
COLUMNS=231
GPG_TTY=/dev/pts/0
HOME=/home/cnb
HOSTNAME=75a8f8ffc766d8cf-x9lbh
KUBERNETES_PORT=tcp://<IP_ADDRESS>:443
KUBERNETES_PORT_443_TCP=tcp://<IP_ADDRESS>:443
KUBERNETES_PORT_443_TCP_ADDR=<IP_ADDRESS>
KUBERNETES_PORT_443_TCP_PORT=443
KUBERNETES_PORT_443_TCP_PROTO=tcp
KUBERNETES_SERVICE_HOST=<IP_ADDRESS>
KUBERNETES_SERVICE_PORT=443
KUBERNETES_SERVICE_PORT_HTTPS=443
LINES=50
OLDPWD=/workspace/source
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
PWD=/workspace/source/app
RAILS_MASTER_KEY=75a74503267d5869281389d73cf8b90b
SHLVL=1
TERM=xterm
_=/usr/bin/env

Much more and the assignments from the pod are present.
So, somewhere in the processes for staging the information is lost ...
PID 1 is

cnb            1  0.0  0.0  14188  2896 ?        Ss   17:14   0:00 /bin/bash -c /cnb/lifecycle/creator -app=/workspace/source/app -cache-dir=/workspace/cache -uid=1000 -gid=1000 -layers=/layers -platform=/platform -report=/layers/report.toml -process-type=web -skip-restore=false -previous-image=epinio-registry.<IP_ADDRESS>.omg.howdoi.website/apps/rails-rails6:75a8f8ffc766d8cf epinio-registry.<IP_ADDRESS>.omg.howdoi.website/apps/rails-rails6:75a8f8ffc766d8cf && curl -X POST http://localhost:4191/shutdown

cat /proc/1/environ ... Master key present

The actual process is pid 7 ... Has master key. Expected
Next down in the chain, PID 28. That is the buildpack's build script, called from the lifecycle/creator.

cnb           28  0.0  0.0  14188  2844 ?        S    17:14   0:00 bash /cnb/buildpacks/atgracey_test/0.0.1/bin/build /layers/atgracey_test /platform /tmp/atgracey_test-577736561/atgracey_test/plan.toml

And

cat /proc/28/environ ... Master key missing

The lifecycle creator calls the build script, etc with a severely reduced environment, and the pod env settings are gone.
Given the reference to /platform/env by  -e /platform/env in the command line for the meta buildpack I suspect that our pod env assignments should be copied there, somehow, before the liefecycle/creator is invoked.

That's some really good debugging!
Based on that research, I looked at the buildpack task and they do some additional set up that I missed when porting:
https://github.com/buildpacks/tekton-integration/blob/main/task/buildpacks/0.4/buildpacks.yaml#L91-L116

Current state: Reworked how the app environment is passed into the pod. Instead of passing it as pod environment variables a properly configured secret is mounted in the pod, in the expected place (/platform/env). The secret is a copy of the app secret holding the app's env, allowing for standard variables to be passed in.
Rebased the long sequence of commits to squash most of the history, and write a nicer commit message.
Attention: While the local rails deployment still aborts in staging it happens further into staging than before, and I have checked and confirmed that the same issue happens for the old tekton-based staging. See the tekton log excerpt below. The job staging reports exactly the same. As such I do not consider this to be an issue of the tekton/job change, but something in either the local environment, or with the rails app itself.

[ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create __rails aborted! __
🕞  [ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create ActiveSupport::MessageEncryptor::InvalidMessage: ActiveSupport::MessageEncryptor::InvalidMessage
🕞  [ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create /workspace/source/app/config/environment.rb:5:in `'
🕞  [ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create
🕞  [ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create Caused by:
🕞  [ec0a811b7cbec444-stage-rlvz5-pod-djrtv] step-create ArgumentError: invalid base64

I also confirmed during job-based staging that the app env is now properly seen by the buildpack.
So, let CI have another go at this.

Oh, you **** ...

🕞  [83ab91cf6b692a92-97hhv] buildpack ======== Output<EMAIL_ADDRESS>========
[1]       🕞  [83ab91cf6b692a92-97hhv] buildpack failed to run detection: read /platform/env/..2022_01_31_11_24_23.439121451: is a directory

Recent log from CI ... At least one of the build packs cannot handle the dir structure of a mounted secret.

Attempting a fix for the previous by

Mounting into the /platform/appenv sibling directory instead, and then
Copying the data to their destination using cp -vL. The L resolves the symlinks, leaving proper files in the destination.

Now buildpacks should see a directory containing just files, one per variable.

That was a bit more involved than I had thought. We actually need two volumes. One for the raw mounted secret, the other for the cleaned up environment. The second is mounted into /platform/env. Why a volume here, not just a plain directory ? Kubernetes apparently mounts its own volume into that directory, leaving the buildpack step to see an empty directory instead of the files just copied into it by unpack-blob. By mounting our own volume over what kubernetes gave us we make our changes properly visible.

Extraction of the staging logs via kube commands looks to be working. The interleaving with other logs may be off somewhat, maybe a stdout/stderr thing (code currently reports to stdout).
In the apps section I find that the issues were with the internal registry:

ERROR: failed to get previous image: connect to repo store "epinio-registry.<IP_ADDRESS>.omg.howdoi.website/apps/namespace-213385109-apps-213381563:526a115c9907d184": Get "https://epinio-registry.<IP_ADDRESS>.omg.howdoi.website/v2/": dial tcp <IP_ADDRESS>:443: connect: connection refused

And looking over the logs of the cli section ... It is the same issue as above, all the time.
So, current issue seems to be flakiness with the internal epinio-registry.
Note, there are tests which do not fail, just flake, and tests which simply pass.
The staging job and pod have the proper credentials to talk to the registry, otherwise everything would fail.
I currently do not see how the change to jobs could have affected such ... Grasping at straws, that the faster staging via a job causes a higher load on the registry as the tests run quicker after each other ?
I will now put this set of issue and PR (here, and helm chart) into review.

Note: I am leaving "changing the pod name to contain identifying info" to a separate PR. This one has taken long enough as it is.

Ok. We have a near green now, with the latest changes, and linkerd disabled for the epinio installation.
The single api test which failed actually involves linkerd:
https://github.com/epinio/epinio/blob/18ea6d167e88ba23bb26d2b6b600ad2defceadc2/acceptance/api/v1/application_show_test.go#L80-L84
So, all green.

Confirmed #1174.

Result from my huddle with @jimmykarily ...
The issue seems to have been a race condition between the buildpack container/step and linkerd sidecar in the staging pod.
While the sidecar spins up networking is blocked.
When the buildpack step runs ahead to talk to the epinio registry, well, connection refused.
Testing on #1174 draft PR with a linkerd annotation to have port 443 bypass the proxy (*) has everything in working order again.
This change will now be pulled into here, with the linkerd suppression thrown out again.

(*) And given that port 443 is TLS, i.e. secured, no need to resecure via linkerd either.

Cherry picked the commit fixing linkerd, reworked and rebased the history.

All :heavy_check_mark:.

All comments addressed, either directly, or by making an issue to handle.


Add yarn test placeholder script

Travis CI seems to run yarn test which I didn't expect. This adds a placeholder test script and the test script definition in packages.json.

closes #185


DM-23354: Deploy xml 4.6 at the summit

The different build numbers on the containers mentioned above are mainly updates on base configuration packages (for b36 and b22) and on the CSC package itself for atdome/b36.
the core packages are all the same though. I'll try to document this more carefully in the future.


Model disappearing then i have to restart it to view it again

Describe the bug
I am using bubble tea to show a display containing X jobs (we could call these JOBS as operations done in graphql which take an estimated 3 to 7 minutes, all these jobs are executed concurrently).
The model looks perfect - it shows what it should show.
But a bug occurs very sporadically which I am having a hard time determining and maybe you guys understand better what could be the cause - it basically consists in that the whole model "disappears" in my terminal at a random moment of the execution, and I have to close the model (exit) and then open it again so I can see the model content again - and then everything works normally. (sometimes I have to do this several times to be able to see the model again, such as i showed in the video)
(in the video you can see this bug perfectly and could even be better understood)
Setup
Please complete the following information along with version numbers, if applicable.

OS: Ubuntu 22.04 | Kali Linux | PopOs (the bug happens in all of these O.S)
Shell: bash
Terminal Emulator: python3
Terminal Multiplexer: terminator

Venti (this binary runs a tea.Model) typing the dash command (you can see it in the video)
func (cmd *dashCmdStruct) run(args []string) error {
	// suppress most logging to avoid messing up the display
	rootCmd.log.SetLogLevel(rtlogger.ErrorLevel)

	p := dash.NewProgram()

	// run the model
	model, err := p.Run()
	if err != nil {
		fmt.Println("Error running Dash:", err)
		os.Exit(1)
	}

	// init the model
	model.Init()

	return nil
}

And here it his the Update function
func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
	var (
		cmd  tea.Cmd
		cmds []tea.Cmd
	)

	switch msg := msg.(type) {

	case tea.KeyMsg:
		if m, cmd := m.handleKeyPress(msg); m != nil {
			return m, cmd
		}
	case tea.WindowSizeMsg:
		m.report.SetSize(msg.Width, msg.Height)
		m.list.SetWidth(msg.Width)
		m.list.SetHeight(msg.Height - 2) // -2 for the status bar
		m.progress.Width = msg.Width - 4 // -4 for the borders
	case []*testjob.Job:
		return m.onJobsLoaded(msg)
	case tickMsg:
		cmd = m.progress.SetPercent(m.percentage)
		return m, tea.Batch(tickCmd(), cmd, loadJobs())

	// FrameMsg is sent when the progress bar wants to animate itself
	case progress.FrameMsg:
		progressModel, cmd := m.progress.Update(msg)
		m.progress = progressModel.(progress.Model)
		return m, cmd
	case error:
		return m.onError(msg)
	case View:
		return m.onViewChange(msg)
	}

	m.list, cmd = m.list.Update(msg)
	cmds = append(cmds, cmd)

	m.report.Viewport, cmd = m.report.Viewport.Update(msg)
	cmds = append(cmds, cmd)
	return m, tea.Batch(cmds...)
}

I will provide more code details if needed
Video triggering the bug:
https://github.com/charmbracelet/bubbletea/assets/47611900/8c4879a8-4a01-480f-9141-48f6d2d69a29

Hey guys, i think found the bug - i am able to keep the model displayed.
Sorry for opening this.


Request a trained model to analyze the role

Dear author, can you provide the corresponding model trained in the paper? I want to try to analyze the concept of role proposed in the paper.

We looked for the training models in past several days, but it seems that we did not save the models. When preparing the figures in the paper, we directly output the latent role presentations.
The results in the paper can be obtained by running our code. We are sorry for the inconvenience.
Please feel free to contact us if you have any further questions.

As far as I know, you should save the model before you can visualize it (in mac/windows) and also get the screenshots and role information in the paper.


[357] - refactor filter managers/beneficiaries by state

This PR fixes #357
Changes
Refactor filter managers/beneficiaries by state.
Instead of using active, removed and locked, use 0 (active), 1 (removed) and 2 (locked).
The endpoint response was changed too.
Tests

@obernardovieira , after validation, please don't merge, I need to align with the front-end team the deploy


Flutter does not detect VS code extension

Steps to Reproduce
When running flutter doctor, the warning message is shown stating that the VS Code extension is not installed even when the plugin is installed and visible from VS Code.
code --install-extension Dart-Code.flutter

Found 'Dart-Code.flutter' in the marketplace.
Installing...
Extension 'Dart-Code.flutter' v2.15.0 was successfully installed!
flutter doctor -v

[!] VS Code (version 1.24.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
ls ~/.vscode/extensions

dart-code.dart-code-2.15.0 dart-code.flutter-2.15.0
Visual studio code also shows that both the dart and the flutter plugin are installed.
Logs
flutter doctor -v

[✓] Flutter (Channel beta, v0.5.1, on Mac OS X 10.13.5 17F77, locale en-IN)
• Flutter version 0.5.1 at /Users/Jerry/Applications/flutter
• Framework revision c7ea3ca377 (5 weeks ago), 2018-05-29 21:07:33 +0200
• Engine revision 1ed25ca7b7
• Dart version 2.0.0-dev.58.0.flutter-f981f09760
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at /usr/local/share/android-sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• ANDROID_HOME = /usr/local/share/android-sdk
• Java binary at: /Library/Java/JavaVirtualMachines/jdk1.8.0_172.jdk/Contents/Home/bin/java
• Java version Java(TM) SE Runtime Environment (build 1.8.0_172-b11)
• All Android licenses accepted.
[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 9.4.1, Build version 9F2000
• ios-deploy 1.9.2
• CocoaPods version 1.5.3
[✗] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.io/setup/#android-setup for detailed instructions).
[✓] IntelliJ IDEA Community Edition (version 2018.1.5)
• IntelliJ at /Applications/IntelliJ IDEA CE.app
• Flutter plugin version 26.0.2
• Dart plugin version 181.4892.1
[!] VS Code (version 1.24.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[!] Connected devices
! No devices available
! Doctor found issues in 3 categories.

Thanks for the report! This issue has been fixed in the master channel but hasn't reached the beta channel yet. It's safe to ignore for now, the extension check is informational only so shouldn't affect your use of VS Code or Flutter.


🍒 [cxx-interop] Improve the warnings for unannotated c++ APIs returning SWIFT_SHARED_REFERENCE types

Explanation: Improved the diagnostic messages introduced by https://github.com/swiftlang/swift/pull/76798.
Scope: Changes new diagnostic messages. Removing warnings from certain cases where users can't suppress the warning easily.
Risk: Low, this reduces the cases where new warnings could trigger.
Reviewer: @egorzhdan  @Xazax-hun @j-hui
Original PR: https://github.com/swiftlang/swift/pull/77397

@swift-ci please smoke test

@swift-ci please test

FYI @fahadnayyar we require full tests (not just smoke tests) on the release branches. I kicked off the full test run for this PR.


Update setup-edison.md

run executable located here
points to many executables
so clarification needed

Incorporated via https://github.com/openaps/docs/commit/94545d7902a4584b8a608ec12f64ae7e7f9978e6


Add wiki images

Overview
I need to add screenshots to the documentation in the wiki. This PR uploads the images I need for the Getting Started Gude.

Added images directly to wiki instead.


error 49999: unknown system-value semantic 'SV_InsideTessFactor'

Are tessellation shaders supported yet?
Is there an example how to map both endpoints to the hull stage?
Also, VulkanUtil::getShaderStage: domain stage is mapped to VK control, hull to evaluation
https://github.com/shader-slang/slang/blob/master/tools/gfx/vulkan/vk-util.cpp#L183
A simple sample I get the error with (v2024.1.21, SPIRV target):
struct HS_CONTROL_POINT_OUTPUT
{
    float4 Position : POSITION;
    float2 TexCoord : TEXCOORD;
};

struct HS_CONSTANT_DATA_OUTPUT
{
    float EdgeTessFactor[4] : SV_TessFactor;
    float InsideTessFactor[2] : SV_InsideTessFactor;
};

struct DS_OUTPUT
{
    float4 Position : SV_Position;
    float2 TexCoord : TEXCOORD;
};

// Domain Shader (DS)
[domain("quad")]
DS_OUTPUT main(HS_CONSTANT_DATA_OUTPUT input, const OutputPatch<HS_CONTROL_POINT_OUTPUT, 4> patch, float2 uv : SV_DomainLocation)
{
    DS_OUTPUT output;
    output.Position = 0;
    output.TexCoord = 0;
    return output;
}

These shaders are supported
it seems that we just did not implement:

SV_InsideTessFactor->TessLevelInner for SPIRV
SV_InsideTessFactor->gl_TessLevelOuter for GLSL

@ArielG-NV Sorry I didn't notice that you already assigned this issue yourself, but I have just fixed this along with the other spirv issue.

Hey, thanks for a quick turnaround!
Unfortunately I still see issues even with the https://github.com/shader-slang/slang/pull/4318, but I might be missing something
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
hlsl.meta.slang(3698): error 29000: unable to parse target intrinsic snippet: .operator[]
(0): error 99999: Slang compilation aborted due to an exception of class Slang::InternalError: unimplemented: Unhandled global inst in spirv-emit:
let  %1 : _     = InputPatch(%VSx5FOUT, %2)

struct VS_OUT
{
    float3 position : POSITION;
};

struct HS_OUT
{
    float3 position : POSITION;
};

struct HSC_OUT
{
    float EdgeTessFactor[4] : SV_TessFactor;
    float InsideTessFactor[2] : SV_InsideTessFactor;
};

// Hull Shader (HS)
[domain("quad")]
[partitioning("integer")]
[outputtopology("triangle_cw")]
[outputcontrolpoints(4)]
[patchconstantfunc("constants")]
HS_OUT main(InputPatch<VS_OUT, 4> patch, uint i : SV_OutputControlPointID)
{
    HS_OUT o;
    o.position = patch[i].position;
    return o;
}

HSC_OUT constants(InputPatch<VS_OUT, 4> patch)
{
    float3 p0 = patch[0].position;
    float3 p1 = patch[1].position;
    float3 p2 = patch[2].position;
    float3 p3 = patch[3].position;

    HSC_OUT o;
    o.EdgeTessFactor[0] = dot(p0, p1);
    o.EdgeTessFactor[1] = dot(p0, p3);
    o.EdgeTessFactor[2] = dot(p2, p3);
    o.EdgeTessFactor[3] = dot(p1, p2);
    o.InsideTessFactor[0] = lerp(o.EdgeTessFactor[1], o.EdgeTessFactor[3], 0.5);
    o.InsideTessFactor[1] = lerp(o.EdgeTessFactor[0], o.EdgeTessFactor[2], 0.5);
    return o;
}

Respectively domain shader complains that OutputPatch is not implemented

I think you are right. We haven't fully supported tessellation stages for SPIRV yet, but I think we are close and I'll make sure these are supported today.

Opened https://github.com/shader-slang/slang/issues/4332 to track development of InputPatch and OutputPatch.


Integrated code lifecycle: Trigger solution build before template build

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I adapted a test to cover the added functionality
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[x] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).

Motivation and Context

A psuh to a test repository triggers the build of the respective solution and template repositories. If the template repository results come in first (which can easily happen if both builds are triggered at the same time) it will display faulty results as it still used the old testcases. Thus, the template repository should only be built after the solution build results are available.
Description

When a push to a test or auxiliary repository is detected by local VC, it triggers only the solution build. Once the build result has been processed, LocalCI then triggers the template build.
Steps for Testing

Prerequisites:

1 Instructor
1 Programming Exercise

Edit the test repository and submit
The solution build should start right away
Once the solution build result is available, the template build is triggered

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Server

Class/File
Line Coverage
Confirmation (assert/expect)

LocalCIResultProcessingService.java
71%
✅

LocalVCServletService.java
91%
✅

Summary by CodeRabbit

New Features

Enhanced build triggering logic to improve the handling of solution and template repositories.

Refactor

Refined method for determining if a build is for a solution of test or auxiliary push.

Tests

Added integration tests to verify the correct order of build job completions.

@Strohgelaender Not sure how easy this is to fix for the other systems as I'm not very familiar with the code.


My partial solution

Gone a bit K&R with the styling - sorry.
There's some error handling missing (eg malloc failure) and I'm maybe  MAX_STR_LEN is a bit too paranoid.
Just got as far as creating the struct but I'm hungry.

Everything I hoped to see happening 👍
Would love to see where it takes you from here.

Ok, so I changed the signature of str_copy such that it creates a new struct because I didn't want to make assumptions about what was in the destination struct.
I am unclear about what str_move is supposed to achieve and what I've implemented has dangers.
¯\(ツ)/¯

I am unclear about what str_move is supposed to achieve and what I've implemented has dangers.

You're on the right lines. I wanted to do something here to demonstrate the thought processes required for moving ownership of memory. I think this is referred to as move semantics in Rust/C++.

Tried something else but I don't think I understand the purpose of str_move yet.

Also my Hungarian is atrocious. I've never been fond of Hungarian, tbh.

Tried something else but I don't think I understand the purpose of str_move yet.

Do we really need allocate the str_t structs?
Have you used C++ much? If so, do we always have to say string x = new string(..) or can we just say string x? Why? And who decides which is used, end-coder or stdlib?

Do we really need allocate the str_t structs?

I meant, do we really need to dynamically allocate the str_t structs within our string library?


CI - Publish wheels

Issue description rewritten by @jgiannuzzi
Python wheels for major platforms (linux-amd64, linux-arm64, macos-amd64, macos-arm64, windows-amd64) should be published automatically when a release is cut.

[x] Ensure that a wheel can be created via pyproject-build (I think that's already the case?)
[x] Enable cross-compilation from macOS via zig cc -target - this will allow to compile each wheel in the most compatible way, as the glibc and musl implementations can be aligned to the most universal manylinux targets
[x] (optional) Add a mage target that

creates a venv
installs the build Python package in it
installs the ziglang Python package in it
gets a target as argument (and defaults to the native target if none given)
calls pyproject-build with the right CC, GOOS, and GOARCH vars for the target

[x] Add a build GitHub Actions workflow that builds the various wheels in parallel on a macos-latest runner (using the matrix statement) and uploads them as artifacts
[ ] Add a publish GitHub Actions workflow that only runs on tag or release creation, downloads the artifacts, and uploads them to PyPI via trusted publishing. It should be tied to a specific GHA environment and this environment should be tied to protected tags.

The rationale for using zig is that we need CGO for using SQLite in mlflow-go, which means that classic Go cross-compilation wouldn't work. Zig ships with all the toolchains, as opposed to gcc/clang that need to install each toolchain independently. However, zig doesn't ship with the toolchains for macOS, so compiling from Linux or Windows for macOS is not possible. Luckily the fastest GitHub runners are actually the macOS ones with Apple Silicon (arm64), which can be used via the macos-latest runner tag. Hence my proposal to cross-compiling with it.

@BenWilson2, as mentioned on the Slack channel, I believe we would need this repository to be registered as trusted publishing in Pypi.

We are trying to publish to PyPi in https://github.com/mlflow/mlflow-go/actions/runs/11837339705/job/33029830166
however, we got:
Error: Trusted publishing exchange failure:
Token request failed: the server refused the request for the following reasons:

* `invalid-payload`: The name 'mlflow-go' is too similar to an existing project. See https://pypi.org/help/#project-name for more information.

This generally indicates a trusted publisher configuration error, but could
also indicate an internal error on GitHub or PyPI's part.

The claims rendered below are **for debugging purposes only**. You should **not**
use them to configure a trusted publisher unless they already match your expectations.

If a claim is not present in the claim set, then it is rendered as `MISSING`.

* `sub`: `repo:mlflow/mlflow-go:environment:release`
* `repository`: `mlflow/mlflow-go`
* `repository_owner`: `mlflow`
* `repository_owner_id`: `39938107`
* `job_workflow_ref`<EMAIL_ADDRESS>* `ref`: `refs/tags/v0.1.0`

Do we need a different name for our project 🫠?
@BenWilson2 @jgiannuzzi @dsuhinin any thoughts on this?

Ben and I had a call yesterday, and PyPi is currently refusing to accept mlflow-go as the package name. Ben opened an issue at https://github.com/pypi/support/issues/5146, but it's unclear how long this will take. Our plan is as follows:

Wait a few days to see if there is a response to the PyPi ticket.
Try a manual publish from Ben's side, which might bypass the name validation.
Rename the package.


Support latest FastAPI and Pydantic 2.0

Description
Updates FastAPI-Tableau to support the latest (at time of writing) version of FastAPI and Pydantic 2.0.
Here are a summary of changes:

In routing.py, updated TableauRoute's code that gets its own schema.

We use the schema to validate requests against the schema and rewrite Tableau-style requests to match what FastAPI expects.
Pydantic 2.0 changed or removed some of the capabilities FastAPI used to get this information. We mirrored FastAPI's use of those functions. FastAPI now uses functions from an internal module named _compat. We import functions from that internal module.

Update user_guide.py to correctly populate parameters from endpoints.

When constructing our user guide, we used to use an internal method of endpoint parameters called _type_display(). This has moved to pydantic._internal._repr as display_as_type().
Added tests to ensure that parameters are properly populated.

Some other small modifications to account for upstream changes, add type annotations, etc.
Re-vendor and modify OpenAPI function from FastAPI (now OpenAPI v5 rather than v3 I think).
Modify docs URLs that have changed.

Originally, I wanted to vendor some of the internal functions we call, after discussing with @mmarchetti. When I investigated more closely, the amount of code I'd have to vendor was significantly higher than we first thought. Instead, I'm going to pin to the current FastAPI version. I think this is a legitimate approach, as this is designed to be a replacement, rather than a supplement, to FastAPI.
Following on from this PR, I'll try to set a GitHub Action that tests against newer-than-pinned versions of FastAPI and Pydantic so we can monitor for breakages.
Addresses #37
Testing Notes / Validation Steps
We should ensure that all our demo content can be published to Connect and works with Tableau as expected. More detailed testing notes to come.

Verified this with @blairj09 using Tableau Desktop.


Reduce ESP partition size in deployment image

It does not fit into 550MiB partition created by Ironic Python Agent. This is relevant when provisioning software RAID UEFI enabled servers.
https://github.com/openstack/ironic-python-agent/blob/stable/yoga/ironic_python_agent/raid_utils.py#L30

Do we need new host images with this change included?

Do we need new host images with this change included?

Yes. Before that's available, one can either virt-customize local image or use IPA with ESP_SIZE_MIB modified to something bigger.

Building in: https://github.com/stackhpc/stackhpc-kayobe-config/actions/runs/6495467216

Is this an upstream issue too? DIB defaults to 550 MiB and IPA too, but the DIB image doesn't fit into what IPA creates? Maybe IPA should be just slightly bigger?

I think it's rather the issue of our layout that we pass to DIB?

I'll rebuild the images as part of https://github.com/stackhpc/stackhpc-kayobe-config/pull/721

Is this an upstream issue too? DIB defaults to 550 MiB and IPA too, but the DIB image doesn't fit into what IPA creates? Maybe IPA should be just slightly bigger?

I think it's rather the issue of our layout that we pass to DIB?

It's probably rounding / calculations error, I saw the ESP partition 0,1MB too big for MD created by IPA. It's easier to change here than a hard-coded value in IPA for us.

Merging as this is not tested in aio


After installing gc-excelviewer, message about it not being installed keeps appearing

While experimenting with the Preview command problems I also found that even after installing the gc-excelviewer the message about it not being installed kept appearing. The message went away after restarting VSCode, but this seems like a bug.
Originally posted by @andycraig in https://github.com/Ikuyadeu/vscode-R/issues/92#issuecomment-582397603

I'm not familiar with this part of the extension but I noticed that we check if the excelviewer extension is installed and active but do not try to activate it ourselves.
https://github.com/Ikuyadeu/vscode-R/blob/39c5b20f65c0cdf16eaa2680f092d68b0241dfc6/src/preview.ts#L118-L120
This probably won't work as intended if the extension is installed but hasn't been triggered to activate yet. This might be fixed by something along the lines of
if (iscsv !== undefined && iscsv.isActive) {
    return true;
} else if(iscsv !== undefined) {
    await iscsv.activate();
    return true;
}

or we skip the isActive check if we don't actually need the extension to be active at that time.

I'm not familiar with this part of the extension but I noticed that we check if the excelviewer extension is installed and active but do not try to activate it ourselves.
https://github.com/Ikuyadeu/vscode-R/blob/39c5b20f65c0cdf16eaa2680f092d68b0241dfc6/src/preview.ts#L118-L120
This probably won't work as intended if the extension is installed but hasn't been triggered to activate yet. This might be fixed by something along the lines of
if (iscsv !== undefined && iscsv.isActive) {
    return true;
} else if(iscsv !== undefined) {
    await iscsv.activate();
    return true;
}

or we skip the isActive check if we don't actually need the extension to be active at that time.


 Does your Logger includes somewhere ActiveSupport::LoggerSilence module ? It throws:
NoMethodError: undefined method `silence' for #<Logger:0x007f89081dbad0>

I took a look at SilentLogger but didn't find a way to use it instead. Any idea?

Rails 5.2.3
Ruby 2.6.5

Same here

Ruby 2.5.5
Rails 5.2.2
lograge (0.11.2)
gelf (3.1.0)

Hi! If you look at the activerecord-session_store README it states:

Please note that you will need to manually include the silencer module to your custom logger if you are using a logger other than Logger and Syslog::Logger and their subclasses:
MyLogger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer

This silencer is being used to silence the logger and not leaking private information into the log, and it is required for security reason.

So in your case you will add smthing like this in /config/environments/<env>.rb
  GELF::Logger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer
  config.logger = GELF::Logger.new(...)

References: main answer, undefined method silence

Hi! If you look at the activerecord-session_store README it states:

Please note that you will need to manually include the silencer module to your custom logger if you are using a logger other than Logger and Syslog::Logger and their subclasses:
MyLogger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer

This silencer is being used to silence the logger and not leaking private information into the log, and it is required for security reason.

So in your case you will add smthing like this in /config/environments/<env>.rb
  GELF::Logger.send :include, ActiveRecord::SessionStore::Extension::LoggerSilencer
  config.logger = GELF::Logger.new(...)

References: main answer, undefined method silence

I just want to point out, if anyone comes across this issue after upgrading activerecord-session_store to 2.0, this no longer works even though their README still says what you quoted.
At some point last year (I think it was a June 2020 commit), they removed ActiveRecord::SessionStore::Extension::LoggerSilencer. There is currently an issue open here, https://github.com/rails/activerecord-session_store/issues/176, where they have offered possible solutions, but none seem to actually work yet.
I forked activerecord-session_store from the latest codebase (which fixes the CVE that sparked the need to release 2.0)  and reinserted ActiveRecord::SessionStore::Extension::LoggerSilencer, so you can find that fork here if you need an immediate solution: https://github.com/egallup02360/activerecord-session_store
You can use it in your Gemfile like so
gem 'activerecord-session_store', git: 'https://github.com/egallup02360/activerecord-session_store.git', ref: 'f876987'

Hopefully they come up with a solution for this relatively soon though, I really dislike the idea of running a fork of something like this in production.


Added better windows support

added function checkOS and added a description tag at the end of the output

Thanks for the PR, but it must pass the build @abrelsfo


[WIP] Add expression testing framework

Submission Checklist

[ ] Run unit tests: ./runTests.py src/test/unit
[ ] Run cpplint: make cpplint
[ ] Declare copyright holder and open-source license: see below

Summary
Addes a testing framework that checks that functions from stan math that are exposed to stan language work when given Eigen expressions as inputs. The tests are generated from the list of signatures stanc3 supports. For now we have a list of signatures that are skipped. As more functions support working with expressions we will remove them from the llist and eventually remove the list itself.
For every signature prim, rev and fwd instantinations are tested (with all scalars of type double/var/fvar<double>). Tests check the following:

signatures can be compiled with expressions arguments
results when given expressions are same as when given plain matrices (including derivatives)
functions evaluate expressions at most once

TODO:

replace list of signatures with a call to stanc3 (once this repo uses stanc3)
add support for generating tests to makefile
make tests run on Jenkins

Intended Effect
How to Verify
Side Effects
If a new function is exposed to stan language it will need to work with eigen expressions. Specifically every argument that can be of Eigen type (matrix, vector and row_vector in stan language) need to also be able to accept expressions. That is unless we add it to the list of exceptions, which should not be allowed - the list should only be getting shorter.
Documentation
None for now. I'll add doc where requested by reviewer.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Tadej Ciglarič
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

replace list of signatures with a call to stanc3 (once this repo uses stanc3)

This is in motion. Currently waiting for https://github.com/stan-dev/stan/pull/2761 which just needs a review. Then I will open a PR that will download stanc3 instead of compiling stanc2 and use that to compile models. Then you can proceed here as far as that is concerned. There will be other PRs that will reorganize some of the testing but that does not conflict with these efforts.

@serban-nicusor-toptal Why are all Jenkins changes on this PR ignored? Is it because @t4c1 does not have write rights to this repo? Thanks.

You are right @rok-cesnovar see this thread.

Is it possible to give @t4c1 the same permissions he has on Math?

Added to the stan team, @t4c1 you should receive an email shortly.

Still unable to change Jenkinsfile here on this branch. Any ideas Nic?

Can you please push a small change now to test ? Thanks

Jenkins still sees it as untrusted.
Loading trusted files from base branch develop at ef2e811498593ab173b807dddaa15cb15ea82d77 rather than 9bd4a6212364d0470c1560f08526193f45669263
Obtained Jenkinsfile from ef2e811498593ab173b807dddaa15cb15ea82d77
‘Jenkinsfile’ has been modified in an untrusted revision

Give me a moment to understand why, if all fails we can try to re-do the PR in a minute,

Is it maybe because the change is coming from a fork?

It should only require write permissions of the author for stan, shouldn't for the fork source.


[Dialog] Height shrinking on dynamic content

While dealing with dynamic content like TODOs on Dialog, the Dialog content area is shrinking. I have used this code
<Dialog
  ref='formDialog'
  title='TODOs'
  actions={customActions}
  autoScrollBodyContent={true}
  autoDetectWindowHeight={true}
  modal={true}>
  <div style={{height: '1000px'}}>
    //todo list dynamically added here
  </div>
</Dialog>

I have removed material-ui/lib/dialog.js#L345 and changed it to below. And this is fixed.
-  var maxDialogContentHeight = clientHeight - 2 * (styles.body.padding + paddingTop + 64);
+  var maxDialogContentHeight = clientHeight - 2 * (styles.body.padding + 64);

@anilCSE could you post your code that generates the todo list items? I'd like to run the whole thing to see the problem you're facing..

@anilCSE : Is this problem solved for you? Let us know.

Closing this issue for now. If you feel this issue has not been addressed completely do feel free to leave a comment below and we will reopen the issue.


Cannot access offset of type string on string

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2
Issue Description
After successful payment via PayPal SmartButton trying to reach /admin/sales/transactions/view/1#stack (Transactions->view), got: Cannot access offset of type string on string
at packages /Webkul/Admin/src/Http/Controllers/Sales/TransactionController.php: 172
Preconditions
Ubuntu, Apache, PHP 8.1.2
Steps to reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected Result
Transaction details page
Actual Result
Cannot access offset of type string on string
Screenshots

Additional context
No response

Hii euge001.
As i checked and found that there is no such issue we are opening drawer for transactions details and there is no such method convertIntoSingleDimArray in TransactionController.php file.

@euge001
As per checking the concern, this method which is applied is not  used, and also for transactions we use drawer.
Hence I am closing the issue,

FYI we just encountered the exact same thing. @euge001 did you manage to fix your issue? Can you please share how you fixed it?

Same issue here @euge001


[pigeon] let FlutterApis be invoked with null callbacks

I had IDL with
@FlutterApi()
abstract class FlutterBookApi {
  void displayBookDetails(Book book);
}

it created Java code
public interface Reply<T> {
  void reply(T reply);
}
public void displayBookDetails(Book argInput, Reply<Void> callback) {
  BasicMessageChannel<Object> channel =
      new BasicMessageChannel<Object>(binaryMessenger, "dev.flutter.pigeon.FlutterBookApi.displayBookDetails", new StandardMessageCodec());
  HashMap inputMap = argInput.toMap();
  channel.send(inputMap, new BasicMessageChannel.Reply<Object>() {
    public void reply(Object channelReply) {
      callback.reply(null);
    }
  });
}

but if I don't care about the result, I can't pass null to callback since it doesn't null check and can crash.
cc @gaaclarke

Bonus points if instead of null, you add overload method that just omits the callback.

@gaaclarke I don't think this ticket is relevant to the https://github.com/flutter/packages/pull/222
This ticket is for the native APIs (such as Java APIs)

Sorry, keeping the languages straight has been a problem for me.  Thanks for the heads up.  I'll update the title.


[API] $.hdb

$.hdb API
https://help.sap.com/doc/3de842783af24336b6305a3c0223a369/2.0.03/en-US/$.hdb.html

Module:
https://github.com/SAP/xsk/tree/main/modules/api/api-xsjs/src/main/resources/xsk/hdb

Module:
https://github.com/SAP/xsk/tree/main/modules/api/api-xsjs/src/main/resources/xsk/hdb

https://github.com/SAP/xsk/pull/319  - resolves this issue


Comment:Chat-Frontend

In the index.hml, I had to use socket.io/socket.io.js instead of /socket.io/socket.io.js in order for the browser to find the resource.

Is your app in a subdirectory, maybe?

It "was" in an subdirectory - maybe just leave a note somewhere that subdirectories will break it and needs either ProxyPass and RewriteRule fiddling. I moved it under its own subdomain now.

We'd definitely merge a PR with a note but to me this is how any webserver relative URL would work (../socket.io/socket.io.js in a subfolder would probably also work).


Make preview work and add onClickVideoFrame callback

Kind of dirty but figured I'd create a PR.

The Toast is the same as when you did it? No?

It is in the sample code.

What I meant was the sample has Toasts already so I just kept it consistent.

You're right, thanks


[Umbrella Issue] Re-document communication methods on the website

[x] Add a page to website for workstreams
[x] Add/link? public calendar from website
[ ] Add link to Slack inviter
[x] Document workstream mailing lists somehow

I'm sure there's things I'm missing cc: @justaugustus, @edwarnicke  – any other thoughts?

@celestehorgan the slack inviter is added already right? https://communityinviter.com/apps/inclusive-naming/invite


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/284 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 45c0ca3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-01-13-07-12-11 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-01-13-07-12-11

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


FQDN: some small performance optimizations around logging

As part of some basic benchmarking, I noticed that we were spending a lot of time in the DNS critical section enriching Hubble logs. Additionally, we were wasting CPU time re-generating information we already had!
So, there are 3 changes:

Change the label representation of some internal types from []string to labels.LabelArray. We already have the LabelArray on hand, so we can save the allocation and string conversion.
Stop generating some logrus.Fields objects when we don't need them
Don't look up identities when we already have them.

All in all, this results in a bit faster dns throughput in an artificial test I set up, as well as reduced CPU usage in the critical path. Log enrichment went from ~6% of CPU to around 2.5%, all while requests-per-second increased.

/test

Thanks for the great feedback, @kaworu and @learnitall. I've updated the PR based on your suggestions, PTAL.

proxy logging: restrict label retrieval time

As suggested by @kaworu, we were not passing a valid context to identity
selection. This leaves us open to arbitrary latency spikes in the DNS
response critical path.

Since these identities are almost certainly already in cache, there is
essentially no risk that we lose log fields. Plus, this protects us
against DNS tail latency spikes.

Signed-off-by: Casey Callendrello<EMAIL_ADDRESS>
nitpicking but Suggested-by: Alexandre Perrin<EMAIL_ADDRESS>would be more idiomatic than using the GitHub handle.

/test

/test


How to generate Android apk?

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ x] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

npm-debug.log.txt
Current behavior

When run "npm run buil.android", work until get this messange, the apk is not generated
Total time: 1 mins 3.788 secs
Successfully prepared plugin nativescript-angular for android.
Successfully prepared plugin nativescript-intl for android.
Successfully prepared plugin nativescript-theme-core for android.
Successfully prepared plugin tns-core-modules for android.
Successfully prepared plugin tns-core-modules-widgets for android.
Project successfully prepared (android)
Executing after-prepare hook from C:\Desarrollo\Source\NASA3\nativescript\hooks\after-prepare\nativescript-dev-android-snapshot.js
Searching for devices...
Starting Android emulator with image Android_Accelerated_x86
Cannot run your app in the native emulator. Increase the timeout of the operation with the --timeout option or try to restart your adb server with 'adb kill-server' command. Alternatively, run the Android Virtual Device manager and increase the allocated RAM for the virtual device.
run android
Expected behavior

Minimal reproduction of the problem with instructions

"npm run buil.android"
What is the motivation / use case for changing the behavior?

I want generate apk without  run emulator
Please tell us about your environment:

Windows 10

Angular Seed Version: 8d0eb877c4d2d700426bcb23734569fdab20f660

Node: node --version =v6.9.5

Hi @wuardyn instead of using the npm script (which is used to create an AoT bundle of the build), just use things manually:
cd nativescript

Then follow these instructions to build a release APK:
https://docs.nativescript.org/publishing/publishing-android-apps
Caution: Take note of the keystore generation and do not lose that keystore so email yourself a copy.

great, thanks for your help


No module named aruudy.lists

When I run this command in python from aruudy.poetry import prosody
I get the error ModuleNotFoundError: No module named 'aruudy.lists'
Python version 3.6
Windows 10 64-bit

It is fixed in release 0.5.1. The problem in version 0.5: I forgot to include arrudy.lists
To update :
pip install aruudy -U
Thanks for your feedback


Fix the automated build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This fixes the failed automated build (https://github.com/openmm/openmm-torch/issues/124#issuecomment-1752801976) following https://github.com/conda-forge/onnxmltools-feedstock/pull/10 as suggested by https://github.com/regro/cf-graph-countyfair/pull/6#issuecomment-1754679462
The build number bump and re-rendering are skipped intentionally.

@mikemhenry could you review?

This is obsoleted by https://github.com/regro/cf-graph-countyfair/pull/6


Proposal on componentization.

@TimBarham as per our discussion. This is the first part of the documentation, some notes on how cordova-simulate structure should be componentized.
I put the documentation in docs/dev instead of devdocs. I also think docs/ could be renamed to doc/ (not part of this change).
For reading convenience, here the link to rendered markdown.
I will be submitting a proposal documenting sim-host <--> ui-host, this PR puts a missing context for the next, more detailed, document.

@TimBarham, added proposal on inter-component communication. For reading: component-communication.md.
Both documents are draft and submitted to for the purpose of discussion. Once we agree on the approach, I can implement this really quick.

I'm also thinking this needs to contain a full and concrete example, something like "echo plugin" cordova uses for documentation, but that needs to come with working code, so it's left out for later.

another idea is to use github wiki


Make label-warning more easily distinguishable from label-danger

Resolves #43

lgtm :+1:

I also think that the label-success color should be changed.

Shouldn't the changes be in site.css? I thought we weren't supposed to make changes to the material bootstrap defaults?
The color looks good though :+1:

Wellll...that'd require I write CSS instead of just using sed. I guess I'll fix it correctly then.

:+1:
@liam-middlebrook what color do you think label-success should be? a lighter shade of green?

Yeah a lighter shade would work nice!

Tried lighter green: clashes horribly.
If you can find a shade you like, patches are welcome.


Fixed bug With MerchantConfig

Due to the _defaultMerchantConfig being a static it meant that values from previous requests where being retained in memory.
This was causing failures in the case of a previous request having an incorrect subId as this incorrect value would always be used to make the request with iDeal throwing an exception on their side.
This is a simple fix removing a null check to ensure that the defaultMerchantConfig is created for each request removing this problem.

But is it still a default config then? The naming would be a bit misleading, how about getting rid of the static field altogether?

We can use as is currently, but I'd rather like to have this clean - even though it's an external library by the amount of changes we already did we kind of got responsible for it. But I'm happy to do the clean-up at a later stage.


Copy Right text BUG

When I click on the "from here", it gives an error.
And there is no statement about the copyright which I entered on the settings page

fixed


Activate performance and network tracing in Sentry

What

Following a discussion with @jasmeet0817 today about sluggishness of Smoothie on his phone, we found out that Sentry has a profiler option.
Activate performance and network tracing in Sentry
Add automatic / custom performance instrumentation

https://docs.sentry.io/platforms/flutter/performance/instrumentation/automatic-instrumentation/

@teolemon fixed by #1616 ?

Nope, some more tooling necessary


nltk required

Adding this plugin I ran into this error
File "plugins\AutoGPT-YouTube.zip\AutoGPT-YouTube\src\autogpt_youtube\youtube_functions.py", line 2, in <module>
ModuleNotFoundError: No module named 'nltk'

so I did
pip install -U nltk

now it seems to work

suggest add to requirements

Also, it seems like you need to add following line to the src/autogpt_youtube/youtube_functions.py to make it work too:
nltk.download('punkt')
Otherwise it would complain:
Resource punkt not found. Please use the NLTK Downloader to obtain the resource: >>> import nltk >>> nltk.download('punkt')


KAFKA-17878: Move ActionQueue to server module

Move ActionQueue to server module.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@chia7712 Hello, PTAL.


💡 change the copy on Adding a poll

Area
Proposal Pillar
Is there new design needed?
No
What?
Change the copy "The poll will be pinned to the top of your comments list, and you can close it whenever you like. Opening a new poll will automatically close the previous one, which will then appear as a comment in the comments feed."

Why?
na
How?
na
(Optional) User Story with acceptance criteria
No response

duplicate with;
https://github.com/IntersectMBO/govtool/issues/1792
https://github.com/IntersectMBO/govtool/issues/1789
https://github.com/IntersectMBO/govtool/issues/1790


Note about VirtualBox install (Linux Machine step not nessecary)

On the latest versions of virtualbox (at least on the latest version of arch linux), you do not need to do the step where you have to change the machine type to linux. It shows the GPU just fine regardless.


[NETBEANS-1074] Module Review groovy.support

Change the license headers of template files and Bundle.properties
Add the license headers to META-INF.debugger* files

Fixed.

Thanks, merging.


Update script to handle error gracefully

Option "set -e" has been removed from the script to avoid existing the script in case any command returns a non-zero exit status.

jenkins -> run tests please

Not required.


parameterized query: add parameterized query to stmt.Query

Added interpolateQuery function for generating query utilizing parameterized queries
Moved the stmt stuff in conn.go into its own file since there's more and more stmt specific stuff --hope you don't mind :)
Defaulting timezone to UTC for now; will send a separate PR that extracts that configuration out of the DSN
Copied how the mysql driver was handling string escapes and time format handling for the interim (we're currently not utilizing those types right now)

BTW, tests on go1.6 is failing b/c i'm using the testing.Run function...Should these uses be removed?


Bug 🐞: Can't run @nrwl/react component generator

Current Behavior
Can't run the command.
How to Reproduce
Try and run the command via the plugin
Envirounment:
Node : 12.20.0
OS   : darwin x64 (MacOS Big Sur)
Webstorm: 2021.2.3
npm  : 7.24.1
nx : 13.0.2
@nrwl/angular : Not Found
@nrwl/cli : 13.0.2
@nrwl/cypress : 13.0.2
@nrwl/devkit : 13.0.2
@nrwl/eslint-plugin-nx : 13.0.2
@nrwl/express : Not Found
@nrwl/jest : 13.0.2
@nrwl/linter : 13.0.2
@nrwl/nest : Not Found
@nrwl/next : Not Found
@nrwl/node : Not Found
@nrwl/nx-cloud : Not Found
@nrwl/react : 13.0.2
@nrwl/schematics : Not Found
@nrwl/tao : 13.0.2
@nrwl/web : 13.0.2
@nrwl/workspace : 13.0.2
@nrwl/storybook : 13.0.2
@nrwl/gatsby : Not Found
typescript : 4.3.5
Additional context
The error stack:
java.lang.NullPointerException
	at com.github.etkachev.nxwebstorm.services.MyProjectService.getProjects(MyProjectService.kt:162)
	at com.github.etkachev.nxwebstorm.services.MyProjectService.getProjectList(MyProjectService.kt:84)
	at com.github.etkachev.nxwebstorm.utils.GenerateFormControl.<init>(GenerateFormControl.kt:20)
	at com.github.etkachev.nxwebstorm.ui.RunSchematicPanel.<init>(RunSchematicPanel.kt:42)
	at com.github.etkachev.nxwebstorm.actionlisteners.SchematicSelectionTabListener.runSelectedLogic(SchematicSelectionTabListener.kt:139)
	at com.github.etkachev.nxwebstorm.actionlisteners.SchematicSelectionTabListener.valueChanged$lambda-2(SchematicSelectionTabListener.kt:125)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:313)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:776)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:727)
	at java.desktop/java.awt.EventQueue$4.run(EventQueue.java:721)
	at java.base/java.security.AccessController.doPrivileged(Native Method)
	at java.base/java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(ProtectionDomain.java:85)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:746)
	at com.intellij.ide.IdeEventQueue.defaultDispatchEvent(IdeEventQueue.java:885)
	at com.intellij.ide.IdeEventQueue._dispatchEvent(IdeEventQueue.java:754)
	at com.intellij.ide.IdeEventQueue.lambda$dispatchEvent$6(IdeEventQueue.java:441)
	at com.intellij.openapi.progress.impl.CoreProgressManager.computePrioritized(CoreProgressManager.java:825)
	at com.intellij.ide.IdeEventQueue.lambda$dispatchEvent$7(IdeEventQueue.java:440)
	at com.intellij.openapi.application.impl.ApplicationImpl.runIntendedWriteActionOnCurrentThread(ApplicationImpl.java:794)
	at com.intellij.ide.IdeEventQueue.dispatchEvent(IdeEventQueue.java:492)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

Yup I believe this has to do with the recent Nx 13 upgrade. Since all the nx.json does not have projects in there anymore. All projects are now declared in the workspace.json or angular.json. I will have a fix for this shortly.

Thank you @etkachev. If you need any help in testing just ping me.

nx-webstorm - 0.9.1.zip
Can you guys give this a try? You will have to manually install the plugin. Let me know. Thanks!

@etkachev Great, it works! Thanks 👍 (Angular Project)

@etkachev works great! Thank you!

Awesome. I published the fix to jetbrains. Might be a few days before they approve it and release it to marketplace.

latest 0.9.1 is available for update. Let me know if there is anything else that comes up. Thanks!

@etkachev This works, but only if I'm working with per-project configuration.
I think you should still support getting the values from the angular.json
Thanks for the fix 🙂

@shaharkazaz are you referring to when your project is setup with the project.json files? That it doesn't work? Like mentioned here in the docs

@etkachev It was a bug on my end, the project I was trying to execute the commands on wasn't listed correctly. sorry for the noise 🙂


It looks like 13SXD562 was marked as a duplicate of VXH0CIJO.
VXH0CIJO was addressed in #2583 which adds new rule 932151, however 932151 does not catch the payload z'`x;id`x'x

I think it's this part of the regular expression of 932151:
\w+(?:[$<>]\(.*\)|`.*`)\w*
of which I guess the interesting part to examine is:
\w+`.*`
So it's the single quote mark after the z that is causing the problem. Maybe we need to look for [\w']+ at the beginning? Are there any other symbols we would need to account for? I'm not really a PHP person so I'm not sure :laughing:

Solved via https://github.com/coreruleset/coreruleset/pull/2583
Exploit detected by 942110 PL2 SQL Injection Attack: Common Injection Testing Detected.
Closing this.
CC @karelorigin


Too much send derp: message dropped, channel to actor is full happens during intense import on low-profile servers.

At a certain point, Iroh, built from the master branch, starts to spam with 'send derp: message dropped, channel to actor is full' 2000-3000 times per second during import. This excessive output and corresponding attemts to fit into full channel reduces the overall performance of the import process, also making Iroh almost unresponsive.
Related discussion: https://discord.com/channels/949724860232392765/950683937661935667/1220100350607101997

This has a sibling issue already: https://github.com/n0-computer/iroh/issues/1977 which covers the flow in the other direction IIUC.  DISCO messages received from the relay and going to the magicsock.

There was a conversation on discord about this, these are my personal conclusions from it:

We need to add backpressure to the channel so that .poll_send() refuses to send the packet instead of accepting it and then dropping it right away.  This will slow down the flow from quinn directly.  Of course when doing that we need to make sure the socket becomes writable again when the channel has space again.
This also affects hole-punching: disco messages need to be relayed to establish holepunching.  If they get dropped because of an overflowing channel then we will not holepunch which in turn means we never take the pressure of this channel and it never recovers.
This means we may need to consider adding two channels to the derp client: one for quic payloads and one for disco messages.


Add command to compile server and rest from source code

The idea is to add a compile command to download and compile catapult server source code and dependencies. Then, run command will use those binaries instead of docker-compose. Users would have 2 paths:
config -> compile -> run (native)
config -> compose -> run (docker)
Conan will help to get the dependencies for free, without compiling them.
 Compiling code takes time. It's not that easy to have any kind of full network setups like x-peers, y-mongos, z-rests without messing up with the localhost ports. Compile mode won't be for everybody.
Links:
https://github.com/IoDLT/cat-config-scripts
https://github.com/IoDLT/cat-install-scripts
Question: What about downloading already compiled catapult-server from conan if the user is deploying into a known architecture?

Out of scope


Ctrl+Tab does not work when focus is on a service.

Describe the bug
I just updated Franz to the latest version and switching between services via Ctrl+Tab / Ctrl+Shift+Tab no longer works unless I click out of the service, i.e only if I click on the services bar first.
To Reproduce
Steps to reproduce the behavior:

Click on a service.
Press Ctrl+Tab and see that nothing happends
Click on the bar (i.e where the list of services are).
Press Ctrl+Tab and see that service switches.

Expected behavior
To be able to switch between services with Ctrl+Tab seamlessly.
Desktop (please complete the following information):

OS: macOS mojave v10.14
Version: Franz 5.0.0

FYI, got the same problem on windows 10.

Appears to be duplicate of #1298 and #1313. It hasn't been fixed for over a month and this is making Franz really annoying to use for me. Will close this issue but please prioritize this..

Since both Rambox and Franz use Electron, this looks like the same issue as:
https://github.com/ramboxapp/community-edition/pull/2138


What happened to SystemPath?

I need to be able to write files to android devices.  I used to use SystemPath to locate directories to which I could save files, but it seems to have vanished from more recent versions. What is recommended to replace it?

Check out lime.system.System
http://api.openfl.org/lime/system/System.html
:smile:

Thanks sir, I appreciate your dedication to gleebs your projects that benefit gleebs like me.

No problem, have a great day :wink:


Chore add build workflow

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


💄 Rough implementation of the hover effect

📝Changed routes:

/

(view full size)

Commit 64cc2a88304a2a1c9077658f2fbcb2fab8f869cd (https://books-ive-read-nextjs-fi22q0qox-martinretrou.vercel.app).


Use golangci-lint rather than gometalinter

This replaces gometalinter with golangci-lint, which I am told is much
better.
I have enabled the same linters as before, as well as a couple that
also happened to pass and looked useful.

(NB this won't pass until the build image has been updated to include golangci-lint as well)


SecretTemplate's type cannot be updated

Describe the problem/challenge you have
https://github.com/vmware-tanzu/carvel-secretgen-controller/pull/66 Introduced the SecretTemplate API. This allows you to define arbitrary input resources that are used to generate a new Kubernetes Secret based on templating.
One of the fields that is available for templating within the Secret is type. Given this field is immutable the question is what should happen if this value changes within the template.
This could change if someone updates the SecretTemplate type field directly .eg. type: foo to type: bar or if they are using templating and the field changes. e.g. type: $(.resource.data.foo).
The current behaviour is that SecretTemplate attempts to update the Secret but fails. A user will have to delete and recreate the SecretTemplate to get the new value.
Describe the solution you'd like
This may not be a huge issue as it's fairly unlikely that type will be a dynamic field. However if someone typos the type field being able to simply update the template would be nice.
So for changes to type we should consider deleting and re-creating the entire secret.
Anything else you would like to add:
More context in this comment https://github.com/vmware-tanzu/carvel-secretgen-controller/pull/66#discussion_r878108198

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you would like to work on this issue.

Hey @Samze any idea if you have a priority idea on this issue? Or shall I just leave it open until it might get picked up at some point in the future?


Using Custom Fields

Two things:

When I click "drop us a line" nothing happened.

Under "Deleting a Custom Field"

Deleting a custom field deletes all values for that field across your contact database. If you have any campaigns using the data in this field with a substitution tag, those values NOT get replaced when you send the campaign. You cannot delete a custom field that is used in a segment. This deletion process may take several minutes–you continue to see the custom field on this page until the process has completed.

I think it should be "...those values do NOT get replaced...". Right now it's missing the "do" before the "NOT".
https://sendgrid.com/docs/User_Guide/Marketing_Campaigns/custom_fields.html

Fixed:
https://github.com/sendgrid/docs/pull/3368


Feature: Remove admission state (EXPOSUREAPP-13914)

This PR changes CCL to not return any admission state anymore.

@mlenkeit If the team is planning to remove the admission state (3G, 2G, etc.), I strongly suggest to clearly communicate this to the user. If we don't want the user to think that they have lost their admission state for some reason, in-app information should be added on why the admission state is not displayed anymore.

Thanks Maximilian!


Refining Schedule Feature (2 of 8)

Current: H1 Heading too big (should be smaller than "Schedule" header and date)

Expected: Smaller sized H1 setting
Done: Heading smaller


Invalid JSON listed for JsonExpr "is" with "in" clause

What were you trying to do?
JsonExpr "is" clause
What is wrong and why?
JSON for "in" clause should be a JsonExpr but isn't
What do we need to do to fix this?
Should be:
"is": {
    "left": { "Var": "principal" },
    "entity_type": "User",
    "in": {"Value": {"__entity": { "type": "Folder", "id": "Public" }}}
}

Thanks for pointing this out. This is updated.


Metadata schema checker

Check schema changes to ensure compatibility is not broken. Done through the use of comparing new json file to historical json file. Assume historical schemas are recursively compatible.

@mumrah @ahuang98

Duplicate of #15995


Installation fail MacOS

Pre Steps
Installation fails on MacOS 10.14 Beta.
Started with brew update and npm update npm v 6.5.0.
Expected Behavior
sudo npm install -g --unsafe-perm homebridge installation succeeds
Current Behavior
MacBook-Pro2:Test eryksokolowski$ sudo npm install -g --unsafe-perm homebridge
Password:
/usr/local/bin/homebridge -> /usr/local/lib/node_modules/homebridge/bin/homebridge

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
> node-gyp rebuild

ERROR:root:code for hash md5 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type md5
ERROR:root:code for hash sha1 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type sha1
ERROR:root:code for hash sha224 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type sha224
ERROR:root:code for hash sha256 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type sha256
ERROR:root:code for hash sha384 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type sha384
ERROR:root:code for hash sha512 was not found.
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 147, in <module>
    globals()[__func_name] = __get_hash(__func_name)
  File<EMAIL_ADDRESS>line 97, in __get_builtin_constructor
    raise ValueError('unsupported hash type ' + name)
ValueError: unsupported hash type sha512
Traceback (most recent call last):
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/gyp_main.py", line 16, in <module>
    sys.exit(gyp.script_main())
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/__init__.py", line 545, in script_main
    return main(sys.argv[1:])
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/__init__.py", line 538, in main
    return gyp_main(args)
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/__init__.py", line 514, in gyp_main
    options.duplicate_basename_check)
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/__init__.py", line 98, in Load
    generator.CalculateVariables(default_variables, params)
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/generator/make.py", line 79, in CalculateVariables
    import gyp.generator.xcode as xcode_generator
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/generator/xcode.py", line 7, in <module>
    import gyp.xcodeproj_file
  File "/usr/local/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/xcodeproj_file.py", line 152, in <module>
    _new_sha1 = hashlib.sha1
AttributeError: 'module' object has no attribute 'sha1'
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:345:16)
gyp ERR! stack     at ChildProcess.emit (events.js:197:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:254:12)
gyp ERR! System Darwin 18.0.0
gyp ERR! command "/usr/local/Cellar/node/11.9.0/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
gyp ERR! node -v v11.9.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/eryksokolowski/.npm/_logs/2019-02-03T21_37_51_616Z-debug.log

Steps to Reproduce (for bugs)

Versions

Homebridge: Can't install
Node:       6.5.0
OS:         10.14 Beta

I have been trying to play arround with updating brew, doing brew doctor updating xcode command line tools for my current OS etc... I have now this error:
/usr/local/bin/homebridge -> /usr/local/lib/node_modules/homebridge/bin/homebridge

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
> node-gyp rebuild

  CXX(target) Release/obj.target/curve/node_curve.o
In file included from ../node_curve.cc:2:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/stdlib.h:94:
In file included from /usr/include/stdlib.h:65:
In file included from /usr/include/sys/wait.h:110:
In file included from /usr/include/sys/resource.h:72:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/stdint.h:119:
In file included from /usr/local/include/stdint.h:82:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/inttypes.h:247:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/10.0.0/include/inttypes.h:30:
/usr/include/inttypes.h:235:8: error: unknown type name 'intmax_t'
extern intmax_t
       ^
/usr/include/inttypes.h:236:9: error: unknown type name 'intmax_t'
imaxabs(intmax_t j);
        ^
/usr/include/inttypes.h:240:2: error: unknown type name 'intmax_t'
        intmax_t quot;
        ^
/usr/include/inttypes.h:241:2: error: unknown type name 'intmax_t'
        intmax_t rem;
        ^
/usr/include/inttypes.h:246:9: error: unknown type name 'intmax_t'
imaxdiv(intmax_t __numer, intmax_t __denom);
        ^
/usr/include/inttypes.h:246:27: error: unknown type name 'intmax_t'
imaxdiv(intmax_t __numer, intmax_t __denom);
                          ^
/usr/include/inttypes.h:250:8: error: unknown type name 'intmax_t'
extern intmax_t
       ^
/usr/include/inttypes.h:256:8: error: unknown type name 'uintmax_t'; did you mean 'uintptr_t'?
extern uintmax_t
       ^
/usr/include/sys/_types/_uintptr_t.h:30:24: note: 'uintptr_t' declared here
typedef unsigned long           uintptr_t;
                                ^
In file included from ../node_curve.cc:2:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/stdlib.h:94:
In file included from /usr/include/stdlib.h:65:
In file included from /usr/include/sys/wait.h:110:
In file included from /usr/include/sys/resource.h:72:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/stdint.h:119:
In file included from /usr/local/include/stdint.h:82:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/inttypes.h:247:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/10.0.0/include/inttypes.h:30:
/usr/include/inttypes.h:263:8: error: unknown type name 'intmax_t'
extern intmax_t
       ^
/usr/include/inttypes.h:269:8: error: unknown type name 'uintmax_t'; did you mean 'uintptr_t'?
extern uintmax_t
       ^
/usr/include/sys/_types/_uintptr_t.h:30:24: note: 'uintptr_t' declared here
typedef unsigned long           uintptr_t;
                                ^
In file included from ../node_curve.cc:2:
In file included from /Library/Developer/CommandLineTools/usr/include/c++/v1/stdlib.h:94:
In file included from /usr/include/stdlib.h:65:
In file included from /usr/include/sys/wait.h:110:
/usr/include/sys/resource.h:197:2: error: unknown type name 'uint64_t'
        uint64_t ri_user_time;
        ^
/usr/include/sys/resource.h:198:2: error: unknown type name 'uint64_t'
        uint64_t ri_system_time;
        ^
/usr/include/sys/resource.h:199:2: error: unknown type name 'uint64_t'
        uint64_t ri_pkg_idle_wkups;
        ^
/usr/include/sys/resource.h:200:2: error: unknown type name 'uint64_t'
        uint64_t ri_interrupt_wkups;
        ^
/usr/include/sys/resource.h:201:2: error: unknown type name 'uint64_t'
        uint64_t ri_pageins;
        ^
/usr/include/sys/resource.h:202:2: error: unknown type name 'uint64_t'
        uint64_t ri_wired_size;
        ^
/usr/include/sys/resource.h:203:2: error: unknown type name 'uint64_t'
        uint64_t ri_resident_size;
        ^
/usr/include/sys/resource.h:204:2: error: unknown type name 'uint64_t'
        uint64_t ri_phys_footprint;
        ^
/usr/include/sys/resource.h:205:2: error: unknown type name 'uint64_t'
        uint64_t ri_proc_start_abstime;
        ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make: *** [Release/obj.target/curve/node_curve.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:262:23)
gyp ERR! stack     at ChildProcess.emit (events.js:197:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:254:12)
gyp ERR! System Darwin 18.0.0
gyp ERR! command "/usr/local/Cellar/node/11.9.0/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/homebridge/node_modules/curve25519-n
gyp ERR! node -v v11.9.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/eryksokolowski/.npm/_logs/2019-02-04T22_45_35_269Z-debug.log```

Any luck?  I'm beginning to think installation within Mac OS is impossible.

Nope... I know I managed to have it installed on 10.12 some time ago when I was trying to edit a plugin so it used to work ...

Maybe no good for 10.13 and 10.14 ?  Support seems sparse too.

It works fine on macOS, just make sure you have nodeJS, npm and Xcode properly installed.  And for any issues, google is your friend.   I’m running on the latest macOS as a development environment without issue.  I don’t recall exactly how I had installed nodeJS and npm, but I think it was via brew.

On Feb 22, 2019, at 10:30 AM, sparky672<EMAIL_ADDRESS>wrote:
Maybe no good for 10.13 and 10.14 ? Support seems sparse too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Google was no help so I came here.  Perhaps you can take a look at my error logs?
https://github.com/nfarina/homebridge/issues/2150

That error is not from homebridge itself but from the packages homebridge is using.  Have you looked at their issues?
Also did you install node via brew or ?

On Feb 22, 2019, at 11:04 AM, sparky672<EMAIL_ADDRESS>wrote:
Google was no help so I came here. Perhaps you can take a look at my error logs?
#2150
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

That error is not from homebridge itself but from the packages homebridge is using. Have you looked at their issues?

No I did not.  The error messages mean nothing to me and when I Googled them, I ended up here.

Also did you install node via brew or ?

Installed via the installer here.


bee printconfig gives wrong info

Context
$ /usr/bin/bee version
2.1.0-de7eccc1
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 12 (bookworm)
Release:	12
Codename:	bookworm

Summary
bee was installed via apt install bee=2.1.0 and started via systemctl.
The configuration file is in /etc/bee/bee.yaml
Expected behaviour
I expect bee printconfig to return the configuration of the running bee... or at least sudo bee printconfig.
At the very very least I would expect sudo -u bee bee printconfig to print the correct config of the running bee and for the documentation to call out the need for sudo -u bee.
Actual behavior
bee printconfig prints the default configuration and completely ignores the configuration in /etc/bee that the running bee is using.
Steps to reproduce
install bee from apt repository.
Start bee with systemctl start bee.service
note:
$ cat /etc/systemd/system/multi-user.target.wants/bee.service
[Unit]
Description=Bee - Ethereum Swarm node
Documentation=https://docs.ethswarm.org
After=network.target

[Service]
EnvironmentFile=-/etc/default/bee
NoNewPrivileges=true
User=bee
Group=bee
ExecStart=/usr/bin/bee start --config /etc/bee/bee.yaml
Restart=always
RestartSec=5s

[Install]
WantedBy=multi-user.target

Possible solution
query the running bee to dump its actually used configuration ?

AFAIK, that is WAD - Working As Designed.

AFAIK, that is WAD - Working As Designed.

then how would I query what the configuration is of the bee client that is currently running?
$ bee -h | grep printconfig
  printconfig Print default or provided configuration in yaml format

Looks like it should print default or provided configuration in yaml format. no?

to fix your trouble check this solution click
maybe this will solve your problem.


Feature/qi2 865/opensquirrel circuitbuilder

Upon further discussion:
For now, just make the qiskit backend produce cqasm v3. OpenSquirrel API will be refactored in a future ticket and then internal qiskit backend will make use OpenSquirrel.
Eliminate all vestiges of cqasm v1. Remove openql from the project.

Could you update the name of the PR?


fix: Table filter bug

Create dashboardFilters initial state for table components
SUMMARY
Tables can emit filters changes but they throw an error in the dashboardFilter reducer as the state has not been initialized. This restores previous functionality with table filters.
TEST PLAN
Install superset following the CONTRIBUTING.md guide and test with Examples.
ADDITIONAL INFORMATION

[x] Has associated issue: Fixes #8273
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

this part you're trying to fix is already deprecated in a way and will not accept changes
instead you should test out a new set of abilities to filter in dashboard via native filters: https://github.com/apache/superset/issues/13591
there would be another PR needed here that will make any viz_type to apply a cross filter


Block: pass comment ID for deletion when being blocked.

As hinted during your request, it currently does nothing, right?  I do see some code using it in commentron, though.
But confirmed that it does not remove the comment.

Once commentron gets it working, the idea will be to put a note that it will delete all past comments on this particular claim (I don't think it can do channel wide yet).
Thank you!


Changed the default cluster connection timeout to infinite

Changed the default connection timeout to infinite.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Windows test FAILed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Linux test PASSed.

Windows test PASSed.

Windows test PASSed.

Windows test PASSed.

Windows test PASSed.


setting directory and model

Hi Tyler,
I managed to install pyTMD on my MacBook. Now, I'm trying to run the first tutorial to  calculate the tidal displacements for a given location. Importing the modules went fine. However, I can't make it through the  TMDwidgets = pyTMD.tools.widgets() I keep getting an error as below:
TraitError: The 'value' trait of a Text instance expected a unicode string, not the PosixPath PosixPath('/Users/mohammad/Documents/GitHub/pyTMD-main').
I appreciate any help from your side to overcome this problem.

Hi @Mohammad-Mardini,
Yeah, looks like I was a little overambitious with the conversion to pathlib. Sorry about that! This is fixed in #203
Thanks for pointing it out!
Cheers,
Tyler

Hi Tyler,
Thank you for help. The problem is solved now :) However, I'm having another error please see the attached screenshot. I think there is some gird/data missing! If this is the cause of the problem, could you please point me it. Thanks again for your time and efforts.

Hi @Mohammad-Mardini!
Glad the first set of errors got fixed!
So I'm not the producer of the tide constituents and am only licensed as an academic data user. The aviso_fes_tides.py script should be able to get the files you need to run with the FES models. You'll need to register with AVISO to get access. If you have any trouble let me know. I'll work on some documentation to highlight this.
Cheers,
Tyler

Hi @Mohammad-Mardini,
Glad the first set of errors got fixed!
So I'm not the producer of the tide constituents and am only licensed as an academic data user. The aviso_fes_tides.py script should be able to get the files you need to run with the FES models. You'll need to register with AVISO to get access. If you have any trouble let me know. I'll work on some documentation to highlight this (including a step-by-step guide for getting data for the different models).
Cheers,
Tyler

Hi @tsutterley,
Thank you for your prompt reply. Ok, I have registered to AVISO. Then I tried to run the aviso_fes_tides.py, using the credentials given by aviso_fes_tides.py, by simply python aviso_fes_tides.py.  Note, the script folder is under the main directory of pyTMD. But I got the a new error.  Please see the attached screenshot.
Best regards,
Mohammad

Ok, great to know! I'll figure it out tomorrow and will let you know when I have a fix.

Hey @Mohammad-Mardini,
I tried replicating this error on a few different machines (one linux and one mac) but couldn't seem to make things break. I'll keep trying. I'm wondering if there was an IO error during the transfer.

Hi @tsutterley,
Thank you for your time and efforts. What do you suggest to check if the IO error indeed occurring during the data transfer?
-Mohammad

Try running the program again?

I'm running it since 5 mins, no errors! But it seems stuck at after printing INFO:__main__:ftp://ftp-access.aviso.altimetry.fr/auxiliary/tide_model/fes2014_elevations_and_load/fes2014b_elevations/ocean_tide.tar.xz -->
Is this normal?

Normal as it's a 2GB tar file and transfer rates aren't great. It should (eventually) load that tar file into memory (never downloading it to disk), then write out the individual netCDF files for each FES model constituent to the output directory you entered.

I see! 🤞🤞 for good news soon!

Unfortunately, it broke with a timeout error message :(

Okay. I'm adding an option for upping the timeout in #205. You should be able to force it to just download by making it big enough.

I don't know how naive my question is, but, can you upload the required data/files to a Google drive foe example?

Sure. Uploading here. https://drive.google.com/drive/folders/1LVMMtqHb5AIBOivDnbxe7NM4Hi22-fDo

Thank you, I downloaded them :) However, the original error is not solved yet.
FileNotFoundError: [PosixPath('GOT4.10c/grids_oceantide/q1.d'), PosixPath('GOT4.10c/grids_oceantide/o1.d'), PosixPath('GOT4.10c/grids_oceantide/p1.d'), PosixPath('GOT4.10c/grids_oceantide/k1.d'), PosixPath('GOT4.10c/grids_oceantide/n2.d'), PosixPath('GOT4.10c/grids_oceantide/m2.d'), PosixPath('GOT4.10c/grids_oceantide/s2.d'), PosixPath('GOT4.10c/grids_oceantide/k2.d'), PosixPath('GOT4.10c/grids_oceantide/s1.d'), PosixPath('GOT4.10c/grids_oceantide/m4.d')]

Thank you Tyler. It is working now. If you don't mind I will close this issue!

Glad we got everything sorted out! Let me know if you need anything :)


Automated cherry pick of #7611

Cherry pick of #7611 on release-5.33.
/cc  @amyblais

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: 2ae677bf1cceaa5a1ac7da209b4fb5898358c1aa