net: l2: ieee802154: radio: inconsistent/non-standard definition of RX timestamp

Describe the bug
The Radio API defines the RX timestamp for IEEE 802.15.4 packets as "the first symbol of the MAC Header" (net_pkt.h, line 102).
But this point in time is not what the standard defines as RX timestamp (see IEEE 802.15.4-2020, section 8.3.4, table 8-90, Timestamp) - see details below in "Expected Behavior". Such a point in time cannot be used for synchronization purposes outside L2 without leaking implementation details of the PHY, see OpenThread workaround below as an example.
The two current drivers supporting RX timestamps implement this timestamp inconsistently:

nRF5 calculates a non-standard timestamp at the beginning of the MHR header internally from the end of the packet timestamp.
DW1000 uses the standard's well-defined synchronization anchor (end of SFD aka RMARKER)

The OpenThread platform API historically used the start of MAC as its timestamp (see @edmont's comment below) but has since turned to the standard definition (see include/openthread/platform/radio.h) "The value SHALL be the time when the SFD was received" (line 209f) and uses it in practice precisely as the standard would define it with a value of macSyncSymbolOffset == 0, see radio.c, line 189: "OpenThread expects the timestamp to point to the end of SFD".
The radio API is based on OpenThread's previous definition, which is not used anywhere anymore. A workaround in radio.c had to be implemented that assumes a specific PHY (which is ok for Open Thread due to its limited choice of PHYs but not for other protocols built above L2 with arbitrary PHYs). The workaround in radio.c can easily be fixed (see below) to improve PHY/MAC encapsulation of the OT stack plus turn to the standard definition for all drivers and other L2 stacks.
To Reproduce
n/a (architecture issue)
Expected behavior
All drivers and call sites should use a common standard conforming concept of RX timestamp.
The IEEE 802.15.4 standard (and OpenThread) define as time synchronization anchor for all PHYs (equivalently):

the end of the start-of-frame delimiter (SFD) field
the end of the synchronization header (SHR)
the start of the PHY header
the RMARKER (IEEE 802.15.4-2020, section 6.9.1)

This is to be measured at the exact moment when this signal is present at the local antenna. Signal propagation delays between the antenna and the timestamp counter must be corrected for by the hardware and/or firmware.
This unique synchronization anchor is then referred to everywhere in the standard (e.g. synchronization with beacons, RSSI measurements, timestamping of received and transmitted frames, synchronization of CCA, ranging, etc.).
More specifically: RX and TX timestamps are defined to be measured relative to the end of the SFD, offset by a hardware specific constant parameter macSyncSymbolOffset (section <IP_ADDRESS>, table 8-94) which L2 requests from the driver to calculate the exact synchronization point if required.
As the current radio API does not report the driver's macSyncSymbolOffset constant, the most obvious definition for any RX or TX timestamp would be the end of the SFD (i.e. drivers correct for their own offset as if their macSyncSymbolOffset was zero). This is the same definition that has been chosen by the OpenThread platform API.
The implementation of this for the RX timestamp is simple (and will be provided as PR by myself if the solution is accepted):

Move the nRF5/PHY-specific workaround in radio.c into the nRF5 driver where it belongs architecturally.
Change the documentation in net_pkt.h so that the timestamp is defined as in the standard assuming macSyncSymbolOffset == 0.

Alternatively macSyncSymbolOffset would have to be reported as a constant attribute by drivers implementing any of the *TIME* capabilities. As macSyncSymbolOffset does not have much use other than saving a few cycles on RX, I would go with the OpenThread solution for now to keep it simple.
TX timestamping must be analyzed separately and is not in scope for this bug report.
Impact
A common concept of timestamping on the RX and TX path is required for TSCH (and other timing sensitive features in the IEEE 802.15.4 standard, like CSL, ranging, etc.) no matter the PHY, band, channel, etc. chosen.
So fixing this is a pre-condition for TSCH implementation.
Logs and console output
n/a
Environment (please complete the following information):
n/a
Additional context
n/a

@fgrandel, thanks for the detailed description. https://github.com/zephyrproject-rtos/zephyr/pull/57013 was added more like a hotfix, since historically the MAC Header start was used as reference for CSL synchronization in Thread, but that changed later on. So I agree with your proposed changes to align to the end of SFD in every place.
CC: @jciupis

since historically the MAC Header start was used as reference for CSL synchronization in Thread

@edmont I didn't know that. That explains perfectly where you come from. Thank you for your fast response! Will provide a PR then.

Just one more bit of information supporting the solution proposed here: TX timestamp also de-facto points to the first symbol of SHR.

TX timestamp (net_pkt.txtime) is de-facto only used at a single place (timed TX in CSL context from OT stack to Nordic's radio driver).
Nordic's driver uses the "start of SHR" definition.

@hubertmis Two questions to you in this context:

Do you know why the nrfxlib driver stamps the last symbol on RX but uses start of SHR on TX?
When fixing this - are you interested in providing a re-usable nrf_802154_shr_timestamp_get() in the nrfxlib driver API (I can provide a PR against your github repo...) or should I simply move the MHR-to-SHR offset into the nrf5 zephyr driver wrapper?

Related discussion about timing and clocks (including but not only) in the radio protocol context: #19030

Do you know why the nrfxlib driver stamps the last symbol on RX but uses start of SHR on TX?

Yes. It's convenient to configure nRF RADIO to capture timestamps of the end of the received frame and begin transmission at the passed timestamp. To simplify the implementation the driver API exposes these HW properties.

When fixing this - are you interested in providing a re-usable nrf_802154_shr_timestamp_get() in the nrfxlib driver API

I think that's the correct way. The nrf_802154 module should expose functions to recalculate the timestamps it passes through API calls to the format expected by the standard (start of SHR) for both TX and RX paths. Or even considering how the standard defines the frame timestamps, maybe existing functions could be modified to recalculate to the expected values internally.
@jciupis, do you think these features could be delivered as a part of nrf_802154?

do you think these features could be delivered as a part of nrf_802154?

Yes, that's the plan. I'd like to provide an API for the nRF driver to conveniently calculate RX timestamp aligned with @fgrandel's suggestion, pointing to the end of SHR. I'd prefer that over offset conversions in the nRF5 Zephyr driver.
Once we agree on a common definition of TX timestamp different than the current one, which in my understanding is out of scope of this discussion, I'll provide alignments for nRF5 driver as well.

@hubertmis @jciupis Thanks for responding so quickly. It's nice that we can fix this upstream in nrfxlib. :-)
So what approach do you prefer? I see two possible solutions:

You want to keep the current timing semantics for ...rx_at() and tx_at() for backwards compat and provide conversion methods so that we can offset those timestamps in the Zephyr driver. In this case you could provide the required changes in nrfxlib and once we have that merged and the west SHA pointer is updated in Zephyr's mainline, I can fix the rest of it in a separate PR.

You want to change the timing semantics for ...rx_at() and ...tx_at() to the standard definition: In this case updating the driver could be done in the background without updating the west SHA pointer. I could then provide a PR for this ticket plus updating the SHA pointer myself in the same change.

Does this work? Is there maybe another way that I'm overlooking?

@edmont Quick question to you - we now discovered (see posts above) that while the RX side is offset internally in the OT stack to point to "start of PHR", the timestamps calculated with this as a base time for CSL timing on the TX side seem to refer to "start of SHR" if I'm not mistaken.
It seems to me then, that currently OT sends packets systematically a little bit too late (exactly the number of symbols in the SHR) as the timing in the standard refers to the symbol boundary in IEEE 802.15.4-2020, section 6.5.2, which is "start of PHR". Do you agree that this might be a timing bug in OT still (which would be automatically fixed by the changes proposed here for the TX path of course)?

Hi @fgrandel, OT is sending the frames at the exact timing since the SHR symbols time is being subtracted here: https://github.com/openthread/openthread/blob/main/src/core/thread/csl_tx_scheduler.cpp#L162
However, the transmission time base time is not properly documented (https://github.com/openthread/openthread/blob/main/include/openthread/platform/radio.h#L276) with regards to the exact moment in the frame it refers to. So it's a good opportunity to do so. We would need to synchronize the changes.

@edmont Nice! Thanks for responding so quickly and of course you're right. So from my side I'd be available to also move that TX workaround to the nRF driver plus propose improved doc on the OT and radio API side, as @edmont proposes. I think this will not only standardize the definition to something obvious but also simplify the OT side a bit.

@fgrandel the approach I'd like to suggest to handle the changes for nRF driver is:

PR to hal_nordic with new API that converts RX/TX timestamps from the format supported by the existing nRF driver API to the format agreed on above (RMARKER). The existing APIs are left unchanged for backwards compatibility. I'll handle this PR.
PR to Zephyr that integrates changes in hal_nordic through a west manifest update. I can handle this one too.

In parallel, in no particular order (although some level of synchronization is going to be necessary):

changes in OT to unify the interpretation of TX timestamp with the IEEE 802.15.4-2020 spec, both doc and implementation
alignments in Zephyr's radio API for timed TX
changes in Zephyr's nRF driver that utilize the new API to expose a unified timestamp interpretation to its users. Can be combined with PR mentioned in point 2.

If I understood you correctly, you're available to take care of these three points. Are there any missing pieces?

If I understood you correctly, you're available to take care of these three points.

@jciupis Correct, I can do the PRs in this area against the OT (docs) and Zephyr repos.

Are there any missing pieces?

No, not that I know of.

@fgrandel assigning this to you since you proposed opening the PRs for this yourself in an earlier comment.

@edmont @jciupis I have tried to make the necessary changes in Zephyr and OpenThread and integrate them with the work @jciupis has done on the NRF side.
OpenThread: https://github.com/fgrandel/openthread/commits/feat/standards-based-timing Zephyr: https://github.com/fgrandel/zephyr/commits/feat/net-unified-timestamp
~Unfortunately I do not currently have the hardware and precise measurement tools to measure timings~ nor could I compile the low-level driver changes as the timing stuff requires the proprietary version of the SL. Therefore there must be bugs in there - I probably mixed up something while shuffling timing offsets back and forth.
@edmont It would be great if you could have a close look and maybe do some measurements on your side to check what's still wrong. Is there a way we can collaborate directly over Discord/WhatsApp or similar, so that I can provide fast fixes in case you find bugs? I'd be available for pairing if that helps.
UPDATE: Got some DevKits now - so just need some help in setting these up (maybe with some kind of sniffer) to do regression tests myself.

I managed to verify the changes in the two branches linked here with nRF5 hardware and I can confirm that they are correctly setting the CSL transmission time.


Inline hints should allow to set a max chars number for hint

An option to choose the max inline hints chars should be available in the config (probably with a default value).
Reason:

Oh, I think it was the old Inline parameters plugin.. (https://github.com/imliam/vscode-inline-parameters)


Regression: Custom node inheriting BaseButton does not appear in Create New Node dialog

Godot version
v3.4.4.stable.official [419e713a2]
System information
Ubuntu 2022.04, GeForce GTX 765M
Issue description
Regression of issue #8034.
Last comment says it's fixed in 3.1 but custom nodes extending abstract classes like BaseButton are still / again missing.
Steps to reproduce
Follow either the steps from the original report or the custom node example with a a script extending BaseButton instead of Button.
Minimal reproduction project
see original report.

The original issue was technically never fixed, but class_name doesn't allow extending abstract classes in 3.x if I'm not mistaken. It will in 4.0 though.

Thanks for the clarification.
Is there any way at all to create a custom button extending BaseButton?
It doesn't need to be a custom node.. at least for my use cases.

Is there any way at all to create a custom button extending BaseButton?

In 3.x at least, not that I know of. You can extend Button and use a StyleBoxEmpty to disable its background still.

Just as an update, it seems as of 4.0, extending a BaseButton still isn't possible. It would be pretty cool if we could have this someday. Up until now, the only way I've ever been able to have a button that is something different than a picture, a link, or exactly one label, is to rewrite the logic (focused, toggle mode, and themes) manually on a custom node extending from MarginContainer.


Added Symbol element

Added some mathematical symbols and a .png of html file opened in the browser

Good work @adlyloukil  :+1:
Thanks for contributing.
Hoping more collaboration in future. :smile:

Thanks! I look forward to possible future collaboration with you 👍


Orchad CMS - Dead lock & Time out issues

We are using Orchard CMS 1.7.2. we recently getting many dead locks, timeouts and assertion failure issue.
Below are my observations:
1.dead lock and time outs are mainly happening at the below query: select userrolesp0_.Id as Id26_, userrolesp0_.UserId as UserId26_, userrolesp0_.Role_id as Role3_26_ from Orchard_Roles_UserRolesPartRecord userrolesp0_ where userrolesp0_userrolesp0_.UserId=@p0
2.The Orchard_Roles_UserRolesPartRecord table is having index on USerId column even though no change in number of timeouts and dead locks
3.Along with this table, dead locks/timeouts/concurrency has been noticed on the following tables: Orchard_Framework_ContentItemRecord and Orchard_Framework_ContentItemVersionRecord (Note : These tables also having indexes)
4.N-hibernate is not able to fetch records when table having more than 1+ million records (in Orchard_Framework_ContentItemVersionRecord) and throwing assertion failure
Because of this, we are getting many complaints from client/end users saying that they are not able to use the site well.
We are not sure how to control/fix them as these are coming from orchard framework level. As of now, We are looking at database isolation level. Some of the orchard articles recommending to change the database isolation level to snapshot (but no concrete justification on this)
Please share your thoughts which are helpful for to us to resolve these issues at the earliest.
Regards,
Kamal

To fix some potential dead locks I would suggest you upgrade to a more recent version which has many fixes regarding data access.
About loading 1 million records with NHibernate the only solution is to simply not do that. I can't imagine a scenario where you'd have to load 1 million content items in memory, or you probably need another solution.

Thank for your suggestions.
The application has to fetch few records from table having 1+ million records.  At this point, most of the times, N-hibernate is throwing error as Assertion failure.

Please provide some repro steps, there is nothing we are aware of that could link to this issue. Also please check that you are not reading too many values, and you have correct indexes in your database.


Adding latest versions of Chromedriver

Adding some of the later versions of Chromedriver to the repository file. Let me know if any other changes are needed.

@andidev can you check these changes?

Thanks for the update. The repository you have just upgraded is though not the one used by this plugin (just the testing of the plugin). The repository used by the plugin in found at https://github.com/webdriverextensions/webdriverextensions-maven-plugin-repository/blob/master/repository-3.0.json
I have added the latest chromedriver 2.25.0 in PR #3 just now (should be available now).
The reson why the test fails is that for some reason in this repo is that the phantomjs driver is not possible to download too many times (though I'm not sure why exactly).

Got it, thanks for the update @andidev


Create pull request feature

Is your feature request related to a problem? Please describe.
It would be nice to have ability to create pull request in repo by opening tab in browser.
Describe the solution you'd like
Press some key in Neogit which will open page with pull request opening in browser.
Additional context
It works in lazygit by pressing o btn.
Super useful.

At this time I have every intention of keeping this agnostic to which git-site you are using. I think something like https://github.com/chrishrb/gx.nvim or the new vim.ui.open() api would better be suited for this.

@CKolkey may you provide an example please?

Haven't got one, sorry. But off the top of my head you could do something like
vim.ui.open = function()
  -- if the filetype is `NeogitStatus`, send the PR URI to get opened
end


BI-1710 - Add support for searching by externalReference to /search/trials endpoint in BreedBase

Description
Implemented the ability to specify externalReferenceId(s) and externalReferenceSource(s) for the GET /trials and POST /search/trials endpoints.
Testing

Create a new trial with external references

make a call to the GET /trials endpoint, and specify externalReferenceId as a query param

Verify that only the trial with that externalReferenceId is returned

Create a new trial with external references

make a call to the POST /search/trials endpoint, and specify externalReferenceIds as a parameter in the body

make a call to the GET /search/trials/{searchId} endpoint

Verify that only the trial with that externalReferenceId is in the result set

Checklist

[ ] Refactoring only
[ ] Documentation only
[ ] Fixture update only
[ ] Bug fix

[ ] The relevant issue has been closed.
[ ] Further work is required.

[x] New feature

[ ] Relevant tests have been created and run.
[ ] Data was added to the fixture

[ ] Data was added via a patch in /t/data/fixture/patches/.

[ ] User-Facing Change

[ ] The user manual in /docs has been updated.

[ ] Any new Perl has been documented using perldoc.
[ ] Any new JavaScript has been documented using JSDoc.
[ ] Any new legacy JavaScript has been moved from /js to /js/source/legacy.

Passed Developer test


go 1.22 now requires go 1.20.6

go 1.22+ now requires go 1.20.6 to compile so it's reflected in readme

the script requires go always, even compiling 1.20.x
so, what is the purpose of gvm if it requires system go?


Move all logic into GameInstance for persistence between levels

Alternatively find a way to utilise saving and loading, but I don't see that working well with rollback.

I found a solution using saving and loading.


wss?

Hi, does this support wss, I only want to use secured.

Please read the docs. They are written for something (for instance, to not have to answer basic questions here).


Skip tasks with excess registrations when syncing

Explicitly handle a rare scenario during sync in which a task has more registrations than it should have according to its application configuration.
This may happen when application configuration is changed (now defining less registrations) and then the deployment this caused is rolled back – the new application configuration still stays in Marathon. Or we end up in some weird state. Since we don't know what to do in such scenario – we don't know which service to deregister – I suggest skipping such task with a log message.
We could also deregister all registrations and register the task again, but this seemed like a bad idea to me – the deployment with this new configuration was rolled back after all!

Coverage increased (+0.05%) to 79.206% when pulling a9b522cbe44ef522faf8780aac0844a07059a75a on skip_tasks_with_excess_registrations_when_syncing into 0e2e5e2c066aed4bfee0eaddfb4929a2eeeda25c on develop.

Coverage increased (+0.05%) to 79.206% when pulling d8262cdc49f447ae414f04222aa6998e9f0bd89d on skip_tasks_with_excess_registrations_when_syncing into 0e2e5e2c066aed4bfee0eaddfb4929a2eeeda25c on develop.

Coverage increased (+0.05%) to 79.206% when pulling 173fea4d4c7a333dee58322b2b31b97f7e110f9b on skip_tasks_with_excess_registrations_when_syncing into 0e2e5e2c066aed4bfee0eaddfb4929a2eeeda25c on develop.


MIGRATE command syntax error?

Can anyone give me a hand understanding what's causing the syntax error: Current version of redli, on a Ubuntu 18.04 box running Redis, trying to migrate the keys from this source redis to DO managed Redis.
root@rogue-01:~# redli -a pasword
Connected to 5.0.5
> MIGRATE test-managed-redis-do-user-1250700-0.db.ondigitalocean.com 25061 "" 0 5000 AUTH managed-password KEYS
(error) ERR syntax error
>

Solved: I wasn't passing the list of keys at the end.


Fix PopupContextMenu in StatusBarPopover

Changes
When PopupContextMenu is used as a nested popup in StatusBarPopover it is problematic because the logic of iTwinUI (Popover) and AppUI (Popup) clash since they portal to different places which then causes various issues (e.g. the nested Popup displays behind the main Popover or opening the nested Popup would close the Popover). As a simple fix I used portalTarget prop which will portal the Popup into the Popover (more specifically, the trigger button which is inside the Popover).
Additionally, PopupContextMenu will be deprecated in AppUI 4.16.0 and can be easily replaced with DropdownMenu or DropdownButton from iTwinUI.
Fixes #909.
Testing
Tested in standalone test-app.

I tested this with other Popup components and even when it is not used in another popup. Everything seems to be working the same as before.


ci: extend timeout for v6 workflow

To verify major upgrade in workflow, it needs
more package build time.

e.g.
https://github.com/fluent/fluent-package-builder/actions/runs/11571106651
This workflow run was canceled.


Annotation-based models for Java

Is it possible for models to understand annotations? For example, I would like to have assertions on the parameters of a method with a particular example. I had a look at the models defined as part of infer and none of them use annotations.
If this is not currently possible, how difficult would it be to implement this? Can anyone point me to where I can get started?

Hi @vikhyat. It is in theory possible to use annotations with Infer but so far we did not need it. Do you have a concrete example in mind where using annotations in the models would be good ?

One example would be Spring's @Autowired annotation, fields with that annotation should be treated as constructor-initialized but currently the Eradicate checker complains that they need to be marked as @Nullable since they are not initialized in the constructor.
The actual use-case for which I posted this question is that I have some methods with @Validated annotations, for which there are a number of constraints I want to apply to the arguments.

Adding support for @Autowired is going to be easy: we already support @Inject with the same effect of not requiring fields to be initialized, so it's just a matter of recognizing one more annotation.
For @Validated, it's hard to sat without knowing what are the constraints and what effect they should have.

My use-case is mostly not null constraints on a bean that is passed to the method that are not enforced globally, it is quite similar to Hibernate Validator. The annotation is similar to how it is done in Dropwizard except that it is applied to the method rather than the argument.
One option I have could be to exclude the beans from Infer and provide an auto-generated model in it's place, but the problem with that is that it will be used for all occurrences of the bean rather than just the places where it is validated.


fix(urllib3): fixing when headers are args (not kwargs)

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


Changed all label formats to match the original.

This PR fixes issue #655. The new labels look like this:

This is how the new labels look.

Alright, I duplicated these changes to the labels in labelfield.svelte, I think it's fair to mention that the selected labels have an altered look if I change them to strictly ACM colors, not sure how you guys feel about this look. Should I revert it back to the original colors?

Let me know.

Should I revert it back to the original colors?

Thanks for following up, @DavidJSolano! I think we should go back to the original colors.

Okay, this is what the final product looks like after reverting the selected labels to their original colors and making them compatible with light/dark mode.


Added a Custom Cursor Effect

Pull Request
Sucessfully added a custom cursor to improve User interaction
Description of Changes
-> Used Recursive pattern of Circles for attractive cursor design .
-> It makes website more attractive and User-Interactive.
Type of Pull Request

[ ] Bug Fix
[X] New Feature
[X] Performance Improvement
[ ] Documentation Update
[ ] Code Refactoring
[ ] Security Enhancement
[ ] Other (Specify): _______________

Screenshots / Demo
https://github.com/user-attachments/assets/756626f2-87c4-4f7a-8168-6567f5fc5306

[X] I have reviewed my code before submission.
[X] I have read and followed the project's Contribution Guidelines.
[X] I have tested the code thoroughly.
[X] I have attached related issue numbers, screenshots, or videos (if applicable).
[X] I have added comments to my code where necessary for clarity.
[X] I have checked the existing code base and my changes do not introduce any breaking changes.
[X] My PR addresses only one issue; I have created separate PRs for other issues.
[X] I have ensured that the PR does not include unrelated files.
[X] I am mindful of the project's coding standards and best practices.

Additional Information
[Include any additional context or relevant information that may help in reviewing this PR.]
Contribution Guidelines Reminder
Thank you for contributing to our project! Please ensure you follow the guidelines below:
Contribution Levels

Task Categories:

Basic Tasks: Primarily focused on small changes like minor bug fixes, documentation improvements, and unit testing. Labeled as Easy.
Intermediate Tasks: Moderate-level tasks that involve enhancing existing features, code refactoring, or performance improvements. Labeled as Medium.
Advanced Tasks: Complex tasks like building new features, significant refactoring, or critical bug fixes. Labeled as Hard.

Acknowledging Effort:

Exceptional contributions will be recognized with a higher contribution level if substantial effort and innovation are demonstrated.

Review Process

PRs will be reviewed by the maintainers, and feedback will be provided. Contributors should address feedback promptly and resubmit the PR if necessary.

Contributors are encouraged to participate in the review process of other PRs for community growth.

We appreciate your time and effort in improving this project and look forward to your valuable contributions!

good @AnujShrivastava01

good @AnujShrivastava01

Thankyou !
Now please assign me labels in pull request also for points evaluation and then merge the pull request .

Please check the level bagde I was assigned level2 for the issue .
Please correct it .
Label Screenshot

Please check the level bagde I was assigned level2 for the issue . Please change it .
Label Screenshot

bro its just normal change added cursor .
although i will assign you level 2 later on it will be assigned as level 1 if they found.....

Ok Bro, Thankyou for your  support and quick reply.

Ok Bro, Thankyou for your support and quick reply.

sure bro tell me your tech stacks speciality, so i will tell you issues if you want

Ok Bro, Thankyou for your support and quick reply.

sure bro tell me your tech stacks speciality, so i will tell you issues if you want

Yes Bro sure , I have just started my Open Source journey and learning day by day.
Currently I have my skills in Enhancing Frontend Using HTML , CSS and JS.
Its great to contribute in your project and work with you.
I have also connected you on Linkedin .


Make use of v8 snapshots on Capitano build step to improve startup time

Related to https://github.com/resin-io/capitano/issues/46

Closing stale issues


inventory: update feature flags based upon usage in inventoryagent.go

What does this PR do?
Update the feature flags documented in README.md based upon how they are set by inventoryagent.go.

Motivation
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Would be nice to have a script generating this README from the definition of the inventory (not asking you to do it :) )

Hmmm, interesting idea, I'll think on it


ECM-607

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/ECM-607
Change description
Added code so batchUpdated judgments with hearings are set as non hearings for updated cases
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Why no unit test?

Created


Add migrate command to convert JSON rules to Bolt language

This should be merged. Are you abandoning this project?


Implement default CoAP request handler

More info in #1662

Interesting failure. This should add 8 bytes per server, if there are 3 server instances (netif, app, dtls), thats 24 bytes. I would say that configuration is razor edge close to undefined behavior due to stack overflow.

Yes, we're running pretty low on RAM for the existing Travis build configuration for CC2538.  I'll take a look to see what we can do there.  Regarding the stack size issue, are you noticing that the allocated stack region is not large enough for current needs?

I am unfamiliar with OpenThread on CC2538, so that was opinionated observation. As maximum stack size is often hard to predict, I tend to use conservative rule of thumb by leaving some gap between the stack and static variables. I guess it's not too different from allocating enough extra stack space.

Can you try rebasing this on master to see if the build checks pass?  Alternatively, we wait until #1745 is merged since that will free up a handful of memory.

Let's wait till #1745

#1745 has been merged.  Feel free to reimplement this feature using the combined CoAP implementation.


Frameit producing incorrect image size for iTunesConnect

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
I used frameit silver to create screenshots with frame however it produced incorrect image sizes for iTunesConnect.
Complete output when running fastlane, including the stack trace and command used
Andis-MacBook-Pro:Home Report Final setiyadi$ frameit silver
[14:36:21]: Added frame: '/Users/setiyadi/Desktop/Home Report Final/fastlane/screenshots/en-US/iPhone6-0ForSale_framed.png'
[14:36:21]: Added frame: '/Users/setiyadi/Desktop/Home Report Final/fastlane/screenshots/en-US/iPhone6-1Sold_framed.png'
[14:36:22]: Added frame: '/Users/setiyadi/Desktop/Home Report Final/fastlane/screenshots/en-US/iPhone6-2Filter_framed.png'
[14:36:22]: Added frame: '/Users/setiyadi/Desktop/Home Report Final/fastlane/screenshots/en-US/iPhone6-3Summary_framed.png'
Andis-MacBook-Pro:Home Report Final setiyadi$ deliver
[14:36:39]: Login to iTunes Connect<EMAIL_ADDRESS>[14:36:42]: Login successful

+--------------------+---------------------------------------+
|                   deliver 1.11.2 Summary                   |
+--------------------+---------------------------------------+
| app_identifier     | com.pfi.coredata.fastlane.Home-Report |
| username           |<EMAIL_ADDRESS>                 |
| screenshots_path   | ./fastlane/screenshots                |
| metadata_path      | ./fastlane/metadata                   |
| app_version        | 1.0                                   |
| ipa                | Home Report.ipa                       |
| skip_binary_upload | false                                 |
| skip_screenshots   | false                                 |
| skip_metadata      | false                                 |
| force              | false                                 |
| submit_for_review  | false                                 |
| automatic_release  | false                                 |
+--------------------+---------------------------------------+

[14:36:42]: Making sure the latest version on iTunes Connect matches '1.0' from the ipa file...
[14:36:43]: '1.0' is the latest version on iTunes Connect
[14:36:43]: Framed screenshots are detected! 🖼 Non-framed screenshot files may be skipped. 🏃
[14:36:43]: 🏃 Skipping screenshot file: ./fastlane/screenshots/en-US/iPhone6-0ForSale.png

[!] Unsupported screen size [613, 1255] for path './fastlane/screenshots/en-US/iPhone6-0ForSale_framed.png'
Andis-MacBook-Pro:Home Report Final setiyadi$

Configuration Files
Please copy the complete content of your Fastfile and any other configuration files you use below:
Fastfile:
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md
# can also be listed using the `fastlane actions` command

# Change the syntax highlighting to Ruby
# All lines starting with a # are ignored when running `fastlane`

# If you want to automatically update fastlane if a new version is available:
# update_fastlane

# This is the minimum version number required.
# Update this, if you use features of a newer version
fastlane_version "1.83.0"

default_platform :ios

platform :ios do
  before_all do
    # ENV["SLACK_URL"] = "https://hooks.slack.com/services/..."

  end

  desc "Runs all the tests"
  lane :test do
    scan
  end

  desc "Submit a new Beta Build to Apple TestFlight"
  desc "This will also make sure the profile is up to date"
  lane :beta do
    # match(type: "appstore") # more information: https://codesigning.guide
    cert
    sigh
    gym(scheme: "Home Report") # Build your app - more options available
    pilot

    # sh "your_script.sh"
    # You can also use other beta testing services here (run `fastlane actions`)
  end

  desc "Deploy a new version to the App Store"
  lane :appstore do
    # match(type: "appstore")
    # snapshot
    gym(scheme: "Home Report") # Build your app - more options available
    deliver(force: true)
    # frameit
  end

  # You can define as many lanes as you want

  after_all do |lane|
    # This block is called, only if the executed lane was successful

    # slack(
    #   message: "Successfully deployed new App Update."
    # )
  end

  error do |lane, exception|
    # slack(
    #   message: exception.message,
    #   success: false
    # )
  end
end

# More information about multiple platforms in fastlane: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Platforms.md
# All available actions: https://github.com/fastlane/fastlane/blob/master/fastlane/docs/Actions.md

# fastlane reports which actions are used
# No personal data is recorded. Learn more at https://github.com/fastlane/enhancer

Environment
fastlane version (run fastlane -v):
1.8.3
Do you use bundler to execute fastlane (i.e. bundle exec fastlane)?
No
Do you use a Ruby environment manager (e.g. chruby, rbenv, rvm)?
No

@asetiyadi thanks for this report! I tired and was unable to reproduce, could you post your Framefile.json?

Hi Sam,
Not sure if you can receive a .zip attachment, attached is the FastlaneTest.zip that consists of fastlane and UITest folder.
It will include all the configuration files.
Regards
Andi

On May 6, 2016, at 5:37 AM, Sam Phillips<EMAIL_ADDRESS>wrote:
@asetiyadi https://github.com/asetiyadi thanks for this report! I tired and was unable to reproduce, could you post your Framefile.json?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub https://github.com/fastlane/fastlane/issues/4508#issuecomment-217417853

@asetiyadi hrm, no, zip attachments don't seem to come through on Github.

Hi Sam,
Upon entering my config file, I found some discrepancies on how I named my UI test and the config in title.strings.  I set them straight and that's it, now I got the title, the frame and the size correctly.  Previously since the title not showing up, the frame size got messed up and producing incorrect size.  All good now.  Thank you for your superb response on this issue.
Regards,
Andi

@asetiyadi great to hear!  Glad you're back up and running 👍


docs(index.js): modified the copy of the index.js

I don't know what you mean, would these messages be suggestions to change?
Em ter., 18 de out. de 2022 às 15:28, Parth Jadhav @.***>
escreveu:

@.**** commented on this pull request.
Left some comments @cristian-sbardelotto
https://github.com/cristian-sbardelotto
—
Reply to this email directly, view it on GitHub
https://github.com/leonardomso/33-js-concepts/pull/291#pullrequestreview-1146325242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A22GZQJ6OSY5M2IZSVZSWITWD3T43ANCNFSM6AAAAAARGOLD2E
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes @cristian-sbardelotto , Sorry for incomplete information. They are meant to be suggestions. Feel free to modify/reject them.

Sure, I'll add all the suggestions, your way is better
Em qua., 19 de out. de 2022 às 16:00, Parth Jadhav @.***>
escreveu:

Yes @cristian-sbardelotto https://github.com/cristian-sbardelotto ,
Sorry for incomplete information. They are meant to be suggestions. Feel
free to modify/reject them.
—
Reply to this email directly, view it on GitHub
https://github.com/leonardomso/33-js-concepts/pull/291#issuecomment-1284446000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A22GZQPLPPQVWDXJTQANOPTWEBAMDANCNFSM6AAAAAARGOLD2E
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the help!
Em qua., 19 de out. de 2022 às 18:30, Cristian Keiber Sbardelotto <
@.***> escreveu:

Sure, I'll add all the suggestions, your way is better
Em qua., 19 de out. de 2022 às 16:00, Parth Jadhav <
@.***> escreveu:

Yes @cristian-sbardelotto https://github.com/cristian-sbardelotto ,
Sorry for incomplete information. They are meant to be suggestions. Feel
free to modify/reject them.
—
Reply to this email directly, view it on GitHub
https://github.com/leonardomso/33-js-concepts/pull/291#issuecomment-1284446000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A22GZQPLPPQVWDXJTQANOPTWEBAMDANCNFSM6AAAAAARGOLD2E
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you for your help
Em qua., 19 de out. de 2022 às 18:30, Cristian Keiber Sbardelotto <
@.***> escreveu:

Sure, I'll add all the suggestions, your way is better
Em qua., 19 de out. de 2022 às 16:00, Parth Jadhav <
@.***> escreveu:

Yes @cristian-sbardelotto https://github.com/cristian-sbardelotto ,
Sorry for incomplete information. They are meant to be suggestions. Feel
free to modify/reject them.
—
Reply to this email directly, view it on GitHub
https://github.com/leonardomso/33-js-concepts/pull/291#issuecomment-1284446000,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A22GZQPLPPQVWDXJTQANOPTWEBAMDANCNFSM6AAAAAARGOLD2E
.
You are receiving this because you were mentioned.Message ID:
@.***>


Restart agent pods after upgrade to 1.22

When a cluster is created with version 1.18 or lower, various
controllers will start a watch for the extensions/v1beta1/Ingress type.
Without this patch, f that cluster is then upgraded to 1.22, those
watches will start to fail because that API is no longer available, and
the controllers using those watches will stop syncing because they don't
see events for the new Ingress type, networking.k8s.io/v1/Ingress.
This patch adds a check to the cluster controller that updates the
cluster version status, and forces the clusterdeploy controller to
restart the cluster agents. Simply forcing the agent to be reapplied
does not necessarily fix the problem because if there is no change to
the cluster agent deployment then the pods won't actually be restarted.
The version checks for an original version of 1.21, rather than 1.18.
This is because it's possible that a cluster was created with version
1.18 or below, then upgraded to version 1.21 or below, where the old
extensions/v1beta1 API would still be available and the controllers
would still work. But then the final upgrade to 1.22 would break the
controlers. So, assume that any upgrade to 1.22 needs the agents
restarted.
https://github.com/rancher/rancher/issues/35766

Actually my idea won't work, since the controllers that need to be restarted are all in the agent, and the clustermanager is not available in the agent nor are there any cluster controllers running in the agent that can detect when the cluster version has changed, so I think this solution is probably best.

@rmweir came up with the idea of annotating the deployment template which automatically causes the pods to restart and doesn't rely on the resource version number, so updated this PR to do that.


Fix typos in configuration

I found another couple of typos in the configuration. I hope it’s okay to open pull requests for such nitpicky changes :slightly_smiling_face:.

Codecov Report

Merging #5398 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5398      +/-   ##
==========================================
- Coverage   37.47%   37.45%   -0.02%
==========================================
  Files         312      312
  Lines       46509    46509
==========================================
- Hits        17427    17419       -8
- Misses      26594    26599       +5
- Partials     2488     2491       +3

Impacted Files
Coverage Δ

models/repo_indexer.go
44.49% <0%> (-3.39%)
:arrow_down:

Continue to review full report at Codecov.

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


Tweakers

Added support for Dutch IT site Tweakers.net
Example: https://tweakers.net/video/9926/new-nintendo-3ds-xl-op-alle-fronten-beter.html
I don't know about the _VALID_URL regex, and I could done the URL parsing cleaner by using regex instead of split.

I commited some changes, but there are still things a bit unclear for me.
Downloading the regular page: https://tweakers.net/video/9926/new-nintendo-3ds-xl-op-alle-fronten-beter.html, with:
player_page = self._download_webpage(url, video_id)

Thanks, merged.


check if name exists before create/pull/copy

TODO

[x] tests

This is waiting on https://github.com/ollama/ollama/pull/3718 which I'll merge after the next release is out so it can bake


Added Windows gamepad support

Gamepad support for Hazel
This pull request contains the basic implementation of a gamepad input system. It only supports regular gamepads, so it isn't possible to connect a guitar hero controller for example.
Why?
Having an interface for gamepad input will be useful if Hazel ever gets ported to console. It's also very important for an engine to support gamepad input, as it is one of the most popular input methods in the industry.
How does it work?
Additional functions have been added to the input class to poll button state, trigger state, joystick state and vibration state.
Some additional enum values have been added to map ABXY buttons to the Dualshock's square, cross, rectangle and triangle button.
The input system handles deadzone values as well.
You may also notice that there is three new functions in the input class: Init, SetEventCallback and Update.
Init is used to initialise input polling libraries (XInput on Windows for example)
SetEventCallback is used so that the input system can send GamepadConnected and GamepadDisconnected events.
Update is used to actually update the state of the gamepads.

Because GLFW doesn't support gamepad vibration.
Why is gamepad connect and disconnect an event? Can't you just add a Input::IsGamepadConnected function?
No, because gamepad connection and disconnection should be handled as notifications : not just a plain old boolean state
Without GLFW, it's not cross-platform, dude...
LOL as if Hazel was cross-platform KEKW (on a serious note, when Hazel will be able to build on both Mac and Linux, I'll add
the necessary backends for Apple's GameController framework and Linux joystick file descriptor)

What could be added

Swap A & B buttons if the controller is a Nintendo Switch gamepad
Get the battery state of the gamepad (XInputGetBatteryInformation)

Nice.  I wasn't aware that glfw doesn't support gamepad vibration

Have you implemented this for mac and linux already?

Have you implemented this for mac and linux already?

Already adressed in the QNA:

LOL as if Hazel was cross-platform KEKW (on a serious note, when Hazel will be able to build on both Mac and Linux, I'll add
the necessary backends for Apple's GameController framework and Linux joystick file descriptor)

Any specific reason for moving the EventCallbackFn?

I think the OP chose to move it so it was easier to use: static void SetEventCallback(const EventCallbackFn& callback);
inside of Input.h and virtual void SetEventCallback(const EventCallbackFn& callback) = 0; inside of window.h

Correct. It's just so that the input system can also access the EventCallbackFn type. No reason not to move it for clarity.


Odata querying

This pr should add odata querying to the google app engine version of the application.
To make better use of this querying capability we will probably need to improve our data import process so that we pull things in as data instead of it all just being strings. That'll be easier to query.

👍


Linux desktop app idle cpu usage is 45%

Details
When our app is compiled to for Linux it never uses less than 45% CPU utilization.  The web-app seems to be worse where chrome/brave threads sit at around 50%.  The web-app percentages are with release builds, the Linux desktop does drop a few percent with a release build, but never below 40%.  I ran the CPU profiler on the Linux app and _drawFrame was taking over 70% of the recorded time.  I could not find a single reference to any of our app code in the displayed profiler output.
I have put log statements in various widget's build functions to see if to see if any of our higher level widget are drawing more than they should, I cannot find the source of the background utilization in our code.
On macos the background utilization is 18% which while better is still significant.
It does seem like we are doing something to trigger such high CPU utilization but so far the profiler has not been helpful at indicating what might be causing the issue.
I have attached the exported data from the profiler.  The data was collected at the default medium sample rate.  I also tried another run at high but the results were similar.
Looking at the raw data I did find 2 traces that had our code.  We have a background timer that refreshes authentication info every 2 minutes.  We have other timer that refreshes some other info about every 10 seconds, but during the sample run that data would not cause any UI updates.
I did run the profiler on android and there was almost nothing, with _RawReceivePort._handleMessage and _startMicrotaksLoop being the top (and only) two routines shown in the Call Tree display.
The profile was captured about 30 seconds after app startup when all of our startup code should be done, the app was just sitting there, not occluded by any other window and the window never received pointer focus.
I did try to see if I could build the app with the master channel but there were two many incompatible dependencies. If it will be useful I can try to see if they can be resolved but they got only worse after my first pass updating versions.
I have attached the exported profiler data for the linux app.
**Target Platform: linux, web, macos **
Target OS version/browser: linux Fedora 37, macos 13.2.1, chrome Version 112.0.5615.165 (Official Build) (64-bit), brave [Version 1.50.125 Chromium: 112.0.5615.165 (Official Build) (64-bit)]
Devices:
Logs

Logs
 csh  …  lochbox  dart analyze                                                                                  master
Analyzing lochbox...                   10.0s
No issues found!
 csh  …  lochbox  cd ../core/                                                                                  master
 csh  …  core  !dart                                                                                  master
dart analyze
Analyzing core...                      2.7s
No issues found!

dart_devtools_2023-04-23_05_36_42.672.json.gz
[✓] Flutter (Channel stable, 3.7.12, on Fedora Linux 37 (Thirty Seven) 6.2.11-200.fc37.x86_64, locale en_US.UTF-8)
    • Flutter version 3.7.12 on channel stable at /server/home/csh/lib/flutter3
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 4d9e56e694 (5 days ago), 2023-04-17 21:47:46 -0400
    • Engine revision 1a65d409c7
    • Dart version 2.19.6
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /home/csh/Android/Sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /server/home/csh/lib/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 15.0.7 (Fedora 15.0.7-2.fc37)
    • cmake version 3.26.3
    • ninja version 1.10.2
    • pkg-config version 1.8.0

[✓] Android Studio (version 2022.1)
    • Android Studio at /server/home/csh/lib/android-studio
    • Flutter plugin version 72.0.2
    • Dart plugin version 221.6096
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.77.3)
    • VS Code at /usr/share/code
    • Flutter extension version 3.62.0

[✓] Connected device (5 available)
    • sdk gphone64 x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
    • sdk gphone64 x86 64 (mobile) • emulator-5556 • android-x64    • Android 12 (API 32) (emulator)
    • sdk gphone64 x86 64 (mobile) • emulator-5558 • android-x64    • Android 12 (API 31) (emulator)
    • Linux (desktop)              • linux         • linux-x64      • Fedora Linux 37 (Thirty Seven) 6.2.11-200.fc37.x86_64
    • Chrome (web)                 • chrome        • web-javascript • Google Chrome 112.0.5615.165

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

I forgot to mention the baseline CPU utilization is being measured with top/bpytop.

Hi @mrcsh
I think it's not easy to figure out where the issue is coming from without a reproduction sample.
It can be caused by 3rd party plugins or wrong implementation in the code. Or it could be an existing animation issue at https://github.com/flutter/flutter/issues/101591 or be related to the general issue at https://github.com/flutter/flutter /issues/88996.
I would recommend narrowing down the issue by isolating the project's modules and finding the most relevant screen/widget then providing more information about the issue that would be helpful for investigation. Thank you!

Thanks, I think I will try to get additional perf info using some of the tools in the other related issues, and see where that leads.  Unless there is an animation that is always running (with no visible effect) I'm not sure the animation issue is relevant.


[Bug]: vllm slows down after a long run

Your current environment
Collecting environment information...
/data/miniconda3_new/envs/vllm/lib/python3.10/site-packages/transformers/utils/hub.py:124: FutureWarning: Using `TRANSFORMERS_CACHE` is deprecated and will be removed in v5 of Transformers. Use `HF_HOME` instead.
  warnings.warn(
PyTorch version: 2.1.2+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.2 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.28.0
Libc version: glibc-2.35

Python version: 3.10.0 | packaged by conda-forge | (default, Nov 20 2021, 02:24:10) [GCC 9.4.0] (64-bit runtime)
Python platform: Linux-6.5.0-21-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA GeForce RTX 3090
GPU 1: NVIDIA GeForce RTX 3090
GPU 2: NVIDIA GeForce RTX 3090
GPU 3: NVIDIA GeForce RTX 3090

Nvidia driver version: 525.147.05
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
架构：                              x86_64
CPU 运行模式：                      32-bit, 64-bit
Address sizes:                      46 bits physical, 48 bits virtual
字节序：                            Little Endian
CPU:                                64
在线 CPU 列表：                     0-63
厂商 ID：                           GenuineIntel
型号名称：                          Intel(R) Xeon(R) Gold 5218 CPU @ 2.30GHz
CPU 系列：                          6
型号：                              85
每个核的线程数：                    2
每个座的核数：                      16
座：                                2
步进：                              7
CPU 最大 MHz：                      3900.0000
CPU 最小 MHz：                      1000.0000
BogoMIPS：                          4600.00
标记：                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cdp_l3 invpcid_single intel_ppin ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm mpx rdt_a avx512f avx512dq rdseed adx smap clflushopt clwb intel_pt avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local dtherm ida arat pln pts vnmi pku ospke avx512_vnni md_clear flush_l1d arch_capabilities
虚拟化：                            VT-x
L1d 缓存：                          1 MiB (32 instances)
L1i 缓存：                          1 MiB (32 instances)
L2 缓存：                           32 MiB (32 instances)
L3 缓存：                           44 MiB (2 instances)
NUMA 节点：                         2
NUMA 节点0 CPU：                    0-15,32-47
NUMA 节点1 CPU：                    16-31,48-63
Vulnerability Gather data sampling: Mitigation; Microcode
Vulnerability Itlb multihit:        KVM: Mitigation: VMX disabled
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:             Mitigation; Enhanced IBRS
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced / Automatic IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Mitigation; TSX disabled

Versions of relevant libraries:
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.24.4
[pip3] onnx==1.15.0
[pip3] onnxruntime==1.15.1
[pip3] rapidocr-onnxruntime==1.3.8
[pip3] torch==2.1.2
[pip3] torchaudio==2.1.2
[pip3] torchvision==0.16.2
[pip3] triton==2.1.0
[conda] numpy                     1.24.4                   pypi_0    pypi
[conda] torch                     2.1.2                    pypi_0    pypi
[conda] torchaudio                2.1.2                    pypi_0    pypi
[conda] torchvision               0.16.2                   pypi_0    pypi
[conda] triton                    2.1.0                    pypi_0    pypiROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.2.6
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	GPU1	GPU2	GPU3	NIC0	NIC1	CPU Affinity	NUMA Affinity
GPU0	 X 	PIX	NODE	NODE	NODE	NODE	0-15,32-47	0
GPU1	PIX	 X 	NODE	NODE	NODE	NODE	0-15,32-47	0
GPU2	NODE	NODE	 X 	PIX	NODE	NODE	0-15,32-47	0
GPU3	NODE	NODE	PIX	 X 	NODE	NODE	0-15,32-47	0
NIC0	NODE	NODE	NODE	NODE	 X 	PIX
NIC1	NODE	NODE	NODE	NODE	PIX	 X

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

NIC Legend:

  NIC0: mlx5_0
  NIC1: mlx5_1

🐛 Describe the bug
vllm slows down after a long run about 12 hour
at the begin :

after 12 hour:

then i reboot vllm: speed recovery, but i loss some data

It seems to have something to do with gpu kv cache 100%

may need to find a way to limit the length of the GPU kv cache, which will cause a long kv retrieval time if it is too long.

可以加下--enforce-eager试下

可以加下--enforce-eager试下

这个模式更快吗？

你好，请教一下，这个问题怎么解决的？


Checksumming

Do we need to provide a checksum for data? Or trust that the transmission strategy handles this?

Nah, it's fine


unsupported operators, including deformable_conv

Problem Description:
I have finetuned detection db_r50_td_tr model and have exported to inference model as well. now i want to convert it into onnx format. while converting im facing this issue
More Information:
(vp_env2) PS D:\hawkingradiation\paddleocr-repo> paddle2onnx --model_dir ./inference/det_db_r50_td_tr_inference_augment_complete/ --model_filename inference.pdmodel --params_filename inference.pdiparams --save_file ./inference/det_onnx/ppocr_v4_model.onnx --opset_version 10  --enable_onnx_checker True
>>
[Paddle2ONNX] Start to parse PaddlePaddle model...
[Paddle2ONNX] Model file path: ./inference/det_db_r50_td_tr_inference_augment_complete/inference.pdmodel
[Paddle2ONNX] Paramters file path: ./inference/det_db_r50_td_tr_inference_augment_complete/inference.pdiparams
[Paddle2ONNX] Start to parsing Paddle model...
[Paddle2ONNX] Oops, there are some operators not supported yet, including deformable_conv,
[ERROR] Due to the unsupported operators, the conversion is aborted.

Inference engine used for deployment:
Why conversion to ONNX format is needed: conversion is needed so that to check if the inference time reduces on CPU, open to any suggestions here
Paddle2ONNX version: 1.0.6
paddleocr Version: <IP_ADDRESS>
onnx version: 1.13.1
onnxruntime version: 1.9.0
open to any suggestions or how to solve this
ThankYou

升级下 Paddle2ONNX 就可以转了，但是 ONNXRuntime 应该是不支持这个算子的，需要考虑运行框架。


Miner silk touch stack creation

The miner seems to call Block#getMetaFromState directly instead of invoking Block#createStackedBlock when creating ItemStacks for silk touched blocks.
The vanilla silk touch logic will also invoke Block#createStackedBlock, as can be seen in Block#harvestBlock.
The problem is that this can set a non-zero metavalue to the ItemStack, even if this is not intended by the block.
This causes issues like CyclopsMC/EvilCraft#515, and possibly also for other blocks like vanilla Redstone when it is activated.

Fixed, thanks


Source code archives outdated

Description
The qgis-latest-ltr.tar.bz2 and qgis-latest.tar.bz2 source code archives contain the source code of QGIS 3.34.6 and QGIS 3.36.2 instead of QGIS 3.34.7 and QGIS 3.36.3, i.e. they are the same as qgis-3.34.6.tar.bz2 and qgis-3.36.2.tar.bz2 instead of qgis-3.34.7.tar.bz2 and qgis-3.36.3.tar.bz2.
Gentle ping @jef-n.

Page URL:
https://qgis.org/en/site/forusers/download.html -> SOURCES tab
https://qgis.org/downloads/?C=M;O=D
https://download.qgis.org/downloads/?C=M;O=D

qgis-latest-ltr.tar.bz2 and qgis-latest.tar.bz2 are still outdated.


Pin pvxs to earlier version

When pandablocks-ioc uses softioc version 4.5.0, the following import error occurs:
.venv) [twj43146@pc0139 PandABlocks-ioc-fork]$ pytest
filename="modules/database/src/ioc/dbStatic/dbLexRoutines.c" line number=276dbRead opening file pvxsIoc.dbd
ImportError while loading conftest '/scratch/twj43146/Programming/PandABlocks-ioc-fork/tests/conftest.py'.
tests/conftest.py:3: in <module>
    from fixtures.mocked_panda import *
tests/fixtures/mocked_panda.py:38: in <module>
    from softioc import builder
.venv/lib/python3.10/site-packages/softioc/__init__.py:25: in <module>
    dbLoadDatabase('pvxsIoc.dbd', pvxslibs.path.dbd_path, None)
.venv/lib/python3.10/site-packages/softioc/imports.py:45: in expect_success
    assert status == 0, 'Expected success'
E   AssertionError: Expected success

This is caused by the latest release of pvxs.
Pinning  "pvxslibs==1.2.4" fixes this problem.
@AlexanderWells-diamond, am I good to do this?

yes

No longer necessary - see #146 .


Dark Mode

@RamakrushnaBiswal fixed conflicts and updated navbar

Summary by CodeRabbit
Release Notes

New Features

Introduced BackToTopButton and Preloader components for improved navigation and loading experience.
Added a ThemeToggle component for easy switching between light and dark modes.
Implemented dark mode support across various components, enhancing visual consistency.

Styling Enhancements

Updated multiple components to support dark mode, including Membership, About, Boardgame, Event, Login, Menu, FeedbackForm, Landing, and more.
Adjusted background and text colors for better visibility and aesthetics in dark mode.

Chores

Added CSS variables and styles for dynamic theming in the global stylesheet.
Configured Tailwind CSS for dark mode functionality.

@jainaryan04 there are some conflicts
and dont use normal css use tailwind css @jainaryan04

converted to tailwind @RamakrushnaBiswal
I have fixed the conflicts now but if you merge any pr before mine there will be conflicts again. please fix them if they arise now because otherwise, we will be going in circles.

@jainaryan04

here the cards are not visible

fixed @RamakrushnaBiswal

@jainaryan04

cahnge it

@RamakrushnaBiswal fixed

@jainaryan04 resolve the conflicts

It won't help
@RamakrushnaBiswal
I resolved them twice before. If you merge any pr till i resolve them, there will be conflicts again
Can you please resolve them ?


Rename the Henson annotation or the Henson generated class

Both have the same name and are confusing users.

Duplicate of #67


Provider view - # of patients: 0

When using the provider view in Pophealth, the number of patients is always zero (and the bar charts always remain gray, even if the measure calculates to be something other than 0/0).
This is caused by the following code block in provider.rb:
       Record.by_provider(self, effective_date)
end```

There is no by_provider method defined on record, only on PatientCache.
Replacing the Record with PatientCache makes the number of patients show, but only when measures have been calculated (which isn't good either).

this was an issue with null start and end dates on provider performances.  I added in a check for null dates to pass them through.  The assumption is that null dates indicate that the provider performance is always valid.

Hi Andre, where is this fixed?
I've pulled in the latest master and there the number of patients is still zero.


Basic dockerization

This application is rather simple to containerize, so I went ahead and done it, as per issue #3
The image is not necessarily slim, and can probably be reduced, but it suits the project's needs
Current configuration
Dockerfile

The docker image used is node:19-bullseye
it copies the repository files into /app and installs the dependencies into the image
The entrypoint is set to npm run
The default command is dev, resulting in npm run dev

Once you build the image and push it to Docker Hub, users will not have to build the image or even clone the repository, just pull it from your repository!

Docker Compose
When using Docker Compose, I have separated the app into three alternate services:

base: The base service contains the image and build instructions to run the app

The image is currently called ghcr.io/deanayalon/free-admin-dashboard-fork, see image name

standalone: Maps port 5173 to the host machine, if not used with a proxy - Not secure, but valid for local use
dev: Mounts the project's files into the container, to reflect changes in real time, for simpler development

Use
docker compose up -d standalone # Run the app locally
docker compose up -d dev        # Run the app in development mode (project mount)

When using a reverse-proxy, simply use the base service.

Image Name
The image is currently named ghcr.io/deanayalon/free-admin-dashboard-fork, it is pushed to my GitHub account. You can pull it and use the app immediately without ever cloning the repository by doing:
docker run -p 5173:5173 ghcr.io/deanayalon/free-admin-dashboard-fork

You can use ghcr.io/Kuzma02/Free-Admin-Dashboard as the image name, and it will autoamtically be linked with your GitHub repository!
Package.json
Modified npm run dev to execute vite --host <IP_ADDRESS>, so the container listens on all addresses

I updated the image's default command to be dev:docker
Alternatively, the package.json does not necessarily require dev:docker to be defined, as the command can be changed to npm run dev -- --host <IP_ADDRESS> instead
This can be done using CMD ["dev", "--", "--host", "<IP_ADDRESS>"] in the dockerfile

The compose file you have merged still uses ghcr.io/deanayalon/free-admin-dashboard-fork as the image
services:
  base:
    image: ghcr.io/Kuzma02/Free-Admin-Dashboard

This should probably be changed to ghcr.io/Kuzma02/Free-Admin-Dashboard so that it can be updated and controlled in your repository, rather than mine
Besides installing Docker, to update the image in the future, all you need to do is
docker compose build base # Rebuild the image with the new files
docker compose push base  # Pushes the image to GitHub Container Registry

To be able to push it, you'll need to authenticate your Docker CLI tool with ghcr.io (For which you will need to create a Personal Access Token)
docker login ghcr.io

I will not be updating the image myself as you continue developing this dashboard
You could also create a GitHub Action that automatically updates the image when a new commit is pushed to this repository

Thank you, Dean.


Update sbt-pgp to 2.0.1

Updates com.jsuereth:sbt-pgp from 2.0.0 to 2.0.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

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

labels: semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master     #356   +/-   ##
=======================================
  Coverage   95.54%   95.54%
=======================================
  Files          15       15
  Lines         764      764
  Branches       45       45
=======================================
  Hits          730      730
  Misses         34       34

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 aeb19a2...d35a7ac. Read the comment docs.


Draft to ease PRAGMA usage (#273 and #265)

First, the Sql is not exported (outside of rusqlite) so no risk for abuse.
Another approach would have been to bind the sqlite3_mprintf but variadic functions are not supported on Rust side.
The most important point is to prevent SQL injection. As PRAGMA statements do not support parameters, it seems wise to help users calling any PRAGMA instead of letting them generate SQL manually.

First, the Sql struct is not exported (outside of rusqlite) so no risk for abuse.

Ah, yeah, I missed that the pragma mod itself wasn't public.

Another approach would have been to bind the sqlite3_mprintf but variadic functions are not supported on Rust side.

We could work around that with some C code potentially, but I'm not sure sqlite3_mprintf would help in this particular case. It looks like it adds %q/%Q/%w/%z, but it sounds like all of those are for properly quoting/escaping SQL strings, which wouldn't work for PRAGMAs.

The most important point is to prevent SQL injection. As PRAGMA statements do not support parameters, it seems wise to help users calling any PRAGMA instead of letting them generate SQL manually.

Agreed! I was suggesting we offer an even more restricted API than this PR adds. Instead of
db.pragma_update_and_check(None, "journal_mode", &"OFF", |row| row.get_checked(0))

for example, we could offer something enum based (untested, but something like):
db.pragma_update_and_check(None, Pragma::JournalMode(JournalMode::Off), |row| row.get_checked(0))

or with more explicit functions (again untested but something like):
let journal_mode = db.set_pragma_journal_mode(None, JournalMode::Off)?;

This would build a bigger API surface (enum cases or functions for each pragma we wanted to support), but would avoid us having to sanity-check a user-provided identifier before building up the pragma string. Would probably be simpler/easier to use, too, not that either is very complicated.

To query the JournalMode without updating it, we will need another enum, no ?
And for PRAGMA which can have an optional parameter, like integrity_check, a ``Pragma::IntegrityCheck(Some(200))` or two different entries ?

Yeah, that's definitely the downside to something more specific. Do you think the route this PR takes is a big win over what users can already do (like you showed on 265)? SQL injection shouldn't be a huge worry with PRAGMAs, since they shouldn't be things that take user input. Maybe we just need to include some examples of pragmas that do and don't return values (like you did on #273) in the docs?

Why is there any problem with PRAGMA ?
Because they don't support variable/parameter.
It is probably the first/big surprise for devops.
How do we simulate parameter for PRAGMA with no restricted input (PRAGMA with restricted input will also take advantage of this) ?
If we prevent using PRAGMA with user input,
Then, how do we expose a service to generate an ERD for a limited set of tables ?
Or how do we expose an API similar to JDBC ?
Or Diesel. Are you sure that the table name will never be boom'-- ?

SQL injection shouldn't be a huge worry with PRAGMAs, since they shouldn't be things that take user input

PRAGMA key, when using the sqlcipher feature conceivably could for some users.


Getting control over report pages using Query Chashing

Hello, I learned from your support page (https://docs.microsoft.com/en-us/power-bi/power-bi-query-caching) that only the landing page of a report is cashed. In a given case of a more complex manegement report in my enterprise enviornment, it would also be great as report / capacity admin to gain control over which specific report pages are to be cashed (and which not). As I learned from this comment here (https://github.com/MicrosoftDocs/powerbi-docs/issues/1124) there should'nt be much effect on the capcity doing this, right? And even though it would, as capacity owner the transparency about the respective memnory consumption (e.g. in the Capacity Metric App) would be than helpful to manage this function as capacity admin adequately.

Thanks for your comment, @schuffa. This looks like product feedback, rather than documentation feedback. You can submit product feedback or feature ideas by going to Power BI Ideas. You can also engage with the Power BI engineering team by being active in the Power BI Community.


FIX: Don't eagerly import bnb for LoftQ

Resolves #1665.
We accidentally added code in loftq_utils.py that eagerly imports bnb, which we want to avoid to prevent CUDA from being initialized too early.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

As I explained in this comment, it is unfortunately not easily possible to write a test for this.


SnapToken is required for permission check

	cr, err := client.Permission.Check(context.Background(), &v1.PermissionCheckRequest{
		TenantId: "t1",
		Metadata: &v1.PermissionCheckRequestMetadata{
			SnapToken: "", // rr --> relationship write response
			//SchemaVersion: schemaVersion, // sr --> schema write response
			Depth: 50,
		},
		Entity:     &v1.Entity{Type: "doc", Id: "1"},
		Permission: "read",
		Subject:    &v1.Subject{Type: "user", Id: "1"},
	})
	fmt.Println("???", cr, err)

???  rpc error: code = Internal desc = ERROR: function max(xid8) does not exist (SQLSTATE 42883)
postresql 14
Document said SnapToken is not required: https://docs.permify.co/docs/api-overview/permission/check-api
Golang users must write a relation before do any permission check, am i the only one got this error? or am i is the only one golang & pg14 user?

Solved: https://discord.com/channels/950799928047833088/1165341696209469451
Tested with v0.5.5-9e39bb8, it fixed.


double click / dbclick show

HTML:
superuser

Js:
username.on('hidden', function (e, reason) {
    username.editable('destroy');
});

username.dblclick(function (e) {
    e.stopPropagation();
    username.editable('show');
});

This works for me.
However, why x-editable not offer double click / dbclick feature ?

Unfortunately, due to this project being supported by volunteers we cannot provide user support at this time. Please try a site like Stack Overflow:  http://stackoverflow.com/questions/tagged/x-editable
Thank you.


Add pytest-randomly seedsetting functionto Makefile

I have made things!
Hey there. This is my first pull request for an open source project. :)
Checklist

[x] I have read contributing guidelines

[x] I'm sure that I did not unrelated changes in this pull request

[ ] I have created at least one test case for the changes I have made

[x] I have added myself to the CONTRIBUTORS.rst

Related issues

Closes #468

Ideas
I've already described my ideas in my first draft in the issues thread.
But what it boils down to:
Give the option to specify the --randomly-seed option from the Makefile to make easier use of this feature.
I have updated the contributing guide to introduce the feature.

@tabassco Hi! Thank you for your interest! Can you please add it to the travis.yml file to make sure that this solution really works? Thank you!

@lk-geimfari Thanks for your reply. I'm not quite sure where I'd merge it as it is only a change to the Makefile that would be run locally and not the travis.yml file. As pytest-randomly is already in the dependencies in poetry it will be used to shuffle the tests everytime without any changes.
If you could point me in the right direction I'll do so right away.

@tabassco Are you saying that it (random shuffling) works on Travis already? Sorry, I'm not quite sure that I understood you correctly.

@lk-geimfari Yes. Exactly.
Taking a look at the job protocol from my run:
============================= test session starts ==============================
platform linux -- Python 3.7.1, pytest-5.4.2, py-1.8.1, pluggy-0.13.1
Using --randomly-seed=1589317930
rootdir: /home/travis/build/lk-geimfari/mimesis, inifile: setup.cfg
plugins: isort-1.0.0, cov-2.8.1, flake8-1.0.5, randomly-3.3.1, mock-3.1.0
collected 4477 items

You can get the randomly-seed from there and plug it into your local setup using the Makefile changes I made. Then you will get the same ordering for the tests to do debugging.

@tabassco Thank you!


Update Overview.md

This change adds the best link for this whole category. It links to the main GitHub repo for scenario samples.
I often find myself going in loops, as the azure.microsoft.com pages also do not link well, to the GitHub repo.
Note the left hand menu, resources, samples, links to here, which proves fruitless and frustrating.

@ProjectPete : Thanks for your contribution! The author, @SteveMSFT, has been notified to review your proposed change.

Thanks. #sign-off


Improve error handling

How to handle exceptions in a webservice, for example when reading XLSX files?

Addressed via logging and failing in the libroclass branch.


Problem with multiple ui-select and ui-disable-choice

When using the ui-disable-choice attribute, the plugin will add the _uiSelectChoiceDisabled property directly to the option model to store if the option is disabled or not. So, when two ui-select share the same list of object, options will be disabled in both ui-select. This is not desirable because a different ui-disable-choice attribute might be defined for different ui-select that share the same data.
A hack would be to clone the options object. But is it possible to have a better fix than that?
Is it possible that this attribute be _uiSelectChoiceDisabled-{{UNIQUE ID OF UI-SELECT}}, that would permit to disable the option only in the given ui-select?
Check this http://plnkr.co/edit/WD8eEo?p=info
For a given Transaction, there is an Account and a Category defined.
Both Account and Category options are issued from the same list. An Account is a Category and vice-versa.
The Category ui-select-choice tag has the ui-disable-choice attribute set to a function that disables the Category if this Category has the same id property as the Account id property.

Select "Category 1" as the account
Select "Category 2" as the category
Select "Category 2" as the account
Try to select "Category 1" as the account (doesn't work)

+1
This is related to bug #656


Windows: Call TerminateProcess, not ExitProcess

Ty Baen-Price explains:

Problem and Resolution:
The following lines of code make use of the Microsoft API ExitProcess:
Apps\Speed.c line 335:	ExitProcess(ret);
Ms\uplink.c line 22: ExitProcess(1);

These function calls are made after fatal errors are detected and
program termination is desired. ExitProcess(), however causes
orderly shutdown of a process and all its threads, i.e. it unloads
all dlls and runs all destructors. See MSDN for details of exactly
what happens
(https://msdn.microsoft.com/en-us/library/windows/desktop/ms682658(v=vs.85).aspx).
The MSDN page states that ExitProcess should never be called unless
it is known to be safe to call it. These calls should simply be
replaced with calls to TerminateProcess(), which is what should be
called for disorderly shutdown.
An example of usage:
TerminateProcess(GetCurrentProcess(), exitcode);

Effect of Problem:
Because of a compilation error (wrong c++ runtime), my program
executed the uplink.c ExitProcess() call. This caused the single
OpenSSL thread to start executing the destructors of all my dlls,
and their objects. Unfortunately, about 30 other threads were
happily using those objects at that time, eventually causing a
0xC0000005 ACCESS_VIOLATION. Obviously an ACCESS_VIOLATION is the
best case scenario, as I'm sure you can imagine at the consequences
of undiscovered memory corruption, even in a terminating process.

And on the subject of TerminateProcess() being asynchronous:

That is technically true, but I think it's probably synchronous
"enough" for your purposes, since a call to TerminateProcess
suspends execution of all threads in the target process. This means
it's really only asynchronous if you're calling TerminateProcess one
some other process. If you're calling TerminateProcess on your own
process, you'll never return from the TerminateProcess call.

Fixes #2489
Was originally RT-4526

Let's see what Appveyor thinks of this...

So a question about the "speed" change: what about this means that we don't know whether it is safe to call ExitProcess or not? After all this is in an app not in the library?
On the uplink change is does beg the question whether we should be calling a function to exit the process at all from within the library. Although I'm not sure what you could do instead in that situation.

Good point about apps/speed.c, will revert that later todqy

Regarding ms/uplink.c and unimplemented, that's basically an assertion that an implementation is required.  We could simply assign NULL to unimplemented slots, but that would hardly be helpful.  Considering this stuff is for doing what happens when firing up a program that's directly linked with a shared library, I would say this is the right thing to do.

At this point, only ms/uplink.c is affected

Merged.
master:
925795995018bddb053e863db8b5c52d2a9005d9 Windows: Call TerminateProcess, not ExitProcess
1.1.1:
4af54c9b9933336d1c48f3012aca92262c36d3a8 Windows: Call TerminateProcess, not ExitProcess

It's an absurdly long commit message for a one line fix...


Online Documentation

I have been through some of the notebooks/tutorials available which are very helpful. I wanted to read the docs so that I can use pysal beyond these examples. However, at https://pointpats.readthedocs.io/en/latest/ I just see another version of the same notebooks/tutorials. I tried to access the source code online e.g. https://github.com/pysal/pysal/tree/master/pysal/explore/pointpats, but, all I see is an __init__.py file.
Accessing the local copies of the source code helped me a lot, though.
Thanks,
Vasilis

hey, sorry for the confusion. as of version 2.2, pysal is a metapackage that wraps a family of tools for spatial analysis. The source code for pointpats lives here and the API docs can be found here

i'm going to close this, but if you're still having issues, please feel free to open an issue over in pointpats


Ubuntu with No-GUI - stucked on Gdrive auth

Hi.
Im using digitalocean / ubuntu VPS.
My vps dont have ubuntu GUI.
so once i type 'google-drive-ocamlfuse' , it is looking for a browser to do the google auth.
Is there anyway to do this.?

Have you checked this wiki page?

Hi Astrada. Thank you for your guidance & YES, successfully done.
(Sorry - if my question may out of your topic)
I can see my gdrive folder inside my home/public_html folder. But is this folder can make use of as public access web folder.?
I tried to change the folder permissions to web user name, but seems failed. Via my normal FTP user access, i cant see my folder there.

Seems i got it.
I have to runt the ocamlfuse command as a 'user account'. So now i can use my drive as my hosting space..


ShannonLoader extension not working.

Hi @grant-h ,
I am trying to load the ShannonLoader extension into ghidra but it ain't working.
I am receiving the following error:

I've tried copying the ShannonLoader into the Ghidra's Extensions folder but it didn't work either.
I am using the latest ghidra version: ghidra_9.1.2_PUBLIC
Awaiting your response.
Thanks.

What value did you set your GHIDRA_INSTALL_DIR to? Did the console have any errors? What system are you using GHIDRA on?

What value did you set your GHIDRA_INSTALL_DIR to? Did the console have any errors? What system are you using GHIDRA on?

My bad I didn't set the path for GHIDRA_INSTALL_DIR.
A very n00b question how do I set the PATH for the same?

What value did you set your GHIDRA_INSTALL_DIR to?

export GHIDRA_INSTALL_DIR=/home/user/Desktop/ghidra_9.1.2_PUBLIC/

Did the console have any errors?

If you mean the Linux console while exporting the GHIDRA_INSTALL_DIR then no It didn't have any error.

What system are you using GHIDRA on?

Lubuntu 18.04.1 x86_64

This doesn't make sense as the version should be picked up by the install dir environment variable. It's not hardcoded anywhere. Are you using the right Java version (11) in Gradle to compile? I can't reproduce this on my end (macOS)

Closing as cant reproduce.


Add support for visualizing bones in Advanced Import Settings

When an imported model Skeleton3D type node is selected, the bones are drawn using lines or octahedrons to provide a clearer reference to their position. Refactored Skeleton3DGizmoPlugin::redraw now uses a static function to generate bone meshes
Update
editor/import/scene_import_settings.cpp
editor/import/scene_import_settings.h
editor/plugins/skeleton_3d_editor_plugin.h
editor/plugins/skeleton_3d_editor_plugin.cpp

Bugsquad edit: This closes https://github.com/godotengine/godot-proposals/issues/6801.
this PR comes from https://github.com/godotengine/godot/pull/76818

Can you merge into one commit?

Could you please amend your commit message to remove this part?
Update
editor/import/scene_import_settings.cpp
editor/import/scene_import_settings.h
editor/plugins/skeleton_3d_editor_plugin.h
editor/plugins/skeleton_3d_editor_plugin.cpp
Conflicts:
	editor/import/scene_import_settings.cpp
	editor/import/scene_import_settings.h

It's not relevant to the changes and doesn't need to be stored in the git history.

@jeronimo-schreyer do you think you will have time to resolve the pending review feedback?
If not, I can try take a look at it later this week

@jeronimo-schreyer do you think you will have time to resolve the pending review feedback? If not, I can try take a look at it later this week

Maybe you should just take over. It would be really nice to have this done for 4.3 and we don't have much more time

Superseded by #96094.


Snyk test is not run on prepush

Inconsistency between connecting-to-services and nearby-services-api in application of snyk.
See comments in https://github.com/nhsuk/nearby-services-api/pull/25 for more detail.

Merged


[CircleGraph] Tidy unused view methods

This will tidy unused view methods.
ONE-vscode-DCO-1.0-Signed-off-by: SaeHie Park<EMAIL_ADDRESS>

from draft #482


Thanks

I just wanted to say thanks, Matt. This is a well done library. I like the NSURLConvertible, it's a nice touch. A pleasure to use!

:+1:


Add extensions to register all APIs with DI

Motivation
Especially with large SDKs it may be cumbersome to register all the API
types one by one.
Modifications

Add AddAllApis and AddAllOtherApis extensions to IApiBuilder.
Add `ApiClientMappingRegistry' to prevent double registration of the
same API.
Apply source formatting to resource file syntax trees.

Results

Consumers may call .AddAllApis() to add all APIs. This method fails
if there are any manual registrations.
Consumers may call .AddAllOtherApis() to register just the APIs that
have not been manually registered.
Both methods accept a configuration action which allows configuration
of the IHttpClientBuilder. The first parameter to this action is the
API interface type which allows variation of behaviors by API.
Since we added InternalsVisibleTo for the client extensions, we can
remove the ref to NullableAttributes from the project.

/merge


Update dependencies and solve deprecated warnings.

ref #999.
cc @raphlinus.

Well, it seems there is a known SSL issue for nightly right now. https://github.com/rust-lang/cargo/issues/3340


Added menu option for grayscale

Added drop-down menu item to invoke grayscale simulation.

@nvkelso The OSX builds do not include this change, do they? Downloadable binary is still at 1.1.4.


Need API for clearing an Input field

The firechat example app wants to clear the input field when you submit a message.
Proposed solution (warning: this is a bit of a yak shave, but it makes Input behave more like our other Material widgets)
Example usage of new Input that clears on submit:
https://gist.github.com/collinjackson/33906003d2eef047b6fc
Add InputValue to editable.dart
https://gist.github.com/collinjackson/b7af2b2fc2349594e1f1
EditableString is designed to be mutable private state, InputValue is immutable and suitable for passing into widget constructors
onUpdated/onSubmitted take an InputValue instead of a String
Rename RawEditableLine -> RawInputLine so that InputValue can live in editable.dart
On build, detect when the passed InputValue doesn’t match the editableString’s InputValue and change the value
RawEditableLine should probably take onUpdated/onSubmitted arguments and manage the EditableString instead of Input?
Rename setSelection on EditableString to _setSelection since only RawEditableLine uses it?

I believe @jason-simmons resolved this with #1747


Fix regex transform failure on FileProperty.fullpath

Also add a test case which fails without the fix.  I don't know if this is the best place to cast PurePath to str, but it has the least impact here while still solving the problem.

recheck

Hi @brendon-boldt , and thank you for this PR! The bug has been fixed in #680 :)

Whoops.  I didn't do my due diligence in checking what was already open.


Spacy versioning error for DrQA

It would be helpful for people if you could add to the readme file that spacy version 2.1.0 is required to run DrQA. This isn't mentioned on their repo either and the repository seems dead so it probably won't be updated there anyways. If one tries to run their script it installs spacy v3 which is incompatible with their code. I had to find out the correct spacy version by looking up the date at which their scripts were written and comparing it to version releases.

Could you open a PR with the change? I will merge it in

Fixed in #58


Created ATM7 HTTP FAQ

Added a simple ATM7 HTTP FAQ so it doesn't have to be explained each time.

I assume we've not heard from the ATM folks since the initial issue? Mildly irritating, though I shouldn't throw stones :).
What's the exact questions people are asking when they come into Discord? I wonder if we can improve the UX elsewhere first, so people don't have to turn to Discord.
For instance, I kinda wonder if we should always add http programs to the path. This way they at least fail with "http not enabled" (which is more searchable) than the generic "not found" error.

I kinda wonder if we should always add http programs to the path. This way they at least fail with "http not enabled" (which is more searchable) than the generic "not found" error.

Sounds like a good idea to me. Or even some kind of stub program that gets aliased with the HTTP program names, but maybe just adding the programs themselves would be simpler.

Though props to that person who described Advanced Peripherals as "irish plethora".  If only anybody had this much to contribute on CC:T's documentation PRs >_>

Bloody hell. If only anybody had this much to contribute on CC:T's documentation PRs!

Yeah but nerds are so much more interested in correcting other nerds! /s
CC:T documentation is pretty fun though.

might this be merged?

insert elevator music here

Commited the changes, thanks for them.

Lovely, thank you!

no problem


drivers: flash: spi_nor: Remove page alignment requirement for read

For flash R/W, the start address need not be aligned on a page size.
Signed-off-by: Rajavardhan Gundi<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #11693   +/-   ##
=======================================
  Coverage   48.23%   48.23%
=======================================
  Files         279      279
  Lines       43291    43291
  Branches    10359    10359
=======================================
  Hits        20881    20881
  Misses      18270    18270
  Partials     4140     4140

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 096e18c...0cf78c1. Read the comment docs.

@rgundi I just noticed that there is a similar restriction on read. Writes do not need to be aligned to a page either. As long as the sector is previously erased and a location is not subsequently written to, it would be possible to write to that location. It would be better to remove the limitation on write as well and return error if the location is not previously erase.

@rgundi I just noticed that there is a similar restriction on write. Writes do not need to be aligned to a page either. As long as the sector is previously erased and a location is not subsequently written to, it would be possible to write to that location. It would be better to remove the limitation on write as well and return error if the location is not previously erase.

I too had noticed this but i wasn't sure if this was needed. In fact, there's even a parameter called write_block_size in flash_driver_api. I also noticed that not all flash drivers have tinkered around with this parameter. So, i believe it's ok to remove this restriction as well. However, i am not sure there's an efficient way to know if the location was previously erased. At least, none of the current drivers have this built-in.

I also pushed another commit which removes the unused block_size parameter.


🚀 [New Feature_LP] - Hover effect on navbar

Description of the enhancement. ✍🏻
Currently, when we hover over the navbar only the opacity of the links changes, this is not that attractive.
Describe briefly how you would implement this enhancement.
Some steps to implement this enhancement-
1. Instead, we can add hover and transition effects on the links to make them more attractive

Screenshots Example (if applicable)
Look at the below videos for more context :
Before :
https://github.com/CatsInTech/GitHub-ReadMe/assets/76463001/b9f805fe-ddb7-4c7b-b47e-708e4bd28010
After :
https://github.com/CatsInTech/GitHub-ReadMe/assets/76463001/e3c5368a-fb20-4b4b-8c9d-8c572d8f4fba
Additional information
@praveenscience , I'd like to work on this issue.
Contributor terms

[X] I agree to follow the Contributing Instructions

hey, i can do this

I can do this under SSOC 2.0

@mukul314 Started this one, let's see if you can do a PR before a day, else it can go to @kaleemullah786 or @Aditya-Lawate-codez, is that alright?

@praveenscience , I have made a PR for the above issue. The PR is #89 .

@mukul314 Use the right PR template and fill it properly, thanks.


As a forgetful user, Taskbook can detect duplicate contacts

... so that I would not accidentally add the same contact again

Already done.