“failed to unify” message should not print identical types

Sometimes Lean says
tactic 'apply' failed, failed to unify
  some type
and
  some type

and some type is visually identical in both cases.
Of course advanced users will be forgiving such silliness, as they know about implicit arguments and things, and can use set_option pp.explicit true. But it is still quite silly for Lean to even say something like this.
Is there a reason it couldn’t compare the delaborated Syntax for equality, and if it is equal, elaborate again with pp.explicit := true (and if that’s still equal maybe with pp.raw := true), printing
tactic 'apply' failed, failed to unify
  @some 1 type
and
  @some 2 type
(shown here with pp.explicit := true to reveal differences)

Context

https://leanprover.zulipchat.com/#narrow/stream/113488-general/topic/rage/near/418885474

Steps to Reproduce
(TODO)
Versions
Certainly around 0.4.5
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

This is not the first time this issue has happened, see here, and here, and a similar issue here

It looks like we're not using addPPExplicitToExposeDiff  in tactics yet, we should

Oh, there functionality is even there already! That’s great :-)

That made it easy to heroically PR a fix minutes shortly after people complained on zulip :-)

Here is another person confused by this kind of issue (looks like exact rather than apply)


proposal for unaligned loads and stores

Implementation PR: https://github.com/apple/swift/pull/41033

@glessard Is this still awaiting an implementation?

The proposal and implementation are ready for review.

I could remove excessive references to drafts, I guess.

Ah, I see the link in the PR summary.  Could you link it from the document, please?  And yeah, if there's any draft cleanup you want to do, please do so.

@rjmccall Ready to go on my end; let me know if you need any other changes.

Thanks.  We'll try to talk about it in the Core Team ASAP and figure out how it's going to be scheduled, but I wanted to make sure you felt it was ready before we started that process.

We'd like to go ahead and run this.  I'll be review manager.  I think two weeks, running from March 30th until April 12th, is probably appropriate; is that okay with you as the author?

Sounds great. Thanks!

loadUnaligned has a debugPrecondition for POD-ness. I'll clarify this in the document and in the doc-comment.

Thank you.

Regarding storeBytes and storeBytesUnaligned: one issue here is that the destination is always, by definition, byte-aligned. At the builtins level, there is also not a different way to spell the two. If we had both functions, they would be identical in release mode, and would otherwise only differ by their set of debugPrecondition calls.

Regarding storeBytes and storeBytesUnaligned: one issue here is that the destination is always, by definition, byte-aligned. At the builtins level, there is also not a different way to spell the two. If we had both functions, they would be identical in release mode, and differ only by their set of debugPrecondition calls.

I mean, we could add a builtin that does an aligned store.  But in fact we don't need to: since you're requiring the type to be POD, you can simply use the existing initialize or assign builtin.

Do you want me to put this into review?  I can just make these same comments in the review thread, and we can see what the community thinks.  Worst case, the Core Team asks you to revise the proposal.

Yes, go ahead. Thanks!


mech nitrogen filtering 2

About the PR
better version of last one
if there is less than 21% oxygen it will start filtering out nitrogen
fixes #19863
Why / Balance
bug making vim eventual deathtrap
Technical details
added TargetOxygen which is checked when filtering against the current oxygen ratio. if there is not enough oxygen it will filter out OverflowGases, which includes nitrogen for basemech.
also moved everything to atmos so it can be reused for other things like gas masks
uses GetFilterAirEvent so it doesnt depend on mech
Media
trust

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

Breaking changes
MechAirFilterComponent renamed to AirFilterComponent
MechAirIntakeComponent renamed to AirIntakeComponent
AirFilterComponent now has the required field overflowGases which should be the same as gases plus a gas to filter out to maintain oxygen ratio. Adding nitrogen is normal for this.
Changelog
no cl no fun

Perhaps at this point it's better to start seeing where code can be shared with vents/scrubbers instead of doing a wholesale copy/pasta.

with these changes it would also be an air alarm and air alarm would need to support saving gases per warning/danger

@metalgearsloth merg

Thank you for making it generic

i was gonna make followup gas mask filtering pr but the body code trolled me or something :trollface:


onPageChanged is not triggered PdfViewerParams on some android devices

I have this logic that users need to read the whole pdf document. In some devices I'm not able to track the page changed and the moved to a new page to activate the button and it's happening on some Android devices.
I was testing on Galaxy Z flip3.

onPageChaged is not reliable way to know the user read the document fully. It uses the amount of the screen space occupied by pages to know which page is the user's main concern... I don't have good word choise for this, main, current, or, anyway the page user is reading.
I think it's not the perfect algorithm but it's not easy to implement a perfect algorithm...
To know whether the user scrolls down to the end of document or not, you can use PdfViewerController.visibleRect and PdfViewerController.documentSize.
If visibleRect.bottom is identical to documentSize, it indicate the user reached the end.
You can check it on a listener function registered using PdfViewerController.addListener.


[AUTO] Adding 2024-11-20-LANL-CovTransformer.parquet with predictions

Adding LANL-CovTransformer predictions for 11/20/2024 prediction date.

The views expressed here are those of the authors and does not necessarily represent the official views of the National Institutes of Health, Los Alamos National Laboratory, or the US Government.

For (probably my) future reference, we broke and (fixed) the normalization steps.
States that now insufficient data for a prediction (as per our model) are no longer reported. The metadata for our model now reflects this minor change in reporting structure.

Perfect! Everything looks good now and I'll merge it in!


Cleanup issues and PR's

There are right now 332 open issues and 59 open pull request.
I'd like to contribute, but I don't know where to start. Maybe we should start labeling issues (295 issues are unlabeled), close very old questions, etc. If I can help, please let me know!

Hi @stephanvierkant I do not work with PHP anymore, that makes it very hard to maintain this repository. I'm trying to keep it stable, that is all. If you wish to contribute, that is great. but I would not like to introduce backward incompatible changes. Anyway, added you to contributors, you may find something interesting to get your hands on, maybe refactor some portions of code. The master branch is somewhat staying at the same state, where 3.0.0 version was being prepared, but it stayed like that for long time.

@l3pp4rd Would it be a good idea, to split this into multiple repositories? It would definitely take some doing, but there is a huge upside.

Individual libs could improve much faster.
The 3.0 branch has a lot of good stuff, but it will never be finished. No one is capable of handling this in their spare time, and I don't think anyone is willing to take this project on company time, but lets hope I'm wrong.

The project could have separate maintainers per extension.
I think more people would be willing to help if they would only be responsible for a small piece. I would be willing to take sortable, which I know pretty well.

A couple of extensions could be dropped.
I would propose dropping
uploadable (and recommending vich uploader for symfony projects, and writing an article about doctrine uploads for non sf users),
translatable (drop this, ask the knp guys to contribute theirs, which is better supported by other bundles)

What do you think?

See #2002


Scanner crashing every time

Started having this issue since 0.9. Every time i try to scan program keep crashing.
Followed all steps/instructions and also checked other crash threads nothing seems to work.
I open game set everything according to instructions and click start scanning as soon as i click scaner sets genshing as main window then it updates itself in navigation section and immideatly crashes. Crash happens right after it updates to state shown in picture.

s

I have this issue as well. The exception I get is the following:
  System.IndexOutOfRangeException
   at InventoryKamera.GOOD..ctor(InventoryKamera.GenshinData)
   at InventoryKamera.Form1.<button1_Click>b__17_0()
   at System.Threading.ThreadHelper.ThreadStart_Context(System.Object)
   at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
   at System.Threading.ThreadHelper.ThreadStart()

Thanks for the report @Leanny have you tried downloading the C++ files?

I have the same issue but it only happens when scanning weapons or characters. Works fine for artifacts

@Andrewthe13th yes, that is not the issue though. After some debugging it looks like Itto and Gorous missing key. I guess this will be fixed with the auto updater from the datamined DB

So everything is updated and nothing happens in error log section it just instantly crashes after my nickname appears at navigation section.

thats what i got so far from runnung original project in Visual studio

@LightningDantes Can you run the debugger and go line by line until it crashes? It's not apparent what's the cause from your screenshot.
@Leanny Yes index range crashes are likely because new things haven't been hardcoded in yet. If you really want to scan the new characters, you can hardcode them yourself into their the lists or you can clone this repo's master branch has the code for the updater. It hasn't been put out as a release yet but I think it should be stable enough.

i have this problem also when using GOOD format, but when using seelie format it works normally

Seems like this causing problem

cuz i added 2 stop points for degug after lane 207 it still running but after 59 i am getting  there is an exception on picture above
also i got something else from debuging

but this two came up after  i did suggested changes from visual studio (changing/fixing format for line 207 and it recomends to set parameter for scraper)

Hmm I don’t believe there should be any parameters for Scraper.cs and the exceptions it’s throwing don’t tell us much either. If you can single out the line in RestartEngines that crash it would tell us more.
As for the last two screenshots, that’s expected behavior if it can’t find any items in your inventory to click. There’s nothing written in to the program to check that it navigates correctly so it may have tried to scan your inventory when the weapons page isn’t open.
Unfortunately my internet has been down today so I can’t exactly do a deep dive on what’s going on. However, I am confident that whatever bugs you’re encountering are fixed on the main branch if you would like to take a look at that.

@Cupcak3 so basicly after starting scan it shows my nickname in navigation tab but in game it never exits paimon menu and it crashes trying to scan

Okay. It sounds like one of two things to me:

You rebound your escape key so it doesn’t exit the menu
You set a breakpoint before the game tries to exit the menu which brings focus away from Genshin. Genshin needs focus for the menu navigation to work. This would make debugging tricky as you’d need to know when Kamera does it’s navigation.

so it runs restart engines fine then it shows nickname and puts mouse into game and immidiatly stops
none of my keys is rebinded

okey for some reason it decided to work this time but all artifacts give "Unable to validate information for artifact"

If you are using the main branch build there’s an additional step before you can scan. It’s automated so under options all you need to do is update every list. Sorry there’s no documentation on that yet.
I will work to make validation errors more specific and helpful.

new release has been made public


Add description about another region SNS topic

Actually, can you add a description about this problem to the FAQ docs? - https://github.com/awslabs/serverless-application-model/blob/master/docs/faq.rst
You should be able to grab this info from the Github Issue you referenced. FAQs are a good place to document such quirks


sys/linux, executor: improve USB descriptions

Change HID descriptions to allow devices to have two interrupt endpoints.
Remove unneeded responses to OUT control requests from descriptions.
Add some debugging code to detect and report missing descriptions.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1284   +/-   ##
=======================================
  Coverage   67.49%   67.49%
=======================================
  Files         120      120
  Lines       21005    21005
=======================================
  Hits        14177    14177
  Misses       6154     6154
  Partials      674      674

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 baa5258...a85b7cf. Read the comment docs.


Pantheon deployment refactor

Made several key changes to Pantheon:

Changed Vagrantfile to update the box and share some extra folders
Reorganized some of the variables in group_vars
Pinned new versions of Kratos, Moirai and DevDash
Created a symlink to Unitybox, our new ansible helper repo
Created pre/post tasks to set the umask to prevent permissions problems during deployment
Disabled Selinux on vagrant (causes unnecessary pain)
Fixed a bug with the creation of the web user
Refactored load_repos role so we now clone at specific versions
Fixed npm install of kratos
Added a bootstrap script to Vagrant to clone Unitybox

Reviewer: @dgreisen-cfpb

looks good. fixed a couple white space issues.


Setup firewall for grizzly cloud

resolution_fixed   |    by<EMAIL_ADDRESS>
Need to add https://github.com/ScottWales/puppet-security

Issue migrated from trac:39 at 2024-01-31 17:26:39 +1100

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>changed resolution from `` to fixed

<EMAIL_ADDRESS>changed owner from `` to bje900

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>commented

Milestone Future deleted

<EMAIL_ADDRESS>removed milestone (was Future)


Feature: Authorization

In this pull request:

Installed CanCanCan gem.
Added a role column to the users table.
Implemented the post deleting functionality and Added the Delete button to the view.
Implemented the comment deleting functionality and Added the Delete button to the view.

@teckim check slack for my message


[DOCFIX] add Clients-go.md of the Chinese version

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16313/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16332/
Test PASSed.


Upgrade to SDK 24

Travis is experiencing some lint problems when we upgrade the library to Android SDK 14 and build tools 24.0.0

Fixed in version 1.3.0


fvgdfgdf

Background

This is where you can provide code examples or context (e.g. past PR's)
e.g.

The following PRs contributed to...

Suggested Design

Here, you can put concrete steps as to what to do next
e.g.

create test file in...

Acceptance Criteria

Goals & criteria for success
e.g.

all existing and new tests should pass

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.

:warning: Broken links detected
The following broken links were found in the Markdown files:

Please fix these broken links as soon as possible.


Add bezier option

See #5

Thanks @nehayward! Which issues are you referring to?
I don't get why the diff is so big – would you mind to update only the lines that you have actually changed and describe what have you done?

Ya I think it might have been the spacing that got changed. I'm not sure
what's causing it, but when the there are a certain amount of points the
bezier gets all messed up. I only added those two functions, the bezier
class and the bools to the chart series.
On Fri, Oct 14, 2016 at 2:43 PM Giampaolo Bellavite <
<EMAIL_ADDRESS>wrote:

Thanks @nehayward https://github.com/nehayward! Which issues are you
referring to?
I don't get why the diff
https://github.com/gpbl/SwiftChart/pull/36/files is so big – would you
mind to update only the lines that you have actually changed and describe
what have you done?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gpbl/SwiftChart/pull/36#issuecomment-253901570, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGa85uzv-m3IJSn2TO5N9_FYNXODEZLGks5qz9ttgaJpZM4KXWzX
.

I see you are using this class by @ Ramshandilya (I wonder what are the best practices when including code from other people).
If you fix the diff I can join in and try to help 😊

Ya I'm not sure maybe we should ask. I'll fix the diff.
On Fri, Oct 14, 2016 at 3:41 PM Giampaolo Bellavite <
<EMAIL_ADDRESS>wrote:

I see you are using this class
https://github.com/Ramshandilya/Bezier/blob/master/Bezier/CubicCurveAlgorithm.swift
by @ Ramshandilya (I wonder what are the best practices when including code
from other people).
If you fix the diff I can join in and try to help 😊
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gpbl/SwiftChart/pull/36#issuecomment-253913848, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGa85mCJ0TaW5nT4J_XOmiXS56rXvs-hks5qz-j0gaJpZM4KXWzX
.

Is this still up for discussion?


Help

Hopefully it is allowed to ask questions here. I would need help please.
I have added the IP address and the code under /config/custom_components/bwt_perla/translations/de.json. Now how do I get further to my entities? Please, best thanks.

Have you read the readme file?

Of course I had read everything. Unfortunately I was on the line, but now I understand. Thank you very much! Now it is very easy.


Defang fix

Issue referenced and fixed by jlord

@profOnno the app code looks good (I'm assuming you tested it locally?) but I'm a little confused as to why there's an entire file being added. Was there just no test for the sanitization before? That doesn't seem correct...

@profOnno so I think it'd be better if the tests were in scrubber-activity-api-test.js. Other than that, though, this looks great :)
Thanks as always!

I went ahead and moved the test myself, since we were getting complaints on the network about this (https://io.jrobb.org/jrobb/note/4my4TsXqTlOV32LylnrhFQ). Thanks, @profOnno!

Thanks!
On May 3, 2016 2:15:50 PM EDT, Alex Jordan<EMAIL_ADDRESS>wrote:

I went ahead and moved the test myself, since we were getting
complaints on the network about this
(https://io.jrobb.org/jrobb/note/4my4TsXqTlOV32LylnrhFQ). Thanks,
@profOnno!

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/e14n/pump.io/pull/1144#issuecomment-216617795

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

@jfrobbins sure.


NullPointerException: null

NullPointerException: null when sorting some columns
To Reproduce

Open Recaf
Load a .jar file (any)
Open any method
Go to method's tab and double click in any column header for sorting the entire column, for example: sorting by name (the double click must be pressed fast)
The following error appears (on cmd.exe console and in the GUI)

Screenshots

Thanks a lot!


Allow consul service state to be overridden

In order to allow playbooks to decide what state to leave the consul service in, the handler needs to accept a parameter for consul_service_state.

Cool, gracias.


Document UTF-8 support with Identity Server

Description
Document how to configure the support for UTF-8 in the identity server.

For 5.7.0 also please consider this.

A valuable ticket lying open for 2 years and there is no progress or no updates ??

Will address this with https://github.com/wso2/product-is/issues/10352
Thanks for reporting.


Call reply.send before return causes ERR_HTTP_HEADERS_SENT error

Describe the bug
On fastify, call reply.send before return causes ERR_HTTP_HEADERS_SENT error.
How to reproduce

Call reply.send before handler returns.
Expected behavior
No response
Code reproduction
getLargeJson: async ({ params, reply }) => {
      const readable = await someService.getData()

      return reply
        .status(200)
        .type('application/json')
        .send(readable);
    }

ts-rest version
"@ts-rest/core": "^3.45.2",
"@ts-rest/fastify": "^3.45.2",
"@ts-rest/open-api": "^3.45.2",

You're not supposed to call .send yourself; ts-rest does it for you, just return { status: 200, body: readable } instead.

I should've created this ticket as a feature request. That's the workaround but it fails type checking.
Also, ts-rest allows handlers to return reply object. I wonder if it is intended since there is no way to respond if you return the object.

Also, sending before return still needed for utilize plugins. @fastify/compress, for example, supports reply.compress method which causes the error when using inside ts-rest handler.

You're not supposed to call .send yourself; ts-rest does it for you, just return { status: 200, body: readable } instead.

There are valid cases when additional tasks have to be run after the response, that the client shouldn't wait for. This case you are actually supposed to do that, so I suggest to reopen this issue.
Other frameworks also support this behavior, like for example Express.js enables using res.send and have additional code in the req handler. Another example: FastAPI has the BackgroundTask feature for this purpose.


"Too many open files" error when bundling - watchman installed, ulimit=2048

I have a lot of images to bundle with my app (a few thousand, as part of an emoji pack), and recently (after further images were added) I'm receiving the following output and error when attempting to create a bundle with the following command:
react-native bundle --platform=android --dev=false --entry-file=index.js --bundle-output=android/app/src/main/assets/index.android.bundle --assets-dest=android/app/src/main/res
result
Loading dependency graph, done.
Loading dependency graph...info Writing bundle output to:, android/app/src/main/assets/index.android.bundle
info Done writing bundle output
error EMFILE: too many open files, open 'C:\Users\cherb\Desktop\dev\Wander\Wander\node_modules\bugsnag-react-native\package.json'.
Run CLI with --verbose flag for more details.
Error: EMFILE: too many open files, open 'C:\Users\cherb\Desktop\dev\Wander\Wander\node_modules\bugsnag-react-native\package.json'

I can see that the latest images have not been added to res and do not show in release builds of the app.
I'm using Git Bash on VSCode on Windows 10.
I have tried increasing the ulimit open files count with ulimit -n 2048. Before increasing the limit, the error showed a different file for the "too many open files" example (unrelated to Bugsnag, the file currently shown).
Watchman is installed, and I have tried re-installing it. Running watchman.exe -v returns 4.9.4. Maybe it's relevant that watchman is not accessible via watchman -v without the .exe? This seems to be how Bash works on Windows.
React Native version:
System:
    OS: Windows 10 10.0.17763
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
    Memory: 2.35 GB / 7.82 GB
  Binaries:
    Node: 10.15.2 - C:\Program Files\nodejs\node.EXE
    npm: 6.4.1 - C:\Program Files\nodejs\npm.CMD
    Watchman: 4.9.4 - C:\Users\cherb\AppData\Local\watchman\watchman.EXE
  SDKs:
    Android SDK:
      API Levels: 23, 25, 26, 27, 28
      Build Tools: 23.0.1, 25.0.0, 25.0.2, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.3, 28.0.2, 28.0.3, 29.0.0, 29.0.1, 29.0.2
      System Images: android-22 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-26 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-29 | Google APIs Intel x86 Atom
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-192.7<IP_ADDRESS>08749
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5

https://github.com/facebook/create-react-app/issues/4540#issuecomment-393310226


TinyMCE 6 styleselect attribute does not work

styleselect attribute
The same configuration works normally in 5 and is not displayed in 6. Does 6 need special configuration?

I believe that changed to styles. For changes regarding the editor and migrating to TinyMCE v6 feel free to visit the editor repo. The Blazor wrapper supports both versions


[Geo] amplify add geo doesn't set region of Amazon Location Service in aws-exports.js using cli v.6.3.1

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm install -g @aws-amplify
If applicable, what version of Node.js are you using?
v16.11.1
Amplify CLI Version
6.3.1
What operating system are you using?
Amazon Linux 2 (Cloud9)
Amplify Categories
Not applicable
Amplify Commands
add
Describe the bug
amplify add geo created an aws-exports.js that does not include the project region into the geo settings of the amplify project. This breaks the rendering of any map (unless the map is explictly instantianted with the region parameter set) since API requests are sent against an in-existing region https://maps.geo.undefined.amazonaws.com/maps/v0/maps/.
/* eslint-disable */
// WARNING: DO NOT EDIT. This file is automatically generated by AWS Amplify. It will be overwritten.

const awsmobile = {
    "aws_project_region": "eu-west-1",
    "aws_cognito_identity_pool_id": "<redacted>",
    "aws_cognito_region": "eu-west-1",
    "aws_user_pools_id": "<redacted>",
    "aws_user_pools_web_client_id": "<redacted>",
    "oauth": {},
    "aws_cognito_login_mechanisms": [
        "PREFERRED_USERNAME"
    ],
    "aws_cognito_signup_attributes": [
        "EMAIL"
    ],
    "aws_cognito_mfa_configuration": "OFF",
    "aws_cognito_mfa_types": [
        "SMS"
    ],
    "aws_cognito_password_protection_settings": {
        "passwordPolicyMinLength": 8,
        "passwordPolicyCharacters": []
    },
    "aws_cognito_verification_mechanisms": [
        "EMAIL"
    ],
    "geo": {
        "AmazonLocationService": {
            "maps": {
                "items": {
                    "geo-test": {
                        "style": "VectorEsriStreets"
                    }
                },
                "default": "geo-test"
            },
            "region": "eu-west-1"
        }
    }
};

export default awsmobile;

The geo section lacks the documented region to be used of Amazon Location Services. Note that the region is setup correctly, since other configuration sections like auth take over the region correctly.
Note that there seem to be related bugs open that interfere with this issue. Issue-8405 of the amplify cli reports that the JSON keys of the geo settings section seem to be inconsistent across different versions, increasing the risk of unsuccessful config parsing.
Issue-57 of maplibre-gl-js-amplify points out that a missing default region for Amazon Location Service (in aws-exports.js's geo section) can break rendering of Maps. (When aws-exports.js does export a default AND devs don't set the region expliclty for every map initialized in JS.)
Expected behavior
amplify add geo should create a config that contains the project region and looks below. I verfied with an older version of amplify cli (6.2.1) that the region used to be passed through successfully.
{
    // ...
    "geo": {
        "AmazonLocationService": {
            maps: {...},
            search_indices: {...},
            region: REGION_OF_AMPLIFY_PROJECT
        }
    }
}

### Reproduction steps

### Reproduce issue with npm<EMAIL_ADDRESS>1. install most recent amplify cli version (npm install -g @aws-amplify)
2. init an amplify project with `amplify init`
3. Add geo feature with defaults by executing `amplify add geo`
4. Generate `aws-exports.js` by pushing to cloud `amplify push`
5. Find that the generated `aws-exports.js` __does not set the the `region` key__ under .geo.AmazonLocationService.

### Proof that older version<EMAIL_ADDRESS>behaved as expected
This works with older amplify version v. 6.2.1 executing the same steps, i.e.
1a. npm rm -g @aws-amplify (clean up previously used version)
1b. install amplify cli  `npm install -g<EMAIL_ADDRESS>`
2. init an amplify project with `amplify init`
3. Add geo feature with defaults by executing `amplify add geo`
4. Generate `aws-exports.js` by pushing to cloud `amplify push`
5. Find that the generated `aws-exports.js`  __does set the the `region` key__ under .geo.AmazonLocationService as expected

### GraphQL schema(s)

-

### Log output

-

### Additional information

_No response_

This has been fixed with amplify CLI v6.4.0. Please run amplify push --force after upgrading the Amplify CLI.


Check the link source are from the same workgraph

Fixed in https://github.com/scinode/node-graph/pull/13.


Nearpod

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a new site support request
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that none of provided URLs violate any copyrights or contain any DRM to the best of my knowledge
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and am willing to share it if required

Region
USA
Example URLs
https://nearpod.com/collaborate/reports/e9e8e2a895dec0ba6c9c850a3c866a2f/421747216?&parent_id
=0&hideStudentNames=1
Provide a description that is worded well enough to be understood
Please add support for Nearpod. It has these collaborative boards where users can post videos and audios. Please add support for downloading video and audio from the site.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[Documents]$ yt-dlp -vU https://nearpod.com/col
laborate/reports/e9e8e2a895dec0ba6c9c850a3c866a
2f/421747216?&parent_id=0&hideStudentNames=1
[debug] Command-line config: ['-vU', 'https://n
earpod.com/collaborate/reports/e9e8e2a895dec0ba
6c9c850a3c866a2f/421747216?&parent_id=0&hideStu
dentNames=1']
[debug] Encodings: locale UTF-8, fs utf-8, pref
 UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.09.01 [5d7c7d6] (pi
p)
[debug] Python 3.9.10+ (CPython 64bit) - macOS-
16.0-iPhone14,5-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg n5.0.1-4-ga5ebb3d2
5e (setts), ffprobe n5.0.1-4-ga5ebb3d25e, phant
omjs present, rtmpdump present
[debug] Optional libraries: Cryptodome-3.15.0,
certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1670 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.09.01, Current version: 2022.09.01
yt-dlp is up to date (2022.09.01)
[debug] [generic] Extracting URL: https://nearpod.com/collaborate/reports/e9e8e2a895dec0ba6c9c850a3c866a2f/421747216?&parent_id=0&hideStudentNames=1
[generic] 421747216?&parent_id=0&hideStudentNames=1: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 421747216?&parent_id=0&hideStudentNames=1: Extracting information
[debug] Looking for Brightcove embeds
[debug] Looking for embeds
ERROR: Unsupported URL: https://nearpod.com/collaborate/reports/e9e8e2a895dec0ba6c9c850a3c866a2f/421747216?&parent_id=0&hideStudentNames=1
Traceback (most recent call last):
  File "/var/mobile/Containers/Data/Application/BE92A072-84F0-4DB1-8FB4-0C9E11A11159/Library/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1459, in wrapper
    return func(self, *args, **kwargs)
  File "/var/mobile/Containers/Data/Application/BE92A072-84F0-4DB1-8FB4-0C9E11A11159/Library/lib/python3.9/site-packages/yt_dlp/YoutubeDL.py", line 1535, in __extract_info
    ie_result = ie.extract(url)
  File "/var/mobile/Containers/Data/Application/BE92A072-84F0-4DB1-8FB4-0C9E11A11159/Library/lib/python3.9/site-packages/yt_dlp/extractor/common.py", line 670, in extract
    ie_result = self._real_extract(url)
  File "/var/mobile/Containers/Data/Application/BE92A072-84F0-4DB1-8FB4-0C9E11A11159/Library/lib/python3.9/site-packages/yt_dlp/extractor/generic.py", line 3078, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://nearpod.com/collaborate/reports/e9e8e2a895dec0ba6c9c850a3c866a2f/421747216?&parent_id
=0&hideStudentNames=1

Request to https://collaborative-api.nearpod.com/sessions/421747216/e9e8e2a895dec0ba6c9c850a3c866a2f?isReport=1 to get board details and cards.
The cards in above request do not contain information about their assets. Just the assetUid (if there is any linked assets). To get the asset information it goes something like:

For every assetUid, request /assets/<assetUid>/type for every card to see if it is a video, audio, image etc.
Request the asset metadata:

If the asset is audio, you need to request /assets/<assetUid>/audios/status
If asset is video, you need to request /assets/<assetUid>/status
The above contain the metadata you want from yt-dlp - formats, title, duration, etc.
Be aware of the different types of video sources - e.g. youtube, amazon(direct link?)
The API suggests it keeps track of different versions

In other words, a lot of requests (what a great API)
Maybe someone can figure something out to return relevant data in less requests.


DataService SendEmail does not attach PDF invoice

The invoice gets emailed and the email references an attached invoice but there is only the link, not an actual PDF attached.
I call something like this:
$qb_client->SendEmail($theInvoice,$customer['email']);

Hi @lmorroni
It should be a download PDF. Could you forward your E-mail to my E-mail address<EMAIL_ADDRESS>

@lmorroni
Hi Larry, thanks for forwarding me the E-mail. The product and sandbox has different view of SendEmail Page. If you use Product QBO, the downloadPDF is within the "Pay Invoice" button. In sandbox, it is in the E-mail itself. If you go to the "Pay Invoice" page, you should see the "download PDF" option there


feat(engines v3): python sdk

#20 introduced mustache templates from open api repo
#21 makes necessary changes to api_client.mustache to accomodate for v3 mapping different response types to response codes
#22 makes necessary changes to api.mustache to accommodate for expressing v3 mapping multiple success response
#23 integrates anapi's concept of ClientErrorResponse with python sdk http response exceptions

Closing in favor of https://github.com/factset/analyticsapi-engines-sdk-generator/pull/28 due to signed commit issue


fix: correctly contstruct path for opening zettels in nested folders

At the moment, only zettels at root level are handled correctly. If there any zettels nested in additional folders (for
example, I have books for bibliograph) - those can not be opened correctly. Note that neuron itself does handle
that - you can still see virtual text with titles for such zettels.
Use plenary's path to not have to deal with pathing logic.

@oberblastmeister mind taking a quick look here? Hopefully nothing controversial.


Pass parameters to the FSI terminal creation routine

Hi,
In latest version there is a regression - not passing extra parameters to FSI terminal process.
This PR fixes it, but only once https://github.com/ionide/ionide-vscode-helpers/pull/5 is accepted, since signature of the createTerminal is wrong in current release.
PS: The build might be broken here, I didn't want to reference different version of Fable.Import.VSCode.fs from my forked repo, but I checked it locally - all works fine.

You need to run paket update and commit lock file to get build green here.

@Krzysztof-Cieslak yep, I know, and will do now, thanks!

All good now.

Released in 2.6.9

Thanks!


Error in training on new data from pretrained model

I want to train an ssd with a new data set with the following script; however, I got error.
I used mxnet-0.12.1.
python ../train.py \
--train-path cat_and_dog.train.rec \
--val-path cat_and_dog.val.rec \
--network mobilenet \
--pretrained ../model/mobilenet-ssd-512 \
--epoch 1 \
--data-shape 512 \
--num-example 3487 \
--num-class 2 \
--class-names cat,dog \
--lr 0.001 \
--wd 0.0 \
--lr-factor 0.2 \
--gpus 0,1

Error message:
Using mxnet as:
<module 'mxnet' from '/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/__init__.pyc'>
Warning: using pre-installed version of mxnet may cause unexpected error...
(export MXNET_EXAMPLE_SSD_DISABLE_PRE_INSTALLED=1) to prevent loading pre-installed mxnet.
[17:27:03] src/io/iter_image_det_recordio.cc:280: ImageDetRecordIOParser: cat_and_dog.train.rec, use 48 threads for decoding..
[17:27:03] src/io/iter_image_det_recordio.cc:333: ImageDetRecordIOParser: cat_and_dog.train.rec, label padding width: 350
[17:27:04] src/io/iter_image_det_recordio.cc:280: ImageDetRecordIOParser: cat_and_dog.val.rec, use 32 threads for decoding..
[17:27:04] src/io/iter_image_det_recordio.cc:333: ImageDetRecordIOParser: cat_and_dog.val.rec, label padding width: 350
INFO:root:Start training with (gpu(0),gpu(1)) from pretrained model ../model/mobilenet-ssd-512
[17:27:04] src/nnvm/legacy_json_util.cc:190: Loading symbol saved by previous version v0.12.0. Attempting to upgrade...
[17:27:04] src/nnvm/legacy_json_util.cc:198: Symbol successfully upgraded!
[17:27:07] src/operator/././cudnn_algoreg-inl.h:106: Running performance tests to find the best convolution algorithm, this can take a while... (setting env variable MXNET_CUDNN_AUTOTUNE_DEFAULT to 0 to disable)
[17:27:11] /home/travis/build/dmlc/mxnet-distro/mxnet-build/dmlc-core/include/dmlc/logging.h:308: [17:27:11] src/operator/nn/./../tensor/../elemwise_op_common.h:122: Check failed: assign(&dattr, (*vec)[i]) Incompatible attr in node  at 0-th output: expected (84,), got (12,)

Stack trace returned 10 entries:
[bt] (0) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x26ab7c) [0x7fb6918c0b7c]
[bt] (1) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x376923) [0x7fb6919cc923]
[bt] (2) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x37797b) [0x7fb6919cd97b]
[bt] (3) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x214d8f2) [0x7fb6937a38f2]
[bt] (4) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2139e69) [0x7fb69378fe69]
[bt] (5) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2090441) [0x7fb6936e6441]
[bt] (6) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(MXImperativeInvokeEx+0x63) [0x7fb6936e67e3]
[bt] (7) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call_unix64+0x4c) [0x7fb74aa3557c]
[bt] (8) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call+0x1f5) [0x7fb74aa34cd5]
[bt] (9) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x3e6) [0x7fb74aa2c376]

Traceback (most recent call last):
  File "../train.py", line 148, in <module>
    tensorboard=args.tensorboard)
  File "/media/storage1/software/github/mxnet-ssd/train/train_net.py", line 354, in train_net
    monitor=monitor)
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/module/base_module.py", line 464, in fit
    allow_missing=allow_missing, force_init=force_init)
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/module/module.py", line 296, in init_params
    _impl(desc, arr, arg_params)
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/module/module.py", line 284, in _impl
    cache_arr.copyto(arr)
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/ndarray/ndarray.py", line 1635, in copyto
    return _internal._copyto(self, out=other)
  File "<string>", line 25, in _copyto
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/_ctypes/ndarray.py", line 92, in _imperative_invoke
    ctypes.byref(out_stypes)))
  File "/media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/base.py", line 146, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: [17:27:11] src/operator/nn/./../tensor/../elemwise_op_common.h:122: Check failed: assign(&dattr, (*vec)[i]) Incompatible attr in node  at 0-th output: expected (84,), got (12,)

Stack trace returned 10 entries:
[bt] (0) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x26ab7c) [0x7fb6918c0b7c]
[bt] (1) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x376923) [0x7fb6919cc923]
[bt] (2) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x37797b) [0x7fb6919cd97b]
[bt] (3) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x214d8f2) [0x7fb6937a38f2]
[bt] (4) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2139e69) [0x7fb69378fe69]
[bt] (5) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(+0x2090441) [0x7fb6936e6441]
[bt] (6) /media/storage1/software/install/miniconda2/lib/python2.7/site-packages/mxnet/libmxnet.so(MXImperativeInvokeEx+0x63) [0x7fb6936e67e3]
[bt] (7) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call_unix64+0x4c) [0x7fb74aa3557c]
[bt] (8) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(ffi_call+0x1f5) [0x7fb74aa34cd5]
[bt] (9) /media/storage1/software/install/miniconda2/lib/python2.7/lib-dynload/_ctypes.so(_ctypes_callproc+0x3e6) [0x7fb74aa2c376]

there are two options to reuse pertained models.

use image classification model, you can specify --pretrained xxx
use pretrained models for detection, which we usually call it fine-tune, use --finetune xxx


ArgMax operation (GPU Implementation)

I need some help. I don't get how to make output tensor with int type?

@vanmaxim Thanks for your contributions!

Could you show that why you need argmax operation on GPU? Is there some shortage in the CPU implementation?
OpenCL is not good at controlflow statements, such as if-else, same as CUDA. So the running may cost more time than CPU.
We haven't support 3-D memory layout for OpenCL Image struct, refer to: https://mace.readthedocs.io/en/latest/development/memory_layout.html. It seems to be hard for mapping dimensions in tensors to Images.

@yejw5 How can I make output tensor with int type?
PS. I'll prepare performance benchmarks.

@vanmaxim Did you mean model's output in MaceTensor::data()？ Currently, only float data type is supported. But you can reinterpret_cast the data pointer as int type.


Investigate security test flakiness

The test test_security_alerts is behaving flaky on ADO. Possibly related to #9385.
Occasional ADO stack trace:
======================================================================
FAIL: test_security_alerts (azure.cli.command_modules.security.tests.latest.test_alerts_scenario.SecurityCenterAlertsTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/testsdk/base.py", line 251, in _in_process_execute
    self.exit_code = cli_ctx.invoke(shlex.split(command), out_file=stdout_buf) or 0
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/knack/cli.py", line 218, in invoke
    exit_code = self.exception_handler(ex)
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/core/__init__.py", line 98, in exception_handler
    return handle_exception(ex)
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/testsdk/patches.py", line 33, in _handle_main_exception
    raise ex
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/knack/cli.py", line 206, in invoke
    cmd_result = self.invocation.execute(args)
vcr.errors.CannotOverwriteExistingCassetteException: No match for the request (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/cli_role_assign000001/providers/Microsoft.Security/alerts?api-version=2015-06-01-preview>) was found. Can't overwrite existing cassette ('/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/command_modules/security/tests/latest/recordings/test_security_alerts.yaml') in your current record mode ('once').

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure_devtools/scenario_tests/decorators.py", line 40, in _preparer_wrapper
    fn(test_class_instance, **kwargs)
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/command_modules/security/tests/latest/test_alerts_scenario.py", line 16, in test_security_alerts
    alerts = self.cmd('az security alert list').get_output_in_json()
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/testsdk/base.py", line 161, in cmd
    return execute(self.cli_ctx, command, expect_failure=expect_failure).assert_with_checks(checks)
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/testsdk/base.py", line 200, in __init__
    self._in_process_execute(cli_ctx, command, expect_failure=expect_failure)
  File "/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/testsdk/base.py", line 256, in _in_process_execute
    raise AssertionError(ex)
AssertionError: No match for the request (<Request (GET) https://management.azure.com/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/cli_role_assign000001/providers/Microsoft.Security/alerts?api-version=2015-06-01-preview>) was found. Can't overwrite existing cassette ('/opt/hostedtoolcache/Python/3.6.8/x64/lib/python3.6/site-packages/azure/cli/command_modules/security/tests/latest/recordings/test_security_alerts.yaml') in your current record mode ('once').

When run locally, these tests pass consistently.

Looking for the relevant contact person on the team.

security


[AutoPR migrate/resource-manager] chore: jsonfmt migrate

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6182

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/3414


What is the GuidPrefix_t of XRCE-DDS Agent's discovery server?

Hi, I want to use XRCE-DDS Agent 's discovery server:
MicroXRCEAgent udp6 -p 2021 -d 7400
but  on client side setup with xml, I don't know how to fill the value of RemoteServer prefix
<RemoteServer prefix=" ">

Hello, @liyue75 in which platform are you using the Micro XRCE-DDS Client?
In general, when you enable the Agent's discovery server, the client just uses broadcast to discover Agent's interfaces, so no configuration is required. Can you share some more details about your use case so we can help you?

@pablogs9 , I'm using  XRCE-DDS to get sensor state, those sensors are running on stm32.
The data is sent from STM32 boards to the agent on the computer. On the computer, I need to run a FastDDS  program to get the data from the agent and send they to influxdb.
On the Client side, I use default transport, <useBuiltinTransports>true<useBuiltinTransports> when create participant from xml.
On computer, I use  FASTDDSv2.3.0 and write a fastdds sub program to get the data.
In my SUB program, If I use default transport, <useBuiltinTransports>true<useBuiltinTransports>, I find the sub can not recv data. The DDS on the same computer should use SHM to transfer data, but my sub program can not get data from Agent,  I think it may be caused by the lower DDS version of the Agent.
So I use udp transport,
<useBuiltinTransports>false<useBuiltinTransports>
<userTransports><transport_id>udp</transport_id></userTransports>

this time, my sub program can get data from the Agent.
When I using udp transport, I find there are lots of multicast packets on the computer, I know they are sent by the discovery process, every interface on the computer sends multicast packets for every entity created by Client.
I dont want these multicast packets on the network, so I want to use a discovery server, each entity sends they discovery message to the discovery server.
I find the Agent can run as a discovery server, but I don't know its guid.  I need the id in my fastdds sub program when I create the sub participant with XML file.

Hello, @liyue75 as explained in the XRCE-DDS Documentation the discovery profile is used by the Client to discover the Agent IP and start the XRCE communication. So, it is not related to DDS or RTPS.

@pablogs9, thanks for your repley.


[DRAFT] [BEAM-2928] Create sideinputHandler in ULR

Creates sideinputHandler inside ULR.  The goal of this PR is to finish step 1 in this doc: http://bit.ly/2EbqCKd
Following things are done:

Setup and pass stateAPI server description.
Created a SideInput Handler (by borrowing ideas from Flink implementation) that does the look up. This mostly implements the skeleton without wiring to job graphs in ULR.
Runner sends back a constant Integer(1), for any SideInput request via state API. Note this part does not completely work yet,  there is error during data transmission with complaints:

Caused by: java.lang.IllegalStateException: java.lang.ClassCastException: java.lang.Integer cannot be cast to [B
at org.apache.beam.sdk.coders.ByteArrayCoder.encode(ByteArrayCoder.java:41)
...
I am still looking into how to make this work, please suggest anything helpful to fix.

Follow this checklist to help us incorporate your contribution quickly and easily:

[X ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


fix: Enhanced gql generation

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


python27Packages.rpyc: 4.1.2 -> 4.1.3

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python2.7-rpyc/versions.
meta.description for python27Packages.rpyc is: '"Remote Python Call (RPyC), a transparent and symmetric RPC library"'.
meta.homepage for python27Packages.rpyc is: '"https://rpyc.readthedocs.org"
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 4.1.3 with grep in /nix/store/drq34ljvwf8c2z9p9nmzp27436g2ll01-python2.7-rpyc-4.1.3

directory tree listing: https://gist.github.com/a862c5b8ab454a87b790922b8ced85fc

du listing: https://gist.github.com/c11e5d39ae3fb2e38dad7d856a9e8266

Rebuild report (if merged into master) (click to expand)

16 total rebuild path(s)
4 package rebuild(s)
4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
4 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
flexget
python27Packages.rpyc
python37Packages.rpyc
python38Packages.rpyc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/drq34ljvwf8c2z9p9nmzp27436g2ll01-python2.7-rpyc-4.1.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python27Packages.rpyc https://github.com/r-ryantm/nixpkgs/archive/f7d8a5dad0a5340ae9baf9590308c878add866c5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/drq34ljvwf8c2z9p9nmzp27436g2ll01-python2.7-rpyc-4.1.3
ls -la /nix/store/drq34ljvwf8c2z9p9nmzp27436g2ll01-python2.7-rpyc-4.1.3/bin

darwin failure likely unrelated


Add optionality to 'opts' in Choo constructor

This PR makes opts parameter in constructor optional, because choo has a code to resolve the parameter if it is undefined: https://github.com/choojs/choo/blob/master/index.js#L20

📦 v6.4.3


IBM Cloud ROKS integrated use of Let's encrypt for routes creates issues with ESO Deployment

I've deployed a validated-patterns based cluster which was derived from multicloud-gitops based on this repo:

https://github.com/butler54/validated-patterns-demos

When deploying I was using 'Red Hat Openshift on IBM Cloud' where:

It was a VPC cluster (e.g. not classic)
The public service endpoints were enabled (e.g. routes are on the internet by default).

Here I was using the default route hostname (described here): https://cloud.ibm.com/docs/openshift?topic=openshift-openshift_routes using an "IBM provided domain".
By default the routes appear to be encrypted with lets encrypt certs including the console. So any re-encrypt/Redirect or edge/Redirect routes have usable public certs.
The adverse impact of this is that by default the External secrets Operator enforces a particular certificate chain.
The following commit enabled the ESO to function. however, does decrease security posture.
https://github.com/butler54/validated-patterns-demos/commit/e427f9911f467891eddabfb199924ee4d540852e
references
https://cloud.ibm.com/docs/openshift?topic=openshift-openshift_routes

Thanks for raising this @butler54 ! At first I was a bit puzzled, because we do have some experimental support for letsencrypt and it all works okay there (I just retested it). I looked into this a bit more and the reason letsencrypt works for us, is because we deploy it with cert-manager which takes care of injecting the letsencrypt CA in the kube-root-ca.crt by default:
oc extract -n golang-external-secrets configmap/kube-root-ca.crt --to=- > /tmp/letsencrypt1.txt  ;  openssl storeutl -noout -text /tmp/letsencrypt1.txt | grep "Issuer:"
# ca.crt
        Issuer: OU=openshift, CN=kube-apiserver-lb-signer
        Issuer: OU=openshift, CN=kube-apiserver-localhost-signer
        Issuer: OU=openshift, CN=kube-apiserver-service-network-signer
        Issuer: CN=openshift-kube-apiserver-operator_localhost-recovery-serving-signer@1699294589
        Issuer: C=US, O=Let's Encrypt, CN=R3
        Issuer: C=US, O=Internet Security Research Group, CN=ISRG Root X1
        Issuer: O=Digital Signature Trust Co., CN=DST Root CA X3

This does not happen on an IBM cloud:
oc extract -n golang-external-secrets configmap/kube-root-ca.crt --to=- > /tmp/letsencrypt2.txt  ;  openssl storeutl -noout -text /tmp/letsencrypt2.txt | grep "Issuer:"
# ca.crt
        Issuer: CN=root-ca-1699287502
        Issuer: CN=root-ca-1699287502
        Issuer: CN=root-ca-1699287502
        Issuer: CN=root-ca-1699287502
        Issuer: CN=cluster-signer-1699287502
        Issuer: CN=cluster-signer-1699287502
        Issuer: CN=cluster-signer-1699287502
        Issuer: CN=cluster-signer-1699287502

So either we find a simple way to inject the letsencrypt CA (or other CAs) into the kube-root-ca.crt configmap when on IBM cloud, or we need to come up with some not-too-horrible way to override which CA gets used in ESO.

(I'll add that the cleanest solution is for IBM to add the CA cert they use automatically to the default CA bundle, but I suspect that might be a bit more complicated ;) )

So via https://github.com/validatedpatterns/multicloud-gitops/pull/303 and https://github.com/validatedpatterns/common/pull/391 I think this one is addressed, so I am going ahead and closing it. Do scream, if you think it needs more work ;)


Add textio.yml file for automated repo administration

This PR adds a standard textio.yml file, to track team stewardship of this repo.

I happened to get a random notification for this PR - good to see you all are still using textio.yml ✨  its one of my best works 😄

@coilysiren It's seriously awesome, thank you!


Update to Python 3.11 and set POWERTOOLS_SERVICE_NAME for Python Lambda functions

Issue: https://github.com/aws-solutions-library-samples/guidance-for-custom-game-backend-hosting-on-aws/issues/7

Updated to Python 3.11.  As part of this platform was updated from manylinux2010_x86_64 to manylinux2014_x86_64, otherwise pip pulls in a very old (1.x) version of pyjwt, likely due to an ffi dependency.
Set POWERTOOLS_SERVICE_NAME so tracing doesn't report service_undefined

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tested and added cdk-lib upgrade to package.json


Fix how we check if a type is generic

Description
This PR fixes how we check if a type is generic, which was preventing us from being able to subclass a generic type while passing a type to it.
T = typing.TypeVar("T")

@strawberry.interface
class Node(typing.Generic[T]):
    id: strawberry.ID

    def _resolve(self) -> typing.Optional[T]:
        return None

@strawberry.type
class Book(Node[str]):
    name: str

@strawberry.type
class Query:
    @strawberry.field
    def books(self) -> typing.List[Book]:
        return list()

NOTE: we should probably also fix this use case:
T = typing.TypeVar("T")

@strawberry.interface
class Node(typing.Generic[T]):
    id: strawberry.ID
    name: T

@strawberry.type
class Book(Node[str]):
    ...

which is still unsupported, but I can be done in another PR

Types of Changes

[ ] Core
[x] Bugfix
[ ] New feature
[ ] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release fixes a bug that prevented from extending a generic type when
passing a type, like here:
T = typing.TypeVar("T")

@strawberry.interface
class Node(typing.Generic[T]):
    id: strawberry.ID

    def _resolve(self) -> typing.Optional[T]:
        return None

@strawberry.type
class Book(Node[str]):
    name: str

@strawberry.type
class Query:
    @strawberry.field
    def books(self) -> typing.List[Book]:
        return list()

🎉  Thanks for the quick fix!


beforeEnter: window.history.state values are absent

Reproduction
https://jsfiddle.net/zcnsje7u/
Steps to reproduce the bug

Click on Home in the reproduction

Expected behavior

The message "in foo: quux is (42)" is displayed.
The message "bar: if you got here from home, things aren't working" is not displayed.

Actual behavior
The message "bar: if you got here from home, things aren't working" is displayed.
Additional information
window.history.state is not populated with values from router.push (or router replace) in the target route's beforeEnter method. There is thus no way inside that method to redirect the user if the value is absent; while it is possible to do so outside of the navigation guard, this requires special handling inside the target component itself.
While this is a trivial example, the use of window.history.state is recommended as an approach when passing large values that do not fit comfortably in (or are otherwise inappropriate for) a query or path parameter. The inability to rely on that approach renders it all but useless.

Unfortunately, this is a History API limitation, and only the current entry can be accessed. You can implement custom solutions to store your state somewhere else and pass some id as a query param like I have seen it done in other framework routers.
FYI this limitation isn't present in the still experimental Navigation API.

| only the current entry can be accessed
Which is what, I believe, I'm trying to do. Set state in route X, access it before route Y is entered.

Oh I thought you were trying to do something else but this is still expected behavior: the state is only set after the navigation is validated, you are still in a navigation guard, the navigation can be cancelled and the state discarded.
History State is quite limited. I would recommend to only use it for data that is only needed while rendering the app (in components).

Ok, thanks for the response. I'll try to figure out a different approach.


Reconnecting loop

If my issue is not meeting contributing guidelines specified above, especially if it's a question or technical issue, please close my issue immediately - I won't complain.
P.S. Sorry for my English

Bug
It starts to loop reconnecting after executing the program.
Description
I was going to farm some cards but when I executed the program it started looping reconnecting to Steam.
Expected behavior
Just work like always.
Actual behavior
It started to loop reconnecting.

Global ASF.json config
{
	"AutoRestart": true,
	"AutoUpdates": true,
	"BackgroundGCPeriod": 0,
	"Blacklist": [],
	"ConnectionTimeout": 60,
	"CurrentCulture": null,
	"Debug": false,
	"FarmingDelay": 15,
	"GiftsLimiterDelay": 1,
	"Headless": false,
	"IdleFarmingPeriod": 8,
	"InventoryLimiterDelay": 3,
	"IPCHost": "<IP_ADDRESS>",
	"IPCPort": 1242,
	"LoginLimiterDelay": 10,
	"MaxFarmingTime": 10,
	"MaxTradeHoldDuration": 15,
	"OptimizationMode": 0,
	"Statistics": true,
	"SteamOwnerID": 0,
	"SteamProtocols": 1,
	"UpdateChannel": 1
}

BotName.json config of all affected bot instances (if more than one)
{"name":"Wolterhon Steam Trading Cards Farm","SteamLogin":"MY_LOGIN","SteamPassword":"MY_PASSWORD","Enabled":true,"SteamUserPermissions":{"76561198360548976":3}}

Help me please Archi.

Please review the guidelines for contributing to this repository.

https://github.com/JustArchi/ArchiSteamFarm/blob/master/CONTRIBUTING.md

If my issue is not meeting contributing guidelines specified above, especially if it's a question or technical issue, please close my issue immediately - I won't complain.

Reason for closing: technical issue is not ASF bug.


Support Solr 9

Solr 9 is released. It would be great with support for that in the operator.
#401 suggests that we're not quite there, but not far from it if it's a complete list of issues?

Thanks for mentioning that issue. It's a little out of date.
The action items there were addressed before the release was made. The release candidates were also tested with the Solr Operator smoke tester to ensure basic functionality.
That does not mean that it is perfectly supported yet, but we will only be able to determine that when people test it out over time. I would suggest giving it a shot if you have a development cluster and see if it works for your use case.

Please report back how it works for your use case @sdlarsen!


add support for passing in array of jobs

i'm with a customer who wants to pass in specific jobs to target within deployments vs doing them all...so, for example, they could specifically target diego cells vs all of cf.  this is an attempt to add that capability based on advice from @calebwashburn.  i'm sure i've missed something, please provide feedback.  :-)

I think it seems reasonable if you want to configure which jobs to include as part of repave.  For instance I could see some customers wanting to do only Diego cells and others doing all vm types.

hmm i probably greatly over-simplified this...  thinking out loud.
if there is just one deployment name (e.g. cf) and a list of jobs (e.g. diego, etc) that should work fine...but what if they pass in multiple deployments.  then the jobs loop would need some extra logic to target multiple sets of jobs across all the specified deployments.
maybe this is best achieved by a readme update, since this is sort of an edge case...or maybe we could  do something funky in the JOBS list like JOBS=a,b,c:d,e,f:etc and then map each colon-delimited list of jobs to the right deployment ("right" just being based on ordering of the DEPLOYMENTS/JOBS lists).
may also be over-thinking this...  thoughts?

hi folks, appreciate input before going down a bad path...  is the above an edge case best "documented around" in your opinion (maybe we should just say if you plan to specify a JOBS list then you only provide a single DEPLOYMENT).  that would solve the immediate need for this customer (only roll diego cells within cf deployment) but may not be ideal in all cases.
or should we do something (not right syntax, just thinking out loud) like this in secrets.yml:
jobs:
    - deploymentX: a,b,c
    - deploymentY: d,e,f
    - deploymentZ: g

then have task.sh work through the list of jobs per deployment?  deploymentX/Y/Z/etc would map to DEPLOYMENTS, and (i suppose) if a list of jobs is not found it would just do all (maintain prior behavior unless specifically overridden).
other implementation ideas?  if we get rough agreement, i'll update the script+readme.  :-)
thanks!

I think it is a fair assumption that customers may want to repave only a set of VMs pre-defined in a list. I'm OK with this PR for the JOBS parameter, however it is incomplete, you are missing the entry for the JOBS parameter in pipeline.yml, otherwise JOBS env var will always be empty in task.sh. Also, please update the main README with the details and usage of the new parameter.
Please resubmit it with those updates at your convenience.
With regards to your question about the deployments vs jobs scenario, I would prefer to keep the current code as-is and avoid the complexity of having to represent a matrix of deployments vs jobs in the parameters.
Another reason is, the current implementation supports that scenario in a simplified manner: if you define one pipeline per deployment with the deployment name jobs you want to restart for that deployment, it works.
One possibility to make the pipeline more robust in that scenario is to change the loop through jobs names and do a check of if <job-name-from-deployment> is in <list-of-jobs-to-restart> then restart job otherwise go-to-next-job--for-current-deployment. But I think we should start a separate Issue entry/discussion for this feature request. Thanks.

I think this may make sense to split to its own task so it’s explicit.  Repave jobs for a deployment which only talks in a deployment and comma delimited list of jobs, no need to pass Singleton job flag.  This makes this absolutely clear what use case this task is used for and removes conditional logic which can cause room for errors due to mis-configuration.

thanks for the input!  going for thinest slice of value...  fixed pipeline.yml (added the missed JOBS param) and updated the readme to match.
happy to work on additional PRs together for splitting to separate task (still needed/preferred?) and make the jobs loop more robust.

Thanks for the PR @deadlysyn !


Error 0x80070003 unsolved after trying all solutions

Windows Version
Microsoft Windows [版本 10.0.22631.3374]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-2
Distro Version
Ubuntu
Other Software
No response
Repro Steps

wsl -l -o
wsl --install Ubuntu

Expected Behavior
Steps to set username and passwd
Actual Behavior
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070003
Error: 0x80070003 ???????????
Press any key to continue...
Diagnostic Logs
WslLogs-2024-04-03_14-21-45.zip
attached is the log

I have tried every means to solve it, but just like what he met mentioned in #11019 ,it didnt work!

According to #11019 , he solved this issue by reinstalling windows. But how to solve this issue without reinstalling?

@LaysonShen: Looking at the logs it might be because the ubuntu app is in a weird state. Does repairing / reinstalling the app help ?

Thank you for your reply. I have reinstalled Ubuntu for several times, but it still didnt work. Moreover, other linux subsystem apps couldnt work neither.
在 2024-04-05 07:21:10，Blue @.***> 写道：
@LaysonShen: Looking at the logs it might be because the ubuntu app is in a weird state. Does repairing / reinstalling the app help ?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you @LaysonShen.
Let's try something else. Can you capture a TDD trace of ubuntu.exe ?
You can find instructions on how to do that in section 12)

Thanks @OneBlue ! I have followed the instructions. However, I have to admit my failure while recording wslservice.exe and the warning info has been shown above.
Then I turned to record ubuntu.exe and this time I successfully record everything helpful.
ubuntu01.zip

Thank you @LaysonShen. Did you capture ubuntu.exe's state from its startup ? Looking at the TDD trace, the trace starts when ubuntu.exe is already exiting so unfortunately it doesn't contain the repro information.

I also have ran into this problem, I opened my own issue but haven't received response #11553 if there is anything that could be provided to help solve this I would like to contribute since I can't reinstall OS at the moment or something like that

I tried what Blue has told me to capture the log of the subsystem but in vain when filtering sessions. I have to give it up LOL
在 2024-05-06 22:34:04，divisa-developer @.***> 写道：
I also have ran into this problem, I opened my own issue but haven't received response #11553 if there is anything that could be provided to help solve this I would like to contribute since I can't reinstall OS at the moment or something like that
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

I tried what Blue has told me to capture the log of the subsystem but in vain when filtering sessions. I have to give it up LOL 在 2024-05-06 22:34:04，divisa-developer @.> 写道： I also have ran into this problem, I opened my own issue but haven't received response #11553 if there is anything that could be provided to help solve this I would like to contribute since I can't reinstall OS at the moment or something like that — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.>

I'm sorry to hear that. If it's worth anything I managed to pin down my problem to a permission issue in the installation of drivers for the Hyper-V devices used by the Virtual Machine Platform, a company wide policy suddenly dissalowed installation of drivers and this messed the Hyper-V devices, after asking the sys admin to modify the policy I could boot up distributions again without issue. I noticed this beacuse in my device manager I found 3 Unkown Devices under the category others, when I tried to update their drivers it gave a message pointing to the permission issue.
I noticed your error message is a bit different than mine but it's worth a try.

I ran into this error while using my own PC. Wish you good luck.
在 2024-05-06 23:13:37，divisa-developer @.***> 写道：
I tried what Blue has told me to capture the log of the subsystem but in vain when filtering sessions. I have to give it up LOL 在 2024-05-06 22:34:04，divisa-developer @.> 写道： I also have ran into this problem, I opened my own issue but haven't received response #11553 if there is anything that could be provided to help solve this I would like to contribute since I can't reinstall OS at the moment or something like that — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.>
I'm sorry to hear that. If it's worth anything I managed to pin down my problem to a permission issue in the installation of drivers for the Hyper-V devices used by the Virtual Machine Platform, a company wide policy suddenly dissalowed installation of drivers and this messed the Hyper-V devices, after asking the sys admin to modify the policy I could boot up distributions again without issue. I noticed this beacuse in my device manager I found 3 Unkown Devices under the category others, when I tried to update their drivers it gave a message pointing to the permission issue.
I noticed your error message is a bit different than mine but it's worth a try.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Change log target from log_panics to panic

Currently when logging it uses the default target, which is the module name log_panics. But in my opinion it would be nicer to change this to panic, since the panic itself really has nothing to do with the module.
2017-08-04T12:32:58.203374+00:00 [ERROR] log_panics: thread 'tests::log_output' panicked at 'oops'
// vs
2017-08-04T12:32:58.203374+00:00 [ERROR] panic: thread 'tests::log_output' panicked at 'oops'

Would a change like that be acceptable?

Sure, that seems like a fine thing to do.

Thanks, any change for a minor update to include the change on crates.io?

Done

Thanks.


Sushi upgrade 22

ISSUE #35 - UPGRADE IG TO SUSH 2.2
- NPM vulnerabilities reduced:  31 to 19
- High vulnerabilities reduced:  8 to 6

CHANGE SUMMARY
- Renamed config.yaml to sushi.config.yaml
- Updated sushi-config.yaml content and comments to match new sushi template
    - Changed release label to draft (see FHIR release label guidance)
    - Added jurisdiction = US (default from sushi init)
    - Added url for publisher (linked to Alvearie site)
    - Set apply-contact false to prevent contacts from being added to resource samples in the IG
    - Remove test-history path and history section, not working as intended
- Changed folder structures for Sushi 2.2
    - Moved fsh files to data/input/fsh folder (Sushi 2.2 style)
    - Moved custom template files to work with Sushi 2.2
        - Moved alvearie-template folder to data/
        - Moved ig.ini file to data/
        - Point ig.ini to new alvearie-template path
        - Moved include folder to data/input
        - Removed ig-data folder (obsolete)
    - Updates to get build working with Sushi 2.2
        - Point ig.ini to new alvea
        - Added data/fsg-generated file to gitignore
    - Removed extra package.json in data folder (obsolete)
    - Removed docs folder (obsolete)
- Fixed profiles that were breaking Sushi 2.2 compiler
    - Removed duplicate extensions
        - InsightValue
        - SourceFieldID
    - Fixed fsh lines with spaces on front before *
        - CDM_Patient_WithDataQaulityInsights_Instance.fsh
        - CDM_Condition_WithNLPInsights_Instance.fsh
        - CDM_Coverage.fsh
    - Fixed broken valueset references
    - Episode extensions modified to use CURRENCY#USD
    - Fixed duplicate inline examples
        - Patient1
        - IBM
        - Genesys (now CommunicationVendor1)
        - AttributedPractitioner2
    - Fixed missing indexes in PatientMatch example
- Npm script and CI build github actions modifications
    - Updated build scripts to work with new folder structures
    - Changed from copy to rsync (works better for mac users)
    - Removed ping from _updatePublisher.sh to prevent CI Build failures
    - Added version info to CI Build github actions output
- Corrected template styles (footer display colors)

ISSUE #64 - CLEAN UP IG QA ERRORS AND WARNINGS
- QA Errors reduced: 127 to 29
- QA Warnings reduced: 126 to 10
- QA Info reduced: 406 to 8
- Broken links reduced: 3657 to 2

CHANGE SUMMARY
- Added ignoreWarrnings.txt file to suppress known warnings that can't be corrected
- Fixed incorrect profile on ConditionInstanceWithNLPInsights
- Corrected code descriptions on ConditionInstanceWithNLPInsights to fix terminology warnings
- Added caseSensitive = true to all code systems
- EngagementCommunicationRequestReason code system changes:
    - Fixed typo on url
    - Fixed long descriptions on two codes
- Changed QualityMeasureExample useContext.code to #focus
- ObservationExample changes:
    - Corrected Typo on #vitalsigns
    - Corrected mm[Hg] codes
    - Use aliased code systems
- CDMPatientDeceasedExample changes:
    - Fix url for local-marital-status coding system
- Correcting code system aliases:
   - http://terminology.hl7.org/CodeSystem/definition-topic

Added missing description to CDMPractionerRole
Use oid code systems where needed to prevent terminology QA errors

Added OID aliases
Use OIDs for appointment serviceCategory and serviceType
Use OIDs for encounter serviceType
Use OIDs for coverage emplyeeJobTitle
Use OIDs for organization SIC on type.coding

Fixed some slicing errors

Added identifier slices for identifier type codes used in examples
Added CareGap slice to Measure / MeasureReport

Fixed patientImportance cardinality error
Fixed broken links and references

Added some examples for references used in examples
Using inline examples as contained references to prevent QA broken link errors
Fixed canonical measure reference errors

Fixed valueset / coding system issues, use system urls and not aliases
Added measureScore to measure reports (required for proportion measures)
Removed WorkAddress profile (obsolete)
Added multiple data types to insight-value extension (had none)
Fixed library example errors

Using compliant names
Added code systems required for data requirements section
Fixed library canonical and reference errors

I think I've looked at all of the changes. 2 general areas to talk about:
1. using url vs.  alias for code system

2. claim sequence invariant issue - still working on logic

Thanks for working on all of this!

Followup 1/21/2022-

The alias was generating warnings so changed to use URL. Need to remember to check QA report after any new changes.
Removing the invariant until it can be resolved.


Edit DeveloperGuide Use cases && zico's information

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (6d122ce) compared to base (1cfa2e1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #47   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


adds phys location column to server tables

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

Which Traffic Control components are affected by this PR?

Traffic Portal

What is the best way to verify this PR?

Open TP and navigate to the following and notice the "Phys Location" column has been added:
https://tp.domain.com/#!/servers
https://tp.domain.com/#!/cache-groups/164/servers
https://tp.domain.com/#!/cdns/164/servers
https://tp.domain.com/#!/delivery-services/164/servers
https://tp.domain.com/#!/phys-locations/164/servers
https://tp.domain.com/#!/profiles/164/servers
https://tp.domain.com/#!/statuses/164/servers
https://tp.domain.com/#!/types/164/servers
This is a simple UI change. No tests or documentation needed.
The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3500/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/3501/
Test PASSed.


Ping @Helpers through the bot

Full Meta Proposal: https://discord.com/channels/268882317391429632/1181816468254507038
Summary:
To prevent the misuse of  @Helpers ping, only the bot should be allowed to ping in a timeout based manner.
TODO: config plugins.clopen ping_timeout 7200

pywright :bleakcat:


Discord link is invalid

Describe the bug
https://fabulously-optimized.github.io/discord
...redirects to...
https://discord.com/invite/fabulously-optimized-859124104644788234
...which doesn't exist.
Please fix that.
Modpack version

Launcher
Minecraft Launcher (vanilla)
Other things

[ ] I updated FO to a newer version (mention, what version to what version)
[ ] I added/removed some mods or resource packs (mention, which ones)

Additional context
No response

What you say is false ?
The link does actually work

Yeah, I can't seem to reproduce this either. The link exists and opens #links in the FO discord.

Link still works. Try opening it in a private/incognito window to see, and ensure your client is up to date.

Cannot reproduce.

UPDATE – probably because of the same reason, Discord deleted my account for using too many VPNs/Tempmails :/

UPDATE – probably because Discord deleted my account & blocked my actual IP for using too many VPNs/Tempmails :/

Probably not for VPNs but for temp mails 100%, they tought banning people with temp mails was a good idea to limit throwaway accounts 💀


Add commandType annotations to cobra.Command

Changes
This commit adds a commandType annotation to all new cobra.Command,
specifically:
		Annotations: map[string]string{
			"commandType": "main",
		},

This is primarily done so the commands and subcommands can be discovered
by tkn CLI which looks for this annotation while collating usage
documentation for all subcommands, see here:
https://github.com/tektoncd/cli/blob/3c9ece67610d3be38cd56847305108effad93c77/pkg/cmd/root.go#L52
Co-authored-by: Piyush Garg<EMAIL_ADDRESS>Submitter Checklist

[x] ♽  Run make test lint before submitting a PR (ie: via pre-push github hook no need to waste CPU cycle on CI
[x] 📖 If you are adding a user facing feature or make a change of the behavior, please make sure to document it
[x] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[x] 🎁 If that's something that is possible to do please make sure to check if we can add a e2e test.
[x] 🔎 If there is a flakiness in the CI tests then make sure to get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it.

See the developer guide for a bit more details.

CC: @piyush-garg

/retest

Codecov Report

Merging #410 (d3c4340) into main (7ca5b2e) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #410      +/-   ##
==========================================
+ Coverage   70.46%   70.52%   +0.05%
==========================================
  Files          50       50
  Lines        2607     2629      +22
==========================================
+ Hits         1837     1854      +17
- Misses        586      589       +3
- Partials      184      186       +2

Impacted Files
Coverage Δ

pkg/cmd/pipelineascode/pipelineascode.go
50.00% <ø> (ø)

pkg/cmd/tknpac/generate/generate.go
45.52% <ø> (ø)

pkg/cmd/tknpac/repository/create.go
52.21% <ø> (ø)

pkg/cmd/tknpac/repository/list.go
83.58% <ø> (ø)

pkg/cmd/tknpac/resolve/resolve.go
67.03% <ø> (ø)

pkg/cmd/tknpac/completion/cmd.go
100.00% <100.00%> (ø)

pkg/cmd/tknpac/repository/root.go
100.00% <100.00%> (ø)

pkg/templates/templating.go
100.00% <0.00%> (ø)

pkg/git/git.go
74.35% <0.00%> (+5.60%)
: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 7ca5b2e...d3c4340. Read the comment docs.

we need to find a way to fix those labels stuff, since we don't have time to know the pipeline label until we check it out 🤔


Move files when we stop ros

when ros is stopped the files in /tmp directory are not moved to the record directory

Fixed


ScrollArea event-based scrolling does not work with scroll-behavior: smooth

Bug report
I'm setting scroll-behavior: smooth so that my custom .scrollTo calls are smoothly scrolled into view. However, it breaks the ScrollArea dragging and trackpad (wheel event) behavior.
Current Behavior
https://user-images.githubusercontent.com/34928425/229560733-d6d9d109-c148-43ae-a793-38000fa6d270.mp4
Expected behavior
https://user-images.githubusercontent.com/34928425/229560654-163cce91-3953-4db4-bf62-3f5e8c761691.mp4
Reproducible example
https://codesandbox.io/p/sandbox/sparkling-waterfall-6pibz1
Suggested solution
ScrollArea should manually set viewport.style.scrollBehavior = 'auto' when the scroll position (.scrollTop and .scrollLeft) is manually set from JS events.
Additional context
Your environment

Software
Name(s)
Version

Radix Package(s)
ScrollArea
1.0.3

React
n/a
18.2.0

Browser
Chrome, Brave, Safari

Assistive tech

Node
n/a

npm/yarn

Operating System

Fixed in #2175


react warning: no-op react state update (possible memory leak)

Warning happened when submitting a transfer (sent it) and automatically being brought back to the Wallet Page.
Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application.  Lets keep this open to check if it occurs again.


Functionality of createTable changed from version 0.4.0 on for DynamoDbEnhancedAsyncClient

With version 0.4.0 the createTableWithIndices functions were removed, because the underlying DefaultDynamoDbTable already is doing the job.
This is working for the DefaultDynamoDbTable  but not for the DynamoDbEnhancedAsyncClient as I can see.  The tests in CreateTableTest  should verify the correct behavior after the removal of the createTableWithIndices - function are not using the async variant.
After replacing the createTableWithIndices - function calls with the recommended createTable - function calls in my application tests are failing because of missing indices.
Below you can find an updated test that I use to reproduce the issue. It is showing me that there are no indices available.
class CreateTableTest {

    private data class Person(
        @DynamoKtPartitionKey val id: Int,
        @DynamoKtSecondaryPartitionKey(indexNames = ["names"]) val name: String,
        @DynamoKtSecondarySortKey(indexNames = ["names"]) val dob: Instant
    )

    private val storage = Storage.InMemory<DynamoTable>()

    private val personTable = DynamoDbEnhancedAsyncClient.builder()
        .dynamoDbClient(
            DynamoDbAsyncClient.builder()
                .httpClient(AwsSdkAsyncClient(FakeDynamoDb(storage)))
                .credentialsProvider { AwsBasicCredentials.create("key", "id") }
                .region(Region.CA_CENTRAL_1)
                .build()
        )
        .build()
        .table("people", DataClassTableSchema(Person::class))

    @Test
    fun createTable() = runBlocking {
        personTable.createTable().await()

        val table = storage["people"].shouldNotBeNull().table

        table.GlobalSecondaryIndexes.shouldContainExactlyInAnyOrder(
            GlobalSecondaryIndexResponse(
                IndexName = "names",
                KeySchema = listOf(
                    KeySchema(AttributeName.of("name"), KeyType.HASH),
                    KeySchema(AttributeName.of("dob"), KeyType.RANGE)
                ),
                Projection = Projection(ProjectionType = ProjectionType.ALL)
            )
        )

        table.LocalSecondaryIndexes.shouldBeNull()
    }
}

Can you please verify if this is correct?

If this is the case on the latest AWS SDK, it would be a curious oversight.  What I would do is open an issue on the AWS SDK, and reintroduce a helper method in the meantime.  I'll perform a test soon.
If you haven't done so already, you can make your own extension method.  This is the old one.
private val defaultProjectionBuilder = { _: IndexMetadata ->
    Projection.builder().projectionType(ProjectionType.ALL).build()
}

fun <T: Any> DynamoDbAsyncTable<T>.createTableWithIndices(
    createProjection: (IndexMetadata) -> Projection = defaultProjectionBuilder
): CompletableFuture<Void> {
    val request = tableSchema().tableMetadata().createTableEnhancedRequestWithIndices(createProjection)
    return createTable(request)
}

I've restored DynamoDbAsyncTable.createTableWithIndices in release 0.8.  I've verified the feature was never added to the DynamoDbAsyncTable, and I am attempting to add the missing feature myself.  Just need to figure out how to get the project to build locally 😅 .

That was super fast. Thank you very much.