RTCP packets and hooking

Hello,
I am testing the RTCP functionality based on the example (https://github.com/ultravideo/uvgRTP/blob/master/docs/examples/rtcp_hook.cc). I modified it slightly so that it only sends 5 frames. Also, I added the deallocation of the session object, as instructed here:
.
My issue is that the RTCP packets are terribly out of sync, and even when we go out of scope and start doing something else, the RTCP receiver reports keep on coming. Because we sent 5 frames, I expect 5 reports, but if we let it play out we receive much more than that:

Moreover, after we stop sending stuff, we just receive RTCP report of the last frame over and over. Inevitably, as we destroy the session, we get an error, because RTCP packets keep on coming:

void receiver_hook(uvgrtp::frame::rtcp_receiver_report* frame)
{
	LOG_INFO("Received an RTCP Receiver Report");

	for (auto& block : frame->report_blocks) {
		fprintf(stderr, "ssrc:      %x\n", block.ssrc);
		fprintf(stderr, "fraction:  %u\n", block.fraction);
		fprintf(stderr, "lost:      %d\n", block.lost);
		fprintf(stderr, "last_seq:  %u\n", block.last_seq);
		fprintf(stderr, "jitter:    %u\n", block.jitter);
		fprintf(stderr, "lsr:       %u\n", block.lsr);
		//fprintf(stderr, "dlsr (ms): %u\n", uvgrtp::clock::jiffies_to_ms(block.dlsr));
	}

	/* RTCP frames can be deallocated using delete */
	delete frame;
}

void appHook(uvgrtp::frame::rtcp_app_packet* packet) {
	LOG_INFO("Received an APP packet!!");
	fprintf(stderr, "Message: '%s'", packet->payload);
        delete packet;
}

int main() {

	uvgrtp::context ctx;
	uvgrtp::session* sess = ctx.create_session("<IP_ADDRESS>");
	uvgrtp::media_stream* sender = sess->create_stream(7777, 8888, RTP_FORMAT_GENERIC, RCE_RTCP);
	uvgrtp::media_stream* receiver = sess->create_stream(8888, 7777, RTP_FORMAT_GENERIC, RCE_RTCP);

	char* message = (char*)"Hello, world!";
	size_t len = strlen(message) + 1;
	auto data = (uint8_t*)message;

	char* name = (char*)"name";
	char* packet_txt = (char*)"pack";
	size_t pack_len = strlen(packet_txt) + 1;
	auto pack_data = (uint8_t*)packet_txt;

	//sender->get_rtcp()->install_app_hook(appHook);

	sender->get_rtcp()->install_receiver_hook(receiver_hook);

	uvgrtp::frame::rtp_frame* frame;
	for (int i = 0; i < 5; ++i) {
		sender->push_frame(data, len, 0);
		std::this_thread::sleep_for(std::chrono::milliseconds(2000));

		frame = receiver->pull_frame();
		fprintf(stderr, "RECEIVED Message: '%s'\n", frame->payload);

		//receiver->get_rtcp()->send_app_packet(name, 0, pack_len, pack_data);

		uvgrtp::frame::dealloc_frame(frame);

	}
	// uncomment to recreate repetitive RTCP packets
	/*
	for (int i = 0; i < 10000; ++i) {
		std::this_thread::sleep_for(std::chrono::milliseconds(20));
	}
	*/

	ctx.destroy_session(sess);
	return 0;
}

rtcp_app hook and manual APP packet sending will demonstrate how synchronous packet exchange could be.
Thank you

Hi,
as Marko already mentioned/implied, the RTCP code was hacked together fast and is quite untested so there be dragons but unfortunately I'm not sure what kind of pressure there is to fix these bugs right now at least.
As for these issues, I'm not sure what you mean by RTCP being horribly out of sync. I believe we set, for simplicity's sake, the RTCP interval to maximum which I believe is around 5 seconds. So every 5 seconds you should receive an RTCP report. RTCP reports are not tied to any single RTP packet but to an interval of time where multiple RTP packets may arrive.
The fix to stop the RTCP reports from coming after sender has stopped streaming is quite simple. The RTCP runner could keep track of when the last packet from the participant was received and if it was more than some interval, it could convert the participant back to receiver (i.e., it wouldn't send RTCP packets to that participant anymore). Somebody should check what the specification has to say about this though.
The crash is probably quite easy to fix. When the session is destroyed, the RTCP destructor must block until the RTCP runner has stopped. This could be done with a condition variable/simple boolean to signal the RTCP runner that is must stop.

Just to clarify, we (Ultra Video Group) will fix all issues users encounter, but we have somewhat limited resources to dedicate to this task (after @altonen graduated). If no one else has time, I will look at this bug when I have the time. Unfortunately, I'm not very familiar with the library.
As far as I understand, RTCP reports are supposed to be sent at constant intervals regardless of how many frames the user sends.

@polusto That is fine, I will submit a pull request to the previous issue (about APP packets) tomorrow. This one is not as critical

I'm closing this issue since there doesn't seem to any direct bugs. There will probably be a slight improvement to the timing accuracy of the reports coming in near future. We may continue the discussion of RTCP reports in issue #59


Improve user feedback on action successes and errors

Add visual cues and messages to user actions results like login success and fail, add document success and fail...


[SPARK-23948] Trigger mapstage's job listener in submitMissingTasks

What changes were proposed in this pull request?
SparkContext submitted a map stage from submitMapStage to DAGScheduler,
markMapStageJobAsFinished is called only in ((https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala#L933 and https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala#L1314);
But think about below scenario:

stage0 and stage1 are all ShuffleMapStage and stage1 depends on stage0;
We submit stage1 by submitMapStage, there are 10 missing tasks in stage1
When stage 1 running, FetchFailed happened, stage0 and stage1 got resubmitted as stage0_1 and stage1_1;
When stage0_1 running, speculated tasks in old stage1 come as succeeded, but stage1 is not inside runningStages. So even though all splits(including the speculated tasks) in stage1 succeeded, job listener in stage1 will not be called;
stage0_1 finished, stage1_1 starts running. When submitMissingTasks, there is no missing tasks. But in current code, job listener is not triggered. We should call it here.

How was this patch tested?
Not added yet.

Test build #89085 has started for PR 21019 at commit 685124a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2120/
Test PASSed.

Test build #89085 has finished for PR 21019 at commit 685124a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89085/
Test FAILed.

Test build #89097 has started for PR 21019 at commit 685124a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2130/
Test PASSed.

Test build #89097 has finished for PR 21019 at commit 685124a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89097/
Test FAILed.

Jenkins, retest this please

Test build #89117 has finished for PR 21019 at commit 685124a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89117/
Test FAILed.

Jenkins, retest this please

Test build #89165 has started for PR 21019 at commit 685124a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2185/
Test PASSed.

@squito @cloud-fan
How do you think this change ?

cc @jiangxb1987

I need to look more closely at the change, but your description of the problem makes sense.  Can you also add a test case?

Test build #89264 has started for PR 21019 at commit 9d369f8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2262/
Test PASSed.

Test build #89264 has finished for PR 21019 at commit 9d369f8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89264/
Test PASSed.

Test build #89426 has started for PR 21019 at commit 42a9b2e.

Thanks comments from Imran and Xingbo.
I made some change and please take another look when you have time.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2368/
Test PASSed.

Test build #89426 has finished for PR 21019 at commit 42a9b2e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89426/
Test FAILed.

LGTM

retest this please

Test build #89437 has started for PR 21019 at commit 42a9b2e.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2378/
Test PASSed.

Test build #89437 has finished for PR 21019 at commit 42a9b2e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89437/
Test PASSed.

merged to master, thanks!

a few minutes after merging this I realized I should have also merged to branch 2.3.  I don't see a way to do that without another PR.  oops.  I opened this, its a clean cherry-pick https://github.com/apache/spark/pull/21085

I guess I rely entirely on the merge script, but in these simple cases I should just do the push directly ...

@squito @jiangxb1987
Thanks for merging.


Minor fixes in documentation

Add missing backslash to multi-line command
Reorder kube-aws commands in README to match Quick Start
Newline end of file

CLA signed and changed email to match my CNCF's

Codecov Report

Merging #1505 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1505   +/-   ##
=======================================
  Coverage   25.47%   25.47%
=======================================
  Files          97       97
  Lines        5001     5001
=======================================
  Hits         1274     1274
  Misses       3580     3580
  Partials      147      147

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 dc29af4...1de00db. Read the comment docs.

@PabloCastellano thank you for your contribution!


Need a blast logo

🚀 Feature Request
Is your feature request related to a problem? Please describe.

We don't have a logo for blast! We really need one, and should put it everywhere!!!

Describe the solution you'd like

Design a logo for blast and put it on:

The website header
The repository readme.md,
The documentation (by adding the html_logo = "path to file" line in the docs/config.py file).

Please design one and post it here as an attachment to a comment. Once we are happy with the design add the file to the base directory of the project!

Here are some sketches I made a few years ago when I initially worked on this project:

Here's another concept:

nice!!!  yeah I really like #1 and the full-scale concept at the bottom.

marking this one as done


dotnet 5 support info

Is this kestrel transport supported for dotnet 5 based aspnet core
If not, could you please state this somewhere in the docs?

I investigated possible run on dotnet 5 and observed error on nginx reverse proxy (and not on dotnet-3.1 or on apache) when enabled your transport package. When disabled kestrel-linux-transport was disabled - error went away.

Same here using this with nginx proxy it seems to fail half the time with a 502 bad gateway error messages.


Missing EmailAddress.php in autoload

Just trying the "Calendar: Retrieve FreeBusyResponse" example from the wiki and the autoload looks to try and run "require_once" for a file "EmailAddress.php" - problem is the letter doesn't exist in the EWSType folder.

I'm pretty sure it should be EWSType_EmailAddressType
Also, 'Address' is now 'EmailAddress'.

This has been resolved on master.


[viogpudo] Customize the INF file

This patch customizes the viogpudo INF file, so it can be configured
by customized Subsystem Vendor ID and device naming for various vendors.
The macros related to the customization are the following,
_INX_SUBSYS_VENDOR_ID
INX_PREFIX_VENDOR
INX_COMPANY
Signed-off-by: Annie Li<EMAIL_ADDRESS>Reviewed-by: Darren Kenny<EMAIL_ADDRESS>

Is there any estimate on when this might be merged into master?
I see a couple of failures in CI, but none of those should be related to this set of changes, or at least I can't see where they are.
is there anything else that needs to be done before a merge is possible?

@darrenkenny Sorry, the PR got lost in the todo list.


deps: use celestia-node v0.6.1, celestia-app v0.11.0

Resolves: #37

Overview

Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

Codecov Report

Merging #38 (26329b7) into main (966e9d4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #38   +/-   ##
=======================================
  Coverage   54.02%   54.02%
=======================================
  Files           2        2
  Lines          87       87
=======================================
  Hits           47       47
  Misses         34       34
  Partials        6        6

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


Problem with utility_meter and sensor with fixed reset value

Home Assistant release with the issue:
0.101.3
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io in Docke
Integration:
https://github.com/home-assistant/home-assistant/tree/dev/homeassistant/components/utility_meter
Description of problem:
I have a problem with the utility_meter component i'm using a sensor with start from 0 to max 65,535  (After 65,535 restart to 0 )  during the restart of sensor i receive  a very strange value into my daily/monthly sensor.
I think the problem it is the logic inside the component to handle only an increasing sensor sensor.

There is something i can do? or there is the possibility to change the code to handle this type of sensor?
Thanks.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
`utility_meter:
  consumo_giornaliero:
    source: energia_gen_energy
    cycle: daily
    tariffs:
      - f1
      - f2`

  consumo_mensile:
    source: energia_gen_energy
    cycle: monthly
    tariffs:
      - f1
      - f2

Traceback (if applicable):

Additional information:

Hi!
What is a strange value ?
The current code skips the rollover. I know that you miss a reading, but many people do not know the value at which the rollover occurs, so I opted for skipping that reading instead of showing a negative readout

Hi @dgomes,
I receive a very high value i don't have a historic graphic to show because i have exclude the recording on this.
For example the first montly value it is very high usually i have from 10K to 50K.

Thanks

282767 is very high for a sensor that rolls over at 65,535 ...
Even considering bit overflow that would be 262140 (still far from 282767)
Can you share the configuration of the source sensor ?

The sensor cam from a PZEM using esphome with custom component to read the UART value.

can you share the custom component (I believe the issue lies there)

I have activate the recording component and i'm lucky because i have received the issue yesterday.
The problem it is two reading at 0 value and after the normal increased value.

At this point i think the problem it is not the reset of counter but some strange reading value at 0.
Custom Component
https://drive.google.com/file/d/1KlMxUf-MDGcmtIdQcubVPOiuQWGiQFR6/view?usp=sharing
I'm trying to improve the dimension of variable to store the total value from PZEM.

Thanks.

Can we then close this issue :) ?

There is the possibility to skip these case? ( some value 0 ) because these case generate an incorrect value into utility meter.

I would use filter https://www.home-assistant.io/integrations/filter/

Thanks a lot


[QA] Other player's avatars are invisible

Explorer Alpha build version:
0.3.250-main-08f208c
Issue Description:

It can happen with different conditions but sometimes we cannot see other avatar around us but we can still see the nametag going around.
Steps to reproduce:
To repro this, we only need to be with more than two players with us in the same place. It can also be checked by teleporting to other place and then back where we were.

During playtests this was not happening anymore.


[Bug]: Handle Redis Configuration Error in Production Build

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
During the production build process (npm run build), an error occurs related to the Redis configuration. Specifically, the error message states that the 'url' property is missing or undefined in the Redis config.
Expected behavior
The production build should proceed without errors related to Redis configuration. It should correctly handle the Redis URL configuration or provide clear instructions on setting up Redis for production environments.
Add ScreenShots
No response
On which device are you experiencing this bug?
Windows
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@iamabhishek2828 Do you wan to work on this?

Yes

@iamabhishek2828 updates?

This issue has been automatically closed because it has been inactive for more than 30 days. If you believe this is still relevant, feel free to reopen it or create a new one. Thank you!

Hello @iamabhishek2828! Your issue #363 has been closed. Thank you for your contribution!


Remove "files" from package.json

If you distribute src folder in the npm package, it's much easier for the editor to generate docs and autocompletion help.
Also with modern build tools like parcel 2 and vite, including autoComplete.js like import autoComplete from<EMAIL_ADDRESS>will allow their JS tools to tree shake, optimize, and compress instead of using babel and rollup.
This is parcel 2 nightly building one of my projects with your current npm package:
dist/www/index.984d4fe7.js                         44.37 KB    1.06s
dist/www/index.8b01772a.js                         10.38 KB    357ms
dist/www/index.39d18735.js                        791.89 KB    5.58s

And this is parcel building from autoComplete.js src directly
dist/www/index.984d4fe7.js                         44.37 KB     57ms
dist/www/index.8b01772a.js                         10.38 KB     15ms
dist/www/index.cd0f409b.js                        787.88 KB    4.69s

This is vite building with current npm package:
dist/assets/index.bb942bdc.js                     66.71kb / brotli: 23.00kb
dist/assets/vendor.38b12ba7.js                    741.66kb / brotli: skipped (large chunk)

And this is vite building with autoComplete.js source directly:
dist/assets/index.7e57cf76.js                     73.48kb / brotli: 24.93kb
dist/assets/vendor.bc244387.js                    731.96kb / brotli: skipped (large chunk)

Hello @folknor,
Thanks for your suggestion, it sounds really interesting and I'll definitely check out that combo after making sure that v10 is stable.
I've included the src folder in the package as per your request in v10.
Cheers and have a nice day! :)


Replace example for generally constrained NLOC

A suggestion for replacing the current example for generally constrained NLOC by a more intuitive one: a point mass moving towards a goal in two-dimensional space while avoiding a circular-shaped obstacle.

@hmcm why not have both? In my opinion I think it would be great to just add another example instead of replace the existing one.

The old example is unfortunately completely meaningless and unintuitive,
this was the motivation why I agreed with hmcm to replace it...
On Sat, Jul 11, 2020, 00:19 acxz<EMAIL_ADDRESS>wrote:

@hmcm https://github.com/hmcm why not have both? In my opinion I think
it would be great to just add another example instead of replace the
existing one.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/ethz-adrl/control-toolbox/pull/149#issuecomment-656918662,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJQVFAJI3JE5IL4RZQYEMDR26HV5ANCNFSM4OWSYH2A
.

I see, sorry for the noise!

Hi @hmcm, I needed to make a few changes ... I did not find a better way than branching your fork once more and making a new PR.. please find it here #155


[BUG] DataLake directory sas generating wrong signature

Generating a sas token for a DataLake Gen2 directory creates an invalid signature.
Using this code:
public StorageDto getSasToken(String folder) {

    // get config values
    StorageSharedKeyCredential credential = StorageSharedKeyCredential.fromConnectionString(storageProperties.getConnectionstring());
    String accountName = storageProperties.getAccountName();
    String containerName = storageProperties.getContainerName();
    String filesystemPath = String.format("https://%s.dfs.core.windows.net/%s/", accountName, containerName);
    String endpoint = filesystemPath + folder;

    // create the DataLakeDirectoryClient
    DataLakeDirectoryClient directoryClient = new DataLakePathClientBuilder()
     .endpoint(endpoint)
     .credential(credential)
     .buildDirectoryClient();

    // set expiration: 1d, permission: read, write, list
    OffsetDateTime expiryTime = OffsetDateTime.now().plusDays(1);
    PathSasPermission pathPermission = new PathSasPermission().setListPermission(true).setWritePermission(true).setReadPermission(true);
    DataLakeServiceSasSignatureValues signature = new DataLakeServiceSasSignatureValues(expiryTime, pathPermission).setStartTime(OffsetDateTime.now());

    // generate SAS Token
    String sasToken = directoryClient.generateSas(signature);

    StorageDto response = new StorageDto(filesystemPath, sasToken);

    return response;
  }

generates the following SAS Uri:
https://***.dfs.core.windows.net/ingestion-test/a709b091-b657-473d-8024-4a5b97aec20b/2020/12/4/?sv=2020-02-10&se=2020-12-05T01%3A37%3A29Z&sr=d&sp=rwl&sdd=4&sig=VhgwjxEhOBbzENbBfTH%2Fsn%2Bus%2B7Y08AJtjT0vO4pX%2Fs%3D
where "Ingestion-test" is the container name
and "a709b091-b657-473d-8024-4a5b97aec20b/2020/12/4" is the folder name.
When using azcopy
azcopy cp .\README.md "https://shopchainmanualdatalake.dfs.core.windows.net/ingestion-test/a709b091-b657-473d-8024-4a5b97aec20b/2020/12/4/?sv=2020-02-10&se=2020-12-05T01%3A37%3A29Z&sr=d&sp=rwl&sdd=4&sig=XXXXXX"
I get the following information:
===== RESPONSE ERROR (ServiceCode=AuthenticationFailed) =====
Description=403 Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature., Details: (none)
Using the .NET code in this doc: https://docs.microsoft.com/en-us/azure/storage/blobs/sas-service-create?tabs=dotnet#create-a-service-sas-for-a-directory  I get a very similar token (only the signature is different):
https://***.dfs.core.windows.net/ingestion-test/a709b091-b657-473d-8024-4a5b97aec20b/2020/12/4?sv=2020-02-10&se=2020-12-04T03%3A12%3A32Z&sr=d&sp=rwl&sdd=4&sig=P%2BFc9vVVX6t1odOzx1OHj%2F3cQqwgAih9UhBLV%2BlRtPY%3D
This one works on AzCopy as expected.
Expectation:
To create a valid SAS token
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

Found out that the issue only occurs when trying to generate SAS for a folder or file inside a folder.
If I change the "folder" parameter to be something like "test.txt" it works, I get the issue if I do "test/test,txt"
Starting to think this is related to encoding of the path. I tried passing in the URLEncoder.encode result, but I get the same issue

Found out the issue only happens when we try to generate a sas for a file inside a folder.
public StorageDto getSasToken() {
  try {
      // get config values
      StorageSharedKeyCredential credential = StorageSharedKeyCredential
          .fromConnectionString(storageProperties.getConnectionstring());
      String accountName = storageProperties.getAccountName();
      String containerName = storageProperties.getContainerName();

      String accountUrl = String.format("https://%s.dfs.core.windows.net", accountName);

      //this one works
      //String path = URLEncoder.encode("test.txt", "UTF-8");
      //this one doesn't
      String path = URLEncoder.encode("test/test.txt", "UTF-8");

      // create the DataLakeDirectoryClient
      DataLakeDirectoryClient directoryClient = new DataLakePathClientBuilder()
        .endpoint(accountUrl)
        .credential(credential)
        .fileSystemName(containerName)
        .pathName(path)
        .buildDirectoryClient();

      // set expiration: 1h, permission: read, write, list
      OffsetDateTime expiryTime = OffsetDateTime.now().plusDays(1);
      PathSasPermission pathPermission = new PathSasPermission().setListPermission(true).setWritePermission(true)
          .setReadPermission(true);
      DataLakeServiceSasSignatureValues signature = new DataLakeServiceSasSignatureValues(expiryTime, pathPermission)
          .setStartTime(OffsetDateTime.now());

      // generate SAS Token
      String sasToken = directoryClient.generateSas(signature);

      StorageDto response = new StorageDto(directoryClient.getDirectoryUrl(), sasToken);

      return response;

    } catch (Exception ex) {
      return null;
    }
  }

It works when I generate a sas for "test.txt", the signature is considered invalid if I generate the sas for "test/test.txt"

@gapra-msft Can you please take a look at this one?

@QuickNS,
Thank you for reporting this issue. I will investigate and get back to you.

@QuickNS
I just noticed that you are using a DirectoryClient to point to a file, Could you use DatalakeFileClient instead and try generating a SAS off that? If you don't use a DatalakeFileClient, we can't distinguish between a subdirectory and a file.

If you still get the error, could you please provide the String to Sign generated by Java and .NET? You can do this by walking down the generateSas stack and placing a breakpoint where the stringToSign method is called.

@QuickNS It looks like you accidentally pasted the whole SAS token without erasing the signature. Just wanted to point that out so you can roll your creds.

please reactivate if you are still hitting this issue.

please reactivate if you are still hitting this issue.


How to veritfy a contract on the mainnet is exactly follows the source code the author open sourced on github?

I use get abi and get code to retrive a contract from the mainnet,
and i found the source code the author announced open sourced on github
how to verify the contract and source code's implementation is identical?

+1

If you compile the source code and hash it using hash256, the hash should be the same as the one you get with cleos get code


Cancelling ClickContainerEvent.Primary removes the item in the clicked slot

This PR is aimed to fix the issue #3862
Tested with SpongeAPI 10.0.0-SNAPSHOT and SpongeVanilla 1.19.4-10.0.0-RC0.
The bug
When the ClickContainerEvent.Primary event is cancelled, the first click on an item is cancelled, but a second click causes the item to disappear. It's not a visual bug, leaving and then reconnecting to the server will not make the item to reappear.
Why this bug happens
During the restore of the container after the first click, many packets are sent to the client to replace the item in the slot and remove the item in the cursor. At the end, the stateId of the container will be 4. But during the second click, the client will send a stateId of 3, leading to the execution of the method AbstractContainerMenu.broadcastFullState instead of AbstractContainerMenu#broadcastChanges, where the items in the container are set without transactions.
This difference of stateId is due to the fact that after the restoration, the stateId of the container is 3, but the tick system will detect an inconsistency between the "carried" item and the "remote carried" item. It will fix it and then send a packet with a stateId of 4 to tell to the client that the "carried" item has been updated. But it has already received the updated item through the cursor restore, so I think that it ignores the packet and so keep the stateId of the container to 3.
The fix
The fix simply sets the "remote carried" and not only the "carried" item during the cursor restore to avoid the inconsistency. A test plugin has been made that cancels the event ClickContainerEvent.Primary to see the bug and that this PR fixes it.

This fix reveals the existence of an another bug: if you spam click 3 times, the event will not be cancelled. This can be used to duplicate items. This problem is fairly the same as this one: the difference of stateId between the client and the server.
After spam clicking 3 times, the client sends not 3 but 4 "click container" packets. The third and fourth packets have the same stateId, whereas the server has updated the container after the third packet and so the stateId.
After spam clicking 2 times, the client sends not 2 but 3 "click container" packets, but this time, the stateId is correct. That's why the item is moved 3 times instead of 2.
I don't know why this happens, if it's a normal behaviour, which happens after some conditions or not, and if I'm the only one who he's affected. I'm still investigating.

I fixed the bug mentioned above.
I don't know if you are ok with the modification of TrackerContainerBridge and the other methods, tell me if you prefer an another way.

Modifications made !


Fetch followed artists by album

A new endpoint to fetch all followed artists by getting liked albums from Spotify. A list of fetchTypes is required to tell which way of importing is requested. I chose not to use enums because I didn't want to have to handle parsing errors here. If a type does not match, just nothing is returned. I duplicated the return type to avoid confusion with the other method. The other one will be removed sooner or later anyway.
Only those artists are returned that the user does not follow yet. This makes the list smaller if the import is triggered again later.

 Artist import endpoints


Resizeable indicator subcharts

I'm submitting a...

[X] feature
[ ] chore
[ ] bug

What is the current behavior
Subcharts have fixed heights and can't be resized by user.
What is the expected behavior
Users being able to resize subcharts by dragging the edges.

any updates on this? I'm using split-pane-react component which handles this.


Expand FC.Util.Extent to a component

Thought it would be nice to have a little more flexibility with the Extent function.
If we created it as component could do something like:
fc.util.extent()
    .includeZero()
    .scale(1.1)(data, ['high', low']);

Like it!

I've wanted something similar ... on a few occasions I've found myself doing something like this:
fc.util.extent(series, function(d) { return 0; }, 'y')

(Notice you do not need to pass an array due to https://github.com/ScottLogic/d3fc/issues/468)
What if extent supported the following:
fc.util.extent(series, 0, 'y')

i.e. if a number is passed, convert to function() { return number; }

passing a number would certainly cover the 0 case, guess felt we got a little more flexibility if we had a component model.
Could then do thing like roundTo(...)

I prefer the idea of a component over heavily overloaded method arguments. Maybe you could take a stab at the API for the component?

yep happy to

An API that allows for the addition of range padding would be useful.

Think @owennw is going to have a go at creating this as first component for him.

I've implemented a function on extent that accepts a number and ensures that is it in the range, making the feature more flexible than just including 0. At the moment it's called "include".
What other features might we want on the extent? For example:
Scale(number): scales the max value up by the input, and the min value down by the same proportion? This wouldn't do anything to 0 though.

I think scale should magnify the range around the average point.

How do we want include and scale to operate together?
For example:
Calculated range is [10, 20], and 30 is included.
The scale is 0.8.
Do we expect the final range to be [11, 30] or [12, 28] (or even anything else)?

I'd be careful using the word scale, it's quite a loaded term in d3. Is
there a good use case for scaling with a value < 1? Does that therefore
suggest a different name?
On Thu, 22 Oct 2015 at 17:53 Nick Owen<EMAIL_ADDRESS>wrote:

How do we want include and scale to operate together?
For example:
Calculated range is [10, 20], and 30 is included.
The scale is 0.8.
Do we expect the final range to be [11, 30] or [12, 28] (or even anything
else)?
—
Reply to this email directly or view it on GitHub
https://github.com/ScottLogic/d3fc/issues/646#issuecomment-150289369.

I'd opt for something along the lines of padding or margin, basically a factor that increase the max and min by a given amount

I don't know if there's a good case for shrinking the range, but as it stands that certainly will be possible. Of course I could also reject inputs < 1.
Padding / margin have their own loaded terms too (despite my best efforts I always forget which is which, too). Would we rather have the min/max of the range increase by a fixed amount as input (e.g. .padding(10) would increase either side by either 5 or 10), or a more complicated function that 'scales' the range by a proportion (e.g. `.scale(1.2) increases the range by 10% on either end). Or both??

I think we should only allow values greater than 0 for the to be names scale function.
I think it fits with trusting the program to allow 0 to 1 where they are trying to hide sections of the range.
Precedence-wise I would scale the range and then verify the included points are within the range. For example if we had include(0) would definitely want it even if shrank range.
Name-wise how about proportion?

There's quite a lot of precedence for this in d3's ordinal scales, maybe it's worth looking over their API as well?

ah, they call it padding through out.

I was going to mention the same idea, do a bit of research. What term do other charting libraries use for this concept? I think 'padding' is quite a common term.
Also, regarding units, it would be great if we could allow for percent and 'domain' units. For example, pad(100) or padPercent(0.1).
However, what happens if you include zero and apply a padding? things start to get complicated! Should this support asymmetrical padding?

Given they call it padding throughout I've implemented the function as .pad(number), where currently the padding will occur, and then the included number is applied.
If desired percent and domain units pad functions can also be implemented -- perhaps in this issue. As for asymmetrical padding, maybe that's worth having a think about and implementing in a later issue?

Summary of discussion:
The pad function can be described by the following examples:
[10, 20].pad(0) -> [10, 20]
[10, 20].pad(1) -> [5, 15]
Whatever drops out of the calculation when pad accepts a negative number is okay.

@jdunkerley can we close that?

I have implemented asymmetrical padding in this function in d3fc-showcase. https://github.com/ScottLogic/d3fc-showcase/blob/develop/src/assets/js/util/domain/padYDomain.js

When calling this with a padding percentage array such as [-0.1, 0.1], we shift the domain by 10%. Could/ should this be span out into a seperate call on fc.util.extent(), or can a user be trusted to figure this out for themselves?


tests: tests/kernel/fatal/exception/ test case fail

Describe the bug
tests/kernel/fatal/exception/ failed in daily test.
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using? it8xxx2_evb
Is this a regression? Yes
specific commit? 77e1e39cfff80c12ac3b0b1b802eee9e61ae79c4
...

To Reproduce
Steps to reproduce the behavior:
west build -p always -b it8xxx2_evb tests/kernel/fatal/exception/ -DCONFIG_STACK_SENTINEL=y -DCONFIG_TEST_HW_STACK_PROTECTION=n -DCONFIG_HW_STACK_PROTECTION=n -DCONFIG_ZTEST=y -DCONFIG_COVERAGE=n -DCONFIG_MP_NUM_CPUS=1 -DCONFIG_ZTEST_NEW_API=y
OR
twister  -p it8xxx2_evb --device-testing --west-flash="../../itetool/loader.sh" --device-serial=/dev/ttyUSB0 -T tests/kernel/fatal/exception/ -vv
Expected behavior
The test can pass
Impact
1 test case fail in daily test
Logs and console output
*** Booting Zephyr OS build zephyr-v3.1.0-3392-g77e1e39cfff8  (delayed boot 1ms) ***
Running TESTSUITE fatal_exception
===================================================================
START - test_fatal
test alt thread 1: generic CPU exception
E: Spurious interrupt detected! IRQ: 11
E: >>> ZEPHYR FATAL ERROR 1: Unhandled interrupt on CPU 0
E: Current thread: 0x80101048 (unknown)
Caught system error -- reason 1
Wrong crash type got 1 expected 0

Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows): Ubuntu20.04
Toolchain (e.g Zephyr SDK, ...) 0.14.1
Commit SHA or Version used: 77e1e39cfff80c12ac3b0b1b802eee9e61ae79c4

In entry_cpu_exception(), the test code runs an illegal instruction from RAM section to trigger an expection on RISCV architecture.
Since IT8xxx2 series run program from flash (text section) directly, run code from RAM section will cause an unknown result.
Can the test code be changed like this?
void entry_cpu_exception(void *p1, void *p2, void *p3)
{
	expected_reason = K_ERR_CPU_EXCEPTION;

#if defined(CONFIG_X86)
	__asm__ volatile ("ud2");
#elif defined(CONFIG_NIOS2)
	__asm__ volatile ("trap");
#elif defined(CONFIG_ARC)
	__asm__ volatile ("swi");
#elif defined(CONFIG_RISCV)
	/* illegal instruction on RISCV */
	__asm__ volatile (".long 0");
#else
	/* Triggers usage fault on ARM, illegal instruction on
	 * xtensa, TLB exception (instruction fetch) on MIPS.
	 */
	{
		volatile long illegal = 0;
		((void(*)(void))&illegal)();
	}
#endif
	rv = TC_FAIL;
}

Looks good to me
The patch can fix the problem on it8xxx2_evb.
Could you please provide a PR?


Support for updated Inquisitor Ascendancy node: Sanctuary

In 3.6.0, the Inquisitor Ascendancy node "Sanctify" was renamed "Sanctuary" and reworked.
This adds support for the new mod "Consecrated Ground you create applies 10% increased Damage taken to Enemies"
From poe wiki:
Renamed from Sanctify and reworked.
Old:
30% chance to create Consecrated Ground when Hit, lasting 8 seconds
30% chance to create Consecrated Ground on Kill, lasting 8 seconds
10% chance to create Consecrated Ground when you Hit a Rare or Unique Enemy, lasting 8 seconds
Consecrated Ground you create grants 40% increased Damage to you and your Allies

Now:
You have Consecrated Ground around you while stationary
Consecrated Ground you create applies 10% increased Damage taken to Enemies
15 Mana Regenerated per Second while on Consecrated Ground

closing, this wasn't quite ready yet.


Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1770":https://sourceforge.net/p/geneontology/ontology-requests/1770

thanks www.movied.org


Add edit task feature, test cases and update UG

Codecov Report

Merging #86 (a3efc17) into master (229c762) will increase coverage by 1.01%.
The diff coverage is 84.78%.

@@             Coverage Diff              @@
##             master      #86      +/-   ##
============================================
+ Coverage     67.04%   68.05%   +1.01%
- Complexity      464      490      +26
============================================
  Files            90       92       +2
  Lines          1605     1700      +95
  Branches        162      179      +17
============================================
+ Hits           1076     1157      +81
- Misses          483      486       +3
- Partials         46       57      +11

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
69.56% <0.00%> (-3.17%)
:arrow_down:

src/main/java/seedu/address/model/task/Task.java
66.66% <25.00%> (+4.16%)
:arrow_up:

.../seedu/address/logic/commands/EditTaskCommand.java
85.71% <85.71%> (ø)

...du/address/logic/parser/EditTaskCommandParser.java
92.30% <92.30%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <100.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
100.00% <100.00%> (ø)

src/main/java/seedu/address/model/TaskList.java
74.28% <100.00%> (+1.55%)
:arrow_up:

...java/seedu/address/logic/commands/ViewCommand.java
73.68% <0.00%> (-13.82%)
: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 229c762...a3efc17. Read the comment docs.

LGTM


Llama3.2-11b-Vision

llama.cpp state: https://github.com/ggerganov/llama.cpp/issues/9643

@hahuyhoang411 @vansangpfiev I'd like to take next sprint to see if we can contribute Vision back to llama.cpp, perhaps by building on top of @ngxson's PR here:
https://github.com/ggerganov/llama.cpp/pull/9687
We need an extensible architecture on top of llama.cpp for multi-modality, especially for Ichigo (e.g. audio encoder)


new Warning! (install to release UNA-10.1.0) + modules of una-master 26.11.2019

-Если отключить Timeline то Предупреждения не будет
-If you disable the Timeline Warnings will not

public function getCode ()
{
$sCode = parent::getCode();
if($sCode !== false)
$sCode = $this->_oTemplate->parseHtmlByContent($sCode, array(
'id' => $this->_aObject['menu_id']
));
    return $sCode;
}

It's a bad idea to install old (even if it's the last released) version of main script and the newest DEV versions of modules.
Could you please test issus (bugs) on ci.una.io/test before creating an issue here.

Current version of main script already has necessary check in constructor of BxBaseMenuInteractive class:
if(empty($this->_aObject['menu_id']))
    $this->_aObject['menu_id'] = $this->_sObject;


gateway-api: Fix flaky conformance tests

Description
The goal is to have unit tests for Conformance test cases for faster feedback and stability. As part of this PR, two flaky tests are fixed :crossed_fingers:.
Testing
Testing was done in https://github.com/cilium/cilium/actions/runs/4402111837, which are having 20+ consecutive successful runs.
gateway-api: Fix flaky conformance tests

Full CI is not required, as the code changes are related to Gateway API/Ingress/Unit tests, which are verified as part of GHA.

/test

/test-1.16-4.19

/mlh new-flake Cilium-PR-K8s-1.16-kernel-4.19

Above failure is fixed in https://github.com/cilium/cilium/pull/24336, which is just merged. Re-run is not required.


Empty synonym list affects entity defined below it

Rasa version: 1.1.8
Python version: 3.6
Operating system (windows, osx, ...): Ubuntu (with WSL)
Issue:
When a synonym section is defined in the NLU training file without any synonyms in the list, the entity belows it gets modified by the synonym.
In nlu.md:
## synonym:alone

## entity:animal
- fish
- cat
- dog
...

When I extract the animal entity, it always takes the value alone.
It's obviously an easy fix for us but I found it curious that it would do this (our nlu.md file being generated from the database, it was quite surprising to see this).
Error (including full traceback):
N/A
Command or request that led to error:
N/A

Thanks for the issue, @amn41 will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

duplicate of https://github.com/RasaHQ/rasa/issues/4304


Help with setup, not converting files to mp4

Hi,
Can I get some help with the install please, I am using Sickrage
https://imgur.com/jd9P7A5
This is an image showing it running the post processing and the script
https://i.imgur.com/ObIwQLo.png
Image showing the file still an .mkv
https://pastebin.com/yAZRTYLd
Edit
I've now added
[MP4]
ffmpeg = /usr/bin/ffmpeg
ffprobe = /usr/bin/ffprobe
and also installed apt-get install ffmpeg -y
This is myt auto Process.ini, I've left it default other than setting the api key for sickrage, do I remove all the other settings I am not using?
In the post processing for sickrage for extra scripts I have /usr/bin/python /sickbeard_mp4_automator/postConversion.py
To install I used the following commands
40  pip install requests
41  apt install python-pip
42  pip install requests
43  pip install --upgrade pip
44  pip install requests[security]
45  pip install requests-cache
46  pip install babelfish
47  pip install "guessit<2"
48  pip install "subliminal<2"
49  pip install stevedore==1.19.1
50  pip install python-dateutil
51  pip install qtfaststart
67  git clone https://github.com/mdhiggins/sickbeard_mp4_automator/
68  ls
69  cd sickbeard_mp4_automator/
70  ls
71  mv autoProcess.ini.sample autoProcess.ini
72  ls
73  vi autoProcess.ini
running the process manually worked
https://i.imgur.com/1jaH18r.png
Permissions on the files and folders
https://i.imgur.com/QJnYsJC.png
Possible permissions issue?

Thanks for your very thorough documentation, its refreshing.
Your setup process looks correct as does your settings, and the fact that Sickrage is attempting to trigger the script suggests that's set up correctly.
I think the next step will be to check info.log after running a Sickrage processing attempt and see if any errors are present that may provide further information
Additionally one more thing to try would be to remove the python path from the Sickrage post processing settings and only point it to the postConversion.py file. Windows tends to need the python path but I would think linux would recognized and respect the shebang and execute the script without the path to the python executable. Perhaps that extra component in the path is causing issue.

So I made a few changes, I set permissions to that whole folder to sickrage and the sickrage group, I also made the change you suggested of removing the python path. What I will do is set the permissions back to root and see if it was just the pyton path.

I'm guessing that change made it work?

Yeah it is working fine noiw, thanks.


in .history(period = 'max') funtion I got this error: AttributeError: 'Index' object has no attribute 'tz_localize'

Hi,
how can I fix the below error?
Code:
google = yf.Ticker("GOOG")
google_data = google.history(period="max")
Error:
--> 279 df.index = df.index.tz_localize("UTC").tz_convert(
280     data["chart"]["result"][0]["meta"]["exchangeTimezoneName"])
282 if params["interval"][-1] == "m":
283     df.index.name = "Datetime"
AttributeError: 'Index' object has no attribute 'tz_localize'
Thanks in advice

Hi, you need to add a line of code to remove the timezone-sensitive DataFrame index
google = yf.Ticker("GOOG")
google_data = google.history(period="max")
google_data.index = google_data.index.tz_localize(None) # Remove timezone

@PatrickDionne That won't fix errors originating inside yfinance
@1uislopez What version are you using? I can't find that code anywhere.

@PatrickDionne That won't fix errors originating inside yfinance
@1uislopez What version are you using? I can't find that code. But I've seen error before and potential fix is in branch fix/prices-index-not-datetime for you to test (instructions #1080)

Sorry I can not find the branch fix/prices-index-not-datetime

I closed this and deleted branch because I considered issue fixed. Create a new issue.

I closed this and deleted branch because I considered issue fixed. Create a new issue.

Oh thx! But I fix it with install pandas at version 1.3.5 (instead of 2.2.2)


pwm: pwm_nrf5_sw: Add prescaler support and use it in the servo_motor sample

These commits fix the inaccuracies regarding PWM frequency and pulse width in the current pwm_nrf5_sw implementation, adds support for specifying a HF timer prescaler value and finally uses the prescaler to enable use of the servo_motor sample on the BBC micro:bit.

Codecov Report

Merging #8068 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #8068      +/-   ##
==========================================
- Coverage   64.54%   64.54%   -0.01%
==========================================
  Files         420      420
  Lines       40131    40131
  Branches     6759     6759
==========================================
- Hits        25903    25902       -1
  Misses      11106    11106
- Partials     3122     3123       +1

Impacted Files
Coverage Δ

boards/posix/native_posix/timer_model.c
98.21% <0%> (-1.79%)
: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 8bbf0fd...b49a475. Read the comment docs.

you'll need to fix the commit message title (too long)

@dbkinder Thank you for reviewing. Which commit has the too long title?

This is an alternative solution to #7474 which is more in line with the proposed addition of the nrfx PWM HW driver in #7334.

@cvinayak Thank, fixed.

我的本意是修复 period_cycles  大于 0xffff 时会引发的bug，在这里，我认为应当在period_cycles大于0xffff 时返回错误
From Google Translate
My intention is to fix the bug that is caused when period_cycles is greater than 0xffff. Here, I think it should return an error when period_cycles is greater than 0xffff.

@findlayfeng Good point, that should be addressed as well. Not sure I understand your math in #7474 regarding this, but perhaps it can just be rebased to address the overflow issue once this has been merged?

@carlescufi please do the needful.

@jhedberg hope you can check the mesh demo buzzer has no regression.

@jhedberg hope you can check the mesh demo buzzer has no regression.

@cvinayak I tested with samples/boards/microbit/sound/ - seems to work fine.


Support custom scrollable container

It solves #97 issue.

hi @borowskt
can you please add tests?

Looks like it should accept HTMLElement | string instead of just string. With ng2 we have a lot of ways to proper element referencing and original ngInfiniteScroll infinite-scroll-container input works this way.

hi @borowskt
I made a major update to the code - so the branch has now conflicts.

Hello @borowskt, @orizens
Do you have any plans to resolve conflicts and merge functionality?

Lot's of work lately :) I will finish it on Monday
On Thu, Jan 26, 2017 at 12:24 PM, Pavel<EMAIL_ADDRESS>wrote:

Hello @borowskt https://github.com/borowskt, @orizens
https://github.com/orizens
Do you have any plans to resolve conflicts and merge functionality?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/orizens/angular2-infinite-scroll/pull/108#issuecomment-275366421,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQDyikY3U-mQMJIugEI4dfouSC9AVu0yks5rWIJ2gaJpZM4LeLRf
.

--
Tomasz Borowski
Software Development Specialist
Confidentiality Note: This message is intended only for the use of the
named recipient(s) and may contain confidential and/or proprietary
information. If you are not the intended recipient, please contact the
sender and delete this message. Any unauthorized use of the information
contained in this message is prohibited.
*Informacja o poufności: *Treść tej wiadomości zawiera informacje
przeznaczone tylko dla adresata. Jeżeli nie jesteście Państwo jej
adresatem, bądź otrzymaliście ją przez pomyłkę, prosimy o powiadomienie o
tym nadawcy oraz trwałe jej usunięcie. Wszelkie nieuprawnione wykorzystanie
informacji zawartych w tej wiadomości jest zabronione.

@pavelalexeenko @borowskt
can you please share a use case where this functionality is useful? a plunkr is mostly welcomed.

@orizens
I have a toolbar and router-outlet under, and an infinite-scroll in this router-outlet.  So I want to make it possible to scroll list using main window scroll, but when I use infinite-scroll in router-outlet, it doesn't work. Works fine, if I render my infinite-scroll for a whole page.
Can't get this using @Hostlistener().

@orizens
I finally found my problem (may be it isn't even connected with current pull request). In root component I have md-sidenav-layout from Material2, and when i use fullscreen directive, my child components don't trigger scroll events.
Sorry for disturbing.

@orizens
Branch rebased, testes added.
In my case I had scrollable element defined outside router-outlet which injects subpages that require infinity scroll.

@borowskt  Thanks - will review and comment.

@borowskt
Thanks for this pr - I releases version 0.3.1.
can you take the time and create a plunkr example for this feature?

I'm interested as well in a plunkr, i can't manage to make it work with angular/material2, a sidenav and a router-outlet


Add support for functional easing

For more control, you can give it an ease parameter, this will override the launch and land, and instead use that for smooth easing
You can give it either a function, or a case-insensitive string with any of the names found at https://easings.net/
If you dont specify an ease parameter, or provide a falsy value, it will default to using the launch and land parameters

Hey by the way - I haven't forgotten this! I'm just using all my time up on finishing the next video currently. Will get around to testing + pulling as soon as I've done the vid! :)

Cool, I've pulled it! Seems to work well!
I have some thoughts on how we could make it even better + simpler to use! I'll make an issue!


feat: common provider for the events system

common provider for events system

feat: New feature (non-breaking change which adds functionality)

TODO: need to discuss the best way to confirm provider is enabled before calling any of the events functions.

overriding methods? -  code still will be duplicated
a proxy function which calls enable before calling any events call?

Checklist

[X] Self-review of changed code
[X] Manual testing
[X] Added automated tests where applicable
[X] Update Docs & Guides
For UI-related changes
[X] Darkmode
[X] Responsive layout

@pavanjoshi914 awesome! can we move the enabled / enable() functionality there too?


Create a model for Game

Task finished, pull request opened #6

Task finished, pull request opened #14


Error "Invalid Host header" behind traefik-reverse-proxy

When I try to make the container accessible via traefik (https) I get the error "Invalid Host header". how can I solve this?

I passed  "- DANGEROUSLY_DISABLE_HOST_CHECK=true" as an environment variable in the docker-compose file. That did the trick

I passed "- DANGEROUSLY_DISABLE_HOST_CHECK=true" as an environment variable in the docker-compose file. That did the trick

This worked for me. Thank you


Note options: Some buttons are cut off

For longer notes there's not enough space to display the lower row of option buttons (copy/move/delete), cause the options drawer doesn't scroll.

This should be fixed in version 2.1.3.

Scrolls now. Phew.


Improve typing of filters

Currently we return _FilterValue, _FilterAnd and _FilterOr from various filter functions. This makes it a type error when using &=and |= which works without problems. Instead all of these functions should return _Filters (which is the baseclass for all the other Filtertypes). The exact subtype is only relevant internally and not for users as all of them can be freely combined

Implemented in https://github.com/weaviate/weaviate-python-client/pull/973


document is not defined

when I use it in node.js，run cv.imread() ，it prompts that the document is not defined。

how you fix this issue??

i get this same issue, how did you fix?


Retry with Power off if graceful shutdown times out

Proposed Changes

I observed that in case Server is stuck in a failed PXE Boot or in other non-responsive state, redfish.GracefulShutdownResetType would not power off the system.
So we can retry with redfish.PushPowerButtonResetType if above fails.

hmm, need to revisit. For Dell it has worked, but on Lenovo I get:
2024-09-17T12:56:20.219583196Z 2024-09-17T12:56:20Z	ERROR	Reconciler error	{"controller": "server", "controllerGroup": "metal.ironcore.dev", "controllerKind": "Server", "Server": {"name":"node006-bb00-system-0"}, "namespace": "", "name": "node006-bb00-system-0", "reconcileID": "1e290486-c86f-4df4-90a5-5c55ab438ae1", "error": "failed to ensure server state transition: failed to ensure server power state: failed to power off server: failed to reset system to power on state: reset type 'PushPowerButton' is not supported by this service"}

lgtm as well.
we could also use metal.ironcore.dev/operation annotation to force a power off. But maybe it is good that this is happening automatically.

lgtm as well. we could also use metal.ironcore.dev/operation annotation to force a power off. But maybe it is good that this is happening automatically.

good point. I'd see metal.ironcore.dev/operation annotation as a troubleshooting tool to perform the operation and also ability to force power off for users that will not use this --enforce-power-off option.


What license governs jquery-linkify?

Hi @uudashr .  I've noticed that you have licensed several other projects under the MIT License but cannot find any licensing information for jquery-linkify.  What license applies to this project?  Thanks very much.

I might put Apache License 2.0

Thank you @uudashr .

@uudashr  Following up on our earlier chat, we're interested in using jquery-linkify but need a clearly-defined license to do so, and as it happens we use Apache 2.0 ourselves.  Would you agree to license the package under Apache 2.0?

Hi, I just added the license to the master

That's great -- thank you, @uudashr


Update changelog for release v0.0.2

Updates the changelog for release

Changelog needs to be added at the top of the previous one. Check the boa changelog as an example


Run npm audit fix

There are still Component Governance bugs for this repo, this is the result of running npm audit fix on these 2 files.

Closing after offline discussion


Freeze/Crashing with MultithreadScheduler

Holoscan 0.6
AGX Orin 64GB igpu
Jetpack 5.1.1
Docker
Python
Experiencing a frozen state or crash when switching to the MulitThreadScheduler.  In our application this will occur sometime between 15 minutes - 3 hours.  To recreated add the MultiThreadScheduler to a Holoscan or Holohub example.
On that fails particularly fast is holoscan-sdk/examples/holoviz/python/holoviz_geometry.py.
I added:
scheduler = MultiThreadScheduler(
      app,
      worker_thread_number=4,
      stop_on_deadlock=True,
      stop_on_deadlock_timeout=500,
      name="multithread_scheduler",
  )
  app.scheduler(scheduler)

Result:
Vizualizer stops but the application is still running.  When attempting to close the application it will attempt stop the scheduler and freeze again.
^C[info] [multi_thread_scheduler.cpp:607] Stopping multithread scheduler
[info] [multi_thread_scheduler.cpp:664] Stopping all async jobs
[info] [multi_thread_scheduler.cpp:147] Dispatcher thread has no more jobs to check
[info] [multi_thread_scheduler.cpp:206] Dispatcher thread has stopped checking jobs
[info] [multi_thread_scheduler.cpp:649] Waiting to join all async threads
[info] [multi_thread_scheduler.cpp:298] Worker Thread [pool name: default_pool, thread uid: 0] exiting.
[info] [multi_thread_scheduler.cpp:348] Event handler thread exiting.
^C^C

[2024-01-09 13:00:48.413][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 1, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 2
[2024-01-09 13:00:48.413][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 358236160, last_no_stop_ts_:357339680, should_stop: 0
[2024-01-09 13:00:48.413][580][holoscan][debug][entity_executor.cpp:644] [C00027] tick codelet holoviz
[2024-01-09 13:00:48.415][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 2, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 1
[2024-01-09 13:00:48.415][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 359982272, last_no_stop_ts_:358236160, should_stop: 0
[2024-01-09 13:00:48.415][583][holoscan][debug][entity_executor.cpp:644] [C00021] tick codelet geometry_layer_object_generator
[2024-01-09 13:00:48.416][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 1, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 2
[2024-01-09 13:00:48.416][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 361540128, last_no_stop_ts_:359982272, should_stop: 0
[2024-01-09 13:00:48.417][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 2, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 1
[2024-01-09 13:00:48.417][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 362264416, last_no_stop_ts_:361540128, should_stop: 0
[2024-01-09 13:00:48.417][582][holoscan][debug][entity_executor.cpp:644] [C00010] tick codelet replayer
[2024-01-09 13:00:48.418][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 1, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 2
[2024-01-09 13:00:48.418][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 363636736, last_no_stop_ts_:362264416, should_stop: 0
[2024-01-09 13:00:48.421][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 1, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 2
[2024-01-09 13:00:48.421][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 366478880, last_no_stop_ts_:363636736, should_stop: 0
[2024-01-09 13:00:48.422][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 0, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 3
[2024-01-09 13:00:48.422][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 367022752, last_no_stop_ts_:366478880, should_stop: 1
[2024-01-09 13:00:48.422][578][holoscan][debug][multi_thread_scheduler.cpp:708] Onhold trend to stop on deadlock for [0] ms
[2024-01-09 13:00:48.423][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 1, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 2
[2024-01-09 13:00:48.423][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 368563488, last_no_stop_ts_:366478880, should_stop: 0
[2024-01-09 13:00:48.423][581][holoscan][debug][entity_executor.cpp:644] [C00027] tick codelet holoviz
[2024-01-09 13:00:48.426][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 2, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 1
[2024-01-09 13:00:48.426][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 371426560, last_no_stop_ts_:368563488, should_stop: 0
[2024-01-09 13:00:48.426][583][holoscan][debug][entity_executor.cpp:644] [C00021] tick codelet geometry_layer_object_generator
[2024-01-09 13:00:48.427][578][holoscan][debug][multi_thread_scheduler.cpp:382] ready_count_: 3, wait_time_count_: 0, wait_event_count_: 0, wait_count_: 0
[2024-01-09 13:00:48.427][578][holoscan][debug][multi_thread_scheduler.cpp:681] timeout: 500, now: 371957024, last_no_stop_ts_:371426560, should_stop: 0
[2024-01-09 13:00:48.427][580][holoscan][debug][entity_executor.cpp:644] [C00010] tick codelet replayer

Hi @genevanmeter,
I think we may have already resolved this issue in our v1.0 development branch. I will try running that example with the scheduler modification you suggest and see if I can reproduce the issue locally.
Do you happen to know if this issue is only seen for the Python version of the app? I ask because one fix that was merged for 1.0 was specifically related to a potential deadlock in multi-threaded Python applications due to an acquisition of Python's GIL during tensor object deletion. If you only see the issue for the Python version of the example, that is likely the cause. Unfortunately, I don't have a good workaround for that GIL-related issue for release 0.6.

@grlee77  Thank you.  We are eagerly waiting for the possible fix.  Our application only uses Python operators so we can't compare to C++.  Our workaround has been to revert back to greedy scheduler and change/remove operators to optimize for the performance we need.

I confirmed just now that on the release branch for 0.6, holoviz_geometry.py modified to use the multi-thread scheduler deadlocks almost immediately. On the current 1.0 internal dev branch, I left the app running for almost two hours before closing it and there were no issues.
(I did this testing on an x86_64 workstation rather than on AGX hardware, but don't think that will make a difference in this case)

Thank you testing. I would gladly test our AGX Orin if I could.

Just posting here to confirm the fix for this is in release v1.0.3.
The relevant entry of the bugs fixed section of the release notes:
4293741 	Python application with more than two operators (mixed use of pure Python operator and operator wrapping C++ operator), using MultiThreadScheduler (including distributed app) and sending Python tensor can deadlock at runtime.

Thanks for confirming @grlee77 . Closing this issue as resolved in the v1.0.3 release.


Drop the Matplotlib-based viewer

Drops the old, deprecated code for the Matplotlib-based viewer. This is merely an alias to mplview left in to aid people migrating to mplview, which is better documented, tested, and cleaned up after the refactor. The workflow itself already requires mplview and uses that directly. Also more than a sufficient amount of time has passed since the deprecation. Hence this goes ahead and drops that code and the associated requirements. Users should be able to easily get mplview themselves should they need it.

Appears matplotlib is still being installed. Likely because scikit-image requires it.

There's also a bit of code in the nanshe/nanshe Docker image, which will need to be shifted to wherever matplotlib does get installed. For now it sounds like it is ok to keep in nanshe/nanshe, but it is something to keep an eye on.
https://github.com/nanshe-org/docker_nanshe/blob/809049be67fcc272e2f5406c914169b1cdd741b0/Dockerfile#L13-L16


Make benchmarks measure cache misses and hits separately

Cache population/misses and cache hits are different animals with different expected runtimes, and thus they should be measured separately.

Codecov Report

Merging #74 (b33651b) into master (775daf0) will decrease coverage by 2.13%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   92.61%   90.47%   -2.14%
==========================================
  Files           3        3
  Lines         406      399       -7
  Branches       50       51       +1
==========================================
- Hits          376      361      -15
- Misses         18       23       +5
- Partials       12       15       +3

Impacted Files
Coverage Δ

src/fscacher/cache.py
84.44% <28.57%> (-5.71%)
:arrow_down:

src/fscacher/tests/test_cache.py
93.35% <100.00%> (-0.25%)
: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 775daf0...b33651b. Read the comment docs.

And let's release since no release for awhile and lots of enhancements for directories handling


Now all xml validates with xmllint

FIX: added missing block name & status into XSD
FIX: typos fixed in models 4, 5, 6, 7, 601 (andatory, note ->notes)
FIX: removed point attributes, which are missing in XSD
FIX: model 160, current can be also negative if e.g. Fronius uses MPPT for a battery

All issues fixed in another pull request.


HEAD commit change of the submodule can't be discard in parent repository

SourceGit.Commands.Discard now can only discard changes in current repo, the HEAD commit change of the submodule needs to execute git reset --hard in the submodule directory.
Reference: https://stackoverflow.com/questions/10906554/how-do-i-revert-my-changes-to-a-git-submodule

It seems good, but git restore is a new API added in git >=2.23, while Debian 10 (end of LTS on June 30th 2024) officially provides git 2.20.1. It is best to indicate this version requirement in README and other places.


Suggestion: add Travis CI

Travis CI is a continuous integration service that performs all kinds of checks upon a git push. As most of my GitHubs have this, I volunteer to add this.
To resolve this Issue I will add a script that just does most of the tests on CRAN.
In other Issues I will suggest to add more tests like (1) code coverage using covr (2) coding style using lintr (3) use of good practices, using goodpractice

Am all in favor of travis, covr, lintr and goodpractice.

Added Travis CI. Although not all tests pass at Travis yet, I close this Issue.


Bug Report: Linux Compile Error

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
happened as soon as i try'ed compilong it
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
Linux x86_64
What version of the game are you using? Look in the bottom left corner of the main menu.
newest
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
compile extension-webm compile Kade Engine
Please describe your issue. Provide extensive detail and images if possible.

when you ask your mom for ice cream but she says no:


[TextField] SelectProps is missing the generic type parameter

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
TextField's SelectProps is missing the generic type parameter.
SelectProps has the following type:
SelectProps?: Partial<SelectProps>;
Where SelectProps is
export interface SelectProps<T = unknown>
  extends StandardProps<InputProps, 'value' | 'onChange'>,
    Omit<OutlinedInputProps, 'value' | 'onChange'>,
    Pick<SelectInputProps<T>, 'onChange'>

so T defaults to unknown.
Please see the example below that describes how it affects user code.
Examples 🌈
I'm getting the following error
Type 'Partial<SelectProps<string | string[]>>' is not assignable to type 'Partial<SelectProps<unknown>>'. Types of property 'onChange' are incompatible. with the code below
          <TextField
            InputProps={{
              'aria-label': `${labelText ?? ''} selector`
            }}
            SelectProps={{
              classes: {
                icon: classes.selectIcon
              },
              disabled,
              displayEmpty: true,
              IconComponent: ChevronDownIcon,
              MenuProps: {
                anchorOrigin: {
                  horizontal: 'left',
                  vertical: 'bottom'
                },
                transformOrigin: {
                  horizontal: 'left',
                  vertical: 'top'
                }
              },
              multiple,
              renderValue,
              value: processedValue
            } as Partial<SelectProps<typeof selection>>}

Motivation 🔦
Type safety and to prevent compiler errors on valid code.

Can you please share a codesandbox with the error? It's easier to test out locally.


Trace and memory: relocation and files

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@736c56c). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head b93c6c7 differs from pull request most recent head b6cd96a. Consider uploading reports for the commit b6cd96a to get more accurate results

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage        ?   62.41%
=======================================
  Files           ?       12
  Lines           ?     1067
  Branches        ?        0
=======================================
  Hits            ?      666
  Misses          ?      357
  Partials        ?       44

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Split DOA to two distinct commands: SQL conversion and YAML generation

[ ] Split run-doa.sh to doa-translate.sh and doa-verify.sh
[ ] Split doa/migrate.sh to ...

The shell scripts will be merged into single Python script in future.


Link to The CoE Starter Kit not correct

[Enter feedback here]

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

ID: 9fe69498-a485-5379-97ff-53fbe029f11a
Version Independent ID: 5b3f9705-9e49-1575-4ae7-0293d12b739c
Content: Set up inventory components - Power Platform
Content Source: power-platform/guidance/coe/setup-core-components.md
Service: power-platform
Sub-service: guidance
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

Thanks for your comment, @rheid. This issue has been assigned to the doc author for review.

Thank you. We have a PR open to address this already
https://github.com/MicrosoftDocs/power-platform/pull/1687
Closing as duplicate.


the correct word would be Syntax, not sintax

Multiple uses of sintax instead of syntax

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

ID: a552efca-0c18-3b2a-fa6e-af1ed276ea08
Version Independent ID: 82f50727-a66f-4faa-1df5-0932ab8f1b7b
Content: Set-CsSlaConfiguration (SkypeForBusiness)
Content Source: skype/skype-ps/skype/Set-CsSlaConfiguration.md
Service: skypeforbusiness-powershell
GitHub Login: @hirenshah1
Microsoft Alias: hirshah

@DEberhardt Thank you very much for the contribution and updating the documentation with PR. @jeff2work Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Raid registering issue tracker

Since we want to publish as soon as possible, some things can be improved after, here is the list :

T-Shirt on participant update
Adding missing tests to improve coverage
Foreign key constraint violation on update security file
Adding delete all team button
Restricting  file size
Don't keep PDF files that won't be linked to a document, if the second call, after the upload fail. We could technically have orphan sensitive documents
Remove the two pdf python dependencies used only for this module
Don't let the module choose path on the file system but use existing utils
Run postupgradeactions as a background task
Maybe remove get_drive_file_manager dependency
run post_update_action in a separate thread

500 for raid/security_file/?participant_id= when updating security file:
sqlalchemy.exc.IntegrityError: (sqlalchemy.dialects.postgresql.asyncpg.IntegrityError) <class 'asyncpg.exceptions.ForeignKeyViolationError'>: update or delete on table "raid_security_file" violates foreign key constraint "raid_participant_security_file_id_fkey" on table "raid_participant"

Error while creating pdf Traceback (most recent call last): File "/app/app/modules/raid/utils/utils_[raid.py](http://raid.py/)", line 142, in save_team_info file_path = pdf_writer.write_team(team) ^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/app/app/modules/raid/utils/pdf/pdf_[writer.py](http://writer.py/)", line 114, in write_team return self.add_pdf() ^^^^^^^^^^^^^^ File "/app/app/modules/raid/utils/pdf/pdf_[writer.py](http://writer.py/)", line 76, in add_pdf writer.write("data/raid/" + self.file_name) File "/usr/local/lib/python3.11/site-packages/pypdf/_[writer.py](http://writer.py/)", line 1274, in write stream = FileIO(stream, "wb") ^^^^^^^^^^^^^^^^^^^^ FileNotFoundError: [Errno 2] No such file or directory: data/raid/Équipe de xxxx.pdf


made some tweaks to the readme

Hi, thank you for making this awesome project and spending time writing great documentation! I made some tweaks to the documentation to make it easier to read, and fix a couple of typos.

Sorry, haven't had much lately. Thank you for your work.


remote snapshot index supports aliases

Description
[Describe what this change achieves]
We want to add support for searchable snapshots within index management which can operate as follows :

Take a snapshot for the index
Rename on restore the index as a searchable snapshot index
Swaps all the aliases from the source index to the restored index and deletes the source index

But found that aliases operation is blocked by REMOTE_READ_ONLY_ALLOW_DELETE cluster block. In order to switch the index smoothly,  We hope that the searchable snapshot index supports aliases operation.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] New functionality includes testing.
[ ] All tests pass
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks @arvinsg! This is definitely a reasonable feature request. Can you add some tests for this feature? Also can you take a look at the failed DCO workflow and properly sign your commits?

Add some test cases, thanks for the advice and review

This would be fantastic, following :)


hello-jni/packaging: update categories

The category looks already fixed with #62, @tjohns asked us not to squat the already overloaded 'Getting Started' category.


styles not working for collapsible

hello, i am trying to implement this module, but seems that not working
<CardStack
 height={this.state.trucksAvailable.length * 100}
 width={400}
 background="#f8f8f8">
 {
        this.state.trucksAvailable.map(truck => {
        return (
         <Card
          key={truck}
          background={this.generateColor()} >
          <TeamMemberCard {...truck} />
           </Card>
  )})}
</CardStack>

const TeamMemberCard = (props) => (

{props.Vendor_str}

{props.VendorName_str}

            <DetailsRow
                title={props.TruckType}
            />

            <DetailsRow
                title={props.Weight_Capacity}
            />

            <DetailsRow
                title={props.Fei_str}
            />

);

const DetailsRow = ({ title }) => {
    return (

                    {title}

    );
};

but seems that the each child have same height than CardStack, it do that all card is in overlapping or so i think

Hey @mealbarracin10, thanks for the issue! Sorry about the extremely delayed response, I haven't been actively maintaining this repo. Out of curiosity, were you able to resolve the issue?


Create a page: how to submit bugs/ideas for iD

Saw the message on hot list, put this together quickly. I'm creating a new page, but maybe you'd want to integrate the content into an existing one.
I also have two screenshots, but I don't know how to get the images into this project. If you let me know, I can upload and integrate into the page.

There is a submit bug/feedback button in the iD browser itself in the lower right hand corner. It is probably better to train users to use the feature in the editor itself. it takes you right to the issues tab of the repository.

good point, forgot about that! added it in


DynamoDB table.get_item does not raise ResourceNotFoundException ClientError for non-existent key

Issue
When trying to test table.get_item on a non-existent record, I expect moto to raise a ClientError with error code ResourceNotFoundException in line with the expected behaviour whilst testing on an actual table.
Instead, get_item function returns an empty item response.
Library Versions Used
moto==5.0.5
boto3==1.34.84
botocore==1.34.84

Sample code
import boto3
from moto import mock_aws
from botocore.exceptions import ClientError

@mock_aws
def test_dynamo_key_not_exists():
    table_name = 'test_table'
    dynamodb = boto3.resource('dynamodb', region_name='us-east-1')
    # Create a test table
    dynamodb.create_table(
        TableName=table_name,
        KeySchema=[{'AttributeName': 'id', 'KeyType': 'HASH'}],
        AttributeDefinitions=[{'AttributeName': 'id', 'AttributeType': 'S'}],
        ProvisionedThroughput={'ReadCapacityUnits': 1, 'WriteCapacityUnits': 1}
    )
    table = boto3.resource("dynamodb", 'us-east-1').Table(table_name)
    with pytest.raises(ClientError) as e:
        resp = table.get_item(Key={'id': '1'})
        print(resp)
    assert e.value.response['Error']['Code'] == 'ResourceNotFoundException'

Expected
The test passes with the ClientError with code ResourceNotFoundException being raised
Actual
py::test_dynamo_key_not_exists failed: @mock_aws
    def test_dynamo_key_not_exists():
        table_name = 'test_table'
        dynamodb = boto3.resource('dynamodb', region_name='us-east-1')
        # Create a test table
        dynamodb.create_table(
            TableName=table_name,
            KeySchema=[{'AttributeName': 'id', 'KeyType': 'HASH'}],
            AttributeDefinitions=[{'AttributeName': 'id', 'AttributeType': 'S'}],
            ProvisionedThroughput={'ReadCapacityUnits': 1, 'WriteCapacityUnits': 1}
        )
        table = boto3.resource("dynamodb", 'us-east-1').Table(table_name)
>       with pytest.raises(ClientError) as e:
E       Failed: DID NOT RAISE <class 'botocore.exceptions.ClientError'>

The client error is not raised. Instead, the following is the response returned
{'ResponseMetadata': {'RequestId': 'YlHqvAhnzbqUct8Asi2FMePwYphmZRR3cdO5JJ3omO6Vg5bTCHhy', 'HTTPStatusCode': 200, 'HTTPHeaders': {...}, 'RetryAttempts': 0}}

Related to the following issues with the change being that this occurs for a non-existent record in a table rather than a non-existent table.
#4880
#4344

Hi @shashanklmurthy, it is confusing, but that is exactly how AWS behaves.
I know that the AWS CLI does throw a ResourceNotFoundException, and maybe other SDK's as well? But boto3 just returns an empty 200 HTTP response.


docs : improved commands routing docs issue #238

What issue does this Pull Request Address?

What issue does this Pull Request Address?

Found it #238

I'm not sure if would be better just to link to https://discordjs.guide/

maybe just add it to the end . like to learn more visit the discord js website and then add the link.

In the read me where do yo

LGTM 👍, thank you for adding this @ChinmayMhatre 😃.

I'm not sure if would be better just to link to https://discordjs.guide/

@AllanRegush yes I think this can be added somewhere on our README for more docs. But in regards to the command routing, I think this PR is ready.
@ChinmayMhatre I'll leave it up to you if you want to add that Discord.js link in the README 😃

In the read me where do you think the link should go?

yeah works for me ! I'll create a pr

yeah works for me ! should I create a new pr or commit to this branch itself?

 Let's do, @BOLT04's recommendation in another PR.

yeah works for me ! should I create a new pr or commit to this branch itself?

 Let's do, @BOLT04's recommendation in another PR.

oh okay , so how do I move forward with ? do I reference this pr in that one?

yeah works for me ! should I create a new pr or commit to this branch itself?

 Let's do, @BOLT04's recommendation in another PR.

oh okay , so how do I move forward with ? do I reference this pr in that one?

Reference this PR in a new issue noting what @BOLT04 mentioned. Then make new PR.
I'll merge this one shortly.

Thank You 👍 for your hard work


Add more presets

the following preset config files could be useful:

1080p
720p
540p
360p
240p
1440p?
They would be easy to implement:

default_to = "presets/default.toml"

[canvas]
width = 1920
height = 1080

1440 was not implemented


Redundant text with herocards and prompts in Facebook Messenger

When you send herocards or prompts, the bots sends redundant text saying "Options" in Facebook Messenger.

Here's the code:
let msg = new builder.Message(session)
  .text('This is what I can help you with :angel:');
  msg.addAttachment(
      new builder.HeroCard(session)
                    .buttons([
                        builder.CardAction.imBack(session, "Add grocery", "Add grocery"),
                        builder.CardAction.imBack(session, "Remove grocery", "Remove grocery"),
                        builder.CardAction.imBack(session, "Change pick-up time", "Change pick-up time")
                    ])
            );

This is a currently known issue (https://github.com/Microsoft/BotBuilder/issues/711) and has been marked as a bug so the team are aware of it. Hopefully there should be a fix soon.

this is fixed.


Master

good

These are customizations to make the page personal, not product contributions. Please make these changes on a fork, not on a pull request. Thanks!


fix: use different current_version value per locale

Not all locales need to be included in every releases. E.g. en could be part of the release 8, while de might use 7 as the latest current version. This could affect whether whether a locale is updated and we need to store the latest fetched version for each locale
Use a unique cache key per distribution/env/locale. This will prevent issues in case the distribution or secret token should be changed during runtime
Change poll_interval_seconds variable to poll_interval. Allows for a more readable usage like poll_interval = 10.seconds. Although this is a breaking change I would avoid a major version bump as the library is not generally available yet

Closing this PR, will pick the issue up again post migration of UI to syntax components.


fix: Allowed Callers list instantiated to an empty string array.

Description

Fixes an edge case where absolute paths are saved in the botproject file
allowedCallersList is now set to an empty array instead of *.

Task Item
Fixes #5193
Fixes #5165

Coverage increased (+0.04%) to 55.361% when pulling f7f5163f66b6a96cd54398d88eb78c9adda36a1f on srravich/allowed-callers-list into 99142b6f52e8f67a3a49d63c5fd3a94925bf336b on main.

Verified following scenarios. all passed.

local skill with empty array[] (call success)
remote skill with empty array[] (expect 401 error)
remote skill with botid setting in array["botid"] (call success)

by the way, old bot still keep old format. only new created bot with this updated skill settings.

@hibrenda Thanks for testing the scenarios.

@clearab @gabog The allowedCallers list is now set to an empty array by default


bug: e2e test cases fail when running together on CI

Currently, running e2e tests one after the other works as on the main branch. But running them in one command (even with 1 thread), fails as shown - https://github.com/keep-starknet-strange/madara/pull/1562

e2e test have been removed with https://github.com/keep-starknet-strange/madara/pull/1580


Release: rust 0.6.2 and python 0.3.3

includes bug fixes with python wheel distribution
and in turn facilitates integration with libraries that depend on lakers-python, such as aiocoap
actually, the rust side doesn't have updates,
but since the triggers to publish python wheels are on github actions, I am just bumping all the versions

Seems to be something in the Actions environment, tried to re-run 0.6.1 and got the same errors.

I've been trying to reproduce that locally, but that fails for lack of PSA, and GitHub runners are not exactly something to easily run locally.
This:
 error: expected one of `!` or `::`, found keyword `static`
 --> /home/runner/work/lakers/lakers/target/debug/build/psa-crypto-sys-3e14593d80be7147/out/shim_bindings.rs:3:11
  |
3 | rewriting static
  |           ^^^^^^ expected one of `!` or `::`

looks almost like a code generator started spewing debug output into the generated code. But those are generated by bindgen.
As presumably the tests run successfully locally, could you try checking in your Cargo.lock file? By now, that's the default anyway

Yeah, I had rustc 1.72.0-nightly in my local machine (and it was working), then after updating to rustc 1.81.0-nightly I got the error.

I'm getting completely different errors on the PSA side; updating bindgen could help, trying that.

I've taken the liberty to push a commit onto your branch to run the CI of this PR again with the update from https://github.com/malishav/rust-psa-crypto/pull/1
Unless Mališa is currently on vacation or there's another reason that PR can't go in quickly, I'd rather avoid having that in the released version (so I can delete the branch again) -- but then again, it's a patched crate, and those are not part of what gets to crates.io anyway.

Hrmpf, that fixed my local build, but not the CI case.
At least I know now why it's on line 3: the top lines are /* automatically generated by rust-bindgen 0.69.4 */ / empty; this means that rewritingt static can really no be part of valid Rust. Still no clue where that comes from precisely.

I can now reproduce locally: things worked on stable but broke on nightly.
I'm tempted to just fix this by editing out the "rewriting static" that's clearly a leftover println somewhere where I can't trace it. What I do know so far is that it does affect rust-psa-crypto.

The issue is tracked to bindgen: https://github.com/rust-lang/rust-bindgen/issues/2852

Wow, and that's a really fresh issue. Thanks for digging and finding it. Let's wait a bit and see if they fix it somewhat fast. Or else I can try to specify a previous rust version in the CI.

… but ultimately came from https://github.com/rust-lang/rustfmt/issues/6210. Rerunning things with a band-aid from https://github.com/malishav/rust-psa-crypto/pull/1 that disables rustfmt.
It is already fixed, but it'll take a day or two to get into the nightlies.
As a whole I think the best way to go is to revert my change (updating rust-psa-crypto's bindgen is still good but not critical for here), pin the nightly instead, and revert that pin after this is out.

All green now; the only change compared to your original branch is now the fixed nightly version, and that the example's nightly version is symlinked to the toolchain version to avoid missing one when changing back.

Great! Thanks for fixing that.


Logbox warning

Thank you for putting this together. I am relatively new to react native so this boilerplate is immensely useful. Just a heads up that upon starting the app, the following warning is displayed:
Compiled with warnings.
/Users/me/dev/react/newapp/src/utils/ignore.js
Attempted import error: 'LogBox' is not exported from 'react-native-web/dist/index'.

Hi @pronouncedJerry , thanks for letting me know. I'm glad to hear you said the boilerplate is useful. Sure, let me fix the logbox import error when I have time, thanks!

Hi @pronouncedJerry , thanks for letting me know. I'm glad to hear you said the boilerplate is useful. Sure, let me fix the logbox import error when I have time, thanks!

same issue

The LogBox issue comes when you run in web browser. Just in case, I added optional chaining to avoid crashes on web #14 .
I would like to keep the Logbox ignore lines for mobile. So please feel free to remove import ignore for web in your convenience, thanks


Alignment of groups in pages

I have a page laid out as 'Fixed', and 2 groups;
The first group is sized 6 x 6 and which contains gauges.
The second group is 3 x 6 and does not contain any widgets.
Together, they look like this;

However, whenever I add something to the second group, the group's height increases, and is no longer bottom aligned with the first group.
By example, I've added a 3 x 3 text box and some random text.

In my example above, the group was 3 x 6, and the widget was only 3 x 3, so why was it necessary to auto increase the size of the group, surely 3 x 3 would fit with room to spare??
Forum discussion - https://discourse.nodered.org/t/alignment-of-groups-in-pages-dashboard-2/86210?u=paul-reed

okay, spotted one issue in the Grid layout, which I've now fixed (this previously wasn't rendering the full height for the 3x6 group), which i know isn't the issue reported here, but it's closely related.

Still trying to find the equivalent problem in Fixed as reported in this particular issue

I've created 2 identical groups, both 3x6, and added an inject node containing some text which feed one group, whilst the other group has no text.
My CSS nodes have been disabled.
[{"id":"9d6edffda7c0d564","type":"ui-text","z":"1326aadbacf36704","group":"4c637ca6c57dac54","order":0,"width":"3","height":"3","name":"","label":"text","format":"{{msg.payload}}","layout":"row-spread","style":false,"font":"","fontSize":16,"color":"#717171","className":"","x":350,"y":360,"wires":[]},{"id":"d6b07fc3709512ec","type":"inject","z":"1326aadbacf36704","name":"","props":[{"p":"payload"},{"p":"topic","vt":"str"}],"repeat":"","crontab":"","once":false,"onceDelay":0.1,"topic":"","payload":"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco","payloadType":"str","x":130,"y":360,"wires":[["9d6edffda7c0d564"]]},{"id":"a63c3b6699a82214","type":"ui-text","z":"1326aadbacf36704","group":"2907cc0695e59ca5","order":0,"width":0,"height":0,"name":"","label":"text","format":"{{msg.payload}}","layout":"row-spread","style":false,"font":"","fontSize":16,"color":"#717171","className":"","x":350,"y":420,"wires":[]},{"id":"4c637ca6c57dac54","type":"ui-group","name":"Misc","page":"d6557c66b0d2c002","width":"3","height":"6","order":-1,"showTitle":false,"className":"","visible":"true","disabled":"false"},{"id":"2907cc0695e59ca5","type":"ui-group","name":"Test","page":"d6557c66b0d2c002","width":"3","height":"6","order":-1,"showTitle":false,"className":"","visible":"true","disabled":"false"},{"id":"d6557c66b0d2c002","type":"ui-page","name":"Energy","ui":"ae3d4aeb3f977a90","path":"/energy","icon":"home","layout":"flex","theme":"a965ccfef139317a","order":-1,"className":"","visible":"true","disabled":"false"},{"id":"ae3d4aeb3f977a90","type":"ui-base","name":"Dashboard","path":"/dashboard","includeClientData":true,"acceptsClientConfig":["ui-notification","ui-control"],"showPathInSidebar":false,"navigationStyle":"temporary"},{"id":"a965ccfef139317a","type":"ui-theme","name":"Default","colors":{"surface":"#ffffff","primary":"#0094ce","bgPage":"#eeeeee","groupBg":"#ffffff","groupOutline":"#cccccc"},"sizes":{"pagePadding":"12px","groupGap":"12px","groupBorderRadius":"4px","widgetGap":"12px"}}]

Thanks @Paul-Reed with code as-is, I expect that behaviour, fix is in the attached PR which I need a review on before I can merge.
Should hopefully be resolved in a release tomorrow, early next week at the latest

Still trying to find the equivalent problem in Fixed as reported in this particular issue

Ah OK, I thought you couldn't reproduce.
There's no rush, I'm using hotnipi's css hack at the moment.


Mac/Chrome: Wallet connection doesn't seem to work

I've received a message from a user today that the wallet connection for the Walrus Sites deployment of Sui dApp Starter https://suidappstarter.walrus.site/ doesn't work on their Mac/Chrome.
However, the Web2 deployment is working fine for them https://demo.sui-dapp-starter.dev/
Unfortunately, I don't have a Mac machine at hand, so I'd appreciate if you guys could test and reproduce. Thank you.

I just checked and cannot reproduce this. Wallet connection with Sui Wallet works fine on my macBook with Chrome.

Does the user have some further details to share (error messages from the console, etc.)?

@mlegner Thanks for testing. Will try to request some more details

@kkomelin checked as well, works for me too on macbook.

Thank for checking @giac-mysten! I will keep you guys updated if I get any new details.

No reply from the user who reported. Let's close for now and reopen/open if anyone else reports.


Probable wrong dependency name in android>build.gradle

Hi Team.
Is there maybe a wrong dependency name in file (as title) build.gradle inside "src/android/build.gradle" for key " **cdvPluginPostBuildExtras.add * " as apply ?
At current version (2.0.2), name is like:
"com.google.gms.google-services"
instead "com.google.gms:google-services" (with ":" between 'gms' and 'google')
Thanks for the attention

@campagna91 thanks for keeping a close eye on things. There are 2 build.gradle files  in an Android project. The build.gradle inside src/android/ augments the app build.gradle, not the project build.gradle.
It's easy to read the Firebase install directions amd miss that (I did the first time I read it). If you look at the app build.gradle modifications it is correct.
The dependency that has the semi colon goes in the project's build.gradle. That is handled in an install script as there's no built in way to augment that file.
Thanks for checking!