addObjectWhereUsed producing a lot of warnings/errors

Describe the bug
This command is producing very unexpected output:
pan-os-php type=address in=api://panorama location=any actions=addObjectWhereUsed:new-host 'filter=(name eq current-host)'
Expected behavior
The new-host address object should be added wherever the current-host address object appears.
Current behavior
I get lots of about address objects or groups in address groups as well as errors about address groups being involved in loop usage:

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: BI-Cluster is added as subgroup to servicegroup: MSSQL-Servers
XML line #630761, XPATH: /config/shared/address-group/entry[@name='BI-Cluster']

** WARNING ** * addressgroup with name: App-Servers-2 is added as subgroup to servicegroup: PRD-App-Servers-1
XML line #633990, XPATH: /config/shared/address-group/entry[@name='App-Servers-2']

** WARNING ** * addressgroup with name: Active-Directory-Clients-2 is added as subgroup to servicegroup: Active-Directory-Clients-1
XML line #630253, XPATH: /config/shared/address-group/entry[@name='Active-Directory-Clients-2']

** WARNING ** * addressgroup with name: All-User-Nets-2 is added as subgroup to servicegroup: All-User-Nets-1
XML line #633131, XPATH: /config/shared/address-group/entry[@name='All-User-Nets-2']

** WARNING ** * addressgroup with name: Domain-Controllers-2 is added as subgroup to servicegroup: Domain-Controllers-1
XML line #629835, XPATH: /config/shared/address-group/entry[@name='Domain-Controllers-2']

** WARNING ** * addressgroup with name: BI-Cluster is added as subgroup to servicegroup: PRD-Windows-Servers-1
XML line #630761, XPATH: /config/shared/address-group/entry[@name='BI-Cluster']

** WARNING ** * addressgroup with name: All-GlobalProtect-Nets is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #632401, XPATH: /config/shared/address-group/entry[@name='All-GlobalProtect-Nets']

** WARNING ** * addressgroup with name: App-Servers-2 is added as subgroup to servicegroup: PRD-App-Servers-1
XML line #633990, XPATH: /config/shared/address-group/entry[@name='App-Servers-2']

** WARNING ** * addressgroup with name: PRD-App-Servers-1 is added as subgroup to servicegroup: PRD-Servers-2
XML line #629572, XPATH: /config/shared/address-group/entry[@name='PRD-App-Servers-1']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: VPN-Employee-Nets is added as subgroup to servicegroup: Exchange-Clients
XML line #628748, XPATH: /config/shared/address-group/entry[@name='VPN-Employee-Nets']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: User-Networks-3 is added as subgroup to servicegroup: SharePoint-Clients
XML line #624931, XPATH: /config/shared/address-group/entry[@name='User-Networks-3']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: Security-Dept
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-NC-NETWORKS
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: Security-Dept
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: SSL-VPN-NC-NETWORKS
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: BI-Cluster is added as subgroup to servicegroup: PRD-Windows-Servers-1
XML line #630761, XPATH: /config/shared/address-group/entry[@name='BI-Cluster']

** WARNING ** * addressgroup with name: VPN-Employee-Nets is added as subgroup to servicegroup: Patch-Clients
XML line #628748, XPATH: /config/shared/address-group/entry[@name='VPN-Employee-Nets']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Security-Dept is added as subgroup to servicegroup: WSUS-Clients
XML line #624807, XPATH: /config/shared/address-group/entry[@name='Security-Dept']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: Security-Dept
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: SSL-VPN-NC-NETWORKS
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: Security-Dept
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-NC-NETWORKS
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Active-Directory-Clients-1 is added as subgroup to servicegroup: Active-Directory-Clients-2
XML line #626760, XPATH: /config/shared/address-group/entry[@name='Active-Directory-Clients-1']

** WARNING ** * addressgroup with name: All-User-Nets-2 is added as subgroup to servicegroup: All-User-Nets-1
XML line #633131, XPATH: /config/shared/address-group/entry[@name='All-User-Nets-2']

** WARNING ** * addressgroup with name: BI-Cluster is added as subgroup to servicegroup: PRD-Windows-Servers-1
XML line #630761, XPATH: /config/shared/address-group/entry[@name='BI-Cluster']

** WARNING ** * addressgroup with name: All-GlobalProtect-Nets is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #632401, XPATH: /config/shared/address-group/entry[@name='All-GlobalProtect-Nets']

** WARNING ** * addressgroup with name: Domain-Controllers-2 is added as subgroup to servicegroup: Active-Directory-Clients-2
XML line #629835, XPATH: /config/shared/address-group/entry[@name='Domain-Controllers-2']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: App-Servers-2 is added as subgroup to servicegroup: PRD-DirService
XML line #633990, XPATH: /config/shared/address-group/entry[@name='App-Servers-2']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: All-User-Nets-1 is added as subgroup to servicegroup: CitrixDeliveryAgents
XML line #624813, XPATH: /config/shared/address-group/entry[@name='All-User-Nets-1']

** WARNING ** * addressgroup with name: All-GlobalProtect-Nets is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #632401, XPATH: /config/shared/address-group/entry[@name='All-GlobalProtect-Nets']

** WARNING ** * addressgroup with name: All-GlobalProtect-Nets is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #632401, XPATH: /config/shared/address-group/entry[@name='All-GlobalProtect-Nets']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup with name: Portal-InfoSec is added as subgroup to servicegroup: VPN-Employee-Nets
XML line #628167, XPATH: /config/shared/address-group/entry[@name='Portal-InfoSec']

** WARNING ** * addressgroup: IN-Exchange-Servers-dupe is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Security-Dept is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: InfoSec-Scanning-Servers is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: V-Agents is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: SIEMs is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Servers-for-Monitoring is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: ssh-clients-1 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: ssh-clients-2 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: ssh-servers-2 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: SSH-Servers-3 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: BI-Servers is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-Windows-Servers-1 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-DirService is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-App-Servers-2 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-LAP is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Scanning-Server-Clients is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: BUILDGRP-RDP-Targets is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-4 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Storage-MGMT is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Portal-Destinations is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-Address-Parsers is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: VPN-User-PAT-Targets is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Cluster-IPMI-All is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Monitoring-Clients is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: PRD-PhoneVal-Servers is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-5 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Web-Servers-6 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: DNS-Servers-External-Queries is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Domain-Controllers-SC is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: DNS-Servers is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: FW-Users-Nets-All-3 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: FW-Users-Nets-All-3-CIFS is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: DNS-Servers-Internal-Queries is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: SSL-VPN-NETWORKS is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: User-Nets-4 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Web-Application-Users-5 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: SSO-Remote-User-Nets is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-7 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-8 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: CitrixDeliveryAgents is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-9 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-10 is maybe not listed as it is involved in a loop usage

** WARNING ** * addressgroup: Clients-11 is maybe not listed as it is involved in a loop usage

** ERROR ** * cannot determine groups dependencies after 40 loops iterations: is there too many nested groups?

In addition, note that for the initials warnings where an XML line is listed.  All of them are listed as a line >600K.  My Panorama XML config maxes out at 329,918 lines.
Possible solution
Unknown - it used to work fine.  I've run this command many, many times and this is the first time I received any kind of error that wasn't due to operator error (typo, missing object, etc.).
Steps to reproduce
run this command, where new-host and current-host are existing address objects, new-host is not currently a member of any address group, and current-host is a member of 3 address groups (none of which are listed in the errors/warnings):
pan-os-php type=address in=api://panorama.domain.com location=any actions=addObjectWhereUsed:new-host 'filter=(name eq current-host)'
Context
I need to setup firewall rules for 6 new hosts that are replacing 6 current hosts.  I only tried the first one.
Your Environment

Version used: 2.0.55
Operating System and version (desktop or mobile): Windows 10

this is NOT a bug, [only one part there is  a typo in the warning "service group"]
this is an improved validation of your configuration.
your addressgroups are creating a loop.
And as you where running this many times before,
one of your last configuration changes brought this up.

you will get the same warning / error if you are running only with DISPLAY, also if no filter is set
pan-os-php type=address in=api://panorama location=any actions=display 'filter=(name eq current-host)'

All of them are duplicates or just the ones that warn about loops?
Also, what about the XML line numbering?

regarding line numbering,
I recommend to load your XML config file into a XML reader where you can generate a pretty XML;
then you will see the line mentioned here.

no clue where you are seeing duplicates, please the loop detection is working this way:

address-group example:

grpA has member grpB
grpB has member grpC
grpC has member grpA

Yeah, I get the loop part of it (we've had that problem before and on Juniper firewalls).  I spot checked one group with which I'm very familiar and didn't find a loop.  It's a group we no longer need (there's an address object in it that is the only thing needed so I'm swapping out the group for that object and I'll see if that one goes away.

I spot checked a few of the flagged groups and none of them had loops nor did they have a large number of members or nested groups.

Deleting a flagged group dropped it out of the list of errors, as expected.

version 2.0.57 - removed the additional loop check

if it is not done - why you are not reopen this issue?

if the actual fix is not helping you, I recommend to to share more detailed information about the behaviour and also demo config where this problem is happen.
Right now, I can not reproduce your situation

I still get the same exact errors on 2.0.58 as I did on 2.0.55 (minus the groups I have since deleted or removed any nested groups).
I can't share it publicly.  Also, my Panorama config has over 300K lines in it.  I've been going through the groups that have the warnings/errors above (which have been sanitized but I know how I translated them) and cleaning them up where I can.  I also checked for loops, too many group members, and groups with a lot of nesting (I don't know what the limit is).  So far, the deepest nesting is 3 (a group within a group within a group), but I haven't checked all of them, yet.

Correction: The "...is maybe not listed as it is involved in a loop usage" lines have disappeared after at least one of my group deletions.

so you are still getting this error message:
"** ERROR ** * cannot determine groups dependencies after 40 loops iterations: is there too many nested groups?"
As this is specific to your configuration file, and I still can not reproduce your exact issue.
I assume you still have my mail address, please let us move there to discuss how we can proceed

Okay, with that fixed, the "...40 loop iterations..." message went away and the command I was trying to run finished.  There were still the Warnings, however.  I'll need to dig into those to see why.
Do the Warning messages happen any time there are nested groups?

in the next version; these warnings will disappear.
and if a loop is found, it will at least point you into the Device-Group where this is happen, and also display the relevant address-groups with its subgroups.
2.0.58 will be reused for this and massiv extended also with also new features and some bugfixes for other parts

please validate with latest version,
version is still 2.0.58, but with extended feature

I ran the same command, but as an action=display and all of the Warnings went away.  Thanks!

then I am closing this


♻ StateMachine refactor and cleanup

Revisiting our experience state machine structure after a long time to see if there is some room for improvements, also trying to figure out testable pieces (unit test), the focus of this change is around deleting the Transitions and changing the StateMachine implementation so that:

State is now an interface, each State implementation needs to handle actions, returning null means that this action is not valid for this state

internal interface State {
    fun take(action: Action): Transition?
}

SideEffect is now an interface, and instead of returning ResultOf<State, Error> sideEffects can produce a nextAction, returning non-null will re-invoke handleActionInternal(newAction) which is basically what was happening before but it is now simpler.

internal interface SideEffect {
    suspend fun launch(processor: ActionProcessor): Action?
}

so for instance ContinuationEffect(action) returns  action, while AwaitEffect(action) returns the action after await is satisfied.

Transition takes in optional error: Error?

internal class Transition(
    val newState: State,
    val error: Error?,
    val sideEffect: SideEffect?
)

StateMachine handleInternal now runs the following

take action into transition
update current state/emit
check if we have any error, if so we emit and return Failure(error)
check if we have any sideEffect, if so we apply and re-invoke handleInternalAction(next), else we Success(state)

With these core changes we have some other small changes too, the most notable one is that ViewModel now listen to RenderingStep instead of BeginningStep this is because instead of relying on State extension functions to produce the metadata we are not moving to BeginningStep with PresentationEffect that does all the work and produce the RenderStep action with valid metadata or else it ReportError
Listening to BeginingStep was important before because we used to callback a "rendering completion" which is no longer done, by the time the ViewModel is collecting state we assume the experience is showing already, so there is not practical difference to this change aside from better organizing the state machine flow.
BeginningExperienceState
|--> next(RenderingStepState, PresentationEffect)
    |--> produceMetadataWithRetry()
    |--> presentingTrait.present()
    |--> return action RenderStep(metadata)

EndingStepState
|--> next(RenderingStepState(index), PresentationEffect(index))
    |--> produceMetadataWithRetry() or produceMetadata()
    |--> presentingTrait.present()
    |--> return action RenderStep(metadata)

other changes:

change: StepReference package to model, since its not a state machine specific class
change: config.experienceListener calls to ExperienceLifecycleTracker to avoid creating extra collects + taking responsibility from state machine
add: getNavigationActions/areStepsFromDifferentGroup/getPresentingTrait/ isValidStepIndex to Experience class
change: RenderStep action now carries metadata to ViewModel
delete: BeginningStepExt and moved all methods to other places (State, SideEffect)
Added: Bunch of unit tests!

After I was done with implementing a the new structure, I followed up with creating tests for each state and sideEffect that we have, and that led me into changing some of the existing code, during every turn I kept checking existing tests and accommodating it accordingly.


Make BitcoindRpc a dyn newtype

LGTM. Wondering about the IBitcoindRpc name. Why the I. Going to defer to @elsirion for merging because I'm not an expert on rust patterns like this ....

After doing a call with @dpc I'm going to merge this. IBitcoinRpc was because he couldn't think of anything better and the I prefix is used in other languages to mean "interface". Renames are welcome!


Breaking change since 6.5.0 - "of" no longer contains value object

Bug Report
Current Behavior
"of" does not contain a value object
Reproduction

REPL or Repo link:
https://stackblitz.com/edit/observable-of-test-breaking-change?file=package.json

Expected behavior
"of" contains value object, or change made to remove as part of a new major version change
Environment

RxJS version: 6.5.x

Additional context/Screenshots
Previously we were using the value object in of for simplifying unit testing. Upgrading to 6.5.x version causes the value object to disappear so all our unit tests now fail. In the reproduction link you can flip between 6.4.0 and 6.5.0 to see it only appearing in 6.4.0.

Property value is internal and not exposed as public interface in 6.4.0 even. It is not breaking change as this was never public interface consumer intended to use.

@kwonoj Looking at ./src/internal/BehaviorSubject.ts neither option is being marked as private. Even the current official doc states both options. Please advice wether the is the intend on deprecating one of them in the future.

@kwonoj This API change/status seems rather poorly documented - no comments/JSDoc tags/indication of any kind in source or mention in web docs for BehaviorSubject.

I'm in hard doubt doc issue if I remeber correctly: we never expose, document anything for this internal property and unless specified as public user shouldn't assunt it's public interface to use.

For clarity issue is about of and plain observable in here, nothing related with subject. Behaviorsubject have proper punliv value and it always exists.

@kwonoj just to clarify, are you saying it is reasonable to access the current value of a BehaviorSubject via the public value getter?
Example:
const myBehaviorSubjectValue = this.myBehaviorSubject.value;

https://github.com/ReactiveX/rxjs/blob/master/src/internal/BehaviorSubject.ts#L19. Behaviorsubject properly expose value getter and it's purpose of behaviorsubject.
https://rxjs-dev.firebaseapp.com/api/index/class/BehaviorSubject

Double clarification this means you can access current last emitted value via behaviorsubject, doesn't mean it's recommended usecases like this issue's case.

Would you be kind enough to add a quick snippet to demo what the recommended method to access this would be based on this example:
const myBehaviorSubjectValue = this.myBehaviorSubject.value;


bug: ListItem elements with no text

Document types that are being processed directly or indirectly by partition_text sometime produce ListItem elements that have an empty text field.
This was meant to be prevented by this line of code, but I suspect ListItems are slipping through by the following mechanism:

ctext is checked, it's not empty
The text is routed as a ListItem
clean_bullets is applied, but this removes everything from the text
The element is returned

This should be validated as a mechanism by which empty ListItems are slipping through, and the loophole should be closed. Perhaps by moving the check for empty text to after the element is created and just before it is returned.

https://github.com/Unstructured-IO/unstructured/pull/1268

Complete with #1252 ; reopen with comment if a different solution is required


Fix for WFLY-16408, Upgrade to Galleon 5.0.1.Final and Galleon plugins 6.0.0.Alpha2

Issue: https://issues.redhat.com/browse/WFLY-16408

Thanks @jfdenise


Remove old bme280, add new combination bmp/bme280

This PR has 3 commits:
The first one removes the older, somewhat broken bme280 driver.
The second adds a new combination bme280 and bmp280 driver that is fully functional, into a new bmp280 UPM library.  These drivers do not rely on the Bosch driver.
The third commit re-adds the bme280 support to the pressure, humidity, and temperature interface c++ examples that were removed in the first commit.

Merged.  Thanks Jon.


Investigate slow first-time page loads on gitbooks.

Asked Mark Norman<EMAIL_ADDRESS>on July 10 about this issue, they escalated to support team.
Had not heard back from them, asked for an update on July 23.

Have not been able to reproduce recently. Closing.


Document TCP/IP support for GELF driver

See https://github.com/moby/moby/pull/34758

Duplicate of #4536 😊


Dan search tests

@david-a-wheeler
In addition to writing tests to cover project filtering, I also updated our gems, refactored the tests to make them more consistent, and updated the fixtures to make them clearer.

Looks great!


ISPN-10953 doc suggestions

@tristantarrant some minor doc edits for the CLI index command

Merged, thanks


Configure basepath to easily use relative paths for resources (css, images and the like)

From an answer to https://github.com/gutschilla/elixir-pdf-generator/issues/19 I think of a new configuration avlue that cwds into the configured directory and calls wkhtmltopdf from there.

Seems like everyone is just generating from either plain HTML with  absolute paths or just from a URL which serves as preview. Unless this pops up again, I'll close this.
This lingers around since ages. Are you OK and still there @lypborges ? ;-)


Add @maico910 as a contributor for code.

Add @maico910 as a contributor for code.

Add @caio-ribeiro-pereira as a contributor for code.


Devise sign_in TestHelper doesn't trigger Warden after_set_user callback

Environment

Ruby **ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-darwin16] **
Rails Rails 4.2.7
Devise 3.5.10

Current behavior
Warden::Manager.after_set_user except: :fetch do |user, warden, _|
  UserSession.deactivate(warden.raw_session['auth_id'])
  warden.raw_session['auth_id'] = user.activate_session(
    ip: warden.request.ip,
    user_agent: warden.request.user_agent
  )
end

# After fetching a user from the session, we check that the session is marked
# as active for that user. If it’s not we log the user out.
Warden::Manager.after_fetch do |user, warden, _|
  unless user.session_active?(warden.raw_session['auth_id'])
    warden.logout
    throw :warden, message: :unauthenticated
  end
end

I have customized Warden::Manager.after_set_user to set server side session which I can later invalidate if user logout from the application. And it is working as expected if I login to the web application.

problem

I am using Devise::Test::ControllerHelpers to use Devise sign_in method. The problem here is, this sign_in() method doesn't trigger Warden::Manager.after_set_user callback.
But Warden::Manager.after_fetch is triggering if I put a breakpoint in both callbacks.
I see devise sign_in method is using warden.set_user method but sign_in test helper is not.

Expected behavior

When I call sign_in :user in my test case, it should trigger Warden::Manager.after_set_user callback.

Important: you should not test Warden specific behavior (like callbacks)
using Devise::Test::ControllerHelpers since it is a stub of the actual
behavior. Such callbacks should be tested in your integration suite instead.

I have found this Important message in the source code: https://github.com/plataformatec/devise/blob/master/lib/devise/test/controller_helpers.rb#L23.
But now that I have thousands of test cases relying on this behavior, if anyone can help me solve this that would be great.

Hello @userrails, thanks for your report.
As you noticed, we run warden in test mode, so this is an expected behavior. I'm not sure what path you should follow. You can abstract the logic to another object and test this object in isolation, or maybe create integration test as the code's comments suggest.
Thanks.


Added Backtracking algorithm (N-Queen Problem) [Go]

Fixes #535
By submitting this pull request I confirm I've read and complied with the below declarations.

[ ] I have read the Contribution guidelines and I am confident that my PR reflects them.
[ ] I have followed the coding guidelines for this project.
[ ] My code follows the skeleton code structure.
[ ] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[ ] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.

Hi! Thanks for showing interest in contributing to this repository.
Please make sure that you have ticked the points mentioned in PR description correctly.
Thanks again!


alert rules disappear if k8s cluster gets rebooted

Bug Description
During testing microk8s cluster went down. Due to some issue cluster entered CrashLoopBackOff and juju restarted it. Once units went up again, alert rules were empty
To Reproduce
restart k8s cluster with deployed prometheus
check alert rules
Environment

Relevant log output
-

Additional context
No response

This is probably because the config hash is in StoredState on the controller.
As discussed, storing the hash inside the workload container may improve resilience.


Need dartk to parse patch files

We should make the normal dartk compiler be able to read in patch files.

The plan is to generate a pre-patched SDK based on the same tool DDC uses.

Closing the issue for now. DartK expects the prepatched SDK.


First step towards removing the state copying

Description
work still todo:

[x] make migration work
[x] cleanup of code
[x] make cleanup mechanism for the new table
[x] update get_resource_details
[x] get_resources_report
[x] remove last_deploy field
[x] update ResourceView

Type errors added / moved
src/inmanta/data/__init__.py:4929: error: Argument 1 to "loads" has incompatible type "object"; expected "str | bytes | bytearray"  [arg-type]

Open opportunities

track last and first version for simpler get_resources_report and  &get_resource_details*
track last_released for easier ResourceView
get rid of  mark deployed for known good state (biggest issue is tracking when a version is deployed and the dashboard)
add new state fields

Deploy result -> derived from last_non_deploying_state
Resource status -> last deployed hash Vs current hash + non-recursive orphans

we may be able to get rid of mark_deployed

Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Pull request rejected by merge tool. This pull request has not been approved yet by some of the requested reviewers. Still awaiting approval from sanderr

Processing this pull request

Merged into branches master in 6c8d45438e67c46e3d6c6b42eb98ed24b4b9cf27

Processing #7238.


udp-proxy-2020 on Raspberry

Dear all
I installed udp-proxy-2020 on a Raspberry 4 (hopefuly correctly) according the guide.
If I ask for the status I get an error like this:

What do I have to do?
P.S. How can I exit the window after receicving this message?
Thanks a lot.

No idea what that means

Conf file:

seems like the error is in your /lib/systemd/system/udp-proxy-2020.service file... what are the contents?

[Unit]
Description=UDP Proxy 20202
Documentation=https://github.com/synfinatic/udp-proxy-2020
Wants=network-online.target
After=network-online.target
[Service]
EnvironmentFile=/etc/udp-proxy-2020.conf
User=root
Group=root
Type=simple
Restart=on-failure
ExecStart=/usr/bin/udp-proxy-2020 $ARGS
LimitNOFILE=10000
TimeoutStopSec=20
UMask=0027
[Install]
WantedBy=multi-user.target

okay, looks like the problem is the \ on the end of the ARGS line.  it should only be there if you have actual args there.

Took it off.
The error persists:

Sorry, the output of that command only confirms it didn't start..  the logs are what gives the hints.  Can you look in the logs and paste in the most recent?

Here we go

Thank you.

Nope, those are the same logs from last time.  notice the timestamps are the same?

I noticed. But I only get this output.  But I rebooted and did everything again. Still only with that timestamp.

you can use the spacebar to scroll to the next page or shift-g to go to the end.  g to go back to the top.  j and k to go down/up one line.

Aha! that looks much better

permission denied   means file permissions are wrong.  most likely you forgot to mark the binary as executable: chmod 755 /usr/bin/udp-proxy-2020

Runs now without error. Thank you so much!!!
Did config remotly.
Will check tomorrow when back in the office.

So I tried now to see whether it starts on reboot.
I get an error message like that

If I start it manually (I have to enter ther password for my user) it works.
P.S. How can I delete the journal/logfile to have a better overview?

Ok I tried. No functionality.
Roon core on my home network. Synology running udp-roxy-2020. Nor error
Other network Raspberry running udp-proxy-2020.
Both location connected with OpenVPN.
Port 1194 forwarded on both netoworks to the server running OpenVPN.
Must I forward any other port e.g. 9003?
Thanks a lot.

okay, sounds like you're doing a site-to-site VPN.   That is what we call "advanced" :)
I really need a basic network diagram to understand how you have things deployed.  Doesn't have to be fancy- draw it on a piece of paper, take a picture with your phone and upload it if you want.
Key things to show:

What device and where is it that is your Roon core
What device and where is it that you are  running the Roon client
What devices are running udp-proxy-2020
What devices are running OpenVPN
What devices (like a firewall??) are in between the above list

For Roon, udp-proxy-2020 only needs to forward udp/9003.

I can't speak to Sonus specifically since I don't know how Roon supports that and I don't own one to test/find out.  Ie: does it use the normal Roon discovery protocol or something else?  If something else, then maybe or maybe not.
I just use Roon on my phone/laptop while traveling.   But yeah, that is the general idea.


data tables order for results

split off from #168
Gene Prediction table
make “Rank” the first column on the left == Move Rank to column 1 and keep the width to the minimum.
Make the widths of columns ‘Entrez’, ‘Symbol’, ‘Probability’, ‘Known/Novel’, and ‘Class-Label’ the same and snug.

Make the ‘Name’ column use the remaining width.

Similarities to BPs/Diseases table: make “Rank” the first column on the left.

Current look (for one tab, the rank size will be similar in the other tables)

there are issues with changing column sizes on the other 3 tables causing the table to colapse into 1/4th of the screen so we will table that for now and submit this

closed with PR #202


removed mcedit.rb

Can’t find a download link on the homepage (only Windows versions) and it uses a download url from a different host than the homepage. Unless someone can point to where the download url lives, we should remove this.

There are two things happening here, as far as I can determine.  mcedit, as our cask currently points to, is actually now mcedit2 (at https://github.com/mcedit/mcedit2) and is indeed currently windows only.  It is apparently under active development (the version for download is named as a demo) and from the readme in the repo, it appears they intend to add support for OSX, but haven't gotten there yet.
mcedit2 is the next version of the now-no-longer-developed https://github.com/mcedit/mcedit mcedit/mcedit which has been picked up under a different name, https://github.com/Khroki/MCEdit-Unified.  MCEdit-Unified appears to be both Mac compatible and under development.
So with all that in mind, I suppose this should be done away with, a replacement mcedit-unified added, and if mcedit2 adds support that can be added at a later date.

Everything sounds good. I’ll merge this in, then. Would you care to add mcedit-unified or would you prefer I do it?

Sorry, could you?  I'm mobile for a while.
~A
On Mon, Jan 26, 2015 at 5:08 PM, Vítor Galvão<EMAIL_ADDRESS>wrote:

Everything sounds good. I’ll merge this in, then. Would you care to add mcedit-unified or would you prefer I do it?
Reply to this email directly or view it on GitHub:
https://github.com/caskroom/homebrew-cask/pull/9042#issuecomment-71572116

Sure thing.


Adds checks for subcommands

Description
Adds some tests that check that the subcommand documentation is correct in the docstring for the spellbot() function. Also adds checks for the readme documentation. And adds checks for the webpage documentation for both commands and subcommands.
Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #226 into master will decrease coverage by 0.25%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #226      +/-   ##
==========================================
- Coverage   97.60%   97.34%   -0.26%
==========================================
  Files          32       32
  Lines        1881     1886       +5
  Branches      272      273       +1
==========================================
  Hits         1836     1836
- Misses         23       27       +4
- Partials       22       23       +1

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.53% <100.00%> (-0.43%)
:arrow_down:

Continue to review full report at Codecov.

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


Added ixo's coingecko_id "ixo"

Tested:
https://api.coingecko.com/api/v3/coins/markets?vs_currency=usd&ids=ixo&order=market_cap_desc&per_page=250&page=1&sparkline=false&price_change_percentage=1h%2C24h%2C7d

Thanks for the fix, I appreciate it!


Test with Acorn (not Acorn-6to5) and fix issue with AssignmentPattern

Acorn-6to5 is not up-to-date. After switching to Acorn, a test failed, showing that escodegen had been improperly handling AssignmentPattern.

When dealing with the AssignmentPattern node type, escodegen was trying to use an operator property. But AssignmentPattern doesn't have an operator property. The tests weren't noticing this because the only relevant tests (compare-acorn-es6) were using Acorn-6to5 instead of Acorn. Acorn-6to5 isn't up-to-date and it wasn't parsing these cases correctly. So two mistakes (one in escodegen and one in Acorn-6to5) were essentially canceling each other out as far as the tests were concerned. Switching to the more up-to-date Acorn resulted in the relevant tests failing.

Nice, thanks.

The issue with AssignmentPattern still appears to be present in version 1.6.1


[2/?] - input: add taproot chan scripts, control block logic, and spending routines

Change Description
In this commit, we add the current draft set of taproot scripts, control block handling logic, and also spending routines.
Reviewers will want to be familiar with the current spec draft before diving in: https://github.com/Roasbeef/lightning-rfc/blob/simple-taproot-chans/bolt-simple-taproot.md
This PR is a draft PR, as it still needs tests for all the new routines (they work in #6877) and also the witness estimate constants to be updated.
Steps to Test
Steps for reviewers to follow to test the change.
Pull Request Checklist
Testing

[ ] Your PR passes all CI checks.
[ ] Tests covering the positive and negative (error paths) are included.
[ ] Bug fixes contain tests triggering the bug to prevent regressions.

Code Style and Documentation

[ ] The change obeys the Code Documentation and Commenting guidelines, and lines wrap at 80.
[ ] Commits follow the Ideal Git Commit Structure.
[ ] Any new logging statements use an appropriate subsystem and logging level.
[ ]  There is a change description in the release notes, or [skip ci] in the commit message for small changes.

📝 Please see our  Contribution Guidelines for further guidance.

Pushed up a new commit that adds exhaustive tests to the new scripts. This tracks the current draft PR, which'll be updated soon to have some small changes (remove implicit CSV 0, etc). Removed it from draft now as unit tests are in place.

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@roasbeef, remember to re-request review from reviewers when ready

@guggero

Running the unit tests with coverage, the following functions don't seem to be covered yet: (though I'm not sure if we are aiming for 100% unit test coverage on these new functions, as they'll be executed in the itests for sure)

So those are tested indirectly. Eg: TaprootCommitScriptToSelf just calls NewLocalCommitScriptTree which is what the test harness uses to gain access to all the script context and not just the output key.

Also I fixed that test flake, I thought it was a key parity issue, but turns out one of the mutation tests ended up modifying the script itself, which meant that a different root was constructed in a later test, causing an incorrect output key to be derived 😅.


Add motionhazard to the outer side of parallel aware join.(fix flaky incorrect results of agg)

For a parallel aware join, we may partition outer data to batches if there is not enough memory.And if a worker has consumed all the data from subnode, it will arrive and wait for others to begin next phase: PHJ_BUILD_DONE.
PHJ_BUILD_DONE means that we has partitioned every thing from PHJ_BUILD_HASHING_OUTER phase and are ready to probe. It's true for Postgres, becuase that if a worker pulls no data from subnode and it leaves PHJ_BUILD_HASHING_OUTER phase, all subnode's data must be processed(ex: a Parallel Seqscan).
As all participants leaves PHJ_BUILD_HASHING_OUTER, the whole data should be pulled completely by these processes, or there may be other siblings who have never participated in phase PHJ_BUILD_HASHING_OUTER at all.
But it's not true for CBDB, a plan node may have Motions behind. Some parallel workers pull no data from subnode, it doesn't mean that other workers have no data to process at all. We need to wait for all parallel workers together instead of build barrier's participants arrived at the current phase before we move to the next phase.
Add an outer_motion_barrier to ensure all parallel workers finish their work of partition outer batches.
As compared to failed case in issue #259 , we will get at least 4-10 times wrong result of a 200-times loop.
With this PR, a 1000-times loop tests show no incorrect results.
Authored-by: Zhang Mingli<EMAIL_ADDRESS>

closes: #ISSUE_Number

Change logs
Describe your change clearly, including what problem is being solved or what feature is being added.
If it has some breaking backward or forward compatibility, please clary.
Why are the changes needed?
Describe why the changes are necessary.
Does this PR introduce any user-facing change?
If yes, please clarify the previous behavior and the change this PR proposes.
How was this patch tested?
Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.
Contributor's Checklist
Here are some reminders and checklists before/when submitting your pull request, please check them:

[ ] Make sure your Pull Request has a clear title and commit message. You can take git-commit template as a reference.
[ ] Sign the Contributor License Agreement as prompted for your first-time contribution(One-time setup).
[ ] Learn the coding contribution guide, including our code conventions, workflow and more.
[ ] List your communication in the GitHub Issues or Discussions (if has or needed).
[ ] Document changes.
[ ] Add tests for the change
[ ] Pass make installcheck
[ ] Pass make -C src/test installcheck-cbdb-parallel
[ ] Feel free to @cloudberrydb/dev team for review and approval when your PR is ready🥳

Results check  of case in #259 , rewrite to sql
select count(i3::numeric), sum(i3::numeric), avg(i3::numeric) from
(SELECT t1.* FROM test_hj_spill AS t1 right JOIN test_hj_spill AS t2 ON
t1.i1=t2.i2) foo;

correct result:
 count |   sum    |         avg
-------+----------+----------------------
 45000 | 22477500 |<PHONE_NUMBER>000000000
(1 row)

without this patch
500 loops, 470 run, 30 failed
gpadmin@i-v3roa8c2:/var/tmp$ cat res.log | grep -i count | wc -l
470
gpadmin@i-v3roa8c2:/var/tmp$ cat res.log | grep -i 45000 | wc -l
440

It's possible that some loops didn't success to run a sql file, because I limit the resource and CBDB got errors like:
psql:p.sql:3: ERROR:  failed to acquire resources on one or more segments
DETAIL:  connection to server at "<IP_ADDRESS>", port 9004 failed: FATAL:  reader could not find writer proc entry
DETAIL:  lock [0,1260] AccessShareLock 0. Probably because writer gang is gone somehow. Maybe try rerunning.
 (seg2 <IP_ADDRESS>:9004)

It's expected, just bypass them.
With this patch:
500 loops, 474run, 0failed
gpadmin@i-v3roa8c2:/var/tmp$ cat res.log | grep -i count | wc -l
474
gpadmin@i-v3roa8c2:/var/tmp$ cat res.log | grep -i 45000 | wc -l
474

loops
success to run sql
wrong results

500
470
30

500
474
0


Top-level breadcrumbs don't look like breadcrumbs

I saw this random floating link about the top-level spec page and got a bit confused. Only when I clicked on another page and saw this:

Did I realise what was going on.
We might want to either:

Not show breadcrumbs when there's only one item.
Show something like "You are here: ..." in front so it's not just a naked hyperlink.
Keep the / on the end if it's just one item.

Or something of that nature to give context to what that actually is, especially since it'll likely be one of the first things people see :)

+1 on not showing breadcrumbs when we are at the root. It doesn't seem helpful to show a link to the current page on its own, and removing it would also reduce the slightly embarrassing number of times we say "Matrix Specification" on https://adoring-einstein-5ea514.netlify.app/.

I've now done this, with https://github.com/wbamberg/matrix-hugo/commit/551b37de91c819445b986b2bed64de5eeb16722d:
https://adoring-einstein-5ea514.netlify.app/

@anoadragon453 , can we close this?


Bug 001

What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Karmada version:
kubectl-karmada or karmadactl version (the result of kubectl-karmada version or karmadactl version):
Others:

Hello 👋 Looks like there was no activity on this issue for last 90 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for 180 days, this issue will be closed (we can always reopen an issue if we need!).


ARROW-10366: [Rust][DataFusion] Do not buffer intermediate results in merge or HashAggregate

This PR is based on https://github.com/apache/arrow/pull/8503 from @jorgecarleitao
This makes merge send batches to a receiver stream as they arrive and, thereby removing the need to wait for each thread to finish collecting all its batches. It improves reported performance in my laptop on the aggregate_query_sql microbenchmark by 5-15%.
It also includes a change to the aggregate code that allows the aggregates to produce output even if there are no results on the first call to poll_next. The change in GroupBy is needed because when we initially tried to run Merge in parallel, we discovered no output was created if the inputs had not completed producing all their values before execution began. There is much more discussion on the topic here: https://github.com/apache/arrow/pull/8503#discussion_r510061337. I will add additional comments on the changes inline below.
Performance
I measured Aggregate Performance with the following command
git checkout master && \
    cargo bench --bench aggregate_query_sql && \
    git checkout alamb/ARROW-10366-collect-in-parallel && \
    cargo bench --bench aggregate_query_sql

Click for performance details
aggregate_query_no_group_by 15 12
time:   [715.38 us 717.64 us 720.07 us]
change: [-5.6870% -4.7352% -3.6834%] (p = 0.00 < 0.05)
Performance has improved.
Found 8 outliers among 100 measurements (8.00%)
2 (2.00%) high mild
6 (6.00%) high severe
Benchmarking aggregate_query_group_by 15 12: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 13.7s or reduce sample count to 40.
aggregate_query_group_by 15 12
time:   [2.4874 ms 2.5029 ms 2.5173 ms]
change: [-13.513% -12.381% -11.184%] (p = 0.00 < 0.05)
Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
3 (3.00%) high mild
2 (2.00%) high severe
Benchmarking aggregate_query_group_by_with_filter 15 12: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 10.2s or reduce sample count to 40.
aggregate_query_group_by_with_filter 15 12
time:   [1.9724 ms 1.9856 ms 1.9982 ms]
change: [-14.759% -10.342% -6.5522%] (p = 0.00 < 0.05)
Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
2 (2.00%) high mild
3 (3.00%) high severe

I will try and run some TPC benchmarks tomorrow. I'm excited to see how this looks. Great work @jorgecarleitao and @alamb

I went through this. Beautiful work, @alamb !

Thanks @jorgecarleitao

My only comment is that we are performing a change in the git submodule testing which I do not think it was intended.

You are correct that that was not intended -- I will fix that

Here are the TPC benchmark results (sf=100) based on the following command.
cargo run --release --bin tpch -- --iterations 3 --path /mnt/tpch/parquet-100GB --format parquet --query 1 --batch-size 4096 --concurrency 24

2.0.0 release
Query 1 iteration 0 took 13779 ms
Query 1 iteration 1 took 13665 ms
Query 1 iteration 2 took 14008 ms

master branch
Query 1 iteration 0 took 20304 ms
Query 1 iteration 1 took 20228 ms
Query 1 iteration 2 took 20737 ms

This PR
Query 1 iteration 0 took 20528 ms
Query 1 iteration 1 took 20688 ms
Query 1 iteration 2 took 20432 ms

I'm not sure why there is such a regression since 2.0.0 and will see if I can track down where this started.

The performance regression is related to removing use of the specialization trait. I filed https://issues.apache.org/jira/browse/ARROW-10453

@andygrove
Do you have the instructions for generating (or would be willing to provide me a copy to) the 100GB parquet dataset you are using in the following benchmark? I would love to try and run it locally and do some profiling to understand more of where time is being spent.
cargo run --release --bin tpch -- --iterations 3 --path /mnt/tpch/parquet-100GB --format parquet --query 1 --batch-size 4096 --concurrency 24

Thanks!
Andrew

@alamb I've been using Apache Spark to do the conversion and re-partitioning, using code in the Ballista project that I've been running manually from my IDE. I wouldn't feel good about checking Scala code into the Arrow Rust project so maybe the short term solution is for me to put a Dockerfile in Ballista for creating a Docker image that can do the conversion. Long term we should write a Rust utility to read CSV and write to Parquet. I'm not sure if the Parquet writer is far enough along yet to support that?

@alamb Could you try this out. You'll need to map volumes in appropriately to read/write from host directories.
docker run -it ballistacompute/spark-benchmarks:0.4.0-SNAPSHOT
  -h, --help   Show help message

Subcommand: convert-tpch
  -i, --input  <arg>
      --input-format  <arg>
  -o, --output  <arg>
      --output-format  <arg>
  -p, --partitions  <arg>
  -h, --help                   Show help message
Subcommand: tpch
      --input-format  <arg>
  -i, --input-path  <arg>
  -q, --query  <arg>
  -h, --help                  Show help message

@alamb Could you try this out. You'll need to map volumes in appropriately to read/write from host directories.

Checking it out...

@andygrove  -- it seems to work well:
$ docker run  -it -v /Users/alamb/Software/tpch_data/SF1:/opt/spark/work-dir ballistacompute/spark-benchmarks:0.4.0-SNAPSHOT convert-tpch  --input lineitem.tbl --input-format tbl --output lineitem.parquet --output-format parquet --partitions 16

Which then produced output like this:
$ ls -l /Users/alamb/Software/tpch_data/SF1/lineitem.parquet/
total 428032
-rw-r--r--  1 alamb  staff     0B Nov  2 06:27 _SUCCESS
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00000-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00001-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00002-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00003-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00004-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00005-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00006-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00007-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00008-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00009-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00010-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00011-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00012-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00013-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00014-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet
-rw-r--r--  1 alamb  staff    13M Nov  2 06:27 part-00015-47e26762-3d8a-4652-ba5e-2e2a620e399e-c000.snappy.parquet

I plan to test out using these files with datafusion later today if I have time, or tomorrow if not.
Thanks again,
Andrew

@jorgecarleitao  and @andygrove  -- I did some profiling of the tpch benchmark (SF10) locally on my laptop this afternoon. I was not able to reproduce a slowdown on this branch
Here is my data from on my laptop (a Macbook Pro):
Test command was: cargo run --release --bin tpch -- --iterations 3 --path /Users/alamb/Software/tpch_data/SF10-parquet-64 --format parquet --query 1 --batch-size 4096
When run against  @ d4121d8a17d9e53ad4421960e357dd2f89771603
Running benchmarks with the following options: TpchOpt { query: 1, debug: false, iterations: 3, concurrency: 2, batch_size: 4096, path: "/Users/alamb/Software/tpch_data/SF10-parquet-64", file_format: "parquet", mem_table: false }
Query 1 iteration 0 took 8913 ms
Query 1 iteration 1 took 10533 ms
Query 1 iteration 2 took 10633 ms

When run against this branch @<PHONE_NUMBER>77022bd0945f98f987993ca7267eb6 (after rebasing against d4121d8a17d9e53ad4421960e357dd2f89771603):
Running benchmarks with the following options: TpchOpt { query: 1, debug: false, iterations: 3, concurrency: 2, batch_size: 4096, path: "/Users/alamb/Software/tpch_data/SF10-parquet-64", file_format: "parquet", mem_table: false }
Query 1 iteration 0 took 9168 ms
Query 1 iteration 1 took 10097 ms
Query 1 iteration 2 took 10199 ms

So in other words I didn't measure a noticeable change on my machine
Qualatatively both this branch and master While kept my CPUs ~ 100% busy and the memory used was fairly consistent
I then spent some time poking around with a profile from XCode Instruments to see where the time was going. Unsurprisingly it is going to memory management (this is the story of my professional life):

I am going to poke some more time seeing if I can figure out any easy to recycle buffers or find a smoking gun

Thanks @alamb for posting the detailed info. I hope I haven't caused confusion with my earlier comments and performance data. I was not seeing a slowdown on this PR compared to master. I was seeing a slowdown in master compared to the 2.0.0 release (which I tracked down to the commit that removed specialization).

Ah -- got it. 👍

@jorgecarleitao -- when I ran the TPCH benchmark Q1 locally on my machine, I found it kept all my cores busy and the memory profile was low. Thus the improvements offered by this PR of running more things in parallel and avoiding buffering don't address the (current) bottleneck in the TPCH query
I theorize (without presenting evidence yet) that this is due to how the query is running and that:

Runtime is  dominated by filtering (not aggregation)
The intermediate results are small (the output of the first stage of grouping is 4 rows)

I have a more detailed performance analysis half written up but it is not coherently enough to share yet . I am hoping for some time later this week to write it up
I (selfishly) think this is an improvement, but I don't have any performance numbers to back that up

Rebased

I had some code that was crashing because of the behavior aggregate had when the wrapped exec first returned Pending when being polled. It know works perfectly with this PR! Thanks Andrew! Once this is merged I have some tests ready in #8658 that validate this!

I plan to merge this tomorrow unless i hear otherwise. @jorgecarleitao  / @andygrove  let me know if you have any concerns

Rebased

The travis CI checks (which aren't for rust specific things) seem to be stuck / not working. I don't think they are related to this PR (e.g. the same thing happened on This unrelated PR ), so merging

FYI @rdettai , this PR has been merged


Add image usage function to client

The image usage function allows more configurable and accurate calculations of the resources held by an image.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3520   +/-   ##
=======================================
  Coverage   44.27%   44.27%
=======================================
  Files         124      124
  Lines       13682    13682
=======================================
  Hits         6058     6058
  Misses       6688     6688
  Partials      936      936

Flag
Coverage Δ

#linux
48.05% <ø> (ø)
:arrow_up:

#windows
39.76% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


[Bug Report]: Bicep compilation fails due to CLI version on ubuntu-agents

Describe the bug
With the latest CLI version (2.37.0) used by the ubuntu-20.04 agents, a bug was introduced that renders the output of the 'az bicep build --f (..) --stdout' command empty, unless an additional '--no-restore' flag is passed along with it.
This is very bad, as the function is used by all our Pester tests (alongside some other logic).
This bug was already brought up here

https://github.com/Azure/azure-cli/issues/22621

And a Pull Request to fix it is open here

https://github.com/Azure/azure-cli/pull/22685

For the time being, we can either try to downgrade the CLI version in our pipeline (to e.g. 2.28.0), use the '--no-restore' flag (which may have side-effects) or wait until the PR is merged and the agents updated.
To reproduce
Run any template with the 'az bicep build --f (..) --stdout' command. The result will be empty.
Code snippet
`'az bicep build --f (..) --stdout'`

Relevant log output
No response

Thanks for reporting this. This was the bug I was chasing.  :-)


Missing implementation for data preprocessing

Hi,
I am aiming to extend the current datasets. However, it is missing the implementation of data preprocessing.
Therefore, I would like to ask if this can be included in the new version?
Thanks

We are currently working on it and plan to include the necessary implementation in the next release. You could keep an eye out for the upcoming version. However, we cannot provide a specific release time at this moment.

We are currently working on it and plan to include the necessary implementation in the next release. You could keep an eye out for the upcoming version. However, we cannot provide a specific release time at this moment.

Really looking forward to see the update.


Better cursor handling

Fix move-cursor, which in the current version doesn't do anything.
Add get-cursor
Add buttons for both operations to the test example.

Fantastic, Thanls!


Support filters on the "worst lines" widget

Suggested filters:

[ ] By operator
[ ] By line number
[ ] By city (src \ dst) - a bit trickier, because we don't directly have the city name from the route_long _name (the name can be מסוף קסטינה <> דרך ירושלים, תל אביב). This can be achieved by identifying the location of the first and the last stop using GIS.

please assign me

@moshe742 hi, do you still work on this issue?

@ArkadiK94 Yep exactly :)
I think the other filters are very important too, but maybe in the dashboard page they'll create too much clutter

Is this issue still relevant?
I didn't understand where is this filter should be in the website.

@UrielOfir it is relevant  :)
It should be on the main (dashboard) page

@NoamGaash It looks like no one is working on this issue, can I asign it to myself ?

@ShayAdler Hi! Can i clarify something?
What can be  achieved by identifying the location of the first and the last stop using GIS?  src/dst points ? But double arrow (<>) indicates a two-way relationship between the locations, isnt it ?

@NoamGaash I filtered routes that don't contain cities, there are about 170 routes, how can I efficiently get the missed cities? I can't use the gtfs_route__line_refs query because each ref_line can have thousands of different routes, for the same reason as all comma separated list of values queries. What queries should I use, maybe combine something? I can group these 170 routes by the same short names and get 50 unique names, then make 50 queries and get the cities, but this operation will be too expensive  for the server.


@!@!Quick Name Correction on American Airlines Ticket

To change the passenger name on an American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮𝓼 booking via the app, log in, go to "My Trips," and select the booking you need to modify. For more complex name changes or if the option isn't available, call 1-888-817-6019 (English) 🎭🦍 or 1-888-817-6019
𝒯𝑜 𝒸𝒽𝒶𝓃𝑔𝑒 𝓉𝒽𝑒 𝓃𝒶𝓂𝑒 𝑜𝓃 𝒶 American 𝓉𝒾𝒸𝓀𝑒𝓉, 𝓋𝒾𝓈𝒾𝓉 𝓉𝒽𝑒 American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 𝓌𝑒𝒷𝓈𝒾𝓉𝑒 𝑜𝓇 𝒸𝑜𝓃𝓉𝒶𝒸𝓉 𝒸𝓊𝓈𝓉𝑜𝓂𝑒𝓇 𝓈𝑒𝓇𝓋𝒾𝒸𝑒 𝓉𝒽𝓇𝑜𝓊𝑔𝒽 🎯🔮🔮 1-888-817-6019 (𝒪𝒯𝒜) 𝒪𝑅 1-888-817-6019 (𝒪𝒯𝒜) 🏀🏀(𝒪𝒯𝒜) 𝑜𝓇 🎯🔮🔮 1-888-817-6019 (𝒪𝒯𝒜). 𝐿𝑜𝑔 𝒾𝓃 𝓉𝑜 𝒻𝒾𝓃𝒹 𝓉𝒽𝑒 "𝒞𝒽𝒶𝓃𝑔𝑒 𝒩𝒶𝓂𝑒 𝒪𝓃 American 𝒯𝒾𝒸𝓀𝑒𝓉" 𝑜𝓅𝓉𝒾𝑜𝓃 𝒶𝓃𝒹 𝒻𝑜𝓁𝓁𝑜𝓌 𝓉𝒽𝑒 𝓅𝓇𝑜𝓋𝒾𝒹𝑒𝒹 𝒾𝓃𝓈𝓉𝓇𝓊𝒸𝓉𝒾𝑜𝓃𝓈 𝒻𝑜𝓇 𝒶 𝓈𝓂𝑜𝑜𝓉𝒽 𝓉𝓇𝒶𝓃𝓈𝒶𝒸𝓉𝒾𝑜𝓃.
𝒞𝒶𝓃 𝓅𝒶𝓈𝓈𝑒𝓃𝑔𝑒𝓇 𝒸𝒽𝒶𝓃𝑔𝑒 𝓉𝒽𝑒𝒾𝓇 𝓃𝒶𝓂𝑒 𝑜𝓃 American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈? 𝒴𝑒𝓈, 𝓎𝑜𝓊 𝒸𝒶𝓃 𝒸𝒽𝒶𝓃𝑔𝑒 𝓎𝑜𝓊𝓇 American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 𝓉𝒾𝒸𝓀𝑒𝓉 𝓃𝒶𝓂𝑒 𝑜𝓃𝓁𝒾𝓃𝑒 𝑜𝓇 𝑜𝒻𝒻𝓁𝒾𝓃𝑒. 𝒰𝓈𝑒 𝓉𝒽𝑒𝒾𝓇 𝓌𝑒𝒷𝓈𝒾𝓉𝑒 𝑜𝓇 𝓂𝑜𝒷𝒾𝓁𝑒 𝒶𝓅𝓅, 𝑜𝓇 𝒸𝑜𝓃𝓉𝒶𝒸𝓉 American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 𝒶𝓉 🎯🔮🔮 1-888-817-6019 (𝒪𝒯𝒜) 𝒪𝑅 1-888-817-6019 (𝒪𝒯𝒜) 🏀🏀 𝑜𝓇 🎯🔮🔮 1-888-817-6019 (𝒪𝒯𝒜) 𝒻𝑜𝓇 𝑔𝓊𝒾𝒹𝒶𝓃𝒸𝑒 𝑜𝓃 𝓇𝑒𝓆𝓊𝑒𝓈𝓉𝒾𝓃𝑔 𝒶 𝓃𝒶𝓂𝑒 𝒶𝒹𝒿𝓊𝓈𝓉𝓂𝑒𝓃𝓉.
To change or correction a name on an American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 ticket, contact their customer service at 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA). They will guide you through the name correction process and applicable fees.
𝕮𝖆𝖓 𝖆𝖎𝖗𝖑𝖎𝖓𝖊𝖘 𝖈𝖍𝖆𝖓𝖌𝖊 𝖕𝖆𝖘𝖘𝖊𝖓𝖌𝖊𝖗 𝖓𝖆𝖒𝖊?
𝔾𝕖𝕟𝕖𝕣𝕒𝕝𝕝𝕪, American 𝕗𝕝𝕚𝕘𝕙𝕥𝕤 𝕒𝕝𝕝𝕠𝕨 𝕟𝕒𝕞𝕖 𝕔𝕙𝕒𝕟𝕘𝕖𝕤 𝕗𝕠𝕣 𝕒 𝕗𝕖𝕖 1-888-817-6019 , 𝕓𝕦𝕥 𝕥𝕙𝕖 𝕖𝕩𝕒𝕔𝕥 𝕡𝕠𝕝𝕚𝕔𝕪 𝕔𝕒𝕟 𝕧𝕒𝕣𝕪 𝕓𝕒𝕤𝕖𝕕 𝕠𝕟 𝕥𝕙𝕖 𝕥𝕪𝕡𝕖 𝕠𝕗 𝕥𝕚𝕔𝕜𝕖𝕥 𝕡𝕦𝕣𝕔𝕙𝕒𝕤𝕖𝕕.
When you need to change the Name on your American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 ticket, simply visit their website or contact their customer service 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA). Enter your booking information in the "My Trips" section and select the flight you want to modify. Choose the option to change the flight Name, follow the instructions to complete the process, and pay any applicable fees. After making the changes, expect to receive a confirmation email from American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈.
How Do I Change Name On American Ticket Using Online Mode?
Go to the American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 website to begin the process of changing the Name on your online ticket.
Can you switch passengers on a American flight?
𝕋𝕠 𝕞𝕠𝕕𝕚𝕗𝕪 𝕥𝕙𝕖 𝕡𝕒𝕤𝕤𝕖𝕟𝕘𝕖𝕣 𝕟𝕒𝕞𝕖 𝕠𝕟 𝕒 American 𝕥𝕚𝕔𝕜𝕖𝕥, 𝕔𝕠𝕟𝕥𝕒𝕔𝕥 American 𝕔𝕦𝕤𝕥𝕠𝕞𝕖𝕣 𝕤𝕦𝕡𝕡𝕠𝕣𝕥 𝕠𝕣 𝕥𝕙𝕖 1-888-817-6019 (𝕆𝕋𝔸) 𝕠𝕣 1-888-817-6019(𝕆𝕋𝔸) 𝕕𝕚𝕣𝕖𝕔𝕥𝕝𝕪. American 𝕘𝕖𝕟𝕖𝕣𝕒𝕝𝕝𝕪 𝕕𝕠𝕖𝕤 𝕟𝕠𝕥 𝕒𝕔𝕔𝕖𝕡𝕥 𝕖𝕟𝕥𝕚𝕣𝕖 𝕡𝕒𝕤𝕤𝕖𝕟𝕘𝕖𝕣 𝕟𝕒𝕞𝕖 𝕔𝕙𝕒𝕟𝕘𝕖𝕤; 𝕙𝕠𝕨𝕖𝕧𝕖𝕣, 𝕞𝕚𝕟𝕠𝕣 𝕔𝕠𝕣𝕣𝕖𝕔𝕥𝕚𝕠𝕟𝕤 𝕠𝕣 𝕖𝕕𝕚𝕥𝕤 𝕞𝕒𝕪 𝕓𝕖 𝕒𝕝𝕝𝕠𝕨𝕖𝕕 𝕨𝕚𝕥𝕙 𝕡𝕣𝕠𝕡𝕖𝕣 𝕡𝕣𝕠𝕠𝕗 𝕒𝕟𝕕 𝕒𝕥 𝕒 𝕔𝕠𝕤𝕥.
Go to "Manage Travel" or "My Trips" 1-888-817-6019 and enter your booking details to log in. Find your itinerary and click on the Name of flight change option. To select a new Name and complete the adjustments, 1-888-817-6019 adhere to the instructions. If you find any issue while making the changes, feel free to communicate with its customer support team at 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA)
How To Change Flight Name On American Air Ticket?
American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 renders your various approaches to amend the Name on your flight ticket. Majorly you can either initiate the process by heading to the official American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 website or connect with its customer support team 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA). Moreover, navigate to the “Manage My Booking” section. Enter your booking information and select the flight you want to change. Follow the instructions to choose a new Name and pay any required fees. Once confirmed, you'll receive an email from American confirming your upNamed travel details.
How To Change Name On Plane Ticket American?
To change flight Name on plane ticket American, you need to log in to their official website or get in touch with their customer support staff by dialing 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA). Locate your booking under "My Trips," input your reservation details and confirmation code. Select the flight you need to change and select the option to adjust the Name. If you are confused, then you can communicate with the staff and get your doubts and issues resolved in no time.

There are a few different ways to change the Name on a American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 ticket. You can change your flight Name through American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈' official website. Log into your account, navigate to the "My Trips" section, enter your booking details and confirmation code. Alternatively, Contact American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈' customer service directly via phone 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA). Provide your booking information and request to change your flight Name through a customer service representative.
Can A Passenger Change Name On American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈?
Yes, passengers can change the Name of their American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 flight. They can do so by visiting the American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 website or contacting customer service 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) directly. Navigate to the "My Trips" section, input your booking details and reference code, and select the flight you wish to change. Choose the option to modify your flight Name, complete any required fee payment, 1-888-817-6019 and await confirmation from American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 via email.
Can You Change The Name On A American Airline?
Yes, changing the Name of your American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 flight is possible. Access their official website or call 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA), their customer service directly. Locate your booking under "Manage My Trip," input your reservation information, and choose the flight Name 1-888-817-6019 change option. Ensure you settle any relevant Name adjustment charges and await confirmation from American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 upon completing the process.
How to Change Passenger Name on American Ticket
How do I change my Name on the American ticket?
Name Correction on American flight Ticket
Change Name on American
Name Correction on American Ticket
how do I change my Name on American ?
Can I Change the Name on the American flight Ticket
How to Change Name on American
How do I change my Name on the American NameReservation?

Name Correction on American Reservation
How can I change my Name on a American ticket?
Can I change my Name on a American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight ticket?
How to Change Name on American Ticket
Can I change the Name on my American ticket?
change passenger Name on American ticket
How do I change the Name on my American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight boarding pass?
How do I change a passenger's Name on a American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight flight?
How do I change the Name on my American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight ticket?
Can I change the Name on my American s reservation?

Name Correction on American flight Ticket
How can I change my Name on a American ticket?
Name Correction on American Boarding Pass
Can I change my Name on a American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight ticket?
Name Correction on American flight Reservation
Name Correction on American Ticket
Change Name on American ticket
How to change the Name on American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 ticket
How to Change Passenger Name on American Ticket
Can I change the Name on my American ticket?
How do I change the Name on my American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight boarding pass?
How do I change a passenger's Name on a American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight flight?
How do I change the Name on my American 𝓐𝓲𝓻𝓵𝓲𝓷𝓮 flight ticket?
How do I change my Name on a American ticket?

Change Name on American flight Ticket
replace passenger Name on American flight ticket
How Can I Modify a Passenger's Name on a American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 Ticket?
The "Manage My Booking" area can be accessed by first visiting the official American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 website. Entering their reservation data, choosing the flight they want to change, and changing the Name as necessary may all be done from here. As an alternative, customers can call American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 customer service directly 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) and ask for a Name modification. With American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈, both options guarantee convenience and adaptability when modifying travel arrangements.
How Can My American Airline Flight Be change name ?
Is the Name of your American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 flight being upNamed? Go to their website or contact customer service 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA). Navigate to "Manage Travel," fill out the details of your reservation, and choose the flight that requires modification. Just follow the directions and pay the applicable fees to reschedule. A confirmation email from American 𝒜𝒾𝓇𝓁𝒾𝓃ℯ𝓈 with your upNamed trip details should arrive soon.
Can I Change My American Name Reservation Name?
Reach out to customer service or visit the 🎯🔮🔮 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) or 🎯🔮🔮 + 1-888-817-6019 (OTA) OR 1-888-817-6019 (OTA) 🏀🏀(OTA) website. After login in, select "My Trips" and input the booking information and confirmation code. Find the button to edit your reservation after selecting your flight. Choose a new Name and pay any necessary fees by following the instructions. To finish the procedure, make sure you get an email or notice on their website confirming the adjustments you made.

/spam


Delete subscription costs 60 seconds, bookkeeper-ml-workers-OrderedExecutor hang, java.util.concurrent.CompletableFuture$UniAccept took too long 60007267 micros to execute.

Describe the bug
2021-01-19 04:07:03,560Z [bookkeeper-ml-workers-OrderedExecutor-0-0] WARN  org.apache.bookkeeper.common.util.OrderedExecutor - Runnable java.util.concurre     nt.CompletableFuture$UniAccept@78d636f0:class java.util.concurrent.CompletableFuture$UniAccept took too long 60007267 micros to execute.

Then the thread's debug log
2021-01-19 04:06:03,552Z [bookkeeper-ml-workers-OrderedExecutor-0-0] DEBUG org.apache.bookkeeper.mledger.impl.EntryCacheImpl - [15e6e9ef41774be2b0823235de86e9ae/routingdata/persistent/routingdata_3e304e9d-3421-42bf-93ee-8d09971c230d-partition-0] Invalidated entries up to 849979:16 - Entries removed: 0 - Size removed: 0
2021-01-19 04:06:03,552Z [bookkeeper-ml-workers-OrderedExecutor-0-0] INFO  org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl - [15e6e9ef41774be2b0823235de86e9ae/routingdata/persistent/routingdata_3e304e9d-3421-42bf-93ee-8d09971c230d-partition-0] [ab7fd922-8c3e-43eb-b4f9-cbf13972a9d2] Deleted cursor
2021-01-19 04:06:03,552Z [bookkeeper-ml-workers-OrderedExecutor-0-0] DEBUG org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://15e6e9ef41774be2b0823235de86e9ae/routingdata/routingdata_3e304e9d-3421-42bf-93ee-8d09971c230d-partition-0][ab7fd922-8c3e-43eb-b4f9-cbf13972a9d2] Cursor deleted successfully
2021-01-19 04:07:03,560Z [bookkeeper-ml-workers-OrderedExecutor-0-0] INFO  org.apache.pulsar.broker.service.persistent.PersistentSubscription - [persistent://15e6e9ef41774be2b0823235de86e9ae/routingdata/routingdata_3e304e9d-3421-42bf-93ee-8d09971c230d-partition-0][ab7fd922-8c3e-43eb-b4f9-cbf13972a9d2] Successfully deleted subscription
2021-01-19 04:07:03,560Z [bookkeeper-ml-workers-OrderedExecutor-0-0] INFO  org.apache.pulsar.broker.service.persistent.PersistentTopic - [persistent://15e6e9ef41774be2b0823235de86e9ae/routingdata/routingdata_3e304e9d-3421-42bf-93ee-8d09971c230d-partition-0][ab7fd922-8c3e-43eb-b4f9-cbf13972a9d2] The subscription was deleted due to expiration

And the pulsar-msg-expiry-monitor hangs too.
To Reproduce
Currently, I have no idea to reproduce the probleam.
Expected behavior
In other environment, it executes quickly.
Desktop (please complete the following information):

OS: Linux, Huawei Euler OS

Additional context
Full debug log
04.zip

@Shoothzj Which version are you using? And is it possible to make a thread dump?

@Shoothzj Which version are you using? And is it possible to make a thread dump?


Run tracer as a web server

This would allow generating trace data for a code base that doesn't have unit tests (or any other good way of running from CLI) by allowing the developer to interact with the application from a browser and exercise the code that way.

While a nice idea, I'm not sure this should necessarily be part of the project?

Fair, I guess we can maybe hint at it in the readme so that people can configure there php+xdebug to output the needed trace files. I might still experiment a bit with it as the project where I have the most use of phpweaver only has 11% unit test coverage (I'm only associated with that project for the remainder of this week).


Performance fix

fix a bug with layer getNumInstance not returning the updated number of instance after layer state changed in the _getRenderFunction

wrong branch


Horizontal and Vertical Draggables

cc @apwilson this is your horizontal dragable

fixes https://github.com/flutter/flutter/issues/1836

lgtm


Support case insensitive filtering

Would be good for createOptions to be case insensitive by default on filtering as generally more useful/expected by people.

Now case insensitive and released in 0.6.0


Adds searchable speaker and adds lazy load to all the pages

Solves #1138 and #1135 .
Test server:
https://open-event-web-appp.herokuapp.com
Adds lazy load to all the pages please check the Screenshots:
Tracks page

Rooms page

Schedule page

Searchable speakers on schedule page

Codecov Report

Merging #1141 into development will not change coverage.
The diff coverage is 100%.

@@             Coverage Diff              @@
##           development    #1141   +/-   ##
============================================
  Coverage        62.98%   62.98%
============================================
  Files                8        8
  Lines             1375     1375
  Branches           213      213
============================================
  Hits               866      866
  Misses             509      509

Impacted Files
Coverage Δ

src/backend/dist.js
56.4% <100%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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

Nice @geekyd Thanks.


Back ported tsflags=nodocs supported into tdnf

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[ ] The toolchain/worker package manifests are up-to-date
[ ] Any updated packages successfully build (or no packages were changed)
[ ] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[ ] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[ ] All package sources are available
[ ] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[ ] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[ ] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[ ] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

The feature to use tsflags and remove docs was added to the upstream project, backporting that feature into mariner.
This patch file had to modified to support tdnf version 3.2.2
https://github.com/vmware/tdnf/commit/cccf140befaf58d44a42f4bbcd5eeba4ddf44a06
Change Log

Change: Ported upstream patch to tdnf in mariner  add support for tsflags=nodocs

Does this affect the toolchain?

NO
Test Methodology

Checked out tdnf code base, went to branch 3.2.2
Applied my patch
Patch  applied fine
I tried to build out all packages on Mariner via sudo make toolkit/build-packages, but having trouble getting it to build

OK, bumped the mainifest files


When flattening edge metadata and propagating to the server end of the edge, reverse the direction of the metadata.

Fixes #373

LGTM


TransformerASR, same dataset, different file order, different result

I use glob.glob() to get the audio file list, then I evaluate speech one by one. However, I find the WER of default order and sorted order by wav length are different. I thought different speeches are independent of each other.  They should get small WER. But the result is different a lot(5.80 vs 5.93). Is there any state that remained due to the previous sentence in TransformerASR? Thanks.

So, this is due to batched decoding. If you want to remove this variability, you should decode with a batch of one (unless you already do that, then there is something weird)

So, this is due to batched decoding. If you want to remove this variability, you should decode with a batch of one (unless you already do that, then there is something weird)

I transcribe speeches one by one. Is there any dependency or cache between continuous transcription?

Interesting ... Could you share your code ?

@sunwell1994 any update to share?

@sunwell1994 any update to share?

Hi, I did not invest in the problem. I just fix the order of evaluation. I am sorry that the whole code is distributed in a private project thus I cannot share it currently. Thanks!


Do not fail based on reinstall

This is a best-effort in detecting that a package was reinstalled in a layer. If the version is the same, but the install time has been updated, this is the likely scenario.
Fixes #966

coverage: 70.928% (-0.3%) from 71.233%
when pulling 11a38cc2adfe096a188f1315385cdc4fcb77d8f7 on bcrochet:hmf-install-date
into 7a9372e8e0348399c71465faaeba13662c11464a on redhat-openshift-ecosystem:main.


Pause Day Cycle making day cycle not continue after a player is online

I'm running the most recent version, with detect afk players and afk sleep also installed. Any idea what could be causing this? the time does not move forward at all

Oh that's super weird! That's one of the things that I specifically tested to make sure it was working. Just tested again to confirm, and with exactly those datapacks installed it works fine for me.
Need some more info:

Do all players online have the afk tag? /tag @a list
Are there any other data packs installed, and what are they?
What game version are you running?
If you're an admin (I'm assuming you are?) can you run the following commands and send the output:

/scoreboard players get #do_daylight_cycle pause_daylight_cycle.dummy
/scoreboard players get #is_paused pause_daylight_cycle.dummy

Is there anything else that might be relevant?

I truly don't know how this could have broken because the only thing that changed was the name of the folders, there literally wasn't a change in the logic at all haha.
My only guess is that while the daylight cycle was paused, somehow the scoreboard that it uses to keep track of how it was paused got reset, so it thought a player had disabled the daylight cycle and didn't resume it. That's what I'm trying to find out with those scoreboard values.
After you've given me the result of those commands, I think this will fix it:
/function pause_day_cycle:uninstall
/gamerule doDaylightCycle true
/reload

But if it breaks after that, I'll have to keep digging to figure out what the problem is. I even tested both cases of upgrading the world from 1.20.x to 1.21 while the daylight cycle is and isn't paused.

Oh one more thing actually. Because datapacks are world based I can actually do this debugging myself if you're willing to send me the world file - I'm @clo4 on Discord if you want to DM it to me.

i was able to patch it just by turning on the gamerule back on and it worked perfectly even when no players are online. not sure why it was paused to start with upon world launch. not sure why but it's fixed now and works as intended.

Extremely strange. I’ll keep a lookout for that on my own worlds too. Thanks for letting me know, appreciate it!


Depend on http:0.1

Other users of the http crate may depend on crates.io or hyperium/http... This ambiguity leads to compile errors. Depending on crates.io allows dependents to override the repo dependency more easily.

Although the Travis build for 68e318d got the green checkmark, it did not actually build successfully due to issue #143.

I looked over the travis logs for 68e318d5aa5c6f6066f2468e77469ca3c79ea872 and ddd3c65c1512a7e505456ffece73907ae4845c67, and it looks like depending on hyperium/http as a Git dependency works fine, but depending on version 0.1 from crates.io does not.


Investigate switching to comrak for markdown rendering

https://github.com/kivikakk/comrak
For normal size articles, it is roughly the same speed as pulldown-cmark but you get an AST so it should be easier. It also supports some GitHub markdown enhancements, which is quite nice.
comrak passes all the tests in the CommonMark spec and is actively maintained as opposed to pulldown-cmark.

If anyone is interested in contributing, all the code lives in the next branch in components/rendering/src/markdown.rs. There should be tests covering all the markdown rendering aspects so it should be fairly straightforward.

I don't have time to contribute right now, but I'd very strongly be in favor.

I started to give a look but I'm not convinced by the maintenance argument:

pulldown-cmark has slightly more PR than comrak;
there is even a release planned;
mdBook (used by cargo doc) depends on pulldown-cmark;

Is there any other benefits from switching to comrak? Maybe making some feature implementation easier?

The main benefit is having a markdown parser that follows the spec without errors. Right now there are some issues with pulldown-cmark in some markdown cases that can't really be worked around.
Another bonus of comrak is the github extensions like checklists

I think it's fine to use comrak if it meets your needs. That said, pulldown-cmark will get 100% cmark compatibility before too long, it's obviously very important. I can't promise when though.

If possible I would prefer avoid changing code that already works with pulldown-cmark. I was under the impression that pulldown-cmark wasn't really actively developed anymore so I stand corrected

@Keats Cool, that's motivating. As background, I made more than two attempts to try to find someone to take over the new_algo branch development, but those didn't pan out. I'm taking another look now and I'm feeling like doing it myself (often, just writing code is more relaxing than coordinating volunteers).
If people are interested in discussing in more detail, I'll create a channel on the xi zulip. Otherwise, they can watch progress on the new_algo branch.


Removing nb_check file from notebooks.

also changing references to !pip install msticpy to %pip install msticpy

There're some failed checks. Can you pls take a look @ianhelle ?


Add code change to 'Account.java'

Failed Tests:

testAddToBalance() from io.diffblue.corebanking.account.AccountDiffblueTest
testExecuteTransaction() from io.diffblue.corebanking.transaction.CashTransactionDiffblueTest

Tests run: 46, Failures: 2, Errors: 0, Skipped: 0


Add new comment-provider vk.com

Add ability to share articles in the social network VK.com by introduction of a new comment-provider.

Here's information on how to implement this https://vk.com/dev/share_details


Can multiple useMDXComponent be used safely?

I'm making a list of mdx content. Each item in the list ends up needing to use useMDXComponent for the card. Is useMDXComponent expensive? Is it alright to use it multiple times on a page?

@khinshankhan Hi, I am not quite sure, if this is an issue. It sounds more as a question, than an error.
Maybe instead of asking help in an issue ticket you want to ask your question in the Contentlayer Discord Channel - probably you get an response / answer much quicker.

Oh I see. I joined and asked, thanks :+1:


Adding new MTC filter that allows the user to specific which term(s) …

…they want to test. Only these terms are tested. Also added new unit test in test_mtc_filter.py

I also tested this assertion (https://www.ncbi.nlm.nih.gov/pmc/articles/PMC8948816/  "The frequency of global developmental delays and intellectual disability/learning difficulties in patients with truncated ANKRD11 gene variation was higher than that in patients with missense variation in the ANKRD11 gene (p < 0.05).", made on only 13 patients) by testing only two terms in the KBG notebook. There was no significance, and so I think we can say that we have fairly ruled this out.

@ielis thanks! I will merge


Showing Recent Messages Framework 'LofeltHaptics' not found | Xcode build error for Unity mobile game

Bug report
Checklist

[* ] I'm using the latest release
[* ] I've searched for existing issue

Environment
We are using Unity 2021.3.27f1 along with Feel 3.13.0, building the game for Android and iOS
Goals
We wanted to integrate haptic feedbacks on our UI buttons and build the game successfully for both Android and iOS
Steps to Reproduce
Integrate Feel into the game, place the functions for the haptic feedbacks on UI buttons and build the game for iOS then try to build it into xcode on a MAC machine
Expected behavior
It should build successfully in xcode then be able to upload it to the app store, on android it works without any problem.
Observed behavior
After it was exported from unity and tried to be built in xcode it will throw this error and the build stop:
Framework 'LofeltHaptics' not found
Linker command failed with exit code 1 (use -v to see invocation)
ld: warning: ignoring duplicate libraries: '-lc++'
ld: framework 'LofeltHaptics' not found
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Code Samples & Details
<Please provide a code sample if necessary, as well as any additional details, to help us track down the issue.>

Paste your \Assets\Plugins\Feel\NiceVibrations\Plugins\iOS\LofeltHaptics.frame file into > Pods\NiceVibrations (create NiceVibrations folder) in your xcode build output. Then link this file as a framework via Xcode

Paste your \Assets\Plugins\Feel\NiceVibrations\Plugins\iOS\LofeltHaptics.frame file into > Pods\NiceVibrations (create NiceVibrations folder) in your xcode build output. Then link this file as a framework via Xcode

That solved the problem for me a year later! Thanks.


Search icon no mouse pointer

When you hover over the search icon on Group Finder the mouse pointer does not change to mouse hand

Just to clarify, the search icon being referred to is the magnifying glass icon on the right:

@johnthepink correct.

@johnthepink thoughts of adding a clickable class to Junction which would handle the on hover?  Or are we just looking for a quick fix and write the css inline?

@edolyne i think we should just turn it from a  in to a , which has the hover state we want by default


Write a Java program to convert feet to centimeters

Description
Write a Java program to convert feet to centimeters
1 foot = 30.48 centimeters
Input  : 5
Output : 152.4

How to contribute

Save the solution in program/ConvertFeetToCentimeters.java file
Add ConvertFeetToCentimeters.java file in convert-feet-to-centimeters folder

!assign

Hey @aritra-tech,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign


Exclude Legacy demos

A number of the demos were written for older versions of the
Web Animations API. We exclude these from the list of displayed
demos, until they use the standard Web Animations API.

lgtm with nit: Refer to this pull request in the TODO comment.


GoCD pipeline backup on destructive operations

Also a patch for some healthcheck operations and DBURI regex

[Current coverage][cc-pull] is 34.72% (diff: 22.44%)

Merging [#25][cc-pull] into [develop][cc-base-branch] will decrease coverage by 1.90%

@@            develop        #25   diff @@
==========================================
  Files             9          9
  Lines           658        671    +13
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits            241        233     -8
- Misses          394        415    +21
  Partials         23         23

Powered by Codecov. Last update [c97e7b9...7c4f187][cc-compare]
[cc-base-branch]: https://codecov.io/gh/cfpb/rhobot/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/cfpb/rhobot/compare/c97e7b9a70eb4776d7f82eac4b3895f18e571c54...7c4f187500ca1cae502eab0481b45e3eea107eba?src=pr
[cc-pull]: https://codecov.io/gh/cfpb/rhobot/pull/25?src=pr


add the -t option which allows the user to specify a dynamo table to use for credential storage

awesome.

merged in 592cc2cec0a0be0a8303279c444ef286dee2c373


Shouldn't be able to add member to org/team they are already in

Org:
$ amp org member ls
[user jgrahamjones @ localhost:50101]
organization name: test
MEMBER         ROLE
jgrahamjones   ORGANIZATION_OWNER
$ amp org member add
[user jgrahamjones @ localhost:50101]
organization name: test
member name: jgrahamjones
Member has been added to organization.

Team:
$ amp team member ls
[user jgrahamjones @ localhost:50101]
organization name: test1
team name: test2
MEMBER
jgrahamjones
$ amp team member add
[user jgrahamjones @ localhost:50101]
organization name: test1
team name: test2
member name: jgrahamjones
Member has been added to team.

Already discussed here, https://github.com/appcelerator/amp/pull/1257#pullrequestreview-40412321.
I don't think this is an issue per se.


add release channels and support levels

Add stable and experimental release channels and Core, Extended, Implementation-specific support levels in line with the gateway-api.
Currently all of the apis we provide are experimental since they are v1alpha1 version.
Also add documentation on the new channels + Support levels and update our NPEP process slightly.

/hold
(to prevent accidental merge)

/lgtm


pudl.output.plants_eia860 gets run... many times in the pudl_out compilation tables

Some of our output tables are mostly just grab x table and denormalize it (merge entity x table into it), but sometimes we do imputations or filling in of data.
There are also two main ways we access the output tables, through the cached table stored within pudl_out._dfs or by re-regenerating the output table directly through the function in pudl.output. The plants_eia860 is being accessed in a few different places via the second method and for a while has had some imputations happening via pudl.output.eia860.fill_in_missing_ba_codes. Because of this, this imputation is being run over and over again.
I've noticed this mostly from re-running the net-gen/fuel allocation process in a notebook over and over and over again (a new pudl_out object will re-run this missing ba codes process 7 times for the mcoe output when the net generation/fuel is allocated).

Closed by #1973?


Microsoft 365 Groups now support nested membership

An update is needed to the Microsoft 365 Group section as Microsoft 365 Groups now support nested membership. Reference Message center change MC394932 .
Therefore the current sentence is incorrect
_'Microsoft 365 Groups don't support nesting with other Microsoft 365 Groups or with distribution or security groups.
[Enter feedback here]'_

Message center change MC394932 Create "Nested" Groups with Azure AD Dynamic Groups (preview) published 24 Jun 2022 now a
This feature will help you better manage group memberships by allowing you to build dynamic Azure AD Security Groups and Microsoft 365 groups based on other groups. For example, you can now create Dynamic-Group-A with members of Group-X and Group-Y.
The groups that define the membership of the dynamic group can be any group type represented in Azure Active Directory, such as user or device security groups, Microsoft 365 groups, and groups synced from on-premises. Unlike existing nested security groups today, memberOf dynamic groups return a flat list of members, so they can be used for licensing assignment and application assignment.

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

ID: 1a37b2d3-f63b-a719-565b-a46f2b1f2e96
Version Independent ID: e267656e-4bda-f03b-4d9c-d5149ec0a227
Content: Compare groups - Microsoft 365 admin
Content Source: microsoft-365/admin/create-groups/compare-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

As much as I welcome the feature announcement, from an administrative point of view I would rather prefer to not directly have the result called nesting in the documentation, at least as long as the feature is still in preview. My reasons would be the following:

Based on the feature description, users of group X and Y would be made direct members of group A. Although the usability of membership information would match that of nested groups, the groups wouldn't be nested for real, and it would always take a dynamic membership refresh interval to work as expected.
As dynamic group membership provides the technical base for this feature, simply calling it nesting would belittle the possibilities, as group A could also consist of just members of group X and Y located in Spain, while excluding all others.
Real nesting would allow groups X and Y to be members of group A, which itself could be a member of group M. I really wonder how (or better: if) this would work with dynamic group memberships and the refresh interval. Would such cases also work within a single interval? Would it take nesting levels - 1 intervals to work?

Nesting would of course make for one good exemplary use case in the documentation, although that would probably be a part of Dynamic group rules syntax.
That's just my two cents though, and I can very much understand if calling is nesting would help promote the feature.

@nikki-c Thank you for your feedback. I have made changes via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9074
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @nikki-c Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.