Does JsonPath supports escaping?

I Know that JsonPath syntax is not very strictly defined :sweat:, but is it possible to select weird field names (containing ' and \)?
My preliminary tests suggest it doesn't work. Am I doing something wrong?
func TestEscaping(t *testing.T) {
	data := map[string]any{
		`\`: "ok",
	}
	expr, err := jp.ParseString(`$['\\']`)
	assert.NoError(t, err)
	assert.Equal(t, []any{"ok"}, expr.Get(data))
}

Thanks!

Sorry for the slow reply. The block syntax is what you want to use as you figured out. I expect that to work so let me look into it and get it fixed.
https://datatracker.ietf.org/doc/draft-ietf-jsonpath-base is the most comprehensive spec so far for JSONPath.

I can confirm it is a bug. I'll try to get it fixed this weekend.

Sorry for the slow reply

Not slow at all. Thank you very much! :pray:

Please try the jsonpath-escape branch. I have to add more tests to get the coverage back up but I believe the issue is fixed.

Tests added and ready to merge.

Awesome, thank you! 🙏
I looked briefly at your branch and it looks great
The only thing I'm not sure about is if you would need to check out-of-bound when handling escaped chars in readEscStr(). E.g.,  \, \x, \x1, \u123

Awesome, thank you! 🙏
I looked briefly at your branch and it looks great
The only thing I'm not sure about is if you would need to check out-of-bound when handling escaped chars in readEscStr(). E.g.,  \, \x, \x1, \u123

I added a few tests to make sure that was covered. It should be in the release I make tonight.

Released v1.18.2

Hey, @ohler55 :
A bit of a complimentary question, is there a function to build JsonPaths?
In particular the escaping bit is never trivial :)
What I have right now is this:
// Valid identifiers in `.fieldName` syntax are not clearly defined anywhere,
// so we use a generic/conservative pattern and fallback to ['fieldName'] otherwise
var validNameRegex = regexp.MustCompile("^[a-zA-Z_][a-zA-Z_0-9]*$")

func NewJsonPathString(path ...any) string {
	ret := "$"
	for _, v := range path {
		switch vv := (v).(type) {
		case int, int8, int16, int32, int64:
			// Array index, use `[index]` syntax
			ret = fmt.Sprintf("%s[%d]", ret, vv)
		case string:
			if validNameRegex.Match([]byte(vv)) {
				// Simple field name, `.fieldName` syntax
				ret = fmt.Sprintf("%s.%s", ret, v)
			} else {
				// Complex field name, use `['fieldName']` syntax
				// FIXME, this is terrible
				vv = strings.ReplaceAll(vv, `\`, `\\`)
				vv = strings.ReplaceAll(vv, `'`, `\'`)
				ret = fmt.Sprintf("%s['%s']", ret, vv)
			}
		default:
			panic(fmt.Errorf("Invalid JSON Path token: %v", v))
		}
	}
	return ret
}

Example:
NewJsonPathString("foo_bar56", 19, "😀", 1, "5")-> "$.foo_bar56[19]['😀'][1]['5']"

This is a completely different issue. Well not really and issue more like a question. Yes there are functions to build JSONPaths. Looks at the Expr functions here: https://pkg.go.dev/github.com/ohler55/ojg/jp
I'll close this issue later today. If you still have questions please start a discussion instead.

Thanks for the pointers!
And sorry for reusing it this issue, I'll open a new one to continue


X sp search authoratative pages

#32 Work
Resource to configure Search Authoritative / Demoted Pages. This resource enables you to configure each page as a distinct resource. The implementation suggested in the Issue above was to configure all the rules using a single resource. This seemed like an easier way to implement, then you could use a for loop in your configuration to generate the configuration for multiple values.
First thing it looks like I will need todo is to fix my spelling..

[x] Change details added to Unreleased section of changelog.md?
[x] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[x] Examples updated for both the single server and small farm templates in the examples folder?
[ ] New/changed code adheres to Style Guidelines?
[x] Unit and Integration tests created/updated where possible?

This change is

@mrpullen,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #530 into dev will decrease coverage by <1%.
The diff coverage is 39%.

@@         Coverage Diff         @@
##            dev   #530   +/-   ##
===================================
- Coverage    88%    87%   -1%
===================================
  Files        80     81    +1
  Lines      7589   7668   +79
  Branches      3      3
===================================
+ Hits       6716   6747   +31
- Misses      870    918   +48
  Partials      3      3

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 a6aea22...2905609. Read the comment docs.

Can someone familiar with the Pester Tests take a look? I have to create some [Microsoft.Office] objects to run the powershell, but I don't know how to mock the New-Object with different parameters / and / or Mock an Enum value..
Tx.

Any ideas on this failure? The issue is that I have an object type in two test files, and once it is loaded once it can't be loaded again.. Should these objects be loaded once outside of a specific test file??

I'm not sure what's triggering this at first glance. Realistically we load every type twice anyway as every single test file gets run twice, once for SP2013 and once for SP2016, so I wouldn't think loading it twice should make a difference. Just for the sake of consistency though, instead of putting it in a separate cs file, could you do it inline like all of our other resources? Here's an example
    Add-Type -TypeDefinition @"
        namespace Microsoft.Office.Server.Search.Administration
        {
            public class SearchObjectLevel {
                public static string Ssa { get { return ""; } }
            }
        }
"@

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

Comments from Reviewable

Added what I hope fill check and stop it from adding a type if a type already exists..

The code coverage stats don't look like they are being picked up here for some reason, but if you can make the other changes here I'll have a look at that and see what we can do.

Reviewed 9 of 13 files at r2, 1 of 4 files at r3, 2 of 2 files at r4.
Review status: all files reviewed at latest revision, 10 unresolved discussions, some commit checks failed.

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.psm1, line 54 at r4 (raw file):

New-Object Microsoft.Office.Server.Search.Administration.SearchObjectOwner($searchObjectLevel)

This shouldn't use positional parameters, use -TypeName and -ArgumentsList

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.psm1, line 150 at r4 (raw file):

            $serviceApp = Get-SPEnterpriseSearchServiceApplication -Identity $params.ServiceAppName
            $searchObjectLevel = [Microsoft.Office.Server.Search.Administration.SearchObjectLevel]::Ssa
            $searchOwner = New-Object Microsoft.Office.Server.Search.Administration.SearchObjectOwner($searchObjectLevel)

Same remark about positional parameters

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.psm1, line 158 at r4 (raw file):

            if($params.Action -eq "Authoratative")
            {
                 New-SPEnterpriseSearchQueryAuthority -Url $params.Path -SearchApplication $serviceApp -Owner $searchOwner -Level $params.Level

Can we wrap this line so we don't go over the 100 character length recommendation in the style guidelines

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.psm1, line 184 at r4 (raw file):

            if($params.Action -eq "Authoratative")
            {
                Set-SPEnterpriseSearchQueryAuthority -Identity $params.ServiceAppName -SearchApplication $ssa -Owner $searchOwner -Level $params.Level

Can we wrap this line so we don't go over the 100 character length recommendation in the style guidelines

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.schema.mof, line 5 at r4 (raw file):

{
    [Key, Description("Search Service Application Name")] String ServiceAppName;
    [Key, Description("Source URI for the crawl mapping")] String Path;

Still says crawl mapping here, shouldn't this be something else?

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.schema.mof, line 6 at r4 (raw file):

Real32

Shouldn't this be "UInt32"

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.schema.mof, line 7 at r4 (raw file):

resourec

Typo here

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.schema.mof, line 8 at r4 (raw file):

    [Write, Description("Level of Authoratative Page, values between 0.0 and 2.0")] Real32 Level;
    [Write, Description("The resourec will either make the page authoratative or demoted based on this value"), ValueMap{"Authoratative","Demoted"}, Values{"Authoratative","Demoted"}] String Action;
    [Write, Description("Ensure the crawl rule is Present or Absent"), ValueMap{"Present","Absent"}, Values{"Present","Absent"}] String Ensure;

Crawl rule again here

Modules/SharePointDsc/DSCResources/MSFT_SPSearchAuthoritativePage/MSFT_SPSearchAuthoritativePage.schema.mof, line 9 at r4 (raw file):

thsi

Typo - @ykuijs can you look at the Access Services 2010 resource I just let through, it has this same typo. If you could pick it up in your bug fix PR that would be awesome :)

Modules/SharePointDsc/Examples/Resources/SPSearchAuthoritativePage/2 - DemotedPage.ps1, line 3 at r4 ([raw file](https://github.com/powershell/sharepointdsc/blob/da723187d91b12feecca0f4fab4f65d21eb69769/Modules/SharePointDsc/Examples/Resources/SPSearchAuthoritativePage/2 - DemotedPage.ps1#L3)):

<#
.EXAMPLE
    This example shows how to create a Search Authoritative Page

Should this say something about it being demoted instead of authoratative

Comments from Reviewable

OK so I can't see where that weird character is, but its preventing the tests from running which is breaking the code coverage result as well. Leave it with me, I'll try to do a pull from your branch and see if I can find it and let you know where it is.

Reviewed 4 of 13 files at r2, 5 of 5 files at r5.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

ok, swap out the two "Add-Type" statements at the top for this, that seems to fix it - the tests are broken after that point, but they run:
Add-Type -TypeDefinition @"
namespace Microsoft.Office.Server.Search.Administration {
    public class SearchObjectLevel {
        public static string Ssa { get { return ""; } }
    }

    public class SearchObjectOwner {
        public SearchObjectOwner(Microsoft.Office.Server.Search.Administration.SearchObjectLevel level) { }
    }
}
"@

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

So I updated the code to
Add-Type -TypeDefinition @"
namespace Microsoft.Office.Server.Search.Administration {
   public enum SearchObjectLevel
   {
       SPWeb,
       SPSite,
       SPSiteSubscription,
       Ssa
   }

   public class SearchObjectOwner {
       public SearchObjectOwner(Microsoft.Office.Server.Search.Administration.SearchObjectLevel level) { }
   }
}
"@

the code expects that SearchObjectLevel to come back as an enum.

Reviewed 1 of 13 files at r2, 2 of 2 files at r7.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

 just need a merge from dev to resolve the conflict on the change log and we are good to go.

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

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

Comments from Reviewable


The rack.monitor not loaded

The rack.monitor isn't loaded when Lambda calls the event Hanami.boot is called.

The reason is rack gem is not loaded correctly.
https://github.com/hanami/hanami/blob/73adf779ed5a75ecfe576cd8f8c1d034791f5102/lib/hanami/app.rb#L160-L163
        if Hanami.bundled?("rack")
          require_relative "providers/rack"
          register_provider(:rack, source: Hanami::Providers::Rack, namespace: true)
        end

Fixed by 16fbce66514c32b8cb23201a2651d9fa8d56bedf


KnotxServer response configuration - wildcards, case-insensitive filtering allowed headers

Description

For a reason we can use:
AllowedHeadersFilter.WildcardCaseInsensitive.create(strings);

I done refactor usage of AllowedHeadersFilter
I also found one thing that worries me. In EndpointOptions we can also set:
private JsonObject additionalHeaders;
private List<Pattern> allowedRequestHeadersPatterns;

by provided setter. But if we use only setAllowedRequestHeaderPatterns, we don't fixing value of additionalHeaders. But in the other way, if we use setAllowedRequestHeaders we also set correct value for allowedRequestHeadersPatterns. In my opinion we should make setAllowedRequestHeaderPatterns private, or even skip this concept. @Skejven @tomaszmichalak  what is your opinion about that ?
Motivation and Context

knotx-commons
#7 and PR-8
knotx-server-http
#41
Screenshots (if appropriate)
Upgrade notes (if appropriate)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I hereby agree to the terms of the Knot.x Contributor License Agreement.

Hi @dragarth-7
Thanks for bringing this problem. Actually, there are several tickets/topic that bring allowed headers filtering topic:

https://github.com/Knotx/knotx-server-http/issues/41
https://github.com/Knotx/knotx/issues/414
https://github.com/Knotx/knotx/issues/401

There is few things to remember about and why restricting headers is present:

We don't want to send all original request (the one that comes from the user to server) be exposed to all external endpoints Knot.x will interact with (e.g. security reasons).
The same goes the opposite direction, we don't want to expose all headers from endpoints to the end user with the final response headers.
Somehow managing headers should be done at each level, that means every time there is interaction with:

external endpoint (http-repository, http-action)
final response to end-user (response-writer)
The current solution remembers Knot.x 1.0 times, and it might be a good time to start discussion how we can refactor the code to make managing headers easier in multiple places.

FYI Knotx/knotx is repository for discussing general (cross-repo) problems and questions.

Please resolve conflicts.

Done. Conflicts are resolved.  Please remember that you need firstly changes from https://github.com/Knotx/knotx-commons/pull/14


Gauge LSP: Execute a scenario

Proposed behavior
Execute a scenario in the specification
Out of scope

Debugging
Parallel

Steps

Provide a Code lens link to a specific scenario in the specification.

Test Cases

[ ] Execute the selected scenario in the specification, show the results in the output window.

Should be available in gauge nightly >= 05-10-2017 and gauge-vscode nightly >= 6-10-2017


Wheel Naming Discrepancy Prevents Installing

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: Linux
Poetry version: Poetry 1.0.0a1
Link of a Gist with the contents of your pyproject.toml file:  N/A

Issue

Building wheel packages with Poetry produces different version naming/wheel packages than bdist_wheel which is preventing other packages from finding it in our private index.
For example, our library package has a version set to:
name = "arroyo-settings"
version = "0+untagged.35.g68290c5"

When building it with Poetry it produces a wheel:
Building arroyo-settings (0+untagged.35.g68290c5)
 - Building wheel
 - Built arroyo_settings-0_untagged.35.g68290c5-py3-none-any.whl

When building with setup.py bdist_wheel:
arroyo_settings-0+untagged.35.g68290c5-py3-none-any.whl

Using the Poetry built wheel, trying to poetry add arroyo-settings in our second project, I get the following: https://gist.github.com/seglberg/e5521c49359ec276ac2a81c0b010af3e

Ah the version in the metadata wasn't transformed 0+untagged.35.g68290c5 (with the +) - which is probably causing the mismatch.

I see this issue as well when using poetry and devpi server - except the devpi server rejects the upload:
poetry publish --repository kristofferb_dev

Publishing my-package (0.1.3-dev0+some_branch) to kristofferb_dev
 - Uploading my-package-0.1.3.dev0+some.branch.tar.gz 100%

[HTTPError]
400 Client Error: filename 'my-package-0.1.3.dev0_some.branch-py3-none-any.whl' does not contain version '0.1.3.dev0+some.branch' for url: https://<our devpi package index>/kristofferb/dev/

The issue seems to be the regex here
https://github.com/sdispater/poetry/blob/6fe7f54a83c59131d4e291084c3496a0c513cfaa/poetry/masonry/builders/wheel.py#L201
that replaces + with _ in the wheel filename.


"/init cmd /C set" contains too few environment variables, some softwares run failed

A brief description

/init cmd /C set contains too few environment variables, some softwares run failed, like shutdown.
$ /init shutdown
The system could not find the environment option that was entered.(203)

$ /init cmd /C 'set COMPUTERNAME=MY-PC & shutdown'
Usage: shutdown ...

And without TMP variable, some softwares run failed.

Your Windows build number

14951.1000

Steps / All commands required to reproduce the error from a brand new installation

/init shutdown

Thanks for reporting this. The processes created out of WSL should have the same environment as the bash.exe window you are running.  This is working for me internally. How are you launching bash? Could you share the output of this command with me?
/init cmd.exe /c set
My output shows TMP, COMPUTERNAME, etc.

It‘s my carelessness. I ran it in wslbridge in mintty. I think the problem was caused by mintty. Thanks.

The problem is certainly not cause by mintty or wsltty. If shutdown depends on the setting of some non-standard environment variable, it's an incompatibility of shutdown (provided the Linux version of shutdown is meant). Setting TMP is not strictly required in a Linux environment, but should normally be set somewhere in the /etc profile scripts. This is a WSL configuration issue.

@mintty The shutdown is shutdown.exe instead of /usr/bin/shutdown in WSL, it requires Windows environment variables,.

it requires Windows environment variables

Nice, so if WSL assumes a set of proprietary settings, they should properly document that somewhere (first issue from my point of view), then we could ask third-party invocation tools (wslbridge) to replicate that behaviour (second issue).
 I don't know what environment variables the software requires, WSL doesn't know neither.

Oh, understood. In that case I suggest to close this issue again :/
and reopen the other one.


Update kustomize build github workflow

Update Kustomize version
Update checkout action to v4
Use cnf-features-deploy release-4.13 branch instead of main

/cherry-pick cars2


Feature/gps plugin jade

I adapted the gps plugin to the new message format in jade.
Therefor I have reverted to earlier commits, where the plugin file was removed.
NOTE: The orientation is not longer a part of the GPS msg (NavSatFix.msg) therefor I had to constantly set it to 0.

I'm not sure if we should take this approach or split NavSatFix out into its own plugin (with both plugins available in indigo and GpsFix removed in jade).

This looks good, but please remove the "arrows" draw style, as the arrows have no meaning without heading. Once that's done, we'll test this and merge it in.
Thanks!

@evenator Hey, I removed the arrows and renamed the plugin. See if you like it.

@malban


[WIP] add a default ipPool for calicoctl image

don't merge
this is for kubernetes self hosted stuff

@gunjan5 let's chat about what this is for :)


Unable to reference the Arn of a Kinesis stream for AWS::DynamoDB::GlobalTable for a AWS::Serverless::Function

cfn-lint version: 0.54.2
Description of issue.
The linter fails when using the !GetAtt function to reference the Arn of a stream from a DynamoDB Global Table. This looks to be because cfn-lint is looking specifically for AWS::DynamoDB::Table and does not accept AWS::DynamoDB::GlobalTable.
Kinesis streams from Global DynamoDB Tables have additional functionality to maintain replicas across all regions, but the records on the stream itself can also function as a regular DynamoDB kinesis stream to trigger Lambda. (some information here: https://aws.amazon.com/blogs/database/how-to-use-amazon-dynamodb-global-tables-to-power-multiregion-architectures/#:~:text=How to create a global table)
See below for a pseudo cloudformation example of a syntax which should be valid:
  MyGlobalTable:
    Type: AWS::DynamoDB::GlobalTable
    Properties:
      AttributeDefinitions:
      - AttributeName: PK
        AttributeType: S
      KeySchema:
      - AttributeName: PK
        KeyType: HASH
      Replicas:
      - Region: eu-west-1
      StreamSpecification:
        StreamViewType: NEW_AND_OLD_IMAGES

  MyFunction:
    Type: AWS::Serverless::Function
    Properties:
      CodeUri: someCodeUri
      Handler: someHandler
      Events:
        Stream:
          Type: DynamoDB
          Properties:
            Stream: !GetAtt MyGlobalTable.StreamArn # <-- this throws an error in cfn-lint
            BatchSize: 1
            StartingPosition: TRIM_HORIZON

Output:
E3008 Property "EventSourceArn" can Fn::GetAtt to a resource of types [AWS::DynamoDB::Table, AWS::Kinesis::Stream, AWS::Kinesis::StreamConsumer, AWS::SQS::Queue] at Resources/MyFunctionStream/Properties/EventSourceArn/Fn::GetAtt

Expected Output:
No errors should output.

https://github.com/aws-cloudformation/cfn-lint/blob/75510c7f091deefa609fd54ba4ba88ad608ff417/src/cfnlint/data/ExtendedSpecs/all/03_value_types/aws_lambda.json#L28-L38


RG405m power saving mode not smooth for playing GBA - RetroArch 64 bit - VBA-M core

The game is not very smooth under power-saving mode. Switching to normal mode makes it run smoothly. Wanna ask if this is expected?
https://github.com/TheGammaSqueeze/GammaOS/assets/2152195/350c298a-ea8c-45cf-8ee8-692941cbc737
I also tried SFC, same issue.

Not sure if this is what we should expect for power saving mode or if its some of my configuration issue?

M-gba requires more power. Use gpsp for power saving mode.
For SFC, use Snes9x EX+ standalone emulator.
For PSX, can use existing pcsx rearmed core, or use Duckstation standalone emulator.

Thanks for the explanation.


[Feature request] Data streaming

Similar to Python requests: https://stackoverflow.com/questions/60343944/how-does-requests-stream-true-option-streams-data-one-block-at-a-time

Streaming would be great to have because it allows user to make decision about skipping body data or buffer fetching to print result to client, which is great in many situations:

Content is too big, like 2GB and memory limitation like 2GB RAM, then if you fetch everything into memory you'll get OOM error
In proxy environment, you may want to abort the body data if the response header doesn't match some of your criterias
I'll try to look into the code and implement this feature, shouldn't be that hard considering we're fetching data from Go TLS Client which probably allow data streaming.

Actually after researching the API of tls-client by bogdanfinn, I can't find any information about stream being made available, so it looks like to have this stream feature we first need to do something with the core first, which is the Go Client https://github.com/bogdanfinn/tls-client-api

Hey, I don't think it'll be possible to add this soon sadly. :/


cmd/prometheus: Issue a warning on 32 bit archs

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

:+1:

@brian-brazil can you +1 the 2 las minor changes I made?

used binary twice in the sentence instead of binary & build
use has not instead of hasn't

:+1:


Create a spredsheet from a list of dicts

This PR implements creating a spreadsheet from a list of dicts which enables creating a spreadsheet from a JSON like so:
json-str |parse-json |to-spreadsheet

Thank you for this!

I didn't yet have time to study this in your code. But, what are or were your general thoughts about cases when some JSON objects have some additional fields, or some have some missing?

@refaktor the current function creates a column for each key it encounters and if the object does not have a key it's set to env.Void. I think this is totally usable but we could think about a function that also accepts the required keys and a default value if a key is not found? What do you think?

This is good for first version. There is a principle in Rye that it never returns null (Void), so I'm thinking that if value is missing, the initial value for that column should be inserted. "" for string, 0 for int, etc ...
Practical use would show what other variations we would want ... maybe one where you explicitly define what columns you want yes, or a strict mode, which would fail if something is missing, but all this will be added if a need arises in the practice.


[Bug] [Dist] The main class org.apache.dolphinscheduler.server.StandaloneServer could not be found or loaded

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The dolphinscheduler project is packaged into a binary package, the Standalone-Server service is started, and the log reports an error: the main class org.apache.dolphinscheduler.server.StandaloneServer cannot be found or cannot be loaded.
What you expected to happen
The Standalone-Server service started successfully.
How to reproduce
the Standalone-Server service is started, and the log reports an error: the main class org.apache.dolphinscheduler.server.StandaloneServer cannot be found or cannot be loaded.
Anything else
No response
Are you willing to submit PR?

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

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I mean this https://github.com/apache/dolphinscheduler/blob/ba4cb9d22ad02ba4927fd23ea811c82639261e69/script/dolphinscheduler-daemon.sh#L91

How did you start the standalone server? Did you use the shell script to start?

Sorry, I found this problem in the test on 1.3.9-prelease, the dev branch does not have this problem, I will submit a PR to 1.3.9-prelease to fix this problem.


Publish latest version to npm

Thanks for this great package. Helps a lot while working locally.
Latest package version on npm is v3.13.3, where as GitHub repo has latest version as v3.16.1. I am interested in AdminAddUserToGroup feature. Can you please publish latest version to npm as well? Thanks

Ugh, thanks for spotting this! It looks like the automatic npm publish stopped working when I updated the pipeline for the multi-arch docker builds 🤦
I'll take a look, thanks.

Fixed, v3.16.2 is now available on npm and future releases will be too.


:lipstick: (smpl) [NO-ISSUE]: Improve mock settings page

📝 Description
List & drawer for mock sessions

❓ Context

JIRA or GitHub link: [NO-ISSUE]

https://github.com/user-attachments/assets/b3de4418-2050-4b0b-ba2c-34fd6fd5b24b

Feature:

✅ Checklist
Pull Requests must pass CI checks and undergo code review. Set the PR as Draft if it is not yet ready for review.

[ ] Covered by automatic tests
[ ] Changeset is provided
[ ] Impact of the changes:

list of the changes

🧐 Checklist for the PR Reviewers

[ ] The code aligns with the requirements described in the linked JIRA or GitHub issue.
[ ] The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
[ ] There are no undocumented trade-offs, technical debt, or maintainability issues.
[ ] The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
[ ] Any new dependencies have been justified and documented.

Nice PR 👍 It would be good to be able to delete or modify one line without resetting all the mock

Thanks, well there is no endpoint yet to delete a specific mock, but i can improve the ui to update a specific by selecting it instead of adding with the same prefix


【代码迁移】任务50：Coneheads: Hierarchy Aware Attention

umbral运行结果截图

penumbral运行结果截图

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zxlzd seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


docs(samples): added copy table and accompanying test

@stephaniewang526

https://source.cloud.google.com/results/invocations/0400fb43-dedb-4891-8a03-5912cc5c27cd/targets/github%2Fjava-bigquery%2Fsamples%2Finstall-without-bom%2Ftarget%2Ffailsafe-reports/tests ptal


Self hosted agent cache download is really slow, task times out

Describe your question
Ive set up a self hosted agent on my Windows computer. When I run the pipeline on the agent, it downloads the job, checks out the repo quickly. Then the next step in the pipeline is a Cache task. For some reason, it is really slow and times out.
Here is the log of the cache task:
Starting: NuGet Cache
Task         : Cache
Description  : Cache files between runs
Version      : 2.198.0
Author       : Microsoft Corporation
Help         : https://aka.ms/pipeline-caching-docs
Resolving key:

nuget                          [string]
"Windows_NT"                   [string]
C:\agent_work\1\s/**/*.csproj [file pattern; matches: 16]
...
...
Resolved to: nuget|"Windows_NT"|m9XAv3djbkqwtmVOc3XHWBH7XIHQUalMVHWtVuARTt4=
Resolving restore key:
nuget        [string]
"Windows_NT" [string]
Resolved to: nuget|"Windows_NT"|**
Using default max parallelism.
Using default max parallelism.
Max dedup parallelism: 192
DomainId: 0
ApplicationInsightsTelemetrySender will correlate events with X-TFS-Session cbb5cf5d-f7e5-4a89-bdc4-08cedef451f2
Hashtype: Dedup64K
Getting a pipeline cache artifact with one of the following fingerprints:
Fingerprint: nuget|"Windows_NT"|m9XAv3djbkqwtmVOc3XHWBH7XIHQUalMVHWtVuARTt4=
Fingerprint: nuget|"Windows_NT"|**
There is a cache hit: nuget|"Windows_NT"|m9XAv3djbkqwtmVOc3XHWBH7XIHQUalMVHWtVuARTt4=
Used scope: 1;964b935e-d82b-4899-be22-b7741e10d7e9;refs/heads/refactor/webview2;5846b47a-a0ef-4c62-9456-20b032863955
Entry found at fingerprint: nuget|"Windows_NT"|m9XAv3djbkqwtmVOc3XHWBH7XIHQUalMVHWtVuARTt4=

7-Zip 24.08 (x64) : Copyright (c) 1999-2024 Igor Pavlov : 2024-08-11
[https://sdfvsblobprodweu290.blob.core.windows.net/dba21690dfddf343afa5ee0ee5f3d35952/CE35CDF62DB861145C51C9D7C603CD109982DDC95E095468C669BA7F22FF7F7B02?sv=2019-07-07&sr=b&skoid=b3c3d5ad-124a-4025-8525-7a7a28563a43&sktid=33e01921-4d64-4f8c-a055-5bdaffd5e33d&skt=2024-11-28T08:41:59Z&ske=2024-11-30T09:41:59Z&sks=b&skv=2019-07-07&se=2024-11-28T11:54:10Z&sp=r&rscl=x-e2eid-46c9460b-c5684bf7-bec7f85d-ae328057-session-cbb5cf5d-f7e54a89-bdc408ce-def451f2] Try 1/5, Task was requested to be canceled.
[https://wcovsblobprodweu265.blob.core.windows.net/dba21690dfddf343afa5ee0ee5f3d35952/46E92E290B18F9AD9982782AC27FC6A22564395EEF9AAB4B1706ADB43EF84CB302?sv=2019-07-07&sr=b&skoid=b3c3d5ad-124a-4025-8525-7a7a28563a43&sktid=33e01921-4d64-4f8c-a055-5bdaffd5e33d&skt=2024-11-28T08:41:59Z&ske=2024-11-30T09:41:59Z&sks=b&skv=2019-07-07&se=2024-11-28T12:08:18Z&sp=r&rscl=x-e2eid-46c9460b-c5684bf7-bec7f85d-ae328057-session-cbb5cf5d-f7e54a89-bdc408ce-def451f2] Try 1/5, Task was requested to be canceled.
[https://o30vsblobprodweu248.blob.core.windows.net/dba21690dfddf343afa5ee0ee5f3d35952/8DC533CE352593F21BB9F41CB762FD0AD91021EA37A63F78AA3B33625EE6660802?sv=2019-07-07&sr=b&skoid=b3c3d5ad-124a-4025-8525-7a7a28563a43&sktid=33e01921-4d64-4f8c-a055-5bdaffd5e33d&skt=2024-11-28T08:41:59Z&ske=2024-11-30T09:41:59Z&sks=b&skv=2019-07-07&se=2024-11-28T12:16:35Z&sp=r&rscl=x-e2eid-46c9460b-c5684bf7-bec7f85d-ae328057-session-cbb5cf5d-f7e54a89-bdc408ce-def451f2] Try 1/5, Task was requested to be canceled.
[https://4ysvsblobprodweu228.blob.core.windows.net/dba21690dfddf343afa5ee0ee5f3d35952/767EA5E02042FDECF3302435DAFD3AC7EC6AC568F00A502F0C52AE191B35EE5302?sv=2019-07-07&sr=b&skoid=b3c3d5ad-124a-4025-8525-7a7a28563a43&sktid=33e01921-4d64-4f8c-a055-5bdaffd5e33d&skt=2024-11-28T08:41:59Z&ske=2024-11-30T09:41:59Z&sks=b&skv=2019-07-07&se=2024-11-28T12:13:52Z&sp=r&rscl=x-e2eid-46c9460b-c5684bf7-bec7f85d-ae328057-session-cbb5cf5d-f7e54a89-bdc408ce-def451f2] Try 1/5, Task was requested to be canceled.
[https://vxpvsblobprodweu275.blob.core.windows.net/dba21690dfddf343afa5ee0ee5f3d35952/EB6FDB2B582D38CECCA11202B562CBB66B5EABA683EB40AC27D3B1039D7AC35B02?sv=2019-07-07&sr=b&skoid=b3c3d5ad-124a-4025-8525-7a7a28563a43&sktid=33e01921-4d64-4f8c-a055-5bdaffd5e33d&skt=2024-11-28T08:41:59Z&ske=2024-11-30T09:41:59Z&sks=b&skv=2019-07-07&se=2024-11-28T11:57:33Z&sp=r&rscl=x-e2eid-46c9460b-c5684bf7-bec7f85d-ae328057-session-cbb5cf5d-f7e54a89-bdc408ce-def451f2] Try 1/5, Task was requested to be canceled.
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.0 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
Downloaded 0.9 MB out of 1,119.7 MB (0%).
As you can see after a while it downloads some.
My company is using a proxy. I included *.blob.core.windows.net in the no_proxy variable, but that does not seem to help.
Here is the agent config log:
Starting: Initialize job
Agent name: 'BP-C-011AE'
Agent machine name: 'BP-C-011AE'
Current agent version: '3.248.0'
Agent running as: 'BP-C-011AE$'
Agent is running behind proxy server: 'http://<IP_ADDRESS>:3128'/
Prepare build directory.
Set build variables.
Download all required tasks.
Checking job knob settings.
Knob: DockerActionRetries = true Source: $(VSTSAGENT_DOCKER_ACTION_RETRIES)
Knob: UseGitLongPaths = true Source: $(USE_GIT_LONG_PATHS)
Knob: NoProxy = some_other_links,.blob.core.windows.net,.dev.azure.com,127.0.0.* Source: ${no_proxy}
Knob: ProxyAddress = http://<IP_ADDRESS>:3128/ Source: ${http_proxy}
Knob: EnableIssueSourceValidation = true Source: $(ENABLE_ISSUE_SOURCE_VALIDATION)
Knob: AgentEnablePipelineArtifactLargeChunkSize = true Source: $(AGENT_ENABLE_PIPELINEARTIFACT_LARGE_CHUNK_SIZE)
Knob: ContinueAfterCancelProcessTreeKillAttempt = true Source: $(VSTSAGENT_CONTINUE_AFTER_CANCEL_PROCESSTREEKILL_ATTEMPT)
Knob: ProcessHandlerSecureArguments = false Source: $(AZP_75787_ENABLE_NEW_LOGIC)
Knob: ProcessHandlerSecureArguments = false Source: $(AZP_75787_ENABLE_NEW_LOGIC_LOG)
Knob: ProcessHandlerTelemetry = true Source: $(AZP_75787_ENABLE_COLLECT)
Knob: UseNewNodeHandlerTelemetry = True Source: $(DistributedTask.Agent.USENEWNODEHANDLERTELEMETRY)
Knob: ProcessHandlerEnableNewLogic = true Source: $(AZP_75787_ENABLE_NEW_PH_LOGIC)
Knob: EnableResourceMonitorDebugOutput = true Source: $(AZP_ENABLE_RESOURCE_MONITOR_DEBUG_OUTPUT)
Knob: EnableResourceUtilizationWarnings = true Source: $(AZP_ENABLE_RESOURCE_UTILIZATION_WARNINGS)
Knob: IgnoreVSTSTaskLib = true Source: $(AZP_AGENT_IGNORE_VSTSTASKLIB)
Knob: FailJobWhenAgentDies = true Source: $(FAIL_JOB_WHEN_AGENT_DIES)
Knob: CheckForTaskDeprecation = true Source: $(AZP_AGENT_CHECK_FOR_TASK_DEPRECATION)
Knob: CheckIfTaskNodeRunnerIsDeprecated246 = True Source: $(DistributedTask.Agent.CheckIfTaskNodeRunnerIsDeprecated246)
Knob: UseNode20ToStartContainer = True Source: $(DistributedTask.Agent.UseNode20ToStartContainer)
Knob: LogTaskNameInUserAgent = true Source: $(AZP_AGENT_LOG_TASKNAME_IN_USERAGENT)
Knob: UseFetchFilterInCheckoutTask = true Source: $(AGENT_USE_FETCH_FILTER_IN_CHECKOUT_TASK)
Knob: Rosetta2Warning = true Source: $(ROSETTA2_WARNING)
Knob: AddForceCredentialsToGitCheckout = True Source: $(DistributedTask.Agent.AddForceCredentialsToGitCheckout)
Finished checking job knob settings.
Start tracking orphan processes.
Finishing: Initialize job
What coudl be the problem?
Thank you!
Versions
vsts-agent-win-x64-3.248.0
Windows 11 Enterprise
Environment type (Please select at least one enviroment where you face this issue)

[x] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Operation system
Windows 11 Enterprise
Version controll system
No response
Azure DevOps Server Version (if applicable)
No response

Hi @SzilkoBalog! Thank you for reporting this issue and sorry for the late response.
Could you please clarify a few things:

did this setup use to work normally for you and then a regression happened or were there timeouts like this from the beginning?
does this problem occur consistently on every pipeline run?

There are no explicit timestamps in the attached logs, so it is not quite clear where exactly the delay occurs and if it is the download operation itself that is slow and leads to timeout, or the process of finding a cache entry?
Ideally, it would be useful to check the pipeline and agent logs with the system.debug variable enabled.  As the logs may contain private information, please, consider opening a ticket on https://developercommunity.visualstudio.com/AzureDevOps and uploading them there in a message marked as viewable by MS only.


Document debug blocks, and how to use them.

Document how to use debug blocks.
Debug blocks
A debug block denotes code is only present in development mode, and elided in production.
debug { printfs here }

Compilation modes
from moc --help:

--release ignore debug expressions in source
--debug respect debug expressions in source (the default)

🎉

@matthewhammer
There is a quick reference on how I have used it:
https://github.com/internet-computer/debug-block.mo
(As far as I know) there is no direct support in DFX to pass arguments to moc (e.g. --debug or --release).

I think you can have a motoko build method in dfx.json and have an attribute "args" with "--debug" below it.

Thanks @ggreif~
This makes it much simpler to set up!
dfx.json
{
    "canisters": {
        "prod": {
            "type": "motoko",
            "main":  "src/main.mo",
            "args": "--release"
        },
        "dev": {
            "type": "motoko",
            "main":  "src/main.mo"
        }
    }
}


iphone opacity & shadow problem

in iPhone, you see shadows (by box-shadow or filter: drop-shadow) through elements with opacity.
it is ok in PC.
[Suggested solution]
1.dont use opacity, and add each color arrays in SCSS.
$green: rgb(0 255 0), rgb(0 240 0), rgb(0 230 0);
...
$colors: $green, $yellow ...;
2.dont use opacity, and use rbg(0 255 0) rbg(0 255 0 / .33) ... and insert it into array.

wont fix because main users use PC and use dont use smat-phone


[FR] -

External
Performance metrics and guarantee of terms in the smart contract / on chain itself.
Due to trust issues indirect calculations might not be accepted by users in certain use cases. As an example imagine a smart contract called "vacation delegation" that you sent ADA to and then through performance metrics delegated automatically to pools. With indirect calculations you would be at the mercy of the contract but with guaranteed on chain metrics you would not.
A side issue but it would be interesting also if you could guaranteeing that a sum would indeed be returned after X amount of time due to some verification that where run on the smart contract.
Describe alternatives you've considered
Oracle solutions would probably be a preferred method as it would decrease information that needs to be on the chain and could serve as a trusted party.

(Also not sure if this be here in cardano-node or some of the other repositories is the more correct place so apologize in advance if not the correct place.)

We have put this in our backlog.


Drastic performance drop while using DXVK on Windows with GTA IV

With DXVK, the game just constantly remains at around 7-10 FPS and GPU usage is 100% with CPU being used around 50%, Without DXVK, the CPU is around 100% and GPU is around 60-70% but with higher FPS (20-35). These performance drops are only prevalent on GTA IV, other games run fine with DXVK on Windows. It should be noted that I have an older Intel CPU (E5800) and 2GB RAM.
Note
I have tested the same game with DXVK on a AMD laptop (R5 2500U/Vega 8) and that is completely fine with around 30 FPS with stable framtimes.
Software information
Grand Theft Auto IV The Complete Edition  V<IP_ADDRESS>
System information

GPU:  Nvidia GT 1030 2GB GDDR5
Driver: 471.41
OS: Windows 8.1 (Build 9600)
DXVK version: 1.9

Apitrace file

https://drive.google.com/file/d/1YMLmQ0nySRehpt6FmQrHq4DVA6j2dSg9/view?usp=sharing

Log files
GTAIV_d3d9.log

DXVK does very poorly on 2GB cards. It's possible that GTA IV managed to exceed the 2GB of VRAM.
There isn't really anything we can do here.

DXVK does very poorly on 2GB cards. It's possible that GTA IV managed to exceed the 2GB of VRAM.
There isn't really anything we can do here.

I am afraid you may be right since I saw the memory bus usage of the card and it was at 80% constant, it never goes that high even with much more intensive games like BF4 and always stays in the range of 10-20% from what I have noticed

Try DXVK master and see if that makes a difference.

I know of #1318, but OP is just playing a 2008 game in full hd.
Cards with less than 4 times as much memory should be fine with it.

I know of #1318

That is about system memory. This is about VRAM.

Sorry for the late reply, I just tested the lastest release/master build and it seems to mitigate some of the issues (around 30 FPS in very less intensive areas) which seems much better than before and I have been trying to install my old retail version from 2008 to test DXVK on that, will update after that

Hey, after countless hours of reading through different forums, threads and posts and also trial and error, I finally found that any GTA IV version which is higher than <IP_ADDRESS> performs very bad with DXVK and from what I can speculate.. it seems to be stressing the memory and the memory bus of the GPU but I don't know why it does so.. Anyways, the older versions of the game (<<IP_ADDRESS>) work fine from what I have tested at least on my Nvidia/Intel system with any DXVK version with DX9 support but I have read many posts, blogs and comments where people were experiencing similar problem to mine with a higher end GPU(GTX 1070, 1060, RTX 2060 Super and so on).
It may be a Nvidia Specific issue since I couldn't find any problems with my AMD APU with that specific game version or maybe I don't know but I think this issue should be known. Though Thanks for the help either way K0bin.

finally found that any GTA IV version which is higher than <IP_ADDRESS> performs very bad with DXVK

There was a bit of a problem with DXVK 1.9, thats fixed now. The game runs very well for me on my Nvidia GPU. As I said, most likely out of VRAM. I think the game has config options to adjust the amount of size of the texture pool the game will use. Try playing around with that.


Feat: run pytest with error

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

Flag
Coverage Δ

unittests
65.10% <100.00%> (+0.03%)
:arrow_up:

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

Files with missing lines
Coverage Δ

pyledger/test_ledger.py
100.00% <100.00%> (ø)


Default delete key handler preventing users from using backspace in other elements on the page

Describe the Bug
When the ReactFlow component is present on a page, the backspace key is intercepted and seems to be prevented from working anywhere else on the page (other than purely native text inputs completely controlled by the browser).
I don't think that document-scope key event handlers should be calling Event.preventDefault().
Your Example Website or App
https://codesandbox.io/s/sharp-tharp-jhxqm4?file=/src/App.js
Steps to Reproduce the Bug or Issue

See above example
Attempt to backspace text out of the text input (it doesn't work)
Set deleteKeyCode={null} in the Flow component
Note that using backspace in the text input now works

Expected behavior
As a user, I should be able to use the backspace key elsewhere in the window.
Screenshots or Videos
No response
Platform

OS: macOS
Browser: Chrome, Safari
Version: 103.0, 15.5

Additional context
The example I provided is using exactly the UI component library that I am using, for maximum accuracy to my use case.

Hi @EzraBrooks, thanks for your issue.
The problem here is that the library you are using is rendering a custom web component (rux-input) and not a standard input field. When you are using a normal input field or another library that just renders an input field, everything works fine. In our keyboard handler, we are checking if you are in an input field or in a textarea, and don't handle the event if that's the case.
However, you can workaround this issue with the library you are using by preventing react flow from receiving the keydown event. You can find a working example here: https://codesandbox.io/s/youthful-bartik-on37r9?file=/src/App.js

It's been a while since I last did this, so I may be misremembering, but aren't you able to add focus capability to any element by giving it a tab index?

yes, but I am not sure if that is what we want (would this mean you need to press tab first before you can delete the elements?). when I think about it, the current behavior makes sense in full screen apps but not so much if you have multiple flows on one page. maybe we can find an example for what the best ux would be here. i'll reopen the issue for now.

In the next major version (v11) you can add a class name nokey to all elements that should be treated as an input field.

@EzraBrooks - I've been stuck on this issue for several days and couldn't quite figure it out. Actually in my case it was even causing a complete re-render at times when pressing the backspace. Your workaround fixes it, thanks so much!

@moklick Can nokey be documented in https://reactflow.dev/docs/? Thanks


Feature Request: Add support for Bot password

Problem
My Switchbot bots don't move. I guess it's because I've set a password for each bot
Solution
In the plugin configuration you should be able to set up the password for each bot
Alternatives
Maybe the plugin could manage just one bot, then I would install multiple instances of the plugin, one for each bot?
Additional context
If you don't set a password for the bots, it's easy to activate them just downloading the Switchbot App

Would have to wait for OpenAPI or BLEAPI (node-switchbot) to support this.

After updating today to plugin 1.6.3, the 2 bots are responding. One of them still has a password, and I had removed the password from the second one, but none of them worked with the previous version of the plugin I had. Thank you for the fix.
They show up in Homekit as electric outlets, I wonder if they could show up as something like a "button" to press. I guess this is not the right place, so I will look for this information in the discussions tab. Thanks again.

@GcoGr I didn't chandeliers anything so must be something on the SwitchBot side.
As for displaying it different, HomeKit has stateless buttons that are only really for automations, so would be practical in this situation.
I can potentially do other device types that HomeKit allows though. Like recently I added garage door and door as options in the latest beta.

v1.7.0 I added multiple different Device Types. If there is something else that you would like me to add let me know by opening another feature request.


Blur non-image views?

Does this only blur UIImageViews? How bout other views? like buttons? Labels?

It is not only UIImageView. It can blur any views.

It is not right. It can blur even if did not use isDeepRendering. The flag is for blur when presented view.

Oh ok. I think I made it to work without using isDeepRendering.
But it seems it cannot blur dynamic views. Im actually trying to blur a camera screen. Example, you are using any camera app using the front lens. So you would see ur face in the screen, right? Then Im trying to blur it. Right now it just show black.

Did you try to blur of camera screen? I think it is impossible unfortunately because movie and camera layer cannot copy. If you want to blur a camera screen, you probably need to take pixel buffer and blur it.


rj_sharpen failed to compile

rj_sharpen failed to compile on Reshade 5.2.2

Glamayre_Fast_Effects.fx too.

Can you find the error message in the logs tab so I can see where it failed?
Also, it requires ReShade.fxh from the default ReShade shader repository that normally gets installed unless you unselect it. Check you have it (https://github.com/crosire/reshade-shaders)

Yes i have ReShade.fxh. Here is log
ReShade.log

On new 5.3.0 version the same error.

Strange. Glamayre_Fast_Effects.fx from main Github page has 800 kb size and 10496 total lines, Glamayre_Fast_Effects.fx from releases has only 100 kb size and 1885 total lines. So from releases works fine. Thnaks nad sorry,

I think you downloaded the github web page about the file instead of the file itself - use either releases, or the raw button to just get one file in github.


gost should be able to run without MQTT server

In issue #85 we finally concluded that lockup also occurred when no MQTT service was running.
But our config had disabled MQTT via config.yaml:
mqtt:
    enabled: false
    host: localhost
    port: 1883

Looks like the code does not reflect this. At least I think gost should be able to run without MQTT so possibly a check on an 'MQTT-disabled' setting to skip the MQTT Publish step.

Nice catch, apparently the moment I updated gost-db to postgres 9.6 I also updated my docker-compose to get MQTT working again and assumed 9.6 did the trick. The MQTT code is work in progress but will update it to fix this problem now.

https://github.com/Geodan/gost/commit/46d160d15070a969b25eaead33c38d0085ac2dc3

The internal MQTT client does not start now when MQTT is set to disabled
MQTT Publish not happening when MQTT is set to disabled
MQTT Publish running in goroutine now (No missing responses when MQTT is enabled but client can't connect to the broker)


"Missing required attribute" error (URL)

trying the terraform integration using a pretty "standard" config for 1password on terraform and I'm getting this error:
**│ Error: Missing required attribute
│
│ on line 1:
│ (source code not available)
│
│ The attribute "url" is required, but no definition was found.
**
my docker compose file is:
version: "3.4"

services:
  op-connect-api:
    image: 1password/connect-api:latest
    ports:
      - "8080:8080"
    volumes:
      - "./1password-credentials.json:/home/opuser/.op/1password-credentials.json"
      - "data:/home/opuser/.op/data"
  op-connect-sync:
    image: 1password/connect-sync:latest
    ports:
      - "8081:8080"
    volumes:
      - "./1password-credentials.json:/home/opuser/.op/1password-credentials.json"
      - "data:/home/opuser/.op/data"

volumes:
  data:

and my terraform file is
terraform {
  required_providers {
    docker = {
      source  = "kreuzwerker/docker"
      version = "2.15.0"
    }
    onepassword = {
      source = "1Password/onepassword"
      version = "~> 1.1.2"

    }
  }
}

provider "onepassword" {
  url = "http://localhost:8080"
}

resource "onepassword_item" "demo_password" {
  vault = "ACTUAL_VAULT_UUID"

  title    = "Demo Password Recipe"
  category = "password"

  password_recipe {
    length  = 40
    symbols = false
  }
}

output of versions
❯ terraform version
Terraform v1.0.8
on darwin_amd64
+ provider registry.terraform.io/1password/onepassword v1.1.2
+ provider registry.terraform.io/kreuzwerker/docker v2.15.0
❯ docker version
Client:
 Cloud integration: 1.0.17
 Version:           20.10.7
 API version:       1.41
 Go version:        go1.16.4
 Git commit:        f0df350
 Built:             Wed Jun  2 11:56:22 2021
 OS/Arch:           darwin/amd64
 Context:           default
 Experimental:      true

Server: Docker Engine - Community
 Engine:
  Version:          20.10.7
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.13.15
  Git commit:       b0f5bc3
  Built:            Wed Jun  2 11:54:58 2021
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.4.6
  GitCommit:        d71fcd7d8303cbf684402823e425e9dd2e99285d
 runc:
  Version:          1.0.0-rc95
  GitCommit:        b9ee9c6314599f1b4a7f497e1f1f856fe433d3b7
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Hi @nnovaes,
Thank you very much for opening a ticket and providing this much detail :open_hands:
I think I was able to reproduce the problem when I provide the Connect token through Terraform's prompt instead of through the OP_CONNECT_TOKEN environment variable. I have opened a PR to address this.
While this is not yet released, you could try setting setting your token through an environment variable:
export OP_CONNECT_TOKEN=<your-token>
terraform apply

If that works for you, I expect the problem to be resolved in the next release of the provider.

it did give me a different error message,
so what ultimately worked was to run export TF_VAR_OP_CONNECT_TOKEN="<TOKEN HERE>" and configure my .tf  like this:

variable "OP_CONNECT_TOKEN" {
}
provider "onepassword" {
  url = "http://localhost:8080"
  token = var.OP_CONNECT_TOKEN
}

i guess the documentation should mention the token is passed from terraform, not the docker services running, so adding TF_VAR before the actual var name is needed.

Good to hear you got it working! :tada:
But it does indeed sound like we need to clear up some things in the docs or in how the provider works. I think that is partly addressed in this PR, but I'd like to figure out if we have to do more than that.
So could you maybe share the error you got when setting OP_CONNECT_TOKEN directly?


changed src of hookdoo, previous img url was broken

This PR modifies the src URL of Hookdoo in README.md, it was formerly seen as:

I have added the new URL currently present on the official page of Hookdoo: https://hookdoo.com/img/Hookdoo_Logo_1.png
P.S. I am new to open-source contribution, I'm all ears to feedback and please let me know if there are any issues you'd like me to solve regarding Python, JavaScript, and Frontend!

Thank you :-)


plugin: find new shield value offset for 5.4

When updating the plugin in aeb55b2945e I couldn't find the shield percentage any more.  If anybody with cheat engine expertise has time to look into this, it'd be much appreciated.

I took a brief look at this a couple of nights ago after raid and from what I can see it has been removed from the EntityMemory region completely. I was able to find it, possibly(?) in a totally separate section of memory. It sits right next to current/maxHP and current/maxMP so it might be related to the party list, although its still valid solo!
You can semi-reliably find it by shielding yourself and doing a 2 byte scan for values between 1 and 100, removing the shield and searching for values equal to 0, and then repeating.

the shield percentage is at 0x1977 on the actor

the shield percentage is at 0x1977 on the actor


Migration to https://github.com/citation-style-language/journals

I just created a new repository, journals, to hold the journal metadata originally maintained in "utilities/generate_dependent_styles", and migrated the files and their git history using "git subtree split", as documented at https://makingsoftware.wordpress.com/2013/02/16/using-git-subtrees-for-repository-separation/.
@adam3smith, @zuphilip, please don't commit or make PRs to "utilities/generate_dependent_styles" anymore. Once I correct the style generation script in "journals" to deal with the new directory structure, and confirm everything works, I plan to delete "utilities/generate_dependent_styles" to complete the migration.

OK, thanks


Using cron style for recurrence in worker

Hi!
How to use cron style in recurrence for worker, or is it possible?
I would like to do:
recurrence { "0 1 * * *"}
But it is not working. Is there some trick how to do it?

no possible, sidetiq use ice_cube as recurrence library


Upload from frontend

Hello. First thanks for this library.
I'm having a problem when trying to upload a file from the frontend. I'm currently doing so like this:
let S3Client = new S3({
      bucketName: 'bucket',
      region: 'eu-east-1',
      accessKeyId: 'id',
      secretAccessKey: 'sec',
      s3Url: 'https://bucket.s3.amazonaws.com/'
});
S3Client
      .uploadFile(document.querySelector('[type="file"]').files[0])
      .then(response => console.log(response))
      .catch(err => console.error(err))

I'm not sure if this is a correct approach? I'm only getting 400 bad request
I have been able to delete a file with the same configuration (S3Client.deleteFile('a.file'))
In readme example, file isn't defined so I'm not sure what's the expected or how should I send the file? Any help?
Thanks!

Turns out the problem was that the URL generated was confusing me and I was using the wrong region. Might be fixed when #45 is


Remove short array syntax for PHP <5.4 support

There is no mention of PHP version requirement, however it must be >5.4 without this update :-(

@hobailey awesome! Thanks for taking the time to contribute.
Currently, we have support for 5.4 and above (this should not be the case, but it's on "the list"), but shortly we will only be supporting 5.6 and above. Generally, we will not support end of life versions: http://php.net/supported-versions.php


Enhance User Interaction with Toast Notifications-#268

Description
Implemented toast notifications to improve user experience across the application. The toast notifications will now appear in the following scenarios:
1 User successfully signs in.
2 User successfully signs out.
3 User successfully signs in with Google.
4 User adds an item to the cart.
5 User adds an item to the wishlist.
6 User deletes an item from the cart.
7 User deletes an item from the wishlist.
8 User deletes all items from the cart.
9 User deletes all items from the wishlist.
10 User moves an item from the wishlist to the cart.
11 User removes an item from the wishlist by clicking the wishlist button twice.
Motivation and Context
This change enhances user experience by providing immediate visual feedback for key actions, ensuring users are aware of the outcomes of their interactions. This addresses the need for clear communication of actions such as signing in, signing out, and managing cart and wishlist items.
How Has This Been Tested?
a) Manually tested each scenario where a toast notification is triggered to ensure correct display and behavior.
b) Verified toast notifications appear with appropriate messages and styles.
c) Ensured notifications do not overlap excessively and are dismissed automatically after a few seconds.
d) Tested on multiple browsers and devices to ensure consistency.
Screenshots (if appropriate):
Add to cart:

Add to whishlist:

Remove from whishlist:

Move to cart:

Clear whishlist:

Remove from cart:

Clear cart:

SignOut:

SignIn:

SignIn with Google:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@mr0nerd great work


Added 3 candidates with 3 nodes

Added:

@kureus:matrix.org (didn't add 2nd handle: @firegrass:matrix.org) with 1 node
@tux_in_tuxedo:matrix.org with 1 node
@plant-a-tree:matrix.org with 1 node

@ironoa Element handles verified with the users


(152117) Change the "All conditions met" task for transfers

Changes
The "All conditions met" task should be called "Authority to proceed" for Transfers.
We have decided to keep the name of the attribute the same (all_conditions_met) for simplicity, but change the accompanying text. We have added an alias authority_to_proceed on the attribute.
We have also added two more checkboxes to this task.
Checklist

[ ] Attach this pull request to the appropriate card in DevOps.
[x] Update the CHANGELOG.md if needed.

Agree with you on the migrations, I did prevaricate on it. They make recreating the db a pain.


refactor: redo all the errors

Closes #264

Current dependencies on/for this PR:

main

PR #318   👈

PR #321

This comment was auto-generated by Graphite.


Attach Lazy ClangImporter Diagnostics as Notes to Sema Diagnostics

Behaviour Change
Clang importer diagnostics that are produced as a result of a reference in Swift code are attached as notes to the Sema produced diagnostic that indicates the declaration is unavailable.
This has the added benefit that ClangDiagnostics are produced as many times as referenced from Swift, whereas previosly it was just the first time.
Old Behaviour
badCFunc()
// CHECK: warning: 'badCFunc' could not be imported
// ...
// CHECK: error: cannot find 'badCFunc' in scope
badCFunc()
// CHECK: error: cannot find 'badCFunc' in scope

New Behaviour
badCFunc()
// CHECK: error: cannot find 'badCFunc' in scope
// CHECK: enote: 'badCFunc' could not be imported
// ...
badCFunc()
// CHECK: error: cannot find 'badCFunc' in scope
// CHECK: enote: 'badCFunc' could not be imported
// ...

As a result of this change, the eager diagnostic mode only produces notes. This is somewhat unorthodox, notes are typically attached to some other error or warning. This can be addressed if it is a concern.
Implementation Change
Previously, diagnostics were produced via calls to one of diagnoseValue, diagnoseMissingMember or diagnoseDeclDirectly from outside the ClangImporter. As I mention, these calls were made outside the ClangImporter, but still relatively close. This had the above "only report diagnostics for the first usage" effect. The aforementioned methods have been consolidated into diagnoseTopLevelValue and diagnoseMemberValue. These calls are made in Sema, directly after the diagnostic reporting the declaration could not be found is produced.
diagnoseTopLevelValue
This method is to be used to look up top level declarations whose containing context
is not known (ex: A C function, a macro etc).
/// Emits diagnostics for any declarations named name
/// whose direct declaration context is a TU.
virtual void diagnoseTopLevelValue(const DeclName &name) = 0;

diagnoseMemberValue
This method is to be used to look up declarations where the context
is required to disambiguate between declarations. Typically, this means
members (the context being the containing decl). You could use this
to diagnose a top level declaration from a specific translation unit,
but there no point in doing so.
/// Emit diagnostics for declarations named name that are members
/// of the provided baseType.
virtual void diagnoseMemberValue(const DeclName &name,
                                 const Type &baseType) = 0;

@swift-ci please smoke test.

@swift-ci please smoke test macOS.

@swift-ci please smoke test.

@swift-ci please smoke test macOS.


[Bug]: min_num_seeds setting not respected

Describe the Bug
I'm experiencing an issue where the min_num_seeds setting in my share_limits config does not seem to be working as expected. I have a torrent with 1 seed and a ratio of 16 that's being paused, even though my min_num_seeds is set to 10 and max_ratio is set to 3 in my group. From my understanding, the torrent should continue seeding because it has fewer than 10 seeds, thus overriding the max_ratio setting.
Config
qbt:
  # qBittorrent parameters
  host: XXXX
  user: XXXX
  pass: XXXX

settings:
  force_auto_tmm: false # Will force qBittorrent to enable Automatic Torrent Management for each torrent.
  tracker_error_tag: issue # Will set the tag of any torrents that do not have a working tracker.
  nohardlinks_tag: noHL # Will set the tag of any torrents with no hardlinks.
  share_limits_tag: ~share_limit # Will add this tag when applying share limits to provide an easy way to filter torrents by share limit group/priority for each torrent
  ignoreTags_OnUpdate: # When running tag-update function, it will update torrent tags for a given torrent even if the torrent has at least one or more of the tags defined here. Otherwise torrents will not be tagged if tags exist.
    - noHL
    - issue
    - cross-seed

directory:
  # Do not remove these
  # Cross-seed var: </your/path/here/>  # Output directory of cross-seed
  # root_dir var: </your/path/here/>  # Root downloads directory used to check for orphaned files, noHL, and RecycleBin.
  # <OPTIONAL> remote_dir var: </your/path/here/>  # Path of docker host mapping of root_dir.
  # Must be set if you're running qbit_manage locally and qBittorrent/cross_seed is in a docker
  # <OPTIONAL> recycle_bin var: </your/path/here/>   # Path of the RecycleBin folder. Default location is set to remote_dir/.RecycleBin
  # <OPTIONAL> torrents_dir var: </your/path/here/>  # Path of the your qbittorrent torrents directory. Required for `save_torrents` attribute in recyclebin
  # <OPTIONAL> orphaned_dir var: </your/path/here/>  # Path of the the Orphaned Data folder. This is similar to RecycleBin, but only for orphaned data.
  cross_seed:
  root_dir: /data/torrents/
  remote_dir: /data/torrents/
  recycle_bin: /data/torrents/.RecycleBin
  torrents_dir: /qbittorrent/data/BT_backup
  orphaned_dir: /data/torrents/orphaned_data

cat:
  # Category & Path Parameters
  # <Category Name> : <save_path>  # Path of your save directory.
  prowlarr: "/data/torrents"
  radarr: "/data/torrents"
  radarr-uhd: "/data/torrents"
  readarr: "/data/torrents"
  tv-sonarr: "/data/torrents"

tracker:
  # Mandatory
  # Tag Parameters
  # <Tracker URL Keyword>:    # <MANDATORY> This is the keyword in the tracker url. You can define multiple tracker urls by splitting with `|` delimiter
  # <MANDATORY> Set tag name. Can be a list of tags or a single tag
  #   tag: <Tag Name>
  # <OPTIONAL> Set this to the notifiarr react name. This is used to add indexer reactions to the notifications sent by Notifiarr
  #   notifiarr: <notifiarr indexer>
  other:
    tag: other

share_limits:
  # Control how torrent share limits are set depending on the priority of your grouping
  # Each torrent will be matched with the share limit group with the highest priority that meets the group filter criteria.
  # Each torrent can only be matched with one share limit group
  # This variable is mandatory and is a text defining the name of your grouping. This can be any string you want
  minSeedToMaxRatio:
    priority: 1
    max_ratio: 3
    min_num_seeds: 10
    cleanup: false
  default:
    priority: 999
    max_ratio: -1
    max_seeding_time: -1
    cleanup: false

recyclebin:
  # Recycle Bin method of deletion will move files into the recycle bin (Located in /root_dir/.RecycleBin) instead of directly deleting them in qbit
  # By default the Recycle Bin will be emptied on every run of the qbit_manage script if empty_after_x_days is defined.
  enabled: true
  # <OPTIONAL> empty_after_x_days var:
  # Will automatically remove all files and folders in recycle bin after x days. (Checks every script run)
  # If this variable is not defined it, the RecycleBin will never be emptied.
  # WARNING: Setting this variable to 0 will delete all files immediately upon script run!
  empty_after_x_days: 30
  # <OPTIONAL> save_torrents var:
  # If this option is set to true you MUST fill out the torrents_dir in the directory attribute.
  # This will save a copy of your .torrent and .fastresume file in the recycle bin before deleting it from qbittorrent
  save_torrents: true
  # <OPTIONAL> split_by_category var:
  # This will split the recycle bin folder by the save path defined in the `cat` attribute
  # and add the base folder name of the recycle bin that was defined in the `recycle_bin` sub-attribute under directory.
  split_by_category: false

orphaned:
  # Orphaned files are those in the root_dir download directory that are not referenced by any active torrents.
  # Will automatically remove all files and folders in orphaned data after x days. (Checks every script run)
  # If this variable is not defined it, the orphaned data will never be emptied.
  # WARNING: Setting this variable to 0 will delete all files immediately upon script run!
  empty_after_x_days: 30
  # File patterns that will not be considered orphaned files. Handy for generated files that aren't part of the torrent but belong with the torrent's files
  exclude_patterns:
    - "**/.DS_Store"
    - "**/Thumbs.db"
    - "**/@eaDir/**"
    - /data/torrents/temp/**
    - "**/*.!qB"
    - "**/*_unpackerred/**"

notifiarr:
  # Notifiarr integration with webhooks
  # Leave Empty/Blank to disable
  # Mandatory to fill out API Key
  apikey: XXXX
  # <OPTIONAL> Set to a unique value (could be your username on notifiarr for example)
  instance:

webhooks:
  # Webhook notifications:
  # Possible values:
  # Set value to notifiarr if using notifiarr integration
  # Set value to apprise if using apprise integration
  # Set value to a valid webhook URL
  # Set value to nothing (leave Empty/Blank) to disable
  error:
  run_start:
  run_end:
  function:
    cross_seed:
    recheck:
    cat_update:
    tag_update:
    rem_unregistered:
    tag_tracker_error:
    rem_orphaned:
    tag_nohardlinks:
    share_limits:
    cleanup_dirs:

Logs
https://pastes.io/download/vnnrjvrbsa
Screenshots
No response
Installation
Docker
Version Number
4.0.2
What branch are you on?
master

please test in the latest develop branch. Should be fixed now 👍

That was quick! Will give that a test today.


[docs] terraform-modules automation

what

bump build-harness tag to 0.14.5
create terraform-module-page template for docs project
create script for automation docs create/update actions

why
https://github.com/cloudposse/docs/issues/228
https://github.com/cloudposse/docs/issues/229
Expected result
Script will create terraform-pages in the content/terraform-modules/ folder

We need documentation to be self-contained. Putting this in the dev folder is the wrong place and causes unnecessary sprawl.
The scripts @vadim-hleif have in this repo were more for hacking together pull-requests, but not for a routine workflow.
The terraform-module-page looks good. Let's move this into the docs project under a templates/ root folder.

@osterman

Putting this in the dev folder is the wrong place and causes unnecessary sprawl.

i've used dev repo because it already contains all terraform-modules repositories and docs repository in one place. So it is very suitable place for orchestration things in my point of view.
Moving this scripts to docs repo will require fetching names and cloning all terraform-modules (and coping code for it)

@osterman i'll try to implement it in docs repo

replaced by https://github.com/cloudposse/docs/pull/247


ENH: Add a parameter to test to improve itk::ResampleImageFilter testing.

Add a parameter to the itkResampleImageFilterTest2.cxx to improve the
itk::ResampleImageFilter class testing.
Specifically, exercise the workflow when the UseReferenceImage boolean
is set to false. Add the corresponding baselines.
Rename the tests and old baselines to account for this new test.
Take advantage of the commit to improve the style of the test:

Use the TRY_EXPECT_NO_EXCEPTION in lieu of try/catch for the sake of
readability/compactness/save typing boilerplate code.
Explicitly call Update() on filters with the help of the
TRY_EXPECT_EXCEPTION macro.
Use the TEST_SET_GET_BOOLEAN macro to test the boolean members.
Move the std::cout messages before the filter calls to Update().

There seems to been some baseline comparison issues with 32-bit systems:
https://open.cdash.org/testDetails.php?test=728564967&build=5731782

I guess it has to do with this division being rounded differently in a 32-bit system vs. the 64-bit system I use.
Not sure I will be able to test any solution, but I'd guess that doing a ceil would make the test image dimension become 320 at the place of 319. I'll submit a patch set later.

Fantastic, thanks @jhlegarreta !

Opened PR #462.


Allows command line argument to define postmeta fields that contain user ids (Fixes #48)

The command line argument uid_fields will tell MU-Migration that there are postmeta fields containing user ids, and that those fields will need to be included in user id mapping on import.
This basically just extends the existing functionality around WooCommerce customer field to include an arbitrary set of postmeta fields. The WooCommerce field is automatically added to the list of user id fields if WooCommerce is active.
Example:
wp mu-migration import all mysite.zip --new_url=example.com/mysite --uid_fields='_content_audit_owner,_another_postmeta_userid_field'

Manual testing indicates that this works. I'd like to develop unit tests, but haven't gotten to it yet.

Both good catches on your suggested changes. I'll make them soon -- in the meantime, re: the tests failing, I note that Travis says they didn't fail but errored due to a composer command erroring. That seems unlikely to be caused by the changes here (didn't see it locally, at least). Have you tried re-running the tests?

@azito122 Tests are passing locally. Could you also come up with a test for this new feature? maybe just add some random meta with user ids and have MU-Migration replace them.

Refactored based on maintainer feedback above. Unit tests are in the works.

@nicholasio FYI I think your Travis builds are dead. Even master is failing when I trigger a build from our fork. Not sure what exactly is broken but you can see the error in the build for this pull request, or, I bet, if you trigger a master build on your own Travis.

Update: seems to be a problem with wp-cli version 2.0.0 and 2.0.1. At least, when I downgraded to 1.5.1 it solved the problem.

The Travis builds are exploding. In addition to to the problem with wp-cli, I found that to get the builds to run I had to clear the Travis cache for that branch in between builds.
In terms of the functionality related to this pull request, here are the tests passing, with code matching what's in this branch now. I added the uid_fields test to the existing update authors test.

@azito122 Let's get the Travis fixed first before merging this. But it looks good overall!

Hmm, travis is still failing on this one. Any idea why?

Looks like travis is passing now.

Yeah, I cleared Travis cache and it seems to have fixed.


Remove the duplication between complete_ operations

Closes https://github.com/bloomberg/powerfulseal/issues/26

Closing in favour of https://github.com/bloomberg/powerfulseal/pull/28


Join command doesn't work in this case

A friend of mine clicked join button after we created a /newsanta but the bot doesn't let him to be added.
I don't know why but it seems it's not working only with him.
I summarize the use case below.

he taps join
the bot redirect him to the private chat.
He taps /start
The bot replies with this:

[Forwarded from Secret Santa bot 🤫🎅🏼🎁]
Hello "name"!
I can help you organize a Secret Santa 🤫🎅🏼🎁 in your group chats :)
Just add me to a chat and use /newsanta to start a new Secret Santa.
Source code here
instead of
🎄 You joined "name" 's Secret Santa!
Now wait for "name" to start it. You will receive your match here, in this chat

Thanks for the report.
To debug the issue, it would be helpful to have at least one of these two information:

the user ID of the user facing the issue
the chat ID of the group where the Secret Santa is being organized

I believe the easiest way to obtain these two numbers is to install kotatogram (a Telegram Desktop fork that lets you see user and chat IDs) and open the user profile, and the chat profile. There, you'll find a number which is the chat/user ID
For example:

It seems it worked in the end.
I drop the info anyway maybe you can see more details.
User: 249384467
Group:<PHONE_NUMBER>

It eventually worked so I will close the issue. The bot now logs the /start command payload to debug other deeplinking issues


Annihilator Rounds still fire without mod

All particle beam weapons - Novalight, Big Bang, Va'ruun Inflictor Rifle and Inflictor Pistol (dubbed Star Shard in-game), except for Novablast in my experience - produce an Annihilator Round effect as though the weapon mod by that name were installed when it is not installed.

Details
Annihilator Rounds is a weapon mod that can be applied to Particle Beam type weapons for a chance to produce residual area-of-effect damage as a glowing ball of sparks on the ground. Even without this mod installed, those weapons listed above produce this effect about once every five or so shots that hit an enemy. The AoE damage affects companions and friendlies which causes obvious issues, and the only way to avoid producing it is simply not to use these weapons, regardless of which mods they have - or don't.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Acquire a Va'ruun Inflictor Rifle or Pistol (in the console, searching help "inflictor" 4 weap will show results for both); without the Annihilator Rounds weapon mod on it, use it on several enemies. Since it is an RNG function, it may take several tries, but you should see a residual ball of white sparks on the ground where the weapon hit an enemy. This effect is supposed to happen only with the Annihilator Rounds mod, a mod that is meant to be used advisedly since it's "banned by nearly every government", not ejected randomly by every weapon of the particle beam variety.

This is intended design. These appear upon critical hits.


[LAB-1022] roadmap memo

From SyncLinear.com | LAB-1022

https://docs.google.com/document/d/1tPtJMVBDDHo_ufXYMEeNJ4QNy-HSbAwyjrVeC-XcqgU/edit


[reorder] handle the case where display fields are out of date

Summary
This will adress "handle the case where display fields are out of date" in issue #3. The idea is to store only the ID's of the referenced collection and query data that is displayed from that collection. In order to pass the queried data to preview without calling onChange (to avoid triggering unsaved changes if there are none) we use local storage, kind of hacky but it works for now.
So basically we store IDs from the referenced collection :arrow_right: control mounts :arrow_right: query fresh data :arrow_right: save to local storage :arrow_right: preview mounts :arrow_right: fetch from local storage. And then reorder the queried items when onChange is called as previously.
Things added:

generateIdentifierFromField to generate a unique key for local storage
sortByOrder sorts and merges incoming values with collection-data (not sure about this one, will probably change) replaced with copy({from, into, key}) which is used to copy fields from fetched data into the ordered array of id_field's

Things to fix:

Draggable list is not synced when component mounts (needs to merge id-array with data-array inside or after diff(?)) [done]
Abstract data handling from user in preview (expose a getData() function, HOC or custom hook?)
Preview mounts before data is saved in local storage

Questions:

Should we clear local storage when control unmount?

Other minor stuff

Removed id_field to be rendered in control if its not in display_fields

Thank you for your work @gurkodil! It works well, I can see updated data without triggering 'unsaved changes' in the UI :+1:
We're duplicating the data in multiple places though 😢 (raw data in localStorage, data order in both component state & neltify cms's redux store). Now looking at this I realized I have committed a 'React sin' by doing onChange and setState at the same time!
I think moving forward we should remove local state, and store data like the following:

Store raw data in localStorage (everything, no extractions)
Store order in the redux store via props.onChange

I think the flow should now be
Control component

On component mount, fetch the raw data. Store it in localStorage.
Extract the id data out of the raw data
Check if there's a props.value. If there's no data, trigger onChange so user has a chance to set it. If there is value, compare it against the id data. If there's modifications (new or deleted entries), diff the 2 then trigger onChange, otherwise do nothing.
In render, extract display_fields out of the raw data from localStorage. Use it to render the controller.
Remove data from localStorage on unmount since we want to fetch fresh data every mount.

Let me know what you think!

Thanks for the valuable inputs @d4rekanguok! 👍
Yeah I can se that, accidentaly left the localstorage to update every change but my initial thought was to only push it once and then reorder it via changes. But aah still kind of problematic when it requires ordering in both preview and control and as u said, storing data at multiple places.
I think the flow you mention sounds good and normalizing the data is a great idea! However I see some problems with storing the order explicity in props.value and some local storage problems:

About "Check if there's a props.value [..]", it doesn't feel user-friendly to trigger "unsaved changes" if the user haven't made any changes inside the widget (even if the referenced collection have changed)? It can be necessary to avoid having to handle unordered items in build for the SSG, but changes can be left unsaved so these cases will have to be considered anyway.
Localstorage saves slower than preview mounts, so we might have to wait for it somehow (Timer? can be ugly)
Fetching from local storage in render might be slow (reads from disk I think)

Hm... Not sure on how to pursue on this one. Yesterday I did some testing with a another "hacky" solution by creating a portal to the preview-splitpane and then render preview via control, which seems to work fine and then we can used a shared state. This comes with some benefits as I can see:

No need for local storage
No need for generateIdentifierFromFieldwhich felt kinda weird
Data will always be in sync between preview and control
No logic in preview

And some downsides:

Render preview in control feels wrong/breaking convention 😅
Preview is pretty deep nested and render updates could be slow? But i haven't noticed any problems with it
Will require the api to change as below

// From this:
const CustomReorderControl = createControl({
  renderListItem: item => <ListComponent item={item} />
})

cms.registerWidget({
    name: 'custom-reorder',
    controlComponent: CustomReorderControl,
    previewComponent: CustomReorderPreview,
  })

// To something like this:
const CustomReorderWidget = createWidget({
  controlListItem: item => <ListComponent item={item} />, // optional
  previewListItem: items => <CustomReorderPreview items={items}/>, // optional
  name: 'custom-reorder' // Optional
})
cms.registerWidget(CustomReorderWidget)

Thanks for your work @gurkodil! The portal approach is super creative, and now that I think of it, not a bad 'hack'. Would you send a new PR for that approach ❤ ?

About "Check if there's a props.value [..]", it doesn't feel user-friendly to trigger "unsaved changes" if the user haven't made any changes inside the widget (even if the referenced collection have changed)? It can be necessary to avoid having to handle unordered items in build for the SSG, but changes can be left unsaved so these cases will have to be considered anyway.

This is a really good point & as we discussed, I've attempted to implement a notice bar that asks user to act when the source data has been modified. There's a few cases to handle:

When there are new entries

When some entries from the source collection are removed

And when source collection is empty.

Localstorage saves slower than preview mounts, so we might have to wait for it somehow (Timer? can be ugly)

This is a really good point as well. We'll probably need some interval stuff 😢 We should beef up our test data so we can have a more realistic dev environment

Fetching from local storage in render might be slow (reads from disk I think)

Let's store the data in local state once we got it from localstorage!
I expose a hook for preview usePreviewData, but maybe a HOC would be nicer since it's align with our existing createControl component...
Let me know what you think!

Great work @d4rekanguok! Really nice progress with the notice bar! Makes it much more understandable than triggering unsaved changes without any user action 👍
Some thoughts on notice bar

When there are new entries
When some entries from the source collection are removed

Since both these cases can happen at the same time, maybe it makes sense to show both the new and deleted items instantly? I did so in the examples below but if you don't think it makes sense we can discard it.
One thing that comes up in my head is that we could probably make it more generic to avoid forcing the developer to handle isRemoved, maybe use badges for items that have been changed? Possible something like this (not sure about the colors though):

Or simpler, with lower opacity for removed items and a blue dot for new:

Other stuff

[ ] Preview HOC: I expose a hook for preview usePreviewData, but maybe a HOC would be nicer since it's align with our existing createControl component...
Better design:

Agreed! I think this works for now, depending on how we choose to do the meta-sharing I guess it can change.

The portal approach is super creative, and now that I think of it, not a bad 'hack'. Would you send a new PR for that approach ❤ ?

Thanks! Absolutely! 🚀

Thanks @gurkodil, the design looks great!! I love the simplicity of using a modal instead. Since the widget entries should always be in sync with its source collection, user will always want the latest update anyway.
Once user click the button, I think we should still highlight the newly added entries, since they need to be ordered. That'll require a more powerful diff function (one that only keep track of modifications vs. just returning the whole data) so let's try to tackle that in another PR 😅

Close, in favor of #16


ToDo

Related software versions

Description
ToDo
testapp is a demo app of the theme to cover all customizations and test them. TODO visual testing?
Package currently needs serious review:
TESTS!
every admin template should be reviewed against its core theme peer
bower takes care of atlantis SCSS but what is minifying/managing javascript files? min versions are out of sync...
login, logout, ... should replace admin templates, instead of requiring adding paths to urls.py - subclass AuthenticationForm instead of custom form as well
...

Use cases
Remove ToDo section from README and track those in issues
References
No response

Part of this is moved to #26


Zls fails to start if automatic updates are enabled

The title says it all. It should be easy to reproduce. Simply try to start zls through vscode without an internet connection

On Windows it will also cause zls to not function at all regardless of the internet connection. If I have automatic updates enabled I get this error after restarting vscode:

After I disabled the feature zls works perfectly fine again!

@BuzzwordChief Could you post your extension logs / any network errors in the Developer Tools console window please?


VR Mode: Reticle / Fuse button / Hotspot 'virtual' click

Hi,
I tried the hotspot example in VR mode (Google Chrome on S7 Edge)
When I gaze at any hotspot, the white circle grows but I do not see any recticle.
I would also like to activate a short timer so gazing at hotspot would trigger a click after a few second (like a "fuse button" as described here: https://www.google.com/design/spec-vr/interactive-patterns/controls.html#).
Is there a way to do that ?
Do someone have code examples of how to capture these events like Hotspot onFocus, etc...
Thanks !

I had the same issue using Google Chrome 57 on a Nexus 5. Thanks!

Same thing using latest version in iOS. Thanks.

@transpirman its east, you have function HotspotRenderer.prototype.focus_ ,   there you can put some setTimeout and add logic if it focus more than 1 sec or whetewer do this this.emit('click', id); if someone focus and than blur just remove it. Should work.
HotspotRenderer.prototype.focus_ = function(id) {
  var hotspot = this.hotspots[id];

  // Tween scale of hotspot.
  this.tween = new TWEEN.Tween(hotspot.scale).to(FOCUS_SCALE, FOCUS_DURATION)
      .easing(TWEEN.Easing.Quadratic.InOut)
      .start();
      timeForHospotClick = setTimeout(function(){
          this.emit('click', id);
      }.bind(this),1000)
  };
HotspotRenderer.prototype.blur_ = function(id) {
  var hotspot = this.hotspots[id];

  this.tween = new TWEEN.Tween(hotspot.scale).to(NORMAL_SCALE, FOCUS_DURATION)
      .easing(TWEEN.Easing.Quadratic.InOut)
      .start();
      if(timeForHospotClick ){
          clearTimeout(timeForHospotClick );
      }
};

Thank you for the code.
Actually I started building on that already, but encountered a problem for which I found a workaround. I will try to post code asap.
The problem : when hotspots are created, the 'update' function in renderer emits a focus event, and no blur event.
The workaround: initialize a FocusEnabled status to false, and delay a true assignment

it will help to you this.hotspots[id].show check it before settimeout in focus

Oh thank you for the tip ! I did not think of it.
What I did so far in hotspot_renderer:
1°) declare
var HoverTimer; // Timeout fn to simulate a click event after gazing at a hotspot
var HoverReadyTimer; // Timeout fn to enable "real" focus event after hotspot creation (avoiding the automatic trigger at creation in HotspotRenderer.prototype.update
var HoverReady = false; // initialize to false to avoid automatic first 'focus' event
2°) HotspotRenderer.prototype.add = function(pitch, yaw, radius, distance, id, image, is_stereo) {
clearTimeout(HoverReadyTimer); HoverReady = false;
HoverReadyTimer = setTimeout(function() { HoverReady = true; }, 500);	// ready for real blur event
...
3°) in HotspotRenderer.prototype.update = function(camera)
if (isIntersected && !this.selectedHotspots[id] && HoverReady) {
this.emit('focus', id); this.focus_(id);
}
4°) HotspotRenderer.prototype.focus_ = function(id) {
var hotspot = this.hotspots[id];
// Tween scale of hotspot.
this.tween = new TWEEN.Tween(hotspot.scale).to(FOCUS_SCALE, FOCUS_DURATION)
.easing(TWEEN.Easing.Quadratic.InOut)
.start();
// color change
var inner = hotspot.getObjectByName('inner');
this.tween = new TWEEN.Tween(inner.material.color).to(ACTIVE_COLOR, ACTIVE_DURATION)
.start();
// Virtual Click (todo: real fuse button)
if (HoverReady == true) { var that = this; HoverTimer = setTimeout(function() { that.emit('click', id); that.up_(id); }, 2000); }
};
I will try to create a proper 'pull request' once I get how to fully use GitHub :-)
Now, any idea about the reticle not showing up ?

@transpirman i found the reason, just need change this in package.json
 from "tween.js": "^16.3.4",
 to "tween.js": "16.3.4",

https://github.com/googlevr/vrview/pull/148/commits/80e7b3410e1ee8fa817370b7285772f8a4689e69

for the reticle, uncomment https://github.com/googlevr/vrview/blob/master/src/embed/world-renderer.js#L279

The false calling of focus that happens when a hotspot is added can be stopped by moving this line https://github.com/googlevr/vrview/blob/master/src/embed/world-renderer.js#L64 two lines down so it is under "this.effect.render..."
This fix is also needed: https://github.com/googlevr/vrview/pull/158/files
Then, adding to the code from fix2015:
`
HotspotRenderer.prototype.focus_ = function(id) {
var hotspot = this.hotspots[id];
// Tween scale of hotspot.
this.tween = new TWEEN.Tween(hotspot.scale).to(FOCUS_SCALE, FOCUS_DURATION)
.easing(TWEEN.Easing.Quadratic.InOut)
.start();
if (this.worldRenderer.isVRMode()) {
this.timeForHospotClick = setTimeout( function () {
this.emit( 'click', id );
}.bind( this ), 1200 )
}
};
HotspotRenderer.prototype.blur_ = function(id) {
var hotspot = this.hotspots[id];
this.tween = new TWEEN.Tween(hotspot.scale).to(NORMAL_SCALE, FOCUS_DURATION)
.easing(TWEEN.Easing.Quadratic.InOut)
.start();
if (this.timeForHospotClick) {
clearTimeout( this.timeForHospotClick );
}
};
`


Move model name definition to each module

Codecov Report

Merging #96 into master will decrease coverage by 0.27%.
The diff coverage is 10.40%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
- Coverage   38.58%   38.30%   -0.28%
==========================================
  Files          82       82
  Lines        9720     9830     +110
  Branches     2046     2061      +15
==========================================
+ Hits         3750     3765      +15
- Misses       5750     5846      +96
+ Partials      220      219       -1

Impacted Files
Coverage Δ

neural_sp/bin/model_name.py
0.00% <0.00%> (ø)

neural_sp/models/lm/gated_convlm.py
0.00% <0.00%> (ø)

neural_sp/models/seq2seq/encoders/build.py
0.00% <ø> (ø)

neural_sp/models/seq2seq/decoders/las.py
50.33% <3.22%> (-1.57%)
:arrow_down:

neural_sp/models/lm/rnnlm.py
81.33% <5.88%> (-9.65%)
:arrow_down:

neural_sp/models/lm/transformer_xl.py
61.01% <6.66%> (-5.04%)
:arrow_down:

neural_sp/models/lm/transformerlm.py
61.11% <8.33%> (-4.23%)
:arrow_down:

neural_sp/models/seq2seq/encoders/conv.py
84.93% <8.33%> (-4.05%)
:arrow_down:

neural_sp/models/seq2seq/decoders/transformer.py
59.02% <8.88%> (-4.16%)
:arrow_down:

...eural_sp/models/seq2seq/decoders/rnn_transducer.py
74.84% <14.28%> (-1.37%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Question about ignore threshold?

The paper points out the importance of balance matching of label assign, but actually in the code you ignores the anchor(pred) whose iou>NEG_IGNORE_THRESHOLD or  < POS_IGNORE_THRESHOLD, which may break the balance?
Also, it seems you ignore the anchor(pred) whose iou > NEG_IGNORE_THRESHOLD  only make sense with max-iou label assign, why you use the the anchor(pred) whose iou is maxium or between NEG_IGNORE_THRESHOLD  and NEG_IGNORE_THRESHOLD ?
https://github.com/megvii-model/YOLOF/blob/86288132be25f0fc54d3c4b35c97a774222c12fc/playground/detection/coco/yolof/yolof_base/yolof.py#L186-L231

Hi, thanks for your questions.
For your first question, in the paper, we say that we construct balance on positive anchors. Thus, setting the threshold for negative anchors does not affect the balance. Add threshold for positive anchors indeed bring risk to ruin the balance on positives. However, we only ignore the unreasonable positive anchors (whose IoU with the gt boxes is lower than 0.15) during training. This ignorance will not affect the distribution of positive anchors too much. We can also achieve our goal that makes gt boxes on various scales participate in training and avoid unreasonable training samples that may hurt the training.
For your second question, I don't fully understand. Here, I provide some explanations on why we set ignore threshold for negatives. In this paper, we obtain positive anchors by choosing gt boxes' k nearest anchors and leave the rest to be negative anchors. However, there may exist negative anchors that have similar distances with the chosen positives. These negatives may cause ambiguity for model training if we just set them as negatives. Given that we want the model's predict boxes on negative anchors to have low IoU with gt boxes, we set a threshold on the IoU between predicted boxes and gt boxes instead of the IoU between anchor boxes and gt boxes. You can check this line. For ignoring, we first ignore all the indexes whose max IoU with gt boxes are greater than the negative threshold (Line197), then for positive, we replace the gt_classes on positive indexes with the target_classes_o (Line201).
Feel free to comment if you have further questions.

@chensnathan Thanks for your detailed response!  I think i missundertood the neg_ignore_thresh (0.7) , actually it is used for ignore the negative anchor instead of positive anchor. Thanks again for your nice work!


profile timezone not settable via client

default is "Etc/UTC"

Use timezone from settings instead. One TZ per installation.


HasConversion is ignored

There is a float column in DB which stores date/time in OADate format. It's represented in model as a DateTime property with HasConversion(d => d.ToOADate(), oa => DateTime.FromOADate(oa)). When comparing the property value with DateTime.Now in query expression, it's translated to GETDATE() and no conversion is made, which leads to incorrect comparison results with no error received.
Further technical details
EF Core version: 2.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows
IDE: Visual Studio 2017

What happens if you assign DateTime.Now to a variable then use that variable inside the query?

It works as expected in this case.

Related #10434

@witua - Can you also post the full query?