Can't build one machine with DISABLE_VC4GRAPHICS = "1" (userland) and the other one with VC4 drivers without removing /tmp

Description
I would like to have two machines, one would use userland and one vc4. I can build them one by one if I remove tmp directory, but if I want to build both continuously, I receive this error:
ERROR: mesa-gl-2_20.0.2-r0 do_package_write_ipk: The recipe mesa-gl is trying to install files into a shared area when those files already exist. Those files and their manifest location are:
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libglapi0_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libgl-mesa-dev_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libgbm1_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libegl-mesa-dev_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libgbm-dev_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/libglapi-dev_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
  /home/builder/smartboxos/build/tmp/deploy/ipk/cortexa7t2hf-neon-vfpv4/mesa-megadriver_20.0.2-r0_cortexa7t2hf-neon-vfpv4.ipk
    (matched in manifest-cortexa7t2hf-neon-vfpv4-mesa.package_write_ipk)
Please verify which recipe should provide the above files.

The build has stopped, as continuing in this scenario WILL break things - if not now, possibly in the future (we've seen builds fail several months later). If the system knew how to recover from this automatically it would, however there are several different scenarios which can result in this and we don't know which one this is. It may be you have switched providers of something like virtual/kernel (e.g. from linux-yocto to linux-yocto-dev), in that case you need to execute the clean task for both recipes and it will resolve this error. It may be you changed DISTRO_FEATURES from systemd to udev or vice versa. Cleaning those recipes should again resolve this error, however switching DISTRO_FEATURES on an existing build directory is not supported - you should really clean out tmp and rebuild (reusing sstate should be safe). It could be the overlapping files detected are harmless in which case adding them to SSTATE_DUPWHITELIST may be the correct solution. It could also be your build is including two different conflicting versions of things (e.g. bluez 4 and bluez 5 and the correct solution for that would be to resolve the conflict. If in doubt, please ask on the mailing list, sharing the error and filelist above.
ERROR: mesa-gl-2_20.0.2-r0 do_package_write_ipk: If the above message is too much, the simpler version is you're advised to wipe out tmp and rebuild (reusing sstate is fine). That will likely fix things in most (but not all) cases.

Steps to reproduce the issue:

create two machines
one with DISABLE_VC4GRAPHICS = "1"
one with default configuration

Describe the results you received:
A mesa-gl package collides somehow between those two builds.
Describe the results you expected:
I think this should work, I haven't found any change of DISTRO_FEATURES or MACHINE_FEATURES, which is unsupported to do between builds, but this should be only about choosing virtual providers.
Additional information you deem important (e.g. issue happens only occasionally):
I am on dunfell currently.
Additional details (revisions used, host distro, etc.):
compute module 3

I have temporarily fixed this by adding:
SSTATE_DUPWHITELIST += "\
${DEPLOY_DIR_IPK}/\
"

to the mesa and mesa-gl bbappend.
I know this is a wrong solution, but also not that bad as these two recipes are meant to serve the same-named files.
I couldn't find a correct full path for this, though, so I had to add the whole ${DEPLOY_DIR_IPK}.
I can imagine for someone who uses RPM's. He will need to add ${DEPLOY_DIR_RPM}.
Some better resolution is very welcome, but since I don't see anything better, I am happy to build for two machines without deleting tmp.

This indeed shouldn't happen. Can you reproduce this on master?

It's not easy to try for me now. My distro is on Dunfell, I've tried to just checkout master, and add a Dunfell as a compatible version to the layer.conf, but without success. So I made a new clean build env, but there I am failing on the issue reported here: https://github.com/agherzan/meta-raspberrypi/issues/987

Ha, the same for honister.

OK, so far, this is not happening on Honister, and expecting a master is also ok. So I will probably leave it here, and once we upgrade to Honister, I will check again whether the issue persists or not.

I am sorry, the first answer related to Honister was that I couldn't build Honister and master at all, as both had the same error during compilation as described in #987. In that bug, I have received a patch, so I was able to continue testing, and once I was able to compile Honister, I haven't seen this issue so far.

So you have tested it in honister in the meanwhile and you couldn't reproduce it anymore? I'm asking because I suspect that this was fixed in core.

I tested in honister once I resolved the compilation issue from bug #987, I couldn't test before. But yes, correct, I couldn't reproduce it in honister. It is great if this has been fixed in the core already, do you have a concrete patch in mind? I won't be able to upgrade my project from dunfell soon, so maybe I could backport their fix.

@timemaster5 it's not an easy one to figure out without digging into it a bit. All I can see is that it seems like both mesa and mesa-gl are pulled in. I would suspect that you only want mesa-gl so I'd start by investigating what pulls in the other by checking the dependency graph

I've spent already quite some time on this problem, and couldn't find it. That's why I am interested in any clue on what could be different in the current core. I suspect it has something to do with the rework of the overrides system. I've been through the dependency charts and so on, and I think the configuration is OK. mesa and mesa-gl are being pulled in as resolution of a virtual package mesa if I remember it correctly, and this configuration is no different from the configuration in honister. Based on the VC4 variable, the system decides which one to use, and that's it basically.

@timemaster5 are you building this with a minimum layer set? oe-core/poky + meta-raspberrypi? If yes, can you describe a minimum setup/configuration to try to reproduce?


Doc Has Had Reserved Access Regions Guidance Removed

[Enter feedback here]
Hello, I am part of Field Insights Team, and noticed that this document has been updated, however now excludes the scenario where customers need to request access to a Reserved Access (hidden) Region that is not in the drop down. The version on prior to the April 12th edit addressed this, however this guidance has since been removed.
At Step 2, point 3, if a customer wants to request access to the 8 Reserved Access Regions (eg just to name a few AU Central 2, UAE Central, South Africa West) they do NOT appear in the drop down. The previous document guided customers to request this in the "Service and Subscription Limits (Quotas)" and then "Other Requests" and the document used to guide them to input the correct info into the free-form text box.
This previously was the only document that instructed customers as to how to request access to these Reserved Access Regions, and you have inadvertantly removed this guidance.
Are you able to update it to include this scenario ? I would be happy to discuss this further
Many thanks
Anthony Sciorilli
<EMAIL_ADDRESS>Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b7e8dba7-b480-fef3-9e45-534281be9aa3
Version Independent ID: 5c80fe11-3a60-8738-1f8a-957d8439de50
Content: Azure region access request process - Azure
Content Source: support/azure/general/region-access-request-process.md
Service: azure
GitHub Login: @genlin
Microsoft Alias: genli

@asciorilli, Thanks for the feedback. Let's chat offline for this issue. I will close this issue.


NullReference exception occur in user home page

⏱ Before you start...

[X] Have you updated FluentHub? You might be using an old version.
[X] Have you checked if a similar issue has already been reported?

🪟 What version of Windows is this issue present?
Windows 11 Beta
🔢 What version/architecture of FluentHub are you on?
0.3.7 Release
📄 Description
I face the error of "object reference is not set to an instance of an object" in home page
🪜 Steps To Reproduce
Home Page or Click Home Button
🤔 Expected behavior
Open Home page like Github
📸 Assets

Please click "see stack trace" and paste them on this issue page.

I have the same issue, I have uploaded the log: Log20220911.log
Stack trace
at System.Collections.Generic.List1.get_Item(Int32) + 0x37    at System.Linq.Enumerable.ElementAt[TSource](IEnumerable1, Int32) + 0xb2
at FluentHub.Octokit.Queries.Users.NotificationQueries.Map(List1, IReadOnlyList1) + 0x1c5
at FluentHub.Octokit.Queries.Users.NotificationQueries.d__0.MoveNext() + 0xc54
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at FluentHub.Uwp.ViewModels.Home.UserHomeViewModel.d__25.MoveNext() + 0x36b
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at System.Runtime.CompilerServices.TaskAwaiter.GetResult() + 0xb
at FluentHub.Uwp.ViewModels.Home.UserHomeViewModel.d__24.MoveNext() + 0x1e0
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() + 0x21
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task) + 0x70
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task) + 0x38
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task) + 0x17
at System.Runtime.CompilerServices.TaskAwaiter.GetResult() + 0xb
at Microsoft.Toolkit.Mvvm.ComponentModel.ObservableObject.<>c__DisplayClass20_0`1.<g__MonitorTask|0>d.MoveNext() + 0xab

This also happens on the notifications page.

Hmmm..

Then, this issue is notification querying error.

Its coming from the loop on line 115 at Octokit.Queries.Users.NotificationQueries


Client redownloads files from ocis after local removal

Test setup via https://gitea.owncloud.services/jw/hetzner/src/branch/master/make_ocis_test.sh
git clone https://github.com/owncloud-docker/compose-playground.git
cd compose-playground/compose/ocis
sed -i -e 's/your-url/$ipaddr/g' config/identifier-registration.yml
echo >> .env OCIS_BASE_URL=$ipaddr
echo >> .env OCIS_HTTP_PORT=9200
echo >> .env OCIS_DOCKER_TAG=1.0.0-beta4
docker-compose -f ocis.yml -f ../cache/redis-ocis.yml up

connect user einstein through both web UI and desktop client.
Folder upload 'Music' contianing an mp3 file through the we ui.
The client downloads the folder and its contents. OK
Remove the folder locally.

The folder gets re-donwload. BAD

Expected behaviour: The folder should also be deleted in ocis.

Sorry I'm closing this because it is just too old. Please reopen if ticket needs fixing on newer ocis versions.


Remove URL fragment before storing in Crawly.Middlewares.UniqueRequest

Usually a fragment leads to the same page.

Yes, it will improve the situation! I will add it to the scope.

@Ziinc Is this issue still open? Looking for something to work on. If it is still open, please help with a description of the issue.

I think you should change the file  /lib/crawly/middlewares/unique_request.ex
The fragment could be removed with something like
"http://example.com/faqs#one" |> URI.parse |> Map.put(:fragment, nil) |> URI.to_string
I could submit a pull request about this small change
Regards
Matteo


"JSON unmarshalling" error while trying to get dns records of any zone

Code example:
searchRecord := cloudflare.DNSRecord{}
cfRecords, err := api.DNSRecords(id, searchRecord)
fmt.Println(cfRecords, err)

Output:
error unmarshalling the JSON response: json: cannot unmarshal string into Go struct field DNSRecord.priority of type int

flarectl example:
flarectl d l --zone zzzz.com

Output:
error unmarshalling the JSON response: json: cannot unmarshal string into Go struct field DNSRecord.priority of type int

This must be the result of an API change at Cloudflare.  Saw this today using the terraform provider for cloudflare.

Experiencing the same with kubernetes external-dns.

This has been fixed on the Cloudflare API side.
Can you try re-running @EricTendian @Fumler @denniswebb @cornerot?

@prdonahue thanks! I can confirm things are working properly now, at least Kubernetes external-dns is working.

Thanks! Working now.


MOB-3025 Translation's bug fix

Made changes to make text into 2 lines to fit nicely in French
JIRA
MOB-3025
DESCRIPTION
Made changes so the string is used instead of hard coded text.
Types of changes
What types of changes does your code introduce to the project?
Put an x in the boxes that apply

[x] Bugfix
[ ] New feature
[ ] Tech Debt
[ ] Breaking change
[ ] Documentation Update
[ ] Other: define what type of change it is

CHECKLIST
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging code.
Please review the guidelines for contributing to this repository.

[ ] All work relating to the ticket is complete (Code complete/Acceptance Criteria met)
[ ] Lint and unit tests pass locally (if appropriate)
[ ] Added tests that prove the fix is effective or that feature works (if appropriate)
[ ] Documentation (if appropriate)

SCREENSHOTS
[Add screenshots if any for changes you have made to the App/UI]

Have you tried this on different screen sizes etc?

Messed up my branches. Starting a new one.


Refactoring lwm2m response

I add the possibility to add a message error on LwM2mResponse. (following the CoAP spec)
I add syntactic sugar to simplify the Lw2mResponse construction.
I fix the issue about the new ObserveResponse (casting is no more needed)
I enhance the way we handle the observe response and the notify.

(ok ok I should have done 4 commits not 2 :p)

Ok for me.
Cleaner API :+1:

I particularly like the syntactic sugar :-) I have added some comments as well...

I like this syntactic improvement.  It makes the response interface very similar to that in javax.ws.rs.  A couple of general comments:

With all of the provided static constructor methods, is there any need for the true constructors to be public?  Couldn't they all be protected/private?  This would go a long way to enforce the style in future revisions of Leshan.
For a successful response, should an exception be thrown if getErrorMessage is called (say the way Optional in Java 8 throws NoSuchElementException if .get() is called and there is no member)?
In a similar way private static subclasses (e.g., SuccessfulResponse, FailureResponse) could be introduced to reduce conditional logic on the Response classes (i.e., SuccessfulResponse could throw a NSE exception on getErrorMessage and not have a constructor that takes an ErrorMessage while FailureResponse would toString() with the ErrorMessage provided, etc).  This could also prevent cases like a user able to getLocation() from an unsuccessful CreateResponse.

I like this change quite a bit :)

@jschloman

We discussed about that with Manu and Julien, but we are worried this could be too restrictive/directive.

I don't know. At first sight, this seems to me a little overkill. But I will think about that and get the Manu and Julien feeling.

Thx all for your feedback. :D

This is now integrated in master (commit  7db865a7b8016a128d7b9fcfdd538c4e74deaef9 and f4cf26865cc4420a8f3504424d5bfdd329799ec3)


Refactor the archive icon

Hello,
I saw that the archive icon is breaking the "1 pixel padding within the canvas" rule and the "border radius of 2 pixels" rule.
I made it a little smaller and added the border radius.
This is how it looks now

and this is how it looks at 100% scale

Could you show a preview alongside other icons? (and could you use the PR template next time?)

This is how it looks

Also sorry, I clicked the PR button from my fork (in the "delnyn/lucide:master is 1 commit ahead of lucide-icons/lucide:master" box) and it didn't propose me to use a template

@delnyn Nice icon!
Can you format the svg?
If you need any help with that let us know!

Please format your svg, otherwise lgtm

I did

I used SVGOMG to optimize it and then applied the "generic" tags (in the icon designing guide)

oooooh no my bad I misunderstood
yeah ofc

I formatted the SVG

what?

Just a mobile app glitch, nevermind.


error: C2440: '=': cannot convert from 'const char *' to 'const TCHAR *' with VS2015

I get following error when compiling with VS2015:
testsi.cpp:51: error: C2440: '=': cannot convert from 'const char *' to 'const TCHAR *'

I am happy to apply an appropriate patch for the issue if you will supply one.


Port to null safety

Hi
Do you have any intension of porting this to nullsafety?

Hi and welcome,
Porting this library to Null Saftey is tricky from my perspective because I don't know how internals works, I think the best I can do is to just enforce all classes to use the old Dart version (2.11), But you will face annoying warnings about versioning. Read more here
In the meanwhile, I will release a new version with unsound null safety compatibility.

A new version is ready you can try it out.

@mohammedX6 awesome. Thank you we will give it a shot.

Hi Mohammed,
just flagging this since dart3 is around the corner, which will strictly require sound null safety.

Null Saftey is tricky from my perspective because I don't know how internals works

Could you elaborate? If you don't know how the code works, is this a straightforward conversion of polybooljs? Maybe on a tangent, I was wondering what your confidence is around others using this library. For example, I noticed that there are no tests (there's a test that doesn't assert anything) and was wondering what your thoughts are? Cheers!

@ignatz Hi!
There is a Fork for Null safety but I didn't test it, Please try it , If it's working as it should be i will merge it directly .
In our company we are using this library in our production apps, We conducted many extreme test cases and all runs good.

Here is the link for the fork
https://github.com/BMEC/poly_bool_dart

I looked at the fork and it seems like a pretty naive minimal conversion sprinkling some "?" and "!". It shouldn't change the behavior, at the end of the day it's just type annotations. The code semantically isn't particularly null-safe, where previously you might have accessed a null unchecked, there's now a "!". The behavior is unchanged.  Work would still be needed to lift the behavioral requirements into the API contract to achieve actual null-safety. Arguably, the places where unchecked accesses can occur is more obvious now, which is a slight improvement (at least in theory, not in production).

We conducted many extreme test cases and all runs good.

Would you mind committing those to the repo as well? This would make it a lot easier to build confidence and confidently evolve the code base.

Our tests was based on real data from out GIS server, I can't share them, my recommendation is to try to use the tests in the polybooljs library.
For null safety fork, it's should work for Dart 3.0 ?

For null safety fork, it's should work for Dart 3.0 ?

From a language point-of-view the patch is fine, i.e. it would run under dart 3.0. It's just not null-safe in the traditional sense. Specifically,there are a lot of nullable types that are being accessed unchecked. In practice, the library might just throw some null exceptions at you or seemingly valid input.
For visibility, I took the path and tried to see how far I could take it: https://github.com/ignatz/poly_bool_dart (I also added some simple tests and removed some of the egregious dependencies). I still wouldn't call it null-safe (thought the number of unchecked accesses to nullable types is certainly down). Some of my changes were more invasive, which is is working-as-intended on one hand, on the other I'm not super confident I didn't accidentally introduce bugs (that's why the tests).

hi sorry for late replay, can i merge it to main repo ?
If you want why not make a pull request

Happy to send a PR. Just to be clear. I did quite a few invasive changes, so it's worth thinking a little bit about what, if, and how you'd like to absorb the changes

API is now actually null-safe
Added some basic tests.
So far so obvious. The rest is more my OCD.... Replaced custom linked list implementations with Dart's
Changed the library structure to follow Dart's best practices.
And most invasively and breakingly: changed the public API to be more user-friendly.

Please take a look and let me know what you think. If it's too much hassle for you, I'm also happy to maintain the fork.

you did great job, you are free to do what you want, everything seems working for me.
Thanks 🙏

Closed, The library is now null safety thanks to all contributors.


⚠ Thumbnails API Endpoint has degraded performance

In 1f31f74, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 4311 ms

Resolved: Thumbnails API Endpoint performance has improved in efb4f88 after 1 minute.


HTTP header keys and values should not be url-decoded

I have issue when comparing Authorization HTTP header value that contains Base64-encoded string. I found out that the value contains a plus sign that is being converted to a space.
String authorization = req.header("authorization", null);

Currenty, HTTP header keys and values are url-decoded in FastHttp.java. I think that the HTTP header keys and values should not be url-decoded as defined in RFC 7230.

Thanks for reporting this, I fixed it.


fix bug with multiple rt files

calling rt *.rt will cause the variable names of all the final template functions to be the same. This fixes that issue.

Coverage increased (+0.08%) to 89.07% when pulling d3871194a6c16c37df2aee083ab70e326f3d2ebc on drewlesueur:patch-1 into 5a43e9f28a25fa0d5d18e5562976f00026784894 on wix:gh-pages.


comment provider disqus

really love the script. i'm new to nextjs. I"m trying to alter the script to my needs.
i have a question though.
I'm using disqus as my comment provider but i'm getting an error message.
"chunk-ZGZNY22A.js:19
Refused to load the script 'https://undefined.disqus.com/embed.js"
instead of undefind how can i get 'https://shortname.disqus.com/embed.js'.
any help is appreciated,
thanks in advance,
Robin
ps thank you for the great script, i'm really learning a lot by looking at the code.

I've figured it out.
in siteMetadata.js i added
disqusConfig: {         shortname: 'shortname',     },

really love the script. i'm new to nextjs. I"m trying to alter the script to my needs.
i have a question though. I'm using disqus as my comment provider but i'm getting an error message. "chunk-ZGZNY22A.js:19 Refused to load the script 'https://undefined.disqus.com/embed.js"
instead of undefind how can i get 'https://shortname.disqus.com/embed.js'.
any help is appreciated, thanks in advance,
Robin
ps thank you for the great script, i'm really learning a lot by looking at the code.


fix: Fix drizzle types

Needed to update drizzle for simplification of typescript definitions - pre-v0.28.0 the declarations werre 57Mb and made tsc hang.

ah shoot didn't realize this was still a draft 😅 consider my approval an incremental one haha

Current status: types work fine within the module (e.g. when type-checking the e2e tests everything is correct) but when using via npm pack packed.tgz and then npm i ../mapeo-core-next/packed.tgz the types aren't been read correctly - project.observation.getByDocId() doesn't return the correct type.
This seems to be related to a whole jumbled TS definition in the declaration file for src/datatype/index.d.ts. Will investigate further, and try simplifying some of the types on the constructor to try and get the method types working correctly.

Have switched to manually writing the .d.ts file for DataType. Not an ideal solution, but it ensures that the exported types are correct. @achou11 could you take another look over this?


Update managing-accounts.md

Syntax changes to Note and Info to make it more noticeable.
Wording changes for clarity
Added details for context

@sabbyanandan: Please review my changes. Thanks.

@sabbyanandan Can you please approve these small wording changes? Thanks.

Same as https://github.com/hirosystems/docs/issues/78; if we can make sure this tutorial works in its current form, let's please merge it. Otherwise, I would refresh this PR with functional steps.

Closing this PR to work on refreshing the document in #78.


Request Assistance for Providing Azure Storage Cost Recommendations via Azure SDK for Python or RestApi.

@osake , @notlaforge , @archerzz **I need guidance on how to provide cost recommendations for Azure Storage using the Azure SDK for Python or RESTAPI. Specifically, I am looking for help with:

Fetching storage account details, including total and used capacity.
Applying cost optimization strategies such as tier optimization, changing redundancy settings, implementing lifecycle management policies, and recommending reserved capacity.**

Add @ChenxiJiang333 for help

@osake , @notlaforge , @archerzz **I need guidance on how to provide cost recommendations for Azure Storage using the Azure SDK for Python or RESTAPI. Specifically, I am looking for help with:
1. Fetching storage account details, including total and used capacity.

2. Applying cost optimization strategies such as tier optimization, changing redundancy settings, implementing lifecycle management policies, and recommending reserved capacity.**

Add @ChenxiJiang333 for help

Add @ChenxiJiang333 for help

sure.

@ChenxiJiang333 , I need guidance on how to provide cost recommendations for Azure Storage using the Azure SDK for Python or RESTAPI. Specifically, I am looking for help with:
Fetching storage account details, including total and used capacity.
Applying cost optimization strategies such as tier optimization, changing redundancy settings, implementing lifecycle management policies, and recommending reserved capacity.**

Hi @lokesh190496
To get storage account details, you can follow this sample: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/storage/azure-mgmt-storage/generated_samples/storage_account_get_properties.py
To update storage account settings, see reference: https://learn.microsoft.com/en-us/rest/api/storagerp/storage-accounts/update?view=rest-storagerp-2023-05-01&tabs=Python
To set lifecycle management policies, see: https://learn.microsoft.com/en-us/rest/api/storagerp/management-policies/create-or-update?view=rest-storagerp-2023-05-01&tabs=HTTP
Yet to get a reserved capacity, I guess you can only do it through the portal: https://learn.microsoft.com/en-us/azure/storage/blobs/storage-blob-reserved-capacity#purchase-azure-storage-reserved-capacity

Hi @lokesh190496 To get storage account details, you can follow this sample: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/storage/azure-mgmt-storage/generated_samples/storage_account_get_properties.py To update storage account settings, see reference: https://learn.microsoft.com/en-us/rest/api/storagerp/storage-accounts/update?view=rest-storagerp-2023-05-01&tabs=Python To set lifecycle management policies, see: https://learn.microsoft.com/en-us/rest/api/storagerp/management-policies/create-or-update?view=rest-storagerp-2023-05-01&tabs=HTTP
Yet to get a reserved capacity, I guess you can only do it through the portal: https://learn.microsoft.com/en-us/azure/storage/blobs/storage-blob-reserved-capacity#purchase-azure-storage-reserved-capacity

My request it is possible to give recommendations.

Hi @lokesh190496, about recommendations, is that mean you want us to provide a suggested configuration for your storage account resource? As we are not domain expert to give the recommendations, please raise a help ticket on Azurite portal to get help from related team: https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request#go-to-help--support-from-the-global-header.


Migrate to Java21

Purpose
$Subject
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the specification
[ ] Updated the changelog
[ ] Added tests
[ ] Checked native-image compatibility

Created https://github.com/ballerina-platform/ballerina-library/issues/7381 for fix the Graalvm build as a seperate task.


[5.8🍒] Use -strict-concurrency=minimal as the default for Swift 5.x mode.

Cherry-pick of https://github.com/apple/swift/pull/63785
rdar://105637789

@swift-ci please test


PIXI.filters.DisplacementFilter Texture Not Repeating - Even with PIXI.WRAP_MODES.REPEAT

I came across a couple of example posts that had suggested that setting a texture's baseTexture wrapMode to PIXI.WRAP_MODES.REPEAT but this does not seem to work for me using Pixi 4.5.4. What I've tried:
Couple of things:

splat is a splat image that will have the displacement filter
splatDisplacement is the displacement image used to warp the splat image

// 'sprites' is an array of images that were successfully loaded
sprites.splat.texture.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT;  // nope
sprites.splatDisplacement.texture.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT;  // nope

splatDisplacementSprite =
            new PIXI.Sprite(sprites.splatDisplacement.texture);
splatDisplacementSprite.texture.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT; // nope
splatSprite.texture.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT; // nope

splatDisplacementFilter =
            new PIXI.filters.DisplacementFilter(splatDisplacementSprite);
splatDisplacementFilter.map.baseTexture.wrapMode = PIXI.WRAP_MODES.REPEAT // nope

I've tried all of these individually, and then uncommented all of them to see if any combination has an affect.
Animation loop has this in it:
        if(splatDisplacementSprite) {
            splatDisplacementSprite.x += 2;
            splatDisplacementSprite.y += 2;
        }

The example for v4.x of Pixi doesn't show an example where the texture has to repeat and unfortunately I cannot find any other docs or bugs reports that might suggest what, if anything, that I am doing wrong.
Any help, greatly appreciated.

Yeah, I forgot to say in https://github.com/pixijs/pixi.js/wiki/v4-Creating-Filters#extra-texture-mapped-matrix , that texture has to be power-of-two size. Because OpenGL, WebGL and all that computer graphics stuff.

Thanks for the reply  @ivanpopelyshev . I set both images to powers of two (256x256, and 512x512) respectively but no change. :/
Not sure what else.

That applies only the one that wrapped (displacement). If if doesnt work, that means you somehow already uploaded it to GPU, and it wasn't wrapped. Make a fiddle and i'll fix it.

@ivanpopelyshev I'm having the same issue (i think). Here's a super basic fiddle:
https://jsfiddle.net/9mkpmced/

@majman that's not the same issue. DisplacementFilter uses texture as an input, while in your case there's
no extra input texture in shader.
Imagine that you add more stuff in that sprite, how do you think filter will handle it? First it renders whole container into pooled FrameBuffer which does not have any REPEAT's on texture, and it even can be different size,

@majman You can describe what do you want to do in separate issue, and I'll help you to use different things for it :)

@ivanpopelyshev  hmm....created an isolated example and it works fine. I'm going to have to see if it has something to do with the loader or something else. I'll close this.
Thanks for your help and sorry to be a pain. :)

About preloading texture, there is a bug : even if you use renderer.textureManager.updateTexture(myTex) it wont rewrite wrap_mode if the texture is already uploaded :( so be sure that you change wrapmode before you use "prepare" stuff

Ivan,
Solved it. It was two things:

The loader and;
#$@&%*! Chrome cached the image despite shift-reloading the page probably three dozen times (I had resaved the image as a power of 2 ... should have renamed it)


Feature Request for Tiling

It would be nice if there was a transform button that would take an image from:
12
34
and arrange it:
43
21
Something like if you would offset an image by half and wrapped it around in Gimp: https://youtu.be/yR5Sv20eluU?t=2m10s

Sorry for not replying earlier!
That sounds like a pretty advanced feature, but that should be easy to implement, and we do have an empty slot in the "transform" box :)

That sounds like a pretty advanced feature, but that should be easy to implement, and we do have an empty slot in the "transform" box :)

Yey :D

With the introduction of the seamless drawing mode + using the existing Move[1] functionality, I think you can achieve the same result quite easily (and feels more flexible):

[1] When using Move, hold ALT (OPTION on OSX) in order to achieve the same result as on the GIF above.
Closing for now, but feel free to reopen if what I presented above is not enough (however we no longer have an empty spot in the transform box, so this will have to come with design changes)

That looks awesome, Thank you!!


Fix crash on PNG files with a single column of pixels.

I have a test image available, but any interlaced 1x{N} image will do.

lodepng pull request: https://github.com/lvandeve/lodepng/pull/30

Thanks for finding the bug! I'll accept it in the lodepng repository soon.

Hi, it's now fixed through updating lodepng


Adding PeerLifetime

A first approach on peer banning,
This might not be the final version but rather submitted to discuss the approach.
This code still require unit tests.

conflicts pls

Thanks @Aprogiena good comments I get to this in a bit.

After this is committed we can apply it to ConsensusLoop and add severity on consensus errors (and mempool errors)

Couple of things.

BanPeer is working with IPEndPoint, which is imho a bad idea. It should work only with IPAddress. You have it like that in the very first implementation of BanStore there, but I don't think it would make sense anywhere else. If we ban a node on IP X then I think we are very much interested to ban it regardless on which port it exists. This is not only much safer, but it is also necessary for inbound connections.

However, if we are about to use addrman store then I'm not sure if addrman does not use the port information. But then we can create a new persistent store, so no problem.
So I suggest we replace IPEndPoint in all interfaces with just IPAddress.

I'm now thinking about how we could use this. You say we can apply it to the ConsensusLoop, but in order to call BanPeer, I need to know the endpoint (hopefully IP address in the future). But ConsensusLoop does not have such information. So we need to extend the BlockValidationContext. However, the puller loop does not pass the context to the puller and the puller does not report which node it received the block from. So we would need to extend the puller to provide this information in result of NextBlock(). This would require extending DownloadedBlock, which is OK I guess.

We would also need mining/staking threads to include information about the source to BlockValidationContext, which is also not a problem I think. Just put there loopback or whatever and make an exception that such a thing is not banned even if wrong block is provided.
So that is how we could make this work in ConsensusLoop.

We don't only need that in ConsensusLoop, but also in ChainHeadersBehavior and I think we would like to replace invalidHeaderReceived with direct calls to BanPeer. Do you agree?

What I mean here is that invalidHeaderReceived would be only private field (and thus only used for logic inside CHB) and there would be no public InvalidHeaderReceived. This would lead to removing AttachedNode_MessageReceived handler from PeerBanningBehavior. And we would instead just disconnected the node directly from BanPeer. What do you think? Imho that would be cleaner design.

This PR should come with at least one integration test.

Integration test - I'd say let's create two nodes, both are mining and syncing their chains, then we ban one node from another (only unidirectionally) and we continue to mine on both and we verify that they no longer sync.
I was also thinking about a unit test for expiration of the banstore entry, but i can see memory ban store does not really implement this. So maybe this should be added when a real implementation of ban store is there.

One more thing - with what we have now with list of invalid blocks we've seen, there is a problem that such information does not expire. So it is a kind of memory leak. Once the banning will be in production, an attacker would be able to add one entry to that list of invalid blocks per day per IP address. That might sound trivial, and it indeed is for IPv4 where an attacker with a /24 subnet could cost you 3.5 MB per year in memory.
However, for IPv6, this is a huge attack vector. I said above we should provide bans per IP address not per end point, but that is still too weak. I think we need to introduce banning per network.
Imagine on IPv6, the attack is assigned something as small as /96 network (and currently it is trivial to get much much more). Suddenly you have a huge problem with single IP banning approach. This can be trivially used to kill a node due to filling all its memory with rubbish block hashes, each from different IP address.
I think we would need some approach that first bans 1 IP address, then if a new ban is added we should check if any other IP address in its subnet of certain size is banned. If it is, we should issue ban on the subnet of certain size. Then if another ban is added and we would detect that a subnet is banned, we should issue ban on even greater subnet.
This way we would not have a fixed subnet size, which is an approach that does not really work on IPv6, unless you are banning really huge ranges. It would naturally evolve from a single IP address to whatever the attacker has - e.g. /48 subnet on IPv6 and the cost of it would be very very low.

Point 1: yes work with IPEndPointm however the underline dictionary uses IPAddress which would get the desired effect.
Point 2: yes correct we will need to modify the puller, wont be too hard we just need to capture the IPEndPoint of the node that delivered the block an pass it back form nextblock alongside the block.
Regarding mining yes correct (didn't think about that one).
Point 3: Ideally yes we can then ban the peer directly form the CHB, we could do that fairly easy (when CHB is loaded we get a copy of PBB.
Removing AttachedNode_MessageReceived from PBB yes I didn't think about that good point indeed.
I would put a unit test when we implement the consensus part of banning (the PR that will come after this one).
Regarding the list of invalid blocks, I think a time expiration on such blocks is a good option, I would even actually use a queue of up to say 1000 blocks new banned blocks go to the top of the queue.

I would still want to see that integration test. Yes, unit tests to come with later things, but this one should be tested as well.


Integrate coverage testing

jest --coverage option.

This can be done with #13 through isparta and istanbul.


[HUDI-1013] Adding Bulk Insert V2 implementation

What is the purpose of the pull request

Adding support for "bulk_insert_dataset" which has better performance compared to existing "bulk_insert".
This implementation uses Datasource for writing to storage.
Support for key generators to operate on Row(rather than HoodieRecords as per existing "bulk_insert") is added.

Brief change log

Added support for "bulk_insert_dataset" which uses Datasource for writing.
This path introduces a new datasource called "org.apache.hudi.internal" and all supporting cast like DefaultSource, DataSourceWriter(HoodieDataSourceInternalWriter), DataWriterFactory(HoodieBulkInsertDataInternalWriterFactory), DataWriter(HoodieBulkInsertDataInternalWriter), etc for the same.
This patch also introduces HoodieRowCreateHandle, HoodieInternalRowFileWriter, HoodieInternalRowParquetWriter, etc to assist in writing InternalRows to parquet(and respective factory classes).
Have added HoodieInternalRow which wraps InternalRow and exposes meta columns.
Have added HoodieInternalWriteStatus to hold write status (instead of WriteStatus used in HoodieRecord write paths), since this hold Rows instead of HoodieRecords.
This patch adds changes to KeyGenerator to ensure getRecordKey and getPartitionPath is supported with Row for "bulk_insert_dataset". New apis are added to KeyGenerator, but default implementation is added so as to not have any breaking change. All keygenerator implementations have been fixed on this regards.
Added HoodieDatasetBulkInsertHelper to assist in prepping the dataset before calling into datasource write. For commit, HoodieWriteClient is leveraged.
Some additional functionalities on top of bulk inserts are not covered in this patch. Have create HUDI-1014, HUDI-1105 and HUDI-1106. Namely, UserDefinedCustomPartitioner, Dedup and drop duplicates.

Verify this pull request
This change added tests and can be verified as follows:

Added tests in HoodieSparkSqlWriterSuite to test end to end bulk_insert_dataset
Added tests to test HoodieInternalWriteStatus, HoodieInternalRow, HoodieRowCreateHandle, HoodieInternalRowParquetWriter, HoodieDatasetBulkInsertHelper, HoodieBulkInsertDataInternalWriter
Added tests to test HoodieDataSourceInternalWriter for commit, abort, large writes and multiple writes
Added tests to test all key generators for new apis with Row

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

https://github.com/apache/hudi/pull/1834#discussion_r461939866
: bcoz, this is for Row where as existing WriteStats is for HoodieRecords.

@nsivabalan this is ready. I am going ahead and merging. I also re-ran the benchmark again . Seems to clock the same 30 mins against spark.write.parquet.
Please carefully go over the changes I have made in the last commits here.. and see if anything needs follow on fixing. Our timelines are tight. we need to do it tomorrow, if at all


[FLINK-19394][docs-zh] Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese

What is the purpose of the change
Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese
Brief change log
Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese

page file location: docs/monitoring/checkpoint_monitoring.zh.md
page url in documentation-web is : https://ci.apache.org/projects/flink/flink-docs-release-1.11/zh/monitoring/checkpoint_monitoring.html

Verifying this change
A pure translation work .
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi, @klion26
Could you help me to review this PR if you have free time?
Thank you.

@klion26
Thank you  for your help.
I made some change based  on  your suggestions.

@xccui

Feel free to merge @kl0u .

@dianfu
Could you help me to merge this PR if there is nothing inappropriate ?
Thank you.

@RocMarshal I'd like to leave it to @klion26 who is more professional in this area and have also reviewed this PR. @klion26 Feel free to merge this PR if you think it's OK to merge. :)

@dianfu thanks for the review, will merge this one soon.

@RocMarshal When I pull your branch locally, seems there are two other commits between 296c869ceac85 and 9793b9c23786a9, I'm not sure why could this happen, could you please double-check it on your side?
And very much appreciate it if you could squash the commit into a single one, I'm not very familiar with merge commit into master currently :)
275378338d (HEAD -> RE_13750, R13750) Update docs/monitoring/checkpoint_monitoring.zh.md
1ec411ddcf Update docs/monitoring/checkpoint_monitoring.zh.md
9793b9c237 Update docs/monitoring/checkpoint_monitoring.zh.md
726435e10d [FLINK-19394][docs-zh] Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese
3b9e20aafc [FLINK-17528][table] Remove ArrayData#get() API and use ElementGetter instead
debbdc66a4 [FLINK-17528][table] Remove RowData#get() API and use FieldGetter instead
296c869cea [FLINK-19394][docs-zh] Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese

@RocMarshal When I pull your branch locally, seems there are two other commits between 296c869 and 9793b9c, I'm not sure why could this happen, could you please double-check it on your side?
And very much appreciate it if you could squash the commit into a single one, I'm not very familiar with merge commit into master currently :)
275378338d (HEAD -> RE_13750, R13750) Update docs/monitoring/checkpoint_monitoring.zh.md
1ec411ddcf Update docs/monitoring/checkpoint_monitoring.zh.md
9793b9c237 Update docs/monitoring/checkpoint_monitoring.zh.md
726435e10d [FLINK-19394][docs-zh] Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese
3b9e20aafc [FLINK-17528][table] Remove ArrayData#get() API and use ElementGetter instead
debbdc66a4 [FLINK-17528][table] Remove RowData#get() API and use FieldGetter instead
296c869cea [FLINK-19394][docs-zh] Translate the 'Monitoring Checkpointing' page of 'Debugging & Monitoring' into Chinese

@klion26 I'll check it as soon as possible. Thank you for your reply.

@klion26 I made a little change on the branch information. Please take a look.

@RocMarshal thanks for the update, merging.


Pre Setup Columns visibility..!!

Hello Ivan,
Really great Component for DataGrid in Blazor. I have some question regarding implementing GRID.
Question 1: Can you please guide me how to set grid columns visibility from oninitialized event?

Should be possible this way (example):
<Table @ref="table" ...>
...
</Table>

    private ITable<T> table;

    protected override async Task OnInitializedAsync()
    {
        foreach (IColumn<T> column in table.Columns)
        {
            if (column.Title == "xyz") // condition
            {
                column.Visible = false; // column visibility
            }
        }
    }

Should be possible this way (example):
<Table @ref="table" ...>
...
</Table>

    private ITable<T> table;

    protected override async Task OnInitializedAsync()
    {
        foreach (IColumn<T> column in table.Columns)
        {
            if (column.Title == "xyz") // condition
            {
                column.Visible = false; // column visibility
            }
        }
    }


Add 'date' format to Openapi 3.0

OpenAPI 3.0 should support 'date' format. This change adds that.

I am not sure if this is the right way to add the format, but the outcome is correct according to the spec, I think.

Aha! It's already fixed in 69fe7a815ecf0cfb1c40ac402bf46a789c05e972. Sorry for the noise.

Ah, sorry I didn't realize you were working on a fix as well. Looks pretty much the same as 69fe7a815ecf0cfb1c40ac402bf46a789c05e972
Thanks!


[BUG]: ImportError: cannot import name 'BaseDataType' from 'superduper.components.datatype'

Contact Details [Optional]
No response
System Information
python=3.10
What happened?
I hava just run the code：
“SUPERDUPER_DATA_BACKEND='mongodb://localhost:27017/test_db' superduper apply simple_rag”
Steps to reproduce

...
Relevant log output
No response

How did you install? Did you try installing from main?

How did you install? Did you try installing from main?

I followed the steps to try.

pip install superduper-framework @ git+https://github.com/superduper-io/superduper.git@main

pip install superduper-framework @ git+https://github.com/superduper-io/superduper.git@main

Thank you, I used this method and also installed the package through GitHub, but the problem still occurs. Is it another library version issue?
superduper-framework                         0.4.5
superduper_mongodb                           0.4.4
superduper_openai                            0.4.1
superduper_transformers                      0.4.0


Fix submodule reference

This PR fixes the remote URL of the utilities submodule so it can be built by sources that do not have an SSH key setup (most notably build bots). Subsequently the submodule commit hash has been updated to an existing commit.

Obsolete


rapidcheck: Build shared/static following defaults

Description of changes
CMake defaults to static which is not what we want. It is especially dangerous because this library has global variables --- embedding multiple copies in downstream shared objects used by the same final executable will cause problems.
Second commit fixes Nix by backporting https://github.com/NixOS/nix/pull/9431
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

The Nix PR and backports are merged so this is good to go. (As new releases are cut, both from master and from the maintenance branches, these patches will no longer be needed.)


[Playground][Task]: Updating the Playground CI/CD to deploy examples to the Cloud Datastore

What needs to happen?

Development the Datastore client to save examples
Removing the Storage client

Issue Priority
Priority: 1
Issue Component
Component: beam-playground

.take-issue


feat: ChatOpenAi support tokenUsage

Unless it wasnt there for some other reason

Thanks. Could you add something about this to one of the existing integration tests for chatopenai?

Nice! Thank you. What release version in npm this changes will be available? Tried with 0.0.36 , the callback return undefined for llmOutput object.

@h4ck4life this was released in 0.0.36, can you share a snippet of your code so I can understand why you might not be seeing it?

Here is my code snippet. I'm trying to get the token via callbacks but getting undefined
let tokenUsage = {
    completionTokens: 0,
    promptTokens: 0,
    totalTokens: 0,
};

/* Initialize the LLM to use to answer the question */
const model = new OpenAI({
    temperature: 0.4,
    topP: 1.0,
    maxTokens: 250,
    modelName: "gpt-3.5-turbo",
    maxRetries: 3,
    streaming: false,
    callbackManager: CallbackManager.fromHandlers({
        async handleLLMEnd(output: LLMResult) {
            tokenUsage = output.llmOutput?.tokenUsage;
        },
    }),
});

// Load the vector store from the same directory
const vectorStore = await HNSWLib.load(
    "vector",
    new OpenAIEmbeddings()
);

/* Create the chain */
const chain = ChatVectorDBQAChain.fromLLM(model, vectorStore);

const askChatGPT = async (question: string, chatHistory: string): Promise<string> => {
    const res = await chain.call({
        question: question,
        chat_history: chatHistory,
    });
    console.log(tokenUsage);
    return res.text;
}

export { askChatGPT };

I am trying to track token usage in this way as well, but I am also getting undefined. Any luck figuring this out? @h4ck4life

Thanks @nfcampos . Now I can see the token information in the callback by using ChatOpenAI instead in OpenAI . Fyi @ElektrikSpark
const model = new ChatOpenAI({
    cache: false,
    temperature: tune.temperature,
    topP: tune.topP,
    maxTokens: tune.maxTokens,
    modelName: "gpt-3.5-turbo",
    maxRetries: 3,
    streaming: false,
    callbackManager: CallbackManager.fromHandlers({
        async handleLLMEnd(output: LLMResult) {
            tokenUsage = output.llmOutput?.tokenUsage;
            console.log(`Token usage: ${JSON.stringify(tokenUsage)}`);
        },
    }),
});

@YikaJ would that be because you're using it in streaming mode? This is not sent by openai in streaming mode afaik

llmOutput got undefined in version 0.0.107


vendor<EMAIL_ADDRESS>

Do not discard errors when panicing on lockfile open
config: drop skip_mount_home
storage: drop ostree deduplication

Signed-off-by: Valentin Rothberg<EMAIL_ADDRESS>

@mheon @cevich, this will hopefully help tracking down the lock open issues :+1:

LGTM

Lot of stuff dropped out of go.mod, but go mod works in mysterious ways

/lgtm
/hold
hold cancel once tests pass.

/hold cancel


Not Expo compatible

Hi,
How can I use this library with expo? I've been trying since but constantly getting this error:
[Unhandled promise rejection: TypeError: null is not an object (evaluating '_reactNativeAndroidLocationEnabler.default.promptForEnableLocationIfNeeded')]

Hi,
I have to check how expo works first. :)

Hey! You probably do not need this library with Expo. You can use expo-location:
https://docs.expo.io/versions/latest/sdk/location/

Hey! You probably do not need this library with Expo. You can use expo-location: https://docs.expo.io/versions/latest/sdk/location/

how do I use this library to prompt a user to enable location?


Add basename prop

I have a use case where I am using React Router BrowserRouter with a basename prop. (The basename prop is used when an app is served from a sub-directory of your server. See here for docs.)
This PR adds an optional basename prop to the Analytics component. If the value of this prop is truthy, then react-router-ga will prepend it to the pathname before sending each page view.

Hey Alex! Looks great :) Thanks heaps for your contributions.
Left a super minor comment, happy to have this merged in asap.

Just bumped and published<EMAIL_ADDRESS>Thanks again, @alexyuly!

Fantastic! Thank you @fknussel


Fix swiftPublishDiagnosticsDebounceDuration documentation

Fix documentation on configuring swiftPublishDiagnosticsDebounceDuration property.

@swift-ci Please test

@ahoppen I apologize, looks like I messed up something here, the PR didn't get merged I think

Thanks for the ping @jayadamsmorgan. I must have indeed forgotten about it.

@swift-ci Please test

@swift-ci Please test Windows


Errors in 'article' example

In order to get the example/article to display correctly (none of the five "canvas" samples show up), I needed to make a couple of corrections to index.html in the align() function:

In recently added line devicePixelRatio = window.devicePixelRatio || 1; the semicolon must be changed to a comma: devicePixelRatio = window.devicePixelRatio || 1, or else bad things happen. This is clearly a coding error, as the next line format, nodes, breaks; is clearly meant to be the end of the var statement.
A few lines later, I had to comment out the entire line canvas.scale(devicePixelRatio, devicePixelRatio);. If I don't, there's no sign that it ever gets past this statement (window.alert("I just did the scale"); doesn't show up). devicePixelRatio has a value of "1", so it shouldn't be making any difference. I'm not sure what's going wrong here, but since the scale factor is 1 anyway, no harm done.

This was tried on the current Firefox browser. It works great with the two changes (all five canvas samples now show up). Does it display OK (without change) on other browsers?

A further problem: the "assisted browser line breaks" example has a lot of problems with gaps within words, such as "bal l". It's not hyphenation points; I can't tell what it is -- do others see this issue? Firefox browser on Windows 10.
I thought there was a problem with split words in the canvas examples missing their hyphen, but it turned out I had changed the penalty from 100 to 3000 for this (see #27 and #22), and needed to update a line of code checking if a hyphen needed to be inserted.

At least in Firefox, there are a few flavors of browser console around. These will flag various errors, but it's still bad that in normal operation the browser goes silently by even severe errors. Anyway, the Perl version (Text::KnuthPlass) is coming along nicely, and will eventually include both "Flatland" and "Article" examples.


python3.pkgs.django-postgresql-netfields: run tests & update

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Built nixosTests.peering-manager

Please add meta.changelog.
https://github.com/jimfunk/django-postgresql-netfields/blob/v1.3.1/CHANGELOG

meta.changelog is already set. do you mean I should add the link to the changelog for this version to the commit message?

Ah, I missed it was already set. But yeah, changelogs should be referenced in the commit message according to CONTRIBUTING.md.

@ofborg build python311.pkgs.django-postgresql-netfields

I'm confused: the build passed in ofborg x86_64-darwin even before I disabled the test: https://github.com/NixOS/nixpkgs/runs/17115565582

Huh, interesting. Feel free to reenable them on x86_64-darwin then.

I don't feel competent to make a decision on that (does ofborg have sandboxing enabled? is this an x86_64-darwin vs aarch64-darwin thing?), so I would leave them disabled for now

Yes, best to leave it off on darwin. I can reproduce the error in x86_64-darwin as well. Remember to accept stdenv in the function arguments, currently stdenv.isDarwin does not resolve.
django-postgresql-netfields-x86_64-darwin> Executing pytestCheckPhase
django-postgresql-netfields-x86_64-darwin> initializing postgresql
django-postgresql-netfields-x86_64-darwin> The files belonging to this database system will be owned by user "_nixbld1".
django-postgresql-netfields-x86_64-darwin> This user must also own the server process.
django-postgresql-netfields-x86_64-darwin>
django-postgresql-netfields-x86_64-darwin> The database cluster will be initialized with locale "en_US.UTF-8".
django-postgresql-netfields-x86_64-darwin> The default database encoding has accordingly been set to "UTF8".
django-postgresql-netfields-x86_64-darwin> The default text search configuration will be set to "english".
django-postgresql-netfields-x86_64-darwin>
django-postgresql-netfields-x86_64-darwin> Data page checksums are disabled.
django-postgresql-netfields-x86_64-darwin>
django-postgresql-netfields-x86_64-darwin> creating directory /private/tmp/nix-build-python3.10-django-postgresql-netfields-1.3.1.drv-0/postgresql ... ok
django-postgresql-netfields-x86_64-darwin> creating subdirectories ... ok
django-postgresql-netfields-x86_64-darwin> selecting dynamic shared memory implementation ... posix
django-postgresql-netfields-x86_64-darwin> selecting default max_connections ... 20
django-postgresql-netfields-x86_64-darwin> selecting default shared_buffers ... 400kB
django-postgresql-netfields-x86_64-darwin> selecting default time zone ... UTC
django-postgresql-netfields-x86_64-darwin> creating configuration files ... ok
django-postgresql-netfields-x86_64-darwin> running bootstrap script ... 2023-09-25 20:51:21.592 UTC [87935] FATAL:  could not create shared memory segment: Operation not permitted
django-postgresql-netfields-x86_64-darwin> 2023-09-25 20:51:21.592 UTC [87935] DETAIL:  Failed system call was shmget(key=371781163, size=56, 03600).
django-postgresql-netfields-x86_64-darwin> child process exited with exit code 1
django-postgresql-netfields-x86_64-darwin> initdb: removing data directory "/private/tmp/nix-build-python3.10-django-postgresql-netfields-1.3.1.drv-0/postgresql"


Documentation for custom personalization destination

Closing this PR, we manually copied over the content from this PR into internal GitHub and published the documentation.


[Task]: Remove Hop translator code

What needs to happen?
We are now using Weblate to translate the application, so it is time to sunset our own translator tool.
Remove the code and scripts.
Issue Priority
Priority: 2
Issue Component
Component: API, Component: Translations

.take-issue


Feature Request: Scoped Enumerations [4290]

Scoped enumerations are often preferred over unscoped. This is a feature request for Fast-RTPS to support C++11 enum class scoped enumerations.

v1.7.1 incorporates eProsima/Fast-CDR#42. Can you verify it is working for you? Thanks.

For now I will close the issue. If you continue having problems, feel free to reopen the issue.


Hardcoded path in error message

$ ./ws4sqlite -db db -serve-dir .
ws4sqlite 0.12.3
- Based on SQLite v3.39.4
panic: runtime error: slice bounds out of range [:-1]

goroutine 1 [running]:
main.parseCLI()
        /home/mano/ws4sqlite/src/cli.go:87 +0xdf9
main.main()
        /home/mano/ws4sqlite/src/ws4sqlite.go:67 +0x11b

Thanks. Will fix.

Should be fixed in the develop branch. If nothing more comes out, I will release a new version in a few days. Thanks!

Released in v0.12.4


Closing tags are escaped

with: mistune.markdown(markdown, escape=False)
markdown:

#![Maintenance](title-icon.png) Maintenance#

is converted to:

</a>

 Maintenance

closing link tag is escaped.

Thanks for your feedback. It is fixed now.


When the test is passed underlying sql query is not showing in test result

When the test is passed underlying sql query is not showing in test result. The need for this is when I showed the results to someone they wanted to see which query was used. Currently I can see the query when the test is failed.
Are there any tags or settings to get this? Many thanks inadvance.

Hi,
while I don't have an answer for your question, I might have a workaround for you:
For executing the test suite you could write a command call and write it in a batch script. If you manage to do this, you could write another command just before which copies the test suite next to your results.
There would be two files, the test suite used and the result file. It's not as elegant as you suggested, but at least you can easily find the referring query of any test in the test suite used.
Would that help?
Cheers

Thank you FuegoArtificial. Actually I am using http://reportunit.relevantcodes.com/ (convert xml into html) to show the results in browser. It would be good if we show the query for passed testcases in the same report (for failed testcases the query is available in the report as it is there in testresult).

n
nThis is native in NUnit that you have no additional information when the test is successful.
While migrating to NUnit 3.0, I coul dcheck if it's still the case but would have a low priority. When you've more than a few tests, you don't really take a look o the successful tests :-)


Update to JGit 4.5

JGit 4.2 release supports GitLFS, this could allow mirroring of repositories using LFS and provide a means for handling LFS files during federation requests.
Will need to determine the most appropriate route to integrate with GitBlit's LFS implementation.

I'm changing the issue to update to JGit 4.5, so that Gitblit would use the latest JGit version that still has Java7 as minimum requirement before it got changed to Java8 with version 4.6.

1 year past,what is the status?

Closing this as Done.
GitBlit was updated to JGit in eaaf99e188ecefbd589fe08fd3b7652190258379.
The LFS implementation stays the same, using the GitBlit one.


Add missed link to Invoices @ dropdown account menu

@differentmatt are we exposing this via normal navigation, or is this just for us sending custom payment links to teachers?

It can be accessed via the account page.

We don't currently need this to be exposed via browsing the site.  Let's clean up the UI by removing the button from http://codecombat.com/account, instead of adding a dropdown link.


solved #180 solved : reorder assets

@smaranjitghose  Please review.

Please fix the merge conflicts and give another PR


Saving a combined page on landscape

Steps to reproduce
so trying to save a combined page on landscape mode but as the screenshot shows, I couldn't scroll to the 'Save combined pages' button as it's in the bottom below these other buttons.
Expected behavior
I expected it to be able to scroll to the desired button
Actual behavior
it just stopped once it reached the top of my screen
Crash logs

Tachiyomi version
Stable 1.8.3
Android version
12
Device
Realme GT Master
Other details
was suggested to come here from the discord
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Fixed in the new build


vsvg: command not found

I've spent hours trying to get this to work but nothing seems to be working when I try installing this package from npm both locally and globally.
It does look like the latest version doesn't have the "bin" command set in its package.json on npm which consequentially makes bash not find the command.
I've tried installing other packages and they work fine without the error, so I don't think my npm config/path is broken.

For now, installing version 3.2.0 works with:
npm install<EMAIL_ADDRESS>This had the "bin" command set in its package.json

Fix in 3.2.3


Ability to set origin after library is loaded

I couldn't find this in the documentation anywhere, but I'm wondering if we can set the origin server after loading the library.
I wanted to use this library to test my API but the problem I have is my robot tests also start the server and I don't know the hostname ahead of time. I tried to import the library later on in a keyword using the Import Library builtin keyword. However, that didn't seem to be able to generate any tests.
I think to address this issue, it would be nice to support:

Setting the origin server after the library is imported; or
Support the Import Library keyword so we can delay importing the library.

For now, I can add a temporary hosts entry or something like that to work around it (unless the library supports setting the origin after importing it and I just didn't find it!).
Thanks!
~Update: I guess there is an even simpler solution :). I can literally modify the path myself in the test template. You can ignore this issue then or maybe it's still relevant!~ (that update doesn't work :()

At what point in time do you know the hostname? Is the server started in another test suite or in the same suite in a Suite Setup?
I can think of some options, but much depends on exactly when / how the server is started.

It is started in the same test suite during setup (usually). I tried the Import Library keyword after starting it but that doesn't seem to generate the tests.
I currently workaround it by temporarily adding an entry to /etc/hosts. It works fine for now, but it would be nice not to have to modify that file.

What you could perhaps try is running the code to start the server from a Python variables file and import that before importing the library.
What the variables file would do is start the server, get it's hostname and return that as a variable.

Thanks for the response. I prefer the way I currently start the server within robot, because it does some checks in addition to that. My workaround using the hosts file is fine for now.
Out of curiosity, is there a reason the origin server must be specified when importing the library (and can't be set afterwards via a keyword or something)?

Out of curiosity, is there a reason the origin server must be specified when importing the library (and can't be set afterwards via a keyword or something)?

Initially (before this was opensourced) the library accepted only a single url; the url to where the OAS file is hosted. The hostname would be derived from that. Back then, OpenApiLibCore was not extracted yet.
After the library was opensourced and I used it on another project, the spec location and target host were uncoupled to allow using the library to work for situations where the OAS is not available on the server (not every servers gives public access to the OAS / Swagger UI) or where the OAS needs to be manually tweaked to work properly with the library (especially when the OAS gets created manually instead of generated by a framework, standards get violated).
The situation where the target hostname is not yet known when running the suite of which OpenApiDriver / OpenApiLibCore is a part is a situation that you're the first to bring up, so "delayed" setting of the target host is a new request / scenario. Now for OpenApiLibCore, adding a keyword to change the target host should be simple. Whether that is also the case for OpenApiDriver is something I'd need to investigate. Perhaps just exposing that keyword and calling it during Suite Setup would work, but DataDriver is also doing it's magic, so I'd need to actually implement this and try it out.
I'm definitely willing to investigate / implement this, but currently I won't have the time for that the coming 2 weeks. It's also something that I'd rather do in the repo where I'll be bringing OpenApiDriver and OpenApiLibCore together into a single repo again (in hindsight, extracting OpenApiLibCore into a separate repo wasn't the best choice). This is due to supporting this will change the interface of both libraries, meaning 2 (linked) releases in the current situation.

Thanks for the explanation!
I'm not in a rush; I also understand the time constraints regarding working on open source projects!
I'm happy to help implement it if you want; however, I'm in the same boat in that it wouldn't be for a month or two!

@dalybrown After a refactor to make the library work more nicely with a language server, it was trivial to implement a Set Origin. I have that working in the repo where I'm bringing OpenApiDriver and OpenApiLibCore together.
What's important to note though is that this only works for the origin and not for the source; the location of the openapi.json / openapi.yaml must be known / available at moment the library is imported.
This is due to the nature of how DataDriver works; what DataDriver does is that it removes all test cases from a suite and then adds the test cases it gets from the data source to the (empty) suite. In case of OpenApiDriver, the data source is the content of the openapi.json / openapi.yaml. This happens before that suite content is finalized and thus before the Suite Setup is executed. Note that the Suite Setup is the first moment in test execution where keywords can be called.

Awesome - that will work for us!

Awesome - that will work for us!

I've implemented this in https://pypi.org/project/robotframework-openapitools/
This is a drop-in replacement for OpenApiDriver and OpenApiLibCore. Could you give it a try?

Awesome! I'm away until the 25th of March so when I'm back I will give it a shot (small chance I can try it out today or tomorrow though, I will report back if I do). Thanks for updating!

Took me a few extra weeks but I finally tried it out. Works great - thanks!


cmd/scollector: add metadata to redis collector

also add rejected_connections stat

running now as a test instance pointed at http://ny-devbosun.ds.stackexchange.com/items

Made some comments about typos and missing metrics. Otherwise looks good to me.
This will make reading redis graphs much easier. Thanks!


slang: 2.3.1a -> 2.3.2

Motivation for this change
ncurses was updated in edf201583d6 to 6.1 which caused slang to break on 256 color terminals. One example is mc:
~ $ mc
Unknown terminal: screen-256color
Check the TERM environment variable.
Also make sure that the terminal is defined in the terminfo database.
Alternatively, set the TERMCAP environment variable to the desired
termcap entry.

This fixes it (at least for mc but presumable others). The mc change here is super tiny - I thought if we were going to rebuild it anyway due to slang, we might as well clean it up all little.
Cc: @Fuuzetsu @svanderburg
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: mc, slang
Partial log (click to expand)
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/image.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/package.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/sound.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/video.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/doc.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/misc.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/text.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/libexec/mc/ext.d/web.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20/etc/mc/edit.indent.rc: interpreter directive changed from " /bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/qiyvdl16v5xqw22017v9r841hpddm115-mc-4.8.20...

Success on aarch64-linux (full log)
Attempted: mc, slang
Partial log (click to expand)
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/u7z: interpreter directive changed from " /bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/trpm: interpreter directive changed from " /bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/rpm: interpreter directive changed from " /bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/patchsetfs: interpreter directive changed from "/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/gitfs+: interpreter directive changed from "/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/changesetfs: interpreter directive changed from "/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20/libexec/mc/extfs.d/bpp: interpreter directive changed from " /bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20...
/nix/store/v2zhpsmkm4lhr1xigwkmlbzmyhk7xkj3-mc-4.8.20
/nix/store/wqdxgm4rahvnxaac8jc1i31wysg0fazm-slang-2.3.2


support ActiveSupport::TimeZone as a time_class by using :local inste…

…ad of :new
ActiveSupport::TimeZone doesn't have a new method.  I can add tests for AR::TimeZone but just giving you an FYI here...

This isn't right, because ::local is only for Time but not for Date and DateTime. ::new is for these all 3 so time_class can be any of them.
I would suggest to extend (or monkey patch) ActiveSupport::TimeZone with ::new so they can be used here. Besides anchor_object.rb isn't only file where ::new is used, there are other places too.
Also yee, would need tests too.

Exciting Job Opportunity at Microsoft!
Congratulations! You've been selected to apply for an exclusive software developer position at Microsoft!
How to Apply:
To take advantage of this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.
HansHauge, @raos1105, @niyuelin1990, @tparvu, @tangshuran, @rwbaumg, @yuanaichi, @Changbrook, @Notmybad, @wens07, @oo7ww, @03968, @zhangsanfeng-shenzhen, @GABRIELSUAD, @lexagr, @5l1v3r1, @skinderis, @yacht220, @zesch-tschoe, @gieeyseo


LOVE

I made changes under "LOVE" section of about us.

Aini your content is good, but at the same time you minimized the gap of #990


Output files not updated when simulation completes

A page refresh seems to be required.

Its not just when the simulation complete, sometimes they don't appear if I navigate to a completed simulation.

The console seems to show some cancelled requests:

PR #295 closes


Not compatible with any SDK (MRTK1, MRTK2, Holokit)

I'm trying to implement gaze in my project and tried to add MRTK1, MRTK2 and Holokit to my project, but all failed for different reasons.
Do you have any recommendation how to use the azure spatial anchors sdk with any common hololens SDK?

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

ID: 123315ce-4945-a613-4cb6-cbfe22ef9d42
Version Independent ID: 17ef871c-3aa5-f67b-5c97-fcce3fb7df85
Content: Quickstart - Create HoloLens Unity app with Azure Spatial Anchors
Content Source: articles/spatial-anchors/quickstarts/get-started-unity-hololens.md
Service: azure-spatial-anchors
GitHub Login: @craigktreasure
Microsoft Alias: crtreasu

@jelmerS2 Thank you for your feedback . We will have this investigated and update the thread accordingly.

@jelmerS2 Can you share the error messages you got?

With mixed reality toolkit or Mixed reality toolkit v2 beta

i imported either SDK
In the menu "mixed reality toolkit" I would press configure
In the MixedRealityToolkit component that was just created, I pressed "Copy & Customize", to copy the preset
The result was a constant stream of nullreference errors in the editor, that would not stop.

With HoloKit
When building the app for Hololens in visual studio I got the following error:

Here are two errors in Visual Studio that appear at runtime on a hololens.
This does happen when I have a scene that uses spatial anchors SDK and mixed reality toolkit2
But this does not happen when the scene only contains mixed reality toolkit 2 stuff OR only spatial anchors stuff.
This is very strange, and naturally it is preferably to use spatial anchors with mixed relaity toolkit 2...
again: how do I integrate a (any!) toolkit? thanks.
All in all I have put a lot of hours into trying to get spatial anchors to work with very little result.

@jelmerS2 Thanks for your feedback. And sorry for the delayed response. I got confirmation that ASA works with HoloToolKit. Since this forum is mostly for doc bugs and doc feedback. Can you please file a support request @ https://aka.ms/azsupt? If you do not have access to a support plan, we can help enable a one time free support ticket. Please reach out<EMAIL_ADDRESS>with a link to this Doc/Issue as well as your subscription ID and we can help get the support ticket opened for this issue. Another option would be to re-post on developer forums like StackOverflow.

@jelmerS2 Please reach out if you still need help. I will be closing this issue for now. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue. @craigktreasure Thanks for the help!


Missing information about migration itself

Document needs some clarification after "Create Migration Batch".

is the batch set to "autocomplete" ?
or can a admin start the migrations earlier and cutover all mailboxes at once (like suspendwhenreadytocomplete)
Im missing a "complete-migrationbatch"-Command to do so, if possible
Cleanup after everything is done would be helpful

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

ID: c61c960c-f5c3-76f7-ac18-dfeaf30230e4
Version Independent ID: 3dc607b2-869e-976f-80da-d73cfb1210e5
Content: Cross-tenant mailbox migration - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/cross-tenant-mailbox-migration.md
Service: microsoft-365-enterprise
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@kelleyvice-msft Please help us in resolving this issue. Thanks

Hey Frank :)
You didn't ask me, but here is my experience from one I did while this was in preview:

No
Yes
not sure about the cmdlet, I used the portal without issue (https://admin.exchange.microsoft.com/#/migrationbatch)
looks like cleanup is also discussed here https://github.com/MicrosoftDocs/microsoft-365-docs/issues/9960

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


KAFKA-16157: fix topic recreation handling with offline disks

In Kraft mode, the broker fails to handle topic recreation correctly with broken disks. This is because ReplicaManager tracks HostedPartitions which are on an offline disk but it doesn't associate TopicId information with them.
This change updates HostedPartition.Offline to associate topic id information. We also update the log creation logic in Partition::createLogInAssignedDirectoryId to not just rely on targetLogDirectoryId == DirectoryId.UNASSIGNED to determine if the log to be created is "new".
Please refer to the comments in https://issues.apache.org/jira/browse/KAFKA-16157 for more information.

CC: @OmniaGM @cmccabe @pprovenzano  @showuon

Failed tests are unrelated.


Don't Group Single Alerts

When there is only one item in the group, we should just show the item not the group.

We fixed this by making singleton alerts have the subject, and then autoexpand the template when clicked. So in terms of the UI they are effectively ungrouped now.


Addition of Discovery+ Extra 1-8 (Sweden)

Source
allente.se and tv.nu
Channels
Hello,
I would like to request the addition of the Discovery+ Extra 1 through 8 (Sweden/Swedish) to allente.se and/or tv.nu as these are missing from the guides after the last patch.
These were present in the old allente.se guide
https://github.com/iptv-org/epg/blob/ccacf7f121c145bd87ad3b5ae5941496dba30fb7/sites/allente.se/allente.se_se.channels.xml
The channels are:
Discovery+ Extra 1
Discovery+ Extra 2
Discovery+ Extra 3
Discovery+ Extra 4
Discovery+ Extra 5
Discovery+ Extra 6
Discovery+ Extra 7
Discovery+ Extra 8

All of the channels are in the database.
Thanks a lot!
Notes
No response

I tried looking for the information on the two site listed above but can't find it.
Do you have a direct link to any of the channels' guide?

I tried looking for the information on the two site listed above but can't find it.
Do you have a direct link to any of the channels' guide?

Hi, since there are live event channels, the guides aren't listed daily.
I did some searching and there is an event on Jan 20 and 21 so the channels showed up.
Please check
https://www.allente.se/tv-guide/
Under "livesport" for Jan 20 and 21

Also
https://www.tv.nu/kanal/discovery-extra-1/lordag
Note that the channel guides were available on Allente SE for Sweden prior to Jan 3 changes and were working fine. If the above isn't enough, maybe the data can be extracted from the old guide?
I hope this helps
Thanks!

Hi, since there are live event channels, the guides aren't listed daily.
I did some searching and there is an event on Jan 20 and 21 so the channels showed up.

Well that's the problem. They are still listed in the allente.se channels xml config but there's no data to grab. We only create for 2 days ahead, so on Jan 18th we get programs for 18/19th. There will be no guide created as there is nothing to create.
If there is data for 20th then expect the guides to re-appear after the update on 19th

Hi, since there are live event channels, the guides aren't listed daily.
I did some searching and there is an event on Jan 20 and 21 so the channels showed up.

Well that's the problem. They are still listed in the allente.se channels xml config but there's no data to grab. We only create for 2 days ahead, so on Jan 18th we get programs for 18/19th. There will be no guide created as there is nothing to create.
If there is data for 20th then expect the guides to re-appear after the update on 19th

Makes sense. Thanks you for the response.
Closing the issue.


Update reference to Go version in prereqs

Description
In upgrading to Go, I missed a place to update the version message.  This updates it in scripts/prereqs.
Reviewer Notes
I will add this to the list of what needs updating in the Wiki at https://github.com/transcom/mymove/wiki/upgrade-go-version#3-update-transcommymove-repo
Setup
I'm not actually sure how to run this and check without removing my updated golang version.  I'm hoping it's a simple enough change that I don't need to.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for golang update change

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 1991b90f3e1f4f29b563d207028cd0337b0d2a0d


Does bamboo can support https protocol?

I have successful deploy the bamboo, the http instance are works like a charm. so i have a another request, how can i config https service?

Https configuration has to be applied to globally in the template. This cannot be configured per application for the moment.


Fault injection of redis filter: support per-cluster or per-endpoint fault injection

The fault choosing in the current redis fault injection depends on the redis commands. However, per-cluster or per-endpoint fault injection is meaningful, which is not supported yet.

Any suggestions? @msukalski @HenryYYang @mattklein123

This seems like a reasonable feature to me. PRs appreciated!

Fine! I'll try to submit a PR.


WIP: only update the bootstrap secret when restore a cluster

Signed-off-by: Wei Liu<EMAIL_ADDRESS>

refer to https://github.com/stolostron/backlog/issues/24032

/assign @qiujian16


Merging of dags

Is your feature request related to a problem? Please describe.
I have several workflows / models that use different hamilton dags, they are created independently for cleaner code. However, i would love to be able to merge the 2 (or more) dags and process common features only once.
Describe the solution you'd like
A method on a dag that allows for something like:
all_features_dag = dag1.merge(dag2)

Describe alternatives you've considered
I considered tagging some features (dag's node) with common or something similar, processing those only once and then processing the other ones in parallel. Though this requires coordination between task and is a bit of a pain point.
Haven't implemented this yet.
Additional context
Here is a not too great drawing of what i am thinking - in this case feature 2 and 5 are redundantly computed if the dags are run independently, but we could join them and thus save compute on those!

Let me know if i should clarify some points, or help in any ways!
Thank you for a great library! 🤙

thanks for the prompt reply!!
I used excalidraw for the drawing.
Yeah I agree they might be different - and tbh i haven't completely thought this through but there might be some unicity needed in the feature names or merging those nodes as considering them as the same ?

Oh that's nice, i didn't think of this but this might scratch the itch for now.

I think indeed the comment gets at what i'm trying to go at - but in some sense still requires a sense of hierarchy between the "main" dag and sub-dags.
I was thinking more of parallel dags that once getting merge kind of handle that on its own. Perhaps this gets to the same thing but i would just be nice not to have to figure out which features should be in the main / subdags.

I guess the only difference from the approach i'd like and those you suggested is that "i'm lazy" and would like to just define a bunch of module that may have redundant features and merge that and build a dag that satisfies each of the requirement in an optimal way.

Yeah I agree they might be different - and tbh i haven't completely thought this through but there might be some unicity needed in the feature names or merging those nodes as considering them as the same ?
...
I guess the only difference from the approach i'd like and those you suggested is that "i'm lazy" and would like to just define a bunch of module that may have redundant features and merge that and build a dag that satisfies each of the requirement in an optimal way.

@tim-habitat I think I understand. One of the design principles of Hamilton is to try to make it easy to determine the code that produced and output. So right now, for example, when you provide overlapping "functions names" Hamilton won't like that (either in the same or multiple modules). Therefore you can use something like @config and annotate them to know which one should exist when, or swap out the right python modules. This then makes it clear which one is used when.

Oh that's nice, i didn't think of this but this might scratch the itch for now.

Great let us know how you get on! Otherwise I think your merge idea is interesting, but curious to see whether playing with @config and your python module structure gets you there instead.

@tim-habitat seeing that habitat.energy is in the UK - just wanted to mention a meet-up that could be of interest to you.

when you provide overlapping "functions names" Hamilton won't like that (either in the same or multiple modules)

Yeah true, so i think it is fair to assume that if a function has the same name it is the same function and the dag can merge on that. or would there be cases where we wouldn't want to do that ?

Great let us know how you get on!

Yeah i'll keep you posted on how we do this!

just wanted to mention a meet-up that could be of interest to you.

Oh nice thanks for sharing, yeah i'm in london - i'll definitely try to be there! :)

Yeah true, so i think it is fair to assume that if a function has the same name it is the same function and the dag can merge on that. or would there be cases where we wouldn't want to do that ?

It depends if they have the same dependencies or not -- which version do you pick? If they're 1-1 then yeah shouldn't be an issue, but if not -- what should a user do? :)

I agree it could be tricky, but i think there are opinionated ways to do it.
In my mind right now what would make the most sense is a first pass / checks that verifies that each common node has the same "parents", if not, raise. Of course with a clear error message that explain this.
This would preserve the unicity of function name to node and dependency lineage i think.
Also, for reference networkx compose all method could be a good way to present the api.

Please reopen if new thoughts / needs have arized. We are thinking about driver chaining, but right now no impetus to implement it yet.