Revert "Add back OpenGLES linking for gdx-setup"

Reverts libgdx/libgdx#6887
As discussed in #6884, don't merge this before the next libGDX release
@obigu could you milestone this?

don't merge this before the next libGDX release

I'm confused, should this be merged right before the release, or after the release?

don't merge this before the next libGDX release

I'm confused, should this be merged right before the release, or after the release?

It just should be in the same timeframe, since gdx-setup isn't part of a release and has it's own nightly builds.


Running Example app results in "ImportError: attempted relative import with no known parent package"

When generating an app with the example yaml file provided generates failing relative imports (using Python 3.8)
The import created from .models import Pets should be updated to from models import Pets
Issue is probably caused here: https://github.com/koxudaxi/fastapi-code-generator/blob/master/fastapi_code_generator/main.py#L75

Hello, any plan on fix this issue??


Warn if not in a virtual environment and .venv is present in the working directory

e.g. to help guard against common mistakes

FWIW, py (the Python launcher for Unix) automatically uses .venv if found without requiring it to be activated.

We also do the same if it's in the current directory (I'm 90% sure)

We also do the same if it's in the current directory (I'm 90% sure)

In my testing this is correct. I have not used Python launcher so I could be wrong but I think the difference will come when running the program if no virtual environment is active. py something.py works but python something.py fails even though the packages were installed into the virtual environment by uv.

We want to improve the handling of venvs, but rather by improving uv to make these more convenient than by showing a warning when it's possible to have workflows that don't require activating the venv (https://github.com/astral-sh/uv/pull/1505#issuecomment-1968506663)


Model signature with Iris Classification Example

What changes are proposed in this pull request?
The PR is to enable an option for the user to log and validate the Model Signature.
Model Signature in MLFlow - https://www.mlflow.org/docs/latest/_modules/mlflow/models/signature.html .
While deploying the model using mlflow-torchserve plugin, users can validate the input against the signature of the model.
If the signature of the input matches with the model signature, prediction process will be performed, otherwise an appropriate error message is thrown to the user.
Updated the Iris Classification example to log the model signature and validate the signature during the deployment phase. The example covers both success and failure scenarios by using different types of inputs.
How is this patch tested?
Existing Unit Tests
Release Notes
Is this a user-facing change?

[x] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release notes for MLflow TorchServe Deployment Plugin users.

(Details in 1-2 sentences. You can just refer to another PR with a description if this PR is part of a larger change.)
What component(s) does this PR affect?
Components

[x] area/deploy: Main deployment plugin logic
[ ] area/build: Build and test infrastructure for MLflow TorchServe Deployment Plugin
[ ] area/docs: MLflow TorchServe Deployment Plugin documentation pages
[x] area/examples: Example code

How should the PR be classified in the release notes? Choose one:

[ ] rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
[ ] rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
[x] rn/feature - A new user-facing feature worth mentioning in the release notes
[ ] rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
[ ] rn/documentation - A user-facing documentation change worth mentioning in the release notes

@harupy I have addressed the review comments. can you please review when you find time


Duplicate types and missing transaction Deserialization

Was planning to use this library to test an end-to-end example of transaction deserialization and verification.
Unfortunately have stumbled across a few hurdles:

Duplicate struct declarations for the same types

I simply want to validate the serialization of a transaction works; specifically the the example here:

unfortunately this requires deserializing the JSON string to the xrpl_api::OfferCreateTransaction struct (which implements serde's Deserialize), then using the xrpl_types::OfferCreateTransaction struct (which implements Serializer)
additionally, there isn't an API to convert xrpl_api::OfferCreateTransaction <-> xrpl_types::OfferCreateTransaction (i.e. a From trait) - hence a manual conversion must be done

Missing implementation of a Deserializer trait

Looking deeper into the serialization - seems like this isn't straightforward

Before trying to implement Deserialization; I was wondering if this implementation is already in the works?
Also are we planning to unify the multiple declarations for the transaction structs?

Re conversion between xrpl_api/xrpl_types types, that might be an interesting feature for the future, I will keep a note.
What is the use case for deserializing an xrpl_types (core server) type?

Re conversion between xrpl_api/xrpl_types types, that might be an interesting feature for the future, I will keep a note.
What is the use case for deserializing an xrpl_types (core server) type?

There are no canonical rust based xrpl client SDK implementations out there, this repo is the closest thing there is to that.
Specifically we want to do on-chain signature verification - given a serialized transaction.
The JavaScript SDK provides this functionality (signature verification), but unfortunately we cannot use JavaScript on-chain.
The chain we have is substrate based. There are additional requirements (i.e the code must be compilable to wasm32-unknown-unknown target) - but we will get to those later (might need to fork repo and put some functionality behind feature flags).

I have implemented support for deserialization (specifically for XUMM wallet based/signed transactions) here: https://github.com/futureversecom/xrpl-sdk-rust/pull/1
^ will be closing this issue.
Do let me know if you'd like me to PR the changes upstream.

Yes, I would love to have a PR, thank you!

I added some comments/nitpicks to your PR.

Thanks for the review; do let me know if you think we should remove the Deserialize and Deserializer traits; at the moment there is only a single implementation (which was initially inspired by xrpl.js deserializer).
On a sidenote, support for the the wasm32 compilation target is very useful (IMO) as it greatly increases code interoperability.
However not all crates support wasm32 (you can verify via cargo check -p xrpl_binary_codec --target wasm32-unknown-unknown)
^ I can remove this from the upstream PR if you'd like, but I will be keeping it in the fork (as it's required for on-chain transaction decoding and verification).

Opened PR here: https://github.com/gmosx/xrpl-sdk-rust/pull/30
feel free to leave any additional comments; will make changes accordingly!

Closing this issue - since has been addressed in https://github.com/gmosx/xrpl-sdk-rust/pull/30


Dummy seed

Add update the seed dummy

@KAYEMBAJONATHAN @MathildaShongwe15 @ReitumetseMoloto We are just holding off on this until this ticket is complete:
The reason being the employee table will be updated and the postal table will be removed, therefore the seed data will no longer be valid. Yielding errors.
Azure board


Bringing back URLDecode to fix regression

@barlevalon what is the reason - now we need a test to prove it:)


Added MiXCR

MiXCR is an ultimate software platform for analysis of Next-Generation Sequencing (NGS) data for immune profiling. It supports all kinds of single cell platforms and technologies for immune profiling, including commercial vendors such as 10x Genomics or BD Rhapsody and any custom protocol (droplet-based, plate-based, combinatorial barcoding etc). Check more at https://docs.milaboratories.com.

Hi @PoslavskySV, thanks for the submission 🎉! This looks pretty good and I would be happy to add it, I just have one small change. Please update the platform to "Java/Kotlin" which I think are the languages MiXCR uses.

Hi @lazappi, I just updated the platform as you suggested. Thank you!

Looks good to me, thanks again for the submission!


terminal not running commands

Type: Bug
PS D:\ADO\terraform-ado> az
az : The term 'az' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included,
verify that the path is correct and try again.
At line:1 char:1

az
~~

CategoryInfo          : ObjectNotFound: (az:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

VS Code version: Code 1.95.2 (e8653663e8840adaf45af01eab5c627a5af81807, 2024-11-07T11:07:22.054Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 5800H with Radeon Graphics          (16 x 3194)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
11.34GB (2.27GB free)

Process Argv
--crash-reporter-id ecab4eb5-fa25-494d-8f01-3b0ce5e21ca0

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

vscode-docker
ms-
1.29.3

debugpy
ms-
2024.12.0

python
ms-
2024.18.0

vscode-pylance
ms-
2024.11.1

remote-containers
ms-
0.388.0

remote-wsl
ms-
0.88.5

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
2e7ec940:31000449
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
jg8ic977:31013176
dvdeprecation:31068756
dwnewjupytercf:31046870
impr_priority:31102340
nativerepl1:31139838
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013
j44ff735:31177056

From outside the external terminal when I ran command
It's working but not working in vscode terminal I tried to restart it but
same issue not sure Whats wrong
On Fri, Nov 8, 2024, 2:33 PM Alberto Santini @.***>
wrote:

If you installed Azure CLI (az) inside the VSCode, from the Terminal, you
need to quit from VSCode and reenter, otherwise the environment variables
(PATH) are not read.
Then, if the problem persists, check your PATH.
You double check the problem, executing the command from the external
Terminal, outside VSCode.
If the issue is still there, VSCode is not guilty.
Hope that helps.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/233389#issuecomment-2464175688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATDN3NT3IXDXEKGVO2A4Y43Z7R453AVCNFSM6AAAAABRM6B24WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRUGE3TKNRYHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Rename from_slice methods to decode

The TaprootMerkleBranch and ControlBlock both have methods on them called from_slice but these methods do more that just basic copy from a slice. decode is a more descriptive name.
Deprecate the from_slice methods and implement decode, on other changes to the logic.
cc sanket1729

Thanks for the review man. I used you suggestions but I went with using only the heading line for the deprecated functions.

post-merge comment. I don't like this because it seems hard to decide when it is decoding vs from_slice. Should we also rename SecretKey::from_slice to SecretKey::decode ? It also does additional checks that point in within curve order. Similar thing for PublicKey::from_slice and Signature::from_slice.
I prefer consistent usage of from_slice and decode. This makes it harder to code as now we need to remember which are decode vs from_slice. Ofcouse, the IDE can fill it for me, but having some sort of logic helps.
I also had decoding associated with "decoding from stream" and from_slice associated with decoding from known slice Encoding with "encoding to io::Write" and serialize/to_vec to serializing to vector.
I am willing to update to my mental model for the above, but we should do it consistently so that it's easy to remember.

@sanket1729 in this case we had a whole list of hashes encoded as a slice, which means we were doing a nontrivial amount of destructuring.
With things like secret keys, or even public keys, the data ultimately "is" just a bytestring. maybe one with some extra restrictions.
I dunno if that distinction makes sense.

Sure, it does some sense. Perhaps, I can persuade you/others to also keep the from_slice method for consistency instead of deprecating it.
If not, no worries, this is not something I feel strongly

On this topic I spent yesterday hacking on a ScriptNum type and I used decode and encode for the logic that is currently in read_scriptint and write_scriptint.

Should we also rename SecretKey::from_slice to SecretKey::decode ?

I think so.

keep the from_slice method for consistency

We should rather rename the others. See also #1491
Constructing an array of non-byte items using from_slice taking u8 is super-confusing. One would normally expect from_slice to take [TapNodeHash] instead. For instance there's a similarly named extend_from_slice method on Vec which takes the items, not bytes. Honestly, when I first saw that API, my first reaction was "WTF?!" and I spent a few minutes in trying to understand it.


[improve][functions] EFFECTIVELY_ONCE mode support input topic batching

Fixes #17061

Main Issue: #17061

Motivation
The batch message will generate the same sequence ID in the function, which will cause a large amount of data in the batch message to be mistakenly deduplicated when EFFECTIVELY_ONCE mode and enable message deduplication.
Modifications
Use Interceptor with org.apache.pulsar.common.intercept.AppendIndexMetadataInterceptor and enable exusingBrokerEntryMetadataToClientEnabled on the broker。In the batch message we can generate the sequenceId of the sequence and get it using record.getSequenceId()
Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@jiangpengcheng @freeznet PTAL


whereDay and whereMonth inconsistent when passing int values

Laravel Version: 5.8.11
PHP Version: 7.3.2
Database Driver & Version: MySQL 5.7.25

Description:
When using whereDay, or whereMonth, passing an integer produces mixed results based on the current date
Steps To Reproduce:
add a whereDay, or whereMonth to a builder instance, when the date is less than 10th of the month, or the month is less than 10.

#28185 PR created

Which database types do these columns have?


How to set 100k mode

Sorry if this is the wrong place to post such a question, I haven't found any user forum etc.
I am trying to set 100k mode per this: "We note that you really should use o3d3xx-dump and o3d3xx-config to create an application that employs the 100k imager"
I must be getting the syntax wrong, but can't find what is correct. I keep getting "XMLRPC value out of range" when I try changing resolution. What should I be setting it to?
Update: I have seen in another post that I should be indeed setting it to 1, but this is yielding the error for me. I am using an o3d303 with lib 0.4.8 and fw1.6.2038

I am very sorry but 100k was removed from the regular O3D30x series.There will be a dedicated O3D3xx version with 100k support. Please contact your local ifm contact or maybe  @garrettplace can provide some details.

Thanks I will discuss with the local IFM, but for now downgrading to firmware 1.4.1639 has worked


mobile: Use true not "true" as an arg to set_bool_value()

mobile: Use true not "true" as an arg to set_bool_value()
Signed-off-by: Ryan Hamilton<EMAIL_ADDRESS>

/assign @alyssawilk

cc: @yanavlasov


[Feature Request] Multiple chat windows

Hello! The original Glass addon didn't have this feature, which made me terribly sad because it would otherwise be the perfect chat addon for me!
I like to have two chat windows onscreen; one for all the actual chatting and its respective tabs, and the other for loot/system messages/etc. so that those things don't clutter up my chat, whilst still always being visible as they occur.
With that in mind, do you think it'd be possible to integrate multiple chat windows into Glass? Same functionality and all on both!
If it's not possible, I understand completely, and if it's already a feature, I apologise for being a doofus and not noticing!
Thank you for your hard work, it's extremely appreciated!!

Since it's already an option in the default UI my addon supports it without interfering with it in any way.

<3

Wahoo! Amazing!!  I recall it not working too well or at all with the original addon, so I am very grateful for whatever tinkering you've done for this feature, intended or not!
Thank you so very much, and I hope you have the loveliest of days!


Cannot find module 'clone' (Gulp 3.9.0 and NodeJS 5.1.0)

I've recently updated to node v5.1.0 in my MacOSX 10.10 and now I'm trying to run gulp in my project and I get this error:
module.js:340
    throw err;
    ^

Error: Cannot find module 'clone'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:289:25)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)
    at Object.<anonymous> (/Users/<MYFOLDER>/node_modules/defaults/index.js:1:75)
    at Module._compile (module.js:425:26)
    at Object.Module._extensions..js (module.js:432:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:313:12)
    at Module.require (module.js:366:17)
    at require (module.js:385:17)

I've cleared the npm cache and reinstalling globally glup sudo npm install -g gulp and reinstalled all my project node modules npm installand the error is still there. I can't execute any gulp task.

Something is wrong with your install. This isn't a gulp problem, most likely a problem with your environment or npm.

Having the same issue, except I'm not on OSX. @cmayo would you mind sharing your solution? If you've found one, that is.


chore(master): release 1.4.11

:robot: I have created a release beep boop
1.4.11 (2023-05-09)
Miscellaneous

deps: update docker docker tag to v23.0.6 (652d945)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-build-deps/releases/tag/v1.4.11 :sunflower:


Would be great to have a bower.json file added.

Thanks for CharFunk!

Would be happy to take a PR for this... I don't know much about bower

+1


DocumentTableCell RowSpan and ColumnSpan should default to 1

Don't expose it as nullable

It is set in the swagger, but not with the right properties for codegen to do this for us. The service team has been contacted and they are looking into adding these changes for their preview.2 update


EEPROM programing failes with 'Error: usb_i2c_read(): LIBUSB_ERROR_TIMEOUT'

I can't finish the build since i'm stuck at programing the EEPROM. Loading the firmware into the SRAM works fine and:
usb_sniffer_win.exe --mcu-sram usb_sniffer.bin
... outputs:
Uploading 3187 bytes into the FX2LP SRAM
...done
But when trying:
usb_sniffer_win.exe --mcu-eeprom usb_sniffer.bin
... i always get:
Programming 3187 bytes into the FX2LP EEPROM (SN:<PHONE_NUMBER>5c08)
Error: usb_i2c_read(): LIBUSB_ERROR_TIMEOUT
I've tried different USB ports (connected to different USB host controllers) and tried programing the EEPROM under Linux too -> but every attempt ends with:
Error: usb_i2c_read(): LIBUSB_ERROR_TIMEOUT
I opted for some alternate parts, since the ones from the BOM weren't available at the time. For example i got a HG24LC128M/TR EEPROM:
https://www.lcsc.com/product-detail/EEPROM_HGSEMI-HG24LC128M-TR_C3007884.html
... and some different oscillators:
https://www.lcsc.com/product-detail/Crystal-Oscillators_Abracon-LLC-ASEDV-26-000MHZ-LR-T_C1857762.html
https://www.lcsc.com/product-detail/Crystal-Oscillators_Abracon-LLC-ASEDV-24-000MHZ-LR-T_C17564399.html
But this shouldn't be a problem... at least: i thought. Or did i miss something and one of these components is the culprit?

The problem would be somewhere around EEPROM. The oscillators are ok if it gets that far.
But this error is strange. This can only happen during data verification, which means that EEPROM worked well enough to at least pretend to write the data.
Build the PC application from code and put debug prints in the loop in the fx2lp_eeprom_upload() to see what iteration fails. This is the only place that does the reads.

Also, EEPROM may be extremely slow. So, I would also try to place os_sleep(10); between fx2lp_eeprom_write() and fx2lp_eeprom_read() in the same function.

I've added some debug prints and got a read error at the start address after a successful write:
Programming 3187 bytes into the FX2LP EEPROM (SN:<PHONE_NUMBER>5c08)
Writing to EEPROM at address 0x0
Write complete.
Reading from EEPROM at address 0x0
Error: usb_i2c_read(): LIBUSB_ERROR_TIMEOUT
Trying to write to the EEPROM a second time results in a write error at the start address:
Programming 3187 bytes into the FX2LP EEPROM (SN:<PHONE_NUMBER>5c08)
Writing to EEPROM at address 0x0
Error: usb_i2c_write(): LIBUSB_ERROR_PIPE
Since i've had a MiniPro TL866 laying around somewhere in the chaos of my home, i desoldered the EEPROM and tried to read it. After selecting an 24LC128 as chip device i was able to read the EEPROM... but it contained only a couple of the first bytes of the BIN file.
I resoldered the EEPROM. And since every writing attempt results in a corrupt firmware: i had to connect the board with the BOOT and VCC test points shorted to be able to give it another try. Loading the firmware into the SRAM worked as always. This time i tried to write to the EEPROM with a added os_sleep(10) between writing and reading... but without any success:
Programming 3187 bytes into the FX2LP EEPROM (SN:<PHONE_NUMBER>5c08)
Writing to EEPROM at address 0x0
Write complete. Sleeping...
Reading from EEPROM at address 0x0
Error: usb_i2c_read(): LIBUSB_ERROR_TIMEOUT
I've now ordered some genuine Microchip 24LC32A EEPROMs from a local seller. Hopefully it'll fix this strange behaviour.

Yeah, I would not spend too much time debugging that. I would assume that something is fishy with those EEPROMs.

Also, you should be able to use the sniffer, you just need to upload the firmware every time you power cycle it. It will not have the correct serial number, but this should not cause any issues.

You were absolutely right:
The 24LC32A EEPROMs arrived today... i replaced the fishy one... and:
Programming 3187 bytes into the FX2LP EEPROM (SN:<PHONE_NUMBER>5c08)
...done
... the problem was fixed.
Thanks for taking the time to help... and for this awesome project!


USMON-652: Addressing verifier issue on RHEL involving kernel 4.18.0-305

What does this PR do?
This PR resolves a verifier issue identified on RHEL with kernel version 4.18.0-305

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

To replicate the issue, execute agent version 7.49.1 with USM enabled on RHEL 8.4 using kernel 4.18.0-305.el8.x86_64 (EC2 AMI ID is ami-02e0bb36c61bb9715). Expect a verifier error related to the socket__classifier_entry function. Subsequently, run version 7.51 and ensure that the verifier error does not recur.
Reviewer's Checklist

[ ] 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.
[ ] 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 if the qa/skip-qa label is not applied.
[ ] 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.

Closing this pull request as it is not relevant to the main branch; the problem has been addressed. You can find the resolution at https://github.com/DataDog/datadog-agent/pull/21230/files (pkg/ebpf/c/bpf_telemetry.h)


bug: Webapp: import traces is not working

Node.js version
18.18.0
NPM version
9.8.1
@checkly/cli version
4.14.0
Steps to reproduce
Sorry for writing about this bug in this repository as I did not manage to find a repo related to APi specifically or WebApp. As this repo is pinned I will leave this bug here
I was trying to enable traces to check it in my WebApp running on EC2 and was following a video from https://www.checklyhq.com/docs/traces-open-telemetry/
I was not able to switch Import traces slider in my https://app.checklyhq.com/settings/account/traces
In webUi I see this

In console I see
Access to XMLHttpRequest at 'https://api.checklyhq.com/accounts/<my account id>' from origin 'https://app.checklyhq.com' has been blocked by CORS policy: Method PATCH is not allowed by Access-Control-Allow-Methods in preflight response.

After I switched plugin to ignore CORS in my browser I got PATCH error as per screenshot attached

I tried a direct curl request
curl --location --request PATCH 'https://api.checklyhq.com/accounts/<my account id>' \
--header 'Accept: application/json' \
--header 'X-Checkly-Account: <my account id>' \
--header 'Authorization: Bearer <my ip token>' \
--header 'Content-Type: application/json' \
--data '{
    "settings": {
        "useChecklyOtelProvider": true,
        "tracing": true
    }
}'

I got 401 error . But other API calls with same headers are working
I tried to call otel API from curl but I got redirection
curl -I https://otel.eu-west-1.checklyhq.com/v1/traces
HTTP/2 301
date: Sun, 08 Dec 2024 17:51:55 GMT
location: https://www.checklyhq.com/docs/traces-open-telemetry

My software:
Brave v1.73.97
Ubuntu 22.04 LTS
What is expected?
It should work I guess
What is actually happening?
see above
Any additional comments?
No response

I see you monitor bugs with sentry and I guess you can easily track this bug in sentry

Hey @sir-alex! Thank you so much for submitting this bug report, we'll look into it.
If you'd like to get updates, please ping in our Slack community or email our support team.
In the future, please only report CLI-related issues in this repo. Thank you!

ok, sorry for wrong repo. I simply did not manage to find the right one. I see problem has been fixed and I can import traces now. Thanks for a fast respond. Appreciated


Support Image History

The history field of the Image Manifest is used as a log of what commands/who made modifications to the image. It's generally added to on a layer-by-layer basis, and if it is maintained correctly, you get nice displays in outputs like docker image history:
➜ docker history container-app-tres:0.0.6
IMAGE          CREATED         CREATED BY                                      SIZE      COMMENT
229b57e74fb4   7 seconds ago   dotnet publish -p:PublishProfile=DefaultCont…   8.38MB    A dynamically-generated image layer from the .NET SDK
<missing>      10 days ago     /bin/sh -c #(nop) COPY dir:b0ac707752044c30c…   21.3MB
<missing>      10 days ago     /bin/sh -c #(nop)  ENV ASPNET_VERSION=7.0.0-…   0B
<missing>      10 days ago     /bin/sh -c ln -s /usr/share/dotnet/dotnet /u…   24B
<missing>      10 days ago     /bin/sh -c #(nop) COPY dir:99ddbc873c8fc6285…   72.9MB
<missing>      10 days ago     /bin/sh -c #(nop)  ENV DOTNET_VERSION=7.0.0-…   0B
<missing>      10 days ago     /bin/sh -c #(nop)  ENV ASPNETCORE_URLS=http:…   0B
<missing>      10 days ago     /bin/sh -c apt-get update     && apt-get ins…   37MB
<missing>      2 weeks ago     /bin/sh -c #(nop)  CMD ["bash"]                 0B
<missing>      2 weeks ago     /bin/sh -c #(nop) ADD file:0eae0dca665c7044b…   80.4MB

and the Docker Desktop image history view:

A simple addition to the AddLayer function makes this possible:
var history = new JsonObject();
history["created"] = DateTime.UtcNow;
history["author"] = ".NET SDK";
history["created_by"] = "dotnet publish -p:PublishProfile=DefaultContainer";
history["comment"] = "A dynamically-generated image layer from the .NET SDK";
config["history"]!.AsArray().Add(history);

But I wanted to slow down and think about what data we would want to embed in this structure. The spec has the following to say about it:

Should we put package/SDK versions in this? Should the created date be the actual now? I know we've been thinking about determinism in this project, but the history property isn't part of digest calculation.

IMO actual-now makes sense to me--determinism is IMO important for the images and not so much for the manifests. I created precedent for that in #89.
SDK version (or "file that is running this calculation's version") makes sense to me and could fit in created_by: .NET SDK container tooling 7.0.100+abcde123.
Wondering how much other stuff from the task parameters we should log and lean toward "don't bother doing everything".

"don't bother doing everything"
💯

My main point with this issue is that we should be in the history somehow, and I think setting the created_by is good enough for now.


The handling of direct_mentions in Cisco Spark needs attention

Botkit automatically removes the @mention from the beginning of the string when ingesting a direct mention.  With Spark, this is done by looking for an HTML tag in the content.
Unfortunately, the different Spark clients encode this HTML in at least 4 different ways, requiring Botkit to get a bit more sophisticated in its approach.

Mac Desktop client: base64 ENCODED person id, AND  tag
"text": "hi Steve from the Mac Desktop client",
"html": "hi Steve from the Mac Desktop client",
iPhone native client: base64 DECODED person id, NO  tag
"text": "Hi Steve S from my iPhone",
"html": "Hi Steve S from my iPhone",
Web client (Chrome Windows): base64 ENCODED person id, NO  tag
"text": "hi my love Valérie from Web client Chome on Windows",
"html": "hi my love Valérie from Web client Chome on Windows",
Windows desktop client: base64 ENCODED person id, AND  tag
"text": "hi my love Valérie from Windows Desktop",
"html": "hi my love Valérie from Windows Desktop",

This is fixed in 0.5.6


python-cryptography is failing to build

http://hydra.nixos.org/eval/1214570?filter=cryptography&compare=1214551&full= suggests that it's an across-the board failure, and the specific issue appears to be a missing dependency:
Searching for py>=1.4.29

Note: Bypassing https://pypi.python.org/simple/py/ (disallowed host; see http://bit.ly/1dg9ijs for details).

Couldn't retrieve index page for 'py'
Scanning index of all packages (this may take a while)

Note: Bypassing https://pypi.python.org/simple/ (disallowed host; see http://bit.ly/1dg9ijs for details).

No local packages or download links found for py>=1.4.29
error: Could not find suitable distribution for Requirement.parse('py>=1.4.29')
builder for ‘/nix/store/fv88n1xjn6lpmj24hsxbwj0vdp4gbnhp-python3.4-cryptography-0.6.1.drv’ failed with exit code 1

@domenkozar last time this package broke you seemed to know what was going on...

py dependency is missing or is too new

So, I tried to do a little work to resolve this.  My conclusions are sufficiently counterintuitive---I found not one or two, but three reasons that seem like they should have broken cryptography all along---that I feel like I must be doing something wrong.
First, as @domenkozar observed, the cryptography dependency on py was not stated in its nix expression.  Since this expression doesn't appear to have changed significantly in the last week or so---that being the last time I installed a package that depended on it---I'm confused as to how it ever successfully built without that.
Adding that dependency, I found that the version of py that is currently in nixpkgs is older than what cryptography expects.  So that's another mark in the "how did cryptography ever build" column.
Finally, having updated the dependency, I found that cryptography now gets a runtime error during testing.  Again, I don't understand how it ever built.
So I am now trying to upgrade cryptography as well.  I will post a PR when I have one, but I am inherently uncertain whether it's the right thing to do because I have no idea why it worked before, or why it broke now.

Apparently, cryptography tests have been broken by #9253.

@vbgl Ah, I see you are likely correct.  It is actually the pytest update that requires an updated version of py.

Yes, that's indeed the case. I didn't do a nox-review wip on pytest so that's likely why I didn't see at the time.

FWIW, if I apply the change revving the py package to 1.4.30 that's linked above, I still get an error in the cryptography test suite, though it seems to be an actual issue with the cryptography package.

There's a much newer package at PyPi, version 1.0:
https://pypi.python.org/pypi/cryptography

And I've actually got a change that upgrades it (and a couple of things it depends on), which I will submit once I've finished running nox-review on it. :)

Likely that one will fail due to python-fedora, #9260

And python-novaclient, which is apparently no longer present on any mirrors.  And others for similar reasons.  But it's easy enough to run it and at least see if any of the failures actually appear to be because of the change in the cryptography package.

I don't know the process---should this stay open until these changes land in a particular branch, or is it remaining open because I didn't ever mention #9274 in the thread?

Can be closed now, yup :)


Win32Exception when trying to benchmark

After updating to 0.7.* having exception when launching benchmark.
http://i.imgur.com/3srpxqo.png

I have updated BenchmarkDotNet in dotnetex to v0.7.4. Now it works fine on my computer. Please, check it.

Nope, continuing failing with same error. Tried as Admin, result the same.

Do you have MSBuild in the %PATH% variable? If not, you should add it because BenchmarkDotNet use it for building each benchmark.

Is it works now?

Had the same issue, but hen added MSBuild to %PATH% the problem with Win32Exception disapeared, but now a ihave another problem: when i run benchmark a i have next error:
Target competition: Crc16Bench
// ***** Competition: Start   *****
// Found benchmarks:
//   Crc16Bench_Data4_Throughput_X64_CurrentJit_NET-Current -w=5 -t=10
//   Crc16Bench_Data4_Throughput_X86_CurrentJit_NET-Current -w=5 -t=10
// **************************
// Benchmark: Crc16Bench_Data4 (Throughput_X64_CurrentJit_NET-Current) [-w=5 -t=10]
// Generated project: c:\Users\danila\Documents\Visual Studio 2013\Projects\Crc16Benchmark\Crc16Bench\bin\Release\Crc16Bench_Data4_Throughput_X64_CurrentJit_NET-Current
// Build:
Build failed
Fail
// Run, Process: 1 / 3
// Platform=X64
// Run, Process: 2 / 3
// Platform=X64
// Run, Process: 3 / 3
// Platform=X64
AverageTime (ns/op): Median = 0; StdDev = Infinity; Min = 0; Max = 0;
OperationsPerSecond: Median = 0; StdDev = Infinity; Min = 0; Max = 0;
// **************************
// Benchmark: Crc16Bench_Data4 (Throughput_X86_CurrentJit_NET-Current) [-w=5 -t=10]
// Generated project: c:\Users\danila\Documents\Visual Studio 2013\Projects\Crc16Benchmark\Crc16Bench\bin\Release\Crc16Bench_Data4_Throughput_X86_CurrentJit_NET-Current
// Build:
Build failed
Fail
// Run, Process: 1 / 3
// Platform=X86
// Run, Process: 2 / 3
// Platform=X86
// Run, Process: 3 / 3
// Platform=X86
AverageTime (ns/op): Median = 0; StdDev = Infinity; Min = 0; Max = 0;
OperationsPerSecond: Median = 0; StdDev = Infinity; Min = 0; Max = 0;
// ***** Competition: Finish  *****
// BenchmarkDotNet=v<IP_ADDRESS>
// OS=Microsoft Windows NT 6.1.7601 Service Pack 1
// Processor=Intel(R) Core(TM) i3-2100 CPU @ 3.10GHz, ProcessorCount=4
// CLR=MS.NET 4.0.30319.18444, Arch=64-bit
Common:  Type=Crc16Bench  Method=Data4  Mode=Throughput  Jit=CurrentJit  .NET=Current

Platform
AvrTime
StdDev
op/s

  X64 | 0.000000 ns | Infinity s |    0 |
  X86 | 0.000000 ns | Infinity s |    0 |

// ***** Competition: End *****

Same here...

@danilanv
You have to remove white spaces, cyrillic symbols etc. in the solution path.
I suppose the second argument should be enclosed in quotes.

@olsh @danilanv I solve my issue as well, for me it was actually the fact the benchmark class was an inner class, which caused BenchMarkDotNet to generate wierd class names with + in the class name...

@mumusan, @danilanv, @damageboy , @olsh
I have implemented new build system via API from the Microsoft.Build assembly. Please, check version 0.7.5.

version 0.7.5 - it works!

Same for me, nicer integration with Life(tm)

@danilanv, @damageboy, great!


Fixes for Windows

Fixes #33
This branch contains several modifications to get the lmdb package building, with all tests passing, on Windows.

Usage of ioutil.TempDir was made portable by allowing the package to use the os-dependent default temporary directory instead of "/tmp".

Translate LMDB return values to the proper syscall.Errno constant values in the internal operrno function. This allows error handling code to be portable as long as named errno constants are used instead of numeric errno literals.

There also needs to be some extra error handling documentation for Windows.


Add files via upload

Your checklist for this pull request
Author name
Author: Navneet Saxena

About your game
What is your game about?
A catching game where the user needs to collects as many fruits can they can in the bowl.

How do you play your game?
Use W to move left and D to move right.

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Hi, I believe I have changed the file name now.


adds txpool function and its local node test

This is my work for this issue. #36
Infura doesn't provide txpool api, so I just tested at local environment synced with rinkeby network.

Thanks, @currybab  for first contrib! 👍
We will review it soon!

Good start! Parsing txpool content into native values (ethereum addresses and transactions) would be a nice addition

Done

Done


bootstrap support Angular2 release candidate

Does Bootstrap supports Angular 2 release candidate?

Supporting JS frameworks is outside Bootstrap's project scope. This is frontend framework that provides you with html and css foundations for your project as well as few basic JS components like dropdowns/modals that are purely representational (ergo control elements styles/visibility).
You should instead ask at Angular 2 communities about known issues in writing views/components using Bootstrap's markup.


[Text Analytics] Implement IsEnvironmentReadyAsync

Resolves: https://github.com/Azure/azure-sdk-for-net/issues/35452.

API change check
API changes are not detected in this pull request.

/azp run net - textanalytics - tests

/azp run net - textanalytics - tests

/azp run net - textanalytics - tests

/azp run net - textanalytics - tests

Looks good, however, the issue seems to be not TA specific. Should the pipeline add some latency after the resource creation step to mitigate it?

We don't want to add latency to all pipelines. Few, hopefully, need it. See my review for what I suggested to inject latency when needed.


Mod doesn't load on server

Minecraft Version
1.19.3
Plusls Carpet Addition Version
0.3.144
Carpet Version
1.4.96+v230201
Fabric API Version
0.74.0
Description
Hello, I was using before the build "plusls-carpet-addition-1.19.3-0.2.5-beta+5fa8bc5.jar", which worked fine.
Now after switching to "PluslsCarpetAddition-1.19.3-0.3.144+85a443e-beta.jar", the mod doesn't work anymore.
Carpet says in the log, it can't load the rules, they are not found.
To Reproduce
/
Expected behavior
No response
Log
No response
Latest CI build tested

[X] I confirm that this problem still exists with the latest CI build.

Now you need to use /pca to manage the rules.

Ohh, oki thanks!


Ensure Site is deleted

closes #35
Summary
Currently when deleting preview on pr close, sometimes the site is not fully deleted and on forge level it's no longer showing up under server sites, but on top nav it's still showing up and not fully deleted, which if pr is closed and reopened again will throws site already error and it won't be possible to preview deployments again.
I haven't properly tested same behavior with database, It may be possible that is reproduced
In order to ensure site is fully deleted I Added a condition that refetch the site until it throws 404 where the error is caught and the status is set to null, otherwise the site status is on removing so it'll just wait until it's fully removed.
Steps to reproduce the issue

Create a site in forge and trigger destroy previes in debug.ts notice that the site is partially deleted and still accessible through id from url and from top nav menu sites
Repeat the same process with the code in this pr, the site will be fully deleted

@bakerkretzmar ping

Going to reach out to Forge support, if deleting the site via their API works correctly this shouldn't be necessary.

Going to reach out to Forge support, if deleting the site via their API works correctly this shouldn't be necessary.

Indeed it works when i tested with direct api call, i assume that the delete process runs during the request and it happens that as soon as the request times out/closes  (maybe due to github actions or axios default config) the delete process in forge is not fully done

This should be fixed by 392dba4ff4e1dc8019011b5724b8f55368ef120f (available in v2.5.0). The underlying issue with Forge is still present and the Forge team is still investigating. It's not the request timing out, it's making the deletion request before the scheduler has been fully uninstalled.


[iOS] Clear nowplayinginfocenter on reset()

When you call reset() the track does not get removed from the MPNowPlayingInfoCenter delegate. If we call the UIApplication.shared's method beginReceivingRemoteControlEvents we can instantiate it when someone adds a track. This is delayed by 500ms so that the UI thread is not blocked when attempting to do the rest of the functionality of adding tracks. By setting it to being able to receive them, we can then execute endReceivingRemoteControlEvents on reset() and clear the MPNowPlayingInfoCenter properly

Would this address #478 ?

@curiousdustin yes it does fix that

@dcvz i left a comment, let me know thanks


Related to the deprecation of MainContext::channel in GTK on Linux.

Is your feature request related to a problem? Please describe.
A warning is currently being issued that glib::MainContext::channel is deprecated in the Linux event loop-related code.
warning: use of deprecated associated function `gtk::glib::main_context_channel::<impl gtk::glib::MainContext>::channel`: Use an async channel, from async-channel for example, on the main context using spawn_future_local() instead
   --> src/platform_impl/linux/event_loop.rs:227:71
    |
227 |     let (window_requests_tx, window_requests_rx) = glib::MainContext::channel(Priority::default());
    |                                                                       ^^^^^^^
    |
    = note: `#[warn(deprecated)]` on by default

warning: use of deprecated associated function `gtk::glib::main_context_channel::<impl gtk::glib::MainContext>::channel`: Use an async channel, from async-channel for example, on the main context using spawn_future_local() instead
   --> src/platform_impl/linux/event_loop.rs:241:55
    |
241 |       let (device_tx, device_rx) = glib::MainContext::channel(glib::Priority::default());
    |                                                       ^^^^^^^

Describe the solution you'd like
We can update the currently used MainContext::channel to async_channel in event_loop.rs on Linux.
Additional context
See: https://gtk-rs.org/gtk4-rs/stable/latest/book/main_event_loop.html or https://discourse.gnome.org/t/help-required-to-migrate-from-dropped-maincontext-channel-api/20922

we are not using gtk4, and we don't plan to anytime soon so I think we can ignore this for the time being.


add missing vector header

Fixes compilation under MSYS2's clang backend.
$ ninja
[2/3] Compiling C++ object src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj
FAILED: src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj
"ccache" "c++" "-Isrc/d3d9/d3d9.dll.p" "-Isrc/d3d9" "-I../src/d3d9" "-I../include" "-fcolor-diagnostics" "-D_FILE_OFFSET_BITS=64" "-Wall" "-Winvalid-pch" "-Wnon-virtual-dtor" "-std=c++17" "-O0" "-g" "-msse" "-msse2" "-msse3" "-mfpmath=sse" "-Wimplicit-fallthrough" "-Wno-unused-private-field" "-Wno-microsoft-exception-spec" "-DNOMINMAX" "-D_WIN32_WINNT=0xa00" "-gstrict-dwarf" "-gdwarf-2" -MD -MQ src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj -MF "src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj.d" -o src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj "-c" ../src/d3d9/d3d9_mem.cpp
In file included from ../src/d3d9/d3d9_mem.cpp:1:
../src/d3d9/d3d9_mem.h:70:36: error: implicit instantiation of undefined template 'std::vector<dxvk::D3D9MemoryRange>'
      std::vector<D3D9MemoryRange> m_freeRanges;
                                   ^
C:/Users/Mangix/scoop/apps/msys2/2022-01-28/clang32/include/c++/v1/iosfwd:260:28: note: template is declared here
class _LIBCPP_TEMPLATE_VIS vector;
                           ^
In file included from ../src/d3d9/d3d9_mem.cpp:1:
../src/d3d9/d3d9_mem.h:71:37: error: implicit instantiation of undefined template 'std::vector<dxvk::D3D9MappingRange>'
      std::vector<D3D9MappingRange> m_mappingRanges;
                                    ^
C:/Users/Mangix/scoop/apps/msys2/2022-01-28/clang32/include/c++/v1/iosfwd:260:28: note: template is declared here
class _LIBCPP_TEMPLATE_VIS vector;
                           ^
In file included from ../src/d3d9/d3d9_mem.cpp:1:
../src/d3d9/d3d9_mem.h:129:53: error: implicit instantiation of undefined template 'std::vector<std::unique_ptr<dxvk::D3D9MemoryChunk>>'
      std::vector<std::unique_ptr<D3D9MemoryChunk>> m_chunks;
                                                    ^
C:/Users/Mangix/scoop/apps/msys2/2022-01-28/clang32/include/c++/v1/iosfwd:260:28: note: template is declared here
class _LIBCPP_TEMPLATE_VIS vector;
                           ^
3 errors generated.
ninja: build stopped: subcommand failed.

Can you please format your commit like the rest in this project then it can be merged.

Maybe better to put in https://github.com/doitsujin/dxvk/blob/master/src/util/thread.h . What do you think?

Why would we put it in thread.h? That header uses no vectors

hrm OK. It's just that this file inherits all its headers from there.

Fixed title.


fix: make opening file path relative to vims cwd (closes #678)

I'm not a lua expert. And I use this plugin and fzf since a week. So there might be issues with my proposal.

I'm not a lua expert. And I use this plugin and fzf since a week. So there might be issues with my proposal.

Unfortunately this will break almost everything when using the plug-in on anything other than cwd.
I’ll take a look it your issue, it’s still not clear to me why editing using full paths causes issues, AFAIK he buffer name is always full path no matter how it was opened.
In any event, if this is to be fixed it should be fixed as part of the open file action.

Unfortunately this will break almost everything when using the plug-in on anything other than cwd.
I tested with builtin, buffers and btags, beside git_files. They all still work for me. I don't see why this should break, as it would strip off the cwd, which should be fine if it doesn't exist?

why editing using full paths causes issues, AFAIK he buffer name is always full path no matter how it was opened.
 expand('%:r') will expand to the full path instead of relative path, causing trouble in my setup.

set statusline+=\ %f also shows the "full" path with ~ instead of relative path, consuming unnecessary space for information I don't care about.
I didn't had the issue with telescope and ctrlp, which I used before.
I've the patch applied in my setup and will comment if I encounter any issue. But maybe my explanation already helps? I'm also fine with having my own patch, but thought this might be an improvement. I won't be disappointed if it would be closed without merging :)

I tested with builtin, buffers and btags, beside git_files. They all still work for me. I don't see why this should break, as it would strip off the cwd, which should be fine if it doesn't exist?

This will not break using default options, this will break when using FzfLua with the cwd option, it will also break when using providers with previews outside of the cwd (LSP symbols, etc) and many others issues I can’t forecast ATM, guaranteed :)
Responding to the rest in #678.

In any event, if this is to be fixed it should be fixed as part of the open file action.

https://github.com/ibhagwan/fzf-lua/commit/592ab9a26176a7f9ebd43faf91d1f5d066e0aa07


Create pre-configured aws-sdk clients from aws.Providers

Currently the underlying sdk is exposed with calls like new aws.sdk.S3(), it would be nice to create sdk clients from aws.Provider objects.

Could you please provide some more details about your use case? I would typically expect aws.Provider objects in your Pulumi program to describe configuration for deploying your application and not necessarily the same provider that you would want to run your client code with?

The aws.sdk object is currently designed to be used only at runtime (e.g. with a callback used as a CallbackFunction).  However - we definitely would like to expand this to be usable also at deployment time.  In the case it is used at deployment time though - it is true that it should be possible to get an instance of the SDK initialized with the same configuration as a given AWS Provider instance (or the ambient default provider).
This is also needed for Policy - I had a note about that (and the general case this issue is tracking) in https://github.com/pulumi/pulumi-policy/issues/122#issuecomment-573111188.

Since we are no longer including the aws.sdk property (https://github.com/pulumi/pulumi-aws/pull/2584), this no longer makes sense.


Debug Logging

A brief glance at the code shows that there is no debug level logging. only exception?
Is this correct, or am i missing something?

Logging could very much be improved, but at the same time it needs to be balanced against performance. In programs that use Stash there are often hundreds of calls to cached objects, and each additional function call that gets added into something like Item creation then gets replicated hundreds of times.
Exception logging is easy and obvious so that's what I started with. Debug logging is a little more complicated because of the performance reasons and needs a more nuanced solution. I think having a DebugDriver that can be put in as a first layer in the Composite driver would be a perfect way to handle things, as it means that when debugging is turned off all of those extra calls simply disappear.

Would it not be possible to have a debug flag set, then around each of the debug method calls have an if() around it, this adds a cost of O(1), which is negligible in terms of the system.


Add option to control which state variables to save / exclude

The monitor option allows us to control which function to save, but there is no way to control which state variable to save.
Rather, DynaSim by default saves every single state variable. We can drastically cut space usage down by a factor of 10 or more by only saving certain state variables (e.g. membrane voltages, specific synaptic/ionic currents).
We should add an option to DynaSim to allow saving only certain state variables.
We can use a similar syntax to monitor

Perhaps the default should be not to save anything other than the voltage unless specified. It is minimalistic but most of the times useful for testing while getting the model in the desired regime of operation. Then the user would re-run the simulation and save the state variables of interest for posterior analyses.

the quickest things that's backwards compatible is to add and include_only and exclude_variables flags. those could be used as regular expressions on the fields of data. this would < 20 lines of code. the former would be the only variables to include, filtering out the rest. the latter would remove the matched variables leaving the rest.

Copying relevant notes from emails:
I think the default should remain saving all state variables and time points; that is the most transparent practice and minimizes confusion about what is returned. However, an option could be added to save only a desired subset of variables. The recommended practice then would be to set downsample_factor >1 and the new dsSimulate option 'variables' (or whatever it's called) to whatever state variables the user wishes to record. I don't think DynaSim should "guess" what variables the user cares about because DynaSim is not meant to be a simulator for only neural models. I imagine guessing which variables are desired by the user could easily create problems once more users begin exploring non-neural models that don't conform to whatever assumptions would necessarily be built into the process of guessing what variables are important.
In contrast, dsPlot "guesses" which variables should be plotted by looking at the list of state variables and plotting the first. Other variables can be plotted by setting the 'variables' option. This approach to "guessing" assumes the first state variable in the master equations (which is the first in the list of state variables) is what the user cares about because, in our case of neural models, that is always the voltage, which happens to be what we care about. Users working with non-neural models can plot other variables by simply calling dsPlot again while setting the 'variables' option (without re-running the simulation). If this approach is used in dsSimulate, then users who care more about other state variables will have to learn about the option and re-run the entire simulation with the 'variables' option set in order to obtain the simulated data they care about; however, before they do that, they'll probably give up on using DynaSim.
Also, in general, we need to be VERY careful about changing default behavior at this point because doing so will likely break existing scripts/workflows that use the default options.

We should avoid adding fields to the specification structure that aren't related to the model specification. "monitors" are already an exception that I was hesitant to make.
My vote is to either:
(1) Add a 'custom_save_variables' option to dsSimulate; with that approach, the scope of the variable to be recorded could be controlled the same as dsPlot where "V" means record "V" in all populations and mechanisms where "V" is defined, "E_V" means record "V" in the "E" pop, etc, or
(2) add the ability to constrain the recorded state variables using the "monitor" keyword; like monitor variables={'V','m'}; the monitor statement could be located either in the master equations or the mechanism file. In a mechanism file the scope would be limited to that mechanism (it would be recorded for all populations containing that mechanism). In the master equations the scope is the entire population ("x" appearing in the master equations or any mechanisms would be recorded), or it can be constrained using mech.variable (like "monitor iNa.m"). That is how scopes are presently defined using monitors.
The more we can be consistent with / leverage the way things are already done, the better. That minimizes the number of new things that must be learned to use dynasim.

I like the idea of it being the same approach used for downsample_factor the most, with scopes determined like dsPlot using the 'variables' option (or possibly 'custom_save_variables' in dsSimulate). However, in dsSimulate (unlike dsPlot which always constrains the variables plotted), the default value should be [], and the saved variables should be constrained iff ~isempty(options.custom_save_variables).

I would tend to prefer extending the monitors function to include state variables. In some ways, the way it is now is counterintuitive - e.g., why should the user be able to control which functions are saved using monitors, but not which state variables?
But perhaps we could do both - enabling monitors and also adding the custom_save_variables flag.
The downside of using only monitors, is that then it's difficult to select conjunctions populations and variables. For example, if you want to track AMPA state variables only in superficial RS cells, there is no way to do this using monitors without creating a separate AMPA mechanism exclusively for this population.
M2C

One other thing to consider, also, is that in dsPlot2, populations and variables are treated as distinct different axes. They are controlled by two separate arguments: populations and variables. This behaviour can be changed, but it is the current functioning and I have found it useful for minimizing the complexity of regexp's required.

I like the idea of doing both. You get the best of both worlds that way and can choose between the less flexible method of using monitors (in master equations or mechanism files) vs the more flexible method of using options in calls to dsSimulate. Doing both requires establishing a way of resolving differences between the lists specified using the two approaches. We could either take the union of the two lists or have one override the other. I think taking the union is the best approach because it is less confusing and only that approach overcomes the downside of using monitors that Dave pointed out.

i'm confused about how monitors would enable exclusion of state vars.

one approach to using monitors to exclude variables is to check if any state variables are being monitored and to exclude all others that are unmonitored. This isn't very transparent, though; e.g., it could cause problems if someone borrows a mechanism from someone else that has a state variable monitor in it and they aren't aware of it; i.e., they could end up confused as to why the data structure doesn't contain all state variables, since they didn't do anything in their run script or self-made mechanisms to constrain the saved variables.
another (better) approach is what Erik suggested, something like monitor exclude_variables={'m'} or monitor include_only={'h'} in a mechanism (e.g., iNa.mech) or master equation. When present in a mechanism file, this has a transparency issue like the example above, but in this case, it doesn't affect all state variables and would be easier for users to troubleshoot (i.e., just look at the mechanism files for whichever state variables are not being recorded). However, this approach has a steeper learning curve for beginners and the counterintuitive use of a monitor to exclude things; the latter could be justified possibly by interpreting a monitor as a process that is configured by whatever comes after the monitor keyword. An important point about the include_only option is that the "only" part should probably mean "only for the scope at which the monitor appears"; for instance, if include_only is used in a mechanism, it shouldn't preclude recording the population voltage or else it will lead to a more difficult troubleshooting scenario like the first one I described.
Using the option 'custom_saved_variables' (with scopes set like variables in dsPlot, dsCalcFR, etc) could be recommended for beginners and monitors for more advanced users.

Another approach to having the ability to include AND/OR exclude variables is to have two new options for dsPlot, something like include_variables and exclude_variables, both with scopes determined like variables in dsPlot, dsCalcFR, etc. The recorded set of variables then would be the set difference between the expanded list of variables from include_variables and the expanded list of variables from exclude_variables. Here's an example where the two are used together:
Given model with populations (X,Y,Z) and dynamics dV/dt, include_variables = 'V', and exclude_variables = 'Y_V':
Result: record only 'X_V' and 'Z_V'
This is currently my favorite solution and what I personally would like to be able to do.

Posting this again with more examples to show the power of this approach:
Another approach to having the ability to include AND/OR exclude variables is to have two new options for dsPlot, something like include_variables and exclude_variables, both with scopes determined like variables in dsPlot, dsCalcFR, etc (all of which call dsSelectVariables to get the list of variables). The recorded set of variables then would be the set difference between the expanded list of variables from include_variables and the expanded list of variables from exclude_variables. Here are some examples where the two are used together:
Given model with populations (X,Y,Z) and dynamics dV/dt, include_variables = 'V', and exclude_variables = 'Y_V':
Result: record only 'X_V' and 'Z_V'
This would be accomplished simply by entering:
data = dsSimulate(spec,'include_variables','V','exclude_variables','Y_V',...);
To additionally record 'm' and 'h' from iNa.mech in pops X and Z but nothing from Y:
data = dsSimulate(spec,'include_variables',{'V','m','h'},'exclude_variables','Y_*',...);
or simply
data = dsSimulate(spec,'include_variables',{'V','m','h'},'exclude_variables','Y',...);
Note that * can also be used in dsSelectVariables to specify lists of variables. For instance, given pops (X1,X2,Y), 'm' and 'V' could be recorded in X1 and X2 like this:
data = dsSimulate(spec,'include_variables',{'X*_V','X*_m'},...);
Again, this way of specifying lists of variables is already used in dsPlot etc via dsSelectVariables.
This is currently my favorite solution and what I personally would like to be able to do.

@jsherfey I like this approach. I also like the idea of making this syntax uniform across the functions you mentioned.
How would this interact with monitors? In your example, if you have monitor variables {'m','h'} in Na.mech, would  'exclude_variables', 'h' override this? I would think the answer would be yes.
If no monitor statements are supplied, would the default then to be monitor all state variables and no functions? (E.g., the way things are currently)
Also, would dsSelectVariables apply only to state variables, or would you be able to use this to customize functions as well?
Lastly, would you have to re-compile every time include_variables and exclude_variables specifications are changed?

this should also be implemented in dsImport


Download Links not working

The download links for the sharepoint migration tool on this page send you to a specified blob can't be found page.

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

ID: 89de996c-fdbe-91dd-75c2-34b7e12b1f49
Version Independent ID: 9961e4fe-6764-d6c8-9bde-271ef81c5a6d
Content: Introducing the SharePoint Migration Tool
Content Source: SharePoint/SharePointServer/migrate-to-sharepoint-online/introducing-the-sharepoint-migration-tool.md
Service: unspecified
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Link has been fixed.

Hi Nick -- have you tried deleting your temp.tmp folder and clearing your cache -- and then retry the download and install?
At this time only SharePoint 2013 is supported.

It does not work at all!! Tried to copy a single file:
TaskID,Message,PendingAction,ExceptionId
"5924dc58-6f91-4491-9226-4c828535f318","The content type 'Folder' cannot be created or updated","ACTION_STOP","0x02060003"
Bummer!

Hi jvalks:  At what point did you receive this error? After you entered your info and clicked Migrate? What steps had you done up to that point?

I entered the credentials and the site location link... I'm using other migration software normal, but since last week that software is getting throttled while uploading... It seems to be fixed today! I have to try if the SharePoint migration tool is also working, that means something spooky was going on at the Microsoft backend (not the first time...)...

Hi, jvalks,
Thanks for your feedbacks.
Please share everything under C:\Users%user<EMAIL_ADDRESS>folder to Migration feature crew<EMAIL_ADDRESS>We would like to work with you to address the issue encountered in your migration.
Migration feature crew team

Done!

@jasond727 - I am afraid this issue went stale in the issues here for documentation. I am sorry on the delay. It has been a long time so I hope you have the issue resolved. If not though, I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Here is a quick video on opening a ticket: https://www.youtube.com/watch?v=puuOy0sEg1Y
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Makes implicit change culture transparent

There have been a number of issues where I've needed to clarify why a specific
change isn't ready or shouldn't be merged. The rules of engagement, and their
rationale, shouldn't require me, as I didn't start Brave and may not be around forever!
My hope is that by stating things like how change works in the root folder, we can
avoid some back and forth due to previously unwritten expectations.
Using rule-of-three for this change :) Here are five issues where I've basically restated the contents above manually. I'd really like to use my time and the time of contributors towards shared change, since in both cases it is a scarce resource.
#273 #265 #238  #226 #194

cc @openzipkin/core

ps the contents of this are a direct port from the same file in Feign, which was created under the exact circumstances as repeat here https://github.com/OpenFeign/feign/blob/master/HACKING.md

will merge this in a day


Use the DpexTargetContext inside dpjit

[X] Have you provided a meaningful PR description?

The PR is a draft for the set of changes to ensure that numba_dpex.dpjit uses its own target that is isolated from Numba's
CPUTarget. The goal of the PR is to ensure numba-dpex does not have unintended side effects on Numba, and also free numba-dpex to implement custom dispatch, caching, function registry support.

[ ] Have you added a test, reproducer or referred to an issue with a reproducer?

[ ] Have you tested your changes locally for CPU and GPU devices?
As yet minimally.

[ ] Have you made sure that new changes do not introduce compiler warnings?

[X] If this PR is a work in progress, are you filing the PR as a draft?'

Fixes #1024

FYA @chudur-budur the changes in my PR will impact your SyclQueue work. I will merge my changes only after you are done.


Get-ForensicAmcache returns error

When I'm trying to run Get-ForensicAmcache it returns frollowing error:
PS C:\Windows\system32> Get-ForensicAmcache -HivePath C:\Windows\AppCompat\Programs\Amcache.hve
Get-ForensicAmcache : Index and count must refer to a location within the buffer.
Parameter name: bytes
At line:1 char:1

Get-ForensicAmcache -HivePath C:\Windows\AppCompat\Programs\Amcache.hve

- CategoryInfo          : NotSpecified: (:) [Get-ForensicAmcache], ArgumentOutOfRangeException
- FullyQualifiedErrorId : System.ArgumentOutOfRangeException,PowerForensics.Cmdlets.GetAmcacheCommand

Thanks for reporting this bug.  I will take a look at it in the morning and hopefully have a fix up soon.

PS C:\Windows\system32> $stacktrace
at System.Text.ASCIIEncoding.GetString(Byte[] bytes, Int32 byteIndex, Int32 byteCount)
at PowerForensics.Registry.NamedKey..ctor(Byte[] bytes, String hivePath, String key) in C:\Users\tester\Documents\Gi
tHub\PowerForensics\PowerForensics\src\Windows\Registry\Cells\NamedKey.cs:line 56
at PowerForensics.Registry.NamedKey.GetSubKeys(Byte[] bytes, String key) in C:\Users\tester\Documents\GitHub\PowerFo
rensics\PowerForensics\src\Windows\Registry\Cells\NamedKey.cs:line 264
at PowerForensics.Artifacts.Amcache.GetInstancesByPath(String hivePath) in C:\Users\tester\Documents\GitHub\PowerFor
ensics\PowerForensics\src\Artifacts\Amcache.cs:line 101
at PowerForensics.Cmdlets.GetAmcacheCommand.ProcessRecord() in C:\Users\tester\Documents\GitHub\PowerForensics\Power
Forensics\src\Cmdlets\Artifacts\Get-Amcache.cs:line 68
at System.Management.Automation.CommandProcessor.ProcessRecord()

I think this should be fixed in https://github.com/Invoke-IR/PowerForensics/releases/tag/1.1.1

@jaredcatkinson We're seeing the same error on a Win7 system using Get-ForensicPrefetch -Fast -Volumename C:
Any thoughts about whether this may be related or is fixed in a more recent version?

@jaredcatkinson After further investigation on the affected Win7 system, we found a .pf file in \Windows\Prefetch that was 0 length. Can you add a check for file size and skip 0 length files? I've never seen a 0 length .pf before, no idea how it got there.


Fixed Typographical Error in guide/english/css/index.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

@thecodingaviator is this PR good to be merged

Yup @iamrajiv

@iamrajiv Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 📝


[qemu] q35: PCI bus rescan code is messing up PCIe hotplug

see https://github.com/kata-containers/runtime/issues/2664

This issue covers a number of separate bugs.  The one that's most relevant (and not covered by other more specific issues) is the problems caused by PCI rescans colliding with hotplug operations.  Can we retitle it to reflect that?

As @jodh-intel pointed out in #782, removing the rescan has the side effect that we don't wait until the hotplugged device is ready before starting the container, which breaks the CI (probably amongst other things).
I'm submitting a new PR which includes @devimc's patch to remove the rescan along with extra logic to have the agent explicitly wait for the VFIO hotplug to complete.

I'm no longer planning to try to get this finished for Kata1, I'll be pursuing it with Kata2 instead.


feat(build): create index canister

Motivation

Changes

Tests

This was a test branch to show @StefanBerger-DFINITY the script of creating index canisters


"mvn package" does not build with-dep.jar

graphhopper version: 09d5fdaf6963 (but using the latest version of ./graphhopper.sh).
$ ./graphhopper.sh clean
$ ./graphhopper.sh web berlin-latest.osm.pbf
## using java 1.8.0_131 (64bit) from
## using existing osm file berlin-latest.osm.pbf
## building graphhopper jar: tools/target/graphhopper-tools-0.9-SNAPSHOT-jar-with-dependencies.jar
## using maven at /usr/local/Cellar/maven/3.5.0/libexec
## now web. JAVA_OPTS=-Xmx1000m -Xms1000m -server
Error: Unable to access jarfile ./web/target/graphhopper-web-0.9-SNAPSHOT-with-dep.jar

Notice that ./graphhopper.sh web calling the package maven-phase, like this:
execMvn --projects web -am -DskipTests=true package

AFAICT the install maven-phase is required to generate the *-with-dep.jar.
I'm not a maven expert, is there some phase other than install that can be called to generate the *-with-dep.jar ? Seems weird to need to "install" in order to get artifacts. Is a change needed to the pom.xml configuration so that package will generate the *-with-dep.jar ?
More generally: going forward, does graphhopper.sh continue to be the recommended way of working with graphhopper?
Thanks.

I'm not sure what your problem is here (mvn clean package builds all necessary artifacts and so does the graphhopper.sh script). Also questions should go to our forum

More generally: going forward, does graphhopper.sh continue to be the recommended way of working with graphhopper?

To develop with graphhopper sources the recommended way is a Java IDE like Netbeans or IntelliJ

graphhopper version: 09d5fda (but using the latest version of ./graphhopper.sh).

We only support the latest master or a previous stable version. No mix or something, although I'm not sure what the problem could be here.


Add information about Kristoff Misquitta

Submission for first assignment.

Hello, thank you for submitting this pull request! Because of previous contributions, this file has a merge conflict. I'll fix it myself for now and in a future meeting I'll explain what that is and show how you'd fix it yourself.


I think Prefer InboundRefernces is going too deep.

I did this:
curl -H "Accept: text/turtle" -H "Prefer: return=representation; include=\"http://fedora.info/definitions/v4/repository#InboundReferences\"" http://<IP_ADDRESS>:8080/fedora/rest/prod/f3/xp/73/2v/f3xp732v > out.ttl

and got this file:  https://gist.github.com/jcoyne/5d153463e1379ac2324b
Should triples like this be in that response?
<http://<IP_ADDRESS>:8080/fedora/rest/prod/f3/rv/0k/sk/f3rv0ksk> <http://pcdm.org/models#hasMember> <http://<IP_ADDRESS>:8080/fedora/rest/prod/0b/a2/dc/77/0ba2dc77-bb4f-4b74-aaf9-17d64a369be2> .

This request also goes pretty slow. I'm seeing 800ms-1100ms.

@jcoyne, which version of F4 is this?

I suspect the issue here is the recent update to add inbound references that are generated by IndirectContainers (https://github.com/fcrepo4/fcrepo4/commit/8845a22fdeffbec07bb0d0abd35ec9fd3854e788).  There may need to be some more logic to make sure the inbound references are from the source node to prevent this from happening.

@awoods 4.3.0

Tracking here: https://jira.duraspace.org/browse/FCREPO-1701

This issue was addressed some time ago via https://github.com/fcrepo4/fcrepo4/commit/731e92ca5ad30e83edae4ef6db972d9e8a1e72fd


<EMAIL_ADDRESS>hash check failed

gpu-z: 2.26.0 -> 2.27.0                                                                                WARN  Scoop uses 'aria2c' for multi-connection downloads.                                                              WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.                                    Updating one outdated app:                                                                                Updating 'gpu-z' (2.26.0 -> 2.27.0)                                                                                Downloading new version                                                                                Starting download with aria2 ...                                                                                Download: [#8ea4c0 432KiB/6.7MiB(6%) CN:1 DL:562KiB ETA:11s]                                                           Download: [#8ea4c0 4.7MiB/6.7MiB(70%) CN:1 DL:2.6MiB]                                                                  Download: Download Results:                                                                                Download: gid   |stat|avg speed  |path/URI                                                                             Download: ======+====+===========+=======================================================                              Download: 8ea4c0|OK  |   2.7MiB/s|C:/Users/Emmett/scoop/cache/gpu-z#2.27.0#http_nl1-dl.techpowerup.com_files_GPU-Z.2.27.0.exe_GPU-Z.exe                                                                                Download: Status Legend:                                                                                Download: (OK):download completed.                                                                                Checking hash of GPU-Z.2.27.0.exe ... ERROR Hash check failed!                                                         App:         extras/gpu-z                                                                                URL:         http://nl1-dl.techpowerup.com/files/GPU-Z.2.27.0.exe#/GPU-Z.exe                                           First bytes: 4D 5A 90 00 03 00 00 00                                                                                Expected:    3a34f1a6fafd10b7019a12c9e9a334fe                                                                          Actual:      a8bf8436caece1a691acad62d83256bb

/close
asd-collab


allow for directory and file extension in hca upload files command

Ticket https://app.zenhub.com/workspace/o/humancellatlas/upload-service/issues/36

Codecov Report

Merging #174 into master will increase coverage by 0.01%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master     #174      +/-   ##
==========================================
+ Coverage    86.9%   86.92%   +0.01%
==========================================
  Files          34       34
  Lines        1291     1308      +17
==========================================
+ Hits         1122     1137      +15
- Misses        169      171       +2

Impacted Files
Coverage Δ

hca/upload/cli/upload_command.py
86.66% <90%> (+0.95%)
:arrow_up:

Continue to review full report at Codecov.

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


found more places that should use active_release

Hi @leochen4891 as per the comments in #1099 the changes to the package resources are not correct; these should use node['bcpc']['hadoop']['distribution']['release']. However, there are some good finds here! Your change to mapred.rb is a good find, your second change for the init scripts to namenode_standby.rb is a good find. It is questionable how we want to handle the changes to setup-mapred-app in resource_manager.rb; I am for staging the new release there (i.e. using release not active_release).

Also, it looks like bach_spark attributes/default should be active_release.

PR updated.


Are there any cost implications for upgrading TFS 2013 to latest Azure DevOps Server?

Will there be any cost implication for upgrading TFS 2013 to Azure DevOps Server 2019?
How can one determine the cost implication of upgrading TFS 2013.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c8a797fb-84e9-fb7b-d2d1-ad5f1f65af8c
Version Independent ID: 03523a91-dfb5-edf3-fa1e-b4551cdb938d
Content: Upgrade requirements - TFS
Content Source: https://mseng.visualstudio.com/DefaultCollection/TechnicalContent/_git/tfs-docs-pr?path=%2fdocs%2fupgrade%2fupgrade-2013%2fupgrade-2013-requirements.md&version=GBlive&_a=contents
Product: devops-server
Technology: tfs-admin
Git Login: aaronhallberg
Microsoft Alias: aaronha

Usually you're getting more stuff for free as you upgrade.

Test hub at the basic level
Package management at basic
...

Got it.


[Bug] Higher CPU Usage with Master (1f937c8) vs v1.2.5

Describe the issue
After updating to Master branch commit 1f937c8 for the update check frequency fix I noticed that my Truenas server is now using a lot more CPU than when using v1.2.5 of this integration.
On v1.2.5 my idle Truenas CPU load is about 1-2%
On 1f937c8 my idle Truenas CPU load is 6-7%
How to reproduce the issue

Install v1.2.5
Wait some time for CPU stats to collect
Install 1f937c8
Wait some time for CPU status to collect
Compare

Expected behavior
CPU usage should remain the same as v1.2.5 where it barely made a difference on the Truenas server. The updates in 1f937c8 seem to have increased CPU usage 3-4x.
Screenshots

The third and fourth screenshots show the difference before and after installing 1f937c8 and the idle CPU baseline being increased 3-4x.
Software versions

Home Assistant version: 2024.1.6
TrueNAS integration version: 1f937c8
TrueNAS version: TrueNAS-13.0-U6.1

Diagnostics data
Not sure what to collect for this,
Traceback/Error logs
No errors or anything in the logs. I have my log level set to warning and there are no entries about this integration. I can collect info/debug logs if they are required to troubleshoot this issue.
Additional context
No other changes were made to the Truenas or Home Assistant server. I had restarted both as well after changing integration versions with no change in CPU usage.

oof, thats a lot. what kind of hardware do you use?
Try changing following like to 60 seconds and let me know.
            update_interval=timedelta(seconds=10),

My Truenas server has an intel xeon E5-1620 v4 so its an older cpu but plenty powerful.
I made the change you mentioned and it seems the cpu usage is back down to how it was before, barely noticeable. I guess 10 seconds was a little too frequent for my system. This integration is working great again now!
Also related to this update_interval, do you plan on making this configurable? Not sure the effort involved as I have not made a home assistant integration before but I feel it might be a feature someone would find useful.
Thanks again! This should be good to close out if you want.

great.
I dont currently have plans to make it configurable. 60 sec is standard, 10 is there just for development.
Its fairly easy to make update interval configurable if its needed, thats not a problem.

The 10 seconds makes sense for development. I have done similar with my projects.
I don't think I need to have the update interval configurable, I was just curious. I don't need to request features I won't use.
I am going to close this out now as it looks like everything has been working fine overnight.


[BW Clock] Larger font size if colon is hidden to improve readability further.

Its now possible to disable the colon in the time -- 2314 instead of 23:14. If the colon is disabled I also show the time with a larger font to improve readability further:
Colon enabled (fullscreen):

Colon disabled (not fullscreen):

Nice - thanks!


Add Developers to Projects

Add developers to relevant projects.
@aybanda is to be added to the fantasy-signals project.

@aybanda You've been added to the fantasy-signals project.
Confirm accepting the invite and I'll close this.


Add ebml extension element

As discussed in the cellar meeting. This subelement would allow a way to store webm data, cppname, etc without mixed such extensions with the rest of the element definition.
So in Matroska's EBML Schema:
<element name="AttachmentLink" path="0*1(\Segment\Tracks\TrackEntry\AttachmentLink)" cppname="TrackAttachmentLink" id="0x7446" type="uinteger" maxOccurs="1" minver="1" maxver="3" webm="0" range="not 0">
  <documentation lang="en">The UID of an attachment that is used by this codec.</documentation>
</element>

would become
<element name="AttachmentLink" path="0*1(\Segment\Tracks\TrackEntry\AttachmentLink)" id="0x7446" type="uinteger" maxOccurs="1" minver="1" maxver="3" range="not 0">
  <documentation lang="en">The UID of an attachment that is used by this codec.</documentation>
  <extension cppname="TrackAttachmentLink" webm="0"/>
</element>

This makes it a lot easier to have a valid EBML Schema for Matroska.

I'd prefer the last commit (type -> purpose) to be a separate PR as it seems unrelated to the extension PR.

I like this idea a lot.
I think it would be nicer to allow more than one extension and have them name. They could also be extended later. For example the cppname could have extra things between library version, comments to add in the code of a particular library, etc.
Something like this:
<element name="AttachmentLink" path="0*1(\Segment\Tracks\TrackEntry\AttachmentLink)" id="0x7446" type="uinteger" maxOccurs="1" minver="1" maxver="3" range="not 0">
  <documentation lang="en">The UID of an attachment that is used by this codec.</documentation>
  <extension type="webm" allowed="0"/>
  <extension type="libmatroska" cppname="TrackAttachmentLink">
    <comment>attachment used by the track content<comment>
  </extension>
</element>

On the other end from the CELLAR meeting I thought we decide to strip the extra attributes from the main document and only maintain that one. The extra attributes would be put in another document (so could be using this format). So do we want to allow all kinds of "meaningless" (in the EBML way) attributes to a normative document ? Or are we just defining the separate document we might keep on the side ?

Based on comments and to clear merge conflicts, this PR is updated. Now the extension element may be repeated, the use of it is fairly ungoverned by EBML. I think the historic details of divx and webm are meaningful in a way to matroska so think it could be interesting to have an unofficial but acceptable way to let them stay in the schema, rather than us working to maintain two documents.

In the case of cppname and divx they are both legacies. Once we settle a way to use them we will not have to update anything after that. I'd favor a separate file saying which EBML path needs a special cppname (which should actually be part of libmatroska, not the Matroska repo) or divx, which we saw will probably remain in the spec but forbidden.
For WebM this should also be a separate file that we may or may not maintain. We do not want it to be part of the normative RFC.

So, that being said, it feels like this extension element would be OK but, at least for Matroska, not in the main normative document. I can understand there would be some cases where it's useful to people, so it's probably a good addition. But we need to do it right as it will be part of the EBML specifiation and XSD validation.
I'm not sure what processContents="skip" in XSD means but it sounds like anything could be put in there. So <extension><webm>1</webm></extension> would be OK ? It might cause problem with our XSLT file being fed such files if we don't know anything about the format.
I would prefer something a little more restrictive like <extension name="webm" value=""/> plus extra attributes allowed. At least tools that have no idea what's in the extensions could still output something meaningful about it.

Rebased so it can be merged.

Yes the skipContents means that the validator ignores the contents, as in EBML I'm thinking of leaving it to the DocType about what goes inside. I am ok with <extension name="webm" value=""/> style although it is more limited, but could avoid skipContents.

@robUx4 I added a @type attribute for the extension element so that it's contextualized in a consistent manner. I preferred @type to @name but don't feel too strongly. Once this is merged, I can adjust the corresponding pr at matroska.

rebased and squashed to address merge conflict


Miscellaneous links

Added various links that I tend to use when I want to practice.
Added some interesting links that were mentioned during the Well architected framework workshop.

👍🏼


chore: move style linters to the style namespace

Redone version of #15971.

Indeed - fixed now.

This PR may incur a merge conflict with #15017.

Thank you!
maintainer merge

Thanks! :tada:
bors merge


Eliminate harmful warnings

Operating System: Linux, later Mac

Description
retval missing in settle.cc: In function ‘int mainer(double*, double*, double*, double*, int*, double*, double*, double*, double*, double*)’:
"control reaches end of non-void function [-Wreturn-type]"
→ add "return 1;" at the end of the function

When calling void nrerror(char error_text[]);
ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
→ modify to void nrerror(const char error_text[]);

There is still one weird occurrence of the second - nrerror()
spline.cc: In member function ‘void Spline::splint(double*, double*, double*, int, double, double*)’:
spline.cc:244:25: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
  244 |   if (h == 0.0) nrerror("Bad xa input to routine splint");
      |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

while in other modules there are no complaints (???)

That issue with spline.cc was caused by locally declared function inside the
void Spline::splint(double xa[], double ya[], double y2a[], int n, double x, double *y)

method


Docs Changes Needed from dbt-snowflake Issue #1033

At a minimum, update body to include a link to the page on docs.getdbt.com requiring updates and what part(s) of the page you would like to see updated.

Same as https://github.com/dbt-labs/docs.getdbt.com/issues/5734

Closed in favor of https://github.com/dbt-labs/docs.getdbt.com/issues/5520


[FEATURE REQ] Better support for function calling in ASP.NET Native AOT applications

Library name
Azure.AI.OpenAI
Please describe the feature.
According to the docs at https://www.nuget.org/packages/Azure.AI.OpenAI, developers should use BinaryData.FromObjectAsJson  with an anonymous C# class for tool definitions. This is not possible when using the library with ASP.NET NativeAOT as reflection-based serialization is disabled in this application type. NativeAOT is a popular .NET application type. It would be great if there was guidance for how to use the OpenAI .NET library in this context.
My current workaround: Specify the JSON schema as a string.

There are more issues related to NativeAOT and Azure.AI.OpenAI. Will close this issue and create another, more complete one.


Prefer blob URLs from cache whenever possible

This bug has two parts:

Currently, in our html rewriter code here and here, we fetch from disk and then generate a blob url. Instead, we should try to check if one already exists for that path in cache and use it and if not, only then generate a new one.
In our css rewriter code, for now whenever we encounter a url("path to file") we replace it with the contents of that file. Instead, we should check for a blob url for that path in cache and use that instead, otherwise, we should generate a new one and use it instead of copy/pasting the entire contents of that file.

cc @humphd

I think that if there isn't a cached BLOB URL already existing, it means that writeFile() was never called for that path, and therefore trying to re-read it isn't going to produce anything new.  One interesting side-effect of doing this change is that it would turn the process into something synchronous, since it's just going to be O(1) lookups.

Assigning to @K-LV, since I want this done in conjunction with #162.


Chat - Quote markdown not applied for multiline text

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.31-8
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause Internal Team
Action Performed:

Launch app
Open a chat
Enter > and in next line enter a text
Eg: >
Ty
Note in preview, quote markdown is displayed
Send the message
Note quote markdown not applied after sending message

Expected Result:
Quote markdown must be applied for multiline text.
Actual Result:
Quote markdown not applied for multiline text.
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/2bb5563c-f4cb-4a56-82e0-64689dc773fc

View all open jobs on GitHub

@kevinksullivan  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to Live Markdown

This is expected behavior except I caught an exceptional case.
When a multiline link is sent as a message, the whole link is pressable .
But when multiline link is sent with quote markdown, it breaks the link and looks like an unexpected behavior.
https://github.com/user-attachments/assets/3a8ed2c0-664b-40a5-8c40-41d51cf456eb

Proposal
Please re-state the problem that we are trying to solve in this issue.

Quote Markdown is not applied to multiline links

What is the root cause of that problem?

Quote markdown was designed to be applied only on the same line so when applying to multiline link it produce unexpected result.

What changes do you think we should make in order to solve the problem?

While processing multiline links first apply newline markdown so now \n in between link is converted to </br> and quote markdown would not consider it as newLine character so quote markdown will be extended till end of the link.
To do the same include newline to filterRules here
https://github.com/Expensify/expensify-common/blob/4f3a1a18de51416cd307a2ea55fa9cca5d402f83/lib/ExpensiMark.ts#L980

Before
https://github.com/user-attachments/assets/cd7fd1ba-415d-4020-abcc-56bf0d4bfdb8

Result
https://github.com/user-attachments/assets/65841a66-9147-4287-b13d-4a3c719a24bc

### What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

I don't think this is unexpected behavior if you go to a second line with no >... Maybe we'd remove the preview on the first line, if anything. But yeah, I'm not certain this is a bug. I am going OOO so I'm going to loop in another BZ, but perhaps we take it to product?

Yeah, I'm going to close given that markdown isn't a priority right now.


Python version??

I am facing challenge while installing explainX library. It keeps failing. Is there any specific python version on which this runs?
I am using 3.10.9

We have yet to test explainX with python 3.10.9. We're doing so along with a few other updates, so sorry for the bother, and please stay tuned. In the meantime, yo can try installing on python 3.7.


chore: trigger windows vhd production pipeline

What type of PR is this?
Trigger containerd&docker pipeline as part of the automation process.
/kind feature

What this PR does / why we need it:
Build a production VHD with the changes made to the latest windows base image version.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on wanqingfu/trigger-winvhd-prod at 61.542%

Totals

Change from base Build<PHONE_NUMBER>:
61.5%

Covered Lines:
2570

Relevant Lines:
4176

💛  - Coveralls


Consider if the route() helper should throw if the route does not exist

Right now, the route() helper returns null if the route does not exist. This means it can a typo in a Blade file like {{ route('hoem') }} will silently link to the same page (href=""). The DX could then be improved by throwing a helpful exception like in Laravel.
There have been some discussion on this before, IIRC, and we may want this in v2 paired with a deprecation warning for legacy uses.

Previous discussion on the matter

https://github.com/hydephp/develop/issues/199
https://github.com/hydephp/develop/pull/200
https://github.com/hydephp/develop/pull/591

This is simple enough to fix by using getOrFail in the function, wondering if we should take more steps to normalize the API, and match to Laravel. I'm not sure if there is ever really a reason to want a null return here, I think if you don't want to catch an exception, it's better to use Routes::exists to check first.

Fixed in https://github.com/hydephp/develop/pull/1741


feat(material/form-field): Adds persistHints option

Fixes #24319

Unsure what's causing the e2e test to fail?

@crisbeto Closing in favor of https://github.com/angular/components/pull/24457.
I think unit tests for _syncDescribedByIds should be a separate task/ticket, since these tests don't currently exist and this PR doesn't change its functionality?


Start.bin -> start2.bin on windows 11

Hi,
Thanks for your work.
Just want to know on windows 11 for manage startmenu it's not only start.bin in C:\Users******\AppData\Local\Packages\Microsoft.Windows.StartMenuExperienceHost_cw5n1h2txyewy\LocalState
With a recent update it's start2.bin

Do you know which Windows 11 version? 22H2, 23H2, 24H2?

hi,
The change starts from version Widows  22H2. Ans u can have a start*.bin * is a number. Windows will take into account the most incremented file.
Note the file .xml seems useless now for win 11, only .json work (on itune).
https://www.smartdeploy.com/blog/customize-default-profiles/#how-to-customize-the-windows-11-start-menu
https://learn.microsoft.com/fr-fr/windows/configuration/start/?pivots=windows-11
https://learn.microsoft.com/fr-fr/windows/configuration/start/layout?tabs=intune-10%2Cintune-11&pivots=windows-11
Sorry for bad english