Step missing for installing C++ redistributables on the endpoints

Hi everyone,
In order to get the teams media optimizations working, the C++ redistributables also need to be installed on the client (Poorly mentioned in "https://learn.microsoft.com/en-us/azure/virtual-desktop/teams-on-avd#prerequisites" - It is not stated, that these C++ redistributables also need to be installed on the clients).
Can you add this step to the prerequisites section?
=> https://learn.microsoft.com/en-us/azure/virtual-desktop/users/connect-windows?tabs=subscribe#prerequisites
Greets Pascal

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

ID: 524e47b0-1295-7d07-0cbc-86e6daaa1358
Version Independent ID: 1cb3a5d8-4f16-583c-ad85-325f7638a873
Content: Connect to Azure Virtual Desktop with the Remote Desktop client for Windows - Azure Virtual Desktop
Content Source: articles/virtual-desktop/users/connect-windows.md
Service: virtual-desktop
GitHub Login: @dknappettmsft
Microsoft Alias: daknappe

@PasqualeMueller2
Thanks for your feedback! We will investigate and update as appropriate.

@dknappettmsft
Can you please check and add your comments on this doc update request as applicable.

@PasqualeMueller2
I'm going to assign this to the document author so they can take a look at it accordingly.

Hi @PasqualeMueller2 - thanks for the feedback. As this is a requirement for using Teams on Azure Virtual Desktop and not for the installation of the Remote Desktop client + connecting to a remote session, I'm not going to add it to the prerequisites section as it's not technically true just to connect. I've added it to the next steps section for those that want to do this.  The change should be visible later today.
We saw the other issue you created and we will expand on the requirement there.
#please-close


Supporting actions/github-script step named outputs

Is your feature request related to a problem? Please describe.
When using https://github.com/actions/github-script with a call to core.setOutput(), the VSCode extension is unable to do autocompletion and shows "Context access might be invalid" warnings.
The usual way of setting outputs with actions/github-script is to use a return statement, which creates a result output, accessible with steps.<step-id>.outputs.result (https://github.com/actions/github-script#reading-step-results).
But we preferred to use the reference to the core API to set named outputs instead.
Example:
- uses: actions/github-script@v6
  id: get-versions
  with:
    script: |
      const currentVersion = 'v1.0.0'
      const nextVersion = 'v1.1.0'

      core.setOutput('CURRENT_VERSION', currentVersion)
      core.setOutput('NEXT_VERSION', nextVersion)
- name: Use versions
  run: |
    echo "Current version: ${{ steps.get-version.outputs.CURRENT_VERSION }}"
    echo "Next version: ${{ steps.get-version.outputs.NEXT_VERSION }}"

Describe the solution you'd like
The VSCode extension should be able to read the inline scripts passed to your official actions/github-script action, and detect all the calls to core.setOutput() to provide autocompletion.
And realistically, I can understand that https://github.com/actions/github-script#run-a-separate-file would be a limitation. Trying to find all require statements, reading those files recursively, etc. would be too much for this VSCode extension. Or you would have to include some kind of bundling to find all the core.setOutput() in the import tree.

By the way, the echo "CURRENT_VERSION=$CURRENT_VERSION" >> $GITHUB_OUTPUT syntax in inline bash scripts is supported. We have autocompletion for steps.<step-id>.outputs.CURRENT_VERSION (that's cool btw! 🤩)
So it may be an extra reason to also support core.setOutput().

Ran into this today. It would be nice to at least mark these in a way to make the error go away if full support cannot be implemented.
If I only need to return one single value, the direct script return makes sense. But if I need to return multiple values, there is no need to return stringified json that then must be parsed using fromJSON to extract them. A direct set output is simply cleaner and more readable.
Its a shame this ticket has been rotting close to a year now. I guess I will just have to live with this extension screaming at me.


Fix rosetta block query

Explain your changes:

Once again changes to the archive DB schema invalidated some queries in Rosetta server. This PR is to update these queries to much current schema.

Explain how you tested your changes:

Run rosetta-cli test suite against an ITN node.

Checklist:

[x] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[x] Modified the current draft of release notes with details on what is completed or incomplete within this project
[x] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[x] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet

!ci-build-me

!ci-build-me

!approved-for-mainnet


Fix webpack hanging

This is an issue with an ancient version of webpack (1.X). It feels like it also might be one with newer versions.
This PR correctly informs Webpack 1.X that an error has happened. It is only relevant when the cache options is enabled, due to webpack.async().

I retrofitted some tests. Apologies, those should have been in my initial PR.

Any chance this could have a merge&release?

Can we find better name for this PR? Would help for release notes. Thanks!
I will merge as soon as I am sure on what is this about (I don't use this module anymore but nobody is maintaining it so, I am trying to help...)

I'm struggling to find a good title. The shortest yet most correct title I can come up with would be: "Fix not returning execution flow control to webpack when cache option is enabled and one of the files has a linting error".
But that's a bit long.
To be fair though, we sinced moved away from using eslint-loader so it's not a pressing matter for us anymore.

struggling to find a good title

I know the struggle; naming things is hard ;)
How about "Returns execflow control to cache enabled webpack on linting error"?

Thank you!


[vcpkg] Remove unused testing files

They were moved to the vcpkg-tool repo in https://github.com/microsoft/vcpkg-tool/pull/69

Thanks for the cleanup 👍


Get-Error situationally places other InnerException information before the Message output line

Update: The issue is not one of missing InnerException information, but one of situationally different ordering.
Steps to reproduce
# OK: The "InnerException :" line is followed by a "Message: " line for the inner exception.
try { 1/0 } catch {}; (Get-Error | Out-String -Stream) -replace '\e[^m]*m' | Select-String -context 0,1 '\bInnerException\b' | % { $_.Context.PostContext } | Should -Match 'Attempted to divide by zero'

# FAILS: The "InnerException :" line is followed by no fields describing the inner exception.
try { [int]::parse('foo') } catch {}; (Get-Error | Out-String -Stream) -replace '\e[^m]*m' | Select-String -context 0,1 '\bInnerException\b' | % { $_.Context.PostContext } | Should -Match 'not in a correct format'

Expected behavior
Both tests should succeed.
Actual behavior
The 2nd test fails:
Expected regular expression 'not in a correct format' to match '        TargetSite : ', but it did not match.

That is, the InnerException:  line wasn't followed by the Message:  line.
Environment data

PowerShell Core 7.0.0-preview.5

@mklement0 I think your test is flawed, your query returns TargetSite which itself is an object and only one member of the InnerException.  Here's what I see running it manually:
PS> [int]::Parse('foo')
MethodInvocationException: Exception calling "Parse" with "1" argument(s): "Input string was not in a correct format."
[30.08ms] /Users/steve/repos/PowerShell [origin/get-error-exception-type]
PS> gerr

Exception             :
    ErrorRecord    :
        Exception             :
            Message : Exception calling "Parse" with "1" argument(s): "Input string was not in a correct format."
            HResult : -2146233087
        CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : FormatException
        InvocationInfo        :
            ScriptLineNumber : 1
            OffsetInLine     : 1
            HistoryId        : -1
            Line             : [int]::Parse('foo')
            PositionMessage  : At line:1 char:1
                               + [int]::Parse('foo')
                               + ~~~~~~~~~~~~~~~~~~~
            CommandOrigin    : Internal
        ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
    TargetSite     :
        Name          : ConvertToMethodInvocationException
        DeclaringType : System.Management.Automation.ExceptionHandlingOps
        MemberType    : Method
        Module        : System.Management.Automation.dll
    StackTrace     :
   at System.Management.Automation.ExceptionHandlingOps.ConvertToMethodInvocationException(Exception exception, Type
typeToThrow, String methodName, Int32 numArgs, MemberInfo memberInfo)
   at CallSite.Target(Closure , CallSite , Type , String )
   at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TRet](CallSite site, T0 arg0, T1 arg1)
   at System.Management.Automation.Interpreter.DynamicInstruction`3.Run(InterpretedFrame frame)
   at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)
    Message        : Exception calling "Parse" with "1" argument(s): "Input string was not in a correct format."
    Data           : System.Collections.ListDictionaryInternal
    InnerException :
        TargetSite :
            Name          : ThrowOverflowOrFormatException
            DeclaringType : System.Number
            MemberType    : Method
            Module        : System.Private.CoreLib.dll
        StackTrace :
   at System.Number.ThrowOverflowOrFormatException(ParsingStatus status, TypeCode type)
   at System.Number.ParseInt32(ReadOnlySpan`1 value, NumberStyles styles, NumberFormatInfo info)
   at CallSite.Target(Closure , CallSite , Type , String )
        Message    : Input string was not in a correct format.
        Source     : System.Private.CoreLib
        HResult    : -2146233033
    Source         : System.Management.Automation
    HResult        : -2146233087
CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
FullyQualifiedErrorId : FormatException
InvocationInfo        :
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : -1
    Line             : [int]::Parse('foo')
    PositionMessage  : At line:1 char:1
                       + [int]::Parse('foo')
                       + ~~~~~~~~~~~~~~~~~~~
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1

The InnerException is there and expanded.  It just happens that the colons align but the indentation shows the members.

Thanks, @SteveL-MSFT  - indeed I missed that the information is there.
What caused the confusion is that TargetSite:  information comes first, which I didn't expect.
How do you feel about making sure that at least the Message : line comes first?

Thinking about pending PR, #11076, I suggest making at least the following two lines (properties) always come first:

Type:  (full exception type name)

Message :

If there are additional properties that are always populated, I suggest listing them before those that are situationally missing.

@mklement0 the current order of the properties is what $_.psobject.properties returns.  We can certainly change the ordering.  It's hard to rely on properties that are always populated as I don't think any of it is enforced and Exceptions and ErrorRecords (particularly nested objects) can be any form.

I think it definitely makes sense to put type and message first - they are always available and the most important properties.
I suggest putting the non-primitive properties (the ones with properties of their own or  "noisy" multi-line values) last, in that order:

Data
TargetSite
StackTrace

Then follow that by a nested, indented InnerException block, as suggested in #11123
My suggestion for the ordering of the remaining standard System.Exception properties - to go between type + message and the  non-primitive ones - is this:

Source
HResult
HelpLink


Idiomatic package names

Issue: https://github.com/Paaff/Filly/blob/master/files/dirContent.go#L1

Good package names are short and clear. They are lower case, with no under_scores or mixedCaps.

Source: https://blog.golang.org/package-names

RIP


add config to adjust the changefeed error stuck duration

Is your feature request related to a problem?
In certain situations, the CheckpointTs of a changefeed may get stuck due to a slow table for a period exceeding 30 minutes. In such cases, if the sink module reports any warning, the changefeed will immediately transition to a failed state. At this point, users must manually restore the changefeed, and data from other tables also needs to be resynchronized starting from the CheckpointTs. This can be quite inconvenient and unnecessary.
Describe the feature you'd like
Add a configuration option that allows controlling the duration for which a changefeed does not transition to a "failed" state when it gets stuck due to an error or due to a slow table.
Describe alternatives you've considered
No response
Teachability, Documentation, Adoption, Migration Strategy
No response

/label area/ticdc


Detect Missing Escape Chars for Passing a String Literal as Script Argument

//script_args.daphne
ARG = $arg;
print(ARG);

daphne script_args.daphne arg=... works for int, bool, float but not for string literal. I tried the following:
arg=test
arg='test'
arg="test"
line 1:0 mismatched input 'test' expecting {'true', 'false', INT_LITERAL, FLOAT_LITERAL, STRING_LITERAL}
[error]: Parser error: unexpected literal

Hi @m-birke, please try escaping the quotation marks to prevent them from being consumed by the shell, i.e. daphne script_args.daphne arg=\"test\".
This requirement is documented, but maybe it should be presented more prominently.
At the same time, I can imagine that one can easily forget the escaping, so it would be great if DAPHNE detected this kind of mistake and printed an understandable error message.

@pdamme thanks, yes it works, I renamed this Issue to implement the detection of missing escape chars and opened another issue for requesting more docs which I already wrote and opened a PR

IMHO, quote characters should only be required, if the provided string has white spaces (or special characters?)
daphne script.daphne text=hello number=5
and
daphne script.daphne text="hello world" number=5


Include fallback mechanism in your code to avoid issues

This API have multiple fallback urls:
Fallback URL:
https://raw.githubusercontent.com/fawazahmed0/quran-api/{apiVersion}/{endpoint}

Pseudo code:
For example, if you want to fetch ben-muhiuddinkhan edition
Fetch<EMAIL_ADDRESS>If above url fails, then try fetching<EMAIL_ADDRESS>If above url fails, then try fetching https://raw.githubusercontent.com/fawazahmed0/quran-api/1/editions/ben-muhiuddinkhan.min.json
If above url fails, then try fetching https://raw.githubusercontent.com/fawazahmed0/quran-api/1/editions/ben-muhiuddinkhan.json

Assala-mu-alai-kum brother .How are you ? Actually, brother, I am new to a flutter, recently I finished the flutter course, so I am trying to practice building an AL-Quran Mobile Apps, That's why I am trying to use your API,In this API I don't find any surah, Just found sura list. So if you can give me the actual API where I can find the ALL Surah and I can complete this project, That would be very helpful for me.
Thank you!

hi, I cannot this api for my telegram bot, I have 403 error, is there any configuration for preventing telegram bot messages

Please could you share the code

Thanks, man, I corrected itI am a new in python, I was cpp coder, and now I am learning python,so I have a typically mistakes, thanks again Sent from Mail for Windows From: Fawaz AhmedSent: Friday, May 12, 2023 9:36 PMTo: fawazahmed0/quran-apiCc: jamsCore; CommentSubject: Re: [fawazahmed0/quran-api] Include fallback mechanism in your code to avoid issues (Issue #27) Please could you share the code—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID<EMAIL_ADDRESS>


axios get 403 error when runing

https://user-images.githubusercontent.com/83738283/117539345-ca178f00-b03c-11eb-81b7-8d176344ae3f.mov
my issue happens when i try to run a specific spec, then I got an 403 error, The issue seems coming from line 50 this file
https://github.com/Synthetixio/synpress/blob/master/helpers.js

@joemaoniu Hey, does it still occur?
Have you followed README, "important" section?
https://github.com/Synthetixio/synpress#-important
Metamask version is hardcoded and frequently updated under supervision to avoid a case when e2e tests break because of CSS classes changes in new version, so all you need is to keep synpress updated in your project. However, you can still override metamask with METAMASK_VERSION environmental variable, for example: METAMASK_VERSION=9.3.0 or METAMASK_VERSION=latest.

@joemaoniu let me know if this still happens, thanks!


[FirebaseCore] Mark duplicate internal API as unavailable in Swift

Context

FirebaseCore/Sources/FIRComponentContainerInternal.h was edited in cl/448585537 after the initial copybara to fix a build issue.
This PR brings that change over to change the code at the source-of-truth so future copybaras don't undo this workaround in g3.
I'm not expecting this to be a problem since Swift clients (of FirebaseCore) will get this API from FirebaseCoreExtension here: FirebaseCore/Extension/FIRComponentContainer.h.

#no-changelog

CI is green for FirebaseCore, FirebaseStorage, and FirebaseFunctions.
I will revert and resolve the CI triggers after after review.


Rename Jewelery Count plugin and add all items with charges/doses to it

Also, possibly add a way to automatically detect items with charges by their name, so we don't have to manually put them all in.

Yup, there's been a consensus on renaming since #624 at least. I was planning a refactor once the 1.3.2 changes are shipped!

Category toggles might be good, like "teleport items", "potions", "other" or something like that

I do like the idea of auto-numbering items using the (#) naming pattern. If we start filtering by item category though, we'll have to hardcode all of the known ones anyway, so it'd only be useful for new items during the dead week after their release.

Well I was thinking doing it by the left click option, so if it has a "drink" op its a potion, if it has "rub" or "teleport" its jewelery, etc. Not sure if every item would fit into that pattern, might be worth trying

Sounds pretty smart on the surface. Time to start making a list 📝

This has been done afaik. Can be closed now?

Partially but the new plugin doesn't work for all items with charges. Not sure if that's still on the table or not

Thus far we've been adding new items as they've been requested. (see the recently-added Ogre Bellows, and open issues/PRs for other items)

I was hoping for a more automatic solution like Soy's PR but yeah I guess we can just do them manually as requested.

Yeah looks like the per-item solution is preferred so I will close this


Clarify documentation about dynamic properties and methods search order scope

Expected Behavior
The documentation should provide consistent information about property and method search scope order for Project and ExtensionAware Objects
Current Behavior
Actually the documentation provides conflicting information about property scope search order:
On Project general documentation the property scope search order is:

Project own properties
extra properties
extensions
convention properties
tasks
extra properties inherited from project's parent

https://github.com/gradle/gradle/blob/7a69f2f3d791044b946040cd43097ce57f430ca8/subprojects/core-api/src/main/java/org/gradle/api/Project.java#L132-L164
But on Project.property and Project.findProperty methods the order is:

Project own properties
extensions
convention properties
extra properties
tasks
properties inherited from project's parent

https://github.com/gradle/gradle/blob/7a69f2f3d791044b946040cd43097ce57f430ca8/subprojects/core-api/src/main/java/org/gradle/api/Project.java#L1367-L1385
and on ExtensionAware interface the order is:

object own properties
Groovy Meta-programming methods like propertyMissing()
extra properties
extensions

https://github.com/gradle/gradle/blob/7a69f2f3d791044b946040cd43097ce57f430ca8/subprojects/core-api/src/main/java/org/gradle/api/plugins/ExtensionAware.java#L67-L82
while for Project dynamic methods the order is:

Project own methods
build file methods
extensions
conventions
tasks
properties defined as closure (so I guess extra properties are last)

https://github.com/gradle/gradle/blob/7a69f2f3d791044b946040cd43097ce57f430ca8/subprojects/core-api/src/main/java/org/gradle/api/Project.java#L191-L213

Sorry for the late reply.
This is a valid documentation issue.


Buffer module ends up in production build

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
I'm not sure is this bug or excepted behaviour but I believe it's a bug. Running yarn build to generate production build, buffer module ends up in production build!
 "scripts": {
    "build": "set NODE_ENV=production rimraf build/dist && webpack -p --config webpack/webpack.config.js && npm run bundle_check",
    "bundle_check": "babel-node bundle_check.js"
  },

Could someone explain me why this package ends up in production build. Am I missing something?

yarn why buffer

yarn why node-libs-browser

And webpack is specified only as dev dependency.
I pointed this issue to WEBPACk but they answered it's connected with axios. Then I searched your repo and found the usage within the HTTP adapter
What is the expected behavior?
Buffer module should not be in production build.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
axios: 0.16.1,
Webpack: 2.5.1
Node: 7.8.0
OS: OSX Sierra
Thanks in advance,
Davor

Fixed via #887.

hey @nickuraltsev, #887 is not fixing this issue.


Pre-select an option in ng-options optgroup (with "group by" property)

I need to pre-select an <option> in a dropdown that has option groups.
Without <optgroup> the ng-model reads the value and selects the right option, somewhy that doesn't work with option groups.
Giving a simple example:
      <select
        ng-model="selected"
        ng-options="option.value group by option.type for option in options"
      ></select>

  $scope.options = [
      {type:'Mini', value:'Option 1'},
      {type:'Mini', value:'Option 2'},
      ];

  //$scope.selected = 'Option 2';
  $scope.selected = {type:'Mini', value:'Option 2'};

and a sandbox
http://plnkr.co/edit/1AZ3VosOhfYkj1fImbwv?p=preview

You can make this work by either setting the model to the item in the options:
$scope.selected = $scope.options[1];
or by using track by:
ng-options="option.value group by option.type for option in options track by option.value"
This is because ngOptions by default watches the model by identity, not by equality. That means if you assign a new object to the model, even if it has the same properties as the item in your collection, ngOptions will not see them as the same (because they are equal, but not identical). This is done mainly for performance reasons, but the docs don't do a very good way of explaining this.


Reload config and logging after SIGHUP

If this issue gets enough interest, SIGHUP handling will be added.

I'm going to remove this, since we log rotate using the lumberjack library anyway.


BE-108 - Exception Handling Mechanism Implementation

Graceful handling of exceptions should be enabled, mappings for not found and server errors should be implemented.

Exception handling mechanism for not found and internal server errors are implemented.


Classes & Generators: Can't combine?

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Yes.
Description
When a script with both a js class and a generator are defined, compilation breaks.
Expected behavior
Compilation of script.
Actual behavior
Browser console logs error message "...Uncaught ReferenceError: _regeneratorRuntime is not defined"
Environment
npm ls react-scripts:
<EMAIL_ADDRESS>/Users/angelbeltran/dev/test/my-app
└──<EMAIL_ADDRESS>node -v:
v7.0.0
npm -v:
4.5.0

Operating system: MacOS Sierra, Version 10.12.4
Browser and version: Google Chrome, 57.0.2987.133 (64-bit)

Reproducible Demo
https://github.com/angelbeltran/my-test-app
I added a simple class and generator to src/index.js.
Clone and enter the project, and run
npm start

When the browser opens, you'll get the error message I got.

Interesting...
When a class is in the same file, the following is produced:
var _marked = [someGenerator].map(_regeneratorRuntime.mark);

Without a class, it is transformed correctly:
var _marked = [someGenerator].map(_regenerator2.default.mark);

Both still contain:

	var _regenerator = __webpack_require__(/*! ./~/babel-preset-react-app/~/babel-runtime/regenerator */ 89);

	var _regenerator2 = _interopRequireDefault(_regenerator);

The stray _regeneratorRuntime is the only token with that name in the entire bundle...
Seems like transform-runtime is misbehaving.
I wonder if we can reproduce this with an even more minimal project strictly using the babel chain ...

This looks related. https://github.com/babel/babel/issues/5655

Looks very related, I see the same stray _regeneratorRuntime. 😄

It would be great to check if it still happens on master.

This does not reproduce in master. I’m assuming it was fixed by disabling CommonJS transform.

I could reproduce using<EMAIL_ADDRESS>with the same sample code provided by @angelbeltran. However I could not reproduce on latest master commit (5ecda24f76a0d64aba98f3f5059e25e1a9695ce6) using npm start.

Thanks, guys!


Logs in Cyphondock Logstash

Hi,
I am using  Cyphondock.
Where is this Cyphondock logstash log folder exist .
Usually tail -f /var/log/logstash/* will display the status of pipeline status and forwarding logs from logstash.
But, in case of Cyphodock, there is not folder /var/log/logstash exists.  where can i find this

All the output from Logstash gets sent to stdout of the container, so you can simply do docker-compose logs -f logstash from within the cyphondock directory, or docker logs -f cyphondock_logstash_1 from anywhere else.

Docker-compose logs -f logstash
logstash_1 | [2017-11-12T00:48:52,064][INFO ][org.logstash.beats.BeatsHandler] Exception: org.logstash.beats.BeatsParser$InvalidFrameProtocolException: Invalid Frame Type, received: 1, from: /<IP_ADDRESS>:48737
logstash_1 | Error grabbing logs: invalid character 'l' after object key:value pair.
I have installed TCP output plugin only in dedicated logstash instance, please let me know if I need to install same plugin on cyphondock logstash, also please help me to with the command to install logstash plugin in Docker-compose.

Any updates on this ?

Cyphondock sends container logs to the Logspout container, which sends them to Logstash, which in turns saves them to Elasticsearch — so they should be viewable in Kibana. As for installing the TCP plugin, that's unfortunately beyond our purview. I would consult Logstash or Docker docs for an answer to that one!


No way to set endians?

I have looked at your readme and your code and i don't see a way to specify what endian we want to use. So far your library is the only one that might properly work in a browser but it falls short compared to bufferpack and jspack because it lacks the ability to choose endians.
I honestly hope someone could fully make a 1 file javascript file that mimicks pythons struct module completely and does not use nodes built in buffer library because that's not available in browsers. Bufferpack was good for me when i only needed node but now i need something that also supports browsers that does not use functions that aren't available in a browser.

struct.push and struct.get take a third boolean argument: littleEndian
`
//big endian
var struct = new Struct();
struct.push("H", 10000);
console.log(struct.array); // --> [39, 16]
//little endian
var struct = new Struct();
struct.push("H", 10000, true);
console.log(struct.array); // --> [16, 39]
console.log(struct.get("H", 0)) // --> 4135
console.log(struct.get("H", 0, true)) // --> 10000
`
struct.set is only a syntax-wrapper for DataView prototype setter-methods (setUint8, setInt16, etc), and does not implement handling of endians; probably should.
I will update the README to cover this.

Thank you.


Add setOpenInNewTask() to AppSettingsDialog.Builder

Added features as discussed in #230.
When using setOpenInNewTask() for building the dialog, the app settings screen will be opened using an Intent that has the flag Intent.FLAG_ACTIVITY_NEW_TASK added before calling startActivityForResult(). Hence the settings Activity will not appear as a screen within the task of the activity it has been opened from but instead as a separate task that can be brought to front and back separately by the user.

@sebkur thanks for the submissioN!

I just pushed a new version

@sebkur LGTM, thank you!


KAFKA-6390: Update ZooKeeper to 3.4.11, Gradle and other minor updates

Updates:

Gradle, gradle plugins and maven artifact updated
Bug fix updates for ZooKeeper, Jackson, EasyMock and Snappy

Not updated:

RocksDB as it often causes issues, so better done separately
args4j as our test coverage is weak and the update was a
feature release

Release notes for ZooKeeper 3.4.11:
https://zookeeper.apache.org/doc/r3.4.11/releasenotes.html
A notable fix is improved handling of UnknownHostException:
https://issues.apache.org/jira/browse/ZOOKEEPER-2614
Manually tested that IntelliJ import and build still works.
Relying on existing test suite otherwise.
Committer Checklist (excluded from commit message)

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

Review by @rajinisivaram and cc @junrao and @theduderog for UnknownHostException improvement.

Retest this please


Add user controls for visualization settings

Add user controls for parameters like the time of day and time of year. This change should include a special "settings" tab or sidebar to add additional arbitrary user controls in the future.

Implemented sliders for the time of day and time of year. No "settings" sidebar yet, current plan is to add more user controls to the bottom of the page.


Bump requirement to numpy>=1.16 to fix build fail.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #199 into develop will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #199      +/-   ##
===========================================
- Coverage    88.35%   88.31%   -0.05%
===========================================
  Files           81       81
  Lines         4201     4201
  Branches       699      699
===========================================
- Hits          3712     3710       -2
- Misses         314      316       +2
  Partials       175      175

Impacted Files
Coverage Δ

mxfusion/util/serialization.py
82.85% <0%> (-2.86%)
:arrow_down:

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 c590ffd...748e472. Read the comment docs.


BSG_KSCrashC.h' file not found

I'm trying to install in iOS manually, but I got this error BSG_KSCrashC.h' file not found. I follow this steps:

npm i --save bugsnag-react-native
Add the project in library
add libBugsnagReactNative.a in linked libraries
add pod Bugsnag
pod install

Is there something that I'm missing?

Thanks for the report, @agrass - I suspect that the library may be added incorrectly.
Does it work without steps 4 & 5 (if you remove pod 'BugsnagReactNative')?
What is the exact error message?
What versions of react-native, bugsnag-react-native, and CocoaPods are you using?

Thanks for the response! yes, now is working. It was a little confusing to install it manually, the project is with another project inside and I was following some instructions of the native iOS without RN. Finally worked without steps 4 and 5 and I removing libBusgsnagStatic.a from the linked libraries.


Update scalatest to 3.1.0

Updates org.scalatest:scalatest from 3.0.8 to 3.1.0.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalatest", artifactId = "scalatest" } ]

Applied Migrations

https://raw.githubusercontent.com/scalatest/autofix/e4de53fa40fac423bd64d165ff36bde38ce52388/3.0.x/rules/src/main/scala/org/scalatest/autofix/v3_0_x/RenameDeprecatedPackage.scala
https://raw.githubusercontent.com/scalatest/autofix/e4de53fa40fac423bd64d165ff36bde38ce52388/3.1.x/rules/src/main/scala/org/scalatest/autofix/v3_1_x/RewriteDeprecatedNames.scala

labels: semver-minor, scalafix-migrations

Coverage remained the same at 68.16% when pulling 0ef7adf81b5870ed7e99e032cc8d2e6d0ff07625 on scala-steward:update/scalatest-3.1.0 into f8aaf846f9c1ea7bddec69eeeb2ac05c5349aee5 on evolution-gaming:master.


Need Update for DnD 5e 3.1.2

Maximum system version is set to 2.4.9 so cannot active maodule. Not sure how to manually remove this requirement(like you can with when a maxium version of Foundry is set)

No such plans, sorry.
Closing this as a duplicate of... a lot of other tickets.


Distorted images from mjpeg stream

I have a couple of cameras that don't provide snapshot URLs, a Tapo C310 and a Sonoff GK-200-MP2C. For these I use an ffmpeg mjpeg stream to provide a source for go2rtc's snapshot API, like:
  duck_cam:
    - rtsp://xxxxxx:xxxxxxxxxxxxxxxx@<IP_ADDRESS>:554/stream1
    - ffmpeg:duck_cam#video=mjpeg

I have picture entity cards in Home Assistant displaying snapshots from the cameras.
This works some of the time, but for both cameras, the snapshot images are often distorted. Here is a series of snapshots for one of them:

The snapshots from both cameras are distorted similarly.
The camera in the screenshots above is the Tapo. I've tried setting its RTSP stream to 1920x1080 and 2304x1296, and both result in snapshots with similar artifacts.
Is there some additional fine tuning I should be providing to the ffmpeg stream?

Broken MJPEG for 2K cameras is "normal" situation https://github.com/AlexxIT/go2rtc/issues/83
Are you sure you have problems with FullHD?


ciao: init at 1.19.0

Motivation for this change
Fixes https://github.com/NixOS/nixpkgs/issues/85081.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/162


[admin] merge to live!

Docs Build status updates of commit 190655b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Alias doesn't work for lowercase English words

As shown in the gif, the English alias cannot work properly in Coinc, but the Chinese alias can.

Just run a quick check and can confirm the bug do exists. This happens because of a wrong lowercase/uppercase transition.
For the time being, please unalias your existed alias, and add them back in ALL UPPERCASE.
I'll work on a fix in the next few days.
Thanks for reporting this issue.

Just run a quick check and can confirm the bug do exists. This happens because of a wrong lowercase/uppercase transition.
For the time being, please unalias your existed alias, and add them back in ALL UPPERCASE.
I'll work on a fix in the next few days.
Thanks for reporting this issue.

im glad to help,coinc is excellent plug in alfred!

@EdwinWang0508 Sorry for the delay. I just published the fix in the latest pre-release. Can you help test if everything is working as expected?

@EdwinWang0508 Sorry for the delay. I just published the fix in the latest pre-release. Can you help test if everything is working as expected?

it's my pleasure :)

it works perfectly!but i found another small bug.
when i choose to unalias some words,it will notice that u do unalias that words,but actually not ,it still remains in alias list.

btw,are u traveling in Bangkok？😀

Latest information,when I use this phrase that should have been removed,workflow shows invalid currency.I guess it could be a cache issue?

it works perfectly!but i found another small bug. when i choose to unalias some words,it will notice that u do unalias that words,but actually not ,it still remains in alias list.

That's weird, I can't reproduce the bug in my environment.
Here are some suggestions:

Try opening the source directory and delete the alias.json, if there is one

Delete the settings.json (more details below), then remove and reinstall the workflow

If none of those work, try providing more information so I can diagnose more in-depth.

Type cur workflow:opendata in Alfred, a directory should open with only one settings.json, and paste the content.
Open the Alfred Debugger then repeat the action of the GIFs you provided, and paste the debug logs.

btw,are u traveling in Bangkok？😀

Yep, you can DM me on any social media if you'd like to chat about it 😊

i try to update from alfred gallery,then it gets right!

Yep, you can DM me on any social media if you'd like to chat about it 😊
do u have wechat?
then i found you're taiwanese,i am chinese😁😁

I try to update in alfred gallery,then it goes to normal!

Yep, you can DM me on any social media if you'd like to chat about it 😊

do you have wechat?
I just found you're taiwanese😁i am chinese too!

do you have wechat? I just found you're taiwanese😁i am chinese too!

I have a WeChat account, but please get in touch with other social media listed on my profile, and we can exchange IDs or such privately.
Sorry I'm not feeling great to leave my contacts public. 😅

The bugs seem resolved, I'll close this and release the fix.
You can still comment if you have trouble contacting me. :)


Updated missing translations for Serbian language, latin and cyrilic script

Strings in app/assets/i18n/_missing_translations_sr.json and app/assets/i18n/_missing_translations_sr_Cyrl.json updated.

Thanks


Ensure that all TODO and FIXME comments have associated issues

I have found that standalone FIXME and TODO comments rarely get addressed and are thus not very valuable.
Do one of the following for every FIXME or TODO comment

Address them
Add an issue that tracks the task
Remove the comment if we are not planning to address it

@swift-ci Please test n

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test Windows


path dependencies are ignored from crates.io

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 54ec3cc has been approved by alexcrichton

:hourglass: Testing commit 54ec3cc with merge afac7fc...

:broken_heart: Test failed - cargo-mac-64

@bors: retry
On Wed, Apr 27, 2016 at 10:15 AM, bors<EMAIL_ADDRESS>wrote:

[image: :broken_heart:] Test failed - cargo-mac-64
http://buildbot.rust-lang.org/builders/cargo-mac-64/builds/1576
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/rust-lang/cargo/pull/2625#issuecomment-215154172

:zap: Previous build results for cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64 are reusable. Rebuilding only cargo-mac-64...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Build failing while deploying to Cloudflare

I'm facing issue when I'm deploying it to cloudflare. Local build is running fine. I don't know what's wrong with it

Environment

Operating System: Windows
Nuxt: 3.13.2
Nitro: 2.10.2
Node: v20.18.0

Minimal reproduction
It's my first time, I'm creating the minimal reproduction. Please forgive me if I made some mistake. I have added all the packages I'm using and my nuxt.config.ts file. For reference if I run the build command locally it's building fine. I even tried to downgrade dependencies but there was no success
https://codesandbox.io/p/devbox/festive-volhard-xf53h4
I even upgraded to latest nuxt ( 3.14 ), but still build is failing
Package.json
{
"name": "nuxt-app",
"private": true,
"type": "module",
"scripts": {
"build": "nuxt build",
"dev": "nuxt dev",
"generate": "nuxt generate",
"preview": "nuxt preview",
"postinstall": "nuxt prepare"
},
"dependencies": {
<EMAIL_ADDRESS>"^3.3.1",
"@nuxt/content": "^2.13.1",
"@nuxt/fonts": "^0.7.2",
"@nuxtjs/sitemap": "^6.1.2",
"@prisma/client": "^5.17.0",
"@quasar/extras": "^1.16.12",
"nuxt": "^3.13.2",
"nuxt-auth-utils": "^0.5.1",
"nuxt-quasar-ui": "^2.1.3",
"prisma": "^5.17.0",
"quasar": "^2.16.6",
"sass": "^1.77.8",
"vite": "^5.3.4",
"vue": "latest",
"vue-sonner": "^1.2.1",
"zod": "^3.23.8"
},
"prisma": {
"seed": "node prisma/seed.js"
},
"packageManager"<EMAIL_ADDRESS>"devDependencies": {
"@iconify-json/devicon-plain": "^1.2.4",
"@iconify-json/flowbite": "^1.2.2",
"@iconify-json/heroicons": "^1.2.1",
"@iconify-json/logos": "^1.2.3",
"@iconify-json/mdi": "^1.2.1",
"@iconify-json/pepicons-pop": "^1.2.1",
"@iconify-json/tabler": "^1.2.5",
"@iconify-json/uil": "^1.2.1",
"@nuxt/icon": "^1.6.1"
}
}

a tip: with a minimal repro like this, you can try removing one module after the other to see which one is causing it, and you can set the preset on the command line.
yarn nuxi build --preset cloudflare-pages

you would find that the module which causes this is @nuxt/icon - likely by importing a node version of consola


⚠ Bofrost.nl has degraded performance

In d3d0e47, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1032 ms

Resolved: Bofrost.nl performance has improved in 225cbe9.


Refactor

Consolidated multiple archs to a single Dockerfile
Switched to Debian on all archs
Upgraded PyPy3 to v5.8.0
Dropped the --node entrypoint argument as unnecessary
Dropped the exposed port 8000 since the default configuration does not use it
Dropped the copying of static qemu binaries to images since they're not needed there

Builds complete for all three archs on my workstation.

because you removed the qemu stuff, this is broken

I don't understand. What exactly is broken? If that is so, can I at least fix it?

needs discussion .. see IRC and https://github.com/crossbario/crossbar-docker/issues/48

@agronholm fwiw/fyi: the contents of this repo has moved to the individual projects repos - this here will be closed.


[HOLD for payment 2024-11-29] Not able to submit expense to Expenses - Expensify CA workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @justinpersaud
Slack conversation (hyperlinked to channel name): expensify_migrate
Action Performed:

Go to Expenses - Expensify CA workspace.
Click on + sign
Click global create
Submit expense
Enter amount
Select the workspace Expenses - Expensify CA workspace
Submit the expense

Expected Result:

in step 2 "submit expense" should be available
Expense should be available in the workspace after created

Actual Result:

in step 2 "submit expense" is not available
Expense disappears in the workspace after created

Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @kadiealexander

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

I'm not able to reproduce this (I'm not in the CA workspace and I can't repro with the AU workspace) - @aldo-expensify or @luacmartins would either of you be able to try reproduce this for us?

Yea, I don't have the submit option from within the workspace chat.

Not overdue!

Confirming the specifics of what is and isn't working:

Via quick action button > scan receipt to Expenses - Expensify NL workspace, I get the "hmm it's not here" page ❌
Via the workspace chat, there is not create expense options ❌
Via global create > create expense > scan > add to workspace, does work ✅

In addition to the above, @justinpersaud cannot view his expense created under flow 3 in NewDot, only in OldDot, while mine show in both

I have done some debugging and its because the accounts payable account is the owner of the policy but its not listed in the whitelisted accounts for expensify accounts. We have this check in place so external people cannot request money from Expensify accounts I think

PR up

Confirming this issue has been resolved for me

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production
[ ] 2b. Reported on staging (deploy blocker)
[ ] 2c. Reported on both staging and production
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎


Fix #808

Fixes # .
Added the suggested changes in Title and Description on ReadOnlyComplexTypesAnalyzer
Added two tests : enumerationsDoesNotCreateDiagnostic() and privateEnumerationsDoesNotCreateDiagnostic()

Coverage increased (+0.01%) to 91.596% when pulling 5729fe362e47fab5492bdb6fb0ec2e7449c7d7a8 on cezarant:readOnly into 285c2d1abaddd53e4cb0a0f491e3c27b849d46b2 on code-cracker:master.


:robot: Add rockylinux/fedora to the test matrix

Signed-off-by: mudler<EMAIL_ADDRESS>Change the --cpu flag when running tests. This is a test to see if CI would be impacted by the change

eeeek nope


WinError 2 cannot find file

While doing the installation I get the next error:
DEBUG (build.py): Running nuget install -o .nuget
Traceback (most recent call last):
File "C:\Bitbucket\mediapipe_repo\MediaPipeUnityPlugin\build.py", line 341, in
Argument().command().run()
File "C:\Bitbucket\mediapipe_repo\MediaPipeUnityPlugin\build.py", line 103, in run
self._run_command(self._build_proto_dlls_commands())
File "C:\Bitbucket\mediapipe_repo\MediaPipeUnityPlugin\build.py", line 43, in _run_command
return subprocess.run(command_list, check=True)
File "C:\Users\user\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 505, in run
with Popen(*popenargs, **kwargs) as process:
File "C:\Users\user\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "C:\Users\user\AppData\Local\Programs\Python\Python39\lib\subprocess.py", line 1420, in _execute_child
hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified
Does some know how to fix this?
Command is: python build.py build --desktop gpu -v

First, you cannot build with GPU enabled on Windows, so please run python build.py build --desktop cpu -v.
And if you have not installed nuget yet, please install it (cf. https://github.com/homuler/MediaPipeUnityPlugin#nuget-cli).

Forgot to add the NuGet PATH variable. Also, I needed to restart my command prompt.
Thank you for the advice Homuler.


Attempting to add multiple instruments at the same time breaks pop-up/input fields

To replicate my issue use the Add->Generator->Connect menu, then click Add->Oscilloscope->Connect, without first adding the Generator.
This will make the Add Function Generator pop-up contain 2 sets of input fields, but they are broken.

Easy fix, use unique imgui ID for all "add XX" dialogs. I don't think any other code actually relies on these being single instance dialogs.
Will work on this later today.


@opentelemetry/sdk-trace-base is not available as npm package

I am getting this error report from our npm install:
npm ERR! code E404
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404 It was specified as a dependency of '@opentelemetry/sdk-node'

and am additionally unable to view the package on the website: https://www.npmjs.com/package/@opentelemetry/sdk-trace-base
It looks like you all have been renaming a lot of packages, so I'm hoping there is just a new name and that this wasn't intentionally unpublished?

Ah interesting!
The link to the package is working on npmjs.com now, and I'm able to see 0.25.0 in the version list: https://www.npmjs.com/package/@opentelemetry/sdk-trace-base.
Might have just been a transient issue with npmjs.com...
Thank you!


[Language]The LRU technique for Python's caching

Topic Suggestion
The LRU technique for Python's caching
Proposed article introduction
Just two lines of code are needed to implement LRU caching in Python: import the @lru_cache decorator and declare it. Using examples, we demonstrate how and why it is beneficial to utilize.
The use of caching can dramatically speed up and minimize the burden on computational resources if utilized correctly. Using the Least Recently Used (LRU) method, the output of functions can be cached using a decorator implemented in the Python standard library module.
Key takeaways

Caching strategies can be implemented using decorators
It's important to know what LRU is and how it works
How to use a decorator to speed up an application
After a predetermined amount of time, how to extend the decorator's usefulness and stop caching

Article quality
In this lesson, we will learn about caching techniques, how LRU caching works in Python, how to use the @lru cache decorator, and how the recursive strategy combined with caching helps to solve the problem rapidly. The article will be comprehensive, and the code provided will be simple to grasp.
References
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Please fill in the Article Quality section so we can review your article.

@louisefindlay23 done

Hi. Your article quality section is more about what you're writing about rather than what makes your article unique? Can you tell me what aspects of your article are not covered elsewhere or your unique perspective that you plan to bring?

@louisefindlay23 The already published articles only cover the introductory part on how caching works. My article will include more and the distinguishing part is that I will include a section on how the recursive approach in combination with caching helps to solve the problems quickly enough. Thanks.

I'm going to close this topic due to inactivity.
It can be reopened.

I'm going to close this topic due to inactivity. It can be reopened.

@ahmadmardeni1 please open this topic. Thanks.


AWS CodeCommit Support

It looks like the plugin supports the following for syncing to Git source control:
Local storage (default)
URL
JSONBin.io
GitHub
GitLab
Azure DevOps
Supernova
Generic Versioned Storage
BitBucket (Planned)
Would it be possible to add AWS CodeCommit to the list of planned support? Thanks!

This request has been moved to our new feedback tool. Go check out https://tokensstudio.featurebase.app/ to create a free account. You can upvote and comment on existing posts, make a new request, and subscribe to be notified when requests move into our roadmap.

Added to Featurebase Roadmap : ↕ AWS CodeCommit - Sync support request


cityscapes backbone

May I ask which backbone you use to train cityscpes for semantic segmentation tasks

Hi,
We have tried CAST-S and CAST-SD on cityscapes.  Please check section A.3 in the paper.

My backbone uses deit_cast_stall_deep, and the pre trained model has a maximum of 2GB model checkpoint_0099.pth.tar. Why is it that after training on the Cityscapes dataset for 400 epochs, miou can only reach 58%？


fix: enable debug logging for cli

Analytics implementation broke debug logging.
This PR fixes it

@craicoverflow Merging this as I want it to be enabled in other PR to help you with verification


STT Training Paramaters

This is a great tool!  One thing I did not see is a way to apply STT training paramaters:  word_type_to_add, customization_weight, strict.  Would be great to have that capability.

Under consideration

Under consideration


cancelTaskForImageDownloadReceipt doesn't really cancel task

Hi,
I'm struggling with canceling image requests when using AFImageDownloader in AFNetworking 3.0.4.
When UIImageView+AFNetworking try's cancelImageDownloadTask then AFImageDownloader doesn't really cancel task. Problem is in following if condition:
        if (mergedTask.responseHandlers.count == 0 && mergedTask.task.state == NSURLSessionTaskStateSuspended) {
            [mergedTask.task cancel];
        }

Having following NSLog clears the situation:
<EMAIL_ADDRESS>%d, mergedTask.task.state: %d", mergedTask.responseHandlers.count, mergedTask.task.state);
It prints out this:

mergedTask.responseHandlers.count: 0, mergedTask.task.state: 0

NSURLSessionTaskState 0 is NSURLSessionTaskStateRunning which should be canceled but it isn't. Is this done by some reason because I would like to cancel all running image requests when UICollectionView makes dequeue process.

Hi @josipbernat
As documented here, the image downloader is optimized to let requests currently in flight finish out, and get their response into the cache, so if you come back around in your dequeueable views, you don't have to issue another request. If the request is pending, it is actually cancelled.


Localizing UI text

Add internationalization (i18n) support to the project, so that we can translate user interface texts. Once the i18n package is configured, determine how to follow-up with user interface internationalization, including the following

menu texts
form texts

Definition of Done
TBD

@Nazarah I added a description to this task.

I think we should add this i18n support for the 1.0 release. We can include Suomi localization as part of that release, or wait for 1.1.0 release. By the way, do we want to use semantic versioning starting at version 1.0?

The Aurelia i18n guide expects localizations to be served by a backend. That might be problematic, since this is a client-only project. Lets see if there is another way to provide the localizations, and consider the bundle size if they are included with the client code.

E.g. might we be able to host i18n files on a CDN or some dedicated location?

I think it's possible to load them from anywhere. These docs backend: {                                  // <-- configure backend settings           loadPath: './locales/{{lng}}/{{ns}}.json', // <-- XHR settings for where to get the files from         }, make it look like the json for localization can be hosted anywhere.


Changes to groupby.size() don't work for all possible groupby

Describe the bug
import pandas as pd
ddf = pd.DataFrame({"a": [1, 1, 2], "b": [2, 3, 2]})
reveal_type(ddf.groupby(["a", "b"]).size())

pyright reports
 - information: Type of "ddf.groupby(["a", "b"]).size()" is "Unknown"
  c:\Code\pandas-stubs\play\issue1044.py:3:37 - error: Cannot access attribute "size" for class "DataFrameGroupBy[tuple[Unknown, ...], bool]"
    Could not bind method "size" because "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to parameter "self"
      "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to "DataFrameGroupBy[tuple[Unknown, ...], Literal[True]]"
        Type parameter "_TT@DataFrameGroupBy" is invariant, but "bool" is not the same as "Literal[True]"
    Could not bind method "size" because "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to parameter "self"
      "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to "DataFrameGroupBy[tuple[Unknown, ...], Literal[False]]"
        Type parameter "_TT@DataFrameGroupBy" is invariant, but "bool" is not the same as "Literal[False]"
    Could not bind method "size" because "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to parameter "self"
      "DataFrameGroupBy[tuple[Unknown, ...], bool]" is not assignable to "DataFrameGroupBy[Timestamp, Literal[True]]" (reportAttributeAccessIssue)

mypy reports
note: Revealed type is "Any"

The fix here is that any of the methods in frame.pyi for def groupby() that return DataFrameGroupBy[SOMETYPE, bool] need to be split into 2 where there is one overload that returns DataFrameGroupBy[SOMETYPE, True] and another that returns DataFrameGroupBy[SOMETYPE, False] like is done with DataFrameGroupBy[Scalar, True] and DataFrameGroupBy[Scalar, False] .
Please complete the following information:

OS:  Windows 11
python version   3.10
version of type checker   pyright 1.1.389, mypy 1.13.0
version of installed pandas-stubs:   Dev version

Additional context
This was missed in PR #1014 by @loicdiridollou .  Hopefully he can fix soon, since the prerelease of VS Code pylance has this bug.

Sorry for that one, PR should be up.


Susy 2.2.2 / node-sass: Span(full)

Hi, I am not sure if this belongs here or in the node-sass repo. Currently I moved over to latest node-sass everything is working fine for the most part, but we get a error when we do this...
@include span(full)
Error message: may only compare numbers
Now we actually have to define it by numbers, ex. 12 of 12, 6 of 12 for it to compile. Was this dropped in support or could possibly be a node-sass issue?
Thanks!

I was able to reproduce this, but we'll need to track it down. Likely a libsass bug, since it works fine in Ruby Sass.

The same error exists on my side. It would be awesome if the error can be found. :+1:

@DenisMir I confirm that if I wrap my gutters in ( ), I am able to use @include span(full) and I get no error.

With the latest node-sass version which uses libsass in version 3.2.0-beta.2 I don't even need the () wrap around my gutters. But the main problem remains. The span mixin eats up all the compile performance. When dropping all the span calls I'm down at a 10-100ms (depending on the machine) but adding the span calls the compile time gets up to over 2 seconds.

Yes, that was a well-documented LibSass bug, that has been fixed. Cheers!


tell the excecutor to poll until wait_ready returns Ready

fixes #11
@averyanalex could you check if this works for you?

Thanks for your help!

Thanks for your help!

I thinked a bit about timers... maybe pass async fn to async pn532 constructor? And run this user-provided function after every ready check?


Can the AI start speaking before the human when using server_vad?

Hi, I'm wondering if there is a way to use the available API calls to start a new conversation, and have the AI immediately say hello to the other person? Most demos I've seen show a person saying something first, and then the AI responding, rather than starting with the AI saying something.
Ideally, I'd like to create a new conversation, submit my system prompt, and have some flag like AI Immediately Starts Speaking, such that I start receiving streaming audio from the Realtime API as soon as the web socket connection is established and the session is created. Notably, I would still like to use server_vad turn detection -- I just want to start at the AI's turn.
If this is possible, I would greatly appreciate advice on how to set it up! And if not, would love to submit that as a feature request :)

client.sendUserMessageContent([{ type: 'input_text', text: Introduce yourself. }]);

Comments here provide the solution. Thanks for asking!


Process Manager and Seat Assignments Aggregate

Is based on PR #11.

Current coverage is 69.20%

Branch #12 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

@alexander-yevsyukov, PTAL (proc manager is covered 50% now).

PTAL.

Done, can I merge it now?

LGTM


Spotify search not working

I have a Spotify premium account installed and configured on my iPhone X. It works properly.
From AnimojiStudio when I click "Configure Karaoke", it opens Spotify, puts a big checkmark across the screen, then goes to the search screen.  No matter what I type in the search screen I get no results, no messages, no errors, no feedback, no nothing.
If I search for a song in Spotify without studio, it works fine.

I am also having this issue

Let's keep Spotify search/auth issues discussion on #17


Unknown entity name: RTItemProjectile-deadlock-stack-salt18

I get the following error:
105.020 Error MainLoop.cpp:1288: Exception at tick 19774: The mod Renai Transportation (1.2.2) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event RenaiTransportation::on_nth_tick(3)
Unknown entity name: RTItemProjectile-deadlock-stack-salt18
stack traceback:
[C]: in function 'create_entity'
RenaiTransportation/control.lua:283: in function <RenaiTransportation/control.lua:75>
111.449 Quitting: user-quit.
112.034 Goodbye
Tell me if you need the mod which is not on the modportal
factorio-current.log
crash renai.zip

It looks like the deadlock-stack-salt is causing the issue. What mod does it come from? I have an idea of what it might be, but I'd like to see how that item is coded

the root object comes from Salt & chorine by brevven and i think deadlock's stacking beltboxes & compact loaders creates it dynamicly

work with the placeholder graphics. i guess the mod load order has something to do with if i judge the changelog right.


OAuth authorization force_verify not working

In OAuth Authorizations there's a field to specify if you want to force user verification, as Twitch docs says:

Specifies whether the user should be re-prompted for authorization. If this is true, the user always is prompted to confirm authorization. This is useful to allow your users to switch Twitch accounts, since there is no way to log users out of the API. Default: false (a given user sees the authorization page for a given set of scopes only the first time through the sequence).

This is not working propertly as PHP is converting the "true" and "false" values to the integers 1 and 0 and what Twitch expects are values as the strings true or false.

Oh, I would have never though this would be problem. Fixed in 2.2.2

Thanks for the fast fix!
Updated to 2.2.2 and working as expected.


fefactor(behavior_path_planner): use debug_marker of interface for pull_over

Signed-off-by: kosuke55<EMAIL_ADDRESS>Description

use debug_marker of interface /planning/scenario_planning/lane_driving/behavior_planning/behavior_path_planner/debug/pullover for pull_over.

Related links

https://github.com/autowarefoundation/autoware.universe/pull/1711
Tests performed

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

[ ] remove old rviz configurations.


Handle json schemas, validate artifact descriptors/definitions.

See LaxarJS/laxar#380: laxar-tooling should handle validation during build-time.
There should be an API to validate the collected artifacts before they are subsequently written to the artifact listing and serialized.
JSON Schema files should be loaded from laxar (to make sure the correct version is used), which possibly requires them to be converted (back) to JSON instead of JS. This also makes theme more easily consumable for some clients.

Upgrade Information
The artifactCollector has been extended to collect schemas, the same way it collects other artifacts. For now, only the flow, page and widget schema files from LaxarJS are collected (and only if there are artifacts of that type present). Schemas are loaded via the readJson callback, so they should be stored as JSON files.
A new setting schemas has been added to the paths object. Its default value is laxar/static/schemas.
A new API artifactValidator has been added. It takes the output of the artifactCollector, compiles the collected schemas with Ajv and validates flow and page definitions and widget descriptors, extending the validated objects with defaults.

Implemented on master (v0.5.0)


Fix issue Print pattern X with stars in c language #4

This should fix issue #4
Thanks @rahulparshi !

your output for Main.c file
*******   *    *
*          *  *
*           **
*******     **
*          *  *
*         *    *
*******  *      *

Expected output is
*******  *     *
*         *   *
*          * *
*******     *
*          * *
*         *   *
*******  *     *

In Main.c start inner for loop from j=1 and try modifying the conditions. It might help you to fix this.

@s37syed are you able to finish this pull request?.


[Bug]: samples won't work in Android Studio Flamingo because kotlinOptions.jvmTarget isn't set

Is there an existing issue for this?

[X] I have searched the existing issues

Is there a StackOverflow question about this issue?

[X] I have searched StackOverflow

Is this an issue related to one of the samples?

[X] Yes, this is a specific issue related to this samples repo.

Sample app
JetNews
What happened?
When upgrading any sample to kotlin 1.8.0 or AGP 8.0.0:
Execution failed for task ':app:compileDebugKotlin'.
> 'compileDebugJavaWithJavac' task (current target is 1.8) and 'compileDebugKotlin' task (current target is 17) jvm target compatibility should be set to the same Java version.
  Consider using JVM toolchain: https://kotl.in/gradle/jvm/toolchain

This could be solved by adding this to app/build.gradle.kts:
    kotlinOptions {
        jvmTarget = '1.8'
    }

Or by changing compileOptions.targetCompatibility to JavaVersion.VERSION_17
Relevant logcat output
No response
Code of Conduct

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

CC https://github.com/android/compose-samples/pull/1114

'compileProdReleaseJavaWithJavac' task (current target is 1.8) and 'kaptGenerateStubsProdReleaseKotlin' task (current target is 17) jvm target compatibility should be set to the same Java version.
Nothing is working for me.

kotlin{
IjvmToolchain(18)
}
above code did not help

You just need to have the same version of kotlin JDK and Java target JDK. The exact fix may vary depending on which gradle task you are changing.
Here is an example https://github.com/android/compose-samples/pull/1110

this answer is useless I tried all the way nothing is working
Caused by: org.gradle.api.GradleException: 'compileProdReleaseJavaWithJavac' task (current target is 1.7) and 'kaptGenerateStubsProdReleaseKotlin' task (current target is 17) jvm target compatibility should be set to the same Java version.
now both versions and error is also saying the same yet error exist.

I tried all the way nothing is working
Caused by: org.gradle.api.GradleException: 'compileProdReleaseJavaWithJavac' task (current target is 1.8) and 'kaptGenerateStubsProdReleaseKotlin' task (current target is 18) jvm target compatibility should be set to the same Java version.
Could you share where and what is to be change?
this code is already written in gradle
compileOptions { sourceCompatibility JavaVersion.VERSION_1_8 targetCompatibility JavaVersion.VERSION_1_8 } kotlinOptions { jvmTarget = '1.8' }

I have the same configuration in most of my modules and I still get this error after upgrading to Flamingo. What are we supposed to do?

https://github.com/android/compose-samples/pull/1114 has been merged, could you pull from latest main and confirm you're able to run samples in flamingo? TY

Removing
kotlinOptions {
        jvmTarget = '1.8'
    }

And setting
compileOptions {
sourceCompatibility JavaVersion.VERSION_17
targetCompatibility JavaVersion.VERSION_17
}
Worked for me! Had this issue straight after updating to Android Studio Flamingo.

Is it safe to just target version 17 and 18 in a production app? Does it depend on what Android API you are targeting? The app I work on targets JDK 1.8 because it's what Hilt's documentation recommends. I would really like to understand what the proper solution for this kind of issue is. According to this thread there has been a change in kapt that no longer picks up the jvmTarget setting. Therefore, the workaround I went with is:
allprojects {
    tasks.withType(org.jetbrains.kotlin.gradle.tasks.KotlinCompile) {
        kotlinOptions.jvmTarget = "1.8"
    }
}

I am just not confident changing the source and target compatibility settings or making use of Gradle's JVM toolchain are the right solutions.

I advise everyone using samples to take the latest version for main, since it sets JavaVersion.VERSION_17. Otherwise you can set it yourself for a project and remove kotlinOptions jvmTarget, because it uses VERSION_17 by default with the latest AGP 8.0.


Waiting lists

Add waiting list information to the boundary report

Was this fully completed in your last commit (0b81da6) @ox-baloo?

Yep sorry - resolved in (https://github.com/the-scouts/geo_mapping/commit/0b81da66a2da3a877f724ac52bd8bff69538a8ca)


Shared icon doesn't disappear in chat model for user1 if the chat is deleted from "Shared with me" by user2

EPAM AI DIAL chat version
0.6.11
What happened?
Preparation:

By user1 prepare shared link for a chat
By user2 click on the link

Steps:

By user2 delete the chat from "Shared with me"
By user1 refresh the browser and check model icon ->  Shared icon is still on model -> bug

Expected result: Shared icon disappears when all the users (which whom it was shared) delete the chat from "Shared with me" structure
Note: Please re-check for Prompts. Folders chats and folders prompts. There is the same issue!
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully. Shared icon disappears from folder/model icon when all the users delete chat/prompt/folder from "Shared with me".


Httpclient gzipped response handling

Description
This version perform the logging mechanism of gzip content responses.
Motivation and Context
Related Known Issue:
"The Logbook HTTP Client integration is handling gzip-compressed response entities incorrectly if the interceptor runs before a decompressing interceptor. Since logging compressed contents is not really helpful it's advised to register the logbook interceptor as the last interceptor in the chain."
Types of changes

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

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

:+1:

@lcostela thank you for your contribution. It'll be part of the next release


Investigate Error messages during test about not being able to update status

Tests are passing, but I consistently see these kinds of messages there that seem a bit troubling. Seems like either the tests are not set up right, or ?? But they add noise to the tests and it would be good to understand if they are real errors, cornercases, or whatever.
=== RUN   TestBasicFlowsSync
[restful] 2017/10/10 15:11:34 log.go:33: [restful/swagger] listing is available at https:///swaggerapi
[restful] 2017/10/10 15:11:34 log.go:33: [restful/swagger] https:///swaggerui/ is mapped to folder /swagger-ui/
E1010 15:11:39.610161    5909 controller_instance.go:1524] ServiceInstance "test-namespace/test-instance": Failed to update status: Operation cannot be fulfilled on serviceinstances.servicecatalog.k8s.io "test-instance": the object has been modified; please apply your changes to the latest version and try again
E1010 15:11:41.032099    5909 controller_binding.go:895] Error updating status for ServiceBinding test-namespace/test-binding
E1010 15:11:46.405983    5909 controller_broker.go:727] ClusterServiceBroker "test-broker": Error updating ready condition: Operation cannot be fulfilled on clusterservicebrokers.servicecatalog.k8s.io "test-broker": StorageError: invalid object, Code: 4, Key: /ce5cb960-ed2b-4d4e-a196-50e5fec13068/clusterservicebrokers/test-broker, ResourceVersion: 0, AdditionalErrorMsg: Precondition failed: UID in precondition: 4e9a4567-adcd-11e7-a2c1-0242c0a80906, UID in object meta:
--- PASS: TestBasicFlowsSync (14.90s)
framework.go:76: Starting server on port: 31375
controller_test.go:1066: controller start
controller_test.go:1078: informers start
framework.go:72: Shutting down server on port: 31375
=== RUN   TestBasicFlowsAsync
[restful] 2017/10/10 15:11:53 log.go:33: [restful/swagger] listing is available at https:///swaggerapi
[restful] 2017/10/10 15:11:53 log.go:33: [restful/swagger] https:///swaggerui/ is mapped to folder /swagger-ui/
E1010 15:11:56.265343    5909 controller_instance.go:1524] ServiceInstance "test-namespace/test-instance": Failed to update status: Operation cannot be fulfilled on serviceinstances.servicecatalog.k8s.io "test-instance": the object has been modified; please apply your changes to the latest version and try again
E1010 15:11:58.065823    5909 controller_instance.go:1524] ServiceInstance "test-namespace/test-instance": Failed to update status: Operation cannot be fulfilled on serviceinstances.servicecatalog.k8s.io "test-instance": the object has been modified; please apply your changes to the latest version and try again
E1010 15:11:59.866583    5909 controller_binding.go:895] Error updating status for ServiceBinding test-namespace/test-binding
E1010 15:12:01.069344    5909 controller_binding.go:895] Error updating status for ServiceBinding test-namespace/test-binding
=== RUN   TestBasicFlowsWithOriginatingIdentity
[restful] 2017/10/10 15:12:26 log.go:33: [restful/swagger] listing is available at https:///swaggerapi
[restful] 2017/10/10 15:12:26 log.go:33: [restful/swagger] https:///swaggerui/ is mapped to folder /swagger-ui/
E1010 15:12:28.782039    5909 controller_instance.go:1524] ServiceInstance "test-namespace/test-instance": Failed to update status: Operation cannot be fulfilled on serviceinstances.servicecatalog.k8s.io "test-instance": the object has been modified; please apply your changes to the latest version and try again
--- PASS: TestBasicFlowsWithOriginatingIdentity (5.33s)
framework.go:76: Starting server on port: 26481
controller_test.go:1066: controller start
controller_test.go:1078: informers start
framework.go:72: Shutting down server on port: 26481

I think what we're seeing is races between the work queue in the controller and the informer cache used by the lister not being updated

@pmorie You are spot on with what the underlying cause of those errors are. They come as a result of updating the status multiple times during a single reconciliation loop. When the reconciliation loop ends, the next reconciliation loop may try to reconcile the resource in its state from the first status update. This reconciliation then fails because the resource being used is out-of-date.

Most of these are now fixed by #1779 and #1819, but the integration tests still print out this error in a few places. The cause of those is legit: the test is deleting the resource in the middle of a reconciliation loop iteration:
I0312 14:21:09.142127    4982 controller_binding.go:192] ServiceBinding "test-namespace/test-binding": Processing
I0312 14:21:09.142147    4982 controller_binding.go:538] ServiceBinding "test-namespace/test-binding": References a non-existent ServiceInstance "test-namespace/nothereinstance"
I0312 14:21:09.142157    4982 controller_binding.go:588] ServiceBinding "test-namespace/test-binding": Updating status
I0312 14:21:09.340507    4982 request.go:480] Throttling request took 398.676693ms, request: DELETE:https://localhost:13299/apis/servicecatalog.k8s.io/v1beta1/namespaces/test-namespace/servicebindings/test-binding
I0312 14:21:09.348214    4982 wrap.go:42] DELETE /apis/servicecatalog.k8s.io/v1beta1/namespaces/test-namespace/servicebindings/test-binding: (7.459368ms) 200 [[___TestCreateServiceBindingInvalidInstance_in_github_com_kubernetes_incubator_service_catalog_test_integration/v0.0.0 (linux/amd64) kubernetes/$Format] [::1]:42182]
I0312 14:21:09.540506    4982 request.go:480] Throttling request took 398.27679ms, request: PUT:https://localhost:13299/apis/servicecatalog.k8s.io/v1beta1/namespaces/test-namespace/servicebindings/test-binding/status
I0312 14:21:09.541676    4982 wrap.go:42] PUT /apis/servicecatalog.k8s.io/v1beta1/namespaces/test-namespace/servicebindings/test-binding/status: (901.963µs) 409 [[___TestCreateServiceBindingInvalidInstance_in_github_com_kubernetes_incubator_service_catalog_test_integration/v0.0.0 (linux/amd64) kubernetes/$Format] [::1]:42182]
E0312 14:21:09.541945    4982 controller_binding.go:591] ServiceBinding "test-namespace/test-binding": Error updating status: Operation cannot be fulfilled on servicebindings.servicecatalog.k8s.io "test-binding": the object has been modified; please apply your changes to the latest version and try again

These log statements are logged when there is a problem with the instance/binding (such as pointing to a nonexistent broker/instance). The controller initially re-tries this multiple times per second and posts a new status every time, even though the status is exactly the same as the current status. Do we want to check this before posting a status (non)update?  Looks like the API server doesn't increase the resource version if you post a status update that doesn't actually change the status.
And one more things: the log statement the object has been modified; please apply your changes to the latest version and try again should probably be logged as INFO instead of ERROR, because it's a fairly normal event (it occurs every time someone updates/deletes the resource while the controller is in the middle of processing the resource). We probably want to use the ERROR level for actual errors, right?

The controller initially re-tries this multiple times per second and posts a new status every time, even though the status is exactly the same as the current status. Do we want to check this before posting a status (non)update?

Yes, the controller should avoid sending no-op updates

Looks like the API server doesn't even increase the resource version if you post a status update that doesn't actually change the status.

Correct, that is working as designed

seems fixed
/close


GeoJSON render styles

I have implemented GeoJSON data with google-maps-ios-utils, but how could I config the rendering styles?
In the documentation of Google Maps APIs, there is seems no example of "GMUStyles" configuration setting.
I tried to setup as following but it's not working on styles change. (geometry features is working well)
----my codes example start here----
geoJsonParser = GMUGeoJSONParser(data: jsonData)
geoJsonParser.parse()
renderer =  GMUGeometryRenderer(map: mapView, geometries: geoJsonParser.features, styles:
[GMUStyle(styleID: "111", stroke: UIColor.red, fill: UIColor.red, width: 0.8, scale: 1.0, heading: 1.0, anchor: CGPoint(x: 0, y: 0), iconUrl: "non", title: "title", hasFill: false, hasStroke: true)])
renderer.render()
----my codes example end here----

GeoJSON does not have a concept of styles like KML so you will need to manually associate the style object to a feature. You could loop through geoJsonParser.features and assign it's .style property to the GMUStyle object.

Thank you @mountainvat, would you help explain how to associate the style object further?
I found that each element in geoJsonParser.feature.style is get-only property which means I cannot assign values to them. e.g. geoJsonParser.features[i].style.strokeColor = UIColor.red
Whether there is a normal style format in geojson text structure so that I can retrieve each .style object directly as the GMUStyle object?

The GMUStyle object is immutable but the property GMUGeometryContainer.style is not. You can assign that property to the style object (https://github.com/googlemaps/google-maps-ios-utils/blob/master/src/Geometry/Model/GMUGeometryContainer.h#L36). GeoJSON does not support styles in its format (at least at the time of writing the library), that's why you'll have to set them manually.

Yeah, I got it finally.
It's really good explanation so that I could understand the model more.
Many thanks to @mountainvat.

Just in case someone else comes across this, I did the following to get it to work:
let path = Bundle.main.path(forResource: "countries", ofType: "json")
let url = URL(fileURLWithPath: path!)
let geoJsonParser = GMUGeoJSONParser(url: url)
geoJsonParser.parse()

let style = GMUStyle(styleID: "random", stroke: UIColor.blue, fill: UIColor.orange.withAlphaComponent(0.5), width: 2, scale: 1, heading: 0, anchor: CGPoint(x: 0, y: 0), iconUrl: nil, title: nil, hasFill: true, hasStroke: true)

for feature in geoJsonParser.features {
  feature.style = style
}

renderer = GMUGeometryRenderer(map: gmapView, geometries: geoJsonParser.features)
renderer.render()

let style = GMUStyle(styleID: "random", stroke: .red, fill: .clear, width: 1, scale: 1, heading: 0, anchor: CGPoint(x: 0, y: 0), iconUrl: nil, title: "Test", hasFill: false, hasStroke: true)

disable fill not working? Plz help

hello. I fixed it by modifying GMUGeoJSONParser class, just set default for Polyline fill color. Plz help my question: I want to highlight geojson feature on click. I dont know the way to do it


Manual code review in RDFMapper class

Manual code review has been done on RDFMapper class. Documentation is also extended with the findings. Necessary refactoring has been done according to the review.

There was an error actually in the build. I will fix it with another pr.


[Feature Request] Mapping over compatible trees

Required prerequisites

[X] I have searched the Issue Tracker that this hasn't already been reported. (comment there if it has.)

Motivation
I often work with nested containers that constitute partially available data. It seems that mapping these in conjunction with other compatible trees is not currently supported by the library.
Simple example:
a = {"one": 1, "two": 2, "three": 3}
b = {"two": 2, "three": 3}

def add(x, y):
    return x + y

optree.tree_map(add, a, b)  # ValueError

It would be nice if we had support for mapping over compatible trees. In particular, I would consider trees compatible when each leaf node of tree b has a corresponding leaf node in a where the path to both is identical.
It might be safer to only enable this functionality with an extra argument or in another function than the plain tree_map, however.
Solution
Referring to my example above, I would expect the solution:
{"one": 1, "two": 4, "three": 6}

Alternatives
Allowing us to broadcast trees would also solve this:
a, b = optree.tree_broadcast(a, b)

b
>>> {"one": None, "two": 2, "three": 3}

optree.tree_map(add, a, b)  # works fine already with these inputs (ignoring the None edge-case)

Alternatively, we could also work around this particular problem by using the tree paths or accessors in a generic way:
def add_to_a(accessor_or_path, value):
    # something like this tree_get which I think does not exist yet
    # an in-place method would also be nice
    result = optree.tree_get(a, accessor_or_path) + value
    return value

optree.tree_map_with_path(add_to_a, b)
# or
optree.tree_map_with_accessor(add_to_a, b)

However, this might be highly inefficient.
Additional context
Perhaps this functionality (or a work-around) already exists. I would be happy to be educated.

Hi @LarsKue, could you elaborate more on the definition of "compatible trees"? A tree can be a nested collection of list/tuple/dict/namedtuple etc., not only nested dicts.
I think the following code could be a workaround for this feature:
import optree

def tree_get(accessor, tree, default=None):
    try:
        return accessor(tree)
    except LookupError:
        return default

tree_x = {"one": 1, "two": 2, "three": 3}
tree_y = {"two": 2, "three": 3}

result = optree.tree_map_with_accessor(lambda a, x: x + tree_get(a, tree_y, default=0), tree_x)
# {'one': 1, 'two': 4, 'three': 6}

# What's the expected output of some_tree_add(tree_x, tree_y) and some_tree_add(tree_y, tree_x)?

@XuehaiPan Thank you for the work-around, this already looks great.
I think there are many good definitions for a compatible tree. In particular, consider the set of leaf nodes, along with their paths (nested dict keys, tuple indices, etc.) in the respective trees. We could consider tree A a supertree of tree B if A's leaf node set is a superset of B's. B is then a subtree of A.
Currently, tree_map only works on trees where these sets are exactly identical, which is a little bit limiting. Ideally, we could specify a supertree and allow mapping over any subtree, or simply map over the intersection of both leaf node sets.
Example: List in Dict
a = {"a": [1, 2, 3], "b": 3}
b = {"a": [1, 2, 3, 4], "c": 4}

# the leaf node sets are
# a: {("a", 0), ("a", 1), ("a", 2), "b"}
# b: {("a", 0), ("a", 1), ("a", 2), ("a", 3), "c"}

add_intersection(a, b)
>>> {"a": [2, 4, 6]}  # note that we only return leaf nodes from the intersection of the leaf node set

# the final leaf node set is
# {("a", 0), ("a", 1), ("a", 2)}

I think there are many good definitions for a compatible tree.

In optree and also JAX PyTree / PyTorch PyTree / DM-Tree, a tree X and its compatible tree Y are that the former one is the prefix tree of the latter one. That is if we replace the leaf node in tree X with some subtree or a leaf, the structure of the new tree can be identical to tree Y.
prefix_tree = {'a': 1, 'b': (2, 3), 'c': {'d': 4, 'e': 5}}
full_tree   = {'a': (1, 1, 1), 'b': ([2, 2], 3), 'c': {'d': 4, 'e': {'f': 5, 'g': 5}}}

We can see that the path of the prefix_tree is a prefix of the path of the full tree.
optree.tree_paths(prefix_tree)
# [
#     ('a',),
#     ('b', 0),
#     ('b', 1),
#     ('c', 'd'),
#     ('c', 'e'),
# ]
# vs.
optree.tree_paths(full_tree)
# [
#     ('a', 0),
#     ('a', 1),
#     ('a', 2),
#     ('b', 0, 0),
#     ('b', 0, 1),
#     ('b', 1),
#     ('c', 'd'),
#     ('c', 'e', 'f'),
#     ('c', 'e', 'g'),
# ]

In particular, consider the set of leaf nodes, along with their paths (nested dict keys, tuple indices, etc.) in the respective trees. We could consider tree A a supertree of tree B if A's leaf node set is a superset of B's. B is then a subtree of A.

This definition will change the node arity of the node. The interaction of {'a': 1, 'b': 2, 'c': 3} / {'a': 1, 'c': 3, 'd': 4, 'e': 5} will get key set {'a', 'c'}. The input dicts have arity 3 and 4, but the result has arity 2.


Remove the underscore from _baseweb.scss

The underscore is only intended for Sass partials. baseweb.scss is where are the partials are collected, so the underscore is not needed.

This update was fixed in the v2.2.2 release.


Make floating button for Add Component/Save Changes

The Add/Edit Component (soon to be Save Changes #143 ) is often overlooked since it is farther down the form and past sometimes optional content. Let's see about adding a floating button in react.
If this is too much work or doesn't behave great, we'll see about changing the behavior of Save Project.

This was pushed to the dev branch. It doesn't look wonderful, but hopefully it helps!

@brettfiedler does this seem OK, any design requests or anything that would make this button stand out more?

I think the biggest design need here is for us to pick a better color scheme 😄 At the very least we need a third "accent" color. Might think about leaning into darker shades of the "dot colors"

Let me think about this and get back to this.
In the meantime, maybe we make the button just a hair bigger? Maybe a 25% increase overall?

OK! The button is 25% larger. Back to you for next steps or if we should make a new issue for a new color profile!

This was merged into main.


DBZ-5615 Mask additional credentials during log events

https://issues.redhat.com/browse/DBZ-5615

@Naros Applied, thanks!


API - Phyton 400 error when running a Product Database Query in DE marketplace

Describe the bug
The customer is getting a 400 error when running a PD endpoint query on Phyton. The parameters are correct, and it appears to be isolated to the DE marketplace only
Paulo has confirmed the bug.
API code & error:
HTTP ERROR: 400 Client Error: Bad Request for URL:
Troubleshooting steps
N/A
Screenshots:

Additional information (optional)
Add any other context about the problem here.

Hi @franzJS00! Thanks for submitting this bug report. Commenting to let you know we've received it and we're looking into it.

@franzJS00 let us know if your issue is resolved in v0.2.0.


Support for creating Initializers

We should have a apiserver-boot create initializer <name> --group <group> --version <version> --kind <kind>  command to bootstrap an initializer for a given resource.

@vaikas-google You working on this?


Add throwing inaccuracy

About the PR

Made throwing have a lower cooldown but also inaccuracy
Why / Balance

Because throwing a lot of things at once is usually useful, but doing it with perfect accuracy on each throw doesn't make a lot of sense.
Technical details

Rotates the throw vector depending on how long it has been since the last throw
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/45953835/70834edf-15f7-4350-ab97-6ccece72d667
Breaking changes

Changelog

:cl:

add: Throwing now has inaccuracy if you throw faster than twice per second.
tweak: Throwing now has a cooldown of 0.125 seconds, down from 0.5 seconds.

what's the purpose in reducing the cooldown from 0.5 -> 0.125? It was implemented specifically to mitigate throwing abuse. Yes, there's an accuracy penalty, but you could still spam throw objects in close combat to reliably hit way quicker than you could otherwise.

what's the purpose in reducing the cooldown from 0.5 -> 0.125? It was implemented specifically to mitigate throwing abuse. Yes, there's an accuracy penalty, but you could still spam throw objects in close combat to reliably hit way quicker than you could otherwise.

cooldown was reduced so there wasn't just No Penalty (since it gets perfect accuracy at 0.5s)
i can raise it if need be

Raise the cooldown to 0.5 seconds and remove the accuracy penalty.

Raise the cooldown to 0.5 seconds and remove the accuracy penalty.

this would literally just be closing the pr

For the janitor's sanity, if this gets added the random chance to miss the disposal bin should probably be removed.

For the janitor's sanity, if this gets added the random chance to miss the disposal bin should probably be removed.

doesn't apply to single throws :trollface:

While I don't agree with the initial throw cooldown, it's there as a bandaid to prevent spam throwing exactly like this.
I still believe having a throw mode and having a stamina cost per throw (with a more clear stamina meter) is a better solution. Since throw mode also allows you to catch things.
Inaccuracy doesn't work too well when the gap is shorter, which throwing an obscene amount of batons and other items with one button combination was one of the main issues.,

give me a second while i add media again


Sealed Secrets README / Bootstrap README should be improved

Is your feature request related to a problem? Please describe.
Deploying the K4S platform for a non-DevOps/Kubernetes/Security expert is cumbersome and poorly documented. The major problem are the Sealed Secrets which need to be re-generated and this part is not well described in README.
Describe the solution you'd like
Here is my description  / guide
Those services fail because they depend on some credentials that are passed as sealed secrets, also described in the K4S official README. We need to re-generate secrets for postgres and keycloak.
Generating secrets works like this:
You write plain text passwords in the templates, then run kubeseal on top of them, which generates the same templates, but those passwords are now encrypted. Kubeseal connects to the cluster, takes its public key (sealed-secrets-controller), and uses it to encrypt the data. You then push those encrypted templates to the repo - GitOps (not the plain text ones!!!). ArgoCD recognizes that there was a change in the repo. The sealed-secret service notices that new sealed secrets are available and decrypts them and updates the kubernetes cluster with plain secrets that kubernetes knows how to handle. Each service has then a “reloader” that notices that the plain secret changed and the new value needs to be used.
ArgoCD runs an update every couple of minutes. Not to wait for that, you can log in to ArgoCD and manually execute sync.
Generate secrets:

cd deployment/mesh-infra/security/secrets/

change the password in the template:

nano templates/keycloak-builtin-admin.yaml

then run:

kubeseal -o yaml < templates/keycloak-builtin-admin.yaml > keycloak-builtin-admin.yaml

Repeat the same process for the postgres:

nano templates/postgres-users.yaml
kubeseal -o yaml < templates/postgres-users.yaml > postgres-users.yaml

Then:

git restore the template files
git add new encrypted files
git push to repo

You can then either wait for the ArgoCD to pick up the changes and propagate them, or manually trigger ArgoCD, by logging in to the ArgoCD and clicking on the “sync apps” in the top left corner.
The status of all the services should be “running” after ~15min.

Thanks for your shorter and more direct version of "how to approach for the first time with sealed secrets".
One more question about argocd secret, I followed your steps (edit keycloak-builtin-admin.yaml and postgres-users.yaml in template folder, run kubeseal, revert those in the template folder and push the new ones and I finally have all "running" with no errors but...
In the argocd dashboard now I have a security warning and clicking on it shows three sealed secret errors:

in argocd-secret section error is: no key could decrypt secret (admin.passwordMtime, oidc.keycloak.clientSecret, server.secretkey, admin.password)
in fams-image section error is: no key could decrypt secret (.dockerconfigjson)
in insight-image section error is: no key could decrypt secret (.dockerconfigjson)

And to conclude I logged out from argocd and now I can no longer log in either with the default credentials or with the new ones