Random "illegal register access" GPU crash while playing Satisfactory:

I get this crash randomly while playing Satisfactory:
Jul  3 11:58:08 guilmon kernel: [drm:gfx_v8_0_priv_reg_irq [amdgpu]] *ERROR* Illegal register access in command stream
Jul  3 11:58:08 guilmon kernel: [drm:amdgpu_job_timedout [amdgpu]] *ERROR* ring gfx timeout, signaled seq=387961, emitted seq=387963
Jul  3 11:58:08 guilmon kernel: [drm:amdgpu_job_timedout [amdgpu]] *ERROR* Process information: process FactoryGame-Win pid 4312 thread FactoryGame-Win pid 4312
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: GPU reset begin!
Jul  3 11:58:08 guilmon kernel: clocksource: timekeeping watchdog on CPU1: hpet wd-wd read-back delay of 121244ns
Jul  3 11:58:08 guilmon kernel: clocksource: wd-tsc-wd read-back delay of 121733ns, clock-skew test skipped!
Jul  3 11:58:08 guilmon kernel: perf: interrupt took too long (4629 > 4483), lowering kernel.perf_event_max_sample_rate to 43200
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: snd_hda_intel 0000:00:14.2: IRQ timing workaround is activated for card #0. Suggest a bigger bdl_pos_adj.
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: \x0alast message was failed ret is 65535
Jul  3 11:58:08 guilmon kernel: amdgpu: Failed to force to switch arbf0!
Jul  3 11:58:08 guilmon kernel: amdgpu: [disable_dpm_tasks] Failed to disable DPM!
Jul  3 11:58:08 guilmon kernel: [drm:amdgpu_device_ip_suspend_phase2 [amdgpu]] *ERROR* suspend of IP block <powerplay> failed -22
Jul  3 11:58:08 guilmon kernel: amdgpu 0000:01:00.0: [drm:amdgpu_ring_test_helper [amdgpu]] *ERROR* ring kiq_2.1.0 test failed (-110)
Jul  3 11:58:08 guilmon kernel: [drm:gfx_v8_0_hw_fini [amdgpu]] *ERROR* KCQ disable failed
Jul  3 11:58:09 guilmon kernel: amdgpu: cp is busy, skip halt cp
Jul  3 11:58:09 guilmon kernel: amdgpu: rlc is busy, skip halt rlc
Jul  3 11:58:09 guilmon kernel: CPU: 7 PID: 120 Comm: kworker/u16:6 Tainted: G           OE     5.18.5-artix1-1 #1 d915910a51fd3468d4148b188d07d15cfc34c35a
Jul  3 11:58:09 guilmon kernel: Hardware name: To be filled by O.E.M. To be filled by O.E.M./M5A99FX PRO R2.0, BIOS 2501 04/07/2014
Jul  3 11:58:09 guilmon kernel: Workqueue: amdgpu-reset-dev drm_sched_job_timedout [gpu_sched]
Jul  3 11:58:09 guilmon kernel: Call Trace:
Jul  3 11:58:09 guilmon kernel:  <TASK>
Jul  3 11:58:09 guilmon kernel:  dump_stack_lvl+0x48/0x5d
Jul  3 11:58:09 guilmon kernel:  amdgpu_do_asic_reset+0x2a/0x470 [amdgpu 44b7b22559c0805dcd17429d634a6646a1917733]
Jul  3 11:58:09 guilmon kernel:  amdgpu_device_gpu_recover_imp.cold+0x537/0x8cc [amdgpu 44b7b22559c0805dcd17429d634a6646a1917733]
Jul  3 11:58:09 guilmon kernel:  amdgpu_job_timedout+0x18c/0x1c0 [amdgpu 44b7b22559c0805dcd17429d634a6646a1917733]
Jul  3 11:58:09 guilmon kernel:  drm_sched_job_timedout+0x76/0x100 [gpu_sched ac0b937704053fac20da4213593e0fdd3b2a6d2a]
Jul  3 11:58:09 guilmon kernel:  process_one_work+0x1c7/0x380
Jul  3 11:58:09 guilmon kernel:  worker_thread+0x51/0x380
Jul  3 11:58:09 guilmon kernel:  ? rescuer_thread+0x3a0/0x3a0
Jul  3 11:58:09 guilmon kernel:  kthread+0xde/0x110
Jul  3 11:58:09 guilmon kernel:  ? kthread_complete_and_exit+0x20/0x20
Jul  3 11:58:09 guilmon kernel:  ret_from_fork+0x22/0x30
Jul  3 11:58:09 guilmon kernel:  </TASK>
Jul  3 11:58:09 guilmon kernel: amdgpu 0000:01:00.0: amdgpu: BACO reset

The screens all go blank. Trying to perform GPU recovery hangs; only solution is magic sysrq reboot.
Before installing amdvlk, there was no stability problem (but the game had to use DX11 emulation that performs badly).
OS: Artix AMD64
GPU: RX590
RAM: 32GB (about 8 used when gaming)
CPU: AMD FX-8320E
The most likely situation to crash is just starting the game, but it also happens at random while playing. Sometimes it'll work for 6+ hours with no problem, others it'll crash 10 minutes in.

Reproduced with the amdvlk-pro driver but not with amdvlk open source driver


Magic links don't work with Outlook "safe links" (ATP)

Inspired by similar issue on an authentication library: https://github.com/FusionAuth/fusionauth-issues/issues/629
Outlook will rewrite email links to direct them towards https://*.safelinks.protection.outlook.com/ with the original url encoded as a query parameter, when a feature named ATP (Advanced Threat Protection) is enabled. It then fetches the link, invalidating the login token.
Expected behavior
Requests from Outlook should not invalidate the token. As a first mitigation, we can try to:

Detect if requests is made from Outlook's robot

By IP: 40.94.16.*, 40.107.235.*
User agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.109 Safari/537.36. Chrome version seems to vary from one request to another, ie. Chrome/64.0.3282.140.
Other specific info: OS, timezone, locale, Accept, Content encoding, etc. see https://amiunique.org/ for some inspiration. We need to add some logs to get these.
Detect user interaction (mouse move/touch, window resize, ...etc)

If we suspect that the page is fetched from an Outlook robot, add a button to manually confirm the signin before triggering the request.

Additional context
Multiple users affected:

https://opencollective.freshdesk.com/a/tickets/9299
https://opencollective.freshdesk.com/a/tickets/10361
https://opencollective.freshdesk.com/a/tickets/10472
https://opencollective.freshdesk.com/a/tickets/10699

Issue reported for Ghost too: https://github.com/TryGhost/Ghost/issues/12347

Issue reported as fixed by two users after https://github.com/opencollective/opencollective-frontend/pull/5476 :tada:


Need to introduce an event before drag start, i.e. "onClick"

Use case 1:
Developing an analysis/play board, I need to be able to record the piece that was clicked (aka select the piece for move). I would then need to use hover for styling the desired target, and again handle the event to move the piece.
Use case 2:
While developing a play board, I want to prevent the user from dragging a piece that is not allowed to be moved. This might be the wrong color, or may be a piece of the correct color with no valid moves.
Suggested signature:
    /**
     * Signature: function(event: { piece: string, square: string, allowDrag: boolean }) => void
     */
    onClick: PropTypes.func,

Note:
allowDrag would default to true and if the function changes the value to false the drag operation would be cancelled. The event would also need to be fired when no piece was clicked to allow click-to-move behavior defined in use case 1 above.

Use case 1:

I'm looking at chess.com and they record the piece that was clicked and allow you to move the piece by clicking the target square.  This is a very nice feature(I use it all of the time on my ipad playing on chess.com)and wasn't implemented in chessboard.js but I definitely want it for Chessboard.jsx.

I would then need to use hover for styling the desired target

So have some sort of unique hover styling over a desired target.  This seems like a useful feature

Use case 2:
While developing a play board, I want to prevent the user from dragging a piece that is not allowed to be moved

Ok, so as of now draggable prop is for the entire board, that is not going to work for your use case.

@willb335 Yes, I expect that draggable would apply to the whole board... this is a good thing and I need it too :)
Here though, I need to make the decision on a piece-by-piece basis at the moment dragging begins on a piece. Some pieces are valid to drag (white's move) and some pieces have valid moves (king has no adjacent empty squares) and so I want to prevent the drag operation from even starting.

Ok, got it.  I'll use your suggested signature.  Not sure when I'll start on this.  I have some time this coming weekend

perhaps draggable prop could take a boolean for the whole board or an object for specific squares.  draggable={ {'e5': false } }.

@csharptest
https://github.com/willb335/chessboardjsx/pull/22/commits/a1ee12fd267c82addc938d703d446a4b31d33e55
I provided the onClick feature and updated Demo.js. Take a look at 'With Move Validation.'

I was looking for this feature for use case 2.
I can't find the Demo.js being referred to here. Neither the src/Demo.js in this repository nor the one at https://codesandbox.io/s/lp32ojqzqm?from-embed seems to have anything relevant.
Does it provide a way to reject the click? I'm thinking the handler should return true if the piece is allowed to be picked up and false if not.

@furrykef
After looking at this again I realized I misunderstood use case 2 from @csharptest.
see #28
I used:  Signature: function( { piece: string, sourceSquare: string } ) => bool
Please check out Demo.js again with the 'Conditionally Disable Drag' demo
Please let me know if this is a solution for you

As I said before, I don't know where the Demo.js you refer to is.

Ok, if you fetch the allow-drag branch I committed in #28 and then run npm run start  that will load the Demo.js component(usually called App.js, it's the root component)
If you want to see the implementation code for allowDrag then go to https://github.com/willb335/chessboardjsx/blob/allow-drag/src/integrations/AllowDrag.js

That looks fine to me. Thanks.

Ok, I'll look into fixing that

cursor: 'not-allowed' implemented for non-draggable pieces and will be available  for 2.0.1


Added localized box detection algorithm for digits.

What?
Added support for box detection when only a localized part of the student information section of the cover page is selected in the MarkUs interface. These changes currently only work with digit detection for the student number section of the cover page.
Why?
The original algorithm would crash often when it was not provided with the full student information section. However, the MarkUs interface seems to expect only a small, more localized section to be uploaded. These changes ensure consistency between the web interface and the expectations of the image processing scripts in this repository.
How?
The cv2 library in Python was used to detect square contours in the image. This involves using bounding boxes of contours and attempting to approximate them as squares. Erroneous contours (non-box contours) are then removed so as to leave only the box contours.

@david-yz-liu thanks for catching that, I've reverted the changes to cnn.py. I've also added some morphological operations to create an image containing only vertical and horizontal lines of the original image, and then use this image to detect box contours. These changes seem to improve box detection. In hindsight, I probably should have waited to complete this pull request before incorporating these changes. I will make sure I don't try to do too many things at once in the future.


Joh/11455

Coverage increased (+0.2%) to 61.067% when pulling b725c7120fff9ab6594e615c43c688f77282f42f on joh/11455 into 2bb55b7cff44755bafd03c5d50e22e336360b787 on master.

Coverage increased (+1.02%) to 61.891% when pulling 94acaef890eefb60f24ea3c8e028b4bf69a9ef8b on joh/11455 into 2bb55b7cff44755bafd03c5d50e22e336360b787 on master.

Coverage increased (+1.02%) to 61.891% when pulling af55c2005512b474809370e9b8ef9a4dd1eb659c on joh/11455 into 2bb55b7cff44755bafd03c5d50e22e336360b787 on master.

Coverage increased (+1.02%) to 61.891% when pulling af55c2005512b474809370e9b8ef9a4dd1eb659c on joh/11455 into 2bb55b7cff44755bafd03c5d50e22e336360b787 on master.


5€ fee on 28€ swap from mint to LN self custody

I hade 100€ in a mint balance.
Swapped 28 to LN
got a 5€ fee
Doesn't feel right ... at all.

That's the cost of transacting on chain. We display the fee before you swap and ask you to confirm it.

Yes, and I agreed to the high fee. That's not the issue.
The issue is that a bunch of sats from a mint to self-custodial lighting needs to be this high?
Also: new people won't know the difference (the balances are also separate in the settings details, but added up in the first screen totals. Which confuses me... if I pay 180000 sats and I have 186000, I still can't pay, b/c 30000 is in some mint.
This is pure chaos.

We'll keep working to communicate this better. There's a pretty big jump in cost and personal responsibility, and related knowledge about how things work, when going from fedimint to self-custodial. The reason for the high fee, for instance, is based on the current mempool: https://mempool.space/


[layouts] Add ErrorBoundary component

This PR:

Adds a new component: ErrorBoundary - it's meant to wrap around other component and if any of it's children experience an error that it can't recover from, the error boundary component will catch it and display an error note instead of crashing the entire app.

The plan will be to add ErrorBoundary to PageLayout in #309 and any other component we feel relevant.
This component is based on the ErrorBoundary component in Playground https://github.com/mapbox/playground/blob/72c751092c5b0fafa8dfac619096dcbf3c24a4ef/src/components/error.js (we'll be able to replace the local component with this one.)
Screenshots
ErrorBoundary

Fixes #271
How to test

QA checklist

[ ] No errors logged to console.
[x] Accessibility score in Chrome DevTools Audit/Lighthouse is 100% with Lighthouse version: #.#.#.
[x] Component is accessible at mobile-size viewport.
[x] Component is accessible at tablet-size viewport.
[x] Component is accessible at laptop-size viewport.
[x] Component is accessible at big-monitor-size viewport.
[ ] Create a PR in a site repo, copy the component, and test it. Push to staging and let the reviewer know they can also test the component there.

Open the test cases app locally on:

[x] Chrome, no errors logged to console.
[x] Firefox, no errors logged to console.
[x] Safari, no errors logged to console.
[x] Edge, no errors logged to console.
[x] IE11, no errors logged to console.
[x] Mobile Safari, no errors logged to console.
[x] Android Chrome, no errors logged to console.

@danswick As part of pre-layouts work, I'd love to add this new ErrorBoundary component to dr-ui this week!

Added an errorNote and errorNoteTitle props to customize the note:


Children As Property Resource Wrapper

Serializes super-flat children into a single property as a JSON object. This tends to be much more performant.
Add Operation of 10,000
nt:unstructured: 8773ms ~> 8s
oak:Unstructured: 6374ms ~> 6s
unsorted property based: 632ms ~> .6s
sorted property based: 1,167ms ~> 1s

Add Operation of 100,000
nt:unstructured: 160,227ms ~> 160s
oak:Unstructured: 87,324ms ~> 87s  - Package built in 101377ms.
property based: 5,240ms ~> 5s - Package built in 4528ms.
sorted property based: 6,646ms ~> 7s

Does not support multi-value properties

What's the use case? I think I get it, but I'm not 100% sure.

@justinedelson its used to model flat trees or high-volume data in a package-able format. Allows for the simplicity of a flat-structure (since you could shard it to make it less flat) and packages very quickly, while maintaining a custom (via comparator) order.
For example, you might have 250 products, each with 5 variations. Each variation has daily data extending back 10 years; that's  250 * 5 * 10 * 365 = 4,562,500 nodes. Serializing to a property all there to be 250 * 5 = 1250 nodes. Trying to package up 4562500 nodes for replication (or even a subset) tends to be fairly costly. This is also nice in that it does inundate Oak Indexes w 10ks ~> millions of new nodes to deal with, flip side is this data isnt very Oak searchable.
Admittedly this is a niche use-case.

Im going to change the API to align more w the "usual" Resource apis.


How to pass a new line for multiline?

Based on the documentation I just need to pass a \n to get a newline.
However when I do it it doesn't generate a new line but just displays te \n.
I tried using the % character as well and same thing: no new line.

Had no trouble doing this with slack-ruby-client. Happy to help there.
client.message channel: channel, text: "Line1\nLine2."


Generate mailbag structure and manifests with Bagit-Python

The problem the component solves
A Mailbag is a bagit "bag" so after all of the content is created, we should use bagit-python to package it and generate manifests
Relevant part of mailbag spec?
2. Overview of a Mailbag
Type of component

[x] Core
[ ] Input
[ ] Attachments
[ ] Derivatives conversion
[ ] Reporting/Exporting
[ ] GUI
[ ] Distribution

Expected contribution

[x] Pull Request
[ ] Comment with proposed solution

Major challenges or things to keep in mind
Unfortunately, we're not going to be able to completely rely on the bagit.make_bag(), as that will include mailbag.csv in the data or payload directory. So I think we need to create the bag structure first, including the data directory and then use bagit-python to generate the manifests, bagit.txt, bag-info.txt, etc.
Looks like you can bag.save(manifests=False) to not write the manifests, so I would try bag.make_bag(manifests=False) then add mailbag.csv and then bag.save(manifests=True) or something like that.

Addressed with #125


Create "Biography" for Yoda

Write text

Biography
Early life and the High Republic Era
"For 800 years have I trained Jedi."
―Yoda, to Luke Skywalker
Born 900 years before the Battle of Endor, Yoda belonged to an ancient and mysterious species and never revealed his homeworld. As he was keenly attuned to the Force, he joined the Jedi Order as a youngling. After undergoing the ritual known as the Gathering, he received his cyber crystal, and Professor Huyang helped him construct his first lightsaber aboard the Crucible. At some point during his time as a youngling, Yoda skipped across the lily pads in the Jedi Temple's meditation pond. Yoda was paired up with a personal mentor. During their time together, Yoda's Jedi Master told him the tale of K'ungfu and Chuang, two great Jedi of the past. Around the age of 100, Yoda was ready to pass on what he had learned. Having attained the rank of Master, he spent the next eight centuries training and tutoring generations of Jedi. It was estimated that over his lifetime, he trained around 20,000 Jedi.
Yoda is a fictional character in the Star Wars universe, first appearing in the 1980 film The Empire Strikes Back. He is a small, green humanoid alien who is powerful with the Force and served as Grandmaster of the Jedi Order. In The Empire Strikes Back, Yoda was voiced and puppeteered by Frank Oz, who reprised the role in Return of the Jedi, the prequel trilogy, and the sequel trilogy. Outside of the films, the character has been mainly voiced by Tom Kane, starting with the 2003 Clone Wars animated television series. Yoda has since become an iconic figure in popular culture due to his distinct pattern of speech and role as a wise mentor.
In his first appearance in the original trilogy, Yoda is described as the mentor of Obi-Wan Kenobi and lives in exile on the swamp planet of Dagobah. He trains Luke Skywalker in the ways of the Force until his death at the age of 900 in Return of the Jedi, though he later returns as a Force spirit. In the prequel trilogy, Yoda leads the Jedi High Council and trains the young generations of Jedi until they are assigned to a master. When the Clone Wars breaks out, he becomes a general in the Galactic Republic's army and leads several legions of clone troopers. Yoda is one of the few Jedi to survive the events of Order 66 at the end of the war, and faces off against Emperor Palpatine, but is outmatched and forced to go into hiding. Yoda's spirit appears again in the sequel trilogy, advising an older Luke and empowering Rey to face a revived Palpatine.


@Counted seems not working correctly

I'm using spring-boot-1.2.5-release, Java Configuration,  metrics-jersey2, and metrics-sping
	compile "io.dropwizard.metrics:metrics-jersey2:3.1.2"
	compile "com.ryantenney.metrics:metrics-spring:3.1.2"

I also use JmxReporter.  When I add @Counted to a bean's method, I found the value does not increase, sometimes it was reset to zero. Sounds like there are multiple instances of the counter in different threads. But @Timed seems working correctly, the count of the timer metric increases.
I don't think metrics-jersey2 will conflict with metrics-spring. I use the annotations for the jersey resources, and the metrics for jersey resources have different names.
I'm not sure where I did wrong.

The default behavior of @Counted is to only count active invocations of the method. Try setting the monotonic=true, and the counter will monotonically increase.

Yes. I did not set monotonic=true. Thanks.


Remove Walletobjects.php manual dependency

Google Wallet API Client library  should not be downloaded and included manually, as it is now installed by composer from github.com/google/apiclient-services, here is the source.
Also, according to Google PHP API Client UPGRADING.md, since version 2.10.0 the Google SDKs classes are namespaced so we should use those.
As discussed in #75

Thanks so much Razvan. As you've discovered we recently made the Wallet API available to the various cloud client libraries, and we're currently in the process of updating samples and documentation to reflect this. More than likely we'll use the work you've done here once we make all these changes available, so stay tuned and thanks again.

Oh, didn't know about the "recently" part, seems I'm here at the right time 🙂
@stephenmcd Are you from the Wallet/Pay team? I opened an issue about missing Enums in the generated PHP SDK, I'm interested in contributing this for including it my lib, do you accept contributions and can you suggest how to do this?

Oh, didn't know about the "recently" part, seems I'm here at the right time 🙂
@stephenmcd Are you from the Wallet/Pay team? I opened an issue about missing Enums in the generated PHP SDK, I'm interested in contributing this for including it my lib, do you accept contributions and can you suggest how to do this?

@razvanphp Interesting, yeah, that seems to be a cross-language thing, not just php.  The discovery generators are converting them to and from strings (and even supporting various formats like snake_case camelCase etc on the way in).  We can investigate why that is happening with the cloud publishing folks.


Feature  production config

Working production config.  Turns off debug, and uses whitenoise for serving staticfiles.  Closes #12 .

Working production config.  Simply set DJANGO_CONFIGURATION=Production in your .env file to enable.


manifest: update hal_nordic revision to include nRF91 anomaly 7 fix

New hal_nordic revision contains updated nrfx with an improvement to nRF91 anomaly 7 workaround.

More recent hal_nordic revision was already pulled-in in https://github.com/zephyrproject-rtos/zephyr/pull/63461


Add more autogenerated documentation to the whitelist

I went through and checked what React Native exports that we don't have any autogen docs for, and added the missing stuff into the whitelist. The extra docs seem useful.
Still not reference-documented because docgen chokes:
AppStateIOS
ART
Keyboard
LinkingIOS
Platform
RecyclerViewBackedScrollView
Touchable
UIManager
I also fixed up a bit of alphabetical order

By analyzing the blame information on this pull request, we identified @caabernathy and @sahrens to be potential reviewers.

This probably solves #8202 in a non automatic way for now. Looks good.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


[Enhancement] Implemented identity feature

This is my naive approach / suggestion to solve #16. The PR adds a public struct: Identity  to ops/mod.rs: 305:
#[derive(Debug, Default)]
pub struct Identity {
    pub access_key: String,
    pub secret_key: String,
}

This struct contains the owned values for access_key and secret_key that are used for signature verification and passes them through prepare and call to all methods of the s3_trait. With this change it will be possible to use the users identity in upstream logic and associate corresponding backend actions to a specific user, enabling custom user management.
Example of the additions to the methods from the s3_trait:
async fn abort_multipart_upload(
    &self,
    _input: AbortMultipartUploadInput,
+    _identity: Identity,
) -> S3Result<AbortMultipartUploadOutput> {
    Err(s3_error!(NotImplemented, "AbortMultipartUpload is not implemented yet"))
}

These additions do not change anything related to the actual signature verification logic and are optional to be used by the implementer.
Feel free to make suggestions / enhancements when this change does not find in your plans. If these changes are accepted they should surely result in a new release since they changed the method signatures in the s3_trait.

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

Thank you! But I think the issue needs a better solution.
My draft:
s3s can implement a type like aws_credential_types::Credentials.
pub struct S3Request<T> {
    pub input: T, // operation input
    pub credentials: Option<Credentials>, // identity information. `None` for anonymous request
    pub extensions: http::Extensions, // forward extensions from the http request
    ...
}

async fn abort_multipart_upload(
    &self,
-   _input: AbortMultipartUploadInput,
+   _req: S3Request<AbortMultipartUploadInput>,
) -> S3Result<AbortMultipartUploadOutput> {
    Err(s3_error!(NotImplemented, "AbortMultipartUpload is not implemented yet"))
}

Yeah, as i've said my approach was more of a hacky prototype solution than a fully fletched out one. Your draft sounds much more ergonomic, if you'd like i could give it a try.

Finished in commit 4039538daf2366c13d19e0cad3c962cf18b1b507. It's a big refactor so I implemented by myself.

Very nice, thanks a lot ! This will make our work a lot easier.


Mocks can cause segfaults/undefined behavior

Required information
Operating system:
Ubuntu 18.04 LTS
crash_patch.zip
Compiler version:
GCC 7.4.0
Observed result or behaviour:
A segfault occurred when trying to load OpenCV lib was used. After some digging the problem could be isolated and is now reproducible with the attached patch file.
The problem is the time mock in iceoryx, which implements a mock of getTime(). On one hand, getTime mock has the same signature as the system function and on the other hand OpenCV is doing a static init using getTime.  As the order of initialization for static variables is undefined OpenCV was sometimes using the real system function and sometimes the mock (which wasn't fully initialized and resulting in a nullptr segfault) .
Expected result or behaviour:
No segfault.
Conditions where it occurred / Performed steps:

Apply attached crash patch (patch -p1 < crash.patch).
Run ./build/posh/test/posh_moduletests --gtest_filter="MqMessage_test*"

Fix:
A fix is ready and basically routing the functions "on demand" and not in initialization.

so this is similar to #327 and there is another static initialization order fiasco ... I fear we have more of these in our codebase


feat: Add MacOS support

add support for https://github.com/microsoft/react-native-macos

Thank you! 🎉


live reload doesnt work with long base path to folder

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.2.0
node: 6.11.0
os: darwin x64
Repro steps.

cd ~/Desktop
ng new foo
cd foo
ng serve

change a source file, and live reload works!
but same thing in a deep base directory fails! e.g.....

cd /Users/rbw/Workspaces/Eclipse 4.6 Neon (from new mini 3.6-Kepler)/www.example.com/WebContent/_AngularCliTutorial/demoApp

do identical actions and live reload never works

The log given by the failure.

Desired functionality.

obviously, live reload should work regardless of path
Mention any other details that might be useful.

With that path, other possible candidates are:

doesnt like spaces in path?
doesnt like special characters in path?

This issue is a duplicate of https://github.com/angular/angular-cli/issues/1610, where these and other limitations are known and being tracked.


Introduce checkstyle plugin

Based on https://github.com/spring-io/spring-javaformat
In this iteration, checkstyles are only enabled for spring-ai-core

merged in 8e758dbd00bf6865edf35f3bdaaeadabe2b250e6


Add support for multiple environments

I was just pulling this in from a fork because it looked useful, do what
you will with it. 😀
On Apr 20, 2017 10:10 PM, "Chris Aubuchon"<EMAIL_ADDRESS>wrote:

@ChrisAubuchon commented on this pull request.
In commands/commands.go
https://github.com/mantl/consul-cli/pull/47#discussion_r112609937:

@@ -19,6 +23,106 @@ type Cmd struct {
Template string
}

+type TomlConfig struct {

Env   string
Dev   ConsulFromFile
Prod  ConsulFromFile
Stage ConsulFromFile
QA    ConsulFromFile
West  ConsulFromFile
East  ConsulFromFile
+}

What about a map[string]ConsulFromFile instead of hardcoding environment
names?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mantl/consul-cli/pull/47#pullrequestreview-33938311,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADetTb0TMFNTX0WBf8aNXrZy80HhhAVks5ryDorgaJpZM4ND1f0
.

Would you mind tweaking this code?
On Apr 20, 2017 10:11 PM, "Chris Aubuchon"<EMAIL_ADDRESS>wrote:

@ChrisAubuchon commented on this pull request.
In commands/commands.go
https://github.com/mantl/consul-cli/pull/47#discussion_r112610039:

+}

+func CheckConsulFile(inputFile string) string {

tempConsulFile := inputFile
if inputFile[:2] == "~/" {

user, _ := user.Current()

homeDir := fmt.Sprint(user.HomeDir + "/")

tempConsulFile = strings.Replace(inputFile, "~/", homeDir, 1)

}
_, err := os.Stat(tempConsulFile)
if err == nil {

return tempConsulFile

}
return ""
+}

Might want to look at https://github.com/mitchellh/go-homedir for homedir
detection. Biggest benefit is that it works in windows as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mantl/consul-cli/pull/47#pullrequestreview-33938408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADetVZNKbuDG2BjVt_ITDZ5IJbTee1_ks5ryDqSgaJpZM4ND1f0
.

Yeah. I'll run with it. It's a cool idea.

Awesome, thanks.
On Apr 20, 2017 10:13 PM, "Chris Aubuchon"<EMAIL_ADDRESS>wrote:

Yeah. I'll run with it. It's a cool idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mantl/consul-cli/pull/47#issuecomment-296069168, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADetQfVBcjFGpheoemYx_SGwS44VfsUks5ryDr3gaJpZM4ND1f0
.


Windows: Support serial ports > COM9

Per #393 FluxEngine can not open the serial port when COM port number is greater than COM9.
Windows requires the use of the DOS UNC path name for COM ports greater than COM9.
Fix: Prefix the port name with \\.\

Windows is awesome. Thanks very much for this.


📄 Add LICENSE

Codecov Report

Merging #2 (ef472d4) into main (4429aa3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   68.47%   68.47%
=======================================
  Files           5        5
  Lines         314      314
=======================================
  Hits          215      215
  Misses         99       99

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 4429aa3...ef472d4. Read the comment docs.


Enhance AutoComplete with faceted search queries seen in VisualSearch.js

Your AutoComplete component is neat but we are missing some cool features here you can see in
https://documentcloud.github.io/visualsearch/
Especially the ability to specify the facets and the x icon next to them for removal.

@binarykitchen we're actually looking at removing application logic from AutoComplete (or splitting it out into a helper) rather than adding more proprietary application logic. Sorry if that isn't what you were hoping to hear!

Thats a good move. But I am rather missing the ability to type multiple search words. Thats a pure UI feature, not application logic I think.

@binarykitchen Ah, you mean like a Chip powered multi-select autocomplete thingamadoodle?

@binarykitchen the pieces are in the making: https://github.com/callemall/material-ui/pull/3870

yess!


Fix mut twice error

Fixes #33

@homu r+

:pushpin: Commit 6ee535c has been approved by martndemus

:zap: Test exempted - status


Default virtualenv dir doesn't look right

Poetry configures something like:
~/.cache/pypoetry/virtualenvs

as the default virtualenv dir not ~/.virtualenvs
So if you try to load this plugin it wont work with a vanilla poetry setup.

Does it make sense to change this?
Does it make sense to add some error messages about this? I had no idea what was going wrong until I starting working through the code with print statements
I think falling back to poetry env info -p if you know the user is in a poetry project directory but can't find anything manually is probably a good idea. Right now I don't know if the code can ever hit this option because the "no venvs" option will get hit first and return

Also I'm pretty sure line 64 should explicitly return None instead of just being a return statement

it is the directory used by vox by default as well. I don't want to change this. It is better set explicitly to change poetry's behavior
It would be better to do the "3" and find the venv path using poetry env info -p

for (1) I may be misunderstanding the purpose of this plugin, I thought it was to automatically activate poetry venv when CDing into a poetry controlled project folder, in which case why would we care about the default vox folder, we would only care about the default poetry folder
For (2/3) I agree with you

I see this as a vox plugin that supports poetry created environments. I personally use the vox commands to delete/upgrade the poetry environments. It auto-completes when kept in the same place.

I have released a new version with the fix. It will fallback to poetry env.

This fails with the following exception (which was initially hidden by a contextmanager that I had to comment out)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.10/subprocess.py", line 420, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/usr/lib/python3.10/subprocess.py", line 501, in run
    with Popen(*popenargs, **kwargs) as process:
  File "/usr/lib/python3.10/subprocess.py", line 969, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.10/subprocess.py", line 1845, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'poetry'

poetry commands work fine when run in the xsh shell

Seems like it needs a full path, changing it to p = Path(run("/home/max/.local/bin/poetry", "env", "info", "-p")) works

Seems like you want from xonsh.lib.subprocess import check_output instead of the one from pythons subprocess library so that it gets resolved using the $PATH set in xonsh

I see this as a vox plugin that supports poetry created environments. I personally use the vox commands to delete/upgrade the poetry environments. It auto-completes when kept in the same place.

Cant the existing autovox plugin handle that? I haven't used xonsh long so I don't know for sure how it works. How do you use the vox commands to create poetry environments?

The env is created by poetry but under $VIRTUALENV_HOME, and it is detected by this plugin. Autovox was slow in my case, so I created this one.


Turn GenerateCompiledExpressionsTempFile into a no-op

This target in $(MSBuildFrameworkToolsPath)\Microsoft.WorkflowBuildExtensions.targets does bad things to builds inside Visual Studio. Specifically, it generates files only when building from within VS, for both real and design-time builds. This leads to all kinds of build problems.
In Dev16 we no longer need these files to exist on disk for Workflow Designer scenarios. We can't modify $(MSBuildFrameworkToolsPath)\Microsoft.WorkflowBuildExtensions.targets because that would break the Workflow Designer for older versions of VS using newer versions of the Framework. Instead, we effectively turn it off by overriding it with a no-op implementation here.

Per offline thread with @pilchie, this is desired for 16.0. Merge will be followed up with a VS insertion to take to the release quarterback.


Collaboration - Activity Threads, Comments, Requests

As part of 0.7 we introduced Activity feed based on the change events that are being generated by bot or user activity with-in
OpenMetadata.   Collaboration is one of the founding principles behind the OpenMetadata.
In 0.9 release we will introduce conversations, threads, referring other users with-in threads and asking questions etc..
Initial thoughts around activity feeds, posts and threads

Users should be able to create posts and refer entities and ask questions etc..
Users should be able to comment on the change log entries. Example User X is edited the description and User Y should be able to reply to the activity thread saying please correct the description. Similarly users can discuss about columns, sample data etc..
We should be able to generate a feed per entity. That is group any historic threads per entity. This can later become knowledge threads and users/owners can flag certain threads as important and pin them in the entity page

@sureshms please add more details

Initial discussions:
Currently, we are using change descriptions based on events to populate the Activity feed on the homepage. We should start using the thread_entity table to persist any activity including the change events. The thread_entity table will need schema changes to store userName, createdTimestamp, entityType, etc.
The MessageParser can be improved to support "#" and "@<user/team>" with advanced searches to get nested suggestions such as <table_name>.<column_name>.<field>. We can use elastic search to index these entity names and user names.
We should also track @Mentions as a separate column in thread_entity to be able to display all the relevant threads to a user on the homepage. When a user interacts on a thread by posting a comment, the user should be automatically added to the @Mentions column so that they get future notifications of any activity on the thread. We should also support "unsubscribe" feature for users to ignore notifications on a thread. But, it need not be part of this epic. We can add that feature later. Also, we will not support the editing of posts in the initial release.
User Experience
Users should be able to start a thread or post comments from two places:

Any of the entity pages such as Messaging Service, Table etc. Users should be able to request a description by clicking the button Request Description next to the description field on the page. They should also be able to comment on an existing thread or create a new thread that should automatically be assigned to the owner of the entity.
From the activity feed on the homepage

Activity Feed:

It should list all the activity that happens around the entities they follow, by either being the owner of an entity or by participating in any thread.

Tasks

Update "thread_entity" schema to support "userName", "createdTimestamp", "entityType", "followers" etc.
Add API(s) to serve thread related requests
Add API to get autosuggestions from elastic search
UI: Get activity from thread_entity API
UI: Create mocks to show icons in entity pages to start a thread
UI: Show autosuggestions for entity names or user/team in posts when "#/@" is used

@darth-coder00 @Sachin-chaurasiya @sanketbambulkar @shahsank3t please take a look at the above details. We need to work on mockups for these.

We have officially merged all the features related to Collaboration that we scoped for this 0.9 release.


32 bit process, loading 64 bit module failure

DWORD64 WINAPI LoadLibrary64(const TCHAR* filePath)
{
	static X64Call LdrLoadDll("LdrLoadDll");
	if (!LdrLoadDll) return 0;

	DWORD64 module = 0;
	DWORD64 status = LdrLoadDll(0, 0, filePath, &module);
	if (!status) return module;

	SetLastError64(status);
	return NULL;
}

1
//There seems to be a mistake in the source code.
//The function should initialize the string.（LdrLoadDll）
RtlInitUnicodeString(A,B);
LdrLoadDll(0, 0,A,x);
2
32 bit process, loading 64 bit module failure
The 32 bit process can load 64 bit modules.
I can't do it. I'm looking for a solution.
yapi and wow64ext， it still won't solve the problem

Sorry for not testing the LoadLibrary64 function that inspired from other enhanced wow64exts.
I think you'd like to see the example below:

Popular LoadLibrary example
    YAPICall LoadLibraryA(hProcess, _T("kernel32.dll"), "LoadLibraryA");
    DWORD64 x86Dll = LoadLibraryA("D:\\x86.dll");
    DWORD64 x64Dll = LoadLibraryA.Dw64()("D:\\x64.dll");
    _tprintf(_T("X86: %I64x\nX64: %I64x\n"), x86Dll, x64Dll);

If failed all the same, check the errcode and post it here again.
Thanks for you kind feedback.
Looking forward to your further reply.
Orca

I'll REMOVE the deprecated LoadLibrary64 ASAP.


strict: true is not working when image is rotated

When you rotate image in strict mode, image can get smaller than crop box.

Are you sure? Can you provide a demo for this issue?

You can try it on http://fengyuanchen.github.io/cropper/. Rotate image and you will see, borders of crop are outside of image.

What strict mode limits is just the canvas (the wrapper of the image).


improved the efficiency of matmul() by completely removing concatenation

[x] Supplements #7000
[x] Tests passed
[x] Passes pre-commit run --all-files

This PR avoids concatenation in both blockwise() and reduction() within the implementation of matmul().  Previously, matmul() avoided concatenation only in blockwise().
Due to the special nature of the preceding blockwise contraction, all chunk-operands of a sum during the reduction phase are expected to have exactly the same shape, with a size of 1 for the reduction axis.  This makes mere element-wise addition of the chunks possible.   Furthermore, the summation result can be merely reshaped to lose the reduction axis when requested.  In principle, both these operations (element-wise addition and reshaping) are performance boosters compared to the default concatenation-mediated reduction which often involves data rearrangement.
The code of matmul() has also been shortened, making it a bit more readable.  This is a direct consequence of ensuring that the position of the new-axis introduced in the output of chunk-function _matmul() [the first argument to blockwise()] is consistent with its expected position in the output of blockwise() (the position of the contraction-axis).
Performance is comparable to the original matmul() implementation in all cases except one, in which this new implementation is approximately 20% faster, namely, "the case where all dimensions are chunked".  This outcome is consistent with the total absence of concatenation-overhead in the new version.   Performance results can be found in this jupyter notebook.
See discussion with @ravwojdyla at https://github.com/dask/dask/pull/7000#discussion_r756926031 for the motivation of this PR.

Can one of the admins verify this patch?

Attention:  @dask, admins
Many CI tests are broken (but not because of this PR), as seen here (#8423) and elsewhere (#8424).

Hi @dask
I need some help debugging this PR for the unit-test dask/array/tests/test_array_function.py::test_unregistered_func[<lambda>6] , which passes with numpy version 1.21.4, but not versions ≤ 1.19.5.
The bug originates from using a list of arrays of type dask.array.tests.test_dispatch.EncapsulateNDArray innumpy.sum which is implemented by numpy.add.reduce().  I have not been able to determine what changed in numpy.add.reduce() between versions 1.19.5 and 1.21.4 that killed this bug.  The error thrown is
ValueError: setting an array element with a sequence.

Yay! CI tests are now operational.  Thanks @dask admin!

Sorry for the lack of response @ParticularMiner. I'll leave review to @ravwojdyla or maybe @eriknw

@ravwojdyla
Using squeeze() would also be my preference.
Unfortunately, again, for the array type dask.array.tests.test_dispatch.EncapsulateNDArray (found defined in dask/array/tests/test_dispatch.py) squeeze()  raises an AttributeError: 'EncapsulateNDArray' object has no attribute 'squeeze'.
So I reluctantly have to revert to using reshape() instead.  Pity!

@ravwojdyla
I have now replaced the following code in _chunk_sum():
        item_type = type(a[0])
        if item_type is np.ndarray:
            out = chunk.sum(a, dtype=dtype, axis=0)
        else:
            a = [item.__array__() for item in a]
            out = chunk.sum(a, dtype=dtype, axis=0)
            out = item_type(out)

with this
        xp = np

        if is_cupy_type(a[0]):
            import cupy

            xp = cupy

        out = reduce(partial(xp.add, dtype=dtype), a)

where reduce is imported from the package functools.   Fortunately, all CI tests pass with these changes.  Moreover, just like in _matmul() I explicitly test for the existence of a cupy type.
The latter code snippet seems to give a slight performance edge over the former making it more than 25% faster (compared to 20% by the former) than than the original matmul in the case where all dimensions are chunked.

Thanks for your detailed review on this @ravwojdyla !  I really appreciate that.
It's getting to midnight here, so I'll likely respond to any more messages tomorrow.

@ravwojdyla
You will find that I added the squeeze() method to EncapsulateNDArray, discarded shape_minus_axis together with one conditional branch of _chunk_sum.  Everything seems to work well with these changes.  Performance seems to be even somewhat faster than before.
I guess the only outstanding issue is the one regarding the cupy type array.
Thanks.

@ParticularMiner perfect, thank you. I suspect that cupy branch is not needed, but there's a chance I don't fully understand why it would be there, maybe let's give @quasiben some time to respond to the comment in https://github.com/dask/dask/pull/8423#discussion_r761452511

Sure @ravwojdyla. Thanks for the great suggestions.

I suspect that cupy branch is not needed, but there's a chance I don't fully understand why it would be there, maybe let's give @quasiben some time to respond to the comment in #8423 (comment)

It seems @jakirkham might also be able to address this question:
As dask.array is right now, is it necessary to test for the cupy chunk type (as @quasiben introduced:
https://github.com/dask/dask/blob/43247806aa385d6f958613f77a5b84c3a763823e/dask/array/routines.py#L335-L347 for dask.array.matmul in #7563) whenever one needs to use the cupy implementation of some numpy function like numpy.add or numpy.sum?
Is there a reason it would be less convenient to use a more centralized approach, like register the particular cupy function implementation in a "lookup registry", as has been done, for example, with concatenate for various chunk types including cupy itself?
https://github.com/dask/dask/blob/43247806aa385d6f958613f77a5b84c3a763823e/dask/array/backends.py#L113-L129

That's fine, @ravwojdyla .  It's not urgent to merge now.

pending resolution of the question about the need for cupy check ( #8423 (comment)). @ParticularMiner that said if you prefer to merge sooner, and open an issue to discuss that branch/question etc, that sounds good to me too.

@ravwojdyla thank you so much for reviewing this work and thank you @ParticularMiner for engaging! I'll broaden the ping on https://github.com/dask/dask/pull/8423#discussion_r761439317 a bit to see if anyone on @dask/gpu knows if that branch is still necessary.

@ravwojdyla
Thanks again for clearing things up!
@jsignell
It appears all outstanding issues here have been addressed, and this PR is now awaiting your deployment.
Many thanks for your effective oversight!

I believe the observed test-failures for Python 3.7 on Windows are not related to this PR.  (Correct me if I'm wrong.)

I agree that the failing tests are unrelated. They are also showing up on main.

Thanks for this work! The end diff looks super clean :)


Failing unit tests

We had the following failures in a CI environment that runs all unit tests after having added kindr and kindr_ros to the workspace. Is this a known problem?
-- run_tests.py: execute commands
  /opt/sdw/devel/lib/kindr_ros/kindr_ros-test --gtest_output=xml:/opt/sdw/build/kindr_ros/test_results/kindr_ros/gtest-kindr_ros-test.xml
[==========] Running 18 tests from 7 test cases.
[----------] Global test environment set-up.
[----------] 2 tests from RosGeometryMsgVectorEigen
[ RUN      ] RosGeometryMsgVectorEigen.convertFromRosGeometryMsg
[       OK ] RosGeometryMsgVectorEigen.convertFromRosGeometryMsg (1 ms)
[ RUN      ] RosGeometryMsgVectorEigen.convertToRosGeometryMsg
[       OK ] RosGeometryMsgVectorEigen.convertToRosGeometryMsg (0 ms)
[----------] 2 tests from RosGeometryMsgVectorEigen (1 ms total)

[----------] 4 tests from RosGeometryMsgPositionEigen
[ RUN      ] RosGeometryMsgPositionEigen.convertFromRosGeometryPoint32Msg
[       OK ] RosGeometryMsgPositionEigen.convertFromRosGeometryPoint32Msg (0 ms)
[ RUN      ] RosGeometryMsgPositionEigen.convertToRosGeometryPoint32Msg
[       OK ] RosGeometryMsgPositionEigen.convertToRosGeometryPoint32Msg (0 ms)
[ RUN      ] RosGeometryMsgPositionEigen.convertFromRosGeometryPointMsg
[       OK ] RosGeometryMsgPositionEigen.convertFromRosGeometryPointMsg (0 ms)
[ RUN      ] RosGeometryMsgPositionEigen.convertToRosGeometryPointMsg
[       OK ] RosGeometryMsgPositionEigen.convertToRosGeometryPointMsg (0 ms)
[----------] 4 tests from RosGeometryMsgPositionEigen (0 ms total)

[----------] 2 tests from RosGeometryMsgRotationQuaternionEigen
[ RUN      ] RosGeometryMsgRotationQuaternionEigen.convertFromRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999875
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0125008% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgRotationQuaternionEigen.convertFromRosGeometryMsg (0 ms)
[ RUN      ] RosGeometryMsgRotationQuaternionEigen.convertToRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999538
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0461902% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgRotationQuaternionEigen.convertToRosGeometryMsg (0 ms)
[----------] 2 tests from RosGeometryMsgRotationQuaternionEigen (0 ms total)

[----------] 2 tests from RosGeometryMsgPoseEigen
[ RUN      ] RosGeometryMsgPoseEigen.convertFromRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 1.00011
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0105983% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgPoseEigen.convertFromRosGeometryMsg (0 ms)
[ RUN      ] RosGeometryMsgPoseEigen.convertToRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999538
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0461902% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgPoseEigen.convertToRosGeometryMsg (0 ms)
[----------] 2 tests from RosGeometryMsgPoseEigen (0 ms total)

[----------] 2 tests from RosGeometryMsgTransformationEigen
[ RUN      ] RosGeometryMsgTransformationEigen.convertFromRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 1.00011
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0105983% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgTransformationEigen.convertFromRosGeometryMsg (0 ms)
[ RUN      ] RosGeometryMsgTransformationEigen.convertToRosGeometryMsg
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999538
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0461902% > 0.01%
" thrown in the test body.
[  FAILED  ] RosGeometryMsgTransformationEigen.convertToRosGeometryMsg (0 ms)
[----------] 2 tests from RosGeometryMsgTransformationEigen (0 ms total)

[----------] 3 tests from RosTfPoseEigen
[ RUN      ] RosTfPoseEigen.convertFromRosTf
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 1.00011
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0105983% > 0.01%
" thrown in the test body.
[  FAILED  ] RosTfPoseEigen.convertFromRosTf (0 ms)
[ RUN      ] RosTfPoseEigen.convertToRosTf
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999875
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0125008% > 0.01%
" thrown in the test body.
[  FAILED  ] RosTfPoseEigen.convertToRosTf (0 ms)
[ RUN      ] RosTfPoseEigen.convertTwoWays
unknown file: Failure
C++ exception with description "[std::runtime_error] /opt/sdw/devel/include/kindr/quaternions/Quaternion.hpp:319: UnitQuaternion() Input quaternion has not unit length.
Comparing Scalars:
Scalar 1: norm() = 0.999538
Scalar 2: static_cast<Scalar>(1.0) = 1
Error was 0.0461902% > 0.01%
" thrown in the test body.
[  FAILED  ] RosTfPoseEigen.convertTwoWays (0 ms)
[----------] 3 tests from RosTfPoseEigen (0 ms total)

[----------] 3 tests from TfConventionTest
[ RUN      ] TfConventionTest.Concatenation
[       OK ] TfConventionTest.Concatenation (0 ms)
[ RUN      ] TfConventionTest.Rotation
[       OK ] TfConventionTest.Rotation (0 ms)
[ RUN      ] TfConventionTest.GeometricalInterpretation
[       OK ] TfConventionTest.GeometricalInterpretation (1 ms)
[----------] 3 tests from TfConventionTest (1 ms total)

[----------] Global test environment tear-down
[==========] 18 tests from 7 test cases ran. (2 ms total)
[  PASSED  ] 9 tests.
[  FAILED  ] 9 tests, listed below:
[  FAILED  ] RosGeometryMsgRotationQuaternionEigen.convertFromRosGeometryMsg
[  FAILED  ] RosGeometryMsgRotationQuaternionEigen.convertToRosGeometryMsg
[  FAILED  ] RosGeometryMsgPoseEigen.convertFromRosGeometryMsg
[  FAILED  ] RosGeometryMsgPoseEigen.convertToRosGeometryMsg
[  FAILED  ] RosGeometryMsgTransformationEigen.convertFromRosGeometryMsg
[  FAILED  ] RosGeometryMsgTransformationEigen.convertToRosGeometryMsg
[  FAILED  ] RosTfPoseEigen.convertFromRosTf
[  FAILED  ] RosTfPoseEigen.convertToRosTf
[  FAILED  ] RosTfPoseEigen.convertTwoWays

 9 FAILED TESTS
-- run_tests.py: verify result "/opt/sdw/build/kindr_ros/test_results/kindr_ros/gtest-kindr_ros-test.xml"

Tested with:

Ubuntu Xenial 16.04 (amd64)
ROS Kinetic installed from released debs and up-to-date at the time of writing
Eigen 3.3 installed from system debs
kindr commit 6f95732 (latest master)
kindr_ros commit 9a0e8bf (latest master)

ping

Ubuntu 16.04 has by default a beta version of Eigen3.
It is unstable.
See https://bugs.launchpad.net/ubuntu/+source/eigen3/+bug/1750794 for more info.
I suggest to use a stable Eigen version and try again.


Unable to Clone the Github NVM repos.

Hii i am unable to clone the repo. it giving me the errors on my docker Ubuntu image. Attached it he error screen shot. Please have a look, and help me if anything m missing

This seems like a DNS issue in your docker image, and not an issue with nvm.
I'm not personally familiar with Docker so I don't know how I can help.


Review Grocery Project Outline

Review client supported documentation on the project by downloading the files/links and going through the material.

Separated tasks by section, needs assigning to students as individual issues.


Media::contentLength

Media::contentLength is a derived attribute which is calculated by Media_2::setContent.

fixed with opencrx-4.2.0+


Apk check superseded tracks

Checklist

[X] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[X] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[X] I've read the Contribution Guidelines
[X] I've updated the documentation if necessary.

Motivation and Context

Rework of #9250 after google-api-client correct version pinning #9445
Description
Same changes as previous PR except no changes in the referenced google-api-client

Any chance of merging this soon? This is blocking our usage of fastlane.


Add email confirmation for new sign-ups

Given the rash of spam posts on the forum, it would be nice to email new users a link that they could click on to verify their accounts. Verifying by email would then "allow" the user to start posting on the forum.

According to this, I think it's possible for us to make the email confirmation conditional:
http://stackoverflow.com/questions/4790685/devise-is-it-possible-to-not-send-a-confirmation-email-in-specific-cases-eve
In user.rb
  # Callback to overwrite if confirmation is required or not.
  def confirmation_required?
    !confirmed?
  end

Looks like this may not currently be needed, removing from milestone.


CLOUD-47850 Refactor disk types, fix integration tests

@schfeca75 plz

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1832/


Use fully qualified Urls for amp-consent x 3

same as #1347 for a different file (sorry using GitHub UI and could not do a single PR)

All of those have now been deployed!


[WIP] Add missing pieces for PR #1692

This PR fixes build and test problems for PR #1692.

Backport of change from Bitcoin upgrading fontconfig in depends to version 2.12.1.
Another upgrade of fontconfig in depends to 2.12.6. This is needed to fix compilation error with glibc 2.25.
Fix for building Qt on systems that use pkgconf instead of legacy pkgconfig.
Backport of Bitcoin PR bitcoin/bitcoin#7814: switch to Python 3.
Backport of Bitcoin PR bitcoin/bitcoin#8139: Fix interrupted HTTP RPC connection in Python 3.5+.
Backport of Bitcoin PR bitcoin/bitcoin#8839: Avoid ConnectionResetErrors during RPC tests.

See individual commits for more detailed descriptions of changes and why they are needed.
These fixes work when building on modern Fedora 26.
However Travis tests may fail for now, but I know why and working on fix for old Ubuntu that Travis uses.

Now only macOS build fails because it's unable to download macOS SDK.

Thanks, i think I'll add Python3 migration to #1692 instead of removing the utf-8 characters though

Python 3 migration (Bitcoin PR bitcoin/bitcoin#7814) is quite intrusive, changing 72 files.
Removing two UTF-8 graphics symbols from strings is much simpler. They make no sense anyway.

OK, I've backported Bitcoin PR bitcoin/bitcoin#7814 that switches tests to use Python 3.

All tests are green except the macOS one: it's unable to download SDK.

@OlegGirko please change the domain in travis.yml from bitcoin.jonasschnelli.ch to static.dashdot.io

@schinzelh: Doesn't work. "404 Not Found".

Why is it not https://bitcoincore.org/depends-sources/sdks which is the current one? #1692 changed it for some reason but it probably shouldn't.

#1692 in general looks quite messy now and should be reworked imo. I'd prefer it to be more like #1697 tbh i.e. backport commits that merge corresponding PRs instead of backporting the whole set of individual commits inside each of these PRs (if a PR is backported with all its commits).

OK, I've just added backport of Bitcoin PR bitcoin/bitcoin#8304 to fix SDK_URL instead of my own fix. Let's see how it works.

@UdjinM6: I'd prefer to have have one Dash PR for one backported Bitcoin PR.
And it would be better to merge each PR without fast-forwarding instead of squashing it into a single commit.

OK, now all tests pass successfully.


[Other | Optimize ]:  flashing screen and lagging

Hi! Thanks for the great plugin.
When using lsp gd, jumping to the source code the plugin should not load the information because the user does not necessarily care about the references in the source code.
It can be filtered out by lsp root_dir to not load relationships outside of the working directory.

Hi! Thanks for the great plugin.

Hi! Thanks for the feedback!
In #39 I added a disable.cond list where you can specify with function on which buffers symbol-usage should not be run.
E.g.:
require("symbol-usage").setup({
  disable = {
    cond = {
      function()
        -- disable for all files in node_modules
        return vim.fn.expand("%:p"):find("/node_modules/")
      end,
      function()
        -- disable for all files outside of the cwd
        return vim.fn.expand("%:p"):find(vim.fn.getcwd())
      end,
    },
  },
})


Spectrogram plugin: make the container parameter optional

Short description
Currently, if you try to create a spectrogram plugin without specifying a container explicitly in the plugin options, the typescript compiler will complain. I'm pretty sure this is a mistake in the type definition, because the implementation seems to cope fine if the container is not specified, and the example does not specify a container. Assuming that's right, this change corrects the type def.
Implementation details
I just made SpectrogramPluginOptions.container optional, and added a test for the case where it is specified (as the case where it wasn't is already covered by existing tests, which did not run into the issue because they are js rather than ts).
How to test it
In Typescript code, try to create a Spectrogram plugin that does not specify a container explicitly.
Screenshots
n/a
Checklist

[x] This PR is covered by e2e tests
[x] It introduces no breaking API changes

Nice, thank you!  Cheers

Done, sorry about that. I also moved the expected output screenshots for my spectrogram tests to where I think they are expected to be when running yarn test. I'm a little confused because:

It seems like yarn cypress and yarn test create/check screenshots in different locations
As far as I can see, the other tests don't have snapshots under version control, so unless I'm missing something, the test run will just take snapshots without comparing?
I've also noticed a little non-determinism with the spectrogram where sometimes there's a few pixels differences between the expected and new spectrogram, resulting in flaky tests.

As a result of the above doubts, this PR probably isn't ready for merge. Let me know if you have any thoughts on the above, else I'll keep digging. :)
Thanks!

Oops. Good catch! looks like I accidentally deleted the snapshots in #3105. I'll restore them in a separate PR.
And yes, I need to figure out why yarn cypress and yarn test create them in different folders.

Hey @rewbs, I've pushed your change here: https://github.com/katspaugh/wavesurfer.js/commit/994a6af3d108b2b8effdc4eebd61c6c11947e1bf
Without the test because snapshotting the entire body seems to be giving inconsistent results.


mseccfg values when Smepmp and Zkr aren't implemented

mstatus.SBE is defined to be a WARL bit that is hardwired to 0 if S-mode is not supported.  The mseccfg CSR defines 5 bits but doesn't have similar language.  Technically, I think that this requires all implementations (regardless of Zkr/Smepmp support) to follow the definitions supplied in Zkr and Smepmp.  I'm sure that's not the intention.
It looks like *envcfg may need something similar for the Sstc and Zicbo* bits.

This detail will be addressed when those specs are integrated back into the mainline.  Of course, as you say, the expectation is that the bits will be hardwired if the corresponding extension is not implemented.


[Disk Manager]: Snapshot: Check chunk contents for zeroes

Sometimes NBS can return chunks full of zeroes yet the zero field is set to false. Check if chunk is actually a zero chunk.

Closing since I made a new PR from a non-main branch in my repo. This PR waits for long-running tests, keeping a PR in my repo's main blocks my developement.


0.26.0 upgrade fails - looks to be a change in the setup.py throwing this error

Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if any
[ x] I have reproduced the issue with at least the latest released version of superset
[ x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.26.0 upgrade
Expected results
perform the update
Actual results
. ./venv25rc3/bin/activate
python -V
Python 2.7.12
pip -V
pip 10.0.1 from /home/derwin/venv25rc3/local/lib/python2.7/site-packages/pip (python 2.7
pip install superset --upgrade
Collecting superset
Using cached https://files.pythonhosted.org/packages/89/36/3dcf6e8e9ab80e94af0bc4e6ddf296d8ece737f52b39f721e0296267a9e8/superset-0.26.0.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-liolni/superset/setup.py", line 19, in
with open('README.md', encoding='utf-8') as f:
TypeError: 'encoding' is an invalid keyword argument for this function
Steps to reproduce
pip install superset --upgrade
I see that setup.py was just updated to include this encoding code - did that change the requirements to install/update setup from 2.7 to 3.x?
Also tried running
wget https://raw.githubusercontent.com/apache/incubator-superset/master/requirements.txt
pip install -r requirements.txt

Fixed in #5338

This has been fixed in 0.26.1, please close.

It seems that the newest distribution 0.26.1 ist not available at pypi.
pip list --outdated --trusted-host pypi.python.org
==>
superset         0.25.6  0.26.0  sdist

0.26.2 is working. thx


Meteor Docker

I'm new to docker.. Can someone please give me a full documentation on how to dockerize meteor? I did my research and it just doesn't work. I'm stuck at running my meteor app in local. Can someone help?
docker run -d
-e ROOT_URL=http://yourapp.com
-e MONGO_URL=mongodb://url
-e MONGO_OPLOG_URL=mongodb://oplog_url
-v /mybundle_dir:/bundle
-p 8080:80
meteorhacks/meteord:base
can someone supply those with the right urls from running it in local? When I try to run it gives a long ID, but when I check docker ps -a it says it exited. So my meteor app didn't run.. Hope someone will shed some light. Thank you!

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Hello,
Would be happy to help, however this is not the right forum for this kind of inquiry.
I would like to invite you to our docker-user mailing list, or #docker on IRC to discuss.
Thanks!


[spaceship] unicode hotfix

see https://github.com/fastlane/fastlane/issues/7229

This would also require to revert the revert commit, right?

@KrauseFx this PR was done before the revert, team is investigating how this could have worked in the past (as same babosa code was used in produce).


RScript.evalShaAsync prepends additional characters to the first argument. seems to be related to the default Codec

Set up:
public final CompletionStage<Long> foo(
        String sha,
        String key,
        long expirationSeconds,
        String... elements) {
    String[] keys = {key};
    String[] arguments = new String[elements.length + 1];
    arguments[0] = String.valueOf(expirationSeconds);
    System.arraycopy(elements, 0, arguments, 1, elements.length);

    // calls inside this function are simplified. please ignore redundant allocations / casts.
    Object[] vals = Arrays.copyOf(arguments, arguments.length, Object[].class);

    // RedissonClient client;
    return client
            .getScript()
            .evalShaAsync(RScript.Mode.READ_WRITE, sha, RScript.ReturnType.INTEGER, Arrays.asList(keys), vals);
}

where:

sha is the hash of a loaded script (via SCRIPT LOAD) command.
key is an alphanumeric string. e.g. "123456789"
expirationSeconds is 3600
elements is a single alphanumeric string. e.g. "foobar1234"

the loaded Lua script contains the following lines:
if not tonumber(ARGV[1]) then
  return {err="invalid expiration. ARGV[1]="..ARGV[1]}
end

in order to parse the expiration.
Expected behavior
the if check in LUA script should not trigger an error.
Actual behavior
script returned error and from the error message ARGV[1] is set to > 3600 with possible whitespace characters (stripped from console output)
Steps to reproduce or test case

load a script (statically or programmatically) contains the argument parse lines above
invoke the method above
observe the application output

Redis version
6.0.5 (AWS ElastiCache, cluster mode)
Redisson version
3.15.3
Redisson configuration
Config cfg = new Config();
RedisURI uri = new RedisURI("redis", "aws-ec-redis-cluster-endpoint", 6379);
cfg.useClusterServers().addNodeAddress(uri.toString());
cfg.setThreads(4);
cfg.setNettyThreads(8);
RedissonClient client = Redisson.create(cfg);

Note on the script:
the same script (without modification) worked with lettuce-core in the same application in a production environment (with thousands of calls per minute running 24/7) and has been proven stable.
for some reason the arguments received on Redis server (from redisson) contains extra characters for the first argument.
Update
I tested a non-default Codec:
cfg.setCodec(StringCodec.INSTANCE);
and the issue is addressed. Not sure if this is expected behavior for the default Codec

You need to use StringCodec for your case, since default codec is a binary codec.

thanks for getting back to me and confirming


[WFCORE-4403] Upgrade Undertow to 2.0.20.Final

Upstream Jira: https://issues.jboss.org/browse/WFCORE-4403
EAP Jira: https://issues.jboss.org/browse/JBEAP-16361

Release Notes - Undertow - Version 2.0.20.Final
        Enhancement

[UNDERTOW-1432] -         Add support for multiple routes in JSESSIONID to reverse proxy

[UNDERTOW-1519] -         DirectByteBufferDeallocator static block error at JDK11

[UNDERTOW-1520] -         Add methods returning the builder to FormParserFactory$Builder

        Bug

[UNDERTOW-1498] -         RequestLimit.exchangeEvent can skip a Exchange

[UNDERTOW-1506] -         Internal Server Error (500) when using directory-listing in FileHandler

[UNDERTOW-1521] -         XNIO IO worker 100% CPU during SSL handshake


Why do i have to run as root?

Hello,
in the example Jenkinsfile it says that i have to run the container as the root user? Why is that? That means it will create any reports, logs and similar as the root user aswell meaning the Jenkins-user (who isn't root) will be unable to remove the workspaces that contains files it may not touch.
    agent {
        docker {
            image 'katalonstudio/katalon'
            args "-u root"
        }
    }

By default, Jenkins will map its user ID to the container, which prevents Katalon Studio and browsers to work properly. You can chmod any files created after the tests so that Jenkins will have enough access permissions.


kernel-modules: inconsistent kernel versions: 5.15.z for 5.18.z

Steps To Reproduce
nixos-rebuild with linux-5.18 on current main branch commit 5a188dfbca95f5c13327a0606e407531329274e1
Last successful rebuild: 692517bf851f2d8d999f0ad50f53fa1d2dd5c8f9
Build log
building '/nix/store/6bp60654n0psn638alm3vjjpi2cy1034-kernel-modules.drv'...
building '/nix/store/4fwvc8fnhqkhxc64dz17imz2hag3x00p-linux-5.18.1-modules.drv'...
created 22 symlinks in user environment
inconsistent kernel versions: 5.15.44
5.18.1
error: builder for '/nix/store/6bp60654n0psn638alm3vjjpi2cy1034-kernel-modules.drv' failed with exit code 1
created 22 symlinks in user environment
inconsistent kernel versions: 5.15.44
5.18.1
error: builder for '/nix/store/4fwvc8fnhqkhxc64dz17imz2hag3x00p-linux-5.18.1-modules.drv' failed with exit code 1
error: 1 dependencies of derivation '/nix/store/wjhk8n0i5r5mgspzyjq0kgh2anmxhvgp-linux-5.18.1-modules-shrunk.drv' failed to build
error: 1 dependencies of derivation '/nix/store/m1yc0n7ng5f71k9rddx0bmxcgfy8g0jz-stage-1-init.sh.drv' failed to build
error: 1 dependencies of derivation '/nix/store/06ada1zx99c7wnx4lxp05xyx993dzf87-initrd-linux-5.18.1.drv' failed to build

Additional context
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.18.1, NixOS, 22.11 (Raccoon), 22.11.git.4cea7a03db4`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos, nixos-small"`
 - channels(fabian): `"home-manager, nixos-19.03-19.03, nixos-19.09-19.09, nixos-20.03-20.03, nixos-20.09-20.09, nixos-21.05-21.05, nixos-21.11-21.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I think the below linuxPackages.kernel uses 5.15 even if user has boot.kernelPackages = pkgs.linuxPackages_latest; in configuration.nix:
  new-lg4ff = callPackage ../os-specific/linux/new-lg4ff {
    inherit (linuxPackages) kernel;
  };

Also I think extraModulePackages / kernelModules should use enable option below. AFAIU it's defined, but not used: module always gets included into user's config:
  options.hardware.new-lg4ff = {
    enable = mkOption {
      type = types.bool;
      default = false;
      description = ''
        Enables improved Linux module drivers for Logitech driving wheels.
        This will replace the existing in-kernel hid-logitech modules.
        Works most notably on the Logitech G25, G27, G29 and Driving Force (GT).
      '';
    };
  };

  config = {
    boot = {
      extraModulePackages = [ pkgs.new-lg4ff ];
      kernelModules = [ "hid-logitech-new" ];
    };
  };

https://github.com/NixOS/nixpkgs/pull/175837 fixes unconditional part. Kernel selection would probably need to be addrssed as well.
/cc @legendofmiracles @MatthiasBenaets

Other modules are usually defined in pkgs/top-level/linux-kernels.nix. Example: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/hardware/video/webcam/facetimehd.nix#L35

Resolved by https://github.com/NixOS/nixpkgs/pull/175837

#175837 fixes unconditional part. Kernel selection would probably need to be addrssed as well.
/cc @legendofmiracles @MatthiasBenaets

I will address the kernel selection issue.


db.BlogPostsCounts is magic

You don't define where or how to setup db.BlogPostCounts after the query type is defined in OnModelCreating so it comes out of thin air with no way to know how you're supposed to define it. If you add a DbSet with the type BlogPostsCount you'll get an invalidOperationException saying that an entity type with the same name already exists. As a result this sample is not at all clear and cannot lead to success for the reader.

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

ID: ace630fc-7310-10b7-39bb-26fcfe3f3950
Version Independent ID: dbe96b32-0589-0f8c-6558-11a065d84dff
Content: Query Types - EF Core
Content Source: entity-framework/core/modeling/query-types.md
Product: entity-framework
GitHub Login: @anpete
Microsoft Alias: anpete

@JohnGalt1717 Thanks for reporting this--we will update the document. BlogPostCounts should be defined as a DbQuery property on the context:
public DbQuery<BlogPostCount> BlogPostCounts { get; set; }

It's defined in the linked complete sampel code on GitHub, but I've also been confused by how the article itself skips over this detail... :-(

I found the sample code in https://github.com/aspnet/EntityFrameworkCore/tree/master/samples/QueryTypes

I agree to the above posts about getting confused with not using the standard DbSet<> that I'm used to but instead using the DbQuery<> for this specialized situation. I read that the doc was supposed to get updated with this new information and that was over a month ago? I spent an hour trying to figure things out myself before deciding to come here for assistance only to luckily find that I'm not the only one who has run into this.
Before thinking that only a few people have had trouble with this... Image the amount of people who completely give up or don't waste the time to come here and find out there's missing info. Supposedly if a web page doesn't load in 3 second then you've already lost 50% of users. Apply that to how many people would spend 10 minutes trying to figure out the missing code after reading the docs thoroughly.
Just a thought.

@DanJ2010 it's actually stated in the doc:

Are mapped on the DbContext through properties of type DbQuery rather than DbSet

But @ajcvickers actually commented that the documentation will include the codeline too.

@janis-veinbergs Sorry but I was pretty frustrated. I didn't see anything about DbQuery and I've followed docs like that for a long time and scanned over them a few times and still didn't see anything. Not sure why I'm blind or it's so hard to find but in the end I didn't see it.

This should be added explicitly in the doc. Thanks for putting in here though.

Hi @divega 👋 - PRs #1592 & #1593 submitted 😊

Thank you @spottedmahn!


Add contributors section somewhere

A lot of people contributed to this before it was moved to GitHub! @chadsansing, @zee-moz, and all the reviewers they had!
We should collect all the names and thank them in the README or maybe a special file (humans.txt?)

PR #46

Still to do: let's collect all the reviewers of the original OLTS and add them somewhere!

Sure, send the names and I'll modify then PR. Still out with some of MozLon
so when I'm back I'll fix it.
On Thu, 17 Nov 2016, 23:08 Abigail Cabunoc Mayes<EMAIL_ADDRESS>wrote:

Still to do: let's collect all the reviewers of the original OLTS and add
them somewhere!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/open-leadership-training-series/issues/44#issuecomment-261399046,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCwjz-ssi1dPvziPX9wRCJf54lOIJBlks5q_N5bgaJpZM4K0XeS
.

Back briefly on my laptop. I've attempted to find the reviewers using the Google Doc edit history however I couldn't find enough time to follow through to finding the GH handles.

oh wow, thanks @MinnSoe! Names alone might be fine for now? I'm not sure if everyone has a github handle

Thanks for assembling this! David Bild, Tim Riches, Holly Robbins, and Robert Schaefer all served as readers and offered significant feedback that shaped the "final" content.

GitHub IDs (except for Tim Riches):

David Bild: @dbild
Tim Riches
Holly Robbins: @hvrobbins
Rob Schaefer: @schae234

@MinnSoe -- I'll let you make the PR?

@acabunoc looks like @betatim has done it. LGTM.


Publish to maven central

Hi Thomas,
Could you please publish the modules intended to be reused (embedded-keycloak-server-spring-boot-support, embedded-keycloak-server-spring-boot-starter and the spring-boot-starter-parent as far as I can see) to maven central so that they can be reused by third party projects without the need to build them from source?
The README could then need an update about how to use this in a consuming project, which I'd be glad to contribute :-) [edit: done, see #20]
Best regards,
Mike

+1, good feature that simplify CI deployments when we use starter fo third party projects

I'll try to find the time to set everything up to be released on maven central.
In the mean-time perhaps one can already consume the library via https://jitpack.io/

The following example based on jitpack works quite well already:
https://gist.github.com/thomasdarimont/7f9fb18c5176f6364fbbb13f5e7cd214

Ok, wow, I didn't know that service before.
I can confirm: it works well.


How to train

Hello, I want to ask how I can train my own data set now. I don't see the training code

See examples/imagenet/main.py

Do we need to always normalize using the Imagenet mean/std values (normalize = transforms.Normalize(mean=[0.485, 0.456, 0.406], std=[0.229, 0.224, 0.225])) when we are using an Imagenet pretrained Efficientnet model from here for training on our different dataset? I have read that the pretrained models available in Pytorch needs this normalization as a preprocessing step (https://discuss.pytorch.org/t/how-to-preprocess-input-for-pre-trained-networks/683).


Couldn't stop click event of a disabled button from bubbling

Description
Three disabled buttons shown in the demo page.
button0 is plain HTML/JS, no click handler triggered after click
button1 is a lit-element, handler triggered after click, seems click event is bubbling.
button2 is a lit-element, i called e.stopPropagation in the inner handler, still cannot stop it from bubbling.
Live Demo
https://stackblitz.com/edit/hy4yqu
Steps to Reproduce
Click button1 and button2
Expected Results
No alert
Actual Results
Alert shown
Browsers Affected

Chrome
Safari 11

Versions

lit-element: v2.1.0
webcomponents: v2.2.10

@mgenware in the case of my-button-v2.ts you are getting very close to what will be needed here, however to stop propagation to listeners on the same element you need to use stopImmediatePropagation() a la https://stackblitz.com/edit/hy4yqu-d6jkx9?file=my-button-v2.ts
Here's the MDN docs on that API https://developer.mozilla.org/en-US/docs/Web/API/Event/stopImmediatePropagation
Good luck!

Wow it works! thanks @Westbrook

Sounds like there's not a LitElement bug here. Closing; please re-open if there is anything outstanding.


Techlahoma Organizer CoC

we're collaborating on making a code of conduct to set expectations for user group leaders and additionally include how to treat and maintain meeting spaces, like Prototek. will update this thurs or sun night too with beginning rough draft.

:boom: in its own repo: https://github.com/techlahoma/CodeofConduct


Check f-contiguous when ndarray have shape like (1, k, 1, 1)

If ndarray is f-contiguous, it should be converted to Array. So, check it when it's both f-contiguous and c-contiguous
  julia> a = np.random.random(py_cast(Py, (1,3,1,1)))
  julia> py_coerce(AbstractArray, a)
  1×3×1×1 Array{Float64, 4}:
  ...

Codecov Report

Merging #37 (83ffc04) into main (6ced228) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   61.92%   61.92%
=======================================
  Files          11       11
  Lines         885      885
=======================================
  Hits          548      548
  Misses        337      337

Impacted Files
Coverage Δ

TyPython/src/CPython.NumPy.jl
92.77% <100.00%> (ø)

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


feat: add option to disable CL

Add a feature to disable the Consensus Layer (CL)/Beacon node, useful for scenarios where only the Execution Layer (EL) client is needed.
Use Case:
Supporting Aura consensus from Nethermind with an EL client only.

I think this PR could be made a lot more simple. You probably can do the checks if the CL image is set to null, and check for that.
If CL img is set to null, then don't spin it up, else do.

Could you add the aura config into examples/aura.yaml instead of network config?

Please run kurtosis lint --format in order to have no linting errors.

@gr8h Thanks for the contribution - some notes

Can we enrich the README about this behavior
While cl_client_image being null does work; this feels like magical behavior which is worse when things aren't documented. How about cl_client_type being set to null? @barnabasbusa does it have to be the image?

While cl_client_image being null does work; this feels like magical behavior which is worse when things aren't documented. How about cl_client_type being set to null? @barnabasbusa does it have to be the image?

Actually cl_client_type being null is probably better, the image being set to null is not logical indeed.

While cl_client_image being null does work; this feels like magical behavior which is worse when things aren't documented. How about cl_client_type being set to null? @barnabasbusa does it have to be the image?

Actually cl_client_type being null is probably better, the image being set to null is not logical indeed.

Okay, Make sense.

@barnabasbusa one thing i noticed is that the el_extra_params value in the yaml file does not override the values in nethermind_****.start file, it only extend it and it duplicate the params, is this an intended behaviour?

is this an intended behaviour?

yes it is.
As its supposed to be extra arguments to a sane default, which would mean that the operation is append, not a find and replace.

Will nethermind be able to take the later values?

Will nethermind be able to take the later values?

I had to change chainspecs path in the constants.star, and removed "--config=none.cfg" on nethermind launch script.


Needed: Implement system test for orphaned subscriptions

See #2080 and #2248
Assigned to Takashi so he can help us get in touch with a service owner that can say "give it up already" or help us keep the test from going flaky.

Hello! :-)
First, a mea culpa; I have not done as good of a job at keeping up with issues as I should have. If you are getting this (I admit it) cut and paste, it is likely because your issue sat for too long.
In this case, I have been in the process of making a radical update to the PubSub library (#3637) to add significant performance improvements and a new surface, which we hope to launch soon. As such, I am clearing out issues on the old library. It is my sincere goal to do a better job of being on top of issues in the future.
As the preceding paragraph implies, I am closing this issue. If the revamped library does not solve your issue, however, please feel free to reopen.
Thanks!


SMARTCONTRACT: Define contract storage

Define contract storage.
Which data fields it should have.
    #[storage]
    struct Storage {
        ...
    }

I will like to do this. I will define define all necessary variables.

Cool. Assigned to you.


Added 2D paint feature

Hi All,
I've modified the paintOverlayPoint function to handle painting of individual slices.
Kindly review the code and let me know if there are any changes to be made.
Thank you,
Barry

I'd rather have it as a key+mouse event while in paint mode.     Otherwise
the user will need to toggle thru multiple mouse modes to get back to 3D
painting.
How about ctrl+mouse for 2D mode?
s
On Wed, Mar 30, 2022 at 3:27 PM Forrest Li @.***> wrote:

@.**** commented on this pull request.
Overall, LGTM. Just two comments right now.
@aylward https://github.com/aylward, is making the 2D paint mode a
separate click mode from the 3D paint mode okay with you?
In ImageViewer/ImageViewer.cxx
https://github.com/KitwareMedical/ImageViewer/pull/109#discussion_r838889174
:

@@ -462,13 +466,26 @@ int parseAndExecImageViewer(int argc, char* argv[])
for (int i = 0; i < workflow.size();) {
auto type = workflow[i++];

 if (type == "p" || type == "P") {

 if (type == "p2" || type == "P2") {

   auto name = workflow[i++];

   auto radiusStr = workflow[i++];

   auto labelStr = workflow[i++];

   std::unique_ptr<PaintStep2D> step(new PaintStep2D());

   step->radius = std::atoi(radiusStr.c_str());

   step->label = std::atoi(labelStr.c_str());

   step->name = name;

   steps.push_back(std::move(step));

 }

 else if (type == "p3" || type == "P3") {

If we make this change, we will need to update the annotation script. I'd
suggest we keep this a non-breaking change by defaulting "p/P" to be 3D.
In QtImageViewer/QtGlSliceView.h
https://github.com/KitwareMedical/ImageViewer/pull/109#discussion_r838890943
:

@@ -54,14 +54,15 @@ using namespace itk;

PAINT = Color the overlay
*/
const int NUM_ClickModeTypes = 5;
-typedef enum {CM_NOP, CM_SELECT, CM_CUSTOM, CM_PAINT, CM_RULER, CM_BOX} ClickModeType;
-const char ClickModeTypeName[6][7] =

{{'N', 'O', 'P', '\0', ' ', ' ', ' '},
{'S', 'e', 'l', 'e', 'c', 't', '\0'},
{'C', 'u', 's', 't', 'o', 'm', '\0'},
{'P', 'a', 'i', 'n', 't', '\0', ' '},
{'R', 'u', 'l', 'e', 'r', '\0', ' '},
{'B', 'o', 'x', '\0', ' ', ' ', ' '}};
+typedef enum {CM_NOP, CM_SELECT, CM_CUSTOM, CM_PAINT3D, CM_PAINT2D, CM_RULER, CM_BOX} ClickModeType;
+const char ClickModeTypeName[7][9] =

{{'N', 'O', 'P', '\0', ' ', ' ', ' ', ' ', ' '},
{'S', 'e', 'l', 'e', 'c', 't', '\0', ' ', ' '},
{'C', 'u', 's', 't', 'o', 'm', '\0', ' ', ' '},
{'P', 'a', 'i', 'n', 't','3', 'D', '\0', ' '},
{'P', 'a', 'i', 'n', 't','2','D', '\0', ' '},

I'm going to nit on the missing spaces in front of '2', '3', and 'D',
since I somehow saw it. :)
—
Reply to this email directly, view it on GitHub
https://github.com/KitwareMedical/ImageViewer/pull/109#pullrequestreview-926616060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACEJL336Z74UXSXWJG6GWLVCSTIPANCNFSM5SCUVTOA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Stephen R. Aylward, Ph.D.
Senior Director of Strategic Initiatives
Kitware: Delivering innovation.

Thank you for the feedback!
I will make the change to ctrl + mouse for 2D.

I'd rather have it as a key+mouse event while in paint mode. Otherwise the user will need to toggle thru multiple mouse modes to get back to 3D painting. How about ctrl+mouse for 2D mode? s
…
On Wed, Mar 30, 2022 at 3:27 PM Forrest Li @.> wrote: @.* commented on this pull request. Overall, LGTM. Just two comments right now. @aylward https://github.com/aylward, is making the 2D paint mode a separate click mode from the 3D paint mode okay with you? ------------------------------ In ImageViewer/ImageViewer.cxx <#109 (comment)> : > @@ -462,13 +466,26 @@ int parseAndExecImageViewer(int argc, char* argv[]) for (int i = 0; i < workflow.size();) { auto type = workflow[i++]; - if (type == "p" || type == "P") { + if (type == "p2" || type == "P2") { + + auto name = workflow[i++]; + auto radiusStr = workflow[i++]; + auto labelStr = workflow[i++]; + + std::unique_ptr step(new PaintStep2D()); + step->radius = std::atoi(radiusStr.c_str()); + step->label = std::atoi(labelStr.c_str()); + step->name = name; + + steps.push_back(std::move(step)); + } + else if (type == "p3" || type == "P3") { If we make this change, we will need to update the annotation script. I'd suggest we keep this a non-breaking change by defaulting "p/P" to be 3D. ------------------------------ In QtImageViewer/QtGlSliceView.h <#109 (comment)> : > @@ -54,14 +54,15 @@ using namespace itk; * PAINT = Color the overlay / const int NUM_ClickModeTypes = 5; -typedef enum {CM_NOP, CM_SELECT, CM_CUSTOM, CM_PAINT, CM_RULER, CM_BOX} ClickModeType; -const char ClickModeTypeName[6][7] = - {{'N', 'O', 'P', '\0', ' ', ' ', ' '}, - {'S', 'e', 'l', 'e', 'c', 't', '\0'}, - {'C', 'u', 's', 't', 'o', 'm', '\0'}, - {'P', 'a', 'i', 'n', 't', '\0', ' '}, - {'R', 'u', 'l', 'e', 'r', '\0', ' '}, - {'B', 'o', 'x', '\0', ' ', ' ', ' '}}; +typedef enum {CM_NOP, CM_SELECT, CM_CUSTOM, CM_PAINT3D, CM_PAINT2D, CM_RULER, CM_BOX} ClickModeType; +const char ClickModeTypeName[7][9] = + {{'N', 'O', 'P', '\0', ' ', ' ', ' ', ' ', ' '}, + {'S', 'e', 'l', 'e', 'c', 't', '\0', ' ', ' '}, + {'C', 'u', 's', 't', 'o', 'm', '\0', ' ', ' '}, + {'P', 'a', 'i', 'n', 't','3', 'D', '\0', ' '}, + {'P', 'a', 'i', 'n', 't','2','D', '\0', ' '}, I'm going to nit on the missing spaces in front of '2', '3', and 'D', since I somehow saw it. :) — Reply to this email directly, view it on GitHub <#109 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACEJL336Z74UXSXWJG6GWLVCSTIPANCNFSM5SCUVTOA . You are receiving this because you were mentioned.Message ID: @.**>
-- Stephen R. Aylward, Ph.D. Senior Director of Strategic Initiatives --- Kitware: Delivering innovation.

Hi Stephen,
Thanks for the feedback. I can make that change to ctrl + mouse click.
But what if the user wants to continuously annotate individual slides? Wouldn't that be better if it was a separate mode?
Let me know your thoughts on this.
Thank you,
Barry

You bring up a really good point!   What about having sub-modes?  That is, we have just one mouse mode for painting, and you use a key command to toggle 2D and 3D painting?
We already do this for the ruler - using "u" to switch between ONSD and normal ruler?    How about having u switch between 2D and 3D paint mode?
And thanks for continuing to think about this!!!

Yes. A toggle switch like that would work well!
I'll work on adding that switch.
On Wed, Mar 30, 2022 at 5:30 PM Stephen R. Aylward @.***>
wrote:

You bring up a really good point! What about having sub-modes? That is, we
have just one mouse mode for painting, and you use a key command to toggle
2D and 3D painting?
We already do this for the ruler - using "u" to switch between ONSD and
normal ruler? How about having u switch between 2D and 3D paint mode?
And thanks for continuing to think about this!!!
—
Reply to this email directly, view it on GitHub
https://github.com/KitwareMedical/ImageViewer/pull/109#issuecomment-1083644008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR7Q76CYIO6TE5BSKTP73LLVCTBYTANCNFSM5SCUVTOA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I was able to add the ctrl + u toggle switch to toggle between 2D and 3D
paint. But, I am having trouble finding a way to repaint the bottom right
text corresponding to the paint type.
The workflow information text: STEP  (PAINT2D), L:  remains pointing
to the one for PAINT2D even after switching to PAINT3D.
On Wed, Mar 30, 2022 at 6:18 PM Bharadwaj Ravichandran (Barry) <
@.***> wrote:

Yes. A toggle switch like that would work well!
I'll work on adding that switch.
On Wed, Mar 30, 2022 at 5:30 PM Stephen R. Aylward <
@.***> wrote:

You bring up a really good point! What about having sub-modes? That is,
we have just one mouse mode for painting, and you use a key command to
toggle 2D and 3D painting?
We already do this for the ruler - using "u" to switch between ONSD and
normal ruler? How about having u switch between 2D and 3D paint mode?
And thanks for continuing to think about this!!!
—
Reply to this email directly, view it on GitHub
https://github.com/KitwareMedical/ImageViewer/pull/109#issuecomment-1083644008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR7Q76CYIO6TE5BSKTP73LLVCTBYTANCNFSM5SCUVTOA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


ETAs

Allow for the querying of estimated arrival times

You may be interested in this collection of repos, specifically this repo.  Perhaps you could pull from their API that has done the heavy lifting or poke around to see how they get their information.

I've seen these repos around and they are amazing! I might add them to the readme as an additional resource for people who may be interested!
For the ETA, I would like to stay with the native PassioGo API. The native ETA request structure is https://passiogo.com/mapGetData.php?eta=3&deviceId=59804559&stopIds=cta1514. (Sorry for not including it in the issue in the first place!) It seems that it's not as straightforward as the other requests, but it seems parseable.
And thanks for your PR! I'll take a look at your work!

I don't know that this requires a new pull request, but I just realized that I incorrectly claimed that the min operation performed in Stop.getNextVehicle() would operate in O(1) time complexity. In reality, that method could simply return Stop.getEtas()[0], and therefore the entire method may not be required.
We may simply want to state in the docstring for Stop.getEtas() that etas are considered to be sorted upon creation.