38 Enhance UI and user experiece.

Updates to SimulationScreen, StatisticsScreen & Graph.
Graph:

The background of the legend of the graphs is now white to ensure the legend is always readable.

SimulationScreen:

The layout now shows the current day more clearly
Our logo is now displayed in the UI
The speed of the simulation can now be adjusted.

StatisticsScreen:

A final graph is now shown
The graph can be exported as JPEG
The simulation can be replayed
The existing parameters can now be modified to make it easier to see small changes
The simulation end reason is now displayed along with a summary.

thanks 4 merge


fix: Clean non-API-specific files when configuring a new API

Fixes #63

(This won't help until https://github.com/googleapis/google-cloud-dotnet/pull/14012 is merged and a new container built, mind you.)


Add mason bee blocks?

http://www.nativebeeconservancy.org/projects/pollinator-houses/
https://thegardendiaries.wordpress.com/2015/03/21/home-sweet-home-providing-the-perfect-habitat-for-mason-bees-2/
http://blog.uvm.edu/wagn/2014/04/18/keep-your-farm-buzzing-3-ways-to-encourage-native-pollinators-this-year/

Could provide a passive boost to plant growth in an area around them. Not sure what the crafting recipe should be.
Tangentially related to #42

Actually, this could be a roundabout way of implementing (part) of #42: native bee blocks could passively grow/spread flowers/tall grass a la Forestry, which could then be harvested and composted.

This could work, maybe the boost increases with flowers around the hive.
It could also have integration with Harvestcraft, Growthcraft, and Forestry, since they all have bees.
The recipe could be two logs, and five wood slabs.

Archives of dead links in OP:

https://web.archive.org/web/20170320171308/http://www.nativebeeconservancy.org/projects/pollinator-houses/
https://web.archive.org/web/20161104153515/https://thegardendiaries.wordpress.com/2015/03/21/home-sweet-home-providing-the-perfect-habitat-for-mason-bees-2/

In addition to flowers, this could be a way to propagate and passively fertilize sweet berries. Maybe we could have our mason bees sometimes propagate melon and pumpkin seeds (if such plants are nearby) to grow wild crops, harvestable apples that grow on oak trees. When paired with the Spice of Life and its forks this could be a good way to passively farm a variety of vegan foods/ingredients.


emojis not loaded

hi
Installed this plugin, but it fails to load the emojis unfortunately.

My config:
WebStorm 2023.2
Build #WS-232.8660.143, built on July 21, 2023
Licensed to xxxxxxxxxx
Subscription is active until xxxx.
Runtime version: 17.0.7+7-b1000.6 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 4060M
Cores: 16
Non-Bundled Plugins:
CMD Support (1.0.5)
org.jetbrains.erlang (0.11.1205)
com.intellij.ideolog (<IP_ADDRESS>)
org.sonarlint.idea (<IP_ADDRESS>538)
com.intellij.properties (232.8660.88)
de.achimonline.github_markdown_emojis (1.3.0)
com.intellij.database (232.8660.129)

The images/icons are fetched from the internet.
Does your workstation/IDE have access to web?
e.g.
https://github.githubassets.com/images/icons/emoji/unicode/1f604.png?v8

yes access to the web (but using corporate proxy)
web access works fine because was able to install the plug-in from the Webstorm plugging manager, and Webstorm upgrade is working fine too

Okay.
The screenshot you provided basically shows the integrated "webpage" which pulls the images from the internet.
Maybe your proxy is blocking access to these resources?!
Do you get the image if you put the example-url from above into a webbrowser on your workstation?

Yes I'm able to access the resource

I'm sorry. I have no other idea/clue why images are not loading on your end.


feat: delete account UI

What does this PR do?
add the delete account UI

Fixes # (issue)
#32

How should this be tested?

Test A
Test B

Checklist

Required

[ ] Filled out the "How to test" section in this PR
[ ] Read How we Code at oss.gg
[ ] Self-reviewed my own code
[ ] Commented on my code in hard-to-understand bits
[ ] Ran pnpm build
[ ] Checked for warnings, there are none
[ ] Removed all console.logs
[ ] Merged the latest changes from main onto my branch with git pull origin main
[ ] My changes don't cause any responsiveness issues

Appreciated

[ ] If a UI change was made: Added a screen recording or screenshots to this PR
[ ] Updated the Formbricks Docs if changes were necessary

hey @Chigala
wanna also add the delete functionality?
we are working with the same tech stack with Formbricks, you can have a look at how we did it here:
https://github.com/formbricks/formbricks/blob/main/apps/web/app/(app)/environments/[environmentId]/settings/profile/actions.ts

@jobenjada I think this is ready for review. Tell me what you think about the tables I'm deleting.

/award 50

/award 100


feat: add localization editor and ide extension

Hi together, let me know what I need to change in order to merge this

Added a localization editor & ide extension for better i18n handling

Faster message extraction
Continuous i18n process improvements
Improved translation status

Editor -> Fink Translation Editor (Open Editor)

IDE Extension

What I added

Added project.inlang/settings.json file to configure the inlang project.
repo id file was generated by the SDK
Added the extension to the vscode recommendations, so every contributor con benefit

All opt-in

I didn't change you setup, everybody can use what they want (opt-in)
It's only a config

Hi @philipbrembeck,

Thanks a lot for contributing! How does the web-application exactly work, does it open a PR with changed/newly added translations?

The editor can push changes or opens a PR based on your rights.

Hi @philipbrembeck,

Thanks a lot for contributing! How does the web-application exactly work, does it open a PR with changed/newly added translations?

The editor can push changes or opens a PR based on your rights.

Sounds nice, I'll bring it to Staging 😊

@philipbrembeck does it make sense to add this status badge?
[![inlang status badge](https://badge.inlang.com/?url=github.com/nilsjacobsen/veganify)](https://fink.inlang.com/github.com/nilsjacobsen/veganify?ref=badge)

can be generated here: https://inlang.com/m/zu942ln6/app-inlang-badge

Yes, I'll add it 😊


Build py.test noop container

hack for allowing integration_test to use more dependencies
at runtime

This code now resides here: https://github.com/dcos/dcos/pull/68


Out of Memory

Hi, I'm trying to run your software against images the same size as your example images, but I'm getting:
qlua: ...de/torch/install/share/lua/5.1/nn/SpatialConvolution.lua:110: cuda runtime error (2) : out of memory at /tmp/luarocks_cutorch-scm-1-664/cutorch/lib/THC/THCStorage.cu:44
Any ideas how to resolve? Thanks!

 If so, reinstalling torch/cutorch/cunn should help:
luarocks install torch
luarocks install cutorch
luarocks install cunn

Just reinstalled using those commands, and got the same error! I'm down to images of size less than 300x300 each, still running out of memory.

I see that my computer has an GT 650M (1 GB), while yours is the 750M (2 GB), so that could be why I'm running out. Thanks for your prompt reply!

Memo to others - for me this is also triggered by using a content image with nearly square aspect ratio. The error generated (on b699593d) is:
iter    10      loss 1.40e+08   lr 1.00e-01     time  1.8
qlua: cuda runtime error (2) : out of memory at /tmp/luarocks_cutorch-scm-1-8358/cutorch/lib/THC/THCStorage.cu:30
stack traceback:
        [C]: at 0x010af261d0
        [C]: at 0x16084be0
        [C]: in function 'mm'
        main.lua:48: in function 'gram'
        main.lua:76: in function 'style_grad'
        main.lua:174: in function 'opfunc'
        /Users/tom/torch/install/share/lua/5.1/optim/sgd.lua:43: in function 'sgd'
        main.lua:219: in main chunk

I'm sure it's the aspect ratio - for example for me a 300x294 content image will fail while a 1200x1000 will succeed. Looking at the code flow this kind of makes sense - even small images are scaled up to the standard size - which if square on my hardware is slightly too large.
This puzzled me for awhile, but there is now is a simple fix on the latest codebase: simply add a size argument at some number slightly smaller that 500 - eg: --size 460. @pents90 - this would also help you since you can similarly dial this number down to reduce the memory footprint.

@dribnet Thanks, I was able to use the --size parameter to get it working!

I use the -image_size ,it works


Adding background video view in android application - Kotlin

Topic Suggestion
Adding background video view in android application - Kotlin
Proposed article introduction
Application design is key when it comes to developing users' applications. Video views have come to play a major role in android development by creating applications that make a user feel lively and most of the time keep logged in. They can be used in an app as a struct on a particular thing or just a plain video.
In this tutorial, I'd like to develop an application showcasing the use of video view as the activity background. Thereafter, I shall implement floating interactive elements such as buttons and card views in the activity.
Key takeaways

Introduction to video view
Creating application project
Working with the XML file
Writing Kotlin code
Testing the app

Article quality
This article will be beginner-friendly that will help the developers know the importance of using video view in their applications activities. It will also use simple Kotlin code with step-by-step procedures to achieve our objective. The readers thereafter can integrate the video view in the application they are building.
References
https://www.youtube.com/watch?v=WLwQ3SJjWfY

Thank you for your response and we thank you for submitting your topic. After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations as previously mentioned on your topic form by our content moderator.
We typically refrain from publishing content that is covered widely on the net or other blogs. As we're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.


Is this still working?

I installed the extension, but the console it is not colorized. There are no errors on Chrome's console.
How does the plugin detect that I'm in Jenkins's console?
My Jenkins version is 1.616

I'm also the problem, the extension of chrome can't work.

Sorry, I don't use Jenkins anymore :confused:
Feel free to adapt the extension to the new Jenkins layout (it should be easy enough).

you can use the fix @ PR #6


Add sorting to accounts tables

Type: generic-feature-enhancement
Why is this change necessary?
Improve user interaction with account settings
This change addresses the need by:
Add sorting capabilities to the account settings
table
Potential Side Effects:
No known side effects

Coverage decreased (-0.1%) to 52.616% when pulling 4b9329b33188af647b06891eaa091e87e3989123 on Bridgewater:9620_SortAccounts into 9f9f7f8ce18bf05e596144161e6ec73093a7172a on Netflix:develop.

Very nice.


Start on splitting web / desktop version

This is a start on a web version. It makes app.js more platform agnostic and creates a desktop.js and web.js for the entry points to each version.
Although further work needs to be done to get it running on the web. I'm hitting this error:
Error: Cannot find module '/Users/Kyle/Documents/www/friends/node_modules/protocol-buffers/brfs' from '/Users/Kyle/Documents/www/friends'

and I imagine a few others are not web friendly. Still looking into solutions around those.

npm run web will start a server on localhost:9966.

@shama cool! if you run standard -F locally it should fix most of the linting errors.

@ngoldman Oh nice, I didnt know about standard -F. Fixed those errors, thanks!

@shama I think it just wraps https://github.com/maxogden/standard-format

Looks good to me, here we go!

@shama awesome start!


Text link doesn't get styled when using color schemes for navbar

I am not sure if this actual a bug, or expected behaviour, but I think this might be worth changing.
Expected behaviour:
When I use certain color schemes (background) in combination with a navbar, while using text, while using plain links.
I expect the link to be colored in a way it's readable, thus enough contrast. Just like the text is readable.

Actual behaviour:
When I use certain color schemes (background) in combination with a navbar, while using text, while using plain links.
The links do not get colored (with enough contrast). Rendering it unreadable while using it in combination with certain background color schemes.

    We gebruiken cookies om de site te analyseren, te verbeteren en voor tracking doeleinden. Als je de website blijft gebruiken gaan we ervan uit dat je dat goed vindt. Lees Meer

I understand I can solve it by adding a color scheme to the link (i.e.: Lees Meer). But I think it should work without it.

Hi @sebask I think this is a bug more than an enhancement. I agree that even though you can add a utility class to change the color of the link, it'd be better to have it work by default.
If you are ok with it I can send a PR to fix it.

@andresgalante Of course! :)

@saas786 Awesome, I've also changed the color of the light theme to be dark black instead of blue, that way it works with any color theme.
Thanks a lot for reporting it.


Queues: massage away 2-way round-robin PIFOs

As part of flag day (#2191), let's do a further piece of cleanup. We really have many different kinds of PIFOs hanging around now, but I'll point out two that should be merged thoughtfully.

This style, developed by me a little while ago, does round-robin scheduling among exactly two sub-queues. Its variable names make it clear that the two sub-queues can themselves be queues, not just FIFOs. In particular, they can be FIFOs or, recursively, PIFOs exactly like itself. This style of PIFO can therefore be elegantly used to make PIFO trees, where of course the PIFO trees are limited to binary-branching PIFO trees where each node wants the round-robin policy. This tree-making superpower has been tested via Python oracle.
Meanwhile, this style, developed by @csziklai this summer, does round-robin and strict scheduling among n sub-queues, not just two. This much has been tested via Python oracle. However, its variable names and testing thus far currently assume that the sub-queues are always FIFOs. That is, the file believes it can orchestrate n FIFOs to make an n-way PIFO, but it does not realize that it can orchestrate lists of PIFOs to make PIFO trees!

I would like to have the second style replace the first style. To do this, we need something like the following to happen:

[x] Change the variable names and comments and such of the second style to accept queues in general, not just FIFOs. I do believe the change needed is modest. The structure is ready for tree-building, it's just been used to make trees of height two where the leaves are all FIFOs.
[ ] Make more ambitious trees, like this file does with the first style. First let's literally just make what's in that file, since that matches the Formal Abstractions paper's motivating example and is byte-to-byte testable versus the existing implementation. Like, it'll pass the runt tests with the same .data and .expect files! Then as an extra file let's make rr(strict(A, B, C), rr(D, E, F), strict(G, H)) or something to show the extended capabilities. Check this via Python oracle.
[ ] If this all works, remove the first style from the codebase and use the second style wherever the first was used!
[ ] Document the second style, removing the documentation of the first style. https://github.com/calyxir/calyx/pull/2223 has begun the documentation task, but it adds on text below the text for the first style. I think we should just stamp out the old and sing the praises of the new! For this reason I'm going to skip on reviewing the PR. @csziklai could you please mark it as a draft for now? I'd like for it to be changed as described here and then review it!

@csziklai just a reminder to heed this advice here too! Do make sure that you branch off from the latest main commit!


Bug - empty enemy ships

I found a game breaking bug that prevents npc's from spawning inside boarded or landed spaceships. Ships that have landed no longer open the landing bays and are empty inside as well as any ships you board are empty. This even prevents starborn from spawning and strangely enough allows players to enter and steal their landed spacecrafts.

Details
I believe this bug is caused by the "kill all npcs on ship or leave ship" misc quest not properly finishing and thus breaking the current save-file. earlier save-files will not be affected and starting another newgame+ instance will also fix this. But once a save is affected by this bug it appears to be broken forever.
I do not know exactly how to replicate this bug.
Game Version
1.10.0 (since launch)
Language
English (EN)
New Game
No
Reproduction Steps
Boarding enemy ships. May have been initiated by a vanguard misc mission or one of the early crimson fleet quests that has you board a trade vessel to kill someone. But wouldn't know for sure as it seems random.

https://www.starfieldpatch.dev/issues/268

Current mod to fix and discussion on the topic:
https://www.nexusmods.com/starfield/mods/5668

I had the same problem in the "Matters Of THe Hart" quest at the smugglers cache.  The smugglers would not exit their two ships to attack you , the quest could not progress .

This is related to https://github.com/Starfield-Community-Patch/Starfield-Community-Patch/issues/221 and has been fixed by Bethesda.


Fixed fails running unit tests VS Code

The bug is from jsdom-no-contextify its Element has no classList
I mocked classList just for run test (all.js)
I suggest not remove Unit Tests from launch.json, it's useful for debug tests.
I suggest leave -g mocha argument for newbies in vscode enviroment (like me)

refers to #26699

@thr0w,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

#26699 was closed for a reason. We don't support running unit tests using vanilla mocha any more. We're using Electron for it. @jrieken already explained that. You can see what happens when running ./scripts/test.sh. You can really see what's happening when running ./scripts/test.sh --debug.
If you get those to work in a launch config, perfect, we'll take it.


Option to prevent cleanup of latest backup

Hi
Can an option be introduced to not remove a backup file if it's the latest backup?
We have a number of "Monthly" client extracts that are essentially data transferred from the main production database to client specific databases. The client specific databases are backed up and sent to clients for their own use. We keep the backup file so we can easily re-transmit to client if needed.
eg:
msdb backup history has a latest backup at 1st Nov,
Backup folder has 1st Nov file
Clean up option on Database Backup set to 48 hours
It's 26th Nov, but we still don't delete the file.
Cheers
Phil

By default DatabaseBackup will only do cleanup after a successful backup. This means that it will always keep the last good backup.


Incorrect supported platform for the Dynamics app

I'd like to point out that the Microsoft Dynamics app is never supported on Windows 7. Please remove Win7 off that list for the app. The other platforms listed for the app are correct.

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

ID: 0073f5a8-daf6-7f14-127f-e0c9d80af78d
Version Independent ID: d6d0afdf-3619-ca03-718b-a40bfd07bab9
Content: Azure Active Directory conditional access settings reference
Content Source: articles/active-directory/active-directory-conditional-access-technical-reference.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@hjohn76 Thanks for reporting! I have assigned the issue to the author to investigate and update as appropriate.

Hi @hjohn76, thanks for the information - fixed.
#please-close


Users are able to add dayoff entries but cannot delete the entries.

What permission governs the dayoff entries or events adding/deleting?
I see the users are able to dayoff entries and they couldn't delete their day off?
Please note the users are part of Azure DevOps Services Contributors group.

We're closing old issues so we can get back to actively maintaining the extension. Feel free to reopen if this is still an issue for you.


Implement super() calls in mixins in analyzer behind a flag

This is the analyzer-specific issue for #23770. That issue has the details.

Hi @bwilkerson and @stereotype441 is there any work here for the analyzer?

@sethladd yes, there is work for analyzer, but hopefully a small amount.  Currently, analyzer reports a compile-time error if:

a class that refers to super is used as a mixin, or
a class that is not derived from Object is used as a mixin.

A flag needs to be created, and these error cases need to be removed when the flag is set.

Thanks for the update! We have an anxious customer looking forward to this feature. :)

Support has now landed in the command-line analyzer and analysis server.

Is this work behind a flag?

It is behind the --supermixin flag in the command-line analyzer. It is available in the analysis server, behind a client-configurable option, but I don't know whether any clients have made it possible for users to enable the option.

cc @devoncarew for how to turn this on via Atom.
Also, can't wait for a .analysis_options to turn stuff like this on without a GUI. :)
What will it take to remove the flag? What signal are we waiting for?

+1 to using the .analysis_options file. Once people start using this feature in a project, every developer on the project will need to turn it on. If they use the analysis server flag, it'll get turned on for other projects as well, whether that's desired or not. It's really something that should be under SCM with the rest of the source.

... how to turn this on via Atom ...

Use the analysis.updateOptions request to set the enableSuperMixins  option to true.

What will it take to remove the flag? What signal are we waiting for?

See https://github.com/dart-lang/sdk/issues/23770 for context.

Also, that way we only need to implement the configurability once... otherwise, we're asking three IDEs (or more?) to build GUIs for these kinds of flags (and, thus, coordinate releases of plugins with SDKs).

See #23770 for context.

Right, thanks. Are you waiting for the spec to be formally blessed by TC52 before removing the flag? (IIR that's "standard operating procedure")

Yes. And waiting for the rest of the tools (ddc and dart2js) to support it. We don't really want to enable a feature like this in some places but not in others.


Change max temp to 40

https://github.com/baerengraben/ioBroker.swiss-weather-api/blob/af7dc464c03987ef7db60a5e01ffaafb7ac6e40d/main.js#L219
In the graph displayed maximum temperature is 30. Meanwhile, in summer it is sometimes much warmer than 30 degrees. Therefore, this value should be increased to 38 or even 40 degrees.
perhaps the min_temp =0 is also not ideal. Better would be max=38 and min =-15

Please set the lower limit to -20 or -25°C

I prefere a dynamic Version. Analyse the Temperature of the Day (24 Values) and define after that the limits.
like yAxis_max = (Max Value of the Day) +5 and yAxis_min = (Min Value of the Day) -5
Perfect Chart for every Temperature :)
Gruss aus Bern

@SwissSteini: Your proposal is implemented. Wait for next release und Gruss aus Bern :)

Fixed with release 1.0.5


Fix login redirect bug

When a user that is not logged in views a dandiset, then clicks Log in, the return URL does not record the dandiset they were viewing. Logging in will bounce them back to a blank dandiset page.
Fixes #349

Already fixed in https://github.com/dandi/dandiarchive/commit/0db3a003625fe9e018ddb300833b4380a3a70f10


Move Instant from ProgressDrawState to ProgressDrawTargetKind::Term

I believe it makes more sense for Instant to be stored in ProgressDrawTargetKind::Term. The Instant is used for checking the rate of drawing only. Now, when it is stored in the draw target it is more accurate (as it actually checks the time of the last draw instead of the time when an update was crated) and reduces memory usage.

This looks nice, would you mind rebasing it?

Thanks!


Add examples views in Coder

We need to add the ability to run multiple examples and to view example dependencies directly in Coder.


Default to 1.7-level bytecode in Java build

rake-compiler defaults to source level and output bytecode version 1.6, corresponding to Java 1.6.
JRuby has not supported 1.6 since the release of JRuby 9.0, five years ago, when we started requiring Java 7.
This PR updates the default versions to "1.7" level for Java 7.
All current supported versions of JRuby require Java 8, but I'm ok with using 1.7 here for any 9.0 stragglers.

Thanks!


Add range options to ContinuousDistributionAnalyzer

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (5173c89) 99.87% compared to head (d6c6e6a) 99.87%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage   99.87%   99.87%
=======================================
  Files          39       39
  Lines         819      832   +13
  Branches       88       90    +2
=======================================
+ Hits          818      831   +13
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


macOS fails to boot

Operating system
VanillaOS 2 (Orchid)
Description
I am trying to run this image through Podman on my system, using podman-compose. When I try to run it, it crashes instantly. This same error occurs on both macOS Ventura and Monterey
Docker compose
services:
macos:
image: ghcr.io/dockur/macos
container_name: macos
environment:
VERSION: "monterey"
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 5900:5900/tcp
- 5900:5900/udp
stop_grace_period: 2m
Docker log
BdsDxe: failed to load Boot0001 "UEFI Misc Device" from PciRoot(0x0)/Pci(0xA,0x0): Not Found
BdsDxe: loading Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
BdsDxe: starting Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
#[EB|LOG:EXITBS:END] _
#[EB.BST.FBS|-]
#[EB|B:BOOT]
#[EB|LOG:HANDOFF TO XNU] _
======== End of efiboot serial output. ========
qemu-system-x86_64: terminating on signal 2 from pid 1 (/usr/bin/tini)
Screenshots (optional)
No response

I got this same error on Arch using docker-compose on both Ventura and Monterey.

I got the same error running on Fedora 40

What kind of partition did you create? I'm not sure if it has something to do with it, but when I selected case sensitive with partitioning my installation didn't work. Since I also changed the OS version from Sonoma to Montery I'm not sure if this was the issue though.

Same issue
services:
  osx:
    image: dockurr/macos
    container_name: macos
    environment:
      VERSION: "sequoia"
      DISK_SIZE: "64G"
      RAM_SIZE: "16G"
      CPU_CORES: "4"
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
      - 5900:5900/tcp
      - 5900:5900/udp
    stop_grace_period: 2m

Failed to boot
BdsDxe: failed to load Boot0001 "UEFI Misc Device" from PciRoot(0x0)/Pci(0xA,0x0): Not Found
macos  | BdsDxe: loading Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
macos  | BdsDxe: starting Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
macos  | #[EB|LOG:EXITBS:END] _
macos  | #[EB.BST.FBS|-]
macos  | #[EB|B:BOOT]
macos  | #[EB|LOG:HANDOFF TO XNU] _
macos  | ======== End of efiboot serial output. ========

neofetch of my ubuntu 24.10 system
OS: Ubuntu 24.10 x86_64
    .ossssssssssssssssssdMMMNysssso.       Host: 80YX Lenovo Y920-17IKB
   /ssssssssssshdmmNNmmyNMMMMhssssss/      Kernel: 6.11.0-8-generic
  +ssssssssshmydMMMMMMMNddddyssssssss+     Uptime: 1 hour, 11 mins
 /sssssssshNMMMyhhyyyyhmNMMMNhssssssss/    Packages: 3594 (dpkg), 138 (brew), 158 (flatpak), 46 (snap)
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Shell: zsh 5.9
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   Resolution: 1920x1080, 1920x1080
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   DE: GNOME 47.0
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   WM: Mutter
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   WM Theme: Adwaita
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Theme: Mojave-dark [GTK2/3]
 /sssssssshNMMMyhhyyyyhdNMMMNhssssssss/    Icons: WhiteSur [GTK2/3]
  +sssssssssdmydMMMMMMMMddddyssssssss+     Terminal: terminator
   /ssssssssssshdmNNNNmyNMMMMhssssss/      CPU: Intel i7-7820HK (8) @ 3.900GHz
    .ossssssssssssssssssdMMMNysssso.       GPU: NVIDIA GeForce GTX 1070 Mobile
      -+sssssssssssssssssyyyssss+-         Memory: 20691MiB / 61937MiB

compose.yml
networks:
    1panel-network:
        external: true
services:
    macos:
        cap_add:
            - NET_ADMIN
        container_name: ${CONTAINER_NAME}
        deploy:
            resources:
                limits:
                    cpus: ${CPUS}
                    memory: ${MEMORY_LIMIT}
        devices:
            - /dev/kvm:/dev/kvm
            - /dev/dri:/dev/dri
        env_file:
            - /etc/1panel/envs/global.env
            - ${ENV_FILE:-/etc/1panel/envs/default.env}
        environment:
            - VERSION=${VERSION}
        image: dockurr/macos:1.14
        labels:
            createdBy: Apps
        networks:
            - 1panel-network
        ports:
            - ${HOST_IP}:${PANEL_APP_PORT_HTTP}:8006
            - ${HOST_IP}:${PANEL_APP_PORT_REMOTE}:5900/tcp
            - ${HOST_IP}:${PANEL_APP_PORT_REMOTE}:5900/udp
        restart: always
        stop_grace_period: 2m
        volumes:
            - ${MACOS_STORAGE_ROOT_PATH}/storage:/storage

env
CONTAINER_NAME="macos-ms"
CPUS=0
CPU_CORES=4
DISK_SIZE="64G"
HOST_IP=""
MACOS_STORAGE_ROOT_PATH="/home/data/macos"
MEMORY_LIMIT=0
PANEL_APP_PORT_HTTP=8122
PANEL_APP_PORT_REMOTE=5900
RAM_SIZE="8G"
VERSION=15
ENV_FILE=/opt/1panel/apps/local/macos/macos/.env

日志
macos-ms  | ======== End of efiboot serial output. ========
macos-ms  | [=3h[=3h[=3hBdsDxe: failed to load Boot0080 "Mac OS X" from PciRoot(0x0)/Pci(0xA,0x0)/VenMedia(BE74FCF7-0B7C-49F3-9147-01F4042E6842,471F93308231524A95328E3E6A1889E8)/\9F4330E7-AE1E-4144-8069-E5A69EECA87C\System\Library\CoreServices\boot.efi: Not Found
macos-ms  | BdsDxe: failed to load Boot0001 "UEFI Misc Device" from PciRoot(0x0)/Pci(0xA,0x0): Not Found
macos-ms  | BdsDxe: loading Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
macos-ms  | BdsDxe: starting Boot0002 "UEFI Misc Device 2" from PciRoot(0x0)/Pci(0x5,0x0)
macos-ms  | #[EB|LOG:EXITBS:END] _
macos-ms  | #[EB.BST.FBS|-]
macos-ms  | #[EB|B:BOOT]
macos-ms  | #[EB|LOG:HANDOFF TO XNU] _

After uninstalling, pull the mirror again and reinstall it. It was successful, but iCloud can't log in.

In my case, the log ends there, but NOVNC never connects to anything.  Just says failed to connect to server.

@QYG2297248353 I created a new version 1.16 that generates unique machine serials when creating a fresh installation. I think that should solve your problem with iCloud.

OK, I'll upgrade right away.

So everyone is talking about "I have the same error" but I dont see any errors in your logs? If something else goes wrong please post a screenshot or a description? But the logs looks good.

This is the only thing in the log that looks like a wrong error
After installation, this error occurs, and the remote desktop will restart in an infinite loop of "Five Languages"


M4 instances missing

M4 instances were just announced.
$ fab build
Parsing instance types...
ERROR: Unable to scrape site data: Expected 12 columns in the table, but got 4
Loading data from www/instances.json...
Rendering to www/index.html...

Done.

Looks like there's no need to screenscrape at all anymore, the official pricing page is backed by JSON data, e.g. http://a0.awsstatic.com/pricing/1/ec2/linux-od.min.js ...

Resolved by #116


loss function issue

Hi,  in a3c.py  the gradients are updated by the total loss, however, in the original paper of A3C, there are two loss functions, one is policy loss function and another is value loss function, they both update the gradient including the weights of CNN.

I think the code should like this
pi_grads = tf.gradients(pi_loss, pi.var_list)
vf_grads = tf.gradients(vf_loss, vf.var_list)

Can you share your opinions on this?

For a weight w shared between both value and policy network, the gradient update should be  w = w - grad(v, w) followed by w = w - grad(p, w) which is same as w =w - grad(v+p, w)(by sum rule), where p and v are policy and value losses respectively.


Enable PooledByteBufAllocator to work, even without a cache

Motivation:
useCacheForAllThreads may be false which disables memory caching
on non netty threads.  Setting this argument or the system property
makes it impossible to use PooledByteBufAllocator.
Modifications:
Delayed the check of freeSweepAllocationThreshold in
PoolThreadCache to after it knows there will be any caches in
use.  Additionally, check if the caches will have any data in them
(rather than allocating a 0-length array).
A test case is also added that fails without this change.
Results:
Fixes #7194

Cherry-picked into 4.1 (d2cb51bc2e86ac485d84cb1a0313b22f4de988d0) and 4.0 (6748c3b891884924992f98750ef2f79b4be16008).
thanks @carl-mastrangelo


fix: Cannot reattach ActivatedRouteSnapshot created from a different …

…route

PR Checklist

[x] The PR title follows our guidelines: https://github.com/NativeScript/NativeScript/blob/master/CONTRIBUTING.md#commit-messages.
[x] There is an issue for the bug/feature this PR is for. To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it.
[x] You have signed the CLA.
[ ] All existing tests are passing: https://github.com/NativeScript/nativescript-angular/blob/master/DevelopmentWorkflow.md#running-the-tests
[ ] Tests for the changes are included.

What is the current behavior?
Navigating between sibling routes may sometimes mess with the cache due to it using the key of a parent route, instead of the complete route.
What is the new behavior?
This now uses the correct key from the farthest children of the route
PARTIALLY Fixes #1993. (this issue has 2 problems on it: 1 is the one fixed in this PR, the other is that route guard/resolvers don't work on back navigation and may break navigation for a PRO)

Please don't merge this yet, as it seems it might not fully solve the issue

@edusperoni @NathanaelA
When debugging this issue in the use case Root => A => B, then back to A it looks like all keys to store/retrieve are generated correctly (also without this fix), but the error is still thrown.
The expression that causes the error to be thrown (curr.value.routeConfig !== result.value.routeConfig in https://github.com/angular/angular/blob/3817e5f1dfeb9de26fa2ea4068a37565f435214f/packages/router/src/create_router_state.ts#L52) will actually show equal properties and values when inspecting, but are not the same object reference. This only occurs with lazy loaded modules (if changed to "regular" route configs it works).
Perhaps the solution direction for this part of the issue is to either use a "deepEqual" instead of reference equality or to somehow make sure that the routeConfig reference stays the same for lazy loaded modules.
Hope this helps, cheers.


lxd/storage/driver/zfs: Fix ListVolumes with custom zpool

Closes #9207
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@tomponline should be good to merge when you're around :)


Type and import

Is there a way to add the Translate type back. I was testing on 1.0.0-experimental.4 for a while now. The interface gets not exported anymore. Due to the refactoring I do not know if the type still exists anymore. I think the export stopped at 1.0.0-experimental.4.1.
import { Translate } from 'next-translate';
The idea is simple to pass the t-function as an argument in functions and have some type safety in when you call the function.
Also, it would be nice to see the useTranslation hook exported through a named export.
Import now
import useTranslation from 'next-translate/useTranslation';
Named export
import { useTranslation } from 'next-translate';
Thx!

1.0.0-experimental.4 is quite old. Can you try 1.0.0-canary.2? which is the last prerelease. Not sure if exist this type but we increased the types. If it's definitely not there either, then we can add it, yes!

I did upgrade to 1.0.0-canary.2 in the afternoon, but the type Translate is not there, also useTranslation is exported through
import useTranslation from 'next-translate/useTranslation';

Looking at the index.d.ts from the sources the Translate interface exists, but the build does not have it. I don`t know the internals how you build the deployment bundle, but its missing there. The builds index.d.ts looks the following, important this is from the esm bundle.
import { ReactElement, ReactNode } from 'react';
export interface TranslationQuery {
    [name: string]: string | number;
}
export interface I18n {
    t(i18nKey: string | TemplateStringsArray, query: TranslationQuery | null | undefined, options: {
        returnObjects?: boolean;
        fallback?: string | string[];
    }): string | object;
    t(i18nKey: string | TemplateStringsArray, query: TranslationQuery | null | undefined): string;
    t(i18nKey: string | TemplateStringsArray): string;
    lang: string;
    loadLocaleFrom?: (language: string, namespace: string) => Promise<I18nDictionary>;
}
export interface I18nProviderProps {
    lang?: string;
    namespaces?: Record<string, I18nDictionary>;
    children?: ReactNode;
    logger?: I18nLogger;
    loadLocaleFrom?: (language: string, namespace: string) => Promise<I18nDictionary>;
}
export interface TransProps {
    i18nKey: string;
    components?: ReactElement[];
    values?: TranslationQuery;
    fallback?: string | string[];
}
export declare type PageValue = string[] | ((context: object) => string[]);
export interface I18nConfig {
    defaultLocale?: string;
    locales?: string[];
    loadLocaleFrom?: (language: string, namespace: string) => Promise<I18nDictionary>;
    pages?: Record<string, PageValue>;
    logger?: I18nLogger;
    loader?: boolean;
    logBuild?: boolean;
}
export interface LoaderConfig extends I18nConfig {
    locale?: string;
    router?: {
        locale: string;
    };
    pathname?: string;
    skipInitialProps?: boolean;
    loaderName?: string;
    isLoader?: boolean;
}
export interface LoggerProps {
    namespace: string;
    i18nKey: string;
}
export interface I18nLogger {
    (context: LoggerProps): void;
}
export interface I18nDictionary {
    [key: string]: unknown;
}
export interface DynamicNamespacesProps {
    dynamic?: (language: string, namespace: string) => Promise<I18nDictionary>;
    namespaces?: string[];
    fallback?: React.ReactNode;
    children?: React.ReactNode;
}

Also exist the useTranslation export<EMAIL_ADDRESS>...

How do you build the lib folder the following fields in package.json are referencing to

Just doing yarn build and it executes the follow:
https://github.com/vinissimus/next-translate/blob/72abc8be67f56cd6e98043c61ad2d7012e6f149d/package.json#L58

what is rare is that the repo examples use the canary.2 version with esm and it works well...! Maybe it would help if you create an example reproducing the error🙏

I think the build is just fine. Translate interface for the t function does not exist anymore, instead it is either
t(
   i18nKey: string | TemplateStringsArray,
   query: TranslationQuery | null | undefined,
   options: { returnObjects?: boolean; fallback?: string | string[] }
): string | object
t(
   i18nKey: string | TemplateStringsArray,
   query: TranslationQuery | null | undefined
): string
t(i18nKey: string | TemplateStringsArray): string

this is just pseudo code not written within an editor.

import { I18n } from 'next-translate';

const translateText = (i18nObject: I18n) => {
   const text = I18n.t(...)
}

It is likely that by migrating the project to TypeScript in version 1.0 some types have changed, some improved, but others may need to be re-added.

I did upgrade to 1.0.0-canary.2 in the afternoon, but the type Translate is not there, also useTranslation is not exported. I consume it through.
import useTranslation from 'next-translate/useTranslation';

However, I'm worried about this. You should import useTranslation with any problem on 1.0.0-canary.1...

For the useTranslation exposing through a named export I think you have to add useTranslate in the index.tsx file, next where you do
https://github.com/vinissimus/next-translate/blob/72abc8be67f56cd6e98043c61ad2d7012e6f149d/src/index.tsx#L3
import nextTranslate from './plugin'
module.exports = nextTranslate => end of file

The index.tsx will run through typescript, so I think it makes sense to switch the module.exports = for es exports. You already have your tsconfig.json and tsconfig-cjs.json profiles.
Maybe you can try to change
import nextTranslate from './plugin'
export default nextTranslate; => maybe a more fancier default export is possible

export { default as useTranslation } from './useTranslation' => export as named export

Well, useTranslation package should be imported from next-translate/useTranslation:
import useTranslation from 'next-translate/useTranslation'

This is deprecated:
import { useTranslation } from 'next-translate'

In fact, in the latest versions (before 1.0) it is already showing a deprecation warning if used like this:

https://github.com/vinissimus/next-translate/blob/master/src/index.js#L9-L11

Reading the comment in index.js makes me thinking, haven`t heard this at all, global imports... Maybe something was misunderstood back then, but doing "named" exports allows "named". This has nothing to do with global imports, I do not really unterstand what those might be in this context.
export { default as useTranslation } from './useTranslation'
allowing imports like
```import {useTranslate} from 'next-translate````
any major library like mui is doing it that way.

@gurkerl83 It is open to debate. I prefer that people use it as a separate package to avoid tree-shaking problems, although webpack in principle does tree-shaking I had experienced some problems in the past making the code heavier. I wanted to be strict and that there was only one way to do the imports, but maybe I'm wrong. What are the advantages of allowing these named exports?
Next.js for example also forces to import Router from next/router, link from next/link...

I don't think I can list the pros and cons of Named Exports in detail myself. I have to admit that my arguments are only based on my experience. My primary aspect is the behavior of development environments. Named exports allow an IDE feature Intelli Sense to automatically display possible import variants.
I cannot make any statements about tree shaking. Of course everything is bound to an object but I think Webpack can separate the sub objects attached to the object if there is no mutual entanglement. This is just an assumption.
I can't say much about NextJ's export variants here either. But I could imagine that these small blocks could be considered as elementary to be integrated in the NextJS project as light as possible. This is also just an assumption.

@gurkerl83 as a conclusion, we can say:

[ ] Add again the type Translate
[ ] Check whether to put back the named exports

To verify the tree-shaking I will test the bundle analyzer with both types of import. Then I value whether to add them again or not.

Thats great! For the Translate type maybe you can reduce those three t functions to just one, with some arguments optional. If not maybe typescript unions and Intersection types help. Thx!

I don't know why. But vscode always suggest to import
import useTranslation from 'next-translate/lib/esm/useTranslation'

instead of
import useTranslation from 'next-translate/useTranslation'

@vimutti77 I saw it recently... I don't know either... The correct one is as you say:
import useTranslation from 'next-translate/useTranslation'

Because it will load ESM or CJS depending on node / browser.
If someone knows how to tell VSCode what "import" is the correct one, please tell here! I'm going to do some research, thanks to report it.

Asked in stackoverflow: https://stackoverflow.com/questions/65145253/how-to-tell-vscode-which-suggested-import-is-correct-for-library-creators
I hope someone will enlighten me. I will still continue my research.

@gurkerl83 I did a prerelease 1.0.0-canary.6 re-adding the Translate type (improved a little bit than 0.20.x). Would you confirm that it works well for you? Thank you! 🙏

Hi thanks for adding this. I use the option to return objects quite frequently, same as the standard variant. Is it possible to change the Generic Typ a bit to support this more properly. I tested the following.
export interface Translate {
  <T = string>(
    i18nKey: string | TemplateStringsArray,
    query: TranslationQuery | null | undefined,
    options: { returnObjects?: boolean; fallback?: string | Array<string> }
  ): T;
  (
    i18nKey: string | TemplateStringsArray,
    query: TranslationQuery | null | undefined
  ): string;
  (i18nKey: string | TemplateStringsArray): string;
}

A little explanation is necessary for the first option. For the first option I added a generic type to specify in case it is applied the return type of the object. A consumer of t can continue without applying any cast operation like here
 const steps = t<Array<Content>>(
    'pages/ai/index:steps',
    {},
    {
      returnObjects: true
    }
  );

My question is for the second and third option. Are there any circumstances that those functions can return a value other than string. I that case I understand the purpose of <R = string> () : R like
t<Null>("translationKey")

otherwise i would remove the generic type arguments for the second and third option like illustrated at the top of the comment.
this is the version of the canary release 6
export interface Translate {
  (
    i18nKey: string | TemplateStringsArray,
    query: TranslationQuery | null | undefined,
    options: { returnObjects?: boolean; fallback?: string | string[] }
  ): string | object
  <R = string>(
    i18nKey: string | TemplateStringsArray,
    query: TranslationQuery | null | undefined
  ): R
  <R = string>(i18nKey: string | TemplateStringsArray): R
}

Does this makes sense, Thx!

@gurkerl83 feel free to PR! to be honest, I am not very familiar with TypeScript yet and I am sure it can be better.

I did make a merge request. Its base is the canary branch. Please see the commit message for more detail.

About the named exports, after adding this to the index.ts file:
export { default as DynamicNamespaces } from './DynamicNamespaces'
export { default as I18nProvider } from './I18nProvider'
export { default as Trans } from './Trans'
export { default as appWithI18n } from './appWithI18n'
export { default as loadNamespaces } from './loadNamespaces'
export { default as useTranslation } from './useTranslation'
export { default as withTranslation } from './withTranslation'

I can see that webpack is not doing well the treeshaking... It's importing unused packages...
With named exports

With separate packages

😔

Are you using the package @next/bundle-analyzer in on of the demos, or the standard bundle analyzer on the library?

I added on the example https://github.com/vinissimus/next-translate/pull/379

This needs some more investigation. From a library point of view, I would argue only library relevant files and dependencies have to be taken into account. Applying the analyser at a running application, with other things involved such as mdx loader might corrupt the result and makes interpretation harder. We might consider to make the build process run through rollup of a similar bundler.

@vimutti77 Finally, I have seen that the auto-import is related to where the TypeScript types are. Moving the types to the root solves it.
Instead of:
{
  "internal": true,
  "main": "../lib/cjs/useTranslation.js",
  "module": "../lib/esm/useTranslation.js",
  "types": "../lib/esm/useTranslation.d.ts"
}

Doing this (After moving the types to the root):
{
  "internal": true,
  "main": "../lib/cjs/useTranslation.js",
  "module": "../lib/esm/useTranslation.js",
  "types": "../useTranslation.d.ts"
}

Solve the problem. Solved package<EMAIL_ADDRESS>
@vimutti77 @gurkerl83 I did a prerelease 1.0.0-canary.7 with the fixed types + fixed auto-import

@aralroca That is great!

@gurkerl83 About the named exports; it is difficult that every export coexists with others. There are exports that have a require('fs') (or other node things) and although webpack does tree-shaking, it first needs to resolve these dependencies, and on the client-side, it can't and is throwing errors about this.
I think for now it is better that they stay as individual packages. Much simpler and this way we avoid conflicts.

I know what you mean, I changed quite some things on my local branch to get it work. The fs problem is best know, and easy to solve. Whats important here is that you have to differentiate the build phase either its browser or server. Each build consits of both. If it is on browser there is no fs.
const getFallback = (isServer) => {
  if (!isServer) {
    return {
      fs: false,
      process: false,
    }
  }
}

Most important when webpack 5 gets used an the project is using it based on the yarn resolution used, node pollifills such as process are not provided anymore. The list goes on, and on.
What made the biggest headache for me was the use of imports in the template sections used, i didn`t realised that in the first place and it was to late to quit e.g.
export function getDefaultAppJs(hasLoadLocaleFrom) {
  return `
  import i18nConfig from '@next-translate-root/i18n'
  import { appWithI18n } from 'next-translate' => **here !!!**

  function MyApp({ Component, pageProps }) {
    return <Component {...pageProps} />
  }

  export default appWithI18n(MyApp, {
    ...i18nConfig,
    skipInitialProps: true,
    isLoader: true,
    ${overwriteLoadLocales(hasLoadLocaleFrom)}
  })
`
}

After a few hours the build was working with named exports, and a rollup build underneath. I clean the stuff I have up and provide a feature, not to merge but for reviewing the approach, because it also splits the loader in a separat build.
I think there is much improvement possible but one step at a time. I`m totally fine with the current export mechanism. Thx!

Ok this sounds really good 🙏

@gurkerl83 Maybe we can move this to version 1.1 so that it doesn't block 1.0, what do you think? ☺

@aralroca Perfect, I currently have to catch up on a few things in other projects and won't be able to clean up and provide my changes until next week.


0.3.1 will not install on Win7 Pro 64 - npm 6.9.0

I cannot get past this error:
> scripts/install.js

'scripts' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `scripts/install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional log
ging output above.

npm ERR!      C:\Users\yeah_that_is_me\AppData\Roaming\npm-cache\_logs\2019-06-20T17_34_32
_761Z-debug.log

Thanks in advance.

Tried again 20190919, same result. I monitored directory creation in %appdata%\npm\node_modules.
In the course of installation, in that directory:

Dir .staging is created;
Dir rpscript is created;
Install errors begin;
Dir rpscript is deleted.
It is too quick for me to tell whether the deletion of step 4 occurs before or after step 3.


Use the username provided in GCP address to ssh

Earlier if the user did something like this:
localUser@localhost:~$ sshcode gcp:userName@InstanceName
it'd still be ssh as localUser@InstanceName , thats because only IP was returned and when that's the case, ssh tries to use the current username for remote machine,
This fixes that.
Tested on my machine(Manjaro Linux).
Signed-off-by: Sahil Soni<EMAIL_ADDRESS>

I don't have gcloud, so could you please explain how this PR makes parseGCPSSHCmd return a host with a username?

The dryRumCmd( gcloud compute ssh --dry-run InstanceName ) (Line 538) return the command used to ssh to the instance directly(without gcluod sdk).
So, the command:
localUser@localhost:~$ gcloud compute ssh --dry-run myInstance
returns something like(comment mentioning it at line 553) :
ssh **ssh flags** locakUser@myInstance
Now this works good if we(sshcode) wanna use the current localUser to ssh, so if we use
localUser@localhost:~$ gcloud compute ssh --dry-run remoteUser@myInstance
returns something like :
ssh **ssh flags** remoteUser@myInstance
Before my commit, we were loosing this username (comment mentioning it at line 556), by returning the userIP, we retain the username provided by the user and ssh to the correct user on the instance.

Ah, thanks for pointing that out, I was reluctant on remove that part becuase it was checking that the IP Address was valid, your comment made me to do some digging, and yes, it NEEDS to be gone, its not just unnecessory, but a bug on its own.
I found out that when an external IP is not assigned to the GCP instance we are trying to ssh to, the dry-run command returns something like this:
ssh **sshFlags**<EMAIL_ADDRESS>As we can see that the last part is not a valid IP and will thus fail that check, but this command is valid(well, there's a bug in gcloud sdk at this point that gives invalid sshFlags at this point in a certain case, will be reporting that to google soon, not throughly tested yet), we can ssh with the dry-run command provided(when corrected),
So, yeah, we should remove that.
Pushing the change and adding this to first message.

Ok, so I was looking testing my changes out for instances with no external IP, and I found that instances with no external IP addresses cant connect to internet.
This means downloading the code-server on the instance is not possible(at least i dont know how to do it). So I think we'll have to download it on our local machine and scp it over to the instance.
I think that is out of scope for this PR, will work on it and raise new PR when I do that.

Looks good to me. These commits need to be signed with PGP before I can merge them though.

Signed :)


Normalize text in messages to reduce token length

Purpose

...
resolve the bug: [Rag sample clasic] Fix or revisit token estimation

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

What to Check
Verify that the following are valid

...
Korean and english words are running out of maximum token limit at the same time.

Other Information

Thanks @Yun-BizQA ! Could you provide an example of Korean text that failed before? And then add a test for that? (See CONTRIBUTING.md for info on running the tests)

The fix addresses the issue here: https://github.com/Azure-Samples/azure-search-openai-demo/issues/543
The maximum context length is reached easily for korean, chinese, japanese etc because python calculates their length wrong. The fix makes their len equal as the way English char counts, so conversation lasts longer before reaching to such err. The test is performed in the local deployment instead of codes.

@Yun-BizQA Can you take a look at the changes I made and see if they make sense? I moved the normalization into the message builder so that its applied to all messages. I also added several tests. Let me know what you think.
also cc'ed @tonybaloney as he has experience with CJK language support.

makes sense to me 👍🏻


chore: compile for docs with all features

Add a configuration so all features are shown in docs.rs.
I think this will help people trying to learn how to use the lib from docs.rs

Looks good, thank you!


ng-model receive value of the input field even if empty

Dear @hakib,
first of all thanks for this great directive and the variety of options for its configuration.
Great job!
My goal is to bind the value of the input to a $scope.variable however it is not binding. My workaround as of now is to assign the term to a $scope.variable when fuzzy_suggest gets called
function fuzzy_suggest(term) {
            if (!term) {
                // filter assignment
                $scope.searchText = '';
                return [];
            }
            // filter assignment
            $scope.searchText = term;

this however only gets called as long as there is at least one character in the input field - not when the input is empty line 128.
function _suggest(term, target_element) {
        $scope.selected_index = 0;
        $scope.waiting_for_suggestion = true;
        if (typeof(term) === 'string' && term.length > 0) {

I am sure that I am missing something and would thus be grateful for a quick hint, what I can do to always extract the value of the input {{input}}
Thanks a lot.
Best,
Karl

Hey @karlkurzer,
I'm not sure I understand what you mean by "bind the value of the input to a $scope.variable" but if your objective is to manipulate / show the value which is currently in the input then you can use the variable you bound to the ng-model.
See this example. You can see that {{dirty.value}} is used by the ng-model and is also used in the template.

Hey @hakib
thank you for your quick answer, the plunker and my code work now.
What i tried was:
$scope.search = ''

<div mass-autocomplete>
<input ng-model="search" mass-autocomplete-item="ac_fuse_options">
</div>

instead of
$scope.search = {value:''}

<div mass-autocomplete>
<input ng-model="search.value" mass-autocomplete-item="ac_fuse_options">
</div>

I guess it needs to be an object...
Thanks,
Karl


'sequential_sleeps' statistic breaks statistics menu for vanilla clients

Vanilla clients get disconnected from the server upon opening the statistics menu when they have an entry for 'minecraft:sequential_sleeps'.
Unfortunately it looks like that statistic is how the mod counts sleeps, and I don't know if there's a different way to do that.
The mod still functions server side, just vanilla clients can't view statistics.

i had taken a break from modding but ill look in to this now :3

srry i got busy then forgot abt this, but its fixed not !


Fixing default Prometheus endpoint for Grafana

Fixes the URL for the Prometheus backend in Grafana
Description
Out of the box, the Helm chart for Grafana uses the incorrect URL for the Prometheus backend. As a result, the following issue will pop up: https://github.com/coreos/kube-prometheus/issues/65. Basically, the naming scheme for the URL should be http://{{ printf "%s" .Release.Name }}-prometheus:9090 to fix this.
Motivation and Context
Ensuring that Grafana works out of the box with the kube-prometheus chart.
How Has This Been Tested?
I had to fix this on my pre-production cluster. The Prometheus service is created as so:
$ kubectl get svc monpreprod-prometheus -o yaml
apiVersion: v1
kind: Service
metadata:
  labels:
    app: prometheus
    chart: prometheus-0.0.50
    heritage: Tiller
    prometheus: monpreprod
    release: monpreprod
  name: monpreprod-prometheus
  namespace: monitoring
spec:
  clusterIP: <cluster-ip>
  ports:
  - name: http
    port: 9090
    protocol: TCP
    targetPort: 9090
  selector:
    app: prometheus
    prometheus: monpreprod-prometheus
  sessionAffinity: None
  type: ClusterIP
status:
  loadBalancer: {}

As you can see, this does not following the scheme in the original dashboards-configmap.yaml file.
Screenshots (if appropriate):

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Sorry for the inconvenience, but the prometheus-operator and kube-prometheus helm charts are currently being migrated to the upstream stable helm charts. Because of this we are currently not accepting any pull requests. We apologize for the inconvenience, but this is necessary for the migration to actually happen. We announced this here: https://github.com/coreos/prometheus-operator/tree/master/helm#warning
You can view the progress and help here and here.
Thank you for your understanding! :slightly_smiling_face:


Could not load type 'OpenQA.Selenium.Internal.IFindsById' from assembly 'WebDriver, Version=<IP_ADDRESS>

Could not load type 'OpenQA.Selenium.Internal.IFindsById' from assembly 'WebDriver, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
Getting above error with below Selenium version:
Environment

.NET Core 3.1
Selenium.WebDriver (4.4.0)
Selenium.Support (4.4.0)
Appium.WebDriver (4.3.2)
Win 10 Visual studio pro 2019

Appium 4.3.2 doesn't support Selenium Webdriver v4 so I'm going to close this. If you're looking for a version that supports Selenium 4, then please try the beta version


Opt-out from operator-sdk in e2e tests

Idea:
This is a follow-up from https://github.com/shipwright-io/build/issues/535#issuecomment-766877003 . We want to reduce the usage of the operator-sdk. At the moment we heavily used on the e2e tests.
Acceptance_Criteria:
Refactor e2e tests to avoid the usage of the operator-sdk in favor of something else. Some alternatives for this could be:

Run a helm install, however we do not yet have the charts for the operator in place.
Run the operator as a process in the background, we do this in integration tests.

Another idea, courtesy of @ImJasonH, is to use the ko tool that Tekton uses as a part of their release engineering. He can demo at the community meeting Monday (Feb 1).

Another idea, courtesy of @ImJasonH, is to use the ko tool that Tekton uses as a part of their release engineering. He can demo at the community meeting Monday (Feb 1).

Assigning this to myself, because we will be working on this soon.


Provide a general error for HTTP requests

Often in the code involving HTTP requests there's a general ugliness regarding how we handle possible exceptions. This results in writing rescue without the expected error classes, which sometimes hides valid exceptions, thus hindering debugging.
The proposed solution is:

We create a general exception, which as arguments accept the error message and the class of the original exception.
This exception can be intercepted close to the perform_request so callers can get a safer and simpler way to handle HTTP responses.
We remove all the rubocop:disable statements regarding the hiding of exceptions.

This is mostly done. The only thing missing is to check whether there are areas on the code which are still using the old way to handle requests. This can be done in 2.5.


Turn on all layers at once

To turn on all layers it takes several clicks.
Instead at the top of the menu there should be a "turn on all" shortcut checkbox.

In my opinion it's becoming increasingly cluttered to have all layers enabled at once, so I'm unlikely to add this feature.


If you could set a shortcut for 5 sec back would be nice.

sure, I can add that next

This is complete. Please download v1.4.0


More tests

Coverage remained the same at 80.284% when pulling 6908308d0fd1e473be5d75f56ffaea1c2568ee54 on more-tests-180911 into 2502e9871ec007ccd71f918d54bd0d7a9449c07a on master.


Training resources during backtests not refilling over time

Expected Behavior
Training resources for the Train method should refill the leaky bucket during backtests based on simulated algorithm time, similar to live limits.
Actual Behavior
Training resources seem to be limited to 60 minutes/backtest, and throws the error after exceeded:
Runtime Error: Algorithm took longer than 10 minutes on a single time loop. CurrentTimeStepElapsed:

Reproducing the Problem
# Reproducing Timeout errors from self.Train

from QuantConnect import *
from QuantConnect.Algorithm import *

import time

class TestMLTrain(QCAlgorithm):

    def Initialize(self):

        self.SetStartDate(2019,1,1)
        self.SetEndDate(2019,6,1) # Since we train every week, at least 4 weeks needed to hit time limit
        self.SetCash(1000)

        # Test on this asset
        self.AddEquity('SPY', Resolution.Minute)

        # Model will be trained every Sunday at 6:30
        self.Train(self.DateRules.Every(DayOfWeek.Sunday),
                   self.TimeRules.At(6,30),
                   self.TrainModel
                   )

    def OnData(self, data):
        pass

    def TrainModel(self):
        """
        Simulate a training process.
        """

        def MyModel():
            """
            Some computational expensive stuff here.
            """
            time.sleep(900) # Simulate training for 15 minutes

        start = time.time()

        self.Log('Training model...')

        MyModel()

        end = time.time()
        elapsed = end-start

        self.Log(f'Training completed in {elapsed:.2f} seconds.')

Logs:

2019-01-01 00:00:00 : | Launching analysis for 944c266695c7377cc8565683e1256ea6 with LEAN Engine v<IP_ADDRESS>.8476
-- | --
2019-01-07 09:31:00 : | Training model...
2019-01-07 09:31:00 : | Training completed in 900.01 seconds.
2019-01-14 09:31:00 : | Training model...
2019-01-14 09:31:00 : | Training completed in 900.01 seconds.
2019-01-22 09:31:00 : | Training model...
2019-01-22 09:31:00 : | Training completed in 900.01 seconds.
2019-01-28 09:31:00 : | Training model...
2019-01-28 09:31:00 : | Training completed in 900.03 seconds.
2019-02-03 06:30:00 : | Runtime Error: Algorithm took longer than 10 minutes on a single time loop. CurrentTimeStepElapsed: 5.0 minutes
2019-02-03 06:30:00 : | Algorithm Id:(944c266695c7377cc8565683e1256ea6) completed in 3901.89 seconds at 0k data points per second. Processing total of 17,324 data points.

System Information
LEAN master v8476 on QC Cloud

Also seems like scheduled training outside of market hours are delayed until market open. In the example, TrainModel() is scheduled every Sunday at 6:30 AM but logs indicate training starts Mondays at 9:31 AM.

Hey @adamw96! Actually by design, it's currently refilling based on realtime, not algorithm time (related discussion https://github.com/QuantConnect/Lean/pull/3743#discussion_r336588789), using algorithm time would mean almost unlimited CPU time in practice since time advances so fast

Thanks @Martin-Molinero - glad to see there's been a discussion on this, but wouldn't refilling on algorithm time make the backtest behavior more consistent with live behavior?
For example, let's suppose in a backtest an algorithm calls .Train() every simulated week, with each call ~15 minutes. Since very few backtests take longer than 24 real hours, we assume no refills for now which implies a fixed 60 minute training capacity per backtest.
If we backtest on 1 month of data, then .Train() gets called 4 times. This means that each call has an effective training capacity of 60/4 = 15 minutes in order for the full backtest to run without throwing a timeout error. However for longer backtests:
Backtest period: 3 months
**Number of Train() calls: ** 12
Effective training capacity per call: 5 minutes
Backtest period: 1 year
**Number of Train() calls: ** 48
Effective training capacity per call: 1.25 minutes
Backtest period: 5 years
**Number of Train() calls: ** 240
Effective training capacity per call: 15 seconds
This implies that for any backtests longer than 1.5 months with regular weekly training, using .Train() results in a lower effective training capacity compared to the 10 minute limit of .OnData(). As such, currently .Train() should only be used for one-time/infrequent calls (i.e. to initialize an algorithm) during a backtest.
As for limiting real CPU time, the current upper limit for a backtest in theory is 10N where N is number of minutes to be backtested (i.e. using the full 10 minutes per minutely call to .OnData()). Suppose the leaky bucket algorithm is 60 minute capacity + 10 min/24 hrs. With refilling based on algorithm time, the upper limit is ~(N-60)*0.0069 which is about 1449x lower than the .OnData() upper limit for any N. If the goal is to limit real CPU time, wouldn't it make more sense to simply add a fixed time limit (i.e. backtests must be completed in 12 hours or less)?

Closing as was before QC made the shift to Organizations and AWS-style resource nodes. Now larger nodes have more significant training limits. It's still based on real-time as basing on backtest time would be practically infinite length backtests.


Carbon gray tag is not displaying correctly

Description
Environment: Product
When I add a Carbon tag to a component and pick the color "gray" in the dropdown, the tag will display as green tag, not gray. This needs to be updated so when gray is selected as the tag color, the tag displays as a gray tag.
This bug seems to appear where ever Carbon tags are being used and isn't component specific.
Component(s) impacted
Components: Several including structured list, CTA section with static card, pricing table, and where ever Carbon tags are being used
Mainly the Tag Group component
Browser
Chrome, Safari, Firefox, Microsoft Edge
Carbon for IBM.com version
latest
Severity
Severity 3 = The problem is visible or noticeable to users but does not impede the usability or functionality. Affects minor functionality, has a workaround.
Application/website
IBM.com AEM Sites
Package
@carbon/ibmdotcom-web-components
CodeSandbox example
?
Steps to reproduce the issue (if applicable)
https://prod-author.roks.cms.cis.ibm.net/content/adobe-cms/language-masters/en/prod-sandbox/testproduct-overview-jw/-test-product-feature—jw.html?wcmmode=disabled
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate issues

@sirkitree Hi there! I'm unable to reproduce the behavior you are mentioning. In the Storybook page, you can change the color to grey and it does not revert back to green.
Likewise, in our Tag Group story, you can use individual Carbon tags with different colors, including grey, and they don't default to being green.
That being said, if the type isn't set, the tag will default to green.
For example


dotnet run

Elizabethrigney and her family were locking together for the next levelbeforeshehadtalkedwithhermotherabouttheprocessofmovingonlne xtcoupleSystem Information (please complete the following information):

Model Builder or CLI Version:
Visual Studio Version (if applicable):

Describe the bug

On which page of the tutorial did you run into an issue (URL):
Clear description of the problem:

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Sorry @Youstarter I don't have enough information to investigate this issue. If you're having a problem please answer the questions, and include screenshots. Thanks!
Closing - happy to reopen if you're able to provide more info.


Exceptions thrown by Executor are wrapped in UndeclaredThrowableException.

Hello,
I've been trying to use ThreadPoolExecutor with limited queue size for REST requests processing. Unfortunately, with tracing enabled RejectedExecutionException is wrapped into UndeclaredThrowableException.
I've googled a little and I've found similar bug raised for spring-cloud-sleuth.
Here is a stack_trace.txt I get when this happens.
I'm using version 0.3.1.

hi @ZawadaZSE, thanks for reporting this issue.
Would you like to submit a patch for it?

Sure ;)

The issue is fixed, any idea when can we get this merged?

Done in #226.
thanks @ZawadaZSE.
I will cut a new version now.

I've noticed that it has been published in the snapshot version. Can we release it as a regular?

my bad I have pushed a wrong tag, triggering release again


remove unused file

review by @adriaanm. if this might havesome value in the future, I
suggest we make an issue documenting that (and linking to the commit
with the removal), rather than just leaving the file lying around.

LGTM -- cleanliness ftw


add redirect instructions to the readme

how to do 301 redirects should be documented in the Readme. See https://github.com/m-lab/m-lab.github.io/issues/65 for details

add details about adding new blog posts, date index pages and category pages
how to do 301 redirects should be documented in the Readme. See #65 for details


Added commands to list messagingSettings, memberSettings and guestSettings.

Addressing #810, #811 and #812

Coverage decreased (-0.09%) to 99.908% when pulling 98c2d5a9348b0d26bb6884042ce0549cfeea8979 on vman:getteamssettings into 8762b1df83812151ff527ee1df0070b4e1ef5d09 on pnp:dev.

Thanks! I'll have a look shortly!

Merged manually. Thank you! 👏


google gemini support function call

💻 变更类型 | Change Type

[x] feat
[ ] fix
[ ] refactor
[ ] perf
[ ] style
[ ] test
[ ] docs
[ ] ci
[ ] chore
[ ] build

🔀 变更说明 | Description of Change

📝 补充信息 | Additional Information

Summary by CodeRabbit

New Features

Enhanced streaming capabilities for chat interactions, allowing for more dynamic responses.
New methods added to the API for usage tracking and model retrieval.

Improvements

Improved logic for displaying plugins based on service provider and model type.
Enhanced handling of request options in the fetch function.

Bug Fixes

Refined error handling in the streaming process to prevent unnecessary processing.

Refactor

Streamlined internal logic for managing tool calls and responses in the chat functionality.


Add documentation for Lisk Service

Description
The documentation content specific to Lisk Service should be added to the lisk-service repository under the docs/ folder.
Motivation
Because the content is closely connected and it improves overall management of the Lisk documentation as a whole.
Acceptance Criteria

Documentation is up-to-date
Documentation is programmatically generated
Documentation can be rendered on the official website straight from the repo

Additional Information
Epic: https://github.com/LiskHQ/lisk-docs/issues/1293

The documentation will be addressed after the initial alpha is tagged so that we already have the API properly-defined and all the necessary information is in-place. This would ensure that we do not need re-work on the documentation.

This issue is a duplicate https://github.com/LiskHQ/lisk-service/issues/1012

https://github.com/LiskHQ/lisk-service/issues/1012


MeshView for developmental atlases

For visualization and analysis in the DOPAMAP project, we would be interested in having MeshView versions for the developmental atlases. In particular, we have used the P14 and P28 atlases found here: Z:\HBP_Atlasing\Developmental_atlases\DeMBA_Developmental mouse brain atlas\DeMBA-v1\01_working-environment\01_Data\Resampled-reoriented-volumes_DeMBAv1-beta and would appreciate MeshView versions with these.
Additionally, it would be good to have versions where the point clouds can be cut from two sides (I have only heard of this feature from Martin / Trygve) so apologies if my description is unclear.

Issue merged: https://github.com/Neural-Systems-at-UIO/MeshView-for-Brain-Atlases/issues/52


use custom index.html

Version
3.0.0-alpha.5
Reproduction link
https://github.com/greenpdx/rustwasm
Steps to reproduce
npm install
npm run serve
What is expected?
I added a index.html in the root directory, I want to use that html file
What is actually happening?
the server is generating it own index.html

I am trying to load a opencv as wasm file inside a js file. I run out of memory if I try to import it because webpack tries to parse and optimize it. The example in opencvjs shows it added in the html file.
I want to use a custom index.html and add it there.

index.html is in the public directory.

@yyx990803 What's the solution if I want to specify a different custom HTML file? For example, in my project I have a demo/ subdirectory which is essentially a small project in its own right. In my project root directory, I define a dev npm script in package.json: vue-cli-service serve demo/src/main.ts.
I want my demo to use a custom HTML file that I've created as <root>/demo/public/index.html. However, running npm run dev makes it bundle <root>/public/index.html instead.

@TAGC for now you will have to use an env variable and conditionally modify the options for the HTML plugin in vue.config.js.

Ah okay, fair enough.

@yyx990803 Which env variable and options?

@hguiney This worked for me
https://github.com/TAGC/vue-expandable-grid/blob/5a3f4562d5151989a71311ba5f16b4d0949a2bc4/vue.config.js#L1-L16

if I wrote npm run dev automatically opened app.vue. But i want change this, namely if I wrote npm run dev in my cmd then opened 'blogs.vue'

Thanks @TAGC, very helpful. I have a similar project works too. For someone who stuck at this place, please also refer to this repo:
https://github.com/tjcchen/vuex-best-practice

Thanks @TAGC, very helpful. I have a similar project works too. For someone who stuck at this place, please also refer to this repo:
https://github.com/tjcchen/vuex-best-practice


Translation strings

Is there a performance reason for not being able to set hideText/showText as options rather than having to override them in a subclass?
Asking since i18n by passing-as-an-option seems to be standard among the Leaflet controls I've encountered.

Not any real big reason, no. It was just the way the original contributor did it  way back when. I see that there is a https://github.com/yohanboniface/Leaflet.i18n plugin out now though, so it might be worth looking at that, I suppose. It's not real high up on my priority list since we only have two strings, but pull reqs are welcome.

I'll happily put together a PR for it. Would the existing subclass method still be desired? I.e. need a patch be backwards compatible when it comes to this?

Nah, since we never really had more than one translation added it won't matter to me if we make this a breaking change. I'll just bump the version number and be happy with that.


Android Basics: lists

URL of codelab
In which task and step of the codelab can this issue be found?
Kotlin course - Mutablelist- list of orders
Describe the problem
unresolved reference: addItem
unresolved reference: addItem
Steps to reproduce?

Go to...
...continues
val order4 = Order(4).addItem(Noodles()).addItem(Vegetables("Cabbage", "Onion"))
ordersList.add(order4)
ordersList.add(

Order(5)
.addItem(Noodles())
.addItem(Noodles())
.addItem(Vegetables("Spinach")))
for (order in ordersList) {
    order.print()
    println()
}

}

Click on...
See error...
the item in BOLD (addItem) get red and it appears this messages
Unresolved reference: addItem
Unresolved reference: addItem

Versions
What version of Android Studio are you using?
Additional information
Include screenshots if they would be useful in clarifying the problem.

@mikfas
Thank you for reaching out to us. The snippet of your code is totally correct.
I suggest to double check this part of your code:
class Order(val orderNumber: Int) {
private val itemList = mutableListOf()
fun addItem(newItem: Item)  : Order   {
    itemList.add(newItem)
    return this
}

Probably you missed :Order and return this
If it doesn't help please share with us the snippet of the whole code not just the fragment.


Nudge for Ventura Delta updates

As you may already know, macOS Ventura will be offered as a delta update on macs running 12.3+. On managed machines, it will be offered as a delta after 30 days of Ventura release date.
While testing Nudge on an un-managed mac where I was able to see the delta update, I noticed that when I have attempttoFetchMajorUpgrade key set to false Nudge will show the following in the logs:
_Device requires major upgrade but attemptToFetchMajorUpgrade is False - skipping download
 Unable to find major upgrade application, exiting Nudge_

If I set attempttoFetchMajorUpgrade key to true, Nudge downloads the full app installer. I also noticed that Under the software update preference pane, offered update also changed from Delta to full app installer as soon as Nudge started downloading the installer. This is not ideal because standard user accounts on macs would not be able to perform this upgrade since the full app installer requires admin rights to install.
Basically, on MDM-managed macs, after 30 days, devices running 12.3 to 12.6 will treat the Ventura update as a minor update.
Nudge needs to be able to present users with just the software update pane without actually expecting an app installer bundle to open for a major upgrade. OR Nudge needs to be able to download the delta update as it can do with minor updates.

I'd clarify that once we get through this one-time-only initial 30 day delay of the delta-upgrades:
"macOS 12.3+ will treat the Ventura upgrade the same a minor update with no app needing to be downloaded."

https://github.com/macadmins/nudge/blob/main/Nudge/Utilities/SoftwareUpdate.swift#L56
This is because Nudge calls softwareupdate with the --fetch-full-installer option when looking for a major upgrade. I missed this during my testing.
I think in this case add another code path to check the required OS > =13 and call softwareupdate with the delta option instead, although I'm not sure on the actual verb for that.
Maybe there is a way we can detect softwareupdate is capable of offering delta updates.

Hi for jamf users there can be a workarround. You can use the actionButtonPath to run jamf self service policy and with that policy you can open software update from the System Settings :D
@dan-snelson has posted a working config.
https://macadmins.slack.com/archives/C02KPHZ7MDX/p1666831304360389?thread_ts=1666817887.774279&cid=C02KPHZ7MDX

@colorenz sure we can do that, but the whole point of delta updates is we dont have to push out 12GB installers

With that workflow you can use it also for the delta Updates. In you jamf policy you run that command to open System preferences.
open 'x-apple.systempreferences:com.apple.preferences.softwareupdate'

Infact if you don’t want to open Jamf you can just use the path to the Softwareupdate.prefpane as the url.
/System/Library/PreferencePanes/Softwareupdate.prefpane I believe. I tested it today and it works all the way back to at least 11.x

@colorenz @Lotusshaney thats a great workaround idea! let me give that a shot today!  Thank you!

With that workflow you can use it also for the delta Updates. In you jamf policy you run that command to open System preferences.
open 'x-apple.systempreferences:com.apple.preferences.softwareupdate'

I love how this method actually brings it to the front, and without any errors! Kudos!

Any suggestions for tackling this within nudge instead of workarounds?

@erikng How complicated would it be to have Nudge use the current OS version to evaluate how to proceed?
If 12.3 or higher use the same workflow for both minor updates and major upgrades.
If 12.2.1 or lower behave like it currently does.

Unsure but doesn't it also depend on how many days it's been since the OS released + if device is mdm enrolled?

The MDM enrollment changing the behavior of delta upgrades was a one-time thing that has now expired, so we don't have to consider that any longer.

I encountered this exact issue today where I wanted users to use the Ventura delta update if available, I initially tried to configure Nudge to not attempt to download the major update, but quickly discovered that just caused Nudge to skip showing its UI if it detected the upgrade was a major upgrade.
Thanks to the suggestions here I was able to develop a slightly different workaround, which is to set majorUpgradeAppPath to /System/Library/CoreServices/Software Update.app. This has the affect of both causing Nudge to not attempt to download the full installer as the configured path already exists on disk, and directs the user to (loads) the Software Update settings pane when Update Device is clicked.
With this I achieved exactly the behaviour I was looking for.

I believe I have solved this issue now. I will cut a new release sometime this week. You should no longer have to set these keys to false and just use it out of the box.


Avg detect last update as "IDP.Generic" virus

Even adding the .exe  as exception i can't install  it ( i deleted it after the update went wrong,file blocked)
No idea what to do, i'll try an early version.
PicoTorrent_20191024225439.log
Log attached

Hi! Thanks for the report - I'll check if I can reproduce this in a clean Windows 10 VM.

Hey man, got the installation fixed after a quick restart, after that, when tryng to launch it, it says "VCRUNTIME140_1.dll" missing
Downloaded the .dll, placed in the directory, working (last update)
Probably an issue by my side, something blocking the installation but no idea, the anti virus seems "quiet" about it now

@Ziffax That's a known issue and has been fixed but not released (yet - I'm working on that release now). You can try the installer from the AppVeyor build until then.
https://ci.appveyor.com/project/picotorrent/picotorrent/builds/28361302/job/kxdpm79bb8ibr4v9/artifacts

NP all good now, thanks for your time, love your program.


Do you also get crashes since 2.0 ?

upload_crashed

those crashes seem to be just you, I'm using this unlocker with 2.0 and it's running fine.

those crashes seem to be just you, I'm using this unlocker with 2.0 and it's running fine.

same, I started to play a little bit more than a week and even 10-hour sessions don't have a crash.


pcre library needs to be upgrade to pcre2

Current issues are:

pcre library is no longer maintained; author working on pcre2
pcre jit support in ATS is incomplete

https://issues.apache.org/jira/browse/TS-3143
https://issues.apache.org/jira/browse/TS-3144

Precompilation is not consistently implemented
Library review

Should probably look at pcre2/je2 (or others)

Whilst looking at some high RPS rate tests libpcre was starting to 'feature' in traces.
The obvious optimisation for libpcre is to use the jit support, which probably lends itself well to constrained lists of regex.
However, given the lack of maintenance on the current library an "architectural review" is probably required before more effort is spent...

It looks like trafficserver precompiles the expressions, replacing the need/benefit of the jit…  Perhaps someone can confirm…
So really this may just be a “what’s the long term plan” question… Given status of pcre project…

There is a new Fedora tracking bug for this issue: https://bugzilla.redhat.com/show_bug.cgi?id=2128391

Just to tell the bot this is not stale, we are going to do this by version 10.0 release.

On the corresponding Debian bug, there is a patch available.

libpcre3-dev has been removed from the Trixie repositories, so trafficserver can no longer be built on Debian testing.


Add Forge Binary to NPM

Component
Forge
Describe the feature you would like
I would like to be able to download forge as a binary npm package for easier integration into Frontend and Node applications.   As an example, if forge is a part of a build pipeline it becomes very awkward to integrate it into a vercel build.
I am happy to contribute to making this happen but I would need some guidance on

Is it even possible to package up forge into a binary?   Would I have to try to package up foundry-up and then execute foundry-up?
A pointer to where to look in foundry to get started

Additional context
My specific use case is Iam working to build a forge inspired ethers.js alternative.  User would be able to write forge scripts.  The API is similar to this
import { mutate } from 'ts-sol'
import { TransferEntireBalance } from './ERC20.s.sol'

....

<button onClick={() => mutate(TransferEntireBalance, { signer })} />

I want to reuse forge as much as possible but if I can't find a way to get forge on NPM I'm likely going to be forced to reimplement cheat codes into a javascript based VM or hardhat.

there's https://github.com/foundry-rs/hardhat/tree/develop/packages/hardhat-forge
and https://github.com/foundry-rs/hardhat/tree/develop/packages/easy-foundryup

Interesting I don't think hardhat-forge is super useful.  We use that at OP Labs and I believe foundry is still a required dependency for it to work.
Easy foundry-up looks promising though.  I will dig into that more later

@tynes any thoughts on this issue?

Tried using easy-foundryup in vercel and looks like there are some issues getting foundry to do it's thing on vercel likely because of missing deps in the container

```
foundryup: done
--
09:09:43.978 | forge: /lib64/libm.so.6: version `GLIBC_2.27' not found (required by forge)
09:09:43.978 | forge: /lib64/libc.so.6: version `GLIBC_2.29' not found (required by forge)
09:09:43.979 | forge: /lib64/libc.so.6: version `GLIBC_2.28' not found (required by forge)
09:09:43.979 | forge: /lib64/libc.so.6: version `GLIBC_2.27' not found (required by forge)
09:09:43.981 | ELIFECYCLE  Command failed with exit code 1.
09:09:44.000 | Error: Command "pnpm run build" exited with 1
```

I think the more general question here is is it possible to package up forge into a binary such that it doesn't require external dependencies to execute?

Vercel is likely using musl. You either need to install glibc or follow these instructions: https://github.com/foundry-rs/foundry#out-of-date-glibc-error-when-running-forge-from-default-foundryup-install
You will not be able to use Forge as you described in your issue though, as Forge is not distributable on npm and is not importable as a module in JavaScript, as we do not support wasm.

@onbjerg you don’t need to be javascript or wasm to be on npm.   Any binary can be on npm

Also I’m trying to use forge as a build tool.    Using it in the browser, that isn’t what I’m trying to do (yet).    I am trying to build a way of using forge like cheat codes in the browser but I’m building that myself in top of Ethereum js.   Forge is for building contracts at buildtime not runtime though

Thanks helpful to know the timeline.   What I think I will do is use that toolchain to create a NODE18 docker image that anybody can build in for now.   It's not perfect, but hopefully works for most folks

FYI you don’t need to bundle anything. You can publish the repository exactly as it is to npm and Github Packages.

Wouldn’t that mean you had to build it though?   It takes a very long time to build

Yes, I was commenting on the fact you that bundling is not a requirement to publish to npm or similar registries.
I think having an action that publishes the precompiled binaries which are available in the release page would be nice.

I think it's worth reopening this issue, or otherwise finding a way to enable the following workflow:
I'm using the Wagmi CLI foundry plugin to generate app code from my foundry project. I want to do this at build time on Vercel, so that configuration (like deployed contract addresses) is always up to date.
I imagine this is one of the most common dev setups right now and it doesn't seem like it's possible.

I think it's worth reopening this issue, or otherwise finding a way to enable the following workflow:
I'm using the Wagmi CLI foundry plugin to generate app code from my foundry project. I want to do this at build time on Vercel, so that configuration (like deployed contract addresses) is always up to date.
I imagine this is one of the most common dev setups right now and it doesn't seem like it's possible.

Indeed, I have the same issue. The only alternative now seems to be committing the generated files which is suboptimal.

I agree this issue is still worth reopening. NPM is for distributing anything not just javascript


Make an official No Chat Reports plugin for Bukkit or Spigot servers

Idea
Make an official No Chat Reports plugin for Bukkit or Spigot servers developed officially by @Aizistral
Reasoning
Good to have official plugin by @Aizistral to be compatible with Bukkit and Spigot
Other Information
No response

Fabric, Forge and Bukkit/Spigot platforms are very different, so you can't expect one dev to do all of them. Have a look at https://github.com/Aizistral-Studios/No-Chat-Reports/wiki/Protecting-Server-Players for various options that are already available.
Duplicate of #11


Create _redirects

👋

Right now if you directly link to a tag page (e.g. fun) the request will fail. This adds a redirect rule so Netlify always serves the index.html.

Thank you so much! :pray:

Not sure if you if you use apache locally for development, but if not, you can remove the .htaccess file too 😄
Thanks for the great resource!


Save profile output to file

Background
It would be really great to be able to save the profiler output to file instead of just printing on the screen. This would be a big help when comparing simulation performance across large number of runs.
Proposed solution
Save profiler info to csv file. The "Core profiler results" could be a table with three columns and the "Model profiler results" could be a table with four columns. These two tables could be stacked on top of each other in a single csv or there could be two separate csvs. Saving to a .txt file would also be fine.

Outputting the profiler results to CSV and JSON does not appear to work.
The following commands do not work: gridlabd xxxxx.glm --profile=CSV and gridlabd xxxxx.glm --profile=JSON
The following command does output the profiler results to a text file named gridlabd.pro: gridlabd xxxxx.glm --redirect profile


Update GEP-1742 to use GEP-2257 Duration type

What type of PR is this?

/kind gep
What this PR does / why we need it:
Completing the update that @kflynn started in #2316.
This PR includes Flynn's changes.
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

Thanks @frankbu!
/check-cla
/ok-to-test

/easycla

Thanks @frankbu!
/lgtm
/approve


Added Documentation for Number of Divisors

📚 Documentation
(A clear and concise description of what the issue is.)
Have you read the Contributing Guidelines on Pull Requests?
YES.

Please assign this to me under GSSOC'21
@HarshCasper

@iamrajiv  Kindly assign this to me


Portfolio-pull request

Adding Readme file
Adding .gitignore file
Adding .github file
Adding .hintrc file
Adding Stylelinterc.json
adding linters set up
Adding HTML&CSS file
Compare and Pull request
Adding reviewed HTML & CSS file
Adding updated Readme file
Create new pull request

Invalid Code Review Request ❌
Hi @Hazard10-eng ,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
You have already opened and reviewed pull request(1), Please use the same pull request link to request a review.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Additional repos and url validation

Added the following functionality:

add/remove additional repositories
front-end URL validation: each repository url must be a valid url; feedback to user generated if invalid.
back-end URL validation: if a url in the request is invalid, the original repository list doesn't change.
added a new npm package called valid-url

Still working on moving some of the code into a custom directive, but so far it's functional!

Hey @agundy or @kmcnellis, could you take a look at this when you have time (no rush)? I'm not sure what is causing the conflict, I've never had that happen to me before.

Lets tackle this during the small group

Looks good! ready to merge once travis finishes


Bug: @metrics.log_metrics decorator does not accept additional arguments passed to wrapped function

Expected Behaviour
The function wrapped by @metrics.log_metrics decorator (e.g. lambda handler) should be able to accept arguments other than the event and lambda context.
Current Behaviour
If an additional positional argument is passed to the wrapped function, the following error is encountered:
TypeError: lambda_handler() takes 2 positional arguments but 3 were given
If an additional keyword argument is passed, this error is raised:
TypeError: lambda_handler() got an unexpected keyword argument 'additional_arg_name'
Code snippet
@metrics.log_metrics
def lambda_handler(event: dict[str, Any], context: LambdaContext, additional_arg_name: Any):
    ...

# This causes a wrong number of positional arguments TypeError
lambda_handler({}, {}, "additional arg value")

# This causes an unexpected keyword argument TypeError
lambda_handler({}, {}, additional_arg_name="additional arg value")

Possible Solution
The decorate() function for log_metrics should take in *args and **kwargs as arguments and then pass them to the lambda_handler() call.
@functools.wraps(lambda_handler)
def decorate(event, context, *args, **kwargs):
    try:
        response = lambda_handler(event, context, *args, **kwargs)
        ...

A similar issue with the @logger.inject_lambda_context decorator was fixed in #1276
Steps to Reproduce
This can be consistently reproduced using the code snippet above.
Powertools for AWS Lambda (Python) version
latest
AWS Lambda function runtime
3.10
Packaging format used
PyPi
Debugging logs
No response

@thegeorgeliu merged 🎉
Let us know if this can't wait until tomorrow for a bug fix release.
Thank you for your first contribution and huge thanks for making both bug report and PR so easy to review ;)

Thanks @heitorlessa! No worries, there's no issues with waiting till tomorrow. Glad the PR made the cutoff for the release!

Great!! FYI - for bugs that have no workaround and block multiple customers, we make a bug fix straightaway (typically ~30m after merge).
Hence the ask ;)
Will ping back upon release.
Have a great evening

Closing as complete https://github.com/aws-powertools/powertools-lambda-python/issues/3119