Unable to sync progress state for external user. Watch state works fine.

Describe the bug
I have an external/invited user (not managed) that I want to sync watch state and progress state across 2 of my servers. I have the users Plex token and syncing watch state works perfectly fine. The watch progress, however, fails with a 401:
[2024-02-17T12:11:47-08:00] DEBUG: HttpClient - Request [ POST: https://<redirect>.plex.direct:38247/:/progress/?key=135161&identifier=com.plexapp.plugins.library&state=stopped&time=511000]
[2024-02-17T12:11:47-08:00] DEBUG: Requesting [Plex: plex_hoh] play sessions.
[2024-02-17T12:11:47-08:00] DEBUG: HttpClient - Request [ GET: https://<redirect>.plex.direct:32400/status/sessions]
[2024-02-17T12:11:47-08:00] INFO: Ignoring [Movie (2023)] for [plex_svalbard_hoh]. Event originated from this backend.
[2024-02-17T12:11:47-08:00] NOTICE: SYSTEM: Sending [1] progress update requests.
[2024-02-17T12:11:47-08:00] ERROR: SYSTEM: Request to change [plex] [Movie (2023)] watch progress returned with unexpected [401] status code.
[2024-02-17T12:11:47-08:00] NOTICE: SYSTEM: Sent [1] watch progress requests.

I ended up manually recreating the request with my network utilities, and indeed I wasn't able to get the /:progress API working. After doing some traffic snooping in the plex app logged in as this user, I noticed that play progress is updated with the /:timeline API. Using the users token, I can successfully update the play progress using this API, which led me to notice it was being used by watchstate at one point(?) but is now commented out: https://github.com/arabcoders/watchstate/blob/master/src/Backends/Plex/Action/Progress.php#L237. Is there a reason it's no longer being used? It seems like it would solve my issue if that API command was used instead of /:progress.
To Reproduce

Expected behavior
Play progress is sync'd across servers for invited users with a valid Plex token.
Basic report

[ Basic Report ]

WatchState Version: 20240201-33b70ca
PHP Version: 8.2.15
Timezone: America/Los_Angeles
Running in Container? Yes
Data Path: /config
Temp Path: /config
Database Migrated?: Yes
Has .env file? No
Is Tasks Runner working? Yes
Report Generated At: 2024-02-17T20:19:53+00:00

[ Backends ]

[ Plex ==> plex_nucd_hoh ]

Is backend URL HTTPS? Yes
Has Unique Identifier? Yes
Has User? Yes
Export Enabled? Yes
Time since last export? 2024-02-17T18:29:50+00:00
Play state import enabled? Yes
Metadata only import enabled? No
Time since last import? 2024-02-17T20:11:36+00:00
Has webhook token? Yes
Is webhook match user id enabled? Yes
Is webhook match backend unique id enabled? Yes
Has custom options? Yes
{"plex_user_uuid":"b6c2efffc1c9653e"}
[ Plex ==> plex_svalbard_hoh ]

Is backend URL HTTPS? Yes
Has Unique Identifier? Yes
Has User? Yes
Export Enabled? Yes
Time since last export? 2024-02-17T18:29:50+00:00
Play state import enabled? Yes
Metadata only import enabled? No
Time since last import? 2024-02-17T20:11:37+00:00
Has webhook token? Yes
Is webhook match user id enabled? Yes
Is webhook match backend unique id enabled? Yes
Has custom options? Yes
{"plex_user_uuid":"b6c2efffc1c9653e"}
[ Tasks ]

[ Import ]

Is Task enabled? No

[ Export ]

Is Task enabled? No

[ Push ]

Is Task enabled? No

[ Progress ]

Is Task enabled? No

[ Backup ]

Is Task enabled? Yes
Which flags are used to run the task? -v
When the task scheduled to run at? 0 6 */3 * *
When is the next scheduled run? 2024-02-19T14:00:00+00:00

[ Prune ]

Is Task enabled? Yes
Which flags are used to run the task? -v
When the task scheduled to run at? 0 */12 * * *
When is the next scheduled run? 2024-02-18T08:00:00+00:00

[ Indexes ]

Is Task enabled? Yes
Which flags are used to run the task? -v
When the task scheduled to run at? 0 3 * * 3
When is the next scheduled run? 2024-02-21T11:00:00+00:00

[ Requests ]

Is Task enabled? Yes
Which flags are used to run the task? -v --no-stats
When the task scheduled to run at? */2 * * * *
When is the next scheduled run? 2024-02-17T20:20:00+00:00

[ Logs ]

[ /config/logs/app.20240217.log ]

[2024-02-17T08:59:35.695022-08:00] logger.ERROR: Request for [plex_svalbard_hoh] libraries returned with unexpected [401] status code. {"headers":{"x-plex-protocol":["1.0"],"content-type":["text/html"],"connection":["close"],"content-encoding":["gzip"],"x-plex-content-original-length":["91"],"x-plex-content-compressed-length":["82"],"content-length":["82"],"cache-control":["no-cache"],"date":["Sat, 17 Feb 2024 16:59:35 GMT"]}} []
[2024-02-17T08:59:51.762083-08:00] logger.ERROR: Request for [plex_svalbard_hoh] libraries returned with unexpected [401] status code. {"headers":{"x-plex-protocol":["1.0"],"content-type":["text/html"],"connection":["close"],"content-encoding":["gzip"],"x-plex-content-original-length":["91"],"x-plex-content-compressed-length":["82"],"content-length":["82"],"cache-control":["no-cache"],"date":["Sat, 17 Feb 2024 16:59:51 GMT"]}} []
[2024-02-17T09:17:42.624664-08:00] logger.ERROR: Request for [plex_svalbard_hoh] users list returned with unexpected [401] status code. [] []
[2024-02-17T09:20:16.487362-08:00] logger.ERROR: Request for [plex_svalbard_hoh] users list returned with unexpected [401] status code. [] []
[2024-02-17T09:24:01.026836-08:00] logger.ERROR: Request for [plex_svalbard_hoh] users list returned with unexpected [401] status code. [] []
[2024-02-17T09:24:01.203290-08:00] logger.ERROR: Request for [plex_svalbard_hoh] users list returned with unexpected [401] status code. [] []
[2024-02-17T10:35:33.118779-08:00] logger.ERROR: SYSTEM: Request to change [plex_svalbard_hoh] [Movie2 (2023)] watch progress returned with unexpected [401] status code. {"item":{"id":990,"type":"movie"},"remote":{"id":"6452","url":"https://<redacted>.c581899f0c9b4245b326e89d50d211f0.plex.direct:32400/:/progress/?key=6452&identifier=com.plexapp.plugins.library&state=stopped&time=547000"}} []
[2024-02-17T12:11:47.779214-08:00] logger.ERROR: SYSTEM: Request to change [plex_nucd_hoh] [Movie (2023)] watch progress returned with unexpected [401] status code. {"item":{"id":991,"type":"movie"},"remote":{"id":"135161","url":"https://<redacted>.7956b00f7c8e4a72af5a7f7ac4434cfc.plex.direct:38247/:/progress/?key=135161&identifier=com.plexapp.plugins.library&state=stopped&time=511000"}} []

[ /config/logs/access.20240217.log ]

access log file is empty or does not exists.

[ /config/logs/task.20240217.log ]

task log file is empty or does not exists.

[ /config/logs/app.20240216.log ]

[2024-02-16T06:00:40.029962-08:00] logger.ERROR: Request for [plex_svalbard_hoh] libraries returned with unexpected [401] status code. {"headers":{"x-plex-protocol":["1.0"],"content-type":["text/html"],"connection":["close"],"content-encoding":["gzip"],"x-plex-content-original-length":["91"],"x-plex-content-compressed-length":["82"],"content-length":["82"],"cache-control":["no-cache"],"date":["Fri, 16 Feb 2024 14:00:40 GMT"]}} []
[2024-02-16T06:00:40.068854-08:00] logger.ERROR: Exception [TypeError] was thrown unhandled during [Plex: plex_nucd_hoh] backup. Error [DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given @ /src/Libs/helpers.php:95]. {"error":[],"library":{"id":"4","title":"Movies","type":"movie","totalRecords":998,"url":{"App\\Libs\\Uri":"https://<redacted>.7956b00f7c8e4a72af5a7f7ac4434cfc.plex.direct:38247/library/sections/4/all?includeGuids=1&type=1&sort=addedAt&X-Plex-Container-Size=1000&X-Plex-Container-Start=0"}},"segment":{"number":1,"of":1.0,"size":1000},"item":{"backend":"plex_nucd_hoh","id":"22176","title":"Wings [Remastered] (1928)","type":"movie"},"exception":{"file":"/opt/app/src/Libs/helpers.php","line":95,"kind":"TypeError","message":"DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given","trace":[]}} []
[2024-02-16T06:00:40.070734-08:00] logger.ERROR: Exception [TypeError] was thrown unhandled during [Plex: plex_nucd_hoh] backup. Error [DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given @ /src/Libs/helpers.php:95]. {"error":[],"library":{"id":"4","title":"Movies","type":"movie","totalRecords":998,"url":{"App\\Libs\\Uri":"https://<redacted>.7956b00f7c8e4a72af5a7f7ac4434cfc.plex.direct:38247/library/sections/4/all?includeGuids=1&type=1&sort=addedAt&X-Plex-Container-Size=1000&X-Plex-Container-Start=0"}},"segment":{"number":1,"of":1.0,"size":1000},"item":{"backend":"plex_nucd_hoh","id":"22175","title":"The Broadway Melody (1929)","type":"movie"},"exception":{"file":"/opt/app/src/Libs/helpers.php","line":95,"kind":"TypeError","message":"DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given","trace":[]}} []

[ /config/logs/access.20240216.log ]

access log file is empty or does not exists.

[ /config/logs/task.20240216.log ]

[2024-02-16T06:00:36-08:00] NOTICE: SYSTEM: Preloading DirectMapper data.
[2024-02-16T06:00:36-08:00] NOTICE: SYSTEM: Preloading DirectMapper data is complete.
[2024-02-16T06:00:36-08:00] NOTICE: SYSTEM: Backing up [plex_nucd_hoh] play state.
[2024-02-16T06:00:38-08:00] NOTICE: SYSTEM: Backing up [plex_svalbard_hoh] play state.
[2024-02-16T06:00:40-08:00] ERROR: Request for [plex_svalbard_hoh] libraries returned with unexpected [401] status code.
[2024-02-16T06:00:40-08:00] NOTICE: SYSTEM: Waiting on [3] requests.
[2024-02-16T06:00:40-08:00] ERROR: Exception [TypeError] was thrown unhandled during [Plex: plex_nucd_hoh] backup. Error [DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given @ /src/Libs/helpers.php:95].
[2024-02-16T06:00:40-08:00] ERROR: Exception [TypeError] was thrown unhandled during [Plex: plex_nucd_hoh] backup. Error [DateTimeImmutable::__construct(): Argument #1 ($datetime) must be of type string, int given @ /src/Libs/helpers.php:95].
[2024-02-16T06:00:43-08:00] NOTICE: SYSTEM: Operation is finished.

Additional context

Hi
I disabled that endpoint because i thought it was causing a reload on player if it happen during someone was playing video on plex. But turns out it's not the source of this issue, the source of the issue still eludes me. Also, Nobody reported issues with this endpoint until you did. I will revert the change hopefully sometime tomorrow.


prompt for username when --username not used

Signed-off-by: HassanAlsamahi<EMAIL_ADDRESS>

Replaced by #641


Report error when load model and use InferenceTitle

Hi, It still has some problems when I use inferencetitle.

I train model with a RPU Preset Config and save model.
I use load_state_dict to load my model when Inference
In inference, I use Inferencetile example in doc

But when inferencing, it pops up errors like below:

Hi @darrenkao:
many thanks for raising an issue!  This error your are seeing is a result of a loading an state dict from a different RPU model. This is not supported in our simulator. In some exceptional cases, it might get expected results (for instance loading a floating point model with an PulsedDevice), but in most cases it makes no sense. We might add an explicit error message for this in future.
The InferenceTile is the only tile we currently have that should be used for modeling hardware that is intended only for inference purposes and do not have any training capabilities (ie no hardware implementation of an update or backward pass).
In the case of hardware only targeting inference, this InferenceTile is used to 1. perform hardware-ware training (using perfect update and backward pass in software) to optimize a DNN for running inferernce on the hardware, and 2. for testing the inference performance of the hardware with a programming noise model and an statistical noise model for investigating drift and retention.
Thus the error you error you got is a result from not using the InferenceTile correctly. It cannot load a state dict that is derived from an RPUConfig preset. These presets are modelling analog hardware that is also capable of training in analog. So the hardware is really very different for InferenceTile versus presets, and we do not support to load a state dict from one to the other.

Very thanks for your nice reply and describe!

Hi @darrenkao,
to 1: Yes, that is indeed possible.  All what you have to do is set the the forward pass to perfect:
rpu_config = InferenceRPUConfig()
rpu_config.forward.is_perfect = True

You also want to avoid weight clipping (clip) or added noise training (modifier) in this case, if you really want to use the floating point case only.
to 2: We currently only have  a single PCM pair for inference.  We might support multiple pairs in future. One way to add this features would be to implement a new MultiplePairConductanceConverter analogous to the single pair converter (see here). That would model the case where analog devices are added in analog (by adding their current) before the ADC.  For digital addition of bit slices, one could make a custom Module, that uses one InferenceTile for each weight slice and then add the output.

Hi @maljoras ,
Thanks for your kindly reply again,
After I studied the document, I have some questions
From my understanding, the behavior of InferenceTile with PCM Noise Model is

flexible PCM Array size (It means assuming array size is the same as size of input tensor and output tensor),

It seems column and row size of each layer is always the same as input  and output kernel size. Despite I try to set parameter in_size and out_size to InferenceTile as a fix number, it will be modified by program automatically.

According to item 1, noise only inject to a weight matrix once due to tile size is infinite (A weight matrix will not divide to several tile to process),
Assume parameter of inp_res is set to 1/256, the input will serial compute 1 bit by 1 bit for 8 cycle and inject 8 different noise due to cycle-by-cycle variation
Are above correct?

Many thanks for your help again!

Hi @darrenkao ,
Indeed, the tile size currently adapts to the layer at hand. We do not have any automatic mapping scheme. However, if needed you could add a custom mapping quite easily, by writing a new module which has 2 or more AnalogTiles and inputs are distributed to those tiles as appropriate in the forward method. For instance:
class MyModule(nn.Module):
def __init__(self,...):
    self.linear1 = AnalogLinear(...)
    self.linear2 = AnalogLinear(...)

def forward(self, x ):
    x_part1, x_part2 = split(x)   # add some logic here
    y1 = self.linear1(x_part1)
    y2 = self.linear2(x_part2)
    return combine(y1, y2)  # add some logic here

Where one needs to add some logic for the split and combine function as appropriate.
In this case noise is added for each tile separately of course.
Regarding 3: The DAC is just a linear quantization modelling a pulse width modulation input, where the input is quantiized and only one mat-vec pass is computed. We do not yet support bit sliced input schemes (however notice that the iterative bound management recomputes the result with scaled inputs if the output was clipped).
Nevertheless, you could again write a simple module like the above to achieve it:
class MyBitSliceModule(nn.Module):
def __init__(self,...):
   self.linear = AnalogLinear(...)

def forward(self, x ):
   x_slices =  make_bit_slices_from_x(x)  # add some logic here
   y_slices = []
   for bit_slice in x_slices:
        y_slices.append(self.linear(bit_slice))
   return combine_slices(y_slices)  # add some logic here

If you decide to write your own module, consider contributing them to the code base! That would be highly appreciated.

@maljoras
Thank you very much for your kindly help again!
I will try to build these, also to ask for advice here when I face problems...
If it works well, it's my honor to contribute

@maljoras
Following about the first problem in this thread, there are new problem.
I use InferenceRPUConfig to train my model and save. After that, I also use InferenceRPUConfig and try to load this model back to do inference, but the class mismatch error pops up like below:

Hi @darrenkao
Can you share a snippet of the code you used to hw-aware train the model. It seems that the training still uses AnalogTile.

Hi @kaoutar55 ,
I have solved the problem. The problem is caused by I didn't give some analog layer parameter rpu_config, so the program default use AnalogTile not InferenceTile. I think it better to warn user about empty rpu_config than using default rpu config due to it's hard to think about this problem directly if I get tile class mismatch error when debuging.

Good point. @darrenkao we will add this warning. Thanks for pointing this out!


feat: add J3 term to model parameters

Adds an optional C30/J3 Parameter to the Gravitational Model, and populated them for EGM2008 and WGS84/EGM96. To maintain backwards-compatibility, I've chosen to default these to 0 rather than Undefined - if they were Undefined, an existing Parameters struct without C30/J3 would suddenly stop being isDefined(), which I don't think is a desirable behaviour. Mathematically, I think this is consistent too.
The values for EGM2008 were manually parsed from the coefficients file provided by GeographicLib, and cross-referenced with Vallado's Fundamentals of Astrodynamics and Applications (Table D-1). The values for WGS84/EGM96 came from the official NIMA spec (Table 5.1).
wgs84fin.pdf

Summary by CodeRabbit

New Features

Introduced additional parameters C30 and J3 to the gravitational model for enhanced precision.
Updated constructors to accommodate new parameters in gravitational modeling.

Bug Fixes

Improved validation and assertions in tests for gravitational model parameters.

Tests

Expanded test coverage for gravitational models to include new scenarios and assertions for C30 and J3 coefficients.

Looks like we're missing a test for the ostream friend function, if you could add one it would be great. Should be fairly straightforward you can copy the pattern from any other test file.


commander.cpp: correct parameter ID NAV_TAKEOFF_ALT to MIS_TAKEOFF_ALT

param_find errors are currently being ignored

Thanks! This parameter is unused, and probably was missed on a previous cleanup. I've fixed this in https://github.com/PX4/Firmware/pull/2373.


[Route Manager] Add transaction support to Route Repositories

To support the following route repo:

RouteRepository
RouteWithSubnetMapperRepository
RouteWithVpcMapperRepository

This is a very practice to create an issue to track pending items. I will assign this item to you @kevin-zhonghao


Update README.md

typo error

Thank you for your contribution! This is a read only mirror, however. Please submit your change using gerrit.
you would need to check out the repo here: https://gerrit.hyperledger.org/r/#/admin/projects/fabric-test and make your edits, pushing them to gerrit
here is a walkthrough for zephyrproject which is much the same WRT setting up an LFID, adding ssh keys, etc:
https://www.zephyrproject.org/doc/1.1.0/collaboration/code/gerrit_accounts.html
feel free to ask on rocket.chat - https://chat.hyperledger.org/ - discuss in #fabric
one note: when you set up your LFID do not use social logins the first time, create the account and you can add social logins later


local audio onError

{ error: { code: -11800, domain: 'AVFoundationErrorDomain' },
target: 5997 }
iOS platform

If you need help with iOS errors please check StackOverflow

I have checked StackOverflow, but it did not help. Only audio sent by one Android device cannot be played on Apple device, and other devices can play normally.


Drop ForceableMetricType type

What does this PR do?

Remove the ForceableMetricType type alias.
Motivation

It's not useful: it's redundant with the logic inside as_metric_with_forced_type(), which is the only place where we process forced_type. Updating this function (as in #6384) would result in the type hint getting out of date with no actual benefit, so we should as well just drop it.
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

@AlexandreYang Ha! Didn't even need to ask for a review. :wink: :+1:


[BUG] VerifyVersionConstantsIT is unreliable

Describe the bug
testLuceneVersionConstant did not detect that when the Lucene version was updated [1] the test was broken
java.lang.AssertionError:
Expected: <9.10.0>
     but: was <9.9.2>

Root Cause https://github.com/opensearch-project/OpenSearch/pull/12392

Related component
Build
To Reproduce

Update the lucene version
Merge change into main
Run the test org.opensearch.qa.verify_version_constants.VerifyVersionConstantsIT.testLuceneVersionConstant

Expected behavior
Test should fail when the version is upgraded, should pass all other times.
Additional Details
No response

[Triage - attendees 1 2 3 4 5]
Thanks for filing, look forward to seeing this fixed

[Triage - attendees 1 2 3 4 5]
Thanks for filing - consider contributing with a pull request

We have a number of "qa" tests that introduce cross-branch dependencies and result in temporary breakages until all branches are in sync, so to some extent this is by design. There are a number of issues discussing this problem and potential improvements:

https://github.com/opensearch-project/OpenSearch/issues/8070
https://github.com/opensearch-project/OpenSearch/issues/2350
https://github.com/opensearch-project/OpenSearch/issues/2533

@peternied I'm going to remove the "flaky-test" label as I think the failure here is a result of the design of the test. That's not to say it can't be improved, but I don't think it's quite the same category as other flaky tests. Let me know if you disagree.

@andrross I disagree - if we rev the Lucene version the test should immediately break, unless I don't understand how this test is executed.
Can you help confirm that the boundary of the changing the lucence version show have the rest request/response validation that can be done without requiring a change in another branch or as part of the backward compatibility suite?  If the answer is the boundary is complex, then I'd be curious why this test should cross that boundary.

if we rev the Lucene version the test should immediately break, unless I don't understand how this test is executed.

@peternied Sorry, I think I misunderstood this issue. It works like this: we have the Version.java file that defines a static mapping of OpenSearch version to Lucene version. What this test does is:
for each version in `BuildParams.bwcVersion.indexCompatible`:
 - Start an OpenSearch cluster for this version
 - call the `GET /` API and verify its response matches what is defined in the Version.java of the local code.

That means if we change the Lucene version on main (aka 3.0), then the test will still pass because it will use the local code to start the 3.0 cluster. If we change the Lucene version on the 2.x branch, then this test will break on main until we update the 2.x version mapping on main. Does that make sense?

The intent of this test, as I understand it, is to make sure the forward branches get updated when an older branch (e.g. 1.x or 2.x) update the version of Lucene that they use.

If we change the Lucene version on the 2.x branch, then this test will break on main until we update the 2.x version mapping on main. Does that make sense?

Thanks for breaking down the mechanics - much appreciated.
Does it make sense that a contribution that was created before an off-branch version change was issued cannot pass CI until it merges from main - I'm having a harder time understanding that choice.
This sounds like rather than having a separate dependency (e.g. another maven package named OpenSearchVersionAllThing) using the multi-branched nature of git is being repurposed to hold this information.  Does that seem like a fair statement?

Does it make sense that a contribution that was created before an off-branch version change was issued cannot pass CI until it merges from main - I'm having a harder time understanding that choice.

This is something that happens quite a lot (see #2350). The fundamental issue is that the thing we're testing compatibility against (e.g. 2.x, the next unreleased minor) is a moving target. The CI failures tell you the code in your PR is not compatible with that next unreleased minor version. I honestly don't have any good ideas for how to detect that the failure is because of the code you introduced versus a separate change you haven't rebased.

https://build.ci.opensearch.org/job/gradle-check/42005/ again with assertion failure between 9.11.1 and 9.11.0

9.11.1 lucene version bump for 2.16 is merged in main as well.


adding lenses for events

@metaml thanks for getting this done, I think it is going to be much nicer to work with those default events.
I also noticed that we could potentially rename a couple of fields and remove those field modifier methods e.g. _records -> _Records and _s3Object -> _object. I feel like the s3 prefix adds some clarity, but i don't feel strongly about this one way or the other. Anyone have a preference?

LGTM

I also noticed that we could potentially rename a couple of fields and remove those field modifier methods e.g. _records -> _Records and _s3Object -> _object. I feel like the s3 prefix adds some clarity, but i don't feel strongly about this one way or the other. Anyone have a preference?

_s3Object -> _object this sounds good
 _records -> _Records i'm not sure about this, but no preference

If no objections, then let's leave as-is for now. Thanks everyone!


Redstone Contact Still Emith Signal beside Not Connected

Describe the Bug
Redstone Contact Still Emith Signal beside Not Connected causing the attached redstone device not turned off while moving.
Reproduction Steps

Build 2 "Redstone Contact" block facing eachother
Attack Iron Door to one of the Contact
Make one of Contact to Move (detail screenshot)

Expected Result
While moving the Redstone Contact block should turned off and make the Iron Door close.
Screenshots and Videos

Crash Report or Log
No response
Operating System
Windows 11
Mod Version
0.4.1
Minecraft Version
1.18.2
Other Mods
No response
Additional Context
No response

The picture taken while moving upward with Stationary Contraption

blocks do not update on contraptions except for a few exceptions

blocks do not update on contraptions except for a few exceptions. A redstone contact in motion is always powered and will power stationary ones as they make contact.

Is it possible to bring air in enclosed contraption when it moved to underwater, currently when submerged the air replaced with water.

no


Test with Clang-14

Description
Update to test with Clang-14.

🚀 [bumpr] Bumped! New version:v0.0.23 Changes:v0.0.22...v0.0.23


shap.summary_plot + plt.savefig + PDF format = blue boxes artifacts in plot

To begin, thank you for this great tool.
versions: shap 0.31.0; matplotlib 3.0.3; Notebook (G. Colab)
when I try to save the plot in PDF format using:
shap.summary_plot(shap_values, X_test, feature_names=feature_names, show=False)
plt.savefig('importance_scatter_xgb.pdf', format='pdf', dpi=1200, bbox_inches='tight')

some strange blue rectangles appear above the data points.
I would appreciate any information to remove them.

Greetings.

Strange. Are you sure you don't have some other bar plot getting built somewhere? Do you get the same issue when you do?:
shap.summary_plot(shap_values, X_test, feature_names=feature_names, show=False)
plt.show()

If not then it seems like a matplotlib issue.

Thank you for your answer.
In the same cell of the Notebook I created two graphs. Using plt.close() after each graph works correctly.
shap.summary_plot(shap_values, X_test, feature_names=feature_names, plot_type="bar", show=False)
plt.savefig('importance_bar_xgb.pdf', format='pdf', dpi=1200, bbox_inches='tight')
plt.close()`

shap.summary_plot(shap_values, X_test, feature_names=feature_names, show=False)
plt.savefig('importance_scatter_xgb.pdf', format='pdf', dpi=1200, bbox_inches='tight')
plt.close()

Thank you.


New rule to organize unit test contents

@yanivefraim I've always used comments to help organize the contents of a particular unit test such as:
// arrange
// act
// assert

or
// given
// when
// then

What do you think about adding a new rule to require similar comments within the test patterns?

Sounds cool. Are those patterns always relevant?

No, not always, for example, on some unit tests there may be nothing to arrage but I don't think it's detrimental/distracting to have the comments there even if there is nothing in that part of the test. I find it's a nice extention point when you do need to add code to arrage the unit test context, it's clear where that code should go.

OK, I'm with you on this (:

Do you have a preference between "arrange, act, assert" and "given, when, then"?

I use "given, when, then", not sure what is better...

OK, thanks, I don't have a strong preference. I typically use AAA, but GWT may fit a little better with the Jasmine BDD style.


Detect and report an FailedPvcNotFound/FailedDataVolumeNotFound VM status

What this PR does / why we need it:
This PR extends the VM status field (.status.printableStatus) with new FailedPvcNotFound/FailedDataVolumeNotFound statuses, reported when a VM with a PVC/DataVolume type of volume is started, and the referenced object does not exist.
Special notes for your reviewer:

This PR also changes the Reason field of the VMI's Synchronized=False condition set when the referenced DataVolume does not exist. Previously, it was (misleadingly) set to FailedPvcNotFound, whereas now it gets set to FailedDataVolumeNotFound.
I deliberately didn't include any new functest, as I believe this is well covered by unit tests.
Related to #6078, #6104

Release note:
Report FailedPvcNotFound/FailedDataVolumeNotFound VM status when PVC/DV-type volumes reference non-existent objects

/retest

/retest

/retest

/retest

@jean-edouard can you give this another look when you have some time?
No actual changes since your prev lgtm, rather than a rebase.

/approve

/approve

Thanks @jean-edouard! Can you also /lgtm this, or do you prefer someone else to take a look too?

Thanks @jean-edouard! Can you also /lgtm this, or do you prefer someone else to take a look too?

Let's have someone else LGTM, in case I missed something.

/retest

/retest

/retest

/cc

/retest

/retest

/retest

/lgtm

/retest

/retest


indent doesn't work with comment (jsdoc)

for this code:
class test {
  method1() {
      let resp = null;
    }
    /**
     * @param {String} id
     */
  method2(id) {
    let resp2 = null;
  }
}

the code is not correctly formatted. I've to remove jsdoc to get it works as intended.
it happens when indent-char = " "

How is it not correct?  Please provide this input, expected, and actual output.

in my .jsbeautifyrc:
{
  "indent_size": 2,
  "indent_char": " ",
}

the correct code format should be like this:

but with jsbeautify, it becomes this:

the const of first method has 6 spaces instead of 4. the close brace of first method is aligned with the jsdoc of 2nd method, which is supposed to have 2 spaces instead of 4.

@gitawego, I have the same problem. beautification will work correctly if you leave a new line before the JSDoc description.
class test {
   method1() {
      let resp = null;
   }

   /**
    * @param {String} id
    */
   method2(id) {
      let resp2 = null;
   }
}

@bitwiseman, I think that after the beautification, the code should look that way. Because method1 and method2 are functions. And in between functions adds a new line, as in the example below.
before beautification
function f1() {}
function f2() {}

after beautification
function f1() {}

function f2() {}

This has the same cause as #1043 and would also benefit from the fix proposed in #1069.


[appcenter-file-upload-client-node] bump version to 1.2.4

This PRs bumps version to 1.2.3 in order to include new changes presented in the following PRs:
https://github.com/microsoft/appcenter-cli/pull/1732
https://github.com/microsoft/appcenter-cli/pull/1726

/azp run


Enhancement Request...Surface Area Configuration

Could a DSC resource be built in order to perform different Surface Area Configurations? Examples in TSQL would include:
exec sp_configure 'show advanced options', 1;
RECONFIGURE;
exec sp_configure 'Agent', 1;
exec sp_configure 'Agent XPs', 1;
exec sp_configure 'optimize for ad hoc workloads', 1
RECONFIGURE;

closing because of release 1.7


Interesting work!

Great work! Wish you could release the code soon so I can try it out.

Hi ChicyChen,
I’m very interested in your work and would like to ask if you could release the code. It would be greatly appreciated.
Thank you for considering this request!
Best regards,

Hi all, thanks for your interest! We'll try our best to release the codes as soon as possible.

Hi all, thanks for your interest! We'll try our best to release the codes as soon as possible.

Congratulations on your acceptance to NeurIPS 2024! It's really an amazingwork~ When do you plan to release the related code and checkpoints?

Thanks for the interests, we should be able to release the code by the end of this month.

Hi all, I just released the code, if you want to take a look. Thanks for your patience!


Please consider to use semantic versioning

Some people might just check status of their dependencies via npm outdated and if they see just patch update, they can assume that they can safely update.
It does not happen to me but I find semver as useful convention :)

will do if I add new feature.


2022.07b Iteration Plan

SAP teams plan to work on the following issues/pull requests in our next iteration:

PoC to use ABAP file formats for interfaces in ABAP git as experimental feature (see abapGit/abapGit#5658; won't be closed with this iteration)

The iteration lasts 2 weeks and ends on August 1, 2022.

Iteration Results
Closed
Open

PoC to use ABAP file formats for interfaces in ABAP git as experimental feature (see abapGit/abapGit#5658; won't be closed with this iteration)


Removed old code needed be the server to serve pdf

I would remove this code immediately but you can wait until the client pdf is ready. If you are sure that the client will handle all pdfs in the 2.1, then there is not need to wait.

Ping
There are problem with the old code when trying to install openslides on jessie. It would be easier if we just removed it.
Now this PR has to be rebased. Please give me a response if I should rebase this pull request or remove it.

I prefer removing Reportlab as soon as possible but I do not know how many PDF views are already ported to JS.

Currently, assignments pdf and users access data pdf is still missing as pdfmake PDF. After this is finished we will make a 2.1-beta1 release.

@ostcar Please rebase. The last server pdf file is the assignment catalog pdf - Sean is working on this. But if you like we can temporary deactivate this pdf to merge this pull request.

Rebased. Ready for last review.

Very good. I will merge this now. :)


Extend awaystore with -chans option, which records channel hilights to the awaystore as well as only PMs

Watch module doesn't store highlights already?

Watch is for generic word tracking, this is particularly for storing hilights you got in channels when you're away and showing them when you return (in the same manner it already does for PMs).

+1


fix travis builds

I'm running into discrepancies with local tests vs travis builds. Can see in #1353 and saw it mentioned in #1354.
The culprit is the lack of a install in the .travis.yml This is a result of the auto-magic of the install when travis sees a yarn.lock file. More info here

Fascinating. That said, yarn worked for me locally. Why can't we use it in CI? Thanks @nw

See: https://github.com/electron/electron/pull/8229#issuecomment-268149167


Add linear Gaussian model type + validation notebook

Adds wrapper type for linear Gaussian models in which filtering can be performed using both Kalman filter (exact) and particle filters (approximate), for the purposes of validating the particle filter implementations. A Pluto notebook validating that the particle filter estimates of the mean and variance parameters of the filtering distributions asymptotically converge to their true values as the ensemble size tends to infinity is also added.

Codecov Report

Merging #237 (4730d07) into master (9b39f66) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4730d07 differs from pull request most recent head 5d5b096. Consider uploading reports for the commit 5d5b096 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master     #237   +/-   ##
=======================================
  Coverage   91.17%   91.17%
=======================================
  Files           7        7
  Lines         374      374
=======================================
  Hits          341      341
  Misses         33       33

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

What version of MPI.jl do you have in the active environment?
]st -m MPI

After #235 you'll need to do
]resolve
up

in the Pkg REPL to re-resolve and update the environmemt.

Yes, that would do it. Thanks Mose. My misunderstanding of how the Project.toml files work. Happy to merge this now @matt-graham, thanks for the updates.


Understanding startup profiling

Hi everyone
After investing some time in improving app-startup-time we got down to quite an acceptable speed.
Thanks to https://www.nativescript.org/blog/deep-dive-into-nativescript-3.1-performance-improvements.
I am asking now if someone could explain to me what this rather big chunk (marked with a black box) does? I assume it has something to do with angular bootstrap, but according to the blog it is just the creation of the activity and the frame which shouldn't take that long.
Thankful for any input.

The 800ms are for the Angular bootstrapping

alright, thanks!
So this can be lowered by using AoT compilation

Yep it should - try building with webpack and AOT
npm i nativescript-dev-webpack --save-dev
tns run android --bundle --env.uglify --env.aot --env.snapshot


[fix] podSpec incorrectly configured in case template exist

closes #1514
With this change PodSpec fields are assigned with values from template only if they are null/empty.

@whynowy - not entirely sure what's breaking. if you could help identify, happy to fix and get this through

@whynowy - not entirely sure what's breaking. if you could help identify, happy to fix and get this through

Thanks @inishchith ! I will take a look.

E2e failed because of our recent e2e test image upgrade. It is now fixed after rebasing. cc: @whynowy

:) Thanks @KeranYang


Need new owner

Hi guys, I didn't want maintain this theme anymore, so maybe someone wants to be owner of this repo?
I will transfer it to you.

Hi! Would you be okay with me upgrading the theme to Bootstrap 4? In that case I can take ownership and update the repo if that's okay with you.

@philippch yes, ok. Can we start with pull request?

@evgenyrodionov I can take over this if you want


Add UDP-related methods on the event loop.

Adds sock_recvfrom and sock_sendto to the event loop. Tests were added, based on the other tests. There's no Proactor support yet, because it requires changing overlapped.c.
I did some quick research and it seems that sendto might block, although it usually does not, hence why I also added it.

IIRC methods like sock_recv appeared exactly for sake of Proactor support, selector-based event loop don't need it. add_reader and family should be enough.
That's why I'm considering that Proactor support is crucial for proposed functions (if we want to have them at all).

I'm not against writing the code for Windows. According to the docs, create_datagram_endpoint cannot be used on Windows (using the Proactor loop). If the code for Windows is written, it would improve support for async udp on Windows.

Honestly I still don't want this at all. The sock_recv() methods etc.
are all kind of the wrong level and were added as compromises. Look at
how asyncio/streams.py provides a much more reasonable adaptation
compared to sock_recv() and sock_send().

I agree that the streams.py API is much better than the sock_* methods. I didn't want to put the UDP code there because it's not a streaming protocol.
What if we added a create_datagram() function that creates a UDP socket wrapper with a sendto and a recvfrom coroutine? The returned object could have a bind(address) method, or we could add another method create_datagram_server(address) that creates the socket and binds it instantly.

On 21 February 2016 at 17:18, Guido van Rossum<EMAIL_ADDRESS>wrote:

Honestly I still don't want this at all. The sock_recv() methods etc.
are all kind of the wrong level and were added as compromises. Look at
how asyncio/streams.py provides a much more reasonable adaptation
compared to sock_recv() and sock_send().

IMHO, comparing the TCP and UDP cases is not reasonable.
For TCP, the high-level interface you want is asyncio's stream API, so it's
a good thing Stream was added to the library.  Otherwise, you have to
manually handle cases where you ask to send N bytes but M<N bytes are
actually sent instead, and similar problem for receiving.
For the case of UDP, the high level interface you could possibly want is
"send this message in its entirety" and "wait until the next entire message
is received", which is exactly the same interface that so called "low
level" datagram sockets provide.
Sure, we could create a high level datagram sending API, but I am not
seeing what value the high level API could add to the plain socket API.
--
Gustavo J. A. M. Carneiro
Gambit Research
"The universe is always one step beyond logic." -- Frank Herbert

IMHO, comparing the TCP and UDP cases is not reasonable.

I am the first to agree. This is why I am resisting the addition of
sock_sendto() and sock_recvfrom().
[...]

Sure, we could create a high level datagram sending API, but I am not
seeing what value the high level API could add to the plain socket API.

My preference is to stick with the status quo. The OP seems to prefer
writing loop.sock_recvfrom() over writing a simple DatagramProtocol
subclass (with or without an adapter so he can write
.receive()). Maybe you can help them with the latter so we
can just close this PR?

What if we added a create_datagram() function that creates a UDP socket
wrapper with a sendto and a recvfrom coroutine? The returned object could
have a bind(address) method, or we could add another method
create_datagram_server(address) that creates the socket and binds it
instantly.

All that functionality already exists in create_datagram_endpoint(),
except for the simple logic that enables you to write "next_packet =
yield from ". Please give that a try rather than sending a
PR.

I will, thanks.


Smile Recognition with Deep Learning

GOAL Smile Recognition with Deep Learning
DATASET https://www.kaggle.com/datasets/chazzer/smiling-or-not-face-data
DESCRIPTION The project uses CNN model and DenseNet model.

Make sure you star the repo🙃


Addon upgrade (partial implementation of #8107)

The minimal version. Probably not much more will go into v1.0, but we may want to resolve several questions.
limitations stated in kube-addon-update.sh.
What is handled:

create new addon services
delete removed addon services
create/delete/update addon replication controllers
update if version is changed, e.g. if the name changes from my-rc-v1 to my-rc-v2
monitoring addon directory (it is checked every 5 minutes) - I don't know if this is necessary

What is missing:

node addons - we hope they will be handled by salt
update when only parameters changed, but the name didn't. (it is particularly important for services
restarting the linux service kube-addon after kube push (somehow it doesn't work)
tests (only manual tests were performed).
robustness (it is bash)
agreement on naming conventions for update - is versioning like "my-rc-v1" a good idea?
Agreement whether the monitoring of the add-on dir is necessary

Future extensions (probably not for)

rewrite in go
put into a pod
perform rolling update of replication controllers

cc @satnam6502  @fgrzadkowski @zmerlynn @roberthbailey @davidopp @lavalamp

Probably needs some polishing. I wanted to create the PR today, so that I can finish it tomorrow.

Nice. I'll review it today.

Fixes #8811

Ref #8107

The approach looks good. If you can get me another version by your evening / my morning, I tend to start pretty early, we can probably go another round before you even sign off tomorrow - hopefully we can tie this up.

LGTM except a few nits.

@bgrant0607 clarified (in a personal email) that version string should be included in RC names

Commits are sqashed. It's ready for merge.

Sorry, I missed a few corrections. Still working on it.

I'm going to include rotation of /var/log/kube-addon.log in my pull request. The check if addons update is pending is called every 10 minutes, so the file may get large after a few years...

I found a way to restart a service by salt - I just have to create two salt states - one for stopping the service and another one for starting it. In the future it should be changed to monitoring files (I don't know why it doesn't work

@zmerlynn, The change is (finally) finished on my side. Please have a look at it once again.

LGTM! Let's try to get this in today.


S23 cluster configuration upgrade

This PR makes the following changes:

hdfs name node ui port change from 50070 to 9870
python in cluster change from 3.7 to 3.8
jre in cluster change from 8 to 11

LGTM

LGTM.


Adafruit_NeoPixel not an attribute to neopixel??

File "/home/pi/Desktop/audio-reactive-led-strip-master/python/led.py", line 15, in
strip = neopixel.Adafruit_NeoPixel(config.N_PIXELS, config.LED_PIN,
AttributeError: module 'neopixel' has no attribute 'Adafruit_NeoPixel'

Which version of the adafruit library do you have installed on your Rpi?

I have the same issue using version 3.3.5 of adafruit-circuitpython-neopixel

Same issue with adafruit-circuitpython-neopixel 6.0.0, do I need to downgrade? If so, to which version?


Added GitHub Activity Callback Defensive Code

Added GitHub activity callback logic to check if the commit data is
valid, added additional commit_model.author.name and
commit_model.commit.author dictionary checks. Test if child elements
exist and if values are present.

Signed-off-by: David Deal<EMAIL_ADDRESS>

Resolved under a separate pull request. Closing.


The status field in the error object is string

To conform with the IPP and the JSON API error object format
the generator should be using string for status.
https://jsonapi.org/format/#errors
This PR updates the pre-canned type used by the generator used by topology and sources when they generate the openapi.json so it can create the JSON API conformant type.
https://github.com/RedHatInsights/sources-api/blob/8782d2103bed61c40b547603089c0978624950e1/public/doc/openapi-3-v3.0.json#L1780

@eclarizio The openapi generator doesn't have any specs, It runs thru app/models and generates an openapi.json for sources and topology. The changes in this PR is part of a precanned section that is inserted into the openapi.json file. This is only for documentation purposes when an error is sent out from the api-server we create an ErrorDcoument that conforms to this spec. It becomes part of the HTTP response body and at that point it doesn't matter if its a string or an integer.
We do have failures when we test the error response from a api request. They are handled separately in this PR
https://github.com/RedHatInsights/sources-api/pull/219
There are tons of failure that is being fixed in that PR.
This PR just fixes the documentation aspect of it so that we dont keep inserting the precanned object with integer status.


Fix issue#10

Added the slider when calling the viewer with more than 5 images or if specifying use_slider=True. The default behavior can always be overwritten by setting use_slider=False. The slider callback function simply calls the change_image function that was used with the Prev / Next buttons to change the image.

Before merging, could we add a numbering system on both modes? Small fonts, showing x/X, where x is the index (1-based) of the current image and X is the total number of images in the stack?

Sure, I'll add that as well.


Unable to save model

import torch
from rwkvstic.load import RWKV
from rwkvstic.agnostic.backends import TORCH_QUANT
runtime_dtype = torch.float32 # torch.float64, torch.bfloat16
chunksize = 4
useGPU = False # False
target = 4
model = RWKV("model/RWKV-4-Pile-1B5-EngChn-testNovel-671-ctx2048-20230216.pth", backend=TORCH_QUANT, runtimedtype=runtime_dtype, chunksize=chunksize, useGPU=useGPU, target=target)
torch.save({'model': model.state_dict()}, './model_name.pth')
error: AttributeError: 'RWKVMaster' object has no attribute 'state_dict'
How to save the quantized model?

You can save a quantized model as such:
cd path/to/model/folder/
python3 -m rwkvstic --pq
You will be prompted to select a model.
After conversion, you will have a model.pqth file in the folder.
You can load this model by calling RWKV("/path/to/model.pqth")

Thanks, how to use cpu to execute python3 -m rwkvstic --pq

if you dont have the ability to use your gpu for the conversion, it may be possible to hide your cuda devices using enviroment variables

if you dont have the ability to use your gpu for the conversion, it may be possible to hide your cuda devices using enviroment variables

Thanks, I have successfully converted, but I still have some questions:
`from rwkvstic.load import RWKV
from rwkvstic.agnostic.backends import ONNX_EXPORT
import torch
model = RWKV("model.pth", backend=ONNX_EXPORT, dtype=torch.float16) # or torch.float32
the model is exported to model_{layers}_{embed}.onnx
the external data is stored in model_{layers}_{embed}.bin`
I want to convert the quantized PTH to ONNX, but there is no code for saving the model ONNX

hey, please upgrade your rwkvstic version, the newest version will now export 2 files, model.onnx, and model.bin, before ending. you can then load up the model.onnx using rwkvstic, or another runtime

Understand, thank you very much

I tried running the exported onnx model using the onnx runtime of the mobile platform, but it doesn't work, but if I export onnx using https://github.com/AXKuhta/RWKV-LM/tree/onnx, it can run on the mobile platform, unfortunately it is not possible to export the model above 1B5.
Looks like this https://github.com/josephrocca/rwkv-v4-web

@ZTMIDGO You may need to alter the program to flatten the feeds, as the rwkvstic onnx  expects a flattened shape


Improving accessibility

Improving keyboard navigation. Now works with time picker too.
Side-effect of keyboard navigation is now shift key needs to be held down for calendar navigation.
Shorthand weekday names are now displayed as abbr tags, with full names as title.
Using a colour with higher contrast for month navigation.
Adding a WAI-ARIA compatible description.
The text field now has the aria-live="polite" attribute, which should update screenreader users as soon as the value changes.
The date picker popper now has the aria-hidden tag, making it invisible to screenreaders and assistive technologies, in order to make navigation more straightforward.
The aria-describedby tag has been added as well as a description of the keyboard navigation etc.

Two things — I see the CI is broken and I will do my best to push the required fix.
Seccondly, this PR is based on a Wordpress Gutenberg ticket. For context, see https://github.com/WordPress/gutenberg/issues/1311

Hi, thanks for making these changes. Seeing the size of the PR, I'll let @aij take a pass on this too.

I can split it up into chunks as well.

No worries, just wanted an extra pair of eyes on this PR. No need to split up if it's up to me.

This comment is about a small part of the PR—the addition of the shift key. A big part of why we chose this datepicker was that picking a date with the arrow keys is convenient and "just works". I wouldn't want to start needing the shift key for this.
I see that the rationale is to preserve cursor movement in the textbox. But on many platforms, shift+left selects a character to the left of the cursor, so this change would still be hijacking native behavior. On Windows, ctrl+left moves a word to the left, and you can combine the two, etc. I don't think there's any modifier that wouldn't conflict with standard input behavior on some OS.
In my mind, standard input box behavior isn't really a focus of this component. There are others that do that well (e.g. react-day-picker). It's kind of a UX decision that different datepickers have made in different ways. I also don't think the choice necessarily harms accessibility—as long as dates are being announced to the screen reader it should be clear that the arrow keys are the "primary" interface.
Hope that made sense.

@johnsoft What I've been seeing on my side is complaints from users who claim the keyboard navigation is hijacking expected UI behaviour. Freeing direct use of the left and right arrow keys makes it possible to move the input cursor inside the associated input element and edit the value that way, without the user ever having to depend on the date picker.

You could add a prop (e.g. keyboardNavigationTarget=input/date/time with date as default) for that.

@aldavigdis I would have to agree with @johnsoft on his assessment of the shift key.  I was just about to start working on a PR for a slightly less invasive change which would only allow the left and right arrow key to change the date if the calendar is open.  This would allow a user to navigate the raw input without inadvertently opening up the calendar (my biggest gripe with the current state of accessibility).  The problem with the shift key is that it will have to be explained, as it will not be intuitive to a user.   Bottom line is that if the calendar is open the keyboard navigation on the calendar makes sense.  If it is not, it should be opened by a keystroke and then gain focus.
Regardless, The improvements you have made here will keep us using this control as they are more restrictive than the current functionality.

My suggestion would be to split this PR, moving the shift+arrow key navigation to another PR but other improvements.
I quite like @wiscow idea of using the arrow keys to navigate only when the date picker is open.
Great contribution @aldavigdis! :)

@aldavigdis Could you fix the conflicting files please?


Consider removing disposed checks for common methods

Tabris.js widgets are generic model objects that delegate certain properties to the native clients. When disposing of a widget, we don't need to suppress setting custom properties or triggering custom events.
Only native calls must be suppressed after dispose. However, a warning might be sufficient.

I like the idea of allowing as much as possible. The purpose of the dispose method only to clear the native resources associated with the proxy, the rest is up to the JS garbage collector.

We plan to always print warnings when accessing a disposed widgets and to improve the warn output via https://github.com/eclipsesource/tabris-js/issues/1663.


Added REMOVE_PATH_PARTS option

Hi,
I added a feature to manipulate the name of the path before computing the album name. Example:
REMOVE_PATH_PARTS = ["/Output$","/\d{4}-(0[1-9]|1[0-2])-([0-9]{2})$"]
The first regex will remove the "Output" name from the path, taking the parent name it's useful when working with Capture One sessions):
"/folder/myAlbumName/Output" -> "myAlbumName"
The second regex will remove the date from the path:
"/folder/myAlbumName/2018/10/12" -> "myAlbumName"
Obviously, it's configurable as it's regex-based.
I also added the name in the album in the dry run output and added a hidden folder created by QNAP NAS devices.
What do you think about it? Do you see that as a useful addition?
Thanks,
Ben

First-off thanks for the changes/suggestion. 😄
Concept

Sounds interesting but also a bit risky, no? A change in "REMOVE_PATH_PARTS" in-between runs may mean a delete/reupload of an entire library.

So the README has to be explained very well.

Tests...

I'm running the basic tests I have. For now some pics are being re-uploaded in some test cases... Have to check!
May have to add some specific tests for this scenario.
Will let you know how things go...

Relation with FULL_SET_NAME. Help in testing...

Could you help out with one test... which is with the relation with the setting FULL_SET_NAME.

We may run in into a situation with FULL_SET_NAME=False and REMOVE_PATH_PARTS=["/Sub$"] and a pic /home/user/media/Album4/Sub/pic041.jpg that the result SetName is in fact NULL.
That would be an issue.
In the example you show I think you're assuming FULL_SET_NAME=False... so in the second example Album4/Sub we have to test if it actually returns Album4 or NULL.

FilePathName
Set/Album Name (without REMOVE_PATH_PARTS)
Set/Album Name (with REMOVE_PATH_PARTS)

/home/user/media/Album4/pic04.jpg
Album4
Album4

/home/user/media/Album4/Sub/pic041.jpg
Sub
Album4 ❗ Confirm

Is this a typo?

❓Quick question: On README this is a typo correct: name to anem the album

@bezineb5 I was going thru testing your code but I am getting the following error:

sqlite3.InterfaceError: Error binding parameter 2 - probably unsupported type.

When executing the following code. Only reason I see is the self.xcfg.REMOVE_PATH_PARTS being a list and not supported by sqlite3. Did I Crew up your code in any way or which python version are you using?
            litedb.execute(con, 'SELECT#145', slockdb, self.args.processes,
                           cur,
                           'SELECT DISTINCT getSet(path, ?, ?, ?) '
                           'FROM files WHERE getSet(path, ?, ?, ?) '
                           'NOT IN (SELECT name FROM sets)',
                           qmarkargs=(self.xcfg.FILES_DIR,
                                      self.xcfg.FULL_SET_NAME,
                                      self.xcfg.REMOVE_PATH_PARTS,
                                      self.xcfg.FILES_DIR,
                                      self.xcfg.FULL_SET_NAME,
                                      self.xcfg.REMOVE_PATH_PARTS,),
                           dbcaughtcode='145')

Hi @oPromessa ,
Thanks for having a look at it!
To answer your question, the change removes the part before finding the album name (the regular expression is not applied on the album name but in the path). So in the example /home/user/media/Album4/Sub/pic041.jpg, the path will be /home/user/media/Album4/Sub, so after the regexp deletion, only /home/user/media/Album4 will stay. Hence the album name will be Album4.
Does that make sense? My goal is to deal with Capture One sessions, where output jpegs are in the Output folder of what I want to be an album. Also, on some albums, I have a split by date, and I want to discard that structure on Flickr albums.
Concerning the DB storage, I think that indeed, I didn't try that part - it seems to be run only after a real flickr import, and not a dry run, right? My bad, sorry for that.
I've commited a fix, but I've seen you have tests, how can I run them locally?

@bezineb5 Had a quick look at your fix and it's good. Will check.
My tests have on Travis-ci.org environment have the api_key/secret values for a flickr account I have. Such settings are defined as "private" settings within Travis-CI environment. That's why they simply don't run with your user.

I guess for the tests to run for you, you would have to associate your repository with  www.travis-ci.org and define those exact same settings (I can send the exact list to you if you want). Then all the instructions in .travis.yml should work. -- Never tried it but it should work.

I upload the resulting logs to a DropBox account... so if you want that... you also need to setup some access to such Dropbox account. Of course those are private settings within your own Travis-Ci environment.

Alternatively what I do is I have a ubuntu 16 virtual machine where I:

git clone the repository
run python setup.py develop
and then I can run it from a "Test folder" by setting upload.ini values:

api/secret, etc...
The FILES_DIR you can point out to the <repository folder>/tests/Test Photo Library which holds my test pics.

Coding...
I've put some protection on the set_name_from_file function.

Not finished yet but you could check the code and the test cases (on docstrings) and the introduction of a scenario for NoAlbum situations.

This can be an issue
Coming back to the example I mention: the problem is when the regular expression is such that you end up with a NULL value for set_name. Those are the protections I'm trying to make. of course one should configure REMOVE_PATH_PARTS correctly but it is prone to user error.
["/Sub$"]and a pic/home/user/media/Album4/Sub/pic041.jpg` that the result SetName is in fact NULL.

That would be an issue.
In the example you show I think you're assuming FULL_SET_NAME=False... so in the second example Album4/Sub we have to test if it actually returns Album4 or NULL.

FilePathName
FILES_DIR
FULL_SET_NAME = False
REMOVE_PATH_PARTS
Set/Album Name (with REMOVE_PATH_PARTS)
Set/Album Name (Code Protected)
Remark

/home/user/media/Album4/pic04.jpg
/home/user/media
False
Album4

media
As it would be NULL. I'm assuming the basedir folder from FILES_DIR ("media")

/home/user/media/Album4/Sub/pic041.jpg
/home/user/media
True
Sub
Album4
Album4

/home/user/media/Album4/Sub/pic041.jpg
/home/user/media
True
'Sub', 'Album4'

media
Again the need to protect the empty value. I'm assuming  the basedir folder from FILES_DIR ("media")

Good catch for the exception handling!
In the docstring, I'm not sure about the regular expression:
'/some/photos', False, ['(/Out)+'])
I'd keep the '$' to indicate the end of line, instead of the '+'
I 'm also thinking that the base path should be protected, to avoid issue when computing the relative path. I'll have a look at this.
In the case the path ends up being empty, what do suggest to do? I would say that in this case we should have the same behaviour as when a picture is located at the root of the FILES_DIR.
How would an album be named for a picture called /home/user/media/pic041.jpg?
By the way, I tested on my data set, and the album creation works fine in the general cases.

In the case the path ends up being empty, what do suggest to do? I would say that in this case we should have the same behaviour as when a picture is located at the root of the FILES_DIR.
How would an album be named for a picture called /home/user/media/pic041.jpg?

My thoughts exactly... in such case the album is set to "media". That was already the case prior to the REMOVE_PATH_PARTS setting. So I'm keeping it.


exception with Azure image training

Workspace info:

Select image classification
Select Azure training env, set up workspace with existing subscription, compute and default experiment name
Select Weather dataset
Start training
Get the Exception above
It looks like the training process continued after I closed out of dialog, but hasn't finished yet so not sure

dup of #927

Dup bug is closed.


fix: Use mariadb repositories for mariadb-client, for #6083

The Issue

#6083

mysql and mysqldump in the ddev-webserver don't handle mysql's new sandbox directive.
How This PR Solves The Issue
This installs the 10.11.8-MariaDB of the mariadb client (mysql) and the related mysqldump in the ddev-webserver
We already have matching versions in the ddev-dbserver, but some tools like drush and craft cms's backup tools use local tools in the ddev-webserver.
Manual Testing Instructions
Try creating database dumps on ddev-webserver and importing them various places
You'll note ddev exec 'mysqldump db >/tmp/db.sql' will create a db.sql that has the dreaded /*!999999\- enable the sandbox mode */
ddev import-db and ddev export-db now strip this, but if processing is done by application-level things like drush, your mileage may vary. The sql dump that you create may not be import-able by a different version of mariadb or mysql.
Automated Testing Overview

Release/Deployment Notes

https://github.com/ddev/ddev/issues/6083#issuecomment-2155559450 explains the situation here and links to the mariadb issue. I don't think we'll be able to use the package-install technique reliably with multiple mariadb-client versions.
I am in fact working on the same thing for mysql though, and am able to build clients for mysql 5.7, 8.0, 8.4. It seems to matter more with mysql in general. We could do the same thing with mariadb-client, but I suspect at this point we don't have to go there.

Nothing about this prevents us from going back and adding per-version clients, this just means we start with the latest 10.11.


fix: rename chart to gateway and reuse gateway dockerhub repo

replace:
helm install eg oci://docker.io/envoyproxy/envoy-gateway -n envoy-gateway-system --create-namespace

with:
helm install eg oci://docker.io/envoyproxy/gateway -n envoy-gateway-system --create-namespace

reuse gateway repo
fix chart release failed

Codecov Report

Merging #1133 (5b07d84) into main (c7dbbdd) will increase coverage by 0.07%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1133      +/-   ##
==========================================
+ Coverage   62.75%   62.82%   +0.07%
==========================================
  Files          72       72
  Lines        9528     9528
==========================================
+ Hits         5979     5986       +7
+ Misses       3137     3131       -6
+ Partials      412      411       -1

see 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update to Particle Libraries 2.0

@krvarma, can you please update this library to the new Particle Libraries 2.0 on your repo and in the Particle web IDE.
Thanks!

Sure @pkourany, I will do it in coming weeks

@krvarma, to save you time, as an Elite I can take ownership of the library and do the updates.  Would this be acceptable?

@pkourany sure, with pleasure.
Thanks!


Very minimal settings dropdown highlighting in Dark Modern

In Dark Modern it's hard to tell what is hovered. Since there's an extremely subtle hover decoration, it looks like something is missing. I like the blue background in Dark+

This looks like it's caused by the settings dropdown list background color unintentionally being too light.
Updated to match other list widgets:


License ?

Hello, under what license is this project released under ? I would like to study it to learn from it. Thank-you.

I created it when i was learning D* lite. You can read  or download if you want.


Strings are not escaped in Cargo.toml created by "cargo new"

Hi,
I'm discovering Rust, so I created a new project with the following command:
cargo new hello_world --bin

The created directory included a pre-configured Cargo.toml file with the project name, version & author field. The author field was retrieved from my git global config so I got the following Cargo.toml:
[package]
name = "hello_world"
version = "0.1.0"
authors = ["Charles "Demurgos" Samborski<EMAIL_ADDRESS>
And here is a problem: my git name includes also my quoted pseudo, but the string is added as-is without being escaped causing cargo run to break (parse error).
I quickly escaped the inner quotes, but I post it here to help you improve the overall experience.
Regards
I installed Rust with the latest Windows x64 installer.
Platform: Windows 8.1 x64
cargo --version
cargo 0.2.0-nightly (efb482d 2015-04-30) (built 2015-04-30)
rustc --version
rustc 1.0.0 (a59de37e9 2015-05-13) (built 2015-05-14)

I believe this is a bug in toml-rs.

Triage: as this has been fixed in commit 9a35735 I suppose this can be closed now?

Indeed, thanks!


bun: 0.7.3 -> 0.8.0

Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Reword the commit message:
bun: 0.7.3 -> 0.8.0

Result of nixpkgs-review pr 251067 run on x86_64-linux 1

1 package built:

bun

ah! sorry - fixed!

No, not fixed.

ah! sorry - fixed!

How about:
git commit --amend
git push --force-with-lease origin cad/update-bun

done


release note for v1.22.1/version_history/current is 404

Sorry we are having trouble building v22 images. cc @phlax who will look tomorrow.

this should be fixed in #21657


fixbuild agg for random distributed agg table in bindRelation phase

pick from master https://github.com/apache/doris/pull/40181
Proposed changes
Issue Number: close #xxx

run buildall


Canceled call reported as Connection reset by peer

This is cosmetic issue, as I found that when remote call canceled baresip will reported it as Connection reset by peer which is not entirely true. Its nitpicking but connection reset sound like hard reset on transport level UDP/TCP and not proper cancel on SIP level
Maybe better wording like Call canceled by remote peer will suit here better.

many thanks for your comments,
I am afraid that I would like to keep the current logic, which is to display the error codes
coming from the SIP stack.
If I have to check different error values, it will be a lot of extra code just
to have better cosmetics.
Alfred

Well maybe just return SIP error then instead current ambiguous text? Like 487 Request Terminated


WAF with ALB Ingress

What would you like to be added/changed?
Add a section to the ingress module which adds a WAF to the ALB with the appropriate annotation.
The WAF should be pre-created in Terraform and the WAF ID exported using the environment variables mechanism. Configure it with the core rule set and SQL protection. Add a custom rule that allows us to demonstrate the WAF working in the retail sample app front end (block viewing a specific product?).

Probably too detailed for this workshop


Create In-Place Variants for new Wedges

PR #201 introduced new wedge operators for various dual cases but as of now these are all out-of-place operators. Mirroring the pp wedges structure, we should include wrappers to provide an in-place and an out-of-place version for dec_matrix_generate and update gensim to unlock this functionality for simulation generation.

Good catch, George. Could you make a quick set of instructions for how to add such support?

Create a wrapper that generates both the in-place and out-of-place functions and returns them as a tuple in that order. See https://github.com/AlgebraicJulia/Decapodes.jl/blob/393e3279e6d8b385dcf07381347578a0de476050/src/operators.jl#L97 as an example, your actual implementation may differ based what the function looks like.
Add the symbols for those wedge products into the set extra_dec_operators here https://github.com/AlgebraicJulia/Decapodes.jl/blob/393e3279e6d8b385dcf07381347578a0de476050/src/simulation.jl#L513 This is just a collection of non-matrix but in-place operators.

That should be it but it's been a while since I've done this and I remember the solution to getting the wedge to work was kinda hacky. Lmk if anything goes awry.

We will hold off on this feature since it would be best to introduce the in-place version of these operators at the same time we add GPU support for these operators. GPU kernels are always in-place.

This issue can be addressed by the current push in CombinatorialSpaces towards kernelizing operators https://github.com/AlgebraicJulia/CombinatorialSpaces.jl/issues/113 In-place operators and their GPU versions are now one-and-the-same.


Implement email functionality for password resets

Added by https://github.com/Bram-Hub/Willow/commit/0fdc1cb4b6826893cebf73af240e56e4a8887ddf.


Fix 003 for : Empty data table for no column field

When there no field in column area, then double click on data cell displays empty table
Added code to get filtered data even if the colIndexes has 0 length
Commented line 177
return colIndexes.indexOf(index) >= 0;
and added following line
return (colIndexes.length === 0 ) || (colIndexes.indexOf(index) >= 0);

When there no field in column area, then double click on data cell displays empty table
File impacted: orb-latest/src/js/orb.ui.pgridwidget.js
Replaced line 177
return colIndexes.indexOf(index) >= 0;
By
return (colIndexes.length === 0 ) || (colIndexes.indexOf(index) >= 0);

When there no field in column area, then double click on data cell displays empty table
Added code to get filtered data even if the colIndexes has 0 length

Tem


Provide the ability to provide monitor_address instead of an interface

This would allow users who don't know what interface to provide to
give an IP address to use for the monitor instead.
Note: the includes are needed in ceph.conf.j2 because without them
jinja2 can not properly evaluate the template and will complain about a
missing 'ansible_interface' variable. The includes allow the template to
be evaluated correctly and then the correct include will be used during
render time.
Signed-off-by: Andrew Schoen<EMAIL_ADDRESS>

Thanks @andrewschoen.
One question, why dedicated jinja include files?

@leseb without the include files jinja fails to evaluate the template because hostvars[host]['ansible_interface'] is undefined. If we use the includes jinja can validate ceph.conf.j2 as a valid template and only render the appropriate include.
I was unable to find another way around this issue.

We must make sure that the user doesn't populate both values (eth and ip). So we might need an extra check for this.
Also I think we need to highlight in the var file that the user must choose one or the other but not both.

Hum wait, actually the first check in place should be sufficient since jinja is also doing a check. We can't possibly get both configured. Never mind :).

@leseb I've added the comments you requested. Defining both is fine, but monitor_interface will get priority if both are provided.

Thanks @andrewschoen


🐛 [Bug report]: Infinite Loading UI for Twitters section

Describe the bug
When twitter widget fails to load their script, the cards won't show and the section is stuck in the loading state.
To Reproduce
Steps to reproduce the behavior:

Go to /
Once in a blue moon twitter widgets will fail to load
Scroll down to Community/Twitter Section
See infinite loading state

Expected behavior
Show error or hide section
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser: Firefox, Chrome

Additional context
May not always fire due to external script being fetched

I have a fix for this as well as for #684 in one PR.

ig this issue got solved @billy-le You can check the website


Fix #1876 - lock down adaptive-expressions version

Q
A

Bug fix?
[X]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes #1876

What's in this Pull Request?
Change locks down version of adaptive-expressions to 4.22.3. A newer version (4.23.0) of that package was breaking AdaptiveCardHost control.

Merged manually, thank you!


Inline to_datetime documentation

Codecov Report

Merging #336 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #336      +/-   ##
==========================================
+ Coverage   94.28%   94.29%   +<.01%
==========================================
  Files          35       35
  Lines        3485     3486       +1
==========================================
+ Hits         3286     3287       +1
  Misses        199      199

Impacted Files
Coverage Δ

databricks/koalas/namespace.py
91.03% <100%> (+0.75%)
:arrow_up:

databricks/koalas/__init__.py
88.88% <0%> (-3.71%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks great. Only astype left.


Adding private repository with basic auth

I want to add a private repository with basic auth, it could be a running chartmuseum with basic auth. I have tried to add<EMAIL_ADDRESS>but that isn't working.

@jonasdkhansen any logs from the API pod that might indicate what's happening? Bear in mind you'll need to wait for the next sync cycle (up to 60 mins by default) for it to actually fetch charts from that repo.


python311Packages.google-cloud-asset: 3.19.1 -> 3.20.0

Changelog: https://github.com/googleapis/python-asset/blob/v3.20.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262027 run on x86_64-linux 1

4 packages built:

python310Packages.google-cloud-asset
python310Packages.google-cloud-asset.dist
python311Packages.google-cloud-asset
python311Packages.google-cloud-asset.dist


is it Ok for alarm structure in cluster mode

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
use java schedule to design alarm plugin whether face some problem in cluster mode? or I don't understand the whole architecture

    public void start(List<AlarmCallback> allCallbacks) {
        LocalDateTime now = LocalDateTime.now();
        lastExecuteTime = now;
        Executors.newSingleThreadScheduledExecutor().scheduleAtFixedRate(() -> {
            try {
                List<AlarmMessage> alarmMessageList = new ArrayList<>(30);
                LocalDateTime checkTime = LocalDateTime.now();
                int minutes = Minutes.minutesBetween(lastExecuteTime, checkTime).getMinutes();
                boolean[] hasExecute = new boolean[] {false};
                runningContext.values().forEach(ruleList -> ruleList.forEach(runningRule -> {
                    if (minutes > 0) {
                        runningRule.moveTo(checkTime);
                        /**
                         * Don't run in the first quarter per min, avoid to trigger false alarm.
                         */
                        if (checkTime.getSecondOfMinute() > 15) {
                            hasExecute[0] = true;
                            alarmMessageList.addAll(runningRule.check());
                        }
                    }
                }));
                // Set the last execute time, and make sure the second is `00`, such as: 18:30:00
                if (hasExecute[0]) {
                    lastExecuteTime = checkTime.minusSeconds(checkTime.getSecondOfMinute());
                }

                if (alarmMessageList.size() > 0) {
                    allCallbacks.forEach(callback -> callback.doAlarm(alarmMessageList));
                }
            } catch (Exception e) {
                logger.error(e.getMessage(), e);
            }
        }, 10, 10, TimeUnit.SECONDS);
    }

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

Why do you think there is issue in cluster mode? How does alarm cause conflicts with cluster? I am not following.

Why do you think there is issue in cluster mode? How does alarm cause conflicts with cluster? I am not following.

in cluster mode, every service instance may  check the alarm rule and  callback send the alarm message.

such as i want to receive one alarm message in 2 minutes but i receive two same messages actually

I don't think you could receive two same messages. It could be alarming for different entities. I think you have understood the data flow.

this class? AlarmStandardPersistence

Read how we do distributed aggregration. Don't guess what we are doing.

Thanks, I will learn it.


modified required cpu to resolve prod issue

Tech debt  ticket JU-519
modified terraform config to match the cpu change

CI: No IaC security warnings found


About Options on Settings View

This PR resolves #754 and #756.

@stuartbreckenridge  This pull request had a merge conflict.  I resolved it, but you could help me out by always syncing your fork before submitting a pull request.
Thanks for the great work.  I hope we see more from you.


Test on Android some Qs

Hi,i am clone this code try run on my Android device but can't runing ,why ?
pushNotification.onDeviceReady({ projectid: "768554443090", appid : "5ABA6-050C3" });

Not described well.


Arduino only works after a debug breakpoint is triggered in Arduino.run

For some esoteric reason,
the Arduino module only works when a breakpoint is triggered in its run method.
The current workaround is to set the breakpoint in the beginning of the Arduino.run method ,
, remove it when it triggers and continue after.

This does not happen anymore.
There is still no clear reason why it happened, nor why it stopped happening.


Reformat return value of EnvironmentPlugin, fixes #5392

What kind of change does this PR introduce?
Bugfix
Did you add tests for your changes?
Yes
Summary
fixes #5392, reformat return value of EnvironmentPlugin.
Enable import with: let {ENV_VAR_NAME} = process.env.
Does this PR introduce a breaking change?
No

@pierreneter We should add the regular process.env.xxx to the defs variable. The reasoning is that if I use process.env.xxx in my code, it will be rewritten to the value of xxx instead of { xxxx: 'bar', other: 'var' }.xxx. This makes sure that code-elimination with uglify still works.
Basically, we should keep the old code and add process.env: JSON.stringify(collected vars) at the end.

Looks like the whole idea creates more problems than it solves. Adding process.env would break EnvPlugins + DefinePlugins or multiple EnvPlugins. I guess a better solution would be to make the parser smarter about destructing.
Closing


[Minor] Make it clearer that no configuration needs to be done.

What changes were proposed in this pull request?
Make it clearer what the default is.
Why are the changes needed?
User clarity.
Fix: # N/A
Does this PR introduce any user-facing change?
No
How was this patch tested?
Build locally

@qqqttt123 would you please double confirm here.


Rearrange and change sidebar buttons displayed

Removes the SQL, Wizard, and Legends icons from map view
Renames cartocss to Design and infowindow to Tooltip
Adds "Add Point" button to map view
Adds "Add Column" and "Add Row" button to table view

See #30 for description and #44 for wireframes.
Screenshots:

There are some additional failing tests here.

+1 on fixing up the tests


Update documentation of Add-PnPField with -AddToAllContentTypes parameter

Type

[ ] Bug Fix
[ ] New Feature
[x] Sample

Related Issues?
None
What is in this Pull Request ?
Update documentation of Add-PnPField with -AddToAllContentTypes parameter.
Fun fact: I couldn't find this parameter in the documentation but I was pretty sure it existed. After a big of digging it turned out that I created the PR last year to add the parameter https://github.com/pnp/PnP-PowerShell/pull/2909 😄

Thanks @jackpoz !


Add Acknowledgement Checklist

As a protection against bots and less-intelligent people, This PR will prevent that.
Additional Info:
Before this change, Contributors can only read(or skip) the Acknowledgement part then delete it which makes people who have less than 1 brain cells contribute in which their contributions doesn't make sense or just defeats the purpose of the acknowledgement lines 1-6. To prevent just that, this PR will prevent people from making useless contributions.

Closed the PR due to misunderstanding between .github/ISSUE_TEMPLATE/request_feature.yml .github/ISSUE_TEMPLATE.md. Will make a new PR that modifies that files.

I'm pretty sure you must have made a mistake here, cause right now this PR just deletes a file...


Picking and highlighting broken on master

When mouse point moved away from a selectable object to no object area, 'onHover' message on layer-browser isn't cleared out.
Auto highlighting is not working.

I just noticed an issue on line https://github.com/uber/deck.gl/blob/master/src/core/lib/draw-layers.js#L200
Reusing parameters here brings in previous layers parameters. We must use a fresh variable each iteration. This was a bug I introduced in my split of draw and pick into two files.
Not sure if your bug is related.

Turns out picking and highlighting are two different problems introduced with picking refactor, preparing a PR.

Resolved in #982


Don't add activity and reference fields to outgoing message

I noticed that Botkit adds some garbage fields to outgoing fields, e.g. {"type":"message","text":"Hi...","channelId":"websocket","conversation":{"id":"98f515de-b10a-4b4c-51b9-d6fa3f0e1688"},"from":{"id":"bot"},"recipient":{"id":"98f515de-b10a-4b4c-51b9-d6fa3f0e1688"}}
So I decided to remove them.
Previous activity check was incorrect and allowed falsy values to be added to channelData.
Reference fields weren't checked at all so they were sent to client.

Actually, I don't like this patch anymore,
I think that applyConversationReference call
at https://github.com/howdyai/botkit/blob/master/packages/botkit/src/botworker.ts#L136 is completely unnecessary, because the next step strips those fields.

I copied the good part of this change to #1708

Another U-turn,  I realized that this change was good.

This fix has been indirectly incorporated into 4.5! Thanks for the contribution!

Thanks, I upgraded to 4.5 and it is working as expected.


404 page on vps server

Hello
I have vps server
install yourls at /var/www/html/naparohode3.ru
because already have site at /var/www/html/
for yourls site make httpd conf

<VirtualHost naparohode3.ru>
DocumentRoot /var/www/html/naparohode3.ru
ServerName naparohode3.ru
ErrorLog /var/log/httpd/naparohode3.ru/error_log
CustomLog /var/log/httpd/naparohode3.ru/access_log combined
</VirtualHost>
my .htaccess

# BEGIN YOURLS
<IfModule mod_rewrite.c>
 RewriteEngine On
RewriteBase /
RewriteCond %{REQUEST_FILENAME} !-f
   RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ /yourls-loader.php [L]
</IfModule>
# END YOURLS

i create mysql base
my config
`<?php
/* This is a sample config file.

Edit this file with your own settings and save it as "config.php"
*/

/*
** MySQL settings - You can get this info from your web host
*/
/** MySQL database username */
define( 'YOURLS_DB_USER', 'john' );
/** MySQL database password */
define( 'YOURLS_DB_PASS', 'pass' );
/** The name of the database for YOURLS */
define( 'YOURLS_DB_NAME', 'baz4' );
/** MySQL hostname.
** If using a non standard port, specify it like 'hostname:port', eg. 'localhost:9999' or '<IP_ADDRESS>:666' */
define( 'YOURLS_DB_HOST', 'localhost' );
/** MySQL tables prefix */
define( 'YOURLS_DB_PREFIX', 'yourls_' );
/*
** Site options
*/
/** YOURLS installation URL -- all lowercase and with no trailing slash.
** If you define it to "http://site.com", don't use "http://www.site.com" in your browser (and vice-versa) */
define( 'YOURLS_SITE', 'http://naparohode3.ru' );
/** Timezone GMT offset */
define( 'YOURLS_HOURS_OFFSET', 0 );
/** YOURLS language
** Change this setting to use a translation file for your language, instead of the default English.
** That translation file (a .mo file) must be installed in the user/language directory.
** See http://yourls.org/translations for more information */
define( 'YOURLS_LANG', '' );
/** Allow multiple short URLs for a same long URL
** Set to true to have only one pair of shortURL/longURL (default YOURLS behavior)
** Set to false to allow multiple short URLs pointing to the same long URL (bit.ly behavior) */
define( 'YOURLS_UNIQUE_URLS', true );
/** Private means the Admin area will be protected with login/pass as defined below.
** Set to false for public usage (eg on a restricted intranet or for test setups)
** Read http://yourls.org/privatepublic for more details if you're unsure */
define( 'YOURLS_PRIVATE', true );
/** A random secret hash used to encrypt cookies. You don't have to remember it, make it long and complicated. Hint: copy from http://yourls.org/cookie **/
define( 'YOURLS_COOKIEKEY', 'modify this text with something random' );
/** Username(s) and password(s) allowed to access the site. Passwords either in plain text or as encrypted hashes
** YOURLS will auto encrypt plain text passwords in this file
** Read http://yourls.org/userpassword for more information */
$yourls_user_passwords = array(
'username' => 'password',
'username2' => 'password2'	// You can have one or more 'login'=>'password' lines
);
/** Debug mode to output some internal information
** Default is false for live site. Enable when coding or before submitting a new issue */
define( 'YOURLS_DEBUG', false );
/*
** URL Shortening settings
*/
/** URL shortening method: 36 or 62 /
define( 'YOURLS_URL_CONVERT', 36 );
/

36: generates all lowercase keywords (ie: 13jkm)
62: generates mixed case keywords (ie: 13jKm or 13JKm)
Stick to one setting. It's best not to change after you've started creating links.
*/

/**

Reserved keywords (so that generated URLs won't match them)
Define here negative, unwanted or potentially misleading keywords.
*/
$yourls_reserved_URL = array(
'porn', 'faggot', 'sex', 'nigger', 'fuck', 'cunt', 'dick', 'gay',
);

/*
** Personal settings would go after here.
*/
`
also give root  chmod -R 757 /var/www/html/naparohode3.ru/*
& mod_rewrite is enabled

httpd -M | grep "rewrite_module"
Syntax OK
rewrite_module (shared)

please help

RewriteBase is wrongly set.
Please see Apache conf documentation: https://httpd.apache.org/docs/2.2/mod/mod_rewrite.html#RewriteBase


Help issue: searching for topic from coding pages

Describe the Issue
Issue Summary:
When attempting to search for help from the code editing page on FreeCodeCamp, the information passed to the forum search contains hyphens in the exercise title. However, most forum posts are titled differently, without hyphens. This results in failed searches as the forum doesn't recognize the hyphenated title format.
Additional Details:
This issue is not a critical bug but rather a quality-of-life improvement. It would enhance the user experience by aligning the search parameters between the code editing page and the forum search functionality.
Please let me know if further information is required. Thank you.
Affected Page
https://forum.freecodecamp.org/search?q=
Steps to Reproduce

Input incorrect code to solve a challenge multiple times until the Help button appears.
Click Help.
Click on the underlined link "check if your question has already been answered on the forum" when the "Ask for Help" prompt appears.
This leads to a forum.freecodecamp.org page automatically searching your issue as learn-basic-javascript-by-building-a-role-playing-game - Step 127.
The search doesn't bring up any instances of forum topics as they are titled differently, e.g., learn basic javascript by building a role playing game - Step 127.
Search for the topic manually after being prompted with "no previously asked similar questions" to trigger the link to "create a new topic."
Copy the previously searched, hyphenated string into the topic creation title form.
Now, forum topics appear, but only after attempting to create a new topic.

Expected behavior
The link passed from the coding page to the help page should be formatted without hyphens in the exercise title to match the typical forum post title format. For example:
search?q=Learn%20Basic%20Javascript%20by%20Building%20a%20Role%20Playing%20Game%20-%20Step%20127

Screenshots
No response
System

Device: Macbook Pro, Windows Desktop
OS: Windows 10, macOS Monterey 12.7.4
Browser: Chrome

Additional context
No response

Apologies for the oversight, I didn't notice the recently raised PR addressing the same issue.


Add out of the box monitoring via Application Insights

An issue we have with the current template is that it does not come with monitoring built in.
To check the logfiles for errors, I have to ssh to the nodes and cat the logs. Once we deploy the cluster
in production this will be difficult to do as only few people in our organisation have access to production.
It's also quite cumbersome and time consuming in general.
Is it possible to update the template so it creates an instance of Azure Application Insights alongside the cluster?
And have the cluster publish health status and any errors and warns in the logs to Application Insights?
This way we can easily access this information via the Azure UI.
https://docs.microsoft.com/en-us/azure/application-insights/app-insights-overview

Hi @russcam I see monitoring is not offered with the free basic license. Makes sense.
Anyway I think adding Application Insights support in Monitoring would be beneficial for deployments on Azure, because Application Insights is the standard monitoring solution on Azure

There is a level of monitoring available under the free Basic license - check out under Kibana > Stack Monitoring.

that really helped thanks @russcam

No worries @mario-paniccia .
I'm going to close this issue as including Application insights in the template is unlikely to be implemented as a feature at this stage. Monitoring for the entire Elastic Stack is part of the stack features and enabled when xPackPlugins is set to Yes (the default).