 (Y/n) · true
Good call, now using TypeScript! 🚀
✔ What would you like to use for Navigation? · None
No problem, skipping navigation for now.
✔ What would you like to use for styling? · Tamagui (experimental)
You'll be styling with ease using Tamagui.
✔ What would you like to use for authentication? · None
No problem, skipping authentication for now.
✔ Which package manager would you like to use? · bun

Your project configuration:
{
  projectName: 'my-expo-app',
  packages: [ { name: 'tamagui', type: 'styling' } ],
  flags: {
    noGit: false,
    noInstall: false,
    overwrite: false,
    importAlias: true,
    packageManager: 'bun'
  }
}

To recreate this project, run:
  npx create-expo-stack@next my-expo-app --tamagui --bun

Initializing your project...

Copying base assets...

Installing dependencies using bun...

Success! 🎉 Now, just run the following to get started:

cd my-expo-app
bun run ios

If you frequently use create expo stack, please consider sponsoring the project ❤
https://github.com/sponsors/danstepanov
➜  test cd my-expo-app
➜  my-expo-app git:(main) bun run ios
$ expo start --ios
Starting project at /Users/dan/Developer/test/my-expo-app
CommandError: It looks like you're trying to use web support but don't have the required dependencies installed.

Please install<EMAIL_ADDRESS>by running:

npx expo install @expo/metro-runtime

If you're not using web, please ensure you remove the "web" string from the platforms array in the project Expo config.

error: script "ios" exited with code 1

no longer an issue as of v2.6.4


Repo sync

Thank you for your contribution!
Before submitting this PR, please make sure:

[ ] If you are fixing something other than a typo, please always create an issue first.
Otherwise, your PR will probably be rejected
[ ] Your addition is not too large. The more files/lOC changed means the longer it'll take to be addressed by me (the maintainer)
[ ] Your code builds clean without any errors or warnings
[ ] You have added user documentation for any additional features
[ ] You have added unit tests which (a) pass and (b) sufficiently cover your changes

Other Notes:
Coravel is created with a specific vision and philosophy:

Simple, fluent and straightforward public APIs
Easy to read documentation that guides users coherently
Building useful features because they really are useful - not just for the sake of adding "more features"

To that effect, don't feel bad when many changes are asked of your PR 😉

repo sync


How will this interact with non-layout-affecting URL bars?

The proposal in https://github.com/bokand/URLBarSizing is something that Safari already does and we're likely to switch to shortly. We should be explicit about how the two interact.
In short, I think the real problem here is that when we say "layout viewport" we're actually referring to the "position: fixed viewport". We've often said that the visual viewport is bounded by the layout viewport but in that many cases, the layout viewport doesn't have much to do with layout. In the case of the static-layout URL bar, hiding the URL bar will increase the height of the visual viewport and the position: fixed viewport (what we currently call the layout viewport), but not the initial containing block (the "real" layout viewport"). Perhaps we need a shift in terminology...

Shrinking the top controls shrinks both the visual and layout viewports but not the layout size. This is only weird because layout viewport sounds like it should affect layout but it should more appropriately be called “position: fixed viewport”.


Must process repeating groups in order?

Coding in C#, it seems that repeating groups must be traversed in their order of appearance in the schema. In other words, group B was specified before group A. I tried to skip over group A and traverse group B directly and got corrupted data. Simply looping over group A before accessing group B solved the problem.
If this is the desired behavior, please make this very clear in the documentation and consider throwing an exception for incorrect traversal?

This is by design, and the same behavior applies in java and C++.
You must read fields, then groups, then variable length fields, in the order specified in the schema. We do not throw exceptions as it would require inserting additional branching in the code (I think we talked about that some time ago and decided yo not do it).
Regarding the doc this requirement appears in the design principles but I agree it should probably be highlighted in a few more places. Martin, Todd what do you think?
Olivier

On 1 Mar 2015, at 18:26, Shay Rojansky<EMAIL_ADDRESS>wrote:
Coding in C#, it seems that repeating groups must be traversed in their order of appearance in the schema. In other words, group B was specified before group A. I tried to skip over group A and traverse group B directly and got corrupted data. Simply looping over group A before accessing group B solved the problem.
If this is the desired behavior, please make this very clear in the documentation and consider throwing an exception for incorrect traversal?
—
Reply to this email directly or view it on GitHub.

Thanks for the quick response. A much more visible mention would definitely save a lot of frustration to newcomers...
I definitely understand wanting to avoid the performance hit associated with the additional branching. May I suggest adding these checks with #IF DEBUG in C# and Java? I admit the value here is limited because people will be normally coding against the release build, but it may still help.

One idea: when doing code-generation, you can simply place #IF DEBUG around sanity checks such as the proper order. This way if a user compiles your code-generated classes with DEBUG they get the error checking. The same can be done in C++ with NDEBUG.
This gives users the best of both worlds: checks in debug builds without compromising on performance in release. Of course, this doesn't take care of checks in the non-generated core (i.e. SBE.dll), in which case the user would have to link to a DEBUG-compiled version. But checks in the generated code should probably provide a lot of sanity?

I think it would be useful to add some compilable checks. C# and C++ would be useful. But Java might be slightly less desirable. We can add a static to get the optimiser to avoid it, but it might balloon the methods a good bit and cause them to not be inlined. So, somewhat tricky to do. Going to make this as an enhancement.

Similarly, if you forget to call next() before you write to a group for the first time you will get a corrupt message as the offset member variable will not have been correctly set. This is another example of something that could be checked for in debug build/debug mode.

The idiomatic way to do this in Java is with assertions.
It would be nice however to be able use the API properly without having to read the schema...
Would you accept a patch to decode sequence dependent fields using callbacks instead of getters? That way SBE could invoke the callback only at the right time.
(Should we move discussion of this more ambitious feature to a new issue?)

Best to move this to a separate discussion. Messages can already be decoded with callbacks using the OTF decoder. Whatever approach is taken it should honour the design principles and preserve performance.
https://github.com/real-logic/simple-binary-encoding/wiki/Design-Principles


(PUP-9362) Remove redundant tests

The setup data for the test preceding the ones removed structured things in such a way that all four tests (including the one not removed) had identical content, and expectations. Rather than run the exact same data through the exact same process, and expect the exact same results, we only do this once now, even though it's not (and never was) individually testing each aspect of the transformation.

CLA signed by all contributors.


Failed to create a new whisper context. core\src\transcribe.rs:37:65 on Windows

What happened?
Vibe has repeatedly crashed and refused to transcribe audio, but will connect to the Youtube to MP3 client successfully. Resetting Vibe and updating to the latest release didn't change this. I believe this may be connected to the user who experienced this two days ago.
Steps to reproduce
Attempt to transcribe any audio into text
What OS are you seeing the problem on?
Window
Relevant log output
failed to open model

Caused by:
    Failed to create a new whisper context.

Location:
    core\src\transcribe.rs:37:65
App Version: vibe 2.6.6
Commit Hash: 8244a16
Arch: x86_64
Platform: windows
Kernel Version: 10.0.19045
OS: windows
OS Version: 10.0.19045
Cuda Version: n/a
Models: ggml-medium.bin
Default Model: "C:\\Users\\dabnj\\AppData\\Local\\github.com.thewh1teagle.vibe\\ggml-medium.bin"
Cargo features: vulkan

{
    "avx": {
        "enabled": true,
        "support": true
    },
    "avx2": {
        "enabled": true,
        "support": true
    },
    "f16c": {
        "enabled": true,
        "support": true
    },
    "fma": {
        "enabled": true,
        "support": true
    }
}

See https://github.com/thewh1teagle/vibe/issues/370


Release notes for 0.13

https://github.com/simonw/llm/compare/0.12...main
Don't forget to mention the smaller weirder changes like:

~LLM_OPENAI_SHOW_RESPONSES is now OPENAI_LOG=debug~
Dropped 3.7 support
api_base still works, even though it now gets passed as base_url

https://llm.datasette.io/en/stable/changelog.html#v0-13

Annotated release notes: https://simonwillison.net/2024/Jan/26/llm/


RandomTreeGen-Aradhya

"This artwork is a tree upside down with leaf (represented as leafs now!) and it is randomly generated by a randSeedGen. I spent around 6 hours coding this and going through the docs and I am happy with the results. Overall, I liked this project a lot and I would love to get a Blot to try this out irl! Thanks, aradhya"
Added Thickness to the branches and made it look more natural.

Any Updates?

Thank you for the submission. This is too little variation on a classic fractal. How could you really give the tree some thickness?

I'm going to close this for housekeeping but please resubmit when you are ready.


Add Autocast Utilities External Plugin

Add Autocast Utilities external plugin.
This plugin displays an overlay when a magic weapon capable of autocasting is equipped. The overlay shows the name and icon of the current spell. If the player's magic level falls below the level required for the spell, the overlay has the option to flash and send a game notification.
Link to plugin.
Thank you for your time!

Related to issue #14503

Hi, I think I fixed the build issue (had the wrong path for the plugin in the .properties file). Please let me know if there's any other help needed on my side.

Does this work for when you run out of runes and try to attack? This happens to me all the time 😅 I accidentally attack something without runes in my inventory and it deselects my autocast without me realizing. I grab runes at some later point to do some combat and I just start whackin' 'em 😆

Does this work for when you run out of runes and try to attack? This happens to me all the time 😅 I accidentally attack something without runes in my inventory and it deselects my autocast without me realizing. I grab runes at some later point to do some combat and I just start whackin' 'em 😆

Hey Adam, I didn't think about that. It currently does not notify when that happens, but I think that can be added. I'm working on another plugin right now that I want to release asap, but after I'm done with that, I'll work on adding that feature in. I'll create an issue on the repo for this and tag you. Thanks for the suggestion!

your package statements are wrong; they should just read com.autocastutilities, not that.
otherwise lgtm

Thanks, abex. I have updated the path of the plugin.

It looks like you renamed your repository

You're right, thank you, raiyni! I seem to have missed that. I've updated the repository name.


Android - ConnectToDeviceAsync never completes, connecting to a different device 'fixes' issue

I've ran into a periodic issue I see from time to time. I'm doing lots of builds, DEBUG and RELEASE and generally have no issues. I have a couple of devices I connect too.
Sometimes, I find on Android that I can no longer connect to a given device. I can scan fine, see the device, but any call to ConnectToDeviceAsync or ConnectToKnownDeviceAsync just never completes....no errors are thrown and the CancellationToken timeout eventually just fires.
If I try to connect to another device (not the previous one used), that works...then trying to reconnect to the previous device that never completed - that then works and everything is 'fine' again.
I can uninstall the app, reinstall the current build, older builds, DEBUG build or RELEASE build and I still can't connect the previous device.
Has anyone seen anything similar?
Heres the trace output when trying to connect to a device that never completes (DeviceConnected never fires)...eventually the cancellation token timeout fires.
[Info] (BluetoothConnectionService.ConnectAsync) Lock Waiting
[Info] (BluetoothConnectionService.ConnectAsync) Lock Acquired
[BluetoothManager] getConnectionState()
[BluetoothManager] getConnectedDevices
[BluetoothGatt] connect() - device: CE:9F:97:95:BE:46, auto: true
[BluetoothGatt] registerApp()
[BluetoothGatt] registerApp() - UUID=5402ae97-da5d-4721-9f97-6ca17c077b43
[BluetoothGatt] onClientRegistered() - status=0 clientIf=11
[OpenGLRenderer] endAllActiveAnimators on 0x70245ca100 (RippleDrawable) with handle 0x70225e7900
<-- Cancellation Token duration expires -->
[BluetoothGatt] cancelOpen() - device: CE:9F:97:95:BE:46
Failed to evaluate expression in conditional breakpoint. Unknown identifier: inflaterFailed to evaluate expression in conditional breakpoint. Unknown identifier: inflater[Info] (BluetoothConnectionService.ConnectAsync) Lock Releasing
[Info] (BluetoothConnectionService.ConnectAsync) Lock Released

Hi @IainS1986,
I'm the friendly issue checker.
It seems like (100.00 %) you haven't used our issue template :cry: I think it is very frustrating for the repository owners, if you ignore them.
If you think it's fine to make an exception, just ignore this message.
But if you think it was a mistake to delete the template, please close the issue and create a new one.
Thanks!

No solution as of yet. Are you experiencing the same issue?
Don't know if this repository is no longer supported anymore.

Had the issue occur again, the only consistent thing I could see between both instances of the bug occurring was that I didn't connect to the device (or use the app) for around a week (holiday).
I've forked the repo, pulled in the code and can see in Device.cs this is executed as expected...
var connectGatt = BluetoothDevice.ConnectGatt(Application.Context, autoconnect, _gattCallback, BluetoothTransports.Le);
However the GattCallback.cs OnConnectionStateChange is never fired.
This continued even with deleting and reinstalling the app.
I tried enabled Bluetooth HCI Snoop log in the developer options, and frustratingly that fixed the issue for now! (so the log is likely not going to help as it worked). I'll keep an eye out but I think it will happen again if I leave the app for "some time", which doesn't happen during normal development hence why I've not really seen this bug occur.

@IainS1986 Hello, Sorry for the later answer, So I have search about this. I have found why this problem. That's because bluetooth le (bluetooth low energy) can't connect with another devices if the last one use bluetooth classic. try to connect your app with bluetooth casque or microphone or another device use ble and it's working .
if you want connect it with another phone for exemple you need to use bluetooth classic ... For android he work without any problem...but for IOS i can't found any solution to connect it
Sorry because I'm not good in English

I don't think thats the issue i've been facing.
I have a device thats built in house, call it Device A. I have another identical device built in house, call it Device B.
The app connects fine to Device A all* of the time, Debug builds, Release builds, etc.
I leave the app for a week (not confirmed this is definitely the way to reproduce it).
I boot the app up again, and try to connect to Device A (the last device it connected too and no other connection to Device A has occurred since. It fails.
The app no longer connects to Device A. The ConnectToDeviceAsync just never completes and the Gatt callback never fires.
I try to connect to Device B, works first time.
I then try to connect to Device A again, works first time, everything is working as normal again.

Update:
Ok, further investigation after what you just said. Its not time, you're right that it's maybe connecting to another device breaks it. I thought it was leaving it for some time, but during that time I connected my phone to some headphones and a car bluetooth dongle. That was what broke it.
Can reproduce the issue.

App connects to Device A fine.
Disconnect app from Device A and close app.
I connect my phone to some Headphones (not sure what Bluetooth layer it uses).
Disconnect headphones.
Boot up app.
Try to connect to Device A as before.
App no longer connects to Device A.
Connect to Device B, works fine.
Connect to Device A, works fine.

OMG, it's verry weird ... for me , I have ESP32 (It's my device A ), she connect without any problem in IOS . she write and read fine... but for my Android phone, He don't found My ESP32 with this plugin!! I don't know why!!! and when I try to connect it with another app like you, He work fine without problem in IOS and Android, Now i try to use another Nuget it's https://github.com/shinyorg/shiny
So for you, if you want connect just with android, I can help you how do it with bluetooth serial( CLASSIC), But with IOS me too i'm working for solution.
If you have any news about connexion tell me... And me too I do.

@IainS1986 I used https://github.com/shinyorg/shiny she work fine without any problem for me in ios and android... try to use it. If you have any problem Tell me .
Good Luck

I think we have different issues @YBENZAKOUR as you were saying you could never connect with Android, whereas mine is all working fine 99% of the time but I was (somehow) getting into a state where the app would no longer connect to a single device.
For others (if anyone reads this), I'm experimenting with a solution that I can't confirm works until I maybe (or maybe don't) see this issue reproduce.
I've turned off AutoConnect and ForceBLE in the connection parameters. Firstly, I believe our device isn't dual mode so ForceBLE is maybe fine, but I'm wondering if AutoConnect was breaking somehow internally.
I've set it to false, BUT FYI this does trigger more Gatt 133 issues sporadically so I've had to implement a simple retry system into our bluetooth task handling.
To combat Gatt 133...

Ensure you're making all bluetooth calls on the Main Thread
Ensure you are doing them synchronously. Do NOT make 2 Bluetooth calls at the same time, or overlap them in any way. Block any Bluetooth calls from starting until the current one is finished.
Do NOT start connecting while you are still scanning
You will STILL get them occasionally, so do a retry system on connecting. So far I've never had it retry more than 3 times to connect.

I'll see if I get the old issue again, but it could be something deep within the OS bluetooth handling on my phone (Moto G6+) with the AutoConnect parameter :shrug:

So, an update after another week has passed.
My current app works as such,
Scan -> Connect.  then after doing that once with a device and getting the GUID, it just goes straight to Reconnect.
Force BLE is OFF and AutoConnect is OFF. (Unlike before).
A week passed, booted the app up, a Reconnect call was started and....no luck. It wouldn't connect.
But going to do another scan, then connect DID work (which before would also not work until I connected to a new device).
So google around a bit more I think I might have figured out whats going on, and think it comes down to just Androids inconsistencies with BLE across manufacturers.
These Medium articles helped shed some light<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Android has an internal BLE Cache which can be cleared at some point. Sometimes when Bluetooth is turned off, sometimes when the phone is turned off, or in my phones case (Moto G6+)....just 'some other time'!
When this happens, using the AutoConnect parameter will not work. Android, at least on Moto, cannot autoconnect to a previously known device if its no longer in the OS Cache. Even though the GUID is right, the internal cache is empty.
Similarly, when using Reconnect API, it builds a BluetoothDevice from the GUID but it can't connect to this if the OS Cache is empty! The medium article mentions you can check for this issue by looking at the BleutoothDevice deviceType to see if its 'Unknown', in which case you can't reconnect. Otherwise a new Scan + Connect should work (and does in my instance).
So. I think this is the issue. I'm going to do another test.
I'm going to switch ForceBLE back on as I think this will mitigate some of the Gatt 133 errors I'm seeing as they may be occuring when it tries to use Classic.
I'm then going to remove calls to Reconnect and instead do a Scan -> Connect loop every time....this is fine for my use case to be fair.
I'll have 2 apps installed on my phone, one that tries to Reconnect, one that does a fresh Scan -> Connect.
We'll see in another week (or if I can figure out how to clear my phones BLE cache properly) to see if App A can't reconnect but App B can and if so, I think thats the culprit!

Update:
Another week passed. Checked the two apps.
App A: AutoConnect=FALSE, called ConnectToKnownDevice -> Failed to connect
App B: AutoConnect=FALSE, did a scan and then ConnectToDevice -> Connected
So far it seems like my suspicions around trying to connect to a known device when the internal OS BLE cache has been cleared does not are correct.
I also suspect this repo is potentially abandoned at this point? Either way, anyone who runs into this...
If you use AutoConnect=TRUE, if the device BLE cache is cleared, ConnectToDevice and ConnectToKnownDevice can fail to ever connect
If you use ConnectToKnownDevice, this can fail to connect if the device BLE cache is cleared, regardless of AutoConnect value
My advice, avoid both if you're not sure, otherwise you will have to do your own handling for fallback to Scan->Connect. I might look to add functionality to this plugin to have a 'check can reconnect' function in Droid that checks if the Device object type is Unknown or not, as that (apparently) lets you know if you can do a ConnectToKnownDevice call or not
I might report back periodically to confirm my findings, or if they change.

Update for anyone - more time has passed and i've seen the versionof the App that tries to do a ConnectToKnownDevice call fail multiple time while the version of the app that does a fresh Scan + ConnectToDevice call work fine.
Pretty confident I've confirmed it to be Android internal BLE cache clearing out, and there for ConnectToKnownDevice will fail and AutoConnect=TRUE will fail.
Propose maybe having a 'CanConnectToKnownDevice' call that for iOS etc just returns TRUE and for Android does a check on the device type to see if its Unknown (not cached), similar too this Android code...
// Get device object for a mac address BluetoothDevice device = bluetoothAdapter.getRemoteDevice(peripheralAddress) // Check if the peripheral is cached or not int deviceType = device.getType(); if(deviceType == BluetoothDevice.DEVICE_TYPE_UNKNOWN) {     // The peripheral is not cached } else {     // The peripheral is cached }


Reading fragments from apolloStore

Hi,
could you please provide example of reading fragments from apolloStore in ver 3.0.0-beta1 ?
You provide method:
apolloStore.toRx3ApolloStore().rxReadFragment
where as parameter Fragment is expected. Generated code extends Fragment.Data. How can I get implementation of Fragment interface. Do we need to implement it ourself?

Hi 👋 !
You can use the generateFragmentImplementations Gradle property
Then you can read your fragment with:
apolloStore.readFragment(fragment, CacheKey(id))

We'll add that to the docs!

Thanks! One step forward :)

Hello, I am trying to read a fragment from apolloStore but I am encountering a cache miss exception and I'm not sure what I'm doing wrong
Before trying to read the fragment I have already executed a query that returns that fragment as part of its response
My fragment
fragment MediaView on Media {
	id
	type

	title {
		userPreferred
		english
		romaji
		native
	}
	coverImage {
		large
	}

	episodes
	chapters
	volumes

	format
	status
	genres
	countryOfOrigin
	seasonYear

	averageScore
	popularity

	startDate {
		year
		month
		day
	}

	isAdult

	bannerImage
}

my extra.graphqls
extend type Media @typePolicy(keyFields: "id")
Am I doing something wrong? Would be very grateful for help


templates: Rename *.handlebars ↦ *.hbs and - ↦ _.

See https://chat.zulip.org/#narrow/stream/6-frontend/topic/Handlebars.20filename.20syntax.

This was merged in #12761 as 3c3471b720cc56651eb7b9eb8b781d951c947f83.


ssue 150, update old node.js version

issue: https://github.com/olivernn/lunr.js/issues/150

@olivernn , sorry for the multi-commit, I'm not familiar with git. Please help to review the code change, if ok, I will separate the two commits.


Minor: Refactor group by state machine

Which issue does this PR close?
Follow on to https://github.com/apache/arrow-datafusion/pull/6932
Rationale for this change
I was a little unhappy about the extract_ok! macro and @tustvold
offered a suggestion on how to remove it:  https://github.com/apache/arrow-datafusion/pull/6932/files#r1267309181
This PR is what I came up with . I am not really sure it is better but wanted to put it up in case others had an opinion

What changes are included in this PR?
Pull out the state machine transition logic in group by into some new functions and remove the extract_ok! macro

Are these changes tested?

Are there any user-facing changes?

I think I am inclined to leave this code alone for now as it has already undergone a bunch of churn and this is just stylistic and I want to move on to other things. If anyone else feels differently, please feel free to open a new PR


Data validation for forms

Login and registration button is enabled only when valid data is entered.

@shankybnl thank you for contribution!
I invited you in organisation with write access to frontend/backend repo
create branches from this repo instead your fork, this will fix cypress tests

Split the pull requests. I still kept this pull request from my form open for the form controls. I will keep adding changes as new branch instead of fork.
https://github.com/Visual-Regression-Tracker/Visual-Regression-Tracker/issues/172 -This is partially doe.
@pashidlos , I have not created a twitter account yet.

@shankybnl thank you for contribution!
I invited you in organisation with write access to frontend/backend repo
create branches from this repo instead your fork, this will fix cypress tests

Thank you @pashidlos for giving the access.

These are the types of validation that are part of this pull request.

These are the types of validation that are part of this pull request.

@suratdas it looks awesome!

@suratdas it looks awesome!


Make documentation on using Templates

We currently use a CustomComponentMap when displaying the ThingBody component. Create documentation on why we have/use that.

This should really be documented under https://picolabs.atlassian.net/wiki/spaces/docs/pages/454098948/Creating+a+New+Manifold+App, and essentially is once the page is finished.


In the summary topic there are two wrong samples

data-summary-options.json is the first sample of the topic and it does not show the same thing that the angular sample do.
data-summary-formatter.json is the sample that is under Formatting summaries section. It does not do what it is expected.

@mddragnev please provide info what is expected or links to angular samples that we can use as references when correcting xplat samples

data-summary-options.json has been updated, but we need to spend time adding templates to the headers to toggle the summaries
https://github.com/IgniteUI/igniteui-xplat-examples/pull/190
"data-summary-formatter.json is the sample that is under Formatting summaries section. It does not do what it is expected."
I do believe we already updated this:
https://staging.infragistics.com/products/ignite-ui-blazor/blazor/components/grids/grid/summaries#formatting-summaries
https://staging.infragistics.com/products/ignite-ui-angular/angular/components/grid/summaries#formatting-summaries


RaycastHit _hit; if statement not working

Sometime between episode 12 and 13, my Raycast stopped detecting hits. Any idea what is happening?
Below is the Shoot(); I added debug.logs to find the error.
[Client]
void Shoot()
{
if (!isLocalPlayer)
{
return;
}
    //WE ARE SHOOTING, CALL THE ONSHOOT METHOD ON THE SERVER
    CmdOnShoot();

    RaycastHit _hit;
    if (Physics.Raycast(cam.transform.position, cam.transform.forward, out _hit, currentWeapon.range, mask))
    {
        Debug.Log("RaycastHit"); //DID NOT PLAY
        if (_hit.collider.tag == PLAYER_TAG)
        {
            CmdPlayerShot(_hit.collider.name, currentWeapon.damage);
        }

        //we hit something call the on hit method ont the server
        Debug.Log("raycasthit"); //DID NOT PLAY
        CmdOnHit(_hit.point, _hit.normal);
    }
    else
    {
        Debug.Log("nohit"); //DID PLAY
    }
}

ugh....
On player prefab, the mask was set to only Weapon, instead of everything but localPlayer and Weapon.


Controller Readiness endpoint

We should have a readiness endpoint so that the k8s controller knows when the pod is ready

Hey @dtomcej, just had a look at the controller Helm chart template and it doesn't seem to expose any ports. In this case where is the need for a readiness endpoint and check if the controller itself does not receive any kind of traffic?
Feel free to correct me and I'll happily implement the readiness endpoint. :+1:

@brennerm, you are correct, it does not expose any ports, but it does have a startup process.
It is possible that we could implement it as an http endpoint, or even have it touch a file once successfully connected to the API etc.
https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/#container-probes
Perhaps we set it as ready, as soon as it successfully deploys all configurations to the mesh nodes.
@SantoDE @mmatur what do you think?
My issue here is that we can have users install Maesh, and see 1/1 ready in the pod list, but it might not be usable at that point. I would like to avoid that.

Had a look at the controller code and this position looks like a good point to start the readiness HTTP endpoint to me.
https://github.com/containous/maesh/blob/master/internal/controller/controller.go#L131
Let me know what you think.

@brennerm,
That looks pretty good IMO. After the base services are created, and right before the main loop starts.
Perhaps we may have the handler created and started on the controller.init method, and then we set it to be ready at that point?

Related to #321


DATAGO-69642: reduce number of log messages sent to ep

User description
What is the purpose of this change?
The purpose of this PR is to reduce the number of log messages sent to ep.
Current behaviour:

Whatever is logged on stdout/file using the logger, we sent the same log message to ep-core
This behaviour is creating a lot of noise in DD

New behaviour proposed in this PR:

execution log for commands are written to a file (Each command will get its own execution log file)
If the command is success, we don't send any log to ep-core
If the command fails, ema reads the execution log file and streams all the log messages to ep-core

NOTE:

Due to the implementation nature, the LOGS in ep-core are not streamed real time

How was this change implemented?
Java code change
How was this change tested?
IT, manual
Is there anything the reviewers should focus on/be aware of?
NA

Type
enhancement, bug_fix

Description

Introduced CommandLogStreamingProcessor for handling execution log files, including streaming logs to EP and file cleanup.
Removed the now redundant CommandLogsProcessor and its route builder, consolidating log handling into the new processor.
Updated TerraformManager to create execution log files and write Terraform command outputs to them.
Enhanced TerraformUtils with methods for execution log handling and converting messages to Terraform-style logs.
Adjusted tests and test configurations to accommodate the new log handling approach.

Changes walkthrough
Relevant filesEnhancement

CommandManager.javaIntegrate CommandLogStreamingProcessor in CommandManager

service/application/src/main/java/com/solace/maas/ep/event/management/agent/command/CommandManager.java
Added CommandLogStreamingProcessor to the CommandManager constructor for log streaming.  Implemented execution log file creation and streaming logic in the configPush method.  Added cleanup logic for execution log files.

+24/-4

CommandLogStreamingProcessor.javaImplement CommandLogStreamingProcessor for Log Handling

service/application/src/main/java/com/solace/maas/ep/event/management/agent/processor/CommandLogStreamingProcessor.java
Created CommandLogStreamingProcessor for handling execution log files.  Implemented methods for streaming logs to EP and deleting execution log files.

+199/-0

TerraformManager.javaImplement Execution Log File Handling in TerraformManager

service/terraform-plugin/src/main/java/com/solace/maas/ep/event/management/agent/plugin/terraform/manager/TerraformManager.java
Added execution log file creation and writing in the execute method.  Modified executeTerraformCommand to write logs to the execution log file.

+62/-15

TerraformUtils.javaEnhance TerraformUtils with Execution Log Handling

service/terraform-plugin/src/main/java/com/solace/maas/ep/event/management/agent/plugin/terraform/manager/TerraformUtils.java
Added utility methods for execution log directory creation and HCL file writing with logging.  Introduced a method to convert generic messages to Terraform-style log messages.

+75/-24

Bug_fix

CommandLogsProcessor.javaRemove CommandLogsProcessor and Route Builder

service/application/src/main/java/com/solace/maas/ep/event/management/agent/processor/CommandLogsProcessor.java
Removed CommandLogsProcessor and associated route builder as it's replaced by CommandLogStreamingProcessor.

+0/-59

Tests

TestConfig.javaUpdate Test Configuration for CommandLogStreamingProcessor

service/application/src/test/java/com/solace/maas/ep/event/management/agent/TestConfig.java
Updated test configuration to mock CommandLogStreamingProcessor.  Adjusted CommandManager bean creation to include CommandLogStreamingProcessor.

+29/-5

CommandManagerTests.javaAdapt CommandManager Tests for Execution Log Handling

service/application/src/test/java/com/solace/maas/ep/event/management/agent/commandManager/CommandManagerTests.java
Modified tests to mock the return of an execution log path from TerraformManager.

+5/-3

Preparing PR description...

Preparing review...

PR Review

 ⏱ Estimated effort to review [1-5]
4, due to the complexity of the changes involving multiple components and the need to understand the context and impact of the modifications on the logging and file management system.

 🧪 Relevant tests
Yes

 🔍 Possible issues

Possible Bug: In TerraformManager.java, the execute method now returns a Path object representing the execution log file. However, if an exception occurs before the execution log file is created or if the file creation fails, this method may return a null Path. This scenario is not handled in the calling code, which could lead to NullPointerException.

Error Handling: In TerraformManager.java, when writing to the execution log file fails, the exception is caught, logged, and then rethrown as an IllegalArgumentException. This could potentially mask the original exception and make debugging more difficult. Consider logging the error and handling it in a way that preserves the original exception context.
 🔒 Security concerns
No

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


Specify scroll view element

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (836bd7a) 89.87% compared to head (e3f42dc) 89.87%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #26   +/-   ##
=======================================
  Coverage   89.87%   89.87%
=======================================
  Files          22       22
  Lines         385      385
  Branches       60       60
=======================================
  Hits          346      346
  Misses         30       30
  Partials        9        9

Impacted Files
Coverage Δ

src/components/VariablePanel/useContentPosition.ts
43.47% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Items actions

Work
Adds some missing Items Actions calls:

decline_refund
raise_dispute
request_resolve_dispute
resolve_dispute
escalate_dispute
send_tax_invoice
request_tax_invoice

Code snippet
refund
item.decline_refund

disputes
item.raise_dispute(user_id: '5830def0-ffe8-11e5-86aa-5e5517507c66')
item.request_resolve_dispute
item.escalate_dispute
item.resolve_dispute

tax invoices
item.send_tax_invoice
item.request_tax_invoice

Coverage increased (+0.03%) to 99.539% when pulling 7c7e877651619360417c5cfcee242f1386400cdc on items_actions into b5c4e211acb0a0dfe3fe32df12b4c93a4ffb14bc on master.

Coverage increased (+0.03%) to 99.539% when pulling 7c7e877651619360417c5cfcee242f1386400cdc on items_actions into b5c4e211acb0a0dfe3fe32df12b4c93a4ffb14bc on master.

Coverage increased (+0.03%) to 99.539% when pulling 7c7e877651619360417c5cfcee242f1386400cdc on items_actions into b5c4e211acb0a0dfe3fe32df12b4c93a4ffb14bc on master.

Coverage increased (+0.03%) to 99.539% when pulling 7c7e877651619360417c5cfcee242f1386400cdc on items_actions into b5c4e211acb0a0dfe3fe32df12b4c93a4ffb14bc on master.

Merging this now. Thanks!


Add support for the no_responders feature

Observed behaviour
The old client enabled the no_responders client feature by default, but this one seems to have it disabled and no way to enable it currently.
This feature has been immensely helpful for request-reply patterns so I'd love to see it added back in. I can take a look at getting an initial PR started for it :)
Expected behaviour
Clients should receive a NatsNoRespondersException (503 status code) when sending a subject message that has a reply_to header but no subscriber is registered to receive that subject.
Server and client version
.NET client v2.0.0
Latest NATS server
Host environment
Latest Linux ARM docker image
Steps to reproduce
Publish a message/request with a reply_to header to a subject that has no subscribers. The reply_to address should receive an empty message with an error code 503 in the header indicating that no eligible subscriber for that subject is registered.

Guess this can be closed now @mtmk? :)


Add sum function

Add a function for summing all elements in the iterable and variants converting the result to double/int.

Thank you for your contribution. I wonder if adding something that can be done with the standard library in a strait-forward way is worth adding 3 methods? Also I am afraid that this will lead to an explosion of other similar operations for multiplication, concatenation, etc.

Tell that my production code, which crashed on an empty list (a corner case) because my company's previous internal implementation (done by me, by the way) used reduce instead of fold 😉
Isn't basically the point of more to implement the missing methods of Dart, so people aren't forced to? When the manually implemented variants can contain very difficult to catch bugs, like the reduce/fold issue already mentioned?
And what cost are you referring to, precisely? I believe the cost is very low and the benefit is, potentially, high.
If sum is so straight-forward to implement, why do most other reasonable languages include it in the standard library (Python, Kotlin (dozen variants!), Scala, I can go on)? Isn't more exactly an extension to Dart's standard library, when the latter is missing features?
Multiplication of an iterable is a much rarer operation, in my experience, most other languages don't include it in the standard library.
If there's no built-in method in Dart for concatenation of Iterable<String>, prefereably with a join-string character defaulting to ", ", then more sounds like a perfect place for such util (just after the standard library).
Please reconsider this pull request.

Reconsidering this in the context of adding various statistical functions.


Include steps when the device fails to join

Hi,
It appears you are documenting troubleshooting steps when a device has partially joined AAD, however, there is no mention of how to deal with devices when they don't get that far.
C:\> dsregcmd /join /debug
dsregcmd::wmain logging initialized.
DsrCmdJoinHelper::Join: ClientRequestId: XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXXDsrCmdAccountMgr::IsDomainControllerAvailable: DsGetDcName No domain controller is available for the specified domain or the domain does not exist: 0x8007054b.
PreJoinChecks Complete.
preCheckResult: DoNotJoin
deviceKeysHealthy: undefined
isJoined: undefined
isDcAvailable: NO
isSystem: YES
keyProvider: undefined
keyContainer: undefined
dsrInstance: undefined
elapsedSeconds: 0
resultCode: 0x1
The device can NOT be joined because a domain controller could not be located.

None of the error messages you've documented relate to my issue.
TIA

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

ID: b9f81d74-8dcb-e644-9647-0d1d0817f975
Version Independent ID: e249cc9d-2120-8868-dbca-bc3b85f38b4b
Content: Troubleshooting hybrid Azure Active Directory joined devices
Content Source: articles/active-directory/devices/troubleshoot-hybrid-join-windows-current.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Looks like your device cant see a Domain Controller on-premises based on the message in your post.

Hi @MicrosoftGuyJFlo,
Thanks for your response.
As mentioned in #48791, I believe I haven't grasped the concepts between "Hybrid Azure AD Join" and "Azure AD Join". Either, I am stupid, along with lots of other people, or the documentation doesn't make the differences clear.
I even went through the whole WHfB selection spreadsheet, aiming for a hybrid, modern manged domain, which resulted in a "Hybrid Azure AD Join" scenario, which lead me to believe all devices would be part of the "Hybrid Azure AD Join".
My understand was that "Hybrid Azure AD Join" and "Azure AD Join" are mutually exclusive. This appears not to be the case.
Many thanks.

Details on device join differences can be found in the What is device identity? overview article at the top of the Table of Contents on the left side of the article. Feel free to reach out if you would like to chat further my email is in my profile.

#please-close


Graceful degradation.

I am frequently browsing documentation in links or lynx, because our target system does not have anything better.
Please, make sure that the website gracefully degrades when the client does not support all the features of the most recent Chrome.
Example:
Old website

Useful and usable, 5 pages of pure content, and I can ever get a full sitemap for free, because the menu automatically expands.
New website

The good thing about the new website is that in links  it has a navigation menu at the top, which is just very user-friendly, and an excellent feature. However, I don't users hardly need two copies of that menu.

The rest I wouldn't consider very helpful.

The front page is not really speaking about what boost is, not nearly as much helping the user to grasp most of the boost (as an organization, and as a set of libraries) logic from just a single page. (And browsing and clicks are very expensive for human brain.)
It is also not giving a direct download link to the most recent release.
I also don't see links to RSS, which is still a useful news source, even though newer standards exist. (I would ask to consider equipping the website with hEntry Semantic Web tags, but those are not yet widely supported.)
The wonderful site map is entirely missing.

And with all that, the pages are more or less equal in size:
-rw-r--r--   1 lockywolf users  26K May 12 09:40 new-boost.txt
-rw-r--r--   1 lockywolf users  27K May 12 09:40 old-boost.txt

For reference, this is how the websites look in netsurf.

Netsurf is not the best browser out there, but it is much more portable than either firefox or chromium, which is why is works on outdated and specialised platforms.
It actually deals with the front page quite okay, but the amount of white space is still disheartening. Please consider increasing information density on the first screen.

The front page is a placeholder which will be entirely redone and I agree it is not conveying the right message(s).


feat(home): home base

☆*~ﾟ⌒(‘-‘*)⌒ﾟ~*☆
fixes https://github.com/robinjoseph08/pokedextracker.com/issues/15 and https://github.com/robinjoseph08/pokedextracker.com/issues/37

the image is just a placeholder for now, i'd like to edit it later and maybe add ~ cool animations ~
pretend the navbar isn't there for the homepage view

also, does this fix #37? if im not logged in and im viewing your dex, shouldn't i see login/register buttons in the top right? or is the only way i can login/register is by going to the homepage?

@robinjoseph08 yep, it does fix that! and i fixed the flexbox stuff. updated, take another look :)

@robinjoseph08 oh sry i misunderstood the question. i think that part would come into whatever we decide to do for the "view-only" page. i'll add an issue for that

looks like the text is cutting it a bit close on mobile. do you think it needs a bit more padding? especially compared to the login/register pages:

it looks like mobile dex view is still a bit wonky from the nav restructuring? it might be outta scope of this PR though:

i'm not gonna be able to give it much more padding without the vertical height getting to large :/ yea it's a struggle
yea, i'm planning on fixing all that stuff when i redo the hover/info UX

gotcha. curse you, mobile. 凸ಠ益ಠ)凸
LGTM!! :shipit:

lol I DON'T GET IT


infer_schema TypeError: '<' not supported between instances of 'str' and 'int'

When pandas reads a column that contains a 'mix' of integers and strings its dtype will be object. But the entries themselves will be cast to int/string individually. So infer_schema falls over at np.unique to a type error.
In line 113 of infer_schema.py, changing to
col_stat['sample_num_uni'] = len(np.unique(sample_data).astype(str))
might work?
Thanks

Hi @SauceCat !
Chrisgchon get pretty close to the answer. I managed to solve the issue changing line 113 of infer_schema.py to:
col_stat['sample_num_uni'] = len(np.unique(sample_data.astype(str)))

Thanks! Sorry I currently couldn’t arrange any spare time on this project.
It would be great if you create a PR and help other people :)
On Tue, 14 May 2019 at 11:57 PM, Daniel Micoski<EMAIL_ADDRESS>wrote:

Hi @SauceCat https://github.com/SauceCat !
Chrisgchon get pretty close to the answer. I managed to solve the issue
changing line 113 of infer_schema.py to:
col_stat['sample_num_uni'] = len(np.unique(sample_data.astype(str)))
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SauceCat/pydqc/issues/8?email_source=notifications&email_token=ADXNPFGNTRQJH6UE7ZRSQXLPVLOP7A5CNFSM4FGJWTO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVL6LWI#issuecomment-492299737,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADXNPFFFONIP2D2QXGD7EGLPVLOP7ANCNFSM4FGJWTOQ
.


Implement Raids/Waves and Raiders.

This implements 1.14's Raid and Wave in the API and adds mixins to implement all the raiders plus almost all their logic. Sadly Raid.Status led to the end of a no AT api8 (Hopefully Mojang makes it public in 1.16 so we can have no ATs).
The Spellcasters (Evoker and Illusioner) will require a bit more care in the future due to the complexity required to the the casting time of the spells from the goalSelector.
Vindicators and Witches are Raiders in 1.14 and have been adapted in their mixins so they are seen as raiders by the api now.

You'll also need to rebase your API commit to the latest api-8.


hideAxis is recently broken

In version 2.2.0, the bug fix in the brushFor function had the side effect of partially breaking hideAxis.  The function works when data is first added to the chart, but if called individually there is an error. The following, taken from the brushing.html demo, demonstrates the issue:
d3.csv('data/cars.csv').then(function(data) {
            parcoords
                .data(data)
                .hideAxis(["name"])
                .composite("darker")
                .render()
                .shadows()
                .reorderable()
                .brushMode("1D-axes");  // enable brushing

            // next line is the problem (added for demonstration)
            parcoords.hideAxis(['year']).render().updateAxes(0);

        });

This gives:
Uncaught (in promise) TypeError: Cannot read property 'yscale' of undefined
    at convertBrushArguments (brushFor.js:29)
    at SVGGElement.<anonymous> (brushFor.js:63)
    at Dispatch.apply (dispatch.js:61)
    at customEvent (on.js:103)
    at Emitter.emit (brush.js:286)
    at Emitter.brush (brush.js:278)
    at SVGGElement.<anonymous> (brush.js:222)
    at Selection.selection_each [as each] (each.js:5)
    at brush.move (brush.js:212)
    at Selection.selection_call [as call] (call.js:4)

In versions prior to 2.2.0, this was not an issue. Now, it seems that the axis being referenced in brushFor.js here
const convertBrushArguments = args => {
    const args_array = Array.prototype.slice.call(args);
    const axis = args_array[0];
    // ordinal scales do not have invert
    const yscale = config.dimensions[axis].yscale;

has already been removed from config.dimensions in sideEffects.js here
.on('hideAxis', d => {
      pc.dimensions(pc.applyDimensionDefaults());
      pc.dimensions(without(config.dimensions, d.value));  // this line
    })

thus, config.dimensions[axis] is undefined.
I would attempt to fix this myself, but I'm not really sure why it's happening. @timelyportfolio  do you have any thoughts on this? Maybe we could delay execution of pc.dimensions(without(config.dimensions, d.value)) until after brushFor has been executed?

@timelyportfolio thank you for the fast response, so glad you were able to figure this out so quickly!


Allow content to shrink in thinner viewports

What does the padding do?

It keeps a little space on the edges when the viewport is less than the max width.

Merge conflict on this now.

As of 2eb6cf9890bacb600f2988e940e5c8b3b433655e, we now set a viewport which should improve the mobile experience. I dislike the wrapping that happens when the main table is too thin. Thank you for the contribution, though.


Add version to UI-SDK

Created by: @bayegan Oct 01 2021 at 12:58 CET
Last updated: Jul 18 2022 at 11:20 CET
ClickUp ID: IK-1598
Background:
The GO SDK adds a header so we can catch on the server side to see which SDK versions are still used. Would be nice if other SDKs would do the same
https://github.com/indykite/jarvis-proto/blob/develop/sdk/identity-sdk-go/identity/identity_client.go#L50
Acceptance criteria:
Add to header in Node SDK and UI SDK
Attachment(s):
https://github.com/indykite/clickup-attachments/blob/master/8b8817ae-bf18-47fc-a990-1e007822fa0d.png

Commented by: @hanny21 Oct 05 2021 at 15:33 CET
tested locally that the version name and action name are added in the ui sdk to request headers.
Attachment(s):
https://github.com/indykite/clickup-attachments/blob/master/8b8817ae-bf18-47fc-a990-1e007822fa0d.png

Commented by: @maprokes Oct 06 2021 at 09:28 CET
@bayegan @dusan-kovacik-profiq As Dusan delivered the part of the fix (UISDK without unit tests) in current sprint I copied this ticket also to the next sprint 1kp4uy9


Safari 13.1/iOS 13.4 is latest

The website still shows Safari 13 and Safari iOS 13.3 as latest, but Safari 13.1 and iOS 13.4 have been released.

Thanks, updated!


Small fixes to widgets

Related to: https://github.com/CartoDB/cartodb/issues/13579, https://github.com/CartoDB/cartodb/issues/12918 and https://github.com/CartoDB/support/issues/913
Description
This PR fixes multiple things related to widgets:

Category widget not using collapsed state when refreshing
Formula widget showing No data available when data was 0 (js falsy values <3) (https://github.com/CartoDB/cartodb/issues/13579)
Widgets not being collapsed In responsive mode (https://github.com/CartoDB/cartodb/issues/12918)
Formula widget showing value instead of title when collapsed (https://github.com/CartoDB/support/issues/913)
Category widget showing stats when collapsed

Screenshots

LGTM!
BUT can you test if this is working on your side? I found very difficult to get to that popup.

Mmmm, maybe that's why it wasn't collapsing in responsive 😂

Just talked to @piensaenpixel.
We're always showing the widget in small viewports and hiding the Toggle button.

@rubenmoya what does it mean?

That means the widgets will always be shown in responsive mode (no toggle), and the toggle button will be hidden.

OK, can I test that as well?

Not yet, let me fix it and deploy it again

:boom: LGTM! Amazing job mate!!!


Incorrect sort order

The order of parsed accessibility markers is in the wrong order. See screenshot, where the section headings are parsed as being at the bottom of the view. This is a SwiftUI view with no animations. Is there a workaround for this?
The order is important to me as I use the accessibility markers to create a markdown representation of the view.

Hi @yonaskolb, thanks for reporting this! Can you provide a code sample that reproduces this issue? Additionally, what iOS version are you testing on?

Thanks for the quick reply @NickEntin. Example project showing the issue attached. I'm using iOS 16.4 in Xcode 14.3
SnapshotExample.zip

The example also highlights a different issue with the rendering of forms, which there is an unresolved issue in SnapshotTesting for https://github.com/pointfreeco/swift-snapshot-testing/issues/667

Any updates on this one?

I haven't had a chance to test on Xcode 14.3 yet, but on Xcode 13.3 it seems to be behaving correctly. I'll dig in further this week.

Still working on this one. There was definitely a change in iOS 16 that we're not currently handling properly. Looking at sample view you provided, in iOS 15 it's backed by a table view and in iOS 16 it's backed by a collection view.

It looks like with collection views it's using an accessibility container with a single element view (that's not an accessibility element) which then has subviews that are themselves accessibility containers each with a single element. Somewhere in this nesting we don't order things the same as VoiceOver.

Fixed by https://github.com/cashapp/AccessibilitySnapshot/pull/169


can't make Login in with Google (Redirect) work.

Thank you for the great package. This is rather a question than an issue. Everything works great when I run the starter example on my Mac except the Log in with Google (Redirect). Just can't make it work.
Can anyone give me an advice? getRedirectResult() returns null. What am I doing wrong?

found the following doc.
https://cloud.google.com/identity-platform/docs/web/redirect-best-practices
closing this issue now. Thx!


Upgrading mesos-actor

Prefer offers with least cpus
Description
The earlier implementation was picking up the first available offer that satisfies the requirements/constraints. Now the offers with least cpus are preferred so that scale downs can happen easily.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3990      +/-   ##
==========================================
- Coverage   85.81%   81.14%   -4.68%
==========================================
  Files         146      146
  Lines        7084     7084
  Branches      440      440
==========================================
- Hits         6079     5748     -331
- Misses       1005     1336     +331

Impacted Files
Coverage Δ

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.1%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-81.82%)
:arrow_down:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
0% <0%> (-58.83%)
:arrow_down:

...la/whisk/core/database/cosmosdb/CosmosDBUtil.scala
92% <0%> (-4%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @Himavanth !


Extract and store the OLTPbench results for nightly build

Fixes #937
Extract and store the OLTPbench results for nightly build

This PR is still in progress.
TODO:
Store script's parameters and arguments for each job
Discuss with @jazzplato about the OLTP_DIR_TEST_RESULT. It is better to only have one folder to store all results.

this PR needs some changes. I'm going to create a new one.


Globally configurable results limiters

Add ability to globally define different result limiters, e.g.
RegexpExamples::Config.max_repeater_variance = 5
RegexpExamples::Config.max_group_results = 10
RegexpExamples::Config.max_results_limit = 20000

Coverage remained the same at 100.0% when pulling c7dc80aef3c7423526f3c6ba101b8a90f69e0639 on globally_configurable_results_limiters into 7ae52785a6f5c9b266759d33881d9de409b50551 on master.

Coverage remained the same at 100.0% when pulling e7e4007d07002ea056ad003e28700963f8cf97d7 on globally_configurable_results_limiters into 7ae52785a6f5c9b266759d33881d9de409b50551 on master.

Coverage remained the same at 100.0% when pulling e7e4007d07002ea056ad003e28700963f8cf97d7 on globally_configurable_results_limiters into 7ae52785a6f5c9b266759d33881d9de409b50551 on master.

Coverage remained the same at 100.0% when pulling 67b1747ce0b22dbff586426c161c27a2d094523d on globally_configurable_results_limiters into 7ae52785a6f5c9b266759d33881d9de409b50551 on master.


How to position title in middle of doughnut?

Greetings!
I need to show some data right in the middle of doughnut (responsive-way). Is there a way to achieve it?

@Hedva thanks for your reply!
If i set legend display to true, custom text renders higher than center. I guess it's because legend renders at top and it's needs to do some maths with Chart.legend.height inside 'drawTotals' function, no question here.
At codepen everything works fine but local i have an error:

Uncaught TypeError: Cannot read property 'display' of undefined

At this string inside 'Chart.Chart.pluginService.register' in 'beforeDraw' function:
if (chart.config.centerText.display !== null &&
How to deal with it?

Yes, it takes the height of the canvas to determine the center of the chart, not the chart itself.
Maybe there is a way to figure out the height of the chart instead of the canvas.


Thank you & Symfony Slack

Thank you for writing this bundle - it works really well and I'm enjoying migrating my Symfony sites to Vite 😄
I wanted to let you know I've created a #vitejs channel on the Symfony Slack as there's starting to be regular questions about using Vite. I'll try and help people if they get stuck, and you'd be very welcome if you want to join Slack.

Great idea, creating this channel !
Have a good day !


Some documents not ready?

I wonder if how-to-write-an-ad-plugin.md is ready?

Hmm. yep.. It's not ready. But you can take a look at this https://github.com/Chimeejs/chimee-plugin-popup/blob/master/README.md


raw_logging.h depends on type definition in logging.h.

Resolve https://github.com/google/glog/issues/712

Thanks! Looks good, although I'll defer approval to Sergiu since my area is technically the Bazel support.


@arghadipmanna101 Sir, i have solved issue regarding profile page plz look at it once.

if it is alright then merge pull request under ggsoc24 label

@arghadipmanna101 plz assign label in all my PRs as it won't count in my contributions in gssco24
Postive regards

I want to work more as contributor as it motivated me when u assign the task but plz label it in both Pr and issues there are total 4-5commits I have done so plz look at it once and level them


Change descriptions for ups broker

To make it easier to differentiate between the service and plan description.

@simonleung8 has approved these changes, although he can't add labels. I am adding an LGTM3 label on his behalf. Also, it looks like the travis build failure was a flake. Merging


i cant't use connect

Expected Behavior

i have this problem please help me think you very much
Actual Behavior
Screenshots and error messages are helpful.
Steps to Reproduce the Problem

Specifications
Server

OS (e.g., windows10):
Java (e.g., Oracle JDK 8u151):
Servlet container (e.g., Apache Tomcat 8.5.42):
webSpoon (e.g., version: <IP_ADDRESS> or commitId: 0211d84b):

Client

OS (e.g., windows10):
Browser (e.g., Google Chrome 64.0.3282.140):

It looks like the reverse proxy is not working.
Please let me know how you deployed webSpoon.

Did you use the Docker version?
If you did not use the Docker version, did you follow the instructions at https://github.com/HiromuHota/pentaho-kettle#without-docker?

Thank you for having solved the Thank God project.


dimports

Try dimport to remove the amd packaging
Get hackernoon style blog for everyone in our group
Start a web ring via tag that aggregates remote site query data

https://github.com/lukeed/dimport

we have a new import / appending routine and this is no longer necessary (tho still cool :) )


Version number is not consistent

On Github master branch the version is 4.1.5
On Rubygem the version is 4.1.6 (https://rubygems.org/gems/permanent_records/)

Apologies for this! I left unpushed commits on my laptop. This is a classic example of not automating my workflow and I've fixed it with a new Rake task. It won't happen again. Thanks for pointing this out!
There's now a version 4.1.7 that has everything on master (and vice versa).


Error Adding Character In Psych Engine

What is your question?
How Can Fix This Error: C:\hostedtoolcache\windows\haxe\4.2.5\x64\std/haxe/xml/Access.hx (line 44)
flixel/graphics/frames/FlxAtlasFrames.hx (line 256)
Character.hx (line 164)
editors/CharacterEditorState.hx (line 925)
editors/CharacterEditorState.hx (line 434)
FlxUIDropDownMenuCustom.hx (line 500)
FlxUIDropDownMenuCustom.hx (line 361)
flixel/ui/FlxButton.hx (line 521)
flixel/addons/ui/FlxUITypedButton.hx (line 1172)
flixel/addons/ui/FlxUIButton.hx (line 362)
flixel/ui/FlxButton.hx (line 507)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)
Uncaught Error: SubTexture is missing attribute width
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

Crash Handler written by: sqirra-rng

I Cant Add Characters Or Load New Character Images

my guy

If you manually edited the .XML or used an external program that makes one outside of Animate, I'm not responsible for this issue and so you should close it.


Use log2 y-axis for SYPD scaling plot

Purpose
Use log10 y axis for SYPD scaling plot

[x] I have read and checked the items on the review checklist.

bors r+

bors r+


Remove usage of legacy url API .query

We should switch to .search which is part of the node api, see https://nodejs.org/docs/latest-v14.x/api/url.html which has more details.
Impacted code
https://github.com/opensearch-project/security-dashboards-plugin/blob/fc7b8cc994046c830d950dc57742e6f1ac542b7b/server/auth/types/basic/routes.ts#L173-L174

[Triage] We should also look into url.path that is used right above this location in the file.

Removing the label for 2.0.0 as the PR hasn't been ready yet.


is this code only running in the CUDA 11.8, can this code running in the 12.4

My cuda's version is 12.4, the bug appears when I try to install 'causal-conv1d'. it seems this code is only woking based on the cuda is 11.8. Dose anyone use 12.4 cuda running this code properly?

I have same problem.

I have same problem.

Actually, I have installed 'causal-conv1d' and 'mamba' based on the cuda 12.4. But I am not sure whether the program will working or not.

@poult-lab
I found that just running the test code of mamba verifies If CUDA is being used or not
import torch
from mamba_ssm import Mamba

batch, length, dim = 2, 64, 16
x = torch.randn(batch, length, dim).to("cuda")
model = Mamba(
    # This module uses roughly 3 * expand * d_model^2 parameters
    d_model=dim, # Model dimension d_model
    d_state=16,  # SSM state expansion factor
    d_conv=4,    # Local convolution width
    expand=2,    # Block expansion factor
).to("cuda")
y = model(x)
assert y.shape == x.shape

if you don't have any errors, then it installed successfully

@poult-lab I found that just running the test code of mamba verifies If CUDA is being used or not
import torch
from mamba_ssm import Mamba

batch, length, dim = 2, 64, 16
x = torch.randn(batch, length, dim).to("cuda")
model = Mamba(
    # This module uses roughly 3 * expand * d_model^2 parameters
    d_model=dim, # Model dimension d_model
    d_state=16,  # SSM state expansion factor
    d_conv=4,    # Local convolution width
    expand=2,    # Block expansion factor
).to("cuda")
y = model(x)
assert y.shape == x.shape

if you don't have any errors, then it installed successfully

Thanks bro, it seems the vim is not working based on the CUDA 12 at all. I have to find a way to shift to the CUDA 11.8.

Is it working only based on the CUDA 11.8?how about CUDA11.7?

Is it working only based on the CUDA 11.8?how about CUDA11.7?

it seems okay, CUDA 11.7

how about CUDA 11.6？I can not find a causal-conv1d  based on CUDA 11.6.

how about CUDA 11.6？I can not find a causal-conv1d based on CUDA 11.6.

probably you have to upgrade your CUDA's version.


Disable Incrementally updates Localizable.strings

Hi,
I don't want incrementally updates localizable.string but this code return "Command /bin/sh failed with exit code 78"
if which bartycrouch > /dev/null; then     bartycrouch interfaces -p "$PROJECT_DIR" -b else     echo "warning: BartyCrouch not installed, download it from https://github.com/Flinesoft/BartyCrouch" fi
What's wrong ?
Thanks in advance :D

I can't really tell without an example project to test this against. But be sure to check if you have the same issue as explained in https://github.com/Flinesoft/BartyCrouch/issues/36. Also your title isn't describing your problem, I'm renaming it to something more suitable.

Closing this, as no answer provided for long time to clarify the issue.


feat: add HEALTHCHECK on Dockerfile

Feature Request
Describe the Feature Request

It would be great to put HEALTHCHECK on our Dockerfile to check if the deployed service is healthy
Describe Preferred Solution
Install curl and add a HEALTHCHECK for the RPC in this file
HEALTHCHECK --interval=3s --timeout=5s --start-period=1s --retries=5 \
  CMD curl --request POST \
    --header "Content-Type: application/json" \
    --data '{"jsonrpc": "2.0", "method": "eth_chainId", "id": 1}' http://${KAKAROT_HTTP_RPC_ADDRESS} || exit 1

Resolved by #337


Feature request: Simplified logger formatter when in AWS SAM Local

Use case
I use aws_lambda_powertools.Logger which works very nicely when looking at logs in Cloudwatch.
Nevertheless, when testing/debugging lambda functions using AWS SAM local invoke, I find the json representation difficult to read and cluttering the console too much.
I know I can swap the logger, or use a custom formatter. But would it not be meaningful to include a simple formatter (instead of the LambdaPowertoolsFormatter that prints out jsons) and use it automatically when not really in Lambda (but e.g. in AWS SAM local invoke) ?
Related issue: #409
Note:

Strictly speaking this request might breach your tenet 1 ("AWS Lambda only"), but it depends where you place AWS SAM local invoke or development & testing of Lambda functions outside Lambda

Solution/User Experience

Add a "simple" formatter that would print a single line, without serializing to json

optionally with a custom format

With that, use the existing json formatter only when really in Lambda

or alternatively, use the simple one when in AWS SAM Local (and similar, if it can be detected).
or make it easy to swap, by e.g. an envvar

Alternative solutions
There exists a solution already and that is to use a custom formatter.

I opened the issue to consider building this in thinking that it might be beneficial for more users developing and testing Lambda functions.

Acknowledgment

[X] This feature request meets Lambda Powertools Tenets
[ ] Should this be considered in other Lambda Powertools languages? i.e. Java, TypeScript

Hey @pmarko1711, thanks a lot for taking the time to help us improve! Would you have the output handy to make it easier to reason the next steps?
I need to spin up a project to test. I can't remember if it's an one line JSON (cost optimization), or indented -- would it be more helpful to have indented JSON when running locally if that's not the case yet?
My initial concern: transparently swapping for a no JSON output could lead to incorrect assumptions how the final output would be when deployed.
Look forward to hearing from you

Example
Demonstrating how it could look like without any code change.
Before
sam local invoke
Invoking app.lambda_handler (python3.9)
Skip pulling image and use local one: public.ecr.aws/sam/emulation-python3.9:rapid-1.56.1-x86_64.

Mounting /Users/lessa/DEV/pt-issue-1527/.aws-sam/build/HelloWorldFunction as /var/task:ro,delegated inside runtime container
START RequestId: ea90d758-5a78-4777-9c46-2017b472fc08 Version: $LATEST
{"level": "INFO", "location": "lambda_handler:39", "message": "This should appear indented and not compacted into a single line...", "timestamp": "2022-09-26 09:03:38,349+0000", "service": "issue-1527"}
END RequestId: ea90d758-5a78-4777-9c46-2017b472fc08
{"statusCode": 200, "body": "{\"message\": \"hello world\"}"}REPORT RequestId: ea90d758-5a78-4777-9c46-2017b472fc08     Init Duration: 1.39 ms  Duration: 1141.96 msBilled Duration: 1142 ms Memory Size: 128 MB     Max Memory Used: 128 MB

After
> sam local invoke
Invoking app.lambda_handler (python3.9)
Skip pulling image and use local one: public.ecr.aws/sam/emulation-python3.9:rapid-1.56.1-x86_64.

Mounting /Users/lessa/DEV/pt-issue-1527/.aws-sam/build/HelloWorldFunction as /var/task:ro,delegated inside runtime container
START RequestId: 9d9eeaaa-6a1f-4355-ac3a-dc20da33f1c6 Version: $LATEST
{
    "level": "INFO",
    "location": "lambda_handler:39",
    "message": "This should appear indented and not compacted into a single line...",
    "timestamp": "2022-09-26 09:02:05,262+0000",
    "service": "issue-1527"
}
END RequestId: 9d9eeaaa-6a1f-4355-ac3a-dc20da33f1c6
REPORT RequestId: 9d9eeaaa-6a1f-4355-ac3a-dc20da33f1c6  Init Duration: 1.04 ms  Duration: 1120.49 ms    Billed Duration: 1121 ms        Memory Size: 128 MB     Max Memory Used: 128 MB
{"statusCode": 200, "body": "{\"message\": \"hello world\"}"}%

Thanks for coming back to be, @heitorlessa .
Right now the output is in this format:
{"level":"INFO","location":"functionB:615","message":"Soemthing happening here","timestamp":"2022-09-26 09:16:26,735+0000","service":"ABC"}
{"level":"INFO","location":"send_message_to_queue:563","message":"{'MessageAttributes': {'somekey1': {'StringValue': 'ABC', 'DataType': 'String'}}, 'MessageBody': '{\"somekey2\": \"someval2\", \"somekey3\": \"true\"}'}","timestamp":"2022-09-26 09:16:26,733+0000","service":"ABC"}
{"level":"INFO","location":"lambda_handler:98","message":"Something else happening here","timestamp":"2022-09-26 09:16:26,735+0000","service":"ABC"}
{"level":"INFO","location":"_functionB:185","message":"Another thing here","timestamp":"2022-09-26 09:16:26,735+0000","service":"ABC"}
{"level":"INFO","location":"_functionC:401","message":"And finally, this happened","timestamp":"2022-09-26 09:16:26,736+0000","service":"ABC"}

In here it looks quite OK, but imagine these messages to be quite longer and auto-wrapped into multiple rows. It's then not that easy to find and read the actual messages while debugging.
When I submitted the ticket I thought of simply outputting (when in AWS SAM Local) something more condensed, where message would be easier to find (since it'd be normally the last item in each row), e.g.:
2022-09-26 09:16:26,735+0000|INFO|ABC|functionB:615|Soemthing happening here
2022-09-26 09:16:26,733+0000|DEBUG|ABC|send_messages_to_queue:563|{'MessageAttributes': {'somekey1': {'StringValue': 'ABC', 'DataType': 'String'}}, 'MessageBody': '{\"somekey2\": \"someval2\", \"somekey3\": \"true\"}'}
2022-09-26 09:16:26,735+0000|WARNING|ABC|lambda_handler:98|Something else happening here
2022-09-26 09:16:26,735+0000|INFO|ABC|_functionB:185|Another thing here
2022-09-26 09:16:26,736+0000|INFO|ABC|_functionC:401|And finally, this happened

For now I've achieved this by initializing a logger with a custom formatter that conditionally overwrote the inherited serialize() method. So in Lambda I get the jsons, but in AWS SAM Local only a more plain output. I guess that's a bit dirty, but still, see (*) below.  I thought of embedding something similar into the LambdaPowertoolsFormatter to  do this either automatically or on request. If one went this route, it'd be nice if there was a default, concise format that one could override (something along format parameter of Python's logging).
Nevertheless, your idea of indenting JSON outputs could be sufficient here. Maybe making use of pprint then? Wouldn't however having multi-line messages cause more issues elsewhere? (it seems a bit non-standard)
(*)

import os
from aws_lambda_powertools.logging.formatter import LambdaPowertoolsFormatter

AWS_SAM_LOCAL = os.getenv("AWS_SAM_LOCAL", "false").lower() == "true"

class MyCustomFormatter(LambdaPowertoolsFormatter):
    def serialize(self, log: dict) -> str:
        """Use serialize"""
        if AWS_SAM_LOCAL:
            # https://awslabs.github.io/aws-lambda-powertools-python/latest/core/logger/#standard-structured-keys
            prefix = "|".join(
                [
                    log[k]
                    for k in ["timestamp", "level", "service", "location"]
                    if k in log
                ]
            )
            suffix = f"\n{log['exception']}" if "exception" in log else ""
            return "{}: {}{}".format(prefix, log.get("message", ""), suffix)
        else:
            return super().serialize(log)

logger = Logger(service="ABC", logger_formatter=MyCustomFormatter())

On Mobile so excuse formatting and briefness

For JSON multi-line, stdlib "json" has an indent param which is what I used for this POC -  it's a no brainer to enable it for local dev ;-)
pprint however could skew results as you have nested objects (plus multiple conversions).
My main worry about plain text is setting the expectations to customers who would see different results in CloudWatch Logs (or elsewhere) -- we have a distinct set of customers persona (dev, Data engineers, Ops, InfoSec) so we always take that into account.
If JSON multi-line suffices for you, we could make this available in the next release!
I could either guide you where to make changes (formatter.py) so we can credit you in the release notes, or if you don't have the bandwidth I can make it later this week.
Once I get my laptop tomorrow,
I'd like to dig on your good comment on "format" method - I'm all for making it easier for customers... and we use standard logging so that's a plus

Thanks. Ok, and happy to do add it myself so that I get familiar with your contribute process (and can contribute with sthg else in the future)

should it get activated automatically (looking at AWS_SAM_LOCAL) or via a parameter / or a new envvar?

Awesome! Let’s use AWS_SAM_LOCAL - if it’s present use a different
indentation (4) otherwise None
On Tue, 27 Sep 2022 at 18:17, Peter Marko @.***> wrote:

should it get activated automatically (looking at AWS_SAM_LOCAL) or via a
parameter / or a new envvar?
—
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/issues/1527#issuecomment-1259735575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBCNHPNLFO577H3IQDTWAMMY5ANCNFSM6AAAAAAQRFESSI
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Bug?] Cluster DNS on Windows

Hi,
Not sure this is even an ACS issue but it seems the best place to create this issue. When I deploy a cluster with the ACS engine all works fine. However it seems that windows pods end up with two network interfaces (inside the container)
If I understand correctly the 172.x one is the default docker NAT interface and the other one is the kubernetes transparent network. Is this even correct ?
Both of them get the <IP_ADDRESS> (my cluster DNS) but sometimes (50%) the <IP_ADDRESS> takes precedence above the the cluster DNS. This is undesirable because there's no DNS server listening on <IP_ADDRESS> or at least all kubernetes services are not resolvable.
Where can I configure docker/kubernetes to not use the DNS server other then the kubernetes one?

    : No
   DNS Suffix Search List. . . . . . : hnzbguap32ne3onlsis2rg1plh.ax.internal.cloudapp.net

Ethernet adapter vEthernet (Container NIC dff8d241):

   Connection-specific DNS Suffix  .   : No
   Autoconfiguration Enabled . . . .     : No
   Autoconfiguration Enabled . . . .  : Disabled

cc: @JiangtianLi / @anhowe
This is by design. The two NICs are intentional. One enables pod-to-pod network and the other serves as a gateway to the Internet.

Where is this at @JiangtianLi ? Can we update this or close it, etc?

@colemickens I'm still having troubles with this. If there's something I can do to troubleshoot, please let me know.

@JiangtianLi can you please provide an update

cc: @anhowe

@skinny We are working on a solution to set the correct DNS server and make the order of NICs deterministic.

@JiangtianLi has this work finished?

@JackQuincy Yes, the fix has gone into v1.5.3.

closed as the fix has gone it.


jiva volume upgrade script sed fails

Description
I'm trying to upgrade from 0.9.0 to 1.0.0 following the instructions.  I cloned the repo and get to the ./jiva_volume_upgrade.sh step which fails due to sed option -u.  Seems like an obvious issue with the sed, but I can't find what the -u switch is supposed to do in the man.  I'm on MacOS, but didn't see anything different.
Expected Behavior
Expected to upgrade jiva volume.

Upgrading Replica Deployment to 1.0.0
error: must specify -p to patch
Failed to patch the deployment pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032-rep | Exit code: 1
Clearing temporary files
Your Environment
helm list | grep openebs
openebs             	6       	Mon Jul  1 21:54:36 2019	DEPLOYED	openebs-1.0.0             	1.0.0                	openebs
kubectl get pv
pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032   5Gi        RWO            Delete           Bound    default/xxx       openebs-jiva-default            31h

Thanks for the issue. The fix is merged #2654 . Please verify the changes.

The upgrade works now, but I think another similar issue with the verify script called within.
"pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032-rep-5758c8db5f" deleted
Upgrading target Deployment to 1.0.0
deployment.extensions/pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032-ctrl patched
replicaset.extensions "pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032-ctrl-569b6dd995" deleted
Upgrading target service to 1.0.0
service/pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032-ctrl-svc patched
Upgrade steps are done on volume pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032
verifying jiva pvc-6949cf81-9b6c-11e9-aaa1-000c297ba032 volume upgrade
Failed validation for controller pod not running
Failed validation for replica pods not running
usage: grep [-abcDEFGHhIiJLlmnOoqRSsUVvwxZ] [-A num] [-B num] [-C[num]]
[-e pattern] [-f file] [--binary-files=value] [--color=when]
[--context[=num]] [--directories=action] [--label] [--line-buffered]
[--null] [pattern] [file ...]
usage: grep [-abcDEFGHhIiJLlmnOoqRSsUVvwxZ] [-A num] [-B num] [-C[num]]
[-e pattern] [-f file] [--binary-files=value] [--color=when]
[--context[=num]] [--directories=action] [--label] [--line-buffered]
[--null] [pattern] [file ...]
usage: grep [-abcDEFGHhIiJLlmnOoqRSsUVvwxZ] [-A num] [-B num] [-C[num]]
[-e pattern] [-f file] [--binary-files=value] [--color=when]
[--context[=num]] [--directories=action] [--label] [--line-buffered]
[--null] [pattern] [file ...]
it must be on a while loop because it keeps spitting out the above usage.

the -P option for grep is not supported by the default grep on mac os x:
○ grep --version
grep (BSD grep) 2.5.1-FreeBSD

Replacing -P with -E works.
Also, it would be better if the scripts have:
#!/usr/bin/env bash

iso
#!/bin/bash

/bin/bash on mac os x is old:
○ /bin/bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin18)
Copyright (C) 2007 Free Software Foundation, Inc.

It doesn't support e.g. declare -A, which is used by pre-upgrade.sh. Changing the shebang allows to install a more recent bash (e.g. with homebrew) and have the scripts use it without modifications.

On mac, I also run into the issue that wc -l prints spaces in front of the number:
running_ctrl_pod_count=$(kubectl get pods -n "$ns" \
-l openebs.io/controller=jiva-controller,openebs.io/persistent-volume="$pv" \
--no-headers | wc -l)

results in leading spaces in running_ctrl_pod_count:
○ echo "pods: $running_ctrl_pod_count"
pods:        1

which causes the comparison with 1 to fail.

Thanks for reporting this. The fixes are available in master #2663  . Please verify the changes.


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 1202 ms

Resolved: Bofrost.fr performance has improved in 428958a after 32 minutes.


Create Chinese Lang File

This is an awsome mod! Thank you to bring it to us!
Here are the langurage file for Chinese.
And I've found that the primal peal in the TB has no language string in en_us file.
Then I wonder if I can post this mod to MCBBS which is the biggest minecraft forum in china.Of course I'll put the author and the curse link in the post.

Thanks a lot for the translation!
Sure, you can put the mod on whatever website you want it to be as long as you keep credits!


fix: remove the comma from const path variable

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closes  #27083

Hi @GursimarSinghChopra
I think the code is valid. See Destructuring

T


Curriculum Activity Button Overflow in Full Screen View

hey @llaske
In the Curriculum Activity  the add Template button (bottom Right corner)  goes out of the box in Full Screen view ,
In normal view it is like this

But in Full Screen View it is like this

Kindly share your thoughts on this. So that i can work on the fix.

Dear @llaske,
Thank you for your feedback on the issue regarding the activity-palette's container in the Calculator activity. I appreciate your taking the time to review the issue and provide your feedback.
If this is considered a rare use case and not a blocking problem, I understand that it may not be a priority to address this issue at this time.
Thank you again for your response