Opcodes do not match - Stereo No Longer Working

Required Info

Camera Model
{D415}

Firmware Version
{<IP_ADDRESS>}

Operating System & Version
{Ubuntu 16.04}

Kernel Version (Linux Only)
{4.13.0-39-generic}

Platform
{upCore}

SDK Version
{ 2.10.4}

Issue Description
Hi All -
After trying a longer cable I get the following message in the rs viewer:
Invalid Value in rs2_get_option_range(options:000002B3641140C0, option:Stereo Baseline, min:0.0001, max:0.01, step:1e-06, def:0.001):
OpCodes do not match! Sent 21 but received -6!
Went back to the original cable (that came with the D415) and still have the same problem. I reinstalled the firmware, same issue; and tried the calibration tool but that fails with "calibration tables on device are not supported". The RGB camera works fine but the stereo looks like it does here #1508. Any way to solve this? Thx
p.s Also tried this on other hardware w/ Ubuntu 16.04, same issue.

[Realsense Customer Engineering Team Comment]
hi @juanBerger,
I think we need to make sure your camera back to normal for the calibration first.
can you please try the calibration tool,https://downloadcenter.intel.com/download/27415/?v=t
with Intel.Realsense.CustomRW.exe -g to recover the calibration table and see if it recovered.
and try calibration again to see if you can calibrate the camera now.

Hi,
Thank you for the response, this appears to have worked. After running
Intel.Realsense.CustomRW.exe -g
I was able to do the calibration. Using the depth quality tool I'm getting an RMS error of about 40mm - is this typical? (measuring a distance of ~3m). Thanks

[Realsense Customer Engineering Team Comment]
hi @juanBerger,
great, 40mm error at 3m is a very good result, so back to your main question, is the issue still there? with long or short cable?
thanks.

Hi - I no longer see the error and both cables work mostly without issue. Frequently though, the camera will appear as a 2.1 device. Simply reversing the orientation of the c side of the plug has been fixing the problem, though. This happens with both cables and on multiple host machines. Thx


not work set align: 'inner'

Expected behavior
not work set align: 'inner' for Y-axes. like that:
scales: {
y: {
ticks: {
align: 'inner',
...
},
},
but for X-axes all work correctly.
i try 4.21 and 4.1.2
Link to  interactive example: https://codepen.io/ishtorman/pen/RwYbeZo
Current behavior
i try 4.2.1 and 4.1.2 and 3.*
Reproducible sample
https://codepen.io/ishtorman/pen/RwYbeZo
Optional extra steps/info to reproduce
No response
Possible solution
No response
Context
No response
chart.js version
v4.2.1
Browser name and version
Chrome
Link to your project
https://codepen.io/ishtorman/pen/RwYbeZo

As described by the documentation it only works on the x axis so this is currently working as designed

I implemented inner align for Y-axes as a trial.
How is this?


Consider prettier

See https://prettier.io/

to do what?  render linked data to users?    relationship to eslint?    This ticket needs a little more context for us to be able to do it.

To pretty the formatting of our code.

See https://www.robinwieruch.de/how-to-use-prettier-vscode

I installed prettier in vs-code (my preferred code editor) and gave it a test drive.   I also already have the eslint extension installed in vs-code.   Some comments:

we currently use eslint as part of the build process, and the default preferred styles of eslint vs prettier are different in places, so if you re-format a file with prettier, you are likely to get eslint complaints (which will show up in the build).  if we were to swtich to prettier, we'd need to look into changing the preferred styles of prettier (and/or get rid of eslint)
eslint does more than enforce formatting, it also detects other issues (like variable declaration problems); prettier does not do this so eliminating eslint in the build process will cause this to be lost
i don't believe this a way to integrate prettier into the build process; it runs as a code editor extension or as a pre-commit hook in github

I see perhaps some of my assumptions about prettier vs elint may not be accurate based on work here?  https://github.com/LD4P/sinopia_api/pull/169/commits/00cd714ec4be8eb2178a565dd424027cfc397e38


New Interface For Chinese Restaurant Website

Description
Chinese Restaurant Website Interface
Fixes:  #793

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
AFTER:

https://deploy-preview-794--cool-front-end-templates.netlify.app/chinese restaurant/

@Abh1shekSingh List the resources from where you have taken the reference.

I my self created this design. Didnt take any reference. This project was already in your repo. I just gave it a new interface

@VidhiBhatt01 @astha2412 @DEEPIKA-SUK Done the Required changes u asked for. @arpit456jain Please Review and merge if possible


JetBrains: duplicate CodyInstalled events

Context https://sourcegraph.slack.com/archives/C059N5FRYG3/p1697568648010619
Here is where we log the CodyInstalled event
https://sourcegraph.com/github.com/sourcegraph/jetbrains/-/blob/src/main/java/com/sourcegraph/telemetry/GraphQlLogger.java?L21
We trigger it on

installation
user changes server
user changes access token

One possible resolution step is to persist which server URL has already received this event so that we never send it again for the same server URL.

@akalia25 Thank you! We are unable to reproduce this locally making it difficult to fix the problem. We are keeping this issue on the board for now but de-prioritizing it since have other items that need more attention. Feel free to bump this if it's an important issue to fix.

Hey @olafurpg, following up here - I originally would have agreed with your assessment that this isn't a top priority, since we're de-duping Cody installs on the analytics side by counting daily unique users who fire this event. But we recently realized that CodyInstalled is firing for the same user across days as well. We're de-duping within days, but not across days (so we're still over-counting installs).
Any chance you can bump this up on the priority list? If not, we can  go ahead and add in some transformations on our end to weed out the duplicates as a band-aid fix, but ideally I'd like to get this fixed in the logging.

I suspect that there are two different issues here.
One is that we're logging CodyInstalled on settings changes (server endpoint and access tokens), which hopefully my PR linked above will fix.
The other is what @akalia25 talks about here:

It is also important to note that this event has been firing multiple times at the exact same timestamp, so it might be possible that the installed event is continuously firing during one of those triggers?

Which is very weird. Clearly users aren't changing their server endpoint multiple times at once. Let's see if this continues after the fix above is merged.

@kelsey-brown @akalia25 are you aware of any existing event names from VS Code that I should replicate in JetBrains for config changes? I've made up new ones there, but want to be consistent if I can CC @Strum355

Here's what I could find in our tables that looked relevant, but it doesn't look like any of these events are actually firing anymore (most haven't fired since November) so if you'd prefer to change the names I think that's fine 👍
CodyVSCodeExtension:codyDeleteAccessToken:clicked
CodyVSCodeExtension:codySetAccessToken:clicked
CodyVSCodeExtension:updateEndpoint:clicked

Thanks! There's always a subtle difference in choosing between the user action ("button clicked") and the underlying effect ("setting changed").
But I agree, the fact that these aren't in use makes me comfortable introducing new event names here.

@akalia25 @kelsey-brown if we continue to see the multiple events with the same timestamp let me know and we can reopen (or open a new issue!)

Hey team - re-opening this thread because unfortunately we think that CodyInstalled is still being logged on non-install events somehow. The reason we think that is because we're still seeing a lot of users firing CodyInstalled repeatedly across multiple days. If you look at our daily cody installs on jetbrains broken out by first-time installers vs repeat installers, it looks like most of the installs on any given day are coming from users who have already installed cody on jetbrains previously. I'd expect this to happen on occasion (maybe a user uninstalled and redownloaded, or something along those lines) but the fact that it's happening repeatedly to me makes it seem like something is broken here.
Another reason I suspect something is off is that we’re seeing CodyInstalled events fire on non-dotcom endpoints - because install occurs before the user can update their endpoint, I’d expect most if not all install events to occur on dotcom.
We'll keep digging on our end to see if we can uncover any more clues about what might be going on here but I wanted to re-open this ticket to flag this for your team as well if you have any ideas about what could be the cause here.
cc @dadlerj and @chenkc805

@taylorsperry just realized you're not on this ticket, so flagging this for you for prioritization purposes. TLDR is that CodyInstalled events are being logged on non-install actions. What we haven't been able to figure out is where exactly this could be happening, we just have a lot of evidence (laid out above) to suggest that that is the case. The metrics affected by this are our JetBrains daily install numbers and funnel conversion metrics. I know you guys have a lot going on so I'm running down ways to get around this on our end to get to accurate numbers in the meantime!

Hey @taylorsperry ,
Assigning to you so this doesn't fall through the cracks. I know it's been a while since we've gotten to this ticket but I think we should prioritize this before GA. Install metrics are super important for a bunch of reasons (looking at new users, understanding retention, etc.) and if they aren't being logged correctly it throws off our understanding of how JetBrains as an IDE is doing.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_92rm6cc0gc/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_92rm6cc0gc/clone/recipe/meta.yaml:48: invalid_url: https://pythonhosted.org/PuLP/ : Not reachable: 404
WARNING: /tmp/abs_92rm6cc0gc/clone/recipe/meta.yaml:50: missing_description: The recipe is missing a description
Errors were found


Fix occasional test failure of F.average by changing lower-bound of weight.sum

Fixed the lower bound of self.w.sum().
(#3766)

I still observe error after applying this PR. Could you repeat-run the unittest test with @condition.repeat(20000) annotated to test_backward_cpu?

Jenkins CI test (for commit 25f76512e91dd7a9637f880b637b39349d43f0b8) succeeded without errors!

LGTM!


Apply BEM to People's private CSS

This app contains globally applied CSS rules in stylesheets:

https://github.com/StarcounterSamples/People/blob/52e0f76cb1ef461816eea6bfc818ccdc1df96cba/src/People/wwwroot/People/css/standalone.css

It also contains inline CSS in HTML templates:

https://github.com/StarcounterSamples/People/blob/master/src/People/wwwroot/People/viewmodels/ConfirmDialogPage.html
https://github.com/StarcounterSamples/People/blob/master/src/People/wwwroot/People/viewmodels/OrganizationPage.html

All private CSS should be converted to BEM-style CSS classes defined in a stylesheet, according to the intent started in the issue https://github.com/Starcounter/Starcounter/issues/3450 and explained in the article http://starcounter.io/guides/web/avoiding-css-conflicts/

Related #50

Fixed in #62

This is not fixed yet. Please read up http://starcounter.io/guides/apps/css-bem/:
.people-top-menu ul li a is not a correct BEM class/selector. It should be .people-top-menu__link
This applies to other selectors in stylesheets as well.

I wait for this fix in develop as well :)

Solution tracked in PR #65

Closing as done.


Question - is there documentation that aggregates all API calls, atomicity, etc.

We are evaluating Weed and it looks like a project with a bright future.  However I am unable to find thorough documentation of the API.  I see this page: https://code.google.com/p/weed-fs/wiki/HttpAPIs and then I found this page: https://code.google.com/p/weed-fs/wiki/HttpAPIs on a completely different site.  I'm unsure which are valid.
We almost rejected Weed completely because we did not see a "move" verb in the API docs, but checking the code, I see that it has been implemented in filer_embedded.go:
func (filer *FilerEmbedded) Move(fromPath string, toPath string) error {
Also, there is no discussion of atomicity.  For example, what happens if one client is reading and another attempts to move or put, or vice-versa?

Please check documentation on github. https://github.com/chrislusf/weed-fs/wiki
The move function is implemented for embedded filer only, and with O(1) complexity. The cassandra or redis version do not support move (because it can not be implemented in O(1) for large folders).
There are no transaction. All operations are atomic where the last operation wins.

I updated the github wiki. Welcome to ask questions if still unclear.


Cython future deprecation

warning: freesas/_distance.pyx:76:0: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310

Done with the upgrade to  meson-python


docs: added contributor over time

Checklist:

[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 main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

This PR aims to add the Contributor Over Time for the FCC.

Hi @juzhiyuan thanks for taking the time to prepare this PR.
Unfortunately, we will not be adding this to the README at this time. I
MHO, a better location for the graph maybe on the contributing docs. But even then, I am not very sure that the landing page or the home page in the docs will be a good fit for the graph.
Let us know if you have other ideas though.


K8SPG-298: Map shutdown field

Handle spec.shutdown field properly.
Checklist:

[x] Have you added an explanation of what your changes do and why you'd like them to be included?
[x] Have you updated or added documentation for the change, as applicable?
[x] Have you tested your changes on all related environments with successful results, as applicable?

[x] Have you added automated tests?

Type of Changes:

[ ] New feature
[x] Bug fix
[ ] Documentation
[ ] Testing enhancement
[ ] Other

What is the current behavior (link to any open issues here)?
https://jira.percona.com/browse/K8SPG-298
What is the new behavior (if this is a feature change)?

[ ] Breaking change (fix or feature that would cause existing functionality to change)

Other Information:

Test name
Status

init-deploy
passed

demand-backup
passed

start-from-backup
passed

monitoring
passed

We run 4 out of 4

commit: https://github.com/percona/percona-postgresql-operator/pull/376/commits/e49841351335fbe07a8ef31e908cb890e015045c
image: perconalab/percona-postgresql-operator:PR-376-e4984135


Selected variant resets every time Android Studio gets updated

Greetings, love the plugin but I noticed every time I update AS(which is a lot because I use beta/canary versions) the selected variants gets reset. I usually don't notice until I ran the app and its the wrong configuration and since I work on a big project it can waste a lot of time. I am not sure if this is something that can be improved, is it possible to make it so the chosen variant persists through AS updates? Thanks!

I really don't think that's a problem with the plugin. There is no persistance done by the plugin. No storage. It's Android Studio's responsibility to remember

he

closing . Post back if you have any new information that may change my opinion


Stuck in the void upon respawning with a "Tried teleporting you to an unloaded ship" message in chat

No mods that may affect spawning other than Waystones

I'm getting this too

Some more information: I was, at the time, running no mods other than VS. I placed a bed, slept, then jumped off the ship and let myself fall to my death. Upon respawning, I got the "Tried teleporting you to an unloaded ship" and was in the void. I manually teleported to the coordinates the ship was supposed to be at, and I was back in the overworld. Additionally, the ship was still there. However, it no longer had collisions, it was like a ghost ship. I could pass straight through it, couldn't build or break blocks, and most importantly, I couldn't interact with the Physics Infuser to disassemble it. This was on build 137, I haven't tested in Master yet.

probably doesn't happen anymore


Memory safety proof for JSON_Validate with function contracts

Memory safety proof for JSON_Validate with function contracts.

@remi-delmas-3000 @nwetzler @jimgrundy @markrtuttle could you review these contracts and provide me feedback on the overall re-structure? You can also run all proofs locally.

Hi ! great to see DFCC getting finally used. A couple of remarks: now the contracts can be separated from the actual code, and be written in the proof harness C file:
// in user_code.c

int foo(char *arr, size_t size)
{
  // body
}

// in harness.c
int foo(char *arr, size_t size) __CPROVER_require(size>0 && is_fresh(arr, size)
;

void harness {
  char *arr;
  size_t size;
  foo(arr, size);
}

Second, it seems like contracts for different functions are (almost) exactly the same, that would be a case for reusing the same contract for different functions (I would need to modify the CLI interface to allow specifying a function-contract mapping explicitly, or at least introducing functions for the common parts and calling the functions instead of having bare expressions in the clauses.

@remi-delmas-3000 If we keep the contracts in the harness files (e.g., harness.c), then we will need to include these files in all proofs (Makefiles) so the proofs would have access to them (e.g., add all harness to a proof that needs to replace multiple contracts). That's why I keep all contracts on a separate file named as core_json_annex.h. That way, we can still keep contracts out of the source code, but it makes easier to all proofs get access to all contracts, they just need to include this new .h file in the harness.
Daniel Kroening told me an idea to define "lemmas" instead of contracts. So we can reuse lemmas that looks like exactly the same across contracts. That would be a great idea to avoid duplication.

@remi-delmas-3000 : Are you ok with this latest version of the PR?

@remi-delmas-3000 : Are you ok with this latest version of the PR?

yes this looks good to me. However some of the bugs discovered by @feliperodri are still not fixed (pending merges), we might want to merge these first and then this one.


[DOCFIX] Update security docs

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4342/
Test PASSed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.


Unknown peripheral device failed to retrieve any information

Hello,
I am trying to connect my BLE device angel sensor) using the index.js . However, It is connected it is throwing an error just after a connection. It is saying after the command like this sudo node index.js  Searching for Angel Sensor connected to Angel Sensor 04453 UUID:<PHONE_NUMBER>13 Address: 00:07:80:03:02:13 noble warning: unknown peripheral<PHONE_NUMBER>13  where uuid is my BLE sensor Mac Id.
Would you please help me to resolve this error? What I am doing wrong :(
Thank you in advance

I am getting the same messages https://github.com/sandeepmistry/noble/issues/443

Hi @alokranjan007,
Could you please provide the following?

simple Node.js app to reproduce the issue
debug logs: sudo DEBUG=* node <file>.js
HCI dump: sudo hcidump -t -x
Kernel version

Hi sandeep,
Thank you for your reply.  I apologise for being late to your queries.
Please excuse me I am out of office and will be unaccessible for next 10 days.  It's a kind of emergency and can't be ignored.
So once I joined back I would provide you all details.
Thank you for your kindness and active response.
Thank you,
Alok
Sent from my Sony Xperia™ smartphone
---- Sandeep Mistry wrote ----
Hi @alokranjan007https://github.com/alokranjan007,
Could you please provide the following?

simple Node.js app to reproduce the issue
debug logs: sudo DEBUG=* node .js
HCI dump: sudo hcidump -t -x
Kernel version

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/sandeepmistry/noble/issues/449#issuecomment-241291088, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AS_YZ6-NaaEaClJpQ7eeXIiV8oD6ugV-ks5qiOhwgaJpZM4JoZhC.

Closing this due to lack of activity for now.


Correct documentated KEM ids for OSSL_HPKE_str2suite

I tried to use "p256" per the documentation and it didn't work, so I've updated the values to match the values in https://github.com/openssl/openssl/blob/8ad98cce41aa8a6278f7ade6ad2f70b80b194b72/include/openssl/hpke.h#L56.
I also made the values consistently lowercase, and added the "exporter" AEAD value.

OK with CLA: trivial


Spanish translation, some text spills over

the intro1_para1 text in the Spanish display spills over and this needs to be fixed. (Translator not on GitHub, will follow up on slack to modify the text to make it shorter).

@miquelvila any updates on this?

Closing as this language string has been removed.


TracingHeadersUsingUnorderedDeliveryWithMultipleTriggers waits for bo…

…th triggers to be ready
Backport of upstream https://github.com/knative-sandbox/eventing-kafka-broker/pull/3000
This should be cherry-picked on release-v1.7

/cherry-pick release-v1.7

@pierDipi Thanks for the reviews.

/test 412-test-reconciler-aws-ocp-412


Failed to CreateOrUpdate service when function is updated.

Create a example function with BaaS Integration configuration, then update kafka server to something else and apply the change again.
Go through the logs of openfunction-controller you will find output like below:
2022-10-27T09:51:33.505Z	ERROR	controllers.Serving.OpenFuncAsync	Failed to CreateOrUpdate service	{"error": "Object default/cron-input-kafka-output is already owned by another Serving controller serving-2mfq5"} 2022-10-27T09:51:33.505Z	ERROR	controllers.Serving	Failed to start serving	{"Serving": "default/serving-vjzgh", "error": "Object default/cron-input-kafka-output is already owned by another Serving controller serving-2mfq5"} 2022-10-27T09:51:33.505Z	ERROR	controller.serving	Reconciler error	{"reconciler group": "core.openfunction.io", "reconciler kind": "Serving", "name": "serving-vjzgh", "namespace": "default", "error": "Object default/cron-input-kafka-output is already owned by another Serving controller serving-2mfq5"}
Maybe the service should be removed before creating a new one.

The service will be deleted when the old serving is deleted because the old serving owns the service.
The reason for this error is that there will be a delay in the deletion operation, so creating a new service with the same name immediately after deleting the old service may hit the cache.
The solution is to use GenerateName to create a random name for the service, just like this.
https://github.com/OpenFunction/OpenFunction/blob/main/pkg/core/serving/openfuncasync/servingrun.go#L432
Do you want to commit a pr to fix this bug?


Show mission on the map

Would be nice to show, on the map, the mission that the sample app will upload to the drone.
@ayameMBS Is it a reasonably easy thing to do?

@JonasVautherin yep there is a branch sample-show-mission. for now this is only an orange trace, but I will add a start and stop flag

Perfect! Feel free to open a PR whenever you feel it can be merged!


build gold foot print mask in healsparse

For this mask, we want to record the set of healpix pixels defined as in gold according to the table CARNERO.Y6A2_FOOTPRINT_GRIZ_N2_ALLCOLUMNS in desci in healsparse. The healsparse mask should have a value of 1 for regions in the survey and no bits set outside.
Let's use nside 131072

I updated the first comment to show the new table name

We don't have to use the higher inside. I actually make the metadetection healsparse masks at this nside.

So all we have is 4096 from gold footprint
Eli pointed out that there isn't a point in upsampling it.  We can just check against it.  This is in the apply_masks.py that I just pushed.

We want the actual mask so people can use it for power spectra. Checking it against a catalog is secondary.

Yes, I need this function to create the combined footprint.


Add Admitted Workloads Field to LocalQueueStatus

What type of PR is this?
/kind api-change

What this PR does / why we need it:
Add AdmittedWorkloads to LocalQueueStatus API and update it in the LocalQueueController.
Which issue(s) this PR fixes:

Fixes #259
Special notes for your reviewer:
Sorry for the silliness with opening up multiple repos.  I have my laptop setup with CLA and github commits but I committed from another instance.  I didn't realize that if 1 commit doesn't have my email, it will fail the check.

/assign

Please fix the latest comment and squash

/ok-to-test


add diagrams to ssp demo content

The previous merge for the demo content in the ssp does not contain the diagrams it used to have. Update ssp-example.json so that they are displayed in the OSCAL Viewer.
Update any other demo content as necessary from NIST's demo content found  here

Okay good start! But we want to avoid a bunch of formatting changes; ideally we'd just be dropping our custom content back in. Do you think that's doable?

Sure, I can try to do that. Would I pull the contents from the merged file on main and then add the custom content?

This task was taken care of in another branch and has been merged


VS IDE gets blurry

VSCode Version: 1.39.2
OS Version: Windows 10 (May Update)

Steps to Reproduce:

Put the cursor on the text editor.
As you can see on the images below, both side and bottom panel get blurry. I already tried reinstalling vs code, and deleting the application folder.

Does this issue occur when all extensions are disabled?: Yes.

/duplicate https://github.com/microsoft/vscode/issues/81391


Remove BackendTime from responses >400

Fixes #663

Hi Prathiba, thanks for commenting. If correctly solved, please not just comment, but also merge feature branch (openBackhaul/issue663) into release branch (AP-v2.0.2_spec). Best regards, Thorsten


Integration with default powerbox

Hi @tlmakinen,
Thanks again for putting this awesome package together!
As we discussed, it would be great to have tighter integration of this with the base powerbox code. As I envision it, the best way will be to maintain this as its own package right here. However, instead of copying the Powerbox class from powerbox, it should inherit from it. We should aim for exact drop-in replacement so that people can do import powerbox_jax as powerbox and get exactly the same results as doing import powerbox (but differentiable!). Where not possible to supplement a given method with the jaxified version, we should raise either an error or warning.
To do this, I think the powerbox module in this package should look something like this:
# powerbox.py
import powerbox as _pb
from jax import numpy as np

# replace numpy with jaxified version.
_pb.np = np

class Powerbox(_pb.Powerbox):
    def delta_k():
        # <jaxified code>

To be clear, you'll only need to define methods where the computation needs to be changed using specific jax commands (eg. boolean indexing).
If the method that needs to be changed is more than a few lines long (and you have to change one line of it in the middle), then please make an issue on powerbox suggesting I split this method up so you can overwrite a smaller method!

Awesome, working on this now (sorry for the delay !). I think the only thing that might be a little difficult is the rewritten dft library. I'll get everything subclassed on my end for the Powerbox script and see how it shakes out.


Description for fields\params in annotations

Would be nice to add description for quick doc (Ctrl + q) like:
--- @param paramName paramType Some important info
or
--- Some important info
--- @param paramName paramType

Usage of annotation @param is :
---@param paramName paramType @comment strings


adding market hours as filters in drEngine

This relates to issue #1221
Without waiting for more comments on the issue, I took the liberty to upload these amendments. These are not fully tested (as market is not open). The reason to upload these now is not to request for a merge (yet), but to solicit comments, debates and testing.
Your thoughts are greatly appreciated. Further amendments will depend on them.

not sure how to combine all the changes together, will do it again.  Sorry.


Documentation auto-deploy

Documentation is now deployed automatically. @thomwolf @julien-c

Great, thanks @LysandreJik!


converters: add helper to limit number of digits

Fixes #910

Codecov Report

Merging #911 (0b7c096) into master (93aa5a2) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #911      +/-   ##
==========================================
- Coverage   91.76%   91.76%   -0.01%
==========================================
  Files          66       66
  Lines        5176     5172       -4
==========================================
- Hits         4750     4746       -4
  Misses        259      259
  Partials      167      167


Possible PortMessage buffer truncation

Is your feature request related to a problem? Please describe.
Data truncation is possible in the snprintf() below
https://github.com/nasa/cFE/blob/5cffc39eccee4902f71809c8963920ea1469e3cd/modules/evs/fsw/src/cfe_evs_utils.c#L548-L551
Describe the solution you'd like
Appropriately allocate size for the message.
Describe alternatives you've considered
Leave as is. The possibility of truncating the message may be low given that AppName, for example, will not likely occupy the max 20 characters that it is allotted.
Additional context
This issue was flagged in JSC 2.1 static analysis.
TimeBuffer: 23 characters (excluding the null terminator)
Spacecraft ID: 10 characters (maximum length for uint32)
Processor ID: 10 characters (maximum length for uint32)
AppName: 20 characters (maximum given size)
Event ID: 5 characters (maximum length for uint16)
Message: 122 characters (maximum given size)
Exact non-data characters: 8 characters
The total exact maximum length is 23 + 10 + 10 + 20 + 5 + 122 + 8 = 198 characters needed, not counting the null terminator.
The defined length for CFE_EVS_MAX_PORT_MSG_LENGTH of 172
Requester Info
Justin Figueroa, Vantage

This is actively happening in the bundle, see https://github.com/nasa/cFS/actions/runs/8971391786/artifacts/1476886717 for sample app, ci lab and to lab initialization messages.  This ends up truncating the new line and makes the log harder to parse with long/clobbered lines:
EVS Port1 1980-012-14:03:20.55226 66/1/SAMPLE_APP 1: Sample App Initialized.Sample App Development Build equuleus-rc1+dev46 (Codename Equuleus), Last Official Release: Sam1980-012-14:03:20.55236 CI_LAB listening on UDP port: 1234
EVS Port1 1980-012-14:03:20.55236 66/1/CI_LAB_APP 3: CI Lab Initialized.CI Lab App Development Build equuleus-rc1+dev61 (Codename Equuleus), Last Official Release: CI Lab EVS Port1 1980-012-14:03:20.55256 66/1/TO_LAB_APP 1: TO Lab Initialized.TO Lab Development Build equuleus-rc1+dev52 (Codename Equuleus), Last Official Release: TO Lab v2.3SCH Lab Initialized.SCH Lab Development Build equuleus-rc1+dev29 (Codename Equuleus), Last Official Release: SCH Lab v2.3.0)

I suggest checking for and replacing last two chars w/ */n or whatever truncation character folks want.
Could also update the lab apps to not put out such a long msg, or increase the limit so the current messages fit.

Marking as a bug due to it missing the new line, so the next message gets added on in the output which could be lost by automatic parsing tools that might expect events/sys log output to be on it's own line.  It's also harder to skim visually.


Increase max length on search bookmark URLs

Someone recently tried to create an alert for an Analyse query, but the resulting URL was 207 characters long and our current maximum is 200 characters.
https://sentry.io/ebm-datalab/openprescribing/issues/858011865/
Is there any reason we shouldn't increase this limit to, say, 10024 chars?
The error log contains enough info for us to create the failed alert and contact the user in question if we wanted to do that as well.

@evansd was this done?

Yes, it's a duplicate of closed issue #2075


Installation process suggestions

I suggest adding the following to the readme and to the docs:
The use of a virtual environment for installing the package, e.g.:
python -m venv my_env
source my_env/bin/activate

followed by pip install
Installing from the source on github, for those wanting latest un-released features or for developers:
pip install git+https://github.com/LMBooth/pybci.git

or
git clone https://github.com/LMBooth/pybci.git
cd pybci
pip install -e .

Ping https://github.com/openjournals/joss-reviews/issues/5706

I also think, even though the minimum Python requirement is implicit in the setup.py code, it would be useful to state this explicitly in the docs, ideally in the Installation section.

Thankyou for these recommendation @jsheunis , i've made them and shall now close this issue. Please drop a comment or re-open if you find any related issues.


Fix some edges cases and clean some codes

The first commit includes only typo fix and code clean.
The second commit fixes two bugs in piece_builder.py

The condition when we skip first few files
Raise an error when header_i is out of index.

Thanks, looks good!
There's just make black lint to run

I'm also adding you to the repo, in case it's needed.
However let's keep collaborating with PRs

Thanks ! I just pushed a new commit with the fix of lint.


Added React JS Crash Course by Brad Traversy

Thanks for making this project Ishan!

Issue #1

Ugh sorry if it's messy , I was just trying to resolve conflicts but I guess I messed up.

@ishandeveloper Hi, just fixed the conflicts . It's possible to merge now ? I'm still learning so hopefully I fixed it :)

Hey @parnexcodes !
Congrats on submitting your pull request.
You PR has been merged and the changes will be reflected on the website in a few minutes,.
Do leave a star, if you liked the repo ;)
Happy hacking ! 🎊


bcbio dir and tool dir

hi,
I assign the same path for /path-to-bcbio and /path-to-bcbio-tool while installing with isolate mode.
The main installation went smoothly without any error message, but when I want to add other tools the following message return.
Do you suggest me to re-install and assign different path?
btw, this error happens when I install GEMINI and CADD, dbnsfp, while it shows nothing when installing GATK.
ValueError: Default tool directory not yet saved in config defaults. Specify the '--tooldir=/path/to/tools' to upgrade tools. After a successful upgrade, the '--tools' parameter will work for future upgrades.

Sorry about the problems. Assigning the same path to both isn't a problem and should work. My guess is that none of the installs finished completely cleanly, since bcbio writes the configuration file with tooldir at the end of the process. My suggestion would be to either re-run the base install process or one of the upgrades with --tooldir specified as the initial value. If everything works cleanly, you should get a /path/to/bcbio/config/install-params.yaml file with tooldir in it, which should get things working for all future calls to bcbio. Hope this helps.

Closing this out because it should be working okay and this is pretty old. Feel free to reopen if it is still an issue. Thanks!


⚠ Turnos 24x7 has degraded performance

In 2ff2e8c, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5939 ms

Resolved: Turnos 24x7 performance has improved in e549a14.


datacarpentry/R-ecology-lesson

tracking issue for https://github.com/datacarpentry/R-ecology-lesson'
preview: https://fishtree-attempt.github.io/R-ecology-lesson/

[x] "blockquote" divs (my parser's fault)
[x] investigate figure in index
[x] fix https
[x] fix index link references

615:episodes/03-dplyr.Rmd:703 [unknown div] blockquote
616:episodes/05-r-and-databases.Rmd:264 [unknown div] blockquote
617:index.Rmd:14 [unknown div] <div style="text-align: center; margin-top: 30px; margin-bottom: 30px;">
625:episodes/01-intro-to-r.Rmd:71 [uninformative link text]: [here](https://stat.ethz.ch/R-manual/R-devel/library/base/html/Reserved.html)
626:episodes/02-starting-with-data.Rmd:186 [uninformative link text]: [here](https://tibble.tidyverse.org/)
627:episodes/02-starting-with-data.Rmd:199 [image missing alt-text]: ./fig/data-frame.svg
628:episodes/02-starting-with-data.Rmd:716 [uninformative link text]: [here](https://www.tidyverse.org/)
629:episodes/03-dplyr.Rmd:561 [image missing alt-text]: fig/tidyr-pivot_wider_longer.gif
630:episodes/03-dplyr.Rmd:600 [image missing alt-text]: fig/pivot_wider_graphic.png
631:episodes/03-dplyr.Rmd:640 [image missing alt-text]: fig/pivot_longer_graphic.png
632:episodes/05-r-and-databases.Rmd:57 [needs HTTPS]: [a website](http://db.rstudio.com/)
633:CITATION.Rmd:65 [needs HTTPS]: [http://esapubs.org/archive/ecol/E090/118/](http://esapubs.org/archive/ecol/E090/118/)
634:CONTRIBUTING.Rmd:136 [uninformative link text]: [here](https://yihui.org/knitr/options)
635:index.Rmd:16 [missing file]: [](./fig/DC-logo-vision.png) [image missing alt-text]: ./fig/DC-logo-vision.png
636:index.Rmd:40 [missing file]: [Before we start](00-before-we-start.html)
637:index.Rmd:41 [missing file]: [Introduction to R](01-intro-to-r.html)
638:index.Rmd:42 [missing file]: [Starting with data](02-starting-with-data.html)
639:index.Rmd:43 [missing file]: [Manipulating, analyzing and exporting data with tidyverse](03-dplyr.html)
640:index.Rmd:44 [missing file]: [Data visualization with ggplot2](04-visualization-ggplot2.html)
641:index.Rmd:45 [missing file]: [SQL databases and R](05-r-and-databases.html)
642:index.Rmd:201 [uninformative link text]: [here](https://datacarpentry.org/R-ecology-lesson/CITATION)
643:instructors/instructor-notes.md:29 [missing file]: [code handout](files/code-handout.R)
644:instructors/instructor-notes.md:185 [needs HTTPS]: [http://colorbrewer2.org](http://colorbrewer2.org)

NOTE: this lesson is in a special status because of the work that MCM has done on the rewrite.

@zkamvar please can you confirm whether we update should styles for this lesson or not? It has a very different repository structure from the others.

absolutely not. This has no relation to styles.

fixed in 7bcaacd59d79a991077a8a7f35b74270e7fdcbd9


Setting the supported locales dynamically

First, I'm very happy I found this today! I implemented i18n support in my project in under an hour :)
I'm wondering if there's a reason the supported locales is a private property? It looks like we have to modify the source file every time we add a new supported language, but it would be nice to be able to leave the original code as is and modify the list dynamically.

I agree with tou, this should be at least protected so we can extend and add new languages on the go. I really line this library too and I found it very handy to combine this with weblate as the translation backend.
I suppose you could send a PR but I don't know if this is still maintained. It did not change in two years but I offer my help here and will definitely look into this.
I'm using it in unity but I've copied the files directly in my project instead of actually using it as a dependency to fit my needs. I didn't want my team to mess with submodules ;)

I still support this library, and would be happy to discuss this improvement.
As @GabLeRoux says (and same as how we use it) we just copy this file directly into projects as it just works a little easier that way.
That being said, changing this to a protected property seems perfectly logical.

Thanks for the quick replies! For myself, I prefer subclassing over modifying directly in case an update comes out and another dev on our team copies that updates into our project without realizing they've overwritten our previous modifications.
I've submitted a PR that makes the changes needed and tested that subclassing works to override the locales for me. Here's what I needed to do:
namespace MyApp {
	public class I18n : Mgl.I18n {
		protected static readonly I18n instance = new I18n();

		protected static string[] locales = new string[] { "en-US", "fr-FR", "es-ES", "de-DE" };

		public static I18n Instance {
			get {
				return instance;
			}
		}
	}
}

Now I just have to replace using Mgl with using MyApp and I've added a new locale.

Awesome, I'll update the readme tonight with the instructions you provided. Definitely makes it easier to use and update. :shipit:

Looks good! Thanks you two!

@jbroadway I know it's been a while, but I worked with this today, and I realized that overriding locales here doesn't work as expected. locales is static so the above example doesn't exactly override the locales inside base class. In other words, static belongs to the class and not the object so above code is only hiding it, oups!
When I tested this, it worked by coincidence as I used fr_FR + en_US, but I tried with fr_CA instead and now it's failing with Missing: locale [fr_CA] not found in supported list where fr_CA comes from my own class, but locales contains base class values.
Quick workaround is still to edit base class directly which is easier, but it doesn't feel right, I'll see if I can do something about it, I should have tested this a bit more, sorry.

 Hmm... What about a SetLocales(locales) method, or adding a locales option to I18n.Configure()?

I think it's the way to go, but this implies changing a few parts of the code that are currently static. If we add a SetLocales(locales), we'll also need a GetLocales().
We'll need to convert
protected static string[] locales = new string[] { "en-US", "fr-FR", "es-ES" };

back to private non static:
string[] locales = new string[] { "en-US", "fr-FR", "es-ES" };

And change the way it's used inside. Example, we'd have to remove static from:
 static void InitConfig()
        {
            if (locales.Contains(_currentLocale))
{

I'm not convinced that using static almost everywhere makes the library easier to use.

@jbroadway @lux Thanks to work from @shinji-yoshida we have a pretty good solution for part of the issue


Update Cargo.toml

This will break our ed25519 code because ed25519-dalek isn't compatible with rand=0.7 yet. Believe me I've tried.

You need to sign your commit with a DCO. This is a Linux Foundation requirement. See https://github.com/hyperledger/ursa/blob/master/CONTRIBUTING.md on how to contribute source code.

This will break our ed25519 code because ed25519-dalek isn't compatible with rand=0.7 yet. Believe me I've tried.

Cool, you fixed the problem at the root.
https://github.com/dalek-cryptography/ed25519-dalek/pull/101

amending the patch for dco does not work... I'll close it after @mikelodder7 PR is merged.
Somebody/I can do the update to rand 0.7 then.
In case you are interested why I am doing this... I am trying to get of multiple versions of the same crate to be compiled into libindy.
See cargo tree -d
Which is tedious and might annoy people. The Rust ecosystem is moving fast so every crate author is behind somewhere all the time.

Unfortunately, we also need num-bigint to update to rand=0.7 before Ursa can.

https://github.com/rust-num/num-bigint/pull/110 fixes this
Get BlueMail for Android
On 21 Oct 2019, 20:55, at 20:55, Michael Lodder<EMAIL_ADDRESS>wrote:

Unfortunately, we also need num-bigint to update to rand=0.7 before
Ursa can.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/hyperledger/ursa/pull/73#issuecomment-544656769

In the ursa call today, we discussed closing this PR. Is there anything else we should discuss before doing so?

I am on vacation, so can't check until Sa.
Libindy uses 0.7
In general I find it kind of unacceptable to be stuck at a lower version (for any crate)
Get BlueMail for Android
On 5 Feb 2020, 16:51, at 16:51, Michael Lodder<EMAIL_ADDRESS>wrote:

In the ursa call today, we discussed closing this PR. Is there anything
else we should discuss before doing so?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/hyperledger/ursa/pull/73#issuecomment-582473417

I don't like being stuck at a lower version either. However, we can't always jump to the latest. Controlled marches.

Understandable, and we do try to be up-to-date as much as possible. However, we have to do so in a controlled way that doesn't break code. We watch the dependencies closely to make sure we can stay up-to-date. Thanks for keeping an eye on this.


Online runkit demo is broken

Starting here: https://www.npmjs.com/package/angular2-grid
Click the link marked "Test angular2-grid in your browser."
it loads page: https://runkit.com/npm/angular2-grid
That page displays an error:
Error: Cannot find module 'rxjs/Subject'

Hey, I don't actually know how to change anything for the runkit page. It's nothing to do with me and is just an autogenerated npm thing, as far as I know. To see it running you can go to bmorton.co.uk/angular, or if you know how to fix the runkit stuff, let me know!


Unable to delete a sensor from an extension

Hi, I've been experiencing some issue when attempting to delete sensors that were created from an extension. The delete button does not delete the sensor, see:
https://github.com/user-attachments/assets/f4a09549-94f1-4901-96ed-45d17c148ed0
So far I think this happens when the extension id (in the xml and the KarooExtension/Device) has special chars like a ., example: com.example.myapplication
The only way to delete such sensor would be to change the id of extension (to myapplication for example), re-add the sensor (which replaces the existing one? - is that also a bug?) and then the delete works.

I was able to reproduce this and your inference is correct, it's due to . character in your extension ID. This comes down to how we manage document IDs suffixes which use . internally. So while the document gets written in correctly with your extension with ., it can get hung-up in caches and on deleting due to the way we split the document ID back out using . separator. It also is why you have a replacement when you change the ID because in some places, they resolve to the same ID as long as the last part of the extension after . matches.
I didn't foresee this but I'd suggest excluding . from your extension ID. And I'll figure out how to document or prevent this.


Forge Script could not estimate Gas

So I am forking olympus, and using forge scripts to deploy it.
It says transaction could not estimate gas, when deploying too many events. I comment them out and it works.
How can I not get this error? Would rather not change the olympus codebase.
It also uses my Ethereum to deploy all contracts, my when it reaches the fail point it does not revert

`// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.7.5;
import "forge-std/Script.sol";
import "../src/BondDepositor.sol";
import "../src/OlympusAuthority.sol";
import "../src/interfaces/IOlympusAuthority.sol";
import "../src/interfaces/IgOHM.sol";
import "../src/interfaces/IERC20.sol";
import "../src/Staking.sol";
import "../src/interfaces/IStaking.sol";
import "../src/Treasury.sol";
import "../src/interfaces/ITreasury.sol";
import "../src/governance/gOHM.sol";
import "../src/sOlympusERC20.sol";
import "../src/OlympusERC20.sol";
//forge script script/DeployContract.s.sol:DeployBondScript --rpc-url $ARBITRUM_RINKEBY_RPC_URL  --private-key $PRIVATE_KEY --broadcast -vvvv
contract DeployBondScript is Script {
OlympusBondDepositoryV2 bondDepositor;
OlympusAuthority authority;
OlympusStaking staking;
OlympusTreasury treasury;
gOHM gohm;
sOlympus sohm;
OlympusERC20Token ohm;

//authority arguments
address governer = address(1);
address guardian = address(2);
address policy = address(3);
address vault = address(4);

//staking arguments
uint256 epochLength = 0;
uint256 firstEpochNumber = 0;
uint256 firstEpochTime = 0;

//treasury arguments
uint256 timelock = 0;

function setUp() public {}

function run() public {
    vm.startBroadcast();

    //Deploy ohm
    ohm = new OlympusERC20Token(governer);

    //Deploy sOhm
    sohm = new sOlympus();

    //Deploy gOhm
    gohm = new gOHM(governer, address(sohm));

    //Deploy Staking TODO: init arguments
    staking = new OlympusStaking(address(ohm), address(sohm), address(gohm), epochLength, firstEpochNumber, firstEpochTime, address(authority));

    // Deploy the Authority
    authority = new OlympusAuthority(msg.sender, msg.sender, msg.sender, msg.sender);

    //Deploy Treasury
    treasury = new OlympusTreasury(address(ohm), timelock, address(authority));

    // Deploy the bond contract
    bondDepositor = new OlympusBondDepositoryV2(IOlympusAuthority(address(authority)), IERC20(address(ohm)), IgOHM(address(gohm)), IStaking(address(staking)), ITreasury(address(treasury)));

    console2.log("ohm address: ", address(ohm));
    console2.log("sohm address: ", address(sohm));
    console2.log("gohm address: ", address(gohm));
    console2.log("staking address: ", address(staking));
    console2.log("authority address: ", address(authority));
    console2.log("treasury address: ", address(treasury));
    console2.log("bondDepositor address: ", address(bondDepositor));

    vm.stopBroadcast();
}

}
`


add support for generating bar plots from ANCOM-BC data

This pull requests adds first pass visualizers for addressing #104, inspired by Figure 6 of the ANCOM-BC paper.
An example .qzv can be viewed here. The example data is from the Human Microbiome Project (HMP 1). It was accessed using Qiita under study id 1928 in 2022. Differential abundance was computed using ANCOM-BC via q2-composition with QIIME 2 2022.11.
Still to-do:

[x] add unit tests
[ ] ~add usage examples~ (likely won't be able to get to this in time)
[x] add new altair dependency to package

This one should be at a state that is good enough for an MVP of this type of visualization. The main index page could use some work, e.g., to have tabs for the different plots rather than links, but the code is pretty well unit and field tested.
The link to the example viz above has been updated after commit c230df8, so gives a current view of what this looks like (that link again is here).

The link to the example viz above has been updated after commit c230df8, so gives a current view of what this looks like (that link again is here).

Just waiting on tests for merge.


Extension host abort during training run

Environment data

VS Code version: 1.36.0
Extension version (available under the Extensions sidebar): 2019.6.22090
OS and version: win 10
Python version: 3.6.5

about the bug
I wanted to run a deep learning model using jupyter server of this extension,but recently the jupyter in the vscode always stuck in  a cell,and then pop up a  message saying "extension host program accidentally aborted ".The cell always contained code of model training(which contained a lot of operations).
the bug I thought
I tried to run the same model in the ipython (in the vscode interminal),it had no problem.So it must be that the jupyter server of this extension has some problem.I think it may be related to the variable display function module of the jupyter server in vscode.When the python runs,the vscode host program also runs,and its cpu utilization is almost the half of the python program.I thought the vscode host program should reduce many unnecessary operations :)

the model I runed is related to https://github.com/keras-team/keras/blob/master/examples/addition_rnn.py

If you close the variable explorer, does the problem go away? The variable explorer shouldn't request any data when it's closed.

Did you happen to have a progress bar or continuous output in your training run? We have an issue where too much text output can crash the extension host

Hopefully this is a dupe of #6222


installation issue

Having the following output while trying to compile in ubuntu 14.04 with Qt 5.2  installation.
///////////////////////////////////////////////////
sajjad@sajjad-G74Sx:~/Documents/Books/Qt/qml/qmlbook-master$ make
sphinx-build -b html -d _build/doctrees   . _build/html
make: sphinx-build: Command not found
make: *** [html] Error 127
//////////////////////////////////////////////////
Any idea ?

It seems that you have not installed the sphinx-doc toolchain. You can always access the contents on qmlbook.org, there is no need to build it locally.


feat: Add support for case-insensitive queries

This PR makes the queries case insensitive.
Description
I made the regex case insensitive and for places where simple comparisons were made, I simply lower-cased the strings.
As explained in #1612 by @claremacrae.
I just didn't found the file to put the changes for the release, please let me know in the comments where I can find it :smile:
Motivation and Context
Fixes: #1612
Fixes: #1503
How has this been tested?
For the tests, I duplicated all the query line to test them in the uppercase format.
Screenshots (if appropriate)
Types of changes

Changes visible to users:

[ ] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[x] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)
[x] Documentation (prefix: docs - improvements to any documentation content for users)
[ ] Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)
[ ] Contributing Guidelines (prefix: contrib - any improvements to documentation content for contributors - see Contributing to Tasks)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[x] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Sorry for the delay @claremacrae
I will do the modifications as you suggested :smile:

I've tested a variety of different search patterns and are all good:
```tasks
Description INCLUDEs trash
description regex matches /TRASH/i
Description REGEX MATCHES /TRASH/i

ExplAIN

has due date
HAS DUE DATE

happens after 1 year ago
HAPPENS AFTER 1 YEAR AGO

PRIORITY ABOVE MEDIUM

The results are all consistent, and absolutely fine.
The only cosmetic thing is that the instructions implemented with FilterInstructionsBasedField and FilterInstruction, such as HAS DUE DATE, do not preserve the case of instruction in the query block... This is definitely not a show-stopper, and I can easily fix that later, if it ever gets to the top of my list...

Thank you @vdusart - this will be greatly appreciated by many users of Tasks on iPhones! 😄


Add tracing logging

Adds tracing logging to the replay binary. The logs have the following structure:

A block span for all transactions in a given block
A transaction span for the execution of a single transaction
An error event is emitted if the status of the RPC and the blockifier execution don't match, or if the transaction fails unexpectedly
An info event is emitted if the status matches

@pefontana I added to the README how to see only error messages, but I couldn't find how to filter only reverted transactions.
I also added rpc_execution_status to the execution result event


Gradle Build Failed

> Configure project :
paperweight-patcher v1.5.11 (running on 'Windows 11')

> Configure project :purpur
paperweight-patcher v1.5.11 (running on 'Windows 11')

> Configure project :purpur:paper
paperweight-core v1.5.11 (running on 'Windows 11')

> Task :purpur:paper:patchCraftBukkit FAILED
> Task :purpur:getPaperUpstreamData FAILED
> Task :getPurpurUpstreamData FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':purpur:paper:patchCraftBukkit'.
> io.papermc.paperweight.PaperweightException: Command finished with 128 exit code: git -c commit.gpgsign=false -c core.safecrlf=false clone --no-hardlinks C:\Users\konst\Documents\LinearPurpur\LinearPurpur\.gradle\caches\paperweight\upstreams\paper\work\CraftBukkit C:\Users\konst\Documents\LinearPurpur\LinearPurpur\.gradle\caches\paperweight\upstreams\paper\.gradle\caches\paperweight\taskCache\patchCraftBukkit.repo

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 9.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

For more on this, please refer to https://docs.gradle.org/8.5/userguide/command_line_interface.html#sec:command_line_warnings in the Gradle documentation.

BUILD FAILED in 16s
2 actionable tasks: 2 executed

This error appears, since you are using Windows to build LinearPurpur.
And solution can be found in https://github.com/PaperMC/paperweight/issues/50


Support high level control e.g. Walk/Get Balance Mode/... in Mujoco to control Unitree G1?

Hello Unitree Team,
Thanks for your awesome project. It helps us a lot.
now we have a question, does the unitree_mujoco can serve as Client API to simulate the G1 high level control.
I found the API from unitree_sdk2 can only work with real robot. Is that correct?
And do you have also suggestion for us, if we want to just simulate the high level control in mujoco without connection to real G1, is that possible? or how can we do it?
Thanks for any feedback

High control and Client API are not support for this project.
This repository mainly used for sim to real verification of low level controller (MPC or RL).


EL-1470 placing behave job into pipeline deployment order

What does this pull request do?
Updating CircleCi config to place Behave tests as part of deployment pipeline.
Any other changes that would benefit highlighting?
Depends on EL-1440
Checklist
- El-1470

@psweeting1 - should we close this, as it's no longer required?


Code split locale chunks per component/route

(👋 Thanks for sharing this great tool!)
I have a handful of components, each with their own vocab folder. Each component is lazy loaded.
I am using VocabWebpackPlugin and I was hoping it would create a locale chunk for each component, but instead it seems to aggregate all translations into a single chunk. This means that users are forced to download all translations, even for components they may never actually see.
I suppose one difficulty in achieving this would be how to add the chunks to the HTML during SSR. Currently this is straightforward because there's only one chunk for the locale and it has a deterministic name, so we can just use the chunkName function, but if we split the locale chunk then this wouldn't be possible anymore.

Yeah, good call out. I agree that this would be nice. There are two axis to possibly chunk on: by language and by chunk. Only the former is supported.
How valuable it is to separate by chunks depends on how big your language catalog gets, in the use-cases I looked at it wasn't a very big benefit . Unlike splitting by language, that is often going to have a much larger benefit to being split out.
Ideally we could split by both axis where appropriate, ideally only splitting out when there is a non-trivial amount of translations.

We ended up building our own tool: https://github.com/unsplash/intlc

👋 I'm curious how you solved this?


Issue #193 - client swallows IO exceptions

Signed-off-by: John T.E. Timm<EMAIL_ADDRESS>

john said he manually verified the fix with test class I provided in issue #193
all 3 CI jobs passed; not sure why there is a phantom "4th job" but this change LGTM


3.4.8+ broke Moulinette File Picker image links for Tokenizer in The Forge

After 3.4.8 I get a "GET 404" error. This is because tokenizer doesn't seem to take the full link from the Moulinette File Picker within the forge. The assets link is not correctly used by Tokenizer. It still works with 3.4.7.
The errors can be seen below.
Edit: the first tokenizer log is when I use the default file browser, that works correctly.

This does not occur with last version of Moulinette & Tokenizer 3.4.7, nor when shift + clicking to skip the tokenizer interface with tokenizer 3.4.11, nor when using the default file browser.
It does happen with any tokenizer version after 3.4.7.
Cheers!

I can't replicate this locally, so suspect this is an interaction with Forge - I'll try and give this a test - if you're willing to reach out to me on Discord to get a bit more info about what you have blanked out in the images that would be super useful.

Hopefully fixed in v 3.4.12


Lenovo driver download logic needs some work

While Downloading driver packages for W10 1909 it selects driver package based on release date not windows version.
Attatched Image:

Picture from Log File:

Lenovo uses "_" to separate BuildVersions in their Driver filenames, I'm thinking that we could use that to our advantage.
Script row 12293:
						$Win10VersionLink = $MatchingLink | Where-Object { 							$_ -match "_"+$($OSVersion)+"_" 						} | Select-Object -Last 1
I Will try that later.

After making that change the script works as intended.

After making the change script works as intended:

Also had to add:
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12

Lenovo download issues have since been resolved. Closing.


HBASE-25923 Region state stuck in PENDING_OPEN

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
7m 56s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 40s
branch-1 passed

+1 :green_heart:
compile
0m 41s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 46s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
1m 47s
branch-1 passed

+1 :green_heart:
shadedjars
3m  4s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 48s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 42s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
3m  6s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  2s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 51s
the patch passed

+1 :green_heart:
compile
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 41s
the patch passed

+1 :green_heart:
compile
0m 46s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 46s
the patch passed

+1 :green_heart:
checkstyle
1m 37s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 49s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 29s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 32s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
2m 57s
the patch passed

_ Other Tests _

-1 :x:
unit
142m 37s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 40s
The patch does not generate ASF License warnings.

192m  2s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3312

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux a66546846c83 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-3312/out/precommit/personality/provided.sh

git revision
branch-1 / 3f8aa4c

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/1/testReport/

Max. process+thread count
4601 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/1/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
6m 59s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
9m 54s
branch-1 passed

+1 :green_heart:
compile
0m 40s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 45s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
1m 47s
branch-1 passed

+1 :green_heart:
shadedjars
3m  2s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 49s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 40s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
3m  4s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  1s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 50s
the patch passed

+1 :green_heart:
compile
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 41s
the patch passed

+1 :green_heart:
compile
0m 45s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 45s
the patch passed

+1 :green_heart:
checkstyle
1m 40s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 52s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 35s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 30s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 43s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
2m 56s
the patch passed

_ Other Tests _

-1 :x:
unit
150m 12s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 42s
The patch does not generate ASF License warnings.

198m 31s

Reason
Tests

Failed junit tests
hadoop.hbase.mapreduce.TestLoadIncrementalHFilesUseSecurityEndPoint

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3312

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 67f15036683e 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-3312/out/precommit/personality/provided.sh

git revision
branch-1 / 0fd6eeb

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/2/artifact/out/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/2/testReport/

Max. process+thread count
4733 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/2/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
6m 55s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
10m  1s
branch-1 passed

+1 :green_heart:
compile
0m 41s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 47s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
1m 48s
branch-1 passed

+1 :green_heart:
shadedjars
3m  2s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 49s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 40s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
3m  7s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m  4s
branch-1 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 52s
the patch passed

+1 :green_heart:
compile
0m 39s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 39s
the patch passed

+1 :green_heart:
compile
0m 46s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 46s
the patch passed

+1 :green_heart:
checkstyle
1m 36s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
2m 52s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
4m 36s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 33s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 41s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
2m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
143m 38s
hbase-server in the patch passed.

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

192m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3312

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 1ca74d11b1f1 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-3312/out/precommit/personality/provided.sh

git revision
branch-1 / a40f458

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/3/testReport/

Max. process+thread count
4776 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3312/3/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


fix: types

Even though this project is now built with typescript, I get this error when I use it on my node app:
Could not find a declaration file for module 'uuid'. './app/node_modules/uuid/dist/esm-browser/index.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/uuid` if it exists or add a new declaration (.d.ts) file containing `declare module 'uuid';`ts(7016)

This fix exports types declaration from package.json.

Sorry, uuid@11 has not actually been published yet.  It's still a work in progress, and uuid@10 (the latest version) doesn't have types included with it yet.   So, for now, you still need to npm install @types/uuid despite what the README says.
Sorry about that!  I expect to have v11 published in the next 1-2 weeks, and you should be good to go then.


Sepolia support

PR Title
Added support for Sepolia test network.
Description
It was added Sepolia test network support to our Network Controller and Transaction controller, also on Transaction controller it was updated the package ethereumjs-common to version 2.6.1  to support Sepolia network
Checklist

[ ] Tests are included if applicable
[ ] Any added code is fully documented

Issue
Resolves #https://github.com/MetaMask/metamask-mobile/issues/5406

Hey @tommasini! Are there any tests we want to update? I found these:

https://github.com/MetaMask/core/blob/b1021fe75cc46e31b23f5d6853fdf87a2a9c0528/packages/network-controller/src/NetworkController.test.ts#L78
https://github.com/MetaMask/core/blob/b1021fe75cc46e31b23f5d6853fdf87a2a9c0528/packages/controller-utils/src/util.test.ts#L36


[BEAM-7389] Update docs with matching code files

Merge after #9664
No content changes, should render exactly the same
R: @rosetn @aaltay
Second phase to update code filenames to match the docs filenames.

Updates the docs
Deletes obsolete code files

TODO:

Wait until #9664 merges so new code files are available
Regenerate notebooks

Staged files will be broken until the new code files are merged

FlatMap: http://apache-beam-website-pull-requests.storage.googleapis.com/9692/documentation/transforms/python/elementwise/flatmap/index.html
ToString: http://apache-beam-website-pull-requests.storage.googleapis.com/9692/documentation/transforms/python/elementwise/tostring/index.html
WithTimestamps: http://apache-beam-website-pull-requests.storage.googleapis.com/9692/documentation/transforms/python/elementwise/withtimestamps/index.html

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Moved all docs source from element-wise to elementwise.
Updated for changes on #9664 to use elementwise instead of element_wise.
TODO: regenerate notebooks

If #9669 lands first, I could update the include buttons from here as well.

Please rebase this.

Rebased to latest version
Added buttons for every snippet

TODO (until #9664 merges)

Generate notebooks
Test and stage website

Tests are failing. Could you check those?

Builds are now passing but I just noticed I had a typo in some closing region tags. Fixing those in #9738 which should fix the page/notebook for WithTimestamps.
I've tested the rest of the notebooks and all of them seem to be working except for the Regex which can't find the Regex transform. I think that will be fixed once the new version is released.

Fixed!

LGTM. Does everything render as before as expected?
We can merge this after all tests pass.

LGTM--everything looks the same.


Creation of CSIDriverInfo object should not loop infinitely

Why are we doing this in kubernetesRegister? After the CSIDriverInfo object is created we should succeed and not continuously retry after sleep.
for {
		verifyAndAddCSIDriverInfo(clientset, csiDriverName, spec)
		time.Sleep(sleepDuration)
}

I have a feeling this hack may have been introduced as a workaround to having flaky object creation because of the flawed retry logic described in #6
/cc @lpabon @msau42

/remove-lifecycle rotten
issue still exists in code

/remove-lifecycle rotten


GPUParticles2D animated particles with random offsets stutter between frames when emitted

Godot version
v4.1.1.rc1.official [e709ad4d6]
System information
Godot v4.1.1.rc1 - Windows 10.0.22621 - Vulkan (Compatibility) - Intel(R) UHD Graphics 620 (Intel Corporation; <IP_ADDRESS>8) - Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz (8 Threads)
Issue description
When a GPUParticles2D has a random animation offset, particles appear to suddenly switch between frames when the particles are first emitted.
CPUParticles2D do not exhibit this behavior, they exhibit what we'd expect. (Particles start on the random frame and do not change)
In the video, CPUParticles2D is on the left, and GPUParticles2D is on the right. Note the stuttering on GPUParticles2D when the particles are first emitted/spawned.
https://github.com/godotengine/godot/assets/77651264/bba15e03-a0c0-4ca0-a4d1-7b2353fb092b
This behavior is also exhibited when the project is run in debug mode.
https://github.com/godotengine/godot/assets/77651264/cdde483a-4ac4-40c9-a291-a45fa495285d
Notable Observations:
Setting anim_offset_min and anim_offset_max to equal values does not exhibit this stuttering behavior.
I discovered this bug in 4.1 stable while trying to pinpoint another visual bug with GPUParticles2D.
Steps to reproduce

Create a GPUParticles2D node
Give it a texture
Give it a CanvasItemMaterial, set particles_animation to on, and set either particles_anim_h_frames or particles_anim_v_frames to an integer value greater than 1.
Give it a ParticleProcessMaterial, set anim_offset_min and anim_offset_max to unequal values.

Minimal reproduction project
ParticleAnimationOffsetIssueMRP.zip

Can you reproduce this with Interpolate disabled and Fixed FPS set to 0 on the GPUParticles2D?

Yes, the behavior persists with these changes, both in editor and in debug

Can you reproduce this with Interpolate disabled and Fixed FPS set to 0 on the GPUParticles2D? See #59072.

Yes, the behavior persists with these changes, both in editor and in debug

This reminds me of https://github.com/godotengine/godot/issues/49767
It could be another issue with calculating random values

Since I'm just getting into the subject, here's what I noticed.
In start():
CUSTOM.z = (tex_anim_offset) * mix(anim_offset_min, anim_offset_max, anim_offset_rand);
And in process():
CUSTOM.z = (tex_anim_offset) * mix(anim_offset_min, anim_offset_max, rand_from_seed(alt_seed)) + tv * tex_anim_speed * mix(anim_speed_min, anim_speed_max, rand_from_seed(alt_seed));
Just like you said with the other issue:

Note how angle_rand is used the first time, but the second time a new random amount is generated

Only this time it's about anim_offset_rand.

Since I'm just getting into the subject, here's what I noticed. In start(): CUSTOM.z = (tex_anim_offset) * mix(anim_offset_min, anim_offset_max, anim_offset_rand);
And in process(): CUSTOM.z = (tex_anim_offset) * mix(anim_offset_min, anim_offset_max, rand_from_seed(alt_seed)) + tv * tex_anim_speed * mix(anim_speed_min, anim_speed_max, rand_from_seed(alt_seed));
Just like you said with the other issue:

Note how angle_rand is used the first time, but the second time a new random amount is generated

Only this time it's about anim_offset_rand.

Ah yes, the rand_from_seed functions need to be called in the same order in both start() and process() otherwise the value will jump like here.


feat(flags): Add LaunchDarkly Integration

Ref:

https://github.com/getsentry/team-replay/issues/492

Do we need to worry about the failing checks, ex "Test Miscellaneous / Miscellaneous (latest) (3.12, ubuntu-20.04) (pull_request)"


Change gradio launch to stop being public by default.

I changed the launch argument "share" to false to stop creating a public link by default, since it has been requested and is a security risk.
Temporary fix until a user friendly way is implemented. If that's even needed.

Thank you for your contribution!


Sorry, I'm not sure where to place this comment but
 in base.html
is misplaced apparently.
Please have a look.

Thank you for bringing this to our attention. We've added a fix.


feat(charset): adds attribute to set charset

I need to work with Blob files with different character set.
Add an optional attribute to set the charset

Awesome pull request, thank you!

Thx for merging.
I have just forgotten to mention this new attribute in the README.md. You'll get another mini pullrequest for docs.
Cheers

Great, thank you!


async import('node-fetch') with v3.1.0 breaks in vite build for sveltejs

Building a Svelte Kit app with Vite, and using the latest v3.1.0 of node-fetch, I receive this error:
[vite-plugin-svelte] ENOENT: no such file or directory, open 'node:http/package.json'
> ENOENT: no such file or directory, open 'node:http/package.json'
Error: ENOENT: no such file or directory, open 'node:http/package.json'
    at Object.openSync (node:fs:585:3)
    at Object.readFileSync (node:fs:453:35)
    at resolveViaPackageJsonSvelte (…vite@2.6.14/node_modules/@sveltejs/vite-plugin-svelte/dist/index.js:1148:39)

I have attempted the async ESM import as suggested in the pinned issue here.  Here is my attempt:
	const fetch =
		typeof window !== 'undefined'
			? window.fetch
			: await import('node-fetch').then(mod => mod.default);

Reproduction
Steps to reproduce the behavior:
# 1. Build a new Svelte Kit app with Vite:
npm init svelte@next my-app # choose `skeleton app`, then say no to all prompts
# 2.
cd my-app && npm i node-fetch
# 3.
touch src/routes/api.js && echo "async function send() {const fetch=await import('node-fetch').then(m => m.default);}" >> src/routes/api.js
# 4.
npm run build

Expected behavior

When I revert to 3.0.0, the app builds fine.

Screenshots

Your Environment

ϕ 00:36:32 ⚭ $ npx envinfo --system --binaries --browsers --npmPackages "{node-fetch,svelte,@sveltejs/*,vite}"

  System:
    OS: macOS 12.0.1
    CPU: (8) arm64 Apple M1
    Memory: 107.44 MB / 16.00 GB
    Shell: 3.3.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 16.13.0 - ~/.nvm/versions/node/v16.13.0/bin/node
    Yarn: 1.22.10 - ~/.asdf/shims/yarn
    npm: 8.1.0 - ~/.nvm/versions/node/v16.13.0/bin/npm
  Browsers:
    Chrome: 96.0.4664.93
    Chrome Canary: 98.0.4751.0
    Firefox: 92.0.1
    Safari: 15.1
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.3
    @sveltejs/kit: next => 1.0.0-next.201
    node-fetch: ^3.1.0 => 3.1.0
    svelte: ^3.44.0 => 3.44.2

Additional context

Many thanks for your labors and attentions!

Hmm, I'm not too familiar with Svelte or Vite. It seems like Vite is for frontend, is the thing that you are building intended to run on Node.js?
From searching around a bit, it seems like this PR that was merged 10 days ago might fix the problem? https://github.com/vitejs/vite/pull/5827
It seems like this went out in the 2.7.0-beta.9 version, could you try that and see if it fixes it?

if v3.0.0 works but 3.1.0 don't then i suspect it might be b/c of the node: prefix/protocol we added and that PR seems to fix it.

for those still scratching their heads:

Core modules can also be identified using the node: prefix, in which case it bypasses the require cache. For instance, require('node:http') will always return the built in HTTP module, even if there is require.cache entry by that name.

https://nodejs.org/api/modules.html

Ah, much thanks for excavating this issue. Glad to see Vite is on it too

@krry did you try the 2.7.0 beta of Vite? Did it work? ☺

@krry did you try the 2.7.0 beta of Vite? Did it work? ☺

Yes, like a charm.


dts: spi: cleanup CONFIG_SPI_x_IRQ_PRI

All of the cases of CONFIG_SPI_x_IRQ_PRI should be DT_SPI_x_IRQ_PRI.
So go ahead and fix them up.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Address part of #11569

Codecov Report

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

@@            Coverage Diff             @@
##           master   #11570      +/-   ##
==========================================
- Coverage   52.98%   48.42%   -4.56%
==========================================
  Files         218      270      +52
  Lines       26858    42126   +15268
  Branches     5951    10139    +4188
==========================================
+ Hits        14231    20400    +6169
- Misses      10188    17645    +7457
- Partials     2439     4081    +1642

Impacted Files
Coverage Δ

subsys/net/ip/trickle.c
64.83% <0%> (-12.8%)
:arrow_down:

subsys/net/ip/net_private.h
70.58% <0%> (-12.75%)
:arrow_down:

subsys/net/ip/6lo.c
70.28% <0%> (-9.14%)
:arrow_down:

subsys/net/ip/net_core.c
65.78% <0%> (-7.74%)
:arrow_down:

subsys/net/ip/net_mgmt.c
74.35% <0%> (-5.1%)
:arrow_down:

subsys/net/ip/ipv6.c
53.69% <0%> (-5.03%)
:arrow_down:

subsys/net/lib/config/init.c
57.77% <0%> (-4.73%)
:arrow_down:

subsys/net/l2/ethernet/arp.c
55.22% <0%> (-4.13%)
:arrow_down:

subsys/net/ip/route.c
47.2% <0%> (-3.36%)
:arrow_down:

subsys/net/ip/net_if.c
62.46% <0%> (-3.31%)
:arrow_down:

... and 131 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 4619f8f...fdc509f. Read the comment docs.


In NativeTransport.EndRequest:  __pthread_mutex_lock: Assertion `mutex->__data.__owner == 0' failed.

Hi,
I'm still doing some excessive load testing to find out why we only reach low rps with our server implementation using HFC behind nginx 1.9.10.
After using Siege in benchmark mode HFC crashed with the following stracktrace.
I was using a simple stress test endpoint with ServiceStack which does simple web requests to another server of mine.
I hope this is helpful for you, I assume it could be a possible bug because happening on higher load:
...
libev.c:322: cmd_error():       A socket error (0x21) occurred on fd 47.
libev.c:180: shutdown_cmdsocket():      Error shutting down client connection on fd 47: 107 (Transport endpoint is not connected)
...
libev.c:317: cmd_error():       Remote host disconnected from fd 14.
libev.c:317: cmd_error():       Remote host disconnected from fd 46.
...
mono: ../nptl/pthread_mutex_lock.c:116: __pthread_mutex_lock: Assertion `mutex->__data.__owner == 0' failed.
Stacktrace:

  at <unknown> <0xffffffff>
  at (wrapper managed-to-native) HyperFastCgi.Transports.NativeTransport.EndRequest (HyperFastCgi.Transports.NativeTransport,ulong,int,int) <0xffffffff>
  at HyperFastCgi.AppHosts.AspNet.AspNetNativeWebRequest.CloseConnection () <0x00058>
  at HyperFastCgi.AppHosts.AspNet.AspNetNativeWebRequest.FlushResponse (bool) <0x00024>
  at System.Web.HttpResponseStream.Flush (System.Web.HttpWorkerRequest,bool) <0x000d0>
  at (wrapper remoting-invoke-with-check) System.Web.HttpResponseStream.Flush (System.Web.HttpWorkerRequest,bool) <0xffffffff>
  at System.Web.HttpResponse.Flush (bool) <0x0021e>
  at System.Web.HttpApplication.OutputPage () <0x00085>
  at System.Web.HttpApplication.PipelineDone () <0x0011b>
  at System.Web.HttpApplication/<Pipeline>c__Iterator1.MoveNext () <0x06636>
  at System.Web.HttpApplication.Tick () <0x00057>
  at System.Web.HttpApplication.Start (object) <0x00272>
  at System.Web.HttpApplication.System.Web.IHttpHandler.ProcessRequest (System.Web.HttpContext) <0x00073>
  at System.Web.HttpRuntime.Process (System.Web.HttpWorkerRequest) <0x002e9>
  at System.Web.HttpRuntime.RealProcessRequest (object) <0x0008f>
  at System.Web.HttpRuntime.ProcessRequest (System.Web.HttpWorkerRequest) <0x0005d>
  at HyperFastCgi.AppHosts.AspNet.MonoWorkerRequest.ProcessRequest () <0x0009f>
  at HyperFastCgi.AppHosts.AspNet.AspNetNativeWebRequest.Process (HyperFastCgi.Interfaces.IWebResponse) <0x00014>
  at HyperFastCgi.Transports.BaseAppHostTransport/<AddBodyPart>c__AnonStorey0.<>m__0 () <0x000bb>
  at System.Threading.Tasks.Task.InnerInvoke () <0x0004a>
  at System.Threading.Tasks.Task.Execute () <0x00068>
  at System.Threading.Tasks.Task.ExecutionContextCallback (object) <0x00054>
  at System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool) <0x0021a>
  at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool) <0x00028>
  at System.Threading.Tasks.Task.ExecuteWithThreadLocal (System.Threading.Tasks.Task&) <0x00174>
  at System.Threading.Tasks.Task.ExecuteEntry (bool) <0x000da>
  at System.Threading.Tasks.Task.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem () <0x00016>
  at System.Threading.ThreadPoolWorkQueue.Dispatch () <0x0021a>
  at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback () <0x00010>
  at (wrapper runtime-invoke) <Module>.runtime_invoke_bool (object,intptr,intptr,intptr) <0xffffffff>

Native stacktrace:

        /usr/bin/mono() [0x49d56c]
        /lib/x86_64-linux-gnu/libpthread.so.0(+0xf8d0) [0x7f6b7ac108d0]
        /lib/x86_64-linux-gnu/libc.so.6(gsignal+0x37) [0x7f6b7a88b067]
        /lib/x86_64-linux-gnu/libc.so.6(abort+0x148) [0x7f6b7a88c448]
        /lib/x86_64-linux-gnu/libc.so.6(+0x2e266) [0x7f6b7a884266]
        /lib/x86_64-linux-gnu/libc.so.6(+0x2e312) [0x7f6b7a884312]
        /lib/x86_64-linux-gnu/libpthread.so.0(pthread_mutex_lock+0x11b) [0x7f6b7ac0b38b]
        /usr/lib/libhfc-native.so.1(+0x55f7) [0x7f6b766755f7]
        /usr/lib/libhfc-native.so.1(end_request+0x112) [0x7f6b766758e2]
        [0x41d6467c]

Debug info from gdb:

=================================================================
Got a SIGABRT while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.
=================================================================

Aborted (core dumped)

Using latest HFC from master branch,
mono --version
Mono JIT compiler version 4.2.2 (Stable <IP_ADDRESS>/996df3c Mon Feb 15 17:30:30 UTC 2016)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
        TLS:           __thread
        SIGSEGV:       altstack
        Notifications: epoll
        Architecture:  amd64
        Disabled:      none
        Misc:          softdebug
        LLVM:          supported, not enabled.
        GC:            sgen

,
cat /hyperfastcgi_1.conf
<configuration>
        <server type="HyperFastCgi.ApplicationServers.SimpleApplicationServer">

                <!-- Host factory defines how host will be created. SystemWebHostFactory creates host in AppDomain in standard ASP.NET way -->
                <host-factory>HyperFastCgi.HostFactories.SystemWebHostFactory</host-factory>

                <!-- <threads> creates threads at startup. Value "0" means default value -->
                <threads min-worker="512" max-worker="4096" min-io="4" max-io="4096" />

        </server>

        <listener type="HyperFastCgi.Listeners.NativeListener">
                <apphost-transport type="HyperFastCgi.Transports.NativeTransport">
                        <multithreading>Task</multithreading>
                </apphost-transport>
            <protocol>InterNetwork</protocol>
            <address><IP_ADDRESS></address>
            <port>9001</port>
        </listener>

    <apphost type="HyperFastCgi.AppHosts.AspNet.AspNetApplicationHost">
                <log level="Debug" write-to-console="true" />
                <add-trailing-slash>false</add-trailing-slash>
    </apphost>

</configuration>

Do you use the latest version of HyperFastCgi? In  https://github.com/xplicit/HyperFastCgi/commit/aeadc0e130c43a979c2e986817c90c1a9bfb73b8 it was fixed the issue when client disconnected while server is sending request data.
If so, can you post a reproducible test case?

Thanks for your pointer, I was under the assumption that I was using exactly this revision, but I was not. My bad. I changed to the correc revision, did the same tests as yesterday, but did not encounter the problem again (anyway, it only happened once during my multiple tests on the weekend, so it seemed to be rare). You can close this, in case I re-encounter this I'll let you know.

I close the issue, if you meet it again open new issue.