removeClippedSubviews breaks TextInput text selection on Android

If you try to long press a TextInput to copy / cut / delete a block of text inside of a ScrollView with removeClippedSubviews set to true, it will not work -- it will just move your cursor to that position. All other interactions with the TextInput work as expected.
Reproduced here: https://github.com/brentvatne/text-selection-repro

Clone and run the repo, try to select text in the ListView part (first two text inputs). Try in ScrollVIew and View inputs and notice that it works.
Add removeClippedSubviews={false} to the ListView and reload, it should now work.

cc @dmmiller @kmagiera

@brentvatne can you put together a quick repro and I'll look in the next few days please?

@dmmiller - https://github.com/brentvatne/text-selection-repro just for you man

@brentvatne Thanks for the repro.  Looking now.  One of the things I found is that if you actually edit the text, it works.  Just go into the TextInput and type a single letter, then try long press and it will work again.

Found the issue.  Sending a PR internally to fix it, but I'll reference this issue

Fixed in the attached commit


Sometimes messages get stuck in the Datalayer for a long time before reaching the Renderer

Messages sent from Inspector to Datalayer sometimes take a long time to be relayer from Datalayer to SceneContext:
https://user-images.githubusercontent.com/2781777/228591540-258ae1b0-7217-492b-a5af-651fb4d007c7.mov

got fixed along the way of different improvements to data layer


Updated README.md

Corrected some grammatical errors.

These are all in the backticked code syntax because they're not referring to "Python, the language" but rather "python, an image on Docker Hub" (and the entire file is generated via https://github.com/docker-library/docs/blob/1a91d48fdb0a6eded8adfef1709d76d51ba1d8dd/generate-repo-stub-readme.sh).


[AF-30] Add apps in organization sidebar

:koala:
Add organization_sidebar to locations
Relates to:

Console - zendesk/zendesk_console#8401
ZAF - zendesk/zendesk_app_framework#425

/cc @zendesk/quokka
References

Jira link: https://zendesk.atlassian.net/browse/AF-30

Risks

None

:+1:

:+1:

:+1:

:+1:


Adding the project logo

Adding th project logo and icon
fixes transitive-bullshit/chatgpt-api#353

Perfect fit @transitive-bullshit

This project is undergoing a major revamp; closing out old PRs as part of the prep process.


Main repository setup.py file not installing submodule dependencies

As pointed out in #156 the main setup.py file does not always instal the required submodule dependencies when using the pip install . command.

This issue only appears with old versions of pip. The solution is to update the pip package.


Make !clean!time! Errors into warnings (or infos) and make them more descriptive

Some people are getting a lot of warnings like:

[Thu Dec 30 19:41:22 2010] [error] [mod_pagespeed <IP_ADDRESS>-293] /var/
mod_pagespeed/cache/!clean!time!:0: open input file (code=2 No such
file or directory)

These are not actually errors and are only a problem if they are happening more
than about 2/hour. We should make the message more descriptive so that is clear.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Jan 2011 at 7:00

Can someone please explain what the !clean!time! file is and why it keeps reappearing after I've completely removed ALL files and subfolders from my mod_pagespeed cache directory? I've been searching for some kind of explanation of this and I can't find anything anywhere. Thanks in advance.

mod_pagespeed has a periodic cache cleaning process to keep the cache from growing too large. By default it runs every 1 hour. That !clean!time! file is used to track when the last time cache cleaning was run, and will be regenerated each time cache cleaning runs. It's essentially an implementation detail, so we don't have that file explicitly documented, but you can read more about the cache cleaning process here.

Thanks for the explanation Jud. Now I know my mod_pagespeed is working as it should w.r.t. that file.


Javascript Syntax Error

I get the following error once I have compiled my code into javascript:

pyjsdl.time.js:867 Uncaught SyntaxError: missing ) after argument list

The error only occurs if I am using the strict conversion option in pyjs. The line in question is this line:
self['__is_instance__'] && typeof self['__setattr__'] == 'function' ? self['__setattr__']('timer', $wnd['setTimeout'](function() {self['run']();}, self['time']);) : $p['setattr'](self, 'timer', $wnd['setTimeout'](function() {self['run']();}, self['time']););
I have tried fixing the error myself, but to no avail.
Any help would be much appreciated

Appreciate your assistance in uncovering the bug that was unfortunately entered in the latest git revision. The code triggering the error is in time.py at line 186 and is probably due to a pyjs conversion error, updated the git repository with this correction:
changed:
self.timer = JS("""$wnd['setTimeout'](function() {@{{self}}'run';}, @{{self}}['time']);""")
to:
timer = JS("""$wnd['setTimeout'](function() {@{{self}}'run';}, @{{self}}['time']);""")
self.timer = timer


Temporal Unit Test Failures

I've fixed a couple tests and I'm doing a pass thru the current unit test failures to try categorizing them and chipping away at the causes.  Note that I've added the FactAttributes.cs file with additional unit test attributes in an attempt to categorize various tests and also to provide an easy way to enable/disable tests.  These replace the [SlowFact] attributes you added.
To enable unit tests, simply comment the Skip = null; line in the attribute constructor.
Re-debugging all of these tests isn't super fun, but I figure we can chip away at different problem categories one at a time and hopefully there won't be too many underlying causes.  We'll just have to gut through these.
--

[x] [Fact_Failing_BadWorkerId]: It looks like a lot of the test failures are due to zero worker ID values being passed in messages from the proxy.  The .NET side is expecting valid positive worker IDs for these messages:
WorkflowInvokeRequest
WorkflowSignalInvokeRequest
WorkflowQueryInvokeRequest
ActivityInvokeLocalRequest
ActivityInvokeRequest
ActivityStoppingRequest

[x] We're seeing a couple exceptions in generated stubs:

[x] NullReferenceException
[x] TargetException

[x] Test_EndToEnd.WorkflowStub_Start_Untyped: This test is hanging while waiting for a WorkflowGetResultReply message from the proxy.  I need to verify first that the workflow actually completed.
[Fact_Failing_Untyped]
[Trait(TestCategory.CategoryTrait, TestCategory.NeonTemporal)]
public async Task WorkflowStub_Start_Untyped()
{
    await SyncContext.ClearAsync;

    // Use an untyped workflow stub to execute a workflow.

    TestWorkflowStub_Execute.Reset();

    var stub      = client.NewUntypedWorkflowStub("TestWorkflowStub_Execute::hello", new WorkflowOptions() { TaskQueue = TemporalTestHelper.TaskQueue });
    var execution = await stub.StartAsync("Jeff");

    Assert.NotNull(execution);
    Assert.Equal("Hello Jeff!", await stub.GetResultAsync<string>());  // <--- TEST HANGS HERE

    // Verify that we're not allowed to reuse the stub.

    await Assert.ThrowsAsync<InvalidOperationException>(async () => await stub.StartAsync("Jeff"));
}

NOTE: The workflow never starts.

[ ] @johncburns1: Several (17) message round-trip tests are failing

[ ] @johncburns1: Interop tests

[ ] @jefflill: All of the replay tests are failing: the workflow method is never called.

JEFF UPDATE: I'm trying to execute the Test_Replay.Nop test which starts a workflow with a single enum parameter: Nop.  The weird thing is that the Temporal portal displays this parameter as base-64 encoded??

I wonder if Temporal changed how parameters are encoded which is why the portal isn't able to decode the base-64 encoded parameters.  The interop tests will clarify this.

JEFF UPDATE 2: All of the replay tests passing now except for the two MutableSideEffect tests (I fixed the others by setting the default task queue in the client settings).

[x] @jefflill: 12 of 32 StubManager tests are failing.
FIXED: I just needed to tweak a faked worker initialization.

[ ] @johncburns1: All of the SyncSignal tests are failing with a NullReferenceException.  This appears to be a JSON message property issue.  The description I get back from a DescribeWorkflowExecution() has all NULL properties.

[x] @johncburns1: Local activity worker ID (you may have already fixed this one).  The .NET side is now passing the WorkerId for ActivityExecuteLocalRequest messages.  You'll need to pass this ID back to me when you send the ActivityInvokeLocalRequest to actually execute the activity.

[ ] The two Settings tests are hanging.  There's a decent chance this is JSON property related.  Let's come back to these after resolving the other JSON issues.

I was missing WorkerID on signal, query, and activity stopping requests.  I added them.  I still need to test.

@johncburns1: FYI: I've started looking at the non-end-to-end tests and fixing those.  I'm adding these as TODO items above as I find them.

@jefflill All inputs and outputs to the workflow are base64 encoded.  This is how it was in Cadence as well.

CLOSING: Temporal is deprecated.  We're waiting for an official client.


get_surface_override_material_count returns '1' even when its property is set to '0'

Tested versions
Godot version 4.3-RC1
System information
Windows 11
Issue description
calling get_surface_override_material_count on a MeshInstance3D returns that a material is assigned even when the property value is empty.
Steps to reproduce
call get_surface_override_material_count() on A mesh with surface material override property empty, but has a surface material assigned.
Minimal reproduction project (MRP)
N/A

The documentation of that function already makes it very clear that this returns the equivalent of the surface count of the mesh and not "used slots" if that is what you assumed for whatever reasons.

For reference, here is the docs page: https://docs.godotengine.org/en/latest/classes/class_meshinstance3d.html#class-meshinstance3d-method-get-surface-override-material-count


inference particular classes

How to use existing Yolox models to inference only detect targets with a specific label, such as only the Person classes.

having the same question....

I alse meet the same problem in the task of Seg, because the first label in coco.py is It.
But I don't know how to solve this problem.
#10675

I alse meet the same problem in the task of Seg, because the first label in coco.py is It. But I don't know how to solve this problem. #10675

I solved it！！！


throw null in Element.get()

Is there a reason to throw null instead of return null here? I'm wondering because in the console it looks like something bad happens when there is no data in localStorage yet. I noticed it when working with <ma-data> Vue 3 component. Maybe it's just me. 🤷🏻♂
https://github.com/madatajs/madata/blob/2e8b048cf36b734d2f30caa9b0d1acd799e22811/backends/basic/local.js#L23

Do other backends throw if the data is not found? If so, probably for that. If not, we should change it.

I looked through the code and found that in most cases (Backend, Dropbox, GithubGist, GithubFile, GoogleDrive) when there is no data to work with, backends return null.
However, I noticed inconsistencies with other backends: Google Calendar, Google Firebase, and Google Sheets should properly handle 404 errors and not throw an exception in those cases.
So it seems that Element.get() should return null and not throw it.


Letter-spacing style props caused Text component to breaks at incorrect positions

Description
After the Android 11 release, some of our users noticed that some texts are rendered wrongly

It renders properly after removing letterSpacing: -0.8 style from the text components

Additionaly, it renders properly on iOS and Android 10 devices
React Native version:
System:
    OS: macOS 10.15.4
    CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
    Memory: 821.89 MB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.12.0 - ~/.nvm/versions/node/v12.12.0/bin/node
    Yarn: 1.7.0 - /usr/local/bin/yarn
    npm: 6.11.3 - ~/.nvm/versions/node/v12.12.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 19, 22, 23, 24, 25, 26, 28
      Build Tools: 23.0.1, 23.0.2, 23.0.3, 24.0.2, 25.0.0, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 27.0.3, 28.0.3
      System Images: android-28 | Google Play Intel x86 Atom, android-30 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.6 AI-192.7<IP_ADDRESS>08749
    Xcode: 11.4/11E146 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: ^0.61.5 => 0.61.5
✨  Done in 6.82s.

Steps To Reproduce

Create Text component with letterSpacing
Run the code on Android 11 device (edit: It's all good on Android 11 emulator, but not on real device)
Text will breaks at incorrect positions

Expected Results
Text should render properly (pic2)

I got the same issue with letter spacing in Android 11 (both emulator and device).
Expected

Actual

I've upgraded the latest version of React Native 0.63.3 but still got the issue.
My workaround solution is round the letterSpacing value to 2 decimals (-0.XX).
Any suggestions?
Thanks for any help.

I faced the same issue.

@sbycrosz @htduy262 Do you still experience this issue? If yes, I will publish the fix in the react-native-improved package https://github.com/fabriziobertoglio1987/react-native-improved. Thanks a lot


Dependabot does not respect version constraints for pip-compile

I'm not sure how general this issue is – maybe it applies to all requirements.txt even without pip-compile, or maybe it's something specific to our exact use of pip-compile. Any feedback appreciated!

Package ecosystem
Python (pip-compile)
Package manager version
pip-compile 6.4.0
Language version
Python 3.8.12
Manifest location and content prior to update

We are using a private repository and I can't disclose the full content of these files. We essentially have a requirements.txt, requirements-dev.txt, and constraints.txt that we generate via pip-compile.
dependabot.yml content

(Truncated & redacted from private repo):
version: 2
updates:
  - package-ecosystem: "pip"
    directory: "[redacted]"
    schedule:
      interval: "weekly"

Updated dependency
pyparsing was updated from 2.4.7 to 3.0.6.
The relevant path in the dependency tree is:

pytest==6.2.5 (via pytest >= 6.2 required by pyproject.toml)
packaging (via packaging required by pytest)
pyparsing==2.4.7 (via pyparsing>=2.0.2,<3 required by packaging)

What you expected to see, versus what you actually saw
Expected: We were hoping Dependabot would be aware of version restrictions in the whole dependency tree, preventing it from performing a major upgrade of pyparsing.
Actual: Dependabot updates every package in the requirements files to the latest possible, seemingly ignoring dependency constraints.
Native package manager behavior

We generate our files using:
pip-compile pyproject.toml --output-file requirements.txt
pip-compile pyproject.toml --extra tests --output-file requirements-dev.txt
pip-compile pyproject.toml --extra tests --strip-extras --output-file constraints.txt

This respects version numbers in dependencies.
Images of the diff or a link to the PR, issue or logs

🕹 Bonus points: Smallest manifest that reproduces the issue

Running the above pip-compile commands with a pyproject.toml with a dependency on pytest >= 6.2 should yield requirements files that trigger the issue. (I haven't tested this, though, don't have dependabot set up to run locally)

I think this is a duplicate of https://github.com/dependabot/dependabot-core/issues/5140. That issue is newer, but it includes repro instructions, and a potential approach to fixing it.
So let me close this in favor of the newer issue. If interested, please subscribe to that other ticket!


Feat: Add More relevant web3 terms

Add more web3 terms
The meaning length must not exceed 280 characters.

Can I just make a pr to this? or is there a particular way for contributing? I have a web3 term in mind and I'd like to add it to this.

Sure, @oneknucklehead assign the issue to yourself, research the web3 term then send a PR

How do I assign this to myself? I guess only you can assign this to me :3

Assigned.

Hey @Olanetsoft, is the issue still open? Can I raise a PR for this?

Hello @Olanetsoft! If you need anymore help with this issue I would love to help.

Hello is this issue still open? I would like to be assigned to this

Hello is this issue still open? I would like to be assigned to this

Yes

Hello @Olanetsoft , I have a new term in my mind, can you assign me the issue too to submit a pr?

Hello @Olanetsoft, is this issue still open. Can i add more web3 terms?

assign me to add more terms if it is still open


ISPN-16753 Simple Server allocation rate reductions

https://issues.redhat.com/browse/ISPN-16753
There are 3 distinct changes here

Don't allocate channel promises when not using access log
Allocate less CompletableFuture from telemetry (reduced by about 40% in my case)
Don't allocate EnumSet or Flag[] per invocation when no flags are provided

Wouldn't the code look more readable if you complete the span in handleGet/Put/Remove?
Don't forget the replace method.

Wouldn't the code look more readable if you complete the span in handleGet/Put/Remove? Don't forget the replace method.

Let me give it a try :+1:

Updated and added for all the methods in CacheRequestProcessor

Updated to use try/finally for the spans which reduced code further.

merged! thanks @wburns !


Run Functional tests before release docker image

Is your feature request related to a problem? Please describe.
Run Functional Tests (workflow call) https://github.com/vmware/repository-service-tuf/actions/workflows/functional.yml before releasing the Docker Images
Describe the solution you'd like

Worker
API
CLI

dev
latest
latest

dev
-1
-1

Describe alternatives you've considered
No response
Additional context
No response

Closed by wrong reference.

Closed by #124


Throw if any query building methods are used with upsertGraph

They have no effect.

Done in v2.0 branch.


chore(master): release 1.1.137

:robot: I have created a release beep boop
1.1.137 (2023-08-01)
Miscellaneous

deps: update dependency @wayofdev/commitlint-config to v3.0.2 (5903891)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.137 :sunflower:


Why does isOutsideRange get called multiple times for each date, and at different times (midnight vs noon)?

react-dates version
<EMAIL_ADDRESS>Describe the bug
isOutsideRange prop callback is passing each calendar day multiple times for some reason
specifically, each date can be called at local time midnight or local time noon, which makes it hard for me to know which one I'm supposed to compare for to return a correct value to isOutSideRange
expected behavior: returns consistent time, either always midnight local time or always noon local time
Source code (including props configuration)
Load and see strange output
<DayPickerRangeController
  isOutsideRange={(date) => console.log('huh', date.format())}
/>

https://user-images.githubusercontent.com/5566843/128084224-beaf1af4-0329-4172-af0f-d4abf7aa34d8.mp4
Desktop (please complete the following information):

OS: mac
Browser chrome
Version 92.0.4515.107 (Official Build) (x86_64)

I'd expect always noon, since midnight doesn't always exist. This seems like a bug.

Hi @ljharb,
Proposed a fix for this issue, please take a look.
I introduced simple cache to mitigate multiple isOutsideRange and isDayBlocked, it works as expected and tests passed, but would like to see your thoughts about this implementation.

A cache isn’t a good idea, because the callback may need to be called multiple times.
the fix is to not use midnight.

Wow, thanks for both of your help! Exciting to see the potential fix!

Hi, wanted to ask if this ever got fixed? I'm still struggling to figure out how to deal with this, as I'm currently assuming noon time, but it'll still give me midnight time sometimes causing my users to see strange bugs

Ah nvm, sorry for the thrash, I see that https://github.com/react-dates/react-dates/pull/2146 was merged in! Thank you! Will test this out more and see what might be going on

@opqpop it's been merged, but not yet released.

Ah okay thanks, any ETA on when that might be?
Meanwhile, I dug into it and threw in something like this in my callsite, which seems to have helped with some of the bugs
const hoursToAdd = reactDateDay.format('H') == '0' ? 12 : 0
const standardizedDay = reactDateDay.clone().add(hoursToAdd, 'hours')

I have similar issue, but with timezones
I notice the same date is called multiple times using different timezones.
any idea what could it be?


Implement StringViewArray and BinaryViewArray reading/writing in parquet

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
This is part of the larger project to implement StringViewArray -- see https://github.com/apache/arrow-rs/issues/5374
In https://github.com/apache/arrow-rs/pull/5481 we added support for StringViewArray and ByteViewArray.
The parquet crate has a reader and writer for reading/writing parquet data to arrow:

reader: https://docs.rs/parquet/latest/parquet/arrow/arrow_reader/index.html
writer: https://docs.rs/parquet/latest/parquet/arrow/arrow_writer/index.html

Describe the solution you'd like
I would like to be able to read a StringViewArray and BinaryViewArray directly from the reader and writer with no data copies (so the raw byte values are not copied).

Add functionality
Add tests

Describe alternatives you've considered
For example, I think we need to add the support to the writer here
https://github.com/apache/arrow-rs/blob/f41c2a4e5a33e482e12351051d77d0e059f28e33/parquet/src/arrow/arrow_writer/mod.rs#L719-L732
Additional context
The reader/writer already handles DictionaryArrays which I think could serve as a model for the view arrays.
@ariesdevil  reports they are working on this feature https://github.com/apache/arrow-rs/issues/5374#issuecomment-2004243100

@mapleFU brought up a good point on https://github.com/apache/arrow-rs/pull/5557#issuecomment-2046484889 that I wanted to record here. The  observation is that that the special handling StringView / BinaryView will be substantially more code in the parquet decoder.
The reason for adding the special case is to avoid a copy
For example as I understand it, from the parquet  encodings doc
...
BYTE_ARRAY: length in 4 bytes little endian followed by the bytes contained in the array
...

So the  data looks like this (length prefix, followed by the bytes):
\3\0\0\0foo\26\0\0\0abcdefghijklmnoprstuvwxyz

To make a StringArray, those bytes must be copied to a new buffer so they are contiguous:
offets: [0, 3, 29]
data: fooabcdefghijklmnoprstuvwxyz

However, for a StringView array, the raw bytes can be used without copying
views: [(len: 3, data:"foo"), (len:26, offset8)]
\3\0\0\0foo\26\0\0\0abcdefghijklmnoprstuvwxyz

In parquet, ARROW:SCHEMA is used to identify the schema and extended info in parquet file. I think the trick point is that, without key-value metadata, both view and string are stored as same thing. So I think we can add optimization for "read string as stringview", but I also think maybe storing a stringview as string can make some legacy reader not confused about it.
For some view, this might work for some encoding( like PLAIN and DELTA_LENGTH_BYTE_ARRAY), but might be tricky for DELTA_BYTE_ARRAY.

In parquet, ARROW:SCHEMA is used to identify the schema and extended info in parquet file. I think the trick point is that, without key-value metadata, both view and string are stored as same thing. So I think we can add optimization for "read string as stringview", but I also think maybe storing a stringview as string can make some legacy reader not confused about it.

This is a good point. Maybe something we could consider is to support setting the desired type directly on the reader
https://docs.rs/parquet/latest/parquet/arrow/arrow_reader/struct.ArrowReaderBuilder.html
Something like
let reader = ArrowReaderBuilder::try_new(...)
  // override any schema declared in the file
  .with_schema(schema)
  .build()?;

If we supported a similar API for the writer than we could write StringViewArary without any extra copies but the metadata stored in the file could say "read this as a StringView" 🤔

Yes, my view is that:

writer side: cast string view/binary view to string/binary, keeping the "large" and metadata. This also not confuse legacy reader
reader side: user can choose read by string or string view

I think the current writer has the very nice property that RecordBatches round trp cleanly -- they can be written to parquet and then re-read and will be equal.
Thus, yy personal preference is that the writer should (by default) write the metadata that matches the data it was passed (so in this case specify StringView)
I think it would be valuable to add an option on the writer to write in "compatiblity" mode (and store string/binary data with StringArray metadata, for example)

This may differ from how we treat "view"( For both list view and string view ). If we think it's a part of string or list, it might better be string/list in metadata. If we regard it as a completely different type, we should regard it as string-view/list-view
For ipc, they're completely different type. For parquet, they stored nothing different, ::arrow::ipc::ReadSchema might work bad in this scenerio, which might regard "large-string-view-with-metadata" to another type. I'm ok with this but I think at least we can testing this

however parquet uses ipc schema here. ::arrow::ipc::ReadSchema might work bad in this scenerio,

Thanks @mapleFU  -- I don't fully understand the concern -- if the embedded metadata in a parquet file says it was written as a StringViewArray I would expect the parqet readers that support that type to read it out as a StringViewArray unless they don't support the type (old readers) or the user specifies they want to read the data from parquet as a different type

unless they don't support the type

IMO at least, a release without StringView/BinaryView's spec (rather than don't support StringViewArray) they might meet a "parse failed" and cannot parse an ARROW:SCHEMA. This doesn't matter if we think it's ok.

unless they don't support the type

IMO at least, a release without StringView/BinaryView's spec (rather than don't support read StringViewArray) they might meet a "parse failed" and cannot parse an ARROW:SCHEMA. This doesn't matter if we think it's ok.

I see -- I think we are both in agreement that the user of the Rust parquet crate should be able to decide how they want to handle this situation. We may just have different opinions on what a better default is.
As long as both options are possible, I could be convinced either way on the default value

Yes, I'm also ok with writing a string view type, and maybe we can generate a test file first

Update here is that we have pretty fast support for reading into StringView and BinaryView arrays now thanks to @ariesdevil and @XiangpengHao  so closing this ticket
There is follow on work tracked in https://github.com/apache/arrow-rs/issues/5904 in case anyone wants to try and make it faster


Add low-power-mode extension

Description
An extension to toggle the low power mode of the macOS without fiddling around the ever-changing macOS settings
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

reopen. was still waiting for review

@pernielsentikaer I have modified the logic to detect which power setting value is available between - lowpowermode and powermode and use the appropriate value.
Special thanks to @erics118 for pointing out this info - https://github.com/raycast/extensions/pull/9227#discussion_r1398305277

You can use High Power Mode on 14-inch MacBook Pro models with M3 Max and 16-inch MacBook Pro models with M1, M2 or M3 Max. - https://support.apple.com/en-in/HT212852

The changes are in this new PR - https://github.com/raycast/extensions/pull/9873
I don't have access to a Mac with M* Max chip but I tested the changes in a MacBook Air with M2 chip and 16 MacBook with M1 Pro chip. Could you please check?


Example suggestion: pattern matching with Dict

See conversation here

Something like this maybe?
myDict =
    Dict.empty {}
    |> Dict.insert "one" 1
    |> Dict.insert "two" 2

findNumber =
    when Dict.get myDict "one" is
        Ok 1 ->
            "One is 1"
        Ok _ ->
            "One is another number"
        Err KeyNotFound ->
            "One not found"

expect findNumber == "One is 1"

I've asked Artur to weigh in

Artur says it's good :)


Missing methods in FormData interface

TypeScript Version:  2.0.3
Code
interface FormData {
    append(name: any, value: any, blobName?: string): void;
}

Expected behavior:
FormData can actually do more than only append data. In general JS I can retrieve data set with append by calling getAll. This method is missing inside of the FormData interface which makes it impossible to use as the compiler won't finish without failures.
So I expect a set of returned data when calling FormData.getAll()
Actual behavior:
module/Application/assets/ts/d3-vanilla-fileupload.ts(274,12): error TS2339: Property 'getAll' does not exist on type 'FormData'.
V2A-16:area51 v2a$ tsc --out public/js/d3-fileupload.vanilla.js module/Application/assets/ts/d3-vanilla-fileupload.ts

module/Application/assets/ts/d3-vanilla-fileupload.ts(274,36): error TS2339: Property 'entries' does not exist on type 'FormData'.
V2A-16:area51 v2a$ tsc --out public/js/d3-fileupload.vanilla.js module/Application/assets/ts/d3-vanilla-fileupload.ts

As you can see the same goes for the method entries.
Is there a specific reason these methods were left out of the FormData interface?

This method isn't supported in all browsers. See https://developer.mozilla.org/en-US/docs/Web/API/FormData/getAll

@RyanCavanaugh Thanks for the quick respond. Any way I can go and work around this? The only way I can think of is going away from TS again. This obviously is something I wanna avoid though.

Just write
interface FormData {
  getAll(): string[]
}

If that would be an option I'd already did it. But why would I wanna edit the original Interface?

You aren't editing the original interface.  Interfaces are open ended by design.  You are simply adding to it because you are asserting you are running in an environment where that method will be present.

Example


getters for write_null_portable are fixed

getters should be snake case not camel case

Windows test PASSed.

Linux test PASSed.


"color-alpha" appear as input field instead of color field (fixed)

:octocat: Hello i using Kirki for the first time ,i wanna use color-alpha field , i create field using  Static method calls.
        Kirki::add_field( '-DIMA-', array(
                  'type'        => 'color-alpha',
                   'settings'    => $control[0],
                   'label'       => $control[2],
                    'help'        => __( 'This is some extra help text.', 'kirki' ),
                    'section'     => $control[3],
                    'default'     => 'rgba(34,85,100,.4)',
                    'priority'    => $priority,
             ) );

And here the output

Note : when i change the 'type'  => 'color-alpha', into 'type'  => 'color', and i remove default value, it work but without opacity ( hex code ).
Thanks.

As stated in the docs:

This field requires that another, normal color control exists in your customizer. If not, then it will not work as it requires some objects loaded when the default color is loaded.

Do you have a normal 'color' control set up?

@mikedidthis Thanks for your reply, yes i use normal color and it work smoothly,
 $wp_customize->add_control(
                    new WP_Customize_Color_Control(
                        $wp_customize,
                        $control[0],
                        array(
                            'label' => $control[2],
                            'section' => $control[3],
                            'settings' => $control[0],
                            'priority' => $priority)
                        )
                    );

Sorry, to clarify, you are using a color control (elsewhere) and color-alpha ?

I am using color in some sections and in other place i use color-alpha.
i know if i use color-alpha with Hex will work fine as the doc say , but it not work with 'type' => 'color-alpha' .i hope the you understand me, thanks.

Yep, that makes sense. I was just confirming the issue wasn't down to the missing color control.

I fixed the problem thanks for your help , i make color-alpha class extends from WP_Customize_Color_Control instead of WP_Customize_Control

@AdelDima do you think you could submit a pull-request for this?

I believe this can be closed, the issue was caused by a typo.

@mikedidthis in your case yes... but I'm not sure about @AdelDima so if there is indeed an issue I'd like to keep this one active until we fix it.
Cheers!

@aristath Sorry I may be confusing everyone here. I never had an issue, I was trying to help @AdelDima . The last message states that they resolved the issue. :tada:

the control class now extends WP_Customize_Color_Control as suggested in the comments above. :+1:


(Bug) Keep data between the steps (Manage Page)

This is an integration PR for the manage section
How to test it and what to test:

Go to the home
Create a crowdsale (Minted or Dutch)
Go to the home, choose the created crowdsale
You should see the created crowdsale
If you refresh the page, you should see the data again

Pull Request Test Coverage Report for Build 2874

58 of 271 (21.4%)  changed or added relevant lines in 27 files are covered.
15 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.3%) to 20.128%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils/blockchainHelpers.js
0
1
0.0%

src/components/manage/index.js
0
1
0.0%

src/App.js
0
2
0.0%

src/stores/TokenStore.js
0
2
0.0%

src/utils/alerts.js
0
2
0.0%

src/stores/GeneralStore.js
5
8
62.5%

src/stores/Web3Store.js
0
3
0.0%

src/components/stepTwo/StepTwoForm.js
6
10
60.0%

src/components/stepTwo/index.js
0
4
0.0%

src/components/Common/TierBlock.js
1
6
16.67%

Files with Coverage Reduction
New Missed Lines
%

src/App.js
1
0.0%

src/components/Common/DutchAuctionBlock.js
1
6.12%

src/components/stepOne/index.js
3
0.0%

src/components/stepFour/index.js
3
0.0%

src/components/Home/index.js
3
0.0%

src/components/stepThree/GasPriceInput.js
4
0.0%

Totals

Change from base Build 2862:
-0.3%

Covered Lines:
877

Relevant Lines:
3736

💛  - Coveralls

@mariano-aguero
Step 1 - Choose crowdsale: list of crowdsales is shown  only for last used Network
Steps:

Set account which has already created crowdsales in Sokol and Ropsten networks.
For each network go to Step1-Choose crowdsale, observe list
In my case list empty for Ropsten

Create any crowdsale in Ropsten network
For each network go to Step1-Choose crowdsale, observe list
Actual result:

Sokol network:  list is empty
Ropsten network:  list contains  previous created crowdsales

@mariano-aguero
Step1-Choose crowdsale: list should be sorted and show the same order after each opening
Now order of crowdsales sometime could be different after refreshing or re-opening page

@dennis00010011b

Step1-Choose crowdsale: list should be sorted and show the same order after each opening

Fixed in this branch, but I think is not related with this PR, see PR  #1046

@dennis00010011b

Step 1 - Choose crowdsale: list of crowdsales is shown only for last used Network

Can you check if the problem persist? if yes, can you make a video of the problem? The env vars for sokol are the same for ropsten? Thanks in advance

@mariano-aguero
Build failed to compile, got error:
Failed to compile. ./src/components/stepFour/index.js   Line 183:  'getNetworkVersion' is not defined  no-undef

@mariano-aguero

Can you check if the problem persist? if yes, can you make a video of the problem? The env vars for sokol are the same for ropsten? Thanks in advance

Problem https://github.com/poanetwork/token-wizard/pull/1103#issuecomment-415080605 still exist. Maybe would be better to make it as separate issue?
You can check with my account , PR ba98116a7d4b98f22f113c59448b9cc69f916d75f35d51f088b64b483fd0b8ca
Video:
https://www.useloom.com/share/31c70da4928e47c29d7043eda4c30d6a
Contract's accounts are different for each network.
.env file:
REACT_APP_ABSTRACT_STORAGE_ADDRESS='{"3":"0x0648b0645574966c32d37bc5be0208a979342677","77":"0x9d126f44151f70a4da03f53fcb485dea6c0b8d64", "42": "0x2b2d1b6867572ed42140b11b2dac7d454bbce73d", "4": "0xb5bfd08585b705c0eec3c9043249902cc9291ad8","99":"0xd70befce3cf1cc88119c8f4eb583ccd4c39d06e2"}' REACT_APP_REGISTRY_IDX_ADDRESS='{"3":"0x417f7d0f10fc18c6e139c06e0db7634b5d5756f4","77":"0x76fafe6f7e2f25e40664cf9f22d6e18b709d0087", "42": "0x1be7505450bcd62b2001db8eefb9be1706db45c6", "4": "0xf9d9365d93a351ea4f9ee712a3b731e1d6440a59","99":"0xda7cd8385be8f293677574925f35fbeb38c35a85"}' REACT_APP_PROVIDER_ADDRESS='{"3":"0xc8bdff0860cc2d0c349208f79e0f3e086effe0cb","77":"0x663d2feb6af69867e0144aab82624edb7af1728b", "42": "0x11bd7d75c081ae7deb9244981a72ceeb340bcbc1", "4": "0x8ca673abb1123a3ffe4000bc813ae2c00d0c7c02","99":"0xef5879ad4ba3fa0312875426902dd384d8c63fcd"}' REACT_APP_REGISTRY_EXEC_ADDRESS='{"3":"0x1e2f3f5e8022cf28804dd69b81629927079f9951","77":"0x6865f64ecb1f0222a0febe5c1fc1b40c7a378da6", "42": "0xef4dc7adb5d13448597e4d4b471b66c4690568d2", "4": "0x13462c0292f3007ee6d0aaf600dbe1db2ebcf4be","99":"0xd7a4deeeb59569a9d00ba78a18d591bfe53807e0"}' REACT_APP_MINTED_CAPPED_IDX_ADDRESS='{"3":"0x21712e030328541ba8a1d044df0317047dd76bb9","77":"0xeeaefd3a69f7d10db92205bbaa54f7c5aa8edf3f", "42": "0xa32057752831f4cc0cc49e7f74bb3cd67237af00", "4": "0xd504e3fb278d36437023fbcdcfd9dd194c71c6c2","99":"0x6f1b623058ffe30b14a6071bbbecaa373cef20c8"}' REACT_APP_MINTED_CAPPED_CROWDSALE_ADDRESS='{"3":"0x9039d36603b92211d2532f7f4ecbeced912172bb","77":"0x1d9702eb87c47f90c8669f3d36322f8cf985f721", "42": "0x3397eef245bd419b9e8cc5bc20e896ff784e6d5d", "4": "0xef4dc7adb5d13448597e4d4b471b66c4690568d2","99":"0x39b586af1b947a31806406d8e7dca12240e149eb"}' REACT_APP_MINTED_CAPPED_CROWDSALE_MANAGER_ADDRESS='{"3":"0x10df208e0b76a4e9f8b54e436022f113d4bae5c3","77":"0x3ab54a53eed7e0d3c2f051e4992bb893e8434e5f", "42": "0xf9d9365d93a351ea4f9ee712a3b731e1d6440a59", "4": "0xefc62352f46c62b8aab3d1a802c0c54fc320130b","99":"0x3f099bd191c3f38d60088c287b18f91b1a84f972"}' REACT_APP_MINTED_CAPPED_TOKEN_ADDRESS='{"3":"0x7b0ab844e92ce6ac36cc844e452766fa11a48c82","77":"0xbe1efbd9134bb5cd0ad3ee6cd2c246e56961ac5c", "42": "0xb5bfd08585b705c0eec3c9043249902cc9291ad8", "4": "0xf0e295c8c1947e834ade8fd3742f51ac44fccba9","99":"0xebd4f6ef12688a35406198ae2a39934fc70e3378"}' REACT_APP_MINTED_CAPPED_TOKEN_MANAGER_ADDRESS='{"3":"0x0a019b617a0e1745494e9f5dbefdd82445fd9035","77":"0x7e9f62b32538255065681c24efd7a5947f258351", "42": "0x8ca673abb1123a3ffe4000bc813ae2c00d0c7c02", "4": "0xa4fec065faa308a08dba10373e78a04ea5cc5117","99":"0x001aed057cb44ff7325d990db3ff69b0761cccab"}' REACT_APP_DUTCH_IDX_ADDRESS='{"3":"0x6d7b8d8b819dfb3b03b1a9b1c376f409f48e11f4","77":"0xfbb8127eeb14be7305a6738aa9e43654a95ad908", "42": "0x13462c0292f3007ee6d0aaf600dbe1db2ebcf4be", "4": "0xd1ad7938b170fd9f79221497178b5abc68924d34","99":"0xcb5e9ded56c9fbf4a309f2d6f4e1bbbeb4e31bf7"}' REACT_APP_DUTCH_CROWDSALE_ADDRESS='{"3":"0x9c5f8263171b83c303c18b7b1977fa28bdd45116","77":"0x3f52577e2772f0adacd9d3e6a0fdb456701bf386", "42": "0x1c6d46dedd61ef982f0938e73975404b8de6d739", "4": "0xd37bfd3632f78608ad82d1a8c253e2b35908e35f","99":"0x729b907b2e4ff8d150bc774f400a3afe4553b468"}' REACT_APP_DUTCH_TOKEN_ADDRESS='{"3":"0xd1A26D4A869d66EEeA894c47b5d4F0B3A6FC6cAb","77":"0x2a5fbbb4ec592bace485d0f653b051d991eeffdc", "42": "0xf188df9b40f5a5598a1d7088021a679312e62d03", "4": "0x4b34e3df0b4803674db01b0eaef937e6022f4610","99":"0x9f057f47e10b1d58ce2531863597d9dc992df07a"}' REACT_APP_TW_PROXIES_REGISTRY_ADDRESS='{"3":"0xf8141d3ef4ca940d0f485d0abbf2b11e6303c484","77":"0x16a3e07d364d410c6c99e82047177e604779eb02", "42": "0x7cbe84100ff284f87ecd4d5830ff788b9a7ce49e", "4": "0xd836f5716217640ab97bd6ee848881c42625a05d","99":"0xc6468767214c577013a904900ada0a0dd6653bc3"}' REACT_APP_PROXY_PROVIDER_ADDRESS='{"3":"0x7ae09e9963c835959b06240ba69b00febdff681a","77":"0x7ae09e9963c835959b06240ba69b00febdff681a", "42": "0x7ae09e9963c835959b06240ba69b00febdff681a", "4": "0x7ae09e9963c835959b06240ba69b00febdff681a","99":"0x7ae09e9963c835959b06240ba69b00febdff681a"}' REACT_APP_REGISTRY_EXEC_ID='{"3": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "77": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "42": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6", "4": "0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6","99":"0xb10e2d527612073b26eecdfd717e6a320cf44b4afac2b0732d9fcbe2b7fa0cf6"}' REACT_APP_MINTED_CAPPED_APP_NAME='MintedCappedCrowdsale' REACT_APP_DUTCH_APP_NAME='DutchCrowdsale' REACT_APP_MINTED_CAPPED_APP_NAME_HASH='0x4d696e74656443617070656443726f776473616c650000000000000000000000' REACT_APP_DUTCH_APP_NAME_HASH='0x447574636843726f776473616c65000000000000000000000000000000000000' REACT_APP_INFURA_TOKEN='kEpzZR9fIyO3a8gTqJcI'

@dennis00010011b

Build failed to compile, got error:

Done

Problem #1103 (comment) still exist. Maybe would be better to make it as separate issue?

Agree to generate a separate issue for this

@mariano-aguero

Problem #1103 (comment) still exist. Maybe would be better to make it as separate issue?
Agree to generate a separate issue for this

Issue isn't reproducible in branch 2.0. It should be fixed before merge into 2.0

@dennis00010011b
I think we can solve this in the integration branch, so we can merge this PR an make a progress


Generate properties for parameters included on path items

Currently, properties are only generated for parameters listed explicitly on each operation (except for simple string properties for route parameters). The OpenAPI spec allows for parameters to be defined at the path level and then overridden for each operation.
This implements that behavior, handling query and header parameters defined at the path level and using the correct schema for route parameters defined at the path level.
Relates to #239

/merge


Add fix to handle max path limit in windows

Windows has a max total path limit of 260 and there can be schemas which don't reach the max path segment size, but the full path reaches max total path limit in windows systems.

I've created duplicate PR (#131) to see that CI passes


Altitude accuracy

While migrating from a different library to this one, I noticed that the Position class is missing altitude accuracy.
I think that in some scenarios this value might be useful.
Same goes from bearing accuracy.

Hi @mbn18, thank you for bringing this to our attention.
We just released a new version of the plugin (10.1.0) that exposes these fields if the underlying APIs supports it. They are fully supported on Android, iOS and macOS. On web and Windows only the altitude accuracy is available, and on Linux neither is available.


Classical MD

https://github.com/NQCD/NonadiabaticMolecularDynamics.jl/blob/6039a6c2e0c9be83868f8f0fc9a04d6fd7e978f6/docs/src/dynamicssimulations/dynamicsmethods/classical.md#L23
In the 'Getting Started' section, it's written that simulation is set up as Simulation{...}, however, this does not seem to be the case here. Is that a mistake?

Ah, I see from another comment that it's the default.  I would probably mention that at the end of the Classical-section and leave it out everywhere else to avoid confusion (keep everything as consistent as possible with no special cases).


Extend jdbc functionality

Adjust web ui to allow definition of jdbc connections
Use host column for jdbc connection url
Use connection models extra for storing jdbc driver path and classname
Use connections extra_dejson to create the jdbc connection
Use DbApiHook as a base for JdbcHook
Make JdbcOperator consistent with other Db ops by using run

Example of creating a connection:

Maybe a little clarification on this. The reason we extended this further was due to the fact, that adhoc queries to jdbc wouldn't work. In short, predefining jdbc connections wasn't possible, because there was no way to define jdbc driver location and jdbc driver classname. Now that's possible and everything works including adhoc queries.

About the UI customization, it's a nice flask-admin cutomization that optimizes for user experience but moves away from vanilla CRUD. I'm not sure if I want to optimize for user experience or ease of maintenance though. We could just document how to use the extras key/values very clearly and get rid of a bunch jdbc custom logic that requires knowledge of flask_admin/javascript/jinja to understand/evolve.
On the other hand it looks really nice and fits the JDBC use case.
What do you think?

I think it's a difficult decision.
That said, if you want to get as many people/companies to use airflow as possible, it's probably worth doing a little UI customisation to get a bigger audience to use it. Usability will go a long way when it comes to that. Good documentation will mitigate this to a certain extend, but I think overall the changes that were necessary weren't that intrusive. Still, your argument is valid and it's a tradeoff worth discussing. Let's see if someone else has an opinion on the matter.
In the meantime I'll fix the other things.

The design is based on the goal of making the setup of JDBC Drivers as simple as possible (via the webinterface and not by diving into the code).

Totally, thanks for taking the time to bend the form into doing this, it also sets a precedent on how to do it if we want to replicate it for other hooks/connections.

Fixing a bug and generalizing the idea here:
https://github.com/airbnb/airflow/pull/201


only store the last version of an app to be deployed

this way we can redeploy old versions

This resolves https://github.com/replicatedhq/kotsadm/issues/1236


Add more feeds to cover all subfields of astronomy & amateur astronomy

Subfield-specific feeds allow for highly filtered views of posts on the Bluesky network. These subfield feeds will all crosspost to the main Astronomy feed, too - meaning that posts in subfield feeds won't be in isolation.
Outline
The 1.0 concept with these feeds will be to have:

Coverage of all subfields of astronomy/astrophysics, including topics relevant to amateur astronomers
About ~2 feeds per astro-ph category
Around ~10 feeds in total

Additional features not currently planned include:

Possible additional coverage of methods (e.g. a machine learning feed)
Additional fun ones (e.g. "observe with me" for live-skeeting your observing night)

Existing feeds

Astrosky - all posts by everyone in the Astronomy network (unfiltered)
Astronomy - all astronomy posts

Planned feeds
Covering science:

Cosmology
Extragalactic astronomy (maybe just "Galaxies"?)
Galactic astronomy (maybe just "The Milky Way"?)
Exoplanets
Planetary science
High-energy astrophysics (is there a better / less technical name?)
Astronomy Instrumentation
Astrostatistics & machine learning (maybe...)
Stars

I'm not sure whether more feeds (e.g. black holes, agn, star clusters) would be a good idea or if it would get too specific (and hence too diluted.) Heck, the above may be too diluted already...
Covering the more fun stuff:

Astrophotography (probably good enough as a catch-all? Could also add a separate discussion feed and keep this just for pictures, though not sure)
Space art (there may already be feeds that cover this, though)

Exoplanets & astrophotography are now added!
I'll lay off on new feeds for a bit because they aren't getting so much uptake, and I don't want to dilute posts / add too much complexity.

This is likely to happen sooner rather than later!
There are now logos in the new branding repo.

This happened last week, closing issue as feeds are now stable & running.


When loading an HTML page of type ajax, the loaded page is set as text/xml, not text/html

When loading an HTML page which starts with a <?, magnific-popup thinks it is of type text/xml and not text/html, and thus, fails to load the page with an error on the console (error in createdocumentfragment() in jQuery).
The loading of the page is done with  .magnificPopup({        type: 'ajax'    }) and the start of the page is:
<?php include(__DIR__ . '/../../../../wp-load.php'); ?>
<div id="popup_type">

To solve the problem I just put an empty line as first line. The response to the browser became text/html :-)

Use dataType: 'html'
.magnificPopup({
   type: 'ajax',
   items: {
    src: 'path/to/content.html'
   },
   ajax: {
    settings: {
	    type: 'GET', // or POST, whatever you need...
	    cache: false,
	    dataType: 'html'
    }
   }
});


Run after build on Mac OSX

I read the documentation  and finished building the SDK for Mac OS, but am having trouble figuring out how to run the build as a desktop application. I located the build output in the DerivedData directory, but within that directory are two more directories (librealsense2/ and ModuleCache.noindex/). I've looked everywhere online and can't find any solution for converting these directories into a .dmg or .app file. How should I go about converting this build output into a format that is runnable on a Mac?

The SDK itself does not run as a launchable application, even on Windows.  Instead, programs created with the SDK (such as sample programs, and SDK tools like the RealSense Viewer) are the ones that are run.
Whilst I have no experience first-hand in building Librealsense apps on OSX with XCode, if I were doing it, in theory, then I would try using the standard methods of exporting a project from XCode.
The link below seems like a good description of how to export a dmg to your computer and not to the App Store.
https://apple.stackexchange.com/questions/174008/xcode-how-to-export-app-solution

[Realsense Customer Engineering Team Comment]
hi @connorgoggins,
do you need more support on this topic?


Bug: @xstate/vue has wrong typings for useMachine

Description
useMachine requires 2 parameters when using @xstate/vue, however that should just be one (options is optional).
xstate source: https://github.com/statelyai/xstate/blob/main/packages/core/src/interpreter.ts#L308
@xstate/vue source: https://github.com/statelyai/xstate/blob/main/packages/xstate-vue/src/useMachine.ts#L50
Expected result
Not to have to pass a required options argument
Actual result
Requires an options argument
Example: const { state, send } = useMachine(machine, {})
Reproduction
N/A (Typescript issue)
Additional context
No response

Please always try to share a repro case in a runnable form - either by providing a git repository to clone or a codesandbox. OSS maintainers usually can't afford the time to set up the repro, even if exact steps are given.

I can no longer reproduce this with a simple example.
It might be a Vite caching problem (older version or something), or a typegen issue on the machine.
I will re-open if I run into it again and can isolate the case.


[GLUTEN-5656][CORE][FOLLOWUP] Support GetStructField with NullLiteralNode as subqueries not executing during validation

What changes were proposed in this pull request?
After #5658 , we met validation failed with q24a ad q24b while using bloomFilter.
How was this patch tested?
manul test

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_5923_time.csv
log/native_master_05_29_2024_588faae35_time.csv
difference
percentage

q1
34.42
33.56
-0.860
97.50%

q2
23.68
23.85
0.175
100.74%

q3
36.81
37.37
0.565
101.54%

q4
32.27
32.42
0.151
100.47%

q5
70.41
69.83
-0.577
99.18%

q6
7.44
7.54
0.106
101.43%

q7
83.41
81.75
-1.655
98.02%

q8
86.90
86.57
-0.332
99.62%

q9
119.79
118.35
-1.444
98.79%

q10
43.94
44.16
0.219
100.50%

q11
20.43
22.23
1.795
108.78%

q12
25.19
26.72
1.525
106.05%

q13
53.98
53.60
-0.378
99.30%

q14
20.75
17.59
-3.163
84.76%

q15
31.67
32.89
1.225
103.87%

q16
14.67
13.41
-1.266
91.37%

q17
101.72
103.62
1.902
101.87%

q18
147.02
144.53
-2.487
98.31%

q19
14.91
13.59
-1.326
91.10%

q20
28.98
29.66
0.671
102.31%

q21
261.19
260.43
-0.769
99.71%

q22
11.96
13.84
1.885
115.77%

total
1271.54
1267.50
-4.039
99.68%


Thanks for the update

yeah there isn't really issues i just wanted to thanks for the new online downloader lol
it legit bypasses windows defender so it's pretty cool i suppose.

Great that it works, the main thing I was trying to achieve was to bypass Windows Defender and that seems to have been very successfully so far at least and hopefully it stays that way.

i hope they don't patch this somehow.

Yeah I'd also like to leave a big thanks for the update, it basically helped alleviate all issues. I tried launching ARK with the "ShooterGame.exe" in the Stealth field, the Miner stopped IMMEDIATELY once the .exe was detected, and started again right after it closed.
The detection rate is also down, even by Malwarebytes which hasn't bothered me about it yet when I tried running the newly compiled Miner.


Client-side validation checks irrelevant form fields

I have a grid with adding and editing, and some values are mandatory. When I try to edit a row, netteForms.js validation won't let me to submit the form, because there is that hidden inline add form with invalid data.
You can try it with this simple example:
    /**
     * @return DataGrid
     */
    protected function createComponentTestGrid()
    {
        $grid = new DataGrid;
        $grid->setDataSource([
            ['id' => 1, 'foo' => 'bar'],
            ['id' => 2, 'foo' => 'baz'],
        ]);
        $grid->addColumnText('foo', 'Foo');
        $editor = $grid->addInlineEdit();
        $creator = $grid->addInlineAdd();
        $editor->onControlAdd[] =
        $creator->onControlAdd[] = function (Container $container) {
            $container->addText('foo')
                ->setRequired();
        };
        $editor->onSetDefaults[] = function (Container $container, $item) {
            $container->setDefaults([
                'foo' => $item['foo'],
            ]);
        };

        return $grid;
    }

The same issue may occur, when you have several rows open for editing and one of them contains an invalid value.
Any thoughts how to solve this?
Approaches that I though about:

Disable all irrelevant form fields, then re-enable them again. Nope, it would cause more problems than it solves.
Use $.nette.ajax() directly. But then the client-side validation wouldn't run at all.
On submit, create a new hidden form element, copy relevant fields into this form and submit it.

These form validations are a bit overkill for inline add in my opinion. I have to think about it more complex way.

I've noticed that you don't create Form (filter) submit button, precisely it's created automatically, we are unable to add the validation scope for filtering. I've managed to overcome this in editing via $container['submit'] provided in callback, is there any possibility to similarly deal with the problem?

I have run into similiar issue with group actions. It was not possible to execute group action combined with inline add and Nette rules.
Although I don't like it, I had to do something like this:
$grid['filter']['group_action']['submit']->setValidationScope([ $grid['filter']['group_action'] ]);

Maybe all submit buttons should have their validation scopes set to their respective containers?

This has been fixed in #451.


Search is case-sensitive

A user will probably expect the search feature to be case-insensitive, because most other search engines are. If they get no hit, they will assume the thing doesn't exit, rather than to try different casings.
A simple solution might be to just remove the case-sensitivity in all searches, but maybe a more complicated solution is preferable, e.g. do a case-sensitive search first and if that has no hits, do a case-insensitive search after. Or maybe start with a case-insensitive search, but put the exact matches higher in the results.

We could downcase all text strings during both indexing and queries (simplest solution for now), but I'd rather not remove capitalization information index time.
I'm planning on making some major changes to search sometime soon (see #226), and I'll need to update tantivy. I'll fix this issue then too.


Fix missing healthbar in freshly loaded chunks.

Fixes an issue whereby healthbar would not immediately appear on mobs from a freshly loaded chunk. E.g, upon joining the world, changing dimension, or after traveling a long distance.
The injection in sendPacket now also scans the content of ClientboundBundlePacket's. They contain packets that needed to be altered, and previously flew under the radar.
This PR is for 1.20. This branch acts as a backport for 1.19.4

Thank you so much! Sorry for late reply, I was away.
I usually only release for newer versions, if you're ok with that.

No problem, I only made that port for my own use, and I have already built my own jar.


Encoding plus sign makes no difference

From @PawelTroka on Sunday, November 12, 2017 5:11:23 AM
Basically this issue is kind of similar to #912, except it's about plus sign ('+') and this time it does not only affects TestHost but shows up also when hosted.
It does not work correctly on TestHost or when hosted and invoked through web browser or any web client for that matter.
It is failing my integration tests where I am using Microsoft.AspNetCore.TestHost v2.0.0 and Microsoft.AspNetCore.WebUtilities v2.0.0. Please notice that my WebApi and WebApi.IntegrationTests are AspNetCore 2.0 but are targeting net461 (due to some of my libraries not yet ported).
Repro:
Unit test:
https://github.com/PawelTroka/Computator.NET/blob/97b666341bf8a9b21fb9dc59f94528ccf830a931/Computator.NET.WebApi.IntegrationTests/CalculateApiTests.cs#L42
Goes through route:
real/{equation}/{x} from method:
https://github.com/PawelTroka/Computator.NET/blob/97b666341bf8a9b21fb9dc59f94528ccf830a931/Computator.NET.WebApi/Controllers/CalculateController.cs#L34
But because encoding plus sign makes no difference we have equation with unencoded plus sign
In images:
When run through browser:
We type encoded url

What reaches our controller however is already decoded

So when we are decoding it in another overload of Get(..) we will attempt to decode already decoded expression, which will make us replace plus sign ('+') with space (' ').
The same happens in test:

Also note that this exists at least from AspNetCore v1.1
Copied from original issue: aspnet/HttpAbstractions#964

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Add mips and s390x builders

Rebased on top of master to pick up toolchain updates.

Rebased.

Fixed the names of the mips toolchains and bumped the docker image.

Superceded.


Deleting old deployment assets

Hi, I have been building a few things with ion this past week and it has been a wonderful experience.
Something I noticed looking through my S3 buckets was that there seems to be an sst-asset-[randomID] bucket which contains the code uploads for my deployments. However, it looks to me as though it is never deleting previous versions as I can see code archives all the way back to my first test, including many that have been fully overwritten. While the size of this bucket is still relatively manageable, I can see it starting to balloon as I continue to deploy if another 5-10MB is added there with every single update.
Is there a reason old instances aren't deleted, and is there a CLI command or project setting that could enable automatic deletions of old archived code assets? Would setting up my own lifecycle deletion policy be a good option?
Thanks!

I think it's a little tricky to track this from what I remember. @thdxr do you know?

i think for deployment speed we don't bother deleting old assets - reassigned to frank he knows more


[Breaking] Install in ~/.local/bin instead of cargo home

uv is not part of the rust toolchain nor is it usually installed or managed by cargo. Similarly, i expect that the majority of users does not have cargo installed. We should instead use a not-rust-associated directory for binaries.
I chose ~/.local/bin, which is mentioned in XDG Base Directory Specification and also used by pipx.

User-specific executable files may be stored in $HOME/.local/bin. Distributions should ensure this directory shows up in the UNIX $PATH environment variable, at an appropriate place.

As testing strategy, i'd create a prerelease and test windows and linux(es), while i'd need someone to test mac.

How does this work across platforms? Like what happens on Windows?

Windows currently uses C:\Users\Ferris\.cargo\bin, i expect this to switch to C:\Users\Ferris\.local\bin.

For what it's worth, I don't have a ~\.local\bin on my Windows machine, but it does look like pipx uses that. Will the installer add it to PATH?

I'm cool with the directory but will it cause problems when used alongside pipx?

Should it be ~/.astral/bin...?

Should it be ~/.astral/bin...?

If everybody did that it would require a PATH entry per vendor, which imo would be bad. Normally installing into ~/.astral and symlinking into PATH would be an option, but that doesn't work on windows. I could see installing it into ~/.cargo-dist/bin or ~/.axo/bin if that helps with the updater.

I've realized this will break people who rely on the current installation location. I'd bundle this with other bigger changes (v0.2 probably) if possible.

Yeah I think this has some upside but also has the potential to be pretty disruptive.

I don't care whatever default location is choosen, as long as you can choose to override it yourself.
see https://github.com/astral-sh/uv/issues/2087
My personal preference is ~/bin because I don't want executables hidden away in hidden folders multiple levels deep.
Why?
If I need to support non technical people on windows with a non-working python-executable in some virtualenv,
I would often need to start explaining them  to "Open File Explorer from the taskbar. Select View > Options > Change folder and search options. Select the View tab and, in Advanced settings, select Show hidden files, folders, and drives and OK."  Just to be able to start seeing a folder.
~/.local/bin is just another personal preference.  In 20 years I only encountered a handful installers that use it. i never actively used pipx.  It's OK as a default.

For what it's worth with rye i went through various different choices and I ended up with one path for all platforms instead of messing with the XDG stuff or trying to find suitable locations on all platforms. It's super confusing when you need to look into different places depending on which machine you are on. I also with with ~/.rye/shims as a place on the path since rye is installing further tools. I fully expect that a future uv will be in the same position and will need to bring it's own folder onto the path.

@woutervh Is .local a hidden folder for you on windows? It looks non-hidden on mine.

@mitsuhiko What made you choose that over .local/bin?

@mitsuhiko What made you choose that over .local/bin?

rye tools install installs additional binaries. Those need to be placed somewhere. If I were to put them into ~/.local/bin I would start to muck around in a folder that is not fully under rye's control.

What would you think about (sym)linking those binaries to ~/.local/bin, from the background that it's the XDG standard that users are expected to have in PATH?

We might want to take into account toolchain management design considerations e.g. rustup / cargo do not have shims in ~/.local/bin.

For me it comes down to a somewhat philosophical distinction: Do we consider uv a tool that gets installed, so it goes where all the installed tools go, or do we consider it a "system" application manager with its own domain that can reasonably expect people to change their PATH for it. Note that people have asking cargo to use the xdg directories.

In Windows, why not use %APPDATA% folder for specific user and Program Files for everyone?

I think we should change this someday still

I'm also strongly in favor of getting out of the cargo brand. If we want to change it, i believe we need to do it in breaking release before uv tool support lands, the sooner the better before external dependencies on the uv install location are created.

@charliermarsh Why were you opposed to doing it in 0.2.0?

I just didn't consider it an important change and hadn't had time to think through the consequences. How will this interact with tool installs? Do we know?

By "important", I mean "pressing" or "time-sensitive".

I don't see how it interacts with tool installs. I agree it's not pressing but I agree with Konsti that there is some time sensitivity in that the longer we wait there may be increased reliance on this install location downstream.

The interaction point is: where will binaries installed by uv be placed?

True that's a consideration and an open question.


Bump and fix createjs version

#190

I can confirm that this fixes #190. Thanks @lutangar!

You're welcome.

:+1:


Using DirectX without plugin

Hello
Is there a way to use you development without plugin (I cannot change the kubelet) ?
Best Regards

Hello @clementrey
unfortunately, I'm alone to push device-manager in kubelet for windows. My PR #80917 was made in August 2019. I will try next week to rebase and update this PR for the next Kubernetes release.

Thank you,
Do you think a solution based on VolumeDevices or VolumeMount could be implemented ?
Best Regards
Clement

I don't think, because Volume Devices/Mount has input validation that don't work with directx class id

Ok thank you reply.
I managed to find a way to update my kubelet with the relased in this repo but I got an error :
The k8s pause container cannot be created: (logs from the docker engine running on the windows host)
'''
'"PAUSE <CON"' is not recognized as an internal or external command,
operable program or batch file.
'''
Do you know the reason of that  ?
Best Regards

on windows server 2019 (1809 or higher) ?

Hi,
I Investigated and I found out that my issue is related with an update of the docker engine. My cluster has been set up for docker 18.xx and the issue happened right after the update to 19.03.
It is not related to your kubelet.
Sorry for bothering
Clement

Hi,
I Investigated and I found out that my issue is related with an update of the docker engine. My cluster has been set up for docker 18.xx and the issue happened right after the update to 19.03.
It is not related to your kubelet.
Sorry for bothering
Clement


ATB-141: Using managed cache policy to disable distribution's cache

Proposed changes
Update the SAM Template and ensure the CloudFront Distribution, called CloudFrontDistribution using the CachingDisabled managed cache policy and removed unnecessary Checkov exclusion
ATB-141
What changed
Update the SAM template, specifically the CloudFront Distributions's Cache Policy Id, to use the managed CachingDisabled cache policy.
Why did it change
INCIDENT-67
Issue tracking
INCIDENT-67
Checklists
Environment variables or secrets

[x] No environment variables or secrets were added or changed

Looks sensible. I'm going to deploy this to the dev environment to check it before approving

Manual Testing & Verification -  COMPLETED
Verification Steps Taken
:check_mark: @Alex Whitehead-Smith & @Taher Kapasi - Recreated the bug in dev environment using equivalent version that was deployed into prod.
:check_mark: @Alex Whitehead-Smith & @Taher Kapasi - Deployed the fixed version into dev environment and retested - unable to reproduce error after repeatedly refreshing for 3 or 4 minutes and ensuring we were both refreshing multiple times a second.
:check_mark: Merged to main, deployed to build and promoted to staging and integration environments.
:check_mark: @Alex Whitehead-Smith & @Taher Kapasi - Retested the fixed version in the integration environment - unable to reproduce error after repeatedly refreshing for 2 minutes and ensuring we were both refreshing multiple times a second.
:check_mark: Enabled the Deploy step in pipeline in the production environment and confirmed that the stack was updated correctly.
:check_mark: @Alex Whitehead-Smith & @Taher Kapasi - Retested the fixed version in the production environment - unable to reproduce error after testing repeated refreshing for 2 minutes as well as ensuring we both were refreshing during the same second.


Modify yorkie-cluster helm chart k8s version (1.24 => 1.23)

What this PR does / why we need it:

Modify yorkie-cluster helm chart k8s version (1.24 => 1.23)

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

Can I get an explanation of why this change is needed?

we want to use 'n3r Dedicated Cluster' to deploy yorkie with the same helm configuration in naver internal environment.
1.23 version is required to use n3r Dedicated Cluster.

@krapie Thank you for checking compatibility.


Add info:
Status: Open

fixed


Add basic analytics

What problem does the pull request solve?
Closes #80 #82 #78
Checklist

[x] I've used the pull request template
[x] I've linked this PR to the relevant issue (if mission work)
[x] I've written unit tests for these changes (if code change)
[n/a] I've updated the documentation in (If any documentation requires updating)

[n/a] README.md
[n/a] Elsewhere (please link)

Sample log output:
Hitting the base URL of a form (e.g. )
[form_visit] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217","method":"GET","form":"Register your UEFA Women's Euro 2022 winner's medal","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Saving the first answer in a form
[first_page_save] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217/110/save","method":"POST","form":"Register your UEFA Women's Euro 2022 winner's medal","question_text":"What is your name?","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Saving a subsequent answer on the form
[page_save] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217/111/save","method":"POST","form":"Register your UEFA Women's Euro 2022 winner's medal","question_text":"What position do you play in?","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Hitting the check answers page for a form
[form_check_answers] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217/check_your_answers","method":"GET","form":"Register your UEFA Women's Euro 2022 winner's medal","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Saving an answer in the change answers flow:
[change_answer_page_save] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217/110/save?changing_existing_answer=true","method":"POST","form":"Register your UEFA Women's Euro 2022 winner's medal","question_text":"What is your name?","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Submitting the form:
[form_save] {"url":"https://forms-runner-dev.london.cloudapps.digital//form/217/submit_answers","method":"POST","form":"Register your UEFA Women's Euro 2022 winner's medal","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:104.0) Gecko/20100101 Firefox/104.0"}

Looks great to me


Fix w&b integration (again)

Describe changes
I implemented/fixed _ to achieve _.
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

Types of changes

[ ] 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)
[ ] Other (add details above)

included in https://github.com/zenml-io/zenml/pull/2124


[ISSUE #3185]remove the arm machine test in travis ci

Make sure set the target branch to develop
What is the purpose of the change
#3185
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.006%) to 54.039% when pulling c63412681a3955a62f3ccf0c6d1837306c355d40 on guyinyou:dev into 36e6687d50b2de14d878dbc68e1b99fe254fed67 on apache:develop.


Deserilization to type String

I have added the column 'created' to the table 'AppUser' in the example-postgresql
CREATE TABLE IF NOT EXISTS AppUser (
    id                      varchar (256) PRIMARY KEY,
    username                varchar (256) NOT NULL,
    email                   varchar (256) NOT NULL,
    password                varchar (256) NOT NULL,
    role                    varchar (256) NULL,
    created                 timestamp DEFAULT now(), -- NEW COLUMN --
    CONSTRAINT              UK_APPUSER_USERNAME UNIQUE (username),
    CONSTRAINT              UK_APPUSER_EMAIL UNIQUE (email)
);

I have modified 'UserDto.hpp' to add the field as String  DTO_FIELD(String, created, "created");
but when I try to test the endpoint of getUserById to select some user, I get the following error
{
  "status": "ERROR",
  "code": 500,
  "message": "[oatpp::postgresql::mapping::Deserializer::deserializeString()]: Error. Unknown OID."
}

as I understand from the error message I have to perform a conversion type to take the timestamp value to string
How can I do this?

Hello @Quijos12 ,
You have to cast timestamp to string on PostgreSQL side, before returning DB result.
Please see this StackOverflow answer - https://stackoverflow.com/questions/15990344/select-date-timestamp-from-postgresql-as-string-char-beware-of-null-value


Decide what Android compile SDK to use in flutter/plugins

With the new option of having a compile SDK version that's controlled by Flutter, we have two options for flutter/plugin examples:

Use that version. This will mean we'll stay up to date without having to manage it, but also means we'll get new warnings as out-of-band errors—which if we fix https://github.com/flutter/flutter/issues/91868 would mean out-of-band tree closures
Hard-code a version. That avoids any out-of-band changes, but means we would need to have a process of manually updating it regularly in order to address new deprecation warnings (which end users will see regardless, so are something we don't want to ignore indefinitely).

/cc @blasten

For the reasons described in https://github.com/flutter/flutter/issues/93813, we should do (2). That tooling will solve the "need a manual update process" issue, because examples will be rolled forward when we update the plugin itself (either to use new APIs, or to fix warnings for clients who are on newer target SDKs).


[BUG] AKS Updating AgentPools with custom route table is not allowed

Rancher Server Setup

Rancher version: v2.6.9
Installation option (Docker install/Helm Chart):

Helm Chart: stable-rancher-2.6.9
Kubernetes Cluster: v1.23.5

Proxy/Cert Details: n/a

Information about the Cluster

Kubernetes version: 1.24.6
Cluster Type (Local/Downstream):

downstream

Hosted = AKS Cluster

User Information

What is the role of the user logged in?

Rancher Admin and Cluster Owner

Describe the bug
For AKS private cluster with network-plugin kubenet, with userDefinedRouting and imported in Rancher no update cluster update operations can be performed.
To Reproduce

Create AKS private cluster in following way:

az aks create --resource-group <RESOURCE_GROUP> --location eastus --name <CLUSTER_NAME> \
                        --network-plugin kubenet --network-policy calico --vnet-subnet-id "/subscriptions/..." \
                        --ssh-key-value /root/.ssh/id_rsa.pub --enable-private-cluster \
                        --enable-managed-identity --assign-identity "/subscriptions/..." \
                        --outbound-type userDefinedRouting

Import cluster in Rancher
Try to update AgentPool size or enable autoscaling.

Result
time="2022-10-21" level=info msg="Updating maximum count in node pool [...] for cluster [....]"
time="2022-10-21" level=error msg="error syncing 'cattle-global-data/...': handler aks-controller: failed to update cluster: containerservice.AgentPoolsClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code=\"CustomRouteTableInvalidUpdateAttempt\" Message=\"Updating custom route table is not allowed. Custom route table was originally \\\"/subscriptions/...\\\" and a newly associated route table of \\\"\\\" was detected. Please revert back to the original route table. Please see https://aka.ms/aks/customrt for more information\", requeuing"
time="2022-10-21" level=info msg="Checking configuration for cluster [...]"

Expected Result

Screenshots

Additional context

QA TEST PLAN
Scenarios

Scenario
Test Case

1
As a standard user, import an AKS cluster with the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases

2
As a standard user, import a private AKS cluster with the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases

3
As a standard user, import an AKS cluster with the second highest K8s version available, upgrade the K8s version to the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases

4
As a standard user, import a private AKS cluster with the second highest K8s version available, upgrade the K8s version to the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases

@sowmyav27 I have finished testing it on Rancher v2.7.2-rc5 with aks-operator and https://github.com/rancher/aks-operator/pull/146

Additionnal QA
Issue: #40087
Problem
For AKS private cluster with network-plugin kubenet, with userDefinedRouting and imported in Rancher no update cluster update operations can be performed.
Solution
We added the code which is passing node pool's subnet id to azure, during perfoming node pool update.
Testing
Create Azure virtual subnet, create AKS private cluster and import it to Rancher. After it try to perform Node Pool update e.g. updating min or max nodes number.
Engineering Testing
Manual Testing

Create Azure Virtual Network

Create AKS private cluster in following way:

az aks create --resource-group <RESOURCE_GROUP> --location eastus --name <CLUSTER_NAME> \
                        --network-plugin kubenet --network-policy calico --vnet-subnet-id "/subscriptions/..." \
                        --ssh-key-value /root/.ssh/id_rsa.pub --enable-private-cluster \
                        --enable-managed-identity --assign-identity "/subscriptions/..." \
                        --outbound-type userDefinedRouting

Import it in Rancher

Try to make Node Pool update from Rancher, e.g. change min or max number of nodes in Node Pool.

Automated Testing
Integration test exists.

Validated this on v2.7-head https://github.com/rancher/rancher/commit/2b2ed443065a9f38c4c5764461be33823eb6fec3. See details below:
ENVIRONMENT DETAILS

Rancher install: Docker
Rancher version: v2.7-head https://github.com/rancher/rancher/commit/2b2ed443065a9f38c4c5764461be33823eb6fec3
Browser: Chrome

TEST RESULT

Scenario
Test Case
Result

1
As a standard user, import an AKS cluster with the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases
:white_check_mark:

2
As a standard user, import an AKS cluster with the second highest K8s version available, upgrade the K8s version to the highest K8s version available, edit the cluster configuration and increase the agentpool size, validate that the agentpool increases
:white_check_mark:

VALIDATION STEPS
Scenario 1

In Azure portal, created the following resources in my resource group:

Virtual network with the default subnet
Route table configured with my default subnet just created
User managed identity with Owner permissions as a role assignment

Navigated to a Azure VM with az installed and already logged into my Azure subscription.

Ensured the Azure VM is in the same region as the VNET and route table.

Ran command az aks create --resource-group <resource group> --location eastus2 --name <cluster name> --network-plugin kubenet --network-policy calico --vnet-subnet-id "/subscriptions/…/Microsoft.Network/virtualNetworks/<virtual network>/subnets/default" --ssh-key-value /…/id_rsa.pub --enable-private-cluster --enable-managed-identity --assign-identity "/subscriptions/…/Microsoft.ManagedIdentity/userAssignedIdentities/<user identity name>" --outbound-type userDefinedRouting --service-cidr <IP_ADDRESS>/16 --dns-service-ip <IP_ADDRESS>
Validated that the private AKS cluster successfully deployed in my Azure resource group.
Navigated to Rancher > Cluster Management > Import Existing > Azure AKS.
Imported my private cluster into Rancher; validated this was successful and the cluster and nodes came up as Active in Cluster Management and Cluster Explorer.
Validated that all pods in the system project page came up as Active.
Edited the cluster's configuration and increased the node pool to have maximum of 5 nodes.
Validated that the node pool increased to 5 nodes and the cluster and nodes are Active in Cluster Management and Cluster Explorer.

Scenario 2

Repeated scenario 1, steps 1-8, except ensured that my private AKS cluster is at v1.24.9 and not v1.25.5.
Edited the cluster's configuration and updated the K8s version to v1.25.5 for BOTH cluster and agent node pool.
Validated that the cluster and nodes upgraded to v1.25.5 in Cluster Management and Cluster Explorer.
Validated that all pods in the system project page remained as Active.
Edited the cluster's configuration and increased the node pool from 5 nodes to 8 nodes.
Validated that the node pool increased to 8 nodes and the cluster and nodes are Active in Cluster Management and Cluster Explorer.


Unable to edit wof:name

I'm trying to update the wof:name value of Zerakpur, India (134/373/049/7/1343730497.geojson), but hitting the following issue:

Steps are fairly simple..

Update the wof:name value from "Zerakpur" to "Zirakpur"
Click out of the edit box so the green check box appears
Hit the "Propose change on Github" button
See error

I also tried updating additional properties along with the wof:name, the same error appeared.

This should be fixed now. I was looking in the wrong spot on the JSON document for wof:name when checking if it was set.

Nice, this is fixed!


%adl listaccounts is not working: 'DataLakeAnalyticsAccountManagementClient' object has no attribute 'account'

Describe the bug
%adl listaccounts is not working
To Reproduce
Steps to reproduce the behavior:
%reload_ext adlmagics
AdlMagics initialized
%adl login --tenant microsoft.com
Waiting for user login...
To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code G3Y73CEWX to authenticate.
User<EMAIL_ADDRESS>logged in to tenant 'microsoft.com'
%adl listaccounts
Listing azure data lake analytics accounts...
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 get_ipython().run_line_magic('adl', 'listaccounts')
c:\program files\python36\lib\site-packages\IPython\core\interactiveshell.py in run_line_magic(self, magic_name, line, _stack_depth)
2129                 kwargs['local_ns'] = sys._getframe(stack_depth).f_locals
2130             with self.builtin_trap:
-> 2131                 result = fn(*args,**kwargs)
2132             return result
2133
 in adl(self, line, cell)
c:\program files\python36\lib\site-packages\IPython\core\magic.py in (f, *a, **k)
185     # but it's overkill for just that one bit of state.
186     def magic_deco(arg):
--> 187         call = lambda f, *a, **k: f(*a, **k)
188
189         if callable(arg):
c:\program files\python36\lib\site-packages\adlmagics\adlmagics_main.py in adl(self, line, cell)
65
66         magic = self.__magics[cmd]
---> 67         return magic.execute(arg_string, cell)
68
69     def __register_azure_magic(self, azure_magic_class):
c:\program files\python36\lib\site-packages\adlmagics\magics\adla\adla_accounts_listing_magic.py in execute(self, arg_string, content_string)
15         self._write_line("Listing azure data lake analytics accounts...")
16
---> 17         adla_acounts = self._adla_service.retrieve_accounts(page_index = args.page_index, page_account_number = args.page_account_number)
18
19         self._write_line("(%d) azure data lake analytics account(s) listed." % (len(adla_acounts)))
c:\program files\python36\lib\site-packages\adlmagics\services\adla_service_sdk_impl.py in retrieve_accounts(self, page_index, page_account_number)
23         for sub in sub_client.subscriptions.list():
24             dla_client = DataLakeAnalyticsAccountManagementClient(self.__token_service.credentials, sub.subscription_id)
---> 25             accounts.extend([AdlaAccount(account.name) for account in dla_client.account.list()])
26
27         accounts.sort(key = lambda account: getattr(account, "name"))
AttributeError: 'DataLakeAnalyticsAccountManagementClient' object has no attribute 'account'
Expected behavior
Account should be listed
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS
Win10
Browser
chrome
Jupyter Version
Python 3.6.2, Jupyter notebook 5.5.0
ADS Notebook Version
adlmagics-0.0.1

Additional context
Add any other context about the problem here.

duplicate with #8

We have updated the dependencies associated with azure-data-lake-store in version 0.0.1, lead to the corresponding interface has changed. You may need to manually update the dependencies locally by typing
pip install -U azure-mgmt-datalake-store
in command prompt , and re-start you jupyter-notebook.
I think this problem related issue#21 . In fact, the reason for this problem is that we updated the interface in version 0.0.1, but did not update the dependencies in the package provided to users. We will sovle this problem fundamentally in version 0.0.2, and you can see progress in  issue#21


[move-prover] Upgrade Z3 version

Bump up the Z3 version from 4.8.9 to 4.8.12

according to the release note, the newest version has some update on the arithmetic solver, and also has many general bug fixes

The running time of MVP_TEST_ON_CI=1 cargo test -p move-prover --release almost remains the same (comparing "before" and "after").

MVP_TEST_ON_CI=1 was given in this benchmark test to exclude the cvc4 tests. The cvc4 tests are not related to the Z3 performance, and the cvc4 tests don't work at the moment.

Commented out an unused line from diem/shuffle/move/examples/main/context.ts because the deno linter complains about it in CI

See https://github.com/diem/diem/runs/4250122831?check_suite_focus=true#step:7:38

Motivation
To upgrade the Z3 version
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
cargo test

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land

:lock: Merge Conflict

/land


[5.4] Blade @stack order undocumented breaking change

Laravel Version: 5.4
PHP Version: 5.6
Database Driver & Version: N/A

Description:
I've just updated from 5.3. to 5.4 and #16325 has introduced an undocumented breaking change.
Consider
// layout.blade.php
@push('js')
<script src="jquery.js"></script>
@endpush

@stack('js')

// child.blade.php
@extends('layout')

@push('js')
<script src="script-that-depends-on-jquery.js"></script>
@endpush

In 5.3, this would work. In 5.4, it does not.
The 5.3 functionality is obviously what is desired: adding common scripts to the layout that are always required, such as jquery, and then pushing to the stack in the called child view to add scripts specific to that view.
With this broken in 5.4, I'm not sure what the utility of the stack actually is.

Please clear your view cache, and if it's still broken, please submit a PR to our upgrading guide.
Thank you. :)

@GrahamCampbell The thing is, #16325 specifically discusses this change and then comes to the conclusion that the 5.4 way is correct, but I don't see how it is. The template inheritance system conventionally allows default in the parent and the overrides/extensions in the child. I'm not sure why it was changed, because it was working as intended in 5.3, and now pushing onto the stack gives you an outcome that is useless for what the stack was created for: assets.
To be honest, if this change remains, the documentation will need a note on stacks saying "don't use these with @extends and @section because you'll get unexpected results".
I think it'd be good to get @jbrooksuk and @mark86092's thoughts on this, and maybe even @taylorotwell's.

Looks like this has went back and forth a few times with the addition then removal of array_reverse. I've always thought of it the way it worked in 5.3 but as Taylor points out, the fact that it is "pushing" onto a stack, it seems to make more sense in the 5.4 usage. Since there are other ways to handle script inheritance in blade, it would probably be best to note this in the documentation and leave as-is. I'm guessing, due to the lack of responses, that not many apps are actually affected by this. However it is early in the release so there will probably be more in the coming weeks.

[...] To me push should render its contents in the order they were pushed. Otherwise its not a push its just a prepend.
Source: https://github.com/laravel/framework/pull/16325#issuecomment-260718331

The 5.4 ordering is correct. Whatever called push first will be displayed first, etc... Many just don't truly understand the ordering of when layouts / sections, etc. are rendered which can be make it seem like its not functioning this way.

In the example at the top of this page... Children are rendered before layouts. So, the childs stuff is first in the stack.

Using Components and slots can sometimes help this make more sense since they provide a more natural ordering of rendering.

@taylorotwell Thank you for your reply. I understand what you are saying about the ordering of rendering of parent/children dictating the order that pushing to the stack is actually happening, but I believe that the example of adding jquery dependent scripts in the child is a (the?) common use for the stack. CSS file order is also important.
Would adding the ability to output the stack in reverse be a solution e.g. @stack('js', true) or @stackReverse('js') or something along those lines? I'm just trying to think how to keep this functionality available for users of @extends/@section as well as @component/@slot. As it is, I'm going to have to convert back @section/@parent, which does the job fine, albeit not quite as elgantly or semantically.
Either way, I think information about this needs adding to the @stack documentation, and it definitely needs adding to the 5.4 upgrade guide (I would open a PR on the documentation, but I worry I'd write it too tersely/not match the tone).

Why even use "push" in your layout? It makes no sense. Just put the scripts in your header tag directly.

@ockle Are you adding the jquery.js script-tag from your layout? The layout file is executed after the view file, and anything pushed in the layout file will be appended after whatever the view or subviews pushes.
If your layout always outputs a jquery.js script-tag, just add it as a normal tag in your header.

// layout.blade.php
<script src="jquery.js"></script>

@stack('js')

@taylorotwell Yes, perhaps that example was too simple. Here's a better one:
// top.blade.php
<script src="jquery.js"></script>
<link href="main.css" rel="stylesheet">
@stack('test')

// middle.blade.php
@extends('top')

@push('test')
<script src="script-that-depends-on-jquery.js"></script>
<link href="css-that-overrides-main.css" rel="stylesheet">
@endpush

// bottom.blade.php
@extends('middle')

@push('test')
<script src="script-that-depends-on-script-in-middle.js"></script>
<link href="css-that-overrides-middle.css" rel="stylesheet">
@endpush

view('bottom')

Now that there are three levels, the inheritance order requirement becomes more apparent. Perhaps it could be argued that this is now less likely to be encountered, maybe even contrived, as I don't think inheriting more than one level is super common, but it is still possible in Laravel and ocassionally useful. In this situation, I think it is clear what the developer is wanting to happen, but to achieve it, it has to be changed to:
// top.blade.php
<script src="jquery.js"></script>
<link href="main.css" rel="stylesheet">
@yield('test')

// middle.blade.php
@extends('top')

@section('test')
<script src="script-that-depends-on-jquery.js"></script>
<link href="css-that-overrides-main.css" rel="stylesheet">
@endsection

// bottom.blade.php
@extends('middle')

@section('test')
@parent
<script src="script-that-depends-on-script-in-middle.js"></script>
<link href="css-that-overrides-middle.css" rel="stylesheet">
@endsection

which is fine, works, and is what we were doing before @stack was introduced. I just feel that this inconsistency in the way @stack works depending on whether you are using @layout/@section or @component/@slot is a bit awkward. Perhaps it has to be too many things to be too many people: to those using each type of inheritance model, as well as to those expecting it to facilitate the example above and to those expecting it to rigidly push in the order it is called.
Hopefully you can see where I'm coming from with this, and apologies for confusing what I'm trying to exhibit here with my over-simplified initial example.

The @extends indicate that the views is the "root-view" (for the lack off better word) and that the file mentioned in the @extends is the layout. As previously mentioned, layouts are rendered after the view. It looks like you're chaining several layout files, which will render every layout in the reverse order compare to what you expect (bottom, middle, top)

5.4 is the correct and expected behaviour IMHO.

If we have a choice to decide either append or prepend on stack/push will be the best!


RPC NOT WORKING

when i was using
curl -X POST http://localhost:10332/ -H 'Content-Type: application/json' -d '{ "jsonrpc": "2.0", "id": 5, "method": "getblocksysfee", "params": [13321] }'
its saying
curl: (7) Failed to connect to localhost port 10332: Connection refused
my neo-cli node is running rpc flag
is there any issues with neo-cli

@KanishTheVista  Be sure your offline sync has completed. During the sync by offline package, your neo-cli will not communicate with any request and the node seems 0. When the use of package has been completed, there'll be numbers on nodes. Then rpc will be fine.


bower: Fix ui-components to 0.0.11

since manageiq-ui-components is not quite semver yet, removing the ~ from the version contstraint and updating to 0.0.11 which was the version used before today.
This should fix ui-classic test failures caused by the new version.
(Updating to 0.0.12 in a separate PR, since that brings in more dependencies.)
(And hoping to remove bower.json from manageiq before then.)

@miq-bot add_label euwe/yes
@simaishi There's no BZ but the build will definitely need this, otherwise no toolbars in the UI..

Since the change is needed to keep the current version unchanged, I think we can get this in without BZ..


If the title changes, the Transform should change as well

Report
Didn't really work in the demo.
When a user changed the notebook title, running ln.context.track() doesn't detect the change.
Version information
NA

This has always been tested and I'm not aware that it was ever broken. It's also working right now:

Tentatively closing. Let's chat about what gave you the impression it wasn't working!