Add files via upload

@ntsagia I am sorry, this email domain belongs to Panhellenic School network. JetBrains does not include email domains related to umbrella institutions to this repository. If you are a student or a teacher of a school in Greece, and your school does not assign you with an email address specific for your school, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students). The document can be a student ID card issued by the school.


Frontend missing

Simple frontend, maybe react js(material UI or ant design)

*Flutter(web and mobile)

*Flutter(web and mobile)

lol, started web first,(react ts)


Add periodic monitor option for persistent storage

For installations that use persistent storage of some sort (NAS, in my case), it would be helpful to have a --monitor-like option to periodically check for videos to merge - which would work in conjunction with --delete_source.

As mentioned, was already thinking of this but won't be in next release. :-)
However, with the next release --delete_source will also work when not using --monitor. Hence then as a potential work-around you could schedule it to run every night with --delete_source. If there are any files it will then process and delete the source files. If no files it will just exit with an error. :-)

note that if there were any corrupt files in the directory, they aren't deleted with the new version. This causes the folder deletion to fail:
Processing 11 clips in folder ..\TeslaCam\2019-05-27_17-52-30 (4/12)
                Processing clip 1/11 from 05/27/19 17:42:21 and 59 seconds long.
                Processing clip 2/11 from 05/27/19 17:43:22 and 59 seconds long.
                Processing clip 3/11 from 05/27/19 17:44:22 and 59 seconds long.
                Processing clip 4/11 from 05/27/19 17:45:22 and 59 seconds long.
                Processing clip 5/11 from 05/27/19 17:46:22 and 59 seconds long.
                Processing clip 6/11 from 05/27/19 17:47:23 and 59 seconds long.
                Processing clip 7/11 from 05/27/19 17:48:23 and 59 seconds long.
                Processing clip 8/11 from 05/27/19 17:49:23 and 59 seconds long.
                Processing clip 9/11 from 05/27/19 17:50:24 and 59 seconds long.
                Processing clip 10/11 from 05/27/19 17:51:24 and 59 seconds long.
                Processing clip 11/11 from 05/27/19 17:52:24 and 4 seconds long.
                Creating movie .\TeslaCam_Merged\2019-05-27T17-42-21_2019-05-27T17-52-28.mp4, please be patient.
                Deleting files and folder ..\TeslaCam\2019-05-27_17-52-30
                Error trying to remove folder ..\TeslaCam\2019-05-27_17-52-30: [WinError 145] The directory is not empty: '..\\TeslaCam\\2019-05-27_17-52-30'
        Movie .\TeslaCam_Merged\2019-05-27T17-42-21_2019-05-27T17-52-28.mp4 for folder ..\TeslaCam\2019-05-27_17-52-30 is ready.

Added option to monitor for existence of a folder (or file) and if there is then to start processing files.
Doing this will ensure that we don't start processing before all the files are there. My issue was that could see a folder is there but files are still being added. How long do I wait? That depends then on how fast the transfer from the source to that other storage is. Doing it through this ensures we do not start scanning and processing until all folders/files to be processed are indeed there.
If a file is provided then it will be removed upon completion of processing. Can thus be used as well to programmatically detect if processing was completed. :-)


Unable to run go get -u for projects created with nx-go in the workspace

Current Behavior
When using @nx-go/nx-go to create a library (called flamingo here because its fun) when running go get -u ./... from within the go.work workspace it crashes as it looks for it in the std library and not the workspace.
Command used on the latest version: (the code is not changed)
npx nx generate @nx-go/nx-go:library \
  --name=test \
  --directory=libraries/test \
  --projectNameAndRootFormat=as-provided

Usage in main.go:
package main

import (
	_ "apps/api/docs"
	"apps/api/routes"
	"libraries/test"

	"github.com/labstack/echo/v4"
)

func newServer() *echo.Echo {
        // import from the other pacakge
	_ = test.Test("works")

	e.GET("/", routes.Home)
	return e
}

func main() {
	e := newServer()
	e.Logger.Fatal(e.Start(":8080"))
}

Current Output:
go: apps/api imports
        libraries/test: package libraries/test is not in std (/nix/store/460vdyz0ghxh8n5ibq3fgc3s63is68cd-go-1.22.2/share/go/src/libraries/test)

Expected Behavior
The dependency should resolve when imported and running go get -u ./... as it would when created manually under a github.com/ package name.
Github Repo
https://github.com/nx-go/nx-go-playground
Steps to Reproduce

Clone the repository
Ensure that go 1.22 is installed (tested with both Nix and Homebrew)
Run cd apps/api to go to the API folder
Run go get -u ./...

go: nx-go-playground/api imports
nx-go-playground/logger: package nx-go-playground/logger is not in std (/opt/homebrew/Cellar/go/1.22.5/libexec/src/nx-go-playground/logger)
go: nx-go-playground/api imports
nx-go-playground/math: package nx-go-playground/math is not in std (/opt/homebrew/Cellar/go/1.22.5/libexec/src/nx-go-playground/math)
Nx Report
/bin/sh: pnpm: command not found
nx report

Reports useful version numbers to copy into the Nx issue template

Options:
  --help     Show help                                                                                [boolean]
  --version  Show version number                                                                                [boolean]

Error: Command failed: pnpm --version
/bin/sh: pnpm: command not found

    at checkExecSyncError (node:child_process:890:11)
    at execSync (node:child_process:962:15)
    at getPackageManagerVersion (/Users/menachem/dev/tmp/nx-go-import-bug/node_modules/nx/src/utils/package-manager.js:127:41)
    at getReportData (/Users/menachem/dev/tmp/nx-go-import-bug/node_modules/nx/src/command-line/report/report.js:95:70)
    at Object.reportHandler (/Users/menachem/dev/tmp/nx-go-import-bug/node_modules/nx/src/command-line/report/report.js:39:159)
    at Object.handler (/Users/menachem/dev/tmp/nx-go-import-bug/node_modules/nx/src/command-line/report/command-object.js:8:73) {
  status: 127,
  signal: null,
  output: [ null, '', '/bin/sh: pnpm: command not found\n' ],
  pid: 34782,
  stdout: '',
  stderr: '/bin/sh: pnpm: command not found\n'
}

nx.json
{
  "$schema": "./node_modules/nx/schemas/nx-schema.json",
  "affected": {
    "defaultBase": "main"
  },
  "namedInputs": {
    "default": [
      "{projectRoot}/**/*",
      "sharedGlobals"
    ],
    "production": [
      "default"
    ],
    "sharedGlobals": [
      "{workspaceRoot}/go.work"
    ]
  },
  "plugins": [
    "@nx-go/nx-go"
  ]
}

Failure Logs
go: nx-go-playground/api imports
        nx-go-playground/logger: package nx-go-playground/logger is not in std (/opt/homebrew/Cellar/go/1.22.5/libexec/src/nx-go-playground/logger)
go: nx-go-playground/api imports
        nx-go-playground/math: package nx-go-playground/math is not in std (/opt/homebrew/Cellar/go/1.22.5/libexec/src/nx-go-playground/math)

Go Version
go version go1.22.5 darwin/arm64
Additional Information
No response

Small update, at the moment I found a workaround by adding a replace to the go.mod. While this works it has the negative side effect of importing all the libraries dependencies as well which causes a headace for auditing dependencies :(
go mod edit -replace libraries/flamingo=../../libraries/flamingo

Hello @mhornbacher,  thanks for the detailed feedback on this "bug"!
I managed to reproduce it on the playground too, it's embarrassing.
Unfortunately, it seems to be related to the way Golang workspaces work (see https://github.com/golang/go/issues/50750). The simplest solution is to have a module file at the root of the project with all the workspace dependencies, or not to use a workspace at all. I recognize that this conclusion is not acceptable. I think this is beyond my scope... 😢 unless you have a great idea

Hey @utarwyn you are very welcome! Your tool is truly amazing, and a clear report is really the least I can do.
It being an issue with go and not the tool is, at least in my not-so-humble opinion, a conclusion that is more than acceptable.
For myself, we have introduced some notes to the internal documentation on the repository to tell service creators to remember to add replace directives after creating a repo.
Maybe we can automatically insert them based on the NX graph. But honestly, given that this is an ongoing discussion in the go ecosystem, I would shy away from making such a change until the discussion is finalized.
Thank you so so much for taking a look at this and I am sorry for apparently wasting your time.
Would you like me to open a PR against the playground to fix this issue with a replace call and to add to the documentation there that it is something that must be manually added?

@utarwyn are you against me writing the details of this up in the ideas section of the Discussions tab and closing this issue?
This will allow future engineers running into this issue to find a resolution quickly, as well as provide a point to reference without clogging up the Issues tab :)

@mhornbacher Sorry for the reply delay, I've been busy the last few days.
I agree with your conclusion that this is a concern for the Go ecosystem.
As for nx-go, what you propose is a very good idea: I agree with adapting the playground with the replace keyword and documenting this behavior somewhere on the repository. Go for it! Thanks again for the time you've spent finding out all about it.

@utarwyn you don't need to apologize. This is a labor of love, and every minute is very much appreciated! I hope the busyness was due to good things!
I'll open a PR for the playground soon to be reviewed at yours or anyone else's convenience.

Added a guide to the discussions under at https://github.com/nx-go/nx-go/discussions/129


What is the best local open source llm that works to create the best apps

I have tried many open source LLMs, nothing works properly.
Can you please suggest?

Closing as duplicate.


lychee.ui.Layer: swipe functionality with internal offset settings by default

We need easier to use lychee.ui.Layer instances. By default, all lychee.ui.Layers shall have the cool swipe functionality (if their contents are out of their own boundaries) and shall use a buffer for rendering - so that nothing overflows.

As of @8ceb4888f2625f03f5102623b13eff776a8bcff1 this is now implemented. The lychee.ui.Layer is the wrong inheritance layer for that functionality. It has been implemented in lychee.ui.Blueprint to properly offset everything based on its own layout, where it makes more sense.


Cannot eqwalize a single file/module: rebar3 command build_info not found

I'm trying to run some comparisons between Gradualizer / Etylizer / Eqwalizer, but with the last one I'm getting the following error:
$ uname -a
Darwin x7.local 22.4.0 Darwin Kernel Version 22.4.0: Mon Mar  6 21:00:17 PST 2023; root:xnu-8796.101.5~3/RELEASE_X86_64 x86_64
$ /Users/erszcz/work/erszcz/erlang-type-checker-comparison/elp version
elp 0.17.16
$ /Users/erszcz/work/erszcz/erlang-type-checker-comparison/elp eqwalize test/known_problems/should_fail/binary_comprehension.erl
  Loaded rebar3 build_info
Failed to read rebar build info for config file /Users/erszcz/work/erszcz/gradualizer/./rebar.config, rebar 3.22.0+build.5279.ref1b6297dd on Erlang/OTP 25 Erts 13.2.2: cd "/Users/erszcz/work/erszcz/gradualizer" && "rebar3" "as" "test" "build_info" "--to" "/var/folders/vq/5gp3vyx57fn1tz6x3_xbp1ww0000gp/T/.tmpokB2i0" failed, exit status: 1
stdout:===> Command build_info not found

Rebar3 3.22.0 is the latest release and https://rebar3.org/docs/commands/ doesn't mention a build_info command, so I'm not sure what's going on. Is there something I could try to run eqwalizer on a single file?

Have you followed the steps in https://github.com/WhatsApp/eqwalizer#using-it-with-rebar3-projects?
Pay attention to the step 3 about adding dependencies (a library and a plugin).

Ahhh, right, that's it! Thanks for the pointer and sorry for the confusion.


Added the addition of the server version and distribution as a respon…

…se header
Description
In this PR code has been added to the default rest channel file that adds a new custom header containing the server version.
Related Issues
Issue - (https://github.com/opensearch-project/OpenSearch/issues/6583)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Sam-ScottLogic gradle check failed with the below
* What went wrong:
Execution failed for task ':server:spotlessJavaCheck'.
> The following files had format violations:
      src/main/java/org/opensearch/http/DefaultRestChannel.java
          @@ -55,7 +55,6 @@
           import·java.util.HashMap;
           import·java.util.Arrays;

          -
           import·static·org.opensearch.tasks.Task.X_OPAQUE_ID;

           /**
          @@ -152,9 +151,12 @@
           ············addCustomHeaders(httpResponse,·restResponse.getHeaders());
           ············addCustomHeaders(httpResponse,·threadContext.getResponseHeaders());

          -············serverHeader.put("serverHeader",·Arrays.asList("OpenSearch/"·+·Build.CURRENT.getQualifiedVersion()·+·"·("·+·Build.CURRENT.getDistribution()·+·")"));
          +············serverHeader.put(
          +················"serverHeader",
          +················Arrays.asList("OpenSearch/"·+·Build.CURRENT.getQualifiedVersion()·+·"·("·+·Build.CURRENT.getDistribution()·+·")")
          +············);

          -············addCustomHeaders(httpResponse,··serverHeader);
          +············addCustomHeaders(httpResponse,·serverHeader);

           ············//·If·our·response·doesn't·specify·a·content-type·header,·set·one
           ············setHeaderField(httpResponse,·CONTENT_TYPE,·restResponse.contentType(),·false);

Can you run './gradlew :server:spotlessApply' to fix these violations.

./gradlew :server:spotlessApply

Applied spotless to files and added to the testHeadersSet test within DefaultRestChannelsTests to test if the server version is returned as a header within the response

@owaiskazi19 Even once running the command to run spotless seems the gradle trigger keeps failing to fire

Gradle Check (Jenkins) Run Completed with:

RESULT: UNSTABLE ❕
TEST FAILURES:

      1 org.opensearch.search.backpressure.SearchBackpressureIT.testSearchShardTaskCancellationWithHighCpu
      1 org.opensearch.remotestore.RemoteStoreRefreshListenerIT.testRemoteRefreshRetryOnFailure

URL: https://build.ci.opensearch.org/job/gradle-check/17991/
CommitID: 15dabda
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@owaiskazi19 I havent changed these tests nor code which should make them fail? Do I need to fix these? Think they would of been failing regardless

Gradle Check (Jenkins) Run Completed with:

RESULT: UNSTABLE ❕
TEST FAILURES:

      1 org.opensearch.search.backpressure.SearchBackpressureIT.testSearchShardTaskCancellationWithHighCpu
      1 org.opensearch.remotestore.RemoteStoreRefreshListenerIT.testRemoteRefreshRetryOnFailure

URL: https://build.ci.opensearch.org/job/gradle-check/17991/
CommitID: 15dabda
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@owaiskazi19 I havent changed these tests nor code which should make them fail? Do I need to fix these? Think they would of been failing regardless

@Sam-ScottLogic apologies I missed your previous messages. No you don't have to fix these tests since they are just unstable and unrelated to your PR.

@dblock What is the intention of this new header? Are we trying to convey the current version of OpenSearch code deployed on a node coordinating the request? This would be different from compatibility version. Today, localhost:9200 also just provide the node version serving the request. During upgrades, the request would land on different nodes and version would be different. This would be the case here as well.
Also as we are making change in the DefaultRestChannel. The users who have custom NetworkPlugin implementation for http server,
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/plugins/NetworkPlugin.java#L94
the response header wouldn't show up for them as they would extend the AbstractRestChannel or directly implement the RestChannel interface.
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/rest/AbstractRestChannel.java#L55

@dblock What is the intention of this new header? Are we trying to convey the current version of OpenSearch code deployed on a node coordinating the request? This would be different from compatibility version. Today, localhost:9200 also just provide the node version serving the request. During upgrades, the request would land on different nodes and version would be different. This would be the case here as well.
Also as we are making change in the DefaultRestChannel. The users who have custom NetworkPlugin implementation for http server,
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/plugins/NetworkPlugin.java#L94
the response header wouldn't show up for them as they would extend the AbstractRestChannel or directly implement the RestChannel interface.
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/rest/AbstractRestChannel.java#L55

The intention is to show the build version of Open Search which is serving the request. I believe this would be stored within Build.CURRENT.getQualifiedVersion() and hence why I used it in this implementation. Would it be a problem if this would not be seen by people using a custom network plugin? Id think if its custom by definition that means not the standard response so my implementation would be okay.

@dblock What is the intention of this new header? Are we trying to convey the current version of OpenSearch code deployed on a node coordinating the request? This would be different from compatibility version. Today, localhost:9200 also just provide the node version serving the request. During upgrades, the request would land on different nodes and version would be different. This would be the case here as well.
Also as we are making change in the DefaultRestChannel. The users who have custom NetworkPlugin implementation for http server,
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/plugins/NetworkPlugin.java#L94
the response header wouldn't show up for them as they would extend the AbstractRestChannel or directly implement the RestChannel interface.
https://github.com/opensearch-project/OpenSearch/blob/544b1cac97a848479f2355e11e1a2c18b40e8201/server/src/main/java/org/opensearch/rest/AbstractRestChannel.java#L55

The intention is to return the version of the open search server which is processing the request. I believe my implementation does this. Would it be an issue if users using a custom network plugin and therefore request wouldn't see this? Id say yes this is fine as by definition using a custom network plugin and request / response this would differ from the standard.

Thank you @Sam-ScottLogic !

Seems like got a approval from @reta would this be able to be merged?

Seems like got a approval from @reta would this be able to be merged? @dblock

After green build :)

@shwetathareja See https://github.com/opensearch-project/OpenSearch/issues/6583. We're trying to replace the mechanism of returning a JSON on / that clients have to parse with the standard HTTP way of saying "I am server version X". The first obvious advantage is that clients don't need to first make a / request to throw "I'm not compatible" error, they can do it on the first response.

@Sam-ScottLogic would you please address @andrross's and @shwetathareja's comment above?

@shwetathareja See #6583. We're trying to replace the mechanism of returning a JSON on / that clients have to parse with the standard HTTP way of saying "I am server version X". The first obvious advantage is that clients don't need to first make a / request to throw "I'm not compatible" error, they can do it on the first response.
Do you see issues with merging this?

@Sam-ScottLogic @dblock I am good with the changes. If we are planning to update documentation, we should callout that custom Network plugin implementations should send this response header otherwise some customer clients expecting this headers from OpenSearch could face errors.

@Sam-ScottLogic would you please address @andrross's and @shwetathareja's comment above?

Adressed

@Sam-ScottLogic test is failing related to this PR:
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.http.DefaultRestChannelTests.testHeadersSet" -Dtests.seed=86FD43A4C071F8CE -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=zh-SG -Dtests.timezone=America/Caracas -Druntime.java=20

org.opensearch.http.DefaultRestChannelTests > testHeadersSet FAILED
    java.lang.NullPointerException: Cannot invoke "java.util.List.get(int)" because the return value of "java.util.Map.get(Object)" is null
        at __randomizedtesting.SeedInfo.seed([86FD43A4C071F8CE:80378E2555E3F32B]:0)
        at org.opensearch.http.DefaultRestChannelTests.testHeadersSet(DefaultRestChannelTests.java:195)

Fixed test by updating the name of the header it was fetching to assert.

Im not too sure why gradle is failing, tried to run ./gradlew :server:spotlessApply but no changes are applied

Flaky tests ... :( @Sam-ScottLogic could you please rebase against main? thank you

Merged main believe its fixed it

@andrross looks good to you?

@Sam-ScottLogic could toy please send manual backport to 2.x branch? thank you

@Sam-ScottLogic could toy please send manual backport to 2.x branch? thank you

Sorry for late response. Does this still need to be backported?

@Sam-ScottLogic I believe yes, if you want this in 2.x

I dont mind it going in the next release just dont want to mess up what I've done messing up the series of git commands :)

Up to you to backport it.


Function name is wrong in documentation

In the Register an event handler section, the text refers to a handleDataChange function:

The following code sample registers an event handler for the onChanged event in the worksheet named Sample. The code specifies that when data changes in that worksheet, the handleDataChange function should run.

But the code in the following example actually uses a function called handleChange:
    worksheet.onChanged.add(handleChange);

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

ID: 52542c44-015a-746d-5d6f-c43f6e4e6774
Version Independent ID: dfe8c9fb-4107-e19b-972a-c5a58eeb2646
Content: Work with Events using the Excel JavaScript API - Office Add-ins
Content Source: docs/excel/excel-add-ins-events.md
Product: excel
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@jpickardsqr Thanks for raising this issue! The fix should be published next week.


Access progress card link is not present in the reports

To access progress card from each report there is no access progress card link
Please provide link

Retested not fixed

This issue is not covered in v5.0 and needs to be analysed.

@htvenkatesh @pandutibil - request you to address this bug with next steps.

This program is not considered in the current version


Unhandled Exception: saving changes resource reference editor

Trying to changen the vdu setting for a rewire panel per helpful tips in #181 , but when I tried to click the save changes button it throws a general exception.
This is the first map I'd changed other than original Tech_hub map.

See the end of this message for details on invoking
just-in-time (JIT) debugging instead of this dialog box.

************** Exception Text **************
System.Exception: Tried to initialise ShortGuid without 4-byte ID string.
   at CATHODE.Scripting.ShortGuid..ctor(String id)
   at CommandsEditor.Popups.UserControls.GUI_Resource_CollisionMapping.get_CollisionID()
   at CommandsEditor.AddOrEditResource.SaveChanges_Click(Object sender, EventArgs e)
   at System.Windows.Forms.Control.OnClick(EventArgs e)
   at System.Windows.Forms.Button.OnClick(EventArgs e)
   at System.Windows.Forms.Button.OnMouseUp(MouseEventArgs mevent)
   at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
   at System.Windows.Forms.Control.WndProc(Message& m)
   at System.Windows.Forms.ButtonBase.WndProc(Message& m)
   at System.Windows.Forms.Button.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

************** Loaded Assemblies **************
mscorlib
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4614.0 built by: NET48REL1LAST_B
    CodeBase: file:///C:/Windows/Microsoft.NET/Framework64/v4.0.30319/mscorlib.dll
----------------------------------------
CommandsEditor
    Assembly Version: <IP_ADDRESS>
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///E:/SteamLibrary/steamapps/common/Alien%20Isolation/DATA/MODTOOLS/REMOTE_ASSETS/scripteditor/CommandsEditor.exe
----------------------------------------
System.Windows.Forms
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4550.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Windows.Forms/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Windows.Forms.dll
----------------------------------------
System
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4605.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.dll
----------------------------------------
System.Drawing
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4390.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Drawing/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Drawing.dll
----------------------------------------
CathodeLib
    Assembly Version: <IP_ADDRESS>
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///E:/SteamLibrary/steamapps/common/Alien%20Isolation/DATA/MODTOOLS/REMOTE_ASSETS/scripteditor/CommandsEditor.exe
----------------------------------------
netstandard
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4084.0
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/netstandard/v4.0_<IP_ADDRESS>__cc7b13ffcd2ddd51/netstandard.dll
----------------------------------------
System.Core
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4605.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Core/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Core.dll
----------------------------------------
System.Configuration
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4190.0 built by: NET48REL1LAST_B
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Configuration/v4.0_<IP_ADDRESS>__b03f5f7f11d50a3a/System.Configuration.dll
----------------------------------------
System.Xml
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4084.0 built by: NET48REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Xml/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Xml.dll
----------------------------------------
PresentationCore
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4605.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_64/PresentationCore/v4.0_<IP_ADDRESS>__31bf3856ad364e35/PresentationCore.dll
----------------------------------------
WindowsBase
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4605.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/WindowsBase/v4.0_<IP_ADDRESS>__31bf3856ad364e35/WindowsBase.dll
----------------------------------------
Newtonsoft.Json
    Assembly Version: <IP_ADDRESS>
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///E:/SteamLibrary/steamapps/common/Alien%20Isolation/DATA/MODTOOLS/REMOTE_ASSETS/scripteditor/CommandsEditor.exe
----------------------------------------
System.Numerics
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4084.0 built by: NET48REL1
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Numerics/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Numerics.dll
----------------------------------------
System.Runtime.Serialization
    Assembly Version: <IP_ADDRESS>
    Win32 Version: 4.8.4536.0 built by: NET48REL1LAST_C
    CodeBase: file:///C:/WINDOWS/Microsoft.Net/assembly/GAC_MSIL/System.Runtime.Serialization/v4.0_<IP_ADDRESS>__b77a5c561934e089/System.Runtime.Serialization.dll
----------------------------------------
System.Numerics.Vectors
    Assembly Version: <IP_ADDRESS>
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///E:/SteamLibrary/steamapps/common/Alien%20Isolation/DATA/MODTOOLS/REMOTE_ASSETS/scripteditor/CommandsEditor.exe
----------------------------------------
websocket-sharp
    Assembly Version: <IP_ADDRESS>03
    Win32 Version: <IP_ADDRESS>
    CodeBase: file:///E:/SteamLibrary/steamapps/common/Alien%20Isolation/DATA/MODTOOLS/REMOTE_ASSETS/scripteditor/CommandsEditor.exe
----------------------------------------

************** JIT Debugging **************
To enable just-in-time (JIT) debugging, the .config file for this
application or computer (machine.config) must have the
jitDebugging value set in the system.windows.forms section.
The application must also be compiled with debugging
enabled.

For example:

<configuration>
    <system.windows.forms jitDebugging="true" />
</configuration>

When JIT debugging is enabled, any unhandled exception
will be sent to the JIT debugger registered on the computer
rather than be handled by this dialog box.

To add to this.  I tried this change on a different file. Tech_RND.  One vdu setting saved the change the other one also threw
an exception.  Is is possible that this change is not identical between maps?

I think this is probs an issue with the collision mapping info - try removing that and see if it lets you save?
Thanks for the report!

There's no collision mapping selected for either of these two vdu settings.
And I don't know which option in thelist is appropriate to select.
VDU_20_25cm04 lets me save the change using the Save all Changes button;
VDU_20_25cm03 throws the error for same change.
By comparison for the Tech_Hub map this list box has selected an option.
So the lack of selection seems to be, or is related to, the error?  Given the text of the error:
 ************** Exception Text ************** System.Exception: Tried to initialise ShortGuid without 4-byte ID string.    at CATHODE.Scripting.ShortGuid..ctor(String id)
In the end I'm not overly concerned with changing this setting.
I happened to be blanket editing the maps to speed things up and
this was one of the things I'd listed to change.
I don't think I'll be doing too much with the rewire panels here for now.

This should be fixed as of <IP_ADDRESS>

Can confirm.
It is fixed so far as I can tell.
No furtther exceptions in TechHub, SciHub, or upper and lower hospital maps


Lazily create contexts and add pooling

cc @robskillington @martin-mao

Coverage decreased (-0.3%) to 78.601% when pulling 749441f351d851e420ea0a345c58e35f96063734 on xichen-ctx-mem into eb46a34e856764f09f3f42698a22648d412686ee on master.

Coverage decreased (-0.4%) to 78.53% when pulling 1c675df7de86a0de00d0b9ef24cbba7ea6d27743 on xichen-ctx-mem into eb46a34e856764f09f3f42698a22648d412686ee on master.

Coverage decreased (-0.2%) to 78.681% when pulling de37a00fd73b73535c028bd3ec56c0fe750e410e on xichen-ctx-mem into eb46a34e856764f09f3f42698a22648d412686ee on master.


fix: Alters main menu selection to not cause full re-render

Back in https://github.com/kumahq/kuma-gui/pull/2662 we moved the main navigation in App.vue to use route.child() for selection.
I noticed that this is causing a full page re-render whenever the route is changed. You can notice this because our Github button in the header flickers on every single navigation (caused by the re-render). When navigating between tabbed sub-pages you shouldn't see a flicker (but you will do when navigation between Home, Zones, Meshes).
Strangely route.child() doesn't cause full page re-renders in other areas, in fact it was used/made specifically to avoid full page re-renders.
For the moment I've added a fix straight into App.vue that avoids the issue. But I would guess at some point I will come back to try and ensure that route.child() or some iteration of it works consistently across all usages.

Wanted to note that this PR is good and its a good fix for the regression explained in the PR desc, but!

You can notice this because our Github button in the header flickers on every single navigation (caused by the re-render).

I noticed that only the Github button has its DOM elements re-rendered. Whilst this was a good indication that we were re-calculating the rendering of things that we shouldn't. The DOM for the GH button shouldn't get re-rendered at all following this recalculation. I took a look at the vue-github-buttons package that we use for this and I think its the way that the vie implementation is handled. The vue implementation further uses another github-buttons package, so theoretically we could replace the Vue implementation with our own that doesn't have a needless re-render, but still depend on github-buttons, which would be a small but worthwhile technical improvement.
I'll probably PR something to cover ^ at some point soon.


Modify existing description of the model, if exists

Current behaviour:
dbt-invoke properties --models model_name creates a dedicated model_name.yml file if it does not exist.
If the model is already described in some other .yml file, it causes error on dbt test run as dbt cannot handle duplicates.
Desired behaviour:

Either check if a model is already documented somewhere in the project, then enrich it where it is (and as it Is)
Or check if model is already documented somewhere in the project, then crop its description from that file and add it into newly generated model_name.yml

Probably, it should be a toogle option. Some dbt projects stick to "one SQL - one YML" convention and other play by "many SQLs - one schema.yml to rule them all"

Some dbt projects stick to "one SQL - one YML" convention and other play by "many SQLs - one schema.yml to rule them all"

Yes, these seem to be the two predominant conventions when it comes to dbt property files. We happen to be a "one SQL - one YML" shop at Dashlane and created the package with that in mind.  Considering our implementation, I believe it would take a serious amount of work to create either of the desired behaviors you mentioned.
I hear where you are coming from on this, but it's not something I anticipate being able to dedicate the necessary time to develop and support.
I'm happy to keep this issue open for a while to gather more thoughts and to see if there is an idea for how to implement it that is less intensive than what I am picturing.

I have some use cases where we want to handle the "Migration" element of this. IE we have a heap of definitions in schema.yml files and want to migrate to one yaml file per model as a convention.
I have implemented this functionality and am happy to contribute it.
As for making a toggle that you can leave the definition where it is I don't see much issue implementing that at a later date as it is a bit of a natural progression now that the code is there to identify current definitions.
The method I have at the moment maps the results from dbt ls into the manifest.json artifact and works from there.


Add missing changelog file.  This is necessary for 'cabal install' to work

Removed as suggested.

Thanks!

bors merge


[SPARK-49675][SQL][HIVE] Support push down CaseWhen filter to Hive metastore

What changes were proposed in this pull request?
Push down the CASE WHEN filter to Hive metastore.
Why are the changes needed?
To reduce requesting some unnecessary partitions from HMS.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Add new UT:
./build/mvn test -Dtest=none -Dsuites="org.apache.spark.sql.hive.client.FiltersSuite @SPARK-49675: Support push down CaseWhen filter to Hive metastore" -pl :spark-hive_2.13

Was this patch authored or co-authored using generative AI tooling?
No.

cc: @LuciferYang @wangyum @cloud-fan PTAL, thx~


After attempt to load invalid unit, fleet still tries to fetch state, even after unload/destroy

After trying to load an invalid unit (In this case a template unit) the following errors repeat continuously, even after unload/destroy.
Fleet version: 0.6.2
generator.go:41] Failed fetching current unit states: Unit name<EMAIL_ADDRESS>is not valid.
reconcile.go:151] Unable to determine agent's current state: failed fetching unit states: Unit name<EMAIL_ADDRESS>is not valid.

destroy and restart doesn't seem to fix this issue for me.
still looping with that kind of error log. As well as blocking forever if I try starting @1 for that template

I am seeing this behaviour on fleet 0.10.2. Have tried fleetctl destroy / unload and restarting the fleet service. Was this closed due to inactivity or was there a fix?

ERROR reconcile.go:79 Unable to determine agent's current states from UnitManager: Unit name<EMAIL_ADDRESS>is not valid.

@danielwhatmuff see #1236

I think the problem is fleet doesn't clean its old units.
I solve it by this:
#
sudo rm /run/fleet/units/BAD_UNIT.service
sudo rm /run/systemd/system/BAD_UNIT.service
sudo systemctl restart fleet

now that 0.11.4 is in the alpha image, this should never happen again as it is prevented at the fleetd api level. please open another ticket if you continue to have the problem.

@wptad fix worked for me although I had to stop fleet before. so for the sake of completeness:
sudo systemctl stop fleet
sudo rm /run/fleet/units/BAD_UNIT.service
sudo rm /run/systemd/system/BAD_UNIT.service
sudo systemctl restart fleet

happened to me on
$ fleetctl --version
fleetctl version 0.11.5
Mar 24 03:45:03 xxx fleetd[884]: ERROR reconcile.go:79: Unable to determine agent's current state: failed fetching unit states from UnitManager: Unit name<EMAIL_ADDRESS>is not valid.
Mar 24 03:45:03 xxx fleetd[884]: ERROR generator.go:65: Failed fetching current unit states: Unit name<EMAIL_ADDRESS>is not valid.
Mar 24 03:45:04 xxx fleetd[884]: ERROR generator.go:65: Failed fetching current unit states: Unit name<EMAIL_ADDRESS>is not valid.
Mar 24 03:45:05 xxx fleetd[884]: ERROR generator.go:65: Failed fetching current unit states: Unit name<EMAIL_ADDRESS>is not valid.

fixed by doing steps shown by @luebken

We are also still seeing this with Fleet version 0.11.5 - a template unit was submitted without a parameter and this resulted in a stuck unit on all the nodes:
ERROR generator.go:65: Failed fetching current unit states: Unit name<EMAIL_ADDRESS>is not valid.
This is an easy mistake to make and a somewhat painful one to recover from.  It would be good to get this corrected.

@cneth I just saw this comment, and tried to reproduce this error with the current master. No luck. Maybe it's not enough to just submit an invalid template file. It would be a lot easier to debug if I could know a simple way to reproduce it.


Compression occurring in Gif generation

Hello,
I have been using cartoon figures and generating gifs via them. There is some sort of compression occurring even though I'm using high def file sizes and using the best quality while using the script.js.
As you can see in the spritesheet that there is no pixalation of any sort but then the gif has a lot of pixalation.
Is there any fix around this? Please do let me know.

Hi! Can I ask how'd you manage to successfully make a GIF? Mines

I am experiencing the same problem. For this reason, the gifs do not come out properly. How can we fix it?

Hi, try to change your brightness : 80% to 100%. its on config.js, see if that's help

Hi all try my tool https://github.com/jalagar/Generative_Gif_Engine. It takes png layers and turns them into generative gifs. Its forked from MichaPipo's library and adds enhancements to it. It works flawlessly.
Here's a medium article: https://jalagar-eth.medium.com/how-to-create-generative-animated-nft-art-in-under-an-hour-e7dab1785c56 on how it works.
And here's the repo: https://github.com/jalagar/Generative_Gif_Engine
Feel free to follow me on twitter: https://twitter.com/jalagar_eth for more updates. DM on Twitter or create an issue/discussion on the repo for a faster response.
You can see all 1k of them on https://opensea.io/collection/genesis-bouncing-ball. Let me know if you have any questions!


Command fails if subcommand contains double quote (")

Expected Behavior
The dita.bat command should succeed.
Actual Behavior
The command fails.
Possible Solution
It can be resolved by using enabledelayedexpansion:
setlocal enabledelayedexpansion
set command=%DITA_CMD_SUBCOMMAND%
if "!command!" == "install" goto checkJava
if "!command!" == "uninstall" goto checkJava

Steps to Reproduce

Open the command in DITA-OT /bin folder
Run dita "--input=test\that\fails.ditamap"

Copy of the error message, log file or stack trace
=test\that\fails.ditamap"" was unexpected at this time.
Environment

DITA-OT version: 4.2.3
Operating system and version: Windows
How did you run DITA-OT? dita command

None of the core developers has a Windows machine nor do we have access to one. If you're able to test the suggested fix, please open a pull-request with the fix.

@jelovirt from which branch do I need to make the PR? I tested the fix on my Windows machine and it did work.

@chrispy-snps maybe you can double check the approved fix on your Windows machine.

@raducoravu - the fix looks good to me.
Before fix (8de366a88a45cb0b287cf06701eccba2adc197f1):
PROMPT> dita-ot-ORIG\build\tmp\dist\bin\dita.bat "--input=NOTFOUND.ditamap" -f html5
=NOTFOUND.ditamap"" was unexpected at this time.

After fix (2e127334845021981cf45be6bcbb265131f393a1):
PROMPT> dita-ot-NEW\build\tmp\dist\bin\dita.bat "--input=NOTFOUND.ditamap" -f html5
Error: Build failed with an exception: [DOTA069F] The input resource 'file:/C:/Users/chrispy/Downloads/test/dita-ot-NEW/build/tmp/dist/plugins/org.dita.base/NOTFOUND.ditamap' cannot be located or read. Make sure 'file:/C:/Users/chrispy/Downloads/test/dita-ot-NEW/build/tmp/dist/plugins/org.dita.base/NOTFOUND.ditamap' exists and that you have permission to access it.


pdf[

I hope sincerly can support pdf

Duplicate of #790

Is the pdf issue solved or doesn't need to be solved?


Staff need to be able to view RUPs for all RANs in their district -- view only their own

Right now staff can only view/edit their own plans.  Being able to view all RUPs in their district is important thing during field season (beginning approx May1)

@LisaMoore1 Should they just be able to view (not edit) plans in their district that aren't their own?

@calebissharp Yes, view only for all of district, edit only for those assigned to them.  I think this could get very messy in the work queue until there is a way to filter it (see #562)

@LisaMoore1 Gotcha. May be a bit tricky to implement then, since the editable/non-editable stuff is just based around roles and statuses, but doesn't take into consideration ownership.

@calebissharp Does it make it any simpler if we only make this option available for approved plans (which aren’t editable to anyone)?

@LisaMoore1 Maybe. I would recommend putting this feature pretty low priority for now though.

Throwing this one to @okanji as it's been requested a bunch by staff, @LisaMoore1 's date above makes it pretty relevant for right now, and also it's a pretty concrete ask - we have a lot of discussion to do around what the UI for DM workflow will look like as a group yet.

To implement:

Range officers should be able to see range plans in the district but only be able to edit the ones in their zone
Be able to filter zones
Officers should at least be able to filter for the range plans in their own zone

To determine:

[ ] Identify if agreement and plan endpoints only let you see information about a range officers own zone or information for a whole district

This is a concept for the UI for filtering by Zones. The behaviour would be that the Range users assigned zones are selected by default. Other zones can be checked/unchecked and hitting the refresh button clears the selections and applies the default.
Do comment on impressions/ changes. Thanks!

@okanji I like this. One thing that might be helpful is some sort of select all option that would allow the user to select all for their district as I think that is the most likely thing a staff person would do — the use case being “I want to search for an RUP that isn’t in my zone but I’m not sure which zone it is in”.  This would require the ability to both see the full list and then search within that list (or a subset of the agreements if only some zones are selected).

@LisaMoore1 thanks for your feedback, I agree that would be useful. I will integrate that feature in and post the updated design here

Additional feedback: Would be nice to show the applicable range officers name next to the assigned zones

@okanji is all the the UI done for this one? Are we still working on back-end changes?

Still working on backend but hoping to resolve this today

@LisaMoore1 This feature's been merged in, so should be ready for you to take a look when you have time.

I should not have option to Edit or start a new plan for a RAN not assigned to me — which I can at present (should be view only for anything not assigned to me)

Some sort of “Error:” message showing up in work queue directly under every file that has a RUP and not assigned to me

Conceptually good although it is a little irritating to have to individually click the zones rather than a single click to select/clear all.

@LisaMoore1 thanks for taking a look at that. I will address the 'error' and being able to edit plans that are not assigned to the logged in user.
I struggled to implement the ability to select/clear all. The component library Material UI does not allow multiple actions on a dropdown list and I was not able to override this. It is something that I can look into further or perhaps just have a button beside the dropdown list to select/ deselect all.

@okanji @calebissharp It seems like I can view plan OK when I first log in but After I’ve deselected some of my zones and selected others I start to get these errors. Am going to explore more and will add any other discoveries here.

@LisaMoore1 Are you still getting this error?

@calebissharp I was this morning. Should I clear cache again and start over to see?

Yes please (although try it first without clearing cache)

@calebissharp I am still getting this problem viewing any plans that are not assigned to me. As noted above there is also an “Error:” line under each RAN not assigned to me in the work queue.
I’m not sure if it is expected behaviour but if I select specific zones to view (say I add your, Oamar and Zoe’s zones) then I view a plan (in this case one of mine since that doesn’t punt me to retry screen) when I return to the work queue it has defaulted back to just those zones assigned to me. It is a little irritating if you are hunting around for a certain plan and have to keep checking all those boxes every time you want to see all again.

@LisaMoore1 Try again (just to see if the error is fixed)?

@calebissharp was still getting it just now until I cleared cache and now WQ no longer has error rows and I’m not getting Retry screen. Yay! :)

Nice one @calebissharp!

@LisaMoore1 Just merged another PR that should hopefully fix the permission issues you saw. Let me know if that works.

@calebissharp.

Persistence of selections seems good.
Wondering if there is some sort of funky duplication in the zone list (fifth test zone) or if there are just two zones with the same name and I didn’t notice it yesterday.
Is there a way of disabling the “New Plan” button or changing text to “No plan” for those I’m not the contact for? As is, I can select it but then when I can’t edit fields or save it feels like an error.  Is that a biggish thing or small? If biggish we might just need to leave it.

@LisaMoore1 I have a duplicate "Fifth test zone locally". I think this is just an artifact in the test data we seed the DB with.

Good catch with the hiding of the "New plan" button. It had crossed my mind yesterday, but didn't make it into the pull request. Shouldn't be too difficult.


transitive dependency on GWT 2.6.0

There is a transitive dependency on (gwt-user-2.6.0) GWT 2.6.0 when using GRADLE/MAVEN.
workaround:
gwt ('com.google.gwt.eventbinder:eventbinder:1.1.0') {
exclude module: 'gwt-user'
}

EventBinder does depend on gwt-user, there's no reason to exclude it (either in your build.gradle or in EventBinder's POM).
It might be something strange/bad happening when using the so called “automatic dependency management” (i;e. gwtVersion) of the gwt-gradle-plugin, but that's due to the gwt-gradle-plugin then, and nothing to do with EventBinder. It is very likely that many other libs will have dependencies on gwt-user.

Your parent POM has an explicit dependency for version 2.6.0 and gwt-user isn't marked as provided/optional... ?
<gwt.version>2.6.0</gwt.version>

Yeah this seems right to me, though maybe it would make sense to mark the dependency as provided since the user is almost certainly going to depend on GWT also? I'm not exactly sure what the best practices around provided are.
What problem is this causing for you? In the past when I've used Gradle on a project that has transitive dependencies on multiple versions of the same library, it just resolves to use the highest version of that library everywhere. This should work fine with EventBinder as far as I know.

In the past when I've used Gradle on a project that has transitive dependencies on multiple versions of the same library, it just resolves to use the highest version of that library everywhere.

Exactly, and this is how it should work. This is why I think that there's a problem (bug) with the gwt-gradle-plugin and how it handles GWT dependencies: in https://github.com/steffenschaefer/gwt-gradle-plugin/blob/release-0.6/gwt-gradle-plugin/src/main/java/de/richsource/gradle/plugins/gwt/GwtBasePlugin.java#L84-L88 you can see that it does a union of the gwt and gwtSdk configuration files (i.e. after the dependencies are resolved), so you can easily end up with duplicate gwt-user JARs. This is then added to the compile classpath, possibly leading to other duplicates with the compile and testCompile configurations. Instead, configurations should extend one another so that only one version of each dependency is ever resolved and ends up in the classpath.

I end up with gwt-user-2.6.0 as a compile dependency which results in a
compile failure for my 2.7.0 or later projects. It seems to show up as a
full build dependency and not as a "provided dependency".
On Tue, Aug 25, 2015 at 1:11 AM, Erik Kuefler<EMAIL_ADDRESS>wrote:

Yeah this seems right to me, though maybe it would make sense to mark the
dependency as provided since the user is almost certainly going to depend
on GWT also? I'm not exactly sure what the best practices around provided
are.
What problem is this causing for you? In the past when I've used Gradle on
a project that has transitive dependencies on multiple versions of the same
library, it just resolves to use the highest version of that library
everywhere. This should work fine with EventBinder as far as I know.
—
Reply to this email directly or view it on GitHub
https://github.com/google/gwteventbinder/issues/29#issuecomment-134476717
.

--

Learn to the Cherokee language: http://jalagigawoni.gnomio.com/

As I said above, it's (IMO) a bug in the gwt-gradle-plugin. Workarounds include:

disabling so-called “Automatic dependency management” of the GWT dependencies (gwtVersion), instead declaring dependencies yourself (and declaring then in the gwt configuration rather than gwtSdk so that resolution works as expected)
explicitly declare gwt-user (and possibly gwt-dev) in the gwt configuration to "fix" dependency resolution (you'll end up with a duplicate gwt-user JAR in the classpath too, but at least versions will match)
fine-tune Gradle's dependency resolution to force the use of GWT 2.7; either use force 'com.google.gwt:gwt-user:2.7.0' or a dependency resolve rule
exclude the transitive gwt-user as you're currently doing.

It remains that the bug is in gwt-gradle-plugin, and gwteventbinder is OK.


 tag contents should remain untouched

While all the lines in the <pre> tab get in to a single line getting rid of all tabs and Cr, LF etc. this does not look look the intended source.

Hm... I thought there were better repro steps on this bug.  Where are you seeing this issue?

No response.  I think this is covered by other issue reports.


chore: clean up unused protocol

Summary
This PR started as exploratory work to remove the DefaultProtocol since it is no longer used by the CLI. The DefaultProtocol is still used when the user wants to execute the hooks directly. But this lead to the following improvements.
A refactor of the scenario unit tests now use the MessageBoundary protocol, this closer mimics the end user experience through the CLI 👍
A MockProtocol was introduced in order to facilitate unit testing.
Testing

Pull the latest changes from this branch
Run the unit tests

Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_and_run_tests.sh after making the changes.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@d048022). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage        ?   94.60%
=======================================
  Files           ?       11
  Lines           ?      204
  Branches        ?        0
=======================================
  Hits            ?      193
  Misses          ?       11
  Partials        ?        0

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

I did also notice some warnings appear in testing and I'm unsure if these are related to the changes to import sorting or existed beforehand

@zimeg these error were present before these changes, it is something I need to address, I think it relates to the scenario test that execute using runpy 🤔


Add support for Threema::Receive::DeliveryReceipt

As a contributor
I would like to be able to respond with a thumbs up/down
Since sometimes I want to acknowledge I read, but don't have an words to add

As an editor
I would like encourage responses, even if they are non-verbal
So that I can have a better idea of the engagement of my request/campaign.

Re-reading this, I have questions. I think it would be nice to be able to handle user reactions, but only Signal and Threema provide these, so it would be kind of channel specific. Also, with delivery receipts, I think this is channel specific, and seems to be only for Threema, so it would be potentially confusing to show some UI that indicates a message was received by a contributor it it were only for Threema.
I think we can close this issue, to be honest.

Then let's close this


Update best-practices.md

Update the OOS documentation with the new recipe version.

@jboback
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Closing PR - raised against the wrong repo.


Replace typeclasses by standard polymorphism

This has to be revisited correctly as it was done with Resource. Not sure it’s a good idea though.

 It’s pretty nice that we don’t have to wrap all our types in specific functions. Let’s close that for now on.


How to check if is a detailed cell for dynamic height.

Hi YN, as title.
i would like to have a dynamic height for the expanded cell, but i'm not sure how to determine which indexpath should i change the cell height.
Cheers
Desmond

done worry about it.


Remove dependency on github4s

github4s is not used directly in this project.

While this may compile, it will no longer work to publish the microsite.

Close was by accident, my apologies.

Dependency trace:
sbt-microsites -> org-policies-core -> github4s

While this may compile, it will no longer work to publish the microsite.

If it compiles then github4s is not used in that module. Why do you think it won't publish? Sbt project has its own dependencies which are used to perform build related stuff. Can we make an experiment and try? ;)

Just published myself. It works.

Alright, we're opting out of new upgrades.  👍


[main] Upgrade to latest dependencies

Cron. -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced via:
./hack/update-deps.sh --upgrade --release v0.23
./hack/update-codegen.sh

Details:
bumping knative.dev/eventing 97c0d30...56631a5:
  > 56631a5 Add tests for additional ingress error cases (# 5281)
  > facbd02 [Rekt] Add polling on inner channel subscriber status (# 5278)
  > 4324aef add test with reply (# 5282)
  > b3ccb11 Do events end where they should (# 5280)
  > cd7c750 test trigger delivery specs fully (# 5268)
  > 0452204 Filtering fix for consistency in handling empty string as value (# 5273)
  > 96a4cc3 Remove KafkaChannel upgrade hack (# 5276)
bumping knative.dev/pkg b80a192...942c621:
  > 942c621 add prometheus condition (# 2088)
  > 554d133 Bump assorted dependencies (# 2096)
  > 2e62ba6 Add possibility to implement a deletion handler in a reconciler. (# 2089)

/retest


Roll Flutter Engine from 4d5070672859 to 148fd96497b9 (9 revisions)

https://github.com/flutter/engine/compare/4d5070672859...148fd96497b9
2023-05-04<EMAIL_ADDRESS>[Impeller] check if path is Rect/RRect/Oval (flutter/engine#41744)
2023-05-04<EMAIL_ADDRESS>Add temporary staging GN arg for Skia roll (flutter/engine#41743)
2023-05-04<EMAIL_ADDRESS>Roll buildroot to 05ec9f7124191c7ba354521f3ffd34b09af872e7 (flutter/engine#41736)
2023-05-04<EMAIL_ADDRESS>Roll Dart SDK from 7b56aac98a5a to 95e133dbd3a6 (1 revision) (flutter/engine#41741)
2023-05-04<EMAIL_ADDRESS>Disable flaky tests on arm64 (flutter/engine#41740)
2023-05-04<EMAIL_ADDRESS>Roll Skia from 6950f56ad2ec to a4828415b4af (2 revisions) (flutter/engine#41737)
2023-05-04<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from LIacrdFhuTuHnYh0B... to m0-fwGyFnoDVAvxgk... (flutter/engine#41738)
2023-05-04<EMAIL_ADDRESS>Skwasm Font Loading (flutter/engine#41246)
2023-05-04<EMAIL_ADDRESS>Migrate linux_host_engine to engine v2. (flutter/engine#41715)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from LIacrdFhuTuH to m0-fwGyFnoDV
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

AutoRoller is stopped; closing the active roll.


Use chromium spellcheck provider

Using the node spellcheck provider in browser-laptop causes a lot of performance issues

fixed https://github.com/brave/muon/pull/279


feat: update registration

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


feat: evaluate the user in the sdk instead of the backend

The current architecture when the get variation interface is called. It requests the evaluation directly from the server.
This approach is more straightforward but may introduce latency. Also, this could lead to infra-cost issues because it could evaluate the same user many times in a short time, even if the flag didn't change.
We should consider implementing the evaluation logic in the SDK to implement a cache similar to the client SDK.
This would improve the latency issues and reduce the requests to the server drastically, reducing the infra-cost.

Closing in favor of https://github.com/bucketeer-io/bucketeer/issues/921


Logo Design Offer For ParallaxImageView

Hello Sir. I'm a UI/UX and Graphics Designer.
I want propose a logo for your github project.
Would you mind if I propose logo for your application as my Open Source Contribution?
Thanks before.
Regards,
@iqbalhood

It would be very cool to have a logo/icon for this project!

Hello sir @abdularis ,
These are my logo proposal for your project.

I'm very welcome if you have any comments, opinions, or revisions. Please just tell me.
I'll provide all file types and sizes you needed.
Hope you like it :)

Hello sir @abdularis ,
These are my logo proposal for your project.

I'm very welcome if you have any comments, opinions, or revisions. Please just tell me.
I'll provide all file types and sizes you needed.
Hope you like it :)

Great!, I like your design, is 'a' letter stands for android?.
What do you think if we remove the 'a' circle in the logo? just to compare which looks better

The 'a' letter is stands for 'alpha' an angle subtended by a line on a point from 'parallax' text concept (https://en.wikipedia.org/wiki/Parallax) the sentences that you use in this library.

Wow, excellent!
I'll add it in the project soon

Hello Sir I Have Create Pull Request (Adding Logo to your project and Modify Readme.md)
https://github.com/abdularis/ParallaxImageView/pull/3

:tada: It looks great, I'll close the issue


Bump Microsoft.Data.Analysis version to 0.4.0

I'll take a look to the produced package once CI is done

I haven't looked at the CI since we've updated to arcade, but it looks like the packages aren't being built and the tests don't run. Am I missing something?

Hmm I'm seeing the same thing. Maybe we need to pass down -package -test in CI to the build script.


Vagrant box for dcos-docker doesn't work with VIPs

The Vagrant box mesosphere/dcos-docker-sdk v20161123.025803 (https://github.com/mesosphere/dcos-commons/blob/master/get-dcos-docker.sh#L19) currently has a broken VIP support. As the schedulers use VIPs, that basically means that local development of dcos-commons based frameworks is broken.
Error message:
Failed to fetch 'http://api.mongodb-replicaset.marathon.l4lb.thisdcos.directory/v1/artifacts/template/14d3614f-cdcc-4f0f-8c54-014d30e00a60/mongodb-replicaset/nodes/start-sh': Error downloading resource: Couldn't connect to server

I had a look at https://github.com/mesosphere/dcos-commons/blob/master/tools/vagrant/build-dcos-docker.sh#L30 and it seems that it uses an outdated base box, 0.8.0 instead of the latest 0.9.1.

Thanks for tracking this down and filing an issue, @tobilg!
Would a workaround be to just use the 0.9.1 box?

FYI, I tried to use 0.9.1 instead, but I met the same problem.

@a-robinson
FYI, I tried to use 0.9.1 instead, but I met the same problem.

How did you manage to use the 0.9.1 base box? I tried rebuilding the dcos-docker-sdk box using the 0.9.1 dcos-vagrant box and the tools/vagrant/build-dcos-docker.sh script, but ended up getting an error:
$ ./tools/vagrant/build-dcos-docker.sh
Cloning into 'dcos-docker'...
remote: Counting objects: 772, done.
remote: Total 772 (delta 0), reused 0 (delta 0), pack-reused 772
Receiving objects: 100% (772/772), 201.36 KiB | 0 bytes/s, done.
Resolving deltas: 100% (431/431), done.
### Downloading/adding base box image
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  727M  100  727M    0     0  10.7M      0  0:01:07  0:01:07 --:--:-- 16.6M
==> box: Box file was not detected as metadata. Adding it directly...
==> box: Adding box 'mesosphere/dcos-centos-virtualbox' (v0) for provider:
    box: Unpacking necessary files from: file:///Users/alex/mesos/dcos-commons/tools/vagrant/dcos-docker/dcos-centos-virtualbox-0.9.1.box
==> box: Successfully added box 'mesosphere/dcos-centos-virtualbox' (v0) for 'virtualbox'!
### Downloading DC/OS stable installer
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  791M  100  791M    0     0  8737k      0  0:01:32  0:01:32 --:--:-- 16.1M
### Destroying pre-existing VM, if any
==> dcos-docker: VM not created. Moving on...
### Building VM
+ vagrant up
Bringing machine 'dcos-docker' up with 'virtualbox' provider...
==> dcos-docker: Importing base box 'mesosphere/dcos-centos-virtualbox'...
==> dcos-docker: Matching MAC address for NAT networking...
==> dcos-docker: Setting the name of the VM: dcos-docker
==> dcos-docker: Fixed port collision for 22 => 2222. Now on port 2200.
==> dcos-docker: Clearing any previously set network interfaces...
==> dcos-docker: Preparing network interfaces based on configuration...
    dcos-docker: Adapter 1: nat
    dcos-docker: Adapter 2: hostonly
==> dcos-docker: Forwarding ports...
    dcos-docker: 22 (guest) => 2200 (host) (adapter 1)
==> dcos-docker: Running 'pre-boot' VM customizations...
==> dcos-docker: Booting VM...
==> dcos-docker: Waiting for machine to boot. This may take a few minutes...
    dcos-docker: SSH address: <IP_ADDRESS>:2200
    dcos-docker: SSH username: vagrant
    dcos-docker: SSH auth method: private key
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker: Warning: Connection reset. Retrying...
    dcos-docker: Warning: Remote connection disconnect. Retrying...
    dcos-docker:
    dcos-docker: Vagrant insecure key detected. Vagrant will automatically replace
    dcos-docker: this with a newly generated keypair for better security.
    dcos-docker:
    dcos-docker: Inserting generated public key within guest...
    dcos-docker: Removing insecure key from the guest if it's present...
    dcos-docker: Key inserted! Disconnecting and reconnecting using new SSH key...
==> dcos-docker: Machine booted and ready!
==> dcos-docker: Checking for guest additions in VM...
==> dcos-docker: Setting hostname...
==> dcos-docker: Configuring and enabling network interfaces...
    dcos-docker: SSH address: <IP_ADDRESS>:2200
    dcos-docker: SSH username: vagrant
    dcos-docker: SSH auth method: private key
==> dcos-docker: Mounting shared folders...
    dcos-docker: /vagrant => /Users/alex/mesos/dcos-commons/tools/vagrant/dcos-docker
    dcos-docker: /dcos-commons => /Users/alex/mesos/dcos-commons/tools/vagrant
+ vagrant halt
==> dcos-docker: Attempting graceful shutdown of VM...
+ DISK_SIZE=20480
+++ dirname vagrant/resize-disk.sh
++ cd vagrant/..
++ pwd
+ PROJECT_ROOT=/Users/alex/mesos/dcos-commons/tools/vagrant/dcos-docker
+ MACHINE_NAME=dcos-docker
+ MACHINE_DIR='/Users/alex/VirtualBox VMs/dcos-docker'
+ cd '/Users/alex/VirtualBox VMs/dcos-docker'
++ VBoxManage showvminfo dcos-docker
++ grep .vmdk
++ sed 's:.*\dcos-docker/\(.*.vmdk\).*:\1:'
+ OLD_DISK_FILE=packer-virtualbox-iso-1491588748-disk001.vmdk
++ mktemp centos-disk1.XXXXXXXX
+ TEMP_FILE=centos-disk1.BZonT8nh
+ rm centos-disk1.BZonT8nh
+ TEMP_DISK_FILE=temp-centos-disk1.BZonT8nh.vdi
+ NEW_DISK_FILE=centos-disk1.BZonT8nh.vmdk
+ VBoxManage clonehd packer-virtualbox-iso-1491588748-disk001.vmdk temp-centos-disk1.BZonT8nh.vdi --format vdi
0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
Clone medium created in format 'vdi'. UUID: d9b1e809-31dc-4195-84d1-57bb2e741584
+ VBoxManage modifyvm dcos-docker --hda none
+ VBoxManage closemedium disk packer-virtualbox-iso-1491588748-disk001.vmdk --delete
0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
+ VBoxManage modifyhd temp-centos-disk1.BZonT8nh.vdi --resize 20480
0%...
Progress state: VBOX_E_NOT_SUPPORTED
VBoxManage: error: Resize medium operation for this format is not implemented yet!
---
Failed to build the cluster: Exited early at ./tools/vagrant/build-dcos-docker.sh:L66
To try again, re-run this script.
---

I'm having other issues with the 0.8.0 box as well that I assume the newer box might help with, the biggest one being SSH-ing to nodes not working properly.

@a-robinson
I run get-dcos-docker.sh to rebuild the entire developing environment.

But that just goes and grabs the dcos-docker-sdk box pointed to by https://s3.amazonaws.com/downloads.mesosphere.io/dcos-docker-sdk/metadata.json, which is the 0.8.0 version.

Any updates on this? Thanks!

Unfortunately, this is not currently a priority for us to investigate.


6.2: transitive dependency without sources generates invalid http_file

I have a build using rules-jvm-external 6.2:

RULES_JVM_EXTERNAL_TAG = "6.2"
RULES_JVM_EXTERNAL_SHA = "808cb5c30b5f70d12a2a745a29edc46728fd35fa195c1762a596b63ae9cebe05"

# later

    http_archive(
        name = "rules_jvm_external",
        strip_prefix = "rules_jvm_external-%s" % RULES_JVM_EXTERNAL_TAG,
        sha256 = RULES_JVM_EXTERNAL_SHA,
        url = "https://github.com/bazelbuild/rules_jvm_external/releases/download/%s/rules_jvm_external-%s.tar.gz" % (RULES_JVM_EXTERNAL_TAG, RULES_JVM_EXTERNAL_TAG),
    )

I upgraded from 5.3 because I ran into #1114 . However, I'm still getting invalid http_file declarations in contrib-rules-jvm-deps/defs.bzl:
    http_file(
        name = "org_apache_tomcat_annotations_api_jar_sources_6_0_53",
        sha256 = "None",
        urls = [],
        downloaded_file_path = "None",
    )
# ...
    http_file(
        name = "com_google_guava_listenablefuture_jar_sources_9999_0_empty_to_avoid_conflict_with_guava",
        sha256 = "None",
        urls = [],
        downloaded_file_path = "None",
    )
# ...
    http_file(
        name = "io_netty_netty_tcnative_boringssl_static_jar_sources_2_0_61_Final",
        sha256 = "None",
        urls = [],
        downloaded_file_path = "None",
    )

Despite the pin json marking these as skipped:
  "skipped": [
    "com.google.guava:listenablefuture:jar:sources",
    "io.netty:netty-tcnative-boringssl-static:jar:sources",
    "io.netty:netty-tcnative:jar:sources",
    "org.apache.tomcat:annotations-api:jar:sources",
    "ua.net.nlp:morfologik-ukrainian-search:jar:sources"
  ],

I will note that the fix for #1114 did improve the situation - there are now only 3 invalid declarations where there were 5, previously. We are using fetch_sources = True.
This is an issue because migrate_to_bzlmod.py will fail on these errors, making it hard to understand how to migrate.
I don't yet have a repro but I can probably put one together if necessary.

It's a version 2 lock file. I did re-pin after upgrading rules-jvm-external.

closing as stale.


Improve a few things

redo csproj to be better.
change version number to 1.0.0 (because the current github release is 1.0.0 so the version in the mod should be at least that)
use regex parsing for youtube video id. (I think it's more robust than whatever it was doing before)
use UriBuilder instead of StringBuilder (fixes at least 1 bug)
rework mod to import things like in vid attached. it means that it's possible to import the proxy vids as 360 or any other way you want.

https://github.com/user-attachments/assets/e2fdc7c6-eda6-4adc-b50a-cf607fa86445

I won't merge this just yet as I want to look at it more in a few hours but may I ask why do you have ?dummyParam=1? Also if you can change it to be v1.1.0 that would be lovely.

Ahh okay also I just noticed I didn't push my latest code to main for v1.0.0 before releasing, that's fine I'll sort it later.

may I ask why do you have ?dummyParam=1?

it's basically to not show the "Video Proxy" button twice. because you go into the same root menu, but you have to detect if it's the 2nd time or first time. you can see the code that checks for it in line 64

Opening this back up as on a rare occasion I get errors like this
Unhandled Exception when updating world: Blue Galaxy - Grid. State Running, Refresh Stage: Connectors, Init State: Finished, SyncTick 12699, StateVersion: 8442212 System.InvalidOperationException: CurrentlyUpdating mismatch., could it be the fact stuff is happening too quickly? like do we need a frame or two delay?

So it seems I just needed a __instance.RunInUpdates(3, () => {});


Support multiple records via GET request

Present ostiapi.get() call does not support general GET parameter queries, merely a single OSTI ID.  Needs to more fully support the OSTI API documented 241.6 API query parameters.

Posted up a change to address this, documented in the README.  Alters the present get method, deprecating the ID-only call, which can be implemented as follows:
change ostiapi.get(id-value, username, password)
to
ostiapi.get({'osti_id':id-value}, username, password)
Will obtain a single record.  The parameter values in the parameters dictionary are included in the README documentation.

Closing this one, documented for posterity.


Infrastructure Module - Team Member Spec - Abhishek Saran[111701001]

The team member specs on Test Harness

Closed in #56


Not working for me

final ch = Chalk();
print(ch.red('red text', ftFace: ChalkFtFace.italic));

I am creating a flutter app and I tried to implement this but it is not working

Hey @VickyBOSS ,  sorry for the inconvenience caused.
which IDE are you using?

Vs code
On Sun, Aug 15, 2021, 1:18 PM Shan Shaji @.***> wrote:

Hey @VickyBOSS https://github.com/VickyBOSS , sorry for the
inconvenience caused.
which IDE are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shan-shaji/chalkdart/issues/17#issuecomment-899010732,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHQJGIUSQ7B3CVJ2O6IKO7LT45WK7ANCNFSM5CCSV4PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

ok, will fix the issues soon

🙏
On Tue, Aug 17, 2021, 6:25 PM Shan Shaji @.***> wrote:

ok, will fix the issues soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/shan-shaji/chalkdart/issues/17#issuecomment-900273516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHQJGIUN5O5LTM6JO6O5MITT5JL5TANCNFSM5CCSV4PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


 and show a comparison chart decoding malicious CBOR messages smaller than 32 bytes.

Codecov Report

Merging #66 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   97.42%   97.42%
=======================================
  Files           6        6
  Lines        1361     1361
=======================================
  Hits         1326     1326
  Misses         23       23
  Partials       12       12

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 998a7cd...6f6c67d. Read the comment docs.


Make Search box font size bigger

Some feedback about that the font size is too small for the search box
Acceptance criteria

Improve readability

@BirteThornquist  You may test in development https://web-global-development-equinor-web-sites-dev.c2.radix.equinor.com/search?query=hel
I did what you suggested, but I didn't change any of the spacings. What do you think?

I think this is good @wenche

Thanks @wenche @BirteThornquist - looks good :smile:


Getting value using regex

What is the issue you are having?
I want to get value using regex expression as a Key
What is BigCache doing that it shouldn't?
It accepts only strings as a key
Minimal, Complete, and Verifiable Example
cache, _ := bigcache.NewBigCache(bigcache.DefaultConfig(time.Minue))
cache.Set("someString.ThatCanBe.searchable_byREGEX", encodedStruct)
cache.Get("hereIwantToPutRegex")

Is there any possibility to get value from cache using regex?

I cannot think about a way to effectively use regex to get entries other than scan whole heap :)
Maybe using a proper indexer (like http://blevesearch.com/) will help.


[Question] Standardization code

Interesting idea & implementation. But not complete if there is no standardization of our code. It's for the quality of our code. Therefore we need to add Javascript Standard rules with eslint-config-standard or standard JS (https://standardjs.com/)

Great idea, thanks. I've added eslint to the Code Climate config.


Add Visu3d to the tools

Visu3d is a library to help build Nerf-like ML models (in TF, Jax and Numpy).
Among things, it includes differentiable camera/rays primitives, Dataclass manipulable as numpy arrays,...
The goal is to allow downstream ML research codebase to avoid re-implementing the same set of primitives and facilitate interoperability across codebases.

Up ? Is there concerns / questions about adding this ?

Thanks for the contribution! One quick question, is there a NeRF example built on top of Visu3d?


Add the ability to enable the 'none' algorithm. Set ability to false by default

Add config option to control the ability to use the 'none' algorithm. Set to false by default.

I recently added similar behavior to JOSE.JWS (see also the Unsecured Signing Vulnerability section in the readme).  It's currently controlled at the global application level instead of a per-token basis, however.

Cool! I didn't think about having it at a per token basis. I just assumed it would be global. Now I'm wondering how it would work per token?

It's definitely the easiest signature format to implement on the signing side of things (if you wanted to bypass the global setting):
jwt = %{ "test" => true }
jws = %{ "alg" => "none", "typ" => "JWT" }

signed_none = [
  jws |> JOSE.JWS.to_binary |> elem(1) |> :base64url.encode,
  jwt |> JOSE.JWT.to_binary |> elem(1) |> :base64url.encode,
  ""
] |> Enum.join(".")

Verifying is also fairly easy:
{verified, jwt, jws} = case JOSE.JWS.expand(signed_none) |> elem(1) do
  %{ "payload" => payload, "protected" => protected, "signature" => "" } ->
    case protected |> :base64url.decode |> JOSE.decode do
      jws = %{ "alg" => "none" } ->
        {true, payload |> :base64url.decode |> JOSE.decode, jws}
      jws ->
        {false, payload, jws}
    end
  _ ->
    {false, nil, nil}
end

Actually I believe we are just fine allowing it only as a global state. In fact I appreciate it being turned off by default.

Also, per token works now on the signing/verifying side with jose if JOSE.unsecured_signing/0 is enabled:
JOSE.unsecured_signing(true)

signed_none = JOSE.JWT.sign(
  %{ "k" => "", "kty" => "oct" }, # jwk
  %{ "alg" => "none" },           # jws
  %{ "test" => true }             # jwt
) |> JOSE.JWS.compact |> elem(1)

true = JOSE.JWT.verify_strict(
  %{ "k" => "", "kty" => "oct" }, # jwk
  ["none"],
  signed_none
) |> elem(0)

Unrelated side note, I'm not sure how I survived this long without realizing that Kernel.elem/2 existed for Elixir.  I kept using :erlang.element/2 throughout my Elixir code which was super clunky.

#112 should resolve this


Triton for windows support is needed as pytorch2 using triton in TorchInductor and TorchDynamo

TorchInductor and TorchDynamo use Triton, and they can't run successfully with device=cuda

@anmyachev You've been using triton on windows IIRC. What's your progress on it?

Hi @Jokeren!
Yes, indeed, we (@gshimansky did most of it) are working on enabling Windows support for Triton: https://github.com/intel/intel-xpu-backend-for-triton/pull/2478. The current status is that we can build the code and successfully pass most of the unit tests on Intel GPU. For Nvidia and AMD, there are still several unimplemented functions for the Windows platform (and a few patches that we can send quite soon, for example, for <dlfcn.h>). @gshimansky please comment here if i missed something.

@anmyachev you've written most of the details. The current status or PR https://github.com/intel/intel-xpu-backend-for-triton/pull/2478 is that all core tests pass on Lunar Lake GPU when used with Meteor Lake GPU skiplist.
For nvidia it is necessary to implement functions third_party/nvidia/include/cublas_instance.h. The code should build on Windows but I had no opportunity to run it on Nvidia GPU so I cannot say whether it works or not.


Support OAuth

https://github.com/gseitz/oauth-provider

This would be cool!
Would it be possible to do in the context of #123?

So we've chosen not to implement OAuth internally to postgrest. We are going to use JWT though so another service can handle the OAuth and can issue a json web token telling postgrest that a user is authenticated. Specifically I've been experimenting with auth0.io for handling other forms of auth and integrating with postgrest.
Closing the issue, but open to questions. :)


Vault secures credentials and separates them from config

Firstly, thanks to everyone who's worked on this lovely bit of software! I've enjoyed setting it up and tinkering with it.
I've been using vault to segregate credentials and config with the idea that I'm comfortable keeping my config in a private repo but rather not leave any credentials in there.  So, I figured I'd share this in case anyone else found this helpful.  Please let me know what you think.

Thanks for this, I think it's a good idea :+1: However, I think the implementation needs some love to tidy it up a bit and make it work for everyone. Right now the changes are a bit too ingrained into the overall structure and setup of the playbook, IMO.

The way the vault variables have been implemented will be difficult to work with once the vault is encrypted, such that you'll never be able to find them with a text editor. It's also not clear from the unencrypted files if the variables are in the vault or not. The generally accepted standard is to do something like this:

# random_application.yml
application_password: {{ vault_application_password }}

# vault.yml
vault_application_password: blahblah

This way, you can instantly see from the unencrypted code that the actual variable will be stored in the encrypted vault.

I'm not clear why you've picked the variables you have to go into the vault, there are some random ones in there that don't seem vault-worthy?

The changes you've made to inventory have made using the vault non-optional.

The instructions included in README.md are way too verbose for the readme, IMO. Ideally I'd like to see a separate doc page for this stuff (as there isn't one you've included right now), and perhaps a note in the Quick Start section about how to encrypt variables, should you choose to.

I'm currently working on some changes that will restructure the default variables and inventory somewhat (#89) - I'd like to work this into it but in such a way that a user can choose what they do/don't encrypt - not what we decide to for them. I'll get it pushed onto a branch soon and it'll start to make a bit more sense!
Thanks again for your work on this!

@davestephens thanks for the kind words and constructive feedback!  A few follow up notes and questions:

I'm curious which vars would be considered to be unnecessarily vaulted.
I meant to mention this in the PR rather than just the README, but the inventory change allows the user to deploy (and automate deploys) without entering the server's sudo password.  But yes, I see how tracking inventory might be a bridge too far.  N.b. one minor bug to be addressed if the ansible_host is to be pulled from vault: on deploy, the console currently displays {ansible_host}rather than the actual value.

TBH, I was suspicious of the vault/non-vault split as I implemented it here, but it seemed to be an expedient way to communicate the concept and hand off a workable implementation.
A quick read through the config layout suggested on #89 makes it sound like it should be feasible to allow users to set vaulted vars without much change to the repo beyond documentation, .gitignore changes and including vault in the requirements.
Feel free to close this PR for now and I'll follow along with #89 to see how we can incorporate vault with that approach.  Thanks again!

I'm going to close this - with the structure of the project, all sensitive info is now contained within inventories/your_inventory/nas.yml and users can choose to vault/encrypt that should they want to. Doing so automatically is out of scope of what I'd like to achieve here.
Thanks for the contribution! 👍

I'm going to close this - with the structure of the project, all sensitive info is now contained within inventories/your_inventory/nas.yml and users can choose to vault/encrypt that should they want to. Doing so automatically is out of scope of what I'd like to achieve here.
Thanks for the contribution! 👍


Property tokens article

I have added OGP images. Can you please check and merge it? @Akira-Taniguchi @CryptoDogg


Parent issue for hub & user contributions

What is this issue?
It's an issue to help coordinate/reference the work that joins hamilton, hamilton/contrib, and hub.dagworks.io for an initial release.

Hub website structure
General

[x] Algolia search integration
[x] Documentation links in hamilton.dagworks.io
[ ] Take a pass at messaging & images
[ ] Surface all commits for a dataflow

Tags

[ ] Create canonical list of initial top level tags
[ ] Surface tags better to help people find things
[ ] Write a github action that checks for tag conformance

Dataflow page

[ ] Add JSON structure for encoding example inputs with a submitted dataflow
[ ] Expose example inputs on dataflow page for cut & paste
[ ] Expose example configuration better

sf-hamilton-contrib package

[ ] Come up with unit testing story for submitted dataflows. We want to showcase testing & ensuring things work.
[ ] Build github actions workflow to publish it (can be manually triggered)
[ ] Add documentation to help people contribute more easily
[ ] Add documentation to hamilton.dagworks.io about sf-hamilton-contrib

Dataflows wanted

[ ] LLM related flows
[ ] ML related flows
[ ] Common data preprocessing flows
[ ] Web request processing flows
[ ] Repackaging some of the Hamilton examples?

sf-hamilton package

[ ] Add documentation on functions we want to expose in dataflows module.
[ ] Add documentation to hamilton.dagworks.io
[ ] Add CLI command / someway to bootstrap a new sf-hamilton-contrib contribution

General Testing

[ ] Test end to end flow of:

User creating a PR
Merging it
Docs being updated
User downloads it

[ ] Test end to end flow of:

User comes to hub
User searches
User gets what they need to get started.

[ ] Test the page on some OS users:

Get their impressions and feedback.

We did most of these. Closing for now.


Add forward declaration for PSAppStoreHeader in BWHockeyViewController

This is to work around a problem with header imports when importing HockeyKit via CocoaPods.

Thanks!


style(FN-1798): standardise prettier config and add to pre-commit commit hook

Note: this PR does not yet include running of prettier across the whole codebase, instead only containing the changes required to update the config (as detailed below) and resolve any linting issues that occur as a result of these changes. This makes this PR much easier to review as these are the important changes that represent actual non-formatting updates. Once approved I will separately add prettier to lint staged and run across the whole project (an extra ~1600 files). This shouldn't need a thorough review as it's only formatting changes. Once all tests pass it should then be good to merge.
To add prettier to the commit hook we should update lint-staged in the the root package.json from
"**/*.{js,ts}": [
  "eslint --fix"
],

to
"**/*.{js,ts}": [
  "eslint --fix",
  "prettier --write"
],

Then additionally run
prettier:fix:all`

to reformat all files in the project.
Introduction :pencil2:
Prettier is an opinionated code formatter that can help keep a consistent code formatting style in the project. There are currently the following issues with how it is used:

The prettier package and config have been added in the root project, but some individual sub-projects (external-api and portal-api) have their own version of prettier and config added. This may lead to inconsistencies if the versions and config diverge
Sometimes running prettier on files results in conflicts with the existing eslint config, so prettier changes need to be reverted to keep the linter happy
Running of prettier is not enforced in any way (either with commit hooks or CI)

Resolution :heavy_check_mark:

Remove the prettier package and config from sub-projects so that formatting across the entire codebase will be based on the root directory prettier installation and config. Updated the README to clarify this point.
Add eslint-plugin-prettier package to the elint config in all projects - this resolves any conflicts between eslint and prettier so we can ensure that prettier can always be run, and shows prettier formatting issues as elint errors
Add running of prettier to the commit hook so that this is always run on staged files even if the dev hasn't run the formatter manually
Move the running of lint-staged to a "pre-commit" hook instead of the "commit-msg" hook.  As lint-staged fixes are currently included in the "commit-msg" hook, this means you you would add a commit, then need to add a further separate commit with any eslint or prettier changes, which may lead to these changes not seen and committed. (Further details of the different types of commit hooks here)
Add a prettier check to the CI pipeline to check that all files are formatted according to our configured prettier formatting style

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

Comparison is base (6845487) 27.22% compared to head (cd15a5d) 12.09%.

Files
Patch %
Lines

azure-functions/acbs-function/acbs/index.js
0.00%
17 Missing and 1 partial :warning:

...ure-functions/acbs-function/acbs-facility/index.js
0.00%
6 Missing and 1 partial :warning:

...unctions/acbs-function/acbs-facility-bond/index.js
0.00%
5 Missing and 1 partial :warning:

...functions/acbs-function/mappings/party/exporter.js
0.00%
4 Missing and 2 partials :warning:

...nctions/acbs-function/acbs-issue-facility/index.js
0.00%
5 Missing :warning:

...unction/activity-create-facility-covenant/index.js
0.00%
5 Missing :warning:

azure-functions/acbs-function/helpers/date.js
0.00%
2 Missing and 2 partials :warning:

...functions/acbs-function/helpers/mandatoryFields.js
0.00%
3 Missing and 1 partial :warning:

...ngs/deal/helpers/get-deal-guarantee-expiry-date.js
0.00%
3 Missing and 1 partial :warning:

...ngs/facility/helpers/get-loan-maximum-liability.js
0.00%
4 Missing :warning:

... and 25 more

Additional details and impacted files
@@                       Coverage Diff                        @@
##           feat/FN-1855/common-package    #2699       +/-   ##
================================================================
- Coverage                        27.22%   12.09%   -15.14%
================================================================
  Files                             1293      465      -828
  Lines                            29879     9243    -20636
  Branches                          4863     1892     -2971
================================================================
- Hits                              8135     1118     -7017
+ Misses                           18919     6783    -12136
+ Partials                          2825     1342     -1483

Flag
Coverage Δ

api-test
0.48% <0.00%> (-0.01%)
:arrow_down:

ui-test
?

unit-test
14.26% <ø> (+3.98%)
:arrow_up:

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

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

PR to be replaced by https://github.com/UK-Export-Finance/dtfs2/pull/3030