Clock skew adjuster and spans from web/mobile

Spans captured on the web/mobile do not have accurate start times as those devices are not synchronized with the server clocks.
Because of this I use timestamp of the moment they are received by the backend.
It's still wrong because:

they are received after they finish so the start timestamp is larger than it should be by at least: span duration + network latency (assuming synchronized clocks between browser and backend)
mobile devices can be offline and go online and sync days later
it's likely that such spans are batched and flushed periodically (with even more delay) to avoid many small http requests which drain batteries

The problem is that clock-adjusted "centers" everything if the web span is too much off:

This is how it looks if the web span is not reported as a root. The positions more or less reflect reality:

Am I doing something wrong?
How to approach reporting mobile/web spans?
Does clock skew adjusted need some changes to support web/mobile spans?
I'm using version 1.2.0

There may be a bug in the clock skew adjustment algorithm. While it does center child span if the parent span's timing is way off, it's not supposed to center everything, i.e. if most server side spans have reasonable relative timestamps their relative positions should be sensible.
It would be helpful if you could post a JSON of the trace that gets centered incorrectly. Unfortunately, we don't have an obfuscation utility, so if you obfuscate manually please make sure to keep the process tags consistent. And you can strip all tags and logs aside from span.kind tags.

Here it is: https://gist.github.com/mabn/ec35eb5d6f4a9789b46fd1373c601352
It looks like this (web is the root span):

we don't have span.kind tags - not sure if it matters?
processes do not have "ip" tag so in clockskew.go hostKey defaults to "" and all are treated like separate hosts (which is ok)

The issue is in calculateSkew. E.g. for such test case:
{
	description: "root starts after all descendants",
	trace: []spanProto{
		{id: 1, parent: 0, startTime: 10, duration: 100, host: "a", adjusted: 10},
		// latency = (100-50) / 2 = 25
		// delta = (10 - 0) + latency = 35
		{id: 2, parent: 1, startTime: 0, duration: 50, host: "b", adjusted: 35,
			logs: []int{5, 10}, adjustedLogs: []int{40, 45}},
		// child fits inside parent - no additional adjustment
		// adjusted = startTime + parentSkew (35) = 45
		{id: 3, parent: 2, startTime: 10, duration: 10, host: "c", adjusted: 45},
	},
},

Initially span 3 fits under its parent - span 2 starts at 0, span 3 starts later at 10. Adjustment starts:

span 1 is not adjusted - starts at 10
span 2 starts before parent so it's adjusted (centered) under parent - it now starts at 35
span 3 no longer fits under its parent - it starts at 10, but parent (span 2) starts at 35 - so it is also adjusted additionally and "centered" under span 2
if there are more span it will repeat for each of them if root span started sufficiently ahead in time

I'm not entirely sure what is the expected behaviour.

In the case of root span coming from web with a delay I'd like to somehow alter only this one - e.g. mark it as having "unreliable clock" and just adjust its start time to match with the rest of the trace. So the algorithm descending from roots might not work here.

In the case where all of them are from a single datacenter with more or less reliable clocks and one is "too far ahead" - not sure. It would be good to somehow detect that one clock is completely off.
One solution that would solve this particular case would be to assume that if initially a child fits under its parent and the parent has to be adjusted then the child is adjusted by the same amount.
It would basically shift portions of the trace without altering relations between spans.
But it doesn't solve the case with web.

This is something Otel is dealing with currently (relevant discussion here https://github.com/open-telemetry/oteps/issues/154) There are similar discussions in the OpenTelemetry js repo as well since it's a bigger issue coming from the web side. Since there is no standard to handle this type of skew, and it would normally be defined in the instrumentation, Jaeger is merely the visualizer.


Question: how do you build the docs at pptr.dev ?

I really like the https://pptr.dev docs, especially how you handle versioning.  Could you please tell me how these were generated?   I would love to be able to leverage this tooling for my typescript OSS project.
I see an entry at https://github.com/GoogleChrome/puppeteer/blob/master/CONTRIBUTING.md#writing-documentation explaining how to write docs, but not how the doc system works/building docs.

The pptr.dev repo has the answers you want: https://github.com/GoogleChromeLabs/pptr.dev/

perfect, thank you.


Couple of typos under Common Name.

There's a couple of typos under Common Name:
Details for the cloud management gateway CSR
Common Name: ClousServiceNameCMG.YourCompanyPubilcDomainName.com
Suggested:
Common Name: CloudServiceNameCMG.YourCompanyPublicDomainName.com
Thanks,
Juan.

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

ID: 778b6484-ea66-9518-778d-cea65fd183fa
Version Independent ID: 1ea23258-33b6-738a-cc46-a4059e0d3f42
Content: Tutorial: Enable co-management for internet devices - Configuration Manager
Content Source: sccm/comanage/tutorial-co-manage-new-devices.md
Product: configuration-manager
Technology: configmgr-comanage
GitHub Login: @mestew
Microsoft Alias: mstewart

Nice catch @juelizon. If you're interested you could also submit the change yourself, it's pretty straight forward and something I try to do on a regular basis to help the PG team. Check this page out for more info, How to contribute. If you need any help, just let me know.

Thanks to @jpcast88 for fixing this typo, which is why the issue closed. The change will be live later this  morning. Thanks @juelizon for catching the typo, we appreciate the feedback!


cleanup .cdsrc.json

See: https://github.com/SAP-samples/cloud-cap-samples-java/pull/262

According to my understanding the file could even be a completely empty JSON object or even removed, as also profile java is auto-configured.

According to my understanding the file could even be a completely empty JSON object or even removed, as also profile java is auto-configured.

ok, I will try without .cdsrc.json.

cool, it works withoud a .cdsrc.json, the cds build works fine and bookshop starts.

Don't we use .cdsrc.json also as an indicator in our Maven Plugin for the root of the project?
Maybe better keep an empty file? If we remove it, we should I think avoid putting config into package.json but rather readd .cdsrc.json if we need it.

The cds-maven-plugin uses beside the .cdsrc.json also the package.json to find the build root. So, this is not problem, the maven build works fine without a .cdsrc.json:
[INFO] CdsMojo: Using /Users/d032842/SAPDevelop/.m2/repository/com/sap/cds/cds-maven-plugin/cache/unpacked/18.18.2/unpacked-18.18.2-darwin-x64.tar.gz/node-v18.18.2-darwin-x64/bin/npx provided by goal install-node.
[INFO] CdsMojo: Using directory containing a package.json as working directory: /Users/d032842/SAPDevelop/git/cloud-cap-samples-java
[INFO] CdsMojo: Executing [/Users/d032842/SAPDevelop/.m2/repository/com/sap/cds/cds-maven-plugin/cache/unpacked/18.18.2/unpacked-18.18.2-darwin-x64.tar.gz/node-v18.18.2-darwin-x64/bin/npx, -c, cds version] in working directory /Users/d032842/SAPDevelop/git/cloud-cap-samples-java
[INFO] CdsMojo: Using configured working directory: /Users/d032842/SAPDevelop/git/cloud-cap-samples-java
[INFO] CdsMojo: Executing [/Users/d032842/SAPDevelop/.m2/repository/com/sap/cds/cds-maven-plugin/cache/unpacked/18.18.2/unpacked-18.18.2-darwin-x64.tar.gz/node-v18.18.2-darwin-x64/bin/npx, -c, cds build --for java] in working directory /Users/d032842/SAPDevelop/git/cloud-cap-samples-java
[INFO] CdsMojo: [cds] - building project [/Users/d032842/SAPDevelop/git/cloud-cap-samples-java], clean [true]
[cds] - cds-dk [7.3.0], cds [7.3.0], compiler [4.3.0], home [/Users/d032842/SAPDevelop/git/cloud-cap-samples-java/node_modules/@sap/cds]

[INFO] CdsMojo: Using directory containing a package.json as working directory: /Users/d032842/SAPDevelop/git/cloud-cap-samples-java
But yes, we can keep an empty .cdsrc.json, to avoid future CDS configurations in package.json.


Viewer/chatter list doesn't close when pressing escape

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
When pressing the escape in the viewers list, the window remains open.
Screenshots
https://user-images.githubusercontent.com/4962764/153733703-fa3444a6-5e37-458c-8e0f-8991948a6f63.mp4
OS and Chatterino Version
Chatterino 2.3.4 (commit fb9c3ad4) on Arch Linux (irrelevant)

Ideally this would match the behaviour of the emote picker as they're both popout windows. Noticed the same issue with settings but that sounds problematic to add unless there was some kind of "are you sure you want to discard" prompt.

Yeah the Settings window is intentional in the absence of a more advanced prompt as you suggested, this I can confirm should close with Escape as other popup windows


Yo-Kai Watch 2 came out in the U.S.!

Since Yo-Kai Watch 2 came out in the US, can you translate all the items and Yo-Kai?

added in version 0.2.2.


Extending LSIF output with arbitrary metadata?

I maintain a debugger that I am adding more source navigation capabilities to. One issue that I've run across is that the view of the source as described in the LSIF and the view of the source as described by a compiler in the debug info do not always match. In my case, embedding information about the version of the compiler into the LSIF would be sufficient to solve this problem.
It's unclear to me whether LSIF is supposed to be considered generally extensible (i.e. people can tack on whatever additional properties they want to any entry) or whether nothing that is not in the specification is supposed to appear. If the latter, would there be an objection to adding a spot for arbitrary key/value pairs on the Metadata node? e.g., in vaguely idl-ish syntax
partial interface MetaData {
  additionalInfo?: map<string, string>,
}

No, I don't think so (short of smuggling the compiler version in as an addition to the tool version or something).


Implement deepCopy as copying, instead of merge

Currently when we copy a message we create an empty message, and then merge the
copied message to the empty message.
This is not optimal as merging is a more complicated operation then copying.
This implement deepCopy as simple deep copy and halves deepCopy benchmark
results.
GeneratedMessage.deepCopy also gets a optional freeze argument for
optionally generating frozen messages. Previously we would have to copy and
then make another pass to mark the message as frozen.
Changes:

deepCopy method added to PbList, PbMap, _ExtensionFieldSet,
UnknownFieldSet, _FieldSet

GeneratedMessage.deepCopy now calls _FieldSet.deepCopy, doesn't use
merging

(I also removed generating clone() overrides, but I will revert it for now
and maybe do it in a separate PR)
Benchmarks:

Before: protobuf_deep_copy(RunTimeRaw):<PHONE_NUMBER>063492 us.
After: protobuf_deep_copy(RunTimeRaw):<PHONE_NUMBER>3067485 us.
Diff: -328.23 us, -51.66%

cl/470187112

This causes a lot of breakage internally. I think one of the reasons is probably that I'm assuming PbList for repeated fields and PbMap for map fields, but map and list allocation is done via GeneratedMessage.createRepeatedField (returns List<T>) and GeneratedMessage.createMapField (returns Map<K, V>). I should check the field type (instead of field value type), and use Map and List API similar to how we merge map and list fields.
Since we can't have nested lists and maps, cloning of nested maps and lists will have to go through GeneratedMessage.deepCopy, which will again look at field types and use List/Map API to clone maps and lists. So we won't need deepCopy in PbMap and PbList types.
Interestingly, we assume PbList and PbMap in _FieldSet._markReadOnly:
void _markReadOnly() {
  if (_isReadOnly) return;
  _frozenState = true;
  for (var field in _meta.sortedByTag) {
    if (field.isRepeated) {
      PbList? list = _values[field.index!];
      if (list == null) continue;
      list.freeze();
    } else if (field.isMapField) {
      PbMap? map = _values[field.index!];
      if (map == null) continue;
      map.freeze();
    } else if (field.isGroupOrMessage) {
      final entry = _values[field.index!];
      if (entry != null) {
        (entry as GeneratedMessage).freeze();
      }
    }
  }

  _extensions?._markReadOnly();
  _unknownFields?._markReadOnly();
}

If I'm right that the breakage here caused by users overriding list and map field types, then I'm not sure how _markReadOnly works. Maybe users that override map and list types never freeze their messages?
We should just rewrite this library..

I managed to fix at least some of the downstream breakage by removing the assumptions in the implementation about repeated fields being PbList and map fields being PbMap.
The fact that we csan't assume this means freezing messages is buggy, for the reasons described in my previous comment above.
Performance is still much better than merge-based implementation. I will update the PR description, and refactor the code a little bit (document, revert some of the changes).

I managed to fix at least some of the downstream breakage by removing the assumptions in the implementation about repeated fields being PbList and map fields being PbMap.

This version fixed some of the previous failures, but introduced new ones.


v10.0.0-rc1 compile error

develop/CyberRT/cyber/statistics/statistics.h:35:10: fatal error: third_party/var/bvar/bvar.h: 没有那个文件或目录
35 | #include "third_party/var/bvar/bvar.h"

fastdds and bvar thirdparties not added

bvar is OK now , but fastdds is still not added.


Update default docker images.

Coverage remained the same at 19.242% when pulling b47cf93f76dbd1ecf3d5659278046219b98b2d76 on heuermh:update-docker-images into 349b0f021a95105e70efe5961e09ca2b07e96edf on bigdatagenomics:master.

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


Safari (mac and iOS) fixes

Pictures on CS case study now showing;
CS case study gallery with correct height;
Nav overlay on easy.money and CS case studyes withow text from background;
Case studies links on work page with correct height;
Clients grid now wraps correctly;

:+1:

:+1: but I'd prefer if @azevedo-252 could review this later

@joaojusto can you please leave it here why there are so many workarounds. And what do you mean with workaround,  do you intend to change this later?

@azevedo-252 I left the comments flagging as workaround because it's in fact a workaround for the bug described here.
And yes I intend to change this later, since it's a Safari specific problem.


Add torchaudio to requirements

The torchaudio package is imported explicitly in mel_extractor.py#L5

thank you for your pr
I marged it


Partial refund

Features

Partial Refund request, index and show.

Amazing how many bugs in templates you've fixed. Good!


AzureServiceTokenProvider.GetAuthenticationResultAsync should accept a cancellation token

Per Azure Design guidelines, GetAuthenticationResultAsync() should accept and honor a cancellation token--both for API consistency and for better resiliency.
@varunsh-msft and @nonik0 should probably have a look.

Hi @binderjoe, we've released support for cancellation tokens in the latest preview release we just released. Please try it out and let us know if you have any feedback.


tests: refactor to use the module system

lib/modules: add evalNixvim
modules/test: init
modules/test: check warnings/assertions
tests: use test module output + drop test-derivation
Revert "tests: Allow to test multiple derivations in a single test derivation"

This is half-tested and not fully thought out, so I've marked as draft for now.
This supersedes #1986 and #1985.
I'll flesh out this description when I have a minute!
The general idea is that all nixvim config evals will include a "test" package in their output config.
evalNixvim is introduced to normalise how we should call evalModules when evaluating nixvim configs.
A evalNixvimTest (name TBC) may also be helpful, to evaluate without checking assertions and then return the test output.
Our tests use this now instead of test-derivation.nix, so that file could probably be moved into lib since that's the only place it's used. We may also consider deprecating and/or rewriting parts of it.
The new test also checks the warnings/assertions, meaning tests will fail if warnings are present. In the future we can expand on this to allow "expecting" assertions, which will make our test system more flexible.
TODO, either in this PR or future work:

[ ] offer evalNixvimTest alongside evalNixvim
[ ] Update documentation
[ ] use this in test-derivation and/or deprecate it
[ ] allow "expecting" and whitelisting assertions/warnings in test modules
[ ] remove usage of tests.dontRun and module = in test cases

Tests failing on warnings seems to be working:
withJson> Unexpected warnings:
withJson> - NixVim(plugins.schemastore): you have enabled `json` schemas, but `plugins.lsp.servers.jsonls` is not enabled.

@mergifyio queue


[W20] Late pre-recorded presentations?

Due to our process, we are currently making final decisions about presentations in the workshop. I understand that we have long missed the deadline so am looking for best practice advice about the presentations currently being prepared.
I'm wondering whether it is still at all possible to use the SlidesLive platform to pre-record the presentations, or whether we should simply ask the presenters to record themselves and send us the video file (for live sharing during the Zoom session)?

@kverspoor  This is a question for SlidesLive. Although at this point, given all the different things they are managing, it may be hard for them to get this done in time. So if the presenter is okay with live sharing during zoom session, that might be the easiest at this point.
If the presenter really prefers to pre-record, then please do the following:

Email Jarda at<EMAIL_ADDRESS>and ask if he can share a link with the presenter to pre-record.
If he agrees, then please ask them to update [this spreadsheet] (https://docs.google.com/spreadsheets/d/1mnkvwYGcramosUflCRlVdgzUPIUSfkqUDpDTa5Faslg/edit#gid=1249246662)
Once Jarda has updated the spreadsheet, please let us know here so that we can update the virtual website with the video ID.

FYI, Jarda has indicated that this can no longer be accommodated, so I think we can close the ticket.


editing rights on the wiki

Hi guys, is it possible that I don't have editing rights for the Affiliate network page on the wiki?
https://wiki.creativecommons.org/wiki/CC_Affiliate_Network

Is this taken care of?

She has edit rights but still can't edit that page. I think it's locked or something maybe @robmyers ?

@mmoreshead yes you are right. I've made it so approved users can edit it. @gwenfranck please try now and let me know if it's OK.

Thanks @robmyers!


Calls to postcss-cli silently fail when the config option is wrong

The title of this issue is... imperfect.
The config option for postcss-cli is used to specify the directory containing postcss.config.js (or other file name recognized by github.com/postcss/postcss-load-config).
If the config directory does not exist, Hugo throws an error, as expected.
Hugo does not throw an error if you (erroneously) include an existing file name that is not compatible with postcss-load-config:
dict "config" "existing-directory/xxx.config.js"

The call to postcss-cli fails silently; it does not do its thing. At this point I'm not sure if this is on our side or upstream.
Ref: https://discourse.gohugo.io/t/43490/2

Test case
git clone --single-branch -b hugo-github-issue-10846 https://github.com/jmooring/hugo-testing hugo-github-issue-10846
cd hugo-github-issue-10846
npm ci
hugo

This should throw an error.
If postcss were actually doing something, the content of public/index.html would be:
::-moz-placeholder {
  color: gray;
}
::placeholder {
  color: gray;
}

When calling postcss-cli from the command line, it fails silently under the same conditions. It even fails silently when --config points to a non-existent directory, while Hugo throws an error.
Given that entire configuration can also be specified within package.json, I'm not convinced this should be reported upstream.


Add yajl_reset_stack() to recover from parsing failure

libbons project currently bundles yajl because it needs to support recovering from yajl parsing failures by resetting the byte stack. See [https://jira.mongodb.org/browse/CDRIVER-597].
Followin patch adds yajl_reset_stack() public function which could satisfy libbson need:
From d42168311963a6827b0c9d547d8efdf0e7fbdcae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?=<EMAIL_ADDRESS>Date: Fri, 24 Apr 2015 14:51:24 +0200
Subject: [PATCH] Add yajl_reset_stack()
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

yajl_reset_stack() resets parser stack and initilize it to
yajl_state_start. This is needed by libbson library to recover from
parsing failure without need to destroy the handle completely.

See <https://jira.mongodb.org/browse/CDRIVER-597>.

Signed-off-by: Petr Písař<EMAIL_ADDRESS>
diff --git a/src/api/yajl_parse.h b/src/api/yajl_parse.h
index 1c25a60..8ba903a 100644
--- a/src/api/yajl_parse.h
+++ b/src/api/yajl_parse.h
@@ -168,6 +168,9 @@ extern "C" {
     /** free a parser handle */
     YAJL_API void yajl_free(yajl_handle handle);

+    /** Flush parser stack to initial state. All other settings are preserved. */
+    YAJL_API void yajl_reset_stack(yajl_handle handle);
+
     /** Parse some json!
      *  \param hand - a handle to the json parser allocated with yajl_alloc
      *  \param jsonText - a pointer to the UTF8 json text to be parsed
diff --git a/src/yajl.c b/src/yajl.c
index d477893..1cb7c51 100644
--- a/src/yajl.c
+++ b/src/yajl.c
@@ -114,6 +114,13 @@ yajl_free(yajl_handle handle)
     YA_FREE(&(handle->alloc), handle);
 }

+void
+yajl_reset_stack(yajl_handle handle)
+{
+    handle->stateStack.used = 0;
+    yajl_bs_push(handle->stateStack, yajl_state_start);
+}
+
 yajl_status
 yajl_parse(yajl_handle hand, const unsigned char * jsonText,
            size_t jsonTextLen)
--
2.1.0

Hope we can get this accepted and released this quarter, it'd be a big help for the MongoDB libbson project if we can use the standard YAJL API instead of hacking into YAJL's internals.

Seems @lloyd is not available.

Hi @lloyd, I could really use some response to this request. I am willing to help in any way you like, whatever you require to prevent you from doing too much work.

hrm.  this patch doesn't seem like it would properly re-initialize lexer state?

Also, to be clear, the motivation for this patch is performance?  Reallocation of the yajl parser for distinct documents is slowing you down?

Yes, we're trying to avoid reallocation when we go from one document to the next, parsing something like:
{"a": 1}{"a": 2}

ok, give me a minute to hand you a patch on a branch to review.

Great! Yeah, I'm looking just a little more deeply at this, and it seems some of the lexer's state should be cleared. This is enough for libbson's use case:
   yh->stateStack.used = 0;
   yajl_bs_push (yh->stateStack, yajl_state_start);

... but it's not fully correct for other users.

@ajdavis how's this look?  https://github.com/lloyd/yajl/tree/yajl_reset

Thanks, let me test this with libbson

:+1:

So far so good. One thing I've wondered about is whether we properly report the error location if we encounter a syntax error after the first document, so I'll add a test for that and report back again. =)

There's some work to do regarding properly showing the location of a parse error (the (right here) ------^ message) but I'm pretty sure the problem's in libbson, not YAJL.
So LGTM!

ok, I'll merge down and roll a new release within a day.

hey @lloyd would you consider adding other maintainers?

Thank you very much for implementing the yajl_reset(). I'm looking forward for the new release.

I would absolutely consider additional maintainers.  Who's interested?
--lloyd

On Sep 25, 2015, at 10:21 AM, Lamont Granquist<EMAIL_ADDRESS>wrote:
hey @lloyd would you consider adding other maintainers?
—
Reply to this email directly or view it on GitHub.

I'd like to try, if you'd provide me with some guidance when I need it.
On Thursday, October 1, 2015, Lloyd Hilaiel<EMAIL_ADDRESS>wrote:

I would absolutely consider additional maintainers. Who's interested?
--lloyd

On Sep 25, 2015, at 10:21 AM, Lamont Granquist<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
hey @lloyd would you consider adding other maintainers?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/lloyd/yajl/issues/161#issuecomment-144904557.

I'm definitely interested.  I'm the primary author of http://github.com/chef/ffi-yajl and would like to get some of the windows build stuff into better shape and stop maintaining internal patches at the very least...

ok, I'll merge down and roll a new release within a day.

@lloyd any news about this ?

@lloyd ping?

:+1: I also implemented this in #177 but am deferring to this PR to be merged.

@lloyd Are there any news to merge the patch in the master branch and roll a new release ? Without this release, it is impossible to integrate some libraries in distributions.
Is there a problem with maintenance of yajl ? It seems that there are some developers ( @lamont-granquist, @ppisar ) that would be happy to maintain it.

It would be great if yajl was maintained so that it can be added in distributions like RHEL. (there is currently no yajl-devel package)


ScoreCAM, SSCAM,ISCAM failed when feeding this image

Bug description
When trying the example with torchcam v0.3.2dev0, the command is
python scripts/cam_example.py --arch resnet18 --class-idx 386 --img ./elephant.jpg --method ScoreCAM
The ScoreCAM seems to crash with no outputs.

By the way, when trying this command, it shows "WARNING:root:no value was provided for 'target_layer', thus set to 'layer4'"
Code snippet to reproduce the bug
python scripts/cam_example.py --arch resnet18 --class-idx 386 --img ./elephant.jpg --method ScoreCAM
Error traceback

Environment
torchcam v0.3.2dev0

Hi @ChordXy 👋
Thanks for reporting this! Two thoughts / questions:

if you got the visualisation, I doubt it has crashed. But it's possible that for this class index, the activation is almost uniform (and very very low). This could be possible if you passed a wrong class index
you closed the issue so I assume you found the source of the problem. would you mind sharing it here as well please? 🙏 always better to document a bit your findings 🙂

Cheers ✌

font{
    line-height: 1.6;
}
ul,ol{
    padding-left: 20px;
    list-style-position: inside;
}

    I just test this and I find that the problem is caused by the version of PyTorch!
    I have 2 devices. The code runs perfectly on my PC with torch==1.9.1 while it fails on my server with torch==1.10.2So I guess maybe the upgrade of torch version modifies some realization of functions leading to this consequence

<EMAIL_ADDRESS>

On 4/2/2022 12:52，F-G<EMAIL_ADDRESS>wrote：

Hi @ChordXy 👋
Thanks for reporting this! Two thoughts / questions:
if you got the visualisation, I doubt it has crashed. But it's possible that for this class index, the activation is almost uniform (and very very low). This could be possible if you passed a wrong class index
you closed the issue so I assume you found the source of the problem. would you mind sharing it here as well please? 🙏 always better to document a bit your findings 🙂
Cheers ✌
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


applyTransition should also be applied to current slide before transition

I was working with a set of differently heighted texts when I came across this. Using fade: true, the current slide doesn't become transparent (opacity: 0) until after the next slide has transitioned in. I believe a nice feature would be to apply transition to the current slide before putting the transition in motion. Currently, applyTransition is only set on the slideIndex, if it was applied to both, then the now-previous slide would also transition/fade out.

This should have been fixed by #1349 - are you running the latest build? (1.5.6)


Better UX for long messages

@tarasglek shared a few chats last night that include really long blocks of text as part of the prompt's context (in both cases, he's asking ChatCraft to create "chain of summaries," which has the LLM iterate on its own work to make it better over time):

https://chatcraft.org/c/tarasglek/mcHmCrkvVpLKwK7FfjSC2
https://chatcraft.org/c/tarasglek/oHZpX9u7nHvyK11CGVGHU

Getting to the replies underneath is painful, since you have to scroll a hundred times.  Apps like Discord already provide a way to deal with this in the UI:

When a message or code block is over some threshold in size, we should switch to showing it like a file attachment.  Having some way to open it as text if you need to would be useful too.

How about re-use the Show More... button when messages are too long and add a new button right to it such as Download Text

That's a good approach, let's start with that.  Luckily, we already have "Copy" and "Download" implemented:

So I guess we need to update the base Message component to truncate the message if its length is over some threshold, and show the "Show More..." button instead.

Oh, right! We have the "Copy" and "Download" implemented 😄 I would like to help implement the "Show More..." button

I like how we do show more in system message. should make sure the 2 are similar i think


Add unit tests & refactor trader agent

[Part of epic #50 "Ship MVP"]
What: refactor & add tests similar to what we did for predictoor, trueval

@calina-c I moved this task to this specific github issue (the one you're working on now). Please link it to your PR and close when done. Thanks:)


Feature Request: the !michael command should have its data stored in a file on s3 or in a DDB table. Then we can retrieve it/update it etc as needed. For instance !michael add Quote Goes Here

Duplicate


Add eugene spotlight

Add README highlighting @metaboulie's contributions
Reflect changes throughout repo

ahhh, thanks for this <3 @Haleshot


Allow compiling without pugixml

In the event that one doesn't need to use libecpint directly to read ECP definitions from files, the pugixml dependency isn't necessary.
I couldn't decide between HAS_PUGIXML or WITH_PUGIXML for the preprocessor definition, and kind of don't like either of them. If you have a better suggestion, or prefer another way to disable compiling these routines, let me know.

Thank you for this, this has saved me a job :)


download of latest dev package raised antivirus alert and is removed

Sounds fairly strange, I'd think of a false positive. Rather annoying.

Seems to be a fairly common false positive :( https://cdromance.org/guides/virus-detected-trojanscript-wacatac-bml/

Didn't have this alert with earlier downloads. Wonder if it is related to
the CrowdStrike fiasco today.
On Fri, Jul 19, 2024, 9:00 AM Cédric Champeau @.***>
wrote:

Seems to be a fairly common false positive :(
https://cdromance.org/guides/virus-detected-trojanscript-wacatac-bml/
—
Reply to this email directly, view it on GitHub
https://github.com/melix/astro4j/issues/373#issuecomment-2239511588, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BGFJTATOVOGH65XZXTKDFOTZNEZZHAVCNFSM6AAAAABLEZP6LKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZZGUYTCNJYHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

No issue here, I downloaded both the "last" 2.6.3 dev, the 2.6.3 and the 2.6.4 dev on multiple machines, albeit probably before microsoft showing us once again its real face.

When these positives have been mentioned in the past, I have scanned multiple revs of jSolEx with clamav (on the Mac) and have never found a detection, .
steve

On Jul 19, 2024, at 13:06, VNP @.***> wrote:
No issue here, I downloaded both the "last" 2.6.3 dev, the 2.6.3 and the 2.6.4 dev on multiple machines, albeit probably before microsoft showing us once again its real face.
—
Reply to this email directly, view it on GitHub https://github.com/melix/astro4j/issues/373#issuecomment-2240035180, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG4JHEE5KM3Z3SBOICQPOQLZNFWSZAVCNFSM6AAAAABLEZP6LKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBQGAZTKMJYGA.
You are receiving this because you are subscribed to this thread.

Retired downloads and no alerts now. Will close issue


Reduce memory allocations when computing accessibilityLabel

Summary:
While investigating the root cause of app hanging on older devices in Instruments, I noticed that the heaviest stack trace was pointing to RCTRecursiveAccessibilityLabel in RCTView.m.

Heaviest stack trace in Instruments

The profiling was done on an iPad (5th generation) running iOS 16.7.4. The app is text heavy which makes the issue more visible than in RNTester for instance.
Before

It turns out that [NSMutableString stringWithString:@""] is initialized in every call of the recursion even though most of the time it's only used to check the length at the end and return nil.
My change only initialize the mutable string if it's going to be used. I applied the same logic to the equivalent Fabric component. It's a small change that improved the accessibility label generation by 60ms in my case.
Changelog:

[IOS] [CHANGED] - Reduce memory allocations when computing accessibilityLabel
Test Plan:
Running the same measurements after the change, computing the accessibility label is not the heaviest stack trace anymore. And the line by line tracing shows that [NSMutableString stringWithString:@""] impact has been significantly reduced.
After

I have been using this change in production thanks to a patch-package and it effectively improved the performances when navigating between screens.
I also tested in RNTester with and without Fabric. For both architectures, I made sure the return value of RCTRecursiveAccessibilityLabel.
Interestingly, when there is no label, the Fabric implementation returns an empty string while the RCTView.m returns nil.
I'm open to align both implementations to return nil if you believe there is no underlying reason requiring the Fabric implementation to return an empty string.

Thanks for contributing! I think getting the two implementations to match and returning nil when there is no label would be desired.

Sorry, it took me a while to address the feedback but I finally took the time to do it.


Add support for Stata

Stata documentation: https://www.stata.com/manuals13/pcomments.pdf
'//%s' works as a linewise comment, too, but only in .do scripts. If you copy-and-paste a '//%s' comment into the console, it generates an error.

Is this language already supported by (n)vim? Like syntax highlighting and stuff.

Nvim provides Stata highlighting by default, and gcc comments linewise with /*%s*/. For example, the following code
sysuse auto, clear
summarize

becomes
/* sysuse auto, clear */
/* summarize */

However, it would be better if gcc commented linewise with *%s, yielding
* sysuse auto, clear
* summarize

because Stata's console accepts *%s comments but rejects (errors) /*%s*/comments.
Please let me know if there is a better way to achieve this behavior or submit this pull request. Thanks!

Nvim provides Stata highlighting by default

Nice.

because Stata's console accepts %s comments but rejects (errors) /%s*/comments.

IIUC, *%s is not valid for trailing comments which means gcA mapping will be useless whereas //%s will work for all (line comment) cases.

Ah! I rarely use gcA, so I had not considered this case. Instead, I use A//. //%s works as a trailing comment but fails in the console.

Character
Mode
Works in console?
Works as trailing comment?

*%s
Linewise
Yes
No

//%s
Linewise
No
Yes

/*%s*/
Blockwise
No
NA

Stata's comment characters and mixed console/script usage make a perfect solution difficult. I frequently copy-and-paste several lines of code into Stata's console, so I prefer *%s to //%s. However, someone who runs full scripts (and never copies-and-pastes) might prefer //%s.

/*%s*/ can be used as trailing comment as per docs. In any case, i would much prefer to have //%s and /*%s*/ for line and block comments respectively.
But, you would loose console support. IDK anything about stata, so I would let you decide.

You're right, and //%s and /*%s*/ are better linewise and clockwise comments, respectively. I checked Stata's built-in editor, and it uses //%s comments. I shouldn't break what Stata and Comment.nvim users expect because of my workflow. I updated my branch's ft.lua.
Thanks for the help and a great package!


Add url prefix for reverse proxies

Thanks for this concise and useful project :)
In a reverse proxy setup, it would be useful to allow a prefix / context-path to be used while generating the secret URL. I have added an environment variable to support this.
I am tempted to use APPLICATION_ROOT or a blueprint but with a limited use case like in this case I am not sure if that really would make sense.  Please review.

Good idea! I just a few bits of feedback.

Thanks @jparise I have updated the default value.
For tests it looks like currently mockredis usage is broken and also it is not longer maintained from what I can see in this commit.
With version 2.9.3
======================================================================
ERROR: test_password_after_expiration (__main__.SnapPassTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests.py", line 99, in test_password_after_expiration
    snappass.redis_client.do_expire()
AttributeError: 'StrictRedis' object has no attribute 'do_expire'

Perhaps migration to fakeredis will be a good idea, unless you are using a fork of mockredis?

Perhaps migration to fakeredis will be a good idea, unless you are using a fork of mockredis?

That works for me. Want to take a crack at it in a separate issue?

Yeah I think it will be betrer to handle that separately as that might entail a bit of restructuring.

Added prefix test and force pushed because of rebase to include fakeredis support.


Put a cap on list objects v1

Fix a bug by removing features that should have been removed in 2.1.0.

riak_cs_list_objects_fsm.erl can be removed?

Removing everything carefully takes time so I just capped it. Removing v1 list objects will sure tempt me to clean up other codes.

...and broke the dialyzer.

so I just capped it.

Understood :smile:  Then, is this PR urgent work?

Never urgent.

It's not urgent but this PR is worth to be added for code simplicity.


Update aws-java-sdk-dynamodb to 1.12.331

Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.1034 to 1.12.331.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #356.


⚠ Bofrost.lu has degraded performance

In dc96a87, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1112 ms

Resolved: Bofrost.lu performance has improved in 4f51368 after 6 minutes.


Add support for opentelemetry 0.26

Adds support for the latest stable opentelemetry & tracing-opentelemetry releases

@asonix Thanks
@LukeMathWalker Please release it 🙏🏼


Add exportable custom plain logger

#1169

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

Comparison is base (2dc8e98) 81.07% compared to head (46f59d4) 79.89%.

Files
Patch %
Lines

pkg/logger/hub.go
45.45%
12 Missing :warning:

pkg/logger/export.go
71.42%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1171      +/-   ##
==========================================
- Coverage   81.07%   79.89%   -1.18%
==========================================
  Files         149      153       +4
  Lines       16967    17321     +354
==========================================
+ Hits        13756    13839      +83
- Misses       2558     2822     +264
- Partials      653      660       +7

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

What's wrong ?

@cyrnicolase

It's so sad!

For creating and manipulating OS-specific paths directly use os.PathSeparator and the path/filepath package.

@suchen-sci @xxx7xxxx Could you please review the code for me?


M3DB: Add concept docs for resource usage and tradeoffs

source: https://help.aiven.io/en/articles/4601105-scaling-m3

Fixed by #47


Create record classes for the FMP

Create record classes to map the Financial Modeling Prep api responses to.  One class per each api response, 1. discounted cash flow, 2. Price Target Summary and 3. Price Target consensus

done


Setup CI

What have been done:

[x] Configured codecov.io;
[x] Added .editorconfig and  .gitattributes for Rust project;
[x] Added PULL_REQUEST_TEMPLATE.md;
[x] Added GitHub Action for audit;
[x] Added GitHub Action for CI;

Close #7

Codecov Report

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

@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage        ?   32.27%
=======================================
  Files           ?       33
  Lines           ?      973
  Branches        ?        0
=======================================
  Hits            ?      314
  Misses          ?      659
  Partials        ?        0

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 67cf9f3...e9d47fe. Read the comment docs.


Stop Converting Tuples To Objects(In a Config File)

I believe all of the config is stored as objects, as I mentioned in Slack. I doubt this would get changed without an overhaul to handle it another way.


CUDA out of memory when creating mask

Hi I used the quad6 dataset and set the grid_dim to 10, 10
However, I get the following cuda_outof_memory error:
RuntimeError: CUDA out of memory. Tried to allocate 18.31 GiB (GPU 0; 23.70 GiB total capacity; 18.57 GiB already allocated; 3.13 GiB free; 18.58 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
I'm using 3090 with 8 cards. I dont understand why it caused the out of memory issue.
Weide

You should have a larger GPU or reduce the number of rays. However, I found the process is very very slow.

decrease the grid dim to size  2 4 or 2 2

the ray_chunk_size and dist_chunk_size options also would reduce memory requirements. You can also try playing with the ray_samples but that might impact accuracy. Otherwise the pregenerated masks should be available with links in the README


Resource cleanup is lacking for failed PRs.

Pick any failed PR.
https://microsofthealthoss.visualstudio.com/FhirServer/_build/results?buildId=10072&view=results as example.
We create resources in azure, but we don't clean them if PR failed somewhere.
There is few outcomes for such PR:

Someone push new version or retry failed job. It succeeded and we clean resources in azure.
Something blocking PR and no one retry failed job. Like this one: https://github.com/microsoft/fhir-server/pull/1702 Resources would remain in azure until retry or new iteration.
PR get closed. Maybe it's dependabot found newer version, maybe whole purpose of PR to run tests and see what would happen. As example: https://github.com/microsoft/fhir-server/pull/1625  Resources in azure would remain until someone would delete them manually.

In case of 2 and 3 we just waste resources. Which can be costly.
I would advocate to cleanup resources even in case of test failures.
As alternative we can add workflow for closing PR and do cleanup in that workflow. That would solve case 3 (2nd case would still be problematic)

AB#81259


Internal compiler error when .NET Native compiles __makeref, __refvalue, TypedReference

When I put the following C# code in a UWP app in VS 2019 16.0.2, it works well when compiled for Debug.  However, when it is compiled for Release (.NET Native), then error ILT0005 and "Internal compiler error" occur.
class MyTestTypedReference
{
	protected int fTestField0 = 123;
	protected int fTestField1 = 456;
	protected string fTestField2 = "TestStr";
	protected int fTestField3 = 0;

	public unsafe void TestMakeTrArray()
	{
		System.TypedReference* trArray = stackalloc System.TypedReference[4];
		trArray[0] = __makeref(fTestField0);
		trArray[1] = __makeref(fTestField1);
		trArray[2] = __makeref(fTestField2);
		trArray[3] = __makeref(fTestField3);
		TestReceiveTrArray(trArray);
	}

	public unsafe void TestReceiveTrArray(System.TypedReference* inParams)
	{
		RealMethod1(__refvalue(inParams[0], int), __refvalue(inParams[1], int), __refvalue(inParams[2], string), out __refvalue(inParams[3], int));
	}

	public void RealMethod1(int inA, int inB, string inC, out int outD)
	{
		outD = 6000;
	}
}

public unsafe delegate void TestDelegateWithTrArray(object inInstance, System.TypedReference* inParams);

The error is:
Internal compiler error: Object reference not set to an instance of an object.

If you comment out TestReceiveTrArray and compile the rest (meaning compile the __makeref stuff but not the __refvalue stuff), then the error is:
Error ILT0005: 'C:\Users\XXXX\.nuget\packages\runtime.win10-x64.microsoft.net.native.compiler\2.1.8\tools\x64\ilc\Tools\nutc_driver.exe
@"C:\Users\XXXX\Documents\TestUwpApp1\obj\x64\Release\ilc\intermediate\MDIL\TestUwpApp1.rsp"'
returned exit code -2147483645

System.TypedReference is low-level but quite useful in certain scenarios such as dynamic programming, Reflection, late-binding / dynamic binding at runtime.
System.Delegate.DynamicInvoke(params object[]) and System.Reflection.MethodBase.Invoke(object, object[]) are useful but costly, especially when executed many times in a loop.  The above code snippet is an example of how TypedReference can be used to dynamically execute a method without the overhead of boxing all of the parameters.  It works great when compiled to CIL, but not when compiled with .NET Native 2.1.8.
In some circumstances, an alternative to TypedReference is the DLR, but if I remember correctly, it was @jkotas who wrote somewhere something like the following comment about the DLR or DLR with .NAT Native: "This stuff is super fragile".  I feel that it is too risky to switch over to a solution that relies upon the DLR.  The DLR is unnecessary in this particular case/project because TypedReference is a much simpler solution that works well.  Some projects do need the full functionality of the DLR, but other projects can get away with TypedReference.
I very much hope that .NET Native will fully support TypedReference.  Given the choice between .NET Native supporting TypedReference or the DLR, then obviously TypedReference is far far easier to support and maintain than the big complex DLR.

We don't have plans to add these instructions to .NET Native at this time. This can change in the future, but since mkrefany/refanyval/refanytype are only accessible through undocumented C# keywords, this was scoped out in the 2.0 release.
We were supposed to error out on this with a proper message, but seems like the bug that was tracking it was missed an is still open. For internal reference, the bug is 404381.

@MichalStrehovsky That's a pity.  Does .NET Native support any other way of doing the equivalent of System.Delegate.DynamicInvoke(params object[]) without boxing each parameter?
I guess this also means that __arglist and System.ArgIterator are likewise not in the plan for .NET Native.

@MichalStrehovsky That's a pity. Does .NET Native support any other way of doing the equivalent of System.Delegate.DynamicInvoke(params object[]) without boxing each parameter?

Recently I've done it with help of new ref variables from C# 7.3, but it requires all args of equal size and to create one struct and method overload for each args count:
public struct Args1
{
public Arg Arg1;
}

public struct Args2
{
public Arg Arg1;
public Arg Arg2;
}

[StructLayout(LayoutKind.Explicit, Size = 8)]
public struct Arg
{
    [FieldOffset(0)] public int Int;
    [FieldOffset(0)] public string String;
}

and then call define methods:
public void Method(ref Args1 args) ...
public void Method(ref Args2 args) ...

[StructLayout(LayoutKind.Explicit, Size = 8)]
public struct Arg
{
    [FieldOffset(0)] public int Int;
    [FieldOffset(0)] public string String;
}

This is not legal. .NET Native misses the validation for this, but CoreCLR and CLR will fail to load such type. You cannot overlap reference type and non-reference type fields. This will cause memory corruptions.
High performance reflection APIs are discussed here: dotnet/corefx#14021, but we don't have anything concrete yet.

You cannot overlap reference type and non-reference type fields

Sure, my mistake.
For reference type one could use GCHandle.Alloc and pass it as IntPtr:
[StructLayout(LayoutKind.Explicit, Size = 8)]
public struct Arg
{
    [FieldOffset(0)] public int Int;
    [FieldOffset(0)] public IntPtr Handle;
}

var stringHandle = GCHandle.Alloc(str);
var arg = new Arg { Handle =  GCHandle.ToIntPtr(stringHandle) }

@MichalStrehovsky OK, thanks, you've helped me decide the path forward.  In order to be compatible with .NET Native, and to avoid taking the risk of relying on the DLR, I think I'll go with a solution that boxes all the parameters to a mutable "box" type instead of System.Object, and then pool and recycle the "mutable boxes", so that the late-binding cost is the same for 10 iterations of a loop as for a million iterations.
By the way, the problem with the DLR seems to be that it is a combination of useful and counterproductive functionality.  For example:  Useful example: Late-binding is useful in certain cases.  Counterproductive example:  If every object can change its type at runtime, then this seems to be a case of claiming "That's not a bug. That's a feature!" and then trying really, really, damn hard to believe it.  For example, TypeScript attempts to solve the problem of the "runtime type morphism feature/defect" in JavaScript.  The lesson learned from TypeScript and other examples is that dynamic languages tend to be a mixed bag of helpful + counterproductive ideas rolled into one.  Microsoft already abandoned IronRuby and IronPython and I find that to be an understandable decision.
Thus the DLR poses a problem for .NET Native / CoreRT because the DLR itself is a problem.  Thus it would be good if CoreRT provides alternatives.  I can see value in supporting __makeref but yes I can also understand if it's decided that it's not the best solution.


Add "remove cursor" action

When using multiple cursors this would allow the user to remove a cursor that is contained in the given target

I'll start looking at this one tomorrow, if no-one's workin' on it.

Go ahead. I have assigned the issue to you.
@knausj85 @pokey @brxck
Now that we are four developers we probably should assign the issues to our self when we are working on them.


fix typos

I'm using a spellchecking extension in VS Code and that's the typos it found.

Codecov Report

Merging #100 (e958848) into master (c11adc1) will not change coverage.
The diff coverage is 88.23%.

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

@fkohlgrueber whats the name of the extension? Thanks 😀

I'm using Code Spell Checker.


MAINT: optimize.minimize: enhance callback for remaining minimize methods and fmins

Reference issue
Follow-up to gh-17486
What does this implement/fix?
Does for other minimize methods and fmin optimizers what gh-17486 did for trust-constr and Nelder-Mead.
Additional information
fmin_ optimizers need to first wrap the callback method (it was already done by minimize for the minimize methods), and the _minimize functions need to use _call_callback_maybe_halt instead of just calling the callback.

2 weeks have pass. Let's get this in.


KeyfileAccount should have a publicKey method

Overview

It will be nice to have a account.public_key method to the KeyfileAccount or AccountAPI object.

Specification

I have something in my mind. If I can get a green signal then I will dive right in.

Approach

Using the keyfile property of the KeyfileAccount object I would like to implement this feature.
Something like this

self.wallet = self.keyfile
passphrase = self._prompt_for_passphrase(default="")
# Decrypt the private key
self.private_key = Account.decrypt(self.wallet, passphrase)

# Derive the public key from the private key
self.pk = keys.PrivateKey(private_key)
self.public_key = self.pk.public_key

Dependencies

eth_account
eth_keys
I believe both of them are already in use.

Yeah, this seems fine. Implementation suggestion would be to add this property to KeyfileAccount:
class KeyfileAccount(AccountAPI):
    ...

    @property
    def public_key(self) -> HexBytes:
        ... # logic you outlined above

I am just perplexed that we have to decrypt an account's private key in order to obtain the public key... that just seems backwards?

I am just perplexed that we have to decrypt an account's private key in order to obtain the public key... that just seems backwards?

Not really, and address is a keccak hash so it is not able to be obtained like this, so you need to to run the key derivation to get the public key again
But yes, you can obtain the public key during the address derivation process

I have made some changes to cache the public key in the keyfile please review this pr https://github.com/ApeWorX/ape/pull/1702


i get gray screen in ios version 6

I defined everything in the instruction.
For android everthing works great for me,
But for ios , i get gray screen with logo of google and without the option to increase or decrease the screen.
i made double check of the bundle Identifier in google side, and for the api key i got.
what can be the problem ?
please assist.
Thanks.
Roy.

Fixed :
I forgot to add Google Maps SDK for iOS


    ORDER BY "Member"."Surname" ASC, "Member"."FirstName" ASC  LIMIT 1  42S22-1054: Unknown column 'Member.ValidationKey' in 'field list'" at /Users/James/Sites/circid-roi/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php line 64 {"exception":"[object] (SilverStripe\ORM\Connect\DatabaseException(code: 0): Couldn't run query:\n\nSELECT DISTINCT "Member"."ClassName", "Member"."LastEdited", "Member"."Created", "Member"."ValidationKey", "Member"."NeedsValidation", "Member"."NeedsApproval", "Member"."PublicFieldsRaw", "Member"."FirstName", "Member"."Surname", "Member"."Email", "Member"."TempIDHash", "Member"."TempIDExpired", "Member"."Password", "Member"."AutoLoginHash", "Member"."AutoLoginExpired", "Member"."PasswordEncryption", "Member"."Salt", "Member"."PasswordExpiry", "Member"."LockedOutUntil", "Member"."Locale", "Member"."FailedLoginCount", "Member"."ProfilePageID", "Member"."ID", \n\t\t\tCASE WHEN "Member"."ClassName" IS NOT NULL THEN "Member"."ClassName"\n\t\t\tELSE 'SilverStripe\\Security\\Member' END AS "RecordClassName"\n\nFROM "Member"\n\nWHERE ("Member"."Email" = ?)\n\nORDER BY "Member"."Surname" ASC, "Member"."FirstName" ASC\n\nLIMIT 1\n\n42S22-1054: Unknown column 'Member.ValidationKey' in 'field list' at /Users/James/Sites/circid-roi/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php:64)"} []

Resolved by removing, installing again and running dev/build from command line


fuzzed_y': undefined method keys' for #Array:0...

The code is:
=begin
The following requires cp'ed from:
  https://github.com/tardate/pdf-reader-turtletext/blob/master/lib/pdf-reader-turtletext.rb
=end
require 'pdf-reader'
require 'pdf/reader/patch/object_hash'
require 'pdf/reader/positional_text_receiver'

require 'pdf/reader/turtletext'
require 'pdf/reader/turtletext/version'
require 'pdf/reader/turtletext/textangle'

=begin
The following from:
  "How to instantiate Turtletext in code"
  https://github.com/tardate/pdf-reader-turtletext
=end
pdf_filename = '../taxforms/f1065-2017.pdf'
reader = PDF::Reader::Turtletext.new(pdf_filename)
=begin
The following from:
  "How to extract text within a region described in relation to other text
  https://github.com/tardate/pdf-reader-turtletext
=end
textangle = reader.bounding_box do
  page 4
end
textangle.text

However, when run with my ruby2.3, it produces error the error in the subject line:
make -k
gem list pdf-reader

*** LOCAL GEMS ***

pdf-reader (1.4.0, 1.1.1)
pdf-reader-html (0.1.0)
pdf-reader-markup (0.0.1)
pdf-reader-turtletext (0.2.2)
ruby how_to_instantiate.rb
/var/lib/gems/2.3.0/gems/pdf-reader-turtletext-0.2.2/lib/pdf/reader/turtletext.rb:53:in `fuzzed_y': undefined method `keys' for #<Array:0x000000013d5058> (NoMethodError)
	from /var/lib/gems/2.3.0/gems/pdf-reader-turtletext-0.2.2/lib/pdf/reader/turtletext.rb:42:in `content'
	from /var/lib/gems/2.3.0/gems/pdf-reader-turtletext-0.2.2/lib/pdf/reader/turtletext.rb:87:in `text_in_region'
	from /var/lib/gems/2.3.0/gems/pdf-reader-turtletext-0.2.2/lib/pdf/reader/turtletext/textangle.rb:134:in `text'
	from how_to_instantiate.rb:28:in `<main>'
Makefile:4: recipe for target 'how_to_instantiate' failed

Is this a bug in pdf-reader-turtletext or am I at fault?
TIA.

@cppljevans Please check out the fork from tkieley. I made a small patch so it will work with PDF Reader > 1.2

Thanks @MatthewSuttles ;however, I'm a newbie and despite reading manpages and googling
for several hours, I'm still unsure how to install this.  I'd guess something like
gem install pdf-reader-turtletext --source=https://github.com/kieleyt/pdf-reader-turtletext
Is that right?

For anyone still having this issue, I submitted a pull request to fix compatibility with pdf-reader 2.4.0. You can find it here: https://github.com/emmeryn/pdf-reader-turtletext


Tracking: sort key support across the system

To make our system leverage the property that our storage provides (merge sort!), we should work on each component to make them fully understand sort keys. To get a minimum e2e test to work, this tracking issue involves the following sub-tasks:

https://github.com/risinglightdb/risinglight/issues/625
https://github.com/risinglightdb/risinglight/issues/589
https://github.com/risinglightdb/risinglight/issues/591
https://github.com/risinglightdb/risinglight/issues/601

Firstly, we should have a user interface to specify the sort mode of tables. This is covered by https://github.com/risinglightdb/risinglight/issues/625. We should refactor catalog to support composite primary keys.
Secondly, we should be able to specify such properties at the storage side, and exploit the storage property to do range scan, which is covered in https://github.com/risinglightdb/risinglight/issues/589. This is not required for a full e2e to run, but is still useful.
Then, we can write some executors to leverage the sort property. We already have merge sort join, and can have things like sort agg.
Finally, the optimizer should know how each executor produces data. We should add a "sorted" property on executors https://github.com/risinglightdb/risinglight/issues/601, so as to generate optimized plans. After that, we can add rules to leverage the sort property from the storage layer. Alternative: do some ad-hoc rewrite on order by + table scan.

cc @Shmiwy will take over the issue for a month. He will work on various parts of the system to gain full sort key support.

Status on this?

Storage now supports range filter scan. Need add support to other layers.


fix: ensure tag oldProps[key] is a string

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Nuxt 3.7 (but this certainly applies to other versions).
With routes:
/
/xxx
/[network] (parameter value)
[...]

When you come from / or /xxx to /[network], regardless of direction, then oldProps[key] is true instead of a string.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Hi @harlan-zw, thank you for launching the test.
Don't hesitate to come back to me if you need anything, thanks.

Can you provide some reproduction details? I don't see the issue https://stackblitz.com/edit/nuxt-starter-hyh4tn?file=pages%2F[network].vue


Update contracts to 0.8 of solidity

Fixed in https://github.com/smartcontractkit/truffle-starter-kit/commit/027093558616723755b5dfd75d711fcb351f0798


[SCB-215] Process @ApiParam

just only process parameter  level, it's enough now.

Coverage decreased (-0.2%) to 86.127% when pulling 15bdec69d548ab8d20e9e0b6a4ebc194b3951c88 on wujimin:process-ApiParam into 4dbc83a5c294f48e3edb06542eaf1758325039b2 on apache:master.

This PR implements @ApiParam to specify parameter description and collectionFormat will be implemented later.


Upgrade tower-lsp, tokio and rust

This should solve test/compilation issues with other PRs.
Did not touch anything feature-wise, just made it compile again.

Heya this is already done in the troy branch (mostly) mind to re-target this to there to void murge conflicts? (we really should look into merging the troy branch)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 23 (0.0%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main.rs
0
1
0.0%

src/lsp_utils.rs
0
6
0.0%

src/backend.rs
0
16
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/backend.rs
1
0%

src/lsp_utils.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
306

💛  - Coveralls


Question about Do not configure a credential in integration assistant

[Enter feedback here]
The integration assistant for AAD is great. I used it to check my AAD app registration.
Here is a question that the assistant report the Discouraged configurations: Do not configure a credential (certificate/secret).
As I will use the AAD app to auth, the credential is required. So what should I do for this warning?

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

ID: 77cdac8c-be88-ce51-532e-0cdbc6607ca7
Version Independent ID: 8c657887-5cfd-e51b-244c-bfe69e2bf152
Content: Best practices for Azure AD application registration configuration - Microsoft identity platform
Content Source: articles/active-directory/develop/security-best-practices-for-app-registration.md
Service: active-directory
Sub-service: develop
GitHub Login: @Chrispine-Chiedo
Microsoft Alias: cchiedo

@xzf0587 Thank you for your feedback . We will investigate and update the thread.

@xzf0587 Configuring the client secret in application's configuration file is not secured as secrets ae hardcoded in configuration file and expose along with source code.  We do not recommend configuring a credential.
Microsoft Identity Platform provide you other ways to store credentials in more secured way.
1.You can use Azure Key Vault to store and retrieve secrets/certificates in more secure way.
2. Managed Identity manages the credentials for you for secure communication.

Closing this issue because this forum is for documentation issues only. To report functionality issues or get help with your code, please visit Microsoft Q&A


Speedup Unit Tests

We had some tests that took a long time to run, this saves some redundant computations taking our tests from > 1m to ~17s.

these optimizations in the test harness code aren't really needed after we have configurable params to set more reasonable numbers for MAX_INPUTS and MAX_OUTPUTS. #130


Unable to Install Shapely Library on OpenWRT x86 Linux

Description:
Problem:
I'm facing difficulties installing the Shapely library on an OpenWRT x86 Linux version, where I cannot use pip due to lack of support. The Python version available is 3.9. I have tried multiple approaches, but none have been successful.
Steps to Reproduce:

Downloaded shapely-2.0.1-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl and extracted it.
Copied both the shapely and shapely.libs folders, which is similar to the process I've used for other libraries that are working.
Executed a Python script but encountered the following error:

from .lib import GEOSException  # NOQA
ModuleNotFoundError: No module named 'shapely.lib'

Searched in the Shapely libraries and found that it's looking for the file lib.cpython-39-x86_64-linux-gnu.so. Renamed it to lib.so and re-ran the Python script, but encountered the error:

ImportError: /rwdata/opt/gt/adv_libs/shapely/lib.so: wrong ELF class: ELFCLASS64 even if the file filename says x86_x64.

Tried with an older version, 1.7.1, which has a different structure. This version doesn't give the shapely.lib error but instead says:

OSError: /rwdata/opt/gt/adv_libs/Shapely.libs/libgeos_c-a68605fd.so.1.13.1: wrong ELF class: ELFCLASS64

Attempted to install from source using the command setup.py install, but it failed because setuptools is missing and cannot be installed.

Expected Behavior:
I would like to successfully install the Shapely library on OpenWRT x86 Linux with Python 3.9.
Additional Information:
Am I missing something crucial in this installation process? Any guidance or help would be greatly appreciated.

Are you able to install numpy? Start with that. I don't now where to begin with OpenWRT (never heard of it before), but with an Alpine docker image you would need to get numpy-1.25.2-cp39-cp39-musllinux_1_1_x86_64.whl then unzip it to /usr/lib/python3.9/site-packages. But shapely doesn't have any musllinux builds (it could), so shapely cannot be installed for alpine. You shouldn't need to rename any libraries.
A better option would probably be to install geos then do GEOS_CONFIG=/path/to/geos-config python3 setup.py install

Closing due to inactivity


Add time stamp option for verbose output

I find the verbose option readable and useful.
What maybe I miss is:

an indication of date/time of the message. Is more for performance tracking purposes than anything else.
a progress counter which would simply output the progress on each of the counters presented in the Processed summary which is shown at the end. Something like. Processing X pic of Y for each of the categories shown on Processed at the end:

Processed: 65171 photos, exported: 4, updated: 0, skipped: 66482, updated EXIF data: 5, missing: 50, error: 0, touched date: 66461

Originally posted by @oPromessa in https://github.com/RhetTbull/osxphotos/issues/79#issuecomment-1019466035

@oPromessa I've added a --timestamp option that works with --verbose.  If enabled, output looks like this:

How's that look?


Fix: stop loading resource after stop() and destroy() methods are called

When working with icecast audio and call stop() or destroy() methods, the source continues to load (forever).
Now it's fixed.
Here you can see an example with icecast
Open dev tools -> Network and look: when the player is stopped the resource is still loading. When I have several icecast stations and switch between them, destroying the player each time, I end up with the situation when every resource is loading via several requests and new requests are pending. It's a serious leak. You can press "Run" in the example several times, starting the playback each time, to see the same situation.
The idea of the fix is discussed here:
https://stackoverflow.com/questions/3258587/how-to-properly-unload-destroy-a-video-element
Also it can be fixed via this.el.src = '', but the variant with load() method is clearer.
P.S.: I haven't tested the code with this fix directly, since I can't build the player on Windows (yarn install isn't work and I have Node.js 10 installed). But the same technique I used in my own code to bypass the bug, i.e. the following code
this.player.core.activePlayback.el.removeAttribute('src');
this.player.core.activePlayback.el.load();

before destroy() and stop(). And it works well.

Coverage increased (+0.3%) to 78.703% when pulling b93cd48dba1a16e4cb56952e1c1515faa54ce9bd on RussCoder:dev into b5d9e24d2d3551b801f123cdea70c98cf790bd2c on clappr:dev.

Thanks, @RussCoder. 🍻


Create Cards Automatically with the Automatic Instance Generation Template

Landscape label: trail1 & trail2 & ap
Scenario label : SCPTest-mcard
Priority : High-prio
Tutorial URL: https://developers.sap.com/tutorials/cp-mobile-cards-automatic-instance-generation.html
Please specify the step you are referring to
Step 2: Create an Automatic Instance card
Steps to reproduce :
1)Click on Create Card template icon
2) provide name as Automatic Instance Card
version as 1.0
destination as com.sap.edm.sampleservice.v2
select "Template Source " as "Template Manager" ( if dialog box pops-up to enter API key , then click on cancel )
Actual result :
In document  it is mentioned to select "Card Template " as " Automatic Instance Generation".
But , i am getting "Card Template "  drop down actions as "single Instance " & "Multiple Instance".

Please change the steps & screenshot attached related to this selection of "card template" value accordingly.
For time being, i continued by selecting " Multi instance " as "card template " value & could successfully create automatic instance card.
Thanks
Mounika

Note :
Alos, Please change screenshots for tutorial 4  w.r.to template view  under   "Step 2: Create a new version of the card"

Hi @i353590
Thank you for raising the issue.
@shahchetali can you please look into the issue?

Hi @mounikat987 ,
I will update the steps and screenshots.
Thanks,
Chetali Shah

@sandeep-tds @shahchetali Has this been finished?

Hi @thecodester , @mounikat987 ,
We will be deleting this tutorial soon as the Automatic Instance generation template no longer exist.
Thanks & Regards,
Chetali

@shahchetali Do you want me to delete it?

@shahchetali Do you want me to delete it?

https://github.com/SAPDocuments/Tutorials/issues/5963

https://github.com/SAPDocuments/Tutorials/issues/5963

Hi @mounikat987
The updated tutorial has been released, can you please verify the changes and update the ticket?
Thank you so much for your support.
Warm Regards
Sandeep T D S

Hi @mounikat987
The updated tutorial has been released, can you please verify the changes and update the ticket?
Thank you so much for your support.
Warm Regards
Sandeep T D S

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Interaction with mutations

I'm trying to use joinMonster to do a fetch for the requested object after a mutation has been called. I'm unsure how to tell it to fetch against the output payload. Any ideas would be appreciated.
mutation {
  addUser (...) {
     user: {
         id
         username,
         profile: {
             email
         }
     }
  }
}

This is what I'm trying but it's not working:
// assigned to the rootMutation.addUser(...)
const CreateUserPayloadType = new GraphQLObjectType({
  name: 'CreateUserPayload',
  fields: () => ({
    user: {
      // the UserType has a mapping to a UserProfileType under the field 'profile'
      // UserType has a sqlTable and fields defined
      type: UserType
    }
  })
})

// assigned to the rootMutation.addUser(...)
const createUserResolver = async (root, args, context, resolveInfo) => {
  // insert the user + profile (objection.js Model)
  const user = await UserModel.query().insertGraph(args.input)

  // should query for user / profile w/ the new id and return it
  // probably doesn't work because the resolveInfo is based around the CreateUserInputType
  // instead of the CreateUserPayload?
  // does joinMonster know when the request is a mutation, to use the payload definition for output?
  return joinMonster(resolveInfo, { id: user.id }, sql => {
    return knex.raw(sql)
  })
}

Post with higher level details - https://github.com/stems/join-monster-demo/issues/5

Pretty much a duplicate of
https://github.com/stems/join-monster/issues/131

Does this works? any update?


[Feat]: Adding the GSAP(JS Library) Animations to the website

Is your feature request related to a problem? Please describe.
For this electionHub website I felt to add some best animations which can help the user feel good
Describe the solution you'd like.
So I will be using the GSAP(Javascript animation library) where I will be adding the best animations to this website
Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

I would like to work on this. So can you please assign this to me!

Liked your idea, but I need more clarification about what changes or animation you going to implement.

Yeah I will be adding the sliding animations on the Navbar like when the server starts or when the page reloads the Navbar elements comes from the top and place in the current place.and add the hero session like it scales and comes out such type of animations I will be adding these are called as microInteractions which can enhance the website interface and give the best experience for the website

okay, you can start working on it.

https://www.linkedin.com/feed/update/urn:li:activity:7180274007328321536/ this is the project which was made by team in hackathon

I will be making like this

I will be making like this

No Problem, take your time.

https://github.com/TechNodes2-0/ElectiveHub/assets/146669517/bf506dbd-1511-480c-968f-29612a599d68
https://github.com/TechNodes2-0/ElectiveHub/assets/146669517/b15a8ce5-9b11-4d79-a42b-ce8fbfa3f59e
is this ok sir . Or should I make more changes I included the Locomotive JS u might not see in the video the smoothness when I push it u can experience it

ElectiveHub.-.Brave.2024-05-11.17-14-13.mp4
ElectiveHub.-.Brave.2024-05-11.17-15-00.mp4
is this ok @Yash636261 . Or should I make more changes I included the Locomotive JS u might not see in the video the smoothness when I push it u can experience it

looks cool just make few changes

make that navbar elements load faster and other animations too (no one likes to wait that much)
on login page keep that illustration image static (bounce effect attracts too much attention which should be on form)

Yeah I will make those changes

I made the necessary changes @Yash636261 . and create a PR

Thank you for assigning this issue to me @Yash636261


Test iframe resizing

This PR adds a test to Chapter 16 that tests what happens when one frame's script resizes, using setAttribute, another frame. (It should force a re-layout in that frame.)
Funny enough, Chapter 14 added setAttribute but somehow Chapter 15 lost it. So this PR first adds setAttribute back into Chapter 15, describes it in the Chapter 15 text, and then adds it to Chapter 16, and describes it in the Chapter 16 text.
For some reason, the test passes without having to protect the frame_width field. Not sure why, will investigate tomorrow.

Ok: I found and fixed one important bug. Basically, in each layout object type, we need to clear the descendant dirty bits by calling self.descendants.set(None). However, in LineLayout, we were doing this right after the recursive call to layout. That seems correct, but actually isn't, because after that recursive call, we change the child's y field, which re-sets the descendant field, and then that won't be cleared. Big mistake! (This will be difference once I merge with main but I think a similar bug will occur.)

Ok, yes, this does work for the reason we thought it would: when you setAttribute, you invalidate every frame (this step is important, see #850) and also the IframeLayout. Then when you modify the IframeLayout, it modifies the frame_width, which in turn means that when you get to the other frame it'll invalidate the DocumentLayout's width.

Code and text look good, just needs rebasing on top of my PR to remove descendants, and a quick lab15 test.

Done. Apologies, had an urgent thing to finish last week.


Can't hear the sound from the Sculk Sensor

Server Version: Minecraft 1.19.2 with Fabric Loader 0.14.9 + Geyser-Fabric 2.0.7-SNAPSHOT (git-a16fc9c07c9c6c2c00e03ba5b0637d89f05a1d4d-a16fc9c)
The Bedrock Edition client doesn't have sound effects when the Sculk Sensor detects motion nearby, but the Java Edition does.
https://user-images.githubusercontent.com/9267814/189312344-4f063ff4-cd1c-49d3-9daa-3e16e7239023.mp4

This is probably just a Geyser issue unrelated to Geyser-Fabric specifically, but regardless, do you still experience this?

This is probably just a Geyser issue unrelated to Geyser-Fabric specifically, but regardless, do you still experience this?

yes


status indicator demo

demo 196

Thanks!

nice sample @choper725

if you want, feel also free to add it to the overview, then it is easier to find for everyone. did it now with https://github.com/abap2UI5/abap2UI5-samples/pull/185

if we continue like this, maybe one day we have the whole UI5 Lib integrated 😅

i will add it.
dont know if you noticed, we have like 95% of the sdk in xml view.. :)

yes i tried to add as much samples as i found to the overview (but i am sure i missed some) and also tried to create somehow a understandable structure, but yes still work in progress, feel free to adjust and add.
ahh nice! yes 95% is a very good value :) we have nearly 200 samples now, great!

yes! we pretty much added most of the controls in the sdk..


Put JsonSerializerOptions as a config

Hello,
I didn't respond to your message since the PR was closed @runeanielsen .
You said :

I think it looks fine to me. I might add support so it can be changed when the service is registered in the future.

Indeed, It's true that now it's in the constructor but we do not have the possibility to add it as a config if we want to inject it.
So, I put it as a config instead of the constructor.

Hi @samihafidiDLW thanks for making the change, I've released a new version 7.11.0. I did not create is as a major version, since I think no one has used this feature yet.
// Rune


Sign-in freezes before two-step verification window on Linux Mint

When you try to sign in with Linux Mint having two-step verification enabled for your Google account, the screen freezes before the two-step verification screen opens, with some grey squares appearing in the corners of the screen, but otherwise the content remains as the previous password enter screen (image 1). If I then resize the window for example, the other parts of the screen turn into black (image 2), but there seems to be no way of going forward from that point.
I tested this with OSX yesterday and the login worked fine with two-step verification enabled there.

On linux it appears the --disable-gpu is required for some flavors.

Closed because I am like 100% that flag will work :+1:

You're correct, it did work. Thanks for the quick help!


fetchgit unable to handle repositories at file:/// URLs

Describe the bug
fetchgit is unable to find or clone a repository if that repository is on my local filesystem. I have determined that nix-fetch-git is running in a very stripped-down environment and is unable to even see things like my /home and /var directories.
Steps To Reproduce
Steps to reproduce the behavior:

Create this file as test.nix, replacing the repository with anything you have on your local filesystem:

let
pkgs = import <nixpkgs> {};
in
pkgs.stdenv.mkDerivation {
name = "fluent-ergonomics";
src = pkgs.fetchgit {
  url = "file:///home/savanni/src/fluent-ergonomics/";
  rev = "d6adea30d2edfb8c7049459ff1e640f0c4def487";
  sha256 = "17vdf91caivz5afg51azk1lg7fgph3mjm6kbvszvkgvjxb4gw0pa";
};
dontUnpack = true;
dontBuild = true;
}

Install it: nix-env -i -f ./test.nix

Expected behavior
I expect it to find and clone my repository.
Screenshots
Additional context
I built the above minimal example and included several sets of outputs, including outputs from embedding ls -l / into nix-prefetch-git on my local copy of the nixpkgs repository:
https://gist.github.com/savannidgerinel/fa5c0db1e7a5227a9fd4fee5d9481d60
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.11, NixOS, 21.05.4719.0fd9ee1aa36 (Okapi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.4pre20211006_53e4794`
 - channels(root): `"home-manager-21.05, nixos-21.05.4719.0fd9ee1aa36, pkgs-19.03-19.03.173691.34c7eb7545d, pkgs-19.09-19.09.2522.75f4ba05c63, pkgs-20.03-20.03.3330.1db42b7fe38, pkgs-20.09-20.09.4407.1c1f5649bb9, pkgs-21.05-21.05.4719.0fd9ee1aa36, pkgs-21.11-21.11.336045.4afca382d80, unstable-22.05pre356180.d5f23787297, pkg-20.09-20.09.2205.65c9cc79f1d"`
 - channels(savanni): `"local"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I'd say this is intended — nixpkgs fixed output derivations are allowed to access the network, but not the local file system, since there is no good reason for nixpkgs itself why they should. For your purposes, I'd recommend using builtins.fetchGit which can fetch local repositories, e. g. builtins.fetchGit /home/savanni/src/fluent-ergonomics should do what you want exactly.


Enhance model check for LLM

Type of Change
bug fix
Description
Enhance model check for LLM
Expected Behavior & Potential Risk
Unet pass test with onnx 1.14.0  onnxrt 1.14.1
How has this PR been tested?
pre-ci, extension test
Dependency Change?
no

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/5066/

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/5079/


Wrong output registered when granting privileges on specific columns

What happened?
When granting access to specific columns the provider does not register the outputs correctly causing the Grant resource to be recreated indefinetely.indefinitely
Example
let database = new postgresql.Database("pulumi-db");

let role = new postgresql.Role('foo');

new postgresql.Grant('foo', {
  database: database.name,
  role: role.name,
  schema: 'information_schema',
  objectType: 'column',
  objects: ['sql_features'],
  columns: ['feature_id', 'feature_name'],
  privileges: ['SELECT']
});

The code above was tested with docker run -ti --rm -p 5432:5432 -e POSTGRES_PASSWORD=password postgres:16. Every update will cause the Grant resource to be recreated.
Output of pulumi about
CLI
Version      3.105.0
Go Version   go1.21.6
Go Compiler  gc
Host
OS       arch
Version  "23.1.3"
Arch     x86_64
The backend part of the about was omitted since it seems not relevant to the issue.
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

I tried the same using the terraform provider, and it works fine:
terraform {
  required_providers {
    postgresql = {
      source = "cyrilgdn/postgresql"
    }
  }
}

provider "postgresql" {
  host            = "localhost"
  port            = 5432
  database        = "postgres"
  username        = "postgres"
  password        = "password"
  sslmode         = "disable"
  connect_timeout = 16
}

resource "postgresql_database" "foo" {
  name = "tf_foo"
}

resource "postgresql_role" "foo" {
  name = "tf_foo_role"
}

resource "postgresql_grant" "foo" {
  database    = postgresql_database.foo.name
  role        = postgresql_role.foo.name
  schema      = "information_schema"
  object_type = "column"
  objects     = ["sql_features"]
  columns     = ["feature_id", "feature_name"]
  privileges  = ["SELECT"]
}

Running a verbose update (-v5 --logflow --logtosdterr) I could see that at some point the output is return but in a second log the output is empty for that attribute. I'm not very familiar with the bridge to figure out where it is getting lost exactly.

Hi @fernandocarletti. Thanks for reporting this bug and the excellent repro! We'll take a look as soon as we can.

I just tried reproing this, and it no longer repros.  I confirmed that it repros as recently as the 3.11.1 release, and was fixed in the 3.11.3 release.
It appears the fix was in improvements in pulumi-terraform-bridge as part of this release: https://github.com/pulumi/pulumi-terraform-bridge/compare/v3.86.0...v3.87.0.

Awesome! Thanks for the heads up, @lukehoban!


IonTextWriterTest relies on private isTopLevel accessor

IonTextWriterTest uses the isTopLevel accessor that happens to be present on all implementations of Writer but which is not part of the Writer API.
This behavior works in Javascript but is (rightly) illegal in Typescript.

This was addressed by https://github.com/amzn/ion-js/pull/462, closing


Gradient on mana orb only shows if first skill is open

Bug on a local branch, putting it here so I don't forget to fix this later as I can't figure out what's causing it.

Affects: Firefox, Chrome, Android Chrome.
Does not affect: Edge, Safari,

Gradients are defined via a unique id in the svg, using multiples of the same svg creates problems because now you are defining the unique id multiple times. Temporary workaround soon while I contact vue-awesome.


Error in the installation

Hello,
Since the last update the installation doesn't work anymore on an Ubuntu 20.04.
Here is the errors that i get.

Thank you for your response!

Hi @Madaa29,
This importlib.resources has been giving us some headaches ;) I've just committed an alternative that is hopefully more robust. If it works out on other platforms as well we'll create a new package.

I tried to install it this morning with the latest package and it still gives me the same error.

I assume you are using the (pip) packaged version of BlueSky? It hasn't been updated yet with the fix from yesterday. I want to be sure first that it works :)

Ok, a new version was just uploaded to pypi. Let us know if it is still causing errors for you!

Hello ! It's working ! The only issue is to open scenario it doesn't work !
Thank you a lot ! :)

Can you be more specific? What are you trying exactly, and what then happens?

When you want to launch a scenario by clicking on "open" nothing is happening and we have the following error

Ah, thanks, that was a remaining bug in the transition to PyQt6. Should be fixed in the latest release.


y

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

?

I assume this issue was opened by mistake. Closing.


Added onTapOutside

Added the ability to add an onTapOutside function that by default releases focus from the textfield, allowing users to easily get out of the textfield.

use intl_mobile_field package


dubbo内存泄露

版本：spring-cloud-starter-dubbo 2.2.3.RELEASE，对应的dubbo版本为2.7.8，nacos的starter也是 2.2.3.RELEASE
jmap发现有个dto占用大量空间，得不到释放
手动触发几次gc仍然不能释放

dump后使用mat分析疑似内存泄露

猜测是rpc调用后，dto对象放到threadlocal里，然后没有释放

can you provider a demo to reproduce ?

i don't find the reason
it present casually

can you provider a demo to reproduce ?

i don't find the reason
it present casually

I mean to trouble you to provide a demo to reproduce this problem. Let me also check what caused it.
Because the problem you mentioned did not appear on my side.

Internal Thread Local will hold the reference of invocation until invoke finished. Please check if there are a lot pending request.

我也遇到了相同的问题, 是不是参数有大对象, 不知道哪个版本解决了这个问题. 试过2.7.13还有同样的问题

I have not received feedback for a long time, so I will close the issues temporarily. If there are any problems, I can open the issues at any time.

2.7.10 版本还是存在该问题，我可以复现

@hongyu-zxj
Does it still exist in the latest version 2.7.15

能说一下，2.7.15具体是在哪里修复的吗？

@hongyu-zxj Does it still exist in the latest version 2.7.15
@CrazyHZM 能说一下之前的版本具体的问题是什么吗？在哪里做了修复？

@hongyu-zxj
我的意思是在最新的2.7.15中是否还存在这个问题？


Unable to get type arguments of class

It looks like reading the type arguments of a class doesn't work. Inner Type of type aliases resolve correctly, but not classes.
type Foo<Type> = Type & {}

class Bar<Type> {}

@inject()
export class Baz {
  foo: Foo<string>   // type.innerTypes contains string

  bar: Bar<string>   // type.innerTypes is empty. type.getTypeParameters() returns Type. type.getTypeArguments() is empty
}

Fixed in<EMAIL_ADDRESS>But type work differently than other generic types. type is resolved to it's definition. So type Foo<T> = T & {} is just union T & {}, cuz Foo does not rly exist.
Check this StackBlitz out.

thanks for the fix!
While I'm here, isn't it possible to get types from the reference to a class ? i.e. getType(Foo) instead of getType<Foo>()

@avin-kavish
It is now in<EMAIL_ADDRESS>It was possible to get type of value getType(new Foo()); just one line and you are able to get type of class itself getType(Foo).

that's great!
I was considering the following about tst-reflect usage,

Can't we use a decorator directly, without having to call a method that returns a decorator?

/**
 * @reflect
 */
function inject<TType extends Constructor>(ctor: TType) {
    const typeInfo = getType<TType>()
}

// usage
@inject
class Foo {}

Can't we collect type info for all classes/interfaces in a project without using decorators? Maybe it's too slow?

@avin-kavish

You can. https://github.com/Hookyns/tst-reflect/blob/main/tests/src/11-generic-class-decorator.ts
Now all the types passed to getType<T>() and types decorated by decorators marked by @reflect are "reflected".

Next version is prepared for generating metadata about all the modules and types.

It's in progress and the latest progress is private until publish of final version.


Page Unresponsive SharePoint file change trigger

Describe the Bug with repro steps

Add Trigger.
Select "when a file is created or modified (properties only)
Select site address from dropdown (https:campusadvhq.sharepoint.com)
Select library name: Operations Shared Documents
Select Folder under "Advanced Parameters". Drill through stops functioning at "Recurring Processes" folder.

Page becomes unresponsive.
What type of Logic App Is this happening in?
Consumption (Portal)
Which operating system are you using?
Windows
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
No
Workflow JSON
No response
Screenshots or Videos

Browser
Chrome
Additional context
This functions fine within Power Automate. Permissions are there. Not sure what is happening.

This is a known issue and we have a fix in PR now that'll be in our next release.


Update csi-secrets-store-driver.md

The command to "verify the azure key vault provider for the CSI driver installation" was incorrect. This command requires that the space be removed to properly filter the labels for the pods.

Learn Build status updates of commit e153123:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit e153123:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

Line 0, Column 0: [Error: git-clone-failed] Failure to clone the repository `https://github.com/Azure-Samples/cognitive-services-speech-sdk#master`. This could be caused by an incorrect repository URL, please verify the URL on the Docs Portal (https://ops.microsoft.com). This could also be caused by not having the proper permission the repository, please confirm that the GitHub group/team that triggered the build has access to the repository.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@Nickomang
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Note that the validation warning must be resolved before merging.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Nickomang
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Note that the validation warning must be resolved before merging.
#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Hello, is it anything I need to do on my end for this merge to be pushed? I'm seeing a "merging is blocked" message here. Thank you!

Learn Build status updates of commit e153123:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/aks/csi-secrets-store-driver.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@jamal-brown,
The error message is cleared now. After @Nickomang's review, we can proceed.
Thanks!

Thank you, @jamal-brown! #sign-off

@jamal-brown & @Court72 - thank you for your help! I'm working on doc updates in a separate PR and have incorporated your change. Thank you for your help! If you could #please-close the PR, that'd be great!