Host not accept issue when after the first game

@TIRTAGT so this issue happen, i try to figure it out why, after the first game (no matter play the hole game or not), quit to main menu and create the game again, this happen
using <IP_ADDRESS> and port 7777, no ipv6 listening (old released version), my Dark Hour Modpack code: 018db5f6-c8d3-c090-1da2-fcb97f6b6070

@TIRTAGT so this issue happen, i try to figure it out why, after the first game (no matter play the hole game or not), quit to main menu and create the game again, this happen using <IP_ADDRESS> and port 7777, no ipv6 listening (old released version), my Dark Hour Modpack code: 018db5f6-c8d3-c090-1da2-fcb97f6b6070

Thank you will try to test this, does this happen if you only have LCDirectLAN enabled too ?
I'm a bit worried that this may be because of MoreCompany that you have, they do not support LAN game and actually breaks joining capability when I'm playing with my community, ended up using BiggerLobby instead.

Vanilla with this mod only doesn't have this issue, so that might be MoreCompany issue, recently they add a function to join LAN session using specified IP address, and can use MoreCompany via LAN

https://github.com/notnotnotswipez/MoreCompany/issues/11

Vanilla with this mod only doesn't have this issue, so that might be MoreCompany issue, recently they add a function to join LAN session using specified IP address, and can use MoreCompany via LAN

It looks like I cannot even start the game using your ModPack, stuck on "Waiting for crew..." while TooManySuits are having an error party.

TooManySuits is spamming the log more or less about every frame making it hard to even read anything before it getting erased out:

My log file after a few minutes of "lobby" gameplay had 146k lines 😮

I will try to disable anything except for LCDirectLAN and MoreCompany for testing.
It would also be great if you can provide me with "as little mod as possible to reproduce the issue"

@TIRTAGT uh u might need to restart the game once, bc UI interface have something wrong i guess but, will test after a dinner

@TIRTAGT uh u might need to restart the game once, bc UI interface have something wrong i guess but, will test after a dinner

Tried restarting a few times already, didn't work but after disabling mods one-by-one it looks like the issue I experienced is because of Lethal Admin, specifically the Require Steam configuration.
 Because your ModPack contains a lot of mods that I'm not really sure where to start looking for incompatibility.

To simplify the investigation process, can you try with as little mod as possible to reproduce the issue ? Because your ModPack contains a lot of mods that I'm not really sure where to start looking for incompatibility.

yes uh a, still working on it, give me an hour

To simplify the investigation process, can you try with as little mod as possible to reproduce the issue ? Because your ModPack contains a lot of mods that I'm not really sure where to start looking for incompatibility.

yes uh a, still working on it, give me an hour

Thank you, no rush.
Maybe try checking if the issue still exists with just LCDirectLAN + MoreCompany (without any other mods active), I cannot test further at the moment because of work stuff, but if the issue do still exist with just both mods active, then that makes it extremely easy for finding the incompatibility as it rules out any other mods in the ModPack.

@CoolLKKPS I saw your comment on your MoreCompany issue report, if you are simply looking a way to have a bigger lobby capacity (without Cosmetics and others stuff bundled together), LCDirectLAN have a great compatibility with BiggerLobby.
Hopefully in the future, the LCDirectLAN's Compatibility wiki will be populated with more mods to make it easier to find mods that will work just fine with LCDirectLAN.
So far, me and my community find BiggerLobby to be "it just works", and as a bonus it keeps the original/vanilla Lethal Company looks too.

yea, i still testing, pretty sure this is not conflict with MoreCompany, i still try to find the conflict

T T really fixing issues is so... so boring, and time smasher

i almost got it, bear with me

@TIRTAGT got it, conflict with this mod: https://github.com/NicholasScott1337/RuntimeNetcodeRPCValidator

@TIRTAGT got it, conflict with this mod: https://github.com/NicholasScott1337/RuntimeNetcodeRPCValidator

Aha, so the issue only seem to occur when you have LCDirectLAN + RuntimeNetcodeRPCValidator ? not related to MoreCompany ?

Aha, so the issue only seem to occur when you have LCDirectLAN + RuntimeNetcodeRPCValidator ? not related to MoreCompany ?

yup, only Runtime_Netcode_Patcher

still working on my modpack, so if u need me, u can call me again
and uh im not sure where's gone wrong but uh, should i create an same issue in there?

RuntimeNetcodeRPCValidator

Nice thank you, will investigate later when I can :)

and to make the things easier: host create a lobby, client join, host pull the lever, and after land the host quit to main menu, and host re-create a lobby, done

still working on my modpack, so if u need me, u can call me again and uh im not sure where's gone wrong but uh, should i create an same issue in there?

For now, I'd recommend not, because after looking through that mod's README, it is an API which if changed just to be compatible with LCDirectLAN may potentially break a LOT of other mods, which is actually against what LCDirectLAN is targeting.
We'd try to check if it is possible to make it compatible with LCDirectLAN, or if the issue is confirmed to be on their side then we ask them, this is because LCDirectLAN is not 10/10 perfect, in fact I only took a few days to learn Unity Game environments before starting this project lol

haha ok, so i going to focus on my modpack now, let's go @TIRTAGT !

and to make the things easier: host create a lobby, client join, host pull the lever, and after land the host quit to main menu, and host re-create a lobby, done

So basically after the host ~rage-~ quitted the lobby while another client is in, and then trying to host a new lobby, anyone trying to join the new lobby get "The host was not accepting connections" ?
Just to confirm, are you testing with Allow Remote Connection or Local options ? Because I will need to be able to reproduce the problem, basically find the problem and start experimenting why that problem exists in the first place.
I actually have some theory that the problem is because I had [ServerRpc] and [ClientRpc] in LCDirectLAN's RPC, those aren't needed because LCDirectLAN directly manages data transport on its own, but I left it there so I remember which code is Server side and which code is Client side, haha.

uh im running 2 clients in the same desktop, so after the ship landed, i just do normal quit to menu, previously i just trying to find compatibility but, yea... this happen, re-create a lobby in LAN mode still, and let the client try to rejoin, and error, no matter Allow Remote Connection is on or off

and that take a lot the time to fix compatibility issue, and now i know that modpack have huge issues, can cause savefile corrupt and weather de-sync, and many more... my g...

Thank you for the information, that will really help when I finally have the time to investigate (which hopefully is soon)

np also enjoy with that >;)

and that take a lot the time to fix compatibility issue, and now i know that modpack have huge issues, can cause savefile corrupt and weather de-sync, and many more... my g...

Yeah unfortunately when combining a lot of mods, you will eventually find 2 or more mods that will modify/patch a game function. The problem is not because that they modify/patch those functions, but how they modify it.
I've seen some that tries to modify the game's code directly, or even replace the entire function, while sometimes this is required for them to work, doing this may introduce a lot of issues on other mods that didn't expect that change.
LCDirectLAN actually is designed like that at my first try since I just want it to be done as fast as possible and didn't even plan to publish it to the world, but I'm glad I spend more time finding another way to achieve what I want with modifying minimum game code as possible. A bit surprised that after adding LCDirectLAN to my community modpack, "it just works".
So hopefully for other people that uses LCDirectLAN, the usage experience is the same as what I experienced.

I've heard about some issues in the game itself, where what the code tells u doesn't equate to what it actually is, so i can completely understand why u will think about that, pretty sure we need some people like PAYDAY 2 "The Fixes" mod to fix many game issues, so we can base on that code and tools to create more good stuff without need to think too much

anyway, thx for ur mod! really, this mod will keep in my modpack for a long time, as long as it still working, even i know still have something can improve like link 3rd-party account for ID verify, but that doesn't really matter for now

I have made some progress ruling out the incompatibility, because even after several testing with only LCDirectLAN + Runtime Netcode Patcher, the issue didn't happen but running your setup (imported in r2modman-Plus using your code) does reproduce the issue.
So at least I know that one (or a combination) of the mods included in your modpack is causing the incompatibility, will continue testing later (because it's 4 AM at the moment).

...... its More Emotes, uhhhhhhhhhhhhhh, got to sleep now............

I'm still investigating the issue, that is a bit weird because More Emotes doesn't seem to modify the approval connection at all.
So, I guess it's not, but nothing can be proven yet, a few mods more to go....

uh maybe... i really tired, maybe tomorrow

uh maybe... i really tired, maybe tomorrow

Yup no worries, I'll try investigating the issue.
So far it looks like a combination of mods, but I'm trying to narrow down the list further to make it easy for me.

Found some interesting code that might be cause of the problem, located in BetterLobby, will try to mess with the code a little and see if removing that specific code actually fixes the problem, lol.
I didn't believe that actually worked, let me try adding that BetterLobby code back and see if the issue returns... Performing sanity check to see if I'm on a dream or removing that interesting line of code actually fixed this issue.....

ok will have a look, starting to update my modpack >:)

Looks like that fix is confirmed, the problem is because of this patch in BetterLobby:
[HarmonyPatch(typeof(StartOfRound), nameof(StartOfRound.OnDisable))]
[HarmonyPostfix]
static void DisconnectOnDisable()
{
    GameNetworkManager.Instance.Disconnect(); // weeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee
}

The Disconnect() temporarily forbids any player to join by setting disallowConnection variable to true, since if the host is trying to disconnect anyone, new players shouldn't join which makes sense.
After the host lobby has been closed/disabled, that variable is set to false so the next lobby the host created will accept connections like usual....
But it looks like due to bad timing, BetterLobby called Disconnect() after GameNetworkManager is already disconnected (the game also calls Disconnect() on its own), the function crashes:
[Error  : Unity Log] NullReferenceException: Object reference not set to an instance of an object
Stack trace:
RuntimeNetcodeRPCValidator.NetcodeValidator.NetworkManagerShutdown () (at <0df0f351a5d94cbb9af1af6290fbdcfa>:0)
RuntimeNetcodeRPCValidator.Plugin.OnNetworkManagerShutdown () (at <0df0f351a5d94cbb9af1af6290fbdcfa>:0)
(wrapper dynamic-method) Unity.Netcode.NetworkManager.DMD<Unity.Netcode.NetworkManager::Shutdown>(Unity.Netcode.NetworkManager,bool)
GameNetworkManager+<DisconnectProcess>d__74.MoveNext () (at <af9b1eec498a45aebd42601d6ab85015>:0)
UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) (at <e27997765c1848b09d8073e5d642717a>:0)
UnityEngine.MonoBehaviour:StartCoroutine(IEnumerator)
GameNetworkManager:Disconnect()
Better_Lobbies.Patches.LobbyConnectionFixes:DisconnectOnDisable() (at C:/Users/vispy/source/repos/Better Lobbies/Patches/LobbyConnectionFixes.cs:55)
StartOfRound:DMD<StartOfRound::OnDisable>(StartOfRound)

^ looks like the stack trace also leaked the original developer path.
so now, the value is not reset, but stays at true, and it affects all lobbies the host will create since GameNetworkManager is re-used, which is why there is a value reset to make sure it's ready for the next lobby.

The way I fixed it while testing is to:

Clone the BetterLobby repository
Removing that whole patch or just the Disconnect() call
Compile/build the BetterLobby source code into a DLL
Use the compiled DLL instead on BepInEx instead of the actual one

I'm not sure if I can send the compiled DLL here for you to test as I don't think that'd be nice...
Probably it's a good idea if you open an issue on their repository for this incompatibility to be fixed, feel free to reference to this issue which definitely will save some time debugging on their end too!

This can be fixed in LCDirectLAN as additional patches, however I refuse to do it, because if I force to override the disallowConnection variable to be always false, there is a high chance that a conflict may arise in the future if the game changes or another mod controls it.
And personally, I'd like to keep LCDirectLAN as that mod who does not override things that isn't required :)

@TIRTAGT i sug don't patch that in ur repository, i will quote this issue and create an issue in there, or u can create a pull request in there really

@TIRTAGT i sug don't patch that in ur repository, i will quote this issue and create an issue in there, or u can create a pull request in there really

Correct, I didn't feel like to do anything at that variable to avoid further conflicts down the line haha, LCDirectLAN should stay the way it is this time. :)
For a pull request, I'm not really sure I'd like to submit "deleted a few lines" XD.
I'd recommend you to open up an issue on their side and quote this issue instead :)

weeeeeeeeeeeee

haha enjoy my modpack! (but a few difficult) "assets great great assets to the company, assets great great great assets to the company"
oh and ice cream lol >;D

those mods that allow u to see the scenery in orbit, if u really like that

SpaceSunShine, https://thunderstore.io/c/lethal-company/p/CatsArmy/SpaceSunShine
SpaceShipDoor, https://thunderstore.io/c/lethal-company/p/Wolf11221/SpaceShipDoor
ShipWindows, https://thunderstore.io/c/lethal-company/p/veri/ShipWindows

Lol all fine! This can only happen if you ever include your pdb file in your Plugin or if you have an embedded pdb file inside your dll. This is located here in VS.

yeeeeeeeeeee >;)

those mods that allow u to see the scenery in orbit, if u really like that

SpaceSunShine, https://thunderstore.io/c/lethal-company/p/CatsArmy/SpaceSunShine
SpaceShipDoor, https://thunderstore.io/c/lethal-company/p/Wolf11221/SpaceShipDoor
ShipWindows, https://thunderstore.io/c/lethal-company/p/veri/ShipWindows

Thank you, definitely will bundle that to for my community mod pack haha !

looks like the stack trace also leaked the original development path, unlocked a new fear of accidentally revealing my local storage path in a C# stack trace.

Lol all fine! This can only happen if you ever include your pdb file in your Plugin or if you have an embedded pdb file inside your dll. This is located here in VS.

Ah I see, I disabled Debug Symbols entirely for the release variant.


Feature Request to Be Able To Switch to App Controls when using Apps like Safari or Chrome or Spotify

There needs to be a way for Pock to either automatically or have a button to switch between the pock layout on the touch bar and the native App Controls on the touch bar. This si very necessary because in apps like Spotify and Safari a lot of the controls are embedded there and then you lose a lot of the functionality which isn't great. One way of solving this would ideally be a button where you can engage or disengage Pock from the touch bar so then if you wish to use the native app controls or you can continue using pock.

Pock already has this capability. You can double press the control key to switch between Pock's layout and the default Touch Bar controls.


Add techstack.yml

See https://github.com/marketplace/stack-file

I tried this 2x and failed, so closing this PR.

Finally got this working #2273


Code: Insert a ZWNJ before after type hints

to prevent the editor from displaying a ligature there.
Fixes #6235

Could you add a comment that it is a ZWNJ?

bors r+


[HOLD for payment 2023-07-17] [$1000] mWeb -  Emoji modal stops working in the chat if the user opens it in the IOU report.

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

Action Performed:

Opened mWeb.
Navigated to the chat.
Sent money and accessed the IOU report page.
Opened the emoji modal
Click Back to return to the chat.
Tried opening the emoji box.

Expected Result:
The emoji modal should open in the chat compose box.
Actual Result:
The emoji modal doesn't open in the chat compose box.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
**Reproducible in production?:**y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/b49ebfe7-bd8d-4183-95e7-d79a040fcb31
https://github.com/Expensify/App/assets/93399543/6b081e2d-30b0-4383-b59c-913a77dfc27c
Expensify/Expensify Issue URL:
Issue reported by: @Tushu17
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686644077744109
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01f59b5426989a9373
Upwork Job ID:<PHONE_NUMBER>557054464
Last Price Increase: 2023-07-03

Proposal
Please re-state the problem that we are trying to solve in this issue.
Emoji picker won't open after going back from a child chat.
What is the root cause of that problem?
This has the same root cause with https://github.com/Expensify/App/issues/20513 and https://github.com/Expensify/App/issues/20847. Emoji picker uses a global ref defined in EmojiPickerAction
https://github.com/Expensify/App/blob/e6c2704eb43d4d31a55eff5d5d1b3e89c5292bc3/src/libs/actions/EmojiPickerAction.js#L3
The ref will be set when ReportScreen is mounted and will be set to null once ReportScreen is unmounted. In our case, the second chat is unmounted which will set the global ref to null.

Open first chat
ref: picker 1
Open second chat
ref: picker 2
Close second chat, report screen is unmounted
ref: null

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

Use local ref instead. Define the ref in ReportScreen and pass it down with Context. We planned to create a ReportScreenContext here https://github.com/Expensify/App/issues/20847 that holds all the ref(s), so we can just wait for it first and then use it for emoji picker ref too.
Every component that uses the global emoji picker ref will now use the emoji picker ref from ReportScreenContext

This is on my test list

Proposal
Please re-state the problem that we are trying to solve in this issue.
Emoji modal stops working in the chat if the user opens it in the IOU report
What is the root cause of that problem?
emojiPickerRef is created as global ref instance here
This ref will be set when ReportScreen is mounted and will be set to null once ReportScreen is unmounted. In our case, the second chat is unmounted which will set the global ref to null.
What changes do you think we should make in order to solve the problem?
We can move EmojiPicker component from ReportScreen here to App here or Expensify component here, so that it won't loose state while re-rendering of report component.
What alternative solutions did you explore? (Optional)
None

I'm catching up from my half day, I'll review soon.

I've confirmed this is an issue and don't believe there is another GH related to it.

Still waiting for proposals here.

Reassigning another bug team member, I'm going OOO until Monday, July 10, and will take it back then if it's not closed by then.
Required action from the 🐛 team: Make sure the proposal process is moving forward, so we can get this one assigned out to fix. If no movement on the proposals by next week, then we may want to consider shopping for a reviewer.

Do you think it'll cause performance regressions? Does the entire app re-render when emoji drawer is reopened?

@rushatgabhane thanks for reviewing my proposal.
As investigated and tested, i didn't observe or notice any performance regressions with proposed update. And it won't impact app to cause re-render when emoji drawer is reopened.
In fact, I think this would be more optimised way of handling emoji picker instance global ref here, IMO.
FYI, we also have similar report action context menu global component ref defined in Expensify component here

@rushatgabhane do we have a viable proposal to choose yet or not?

PR is up for review
cc: @rushatgabhane @michaelhaxhiu

Alright, removing the other 🐛 team member. I'm back online and see that this is our current state:
Waiting for automation here

To prepare for payment: @Tushu17 can you please apply to the job? I don't think that automation accounted for you finding this 🐛. Thanks!
https://www.upwork.com/jobs/~01f59b5426989a9373

@alexpensify Okay, I've applied for the job.

I'll work on the payment process tomorrow.

@Tushu17 - I've prepared the payment in Upwork, please accept and I can complete the next steps
@payal-lathidadiya - I've sent the payment via Upwork
@rushatgabhane - please request payment in NewDot, thanks!

@alexpensify I've accepted the offer. Thank you!

Everyone has been paid here.

@rushatgabhane - When you get a chance, can you please complete the checklist:
https://github.com/Expensify/App/issues/21101#issuecomment-1629882520
Thank you!

@rushatgabhane - I'm checking in to see if the checklist is ready and we can close this out. Thanks!

@anmurali - we are on hold for C+ payment and will give the 🟢 as soon as the checklist has been completed.

Note: you can pay the job out before the checklist is completed. We just need to ensure the checklist is done before the GH is closed.

WIP ⚠

The PR that introduced the bug has been identified. Link to the PR:

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Determine if we should create a regression test for this bug.

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again

Made a manual request here https://staging.new.expensify.com/r/2734383403522123

Thank you @rushatgabhane! @anmurali all set for payment here. I'm going to close this GH since all actions have been completed.

@alexpensify Can you please summarize the appropriate individual payments for all parties involved in this issue? This is holding up @rushatgabhane's NewDot payments. More information on this compliance process in Slack.

Here is the payment summary:

External issue reporter: @Tushu17 $250
Contributor that fixed the issue: @payal-lathidadiya $1500
Contributor+ that helped on the issue and/or PR: @rushatgabhane $1500

Upwork Job:
https://www.upwork.com/jobs/~01f59b5426989a9373
*If applicable, the bonuses will be applied on the final payment
Extra Notes regarding payment: All set here @JmillsExpensify! This one was pre-summary days. Everyone has been paid here except @rushatgabhane. Thank you!

Reviewed details for @rushatgabhane. This is accurate based on summary from Business Reviewer and approved for payment in NewDot.


Getting started guides needed for devices

https://docs.resin.io/nitrogen8mm-sbc/nodejs/getting-started/#adding-your-first-device -> https://boundarydevices.com/product/nitrogen8m-mini/ uses sd-card for flashing as mentioned in instructions
https://docs.balena.io/jetson-xavier-nx-devkit-sd-card/nodejs/getting-started/#adding-your-first-device -> https://developer.nvidia.com/embedded/jetson-xavier-nx-devkit uses sd-card for flashing as mentioned in instructions
https://docs.balena.io/jetson-xavier-nx-devkit-emmc/nodejs/getting-started/#adding-your-first-device -> https://developer.nvidia.com/embedded/jetson-xavier-nx uses jetson-flash for flashing as mentioned in instructions
http://docs.resin.io/imx8m-var-dart/getting-started/#adding-your-first-device -> https://shop.variscite.com/product/evaluation-kit/dart-mx8m-evaluation-kits/#_ga=2.226744669.1773822250.1607418338-1994371556.1599813799&_gac=1.83905259.1607418338.Cj0KCQiA5bz-BRD-ARIsABjT4ngl0O6yzgsxteCRTZZ22sGiw6dpbp0Bj4mAIvB-oaiJyC6AqeIJ6xwaAsn-EALw_wcB#utm_source=https://www.google.com/#utm_medium=N/A#utm_campaign=N/A#utm_content=N/A#utm_term=N/A uses sd-card for flashing as mentioned in instructions
http://docs.resin.io/imx8mm-var-dart/getting-started/#adding-your-first-device -> https://shop.variscite.com/product/evaluation-kit/dart-mx8m-mini-evaluation-kits/#_ga=2.4914803.1773822250.1607418338-1994371556.1599813799&_gac=1.251493620.1607418426.Cj0KCQiA5bz-BRD-ARIsABjT4ni3PPzxBT_Ypdo8a_ItzN77bWUJs8Q4_WlAQteaAVdASwQ8ILX5dNgaAoZ3EALw_wcB#utm_source=https://www.google.com/#utm_medium=N/A#utm_campaign=N/A#utm_content=N/A#utm_term=N/A uses sd-card for flashing as mentioned in instructions
https://docs.balena.io/nsnopc-t4/nodejs/getting-started/#adding-your-first-device -> https://www.friendlyarm.com/index.php?route=product/product&product_id=225 uses sd-card for flashing as mentioned in instructions
https://docs.resin.io/jetson-nano-emmc/nodejs/getting-started/#adding-your-first-device -> https://developer.nvidia.com/embedded/jetson-nano uses jetson-flash for flashing as mentioned in instructions

Getting Started guide needed for Jetson Nano EMMC device.
Related issue: https://jel.ly.fish/de14c712-647c-4a9e-bf86-d44bd926fc94

Add Getting Started guide for each device in a separate PR for easier reviews.

I am thinking to break down this issue fully into individual pieces. Will create many issues from so more folks are able to pick these up in the release party. @acostach Can you check if there are any more devices we are missing? Open a issue for each one that we are missing so folks can pick it up and solve that.


Explain installation of package, dependencies

Some more hints in the README on how to pull in the dependencies and run this would be helpful.
So far this worked for me to avoid import errors:
pip install scipy networkx PyQt5

Then, since I don't see a pyproject.toml  or setup.py, I just tried to run python from the root directory of the project, as cloned via git.
But I'm guessing that I'm getting different versions than you are, since I get these errors:
Python 3.8.0 (default, Oct 28 2019, 16:14:01)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import matchingmarkets as mm
>>> import numpy.random as rng
>>> newsim = mm.simulation(time_per_run=100, max_agents=5000, arrival_rate=15,typeGenerator=rng.randint,
   compatFct=mm.stochastic_neighborSameType,crit_input=3, numTypes=5)
>>> newsim.graph(plot_time=0.8)
Attribute Qt::AA_EnableHighDpiScaling must be set before QCoreApplication is created.
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/srv/s/games/MatchingMarkets.py/matchingmarkets/simulations.py", line 254, in graph
    newMarket.update(metaAlgorithm=self.metaAlgorithm,
  File "/srv/s/games/MatchingMarkets.py/matchingmarkets/market.py", line 393, in update
    nx.draw_networkx_edges(self.Graph, self.graph_pos,
TypeError: draw_networkx_edges() got an unexpected keyword argument 'font_size'
>>>

It looks like I have networkx version 2.5.

Note also that I ran the tests as described at https://github.com/QuantEcon/MatchingMarkets.py/tree/master/tests
and they all worked up to Plotting Tests, where I got the same error I noted above.
...
Non-Stochastic brute search
Optimization terminated successfully.

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/srv/s/games/MatchingMarkets.py/matchingmarkets/tests.py", line 154, in FullTest
    market.update(discount=rng.random_sample,
  File "/srv/s/games/MatchingMarkets.py/matchingmarkets/market.py", line 393, in update
    nx.draw_networkx_edges(self.Graph, self.graph_pos,
TypeError: draw_networkx_edges() got an unexpected keyword argument 'labels'

My PR #8 seems to resolve the API issue.
Now I'm seeing this PULP_CBC_CMD test failure:
...
Plotting Tests complete
============================
Pulp Test
         Testing zero subtraction
         Testing inconsistant lp solution
Test failed: var x == 0.0 != 4
Tests failed for solver <matchingmarkets.algorithms.pulp.solvers.PULP_CBC_CMD object at 0x7efc65d16f40>
* Solver <class 'matchingmarkets.algorithms.pulp.solvers.PULP_CBC_CMD'> failed.

Note also that several messages that seem like warnings, simply because I didn't install all the solvers, also come out, e.g.:
Solver <class 'matchingmarkets.algorithms.pulp.solvers.CPLEX_DLL'> unavailable

Hi,
Sorry about taking so much time to respond. I haven't worked on this project in a while.
I added a setup.py you should be able to run normally. If it sees more use, I can add it to pip without too much trouble.

Excellent - thank you!
The updates fixed the error I encountered in testing,  and all the tests now pass.
I note that I got what seemed like a more complete test by running FullTest(), which may be worth mentioning some more in the documentation.  The FullTest did surface a warning or two, which may or may not be worth some attention.
[run FullTest manually]
...
Non-Stochastic brute search
Warning: Maximum number of function evaluations has been exceeded.
Warning: Either final optimization did not succeed or `finish` does not return `statuscode` as its last argument.
Results:  [6.76774276]

...

Attribute Qt::AA_EnableHighDpiScaling must be set before QCoreApplication is created.
/srv/s/games/MatchingMarkets.py/matchingmarkets/market.py:332: RuntimeWarning: divide by zero encountered in double_scalars
  k=(1/(0.9*np.sqrt(len(
iteration done

(That divide by zero line RuntimeWarning was copied in from a run before my full install, so it may be spurious.)
Thanks again!

Yeah that warning is because it would be too long to set up a brute force search that converges in the test suite


No instruction for building

Hi,
For those who are not familiar with rust, can you provide instructions for compilation of Noq, especially under Windows?
I am getting an error when running cargo build:

note: LINK : fatal error LNK1181: cannot open input file 'advapi32.lib'

When running rustc main.rs, I get an error regarding termion:

error[E0433]: failed to resolve: maybe a missing crate termion?

Thanks.

Seems to be a broken SDK (see this issue).
In any case, this crate does not run on windows as is. See #8 for more information.

Hiya, not the maintainer but, this crate is using termion which is a crate for ANSI terminals only.
It was probably chosen primarily for its small size and convenient API, but there is another - larger - crate crossterm providing the same functionality in a cross-platform way.
The maintainer uses Debian Linux and (while I of course can't speak for him) I think is unlikely to switch to crossterm but maybe someone could open a PR or a fork since the changes are fairly trivial to make. e.g. newdoku

Fixed in #15


[Bug] discord stuck in update loop after installing bd

Describe the Bug
after i installed betterdiscord, the discord app would just check for updates, then get stuck on an update loop, this seems to be an issue with bd due to the fact that this only starts happening when i install betterdiscord.
To Reproduce
startup bug, literally only start the bloody thing
Expected Behavior
last time i used bd, which was a good six months ago, it worked perfectly, which is normally what i would expect (i don't even know if i'm doing this right...)
Discord Version
Stable

Try using the repair option of the installer and launch Discord without reinstalling BD.


PowerToys automatic startup not working

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
Settings
Steps to reproduce
PowerToys is not automatically starting up at login, or system boot. I turned on the option but it doesn't work.
✔ Expected Behavior
PowerToys is not automatically starting up at login, or system boot. I turned on the option but it doesn't work.
❌ Actual Behavior
It stays disabled or non-running when I start up my PC.
Other Software
No response

Can you check in Task Scheduler if you have a scheduled task inside "PowerToys" in "Task Scheduler Library"? That option is supposed to create one.
A /bugreport may be helpful as well.


Head Tags are not working. Tried both static and dynamic tags as explained in the doc. Still no luck.

I'm trying customize the annotation 'Index - Nextra' to 'Index - Company', I'm unable to do with the instructions below.
https://nextra.site/docs/docs-theme/theme-configuration#dynamic-tags-based-on-page

Try this if it work for you
export default {
  head: () => {
      ...
    },
    useNextSeoProps() {
        const { frontMatter } = useConfig();

        const { asPath } = useRouter()

        if (asPath !== '/') {
            return {
                titleTemplate: '%s - Company name',
            }
        }
    },
    ... // other configs
}

}

Here is Doc
https://nextra.site/docs/docs-theme/theme-configuration#seo-options

Beautiful. That solved it.

Thank you

Saved. Thank you.


⚠ Bofrost.fr has degraded performance

In a0dde10, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1268 ms

Resolved: Bofrost.fr performance has improved in ec25c0d after 2 minutes.


Training dataset method for Micro speech

Hello and thank you for reading this comment
I'm quite new to ML in general, I've evaluated and did some tinkering of micro_speech on arduino BLE 33.
At the moment, I've tried detecting more than 2 words on the plateform (up, left, right, down). I achieved well performance metrics on training using the provided tf scripts (around 90%) so, I was surprised to see of inacurracy of the inference on arduino target. (as reference, you can check out this video https://www.youtube.com/watch?v=ImrYfkDbC_0 which is more or less what I've experienced )
I had this tought, what if I used recorded samples from the embedded microphone for the dataset training instead of the pretty ones we use on our laptops ?
Shouldn't it be more representative of what the ML system should learn ?
Cheers
LB

Assigning to @ddavis-2015 to answer Arduino related questions.

Was the version of TensorFlow Lite Micro you are using from the Arduino-Examples repository?  Or from the Arduino IDE library download?  The Arduino-Examples repository had the micro-speech example updated in Dec. 2021 to fix some issues with the code base that affected recognition accuracy.
You are more than welcome and encouraged to bring your own data for training your model.


Feature request: Store settings on ini file

Hi, it would awesome if there exist an option for store the settings on a .ini files instead of  the registry, so it become portable and allow the user to launched the program from an usb pendrive.
Cheers.

Use the dedicated forum: https://www.qlcplus.org/forum/viewforum.php?f=18
Closing.

Use the dedicated forum: https://www.qlcplus.org/forum/viewforum.php?f=18
Closing.

Awesome.

Awesome.


OreExcavation compatibility problems

Version:  vanillahammers-1.14.4-1.0.jar
Forge: 28.2.5
Minecraft: 1.14.4
Mods:

Vanilla Hammers
Ore Excavation

Problem:
Ore Excavation apparently thinks that your tools are shovels of sorts, because I can excavate sand, grass, dirt without problems. When it comes to stone on the other hand, it only does the 3x3.
I'd like to use obsidian hammers to make long 3x3 tunnels, without having to remove every layer by hand. I already tried different means of adding your tools to ore-excavation using their tool overrides, Tool whitelist and alternate tool checks. None of those worked.

 Sorry!


[1.x] Replace GLOB_BRACE with more robust solutions

Replaces glob brace usages as some Linux distros do not support it, for instance Alpine, which hurts Docker support.
Related issues: (Which this will probably fix)

https://github.com/hydephp/framework/issues/662
https://github.com/hydephp/hyde/issues/270

Will fix https://github.com/hydephp/develop/issues/2063

I think it makes sense for a file finder to handle file extensions in a case-insensitive manner, especially in scenarios where the target operating system or use case does not enforce case sensitivity. This is particularly important because:

Cross-Platform Consistency: Different operating systems handle case sensitivity differently. For example:

Windows and macOS treat file extensions as case-insensitive.
Linux is case-sensitive. Making the file finder case-insensitive ensures predictable behavior across all platforms.

User Convenience: Users might unintentionally create files with mixed-case extensions (e.g., .MD vs .md). A case-insensitive approach avoids confusion and ensures files are found regardless of how the extension is written.

Practicality: When searching by extension, the intent is generally to match all files of a certain type, not to differentiate based on casing.

I think we should sort the output in the method too to further normalize different file systems. It takes just 0.010ms to sort 100 files, so that's fully acceptable.

Better to have paths relative to the project root to follow the conventions, and how we will use it. Especially since it's a drop in replacement for the smart glob method which is relative to the project.

Okay, I think this is ready for merge, and assuming the root cause of the two suspected related issues is GLOB_BRACE, this should fix those.

@clifmo could you confirm if this branch fixes your issue https://github.com/hydephp/framework/issues/662? (unless the cause was Docker being weird, this should fix your issue, as it doesn't match directories)
@sobi3ch could you confirm if this branch fixes your issue https://github.com/hydephp/hyde/issues/270 (pretty sure it will)

To test this, run the following command in your project terminal:
composer require hyde/framework:dev-replace-glob-brace-preview

Then rerun the reproduction steps that led to your issues.
If either of you don't have time to test it, that's totally okay, just let me know. I will be going out of town for a few days and will hold of on merging this until then as I can't properly support a release rollout until the end of the week.

@caendesilva works like a charm

@sobi3ch Thank you so much for taking the time to test it! I'll be going ahead and merge this, and I will try for a weekend release, if not earlier!


SSZ vector wrap-around

To simplify the presentation we could implement index wrap-around for vectors. That is, vector[i] would be equivalent to vector[i % len(vector)]. This would shorten about 20 lines, and potentially improve readability. Examples below:

state.latest_block_roots[slot % SLOTS_PER_HISTORICAL_ROOT] => state.latest_block_roots[slot]
state.latest_randao_mixes[epoch % LATEST_RANDAO_MIXES_LENGTH] => state.latest_randao_mixes[epoch]
state.latest_active_index_roots[epoch % LATEST_ACTIVE_INDEX_ROOTS_LENGTH]
state.latest_slashed_balances[current_epoch % LATEST_SLASHED_EXIT_LENGTH]
state.latest_state_roots[state.slot % SLOTS_PER_HISTORICAL_ROOT]

I'd say that sacrifices readability too much. It pushes the spec in the direction of "you have to read these other five sections first to understand any of this".

I agree with @vbuterin here, and also, the advantage of having explicit wraparound (as in: having to write out the modulo operation) is that tests can catch out of bounds errors. Wraparound turns this into a silent failure, with much worse debuggability.

The goal was to increase readability, which seems to not have been achieved 😂


[Website] : Improve Lighthouse Score

Description
Change video format from .gif to .webp
This PR fixes #973
Notes for Reviewers
I would delete the original .gif files during final push, for now I have left them as it is!
Signed commits

[x] Yes, I signed my commits.

Hey @Akshun-01  Let's discuss this on website call today at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes.

@Akshun-01 Thankyou for raising a draft PR, can you share the progress here?

2nd Update :
Changed the images (only among the ones pointed by analysis) from .png to .webp format
Before

After

Hey @Yashsharma1911, I was confused about this one.
Do I need to change the size(dimensions) of the original image?
Won't it be troublesome, could you please explain this point!

There can be a couple of solutions you might like to read this blog https://developer.chrome.com/docs/lighthouse/performance/uses-responsive-images, I guess it's about serving image that is appropriate to screen it can differ based on resolution
I'll recommend to not hander the original image rather different size version of original images and you can use srcset attribute to pass multiple images for resolution. Lemme know your findings

2nd Update :
Changed the images (only among the ones pointed by analysis) from .png to .webp format
Before

After

This is Good 👏

Helo!
[UPDATE] : Performance is at 72 (slight decrease idky)
- Changed some other heavy images from png to webp
- Resized the main meshmap gif and added responsive alternative for it
Now, these are the main problem points being highlighted:

So, I think next we can focus on

text compression
prioritising what to ship first (will need to look into it in detail)

@Yashsharma1911, I don't think we can improve it much now, as it is already touching 70 now. It would need lot of optimizations to reach to 80. So, how should we proceed with the issue?

@Yashsharma1911, I don't think we can improve it much now, as it is already touching 70 now. It would need lot of optimizations to reach to 80. So, how should we proceed with the issue?

Thankyou @Akshun-01 it would be nice if you can present this in tomorrows website meeting

@Akshun-01 any update here?

@Yashsharma1911 I don't have much experience at reviewing , but I will need to check what the issue is, then I will review it at night, Thanks

Hey @Yashsharma1911 , I tried some other suggestions given by the report. But now rather than increasing it is rather going the opposite way.
Also the average performance was way better after the second commit ( ie after changing only the formats)
After I changed the image sizes and added multiple images for responsive image feedback it is giving inconsistent reading everytime I check. (ranging from 50 - 75 ).
So, I doubt making that change was a wise choice. It is my first time making these optimizations, so I also don't know if we should consider all feedback or do the relevant ones only.
Could you please check and see how should I proceed with this now.
Thanks!^^
@Raj100679  would appreciate your feedback and insight on this!

@Akshun-01 Sorry I am stuck with some other work, @Yashsharma1911 Can you check it

Thanks for working through this, @Akshun-01.

@Akshun-01 still working on this?

@Akshun-01 still working on this?

Got any solution?

following up again, @Akshun-01 are you still working on this?

Hi @sudhanshutech, sorry for the late reply.
Actually, I was waiting for someone to review the already-made changes.
Also as I mentioned earlier, after a certain point changing a lot of things was also not helpful that much. The thing is it is asking to resize and change the format of images and we have used a lot of images. So, I worked on the ones that were really heavy and were affecting the performance greatly.
So, I think pointlessly making any more image compressions or putting a lot of time into scaling images is fruitful only if we believe that would make a great difference.
That's why I believe some review would be really helpful before proceeding.

@Yashsharma1911
What needs to be done here? I have already discussed this in the weekly meet twice now.
And someone was required to review this and update with anything that needs to be done.
If you or anyone else could take out some time. It would be great!
I have given in depth report for every change I made, so anyone can refer to it to decide what needs to be done.

Great thanks then, I'll look in the evening, till I'll suggest you to parallel work on others issues too if you want as this seems to block you.
Message from mobile

Great thanks then, I'll look in the evening, till I'll suggest you to parallel work on others issues too if you want as this seems to block you.
Message from mobile

Yeah, I was stuck with a lot lately. So didn't get time. I will start with other issues as well now.
Do you have any in mind, I would love to help!
Also,  I wanted to help in the backend part too, so if you know any beginner-friendly ones, I would appreciate it!
Thanks ^^

@Akshun-01 quality of this homepage GIF becomes too poor

Yeah, it's not good.
One thing I can do is. I now know what things help the most in performance, so I can close this one and work on the latest branch and improve it there.
I will only make the necessary changes in that rather than optimizing and resizing every image.
What's your say?

sounds helpful @Akshun-01 , before creating a pr it would be good if you have documented the changes first like what is the issues and what is the solution and series them according them to priority. So in this way it will not take that much time and will easy to review.


Fix WinRT build break

cc @danmosemsft

Odd. Not sure how I missed that, and it went through CI also.

No problem. The CoreRT CI does not build the ProjectN UWP flavors with WinRT.


Refined documentation by adding and removing few character

At some places dot was used instead of colons- fixed those.
At one line (in the end) a bar character ('|') was present, removed that.

Thanks!


_saved_state and _previous_revision_id saved in DB

These fields should not be stored in the database

Hey,
Wow, this is a bug definitely. I'll check and fix it asap. Thank you!

Hi @nichitag ,
Sorry for the delay. I can not reproduce this bug. Could you pls try, if it still exists? If so, pls give me a code snippet, that reproduces it. Thank you!


use transparent images in app manifest

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 99.67%

Files with Coverage Reduction
New Missed Lines
%

src/modules/olMap/handler/draw/OlDrawHandler.js
1
98.6%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.01%

Covered Lines:
13513

Relevant Lines:
13515

💛  - Coveralls


command which prints build directory

I would like to know where next build will stored, it would be nice to have a command which prints it, i.e.
leeway build dir "myapp:mypackage"
/tmp/build/myapp.mypackage.abcd

I used something like leeway describe "//:init" -t "/tmp/build/{{ .Metadata.Name }}.{{ .Metadata.Version }}" for now, but it is rather close, since we transform name to be valid dir name

leeway describe "//:init" -t "/tmp/build/{{ .Metadata.FilesystemSafeName }}.{{ .Metadata.Version }}" not prints the correct name


Update the course email template to link to the API to allow users to unsubscribe (link)

Update the course email template to link to the API to allow users to unsubscribe (link)

I have verified that no emails are being sent from the Staff Instructor Dashboard Email section if one has unsubscribed emails from MITxOnline dashboard but only sent to subscribed users

I have verified that the unsubscribe link in the email footer is working fine. Here are the screenshots of it

As everything is working fine, I am gonna close it on behalf of my last comment.


Ensure full coverage of KV management methods

Overview
The KV ADR has been updated here:  https://github.com/nats-io/nats-architecture-and-design/pull/283
Please make sure that your client covers all the methods mentioned in the change:
// KeyValueManager is used to manage KeyValue buckets. It provides methods to
// create, delete, and retrieve.
type KeyValueManager interface {
    // KeyValue will lookup and bind to an existing KeyValue bucket.
    // Name can be `get_key_value`, or whatever name is idiomatic in given language.

    CreateKeyValue(ctx context.Context, cfg KeyValueConfig) (KeyValue, error)

    // UpdateKeyValue will update an existing KeyValue bucket with the given
    // configuration.
    UpdateKeyValue(ctx context.Context, cfg KeyValueConfig) (KeyValue, error)

    // CreateOrUpdateKeyValue will create a KeyValue bucket if it does not
    // exist or update an existing KeyValue bucket with the given
    // configuration (if possible).
    CreateOrUpdateKeyValue(ctx context.Context, cfg KeyValueConfig) (KeyValue, error)

    // DeleteKeyValue will delete given KeyValue bucket.
    DeleteKeyValue(ctx context.Context, bucket string) error

    // KeyValueBucketNames is used to retrieve a list of key value bucket
    // names.
    KeyValueBuckerNames(ctx context.Context) KeyValueNamesLister

    // KeyValueBuckets is used to retrieve a list of key value bucket
    // statuses.
    KeyValueBuckets(ctx context.Context) KeyValueLister
}

Keep the names idiomatic for your language.
Clients and Tools

[ ] Schemas @ripienaar
[ ] CLI @ripienaar
[ ] Terraform @ripienaar
[ ] Github Actions @ripienaar
[ ] Go @piotrpio
[ ] Java @scottf
[ ] JavaScript @aricart
[ ] .Net @scottf
[ ] C @levb
[ ] Python @wallyqs
[ ] Ruby @wallyqs
[ ] Rust @Jarema
[ ] .Net V2 @mtmk

Other Tasks

[ ] docs.nats.io updated @bruth
[ ] Update ADR to Implemented
[ ] Update client features spreadsheet

Client authors please update with your progress. If you open issues in your own repositories as a result of this request, please link them to this one by pasting the issue URL in a comment or main issue description.

Note that we're not implementing CreateOrUpdateKeyValue for .NET since there is no equivalent JetStream API for managing streams. We can implement it after an atomic create-or-update-stream function is implemented server side.


Exclamation mark on the end of method argument

Hi
On section "Test with WebApplicationFactory or TestServer", chapter "ASP.NET Core 6", line
await context.Response.WriteAsync(helloService.HelloMessage!);
Is a exclamation mark on the end, witch seems wrong.

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

ID: 20acee3e-8b6e-b3fb-8271-661e1a287bdd
Version Independent ID: 20acee3e-8b6e-b3fb-8271-661e1a287bdd
Content: Code samples migrated from ASP.NET Core 5.0 to the the new minimal hosting model in 6.0
Content Source: aspnetcore/migration/50-to-60-samples.md
Product: aspnet-core
Technology: aspnetcore-migration
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hi @mwinkler. That's the ! (null-forgiving) operator. In this example, IHelloService.HelloMessage is nullable (string?), but we know in the example that it won't be null when we pass it into WriteAsync.
@Rick-Anderson The examples here switch between public string? HelloMessage {  get; set; } and public string HelloMessage => ...;. From what I can see here, the MockHelloService won't compile. Do you want me to fix this up?


How to use links like you do with Projects on your site?

Hey,
Not really an issue. I'm just wondering how you created links instead of posts under projects on your own site? (When you click projects, and each list item is a link to a github project)
This theme is fantastic, thank you for making it!

I see that I can simply copy and modify the project/index.html file my bad


Coverage Reporting

Add coverage reporting to codecov.io and generate HTML coverage reports locally

Coverage is now reporting on codecov.io! See coverage for this branch

This is great.  I would recommend changing where the html coverage is emitted to avoid putting it directly in the project directory.  That is, I'd change
coverage html

to
coverage html -d {toxworkdir}/htmlcov

so it ends up at .tox/htmlcov.
Additionally, I use a tool that uses the xml for showing coverage.  It would be handy for my existing workflow to also add
coverage xml -o {toxworkdir}/coverage.xml

I updated the tox.ini to place the coverage reports (HTML and XML) in the .tox directory

Looking at the reports, let's also turn on branch coverage reporting.  This can be done by adding the appropriate flag in the tox.ini file:
[coverage:run]
branch = True


Add Pulumi

Name with link
Pulumi
https://www.pulumi.com/
Description
An infrastructure as code tool/platform, similar to AWS CDK or Terraform
Link to the vector file, preferably from an official source
https://www.pulumi.com/images/logo/logo-on-white.svg

Added in da96ad61116f1c9178dee389ae55254277a45eb3
Thanks


Question: callback.global

Hi Thierry,
I use xajax callback funktions in javascript.
<script type="text/javascript">
//<![CDATA[
  xajax.callback.global.onRequest = function() { $('#loading').css('display','block'); }
  xajax.callback.global.beforeResponseProcessing = function() { $('#loading').css('display','none'); }
//]]>
</script>

Have you an equivalent in jaxon?
I saw that you have callbacks pre-processing and post-processing, but that are on server.
Regards,
Joc

Hi Joc,
There've been no change in the javascript functions.
You just have to replace xajax with jaxon, and it should be working the same way as before.
PS: If you are using jQuery, you may want to checkout the jaxon-jquery package https://github.com/jaxon-php/jaxon-jquery. It provides a jQuery-like API to modify the webpage content and presentation with PHP.
Regards,
Thierry

Any feedback on this issue?

Sorry, the github mail system did not working well, I got only the close mail.
I will try it.

You just have to replace xajax with jaxon, and it should be working the same way as before.
That was what I try first, but it didn't work, so I make the issue.
But now it is working.


jvm options ( such as Xmx for example ) used by pinot processes should be logged

jvm options ( such as Xmx for example ) used by pinot processes should be logged.  Currently, the only way to tell what Xmx pinot is using is to do a ps and grep on the result.  But, that's for a currently running process.  For past jvm options used, it is difficult, if not impossible to tell what Xmx and other jvm options were used for the pinot process.

EDIT - not just jvm options but whatever other options are used to start the pinot process

Please assign this issue to me. Thanks

Here is the PR for the change.

@thatran - You could also use this swagger api to get the jvm configs
curl -X GET "http://localhost:9000/appconfigs" -H "accept: application/json"
jvmConfig": { "args": [ "-javaagent:/Applications/IntelliJ IDEA CE.app/Contents/lib/idea_rt.jar=50367:/Applications/IntelliJ IDEA CE.app/Contents/bin", "-Dfile.encoding=UTF-8" ], "garbageCollectors": [ "G1 Young Generation", "G1 Old Generation" ], "envVariables": { "__CFBundleIdentifier": "com.jetbrains.intellij.ce", "PATH": "/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin", "SHELL": "/bin/zsh", "MANPATH": "/opt/homebrew/share/man::", "HOMEBREW_CELLAR": "/opt/homebrew/Cellar", "OLDPWD": "/", "USER": "manishswaminathan", "HOMEBREW_PREFIX": "/opt/homebrew", "COMMAND_MODE": "unix2003", "TMPDIR": "/var/folders/xj/yl13brv573g9_v_vhxx7knx00000gn/T/", "SSH_AUTH_SOCK": "/private/tmp/com.apple.launchd.lyuZIHVl7g/Listeners", "XPC_FLAGS": "0x0", "__CF_USER_TEXT_ENCODING": "0x1F5:0x0:0x0", "LOGNAME": "manishswaminathan", "LC_CTYPE": "en_US.UTF-8", "HOMEBREW_REPOSITORY": "/opt/homebrew", "JAVA_MAIN_CLASS_1666": "org.apache.pinot.tools.Quickstart", "PWD": "/Users/manishswaminathan/projects/forked/pinot", "XPC_SERVICE_NAME": "application.com.jetbrains.intellij.ce.975373.976077", "INFOPATH": "/opt/homebrew/share/info:", "HOME": "/Users/manishswaminathan" },


Set wireless interface

The GoProController allows you to set a different network interface through an environment variable.
env GOPRO_WIFI_INTERFACE=wlan1

It would be nice if goproctl allowed you to either set it in the parameters or use (the same?) env variable.

This was added in 92db6cafddfccf2b3caaa3769d90246a68d0d0e8.


cointop: init at 2018-05-03

Things done

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

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

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

Not added myself as maintainer because of reasons explained in https://github.com/NixOS/nixpkgs/pull/38504

@xeji done.

Success on x86_64-linux (full log)
Attempted: cointop
Partial log (click to expand)
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/unix/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/unix/mkerrors.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mksysnum_plan9.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mkerrors.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
/nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/install.sh: interpreter directive changed from "/bin/bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/3p9ghhsglcidz85hdzs89aqjvgjgnfrj-cointop-unstable-2018-05-03...
/nix/store/gc005z90lvs8fd4rssk7r6gqlr2q3c4y-cointop-unstable-2018-05-03-bin

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: cointop
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: cointop
Partial log (click to expand)
strip is /nix/store/gp7fylxwn18b7pl2c18ks89hsiaxyfvf-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/install.sh: interpreter directive changed from "/bin/bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mkerrors.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/plan9/mksysnum_plan9.sh: interpreter directive changed from "/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/unix/mkall.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03/share/go/src/github.com/miguelmota/cointop/vendor/golang.org/x/sys/unix/mkerrors.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/8rd1dbvcnwddz38l9kp30x5qmaj6wjhv-cointop-unstable-2018-05-03...
/nix/store/3q0zh1x9fy1m8mg4rhcy16zixv1fpshj-cointop-unstable-2018-05-03-bin


added issue/pr stale workload automation

Related issue: https://github.com/stolostron/backlog/issues/22054
Signed-off-by: dislbenn<EMAIL_ADDRESS>

/unhold


Template Documentation

The entire template should contain explanation on how to use the template.

The template does not include any new environment definitions except the bibliographic environment. This means the documentation is short and is included in the first chapter of template.tex.


Added internal SSL listener test

Description
Added SSL support for internal listener E2E tests.
Type of Change

[x] Other (E2E testing)

Checklist

[x] I have read the contributing guidelines
[x] I have verified this change is not present in other open pull requests
[x] All code style checks pass
[x] New code contribution is covered by automated tests
[x] All new and existing tests pass

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.davkovac seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Issue #26 copy .crt file to temp folder i case of PHAR execution

This fixes issue #26
Added Curl::extractPharCacert(string $pharCacertPath)
Changes Curl::__construct()

Fixed in 673985f549eba8f52e258a49e8c24840a6fb10ea

Do you have a release date in mind, so this fix is available via composer


Correct server/client time offset

Server notifies client of time at hub connection and an offset is calculated. Any client created dates are corrected by the calculated offset. Possible solution to #950

Can you rebase this? It's been like a year I know 😢

Replaced with #1240


#299 introduces a regression that breaks rustls + tokio support

Describe the bug
Previously the with-tokio feature could be used with tokio-rustls-tls - allowing for easy static compilation with musl. After #299 was merged, with-tokio will always pull in native-tls; which forces an openssl dependency
To Reproduce
Upgrade from 0.33.0 to 0.34.0-beta2 with only the  with-tokio and tokio-rusttls-tls features enabled, and you will no longer be able to compile with musl
Expected behavior
rusttls to be used

Ran into the same issue. This is still an issue with the latest beta3 release.

Fixed as of 0.35 at least


Absolute value of x inside log(), ln() and cot() function gives decimal error

Function log10(abs(x)), ln(abs(x)), cot(abs(x)), at negative value of x it doesn't transform into positive, I get decimal error. Works with sqrt(abs(x)).
Thank you.

Did not include case where x is 0.


[Test Release] Add docroot-only options to reduce the size of webapi-…

…service-docroot-tests
Apks built in webapi-service-docroot-tests are not used at all. So skip the
apk build when packing this test suite. docroot-only option was add to statisfy
this request.
Verified that the test suite can be built successfully for docroot.
BUG=https://crosswalk-project.org/jira/browse/CTS-1741

LGTM


deploy_contract_anvil: Error: deserialization error: missing field type at line 1 column 976

Executing task: cargo run --package examples-anvil --example deploy_contract_anvil
Finished dev [unoptimized + debuginfo] target(s) in 0.19s
 Running `target/debug/examples/deploy_contract_anvil`

Anvil running at http://localhost:44993
Error: deserialization error: missing field type at line 1 column 976
Caused by:
missing field type at line 1 column 976
Location:
examples/anvil/examples/deploy_contract_anvil.rs:56:9
https://github.com/alloy-rs/examples/blob/2f109ec5dd66da586e2006e74061d6e19f3ee9b2/examples/anvil/examples/deploy_contract_anvil.rs#L56

fixed by upgrading  anvil (foundryup)


Best Model

hi, what's your best model?

Hey! Don't exactly recall the model selection results, but what I can say for sure is that it was a random forest using only numerical features. As far as height as depth were concerned, I used 5 for both. Unfortunately I didn't get to detail how to use the model selection script, but running that(time costly) should exactly get the best model.
Feel free to follow up with any other questions!


Add a floppy disk save icon!!!!!!!!!

Where is the typical floppy disk save icon?

If you want one you can try asking the heroicons project itself.


Debugger: Wrong current line in source map with inlined source

Test item #8193

VSCode Version: aweinand/asyncsourcemaps: Code - Insiders 1.3.0-insider (392b89ffeebbc51a8574142f9629ff4a6eac15b9, 2016-06-27T22:37:58.724Z)
OS Version: Linux x64 3.13.0-34-generic

Steps to Reproduce:

Open Code with an extension project
Place breakpoint in 'activate' function of the extension
Start debugging
When the breakpoint is hit, step out
Now the current line (correctly) is on TPromise.as(...):
try {
return TPromise.as(extensionModule.activate.apply(global, [context]));
} catch (err) {
return TPromise.wrapError(err);
}
Step in -> Correctly steps into TPromise.as()
Step out -> Correctly steps out of TPromise.as(), but incorrectly shows the current line on TPromise.wrapError(...)
Step in -> Correctly leaves the function (verifies we're not actually in the 'catch' case)

Should work now.


Is it possible to bundle for iOS JavascriptCore?

Current Behavior
I'm trying to bundle my package for use in Apple's iOS JavascriptCore context. This context doesn't have browser globals such as window and document, but it also doesn't have Node globals such as require.
I'm finding that if I bundle for the browser then, naturally, my underlying dependencies expect things like document to be there. And if I bundle for Node, then the result attempts to require or import the dependencies at runtime.
Desired Behavior
Perhaps I'm missing a trick here, but I feel like I need to create a sort of hybrid bundle that runs as if it is in Node, but it contains all the dependencies inline (like you would do for a browser bundle).
Is that possible already? Or is there a better solution?
Suggested Solution
Hopefully what I want to achieve is already possible with one of the existing module formats, but if not perhaps iife would work? #788
Who does this impact? Who is this for?
People, like me, who want to be able to create a package that can be used by both the front end dev team and also the native app dev team.
Describe alternatives you've considered
I've attempted re-bundling the TSDX output with webpack, which seems like it should be unnecessary.
Additional context
Here's my SO question with a bit more detail: https://stackoverflow.com/questions/67193663/how-to-bundle-tsdx-package-for-ios-javascriptcore
The bundle for the React app works great; TSDX made that super easy - thank you.

but it contains all the dependencies inline (like you would do for a browser bundle).
Is that possible already?

Yes. Bundling or not is independent of your target, you just need to unset the default externals. You can see an example of this in https://github.com/formium/tsdx/issues/179#issuecomment-605502952 .
That question is UMD-specific, so I check for UMD specifically in that snippet, but it works outside of UMD as well.

with one of the existing module formats, but if not perhaps iife would work? #788

Per above, externals / bundling deps is also independent of module format. I don't know what format iOS JavaScriptCore supports but you'd want to output to a compatible one.


fix closing of README code block

By having the ``` markup on an existing line, the github web rendering of the code block included the characters.

Coverage remained the same at 65.947% when pulling 2647fa953069014ac999431d8c0901a2ad39abe9 on pdelre:patch-1 into 90fd48a58850ba37efe58e8f12515865b678ef93 on icicleio:master.


rtorrent : error opening terminal : unknow

It's me, again :p
With my pull, I can run rtorrent container but it crash all time with this error :
Error opening terminal: unknown.
addgroup: group 'torrent' in use
And... I doesn't know what is error :/

Found error, I make a pull request ;)


Use shenandoah or zgc

do itttttt

with -XX:+UseShenandoahGC or -XX:+UseZGC
check if your jvm supports it first

ok I'll check

you have to remove arguments relating to g1gc while using them tho


Methods which never return (like run) should be marked as -> Never

As per PR #150, which needs to be fixed in more places.
Note that this is API semver major, so for 2.x.

Of course you should be very sure whether you want the Never behaviour, but if you do - as documented -, it should return Never.
Why might you not want Never? I don't know, maybe because some Lambda's would want to switch the bootstrap for some crazy reason?

lets do this with other breaking API changes for 1.0


[issue-368]: Make Catalogue Server, AAA Server basepaths configurable

Made the Catalogue Server, AAA Server base paths configurable by changing the following files:

config-dev :

 "commonConfig" : {
        "dxApiBasePath" : "/ngsi-ld/v1",
        "dxCatalogueBasePath": "/iudx/cat/v1",
        "dxAuthBasePath": "/auth/v1"
    }

Can one of the admins verify this patch?

ok to test

retest this please


add Boolean widget

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots
[x] Tag @ferdi or @pdpinch for review

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
closes #145
What's this PR do?
This adds a new widget type for setting a Boolean value in the site config. I made a few other changes to parts of the code that I touched when implementing it too:

I created a new enum type called WidgetVariant that captures the different possible widgets. The enum will be transformed into string constants at compile time by typescript.
I slightly refactored how props get from the ConfigField down to the SiteContentField, creating a widgetExtraProps function in lib/site_content.ts which switches on the WidgetVariant and grab the extra props we want out of the ConfigField object for each type.

Anyhow, I think these changes increase clarity a little bit :)
How should this be manually tested?
You can use the override_site_configs management command to add a Boolean field to the site config (I checked in a change to add one to the localdev/starters/site-config-override.yml file). Then check out the metadata config section and make sure you can edit the boolean field.
Screenshots (if appropriate)

Codecov Report

Merging #168 (be38766) into master (815c140) will decrease coverage by 4.43%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   86.38%   81.95%   -4.44%
==========================================
  Files          97       44      -53
  Lines        2395     1308    -1087
  Branches      353      148     -205
==========================================
- Hits         2069     1072     -997
+ Misses        308      219      -89
+ Partials       18       17       -1

Impacted Files
Coverage Δ

static/js/lib/redux_query.ts

static/js/pages/App.tsx

static/js/lib/util.ts

static/js/components/forms/SiteForm.tsx

static/js/query-configs/websites.ts

static/js/pages/SitePage.tsx

static/js/components/forms/validation.ts

static/js/components/SiteCollaboratorList.tsx

static/js/components/Dialog.tsx

static/js/lib/ckeditor/turndown.ts

... and 35 more

Continue to review full report at Codecov.

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


Change color the link when I change the background color of the cell

Is your feature request related to a problem? Please describe.
When I colorized the row and the row is a link to another table, the text in the row is blue, therefore which I use a strong color behind of row, the text is hidden. It would be great if we could change the color of the link as well.
Describe the solution you'd like
I  would like to change the color of the link (default is blue)
Describe alternatives you've considered
Invert color automatically of the link conform I change the background of the cell. The link will contrast the background color.
Additional context
https://github.com/dbeaver/dbeaver/issues/9221

thanks for suggestion


Feature: Ability to change the default identity

Hello,
It would be nice if you can change the default identity under Settings > Accounts because in most cases the login name is not identical to the sender name. If you make a mistake you send a message with your login name which in turn is not present in the alias table.
Thanks in advance.
Karsten

You can, just slide the Identity of your choosing above the other ones :)

Pretty cool. Thank you.

I agree, this is really cool, though not particularly obvious, I missed this trick too!

👍 to this. It's a slick way to implement that, though I had to google to find this thread to know how to do it!
Bumping this so that hopefully more people see this when they also go googling.


Can't delete a preset

Hi,
I have created a preset, to add to my list of preset, but I can't delete it.
I tried doing ",p rmpreset", and it asked me to type the name of the preset, but when I type it there is not response after that.
I don't know why it does not work, maybe because I added an emoji in the name?
Here is a screenshot showing the problem.
By the way, the ",p help" is not very  clear, could you add the ,p rmpreset command in it? I had to read the code to find the command.
Thank you

Hi SoniaMalki! Did you find the answer to your query?
You can delete a saved pattern (previously called a preset) with ,pdelpattern <pattern name>, e.g. ,pdelpattern Anime doro <:koiniboule:832304030351884298>.
In general, you can use ,phelp to obtain more detailed usage information about individual commands, e.g. ,phelp delpattern.


Dogfight mob type dosen't chase straight to player

The mobs when in "dogfight" mode chase the player, but follow him in kind of a grid pattern, and don't walk diagonally until very close to a player.
However the "dogshoot" mobs when chasing go straight toward the player.

With my dogfight mob I had pathfinding set to zero to just turn it off for a second, but apparently the 0 value does the same thing as the 1 value so that was the problem on dog-fighters. Still the dogshoot mobs were not using pathfinding on chase. Do they have pathfinding?

Pathfinding only happens with mobs that follow you when attacking, will look into that...


merge main-v0.13.0 into main

This change is

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (79e16b0) 68.89% compared to head (7f26b4e) 68.97%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

crates/blockifier/src/test_utils/declare.rs
96.29%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1198      +/-   ##
==========================================
+ Coverage   68.89%   68.97%   +0.07%
==========================================
  Files          57       58       +1
  Lines        7092     7113      +21
  Branches     7092     7113      +21
==========================================
+ Hits         4886     4906      +20
- Misses       1791     1792       +1
  Partials      415      415

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


Support other pass types, and functionalities

Hello!
I think this could be a gap filling library for python. I use Google's examples, but this could be better if it's simplifies the process.

Does this library support private generic, generic, loyalty, and other pass types?

Does it support SmartTap functionality?

Does it support api endpoint callbacks (add, remove), and push notifications?

THX!

Hi there, sorry for the late reaction. Currently this library only supports Event Ticket pass types. I created this library to use internally in one of our applications and features like Smart Tap and API endpoint callbacks where not necessary, that's why they are not included and I don't plan on including them any time soon. Feel free to send a MR if you want to add these features to the package!


docs: Change file name to be consistent

Following get-started tutorial
it starts to create the file named 90s-mixtape.md
touch posts/my-first-post.md
touch posts/90s-mixtape.md

But when try to set the content the file name is mentioned as posts/90s-mix-cdr.md

This looks like duplicate of https://github.com/remix-run/remix/pull/585


add hostvisibility information to karmor probe

karmor probe should show if the host visibility for KubeArmor is enabled or not

Hi @Ankurk99 @daemon1024 , I would like to work on this issue. To restate my understanding
Requirement: If the kubearmor-visibility annotation is set on the node, the karmor probe should display this parameter in its output.
Sample output 1:
Node 1 :
        OS Image:                       Ubuntu 20.04.5 LTS
        Kernel Version:                 5.15.0-52-generic
        Kubelet Version:                v1.25.2
        Container Runtime:              docker://20.10.18
        Active LSM:                     AppArmor
        Host Security:                  false
        Container Security:             true
        Container Default Posture:      block(File)             block(Capabilities)     block(Network)
        Host Default Posture:           block(File)             block(Capabilities)     block(Network)
==>	Host Visibility Level:          Process                 File                    Capabilities            Network

Sample Output 2:
Node 1 :
        .....
        Container Default Posture:      block(File)             block(Capabilities)     block(Network)
        Host Default Posture:           block(File)             block(Capabilities)     block(Network)
==>	Host Visibility Level:          Process                 File

Sample Output 3:
Node 1 :
       .....
        Container Default Posture:      block(File)             block(Capabilities)     block(Network)
        Host Default Posture:           block(File)             block(Capabilities)     block(Network)
==>	Host Visibility Level:          None

The dev work is to be done on the probe package.

@legorie Thanks for your interest. Yes your understanding is correct and the sample outputs are what we expect.

Hi @Ankurk99 , After initial analysis, it looks like we need to update the code in the core KubeArmor code too here ..
https://github.com/kubearmor/KubeArmor/blob/main/KubeArmor/core/karmorprobedata.go
type KarmorData struct {
	OSImage                 string
	KernelVersion           string
	KubeletVersion          string
	ContainerRuntime        string
	ActiveLSM               string
	KernelHeaderPresent     bool
	HostSecurity            bool
	ContainerSecurity       bool
	ContainerDefaultPosture tp.DefaultPosture
	HostDefaultPosture      tp.DefaultPosture
        HostVisibility          tp.HostVisibility << a new type ? >>
}

in types.go
// HostVisibility Structure
type HostVisibility struct {
	Process      ProcessType      `json:"process,omitempty"`
	File         FileType         `json:"file,omitempty"`
	Network      NetworkType      `json:"network,omitempty"`
	Capabilities CapabilitiesType `json:"capabilities,omitempty"`
}

Do you think I'm in the right direction ? I would be glad to make the change in the core package too, to test them together.

Hi @legorie, Sorry for the late response. Yes, you seems to be on the right track. Can you please create a draft PR with your changes and then we can suggests there?

Hi @Ankurk99, No worries, thanks for the suggestion. I've created an issue (enhancement) in the core project. If the type and details are okay, please assign this to my name. I'll update the changes to the karmorprobedata.go in that issue.


Draft: MudMenu: Allow hiding the activator for programmatic use

Description

see https://github.com/MudBlazor/MudBlazor/pull/10122#issuecomment-2506505109
Before:

After:

How Has This Been Tested?

visually
Type of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (fix or improvement to the website or code docs)

Checklist

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

@ScarletKuro do you think this is the right approach? Also I'm not sure about the parameter name Hidden, any better ideas?
If you think this is OK I can add tests and docs tomorrow.

Had to get caught up, question- Shouldn't that button always be hidden or visibility collapse or something when not in a popover? I think could probably solve it all with just a css class when laid out in that manner under a popover to not show it and disable it but the popover version would work fine

@versile2 @henon @danielchalmers
What do you guys think? I think this solution is good for edge cases. Ideally, it would be great if the DataGrid natively supported context menus on rows and allowed customization. Then MudMenu near the DataGrid with PositionAtCursor="true" wouldn't be necessary. However, since there isn't such support, I think the proposed solution is the only workaround.

 Why do we need a second property?

When the activator content is null a default button is shown. That's the main use of MudMenu.

 Why do we need a second property?

When the activator content is null a default button is shown. That's the main use of MudMenu.

Right, no activator content or label = hidden?
<MudMenu>
    <MudMenuItem>Enlist</MudMenuItem>
    <MudMenuItem>Barracks</MudMenuItem>
    <MudMenuItem>Armory</MudMenuItem>
</MudMenu>

Right, no activator content or label = hidden?

I like this idea.

Do people use the empty button that's shown in your Before example?

I hope not. I think this behavior qualifies as unintended / buggy.

Right, no activator content or label = hidden?

I like this idea.

We can do this but we should explicitly document it with an example in the docs.

Thanks @meenzen


Support for invoke the flushStatements via Mapper interface.

I want to support invoke the SqlSession#flushStatements() via Mapper interface.
As below:

BoundAuthorMapper mapper = session.getMapper(BoundAuthorMapper.class);
Author author = new Author(-1, "cbegin", "******"<EMAIL_ADDRESS>"N/A", Section.NEWS);

mapper.insertAuthor(author);
mapper.insertAuthor(author);
mapper.insertAuthor(author);
mapper.insertAuthor(author);
mapper.insertAuthor(author);

List<BatchResult> results = mapper.flush();

Merged. Thank you!

@emacarron
Thank you for merging !!
If possible, i want to backport to 3.2.x.
How do you think ?

Yep. No problem. You can do it yourself and make another PR but this time
for the 3.2.x branch.
2015-03-02 17:31 GMT+01:00 Kazuki Shimizu<EMAIL_ADDRESS>
@emacarron https://github.com/emacarron
Thank you for merging !!
If possible, i want to backport to 3.2.x.
How do you think ?
—
Reply to this email directly or view it on GitHub
https://github.com/mybatis/mybatis-3/issues/356#issuecomment-76744987.

Hi @emacarron .
I tried a testing at 3.2.x branch using mvn test.
But testing is failed.
why ?
Results :

Tests in error:
  BindingTest.setup:61 » SQL Failed to start database 'ibderby' with class loade...
  DynamicSqlSourceTest.shouldTrimSETInsteadOfCOMMAForBothConditions:276->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldConditionallyDefault:93->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldConditionallyIncludeWhere:71->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldDemonstrateSimpleExpectedTextWithNoLoopsOrConditionals:53->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimNoWhereClause:261->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREANDWithCRLFForFirstCondition:166->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREORWithCRLFForFirstCondition:205->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfANDForFirstCondition:138->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfANDForBothConditions:246->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimNoSetClause:291->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREANDWithLFForFirstCondition:153->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldSkipForEachWhenCollectionIsEmpty:326->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREORWithLFForFirstCondition:192->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREANDWithTABForFirstCondition:179->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldIterateOnceForEachItemInCollection:309->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldPerformStrictMatchOnForEachVariableSubstitution:343->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREORWithTABForFirstCondition:218->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldDemonstrateMultipartExpectedTextWithNoLoopsOrConditionals:61->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldConditionallyChooseFirst:108->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldConditionallyExcludeWhere:82->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfORForSecondCondition:231->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DynamicSqlSourceTest.shouldConditionallyChooseSecond:123->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  BatchExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  CachingBatchExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  CachingReuseExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  CachingSimpleExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  ReuseExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  SqlRunnerTest.shouldInsert:59->BaseDataTest.runScript:61 » SQL Failed to start...
  SqlSessionManagerTest.setup:40->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  SqlSessionTest.setup:62->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  DuplicateResourceTest.setup:35->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  ManyAnnoTest.testGetMessageForEmptyDatabase:35->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
  NestedQueryCacheTest.setUp:43->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL

Tests run: 692, Failures: 0, Errors: 35, Skipped: 12

try mvc clean, sometimes the derby db is not properly ended and new test
runs fail
2015-03-02 18:30 GMT+01:00 Kazuki Shimizu<EMAIL_ADDRESS>
Hi @emacarron https://github.com/emacarron .
I tried a testing at 3.2.x branch using mvn test.
But testing is failed.
why ?
Results :
Tests in error:   BindingTest.setup:61 » SQL Failed to start database 'ibderby' with class loade...  DynamicSqlSourceTest.shouldTrimSETInsteadOfCOMMAForBothConditions:276->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldConditionallyDefault:93->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldConditionallyIncludeWhere:71->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldDemonstrateSimpleExpectedTextWithNoLoopsOrConditionals:53->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimNoWhereClause:261->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREANDWithCRLFForFirstCondition:166->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREORWithCRLFForFirstCondition:205->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfANDForFirstCondition:138->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfANDForBothConditions:246->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimNoSetClause:291->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREANDWithLFForFirstCondition:153->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldSkipForEachWhenCollectionIsEmpty:326->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREORWithLFForFirstCondition:192->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREANDWithTABForFirstCondition:179->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldIterateOnceForEachItemInCollection:309->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldPerformStrictMatchOnForEachVariableSubstitution:343->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREORWithTABForFirstCondition:218->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldDemonstrateMultipartExpectedTextWithNoLoopsOrConditionals:61->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldConditionallyChooseFirst:108->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldConditionallyExcludeWhere:82->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldTrimWHEREInsteadOfORForSecondCondition:231->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DynamicSqlSourceTest.shouldConditionallyChooseSecond:123->createDynamicSqlSource:357->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  BatchExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  CachingBatchExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  CachingReuseExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  CachingSimpleExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  ReuseExecutorTest>BaseExecutorTest.setup:51->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  SqlRunnerTest.shouldInsert:59->BaseDataTest.runScript:61 » SQL Failed to start...  SqlSessionManagerTest.setup:40->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  SqlSessionTest.setup:62->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  DuplicateResourceTest.setup:35->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  ManyAnnoTest.testGetMessageForEmptyDatabase:35->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL  NestedQueryCacheTest.setUp:43->BaseDataTest.createBlogDataSource:85->BaseDataTest.runScript:61 » SQL
Tests run: 692, Failures: 0, Errors: 35, Skipped: 12
—
Reply to this email directly or view it on GitHub
https://github.com/mybatis/mybatis-3/issues/356#issuecomment-76757563.

sorry.. above issue has been resolve to removing a ibderby  directory.


A few images not downloaded from iCloud

I'm testing this (wonderful) script. Works very well but I noticed an inconsistency on the number of images downloaded from my iCloud account: 827 images on iCloud, 825 images downloaded.
So started to investigate and discovered:

if I go to "Recently discarded" album (iPhone) and recover one picture, wait for sync to iCloud, the same picture remains in the "Recently discarded" album on iCloud web UI and this script does not get it
if I go to "Recently discarded" album (iCloud web UI) and recover one picture here, this script gets it

I then reviewed all pictures and found the two missing pictures (present on iCloud, missing locally): they where pictures older than a few months, that for sure I did discard and recover after a few days.
So it seems to me (and I'd like others to comment if already seen):

when I discard a picture on the iPhone, and recover that picture, the same does not gets immediatly recovered on iCloud
(perhaps) after the recovery time the picture starts being visible in iCloud but still not downloaded

Thanks again for reporting this issue. It sounds like it might be a problem with the iCloud API. I'm going to close this issue, but hopefully Apple will fix it, or someone will find a way to fix it in the pyicloud library that I'm using.


[QUESTION] Need funds to sign transaction?

So I've tried the uport-react-native-demo awhile back but when I've created my own app where I tried to write to the blockchain, my other Uport account wasn't able to "Sign Transaction" due to Insufficient Funds.
I've also thought that I've used my primary Uport account to create the Uport App on the App Manager but then I realized that this does not happen on the uport-react-native-demo where I was able to "Sign Transaction" to a different smart contract address without having ETH on me which was the Uport Demo.
Just wanted to know if there's a work around to this.
Results
Executed the same function on separate phones with different Uport accounts but both showed different actions on the Uport app.
Primary Uport Account

Secondary Uport Account (Problem)

Just found this out Meta Transaction Relaying Server though not sure how to implement it yet hehe


Fixer - fix handling of empty file

If file is empty then many fixers crash during operating on empty collection.
What's more there is no need to lint empty file etc.

:+1:

What about the new line at eof fixer?

BTW, found on #902 regression

What about the new line at eof fixer?

We add a new lines only for non-empty files

See https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/master/Symfony/CS/Tests/Fixer/PSR2/EofEndingFixerTest.php#L63

merged

We add a new lines only for non-empty files

Just checking. :)